Build NOTE: no visible binding for global variable about pacta.multi.loanbook.analysis HOT 7 CLOSED

using group_id <- name_abcd <- sector <- activity_unit <- region <- scenario_source <- scenario <- year <- NULL inside the function fill_missing_direction() is one hack to get around the build notes (see tidyverse/tidyr#971 (comment))

the other is to add these to imports.R (see tidyverse/tidyr#971 (comment))

utils::globalVariables("group_id")
utils::globalVariables("name_abcd")
utils::globalVariables("sector")
utils::globalVariables("activity_unit")
utils::globalVariables("region")
utils::globalVariables("scenario_source")
utils::globalVariables("scenario")
utils::globalVariables("year")

seems like these will be the only options until tidyverse figures out and implements a very different strategy (see tidyverse/tidyr#971 (comment))

from pacta.multi.loanbook.analysis.

I don't really love either, but would probably lean towards the first option as it seems like it does not play with global variables across the entire package for no reason.

from pacta.multi.loanbook.analysis.

I have not been able to figure out how to get rid of the NOTE relating to function fill_missing_direction(). Any attempt I have made at fixing the no visible binding for global variable issue in two nested tidyr functions has lead to ERRORs in R CMD... any ideas?

from pacta.multi.loanbook.analysis.

Wanna schedule a pair program with me on Thursday? Need a bit more context to help 😊

from pacta.multi.loanbook.analysis.

I know I'm late to the party here, but is there a reason to not use quoted strings, rather than bare column names?

from pacta.multi.loanbook.analysis.

I know I'm late to the party here, but is there a reason to not use quoted strings, rather than bare column names?

tried that, it leads to errors

from pacta.multi.loanbook.analysis.

I know I'm late to the party here, but is there a reason to not use quoted strings, rather than bare column names?

because it uses "<[data-masking]>"
https://tidyr.tidyverse.org/reference/expand.html
https://tidyr.tidyverse.org/reference/tidyr_data_masking.html

from pacta.multi.loanbook.analysis.