search

ashander / ftprime

Forward-time simulation of the msprime data structure (for development)
2 stars 1 forks source link

Revisit / document the behavior `.sequence_length` attribute #45

Closed ashander closed 6 years ago

ashander commented 6 years ago

introduced in #44 with behavior: if you provide edges= or tree sequence= in the constructor, the argument =sequence_length does nothing, but if you do not provide either you must provide a squence_length

ashander commented 6 years ago

@petrelharp said

I think that sequence_length if passed in explicitly should override ts.sequence_length.

which does seem good and will just require a small change in flow control in the init and possibly a tweak or two to the tests