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

removed auto val reduce #2462

Merged
merged 1 commit into from
Jul 2, 2020
Merged

removed auto val reduce #2462

merged 1 commit into from
Jul 2, 2020

Conversation

williamFalcon
Copy link
Contributor

@williamFalcon williamFalcon commented Jul 2, 2020

Fixes #2456

We're undoing the auto val reduce. However this will be added in a few PRs from now via structured results where users will have the option to enable for a specific metric.

@mergify mergify bot requested a review from a team July 2, 2020 10:39
@codecov
Copy link

codecov bot commented Jul 2, 2020

Codecov Report

Merging #2462 into master will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #2462   +/-   ##
======================================
  Coverage      88%     89%           
======================================
  Files          69      69           
  Lines        5530    5517   -13     
======================================
- Hits         4891    4888    -3     
+ Misses        639     629   -10     

@Borda Borda added bug Something isn't working feature Is an improvement or enhancement labels Jul 2, 2020
@Borda Borda added this to the 0.8.x milestone Jul 2, 2020
@mergify mergify bot requested a review from a team July 2, 2020 10:47
@williamFalcon williamFalcon merged commit afdfba1 into master Jul 2, 2020
@Borda Borda deleted the reduce branch July 2, 2020 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature Is an improvement or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multi-gpu training triggers CUDA out of memory error
2 participants