-
Notifications
You must be signed in to change notification settings - Fork 487
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
Figure out what yarn start
in plexus is supposed to do
#2093
Comments
Started going through this issue about an hour ago and my understanding so far is yarn start performs the following functions in sequence:
|
@Wise-Wizard I understand what it does mechanically, but what was it intended to do? The resulting web site it starts is blank. |
Yes, according to the ReadMe I believe that if configured properly it should display a Demo Directed Graph on that URL (plexus/demo) and the user can experiment around with the Vertex and Edges to create their own Graph. |
@yurishkuro , like I mentioned above the reason for it showing Cannot Get / is a missing index.html file, i made some changes in webpack and added a test index.html file and it is rendering the webpage. |
the demo has a bunch of graph examples in it, can you get it to work? |
Sure, will look into this. |
My advice would be to look in git history when it was introduced, maybe checkout those old versions and try to run to see what it was doing. |
Got it, will look into that |
Hi @yurishkuro , looked through the previous commits. There was a commit 5 years ago, where yarn start's original purpose was to create a React App using "start": "nwb serve-react-demo" command. Main purpose according to readme was viz.js was used, in a WebWorker, to generate GraphViz as plain-text output which is then parsed and provided to a React component which does the rendering. |
@yurishkuro Is this related to this error? 13:33:08 [vite] Internal server error: Failed to resolve import "./layout.worker.bundled" from "..\plexus\src\LayoutManager\Coordinator.tsx". Does the file exist?
Plugin: vite:import-analysis
File: C:/dev/jaeger-ui/packages/plexus/src/LayoutManager/Coordinator.tsx:20:25
1 | import * as convCoord from "./dot/conv-coord";
2 | import convInputs from "./convInputs";
3 | import LayoutWorker from "./layout.worker.bundled";
| ^
4 | import {
5 | ECoordinatorPhase,
at formatError (file:///C:/dev/jaeger-ui/node_modules/vite/dist/node/chunks/dep-R0I0XnyH.js:63410:46) This happens when, I run |
In
packages/plexus/package.json
there is astart
command. It does not work currently unless the changes below are applied (committed in #2094 for easier reuse). However, even with these changes a web server is started that doesn't seem to do anything, a request to http://localhost:5000/ only showsCannot GET /
.What is supposed to happen with this command? Is it perhaps just a remnant of auto-generated webpack before Plexus was a separate module? This ticket is to figure this out and post findings (e.g. look through the history of commits to see the original intent).
The text was updated successfully, but these errors were encountered: