[SyneRBI/SIRF] Fix STIR.AcquisitionModel base (missing `__del__`). (PR #1387)
Evgueni Ovtchinnikov
Changes in this pull request
Testing performed
Related issues
Checklist before requesting a review
- I have performed a self-review of my code
- I have added docstrings/doxygen in line with the guidance in the developer guide
- I have implemented unit tests that cover any new or modified functionality
- The code builds and runs on my machine
- CHANGES.md has been updated with any functionality change
Contribution Notes
Please read and adhere to the contribution guidelines.
Please tick the following:
- The content of this Pull Request (the Contribution) is intentionally submitted for inclusion in SIRF (the Work) under the terms and conditions of the Apache-2.0 License.
You can view, comment on, or merge this pull request online at:
https://github.com/SyneRBI/SIRF/pull/1387
Commit Summary
- d95c7bb updated files for release 3.9.0 [ci skip]
- 54505e7 Merge branch 'master' of https://github.com/SyneRBI/SIRF
- dedf83c Merged branch 'master' of https://github.com/SyneRBI/SIRF
- 17867e4 Merge branch 'master' of https://github.com/SyneRBI/SIRF
- 5a1e82f Merge branch 'master' of https://github.com/SyneRBI/SIRF
- a00488d Merge branch 'master' of https://github.com/SyneRBI/SIRF
- 6d1d3e5 Merge branch 'master' of https://github.com/SyneRBI/SIRF
- 7179124 added missing __del__ in STIR.AcquisitionModel
File Changes
(1 file)
- M src/xSTIR/pSTIR/STIR.py (5)
Patch Links:
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Kris Thielemans
@KrisThielemans approved this pull request.
Hi
can you add a line to CHANGES.md. Please squash-merge then (editing the suggested message). In the future, you probably need to start from a fresh checkout to avoid the multiple merge commits.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
Evgueni Ovtchinnikov
@evgueni-ovtchinnikov pushed 1 commit.
- 74c1376 [ci skip] updated CHANGES.md
—
View it on GitHub or unsubscribe.
You are receiving this because you are subscribed to this thread.
Evgueni Ovtchinnikov
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.