Skip to content

global variable reorganization#80

Merged
suz-estella merged 1 commit intodevelopmentfrom
suz-globalVars
Mar 29, 2026
Merged

global variable reorganization#80
suz-estella merged 1 commit intodevelopmentfrom
suz-globalVars

Conversation

@suz-estella
Copy link
Copy Markdown
Contributor

I've added a section to our package defining R file R/CBMutils-package.R that declares commonly used variables with utils::globalVariables between all of our functions. These were often declared in multiple files, or used in multiple files but only declared in some, etc. This will make it easy to use these variables anywhere.

This includes:

  1. data.table special variables (e.g. :=)
  2. Common spatial variables (e.g. x)
  3. Common keys (e.g. pixelIndex)
  4. Boudewyn table columns (e.g. juris_id)
  5. CBM defaults database columns (e.g. spatial_unit_id)
  6. CBM-EXN cbm_vars columns (e.g. row_idx)

We can add to this over time as we see fit.

@suz-estella suz-estella merged commit 5ae3eb6 into development Mar 29, 2026
10 of 12 checks passed
@suz-estella suz-estella deleted the suz-globalVars branch March 29, 2026 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants