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

Eliminate the use of the cv namespace #124

Open
advait-0 opened this issue Feb 22, 2024 · 2 comments
Open

Eliminate the use of the cv namespace #124

advait-0 opened this issue Feb 22, 2024 · 2 comments

Comments

@advait-0
Copy link
Contributor

Users especially those new to C++ often face confusion and difficulties in understanding the proper usage of OpenCV classes due to the common practice of using the cv namespace. To improve clarity and encourage best practices, I think it is suitable to eliminate the use of the cv namespace in our examples.

Possible solution to this problem could be:

  • Encouraging Explicit Namespace Usage: Update documentation and examples to explicitly use the cv:: prefix for OpenCV classes (e.g., cv::Mat, cv::VideoCapture) instead of relying on the cv namespace. This will also allow users to have a better understanding of the working of C++ as using namespaces abstracts some stuff.
@AryanNanda17
Copy link
Contributor

I will start working on this issue and will do the PR by midnight.

@AryanNanda17
Copy link
Contributor

@advait-0, there are already some existing PRs that should be merged before I do the namespace change else the test cases would fail because of merge conflicts.

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

No branches or pull requests

2 participants