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

Dev/augmentations #31

Closed
wants to merge 10 commits into from
Closed

Dev/augmentations #31

wants to merge 10 commits into from

Conversation

rhoadesScholar
Copy link
Member

No description provided.

@rhoadesScholar rhoadesScholar linked an issue Feb 8, 2024 that may be closed by this pull request
@rhoadesScholar
Copy link
Member Author

@adjavon Need to replace dacapo.ElasticFuse with gunpowder's new elastic augmentation. Please! :)

Copy link

codecov bot commented Feb 15, 2024

Codecov Report

Attention: Patch coverage is 88.88889% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 64.98%. Comparing base (bc75625) to head (2ae896b).
Report is 178 commits behind head on dev/main.

❗ Current head 2ae896b differs from pull request most recent head d492505. Consider uploading reports for the commit d492505 to get more accurate results

Files Patch % Lines
...ents/trainers/gp_augments/gaussian_noise_config.py 88.88% 1 Missing ⚠️
.../experiments/trainers/gp_augments/simple_config.py 85.71% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           dev/main      #31      +/-   ##
============================================
+ Coverage     64.30%   64.98%   +0.67%     
============================================
  Files           181      179       -2     
  Lines          5483     5380     -103     
============================================
- Hits           3526     3496      -30     
+ Misses         1957     1884      -73     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rhoadesScholar rhoadesScholar changed the base branch from main to dev/main February 15, 2024 21:50
@rhoadesScholar
Copy link
Member Author

@adjavon is this pretty much ready to merge? any breaking changes?

@adjavon
Copy link
Contributor

adjavon commented Mar 1, 2024

@adjavon is this pretty much ready to merge? any breaking changes?

No breaking changes as far as I can tell, but the tutorials still need to be updated.

@mzouink mzouink deleted the branch dev/main May 9, 2024 15:33
@mzouink mzouink closed this May 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Gaussian Noise Augmentation
3 participants