cmu-phil / py-tetrad

Makes algorithms/code in Tetrad available in Python via JPype
MIT License
50 stars 9 forks source link

Package missing in R installation instructions #14

Closed annennenne closed 6 months ago

annennenne commented 7 months ago

I noticed a minor issue in the installation instructions for r-pytetrad (which are otherwise great, thank you!):

use_python("[...path...to...your...Python...installation...]")

ought to be

reticulate::use_python("[...path...to...your...Python...installation...]")

or even

reticulate::use_python(Sys.which("python"))

And I'm not sure if you are aware of this, but you can source in a specific R script using source() (in the final step):

setwd("~/py-tetrad/pytetrad")
source("R/sample_r_code2.R")
jdramsey commented 7 months ago

Awesome, thanks! :-) :-)

On Thu, Dec 7, 2023 at 6:01 AM Anne Helby Petersen @.***> wrote:

I noticed a minor issue in the installation instructions for r-pytetrad (which are otherwise great, thank you!):

install.packages("reticulate") install.packages("DiagrammeR") install.packages("psych") use_python("[...path...to...your...Python...installation...]")

ought to be

install.packages("reticulate") install.packages("DiagrammeR") install.packages("psych") reticulate::use_python("[...path...to...your...Python...installation...]")

And I'm not sure if you are aware of this, but you can source in a specific R script using source() (in the final step):

setwd("~/py-tetrad/pytetrad") source("R/sample_r_code2.R")

— Reply to this email directly, view it on GitHub https://github.com/cmu-phil/py-tetrad/issues/14, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACLFSRYKMFZCPEQNWISLB5TYIGOYDAVCNFSM6AAAAABAK3QXOOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZTANBYG42DQMQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jdramsey commented 6 months ago

Anne,

Sorry I didn't forget about your issue! I will fix it! I ended up getting busy over break on a paper and got away from working on Tetrad for a bit...

Hope all is well!

Joe

On Thu, Dec 7, 2023 at 9:12 AM Joseph Ramsey @.***> wrote:

Awesome, thanks! :-) :-)

On Thu, Dec 7, 2023 at 6:01 AM Anne Helby Petersen < @.***> wrote:

I noticed a minor issue in the installation instructions for r-pytetrad (which are otherwise great, thank you!):

install.packages("reticulate") install.packages("DiagrammeR") install.packages("psych") use_python("[...path...to...your...Python...installation...]")

ought to be

install.packages("reticulate") install.packages("DiagrammeR") install.packages("psych") reticulate::use_python("[...path...to...your...Python...installation...]")

And I'm not sure if you are aware of this, but you can source in a specific R script using source() (in the final step):

setwd("~/py-tetrad/pytetrad") source("R/sample_r_code2.R")

— Reply to this email directly, view it on GitHub https://github.com/cmu-phil/py-tetrad/issues/14, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACLFSRYKMFZCPEQNWISLB5TYIGOYDAVCNFSM6AAAAABAK3QXOOVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAZTANBYG42DQMQ . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jdramsey commented 6 months ago

I finally made these changes!!! Thank you so much!!!

They were so easy; I had just gotten overwhelmed.

By the way, we're going to release a new version of Tetrad soon, version 7.6.2, maybe in a week or two. There will be many updates.

All best,

Joe

annennenne commented 6 months ago

Thanks, and no problem!