add openephys tag to models

[rcpeene]

The changes are minimal. Let me know if anything else needs to be done :)

[jtyoung84]

There might be one or two things that need to be updated. I can push the updates.

[rcpeene]

Will need to pull in #240 and re-test when that's merged, as it has potentially conflicting changes.

In the meantime,
could @arielleleon or @Ahad-Allen review this?

The behavior table produced looks like follows; I'm not quite sure if its right...
1416080419_752311_20250123_stim_epochs.csv

[rcpeene]

This should be nearly ready to merge but three questions:

• @jtyoung84 previously said he had work to do on integrating this Etl. I don't recall the details, has this been addressed?
• The linter is showing two methods are 'too complex' but I can't conceive of any reasonable way to break them up. What to do about this?
• One method is missing a docstring. src\aind_metadata_mapper\open_ephys\utils\behavior_utils.py: remove_short_sandwiched_spontaneous. @Ahad-Allen are you able to document this?

[jtyoung84]

This should be nearly ready to merge but three questions:

• @jtyoung84 previously said he had work to do on integrating this Etl. I don't recall the details, has this been addressed?
• The linter is showing two methods are 'too complex' but I can't conceive of any reasonable way to break them up. What to do about this?
• One method is missing a docstring. src\aind_metadata_mapper\open_ephys\utils\behavior_utils.py: remove_short_sandwiched_spontaneous. @Ahad-Allen are you able to document this?

• We're moving towards having people generate the session.json files locally, but it shouldn't be too difficult to have it run in the transfer service once you are finished.
• I'd recommend breaking up very long functions or deeply nested functions into smaller methods. As a temporary workaround, you can annotate the methods with # noqa: C901 to skip that check, but please open a tech-debt ticket to re-organize it later.

[Ahad-Allen]

I added the doc string for that missing func :)

[mekhlakapoor]

Checking in on this, is this still being worked on @rcpeene @Ahad-Allen ?

[Ahad-Allen]

Checking in on this, is this still being worked on @rcpeene @Ahad-Allen ?

Yes, sorry for this getting out of scope: we found two huge bugs in the stim table generation code that needed to be fixed that affect all behavior sessions. I am updating the test cases for this today and then we should be done :)