Skip to content

Don't catch ProblemDefinitionError in MPCQP constructor#12

Merged
stephane-caron merged 2 commits intomainfrom
change/mpcqp_try_except
Nov 17, 2025
Merged

Don't catch ProblemDefinitionError in MPCQP constructor#12
stephane-caron merged 2 commits intomainfrom
change/mpcqp_try_except

Conversation

@stephane-caron
Copy link
Copy Markdown
Owner

Closes #11

@stephane-caron stephane-caron force-pushed the change/mpcqp_try_except branch from 6c8ff99 to ac4a5ca Compare November 17, 2025 16:01
@stephane-caron stephane-caron changed the title Don't catch ProblemDefinitionError in MPCQP ctor Don't catch ProblemDefinitionError in MPCQP constructor Nov 17, 2025
@stephane-caron stephane-caron merged commit 66806a6 into main Nov 17, 2025
8 checks passed
@stephane-caron stephane-caron deleted the change/mpcqp_try_except branch November 17, 2025 16:48
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 19436111462

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 79.434%

Totals Coverage Status
Change from base Build 19435947277: 0.3%
Covered Lines: 421
Relevant Lines: 530

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 19435911929

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 79.545%

Totals Coverage Status
Change from base Build 19435878418: 0.3%
Covered Lines: 420
Relevant Lines: 528

💛 - Coveralls

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 19435974098

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 2 of 2 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 79.434%

Totals Coverage Status
Change from base Build 19435947277: 0.3%
Covered Lines: 421
Relevant Lines: 530

💛 - Coveralls

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.

Error catching if goal/target state is not specified

2 participants