Closed
Conversation
* code: expose knitro context and modify _extract_values * doc: update release_notes.rst * code: include pre-commit checks
* code: re-introduce knitro context closure and export solver quantities * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * code: place quantities calculation elsewhere * code: place quantities calculation elsewhere * code: add new quantities extracted from knitro context * code: add int * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * code: add release notes --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Fabian Hofmann <fab.hof@gmx.de>
* Strengthen masked IIS regression test * Fix Xpress IIS mapping for masked constraints * Fix typing in masked IIS regression test
* fix: use xarray.Dataset copy instead of constructor since the latest xarray version, passing a Dataset as `data_vars` to the Dataset constructor is not supported. transpose and assign_coords already returns a new dataset. * Revert "fix: temporarily constrain xarray" This reverts commit 545b563. --------- Co-authored-by: Jonas Hoersch <jonas.hoersch@openenergytransition.org>
* fix: blacklist highs 1.14.0, relax python constraints * Update release_notes.rst * fix: temporarily constrain xarray * fix: fix types in expressions.merge * fix: more types problems * fix: rely on issubclass
FBumann
requested changes
Apr 21, 2026
Co-authored-by: Felix <117816358+FBumann@users.noreply.github.com>
Collaborator
Author
|
@FBumann wanna have another look? |
FBumann
approved these changes
Apr 21, 2026
Collaborator
|
@FabianHofmann No, the xarray one alongside highs issue was destined to slip through, as they both tried to get CI going before the other merged. The rest should not have introduced such issues |
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
5 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.
Summary
Patch release cherry-picked onto
v0.6.6. Contains four fixes; no new features.xarray.Dataset.copyinstead of constructor (xarray compatibility)pyproject.tomlpin + release note; the PR's follow-up type cleanups inexpressions.py/tests depend on refactors (Mergeable,issubclass) not present onv0.6.6.Branch based off
v0.6.6, notmaster, to avoid dragging in unreleased feature work.Test plan
ruff check/ruff format --checkcleanpytest(excludingtest/remote— needsrequests/google-cloud-storageextras): 1582 passed, 232 skippedv0.6.7to trigger PyPI publishVersion 0.6.7release-notes block tomasterand remove the four corresponding bullets from theUpcoming Versionsection