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

Mua notebook #998

Merged
merged 5 commits into from
Jun 5, 2024
Merged

Mua notebook #998

merged 5 commits into from
Jun 5, 2024

Conversation

denissemorales
Copy link
Contributor

added some documentation

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@CBroz1
Copy link
Member

CBroz1 commented Jun 5, 2024

Thanks @denissemorales ! If you're able to run jupytext as described here, that will help us keep the python scripts in sync

@denissemorales
Copy link
Contributor Author

I think I did it, but let me know if there are any issues!

@edeno edeno merged commit d8e5196 into LorenFrankLab:master Jun 5, 2024
7 checks passed
edeno pushed a commit that referenced this pull request Jun 18, 2024
* Give UUID to artifact interval

* Add ability to set smoothing sigma in get_firing_rate (#994)

* add option to set spike smoothing sigma

* update changelog

* Add docstrings to SortedSpikesGroup and Decoding methods (#996)

* Add docstrings

* update changelog

* fix spelling

---------

Co-authored-by: Samuel Bray <samuelbray@som-dfvnn9m-lt.ucsfmedicalcenter.org>

* Add Common Errors doc (#997)

* Add Common Errors

* Update changelog

* Mua notebook (#998)

* documented some of mua notebook

* mua notebook documented

* documented some of mua notebook

* synced py script

* Dandi export and read (#956)

* compile exported files, download dandiset, and organize

* add function to translate files into dandi-compatible names

* add table to store dandi name translation and steps to populate

* add dandiset validation

* add function to fetch nwb from dandi

* add function to change obj_id of nwb_file

* add dandi upload call and fix circular import

* debug dandi file streaming

* fix circular import

* resolve dandi-streamed files with fetch_nwb

* implement review comments

* add admin tools to fix common dandi discrepencies

* implement tool to cleanup common dandi errors

* add dandi export to tutorial

* fix linting

* update changelog

* fix spelling

* style changes from review

* reorganize function locations

* fix circular import

* make dandi dependency optional in imports

* store dandi instance of data in DandiPath

* resolve case of pre-existing dandi entries for export

* cleanup bugs from refactor

* update notebook

* Apply suggestions from code review

Co-authored-by: Chris Broz <Chris.Broz@ucsf.edu>

* add requested changes from review

* make method check_admin_privilege in LabMember

---------

Co-authored-by: Chris Broz <Chris.Broz@ucsf.edu>

* Minor fixes (#999)

* give analysis nwb new uuid when created

* fix function argument

* update changelog

* Fix bug in change in analysis_file object_id (#1004)

* fix bug in change in analysis_file_object_id

* update changelog

* Remove classes for usused tables (#1003)

* #976

* Remove notebook reference

* Non-daemon parallel populate (#1001)

* initial non daemon parallel commit

* resolve namespace and pickling errors

* fix linting

* update changelog

* implement review comments

* add parallel_make flag to spikesorting recording tables

* fix multiprocessing spawn error on mac

* move propert

---------

Co-authored-by: Samuel Bray <samuelbray@som-dfvnn9m-lt.ucsf.edu>

* Update pipeline column for IntervalList

---------

Co-authored-by: Samuel Bray <sam.bray@ucsf.edu>
Co-authored-by: Samuel Bray <samuelbray@som-dfvnn9m-lt.ucsfmedicalcenter.org>
Co-authored-by: Chris Broz <Chris.Broz@ucsf.edu>
Co-authored-by: Denisse Morales-Rodriguez <68555303+denissemorales@users.noreply.github.com>
Co-authored-by: Samuel Bray <samuelbray@som-dfvnn9m-lt.ucsf.edu>
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.

3 participants