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

QR decomposition proposition #645

Closed
unordinateur opened this issue Jun 28, 2017 · 3 comments
Closed

QR decomposition proposition #645

unordinateur opened this issue Jun 28, 2017 · 3 comments
Assignees

Comments

@unordinateur
Copy link

Hi!

While working with glm on a project; I encountered a need to do RQ (and QR) decomposition of some matrices.

I coded some functions to do that; that I would like to share them back to glm so you can integrate them to the library.

How can I do that? (That would be the first time I contribute to an open source project, so I'm not familiar with the usual procedures for that.)

Thank you!

@Groovounet Groovounet self-assigned this Jun 28, 2017
@Groovounet
Copy link
Member

Hi,

For that you need to do a pull request (PR). To write the feature, create an extension in here https://github.com/g-truc/glm/tree/master/glm/gtx, let's say call it qr_decomposition and create a test that should go here https://github.com/g-truc/glm/tree/master/test/gtx.

When you create the issue, the test infrastructure of the repository will run the test so you can see it's ok on others platforms that you own platform.

If the PR is not perfect, it's ok, do what you thing you should do and I'll take it from there if it's reasonable.
I might comment the PR for involvement in which case you might have to do a second iteration.

Thanks for contributing!

@unordinateur
Copy link
Author

Great, thanks!
I'll tell you when it's done.

@unordinateur
Copy link
Author

So I just did a pull request with my implementations! (PR #654)

After a few fixes, it now passes all the tests, so it should be able to be merged without any problems! :)

Vincent

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