Conversation
…nd overrides
- Update type declarations for all 7 active backend options in ADNLPModeler
- Update validate_backend_override to accept nothing, Type{<:ADBackend}, or ADBackend instances
- Add comprehensive tests covering all three forms (nothing, Type, instance)
- Update docstring with clear examples for each usage pattern
- Remove references to commented-out NLS residual backend options from tests
- Add detailed report in .reports/2026-02_14_backend/
This aligns with ADNLPModels.jl API which accepts both types (constructed internally)
and pre-constructed instances for backend overrides.
Fixes #27160a
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #70 +/- ##
==========================================
Coverage ? 96.04%
==========================================
Files ? 38
Lines ? 835
Branches ? 0
==========================================
Hits ? 802
Misses ? 33
Partials ? 0 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
- Add Test. prefix to bare @testset macros in test_enhanced_options.jl - Fix validate_backend_override tests to use ADBackend subtypes instead of Float64/Int - Add FakeCoverageBackend for proper ADBackend testing - Update test expectations to match new validation logic All tests now pass: 175/175
- Add v0.3.1-beta section to CHANGELOG with backend override enhancements
- Note no breaking changes in v0.3.1-beta in BREAKING.md
- Document Type{<:ADBackend} and ADBackend instance support
- Include technical details and usage examples
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.