search

tsattler / RansacLib

Template-based implementation of RANSAC and its variants in C++
BSD 3-Clause "New" or "Revised" License
356 stars 48 forks source link

Remove usage of ModelVector.size() #8

Closed Anton3 closed 4 years ago

Anton3 commented 4 years ago

Fixes #7

Anton3 commented 4 years ago

My Solver only produces a single Model, so for optimization purposes, I had to create a SingleList<T> class, which is basically T + operator[]. It might be a rather common scenario. Does it make sense to put it somewhere in the lib?

Edit: actually, ModelVector is only created once per EstimateModel(), so std::vector should be OK, no need for premature optimization.

tsattler commented 4 years ago

Looks good to me, thank you very much!