Pin co-developed dependencies to the OpenAPI.jl 1.x branches - #300
Merged
Merged
Conversation
The OpenAPI model packages pinned `jd/openapi_deps_update`, which predates the 0.1.0 release, its precompilation fix and the Julia 1.13 work; point them at main. InfrastructureSystems moves from the jd/openapi-1x-suite branch to IS4, where it merged. Restore the CI Julia matrices to '1' now that the 1.13 fix has landed.
…ilds again PowerSystems #1783 replaced must_run with commitment_mode and the Bool status with OperationalStates, and left no shim. The feedforwards' _is_must_run trait moves to utils/psy_utils.jl and reads commitment_mode == MUST_RUN for ThermalGen and HydroPumpTurbine (false for anything else, so SELF_SCHEDULED and RELIABILITY stay committable); is_online reads status as ONLINE or STARTUP. The 23 must-run sites and the eleven Bool status reads go through them, as do the test utilities' duration checks, and IOM.get_must_run forwards to the trait for IOM's own start-up and ramp code. SwitchedAdmittance lost its Y field: the shunt model spans the blocks and takes solved_admittance, else the engaged blocks, as the fixed susceptance. FixValueParameter metas were string(::Type), which carries the module prefix in the test workers now that the variable types live in IOM; the four sites use nameof like the other feedforward metas. Tests construct with the enums and the current shunt fields, wrap their HVDC and converter losses in LossCurve on the component base the readers documented, and the FACTS shunt test asks for BYP instead of a nothing mode the OpenAPI 0.1.0 schema cannot encode. The test environment pins InfrastructureOptimizationModels to rh/cost_coefficient_ratio (IOM #166) until it merges: IOM main still calls the convert_cost_coefficient signature InfrastructureSystems removed.
Follow PowerSystems #1783 and the OpenAPI 1.x structs so the suite builds again
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
2 participants
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.
Pins the co-developed dependencies to the branches that carry the OpenAPI.jl 1.x work, so this repo's environments resolve from GitHub again:
jd/openapi-1x-suitejd/openapi-1xjd/openapi_deps_update(bysubdir)[sources]only;[deps]and[compat]are untouched. Temporary until the OpenAPI packages are registered and the psy6-line branches merge. Verified by deleting the manifests and instantiating the test environment from GitHub, then compiling the package.🤖 Generated with Claude Code
https://claude.ai/code/session_01QDkZStcFzebhh9XFpPKRSo