search

AllenInstitute / npc_lims

Tools to fetch and update paths, metadata and state for Mindscope Neuropixels sessions, in the cloud.
https://alleninstitute.github.io/npc_lims/
MIT License
0 stars 0 forks source link

Enh/codeocean sdk #21

Open mochic opened 2 months ago

mochic commented 2 months ago

Switches from aind-codeocean-api to codeocean. We have redundant objects in the form of JobStatus CapsuleComputationAPI that we should consider refactoring out. The codeocean analog provided is Computation. Some of our code relies on JobStatus, CapsuleComputationAPI being able to be iteratively serialized (all fields are optional) whereas Computation doesn't allow for this (requires some values to be present for validation). We should consider how we want to change this. Either by using a Computation subclass that allows us to do this or by changing the way we're creating/using these objects (ie: not iteratively serializing it). doctests for queue and codeocean_utils appear to be passing but we should identify what additional testing is necessary and perform it.

mochic commented 2 months ago

@bjhardcastle @arjunsridhar12345 removed the typed dicts associated with the queue, please review when you have time. There's a lot of redundant code but I refactored out some of the redundancies. Factoring out all the redundancies seems like it goes out of the scope of this pr.