Skip to content

adds autowater logic#49

Open
micahwoodard wants to merge 1 commit intomainfrom
feat-autowater
Open

adds autowater logic#49
micahwoodard wants to merge 1 commit intomainfrom
feat-autowater

Conversation

@micahwoodard
Copy link
Collaborator

No description provided.

is_autowater_trial = self._are_autowater_conditions_met()
is_left_autowater = is_left_baited and is_autowater_trial
is_right_autowater = is_right_baited and is_autowater_trial

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@bruno-f-cruz I'm not sure we can implement auto water exactly how it is done in the original dynamic foraging code. Reading through the code, it seems like autowater can be given on the right or left side simultaneously. I'm looking here. I don't see how we could do this currently since the variable is_auto_response_right specifies one or the other or neither.

Copy link
Member

Choose a reason for hiding this comment

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

Check here #35 (comment)

I don't think it is

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ah okay reading through this thread i have more questions so I will move discussion there. Thanks!

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.

2 participants