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

Read error #44

Closed
JPDarby opened this issue Sep 21, 2023 · 7 comments
Closed

Read error #44

JPDarby opened this issue Sep 21, 2023 · 7 comments

Comments

@JPDarby
Copy link

JPDarby commented Sep 21, 2023

I get a segfault trying to read .xyz files with ExtXYZ version 0.1.13.

I think it's caused by extxyz_jll version 0.1.3+0 because if I downgrade that to 0.1.0+0 everything works.

@jameskermode
Copy link
Member

Oops. I know why this is and will fix ASAP. I just did the update to ExtXYZ_jll to add support for plain XYZ format. As an interim fix I'll change Project to use old jll version.

@jameskermode
Copy link
Member

Should be fixed by #45. Once CI completes I'll do a new release of ExtXYZ to stop this affecting others.

@JPDarby
Copy link
Author

JPDarby commented Sep 21, 2023

thanks

@jameskermode
Copy link
Member

Seems to work - would you be happy to use ExtXYZ.jl dev-ed for now, and I'll do a new release later once I've added the small extra bit of code needed to resolve #35?

@cortner
Copy link
Member

cortner commented Sep 21, 2023

dev doesn't work for me but downgrading extxyz_jll does.

@jameskermode
Copy link
Member

I hadn't merged the PR. Dev really should work now. I'll do a new release within the next day or two.

@jameskermode
Copy link
Member

New release v0.1.14 now made to General. Please re-open if issue persists.

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

No branches or pull requests

3 participants