-
Notifications
You must be signed in to change notification settings - Fork 12
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
update setonix configuration #362
Conversation
None of the modified files are used by the CI, so it should not be necessary to run the GPU tests for this PR. |
(Somehow I created something unmergable on the old branch, so this is a new PR.) |
@psharda Can you see if this works for you? |
ping @psharda |
ah sorry , just saw this now |
I merged this branch in my popiii branch. I get some warnings that I didnt get before on Setonix:
|
All of the warnings are coming from the Ascent build. If you turn off Ascent, they should go away. |
So then should ASCENT be OFF by default in setonix.cmake? |
I think it should be on by default, unless you don't think you'll want to run with it on? |
Since Ascent is OFF by default, I've commented out that line. Users can un-comment or set |
/azp run |
/azp run |
@BenWibking seems that we need > 60 minutes on avatargpus now? |
hmm it does look like it. can you increase the timeout to, say, 90 minutes? |
oh but it's your branch |
ah lol, oops. too many PRs, sorry. |
/azp run |
/azp run |
@psharda can you approve? |
Approved from 37000 ft above sea level |
Update the Setonix build and runtime configuration following the September 2023 upgrade.