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

Merge to main #172

Merged
merged 20 commits into from
Aug 16, 2023
Merged

Merge to main #172

merged 20 commits into from
Aug 16, 2023

Conversation

MateoLostanlen
Copy link
Member

Everything works smoothly on saint peray, we can merge on main

MateoLostanlen and others added 19 commits April 7, 2023 14:54
* Update of the main branch before the new deployment in Ardeche (#145)

* ci on develop (#134)

* update develop tour (#135)

* fix args (#136)

* fix: Fixed script arg name in src/run.py (#138)

* clean backup by size (#141)

* clean backup by size

* black

* missing deps

* drop function

* drop function 2

* Day only (#142)

* sunset_sunrise script

* update output file path

* check if day time

* put back real api

* style

* use datetime timedelta

* Make all params availables from run (#144)

* make all params available in run

* make params availables

* put back api

* style

* put back alert_relaxation to 2

* Yolov5 (#143)

* switch to yolov5

* missing import

* style

* fix tests

* unused import

* update readme

* letterbox transform

* do not resize with pillow before pred

* style

* downgrad opencv

* wrong img name

* long line

* missing deps

* model path

* header

* lib for opencv

* create model folder

* update init

* remove hub deps

* update threshold

* feat: Added promtail service

* feat: Added promtail config file

* feat: Renamed variable

* feat: Renamed vars

* docs: Updated setup explanation

* feat: Added restart option

---------

Co-authored-by: Mateo <[email protected]>
* set margin to 0

* drop utc

* add ir strategy

* update docstring

* fix style
* add bbox

* fix inference

* style

* fix vision

* style vision

* fix engine

* keep all preds

* speed up

* pass dummy loc

* switch to v8

* fix tests

* test

* drop dummy test

* new api version

* unsed import

* install git

* code quality

* use bbox branch

* use apt get

* alert relaxation 3
* new alert system

* fix unitests
* add occlusion mask

* trest mask

* missing args

* use cam_key

* update test

* add a test

* clip values
* limit bbox size

* style
@MateoLostanlen MateoLostanlen added type: bug Something isn't working type: enhancement New feature or request type: new feature labels Aug 16, 2023
@MateoLostanlen MateoLostanlen requested a review from a team August 16, 2023 16:06
@MateoLostanlen MateoLostanlen self-assigned this Aug 16, 2023
@ghost
Copy link

ghost commented Aug 16, 2023

Nice of you to open a PR 🙏
When you're ready and want to get it reviewed, post a comment in this Pull Request with this message: /quack review

@codecov
Copy link

codecov bot commented Aug 16, 2023

Codecov Report

Merging #172 (61a3c37) into main (fe82a48) will decrease coverage by 1.90%.
Report is 11 commits behind head on main.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #172      +/-   ##
==========================================
- Coverage   89.87%   87.97%   -1.90%     
==========================================
  Files           5        6       +1     
  Lines         237      366     +129     
==========================================
+ Hits          213      322     +109     
- Misses         24       44      +20     
Flag Coverage Δ
unittests 87.97% <87.50%> (-1.90%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
pyroengine/sensors.py 88.88% <ø> (ø)
pyroengine/core.py 53.84% <27.27%> (-9.49%) ⬇️
pyroengine/engine.py 89.62% <84.52%> (-3.55%) ⬇️
pyroengine/utils.py 95.55% <95.55%> (ø)
pyroengine/__init__.py 100.00% <100.00%> (ø)
pyroengine/vision.py 100.00% <100.00%> (ø)

@MateoLostanlen MateoLostanlen merged commit 0782fee into main Aug 16, 2023
20 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working type: enhancement New feature or request type: new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants