search

ForwardSimulation / forrustts

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

Nested list access performance #143

Closed molpopgen closed 3 years ago

molpopgen commented 3 years ago

All of the current data access functions are range checked, yet we generate the final return value with [], which is also range checked. The second accesses should use get_unchecked.

molpopgen commented 3 years ago

Another idea is to match on get, returning an error when None is seen.

molpopgen commented 3 years ago

Going to close this for now. It is hard to argue this is a major bottleneck, and the extra safety is probably a good thing.