add openephys tag to models #237
Conversation
|
There might be one or two things that need to be updated. I can push the updates. |
|
Will need to pull in #240 and re-test when that's merged, as it has potentially conflicting changes. In the meantime, The behavior table produced looks like follows; I'm not quite sure if its right... |
…tadata-mapper into openephys_patch
…thub.com/AllenNeuralDynamics/aind-metadata-mapper into enhancement/passive_ophys_behavior_block
…pty parameter sets, and prevent parameter sets with more than 1000 unique values
|
Checking in on this, is this still being worked on @rcpeene @Ahad-Allen ? |
Yes, sorry for this getting out of scope: we found two huge bugs in the stim table generation code that needed to be fixed that affect all behavior sessions. I am updating the test cases for this today and then we should be done :) |
| self.recording_dir = npc_ephys.get_single_oebin_path( | ||
| self.session_path | ||
| ).parent | ||
| # sessions_root = Path(self.job_settings.sessions_root) |
There was a problem hiding this comment.
if this code isnt being used, you can probably just remove it
The changes are minimal. Let me know if anything else needs to be done :)