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

Updates to enable Eyesy web editor and improve install experience #2

Merged
merged 17 commits into from Nov 16, 2020

Conversation

@timcosgrove
Copy link
Contributor

@timcosgrove timcosgrove commented Nov 16, 2020

This aims to enable the webapp. It also aims to improve the install experience, so that new users should be able to run a single command to get all dependencies installed.

To test this (preferably with a clean Fates/Sidekick install):

cd ~/sidekick/patches
git clone git@github.com:timcosgrove/Eyesy_for_fates.git Eyesy
cd Eyesy
git checkout feature/enable_webapp #This is necessary during testing
./deploy.sh

The deploy script takes several minutes, as it is now installing all the Debian packages as well as Python and Node packages. Subsequent runs are much faster.

If everything went well, Eyesy should run, and the web editor should be available at http://norns.local:8080/

Changes:

  • deploy.sh is expanded to install all dependencies, including
    ** Debian python dependencies & nodejs
    ** Python packages psutil and cherrypy (necessary for the webapp)
    ** Node packages (necessary for the webapp)
  • Python, PD, the Webapp, and the NodeJS websocket are all set up as systemd services, put into place by deploy.sh, started by run.sh and stopped by stop.sh
  • The webapp has adjustments to deal with the different path structure of Eyesy on Fates; additionally, all the settings code was ripped out, as on C&G Eyesy it's intended for managing Wifi access and other hardware-specific settings.
  • web/run.sh removed, as we've moved it to services
  • All *.pyc files removed from the repo (they are generated and should not be in a repository).
@Lcchy
Copy link
Owner

@Lcchy Lcchy commented Nov 16, 2020

Thanks a lot for your work, this is amazing!

@Lcchy Lcchy merged commit 7be60e0 into Lcchy:master Nov 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants