Skip to content

Updates from master#763

Merged
jberthold merged 5 commits into
feature/p-tokenfrom
master
Oct 22, 2025
Merged

Updates from master#763
jberthold merged 5 commits into
feature/p-tokenfrom
master

Conversation

@jberthold
Copy link
Copy Markdown
Collaborator

  • Python code refactoring
  • decoding structs
  • handle functions with empty body
  • Fix for statistics (several paths)
  • nix overlay fix

tothtamas28 and others added 5 commits October 21, 2025 06:01
Split method `make_call_config` into `_make_concrete_call_config` and
`_make_symbolic_call_config`.
1. Handle empty-body functions for linking and proving;
2. Log empty-body function existence when logging the reduced items
table size;
3. Quick-compact for Statics, VTable and Function encoding.
Previously, when RV nix derivations were using `poetry2nix`, overriding
a nix input in nix was also overriding the python code and resources
used in `poetry2nix`. This allowed for easy overrides of python and K
semantics changes by utilizing nix overrides.

With the migration to `uv` and `uv2nix`, this was behavior was removed
due to the respective code not documenting it's purpose. In addition,
due to `uv2nix` relying on state-of-the-art python nix packaging tooling
`pyproject-nix` instead of `nixpkgs`, ensuring identical override
behaviour requires more nix logic.

This PR re-introduces this kind of behaviour with `uv2nix`, though it
depends on newly introduced overlays in the k nix flake.

### Example
You can override k by running: `kup install kmir --override k-framework
~/path/to/local-k-checkout`. This will now also build kmir with the
python and K code in `~/path/to/local-k-checkout`.

---------

Co-authored-by: Jost Berthold <jost.berthold@gmail.com>
Co-authored-by: Tamás Tóth <tothtamas28@users.noreply.github.com>
Co-authored-by: Jost Berthold <jost.berthold@gmail.com>
@jberthold jberthold marked this pull request as ready for review October 22, 2025 00:47
@jberthold jberthold merged commit 7c50f5c into feature/p-token Oct 22, 2025
8 checks passed
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.

5 participants