Skip to content

Removes deprecated absorption param#430

Closed
fakufaku wants to merge 11 commits into
masterfrom
fix/remove_absorption_param
Closed

Removes deprecated absorption param#430
fakufaku wants to merge 11 commits into
masterfrom
fix/remove_absorption_param

Conversation

@fakufaku

@fakufaku fakufaku commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator

This PR removes the deprecated absorption parameter from various methods in Room and ShoeBox.

  • Are there docstrings ? Do they follow the numpydoc style ?
  • Have you run the tests by doing nosetests or py.test at the root of the repo ?
  • Have you checked that the doc builds properly and that any new file has been added to the repo ? How to do that is covered in the documentation.
  • Is there a unit test for the proposed code modification ? If the PR addresses an issue, the test should make sure the issue is fixed.
  • Last but not least, did you document the proposed change in the CHANGELOG file ? It should go under "Unreleased".

@fakufaku fakufaku closed this Apr 6, 2026
@fakufaku fakufaku deleted the fix/remove_absorption_param branch April 6, 2026 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant