znamlab / cottage_analysis

Potentially temporary repository to use DLC to track eye position
MIT License
0 stars 0 forks source link

Pass full session name to `synchronisation` methods #27

Closed znamensk closed 1 year ago

znamensk commented 1 year ago

Currently the methods in synchronisation accept separate mouse and session arguments (e.g. BRYC1234.1a and S20230101). However, on flexilims the sessions are already named e.g. BRYC1234.1a_S20230101. So if we grab the session names from flexilims, we'd have to parse them and concatenate them again.

Proposal to simply pass the full session name every time?

ablot commented 1 year ago

Passing full name sounds good.

If we decide to parse, we should remember to use mouse, session = session['genealogy']. One day, one mouse or a session will have a underscore for some reason.

znamensk commented 1 year ago

We are now passing recording Series / name rather than session + mouse.