-
Notifications
You must be signed in to change notification settings - Fork 0
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
Connects to #361. PASS1B-06 metabolomics DEA search result updates. #363
Conversation
… layout as a result
…data updates signup component and the affected page UIs
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.
checked the metabolomics data search and the subscription link. They look good.
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.
Mostly looks good to me. My one piece of feedback is that the "module" prior to the office hours/signup "module" doesn't indicate that the user can scroll further down to see the office hours signups/data release subscription.
Also, I feel like the wording here can be workshopped:
Deep dive into our source codes integral to the MoTrPAC 6 month old rats performing endurance training exercise study, from ingestion to QC and from processing to analysis.
Specifically, "...into our source codes integral..." reads awkwardly to me.
@cteng585 thank you for the comments. How would you change the wording for the code repo section? |
Key Changes:
metabolite
field used for retrieving data has now been changed tometabolite_refmet
due to re-indexing metabolite DEA results usingmeta-regression
datasets.