Resolve "Change NMDBDataAttacher to not require a CRNSDataHub"
requested to merge improvement/25-change-nmdbdataattacher-to-not-require-a-crnsdatahub into development
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