Skip to content

Add R wrapper#45

Open
jared321 wants to merge 8 commits into
mainfrom
35AddRWrapper
Open

Add R wrapper#45
jared321 wants to merge 8 commits into
mainfrom
35AddRWrapper

Conversation

@jared321

@jared321 jared321 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

This brings in changes originally made on mp-codemerge. The main goal is to get an R wrapper that works with the most modern regression functionality (from BitBucket OpenBT), but that does not provide model mixing functionality (from OpenBTMixing).

Tentative PR Self-review

  • Review all changes that I can (and that won't be much)
    • Updated link in User Guide's example works (tested on guide rendered by RTD in this PR).
  • Review updated content in docs as rendered by RTD for the PR
  • Confirm all actions passing
    • Check link actions fails because I added a link to a file on main that will only be on that branch after we merge this branch.

@mpratola Please see some basic questions below. Obviously there's not much that I can do to help review the content of this. If you think it's best, please add @jcyannotty as a reviewer and ping him to see if he has time to review within our deadline.

@jcyannotty We need to finish the release in the coming days. Will you have time to review this soon?

Matt Pratola and others added 5 commits July 6, 2026 16:47
directly from the github repo.  Includes the openbt.R wrapper which
is a merged of version from Pratola's repo and Yannotty's version which
has some model mixing interface code.

(cherry picked from commit 390e400)
The break was due to the movement of the target file to a new folder.
Strangely, the check link action didn't fail for the PR, but did on the merge
commit.
@jared321 jared321 self-assigned this Jul 6, 2026
Comment thread Ropenbt/DESCRIPTION Outdated
Comment thread Ropenbt/DESCRIPTION
@jared321 jared321 requested a review from mpratola July 8, 2026 20:23
This was referenced Jul 10, 2026
@jared321 jared321 requested a review from jcyannotty July 14, 2026 12:31
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.

2 participants