Adding a simple dat.gui controller #44
Conversation
| @@ -2,11 +2,17 @@ | |||
| <head> | |||
| <title>{{ model_name }} (Mesa visualization)</title> | |||
| <script src="//ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> | |||
| <script src="/static/dat.gui.min.js"></script> | |||
There was a problem hiding this comment.
Can we get documentation for the object structure in this file? I'm a bit lost.
There was a problem hiding this comment.
Added; see if that makes sense of it.
There was a problem hiding this comment.
Documentation looks good to me, but I will defer to @kfuruya, b/c I am not strong in the javascripts.
|
@dmasad -- When I tried to run this, I had an error... When I visited... http://localhost:8888/ws Am I missing something? |
|
To load the visualization, you're supposed to only open http://localhost:8888 Try without the /ws and see if it works for you. |
|
@dmasad when I ran it I got the errors in Terminal. The gui loads when I visit http://localhost:8888, but when I try to do stuff it throws those errors, which is how I got there to begin with. |
|
Okay, I think I found it. I couldn't replicate it exactly, but I got a similar error (WARNING:tornado.access:403 GET /ws (127.0.0.1) 0.38ms) when I pointed my browser to http://localhost:8888 It works fine when I go to http://127.0.0.1:8888 I realized the html template is hard-coded to point to "ws://127.0.0.1:{{ port }}/ws" as opposed to localhost. I assumed these were completely interchangeable, but apparently not. A) Try going to http://127.0.0.1:8888 and seeing if it works for you. |
|
I created a ticket for that as a holder, so we can get this merged in. |
Adding a simple dat.gui controller
|
Oh, and what you mentioned worked, so I merged. :-) |
Resolves Issue #38 and adds a dat.gui interface to the TextVisualizationServer.