Skip to content

Architecture coverage test#790

Closed
sofiia-chorna wants to merge 15 commits intometatensor:mainfrom
sofiia-chorna:architecture-coverage-test
Closed

Architecture coverage test#790
sofiia-chorna wants to merge 15 commits intometatensor:mainfrom
sofiia-chorna:architecture-coverage-test

Conversation

@sofiia-chorna
Copy link
Collaborator

@sofiia-chorna sofiia-chorna commented Oct 8, 2025

  • collect coverage for the architectures, make them visible in codecov.io
  • for threshold, use only utils & cli
  • visual badge for coverage of the architectures in the doc

Please go the the Preview tab and select the appropriate PR template:


📚 Documentation preview 📚: https://metatrain--790.org.readthedocs.build/en/790/

@sofiia-chorna sofiia-chorna marked this pull request as ready for review October 8, 2025 10:35
@sofiia-chorna
Copy link
Collaborator Author

Having the issue with coverage check not appearing for PR from the fork, I think I'd need to create a branch in this repo, cherry pick the changes from this forked branch and recreate the PR, unless you have ideas. Perhaps, it is possible to change the branch to merge within the same PR...

@Luthaf
Copy link
Member

Luthaf commented Oct 8, 2025

You should be able to push your branch to this repo and re-open a PR from there

@sofiia-chorna
Copy link
Collaborator Author

Moving this PR to #803

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