Bugfix/351 residual flipping #354
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button.
|
This PR is being deployed to Railway 🚅 Lite: ◻️ REMOVED |
This was
linked to
issues
Dec 18, 2023
tmelliott
added a commit
that referenced
this pull request
Dec 20, 2023
* fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com>
tmelliott
added a commit
that referenced
this pull request
Dec 21, 2023
* fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version (#357) * dynamic config (#358) Co-authored-by: Charco Hui <charcohui93@gmail.com> * update version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> Co-authored-by: Charco Hui <charcohui93@gmail.com>
tmelliott
added a commit
that referenced
this pull request
Dec 21, 2023
* Release/2023.11.9000 (#356) * fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> * update version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com>
Merged
tmelliott
added a commit
that referenced
this pull request
Dec 22, 2023
* fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version (#357) * dynamic config (#358) Co-authored-by: Charco Hui <charcohui93@gmail.com> * Release/2023.12 -> dev (#361) * Release/2023.11.9000 (#356) * fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> * update version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> * remove multivariate module * activate multivariate * add iNZightMultivariate to install * Update NEWS * update default env * Format and tidy code (#362) * fix(ui): Update mark_html() call to generate HTML in-place - remove HTML files - clear some other files - update gitignore * Format all .R files * Remove commented code chunks * update news --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> Co-authored-by: Charco Hui <charcohui93@gmail.com>
tmelliott
added a commit
that referenced
this pull request
Dec 22, 2023
* fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version (#357) * dynamic config (#358) Co-authored-by: Charco Hui <charcohui93@gmail.com> * Release/2023.12 -> dev (#361) * Release/2023.11.9000 (#356) * fix release generator script * Track 'iNZightVIT/Lite2' changes (#293) * Create new branch tracking iNZightVIT/Lite2 changes * fix conflicts * modes? * ui fix * fix import * fix lite2 env variable * prevent dataset to show before confirm * fix url import * remove comments * fix rdata data selection bug * fix(server): LITE_ -> CAS_; bug in decryption code; import wkb, openssl - Replace LITE_ with CAS_ (for now) given this is still a hard-coded solution - fix parentheses in the raw2char / aes_cbc_decrypt logic - remove wkb code and just import the wkb package (minimal overhead) - remove library calls, use namespacing instead (openssl, wkb) * wrap LITE2 if * add function sample_if_lite2 * fix lite2 UI * fix datetime --------- Co-authored-by: Charco Hui <phui285@gmail.com> * lite_config * fix(server): Pre-build df and assign to rvalues It seems that row.names() and colnames() are triggering the reload, and by refactoring slightly the loading-loop seems to go away. * prevent reading ENV unless required * delim option on failed import (#340) * remove keys after reading data * rename plot_selector (#342) * Update NEWS.md * (shared): Generalize handling of config Store config as an unaltered list; display version info on landing page. Minor refactor of some parameter handling. * fix(shared): Store config in session$userData Store LITE_VERSION and LITE_CONFIG in the session$userData object * fix(shared): Find remaining variables to prefix * fix(ui): Remove export data for CAS * fix(shared): Uncommnet export data component * fix(shared): Disable menu options in CAS mode * remove gpairs dependency * update news * remove gpairs from imports (closes #25) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * Remove log error message (#348) * fix(server): Set shiny host to 0.0.0.0 * update the correct dockerfile * fix(server): Disable loading R directory (does not exist) * fix arg * fix(server): Suppress warnings produced by iNZightMapRegions() * additional import options (#353) Co-authored-by: Mona Lisa <email@example.com> * feat(ui): #352 Track disconnects (#355) * feat(ui): #352 Track disconnects Closes: #352 * Bugfix/351 residual flipping (#354) * fix(ui): 351 Use observeEvent instead of observe and isolate After swapping v1 and v2, some reactive connection awakens and gets triggered by the save residuals button. * remove terourou * Data lifecycle management (#349) * fix(server): #304 Delete data after importing * bump version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> * update version --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> * remove multivariate module * activate multivariate * add iNZightMultivariate to install * Update NEWS * update default env * Format and tidy code (#362) * fix(ui): Update mark_html() call to generate HTML in-place - remove HTML files - clear some other files - update gitignore * Format all .R files * Remove commented code chunks * update news --------- Co-authored-by: Charco Hui <phui285@gmail.com> Co-authored-by: Charco Hui <charco.hui@auckland.ac.nz> Co-authored-by: Mona Lisa <email@example.com> Co-authored-by: Charco Hui <charcohui93@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.