Change request #1511
Option to truncate GObservations dump into logger
| Status: | Closed | Start date: | 07/09/2015 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assigned To: |  Mayer Michael | % Done: | 100% | |
| Category: | - | |||
| Target version: | 1.0.0 | |||
| Duration: |
Description
I am using a large GObservations container, which includes about 200 (or more) GCTAObservations. Currently, if we are interested in the log file, one has to browse incredibly long to find the part of interest, since the dump of GObservations into the log file takes a huge space. Maybe it could be interesting to some users to truncate this by just printing the number of observation in the container (even though a large verbosity is desired).
Recurrence
No recurrence.
History
#1
Updated by Knödlseder Jürgen over 9 years ago
I had indeed also the feeling that the default log was too verbose. There are verbosity levels implemented, hence we may switch to a less verbose printing. This can in fact be done in the GObservations::print() method, so that it will be homogenous across all tools.
#2
Updated by Mayer Michael over 9 years ago
- Status changed from New to Pull request
- Assigned To set to Mayer Michael
- % Done changed from 0 to 100
I agree, I reduced the chattiness of GObservations::print() and added the number of model in the container to the output. Change is on branch 1511-reduce-GObservations-verbosity
#3
Updated by Mayer Michael over 9 years ago
should mention that the branch is in gammalib.
#4
Updated by Knödlseder Jürgen over 9 years ago
- Project changed from ctools to GammaLib
#5
Updated by Knödlseder Jürgen over 9 years ago
This pull request has also a modification in GApplication in. Is this related to this issue?
const GApplicationPars& pars(void)const{return m_pars;};
#6
Updated by Mayer Michael over 9 years ago
Apparently, I mixed up some commits. Apologize for this, it is completely unrelated, you can ignore it. However, it would be nice to have such a function in GApplication after all 
#7
Updated by Knödlseder Jürgen over 9 years ago
- Status changed from Pull request to Closed
- Target version set to 1.0.0
Merged into devel (without the change of GApplication).
What would be the use case for having a pars() method?