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

Checking for missing S3 method exports hangs "forever" if you have large data objects in the package #1593

Open
jranke opened this issue Feb 8, 2024 · 2 comments · May be fixed by #1594
Open

Comments

@jranke
Copy link
Contributor

jranke commented Feb 8, 2024

In package yet unpublished I have a large data object (the .rda file has about 6 MB). With current roxygen versions, the namespace roclet hangs here

https://github.com/r-lib/roxygen2/blob/main/R/namespace.R#L402

This is apparently very inefficient if you have large objects in s3objects. If I remove the data object from the package, namespace generation proceeds swiftly as usual.

jranke added a commit to jranke/roxygen that referenced this issue Feb 9, 2024
@jranke jranke linked a pull request Feb 9, 2024 that will close this issue
@alexpghayes
Copy link

alexpghayes commented Mar 28, 2024

I'm running into this as well with https://github.com/alexpghayes/latentnetmediate and it's pretty debilitating. Downgrading to version 7.2.0 resolved my issue temporarily.

Edit: #1594 also resolves my issue. Thanks!

haziqj added a commit to haziqj/lavaan.bingof that referenced this issue Apr 1, 2024
@jranke
Copy link
Contributor Author

jranke commented Apr 23, 2024

Hi, I wonder if noone with commit rights for roxygen is affected by this bug? What about @krlmlr or @DavisVaughan , could you maybe have a look?

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 a pull request may close this issue.

2 participants