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

Name all ConfigureHeadstage nodes the same #371

Open
ChucklesOnGitHub opened this issue Nov 11, 2024 · 1 comment
Open

Name all ConfigureHeadstage nodes the same #371

ChucklesOnGitHub opened this issue Nov 11, 2024 · 1 comment
Assignees
Milestone

Comments

@ChucklesOnGitHub
Copy link

This is probably unattainable, but here goes. It gets confusing to sort through headstages and devices but this could be easier if the neuropixels headstages configure nodes were called ConfigureHeadstageNeuropixelsXXX so all the nodes to configure headstages appear listed together.

image

Or maybe hide the ConfigureDevice nodes in a library for developers? Just thought I'd ask :)

@jonnew
Copy link
Member

jonnew commented Nov 15, 2024

This is good idea and actually might be kinda easy while maintaining backwards compatibility using the the renaming trick here:

public class ConfigureNeuropixelsV1eBno055 : ConfigurePolledBno055

@jonnew jonnew added this to the 0.5.0 milestone Nov 15, 2024
@jonnew jonnew self-assigned this Nov 15, 2024
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

No branches or pull requests

2 participants