Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change default of acidity to None #69

Merged
merged 6 commits into from
Jun 14, 2024

Conversation

pm-blanco
Copy link
Collaborator

Changes the default value of acidity in define_particle and set_particle_acidity to None:

  • The behavior of setting acidity = "inert" is now substituted by acidity = None.
  • A DeprecationWarning is printed to screen if the user sets acidity = "inert", warning the user that this keyword is obsolete and may be deprecated in the future. pyMBE proceeds by replacing acidity = "inert" to acidity = None for backwards compatibility.

@pm-blanco pm-blanco requested a review from paobtorres June 10, 2024 14:53
@pm-blanco pm-blanco self-assigned this Jun 10, 2024
@pm-blanco pm-blanco added this to the first stable version milestone Jun 10, 2024
@pm-blanco pm-blanco requested a review from davidbbeyer June 14, 2024 09:01
Copy link
Contributor

@paobtorres paobtorres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did a few testing and everything seems to work perfect.

@pm-blanco pm-blanco merged commit 0472342 into pyMBE-dev:main Jun 14, 2024
3 checks passed
@pm-blanco pm-blanco deleted the change_default_acidity branch August 29, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants