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

Github Action Fixes #1202

Merged
merged 8 commits into from Sep 30, 2023
Merged

Conversation

jgray-19
Copy link
Contributor

@jgray-19 jgray-19 commented Sep 28, 2023

Fixes

  • Remove all uses of outdated ubuntu-18.04, which reached end of life on May 31, 2023
  • Update the following actions due to Node 12 to Node 16 change
    • Setup python v2 -> v4
    • download-artifact v2 -> v3
  • Update old_thick_bend to true
  • Revert leir.ptc.nonlin.cfg changes forcing test to not run (overall, just reduce the tolerances)
  • Add more beta min max tolerances

After all of these fixes, all of the actions will pass, without warnings. Also, the actions will take somewhere around 20 minutes, instead of a day, since the actions do not hang waiting for an Ubuntu-18.04 machine.

@rdemaria
Copy link
Contributor

Ok for me, thanks! It seems complete, can I merge?

@jgray-19
Copy link
Contributor Author

Yes, it's complete, all checks pass, so as long as you're happy with the changes, should be all good.

@rdemaria rdemaria merged commit 2f1e95e into MethodicalAcceleratorDesign:master Sep 30, 2023
39 checks passed
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.

None yet

2 participants