Skip to content

closes #72 add esub example article#96

Merged
rossfarrugia merged 5 commits intomainfrom
72_esub
May 6, 2025
Merged

closes #72 add esub example article#96
rossfarrugia merged 5 commits intomainfrom
72_esub

Conversation

@rossfarrugia
Copy link
Copy Markdown
Contributor

@rossfarrugia rossfarrugia commented Apr 9, 2025

Pull Request

add esub example article

Before you submit your pull request, take a look at the following checklist. Many thanks for your contribution!

  • Title: Place Closes #<insert_issue_number> at the beginning of your PR title. Use the Edit button in the top-right if you need to update.
  • Linked Issue: Ensure the related issue is linked in the "Development Section" on the right-hand side.
  • First Contribution: If this is your first contribution, add yourself to the DESCRIPTION file.
  • Impact on Examples: If your updates impact any examples, review locally for warnings or errors in the impacted example pages.
  • Merge Conflicts: Developers should address any merge conflicts and merge upon successful review.
  • New Packages: If new packages were used, ensure they are included in the DESCRIPTION file's Imports section.
  • Updated Examples: If you added or updated an example, ensure it runs on the latest CRAN release versions of all packages used.
  • Testing Instructions: Provide instructions on how to test the code if necessary.

@rossfarrugia rossfarrugia linked an issue Apr 9, 2025 that may be closed by this pull request
@rossfarrugia rossfarrugia requested a review from bms63 April 9, 2025 14:28
Comment thread esub/esub.qmd Outdated
Comment thread esub/esub.qmd
Comment thread esub/esub.qmd Outdated
Comment thread esub/esub.qmd
Copy link
Copy Markdown
Collaborator

@bms63 bms63 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome - just some minor comments feel free to use or not, but looking forward to sharing this widely!!

@rossfarrugia
Copy link
Copy Markdown
Contributor Author

Thanks @bms63 - great comments! I've updated for all. I'll give it the rest of April for my colleagues and others from our examples to team to comment on this PR, and then we can merge and publish early next month 🙏

Comment thread esub/esub.qmd
- [`{xportr}`](https://atorus-research.github.io/xportr/): delivers the SAS transport file (XPT) and eSub checks.
- [`{pkglite}`](https://merck.github.io/pkglite/): enables exchange of closed source R packages via text files.
- [`{datasetjson}`](https://atorus-research.github.io/datasetjson/): experimental package to deliver Dataset-JSON.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, @rossfarrugia . Thanks for allowing our review. Could metacore and metatools be considered packages useful for R Submission as well?

Both {metacore} and {metatools} play a vital role in the open-source R landscape for regulatory submissions by empowering users to proactively manage data quality and conformance. By leveraging these tools, we can get closer to ensuring our submitted data aligns with cross-industry standards like CDISC and passes validation checks by tools such as Pinnacle 21.

- metacore (https://atorus-research.github.io/metacore/): This package provides a robust framework for defining and managing metadata, which is crucial for ensuring data structure and content adhere to regulatory guidelines. Its capabilities in defining data dictionaries, controlled terminologies, and variable attributes can greatly assist in the creation of submission-ready datasets.

- metatools (https://pharmaverse.github.io/metatools/): Developed within the Pharmaverse ecosystem, metatools offers a collection of utilities designed to work with metadata, including functionalities for data validation, transformation, and reporting. Its focus on practical applications for clinical data makes it a strong candidate for inclusion in our eSUB toolkit.

I see metacore is in the ADSL example below.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll add a general paragraph here for other pharmaverse packages as trying to not repeat too much that's already in the ADaM example pages

Comment thread esub/esub.qmd
collate(file_ectd(), file_auto("inst")) %>%
prune("R/zzz.R") %>%
pack(output = "ectd/r0pkg.txt")
```
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For these RG examples mentioned here, when submitting closed-source packages, here are the links to their respective ADRGs if anyone would like to see the actual examples.

Pilot 1, using {pkglite} - https://github.com/RConsortium/submissions-pilot1-to-fda/blob/main/m5/datasets/rconsortiumpilot1/analysis/adam/datasets/adrg.pdf
Pilot 3, using compressed .zip approach - https://github.com/RConsortium/submissions-pilot3-adam-to-fda/blob/main/m5/datasets/rconsortiumpilot3/analysis/adam/datasets/adrg.pdf

Comment thread esub/esub.qmd Outdated
pack(output = "ectd/r0pkg.txt")
```

It is worth monitoring this
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
It is worth monitoring this
For current ADRG guidance, regardless of the programming language used for regulatory submissions, ensuring your documentations meet the ADRG completion guidelines found here [PHUSE ADRG Package](https://advance.hub.phuse.global/wiki/spaces/WEL/pages/26804660/Analysis+Data+Reviewer+s+Guide+ADRG+Package)
will be a great foundation to your open-source submission journey. As we are heading into this open-source R submission journey, it is worth monitoring this
[PHUSE Open-Source Metadata Documentation WG](https://phuse-org.github.io/OSDocuMeta/)
to see how guidance evolves in this documentation space.

@rossfarrugia
Copy link
Copy Markdown
Contributor Author

@bms63 some minor updates made for review from my colleagues. Do you want to take a quick final look and then merge/approve?

I'm away monday for UK bank holiday so if you publish this before i get back feel free to share the link out on pharmaverse slack to announce this new example. I hope people will find it useful and we can add to it over time as we see more industry submission experiences shared.

@bms63
Copy link
Copy Markdown
Collaborator

bms63 commented May 3, 2025

Thanks @rossfarrugia for updates. it looks good to me. feel free to publish when you get back from holiday.

@rossfarrugia rossfarrugia merged commit cb28764 into main May 6, 2025
2 checks passed
@rossfarrugia rossfarrugia deleted the 72_esub branch May 6, 2025 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Idea: add eSubmission example for submitting in R

3 participants