search

traversc / qs

Quick serialization of R objects
397 stars 19 forks source link

segfault: RApiSerialize update requires rebuilding/reinstalling qs from source #96

Closed achubaty closed 3 months ago

achubaty commented 3 months ago

just dropping a note here for others, that following an update today to RApiSerialize, I needed to reinstall qs (from source!) to get rid of the following errors on various platforms:

 *** caught segfault ***
address (nil), cause 'memory not mapped'
 *** caught segfault ***
address (nil), cause 'invalid permission'
toomish commented 3 months ago

Same problem:

> qs::qsave(df, "data.qs")

 *** caught segfault ***
address 0x19, cause 'memory not mapped'

Reinstall qs from source helped to get rid of the error.

traversc commented 3 months ago

Thanks, yes please re-install qs if you have this issue

mattfidler commented 3 months ago

Do you have to reinstall from source or is it simply from CRAN?

mattfidler commented 3 months ago

Note for my runnners they install from the mac binaries and it still crashes.

achubaty commented 3 months ago

Need to install from source (can be from CRAN or GitHub):

install.packages("qs", type = "source")
## or
remotes::install_github("traversc/qs")

For GitHub Actions runners, the easiest thing to do was to install from GitHub for this package, e.g.,

- uses: r-lib/actions/setup-r-dependencies@v2
        with:
          extra-packages: |
            any::rcmdcheck
            traversc/qs
          needs: check
traversc commented 3 months ago

Thanks for all the reports. New qs version out on CRAN so hopefully this issue is moot now.