-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Not showing meta-analysis experiments in creating score set dropdown menu. #347
Not showing meta-analysis experiments in creating score set dropdown menu. #347
Conversation
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.
Thanks for the changes Estelle! Would you also be able to add a couple tests for the newly searchable meta analysis field? It might require a new meta analysis score set to be added to the test DB.
@@ -43,45 +43,6 @@ | |||
) | |||
|
|||
|
|||
# TODO: Rewrite this function. |
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.
Lets retain this function-- I think it still has utility for just listing all experiments.
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.
Sure. I'll modify it so that we don't need to send the weird q to it.
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.
I think it is ok to just leave it as is for this PR so we can get it released. We can just handle it at some point later-- it isn't hurting anything right now.
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.
Sorry for the delay on this Estelle, looks good!
API codes for VariantEffect/mavedb-ui#296