-
Notifications
You must be signed in to change notification settings - Fork 11
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
FEATURE FREEZE? #49
Comments
Sure. If it is getting a lot of interest, then I suppose we should do releases. We can target issues to planned releases. Speaking of deployment - if modularization is a big priority - that is, make it so that any number of nupic.visualizations can be dropped on a page with a minimum of configuration - then we should make that a priority issue. It isn't really designed that way right now. Right now, it is designed to be an entire stand-alone application, as opposed to being a part of some other app. |
Yes, and I much prefer it to be a drop-in charting tool. I think it can be Matt Taylor On Fri, Dec 11, 2015 at 8:41 AM, Jeff Fohl [email protected] wrote:
|
Since @rhyolight needs streaming as well, perhaps we should make that the highest priority? |
My concern when creating this issue was simply that too much up-front engineering would be done before we had the charting system actually being used somewhere. I think I had the wrong impression that the end goal was a drop-in JavaScript library that could be used for live-streaming temporal data. If you guys are willing to do the work now to ensure I can use this charting system in my own web applications, I would be happy. Otherwise, I'll continue to use Dygraphs directly. |
I want to make it something you can use Matt. We just need to define the interface. |
@rhyolight what's the missing parts for you? The #40 and #17 ? |
I think we also need to know what to leave out. What would be the responsibility of the charting module, and what would be the responsibility of the parent app? What UI elements do we include, etc. |
All I need right now is #40. Streaming can be done later. |
Can we chat on Gitter? I need more detail. |
I would add the file selection to the UI, actually the existing app minus the header... |
@breznak @jefffohl I think this project is very cool, and I'm glad you are working on it. I'm ready to use it if it in more than one project, including River View.
But, at this point I think you should focus on getting this codebase ready for production and stop all work on new features until it is actually deployed and being used in a few places. We are going to find bugs and new issues as we attempt to deploy this charting tech, and it s better to find out any big issues now before any further architecture changes are made that might somehow affect deployment strategies. (I am talking about issues like #42, #43 & #48.)
What do you guys think?
The text was updated successfully, but these errors were encountered: