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 Pad as a real op #1405
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: s (a few hours) Blocked by: nothing
What to do
In skainet-io/skainet-io-onnx/src/commonMain/kotlin/sk/ainet/io/onnx/OnnxImport.kt (toOperation()), Pad currently maps to OnnxPlaceholderOperation, which throws when the graph is executed. Map it to the SKaiNET pad2d operation for 4-D constant-mode padding (reject other modes with a clear error) 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 Pad 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