ladybug-blender icon indicating copy to clipboard operation
ladybug-blender copied to clipboard

JSON doesn't include default value?

Open Moult opened this issue 5 years ago • 2 comments

https://github.com/ladybug-tools/ladybug-grasshopper/blob/master/ladybug_grasshopper/json/LB_Direct_Sun_Hours.json#L58-L64

Shouldn't there be a default value? Ping @mostaphaRoudsari and @chriswmackey

Moult avatar Oct 05 '20 21:10 Moult

Another example?

https://github.com/ladybug-tools/ladybug-grasshopper/blob/master/ladybug_grasshopper/json/LB_Analysis_Period.json#L52-L77

Moult avatar Oct 05 '20 21:10 Moult

Ideally there should be one but I think @chriswmackey is relying on the default values in the Python libraries and doesn't expose them inside Grasshopper. This should be an easy fix. We should probably transfer this issue to the other repository.

mostaphaRoudsari avatar Oct 06 '20 01:10 mostaphaRoudsari