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

Comments about the release http://weppy.herokuapp.com/ #1

Open
santoshphilip opened this issue Nov 26, 2016 · 5 comments
Open

Comments about the release http://weppy.herokuapp.com/ #1

santoshphilip opened this issue Nov 26, 2016 · 5 comments

Comments

@santoshphilip
Copy link
Owner

leave any comments about the release at
http://weppy.herokuapp.com/

@katsuya0719
Copy link

Hi Santosh!
I tried to use your heroku app and have question.
Do I still need to install graphviz in my local pc?

@santoshphilip
Copy link
Owner Author

Katsuya,

I am pretty sure you need it. I have graphviz installed locally, so I don't notice

It should be easy to recode weppy so that it does not need graphviz. I am thinking of putting in a couple of try ... except statements to skip over the garphviz requirement.

Would you like me to take a stab at it ?
I am intending to do some work on weppy over the next week - trying to develop some HVAC modules for eppy

@santoshphilip
Copy link
Owner Author

santoshphilip commented Apr 16, 2017

I looked through the code to see I could remove the graphviz requirement. It is possible to remove it, but will take some work :-(

@katsuya0719
Copy link

Is it ok for you to implement that? I think many ppl who have not used weppy will face this problem and could be the barrier for them to try to use.

@santoshphilip
Copy link
Owner Author

I think I can do this, since I plan to work with weppy over the nextt few weeks.

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