Aneta Siemiginowska
Aneta Siemiginowska
@hamogu it looks like I build the main branch. Now, I build this PR's branch but I still have issues running the pytest. It looks like `quick.rst` is not in...
@hamogu Thanks. I updated and build the branch and this test passes. Yes, this PR is not to fix all the tests. I do think that it improves the overall...
Yes. I realized that this is the change to the chi2 statistics that have been implemented in XSPEC at some point and there was no corresponding update in Sherpa. The...
Yes, I agree with @DougBurke and to make `chi2datavar` behave consistently with xspec, so `chi2xspecvar` becomes a synonym of `chi2datavar`. @dtnguyen2 is the update to `chi2datavar` so easy?
This is tricky. Sherpa has 'set_analysis' which defines the analysis space and the y-axis for plotting. For ungroup data it is straightforward: `set_analysis(1,'energy','counts',0)` `set_analysis(1,'channel','counts',0)` would result in plotting `counts` on...
@olaurino In the previous chat I said: _I was voting for total number of counts in a new group instead of the division per channel, because if you group to...
There was a comment from @DougBurke at #39 about ignoring the setting at `set_analysis()`. Has this been resolved?
I was just looking at XSPEC code and it looks like they ignore any channels with 0 values in calculating the standard `chi2`, so after checking the channels for any...
@dtnguyen2 It looks like this needs to be rebased.
@DougBurke yes, I'll try with the 4.15 and post it here.