Skip to content

Devel dev#22

Merged
smritia merged 11 commits into
develfrom
devel_dev
Mar 13, 2026
Merged

Devel dev#22
smritia merged 11 commits into
develfrom
devel_dev

Conversation

@smritia

@smritia smritia commented Jan 22, 2026

Copy link
Copy Markdown
Collaborator

Move from @itsmesushanth 's forked repository

This pull request introduces several enhancements and updates to the package, focusing on improved survival analysis functionality, dependency management, and minor bug fixes. The most significant changes are the addition of new exports and imports related to survival analysis, updates to dependencies, and a small bug fix in the forest plot code.

Survival Analysis Enhancements

  • Added new exports to NAMESPACE for survival analysis functions: ph_plot, schoenfeld_plot, surv_pre_processor, and survfit_summary, enabling their use outside the package. [1] [2]
  • Imported the survival package in both DESCRIPTION and NAMESPACE, ensuring access to survival analysis tools and functions. [1] [2]

Dependency Updates

  • Updated RoxygenNote in DESCRIPTION from 7.3.2 to 7.3.3, reflecting the use of a newer version for documentation.

Bug Fixes and Code Quality

  • Fixed the join relationship in adae_risk_summary to specify relationship = "many-to-many", improving data merging accuracy.
  • Corrected the ae_forest_hlt_sig plot code: replaced size with linewidth for point sizing and removed the redundant key aesthetic.

Build and Packaging

  • Removed R/surv_utils.R from .Rbuildignore, ensuring this file is included in the build process.

itsmesushanth and others added 3 commits November 27, 2025 10:17
@itsmesushanth itsmesushanth self-assigned this Jan 22, 2026
@smritia
smritia requested a review from kallea03 January 23, 2026 05:17
@smritia smritia closed this Jan 23, 2026
@smritia smritia reopened this Jan 23, 2026
@smritia smritia closed this Mar 12, 2026
@smritia smritia reopened this Mar 12, 2026
@smritia
smritia merged commit e8a1c54 into devel Mar 13, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants