Force ascending order of independant variables in Table:IndendependentVariable

[jmarrec]

Issue overview

I had noted it here as a question: #4652 (comment)

Current Behavior

We get a bunch of errors on the SDD ReverseTranslator tests of openstudio-resources due to this

   ** Severe  ** Table:IndependentVariable \"{172A710A-926C-44EC-9AC3-1E694B8332A0}\": axis is not sorted.",
   **  Fatal  ** Btwxt: Errors discovered, program terminates.

Expected Behavior

Worked as 3.4.0, stopped working at 3.5.0 when we moved from Table:MultiVariableLookup

https://github.com/NREL/OpenStudio-resources/blob/46a3246cd12026af71fc7db11d990697bf002b9b/test/010112_SchSml_PSZ16.xml_3.4.0_out.osw#L2

https://github.com/NREL/OpenStudio-resources/blob/46a3246cd12026af71fc7db11d990697bf002b9b/test/010112_SchSml_PSZ16.xml_3.5.0_out.osw#L1-L7

Steps to Reproduce

make a non sorted one. try to run in E+.

Possible Solution

Force ascending order. Determine if should be done in Model or at ForwardTranslation time.

Details

Environment

Some additional details about your environment for this issue (if relevant):

• Platform (Operating system, version): all
• Version of OpenStudio (if using an intermediate build, include SHA): 3.5.0+

Context

Noticed this today when I ran the OpenStudio-resources testing with 3.6.1 official.

[jmarrec]

The API for this object is really bad, having to add the independent variables first, then the output values.
And figuring out how to set the TableLookup outputValues if you have more than one independent variable... cf https://github.com/NREL/OpenStudio-resources/blob/46a3246cd12026af71fc7db11d990697bf002b9b/model/simulationtests/tablelookup.rb#L40-L71

We should probably provide some kind of TableLookup::fromTable that could take an array of [X1, ..., Xn, Y] doubles or something. In there it'd be easy to check for ascending order and fix it automagically if not

Quick "fix"

The quick way to address this issue is to have the TableIndependentVariable::setValues throw a loud Warn and return false if the passed values aren't sorted. We can't automatically "fix" the order since the independent variables and dependent output values aren't passed in tandem.

SDD ReverseTranslator

Probably need some work in the SDD ReverseTranslator to sort the values taken from the xml.