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

Improvements #5

Merged
merged 13 commits into from
Apr 11, 2024
Merged

Improvements #5

merged 13 commits into from
Apr 11, 2024

Conversation

LightDestory
Copy link
Member

@LightDestory LightDestory commented Apr 6, 2024

This PR will address the following:

  • Restructuring to use Pythonic-paths
  • Add missing dependency on Requirements
  • Add target version for depenencies on Requirements
  • Remove decorators in favour of registering handlers
  • Fix inefficient camera access, saving a lot of time
  • Add Peek command
  • Registering commands for UI friendly
  • Lint/Format Pipeline

fix: Delay on stop due to join
feat: status command
feat: bot command declarations
fix: Camera access improvements
fix: revert from multiprocessing to thread
@LightDestory LightDestory requested review from DefEnge and Helias April 6, 2024 14:33
@LightDestory
Copy link
Member Author

I don't have the required oauth credentiuals to actually test the image upload. Everything else should be working.

src/main.py Outdated Show resolved Hide resolved
src/main.py Outdated Show resolved Hide resolved
src/main.py Show resolved Hide resolved
src/main.py Outdated Show resolved Hide resolved
@LightDestory
Copy link
Member Author

@Helias 44f96aa should close #6

@LightDestory LightDestory requested a review from Helias April 11, 2024 09:44
@DefEnge DefEnge merged commit 1b303e0 into main Apr 11, 2024
2 checks passed
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.

3 participants