Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Rename valid_ to val_ #197

Merged
merged 2 commits into from
Mar 30, 2021
Merged

Rename valid_ to val_ #197

merged 2 commits into from
Mar 30, 2021

Conversation

carmocca
Copy link
Contributor

What does this PR do?

Fixes #137

@codecov
Copy link

codecov bot commented Mar 29, 2021

Codecov Report

Merging #197 (fb46458) into master (ba34bf4) will not change coverage.
The diff coverage is 77.77%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #197   +/-   ##
=======================================
  Coverage   84.30%   84.30%           
=======================================
  Files          55       55           
  Lines        2447     2447           
=======================================
  Hits         2063     2063           
  Misses        384      384           
Flag Coverage Δ
unittests 84.30% <77.77%> (ø)

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

Impacted Files Coverage Δ
flash/text/classification/data.py 81.81% <ø> (ø)
flash/text/seq2seq/core/data.py 83.60% <ø> (ø)
flash/text/seq2seq/summarization/data.py 85.71% <ø> (ø)
flash/text/seq2seq/translation/data.py 90.00% <ø> (ø)
flash/vision/embedding/model.py 73.17% <0.00%> (ø)
flash/data/data_module.py 76.76% <58.33%> (ø)
flash/tabular/classification/data/data.py 88.63% <75.00%> (ø)
flash/vision/classification/data.py 87.65% <90.00%> (ø)
flash/data/process.py 77.88% <100.00%> (ø)
flash/vision/classification/model.py 95.45% <100.00%> (ø)
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba34bf4...fb46458. Read the comment docs.

@carmocca carmocca enabled auto-merge (squash) March 30, 2021 13:40
@Borda Borda disabled auto-merge March 30, 2021 13:54
@Borda Borda merged commit 0b967bb into master Mar 30, 2021
@Borda Borda deleted the valid-to-val branch March 30, 2021 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename valid_ to val_
3 participants