-
Notifications
You must be signed in to change notification settings - Fork 0
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
Refactor target placement #101
Conversation
to standardize logic of indicating elements of gratings etc. as targets, which is a big par of the `stimuli` Also allows for negative element indices, counting backwards from the highest element index Closes #12
actually places targets in stimulus-dict
One consideration: should |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good, but there are some non-PR-related issues in the demos (see issue #102 ). Also not all style-checks pass because bare except statements are used.
Adds more general-purpose functions
mask_targets()
andplace_targets()
which can be used to designate target "elements" (bars, rings, frames, etc.) from an existing mask.This has been integrated into
waves
,pinwheels
,rings
andwhites
, and been used to addrings.circular_generalized()
andbullseye.circular_generalized()
.This single-implementation of target placement has the advantage that it all works the same: first element is
1
, target_indices can be negative (counting "backwards" from the last bar/ring/etc.).To implement this, some of the public-facing API has to change, as indicated by the fact that the
papers
have to be updated. Thus, this technically constitutes a "breaking" change that would require a major version bump....