Feature #1247

Add operators +|-|*|/ to GSkymap

Added by Mayer Michael over 10 years ago. Updated over 10 years ago.

Status:ClosedStart date:07/08/2014
Priority:NormalDue date:
Assigned To:Mayer Michael% Done:

100%

Category:-
Target version:3rd coding sprint
Duration:

Description

To compute residual maps, the simple operations to subtract two sky maps would be easy to implement. Doing so, we could add further mathematical operators like “plus”, “division”, and “multiplication”.
We could also add functions which scale the entries of the map by a factor or take the square root of it.

Recurrence

No recurrence.

History

#1 Updated by Mayer Michael over 10 years ago

  • Assigned To set to Mayer Michael

I’ve added functionality supporting *= and /= operators, as well as the friend sqrt function.
The branch is in the central repo, called 1247-add-operators-and-friends-to-GSkymap

#2 Updated by Knödlseder Jürgen over 10 years ago

I was about to merge in your change, but I recognized that there is some change in GModelSpatialRadialGauss. Any clues where this change comes from?

 double GModelSpatialRadialGauss::theta_max(void) const
 {
     // Return value
-    return (sigma() * gammalib::deg2rad * 5.0);
+    return (sigma() * gammalib::deg2rad * 1.7941227);
 }

#3 Updated by Mayer Michael over 10 years ago

oups, I might have played around with this value to check the computation time in the past. I don’t know why this has been checked into repository at all. So you can just drop that. Sorry for the confusion

#4 Updated by Knödlseder Jürgen over 10 years ago

  • Status changed from New to Resolved
  • Target version set to 3rd coding sprint

No worries. Merged in the change.

#5 Updated by Knödlseder Jürgen over 10 years ago

  • % Done changed from 0 to 100

#6 Updated by Knödlseder Jürgen over 10 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF