Skip to content

Cleanup THROW mechanism - #207

Open
lpottier wants to merge 2 commits into
developfrom
206-revamp-ams-debug-throw
Open

Cleanup THROW mechanism#207
lpottier wants to merge 2 commits into
developfrom
206-revamp-ams-debug-throw

Conversation

@lpottier

@lpottier lpottier commented Nov 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

This PR clean up and clarify THROW in AMS. See issue #206

  • THROW actually throw an exception.
  • Default logging moved to warning (from error).
  • Cleanup some code using std::experimental::filesystem

@lpottier
Modified THROW to acctualy throw exception. Now we throw when a model…
98d9b36 
… is not loaded

Signed-off-by: Loic Pottier <pottier1@llnl.gov>
@lpottier
removed some existing exit() and set default logging to warning (from…
bdd7995 
… error)

Signed-off-by: Loic Pottier <pottier1@llnl.gov>
@lpottier
lpottier deployed to github-pages November 3, 2026 17:34 — with GitHub Actions Active

This branch was successfully deployed

1 active deployment
github-pages — bdd7995d Deployed Nov 3, 2026 by lpottier via build-docs #119
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@YohannDudouit YohannDudouit Awaiting requested review from YohannDudouit
@rblake-llnl rblake-llnl Awaiting requested review from rblake-llnl

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@lpottier