Ben Straub comments

Results 336 comments of


                                            Ben Straub

Closes #2426 Update NFRLT to reduce duplicates

@jeffreyad sorry jeff! could you look into fixing this merge conflict

Closes #2325 Added `id_vars` to `derive_vars_transposed()` and `derive_vars_atc()`

Many thank yous @gg106046 !! I know you are busy!!

Update actions to use new versions and ubuntu release

Yes I can set up.

Create vignette on generating renv lockfile from log file

@parmsam-pfizer any ideas on the failing actions? I think it says lines 20-24 which are just library calls. @kodesiba @nicholas-masel want to do a mini-release before winter holidays?? get this...

Feature Request: Move Addin packages to Suggest

@nmasel @kodesiba @parmsam-pfizer ![Image](https://github.com/user-attachments/assets/eca0a3da-53fa-4df5-9276-3253fef88a0f) stats and methods I think comes with base R installation. Do we need them in the description file? The x'd ones are Addin related. Okay to...

Feature Request: Move Addin packages to Suggest

oh cool - yeah can you remove in your branch - it might be a while until we drop the stuff for the addin

Closes #2694: added `id_vars_ref` argument to `derive_locf_records()` function and Closes #2680: `analysis_var_fill` argument to `derive_locf_records()` function

Hi @gg106046 just checking in on progress of this PR?

Closes #2694: added `id_vars_ref` argument to `derive_locf_records()` function and Closes #2680: `analysis_var_fill` argument to `derive_locf_records()` function

Hi @gg106046 - we are aiming to release admiral on June 13th - ideally all PRs should be in the week before. Just checking that you can have this completed...

Let's adopt Snapshot Testing for Message/Warnings/Errors

@atorus-research/xportr-development-team get the band back together in 2025? Up our Testing game. Clean up some documentation and make some nicer exmaples. Profit?

Automate releases

oooh i like this!!! Thanks @vedhav you are always amazing!