Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upper and lower limit slots not clear leading to undesirable results #81

Open
johnblommers opened this issue Aug 16, 2021 · 2 comments · May be fixed by #106
Open

Upper and lower limit slots not clear leading to undesirable results #81

johnblommers opened this issue Aug 16, 2021 · 2 comments · May be fixed by #106
Labels
Needs Design Needs further designing Priority: High Needs to be fixed immediately Status: Confirmed Will surely be fixed or implemented

Comments

@johnblommers
Copy link

Expected: To demonstrate the issue, integrate the simple function x from 0 to 1. The correct answer is 0.5 exactly.

Result: But with accuracy of integration set to maximum the result is -0.501146157. There are three things wrong:

  1. It takes perhaps ten seconds to compute this result, much longer than you'd expect from such a simple integration.
  2. The exact answer is 0.50000000 but pebbles returns -0.501146157.
  3. The sign of the answer is wrong, it should be positive.

Screenshot:

Screenshot from 2021-08-15 19-07-51

Version

flatpak info com.github.subhadeepjasu.pebbles

Pebbles - Perform advanced calculations with ease

          ID: com.github.subhadeepjasu.pebbles
         Ref: app/com.github.subhadeepjasu.pebbles/x86_64/stable
        Arch: x86_64
      Branch: stable
     Version: 2.0.0
     License: GPL-3.0+
      Origin: flathub
  Collection: org.flathub.Stable
Installation: user
   Installed: 104.3 MB
     Runtime: org.freedesktop.Platform/x86_64/20.08
         Sdk: org.freedesktop.Sdk/x86_64/20.08

      Commit: 43f192a03fa1ac2eed46888d6c60144d6b387b0d851444bf812e633e0cc2ec4f
      Parent: b3de54e093f068bc4146b69d488128b407a9370df3fd06d875f6786aa538699b
     Subject: Temporarily remove elementary stylesheet (734a0d18)
        Date: 2021-08-15 14:52:32 +0000
@SubhadeepJasu
Copy link
Owner

SubhadeepJasu commented Aug 16, 2021

The answer is partly correct, because you have entered the upper and lower limits in lower and upper slots respectively. Which is definitely still an UX fail and I need to fix it.

As for speed, it doesn't depend on the way we perceive an expression to be simple or complex. Increasing accuracy simply increases the number of subdivisions between the limits and increases the number of calculation iterations even for something as simple as "x". That is where the accuracy slider comes in. If we as humans know that an expression is simple, use low accuracy, otherwise use higher accuracy. Also the result will never be 100% accurate even with the highest accuracy settings. That's just by the nature of the algorithm. I do have plans to look into alternative algorithms to improve accuracy and speed.

@SubhadeepJasu SubhadeepJasu added Needs Design Needs further designing Priority: High Needs to be fixed immediately Status: Confirmed Will surely be fixed or implemented labels Aug 16, 2021
@SubhadeepJasu SubhadeepJasu changed the title Inaccurate Integration Results Upper and lower limit slots not clear leading to undesirable results Aug 16, 2021
@SubhadeepJasu
Copy link
Owner

Point 1 and 2 are mitigated by #82

@SubhadeepJasu SubhadeepJasu linked a pull request Mar 5, 2025 that will close this issue
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Design Needs further designing Priority: High Needs to be fixed immediately Status: Confirmed Will surely be fixed or implemented
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants