Skip to content

Feat bias autowater#1644

Open
micahwoodard wants to merge 2 commits intocurriculum-bug-fixfrom
feat-bias-autowater
Open

Feat bias autowater#1644
micahwoodard wants to merge 2 commits intocurriculum-bug-fixfrom
feat-bias-autowater

Conversation

@micahwoodard
Copy link
Collaborator

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:

  • treats bias watering like autowatering

What issues or discussions does this update address?

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

Describe any manual update steps for task computers

Was this update tested in 446/447?

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

@micahwoodard micahwoodard changed the base branch from main to curriculum-bug-fix October 27, 2025 20:45
@micahwoodard
Copy link
Collaborator Author

@hanhou @alexpiet this is a pr to treat the bias watering algorithm like autowater. I wanted to get your eyes on it to make sure how I implemented works within the current framework for how auowater is handled. Thanks!

else:
self.CurrentAutoRewardTrial = [0, 0]
elif self.bias_AutoWater != [0, 0]: # auto water due to bias
np.logical_or(self.bias_AutoWater, self.CurrentAutoRewardTrial)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are you setting the output of this logical_or to a variable?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh nice catch I meant to set to CurrentAutoRewardTrial

@alexpiet
Copy link
Collaborator

@micahwoodard it seems good to me, but I haven't tested it. Its been such a long time since I looked at this part of the code, I don't have much confidence in my review

@hanhou
Copy link
Collaborator

hanhou commented Oct 30, 2025

No apparent issues from my side. @XX-Yin could you please check if the new logic Micah implemented is compatible with our previous autowater algorithm?

@micahwoodard is it possible to use the simulation functionality (with some bias) to test it out?

@XX-Yin
Copy link
Collaborator

XX-Yin commented Oct 30, 2025

@micahwoodard @hanhou I haven’t caught up on this thread yet, but the simulation functionality should be working.

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