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.

Sign up for GitHub

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

Jump to bottom

minor - drop unneeded casts #1667

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

minor - drop unneeded casts #1667

wants to merge 3 commits into from
+15 −14

Conversation

jeremylt
Copy link
Member

oops

@jeremylt jeremylt self-assigned this Sep 19, 2024
@jrwrigh
Copy link
Collaborator

jrwrigh commented Sep 19, 2024

For the errors on Noether in the fluids example:

[WARNING] yaksa: 1 leaked handle pool objects

I've only recently seen those on Aurora. Possibly due to latest PETSc main?

@jeremylt
Copy link
Member Author

Yeah, looks like PETSc is leaking something somewhere maybe? Its for all PETSc examples

@jeremylt
Copy link
Member Author

jeremylt commented Sep 19, 2024 •
edited
Loading

This isn't expressing in Ratel CI - I wonder if there's something fishy with the int64 builds on Noether? Or maybe a package included?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jrwrigh jrwrigh jrwrigh approved these changes

Assignees

@jeremylt jeremylt

Labels
1-In Review minor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jeremylt @jrwrigh