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

Move to OpenCV for file I/O #154

Open
Correct-Syntax opened this issue Feb 14, 2022 · 1 comment
Open

Move to OpenCV for file I/O #154

Correct-Syntax opened this issue Feb 14, 2022 · 1 comment
Assignees
Labels
core Related to the core renderer and/or backend feature New feature or request high priority This should be worked on first above other issues with lower a priority in progress This issue/PR is in progress. Your input is always welcome python This issue includes writing Python code

Comments

@Correct-Syntax
Copy link
Member

Currently, we do not have pre-built python wheels for OpenImageIO. OpenImageIO is only used for image I/O so it is not necessarily a hard dependency. OpenCV doesn't provide all of the same support for the formats OIIO does, but that is the tradeoff. :/

This issue should implement cv2 (which is already a dependency) as a fallback for image Input and output when OIIO is not installed. It should also alert the user that the image import and export may be limited because of using cv2 rather than OIIO.

@Correct-Syntax Correct-Syntax added feature New feature or request good first issue Good for new contributors core Related to the core renderer and/or backend python This issue includes writing Python code high priority This should be worked on first above other issues with lower a priority ready Non-blocked issue ready to be implemented labels Feb 14, 2022
@Correct-Syntax Correct-Syntax self-assigned this Mar 7, 2022
@Correct-Syntax Correct-Syntax moved this from To do to In progress in v0.6.0 alpha MVP Mar 7, 2022
@Correct-Syntax
Copy link
Member Author

This is actually mostly done in the master branch. I have to cleanup a few areas though.

@Correct-Syntax Correct-Syntax changed the title Use OpenCV as a fallback for OpenImageIO, file I/O Move to OpenCV for file I/O May 3, 2023
@Correct-Syntax Correct-Syntax added in progress This issue/PR is in progress. Your input is always welcome and removed good first issue Good for new contributors ready Non-blocked issue ready to be implemented labels May 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Related to the core renderer and/or backend feature New feature or request high priority This should be worked on first above other issues with lower a priority in progress This issue/PR is in progress. Your input is always welcome python This issue includes writing Python code
Projects
Status: No status
v0.6.0 alpha MVP
In progress
Development

No branches or pull requests

1 participant