Skip to content

Port transforms.LinearTransformation to prototype.transforms #5538

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

Closed
Tracked by #5520
pmeier opened this issue Mar 3, 2022 · 3 comments · Fixed by #6458
Closed
Tracked by #5520

Port transforms.LinearTransformation to prototype.transforms #5538

pmeier opened this issue Mar 3, 2022 · 3 comments · Fixed by #6458

Comments

@pmeier
Copy link
Collaborator

pmeier commented Mar 3, 2022

cc @vfdev-5 @datumbox @bjuncek

@yassineAlouini
Copy link
Contributor

@pmeier Out-of-curiosity, is any help needed for this feature or does it require too much knowledge/not advised for new contributors? 🤔

@pmeier
Copy link
Collaborator Author

pmeier commented Mar 31, 2022

The removal of the "help wanted" label is explained in #5520 (comment). In addition, #5336 (comment) also applies. Thus, at this time, it is probably not a good idea to start. This has nothing to do with the knowledge / experience though. I'm happy to assign you as soon we have ironed out the problems.

If you are looking at ways to contribute, you might also have a look at all issues tagged with the "help wanted" label. That being said, please be aware that we (torchvision maintainers) are experiencing a heavy internal "load" at the moment that takes a lot of our time. I'm not saying this to discourage contributions, but rather as explanation why reviews might take some time.

@yassineAlouini
Copy link
Contributor

Thanks for the details and the pointers @pmeier.
I think I found something to contribute in the meantime.
Please let me know once things are a bit "lighter" on your side and you have more time. 👌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants