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

Separate conversion folder for benisty_2024 and lohani_2022 #14

Merged
merged 8 commits into from
May 16, 2024

Conversation

alessandratrapani
Copy link
Collaborator

@alessandratrapani alessandratrapani commented May 10, 2024

I needed to separate the project in two:

  • one (already complete) for the Lohani22 paper that has only meso imaging + behavior,
  • one for Benisty24 that has everything Lohani22 has, but different metadata and additional data streams: only 2p; suite2p segmentation, dual 2p-meso imaging, manual segmentation

NB: this PR needs to be reviewed before #15 and #17

Copy link
Collaborator

@weiglszonja weiglszonja left a comment

Choose a reason for hiding this comment

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

looks ok, I'm guessing you renamed benisty_2022 to lohani_2022 in a follow-up, because here I see benisty_2022 and benisty_2024

@alessandratrapani
Copy link
Collaborator Author

alessandratrapani commented May 16, 2024

looks ok, I'm guessing you renamed benisty_2022 to lohani_2022 in a follow-up, because here I see benisty_2022 and benisty_2024

Yeah it's a bit confusing. At the time I created this repo the reference paper was a pre-print--> Benisty2022, now it got published and it's Benisty2024. However the first dataset shared belong to Lohani2022, so now I created 2 conversion projects lohani_2022 and benisty_2024 and remove benisty_2022

@alessandratrapani alessandratrapani merged commit 214e6b1 into main May 16, 2024
1 of 2 checks passed
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.

2 participants