search

AllenNeuralDynamics / aind-subject-nwb

MIT License
0 stars 0 forks source link

Some improvements #3

Closed alejoe91 closed 8 months ago

alejoe91 commented 8 months ago

@Ahad-Allen I did a bit of cleaning and extended the capabilities of the capsule.

Now it can either get an asset name and use the database, or use data assets attached to the capsule. In addition I added the backendargument.

A secon improvement is to use the data_description instead of the processing to get the session start time, since that is more reliable :) Let me know what yout think!

alejoe91 commented 8 months ago

@Ahad-Allen how did you apply the linting? Did you use black? I'm asking because it doesn't look "familiar" and the last commit actually wrongly formatted a string so the code was failing

Ahad-Allen commented 8 months ago

Strange! I used autopep8 for these. Your fixes look good. I think we can merge this and change the session_id once the ophys schema is finalized.