Skip to content

Comments

feat: implemented R sessions api#2

Merged
ymarcon merged 7 commits intomasterfrom
feat/sessions
Feb 18, 2026
Merged

feat: implemented R sessions api#2
ymarcon merged 7 commits intomasterfrom
feat/sessions

Conversation

@ymarcon
Copy link
Member

@ymarcon ymarcon commented Feb 18, 2026

No description provided.

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 pull request implements an R sessions API for the datashield-opal package, introducing server-side R session management capabilities. The changes include a new OpalRSession class, updates to OpalConnection for session handling, and modifications to dependencies and project configuration.

Changes:

  • Implements R sessions API with lifecycle management (start, check status, close)
  • Updates dependencies from git URLs to pinned package versions
  • Adds development dependencies for data visualization (matplotlib, pandas)
  • Adds Jupyter notebook example demonstrating the new functionality
  • Modifies test to use different resource identifiers

Reviewed changes

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

File Description
pyproject.toml Version downgrade to 0.2.0, dependency updates to pinned versions, new dev dependencies, wheel packaging exclusions
datashield_opal/impl.py New OpalRSession class implementing RSession interface, modified OpalConnection with session management, added authentication check
tests/test_api_analysis.py Updated resource ID in test from RSRC.CNSIM1 to RSRC.CNSIM2 for server2
examples/datashield.py.ipynb New comprehensive example notebook demonstrating DataSHIELD operations including sessions, resource assignments, and visualizations

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

@ymarcon ymarcon merged commit 0c80974 into master Feb 18, 2026
19 of 22 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.

1 participant