Skip to content

Commit 1b714ef

Browse files
authored
Merge pull request #80 from angelina-momin/fix-no-codechecks-column
2 parents 4ac2111 + bb52e18 commit 1b714ef

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

R/utils_render_table_codecheckers.R

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,13 @@ create_all_codecheckers_table <- function(register_table){
2222
mutate(`codechecker_name` = recode(Codechecker, !!!CONFIG$DICT_ORCID_ID_NAME))
2323

2424
# Adding no. of codechecks column
25+
# Count no. codechecks per Codechecker
26+
codecheck_counts <- register_table %>%
27+
count(Codechecker, name = "no_codechecks")
28+
29+
# Join no_codechecks column to new_table
2530
new_table <- new_table %>%
26-
group_by(Codechecker) %>%
27-
mutate(`no_codechecks` = sum(register_table$Codechecker == Codechecker))
31+
left_join(codecheck_counts, by = "Codechecker")
2832

2933
# Rename the column using the key-value pairs from the CONFIG list
3034
col_names_dict <- CONFIG$NON_REG_TABLE_COL_NAMES[["codecheckers"]]

0 commit comments

Comments
 (0)