Ben Straub
Ben Straub
> @bundfussr how do you suggest solving the computational complexity complaint of the lint check? oh no - does this update increase the complexity? we already increased it a few...
It looks like the backticks in the snapshot tests are causing the R-CMD checks to fail?
@StefanThoma are you able to fix these snapshot tests - was there anything else we needed to finalize here?
UPDATE: I made this into separate issue to minimize scope creep I asked the fancy AI for additional tests that we might be missing Leap year came up! WDYT think...
oooohhh i like this!! @pharmaverse/admiral @pharmaverse/admiral_comm what do you think!?! I think this would be a fun function to spin up, but would take some work to finish it out!!...
> > The `definition` data @bundfussr mentioned above is similar to codelist in metadata, [`create_cat_var`](https://pharmaverse.github.io/metatools/reference/create_cat_var.html) and [`create_var_from_codelist`](https://pharmaverse.github.io/metatools/reference/create_var_from_codelist.html) from `metatools` can create paired variables dependent on good quality of metadata/codelist. >...
HI Stefan - reviewing the discussion it looks like @bundfussr original proposal was agreed upon. We can discuss at next week's meeting if you would like?
HI @jimrothstein this looks good so far and creating a draft and asking for feedback seems appropriate to me. I would just make sure that the example data where it...
Hey Jim - You can play fast and loose with this. The actions we have set up will crash out with all our tests and examples if things are not...
> @bms63 Ready for 1st serious review. > > Note: > > * example_qs.rda _ONLY_ looking good > * **documentation** requirements? roxygen? I think we could benefit from describing the...