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

Update development notes #75

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Ghesselink
Copy link
Contributor

Relatively simple method to run React frontend and Flask backend on a local machine without using docker.

Update and install submodules

~~~
submodule init && submodule update
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

git submodule update --init --recursive

Set environment to 'development'

~~~
export environment=development
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

export is for linux/wsl, on windows use set


Setup local environment with the following packages
~~~
pip install lark-parser==0.12.0 numpy ifcopenshell flask flask-cors numpy gunicorn rq redis SQLAlchemy psycopg2 psycopg2cffi sqlalchemy-utils Flask-BasicAuth flasgger requests XlsxWriter requests_oauthlib Authlib requests argparse orjson --upgrade && conda install flake8
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

psycopg2 psycopg2cffi are not needed, why not use pip to install flake8 as well?


Depending on the OS, change ui/package.json. For instance to 'bash run_debug.sh' in case of a linux system.
~~~
"api": "cd ../application && run_no_debug.bat"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Consider adding api-windows and api-linux or sth instead of just api to package.json so that no local change need to be made.

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

Successfully merging this pull request may close these issues.

2 participants