Petar Mlinarić

Results 146 comments of Petar Mlinarić

> > I'm not sure that would work well in general. You might want different numbers and locations of points in different situations. I would guess that for plotting you...

> So the minimal requirement to achieve the latter would be to interpret the `w_limits` as Hz if `Hz=True` in the plotting options and use your adaptive method. This would...

> Not sure how useful that would be. That library only seems to support coo and dok formats. So to use any linear solver, the matrices would have to be...

Using PyTorch for sparse tensors sounds a bit like overkill to me... From the documentation of [torch.sparse](https://pytorch.org/docs/stable/sparse.html), it seems PyTorch only supports the COO format. Also, there is a warning...

> > Using PyTorch for sparse tensors sounds a bit like overkill to me... > > Why? As far as I understand it pytorch/tensorflow are basically tensor libraries with some...

> Will the current workaround break with 0.5.0? No, it shouldn't. > Could we branch the code on the slycot version instead? That might be better, to support more Slycot...

> Maybe you could also look at the warnings we get from slycot when you get to this? > Starting in line 1477 > [conda.log.txt](https://github.com/pymor/pymor/files/9165378/conda.log.txt) @lbalicki might know what to...

Hi Ale! :smile: Thanks for pointing out psssMOR, I wasn't aware of it. It should be straightforward to add a class for parametric LTI systems in pyMOR, but it would...

@sdrave I see your point, it's not like `LTISystem.__init__` is checking that the operators are non-parametric. After thinking about it, it seems to me that the `LTISystem` class could work...

> If you dislike the changes, we can also revert the last commit. @pmli, what do you think? It all sounds reasonable to me.