aeon icon indicating copy to clipboard operation
aeon copied to clipboard

[ENH] Included RSAST as a alternative to SAST (2.0)

Open nirojasva opened this issue 10 months ago • 7 comments

What does this implement/fix? Explain your changes.

I included a transformer RSAST and a classifier RSASTClassifier which are tools to use in time series classification. The method RSAST is based on shapelets in order to transform dataset of Time Series.

Does your contribution introduce a new dependency? If yes, which one?

Yes, I used f_oneway, DegenerateDataWarning, ConstantInputWarning from the "scipy.stats" toolkit. Also, acf and pacf from the "statsmodels.tsa.stattools" implementation.

PR checklist

For all contributions
  • [ ] I've added myself to the list of contributors. Alternatively, you can use the @all-contributors bot to do this for you.
  • [x] The PR title starts with either [ENH], [MNT], [DOC], [BUG], [REF], [DEP] or [GOV] indicating whether the PR topic is related to enhancement, maintenance, documentation, bugs, refactoring, deprecation or governance.
For new estimators and functions
  • [ ] I've added the estimator to the online API documentation.
  • [ ] (OPTIONAL) I've added myself as a __maintainer__ at the top of relevant files and want to be contacted regarding its maintenance. Unmaintained files may be removed. This is for the full file, and you should not add yourself if you are just making minor changes or do not want to help maintain its contents.
For developers with write access
  • [ ] (OPTIONAL) I've updated aeon's CODEOWNERS to receive notifications about future changes to these files.

nirojasva avatar Apr 08 '24 06:04 nirojasva

Thank you for contributing to aeon

I have added the following labels to this PR based on the title: [ $\color{#FEF1BE}{\textsf{enhancement}}$ ]. I have added the following labels to this PR based on the changes made: [ $\color{#BCAE15}{\textsf{classification}}$, $\color{#41A8F6}{\textsf{transformations}}$ ]. Feel free to change these if they do not properly represent the PR.

The Checks tab will show the status of our automated tests. You can click on individual test runs in the tab or "Details" in the panel below to see more information if there is a failure.

If our pre-commit code quality check fails, any trivial fixes will automatically be pushed to your PR unless it is a draft.

Don't hesitate to ask questions on the aeon Slack channel if you have any.

aeon-actions-bot[bot] avatar Apr 08 '24 06:04 aeon-actions-bot[bot]

Hello, I have another question, I'm using two new libraries "statsmodels" and "scipy". How can I included in aeon these libraries since it seems it is generating conflict? Thanks,

nirojasva avatar Apr 08 '24 08:04 nirojasva

scipy is fine, statsmodels will have to be added as a tag i.e. https://github.com/aeon-toolkit/aeon/blob/main/aeon/classification/shapelet_based/_ls.py#L95

Check out the contributing guide for some tips. The developer install would be helpful for pre-commit.

MatthewMiddlehurst avatar Apr 08 '24 13:04 MatthewMiddlehurst

The tag also has to be included for the transformer, and the imports must be inside the function rather than at the top of the file.

MatthewMiddlehurst avatar Apr 11 '24 10:04 MatthewMiddlehurst

I think the remaining failure is because of the examples. No good way to get around this unfortunately, I would recommend using # doctest: +SKIP. The QUANT example does this in interval_based.

MatthewMiddlehurst avatar Apr 15 '24 22:04 MatthewMiddlehurst

Ok @MatthewMiddlehurst thanks, it seems the validation worked for the library issues. However there are still errors in "pre-commit" and there is no a detail about why they are caused.

nirojasva avatar Apr 16 '24 14:04 nirojasva

These are our code quality checks, for the most part it is probably white space. I would recommend following the pre-commit section of the dev install guide or I can run it through at some point.

MatthewMiddlehurst avatar Apr 16 '24 14:04 MatthewMiddlehurst

We are releasing next week, this should be available then.

MatthewMiddlehurst avatar May 18 '24 17:05 MatthewMiddlehurst