Conversation
|
Previously, using this signal-slot configuration: the basic-root-writer contained this histogram for time slice Now, using the same slot configuration, the basic-root-writer contains these histograms for time slice |
| return hist; | ||
| } | ||
|
|
||
| TH1D* KTTimeSeriesFFTW::CreateHistogramImag(const std::string& name) const |
There was a problem hiding this comment.
Should the previous function be changed to CreateHistogramReal to maintain consistency?
There was a problem hiding this comment.
It would provide consistency, but would require more changes upstream in the KTDAC class. So the benefit to the present approach is that it is a minimal addition that applies to one configuration. Another possibility would be to add a new function called CreateHistogramReal(), but without deleting the CreateHistogram() function that is in use by other classes.
3 participants
This is a change to fftw time series output histograms. Previously, only the real part of the time series was accessible in the basic-root-writer output after configuring: