[Bug 1836768] [NEW] Locus need some optimization

classic Classic list List threaded Threaded
9 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] [NEW] Locus need some optimization

Hliaire Fernandes
Administrator
Public bug reported:

It looks like DrGLocusMorph samples a lot of off screen samples. Later
discarded, though. But it consumes a lot of computer power.

Test with this script, with a value larger than 6

| figure f a b c|
figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
figure courbeDe: f de: -10 a: 10

** Affects: drgeo
     Importance: Medium
         Status: New


** Tags: core

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  New

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
Once can use

sample1 pointScreen - sample2 pointScreen) abs max > 5

in place of the squaredDistance

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  New

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
5 to be replace by an alternal maxScreenDistance

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  New

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
Relaxing a bit the Locus parameters helps a bit:

LocusMorph>>initialize
   super initialize.
   minAbscissaDelta := 1e-4.
   maxScreenDelta := 64.
   maxAbscissaDelta := 1e-1

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  In Progress

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
Analysis show replacing message #squaredDistance has minimal impact.
So no need to replace it.

** Changed in: drgeo
       Status: New => In Progress

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  In Progress

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
One idea is to make the minAbscissaDeltra related to the zoom factor on the sketch.
When zooming in, more accuracy is needed for the mobile abscissa

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  In Progress

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
** Changed in: drgeo
    Milestone: 19.06 => 19.09

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  In Progress

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
** Changed in: drgeo
       Status: In Progress => Fix Committed

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  Fix Committed

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions

Reply | Threaded
Open this post in threaded view
|

[Bug 1836768] Re: Locus need some optimization

Hliaire Fernandes
Administrator
In reply to this post by Hliaire Fernandes
** Changed in: drgeo
       Status: Fix Committed => Fix Released

--
You received this bug notification because you are a member of DrGeo
developers, which is subscribed to Dr. Geo II.
Matching subscriptions: Dr. Geo bug report
https://bugs.launchpad.net/bugs/1836768

Title:
  Locus need some optimization

Status in Dr. Geo II:
  Fix Released

Bug description:
  It looks like DrGLocusMorph samples a lot of off screen samples. Later
  discarded, though. But it consumes a lot of computer power.

  Test with this script, with a value larger than 6

  | figure f a b c|
  figure := DrGeoFigure nouveau afficherAxes afficherGrille echelle: 50.
  a := figure decimal: 1 a: 5 @ -1 min: -8 max: 8 nom: 'a' afficherValeur: true.
  b := figure decimal: 1 a: 5 @ -2 min: -8 max: 8 nom: 'b' afficherValeur: true.
  c := figure decimal: 1 a: 5 @ -3 min: -8 max: 8 nom: 'c' afficherValeur: true.
  f := [:x | a valeur * x squared + (b valeur * x) + c valeur].
  figure courbeDe: f de: -10 a: 10

To manage notifications about this bug go to:
https://bugs.launchpad.net/drgeo/+bug/1836768/+subscriptions