Use Malt instead of Distributed.jl #48

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.

Sign up for GitHub

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

Jump to bottom

Merged

maleadt merged 3 commits into main from vc/malt

Oct 16, 2025

Member

vchuravy commented Oct 15, 2025

No description provided.

vchuravy requested review from giordano and maleadt

October 15, 2025 14:36

vchuravy commented

View reviewed changes

src/ParallelTestRunner.jl Outdated Show resolved Hide resolved

vchuravy commented

View reviewed changes

src/ParallelTestRunner.jl Outdated Show resolved Hide resolved

vchuravy commented

View reviewed changes

src/ParallelTestRunner.jl Outdated Show resolved Hide resolved

vchuravy commented

View reviewed changes

src/ParallelTestRunner.jl Show resolved Hide resolved

Collaborator

giordano commented Oct 15, 2025

The EOFError: read end of file is gone? Also, this seems to be slightly faster, at least on Linux and Windows (less so on macOS)?

Member Author

vchuravy commented Oct 15, 2025

Yeah, Malt doesn't use taskmonitor (for better and for worse), so we just get the Malt.TerminatedWorkerException

maleadt reviewed

View reviewed changes

src/ParallelTestRunner.jl Show resolved Hide resolved

vchuravy commented

View reviewed changes

src/ParallelTestRunner.jl Outdated Show resolved Hide resolved

Member Author

vchuravy commented Oct 16, 2025 •

edited

Loading

So one issue is that CUDA.jl starts a worker under the sanitizer #51 (comment)

See https://github.com/JuliaGPU/CUDA.jl/blob/f7deec6a62cddfbba0d87757175166d9f5e8cb16/test/runtests.jl#L212-L221

Malt currently does not allow for that customization..

Edit: Opened a PR for that JuliaPluto/Malt.jl#98

vchuravy mentioned this pull request

Add option to customize the exename JuliaPluto/Malt.jl#98

Merged

vchuravy and others added 3 commits

October 16, 2025 08:50


          Use Malt instead of Distributed.jl

c7eb21d


          Simplifications.

99fe417


          More simplifications: rely on Malt.isrunning.

11f4f78

maleadt force-pushed the vc/malt branch from 9e0319e to 11f4f78 Compare

October 16, 2025 06:57

maleadt merged commit 681ae1a into main

20 checks passed

maleadt deleted the vc/malt branch

October 16, 2025 11:14

vchuravy mentioned this pull request

Allow users to transform the test expression for files #53

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet