Skip to content

Add nlopt as extra peg it#1274

Merged
ksbeattie merged 11 commits intomasterfrom
nlopt-issue-1273
Feb 24, 2026
Merged

Add nlopt as extra peg it#1274
ksbeattie merged 11 commits intomasterfrom
nlopt-issue-1273

Conversation

@boverhof
Copy link
Copy Markdown
Member

@boverhof boverhof commented Dec 10, 2025

Fixes/Addresses:

#1273

Summary/Motivation:

Add extras dependency so can peg nlopt to a version that is compatible with FOQUS matplotlib 2.7.5/numpy

Changes proposed in this PR:

  • setup.py add nlopt to extras

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.

@ksbeattie ksbeattie added the Priority:High High Priority Issue or PR label Dec 10, 2025
This was referenced Dec 19, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 39.89%. Comparing base (57fb2e2) to head (49bf399).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1274      +/-   ##
==========================================
+ Coverage   39.87%   39.89%   +0.02%     
==========================================
  Files         165      165              
  Lines       37747    37747              
  Branches     5865     5865              
==========================================
+ Hits        15052    15060       +8     
+ Misses      21597    21587      -10     
- Partials     1098     1100       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Member

@sotorrio1 sotorrio1 left a comment

Choose a reason for hiding this comment

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

LGTM

@ksbeattie ksbeattie enabled auto-merge (squash) February 24, 2026 21:27
@ksbeattie ksbeattie merged commit 5fd975d into master Feb 24, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority:High High Priority Issue or PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants