This reformats the whole repo. Right now the formatter is broken so don't worry about it. Revert that.

@ChrisRackauckas Thanks! I reverted the formatter commit.

@ChrisRackauckas Question for you. It seems like the existing code in MTKFMIExt > parseFMIVariableName() mutates the "name" input variable. Is that OK? My understanding is that this is considered bad practice as Julia passes by reference. Also if it the input is mutated, the convention is to add ! to the function name, right?

I don't see where it mutates it. It creates a view but then creates a Symbol, which is effectively an interned string. But I don't see it actually mutating the string? Put to the line that you're worried about.

Though it probably shouldn't create the symbol there for performance reasons on large models, but that's a minor detail.

Ok, I was concerned about thename = @view name[....] with name on the left side of an assignment. I guess that's ok with@view macro?

that just makesname be a reference to a subset of what was previouslyname, no mutation there.

The test doesn't work

@AayushSabharwal what is the problem with the test on your end?