wri / gfw-raster-analysis-lambda

GFW Raster Analysis in AWS Lambda
8 stars 4 forks source link

Keep ints as ints #81

Closed jterry64 closed 8 months ago

jterry64 commented 8 months ago

Pull request checklist

Please check if your PR fulfills the following requirements:

Pull request type

Please check the type of change your PR introduces:

What is the current behavior?

The new version of pandas occasionally converts int values to floats, making the results return as .0

What is the new behavior?

Check if float return values can be integers, and convert to integers if so.

Does this introduce a breaking change?

codecov-commenter commented 8 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e730c98) 94.58% compared to head (91c7553) 94.52%.

Files Patch % Lines
raster_analysis/tiling.py 83.33% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #81 +/- ## =========================================== - Coverage 94.58% 94.52% -0.06% =========================================== Files 19 19 Lines 1256 1261 +5 =========================================== + Hits 1188 1192 +4 - Misses 68 69 +1 ``` | [Flag](https://app.codecov.io/gh/wri/gfw-raster-analysis-lambda/pull/81/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wri) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/wri/gfw-raster-analysis-lambda/pull/81/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wri) | `94.52% <83.33%> (-0.06%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=wri#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.