Skip to content

Change remotes to pharmaverse repositories - #244

Open
shajoezhu wants to merge 3 commits into
mainfrom
shajoezhu-patch-1
Open

Change remotes to pharmaverse repositories#244
shajoezhu wants to merge 3 commits into
mainfrom
shajoezhu-patch-1

Conversation

@shajoezhu

Copy link
Copy Markdown
Contributor

Updated package remotes to use pharmaverse repositories.

Updated package remotes to use pharmaverse repositories.

Signed-off-by: Joe Zhu <sha.joe.zhu@gmail.com>
@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files  274 suites   14m 43s ⏱️
411 tests 219 ✅ 192 💤 0 ❌
676 runs  267 ✅ 409 💤 0 ❌

Results for commit fca23ac.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
table_ent 💚 $16.05$ $-1.18$ $0$ $+4$ $0$ $0$
tsflab01apart3 💚 $15.61$ $-1.14$ $0$ $+1$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
tsflab01apart3 💚 $15.61$ $-1.14$ tsflab01a

Results for commit 283548a

♻️ This comment has been updated with latest results.

Signed-off-by: Joe Zhu <sha.joe.zhu@gmail.com>
@shajoezhu
shajoezhu enabled auto-merge (squash) August 3, 2026 05:12
@gmbecker

gmbecker commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Apparently my write access for this repo (scda.test) has been removed, I tried to push a modification to your branch. please point to junco@dev (instead of main as it is now) in this branch and rerun the tests @shajoezhu

@shajoezhu

Copy link
Copy Markdown
Contributor Author

Apparently my write access for this repo (scda.test) has been removed, I tried to push a modification to your branch. please point to junco@dev (instead of main as it is now) in this branch and rerun the tests @shajoezhu

hi @gmbecker , your access has not changed, can you try again? you still have maintainer role i just checked

Comment thread DESCRIPTION Outdated
Signed-off-by: Joe Zhu <sha.joe.zhu@gmail.com>
@gmbecker

gmbecker commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Looks like that cleared up the majority of the failures, there's one left that I'm having trouble recreating locally:

> source("~/gabe/checkedout/nestpkgs/scda.test/tests/testthat/setup.R")

Attaching package: ‘lubridate’

The following objects are masked from ‘package:base’:

    date, intersect, setdiff, union

Reducing levels of adae_pharmaverse for variable AEDECOD: keeping 12 levels out of 242 levels. Total kept (%): 38
Reducing levels of adlb_pharmaverse for variable PARAM: keeping 15 levels out of 47 levels. Total kept (%): 42
> source("~/gabe/checkedout/nestpkgs/scda.test/tests/testthat/setup-jnj.R")
Reducing levels of adae_jnj for variable AEDECOD: keeping 33 levels out of 246 levels. Total kept (%): 62.7
> test_file("~/gabe/checkedout/nestpkgs/scda.test/tests/testthat/test-tsfae13.R")
Reducing levels of adae_jnj for variable AEDECOD: keeping 33 levels out of 246 levels. Total kept (%): 62.7
Reducing levels of adae_jnj for variable AEDECOD: keeping 33 levels out of 246 levels. Total kept (%): 62.7
Reducing levels of adae_pharmaverse for variable AEDECOD: keeping 12 levels out of 242 levels. Total kept (%): 38
Reducing levels of adlb_pharmaverse for variable PARAM: keeping 15 levels out of 47 levels. Total kept (%): 42

══ Testing test-tsfae13.R ══════════════════════════════════════════════════════
[ FAIL 0 | WARN 0 | SKIP 0 | PASS 0 ]Write Path:/var/folders/pr/sc_nk3d92wxbs4dh53gf53wc0000gn/T//Rtmp2GOwbn/TSFAE13

will continue to investigate. And I'll look at the access thing as well.

@gmbecker

gmbecker commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

I can now recreate the last failure it looks like an expected change but I need to confirm with the team. will do so tomorrow morning

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.

2 participants