Wrong units in radiation analysis component outputs description

[mostaphaRoudsari]

http://www.grasshopper3d.com/xn/detail/2985220:Comment:1073935

"The RadiationAnalysis component mentions it returns values in Wh/m2 and Wh, but the generated legend says kWh/m2. The latter seems to be right."

@chriswmackey this is actually half true. If the sky is generated for a single HOY the result will be in Wh/m2. In case of annual sky the values will be kWh/m2. I'm not sure how do we want to manage this?

[chriswmackey]

Oh gosh. I had experienced this problem before but forgot to post it. I
think that we should put a conversion factor for the cases where we get
Wh/m2 in order to make things consistent. I set all of the E+ results to
be in kWh so I think it is worthwhile making kWh a standard across all
analysis components. I'll try to get to this and fix the description
sometime this weekend unless you get to it first, Mostapha.

-Chris

On Wed, May 21, 2014 at 9:09 PM, Mostapha Sadeghipour Roudsari <
notifications@github.com> wrote:

http://www.grasshopper3d.com/xn/detail/2985220:Comment:1073935

"The RadiationAnalysis component mentions it returns values in Wh/m2 and
Wh, but the generated legend says kWh/m2. The latter seems to be right."

@chriswmackey https://github.com/chriswmackey this is actually half
true. If the sky is generated for a single HOY the result will be in Wh/m2.
In case of annual sky the values will be kWh/m2. I'm not sure how do we
want to manage this?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/62
.

[mostaphaRoudsari]

For some reason it is back! http://www.grasshopper3d.com/group/ladybug/forum/topics/one-doubt-about-radiation-data

Can you take care of this once more @chriswmackey? Thanks.

[chriswmackey]

@mostaphaRoudsari , I take a look into it and have it taken care of for the release. I think that we each thought that the other was going to fix the issue and neither of us ended up addressing it.

[mostaphaRoudsari]

@chriswmackey thanks!

[chriswmackey]

Hello Everyone,
This is now fixed. The selected sky matrices now appear in kWh.

Enjoy!
-Chris