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

Attempt to fix pytraj test. #1102

Merged
merged 1 commit into from
Sep 6, 2024
Merged

Attempt to fix pytraj test. #1102

merged 1 commit into from
Sep 6, 2024

Conversation

drroe
Copy link
Contributor

@drroe drroe commented Sep 6, 2024

See e.g. https://github.com/Amber-MD/cpptraj/actions/runs/10635870076/job/29486441993.

Try pinning numpy to 2.0.0 since 2.1.0 seems broken for pytraj.

@drroe drroe added the CI Framework Issues related to any CI tests (GitHub Actions, Jenkins, etc.) label Sep 6, 2024
@drroe drroe self-assigned this Sep 6, 2024
@drroe
Copy link
Contributor Author

drroe commented Sep 6, 2024

Yep, numpy 2.1.0 was the problem. Python versioning is a developer's nightmare.

@drroe
Copy link
Contributor Author

drroe commented Sep 6, 2024

Ugh the appveyor failure is their CI framework:

C:\projects\cpptraj>exit /b 0 
Collecting artifacts...
Found artifact 'cpptraj-50551057.zip' matching 'cpptraj-50551057.zip' path
Uploading artifacts...
[1/1] cpptraj-50551057.zip (20,973,067 bytes)...100%
Error uploading artifact to the storage: Remote server returned 500: Internal Server Error

It may be time to part ways with appveyor.

@drroe drroe merged commit db9e1ae into Amber-MD:master Sep 6, 2024
10 of 11 checks passed
@drroe drroe deleted the fix.pytraj.test branch September 6, 2024 13:42
@hainm
Copy link
Contributor

hainm commented Sep 6, 2024

Python versioning is a developer's nightmare.

I agree completely.
I've filed Amber-MD/pytraj#1667 to resolve that.
I write more Golang recently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Framework Issues related to any CI tests (GitHub Actions, Jenkins, etc.)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants