You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[Lane 2 · kotlin-core] ONNX importer — import ONNX Silu as a real op #1402
Sub-issue of #1400 ([Feature]: ONNX importer — replace placeholder ops with real operations, stream tensors in OnnxModelReader).
Lane: 2 · Kotlin core Skill needed: Kotlin. No ONNX-protobuf knowledge needed — the node is already parsed; you only map it. Size: xs (under 1 hour) Blocked by: nothing
What to do
In skainet-io/skainet-io-onnx/src/commonMain/kotlin/sk/ainet/io/onnx/OnnxImport.kt (toOperation()), Silu currently maps to OnnxPlaceholderOperation, which throws when the graph is executed. Map it to the SKaiNET silu operation instead, copying how RELU / CONV map to real operations.
Add a one-node import + execute test next to the existing importer tests in skainet-io/skainet-io-onnx/src/commonTest.
Run ./gradlew :skainet-io:skainet-io-onnx:jvmTest.
Acceptance
A one-node Silu graph imports and executes with the expected values
Result reported back on the parent issue
Notes
If something here turns out to be wrong or much bigger than the size label, comment on this issue or open a specific bug instead of blocking the parent.
Activity