Skip to content
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

node main predictor.py not rendering anything? #1

Open
davidleejy opened this issue Apr 5, 2023 · 8 comments
Open

node main predictor.py not rendering anything? #1

davidleejy opened this issue Apr 5, 2023 · 8 comments

Comments

@davidleejy
Copy link

Hi there 😀. I tried running node main predictor.py which I thought would render something similar to the figure in the readme. The server spins up fine but nothing appears to be rendered despite having tried both FF & Chrome.

Thanks for creating this project and wishing you a pleasant week ahead.

@tom-draper
Copy link
Owner

Hi, thanks for your interest! The problem should be corrected now. It was a minor regular expression error causing the issue. Let me know if you have any other issues/queries/suggestions. All the best!

@davidleejy
Copy link
Author

@tom-draper Thanks for looking into this :) Which commit should users check out if they'd like to try this out? Latest commit (6 Apr 2023) on main branch (cfac64e) doesn't appear to be rendering anything though :(

@tom-draper
Copy link
Owner

tom-draper commented Apr 6, 2023

That commit should be the correct one. I've just tested it and it works on my end. Are you getting any error messages? It may take a while to render the graph depending on the size of the Python file. Even while it's loading you should be able to see the class names in the bar across the top.

Also, maybe try npm install to get the same version of express in the package.json

@tom-draper tom-draper reopened this Apr 6, 2023
@davidleejy
Copy link
Author

I may be doing something wrong hmm..

Do i need to uninstall express globally and then run npm install after cd-ing the terminal into this repo?

@tom-draper
Copy link
Owner

tom-draper commented Apr 6, 2023

Is the server running fine? Does the program print out http://localhost:8080? If so, it's probably not a problem with express.
The other consideration is with the version of node you're using? I tested it with node v18.3.0 and it worked correctly. It could be a compatibility issue if you're using an older version of node.

@davidleejy
Copy link
Author

davidleejy commented Apr 6, 2023

Yep, the program does print out the url.

Re-ran it with node v18.15.0. Nothing is being rendered on both FF & Chrome after about 30-45 seconds.

express @ 4.18.2

@tom-draper
Copy link
Owner

Hm okay. Sorry about this, I'll look into this issue and get back to you

@tom-draper
Copy link
Owner

If you open the developer tools in Chrome and look at the console do you see any errors?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants