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

Remove Matrix (at least for now) #11

Open
kabasset opened this issue Jun 25, 2023 · 1 comment
Open

Remove Matrix (at least for now) #11

kabasset opened this issue Jun 25, 2023 · 1 comment
Assignees
@kabasset
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@kabasset
Copy link
Owner

The class was introduced for Affinity, which now relies directly on Eigen. Implementing a full matrix arithmetic/service base would be a nightmare and Eigen would probably be better in every respect.
@kabasset kabasset added enhancement New feature or request good first issue Good for newcomers labels Jun 25, 2023
@kabasset kabasset self-assigned this Jun 25, 2023
@kabasset
Copy link
Owner Author

On open question is: is it cheap to bypass Eigen completely in Affinity, given the limited usage (inverse() only is hard).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kabasset kabasset

Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kabasset