Hi Ben, <div><br></div><div>I have a few questions/comments on the branch:</div><div><br></div><div>- It seems there are some streaming changes as well, such as streamwork.py and archive.js has some websocket stuff added? Should this stuff be added on this branch?</div>
<div>- What is the license of utils/xml2json.py and web/apps/oldworkflow/hint-textbox.js? Need to be careful about adding non BSD stuff.</div><div>- Do we need the old workflow code?</div><div><br></div><div>Looks good otherwise.</div>
<div><br></div><div>Thanks,</div><div><br></div><div>Chris</div><div><br><br><div class="gmail_quote">On Wed, Jul 10, 2013 at 5:36 PM, Ben Burnett <span dir="ltr"><<a href="mailto:benjam.arlyn@gmail.com" target="_blank">benjam.arlyn@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Branch 54-workflow-support-fix is ready for review. After creating a layer, you should see a workflow button. That brings up the workflow canvas, and with a default pipeline (for now). You can drag modules, and drag connections. The drag and drop to add new modules seems to clash with Chris's drag'n'drop implementation and so it doesn't work on the archive page, and needs to be fixed. It does work on </span><a href="http://www.localhost:8080/apps/archive/workflow.html" style="font-family:arial,sans-serif;font-size:13px" target="_blank">http://www.localhost:8080/apps/archive/workflow.html</a><span style="font-family:arial,sans-serif;font-size:13px"> however.</span><span class="HOEnZb"><font color="#888888"><div style="font-family:arial,sans-serif;font-size:13px">


<br></div><div style="font-family:arial,sans-serif;font-size:13px">-Ben</div></font></span></div>
<br>_______________________________________________<br>
Opengeoscience-developers mailing list<br>
<a href="mailto:Opengeoscience-developers@public.kitware.com">Opengeoscience-developers@public.kitware.com</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers</a><br>
<br></blockquote></div><br></div>