wri / gfw-raster-analysis-lambda

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

SQL Parity #54

Closed jterry64 closed 2 years ago

jterry64 commented 2 years 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?

AS, LIMIT and ORDER BY are unsupported and will be ignored.

What is the new behavior?

Does this introduce a breaking change?

codecov[bot] commented 2 years ago

Codecov Report

Merging #54 (bdc4cde) into develop (cffb583) will increase coverage by 2.92%. The diff coverage is 98.43%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #54      +/-   ##
===========================================
+ Coverage    91.57%   94.50%   +2.92%     
===========================================
  Files           20       19       -1     
  Lines         1140     1146       +6     
===========================================
+ Hits          1044     1083      +39     
+ Misses          96       63      -33     
Flag Coverage Δ
unittests 94.50% <98.43%> (+2.92%) :arrow_up:

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

Impacted Files Coverage Δ
raster_analysis/query.py 96.53% <96.15%> (+0.40%) :arrow_up:
lambdas/raster_analysis/src/lambda_function.py 76.92% <100.00%> (ø)
raster_analysis/globals.py 100.00% <100.00%> (ø)
raster_analysis/query_executor.py 92.47% <100.00%> (ø)
raster_analysis/tiling.py 99.07% <100.00%> (+0.06%) :arrow_up:
tests/e2e/test_raster_analysis.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 cffb583...bdc4cde. Read the comment docs.