-
Notifications
You must be signed in to change notification settings - Fork 3
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
Data ag kuppe #109
base: main
Are you sure you want to change the base?
Data ag kuppe #109
Conversation
Also #32 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove this file
|
||
|
||
BAD_GENES = ["eGFP","mCherry2","tdToma"] | ||
META_DICT = {"technology":"Merfish"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change to "MERFISH"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given that this does not download a public dataset but rather curates data that already needs to be present I am not sure if it makes sense to ad the script to the repo?
Don't get me wrong we can definitely still use the data!
@naveedishaque what is our take on this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure. I would say we keep it as it is required to take the data from the current source. IF the sources changes, then we don't need the script, but right now we do. Maybe tag the dataset as a "v1"?
This code setups our heart and kidney dataset.
This is preliminary as our datasets are currently behind credentials in our bucket, need to transfer them to git LFS or similiar at some point.
Code should in principle work with any Vizgen Merscope output