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

Ensure schedule information is set properly #653

Open alexpiet opened 1 month ago

alexpiet commented 1 month ago
alexpiet commented 1 month ago

We can address https://github.com/AllenNeuralDynamics/aind-behavior-blog/issues/559 at the same time. Proposed solution:

When we move to loading this information from SLIMS, we should have the GUI set all of those fields automatically. But for now, lets just verify they are correct

micahwoodard commented 2 weeks ago

@alexpiet for check if FIP on/off should I be checking self.FIP_started? I am unclear about which variable indicates if FIP is on or off

alexpiet commented 2 weeks ago

@micahwoodard Good question. You should check if self.PhotometryB.currentText()=='on' This is the GUI toggle button that sets whether to run FIP (FIP = Fiber Photometry, fyi). FIP_started tells us whether the FIP process has started already for this session, but we already check that if PhotometryB is "on". Since you are checking the schedule, you dont care if the FIP has started or not, only that the toggle has been set to eventual start.

micahwoodard commented 2 weeks ago

Oh okay I should've thought harder about FIP and photometry. If the schedule is blank in the FIP column, does that mean not photometry?

alexpiet commented 2 weeks ago

Yes, if the schedule is blank, then no photometry