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

Add JUMP analysis figure #55

Merged
merged 9 commits into from
Mar 8, 2024
Merged

Conversation

gwaybio
Copy link
Member

@gwaybio gwaybio commented Feb 11, 2024

main_figure_5_jump_application

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@gwaybio gwaybio requested a review from jenna-tomkinson March 7, 2024 15:53
@gwaybio gwaybio marked this pull request as ready for review March 7, 2024 15:53
@gwaybio gwaybio changed the title [WIP] Add JUMP analysis figure Add JUMP analysis figure Mar 7, 2024
Copy link
Member

@jenna-tomkinson jenna-tomkinson left a 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 😄

Copy link
Member

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.

Copy link
Member Author

Choose a reason for hiding this comment

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

will do!

Copy link
Member

Choose a reason for hiding this comment

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

Some comments:

  1. 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.
  2. 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.

Copy link
Member Author

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.

@gwaybio
Copy link
Member Author

gwaybio commented Mar 8, 2024

Updated figure 5

main_figure_5_jump_application

Updated sup fig

Uploading supplementary_jump_enrichment_shuffled_and_extended.png…

@gwaybio gwaybio merged commit 18488dd into WayScience:main Mar 8, 2024
@gwaybio gwaybio deleted the jump-umap-figure branch March 8, 2024 13:03
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