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

Code tree reorganization #4

Open
apjanke opened this issue Jan 15, 2021 · 0 comments
Open

Code tree reorganization #4

apjanke opened this issue Jan 15, 2021 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@apjanke
Copy link
Owner

apjanke commented Jan 15, 2021

Implement code organization updates towards the goal of fewer files per folder (package folders or otherwise). For instance:

  1. In Utils package: consider separating out the pure coding utilities from the domain-oriented npix utilities
  2. In root package: consider separation of some/all functions from the classes, perhaps via an additional sub-package
  3. In Neuropixels package: consider separation of related function such as for file reading/writing/exporting (e.g. readNPY & writeNPY) & consider whether some functions (e.g. runKilosort2) can become public methods of classes defined in the package
@apjanke apjanke self-assigned this Jan 15, 2021
@apjanke apjanke added the enhancement New feature or request label Jan 15, 2021
@apjanke apjanke added this to Needs triage in neuropixel-jankify via automation Jan 15, 2021
@apjanke apjanke moved this from Needs triage to High priority in neuropixel-jankify Jan 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
neuropixel-jankify
High priority
Development

No branches or pull requests

1 participant