Skip to content

CI: fix test workflow#60

Open
mrakitin wants to merge 2 commits into
mainfrom
ci-fix-test-workflow
Open

CI: fix test workflow#60
mrakitin wants to merge 2 commits into
mainfrom
ci-fix-test-workflow

Conversation

@mrakitin

@mrakitin mrakitin commented Nov 1, 2025

Copy link
Copy Markdown
Member

This PR cleans up the disk space to allow for the installation of all libraries and pulling the docker images used for testing.

Got some ideas from https://carlosbecker.com/posts/github-actions-disk-space/.

@mrakitin
mrakitin requested a review from maffettone November 1, 2025 17:16
@mrakitin

mrakitin commented Nov 4, 2025

Copy link
Copy Markdown
Member Author

@maffettone, @ronpandolfi, the tests are unblocked now. Could you have a look at the tsuchinoko-related failures? I don't think I know enough to be able to debug.

@ronpandolfi

Copy link
Copy Markdown
Contributor

@mrakitin Looks like its a result of changes in gpcam and fvgp. I've fixed it in tsuchinoko for now.

Is there a good way to put dict data in an event doc field? One of the state variables for the gp is a kwargs that gets passed to the acquisition function. I've fixed it temporarily by making it a string. I'd rather not merge it in, since its open-form kwargs and could have conflicts.

@prjemian

prjemian commented Nov 8, 2025

Copy link
Copy Markdown

@mrakitin mrakitin closed this Dec 16, 2025
@mrakitin mrakitin reopened this Dec 16, 2025
@mrakitin

Copy link
Copy Markdown
Member Author

Power-cycled to trigger the CI.

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