popsim-consortium / stdpopsim

A library of standard population genetic models
GNU General Public License v3.0
125 stars 87 forks source link

deprecate `--length-multiplier` CLI option #1349

Open grahamgower opened 2 years ago

grahamgower commented 2 years ago

--length-multiplier doesn't compose well with other options, and specifying a fixed length is an easier way to get short-running tests.

grahamgower commented 2 years ago

Also deprecate/remove the length_multiplier options to Species.get_contig() and Contig.species_contig(). No one should be using these outside our test suite.

jeromekelleher commented 2 years ago

SGTM

nspope commented 2 years ago

Length multiplier via -l is used pretty heavily in the CLI portion of the tutorial, these examples should use --left and --right once #1348 is merged.