AllenNeuralDynamics / dynamic-foraging-task

Bonsai/Harp workflow for Dynamic Foraging with Python GUI for visualization and control
MIT License
5 stars 4 forks source link

show sessions with old format #515

Closed XX-Yin closed 3 weeks ago

XX-Yin commented 3 weeks ago

Pull Request instructions:

Describe changes:

Make the GUI can load sessions with old folder structure to the session list. image

What issues or discussions does this update address?

The old session with a different folder structure cannot be loaded to the session list.

Describe the expected change in behavior from the perspective of the experimenter

No

Describe any manual update steps for task computers

No

Was this update tested in 446/447?

Tested on my own computer

alexpiet commented 3 weeks ago

Do you have any active mice that use the old folder structure? Why incorporate this feature?

XX-Yin commented 3 weeks ago

Do you have any active mice that use the old folder structure? Why incorporate this feature?

Yes, we have collected data with old structure that can't be displayed in the session list.

alexpiet commented 3 weeks ago

But isn't that data from more than several weeks ago? I assume that means its from non-active mice. If that is the case, then why are you viewing it on the GUI?

XX-Yin commented 3 weeks ago

But isn't that data from more than several weeks ago? I assume that means its from non-active mice. If that is the case, then why are you viewing it on the GUI?

Yes, that data is several weeks ago.

Sometimes, it's more straightforward to check parameters and behavior performance of that session from the GUI.

hanhou commented 3 weeks ago

FYI, the 'Streamlit!' button now opens all training history of current mouse right away. See this PR https://github.com/AllenNeuralDynamics/dynamic-foraging-task/issues/432

XX-Yin commented 3 weeks ago

@alexpiet @hanhou This PR has no harm to the main code. Just an additional function.