-
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
Add JUMP analysis figure #55
Conversation
gwaybio
commented
Feb 11, 2024
•
edited
Loading
edited
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
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.
LGTM! I made some small comments for you to address, but feel free to merge after 😄
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.
Clean and simple figure! Only nit-pick comment is that panel A has a bit more whitespace than panel B, especially on the left side. Might consider trying to reduce the whitespace more if possible.
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.
will do!
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.
Some comments:
- What is the difference between panels B and C? Is it that the B is the final model and C is shuffled? I can see they both focus only on compound perturbation which makes since as other perturbations did not show much enrichment at all.
- The y-axis titles are very close to the panel letters. Maybe it would work if the top plot for each panel could be moved in alignment with panel A, which might help with the whitespace.
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.
This is great feedback!
B and C have to do with time points. I add a ggtitle()
to make this more clear.
I've updated the y axis titles and have moved panel A alignment closer to the label.