AllenNeuralDynamics / aind-slims-api

AIND-specific access to data stored in SLIMS
MIT License
0 stars 1 forks source link

Enh/2/behavior sessions 2 #16

Closed mochic closed 2 months ago

mochic commented 2 months ago

Addresses https://github.com/AllenNeuralDynamics/aind-slims-api/issues/2 and https://github.com/AllenNeuralDynamics/aind-slims-api/issues/3.

Uses a different pattern in behavior_session which I'd like to port over to other fetching/writing methods and also partially add to the core client so that table references can be resolved by the core client code.

Also, I'd like to return all records fetched from all fetching code and maybe have the return thresholding as some client level option (ideally to fetch less via the query params if that exists on the slims api ) or for that to be on the user also returning a list of validated and unvalidated data like fetch_behavior_session_content_events

Also, we should change the way Record name aliases are serialized/deserialized for better naming but that might be better for a separate pr.

mochic commented 2 months ago

Can't add reviewers, if either of y'all could take a look at this it would be helpful. @dyf @patricklatimer @jtyoung84 @bruno-f-cruz