insightsengineering / teal.modules.clinical

Provides teal modules for the standard clinical trials outputs
https://insightsengineering.github.io/teal.modules.clinical/
Other
32 stars 17 forks source link

nestcolor package dependency issue #1199

Closed zwu-cims closed 4 months ago

zwu-cims commented 4 months ago

What happened?

Since nestcolor is not a reverse import for teal, there are dependency errors when using some modules. Could you add it to import instead of suggest?

sessionInfo()

No response

Relevant log output

there is no package called 'nestcolor'
when evaluating qenv code:
all_colors <- stats::setNames(nestcolor::color_palette(length(full_vita), "stream"), full_vita)

Code of Conduct

Contribution Guidelines

Security Policy

shajoezhu commented 4 months ago

dear @zwu-cims , thanks for raising this. I was wondering which specific issue have you experienced the issue? thanks!

zwu-cims commented 4 months ago

When I was trying to run the patient-profile demo on my own R studio, the vitals tab showed this error message.

ayogasekaram commented 4 months ago

When I was trying to run the patient-profile demo on my own R studio, the vitals tab showed this error message.

Hey there, I think the screenshot is missing in your comment. Would you be able to attach it or paste the code + error message in a formatted comment? Thank you!

shajoezhu commented 4 months ago

When I was trying to run the patient-profile demo on my own R studio, the vitals tab showed this error message.

hi @zwu-cims , I think you are referring to teal.gallery app right?

in which case, you should use renv to reinstate the environment, where nestcolor is included, https://github.com/insightsengineering/teal.gallery/blob/main/patient-profile/renv.lock#L1640

zwu-cims commented 4 months ago

Yes, the previous renv I used was not correct. Thank you so much for your comment!

shajoezhu commented 4 months ago

cool man. I will close this issue for now. Thanks!