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

support conversions from/to other frameworks #178

Open
johnnychen94 opened this issue Apr 11, 2022 · 2 comments
Open

support conversions from/to other frameworks #178

johnnychen94 opened this issue Apr 11, 2022 · 2 comments
Labels
enhancement good first issue Would make a good contribution from new contributors

Comments

@johnnychen94
Copy link
Member

We do have colorview/channelview/rawview to build these functionalities, but from a user's perspective, it would be much much easier to understand and use things like:

  • to_matlab/from_matlab
  • to_torch/from_torch

Because other framework doesn't always contain colorspace information on their numerical array, assumptions must be made. For instance:

  • for MATLAB we can assume that 3D numerical array with size (m, n, 3) is an RGB image of size (m, n).
  • for deep learning frameworks like pytorch, we can assume that 4D tensor with size (w, h, 3) is an RGB image of size (m, n).

These utilities do not need to cover every use case and do not need to provide 100% consistency; they're meant to help users to migrate the old codes from other frameworks.

@johnnychen94 johnnychen94 added enhancement good first issue Would make a good contribution from new contributors labels Apr 11, 2022
@Jay-sanjay
Copy link
Contributor

sir, can you precisely tell what actually I need to do to fix this issue

@ashwani-rathee
Copy link
Member

Johnny was working on it here: #179

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement good first issue Would make a good contribution from new contributors
Projects
None yet
Development

No branches or pull requests

3 participants