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

Considering ion charge when deducing reactions #41

Open
samblau opened this issue Dec 11, 2020 · 0 comments
Open

Considering ion charge when deducing reactions #41

samblau opened this issue Dec 11, 2020 · 0 comments
Assignees

Comments

@samblau
Copy link
Member

samblau commented Dec 11, 2020

We currently ignore the charge of a metal ion when deducing reactions. This can be a problem for lithium systems when Li0 is present in the network, which allows you to take a Li+ starting molecule and reduce it to Li0 and then combine that Li0 with an EC0 to obtain Li+EC-, since only the total molecular charge is checked. But by just removing the Li0 from the network, this can mostly be avoided. However, for magnesium systems the problem becomes more prevalent because Mg2+ and Mg1+ are both legitimate species. In principle, we could track the metal ion charge via partial charges or Critic charges and then disallow reactions in which the metal charge is changing in order to avoid reactions like e.g. Li0 + EC0 -> Li+EC-. However, for the Mg system, maybe we don't want to disallow those types of reactions because internal charge transfer can and does happen, e.g. Mg+TFSI- -> CF3 + Mg2+TFSI-(without CF3). I don't know the solution, but I wanted to open this issue so that we don't forget the underlying problem.

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

3 participants