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

Add TEDA-CDD (Concept drift detector based on TEDA) #1577

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

yuritpinheiro
Copy link

I would like to contribute with a new drift detector.

TEDA-CDD is a evolving memory efficient drift detector.

@hoanganhngo610
Copy link
Contributor

hoanganhngo610 commented Jul 27, 2024

@yuritpinheiro Hi Yuri. First of all, thank you very much for your initiative to contribute to River by implementing TEDA-CDD. However, if I'm seeing it correctly, you have not added any line of code into the teda_cdd.py file. If this PR remains at this state, I regret to say that I will have to close this PR.

@yuritpinheiro
Copy link
Author

I'm sorry. I needed the link to the PR but i'm still working on the source. This week, I will add the code properly.

@hoanganhngo610
Copy link
Contributor

@yuritpinheiro Thank you so much for your response! I look forward to your contributions!

@hoanganhngo610
Copy link
Contributor

Hi @yuritpinheiro.

First of all, thank you very much for your implementation, and congratulations on the great work published on IEEE Access.

I see this PR is still set as a work in progress, as such, may I ask have you finished with the implementation? If yes, I would proceed to make it ready for review, and start the review process ASAP!

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.

2 participants