-
Notifications
You must be signed in to change notification settings - Fork 165
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
Porting WEASEL 2.0 to pyts #155
Comments
Thanks for taking the effort! A couple of comments:
See: Alphabet Size
See: Window sizes
See: Ensemble
See: Ensemble
See: SFA with Variance
See: [Binning Strategy] Hope, this helps. IMO: The most critical parts should be alphabet_size, window-size, differences, and variance in SFA. |
Hi, Sorry for the delayed response, I saw the notification and forgot about it... First, thanks @aglenis for the effort and thanks @patrickzib for the feedback! I will need to look at the paper and the source code to provide more detailed, but I will answer some points first.
In general, I like having more hyper-parameters (even if the values are fixed in the original paper) because it might be useful to change these values for other datasets (many people have their own datasets and don't work on the UCR/UEA archive), but I try to keep the default values as close as possible to the ones in the original publication. I'm very interested in adding WEASEL 2.0 to |
FYI, WEASEL 2 is in aeon and we have run it to test results |
Description
I am trying to port WEASEL 2.0 (https://github.com/patrickzib/dictionary) to pyts however I failed to match WEASEL 2.0 accuracy results.
I don't know if its a bug in my code, or something like bad hyperparameters on my part.
Steps/Code to Reproduce
My code for WEASEL 2.0:
Versions
NumPy 1.21.6
SciPy 1.5.4
Scikit-Learn 1.0.2
Numba 0.55.2
Pyts 0.12.0
The text was updated successfully, but these errors were encountered: