Repository navigation
Store the rewards of left footshock probes as left rewards - #75
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
15 of 25 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #73
Details (AI-generated)
seiler_2024_metadata.yaml,Footshock Degradation Leftnow maps array D toleft_reward_timesand array B, which the program never writes, toright_reward_times.Seiler2024CSVBehaviorInterfacetakes the names forLeftRewardTs(B) andRightRewardTs(D) from the same map.tests/test_footshock_rewards.py), and a left footshock CSV laid out like the real exports (tests/test_csv_footshock.py).pytest tests: 32 passed; both new tests fail onmain.left_shock.ipynb) shows the program lines that write B and D. In the 159 published left footshock files with rewards, all 8822right_reward_timesfall on left nose pokes. It reconverts the 154 left footshock probes the fixed conversion still writes (the other 8 are "Unknown" duplicates removed in Skip duplicated DLS Excitatory sessions #48). Each now hasleft_reward_timesand noright_reward_times, and after the rename every dataset matches the published file, exceptfootshock_timesin the 8 CSV sessions, which Read footshock times from CSV exports #54 added.🤖 Generated with Claude Code