akrherz / iem-web-services

Code servicing IEM API requests
Apache License 2.0
0 stars 1 forks source link

rollback pandas changes until tuple support is resolved #90

Closed akrherz closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #90 (4dc2ad3) into main (5bbb9e1) will increase coverage by 0.27%. The diff coverage is 42.10%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
+ Coverage   75.70%   75.97%   +0.27%     
==========================================
  Files          43       43              
  Lines        1564     1565       +1     
==========================================
+ Hits         1184     1189       +5     
+ Misses        380      376       -4     
Impacted Files Coverage Δ
src/iemws/services/currents.py 75.51% <31.25%> (+6.88%) :arrow_up:
src/iemws/services/cow.py 91.17% <100.00%> (+0.04%) :arrow_up:
src/iemws/services/mos.py 47.36% <100.00%> (+1.42%) :arrow_up:
src/iemws/services/raobs_by_year.py 52.17% <100.00%> (+2.17%) :arrow_up:

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 5bbb9e1...4dc2ad3. Read the comment docs.

akrherz commented 2 years ago

awaiting new conda-forge build of pyiem conda-forge/pyiem-feedstock#8

akrherz commented 2 years ago

This to this SO response, we are trying to roll again with sqlalchemy

lgtm-com[bot] commented 2 years ago

This pull request introduces 2 alerts when merging 4dc2ad3f55b3bad6963b037863daefec3c1eb561 into 5bbb9e1986393b09ad90f4f4fb2c1786216b5dd1 - view on LGTM.com

new alerts: