Skip to content
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

Comment out video section for mini-rec in Update 20_Position_Trodes.ipynb #767

Merged
merged 2 commits into from
Jan 16, 2024

Conversation

edeno
Copy link
Collaborator

@edeno edeno commented Jan 12, 2024

Description

Comment out video section for mini-rec as it does not come with a video file by default.

Fixes #716

Checklist:

  • This PR should be accompanied by a release: (yes/no/unsure)
  • (If release) I have updated the CITATION.cff
  • I have updated the CHANGELOG.md
  • I have added/edited docs/notebooks to reflect the changes

@edeno edeno requested a review from CBroz1 January 12, 2024 21:54
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@edeno edeno requested a review from samuelbray32 January 12, 2024 21:54
@edeno edeno changed the title Update 20_Position_Trodes.ipynb Comment out video section for mini-rec in Update 20_Position_Trodes.ipynb Jan 12, 2024
@CBroz1 CBroz1 merged commit a8dcea8 into master Jan 16, 2024
9 checks passed
@edeno edeno deleted the fix-trodes-position-tutorial-video branch January 16, 2024 22:38
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.

VideoFile fails to insert in the tutorial using the minirec file
2 participants