-
Notifications
You must be signed in to change notification settings - Fork 35
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
_addHeaderLines is not a function error when first accessing server #44
Comments
I think that's the stdout/stderr coming from the browser process. You can probably ignore it. |
@soumith Definitely not from the browser process -- if i start with lgfx.start, it gets thrown in the same console window as soon as the browser tries to access the port. The browser displays a blank page with the "light" tab on the right. Trying to open html directly from static/data, I get an empty shell with svg and png save buttons, but no actual charts. Thanks for the pointer to display, i'll take a look! |
@elbamos this is somewhat the reason why we are phasing out gfx.js, it has too much machinery for what is supposed to be super simple. display is lean, easy to install, better to use. |
@soumith Yeah took about a minute to switch to. That said, it wouldn't kill the author to include some useful chart examples. Thanks! |
@elbamos i'll work on it. all itorch Plots can be used in display with small changes (which I will send as a PR soon), and I think the bokeh plots from itorch's Plot.lua are nicer anyways (and are documented with usage). |
@soumith yeah they are nicer, but its rather difficult to get them output from a console session, no? itorch times out, at least for me, on neural net training operations that take longer than 15 minutes, which makes its display functions beside the point for my use case. |
@elbamos the plotting system is independent of itorch altogether. iTorch does not even need to be installed or running. I'll finish that work in the next couple of days and make it compatible with display. |
I think this is pretty self-explanatory:
Any advice? Thanks!
The text was updated successfully, but these errors were encountered: