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

BR_TriGrid3D info not updated in the "SQL fonctions" section on the Wiki #67

Closed
GCorbeau opened this issue Feb 25, 2019 · 3 comments
Closed
Assignees

Comments

@GCorbeau
Copy link

In the wiki, it is written :

BR_TriGrid3D(Geometry computeEnvelope,VARCHAR buildingsTable, VARCHAR heightFieldName, VARCHAR sourcesTable, VARCHAR sourcesTableSoundFieldName, VARCHAR groundTypeTable, VARCHAR demTable, double maximumPropagationDistance, double maximumWallSeekingDistance, double roadsWidth, double receiversDensification, double maximumAreaOfTriangle, int soundReflectionOrder, int soundDiffractionOrder, double wallAlpha)

In stead of :

BR_TriGrid3D(String buildingsTable, String heightFieldName, String sourcesTable, String sourcesTableSoundFieldName, String groundTypeTable, String demTable, double maximumPropagationDistance, double maximumWallSeekingDistance, double roadsWidth, double receiversDensification, double maximumAreaOfTriangle, int soundReflectionOrder, int soundDiffractionOrder, double wallAlpha)

wich is correct in the OrbisGis's "show SQL fonction list" tab.

@qhesry
Copy link

qhesry commented Mar 14, 2019

Hello.

In the source code, we have 2 different method signature for this functions.
One with the ComputeEnvelope parameter and one without, because it's optional.

I've some trouble to understand where is the error here.
Can you give more details, on where is the error and where have you find your correction ?

Thanks you.

@GCorbeau
Copy link
Author

Hello,
I'm currently working with the guys working on the NM project and they told me that le version with the ComputeEnveloppe is an ancient one. The new one (described in the fonction tab of Orbisgis), without enveloppe, works perfectly.

@nicolas-f
Copy link
Member

The signature

BR_TriGrid3D(Geometry computeEnvelope,VARCHAR buildingsTable, VARCHAR heightFieldName, VARCHAR sourcesTable, VARCHAR sourcesTableSoundFieldName, VARCHAR groundTypeTable, VARCHAR demTable, double maximumPropagationDistance, double maximumWallSeekingDistance, double roadsWidth, double receiversDensification, double maximumAreaOfTriangle, int soundReflectionOrder, int soundDiffractionOrder, double wallAlpha)

is implemented in https://github.com/Ifsttar/NoiseModelling but not in https://github.com/pierromond/NoiseModelling so this issue is not relevant

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

4 participants