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.

Sign up for GitHub

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

Jump to bottom

Feature/performance metrics #39

Open
wants to merge 54 commits into
base: release
Choose a base branch
from
Open

Feature/performance metrics #39

wants to merge 54 commits into from

Conversation

nousayma-nx
Copy link
Collaborator

Counting object in video feature.

Packophys and others added 30 commits November 21, 2022 15:48
@Packophys
switch mapping classes as option
0f941b5
@Packophys
fix bug test csv update DB
002f995
@Packophys
lint data processing
654e0c9
@Packophys
Merge branch 'dev' into feature/debug_dev
dfabc91
@Packophys
add azure ml work
907bae4
@Packophys
lint azure ml
1a92ddf
@Packophys
factorize training code
e82f80c
@Packophys
fix bug generate yolo files
014661a
@Packophys
add crypted credentials for training and azure tests
a09ac9a
@Packophys
add base 64 encryption and update training functions with credential …
1a2db1a 
…options
@Packophys
add tests for azure ml
ee29aaa
@Packophys
update training test DB
be8b86e
@Packophys
lint training and azure
abc2cc8
@Packophys
lint training and azure tests
9b71ee7
@charlesollion
adding dependencies for azure
e633f66
@Packophys
Merge branch 'feature/debug_dev' of https://github.com/surfriderfound…
3d0d62e 
…ationeurope/surfnet into feature/debug_dev
@Packophys
bump up package version to 2.0.0a0
b050e6a
@Packophys
update tests using credentials with .env file
514bae8
@Packophys
Merge pull request #32 from surfriderfoundationeurope/feature/debug_dev
744e97d 
Merge Azure ML work
@Packophys
fix test errors
eaa297c
@charlesollion
adding dependency (but tests fails)
5d2e3c6
@Packophys
Merge branch 'feature/debug_dev' of https://github.com/surfriderfound…
08cbb0c 
…ationeurope/surfnet into feature/debug_dev
@Packophys
Merge pull request #33 from surfriderfoundationeurope/feature/debug_dev
1ad42fc 
Feature/debug dev
@charlesollion
adding python resize image
71a8a32
@Packophys
remove python-resize-image
d40fc00
@Packophys
resize images
6cbd544
@Packophys
bump prepatch version
f56e24a
@Packophys
update test with bboxes 10 or 12 classes
5ba4114
@Packophys
update test with bboxes 10 or 12 classes
b098786
@Packophys
add bboxes table option
b5b2ca0
Packophys and others added 17 commits December 9, 2022 16:56
@Packophys
Merge pull request #34 from surfriderfoundationeurope/feature/debug_dev
bd04459 
Feature/debug dev
@Packophys
updates for Azure ML
0b0488b
@Packophys
bump up version to 2.2.2.a0
b7b2561
@Packophys
Merge pull request #35 from surfriderfoundationeurope/feature/debug_dev
ddb35b1 
Feature/debug dev
@charlesollion
fixing yolo inference with hardcoded classes
9a4104c
@charlesollion
Merge pull request #36 from surfriderfoundationeurope/feature/update_…
8bfdb36 
…yolo

Feature/update yolo
@charlesollion
switch to yolo for serving
ce455fc
@charlesollion
update poetry packages
43b8f2a
@charlesollion
bump version
471dbd3
@charlesollion
fix torchvision version fixed
8e32c5b
@charlesollion
fix tests for including latest yolo instead of previous models
ffece51
@charlesollion
Merge pull request #37 from surfriderfoundationeurope/feature/update_…
27d94dd 
…serving_yolo

Feature/update serving yolo
Implement detected object count.
874ac83
Add compare argument.
0d78be6
Add count object scripts.
14eb557
Move count utils to tracking module.
f38b578
Add unit test for object count in video.
e92ef0a
charlesollion
charlesollion reviewed Jun 13, 2023
View reviewed changes
Copy link
Contributor

@charlesollion charlesollion left a comment

Choose a reason for hiding this comment

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

Hello! Thanks for the work.
Could you please add a line in the README.md to explain how to run the counting on videos?

@nousayma-nx
Copy link
Collaborator Author

@charlesollion done. I added some code to handle errors for missing file.

@charlesollion
Copy link
Contributor

@nousayma-nx tested, everything works as intended, and ready to merge, thanks!!
However, the process to update this repo is the following: we update the dev branch first with new features rather than release, could you reopen your merge request to the dev branch?
You will need to bump the poetry version as well.
Thank you very much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@charlesollion charlesollion charlesollion left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nousayma-nx @charlesollion @Packophys