Per Östlund

I've implemented a possible solution in #10541, though I'm not entirely sure it's correct. What we do now is first remove inside connectors with min = 0, then select the...

> LGTM. > > BTW, stream connectors are extensively used in the Buildings library, for which we are carrying out verification against Dymola-generated results, so if there's something wrong, chances...

> I added that uses by hand. Ok, that uses annotation should have been in the package. Nevertheless, this use of _uses_ is well accepted by OM (at least since...

> it seems that there is coherency in the OM behaviour (and therefore in code having uses() outside top-level classes): in fact, [#9825 (comment)](https://github.com/OpenModelica/OpenModelica/issues/9825#issuecomment-1333614232) says that OM does not consider...

> This is weird. There is actually no conversion involved in this case: 950 has an exact representation in IEEE 754 double precision, so multiplying or dividing it by 1.0,...

It's not fixed by #8094, though it improves the error message a bit: ``` Error: Internal error NFConnection.split got unbalanced connection connect(ad1.hb, h): lhs: {ad1.hb.position} rhs: {h.position, h.pb.speed} ``` Of...

I had a closer look at this, and the issue is that we don't properly support arrays of expandable connectors. Having an array of expandable connectors means that the elements...

> @perost can you elaborate on the following sentance, in particular, why the elements of the array would have different types, and what it does mean ? > > >...

The model works fine on my machine and uses about 11 GB of RAM to flatten, so there's at least no other issue than using a lot of memory and...

> > @perost this model uses the `RectangularZoneTemplate`, which includes many conditional components. If they are flattened and not simply ignored then that could be part of the explanation. I've...