Skip to content

Conversation

@ymarcon
Copy link
Member

@ymarcon ymarcon commented Nov 3, 2025

No description provided.

@ymarcon ymarcon requested a review from Copilot November 3, 2025 16:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds session management functionality to the DSOpal package, implementing support for DataSHIELD Interface (DSI) session operations. The changes enable asynchronous remote R session creation and state monitoring.

  • Adds new OpalSession class implementing DSI session management
  • Implements session lifecycle methods (dsHasSession, dsSession, dsIsReady, dsStateMessage)
  • Updates dependency versions and adds utils package dependency
  • Updates examples to demonstrate new session management functionality

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
R/OpalSession.R New file defining OpalSession class and implementing session state methods
R/OpalConnection.R Adds session management methods and updates dsConnect to pass "datashield" context
NAMESPACE Exports new session-related classes and methods
DESCRIPTION Updates package dependencies (opalr, DSI versions) and adds utils dependency
man/*.Rd Generated documentation for new session management methods
inst/examples/datashield.R Updated example demonstrating session management usage
inst/examples/datashield-api.R Updated example with session lifecycle demonstrations

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ymarcon ymarcon merged commit 3502c41 into master Nov 3, 2025
0 of 3 checks passed
@ymarcon ymarcon deleted the feat/session branch November 3, 2025 16:41
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