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

Workaround for SerialMaster #1258

Closed
wants to merge 2 commits into from
Closed

Workaround for SerialMaster #1258

wants to merge 2 commits into from

Conversation

pmrv
Copy link
Contributor

@pmrv pmrv commented Dec 19, 2023

pyiron/pyiron_atomistics#1265 flagged a failing test for SerialMasterBase, that tries to re-run a master job and fails because the server object is locked. This is a hack to make it work again.

@pmrv pmrv added the bug Something isn't working label Dec 19, 2023
@pmrv pmrv added performance Things are slow integration Start the integration tests with pyiron_atomistics/contrib for this PR and removed performance Things are slow labels Dec 19, 2023
@pmrv
Copy link
Contributor Author

pmrv commented Dec 20, 2023

Closed in favor of removing SerialMaster which is the only thing that failed after bumping base to 0.6.14.

@pmrv pmrv closed this Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working integration Start the integration tests with pyiron_atomistics/contrib for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant