Skip to content
This repository was archived by the owner on Dec 8, 2023. It is now read-only.

Conversation

kabilar
Copy link
Collaborator

@kabilar kabilar commented Jul 8, 2022

Updates

  • Update README
    • Add link to YouTube tutorial
    • Add documentation on the ephys_precluster module
    • Update image links to raw file
  • Update text in Jupyter notebooks
  • Jupyter notebooks included updates with black code formatting. There were conflicts between jupytext and black. I ended up creating a new python script with jupytext, overwriting the previous python script, and then formatting the python script with black. It would also be worth a review of the ipynb files to make sure no issues arose with jupytext or black.
  • Remove electrode localization schema activation from pipeline.py since this is included in localization.py.

TODO

  • Once this pull request is merged, please create a tag, push the tag to upstream, and manually create a new release. Thanks!

@kabilar kabilar added documentation Improvements or additions to documentation enhancement New feature or request labels Jul 8, 2022
@kabilar kabilar added this to the 2022Q3 milestone Jul 8, 2022
@kabilar kabilar linked an issue Jul 11, 2022 that may be closed by this pull request
@sidhulyalkar sidhulyalkar merged commit 374b016 into datajoint:main Jul 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Link to video tutorial
2 participants