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

Baselines for LargeMix #444

Merged
merged 16 commits into from
Aug 26, 2023
Merged

Baselines for LargeMix #444

merged 16 commits into from
Aug 26, 2023

Conversation

DomInvivo
Copy link
Collaborator

Changelogs

  • Added the baselines for LargeMix on the website.

Checklist:

  • Was this PR discussed in an issue? It is recommended to first discuss a new feature into a GitHub issue before opening a PR.
  • Add tests to cover the fixed bug(s) or the new introduced feature(s) (if appropriate).
  • Update the API documentation is a new function is added, or an existing one is deleted.
  • Write concise and explanatory changelogs above.
  • If possible, assign one of the following labels to the PR: feature, fix or test (or ask a maintainer to do it for you).

Closes #372

@DomInvivo DomInvivo added the documentation Improvements or additions to documentation label Aug 26, 2023
@codecov
Copy link

codecov bot commented Aug 26, 2023

Codecov Report

Merging #444 (d1e8cb9) into main (f96c1b6) will decrease coverage by 0.16%.
The diff coverage is 59.11%.

@@            Coverage Diff             @@
##             main     #444      +/-   ##
==========================================
- Coverage   65.52%   65.37%   -0.16%     
==========================================
  Files          90       92       +2     
  Lines        8218     8253      +35     
==========================================
+ Hits         5385     5395      +10     
- Misses       2833     2858      +25     
Flag Coverage Δ
unittests 65.37% <59.11%> (-0.16%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
ipu 49.14% <ø> (ø)

@DomInvivo DomInvivo merged commit 8358a0d into main Aug 26, 2023
7 checks passed
@cwognum
Copy link
Collaborator

cwognum commented Aug 28, 2023

@DomInvivo Please be careful with these PR. It seems this branch was a continuation of mine and besides having WIP code in the main branch now, it could've also led to nasty merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update the baseline page with LargeMix results
2 participants