Matt Fishman

> For `BlockDiagonal` matrices, I think it is quite natural that `U` and `V` are `BlockDiagonal`, but I am not so sure how to define `S`. Currently, it is a...

Hi @corbett5, I'm surprised that actually works using `replaceinds`, it's a bit outside the scope of what the function was meant for. However, I guess because it is implemented using...

We don't have time to work on that right now but we'd be happy to accept a PR, it shouldn't be too hard to add support for that with judicious...

Thanks for the summary of what you tried, that basically sounds like the approach we would use. That's too bad that you didn't see speedups, maybe some operations in the...

@LHerviou I don't think any rewrites going on in ITensors.jl/NDTensors.jl would affect the design of this package. Some of the unit cell tag logic may get incorporated more generally/systematically into...

No worries, I also haven't had time to work on this package recently, as I'm sure you've noticed. I just wanted to make sure there weren't any misconceptions about our...

@mofeing indeed I imagine these kinds of graph data structures are fundamental to designing general tensor networks. Why do you say you can't use Dictionaries.jl? Maybe we could cut down...

I don't think we have many timers inside the codebase right now anyway, what were you hoping to time? As an alternative, I would recommend using a [profiler](https://docs.julialang.org/en/v1/manual/profile).

I don't remember testing it, I haven't used `TimerOutputs` in a long time, we were mostly using it in the codebase for debugging some performance issues early on when developing...

I'll keep this issue open as a reminder to remove `TimerOutputs` from the codebase. @MazenAli to summarize, I would recommend using a profiler instead, or dev'ing the package and adding...