Skip to content

Try to add a sweat test#46

Open
asinghvi17 wants to merge 5 commits intomainfrom
as/sweat_test
Open

Try to add a sweat test#46
asinghvi17 wants to merge 5 commits intomainfrom
as/sweat_test

Conversation

@asinghvi17
Copy link
Member

The idea is to try regridding each grid against each other grid we support to see where problems crop up. I think this found quite a few though

@asinghvi17
Copy link
Member Author

Ok looks like we're going to want to:

  • Perform quadrature over the example field in spherical barycentric space, to get the average value over the cell rather than the value at the midpoint (particularly applicable for the gulf stream field)
  • Just use regular vectors of the right length to contain data and regrid into/out of, instead of using the native field types, since they are quite difficult to write to in practice
  • Make sure all fields are shifted such that their minimum is 0, so we can test integral preservation more easily

These should let most of the tests pass, I hope...

@navidcy
Copy link
Collaborator

navidcy commented Mar 5, 2026

The CI passes?

@asinghvi17
Copy link
Member Author

It does but the rtol is too high, 1e-2. I'd rather get the "right value" of the field analytically there.

@asinghvi17
Copy link
Member Author

Yup definitely a lot of weirdness this is uncovering.

@asinghvi17
Copy link
Member Author

Wait what?! It completely fails locally

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.

2 participants