Skip to content

Install shiny and shinyBS so srtsim can load - #21

Closed
rcannood wants to merge 2 commits into
mainfrom
fix/srtsim-dependencies
Closed

Install shiny and shinyBS so srtsim can load#21
rcannood wants to merge 2 commits into
mainfrom
fix/srtsim-dependencies

Conversation

@rcannood

Copy link
Copy Markdown
Member

srtsim produced no results at all in run_2026-07-11_18-00-02 -- 0 out of 390, failing on every dataset with

Error: package or namespace load failed for 'SRTsim' ...
 there is no package called 'httpuv'

It never got as far as reading its input, and errorStrategy 'ignore' turned that into a silent absence rather than a failure.

SRTsim imports shiny and shinyBS, neither of which ended up in the image, and httpuv is what shiny pulls in. Checked inside the image:

MISSING: shiny, shinyBS

Installing both explicitly makes library(SRTsim) load.

Checklist before requesting a review

  • I have performed a self-review of my code

  • Check the correct box. Does this PR contain:

    • Breaking changes
    • New functionality
    • Major changes
    • Minor changes
    • Bug fixes
  • Proposed changes are described in the CHANGELOG.md

  • CI Tests succeed and look good!

rcannood added 2 commits July 28, 2026 14:20
srtsim produced no results at all in run_2026-07-11_18-00-02 -- 0 of 390,
failing on every dataset with

  Error: package or namespace load failed for 'SRTsim' ...
   there is no package called 'httpuv'

It never got as far as reading its input. SRTsim imports shiny and shinyBS,
neither of which ended up in the image, and httpuv is what shiny pulls in.
Installing both explicitly makes library(SRTsim) load.
@rcannood

rcannood commented Jul 28, 2026

Copy link
Copy Markdown
Member Author

This is no longer an issue, so closing.

@rcannood rcannood closed this Jul 28, 2026
@rcannood
rcannood deleted the fix/srtsim-dependencies branch July 28, 2026 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant