Skip to content

Matrix allocation optimization #321

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
Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Matrix allocation optimization #321

wants to merge 2 commits into from

Conversation

DanielLacina
Copy link
Contributor

No description provided.

@DanielLacina DanielLacina requested a review from Mec-iS as a code owner July 13, 2025 04:38
@DanielLacina
Copy link
Contributor Author

This is not finished but I in order to use the MatrixPool, the Dense Matrix could only accept types that have static lifetimes.

@DanielLacina
Copy link
Contributor Author

There's some situations where if you pass in borrowed values , the static lifetime condition is not satisifed.

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jul 13, 2025

i don't think this is the right way to go.
Let's think about this for some time to have a better understanding

@DanielLacina
Copy link
Contributor Author

Ok

@Mec-iS
Copy link
Collaborator

Mec-iS commented Jul 13, 2025

or at least we need a multi-step roadmap to gradually implement this.

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

@Mec-iS Mec-iS Awaiting requested review from Mec-iS Mec-iS is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DanielLacina @Mec-iS