Skip to content

Add ACT isotherm #415

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Add ACT isotherm #415

wants to merge 5 commits into from

Conversation

WFlynnZ
Copy link
Collaborator

@WFlynnZ WFlynnZ commented May 20, 2025

This PR aims to add the ACT isotherm, a modified Langmuir isotherm with pH (or salt) dependence on the Qmax and Keq parameter.


  • Wait for the paper to show up online.
  • Add isotherm code.
  • Add isotherm document.
  • Add tests.

@jbreue16
Copy link
Contributor

You should rebase onto master before making further changes, since binding tests have been added to the CI

@WFlynnZ WFlynnZ force-pushed the feature/add_ACT_isotherm branch from 3ae52e4 to 9e85815 Compare May 23, 2025 04:34
@jbreue16
Copy link
Contributor

Ive changed the file encoding to utf-8, which is our standard encoding and which was also the reason why it didnt build for the mac CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

2 participants