armgilles / vcub_keeper

Analyse de l'activité des stations Vcub dans la métropole de Bordeaux afin de détecter en amont les stations hors service
https://vcubwatcher.herokuapp.com/
MIT License
2 stars 0 forks source link

Using custum breakpoint calcul, no more pandas cut and add unit test … #145

Closed armgilles closed 1 month ago

armgilles commented 1 month ago

closed #144

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 50.00000% with 5 lines in your changes missing coverage. Please review.

Project coverage is 70.03%. Comparing base (03fd370) to head (7227ffb). Report is 1 commits behind head on dev_polars.

Files with missing lines Patch % Lines
src/vcub_keeper/create/creator.py 50.00% 5 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev_polars #145 +/- ## ============================================== + Coverage 68.45% 70.03% +1.57% ============================================== Files 10 10 Lines 317 317 ============================================== + Hits 217 222 +5 + Misses 100 95 -5 ```

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

codspeed-hq[bot] commented 1 month ago

CodSpeed Performance Report

Merging #145 will degrade performances by 5.02%

Comparing no_more_pd_in_create_station_profilage_activity (7227ffb) with dev_polars (03fd370)

Summary

⚡ 2 improvements ❌ 1 (👁 1) regressions ✅ 8 untouched benchmarks

Benchmarks breakdown

Benchmark dev_polars no_more_pd_in_create_station_profilage_activity Change
test_benchmark_get_transaction_all 1.8 ms 1.5 ms +18.19%
test_benchmark_get_transaction_in 2 ms 1.9 ms +8.09%
👁 test_benchmark_get_transaction_in_big 47.5 ms 50 ms -5.02%