Gilead-BioStats / gsm

Good Statistical Monitoring R Package
https://gilead-biostats.github.io/gsm/
Apache License 2.0
39 stars 9 forks source link

Bugfix: Analysis_Summary in spec causes workflow failure #1837

Closed lauramaxwell closed 1 month ago

lauramaxwell commented 2 months ago

Expected Behavior

spec should run through checkSpec() and pass

Current Behavior

Analysis_Summary is not found in data because it lives inside lAnalysis

Possible Solution

Update checkSpec to skip Analysis_* data frames for now, until this spec formatting is settled, or look inside lAnalysis list for data frame

Steps to Reproduce

1. 2. 3. 4.

Context (Environment)

Possible Implementation

Additional Comments

jwildfire commented 2 months ago

I'll try to address this in the upcoming workflow re-factor ...

lauramaxwell commented 1 month ago

Fixed in #1841