Skip to content

Commit 9784705

Browse files
committed
fix bug due to unrequired local
1 parent 3a1110f commit 9784705

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

R/review.R

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -179,8 +179,7 @@ REV_load_annotation_info <- function(folder_contents, review, dataset_lists, fsa
179179
})
180180
base_info <- RS_load(contents, deltas) # TODO? Call this RS_load_memory and write an RS_load() that works with fnames
181181
dataset_hash <- RS_hash_data_frame(dataset)
182-
if (!identical(dataset_hash, base_info[["contents_hash"]])) {
183-
local({
182+
if (!identical(dataset_hash, base_info[["contents_hash"]])) {
184183
new_delta <- RS_compute_delta_memory(state = base_info, dataset)
185184
deltas[[length(deltas) + 1]] <- new_delta
186185
base_info <- RS_load(contents, deltas)
@@ -196,8 +195,7 @@ REV_load_annotation_info <- function(folder_contents, review, dataset_lists, fsa
196195
contents = new_delta
197196
)
198197
)
199-
message(sprintf("Produced new delta %s", fname))
200-
})
198+
message(sprintf("Produced new delta %s", fname))
201199
}
202200
} else {
203201
contents <- RS_compute_base_memory(dataset_review_name, dataset, id_vars, tracked_vars)
@@ -234,6 +232,7 @@ REV_load_annotation_info <- function(folder_contents, review, dataset_lists, fsa
234232
})
235233

236234
# Compute reverse mapping (which is a more useful representation for the running app)
235+
browser()
237236
dataset_to_state_row_mapping <- local({ # TODO: Is this the right name?
238237
res <- integer(length(state_to_dataset_row_mapping))
239238
res[state_to_dataset_row_mapping] <- seq_along(state_to_dataset_row_mapping)

0 commit comments

Comments
 (0)