Skip to content

DAQ Reversal Stimulus - #203

Merged
monikascholz merged 18 commits into
masterfrom
issue_reversal-stimulus
Jul 15, 2026
Merged

DAQ Reversal Stimulus#203
monikascholz merged 18 commits into
masterfrom
issue_reversal-stimulus

Conversation

@takkasila

Copy link
Copy Markdown
Contributor

No description provided.

@takkasila
takkasila requested a review from monikascholz July 14, 2026 16:26
@takkasila takkasila self-assigned this Jul 14, 2026
@takkasila takkasila added feature New feature or request Medium Medium priority issue labels Jul 14, 2026
@takkasila takkasila changed the title Issue reversal stimulus DAQ Reversal Stimulus Jul 15, 2026
@takkasila
takkasila marked this pull request as ready for review July 15, 2026 12:18
Comment on lines +561 to +566
def detectReversal(self, trail: np.ndarray) -> bool:

# Get last M (trial limit) vertices and
# apply transformation to each row vertex
croppedTrail = trail[-self.trailLimit::, :]

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The function that detects reversal event

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

Comment thread glowtracker/settings/gui_settings.json Outdated
Comment on lines +561 to +566
def detectReversal(self, trail: np.ndarray) -> bool:

# Get last M (trial limit) vertices and
# apply transformation to each row vertex
croppedTrail = trail[-self.trailLimit::, :]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

@monikascholz
monikascholz merged commit 5585048 into master Jul 15, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request Medium Medium priority issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants