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

codecov changes to workflow #435

Open
wants to merge 74 commits into
base: master
Choose a base branch
from

Conversation

MukundhMurthy
Copy link
Contributor

Changes made to python-plain-run-test.yml

Uses built in Codecov utility https://github.com/codecov/codecov-action for Github actions

Codecov bot should become active once the app is installed

Copy link
Contributor

@Ukyeon Ukyeon left a comment

Choose a reason for hiding this comment

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

Codecov provides visualizations and reports of the test coverage data, allowing developers to analyze the effectiveness of our test suite in detecting errors or issues in the codebase.

Copy link
Contributor

@Ukyeon Ukyeon left a comment

Choose a reason for hiding this comment

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

@Xiaojieqiu
Please review.

Ukyeon and others added 28 commits May 15, 2023 17:13
bug fix for uninitialization of a variable, invalid_ids
Multiprocessing context 'fork' only works in Mac OS
error raised while writing key 'notfound' in h5py
add the logging info when X_pca, X_umap are added to adata.obsm
Graph Calculus Cooptimization fix
Add X_PCA as special key in DKM and remove .obsm.["X"]
@Sichao25
Copy link
Collaborator

Sichao25 commented Jul 6, 2023

Hi, @MukundhMurthy Thank you for your contribution and the effort you put into your pull request. We appreciate your work on implementing the feature.

Because we need to include the codecov in the next release soon, we fix all issues here in another pull request. So this pull request will be closed. Apologize for any confusion or overlap that may have occurred. If you have any other ideas or features you would like to work on, please let us know.

@Xiaojieqiu
Copy link
Collaborator

Thanks all! If everyone is okay with this. I will close this pull request and let Sichao to complete the relevant tasks on a different branch.

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.

5 participants