quintel / etmoses

Online decision support tool to create local energy situations for neighbourhoods, cities and regions with a time resolution of 15 minutes created and maintained by Quintel – Not maintained
https://moses.energytransitionmodel.com
MIT License
11 stars 3 forks source link

Y-axis doesn't update #1429

Closed ThomBuijs closed 8 years ago

ThomBuijs commented 8 years ago

I am using this LES, but when I look at the results, the Y axis unit for node MSR wageningen campus isn't updated. It shows me values in kW which would be logical in MW.

grdw commented 8 years ago

screen shot 2016-09-15 at 16 50 39

I'm not able to reproduce this behavior exactly. I just looked at your LES and I see the units in MW. Did you do anything else than just look at the results? For example; update the strategies, selected another week, changed from 'Total' view to 'Stacked' and back?

ThomBuijs commented 8 years ago

I did play a bit with the the view to see a whole year and a week, and stacked and total view. However right now I get the following problem that the unit on the Y-axis is MW also for other nodes while the order of magnitude for the values are in kW.

units wrong

knipsel2

So right now the node MSR Wageningen Campus is displayed correctly, but the others aren't.

grdw commented 8 years ago

I'll fix this with some priority 👍

grdw commented 8 years ago

@ThomBuijs these changes won't be visible directly on production though. For now they're on the beta server.

@antw is it ok, if I cherry-pick this single commit to production?

antw commented 8 years ago

@antw is it ok, if I cherry-pick this single commit to production?

Sure thing! It's easy to rollback, so go for it. :+1:

If this can also go to production, you can just merge master into the production branch rather than cherry-picking. I've done that a couple of times this week with small fixes.