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

Add .libPaths() to the test.data.table() diagnostic #6250

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

MichaelChirico
Copy link
Member

@MichaelChirico MichaelChirico commented Jul 12, 2024

Motivated by r-cmd-check-occasional still failing, which I no longer understand why. See ubuntu-latest log here:

https://github.com/Rdatatable/data.table/actions/runs/9908622817/job/27374799289

We get:

test.data.table('other.Rraw') failed to attach required package(s): ggplot2, hexbin, plyr, dplyr, caret, gdata, nlme, sf, nanotime.

And indeed these packages are missing from "all installed packages". But the previous step (Install dependencies) has those packages installing without error, e.g.

* DONE (ggplot2)

So now I'm wondering if it installed to a different library than is used by the subsequent step (?), otherwise that installed path is removed in between steps (it has _temp in it after all: /home/runner/work/_temp/Library). Anyway this diagnostic seems useful to include.

Copy link

Comparison Plot

Generated via commit bb5c930

Download link for the artifact containing the test results: ↓ atime-results.zip

Time taken to finish the standard R installation steps: 12 minutes and 8 seconds

Time taken to run atime::atime_pkg on the tests: 3 minutes and 24 seconds

@tdhock
Copy link
Member

tdhock commented Jul 13, 2024

great thanks

@MichaelChirico MichaelChirico merged commit cbd6564 into master Jul 13, 2024
3 of 4 checks passed
@MichaelChirico MichaelChirico deleted the rcmd-occasional-libpaths branch July 13, 2024 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants