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

CI: dynamic testing #764

Merged
merged 16 commits into from
Jan 18, 2022
Merged

CI: dynamic testing #764

merged 16 commits into from
Jan 18, 2022

Conversation

Borda
Copy link
Member

@Borda Borda commented Jan 17, 2022

What does this PR do?

Addresses #678 (comment)

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@Borda Borda added enhancement New feature or request test / CI testing or CI labels Jan 17, 2022
@Borda Borda added this to the v0.8 milestone Jan 17, 2022
@codecov
Copy link

codecov bot commented Jan 17, 2022

Codecov Report

Merging #764 (063a955) into master (fd6eebf) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #764   +/-   ##
=====================================
  Coverage      71%    71%           
=====================================
  Files         171    171           
  Lines        6926   6926           
=====================================
  Hits         4917   4917           
  Misses       2009   2009           

@Borda Borda force-pushed the ci/dynamic-testing branch 2 times, most recently from 9c1d7ba to 613800e Compare January 17, 2022 10:43
@Borda Borda marked this pull request as ready for review January 17, 2022 12:50
@Borda Borda force-pushed the ci/dynamic-testing branch from dd73afc to 19032b8 Compare January 17, 2022 13:16
@Borda Borda force-pushed the ci/dynamic-testing branch from 19032b8 to 8c2767c Compare January 17, 2022 18:48
@Borda Borda force-pushed the ci/dynamic-testing branch from f6ff8f1 to 38033a0 Compare January 17, 2022 19:54
.github/workflows/ci_test-full.yml Outdated Show resolved Hide resolved
.github/workflows/ci_test-conda.yml Outdated Show resolved Hide resolved
@Borda Borda enabled auto-merge (squash) January 18, 2022 15:22
@SkafteNicki
Copy link
Member

Should we have some tests that the assistant is doing what it is supposed to do?

@Borda
Copy link
Member Author

Borda commented Jan 18, 2022

Should we have some tests that the assistant is doing what it is supposed to do?

I had checked it on other pr #764 and if we merge it we can also test it on #773 and #769

@Borda Borda disabled auto-merge January 18, 2022 18:41
@Borda Borda merged commit b8e67f9 into master Jan 18, 2022
@Borda Borda deleted the ci/dynamic-testing branch January 18, 2022 18:42
@Borda
Copy link
Member Author

Borda commented Jan 18, 2022

seems to be working as expected... run just text-domain for this related deprecation pruning
image
https://github.com/PyTorchLightning/metrics/pull/773/checks

@Borda
Copy link
Member Author

Borda commented Jan 18, 2022

or the similar just fro improving one classification metric:
https://github.com/PyTorchLightning/metrics/pull/769/checks

@mergify mergify bot added the ready label Jan 18, 2022
@Borda
Copy link
Member Author

Borda commented Jan 18, 2022

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready test / CI testing or CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants