Replies: 2 comments 3 replies
-
also it seems to me that member variable m_A_ordered, m_index_map, m_active_c_up and m_active_c_down are obsolete. I commented out related code, and it worked just fine, with execution time shortened a little bit (~20 us). |
Beta Was this translation helpful? Give feedback.
-
I can't immediately recall now as it has been a few years since we worked on this implementation. If I recall correctly thought, it's about selecting the constraints to add to the Seidel solver. In the optimal case, we select the optimal constraint right away. Regarding time-complexity though, since the random variant of Seidel LP algorithm is linear w.r.t the number of constraints anyway, so this does not give a theoretical benefit. However practically I think it should help quite a bit. |
Beta Was this translation helpful? Give feedback.
-
Hi, there.
While I was reading the seidel.cpp file, I saw the following comment:
// warmstarting feature: one in two solvers, upper and lower,
// is be selected depending on the sign of g[1]
What does it mean by warmstarting feature? I didn't see how it's working or perhaps it is a todo?
Also, I checked the Seidel's linear algorithm(https://www.cs.cmu.edu/~avrim/451/lectures/lect1021-lpII.pdf), and did't see any warm-start trick, can someone explain the idea?
Beta Was this translation helpful? Give feedback.
All reactions