Skip to content

Commit 0a8c901

Browse files
authored
Merge pull request #623 from xpsi-group/disk_obscuration
Disk obscuration
2 parents 4ac36f9 + 0e50a90 commit 0a8c901

28 files changed

Lines changed: 1869 additions & 797 deletions

CHANGELOG.rst

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,59 @@
1+
[v3.2] - 2025-12-01
2+
~~~~~~~~~~~~~~~~~~~
3+
4+
Summary
5+
^^^^^^^
6+
7+
- The main addition is an optional ray-blocking feature due to an accretion disk, following Ibragimov & Poutanen (2009)(`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__). Additional minor updates are listed below.
8+
9+
10+
Fixed
11+
^^^^^
12+
13+
- The Poisson likelihood with a fixed background was modified so that it can have the correct normalization if the user provides the precomputed ln-data-factorial term (as already done for the default background marginalized likelihoods). In addition, the normalizations of the Gaussian ln-likelihood functions (in the Gaussian extensions) were fixed to have the correct sign. (`#614 <https://github.com/xpsi-group/xpsi/pull/614>`__)
14+
- Fix issue (`#620 <https://github.com/xpsi-group/xpsi/issues/620>`__) where the interstellar attenuation could be greater than one, although it should not. (`#622 <https://github.com/xpsi-group/xpsi/pull/622>`__)
15+
- Fix issues (`#534 <https://github.com/xpsi-group/xpsi/issues/534>`__) and (`#574 <https://github.com/xpsi-group/xpsi/issues/574>`__). (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
16+
17+
18+
Added
19+
^^^^^
20+
21+
- Added details of 3 new published papers and 1 new submitted paper. (`#612 <https://github.com/xpsi-group/xpsi/pull/612>`__)
22+
- Added the possibility to add custom legend labels in the corner plot. This has become necessary as the multi-modal cornerplot gives default name to new modes. This can be done with a new parameter in the ``CornerPlotter.plot()`` function such as ``legend_labels=['list','of','my','labels']``. (`#619 <https://github.com/xpsi-group/xpsi/pull/619>`__)
23+
- Now, if the priors are identical (i.e. ``priors_identical=True`` in ``CornerPlotter.plot()``), plot them in black. (`#619 <https://github.com/xpsi-group/xpsi/pull/619>`__)
24+
- New ``TestRun_Disk_Occultation.py`` and ``TestRun_Disk_Occultation_IQU.py`` have been added which demonstrate the usage of the disk blocking. (`#622 <https://github.com/xpsi-group/xpsi/pull/622>`__)
25+
26+
27+
Changed
28+
^^^^^^^
29+
30+
- In the example modules,``CustomHotregion_Accreting.py`` has been modified to accomodate the linear model from Molkov et al. 2024 as a "user" option, alongside the Bobrikova atmosphere as the "Num5D" option. (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
31+
- In the example modules, ``CustomPhotosphere.py`` can now accomodate disk obscuration. (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
32+
- The azimuthal integrators (regular, IQU, and split) now feature the ``R_in`` keyword argument for disk blocking of rays due to being obscured by an accretion disk. The math used is taken from Ibragimov & Poutanen (2009), appendix B. `R_in` has a default value of `1e6` (corresponding to 1000 km), at which point the disk is so large that practically speaking no blocking would occur. In that case however, the blocking calculation is skipped, ensuring no extra computation time is introduced here, e.g. for RMPs. (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
33+
- ``HotRegion.py`` and ``HotRegions.py`` now feature the ``R_in`` keyword arguments, which are passed to the integrators. Default values are ``1e6``, in which case blocking calculations are skipped. (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
34+
- The ``integrator_for_azimuthal_invariance.pyx`` now features new comments that detail how the disk obscuration works as well as other comments that explain in more detail how the intensity calculation works. (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
35+
- Updated instrument trimming: replaced the threshold-based method with a tolerance-based one. Each channel now uses a cumulative response computed over the full energy range. Trimming occurs at the first energy where the cumulative responses of all channels exceed the (1 − tolerance) quantile. (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
36+
- Changed the structure of ``InstrumentPileup`` in order to simplify it. (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
37+
38+
39+
Removed
40+
^^^^^^^
41+
42+
- An unused and incorrect Poisson IQU likelihood extension was removed. (`#614 <https://github.com/xpsi-group/xpsi/pull/614>`__)
43+
- The "threshold" argument for Instrument trimming has been removed and replaced by "tolerance" as the method used for trimming has been changed. (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
44+
45+
Attribution
46+
^^^^^^^^^^^
47+
48+
- Yinghan Mao (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
49+
- Bas Dorsman (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__)
50+
- Tuomo Salmi (`#623 <https://github.com/xpsi-group/xpsi/pull/623>`__, `#614 <https://github.com/xpsi-group/xpsi/pull/614>`__, `#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
51+
- Lucien Mauviard (`#622 <https://github.com/xpsi-group/xpsi/pull/622>`__, `#619 <https://github.com/xpsi-group/xpsi/pull/619>`__, `#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
52+
- Anna Watts (`#612 <https://github.com/xpsi-group/xpsi/pull/612>`__)
53+
- Denis Gonzalez-Caniulef (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
54+
- Pierre Stammler (`#588 <https://github.com/xpsi-group/xpsi/pull/588>`__)
55+
56+
157
[v3.1] - 2025-05-23
258
~~~~~~~~~~~~~~~~~~~
359

changelog.d/588.changed.rst

Lines changed: 0 additions & 3 deletions
This file was deleted.

changelog.d/588.fixed.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/588.removed.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/589.atttribution.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/612.added.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/612.attribution.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/614.attribution.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/614.fixed.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

changelog.d/614.removed.rst

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)