Skip to content

Account for several windspeed measurements falling on a single cell #10

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

Open
wants to merge 3 commits into
base: ros1
Choose a base branch
from

Conversation

andresgongora
Copy link

Problem: If GADEN uses a lower spatial resolution than OpenFoam, several windspeed measurements may fall into a single cell. This is no problem if you use the same resolution for both simulations (say 10cm cubes).

Fix: I've updated the openFoam_to_gaden script to account for this by simply averaging all measurements that fall on a single GADEN cell.

Q: Is there any feature-request page or changelog I should update? I've already removed the appropriate ToDo lines in the script, but I'm unaware if there are other files that should be updated.

@JGMonroy
Copy link
Member

Hi @andresgongora,
The idea is good. Yet we have "removed" the Matlab script from the current GADEN implementation. At the moment we read directly the CAD models of the environment (.stl format) to generate the Occupancy3D and to re-format the wind data (check the new GADEN_preprocessing node).

Maybe it deserves to implement something similar in the new node, I'll keep this pull request open to don't forget it.

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

Successfully merging this pull request may close these issues.

2 participants