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

MAP analysis with Mitocheck Single cell dataset #1

Merged
merged 38 commits into from
Jan 23, 2024

Conversation

axiomcura
Copy link
Member

@axiomcura axiomcura commented Dec 6, 2023

This PR introduces the application of single-cell data into the copairs mAP pipeline. For this implementation, we utilized the Mitocheck dataset, which is available here.

Note to reviewers:
Please only focuse on the .py since they are the nbconverted notebooks, figures, and README documentation.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@axiomcura axiomcura marked this pull request as ready for review January 22, 2024 20:48
Copy link
Member

@MikeLippincott MikeLippincott left a comment

Choose a reason for hiding this comment

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

The plots look really good! Exciting to see this implementation. Please read and address comments prior to merging. I am approving as comments don't change the function of the code rather clean it up and document it better. Great job.

.gitignore Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
.pre-commit-config.yaml Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
notebooks/mitocheck-map-plotting.py Outdated Show resolved Hide resolved
notebooks/mitocheck-map-plotting.py Show resolved Hide resolved
notebooks/mitocheck-map-plotting.py Show resolved Hide resolved
notebooks/mitocheck-map-plotting.py Outdated Show resolved Hide resolved
notebooks/mitocheck-map-analysis.py Show resolved Hide resolved
axiomcura and others added 7 commits January 23, 2024 11:39
Co-authored-by: Mike Lippincott <58147848+MikeLippincott@users.noreply.github.com>
Co-authored-by: Mike Lippincott <58147848+MikeLippincott@users.noreply.github.com>
Co-authored-by: Mike Lippincott <58147848+MikeLippincott@users.noreply.github.com>
@axiomcura
Copy link
Member Author

Hey @MikeLippincott thanks a lot for your review! Was very helpful! Will be merging now!

@axiomcura axiomcura merged commit e710de2 into WayScience:main Jan 23, 2024
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.

None yet

2 participants