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

upd diagram #30

Merged
merged 3 commits into from
Nov 22, 2024
Merged

upd diagram #30

merged 3 commits into from
Nov 22, 2024

Conversation

KulaginVladimir
Copy link
Owner

This PR updates the potential energy diagram, but we need to hear @ehodille opinion.

@ehodille
Copy link

ehodille commented Nov 8, 2024

How can I visualise the potential energy diagrams ?

@KulaginVladimir
Copy link
Owner Author

@ehodille, here you are. Should we have 2 diagrams based on your sketch?

image

potential_diagram_new.pdf

@ehodille
Copy link

ehodille commented Nov 8, 2024

Ha yes, that is perfect.
I think only this diagram is good.
The other diagram is not necessary, especially if you mention in the paper the relationship between the kinetic energy barrier and the heat of solution

@KulaginVladimir
Copy link
Owner Author

@ehodille, alright

@RemDelaporteMathurin
Copy link
Collaborator

How can I visualise the potential energy diagrams ?

Simply go to the "Files changed" tab:

image

@KulaginVladimir
Copy link
Owner Author

KulaginVladimir commented Nov 14, 2024

@ehodille
In the paper, we also state that sputtering and desorption of atoms are possible. What do you think, should we add $2E_H$ energy level to this diagram? In order to somehow note a possibility of atomic release?

@KulaginVladimir
Copy link
Owner Author

@ehodille, what do you think of this diagram?

image

@KulaginVladimir KulaginVladimir merged commit 16778e0 into main Nov 22, 2024
1 check passed
@KulaginVladimir KulaginVladimir deleted the pot_diagram_upd branch December 3, 2024 21:50
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

Successfully merging this pull request may close these issues.

3 participants