EDDTableAggregateRows - override actual_range with min/max calculated on all chidren

[MarcoAlbaETT]

Description

In EDDTableAggregateRows class override actual_range in tSourceAtts Attributes with min/max values calculated on all chidren.
This is to fix the problem about range for time,latitude,longitude and depth that is not the min/max of all the child but the range from the first child.

Fixes #145

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

Checklist before requesting a review

• I have performed a self-review of my code
• My code follows the style guidelines of this project
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings

[ChrisJohnNOAA]

I'm running this through the full test set.

Are the data_min and data_max changes needed to fix the min/max values problem?

[ChrisJohnNOAA]

The tests all passed.

[MarcoAlbaETT]

I'm running this through the full test set.

Are the data_min and data_max changes needed to fix the min/max values problem?

Not really, but since I'm setting the actual_range and EDVTimeStamp is using that as first value, I think setting data_min and data_max is not needed.

[ChrisJohnNOAA]

@MarcoAlbaETT I have a test that ensures the new code is working here: 532bb66

One thing to note, I'm using TS_SLEV_TAD_19 and TS_SLEV_TAD_1888 in the test which are different than the example in the issue.

[MarcoAlbaETT]

@ChrisJohnNOAA thank you very much for the test and sorry for not doing it

[ChrisJohnNOAA]

Thank you for doing the change!