Skip to content

Commit

Permalink
add model with import factors to testing
Browse files Browse the repository at this point in the history
  • Loading branch information
bl-young committed Sep 7, 2023
1 parent cad065d commit 5195143
Showing 1 changed file with 9 additions and 1 deletion.
10 changes: 9 additions & 1 deletion .github/workflows/R-CMD-check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ jobs:
config:
- {os: macOS-latest, r: 'release'}
- {os: windows-latest, r: 'release'}
#- {os: ubuntu-latest, r: 'devel', http-user-agent: 'release'}
- {os: ubuntu-latest, r: 'release'}
- {os: ubuntu-latest, r: 'oldrel-1'}

Expand Down Expand Up @@ -129,6 +128,15 @@ jobs:
useeior::print2RValidationResults(m)
shell: Rscript {0}

- name: Build and validate models - USEEIOv2.0-s-GHG_IF summary model with import factors
if: always()
run: |
require(devtools)
devtools::load_all()
m <- useeior::buildModel("USEEIOv2.0-s-GHG_IF")
# useeior::printValidationResults(m)
shell: Rscript {0}

#- name: Show testthat output
# if: always() # step will run even if previous steps fail
# run: find check -name 'testthat.Rout*' -exec cat '{}' \; || true
Expand Down

2 comments on commit 5195143

@bl-young
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jvendries I went ahead and added this model spec to the test build (I think the validation we have right now happens during model build correct so no specific additional validation is warranted yet?)

@jvendries
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct, validation happens during model build right now, but I'll move it to validation functions soon.

Please sign in to comment.