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

Improve documentation of smoothness() #519

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

DominiqueMakowski
Copy link
Member

@DominiqueMakowski DominiqueMakowski commented Jun 24, 2024

Close #374

@IndrajeetPatil IndrajeetPatil requested a review from mattansb June 24, 2024 13:17
R/smoothness.R Outdated Show resolved Hide resolved
R/smoothness.R Show resolved Hide resolved
@etiennebacher etiennebacher changed the title [docs] document smoothness #374 Improve documentation of smoothness() Jun 24, 2024
@strengejacke
Copy link
Member

Waiting for Dom to update news or add tests...

image

Comment on lines 32 to 33
#' @return Value of smoothness.
#' @references https://stats.stackexchange.com/questions/24607/how-to-measure-smoothness-of-a-time-series-in-r
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be useful to have some details on the range of the output. Based on the examples I thought it would be between 0 and 1 but one of the answers in the CrossValidated post linked says it can go to -1. What does it mean to have a smoothness of -0.5?

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.

Doc of smoothness() looks quite empty
3 participants