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

Time series metrics fails with variable length timeseries #42

Closed
fealho opened this issue Jan 27, 2021 · 0 comments · Fixed by #43
Closed

Time series metrics fails with variable length timeseries #42

fealho opened this issue Jan 27, 2021 · 0 comments · Fixed by #43
Assignees
Labels
bug Something isn't working
Milestone

Comments

@fealho
Copy link
Member

fealho commented Jan 27, 2021

The LSTM classifier doesn't support variable length time series unless they are sorted from longest to shortest. Since we don't need ONNX compatibility, we can remove this restriction:

RuntimeError: lengths array must be sorted in decreasing order when enforce_sorted is True. You can pass enforce_sorted=False to pack_padded_sequence and/or pack_sequence to sidestep this requirement if you do not need ONNX exportability.

@fealho fealho added the bug Something isn't working label Jan 27, 2021
@csala csala added this to the 0.1.2 milestone Jan 27, 2021
@csala csala closed this as completed in #43 Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants