mobility-team / mobility

Mobility, an open-source library for mobility modelisation
MIT License
16 stars 11 forks source link

Improve docstrings #34

Closed Mind-the-Cap closed 1 year ago

Mind-the-Cap commented 1 year ago

Improve docstrings and comments for

Also:

codecov[bot] commented 1 year ago

Codecov Report

Merging #34 (c71eb27) into main (64718fc) will increase coverage by 0.07%. The diff coverage is 83.33%.

@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   94.69%   94.76%   +0.07%     
==========================================
  Files          10       10              
  Lines         490      497       +7     
==========================================
+ Hits          464      471       +7     
  Misses         26       26              
Impacted Files Coverage Δ
mobility/parsers/entd_2008.py 99.33% <ø> (ø)
mobility/get_survey_data.py 91.48% <33.33%> (-1.85%) :arrow_down:
mobility/parsers/emp_2019.py 99.47% <100.00%> (+<0.01%) :arrow_up:
mobility/safe_sample.py 90.00% <100.00%> (+1.11%) :arrow_up:
mobility/trip_sampler.py 88.23% <100.00%> (+1.47%) :arrow_up:
test/test_parsers.py 100.00% <100.00%> (ø)
test/test_trip_sampler.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Mind-the-Cap commented 1 year ago

L'échantillonnage de pandas a un paramètre random_state que nous pourrions utiliser pour avoir des valeurs stables pour les tests, qu'en pensez-vous ?