Skip to content

Fix formation energy - #12

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

Fix formation energy#12
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.

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

Copy link
Copy Markdown
CollaboratorAuthor

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
CollaboratorAuthor

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.

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

@shepplestoneshepplestone 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 mainOct 7, 2024
@nedtaylor
nedtaylor deleted the fix_formation_energy branch October 7, 2024 12:17
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to use energy above hull

2 participants

@nedtaylor@shepplestone