-
Notifications
You must be signed in to change notification settings - Fork 38
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
Refactor of the code #6
Open
nos86
wants to merge
33
commits into
calebolson123:master
Choose a base branch
from
nos86:dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
7e15783
style: :recycle: Improved .gitignore to accept new files and new stru…
nos86 f3972d5
refactor: :recycle: moved SleepyBaby as package
nos86 50b3000
refactor: :construction: Refactoring code in smaller and consistent c…
nos86 31b0777
perf: :zap: helper has been refactored and optimized
nos86 ccece2a
chore: :bug: Ignore supporting file of jupyter
nos86 fd56431
refactor: :art: Refactoring
nos86 4771913
refactor: :art: WIP: first draft of Decision logic
nos86 f5749d2
refactor: :art: Created Frame class
5623d52
refactor: :technologist: Threshold for recognition has been moved as …
nos86 f113419
refactor: :art: media_analysis moved directly inside __init__
nos86 2d1ec03
refactor: :art: Add possibility to show additional parameters on screen
nos86 5f6a274
refactor: :art: implementation of decision logic
nos86 46ca184
added example for main.py
nos86 1e273ff
feat(cli): :sparkles: Add command line configuration for script execu…
a9d0ac1
feat: Refactor logging setup and options
734d533
refactor: Refactor video processing for multithreading.
40257d6
refactor: Refactor progress bar function signature and style.
5c82154
refactor: Refactor progress bar functionality in frame.py
7fa817b
Merge branch 'feature/text_n_bar' into dev
0085a27
fix: :bug: Add `argparse` import and remove debug `print` statement i…
727aa3e
refactor: Move attributes in function signature
c9dd314
refactor: Refactor main.py into class and add CLI support via Fire.
4a4329e
Merge branch 'feature/arguments' into dev
5438aaa
refactor: Refactor source parameter in app.live() method
4964d5c
refactor: Improve constructor functionality and readability in main.py.
ab43dc4
feat: Add face detection and tracking confidence to SleepyBaby initia…
0673da9
feat: implement new `photo` method for image processing
fab7cd6
feat: Add working area configuration
nos86 59a1a61
feat(frame): ✨ add gamma correction to the augmented frame generation
nos86 711bb8c
refactor: 🚀 code to simplify get_height calculation
nos86 a040c9e
style: Remove unused dependencies
nos86 cc1949d
refactor: 🔮 Refactored app.photo() and app._process_streaming() to ma…
nos86 b666027
feat: 🚀 Add debug variable to show baby status
nos86 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,14 +1,12 @@ | ||
| * | ||
| !.gitignore | ||
| !LICENSE | ||
| !README.md | ||
| !main.py | ||
| !helpers.py | ||
| !webapp | ||
| !webapp/** | ||
| __pycache__ | ||
| node_modules | ||
| build | ||
| !sleep_logs.csv | ||
| !*.ipynb | ||
| !requirements.txt | ||
| !.env_sample | ||
| *.log | ||
| .env | ||
| .ipynb_checkpoints | ||
| baby | ||
| *.jpg | ||
| *.jpeg | ||
| *.png | ||
| *.avi | ||
| *.mp4 |
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, when we eventually get around to updating the README I want to mention that users should play around with this. The smaller the area, the less demanding it is on client cpu/gpu. Can be computationally expensive & pointless to pass the entire image through the models
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As soon as we freeze the modification, I'll comment the code and update all related file (including README) to be aligned with new structure