Skip to content

Conversation

@ktbolt
Copy link
Collaborator

@ktbolt ktbolt commented Dec 11, 2024

I've fixed writing integral quantities (see #309).

I've also updated svMultiPhysics/tests/cases/fluid/pipe_RCR_3d to write integral quantities since this is not tested anywhere.

@ktbolt ktbolt requested a review from dcodoni December 11, 2024 23:23
@ktbolt
Copy link
Collaborator Author

ktbolt commented Dec 12, 2024

The errors are caused by exceptions thrown by the integration functions (e.g. all_fun::integ()). This was caused by initializing the output types to true like the original code did, changed that.

Copy link
Contributor

@dcodoni dcodoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good! I only have one comment/question.

@dcodoni dcodoni merged commit 6ecb75f into SimVascular:main Dec 18, 2024
5 checks passed
@ktbolt ktbolt deleted the boundary-integral-file-time-step_309 branch January 8, 2025 20:24
divyaadil23 pushed a commit to divyaadil23/svFSIplus that referenced this pull request Jan 29, 2025
* Change file name.

* Add functionality to write integral quantities and replace using wtn Boolean vector.

* Remove initialization of output type options to true.

* Fix test for integrals.

* Remove redundant loop.
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