Skip to content

Conversation

@nedtaylor
Copy link
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
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
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
Copy link
Collaborator

@shepplestone shepplestone left a comment

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
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

3 participants