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

Update for NonlinearOperator change #321

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

odow
Copy link
Contributor

@odow odow commented Sep 7, 2023

We swapped the order of arguments to NonlinearOperator. Otherwise, tests as passing with jump-dev/JuMP.jl#3489

@odow
Copy link
Contributor Author

odow commented Sep 7, 2023

(You might want to hide the whitespace to see the changes.)

@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Merging #321 (690908c) into jump_nlp (ac40507) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           jump_nlp     #321   +/-   ##
=========================================
  Coverage     99.77%   99.77%           
=========================================
  Files            36       36           
  Lines          6573     6573           
=========================================
  Hits           6558     6558           
  Misses           15       15           
Files Changed Coverage Δ
src/nlp.jl 100.00% <100.00%> (ø)

📢 Have feedback on the report? Share it here.

@pulsipher
Copy link
Collaborator

Thanks for drafting this

@pulsipher pulsipher merged commit 2711001 into infiniteopt:jump_nlp Sep 8, 2023
8 of 9 checks passed
pulsipher added a commit that referenced this pull request Sep 15, 2023
* Update to use NonlinearExpr in src

* update optimizer_index for constraints

* Begin updates for docs and tests

* Update datatypes and tests

* Add registration tests

* Added more tests

* Reimplement all tests

* Update nlp types

* added comment

* Update package dependencies

* incorporate `JuMP.flatten`

* fix @rewrite

* Update w/ JuMP

* remove use of `MA.@rewrite`

* Update user defined functions

* Update registration tests

* `flatten` -> `flatten!`

* Update registration tests

* Prepare tests for CI

* fix syntax bug

* test fix

* minor fixes

* Minor doc fix

* minor fix

* Update to JuMP changes

* fix `build_measure`

* Minor fixes

* fix tests

* Update to JuMP's latest release

* Fix doctests

* Another doctest fix

* Fix header link

* Update for NonlinearOperator change (#321)

* test fix

* Add AST mapper

* Update dep test

* fix tests

* minor fixes

* minor test fix

* update to JuMP 1.15

---------

Co-authored-by: Oscar Dowson <[email protected]>
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