Skip to content

Conversation

Ericgig
Copy link
Member

@Ericgig Ericgig commented Jan 9, 2025

We have benchmarks for random sparse and dense matrices, but both are bad scenario for the Dia format.
I added a tridiagonal matrix case to benchmark the Dia operations in situation where they are useful.

Copy link
Contributor

@hodgestar hodgestar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a minor black formatting issue (easily fixed) and it looks like mambaforge needs replacing with miniforge in this repo too, but the actual change looks good.

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

Successfully merging this pull request may close these issues.

2 participants