Conversation
typoumet
left a comment
typoumet
left a comment
There was a problem hiding this comment.
I have added a number of comments for your consideration.
| result_tooltip_vars = NULL, | ||
| trt_annotation_vars = NULL, | ||
| trt_annotation_x = NULL, | ||
| result_annotation_vars = NULL, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| \item{trt_annotation_x}{`[numeric(1)]` X-axis position for a left-aligned fixed exposure annotation column. | ||
| If NULL, the exposure annotation is placed after each exposure bar end.} | ||
|
|
||
| \item{result_annotation_vars}{`[character(n)]` Character vector of variables for fixed text annotation on response points.} |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| x_rng_upper = NULL, | ||
| trt_annotation_vars = NULL, | ||
| trt_annotation_x = NULL, | ||
| result_annotation_vars = NULL, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| \item{trt_annotation_x}{`[numeric(1)]` X-axis position for a left-aligned fixed exposure annotation column. | ||
| If NULL, the exposure annotation is placed after each exposure bar end.} | ||
|
|
||
| \item{result_annotation_vars}{`[character(n)]` Character vector of variables for fixed text annotation on response points.} |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| result_tooltip_vars = NULL, | ||
| trt_annotation_vars = NULL, | ||
| trt_annotation_x = NULL, | ||
| result_annotation_vars = NULL, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| result_tooltip_vars = result_tooltip_vars, | ||
| trt_annotation_vars = trt_annotation_vars, | ||
| trt_annotation_x = trt_annotation_x, | ||
| result_annotation_vars = result_annotation_vars, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| #' @param trt_annotation_vars `[character(n)]` Character vector of variables for fixed text annotation on exposure. | ||
| #' @param trt_annotation_x `[numeric(1)]` X-axis position for a left-aligned fixed exposure annotation column. | ||
| #' If NULL, the exposure annotation is placed after each exposure bar end. | ||
| #' @param result_annotation_vars `[character(n)]` Character vector of variables for fixed text annotation on response points. |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| result_tooltip_vars = NULL, | ||
| trt_annotation_vars = NULL, | ||
| trt_annotation_x = NULL, | ||
| result_annotation_vars = NULL, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| checkmate::assert_character(result_tooltip_vars, null.ok = TRUE) | ||
| checkmate::assert_character(trt_annotation_vars, null.ok = TRUE) | ||
| checkmate::assert_number(trt_annotation_x, null.ok = TRUE) | ||
| checkmate::assert_character(result_annotation_vars, null.ok = TRUE) |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
| result_tooltip_vars = result_tooltip_vars, | ||
| trt_annotation_vars = trt_annotation_vars, | ||
| trt_annotation_x = trt_annotation_x, | ||
| result_annotation_vars = result_annotation_vars, |
There was a problem hiding this comment.
As discussed with Ming, this argument is no longer required, and it should be removed.
|
As suggested, the |
2 participants
No description provided.