Skip to content

Conversation

@tsbinns
Copy link
Contributor

@tsbinns tsbinns commented Feb 14, 2023

Implementation of comments/suggestions in merged PR #11458.

  • Fixed incorrect default values in the docstring.
  • Reworded the docstring to better reflect the behaviour of the class.
  • Changed the naming of internal variables for consistency and clarity.
  • Changed a logging message to more accurately reflect the behaviour of the code.

Cheers, Thomas!

@tsbinns tsbinns closed this Feb 14, 2023
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.

2 participants