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

Add NP1 and NP2 Headstage Tutorials #64

Closed
wants to merge 11 commits into from
Closed

Add NP1 and NP2 Headstage Tutorials #64

wants to merge 11 commits into from

Conversation

cjsha
Copy link
Member

@cjsha cjsha commented Sep 19, 2024

As of commit 7d0152a, the tutorials lack integration with GUI. I think this should be added by @bparks13 before merging.

Aside from that, feel free to review and make changes as you see appropriate.

"Real world" content is outside of the scope of this PR.

cjsha and others added 8 commits September 6, 2024 17:04
- This change implements breakout tutorial only
- This provides scaffolding for other hardware tutorials (np ones are mostly
  written, they need to be restructured & moved over from another)
- Changed log severity from warning to suggestion
- Added `Devices` property category
- Capitalized sentence start
- Use more original breakout board content from jon
- Write like a human
- Better flow throughout and across pages
- Parahraphs instead of bullet points
- Breakout board schematic
- workflow has its own page so it can be at the top of page
- add real-world content
- port status and bno pages are created using templates
- configuration page is hybrid
- the np probe page is custom content written in the markdown file
- they configuration pages require better integration with GUI
@cjsha cjsha requested a review from bparks13 September 19, 2024 13:47
@cjsha cjsha assigned bparks13 and unassigned bparks13 Sep 19, 2024
@cjsha cjsha requested a review from jonnew September 19, 2024 14:04
- After merging, the configuration template changed. This required
  making a couple changes to the configuration pages
- Additional comments regarding setting buffersize property
- Some changes to the getting-started pages
  - don't need to install OpenEphys.Onix1 if OpenEphys.Onix1.Design is
    being installed
  - link directly to hardware guides instead of describing their
    existence on the getting-started page
@bparks13
Copy link
Member

Due to merge conflicts, development of this branch has been migrated to #71

@bparks13 bparks13 closed this Sep 25, 2024
@cjsha cjsha deleted the issues-35-36 branch October 3, 2024 21:19
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.

Tutorial: Neuropixels 2.0e headstage Tutorial: Neuropixels 1.0e headstage
2 participants