UWCubeSat / lost

Lost: Open-source Star Tracker
42 stars 7 forks source link

Pyramid Refresh: Investigate extra shortlisting for first candidate (`i`) #82

Open markasoftware opened 1 year ago

markasoftware commented 1 year ago

Could improve performance depending on parameters. Possibly should automatically detect parameters or make a configurable CLI option.

markasoftware commented 1 year ago

Still technically possible under the refreshed pyramid, and actually because of smart pyramid selection might be more effective...but on the other hand, the vast majority of the work appears to be spent just computing the pair distance hashmaps, and then the actual iteration is pretty quick...sorting the lists again would likely take longer than just doing the iteration.