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 LeakyRelu as a real op #1401
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()), LeakyRelu currently maps to OnnxPlaceholderOperation, which throws when the graph is executed. Map it to the SKaiNET leakyRelu operation with its alpha attribute (default 0.01) 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 LeakyRelu 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