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

Limiting design size maximum #1244

Merged
merged 2 commits into from
Oct 1, 2024
Merged

Conversation

sotorrio1
Copy link
Member

Fixes/Addresses:

It address issue #1240

Summary/Motivation:

The spin boxes to set the design size in the SDoE Analysis Dialog (for either USF, NUSF or IRSF) should be set to a maximum of the size of the candidate set minus one to avoid issues with the SDoE algorithm.

Changes proposed in this PR:

  • Previously design size was set to a maximum equal to the candidate set size.
  • Now design size will be set to a maximum that is equal to the candidate set size minus one.

Legal Acknowledgement

By contributing to this software project, I agree to the following terms and conditions for my contribution:

  1. I agree my contributions are submitted under the copyright and license terms described in the LICENSE.md file at the top level of this directory.
  2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

…t minus 1 so users can't run USF, NUSF, or IRSF with a design size equal or greater than the candidate set size.
@sotorrio1 sotorrio1 added SDoE Sequential Design of Experiments FOQUS GUI Issues related to information displayed on the GUI, or broken buttons labels Sep 24, 2024
@sotorrio1 sotorrio1 self-assigned this Sep 24, 2024
@sotorrio1 sotorrio1 linked an issue Sep 24, 2024 that may be closed by this pull request
@sotorrio1 sotorrio1 requested a review from bpaul4 September 24, 2024 23:21
@sotorrio1
Copy link
Member Author

@bpaul4 I knew there was a check somewhere in the code, problem was that it was set to a maximum equal to the candidate set size. Now I'm limiting the maximum to candidate set size minus 1.

Let me know if this works for you (you shouldn't be able to set your design size to the same number as the candidate set size).

Copy link

codecov bot commented Sep 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.78%. Comparing base (2e54f98) to head (5edc9da).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1244      +/-   ##
==========================================
+ Coverage   38.75%   38.78%   +0.02%     
==========================================
  Files         164      164              
  Lines       37139    37139              
  Branches     6164     6164              
==========================================
+ Hits        14392    14403      +11     
+ Misses      21611    21597      -14     
- Partials     1136     1139       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@bpaul4 bpaul4 left a comment

Choose a reason for hiding this comment

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

It works perfectly! Thanks for making this change.

@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Oct 1, 2024
@ksbeattie ksbeattie enabled auto-merge (squash) October 1, 2024 19:57
@ksbeattie ksbeattie merged commit 6e306a6 into CCSI-Toolset:master Oct 1, 2024
28 of 29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FOQUS GUI Issues related to information displayed on the GUI, or broken buttons Priority:Normal Normal Priority Issue or PR SDoE Sequential Design of Experiments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SDOE NUSF Bug When Design Size Equals Candidate Set Size
3 participants