datajoint-company / bl_pipeline_python

Python Data pipeline for BrodyLab
0 stars 2 forks source link

Move Kilosort results to a subdirectory for `sessid` 758539 #13

Closed kabilar closed 2 years ago

kabilar commented 3 years ago
kabilar commented 3 years ago

Hi @Alvalunasan, would this be possible for sessid 758539? If there are more cases, I will have to think about a smarter way for the element to handle this situation. Thanks.

Alvalunasan commented 3 years ago

I think we can enforce to have kilosort results in a subdirectory. I will talk with ephys team about it.

Alvalunasan commented 3 years ago

![Uploading Screen Shot 2021-10-21 at 9.10.59 AM.png…]()

Alvalunasan commented 3 years ago
Screen Shot 2021-10-21 at 9 10 59 AM
kabilar commented 3 years ago

Only instances of session directory containing kilosort results that are not in a subdirectory:

image1

Session 751666 possibly does not have an issue because there is also other directory with some kilosort results for that subject and date: Thomas/T219/T219_2019_12_05/spikesort_2019_12_09_01_01_01_ks2jrc_thomas

Alvalunasan commented 3 years ago

Hi Kabi, Adrian agreed to correct these sessions (by creating a SORT_ID) subfolder. Also, the SORT_ID subfolder should be enforced in the future

kabilar commented 3 years ago

Great. Thank you.

kabilar commented 2 years ago

Hi @Alvalunasan, I was going to transfer this Issue to the new public vathes fork, and privately archive the current detached repository. Is there any information in this Issue that we would not want to be in the public repository? If so, I can delete those comments and then transfer the issue. Thank you.

Alvalunasan commented 2 years ago

Hi @kabilar no I think it is fine to let all the Issues you find on the public repo. Thanks for doing this.

Alvalunasan commented 2 years ago

Ephys pipeline in brodylab is not used at all so I will close this issue.