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

Update LDENPointNoiseMapFactory.java - Increase Precision for Numeric Columns in forgeCreateTable Method #719

Open
IsotoCedex opened this issue Jan 7, 2025 · 1 comment
Labels

Comments

@IsotoCedex
Copy link

Description:
This change updates the precision of numeric columns from (5,2) to (9,2) in the forgeCreateTable method.

See acf9ae7

Reason for Change:
The previous precision (5,2) caused the error:
"Value too long for column 'HZ8000 NUMERIC(5, 2)'"
This issue typically occurs when the propagation distance exceeds 3000 meters, resulting in values that exceed the allowed range.

Proposed Fix:
Update the numeric(5,2) columns to numeric(9,2) for fields LAEQ, LEQ, and frequency-related columns (HZ*).
This ensures compatibility with larger numerical values generated in scenarios with significant propagation distances.

Impact:
Resolves the numerical precision issue for long propagation distances.
Prevents runtime errors when inserting large numerical values into frequency (HZ*) and acoustic level (LAEQ, LEQ) columns.

@IsotoCedex IsotoCedex added the bug label Jan 7, 2025
@nicolas-f
Copy link
Member

In the last version we switch to the REAL type

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants