Skip to content

Tests cuda#690

Draft
alinelena wants to merge 16 commits intostfc:mainfrom
alinelena:tests_cuda
Draft

Tests cuda#690
alinelena wants to merge 16 commits intostfc:mainfrom
alinelena:tests_cuda

Conversation

@alinelena
Copy link
Copy Markdown
Member

adding posibility some of the tests to run on cuda. create a workflow for them.. shall we force the other tests to run cpu only?

@alinelena alinelena force-pushed the tests_cuda branch 2 times, most recently from 1e3a424 to 05c5f78 Compare March 29, 2026 05:37
@alinelena
Copy link
Copy Markdown
Member Author

needs !689 to get in

@ElliottKasoar ElliottKasoar marked this pull request as draft March 31, 2026 17:49
@ElliottKasoar
Copy link
Copy Markdown
Member

ElliottKasoar commented Mar 31, 2026

I think this is based off of #689, so we should probably merge that first and rebase.

Tests are looking better re-running now. I'm wondering if the error was from mlco2/codecarbon#1122, which has been fixed as of a release yesterday. It says it was a warning, but perhaps the underlying KeyError meant the emissions tracker wasn't set up, and so we kill it.

Looks like we need to loosen the dispersion comparison slightly too.

@alinelena alinelena force-pushed the tests_cuda branch 2 times, most recently from 55cfbf9 to b279a69 Compare April 10, 2026 15:02
@ElliottKasoar ElliottKasoar added the testing Unit testing or CI label Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Unit testing or CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants