Skip to content

Commit

Permalink
implement warning handling for too many rows in mini-unit table
Browse files Browse the repository at this point in the history
  • Loading branch information
ha0ye committed Feb 26, 2024
1 parent 886d698 commit 23c47cc
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 2 deletions.
6 changes: 5 additions & 1 deletion R/interface_docs.R
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,11 @@ extract_roadmap_mini_units <- function(content)
tt <- table(table_cells$doc_index)
mini_unit_doc_index <- max(as.numeric(names(tt)[tt == max(tt)]))
mini_unit_table <- subset(table_cells, doc_index == mini_unit_doc_index)
stopifnot(NROW(mini_unit_table) == 5 * (num_mini_units + 1))
stopifnot(NROW(mini_unit_table) >= 5 * (num_mini_units + 1))
if (NROW(mini_unit_table) > 5 * (num_mini_units + 1))
{
warning("Table of Mini-Units has too many rows")
}
mini_unit_table$text[seq(from = 2, length.out = num_mini_units)]
}

Expand Down
5 changes: 4 additions & 1 deletion R/processing.R
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,10 @@ sync_statuses <- function(roadmap_url, tracker_url, tracker_sheet, unit_id = "")

}, error = function(e) {
log_action(e$message, url = roadmap_url, type = "ERROR")
})
}, warning = function(w) {
log_action(w$message, url = roadmap_url, type = "WARNING")
}
)

invisible()
}
Expand Down

0 comments on commit 23c47cc

Please sign in to comment.