Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API enhancements #1312

Closed
manuel-loew-infineon opened this issue Sep 20, 2021 · 4 comments
Closed

API enhancements #1312

manuel-loew-infineon opened this issue Sep 20, 2021 · 4 comments
Labels
DSP enhancement moved moved to new github repository

Comments

@manuel-loew-infineon
Copy link

Hi,

are there API functions in CMSIS DSP for

  • fill f64: to be used to initialize a complex f32 array with constant values, similar to arm_fill_f32 for real values?
  • extract the real/imaginary parts of a complex array into a real array?
  • sum up all elements of a real vector into a scalar value?

Maybe I missed that one of the above functionality can be mapped to an existing function in the CMSIS DSP API?

Best regards
ml-0

@christophe0606
Copy link
Contributor

@ml-0 arm_fill_f64 was added very recently in the develop branch so it is now available.

You're right that we do not have the other two functions and with vectorization (M55/Neon) it would be interesting to have them.

@christophe0606
Copy link
Contributor

@ml-0 Hi,

Another question : For the accumulation, which datatype would interest you more ? f32 ? f16 ? q31 ? q15 ?

And if accumulating fixed points: what should be the format of the output ? For instance : accumulating a q31 into a q64 ? Accumulating a q15 into a q31 ?

Thanks.

@manuel-loew-infineon
Copy link
Author

Currently, f32. Consequently, I did not think of the output format of the element sum in q31/q15. In my use case I do not even use the absolute value. I just use the result in comparisons of several element sums to find the largest one.

So, if I did the quantized implementation, the best fit would be accumulating q31 into q63 because I am heading for speed, avoiding any scaling/division of the result.

@christophe0606
Copy link
Contributor

We are moving CMSIS-DSP to a new repository : https://github.com/ARM-software/CMSIS-DSP

I have recreated the github issues on the new repository (sometimes merging a few related ones). You can see the link just above this comment (... mentioned this issue ...).

So, I am closing the github issue here.

@christophe0606 christophe0606 added the moved moved to new github repository label Jun 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DSP enhancement moved moved to new github repository
Projects
None yet
Development

No branches or pull requests

2 participants