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

Add Perspective Transformation Refactor a lot (but not all) JS style to Python style code deprecate weird features #3

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

WasinUddy
Copy link
Contributor

Change Logs

Added

  1. Perspective Transformation: Implemented perspective transformation to provide a more realistic look to the frame, enhancing visual appeal and alignment.
  2. New Assets Structure: Restructure the asset folder for maintainability and order
  3. Frame Data Pattern: Introduced a frame data pattern to support the new perspective transform feature, ensuring accurate and visually coherent transformations.

Changed

  1. OpenCV Approach: Optimized and simplified the OpenCV approach to be more straightforward, enhancing performance and readability.
  2. Naming Refactor: Conducted a partial refactor of variable and function naming JS -> PY style

Deprecated

  1. OpenCV Functions: Deprecated several unnecessary OpenCV functions to streamline the codebase and enhance efficiency.
  2. PosOptions Usage: Deprecated the use of PosOptions due to weird stuff.
  3. CropOptions Usage: Deprecated the use of CropOptions due to weird stuff

Notes

  • The updated code is expected to function as intended but has not been thoroughly tested with Flask. Some adjustments may be required to accommodate deprecated functions and ensure compatibility with the API.

@WasinUddy
Copy link
Contributor Author

To be honest everything is weird features LMAO

@leomotors leomotors merged commit 07c2606 into leomotors:main Oct 5, 2023
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.

2 participants