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

Add 60 minute tutorial notebook #109

Merged
merged 8 commits into from
May 16, 2023
Merged

Add 60 minute tutorial notebook #109

merged 8 commits into from
May 16, 2023

Conversation

kushalbakshi
Copy link
Collaborator

@kushalbakshi kushalbakshi commented Mar 14, 2023

In this PR, I've made the performed the following:

  • Add a 60 minute tutorial notebook for workflow-calcium-imaging
  • Test the 60 minute tutorial notebook end-to-end locally using Dev Containers in VS Code.
  • Update CHANGELOG
  • Update version.py

@kushalbakshi kushalbakshi marked this pull request as ready for review March 14, 2023 21:37
@kushalbakshi
Copy link
Collaborator Author

@kabilar, besides the usual checks, please let me know if I should continue to fill out the event alignment schemas for this tutorial notebook. I'm not sure if we're keeping that or if that is being dropped from this version of the workflow

@kabilar kabilar self-assigned this May 1, 2023
Copy link
Collaborator

@kabilar kabilar left a comment

Choose a reason for hiding this comment

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

Thanks @kushalbakshi! Great work. I have some suggestions for the text that I will submit as a subsequent pull request since commenting on notebooks in GitHub is not currently supported.

@kabilar
Copy link
Collaborator

kabilar commented May 16, 2023

@kabilar, besides the usual checks, please let me know if I should continue to fill out the event alignment schemas for this tutorial notebook. I'm not sure if we're keeping that or if that is being dropped from this version of the workflow

Thanks. We will keep those schemas in this workflow but can add them to a separate notebook to keep this tutorial notebook streamlined.

@kabilar kabilar merged commit 482f203 into datajoint:main May 16, 2023
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.

None yet

2 participants