Skip to content

8.29.5 - #924

Merged
bimac merged 8 commits into
iblrigv8from
fix/trainingCW_old_logic
Jun 23, 2026
Merged

8.29.5#924
bimac merged 8 commits into
iblrigv8from
fix/trainingCW_old_logic

Conversation

@bimac

@bimac bimac commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

No description provided.

…st convention

Introduce `_iblrig_tasks_trainingChoiceWorld_c0_CW` protocol that reproduces TrainingChoiceWorld behavior from IBLRIG versions 8.10.0-8.23.3. Zero-contrast trials always place stimulus on the left (clockwise turn rewarded) instead of random side assignment. Enables consistency with subjects trained under legacy convention.
@bimac bimac linked an issue Jun 17, 2026 that may be closed by this pull request
5 tasks
bimac added 4 commits June 17, 2026 10:48
…ontrast convention

Introduce `_iblrig_tasks_trainingPhaseChoiceWorld_c0_CW` protocol that reproduces TrainingPhaseChoiceWorld behavior from IBLRIG versions 8.10.0-8.23.3. Zero-contrast trials always place stimulus on the left (clockwise turn rewarded) instead of random side assignment. Enables consistency with subjects trained under legacy convention.
Extract stimulus position assignment logic into `_map_signed_contrast_to_position()` method across base and C0_CW task classes. Simplifies `next_trial()` implementation and improves code reusability. Update docstring references accordingly.
@bimac
bimac requested a review from k1o0 June 17, 2026 09:25
@bimac bimac self-assigned this Jun 17, 2026
Comment thread iblrig_tasks/_iblrig_tasks_trainingChoiceWorld_c0_CW/task.py Outdated
@coveralls

coveralls commented Jun 22, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 50.252% (+0.1%) from 50.121% — fix/trainingCW_old_logic into iblrigv8

@bimac bimac changed the title Add legacy TrainingChoiceWorld variant 8.29.5 Jun 23, 2026
@bimac
bimac merged commit a0a031e into iblrigv8 Jun 23, 2026
4 checks passed
@bimac
bimac deleted the fix/trainingCW_old_logic branch June 23, 2026 09:50
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.

implement trainingChoiceWorld_c0_CW and trainingPhaseChoiceWorld_c0_CW

3 participants