-
Notifications
You must be signed in to change notification settings - Fork 30
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
Bad @brief in energy_ratio_by_chunks #106
Comments
@Nexxel excuse me, but do you know how is segment_focus used in the calculation? |
Hi @billydentsu, This operation divides the sum of square elements within a chunk by the sum of square elements of the whole time series. Thus, the segment_focus parameter receives the segment number you want to compute. Cheers. |
Hi @avilchess, I see. Thanks for explaining. |
@billydentsu |
Describe the bug
The actual @brief of energy_ratio_by_chunks in khiva_c is
@brief Calculates the sum of squares of chunk i out of N chunks expressed as a ratio.
Expected behavior
The @brief should be:
@brief Calculates the sum of squares of chunk i out of N chunks expressed as a ratio
The text was updated successfully, but these errors were encountered: