Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Last obs before exposure flag and Baseline Flag #18

Closed
KoulAshish opened this issue Nov 1, 2023 · 5 comments · Fixed by #49
Closed

Last obs before exposure flag and Baseline Flag #18

KoulAshish opened this issue Nov 1, 2023 · 5 comments · Fixed by #49
Assignees
Labels
enhancement New feature or request

Comments

@KoulAshish
Copy link

Feature Idea

Hi All,

Starting from SDTMIG 3.3 a new variable --LOBXL (Last Observation Before Exposure Flag) has been added to finding domains with a view to replace --BLFL (Baseline Flag).

Should we tackle both together or is there a separate thread for --LOBXL.

Source: https://www.cdisc.org/standards/foundational/sdtmig/sdtmig-v3-3/html#Observations+and+Variables
Section: 4.5.9

Relevant Input

No response

Relevant Output

No response

Reproducible Example/Pseudo Code

No response

@KoulAshish KoulAshish added the enhancement New feature or request label Nov 1, 2023
@github-project-automation github-project-automation bot moved this to Product Backlog in sdtm.oak R package Nov 1, 2023
@KoulAshish KoulAshish changed the title Feature Request: <Last obs before exposure flag and Baseline Flag> Last obs before exposure flag and Baseline Flag Nov 1, 2023
@MaguluriVenkata
Copy link

I suggest separating both, it's not necessary that both follow identical rule and also maintenance also is easy in future to have a dedicated space.

@rammprasad
Copy link
Collaborator

@rammprasad to complete the issue description.

@yli110-stat697
Copy link
Collaborator

@rammprasad to complete the issue description.

Hi @rammprasad can we define the desired input and output of the function?

@rammprasad
Copy link
Collaborator

@rammprasad
Copy link
Collaborator

@kamilsi - The function call in {sdtm.oak} can be oak_derive_blfl(ds_in, der_var, ref_var, ref_dat)

ref_dat will replace the code in the section below https://github.com/pharmaverse/roak_pilot/blob/main/R/oak_derive_blfl.R#L201 to https://github.com/pharmaverse/roak_pilot/blob/main/R/oak_derive_blfl.R#L210

The rest of the derivation logic is almost the same.

@kamilsi kamilsi linked a pull request May 8, 2024 that will close this issue
@rammprasad rammprasad self-assigned this May 14, 2024
@ramiromagno ramiromagno moved this from Product Backlog to Done in sdtm.oak R package Jul 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

5 participants