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

Path/str into cv2.imread bug #217

Closed
wants to merge 1 commit into from
Closed

Path/str into cv2.imread bug #217

wants to merge 1 commit into from

Conversation

ainureg
Copy link
Collaborator

@ainureg ainureg commented Jan 25, 2024

Pull Request Template for InnoFW

Title: в файле innofw/core/datasets/image_infer.py:39 попытка использовать path-объект в cv2.imread

Description

Please provide a detailed description of your changes, including:

  1. What issue(s) are you addressing?
  2. How have you tested your changes?
  3. If applicable, any new dependencies added and why they are necessary.

Pre-commit Hook Activation and Linter Checks

Please ensure you have activated the pre-commit hook to enable automatic linting checks before submitting your PR. This helps maintain code quality and consistency throughout the project.

Documentation

Please ensure you have documented the functionality of your changes, especially if it introduces new features or affects existing ones. Proper documentation ensures that the project remains maintainable and understandable for other contributors.

Changes

  • List major changes made in bullet points
  • Include any minor changes or refactoring

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • [ x] Bug Fix
  • Optimization
  • Documentation Update

Checklist

  • I have tested my changes locally and all tests passed
  • I have updated the documentation (if necessary)
  • I have added new dependencies (if necessary)
  • [ x] My changes do not introduce any breaking changes or conflicts with existing code
  • I have activated the pre-commit hook and ensured that linting checks pass

Added/updated tests?

  • Yes
  • [ x] No, and this is why: please replace this line with details on why tests
    have not been included
  • I need help with writing tests

Additional Notes

Provide any additional information or context that may be helpful for the reviewer.


Thank you for your contribution to InnoFW! Your efforts help make this project better for everyone.

@ainureg ainureg requested a review from BarzaH January 25, 2024 11:31
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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