Sketcher: Switch to perpendicular based instead of "de la Hire" in setting minor axis of ellipse
Signed-off-by: Yash Suthar <yashsuthar983@gmail.com>
This commit is contained in:
committed by
Kacper Donat
parent
54ec582ce9
commit
925eb1df23
@@ -395,7 +395,7 @@ private:
|
||||
auto lprojy = projy.Length(); // Py = b sin t
|
||||
|
||||
if (lprojx > firstRadius) {
|
||||
secondRadius = 0.0;
|
||||
secondRadius = lprojy;
|
||||
}
|
||||
else {
|
||||
double t = std::acos(lprojx / firstRadius);
|
||||
|
||||
Reference in New Issue
Block a user