time-series-machine-learning / tsml-java

Java time series machine learning tools in a Weka compatible toolkit
GNU General Public License v3.0
158 stars 120 forks source link

ts reader now uses scanner #519

Closed ABostrom closed 3 years ago

ABostrom commented 3 years ago

This is the draft PR for the new and improved TS Reader that uses scanner.

Needs testing thoroughly. but works on both the univariate and multivariate datasets.

Next up is checking the weird special character cases and scientific types.

ABostrom commented 3 years ago

@MatthewMiddlehurst let me know. But i think we're good to go.

ABostrom commented 3 years ago

@MatthewMiddlehurst any idea why ci is failing. doesn't give me a lot to go on.

MatthewMiddlehurst commented 3 years ago

No, didnt even notice we had them again past this pr. Probably more likely its an issue with the test than the code.