Use elliptic curve as a default implicit curve
ClosedPublic

Authored by yurchor on May 13 2019, 4:43 PM.

Details

Summary

The current default implicit plot 'y∙sinx + x∙cosy = 1' scales badly on the big area. This can be mitigated using some simpler but famous plot.

BUG: 201515

Test Plan
  1. Start a new instance of KmPlot
  2. Ctrl+Scroll down until some big numbers appear (500 or something like this)
  3. Choose "Create -> Implicit plot".
  4. No hanging, KmPlot is still accessible.

Diff Detail

Repository
R334 KmPlot
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
yurchor created this revision.May 13 2019, 4:43 PM
Restricted Application added a project: KDE Edu. · View Herald TranscriptMay 13 2019, 4:43 PM
Restricted Application added a subscriber: kde-edu. · View Herald Transcript
yurchor requested review of this revision.May 13 2019, 4:43 PM
aacid added a subscriber: aacid.May 14 2019, 9:02 PM

it's kind of ugly though, is there any chance to make the other one faster?

it's kind of ugly though, is there any chance to make the other one faster?

May it be an elliptic curve?

y²=x³−x+1

It is obvious that any equation with periodical functions should make KmPlot stuck at some scaling.

On the other hand, all functions with asymptotic behavior are "ugly" at a large scale.

There are some compendiums of "famous" functions (e. g. [1]), but I failed to find something really interesting on an arbitrary scale. That is the main reason for this review request.

[1] https://elepa.files.wordpress.com/2013/11/fifty-famous-curves.pdf

aacid added subscribers: cfeck, apol.May 16 2019, 8:58 PM

y²=x³−x+1 looks a bit nicer i think

Anyone else? @apol you do math apps when very bored and @cfeck i think you're a ¿maths teacher?

For my school students, I would use a simple x²+y²=25. We don't teach elliptic curves before university.

yurchor updated this revision to Diff 58483.May 22 2019, 4:27 PM

Use elliptic curve

yurchor retitled this revision from Use kappa curve as a default implicit curve to Use elliptic curve as a default implicit curve.May 22 2019, 4:28 PM
aacid accepted this revision.Jun 9 2019, 10:50 PM
This revision is now accepted and ready to land.Jun 9 2019, 10:50 PM
This revision was automatically updated to reflect the committed changes.