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

fehmpytests for barometric fails #73

Open
millerta opened this issue Sep 4, 2024 · 2 comments
Open

fehmpytests for barometric fails #73

millerta opened this issue Sep 4, 2024 · 2 comments

Comments

@millerta
Copy link
Collaborator

millerta commented Sep 4, 2024

Re George, this VV test fails and is now skipped in the VV Test Suite.
The test is actually a combination of baro_trans and baro_vel which can be added into fehmpytests as 2 test cases.
Compare output from each to the output for VV versions.

@xiang112233
Copy link
Collaborator

If you use the "main rip.f' file under the ./src/PC folder to re-compile the win-executable, it passes the Barometric test.

@millerta
Copy link
Collaborator Author

Construct one or more simplified versions of this test for fehmpytests and keep the full version in the VV Test Suite. Some parts of this test work, but not the full complicated version in VV Barometric which uses input from other runs.

Output files from linux and windows compare successfully with manual checks in the VV test suite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants