Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUILDING_HEIGHT_DIST indicator name #1002

Closed
ebocher opened this issue Nov 5, 2024 · 4 comments
Closed

BUILDING_HEIGHT_DIST indicator name #1002

ebocher opened this issue Nov 5, 2024 · 4 comments
Assignees

Comments

@ebocher
Copy link
Member

ebocher commented Nov 5, 2024

At grid scale we compute the distribution of horizontal roof area fraction for each layer of the canopy. eg levels : [0, 5, 10, 15, 20, 25, 30]

See :

https://github.com/orbisgis/geoclimate/blob/master/geoindicators/src/main/groovy/org/orbisgis/geoclimate/geoindicators/WorkflowGeoIndicators.groovy#L1808

The name used to define this indicator is not very clear.

Moreover, this indicator is not mentioned in the documentation : https://github.com/orbisgis/geoclimate/wiki/Grid-indicators

@j3r3m1

@j3r3m1
Copy link
Collaborator

j3r3m1 commented Nov 6, 2024

Yes we can change the name to BUILDING_HEIGHT_DISTRIBUTION.
We can also add the building average distance to have all TARGET inputs.

@j3r3m1 j3r3m1 self-assigned this Nov 6, 2024
@j3r3m1
Copy link
Collaborator

j3r3m1 commented Nov 6, 2024

Documentation updated for BUILDING_HEIGHT_DISTRIBUTION

@j3r3m1
Copy link
Collaborator

j3r3m1 commented Nov 6, 2024

Fixed in #1003 + documentation OK

@ebocher
Copy link
Member Author

ebocher commented Nov 7, 2024

Fixed

@ebocher ebocher closed this as completed Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants