search

ariesteam / aries

http://www.ariesonline.org
GNU General Public License v3.0
6 stars 1 forks source link

fix uncertainty labeling in netCDF outputs #35

Closed kbagstad closed 12 years ago

kbagstad commented 12 years ago

@fvilla

kbagstad commented 12 years ago

Uncertainty statements in netCDF outputs currently read "AestheticViewProvision:aestheticServiceUncertainty. Please move the word "uncertainty" to its correct place in the concept name.

kbagstad commented 12 years ago

Reopening this one with some regret. Have you tested it out yourself? I'm doing some model runs just now and have the bad old syntax rather than the new, clearer one.

fvilla commented 12 years ago

yes, I tested it and it's very unlikely that the old behavior has stayed in, as it was encapsulated in one single function that definitely doesn't do that. Have you updated thinklab? it's there, not in ARIES.

fvilla commented 12 years ago

if it remains after updating thinklab, please send a model I can run that produces that.

kbagstad commented 12 years ago

Jesus, I'm an idiot. I've pulled and merged the latest from thinklab but not built, so the latest hasn't been incorporated when I run (at least on my desktop). OK, I claimed not to be an idiot on the last one, but will claim full idiocy on this one. Thanks and looks like we'll keep trading emails.

Ken

On Fri, Jan 27, 2012 at 8:11 AM, Ferdinando Villa reply@reply.github.com wrote:

yes, I tested it and it's very unlikely that the old behavior has stayed in, as it was encapsulated in one single function that definitely doesn't do that. Have you updated thinklab? it's there, not in ARIES.

Reply to this email directly or view it on GitHub: https://github.com/ariesteam/aries/issues/35#issuecomment-3688073

kbagstad commented 12 years ago

Uh, yeah, closing per the last comment. Building thinklab helps.