You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@ddsjoberg , what do you think? Should this be the next task for the error messaging group?
@bundfussr sounds great! I like the suggested messaging. We can also capture the error from filter() and print our custom message followed by the message that filter() throws as well.
(Re-reading and seeing that is exactly what you suggested haha)
I am not sure if this is an issue for admiraldev. I would expect that most of the updates are in admiral (unless we implement a helper function like process_set_value_to() in admiraldev).
I was thinking we could have a function like safe_filter() that captures any errors from dplyr::filter() and gives the appropriate error message? But that really depends on how many different contexts we're trying to cover...anyway, let's discuss
@ddsjoberg , what do you think? Should this be the next task for the error messaging group?
@bundfussr sounds great! I like the suggested messaging. We can also capture the error from
filter()
and print our custom message followed by the message thatfilter()
throws as well.(Re-reading and seeing that is exactly what you suggested haha)
Originally posted by @ddsjoberg in pharmaverse/admiral#2391 (comment)
The text was updated successfully, but these errors were encountered: