Skip to content

[inferno-ml-server] Fix argument position when applying scripts#188

Merged
ngua merged 8 commits intomainfrom
rory-imls-fix-arg-positions
Dec 8, 2025
Merged

[inferno-ml-server] Fix argument position when applying scripts#188
ngua merged 8 commits intomainfrom
rory-imls-fix-arg-positions

Conversation

@ngua
Copy link
Contributor

@ngua ngua commented Dec 8, 2025

He had an issue where inferno-ml-server expected function arguments to be in a specific order, but we can't always expect that (as a subtle bug recently demonstrated). So this fixes it to extract all arguments from the Lam in the VCFunction, then do a Map.lookup on all inputs/outputs/models to reconstruct the correct ordering to apply the function. This makes inferno-ml-server work with any argument ordering, which is far more resilient and doesn't rely on fragile ordering/sorting guarantees to be synchronized everywhere

@ngua ngua merged commit fe19064 into main Dec 8, 2025
1 check passed
@ngua ngua deleted the rory-imls-fix-arg-positions branch December 8, 2025 05:35
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