Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate sniffer sensor into the workflow #666

Open
XX-Yin opened this issue Aug 26, 2024 · 4 comments
Open

Integrate sniffer sensor into the workflow #666

XX-Yin opened this issue Aug 26, 2024 · 4 comments
Assignees

Comments

@XX-Yin
Copy link
Collaborator

XX-Yin commented Aug 26, 2024

@galenlynch @yonibrowning plan to integrate the sniffer sensor into the bonsai workflow

@alexpiet
Copy link
Collaborator

What are they using the sniffer sensor for? Should it be in the task workflow, or in a separate workflow that runs in parallel like the FIP workflow?

@bruno-f-cruz Do you have any insight here?

@bruno-f-cruz
Copy link
Collaborator

bruno-f-cruz commented Aug 26, 2024

No idea, I think the last time I checked what they were doing they implemented following the documentation we wrote:

https://allenneuraldynamics.github.io/Bonsai.AllenNeuralDynamics/articles/sniff-detector.html

nuget package is here:

https://www.nuget.org/packages/AllenNeuralDynamics.SniffDetector/0.1.0-preview06032024

Not sure if this is a problem or not (I think it may since you are running 2.7.x last time i checked), but this package requires bonsai core >= 2.8

@alexpiet
Copy link
Collaborator

The sniffer data will be analyzed offline, and not used in real time for task control. Therefore we should operate a second bonsai workflow in parallel, like the FIP data.

@XX-Yin
Copy link
Collaborator Author

XX-Yin commented Aug 27, 2024

It's ok to integrate the sniffer sensor in the same bonsai workflow even we only use it for offline analysis. The reason we separated the FIP workflow from the main workflow is that the FIP workflow does not run stably when they are combined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants
@yonibrowning @galenlynch @bruno-f-cruz @alexpiet @XX-Yin and others