Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

R sharp r6 #1417

Merged
merged 108 commits into from
Jul 23, 2024
Merged

R sharp r6 #1417

merged 108 commits into from
Jul 23, 2024

Conversation

PavelBal
Copy link
Member

@PavelBal PavelBal commented May 28, 2024

Fixes #1413
Fixes #1395
Fixes #1422
Fixes #1432
Fixes #1449

@PavelBal
Copy link
Member Author

Fine for me. Then we should increment DESCRIPTION to 12.1 now as well

@Felixmil what is the workflow for this? Right now, the version in DESCRIPTION is 12.0.0.9003. Do I manually change it to 12.1.0?

@Felixmil
Copy link
Contributor

Felixmil commented Jul 18, 2024

Fine for me. Then we should increment DESCRIPTION to 12.1 now as well

@Felixmil what is the workflow for this? Right now, the version in DESCRIPTION is 12.0.0.9003. Do I manually change it to 12.1.0?

Not really manually, we use the {usethis} package to handle that. This is the way workflow: https://dev.open-systems-pharmacology.org/r-development-resources/collaboration_guide#releasing-versions

Usually we create a separate PR to achieve this but nothing's wrong to do it directly in this one.

I will take care of it.

R/init-package.R Outdated Show resolved Hide resolved
@@ -6,113 +6,47 @@
#' sim <- loadSimulation(system.file("extdata", "Aciclovir.pkml", package = "ospsuite"))
#'
#' # looking at a reference to `.NET` simulation object
#' sim$ref
#' sim$pointer
Copy link
Member

Choose a reason for hiding this comment

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

@PavelBal Why did we rename this to pointer?
Where is the NetObject class coming from?

Copy link
Member Author

Choose a reason for hiding this comment

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

NetObject is implemented in {rSharp}: https://www.open-systems-pharmacology.org/rSharp/articles/user-guide.html#object-pointers-and-the-netobject-class

In rClr, ref was an S3 object. Now, it is the external pointer externalptr that is held by NetObject.

@PavelBal
Copy link
Member Author

Woop-wooop, all tests and checks are passing under Windows AND Linux!

@Yuri05 @msevestre can we merge and create a pre-release? :)

@Yuri05 Yuri05 merged commit f610a38 into main Jul 23, 2024
6 checks passed
@Yuri05 Yuri05 deleted the r-sharp-r6 branch July 23, 2024 13:47
@Yuri05
Copy link
Member

Yuri05 commented Jul 23, 2024

Woop-wooop, all tests and checks are passing under Windows AND Linux!

@Yuri05 @msevestre can we merge and create a pre-release? :)

cool! just merged!
Sure, let's create a prelease (need then also to create releases or pre-releases of ospsuite.utils and tlf)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants