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

[Feature Request]: Temperature effects in Pourbaix analysis? #3526

Open
CifLord opened this issue Dec 19, 2023 · 2 comments
Open

[Feature Request]: Temperature effects in Pourbaix analysis? #3526

CifLord opened this issue Dec 19, 2023 · 2 comments

Comments

@CifLord
Copy link
Contributor

CifLord commented Dec 19, 2023

Problem

At the moment, all analysis in pourbaix_diagram.py assumes that we are operating under room temperature. As a result, we assume all free energies will have a prefactor of 0.0591 for the pH variable i.e. pH * k_B * T * ln(10) (in eV) where T= 298K (RT) and k_B * T * ln(10) = 0.0591.

Is there a (chemical or implementation) reason why we do not make this more generalized by adding a temperature variable in the Pourbaix diagrams? e.g. maybe Pourbaix diagrams are for room temperature analysis only and doesn't make sense beyond that temperature?

Maybe we can set it to a default of T = 298K so the behavior doesn't change, but still allow users to modify if they want to analyze aqueous electrochemical systems under different temperature conditions? If this is reasonable, I can submit a small pull request later to implement

Proposed Solution

Add a temperature variable that changes the pH prefactor to any method or function that uses "PREFAC". Default all temperature variables to 298K so that the PREFAC value stays at 0.0591 unless the user manually changes temperature.

Alternatives

No response

@montoyjh
Copy link
Contributor

montoyjh commented Dec 19, 2023 via email

@CifLord
Copy link
Contributor Author

CifLord commented Dec 19, 2023

This is a great idea that I'd love to see implemented.

On Mon, Dec 18, 2023, 6:34 PM Richard Tran @.> wrote: Problem At the moment, all analysis in pourbaix_diagram.py assumes that we are operating under room temperature. As a result, we assume all free energies will have a prefactor of 0.0591 for the pH variable i.e. pH * k_B * T * ln(10) (in eV) where T= 298K (RT) and k_B * T * ln(10) = 0.0591. Is there a (chemical or implementation) reason why we do not make this more generalized by adding a temperature variable in the Pourbaix diagrams? Maybe we can set it to a default of T = 298K so the behavior doesn't change, but still allow users to modify if they want to analyze aqueous electrochemical systems under different temperature conditions? If this is reasonable, I can submit a small pull request later to implement Proposed Solution Add a temperature variable that changes the pH prefactor to any method or function that uses "PREFAC". Default all temperature variables to 298K so that the PREFAC value stays at 0.0591 unless the user manually changes temperature. Alternatives No response — Reply to this email directly, view it on GitHub <#3526>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRRSNHQAZAZZTRFR6OKK4DYKD4KRAVCNFSM6AAAAABA2OSIPWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGA2DOOBQGYYDKOI . You are receiving this because you are subscribed to this thread.Message ID: @.>

Sounds good, I was curious what your thoughts on this was going to be. I'll try to make some time this week to push it through. It shouldn't be too much trouble I think

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

No branches or pull requests

2 participants