Skip to content

Updated SAS weighted logrank test page - #625

Open
dimpz4295 wants to merge 5 commits into
PSIAIMS:mainfrom
dimpz4295:qmd_updates
Open

dimpz4295 wants to merge 5 commits into
PSIAIMS:mainfrom
dimpz4295:qmd_updates

Conversation

@dimpz4295

Copy link
Copy Markdown

Updated the SAS weighted log-rank test page to include table with numerical outputs from the SAS procedures using sample data.

@abiterry

abiterry commented Oct 6, 2026

Copy link
Copy Markdown
Collaborator

Hi @dimpz4295, thanks so much for contributing these two PRs! Before we review the content, please could you pull down from the main CAMIS branch into your forked branch and add library(tibble) to your package calls for both PRs?

@dimpz4295

Copy link
Copy Markdown
Author

Hi @abiterry yes I can! I apologize for that. I'll do that as soon as I can and fix the errors.

@dimpz4295

dimpz4295 commented Oct 11, 2026 •

Copy link
Copy Markdown
Author

Hi @abiterry! I've pulled the latest changes from the main CAMIS branch into my ancova-fix branch and 'weighted log rank SAS page' branch, added library(tibble) to the package calls, and pushed the updates to my fork. Thank you.

Comment thread Comp/r-sas_ancova.qmd
```

# ANCOVA Comparison
# Summary

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please can you add the below into this r-sas_ancova.qmd as its failure when run. I think you added it to the logrank one but not this one. Thanks!

#| label: setup
#| include: false
library(tibble)

@DrLynTaylor

Copy link
Copy Markdown
Collaborator

Hi @dimpz4295 - just saw you pushed ANCOVA in a separate PR that now runs clean. Can you just remove ANCOVA change from this PR, then we can bring them both in separately. Thanks !

This branch has not been deployed

No deployments
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.

3 participants