Change remotes to pharmaverse repositories

[shajoezhu]

Updated package remotes to use pharmaverse repositories.

[github-actions]

Unit Tests Summary

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

Results for commit aa3700c.

♻️ This comment has been updated with latest results.

[github-actions]

Unit Test Performance Difference

Test suite performance difference

Test Suite	$Status$	Time on main	$±Time$	$±Tests$	$±Skipped$	$±Failures$	$±Errors$
pagination_table	💚	$14.34$	$-3.23$	$-9$	$+6$	$0$	$0$
table_aet02	💚	$8.49$	$-1.11$	$0$	$+12$	$0$	$0$
table_aet03	💚	$5.15$	$-1.11$	$0$	$+1$	$0$	$0$
table_aet04	💚	$12.99$	$-1.86$	$0$	$+8$	$0$	$0$
table_aet04_pi	💚	$6.91$	$-1.01$	$0$	$+9$	$0$	$0$
table_ent	💚	$16.05$	$-3.57$	$0$	$+4$	$0$	$0$
table_lbt05	💚	$7.25$	$-1.30$	$0$	$+3$	$0$	$0$
table_lbt14	💚	$32.93$	$-4.50$	$0$	$+5$	$0$	$0$
tpk02part1	💚	$8.73$	$-1.80$	$0$	$+1$	$0$	$0$
tpk02part2	💚	$8.83$	$-1.42$	$0$	$+1$	$0$	$0$
tsfae02a	💚	$9.76$	$-1.88$	$0$	$+1$	$0$	$0$
tsfae03a	💚	$8.05$	$-1.65$	$0$	$+1$	$0$	$0$
tsfae04a	💚	$5.72$	$-1.00$	$0$	$+1$	$0$	$0$
tsfae06a	💚	$5.91$	$-1.07$	$0$	$+1$	$0$	$0$
tsfae06b	💚	$18.76$	$-3.63$	$0$	$+1$	$0$	$0$
tsfae16part1	💚	$10.60$	$-1.57$	$0$	$+1$	$0$	$0$
tsfae16part2	💚	$10.74$	$-2.14$	$0$	$+1$	$0$	$0$
tsfae21apart1	💚	$8.34$	$-1.31$	$0$	$+1$	$0$	$0$
tsfae21apart2	💚	$8.42$	$-1.74$	$0$	$+1$	$0$	$0$
tsfae21apart3	💚	$8.50$	$-1.65$	$0$	$+1$	$0$	$0$
tsfae21apart4	💚	$8.54$	$-1.70$	$0$	$+1$	$0$	$0$
tsfae21bpart1	💚	$11.35$	$-1.82$	$0$	$+1$	$0$	$0$
tsfae21bpart2	💚	$11.15$	$-2.21$	$0$	$+1$	$0$	$0$
tsfae21bpart3	💚	$11.27$	$-2.14$	$0$	$+1$	$0$	$0$
tsfae21bpart4	💚	$11.54$	$-2.24$	$0$	$+1$	$0$	$0$
tsfae21d	💚	$7.99$	$-1.49$	$0$	$+1$	$0$	$0$
tsfae22apart1	💚	$9.95$	$-2.02$	$0$	$+1$	$0$	$0$
tsfae22apart2	💚	$10.00$	$-1.72$	$0$	$+1$	$0$	$0$
tsfae22apart3	💚	$10.06$	$-1.78$	$0$	$+1$	$0$	$0$
tsfae22apart4	💚	$9.69$	$-1.59$	$0$	$+1$	$0$	$0$
tsfae22bpart1	💚	$6.00$	$-1.09$	$0$	$+1$	$0$	$0$
tsfae22bpart2	💚	$6.07$	$-1.14$	$0$	$+1$	$0$	$0$
tsfae22cpart1	💚	$8.68$	$-1.60$	$0$	$+1$	$0$	$0$
tsfae22cpart2	💚	$8.87$	$-1.83$	$0$	$+1$	$0$	$0$
tsfae22cpart3	💚	$8.57$	$-1.44$	$0$	$+1$	$0$	$0$
tsfae22cpart4	💚	$8.60$	$-1.76$	$0$	$+1$	$0$	$0$
tsfae24fpart1	💚	$12.67$	$-2.46$	$0$	$+1$	$0$	$0$
tsfae24fpart2	💚	$12.23$	$-2.19$	$0$	$+1$	$0$	$0$
tsfae24fpart3	💚	$12.34$	$-2.45$	$0$	$+1$	$0$	$0$
tsfecg01apart2	💚	$5.06$	$-1.10$	$0$	$+1$	$0$	$0$
tsfecg01part1	💚	$9.13$	$-1.35$	$0$	$+1$	$0$	$0$
tsfecg01part2	💚	$9.22$	$-1.68$	$0$	$+1$	$0$	$0$
tsflab01apart1	💚	$15.29$	$-2.32$	$0$	$+1$	$0$	$0$
tsflab01apart2	💚	$15.06$	$-3.37$	$0$	$+1$	$0$	$0$
tsflab01apart3	💚	$15.61$	$-3.74$	$0$	$+1$	$0$	$0$
tsflab01part1	💚	$6.47$	$-1.34$	$0$	$+1$	$0$	$0$
tsflab01part2	💚	$6.45$	$-1.37$	$0$	$+1$	$0$	$0$
tsflab03a	💚	$10.63$	$-1.83$	$0$	$+1$	$0$	$0$
tsflab07	💚	$17.98$	$-1.79$	$0$	$+1$	$0$	$0$
tsfvit01apart1	💚	$6.93$	$-1.10$	$0$	$+1$	$0$	$0$
tsfvit01apart2	💚	$7.07$	$-1.20$	$0$	$+1$	$0$	$0$
tsfvit01apart3	💚	$7.07$	$-1.11$	$0$	$+1$	$0$	$0$
tsfvit01apart4	💚	$7.20$	$-1.32$	$0$	$+1$	$0$	$0$
tsfvit01part2	💚	$4.05$	$-1.16$	$0$	$+1$	$0$	$0$
tsicm04	💚	$4.44$	$-1.06$	$0$	$+1$	$0$	$0$
tsicm05	💚	$7.64$	$-1.73$	$0$	$+1$	$0$	$0$
tsicm06	💚	$6.94$	$-1.10$	$0$	$+1$	$0$	$0$
tsids02a	💚	$7.92$	$-1.16$	$0$	$+1$	$0$	$0$

Additional test case details

Test Suite	$Status$	Time on main	$±Time$	Test Case
pagination_table	💚	$5.07$	$-1.20$	Pagination_works_also_if_table_is_decorated
pagination_table	💚	$3.75$	$-1.03$	Pagination_works_for_page_types
table_aet03	💚	$5.15$	$-1.11$	AET03_variant_1_is_produced_correctly
table_ent	💚	$4.63$	$-1.11$	ENT01_IT_is_produced_correctly
table_ent	💚	$4.87$	$-1.11$	ENT02_IT_is_produced_correctly
table_lbt14	💚	$6.55$	$-1.19$	LBT14_variant_2_LOW_works_as_expected
table_lbt14	💚	$8.78$	$-1.92$	LBT14_variant_5_HIGH_with_fillings_works_as_expected
tpk02part1	💚	$8.73$	$-1.80$	tpk02part1of2
tpk02part2	💚	$8.83$	$-1.42$	tpk02part2of2
tsfae02a	💚	$9.76$	$-1.88$	tsfae02a
tsfae03a	💚	$8.05$	$-1.65$	tsfae03a
tsfae04a	💚	$5.72$	$-1.00$	tsfae04a
tsfae06a	💚	$5.91$	$-1.07$	tsfae06a
tsfae06b	💚	$18.76$	$-3.63$	tsfae06b
tsfae16part1	💚	$10.60$	$-1.57$	tsfae16part1of2
tsfae16part2	💚	$10.74$	$-2.14$	tsfae16part2of2
tsfae21apart1	💚	$8.34$	$-1.31$	tsfae21apart1of4
tsfae21apart2	💚	$8.42$	$-1.74$	tsfae21apart2of4
tsfae21apart3	💚	$8.50$	$-1.65$	tsfae21apart3of4
tsfae21apart4	💚	$8.54$	$-1.70$	tsfae21apart4of4
tsfae21bpart1	💚	$11.35$	$-1.82$	tsfae21bpart1of4
tsfae21bpart2	💚	$11.15$	$-2.21$	tsfae21bpart2of4
tsfae21bpart3	💚	$11.27$	$-2.14$	tsfae21bpart3of4
tsfae21bpart4	💚	$11.54$	$-2.24$	tsfae21bpart4of4
tsfae21d	💚	$7.99$	$-1.49$	tsfae21d
tsfae22apart1	💚	$9.95$	$-2.02$	tsfae22apart1of4
tsfae22apart2	💚	$10.00$	$-1.72$	tsfae22apart2of4
tsfae22apart3	💚	$10.06$	$-1.78$	tsfae22apart3of4
tsfae22apart4	💚	$9.69$	$-1.59$	tsfae22apart4of4
tsfae22bpart1	💚	$6.00$	$-1.09$	tsfae22bpart1of2
tsfae22bpart2	💚	$6.07$	$-1.14$	tsfae22bpart2of2
tsfae22cpart1	💚	$8.68$	$-1.60$	tsfae22cpart1of4
tsfae22cpart2	💚	$8.87$	$-1.83$	tsfae22cpart2of4
tsfae22cpart3	💚	$8.57$	$-1.44$	tsfae22cpart3of4
tsfae22cpart4	💚	$8.60$	$-1.76$	tsfae22cpart4of4
tsfae24fpart1	💚	$12.67$	$-2.46$	tsfae24fpart1of3
tsfae24fpart2	💚	$12.23$	$-2.19$	tsfae24fpart2of3
tsfae24fpart3	💚	$12.34$	$-2.45$	tsfae24fpart3of3
tsfecg01apart2	💚	$5.06$	$-1.10$	tsfecg01a
tsfecg01part1	💚	$9.13$	$-1.35$	tsfecg01
tsfecg01part2	💚	$9.22$	$-1.68$	tsfecg01
tsflab01apart1	💚	$15.29$	$-2.32$	tsflab01a
tsflab01apart2	💚	$15.06$	$-3.37$	tsflab01a
tsflab01apart3	💚	$15.61$	$-3.74$	tsflab01a
tsflab01part1	💚	$6.47$	$-1.34$	tsflab01
tsflab01part2	💚	$6.45$	$-1.37$	tsflab01
tsflab03a	💚	$10.63$	$-1.83$	tsflab03a
tsflab07	💚	$17.98$	$-1.79$	tsflab07
tsfvit01apart1	💚	$6.93$	$-1.10$	tsfvit01a
tsfvit01apart2	💚	$7.07$	$-1.20$	tsfvit01a
tsfvit01apart3	💚	$7.07$	$-1.11$	tsfvit01a
tsfvit01apart4	💚	$7.20$	$-1.32$	tsfvit01a
tsfvit01part2	💚	$4.05$	$-1.16$	tsfvit01
tsicm04	💚	$4.44$	$-1.06$	tsicm04
tsicm05	💚	$7.64$	$-1.73$	tsicm05
tsicm06	💚	$6.94$	$-1.10$	tsicm06
tsids02a	💚	$7.92$	$-1.16$	tsids02a

Results for commit 0219a23

♻️ This comment has been updated with latest results.

[gmbecker]

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]

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

[gmbecker]

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]

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

[munoztd0]

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

Hey guys , I am back, these are indeed expected changes from dev that we didn't put in the tests yet. Here I implemented the very small change. Let me know if you want it in in another PR or anything @shajoezhu .

[munoztd0]

lgtm