Skip to content

Add deployment flow on MAGIA V2 (upstream) - #59

Open
callme-sam wants to merge 316 commits into
mainfrom
sc/onnx_deeploy_pr_upstream
Open

Add deployment flow on MAGIA V2 (upstream)#59
callme-sam wants to merge 316 commits into
mainfrom
sc/onnx_deeploy_pr_upstream

Conversation

@callme-sam

Copy link
Copy Markdown
Contributor

No description provided.

Dequino and others added 30 commits March 5, 2026 15:39
If compiled with optimization (-O1/2/3), the compiler prematurely
reuses a scalar register to load the multiplier constant
(COEF) before the second vector initialization (vfmv.v.f v8, %0)
is completed.

This register contention causes the vector accumulator v8 to be
initialized with the coefficient value instead of ZERO,
invalidating the reduction.

Fixed by enforcing explicit vector register allocation.
Limitations:
- no auto_pad
- no ceil_mode (floor only)
- no asymmetric padding
- output length must be precomputed
- stride must be multiple of power-of-two
Limitations:
- no auto_pad
- no ceil_mode (floor only)
- no asymmetric padding
- no storage order
- no indices output
- output length must be precomputed
- stride must be multiple of power-of-two
callme-sam and others added 25 commits August 6, 2026 10:40
Previously only axis=-1 where supported
Accumulation in fp32 is under #ifdef and can be enabled with #define
Implemented to test difference in accuracy
scalar version of sigmoid is under #ifdef and can be enabled with #define
Implemented to test difference in accuracy
Both versions are under #ifdef and can be enabled with #define
Implemented in an attempt to improve the accuracy of the yolo deployment
For better accuracy enable the scalar/wide sigmoid via #define in
kernel task source file
The data translation unit only included <stdint.h>, so 'float16' was
undefined there when compiling with GCC_MULTILIB
The public header was committed empty by mistake. GCC_PULP only warns
and links anyway, but GCC_MULTILIB rejects it as an error
@callme-sam
callme-sam requested a review from Dequino September 2, 2026 15:13
@callme-sam

Copy link
Copy Markdown
Contributor Author

test results
deployment_test_results.xlsx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants