search

ForwardSimulation / forrustts

Forward time simulation, rust, and tree sequences.
https://docs.rs/forrustts
MIT License
3 stars 0 forks source link

Tables mod should be pub. #133

Closed molpopgen closed 3 years ago

molpopgen commented 3 years ago

For cases where tables are worked on via dumped vectors, several currently private functions become useful.

Thus, the mod should be pub, and we only import a few key types into the module root/prelude.

molpopgen commented 3 years ago

This one should be kicked down the road until some other aspects of the data models are sorted out, such as where to store mutation times--should we require them to be a property, or should they become part of MutationRecord?

molpopgen commented 3 years ago

such as where to store mutation times--should we require them to be a property, or should they become part of MutationRecord?

This was done in #153

molpopgen commented 3 years ago

Actually, this stuff is all pub enough--the package says pub use tables::* and the stuff we want in there is also pub.