ci(nixos): TCN load probe (catch free(): invalid size) - #80
Open
ajlennon wants to merge 38 commits into
Open
Conversation
Julian mid 993: tcn_load_probe aborts at CreateSession on Nix while ci-smoke loads. Gate the probe on the self-hosted runner. Assisted-by: Cursor
Inline free() grep parentheses broke the nixos job before the probe ran. Assisted-by: Cursor
Split Nix ORT vs Godot-only free() (Julian 993 / CI repro). Assisted-by: Cursor
Assisted-by: Cursor
Assisted-by: Cursor
Prove TCN load under Nix Godot against #26 before it lands on main. Assisted-by: Cursor
Branch pin broke once #26 deleted the feature ref. Assisted-by: Cursor
Member
Author
|
Re-running nixos after godot-onnx-loader #27 (Godot-matched libstdc++) landed on main. CreateSessionFromArray alone did not clear Nix Godot TCN free(). |
Member
Author
RPATH cannot replace a libstdc++ Godot already mapped; preload the copies we ship beside ORT as a last cheap probe before ORT rebuild. Assisted-by: Cursor
Preloading bundled libstdc++ aborted Godot — disable by default and print wrapper head so we can match the real runtime. Assisted-by: Cursor
Assisted-by: Cursor
Julian mid 1034 — report which lever clears free() under Nix. Assisted-by: Cursor
Assisted-by: Cursor
Julian mid 1067 — stop mixing MS ORT into the Nix path. Assisted-by: Cursor
Assisted-by: Cursor
Assisted-by: Cursor
Reuse the loader helper's resolved store symlink instead of selecting an arbitrary ONNX Runtime version from /nix/store.\n\nAssisted-by: Codex
Use the repository's locked nixpkgs revision for Godot and ONNX Runtime instead of the moving nixos-26.05 branch.\n\nAssisted-by: Codex
Assisted-by: Codex
Assisted-by: Codex
Assisted-by: Codex
Gate full TCN inference in the Godot smoke so editor playback dependencies cannot pass on load-only coverage.\n\nAssisted-by: Codex
Embed explicit normalization, vocabulary, provenance, and doc fields; verify the saved ONNX contract after migration.\n\nAssisted-by: Codex
Migrate the smoke sanity/export consumers off model.json and gate the embedded schema-2 contract in CI.\n\nAssisted-by: Codex
Add causal sparse smoothing and a tested one-byte 20 ms transition protocol before evaluating a learned stage-B mapper.\n\nAssisted-by: Codex
Preserve the 68-label stressed ARPA training inventory while reporting stress-collapsed PER and transition-boundary error.\n\nAssisted-by: Codex
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.
Summary
tcn_load_probe.gdon nixos-self-hosted after mel smokefree(): invalid size/ Abort (Julian mid 993)Test plan
Assisted-by: Cursor