Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Interaction between XIOS and doctest? #740

Open
jwallwork23 opened this issue Nov 20, 2024 · 0 comments · May be fixed by #757
Open

Interaction between XIOS and doctest? #740

jwallwork23 opened this issue Nov 20, 2024 · 0 comments · May be fixed by #757
Assignees
Labels
bug Something isn't working ICCS Tasks or reviews for the ICCS team

Comments

@jwallwork23
Copy link
Contributor

          As mentioned in the docstring for `assertIsClose`, there is a bizarre bug that crops up if I try to call `REQUIRE` here. I derived a minimum failing example [here](https://github.com/nextsimhub/nextsimdg/blob/bugfix731_xios-tests-mfe2/core/test/XiosReadWrite_test.cpp) but figuring out the exact issue was taking a while and I feel that this isn't important to track down right now, unless the reviewer(s) think so. If we are happy to leave it for now then I'll open an issue as a reminder.

Originally posted by @jwallwork23 in #738 (comment)

@jwallwork23 jwallwork23 self-assigned this Nov 20, 2024
@jwallwork23 jwallwork23 added bug Something isn't working ICCS Tasks or reviews for the ICCS team labels Nov 20, 2024
@jwallwork23 jwallwork23 linked a pull request Dec 11, 2024 that will close this issue
12 tasks
@jwallwork23 jwallwork23 moved this from Todo to Review required in neXtSIM_DG overview Dec 11, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working ICCS Tasks or reviews for the ICCS team
Projects
Status: Review required
Development

Successfully merging a pull request may close this issue.

1 participant