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

Fix 0.0 value floats #452

Merged
merged 1 commit into from
Apr 26, 2021
Merged

Fix 0.0 value floats #452

merged 1 commit into from
Apr 26, 2021

Conversation

wild-endeavor
Copy link
Contributor

Signed-off-by: wild-endeavor [email protected]

TL;DR

Converting a float of 0 fails currently.

Type

  • Bug Fix
  • Feature
  • Plugin

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Complete description

Using None check.

Tracking Issue

https://flyte-org.slack.com/archives/CREL4QVAQ/p1619449414338000

Signed-off-by: wild-endeavor <[email protected]>
@codecov
Copy link

codecov bot commented Apr 26, 2021

Codecov Report

Merging #452 (c8edea9) into master (d77a773) will increase coverage by 0.01%.
The diff coverage is 87.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #452      +/-   ##
==========================================
+ Coverage   84.39%   84.40%   +0.01%     
==========================================
  Files         344      344              
  Lines       25835    25841       +6     
  Branches     2114     2114              
==========================================
+ Hits        21804    21812       +8     
+ Misses       3458     3457       -1     
+ Partials      573      572       -1     
Impacted Files Coverage Δ
flytekit/core/type_engine.py 82.67% <50.00%> (+0.65%) ⬆️
tests/flytekit/unit/core/test_type_engine.py 100.00% <100.00%> (ø)

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 d77a773...c8edea9. Read the comment docs.

@wild-endeavor wild-endeavor merged commit 5d03584 into master Apr 26, 2021
max-hoffman pushed a commit to dolthub/flytekit that referenced this pull request Apr 29, 2021
Signed-off-by: wild-endeavor <[email protected]>
Signed-off-by: Max Hoffman <[email protected]>
max-hoffman pushed a commit to dolthub/flytekit that referenced this pull request Apr 29, 2021
Signed-off-by: wild-endeavor <[email protected]>
Signed-off-by: Max Hoffman <[email protected]>
max-hoffman pushed a commit to dolthub/flytekit that referenced this pull request Apr 29, 2021
Signed-off-by: wild-endeavor <[email protected]>
Signed-off-by: Max Hoffman <[email protected]>
max-hoffman pushed a commit to dolthub/flytekit that referenced this pull request May 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants