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

Include TWDTW classification in sits #1021

Closed
gilbertocamara opened this issue Oct 12, 2023 · 2 comments
Closed

Include TWDTW classification in sits #1021

gilbertocamara opened this issue Oct 12, 2023 · 2 comments

Comments

@gilbertocamara
Copy link
Contributor

Describe the new API function requested
Include the new version of the dtwSat package by @vwmaus in sits. The previous version of dtwSat used the S4 model, which made it difficult to integrate the algorithm in sits. The new version uses S3, thus making the integration of dtwSat into sits.

Associated sits API function
Develop a new sits_dtw() that can be used in connection with sits_train().

@vwmaus
Copy link
Collaborator

vwmaus commented Oct 19, 2023

Implementation steps:

  • Build a test set
  • Create a new method in dtwSat that supports inputs from sits and returns a twdtw_knn1 model. The new method is expected in dtwSat CRAN release 1.0-2.
  • Integrate the twdtw_knn1 to sits_twdtw()

@OldLipe
Copy link
Contributor

OldLipe commented Jun 18, 2024

Move this issue to #1099.

@OldLipe OldLipe closed this as completed Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

4 participants