test(e2e): Add neural-network integration test.#385
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
WalkthroughAdds a layered neural-network workload to the Huntsman e2e crate, including deterministic wiring, MessagePack payload helpers, Spider execution validation, and exclusion of e2e tests from the unit-test executor. ChangesNeural network end-to-end validation
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant test_nn
participant NeuralNetwork
participant SpiderTestDriver
participant Spider
test_nn->>NeuralNetwork: simulate deterministic inputs
test_nn->>NeuralNetwork: build task graph
test_nn->>SpiderTestDriver: submit graph and encoded inputs
SpiderTestDriver->>Spider: execute dense-neuron tasks
Spider-->>SpiderTestDriver: return outputs
SpiderTestDriver-->>test_nn: decode output payload
test_nn->>test_nn: compare outputs with simulation
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Description
This PR adds neural-network integration test, with the following change:
tests/huntsman/e2e/src/nn: New mod contains neural network helpers.neuron.rs:Neuronactivation enums and mapping to task name and evaluate function.wiring.rs: Wiring outputs to inputs.network.rs:NeuralNetworkstruct that holds all neural network information and conversion toTaskGraph.tests/huntsman/e2e/payload_serde.rs: Helpers that serialize/deserialize taskmsgpackinput/output payload.taskfiles/test.yaml: Excludese2etests fromtest:rust-unit-teststask.Checklist
breaking change.
Validation performed
Summary by CodeRabbit