Should `population()` have default values?

epiverse-trace / epidemics

A library of published compartmental epidemic models, and classes to represent demographic structure, non-pharmaceutical interventions, and vaccination regimes, to compose epidemic scenarios.

https://epiverse-trace.github.io/epidemics/

Other

8 stars 2 forks source link

Should `population()` have default values? #123

Closed jamesmbaazam closed 11 months ago

jamesmbaazam commented 11 months ago

The demography_vector argument of population() currently has a default value of 67e6. Shouldn't it be empty to avoid accidental use?

pratikunterwegs commented 11 months ago

Yes, thanks for spotting. Can fix, or please feel welcome to make a PR with a fix.