The issue was that ophys cells table did not filter by passed only, so it was returning more experiments than in the ophys experiments table.
Adds passed_only arg to BehaviorProjectLimsApi class constructor rather than in instance methods since passed_only applies to all table the class returns
ophys_cells_table now supports passed_only flag
cleans up logic to filter by passed only in sql rather than in python
only pulls session type from pkl file, not from DB, in order to cleanup logic
Testing
Tested manually to confirm that experiments marked as failed are now excluded when passed_only=True
Note: updating old tests with new ones since updating the old tests would be harder than writing new ones
Addresses #2242
The issue was that ophys cells table did not filter by passed only, so it was returning more experiments than in the ophys experiments table.
BehaviorProjectLimsApi
class constructor rather than in instance methods sincepassed_only
applies to all table the class returnsTesting Tested manually to confirm that experiments marked as failed are now excluded when
passed_only=True
Note: updating old tests with new ones since updating the old tests would be harder than writing new ones