Skip to content

Conversation

@micahwoodard
Copy link
Collaborator

@micahwoodard micahwoodard commented May 22, 2025

Pull Request instructions:

  • Please follow the update protocol
  • Answer the questions below in detail. Your responses will be emailed to experimenters.
  • If the experimenters must do anything new, provide detailed step by step instructions on the wiki
  • If computer maintainers need to manually update anything, provide detailed step by step instructions
  • Use markdown syntax in order for your comments to be rendered reliably in the email: "1." instead of "1)", use four spaces for indents.
  • If you use the keyword "skip email" in the title, it will skip the email updates
  • Merges from "develop" into "production_testing" should use the keyword "production merge" in the title for reliable indexing of updates
  • Merges from "production_testing" into "main" should use the keyword "update main"

Describe changes:

  • adds fast retract feature

What issues or discussions does this update address?

Describe the expected change in behavior from the perspective of the experimenter

  • if fast retraction is enabled, lickspout will fast retract and unretract at specified speed and wait time

Describe any manual update steps for task computers

  • None

Was this update tested in 446/447?

  • 446/7

Does this update impact downstream processing by adding new saved files, or changing their format? If so, have you documented changes?

  • No

@micahwoodard
Copy link
Collaborator Author

Users can configure fast retract using widgets in parameters tab
image

Enabling fast retract is at the bottom of the tab
image

@XX-Yin
Copy link
Collaborator

XX-Yin commented Jun 11, 2025

You only need to send a copy of licks to a different address. Is this what you changed?

@XX-Yin I didn't change any existing bonsai nodes, only added two that merge the lick signal from the lickety split board and Janelia board. I merged the two so if either emits a one (was licked), a message will be written to channel two. I did it this way because I only want to write a message if the lickspout was actually licked. Using the time subject like you sent doesn't work because those are emitted regard of whether the corresponding lickspout was licked. I believe I tried that in an earlier version. Let me know if I am misunderstanding your question though

image

This is ok. either way should work. The time subject will only send the timestamps when there are licks.

@XX-Yin
Copy link
Collaborator

XX-Yin commented Jun 11, 2025

@micahwoodard Can we set other parameters (speed and when to retract) like we discussed before?

@micahwoodard
Copy link
Collaborator Author

micahwoodard commented Jun 11, 2025

Users can configure fast retract using widgets in parameters tab image

@XX-Yin yes you can configure speed and wait time. Are there other parameters people want to be able to control as well?

@micahwoodard
Copy link
Collaborator Author

@XX-Yin @alexpiet this is a vid showing what this looks like. I'm testing with @ellahiltonvano today with an actual mouse.

2025-06-18.08-06-20.mp4

@micahwoodard
Copy link
Collaborator Author

@ellahiltonvano and I tested this last week with a mouse and there were some unusual bugs. Still working on investigating and fixing.

@micahwoodard micahwoodard changed the base branch from curriculum-dev-branch to curriculum-bug-fix June 26, 2025 17:29
@micahwoodard
Copy link
Collaborator Author

finished testing with @ellahiltonvano today with a mouse so merging into curriculum branch and closing sipe ticket

@micahwoodard micahwoodard merged commit 60b09e4 into curriculum-bug-fix Jul 17, 2025
@micahwoodard micahwoodard deleted the feat-fast-retract branch September 10, 2025 20:52
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

Successfully merging this pull request may close these issues.

4 participants