Skip to content

Fix formation energy - #12

Merged
nedtaylor merged 11 commits into
mainfrom
fix_formation_energy
Oct 7, 2024
Merged

nedtaylor merged 11 commits into
mainfrom
fix_formation_energy

Conversation

@nedtaylor

Copy link
Copy Markdown
Collaborator

Use of formation energy as the weighting to sum distribution functions (DFs) to generate the generalised distribution function (gDF) is flawed due to the incorrect comparison of structures with differing stoichiometries.

This is extremely prevalent when considering choice of reference energies. If using isolated atom energies for a binary compound material, then one element species will have a much more favourable formation energy for its bulk (i.e. carbon) than the other (i.e. oxygen). This means that the second element in its bulk phase will contribute nothing to the gDF, which is not what should happen.

@nedtaylor nedtaylor added the enhancement New feature or request label Sep 25, 2024
@nedtaylor nedtaylor added this to the Version 1 milestone Sep 25, 2024
@nedtaylor nedtaylor self-assigned this Sep 25, 2024
@nedtaylor

Copy link
Copy Markdown
Collaborator Author

Another addition made to this branch is that the formation energy of the host structure is subtracted from the generated structures. This may be counterintuitive, but it is to show the favourability of placing an atom in the specific host structure, without any bias of unfavourability of the initial host (i.e. energy cost of termination, etc).

@nedtaylor

Copy link
Copy Markdown
Collaborator Author

Another addition made to this branch is that the formation energy of the host structure is subtracted from the generated structures. This may be counterintuitive, but it is to show the favourability of placing an atom in the specific host structure, without any bias of unfavourability of the initial host (i.e. energy cost of termination, etc).

Actually, it negates the energy of formation of the host structure. This is the important value to subtract as that contains the formation energy of the terminations.

@nedtaylor nedtaylor linked an issue Sep 27, 2024 that may be closed by this pull request

@shepplestone shepplestone left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

In principle approved, deep code review due 12/10/2024

@nedtaylor
nedtaylor merged commit efbcf12 into main Oct 7, 2024
@nedtaylor
nedtaylor deleted the fix_formation_energy branch October 7, 2024 12:17

This branch had an error being deployed

1 failed deployment
github-pages — ed7b1a9e Deployed Sep 27, 2024 by nedtaylor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to use energy above hull

2 participants