Skip to content
This repository has been archived by the owner on Jun 7, 2023. It is now read-only.

Add notebook for Allen Institute workshop #83

Merged
merged 12 commits into from
Sep 21, 2022
Merged

Conversation

kabilar
Copy link
Collaborator

@kabilar kabilar commented Sep 21, 2022

No description provided.

Copy link
Contributor

@CBroz1 CBroz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left minor feedback. Will merge early morning regardless and coordinate push of new image

notebooks/py_scripts/2022-allen-institute-workshop.py Outdated Show resolved Hide resolved
session_datetime='2022-09-01 19:16:44',
scan_id=0,
paramset_idx=1,
processing_output_dir='subject3/210107_run00_orientation_8dir/suite2p_1'))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See above re absolute path on code book /home/inbox/{djarchive-version}

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe that the directory attribute within the table should be treated as a relative path, but I could be mistaken.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, no, you're right. relative in this table

CHANGELOG.md Outdated Show resolved Hide resolved
@kabilar
Copy link
Collaborator Author

kabilar commented Sep 21, 2022

Thanks @CBroz1. I also added suite2p to the requirements so that it could be triggered within the notebook. It is not a problem if it runs slow with the resources currently allocated on codebook, as I just want the analysis to start and then I will end the presentation.

@CBroz1 CBroz1 merged commit eae2113 into datajoint:main Sep 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants