Skip to content

[tmva][sofie] Add unit test for ONNX GELU operator parsing#21357

Open
MrViiV wants to merge 1 commit intoroot-project:masterfrom
MrViiV:add-gelu-test
Open

[tmva][sofie] Add unit test for ONNX GELU operator parsing#21357
MrViiV wants to merge 1 commit intoroot-project:masterfrom
MrViiV:add-gelu-test

Conversation

@MrViiV
Copy link

@MrViiV MrViiV commented Feb 23, 2026

Summary

This PR adds a unit test for the ONNX GELU operator in the SOFIE ONNX parser.

GELU is widely used in modern neural network architectures such as Transformers and BERT. Adding a dedicated unit test helps validate the parser behavior, improve test coverage, and prevent regressions as GELU operator support evolves.

Changes

Added TestGelu.cxx in tmva/sofie/test

Added gelu.onnx test model

Notes

I am happy to update the test to follow any specific ROOT testing conventions if needed.

@MrViiV MrViiV requested a review from lmoneta as a code owner February 23, 2026 19:15
@guitargeek guitargeek changed the title [TMVA][SOFIE] Add unit test for ONNX GELU operator parsing [tmva][sofie] Add unit test for ONNX GELU operator parsing Feb 23, 2026
@lmoneta lmoneta requested a review from sanjibansg February 26, 2026 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants