Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update vignettes #67

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Update vignettes #67

wants to merge 2 commits into from

Conversation

slucey
Copy link
Member

@slucey slucey commented Sep 27, 2023

This adds the Anchovy Bay and REcosystem data sets only.

@slucey slucey linked an issue Sep 27, 2023 that may be closed by this pull request
@slucey
Copy link
Member Author

slucey commented Sep 27, 2023

I'm looking into the failing gitleaks action that is failing everytime.

This was referenced Sep 27, 2023
@slucey slucey linked an issue Sep 27, 2023 that may be closed by this pull request
@kaydin
Copy link
Collaborator

kaydin commented Oct 11, 2023

@slucey do you want the raw R code files for these examples available for the user to inspect? Or just the compiled rdata files?

@slucey
Copy link
Member Author

slucey commented Oct 13, 2023

I think just the compiled rdata files. Although I think it is fine to have the scripts that created them in the data-raw folder.

@slucey
Copy link
Member Author

slucey commented Oct 18, 2023

@kaydin I'm talking with @rklasky about unit testing and I think we definitely want the script that creates these data sets available so we can build them as part of the unit test.

@kaydin
Copy link
Collaborator

kaydin commented May 5, 2024

@rklasky I've updated all of my packages and I'm still only seeing 34 tests but no warnings/errors. Here's what is output.

==> Rcpp::compileAttributes()

  • Updated R/RcppExports.R

==> Sourcing R files in 'tests' directory

[1] "here::here: C:/Users/kerim.aydin/Work/src/sean_rpath/main_2023/Rpath"
[1] "OUTPUT_DATA_DIR: C:/Users/kerim.aydin/Work/src/sean_rpath/main_2023/Rpath/tests/testthat/data/output"
[1] "INPUT_DATA_DIR_BASELINE: C:/Users/kerim.aydin/Work/src/sean_rpath/main_2023/Rpath/tests/testthat/data/input"
[1] "INPUT_DATA_DIR_CURRENT: C:/Users/kerim.aydin/Work/src/sean_rpath/main_2023/Rpath/tests/testthat/data/input"
[1] "------------------ Rpath Object Tests ------------------"
[1] "------------------ Forced Biomass Tests (Jitter) ------------------"
[1] "------------------ Forced Biomass Tests (Stepped) ------------------"
[1] "------------------ Forced Effort Tests (Jitter) ------------------"
[1] "------------------ Forced Effort Tests (Stepped) ------------------"
[1] "Completed 34 test(s)."
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 33 ]

Tests complete

@rklasky
Copy link
Collaborator

rklasky commented May 6, 2024 via email

@kaydin
Copy link
Collaborator

kaydin commented May 8, 2024

I just ran the tests locally on my machine and I had 54 tests. How did you generate the Unit Tests? Via a push or pull request?

Ran locally using the Rstudio test button (confirmed that my local dev is synced with origin/dev)

@rklasky
Copy link
Collaborator

rklasky commented May 9, 2024 via email

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.

Clean up Dev branch Add dataset
3 participants