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

reference to the presentation of xyz2mol #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nbehrnd
Copy link

@nbehrnd nbehrnd commented Jul 21, 2022

The additional link to the slides and the recording of Jan Jensen's presentation of xyz2mol conveys details how the program processes the data and showcases known limitations of the implementation.

@andersx andersx requested a review from charnley July 22, 2022 05:57
@nbehrnd
Copy link
Author

nbehrnd commented Oct 17, 2022

Today, I still appreciate Jan Jensens's succinct presentation of xyz2mol -- covering aspects why he was interested and examples where the implementation does encounter difficulties and should be used with care. Is there a major improvement of the program in progress/a publication in preparation which aims to exceed the functionality presented which could explain why the suggested addition of a link to the video and slides is pending revision?

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.

1 participant