Remove `from_json` functionality.

AllenInstitute / AllenSDK

code for reading and processing Allen Institute for Brain Science data

https://allensdk.readthedocs.io/en/latest/

Other

333 stars 149 forks source link

Remove `from_json` functionality. #2607

Open morriscb opened 1 year ago

morriscb commented 1 year ago

We'd like to stop relying on LIMS2 strategies for data releases/packaging. This is nominally done by calling from_lims on jsons produced by these strategies. These would now be unneeded and as no end user is expected to interface with the data via this method, we should remove the functionality to simplify the code and testing.

[ ] Locate all from_json methods on the various data objects.
[ ] Remove unittests.
[ ] Remove from_json code.

mikejhuang commented 1 year ago

The key requirement of this ticket is to convert nwb writers to gather data from_lims instead of from_json. The from_json methods are still used by the test cases to load fixtures and are retained.