-
Notifications
You must be signed in to change notification settings - Fork 17
Separate CARDs section from TLG section #115
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
Separate CARDs section from TLG section #115
Conversation
Co-authored-by: jeffreyad <112705781+jeffreyad@users.noreply.github.com>
Co-authored-by: jeffreyad <112705781+jeffreyad@users.noreply.github.com>
| ``` | ||
|
|
||
| ## Introduction | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add more information about CDISC Analysis Results Data and {cards} package in the introduction
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@copilot Add more information about CDISC Analysis Results Data and {cards} package in the introduction at beginning of CARDs section.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added comprehensive information about CDISC Analysis Results Data and the {cards} package in the introduction. The enhanced introduction now includes:
- A dedicated section explaining CDISC ARD with details on reproducibility, automation, interoperability, and regulatory submissions
- A detailed description of the {cards} package highlighting its key features and capabilities
- Clear context for the ARD-based workflow demonstrated in the examples
Commit: 2b25a94
…ation Co-authored-by: jeffreyad <112705781+jeffreyad@users.noreply.github.com>
|
@copilot rename the files in cards so they don't have the same name as the ones in TLGs, for example cards_demo |
…flicts Co-authored-by: jeffreyad <112705781+jeffreyad@users.noreply.github.com>
Renamed the files in the cards directory to avoid naming conflicts:
The TLG directory files remain as Commit: 07b682d |
|
@copilot in the TLG section add a "See also" comment and link to CARDs example |
Co-authored-by: jeffreyad <112705781+jeffreyad@users.noreply.github.com>
Added a "See also" section at the end of Commit: 23165bc |
|
Haha do you even need us anymore Jeff! |
Pull Request
The
tlg/demographic.qmdfile mixed traditional TLG examples (rtables/tern) with modern ARD-based examples (cards/gtsummary). This PR separates them into distinct sections for clearer navigation and enhanced documentation.Changes
New
cards/section:cards/index.qmd- Section landing pagecards/cards_demographic.qmd- Demographics table using {cards} and {gtsummary} packages for ARD-based workflows, with comprehensive introduction explaining CDISC Analysis Results Data (ARD) and the {cards} packageUpdated
tlg/demographic.qmd:Configuration updates:
cardssection to_quarto.ymlsidebar (positioned after TLG)inst/WORDLISTEnhanced Documentation:
cards/cards_demographic.qmdexplaining CDISC Analysis Results Data (ARD) standard including benefits for reproducibility, automation, interoperability, and regulatory submissionsFile Naming:
cards_demographic.qmdandcards_demographic.Rto avoid naming conflicts with TLG files (demographic.qmdanddemographic.R)The sidebar navigation now presents:
Before you submit your pull request, take a look at the following checklist. Many thanks for your contribution!
Closes #<insert_issue_number>at the beginning of your PR title. Use the Edit button in the top-right if you need to update.DESCRIPTIONfile.DESCRIPTIONfile'sImportssection.Original prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.