switch parse_costs! and parse_constraints

[ctessum]

It seems like they are currently backward

[ChrisRackauckas]

@AayushSabharwal better test this?

[AayushSabharwal]

I'm confused, because we have a test for this and it passes both before and after the swap on CI, but fails after swapping locally.

[AayushSabharwal]

@ctessum how did you run into this? Are you using MTK.get_gui_metadata?

[ctessum]

I don't have a specific case of it failing, I was just looking through the code and it seemed wrong.

[AayushSabharwal]

I see. So the functions are identical, and because they were passed the correct arrays it worked. i.e. in the @costs case, it called parse_constraints! but passed it the costs array. Constructing the system thus worked fine, it's just that in the Model struct the values are backward.