Openmeteo forecast probability of precipitation is not returning an accurate value
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 80/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript
- Área
- backend
Línea de trabajo
El issue señala openmeteo.js, la definición de dailyParams alrededor de las líneas 71–89 y el cálculo de la probabilidad de precipitación alrededor de la línea 424. Lee primero esas secciones y comprueba cómo se asignan los valores diarios de la API de Open-Meteo al resultado meteorológico. El trabajo está terminado cuando la probabilidad indicada usa el valor precipitation_probability_max de la API; ejecuta las pruebas pertinentes del módulo meteorológico si están disponibles.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Environment
MagicMirror² version: 2.38.0
Node version: 24.22.0
npm version: 11.21.0
Platform: docker
Which start option are you using?
node --run start
Are you using PM2?
I don't know
Module
weather
Have you tried disabling other modules?
- Yes
- No
Have you searched if someone else has already reported the issue on the forum or in the issues?
- Yes
What did you do?
The current version of openmeteo.js calculates probability of precipitation using daily hours of precipitation divided by 24 hours to arrive at a percentage.
Line 424: precipitationProbability: weather.precipitation_hours != null ? parseFloat(weather.precipitation_hours * 100 / 24) : null,
This calculation doesn't reflect the actual probability of precipitation.
What did you expect to happen?
I expected openmeteo.js to pull precipitation_probability_max from the openmeteo API and report that as the probability of precipitation.
What actually happened?
The current calculation of probability of precipitation as described above is is used and returns an inaccurate result.
Additional comments
Possible fix (that worked for me) would be to:
- Add "precipitation_probability_max" to dailyParams (Lines 71-89)
- Modify line 424 to:
precipitationProbability: weather.precipitation_probability_max != null ? parseFloat(weather.precipitation_probability_max) : null,
Here's a copy of openmeteo.js that I'm currently using.
Participation
- I am willing to submit a pull request for this change.
- Lenguaje dominante
- JavaScript
- Estrellas
- 24.1k
- Forks
- 4.6k
- Merge medio
- 5 h 36 min
- PR fusionados (30 d)
- 18
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de MagicMirrorOrg/MagicMirror
-
bug
Dificultad 3/5 Medio día Aptitud para principiantes 69/100
MagicMirrorOrg/MagicMirror#4311 · 5 comentarios ·
Los mantenedores suelen responder en 1 día
-
[Bug] Black screen after upgrade to MagicMirror² 2.38.0 on Raspberry Pi 3 with Electron 44.4.5Abiertobug ready (coming with next release)
Dificultad 4/5 3-5 días Aptitud para principiantes 43/100
MagicMirrorOrg/MagicMirror#4287 · 16 comentarios · 3 reacciones ·
Los mantenedores suelen responder en 1 día
-
enhancement PR Welcome
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
MagicMirrorOrg/MagicMirror#3808 · 23 comentarios · 2 reacciones ·
Los mantenedores suelen responder en 1 día
-
arm32 is dead (sadly)Abiertodependencies not ours
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
MagicMirrorOrg/MagicMirror#3796 · 30 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Proposal: Wiki RestructureQuizá libre de nuevo @KristjanESPERANTO la tomó hace 153 días y no hay ningún pull request abierto. Abiertounder development under investigation
MagicMirrorOrg/MagicMirror#2545 · 24 comentarios · 2 asignados ·
Los mantenedores suelen responder en 1 día
Todos los issues de MagicMirrorOrg/MagicMirror
Issues similares
-
accessibility bug html websites
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
quarto-dev/quarto-cli#15187 ·
Los mantenedores suelen responder en 1 día
-
bug needs triage P3
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
pbakaus/impeccable#1143 ·
Los mantenedores suelen responder en 1 día
-
status: waiting for maintainer
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
mui/material-ui#49547 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
capricorn86/happy-dom#2544 ·
Los mantenedores suelen responder en 1 día
-
next() is called twice when the request closes after busboy fails to startPosiblemente ocupada @NgoQuocViet2001 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100