internal relayout method called with undefined
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 18.3k
- Forks
- 2k
- Merge medio
- 2 d 12 h
- PR fusionados (30 d)
- 28
Descripción
We have observed a relayout failure immediately after this warning:
WARN: Relayout fail. undefined undefined
Non-Error promise rejection captured with value: undefined
coming from: https://github.com/plotly/plotly.js/blob/24b6f75e2d3b58cb1cd9cdd850894720404373d3/src/plot_api/plot_api.js#L1774
I'm not sure which exact circumstances triggered this failure, but the relayout internal function probably should not reject if astr is undefined, as this simply means there is nothing to update (recording the warning seems fine as it helps to diagnose call sites which are not correctly omitting the relayout, but it should not cause the function to fail).
I suggest that this branch should return successfully after logging a warning if astr is undefined / null.
It would also be useful for diagnosing the root cause further if the warning could include a stack trace.
Plotly.js version: 2.26.2
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.
Línea de trabajo
Comienza en src/plot_api/plot_api.js alrededor de la línea 1774 y sigue cómo un valor undefined o null de astr llega a la función interna relayout. Verifica el comportamiento de la advertencia y confirma que esta ruta se completa correctamente en lugar de rechazar; considera cómo un stack trace ayudaría al diagnóstico y comprueba el comportamiento relevante de relayout antes de definir la finalización.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript
- Área
- data-visualization
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 45/100