Bug #2495

A q-mode parameter is set to h after quering

Added by Knödlseder Jürgen almost 6 years ago. Updated almost 6 years ago.

Status:ClosedStart date:06/05/2018
Priority:HighDue date:
Assigned To:Knödlseder Jürgen% Done:

100%

Category:-
Target version:1.6.0
Duration:

Description

Andreas reported in #2305 that after using a q-type parameter in a parameter file, it was set to hidden. This is probably a side effect of the current logic where after a query a parameter is set to hidden so that it is not queried again. Querying should be better handled by an explicit flag, and note by changing the type to hidden.


Recurrence

No recurrence.

History

#1 Updated by Knödlseder Jürgen almost 6 years ago

It turned out that when changing a parameter from a to q, the parameter is no longer queried. The parfile

ra,  r, q, 83.63,0,360, "RA of pointing (degrees)" 
dec, r, q, 22.51,-90,90, "Dec of pointing (degrees)" 
results in
$ ctobssim
Radius of FOV (degrees) (0-180) [5.0] 
Start time (UTC string, JD, MJD or MET in seconds) [2020-01-01T00:00:00] 
Stop time (UTC string, JD, MJD or MET in seconds) [2020-01-01T00:30:00] 
Lower energy limit (TeV) [0.1] 
Upper energy limit (TeV) [100.0] 
Calibration database [prod2] 
Instrument response function [South_0.5h] 
Input model definition XML file [$CTOOLS/share/models/crab.xml] 
Output event data file or observation definition XML file [events.fits] 

#2 Updated by Knödlseder Jürgen almost 6 years ago

  • Subject changed from A q-type parameter is set to h after quering to A q-mode parameter is set to h after quering

#3 Updated by Knödlseder Jürgen almost 6 years ago

  • Status changed from New to Pull request
  • % Done changed from 0 to 100

Bugs were corrected, will now integrate the code.

A new method GApplicationPar::was_queried() was added that informs whether a given parameter was actually queried. A new flag was added as protected member that keeps track of querying.

#4 Updated by Knödlseder Jürgen almost 6 years ago

  • Status changed from Pull request to Closed

The code was merged into devel.

Also available in: Atom PDF