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.

Sign up for GitHub

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

Jump to bottom

Replace Eigen Library with Custom Matrix Multiplication Implementation in C++ Code #11

Open
MrNeRF opened this issue Aug 18, 2023 · 6 comments
Open

Replace Eigen Library with Custom Matrix Multiplication Implementation in C++ Code #11

MrNeRF opened this issue Aug 18, 2023 · 6 comments
Labels
easy good first issue Good for newcomers help wanted Extra attention is needed

Comments

@MrNeRF
Copy link
Owner

MrNeRF commented Aug 18, 2023 •
edited
Loading

In our project, we currently utilize the Eigen library for matrix operations. We want to be light on dependencies and we are considering replacing Eigen with our own implementation of matrix multiplication.

So, make it happen and make some tests :)
@MrNeRF MrNeRF added the good first issue Good for newcomers label Aug 18, 2023
@MrNeRF MrNeRF changed the title Replace Eigen Library with Custom Matrix Multiplication Implementation Replace Eigen Library with Custom Matrix Multiplication Implementation in C++ Code Aug 18, 2023
@MrNeRF MrNeRF added easy help wanted Extra attention is needed labels Aug 18, 2023
@DapengFeng
Copy link

DapengFeng commented Sep 10, 2023 •
edited
Loading

Does it mean to replace Eigen with libtorch?

@MrNeRF
Copy link
Owner Author

MrNeRF commented Sep 10, 2023

No. I would suggest with own matmul and I believe quaternion ops. Its not much. The goal is to reduce deps from other libs to make it as self contained as possible.

@MrNeRF
Copy link
Owner Author

MrNeRF commented Sep 10, 2023

Same for glm. There might be overlapping operations with eigen that are used.

@MrNeRF
Copy link
Owner Author

MrNeRF commented Sep 11, 2023

Are you working on this?

@DapengFeng
Copy link

Are you working on this?

Yeah, I will try it.

@MrNeRF
Copy link
Owner Author

MrNeRF commented Oct 12, 2024

I would still like to remove Eigen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
easy good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DapengFeng @MrNeRF