add hotgym data for NuPIC example

htm-community / nupic.visualizations

Web application for interactive graphs, anomaly highlighting and online monitoring.

MIT License

17 stars 11 forks source link

add hotgym data for NuPIC example #69

Closed breznak closed 8 years ago

breznak commented 8 years ago

[x] @rhyolight here is the full example file from hotgym
[x] the windowing seems to cause problems, I'd suggest disabling for now until we need it in online monitoring PR (eg anomalies don't highlight with it, probably should add UI control so users can choose, as the hotgym example gets truncated too, ...)
[x] fixed the "fixed 2 rows skipped in header" for non-OPF files
[x] fix/workaround problem with OPF files having a reset field (causes nonmonotonic timestamps)

Fixes #10 Fixes #70

breznak commented 8 years ago

@jefffohl this is ready for your review. Functionality is complete. Resolves a couple of issues. But if you have already a PR in progress, this can wait, or I'll help you resolving the merge conflicts..

breznak commented 8 years ago

@rhyolight this branch is ready to test, if you wanna give it a shot.

rhyolight commented 8 years ago

@breznak where exactly did you get that data file? The predictions look bad.

rhyolight commented 8 years ago

Chart looks good to me :+1:

breznak commented 8 years ago

where exactly did you get that data file? The predictions look bad.

Thanks. [discussed on chat, the file is from "official nupic" hotgym example.]

jefffohl commented 8 years ago

Thanks @breznak - can you explain how windowing is causing problems?

breznak commented 8 years ago

@jefffohl

can you explain how windowing is causing problems?

it is not, actually. I've tried to cross that out in the PR description, the problem was caused by the reset fields, which is fixed as a part of the PR.

jefffohl commented 8 years ago

:+1: Looks good @breznak. Thanks!

One thing that really needs updates is the notifications system. We have a lot of user feedback to show now that tell the user how the app is interpreting the data. I am starting to think we need a scrolling console or something similar in the UI. Having all the alerts pile up the way they are now is not working well. We should probably have that addressed before release v1.0