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.

Sign up for GitHub

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

Jump to bottom

Fix error if running only locally without internet #157

Closed
arunge opened this issue May 17, 2023 · 2 comments
Closed

Fix error if running only locally without internet #157

arunge opened this issue May 17, 2023 · 2 comments
Assignees
@arunge
Labels
bug Something isn't working Support: Bug fixing

Comments

@arunge
Copy link
Contributor

arunge commented May 17, 2023

Via e-mail:

... I consistently encounter "HTTP error 500" after entering the local server address.
@arunge arunge self-assigned this May 17, 2023
@arunge arunge mentioned this issue May 17, 2023
Merged
@arunge arunge added the bug Something isn't working label May 17, 2023
@arunge
Copy link
Contributor Author

arunge commented May 23, 2023

I am still trying to identify the problem. I fixed an error that the app does not crash anymore when used offline (version 23.4.1.18). When running R locally (.e.g with RStudio) it works.
However, when using the app offline with docker, it reacts very slow, one has to wait ~30 seconds until the import module loads. I already added some timeouts when testing for an available internet connection. However, there is still something not working as intended.

So this is not solved yet. Thanks for your patience!

@arunge arunge mentioned this issue Jun 6, 2023
Merged
arunge added a commit that referenced this issue Jun 6, 2023
@arunge
#157 try new dataTools
c6f8b44
@arunge arunge mentioned this issue Jun 7, 2023
Merged
arunge added a commit that referenced this issue Jun 7, 2023
@arunge
#157 reduce API calls in app
0b85fcf
@arunge
Copy link
Contributor Author

arunge commented Jun 7, 2023 •
edited
Loading

@isomemo With version 23.06.1 on ios-app beta I reduced the parts in the code where the app checks for an internet connection, and removed bugs that lead the app crash when using it offline.

A check for an internet connection tries to reach a webpage until a timeout, if the page could not be reached within this time, the app assumes that there is no internet connection. Although, the timeout is only 2 seconds and I do only selectively check for a connection, the app is very slow when working offline with docker. I don't know why. There are no such problems when working offline with RStudio.

If one really needs to work offline I therefore would recommend to try the local R installation.

Hopefully, this does solve the issue.

@arunge arunge closed this as completed Jun 7, 2023
@arunge arunge mentioned this issue Sep 4, 2023
Merged
1 task
arunge added a commit that referenced this issue Oct 17, 2023
@arunge @f-lukas
@jan-abel-inwt @mgross
Release version 23.10.0 (#181)
f9dc441 
* Feat/13 leaflet point symbol (#130)

* add function to draw symbols on leaflet map, add test

* try adding symbols to UI

* named pchChoices

* update test for pointSize

* add logic to create symbols based on a character column, update and add tests

* add symbol width

* apply pchChoices in other tabs

* fix docu, update tests, prepare symbol legend, add symbol width

* add setSymbolLegend and test

* make the symbol legend show an icon

* fix size of symbols in legend, rename some values

* fix test

* fix namespace

* fix tests after renaming

* try standard path in tests in getSymbolLegend

* update UI, shift update of legends

* remove test

* fix tests

* add size legend input, fix if negative minSize

* prepare pointSize legend

* refactor mapping values to sizes in pxl

* fix docu

* fix name

* create legend values for size

* extract helper functions for legend

* optimize update of data on map

* update news.md

* update news.md

* unique icons in size legend

* Feature/54 requested clustering changes (#136)

* bugfix wrong number of cluster due to wrong df order

* renaming and changing default cluster number

* requested renaming changes

* renaming and adding spatial cluster centers as option

* link to discussion

* clustering description

* description

* Update R/01-estimateMap.R

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Apply suggestions from code review

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Version 23.05.1: add "fixed" option (#150)

* add "fixed" option

* fix test

* add issue

* add user - possible fix for legend icon bug

* delete user - possible fix for legend icon bug

* try absolute path to www

* Version 23.05.2: Feat/133 borders and names on maps (#151)

* use maps without borders or names, add new maps

* add issue

* add info icon

* update helpText for interactive mapType

* Feat/return gif (#152)

* prepare gif

* prepare functions for gif loop

* update UI, select ipnut to choose between with/without gif

* add issue

* extract functions exportGraphic

* add options to export gif

* clean up

* update docu and tests

* clean up

* try hard path for icons

* interactive map: try path to dist, autoformat, update UI for points, fix sizes for legend

* fix test

* most recent DataTools

* Feature/54 clustering changes (#159)

* description and namespace

* function documentation

* renaming columns

* input changes and excel column changes

* KernelR renaming

* plot changes

* add missing package

* Fix/missing package (#160)

* fix missing package

* description

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* Version 23.06.0: Bug/161 leaflet map issues (#162)

* new button reload points

* add fixed size option

* remove new button, it did not help

* update news.md

* add issue to news

* use original names from map providers

* most recent dataTools

* try dataTools with optimized checks for internet connecton

* #157 try new dataTools

* try fix in sheet selection in import

* Version: 23.06.1: optimize app initialisation (#165)

* optimize app initialisation

* update news.md

* #157 reduce API calls in app

* Version 23.06.2: Feat/102 new iso memo api endpoints (#166)

* add new input mappingId

* update test

* rename schema labels

* update news.md

* add schema to save/load data selection

* fix text when empty

* fix missing export

* version up

* add catching error when exporting plots, functions to create gifs from svg and pdf (#171)

* add missing package

* add note

* Version 23.06.3: use always jpeg as base for gif file (#176)

* use always jpeg as base for gif file

* update news.md

* Version 23.06.4: Feat/169 pixel size (#170)

* extract functions to calculate center estimates

* apply new functions in plotMap functions

* update parameter of centerEstimateServer

* fix in plotExport

* remove parameters that are not needed anymore

* remove parameters that are not needed anymore

* add grid length to centerEstimate output

* add some comments

* update news.md

* update news

* fix text

* version up

* change news.md

* Missings assignR (#178)

* New feature for AssignR: Imputatiopn of missings

* Missings assign r refactored (#179)

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* #164 update description for custom maps

* Feat/apply model import from data tools (#184)

* exchange downUploadButton module with importData and downloadModel modules

* add config yaml

* fix news.md

* Feat/119 catch outer errors (#187)

* add tryCatch to wrappers

* version up

* fix version

* fix alignment

* update skin name

* Operator multi (#185)

* New feature for AssignR: Imputatiopn of missings

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

* multi batch user defined maps

* fix

* merge changes from beta (#186)

* new versioning

* add news, fix duplicate uid name

* fix bugs, global vars

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

---------

Co-authored-by: Lukas Fuchs <86153843+f-lukas@users.noreply.github.com>
Co-authored-by: Jan A <jan.abel@inwt-statistics.de>
Co-authored-by: Marcus Groß <marcus.gross@inwt-statistics.de>
arunge added a commit that referenced this issue Nov 7, 2023
@arunge @f-lukas
@jan-abel-inwt @mgross
push version 23.10.1 (#190)
f549072 
* Feat/13 leaflet point symbol (#130)

* add function to draw symbols on leaflet map, add test

* try adding symbols to UI

* named pchChoices

* update test for pointSize

* add logic to create symbols based on a character column, update and add tests

* add symbol width

* apply pchChoices in other tabs

* fix docu, update tests, prepare symbol legend, add symbol width

* add setSymbolLegend and test

* make the symbol legend show an icon

* fix size of symbols in legend, rename some values

* fix test

* fix namespace

* fix tests after renaming

* try standard path in tests in getSymbolLegend

* update UI, shift update of legends

* remove test

* fix tests

* add size legend input, fix if negative minSize

* prepare pointSize legend

* refactor mapping values to sizes in pxl

* fix docu

* fix name

* create legend values for size

* extract helper functions for legend

* optimize update of data on map

* update news.md

* update news.md

* unique icons in size legend

* Feature/54 requested clustering changes (#136)

* bugfix wrong number of cluster due to wrong df order

* renaming and changing default cluster number

* requested renaming changes

* renaming and adding spatial cluster centers as option

* link to discussion

* clustering description

* description

* Update R/01-estimateMap.R

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Apply suggestions from code review

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Version 23.05.1: add "fixed" option (#150)

* add "fixed" option

* fix test

* add issue

* add user - possible fix for legend icon bug

* delete user - possible fix for legend icon bug

* try absolute path to www

* Version 23.05.2: Feat/133 borders and names on maps (#151)

* use maps without borders or names, add new maps

* add issue

* add info icon

* update helpText for interactive mapType

* Feat/return gif (#152)

* prepare gif

* prepare functions for gif loop

* update UI, select ipnut to choose between with/without gif

* add issue

* extract functions exportGraphic

* add options to export gif

* clean up

* update docu and tests

* clean up

* try hard path for icons

* interactive map: try path to dist, autoformat, update UI for points, fix sizes for legend

* fix test

* most recent DataTools

* Feature/54 clustering changes (#159)

* description and namespace

* function documentation

* renaming columns

* input changes and excel column changes

* KernelR renaming

* plot changes

* add missing package

* Fix/missing package (#160)

* fix missing package

* description

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* Version 23.06.0: Bug/161 leaflet map issues (#162)

* new button reload points

* add fixed size option

* remove new button, it did not help

* update news.md

* add issue to news

* use original names from map providers

* most recent dataTools

* try dataTools with optimized checks for internet connecton

* #157 try new dataTools

* try fix in sheet selection in import

* Version: 23.06.1: optimize app initialisation (#165)

* optimize app initialisation

* update news.md

* #157 reduce API calls in app

* Version 23.06.2: Feat/102 new iso memo api endpoints (#166)

* add new input mappingId

* update test

* rename schema labels

* update news.md

* add schema to save/load data selection

* fix text when empty

* fix missing export

* version up

* add catching error when exporting plots, functions to create gifs from svg and pdf (#171)

* add missing package

* add note

* Version 23.06.3: use always jpeg as base for gif file (#176)

* use always jpeg as base for gif file

* update news.md

* Version 23.06.4: Feat/169 pixel size (#170)

* extract functions to calculate center estimates

* apply new functions in plotMap functions

* update parameter of centerEstimateServer

* fix in plotExport

* remove parameters that are not needed anymore

* remove parameters that are not needed anymore

* add grid length to centerEstimate output

* add some comments

* update news.md

* update news

* fix text

* version up

* change news.md

* Missings assignR (#178)

* New feature for AssignR: Imputatiopn of missings

* Missings assign r refactored (#179)

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* #164 update description for custom maps

* Feat/apply model import from data tools (#184)

* exchange downUploadButton module with importData and downloadModel modules

* add config yaml

* fix news.md

* Feat/119 catch outer errors (#187)

* add tryCatch to wrappers

* version up

* fix version

* fix alignment

* update skin name

* Operator multi (#185)

* New feature for AssignR: Imputatiopn of missings

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

* multi batch user defined maps

* fix

* merge changes from beta (#186)

* new versioning

* add news, fix duplicate uid name

* fix bugs, global vars

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Update cosign-installer version

* apply functions from IsoMemo instead of DataTools (#189)

* apply functions from IsoMemo instead of DataTools

* apply getFields instead of getMappingTable

* fix parameter name

* use IsoMemo::getData instead of DataTools::getRemoteData

* version up

* version up

* inc req DataTools version

* fix version

* apply has_internet before call to API

* add has_internet

* catch empty cases of mappingTable, reload mappingTable

* apply reloadMappingTable

* remove comment

* catch error if internet disappears

---------

Co-authored-by: Lukas Fuchs <86153843+f-lukas@users.noreply.github.com>
Co-authored-by: Jan A <jan.abel@inwt-statistics.de>
Co-authored-by: Marcus Groß <marcus.gross@inwt-statistics.de>
Co-authored-by: Jan Abel <106665518+jan-abel-inwt@users.noreply.github.com>
arunge added a commit that referenced this issue Dec 20, 2023
@arunge @f-lukas
@jan-abel-inwt @mgross
Release Version 23.12.0 (#198)
149ab78 
* Feat/13 leaflet point symbol (#130)

* add function to draw symbols on leaflet map, add test

* try adding symbols to UI

* named pchChoices

* update test for pointSize

* add logic to create symbols based on a character column, update and add tests

* add symbol width

* apply pchChoices in other tabs

* fix docu, update tests, prepare symbol legend, add symbol width

* add setSymbolLegend and test

* make the symbol legend show an icon

* fix size of symbols in legend, rename some values

* fix test

* fix namespace

* fix tests after renaming

* try standard path in tests in getSymbolLegend

* update UI, shift update of legends

* remove test

* fix tests

* add size legend input, fix if negative minSize

* prepare pointSize legend

* refactor mapping values to sizes in pxl

* fix docu

* fix name

* create legend values for size

* extract helper functions for legend

* optimize update of data on map

* update news.md

* update news.md

* unique icons in size legend

* Feature/54 requested clustering changes (#136)

* bugfix wrong number of cluster due to wrong df order

* renaming and changing default cluster number

* requested renaming changes

* renaming and adding spatial cluster centers as option

* link to discussion

* clustering description

* description

* Update R/01-estimateMap.R

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Apply suggestions from code review

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Version 23.05.1: add "fixed" option (#150)

* add "fixed" option

* fix test

* add issue

* add user - possible fix for legend icon bug

* delete user - possible fix for legend icon bug

* try absolute path to www

* Version 23.05.2: Feat/133 borders and names on maps (#151)

* use maps without borders or names, add new maps

* add issue

* add info icon

* update helpText for interactive mapType

* Feat/return gif (#152)

* prepare gif

* prepare functions for gif loop

* update UI, select ipnut to choose between with/without gif

* add issue

* extract functions exportGraphic

* add options to export gif

* clean up

* update docu and tests

* clean up

* try hard path for icons

* interactive map: try path to dist, autoformat, update UI for points, fix sizes for legend

* fix test

* most recent DataTools

* Feature/54 clustering changes (#159)

* description and namespace

* function documentation

* renaming columns

* input changes and excel column changes

* KernelR renaming

* plot changes

* add missing package

* Fix/missing package (#160)

* fix missing package

* description

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* update checkout, login-action, metadata-action & build-push-action to newer version & enable running workflow manually

* Version 23.06.0: Bug/161 leaflet map issues (#162)

* new button reload points

* add fixed size option

* remove new button, it did not help

* update news.md

* add issue to news

* use original names from map providers

* most recent dataTools

* try dataTools with optimized checks for internet connecton

* #157 try new dataTools

* try fix in sheet selection in import

* Version: 23.06.1: optimize app initialisation (#165)

* optimize app initialisation

* update news.md

* #157 reduce API calls in app

* Version 23.06.2: Feat/102 new iso memo api endpoints (#166)

* add new input mappingId

* update test

* rename schema labels

* update news.md

* add schema to save/load data selection

* fix text when empty

* fix missing export

* version up

* add catching error when exporting plots, functions to create gifs from svg and pdf (#171)

* add missing package

* add note

* Version 23.06.3: use always jpeg as base for gif file (#176)

* use always jpeg as base for gif file

* update news.md

* Version 23.06.4: Feat/169 pixel size (#170)

* extract functions to calculate center estimates

* apply new functions in plotMap functions

* update parameter of centerEstimateServer

* fix in plotExport

* remove parameters that are not needed anymore

* remove parameters that are not needed anymore

* add grid length to centerEstimate output

* add some comments

* update news.md

* update news

* fix text

* version up

* change news.md

* Missings assignR (#178)

* New feature for AssignR: Imputatiopn of missings

* Missings assign r refactored (#179)

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* #164 update description for custom maps

* Feat/apply model import from data tools (#184)

* exchange downUploadButton module with importData and downloadModel modules

* add config yaml

* fix news.md

* Feat/119 catch outer errors (#187)

* add tryCatch to wrappers

* version up

* fix version

* fix alignment

* update skin name

* Operator multi (#185)

* New feature for AssignR: Imputatiopn of missings

* add missing package to description

* extract functions "addUncertainty", "imputeMissingValues" and "notEmpty"

* format

* fix sprintf

* multi batch user defined maps

* fix

* merge changes from beta (#186)

* new versioning

* add news, fix duplicate uid name

* fix bugs, global vars

---------

Co-authored-by: Antonia Runge <antonia.runge@inwt-statistics.de>

* Update cosign-installer version

* apply functions from IsoMemo instead of DataTools (#189)

* apply functions from IsoMemo instead of DataTools

* apply getFields instead of getMappingTable

* fix parameter name

* use IsoMemo::getData instead of DataTools::getRemoteData

* version up

* version up

* inc req DataTools version

* fix version

* apply has_internet before call to API

* add has_internet

* catch empty cases of mappingTable, reload mappingTable

* apply reloadMappingTable

* remove comment

* catch error if internet disappears

* new version of DataTools

* fix missing data after model upload

* inc upload size

---------

Co-authored-by: Lukas Fuchs <86153843+f-lukas@users.noreply.github.com>
Co-authored-by: Jan A <jan.abel@inwt-statistics.de>
Co-authored-by: Marcus Groß <marcus.gross@inwt-statistics.de>
Co-authored-by: Jan Abel <106665518+jan-abel-inwt@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@arunge arunge

Labels
bug Something isn't working Support: Bug fixing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@arunge