Skip to content

Commit 1b874b6

Browse files
committed
Bug fix to deal with only NA values along one river
1 parent 70bee15 commit 1b874b6

2 files changed

Lines changed: 14 additions & 9 deletions

File tree

R/extend_river_table.R

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ extend_riverTable <- function(
4747
data_table <- aggregated_data[aggregated_data$qsimVis_ID == river_id &
4848
!is.na(aggregated_data$qsimVis_ID),]
4949

50-
if(nrow(data_table) > 0L){
50+
if(nrow(data_table) > 0L & any(!is.na(data_table[[varName]]))){
5151
# apply results to closest verknet node, if not already defined
5252
km_verknet <- river_table$km
5353
for(i in seq_len(nrow(data_table))){

R/value_to_classes.R

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,17 +12,22 @@
1212
#'
1313
#' @export
1414
#'
15-
value_to_classes <- function(river_list, classBreaks, colorVector = NULL){
16-
15+
value_to_classes <- function(
16+
river_list, classBreaks, colorVector = NULL
17+
){
1718
output_list <- lapply(names(river_list), function(N){
1819
r <- river_list[[N]]
1920
x <- r$data
20-
x$value_class <- cut(
21-
x = x$value,
22-
breaks = classBreaks,
23-
include.lowest = TRUE,
24-
ordered_result = TRUE
25-
)
21+
x$value_class <- if(any(!is.na(x$value))){
22+
cut(
23+
x = x$value,
24+
breaks = classBreaks,
25+
include.lowest = TRUE,
26+
ordered_result = TRUE
27+
)
28+
} else {
29+
NA
30+
}
2631
list("data" = x,
2732
"pp" = r$pp
2833
)

0 commit comments

Comments
 (0)