search

marccanby / linguiphyr

Linguistic Phylogenetic Analysis in R
Apache License 2.0
2 stars 0 forks source link

consider selectively silencing warnings rather than all of them. #6

Closed SimonGreenhill closed 10 months ago

SimonGreenhill commented 10 months ago

Brute-force silencing all warnings in server.R because of a noisy/problematic package is probably a bad idea as it'll make debugging harder in future.

Can you selectively suppress the calls that generate the warnings e.g.

suppressWarnings({
   plotly::whatever()
})
marccanby commented 10 months ago

I have fixed this by silencing just the warning in the renderPlotly function. This is now pushed on the main branch. Hence, I am no longer silencing all warnings as I was before.

SimonGreenhill commented 10 months ago

great, thanks!