Skip to content

Resolve "Change NMDBDataAttacher to not require a CRNSDataHub"

Description

The NMDBDataAttacher currently takes as a argument the CRNSDataHub. Change this so it simply takes a dataframe. This makes it more flexible and we can adjust the CRNSDataHub to use it internally on the attached DataFrame in the Hub.

Related Issue

Closes #25 (closed)

Proposed Changes

  • Change class so that it takes a dataframe

Testing

  • Test it works

Checklist

  • I have tested the changes thoroughly
  • I have updated the relevant documentation
  • I have added necessary comments and explanations in the code
  • I have considered backward compatibility, if applicable
  • I have reviewed the code changes myself

Screenshots

Additional Notes

Edited by Daniel Power

Merge request reports

Loading