Skip to content

Conversation

mcranium
Copy link
Contributor

I found that Revticulate did not work on my desktop Linux system. After a bit of digging, I found out that the function callRev() expects there to be 13 lines of banner code after executing a temp file. On my system (Fedora) this is not the case, leading to an error message, rendering the entire package non-functional.
I implemented a quick and dirty check for this banner, to only remove these lines if present. Note: I have not tested it on MacOS or Windows. Could you please look into this?

Also, there was an issue with retrieving the file path of the temporary rev file, which I fixed (again, only tested on my system).

Best,
Mario

@bredelings
Copy link

The development version of revbayes (v1.4.0-preview) recently changed to have a different command-line interface. One thing that changed along with this is that we don't print the header (aka "banner") you run rb script.Rev. So this change probably makes sense. I'll let someone who has more experience with Revticulate decide whether to merge it though.

The new command-line interface looks like this:

$ rb -h
Bayesian phylogenetic inference using probabilistic graphical models and an 
interpreted language 

Usage: rb [options]
       rb [options] file [args]
       rb [options] -e expr [-e expr2 ...] [args]

OPTIONS:
  -h,      --help                  Print this help message and exit 
  -v,      --version               Show version and exit 
  -j,      --jupyter               Run in jupyter mode 
  -q,      --quiet                 Hide startup message (if no file or -e expr) 
  -i,      --interactive           Force interactive (with file or -e expr) 
  -c,      --continue              Continue after error (with file or -e expr) 
  -s,      --seed UINT             Random seed (unsigned integer) 
  -o,      --setOption TEXT ...    Set an option key=value (See ?setOption for 
                                   the list of available keys and their 
                                   associated values) 

Expressions (one or more '-e <expr>') may be used *instead* of 'file'. 
See http://revbayes.github.io for more information. 

@mcranium mcranium mentioned this pull request Sep 22, 2025
@davidcerny
Copy link
Contributor

My understanding is that this PR has been superseded by PR #7.

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.

3 participants