pyronear / pyro-platform

Detection & monitoring platform of wildfires
https://platform.pyronear.org/
Apache License 2.0
10 stars 11 forks source link

refactor: Refactored project & updated dependencies #81

Closed frgfm closed 2 years ago

frgfm commented 2 years ago

This PR introduces the following modifications:

The new project organization is as follows:

app
├── assets
│   ├── css
│   │   └── styles.css
│   ├── data
│   │   └── group_correspondences.json
│   └── images
│       ├── background.png
│       ├── favicon.ico
│       ├── pyro_alert_icon.png
│       ├── pyro_alert_off.png
│       ├── pyro_alert_on.png
│       ├── pyro_fire_logo.png
│       ├── pyro_oldfire_icon.png
│       ├── pyro_site_icon.png
│       └── up_arrow_png.png
├── components
│   └── navbar.py
├── config.py
├── main.py
├── pages
│   ├── device_status.py
│   ├── homepage.py
│   ├── risks.py
│   └── screen.py
├── services
│   ├── api.py
│   └── __init__.py
└── utils
    ├── alerts.py
    └── _utils.py

The folders & files are:

One thing that will be left to do is reduce the size of main (moving callbacks or creating subfunctions of utils) because it's way too massive.

Closes #35 Closes #80

Any feedback is welcome!

frgfm commented 2 years ago

Deployed on Heroku, everything seems to be running smoothly :+1:

frgfm commented 2 years ago

ping @Akilditu

pechouc commented 2 years ago

I have checked the changes since the last comments and everything looks good to me. I have also tried to run the app locally after a few changes to prepare another PR and it seemed to work smoothly. I haven't gone into the details and we might want to wait for @Akilditu's green light but, that being said, I am happy to approve the changes if you wish to move forward quickly.