Skip to content

Commit 958fc7c

Browse files
authored
Merge branch 'main' into 886-r-script-with-the-custom-time-duplicate-management
2 parents 8d47fc0 + f3a40ab commit 958fc7c

File tree

3 files changed

+14
-12
lines changed

3 files changed

+14
-12
lines changed

NEWS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,7 @@ identify difference between missing and not requested (#934)
5656
* Creation of intervals reworked to prevent doses being combined if no samples are taken post dose (#963)
5757
* Parameter selection no longer resets after changes to NCA setup and slope selector- apart from changes to analyte and pcspec that change the study types detected (#1008)
5858
* Prevent a crash when selecting already defined identity variables (i.e, `DOSETRT`) for the `Additional Grouping Variables` in the `Mapping Tab` (#1060)
59+
* Filtering will now correctly also affect all the input widgets in NCA setup (#1092)
5960

6061
# aNCA 0.1.0
6162

inst/shiny/modules/tab_nca.R

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -164,11 +164,12 @@ tab_nca_server <- function(id, pknca_data, extra_group_vars, settings_override)
164164
remove_pp_not_requested()
165165
},
166166
warning = function(w) {
167-
log_warn("Warning during NCA calculation: {conditionMessage(w)}")
168-
pknca_warn_env$warnings <- append(
169-
pknca_warn_env$warnings,
170-
.parse_pknca_warning(w)
171-
)
167+
parsed <- .parse_pknca_warning(w)
168+
if (!is.null(parsed)) {
169+
log_warn("Warning during NCA calculation: {conditionMessage(w)}")
170+
pknca_warn_env$warnings <- append(pknca_warn_env$warnings, parsed)
171+
}
172+
invokeRestart("muffleWarning")
172173
})
173174

174175
# Display unique warnings thrown by PKNCA run.

inst/shiny/modules/tab_nca/setup/settings.R

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -226,11 +226,10 @@ settings_server <- function(id, data, adnca_data, settings_override) {
226226
# Additional settings (PCSPEC and ATPTREF handled later)
227227
updateSelectInput(session, inputId = "method", selected = settings$method)
228228

229-
if (!is.null(settings$bioavailability) &&
230-
adnca_data()$dose$data$std_route %>%
231-
unique() %>%
232-
length() > 1) {
233-
updateSelectInput(session,
229+
dose_routes <- unique(adnca_data()$dose$data$std_route)
230+
if (!is.null(settings$bioavailability) && length(dose_routes) > 1) {
231+
updateSelectInput(
232+
session,
234233
inputId = "bioavailability",
235234
selected = settings$bioavailability
236235
)
@@ -295,8 +294,9 @@ settings_server <- function(id, data, adnca_data, settings_override) {
295294
})
296295

297296
# Update Downstream Inputs (Profile & Specimen)
298-
observeEvent(input$select_analyte, {
299-
req(data())
297+
# React to both analyte selection and data changes
298+
observeEvent(c(input$select_analyte, data()), {
299+
req(data(), input$select_analyte)
300300

301301
settings <- settings_override()
302302

0 commit comments

Comments
 (0)