-
Notifications
You must be signed in to change notification settings - Fork 1
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
Updates v0.3.5 #484
base: master
Are you sure you want to change the base?
Updates v0.3.5 #484
Conversation
R CMD check fails with error below. It indicates there are some regions for which there are observations in the data, but no shape data are available for plotting them in maps.
|
I'm checking the pre upload files if they are ok I'll upload them again and check the upload flow in the process. |
@mvarewyck am I correct this issue occurs with schade data ? |
The issue does occur in the pre upload data. This is a data issue and should be fixed by the next update (see https://github.com/inbo/backoffice-wild-analyse/issues/1157) |
Yes it is. |
R prints the following warning:
|
The issue seems to affect all boxplot plots like FIGUUR: Gewicht per jaar en FIGUUR: Onderkaaklengte per jaar (INBO of Meldingsformulier) |
Its seems the RAM usage of my docker explodes to > 11Gb of the 12Gb available. |
I was able to reduce the inflation of my docker RAM by switching from wsl2 to hyper-V however the issue with loading the Landkaart remains. when looking at the chrome devtools I see the following error:
PS run command:
|
Ok important to know "Gerapporteerd aantal schadegevallen" of the same docker build does load. As does the wbe map however it only loads after switching the wbe displayed. Does this indicates the app needs user input to start loading? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mvarewyck the app apperently needs user input before loading some graphs/maps while other like the main landkaart doesn't seem te load at all, see #484 (comment) & #484 (comment)
It's all due to this commit 262589e, including lightweight plotly . I'm looking into it (reverting or fixing). Known issue: plotly/plotly.R#2119 |
@SanderDevisscher Docker image works fine for me now. I reverted the commit, given there is no easy solution and the planned re-design of the app. |
Include latest changes before re-designing the app.