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

(De)Protonate leads to deleted .CHRG file #373

Open
hoelzerC opened this issue Nov 12, 2024 · 2 comments
Open

(De)Protonate leads to deleted .CHRG file #373

hoelzerC opened this issue Nov 12, 2024 · 2 comments
Labels
wontfix This will not be worked on

Comments

@hoelzerC
Copy link

Describe the bug
The .CHRG files is deleted during (de)protonation.

To Reproduce
Create a folder with struc.xyz and .CHRG (arbitrary charge) and run crest struc.xyz --protonate or crest struc.xyz --deprotonate.

Expected behavior
Leave existing files intact, i.e. not delete the .CHRG file.

@hoelzerC hoelzerC added the bug Something isn't working label Nov 12, 2024
@hoelzerC
Copy link
Author

Add: When charge != 0 the file is not deleted, but altered. Wondering why crest is modifying these files at all.

@pprcht
Copy link
Contributor

pprcht commented Nov 13, 2024

We have used it in the past to track the charge of the protonated structure, which is the modified file you mention. It is user-responsibility to provide the correct input charge, and this should be done via the cli (-chrg), not hidden files.

That said, I won't modify the legacy code since it has done this thing for four years now and it's not a bug. Furthermore the new (calculator rather than xtb-based) implementation is already written. It is not yet the default, but can be selected with --newversion

@pprcht pprcht added wontfix This will not be worked on and removed bug Something isn't working labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants