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

Tests fail when executed in parallel #61

Closed
carlosuc3m opened this issue Aug 12, 2024 · 5 comments · Fixed by #64
Closed

Tests fail when executed in parallel #61

carlosuc3m opened this issue Aug 12, 2024 · 5 comments · Fixed by #64
Assignees
Labels
priority This is a priority

Comments

@carlosuc3m
Copy link
Member

Hello @StRigaud ,
I am working on the tests an finding that when the more than 4-5 tests are launched in parallel, there is a segmentation fault.

I don't think the GPU is running out of space because the tests create only 2x2 arrays, so I don't know what is happening.

I wil try to research more on this, I don't even know if it is reproducible in c++ or python but I just wanted to comment it to you to see if you had any idea why this is happening.
REgards,
Carlos

@carlosuc3m
Copy link
Member Author

Ok it is not because they are being executed in parallel but because something happening with Icy.
I am going to keep digging on this, but Is suspect it has to do with the weird classloading system that Icy has.

@carlosuc3m carlosuc3m mentioned this issue Aug 13, 2024
@StRigaud
Copy link
Member

idk if related but trying to build with maven in the clij3 repo is failing with a lot of error related to Icy.

@carlosuc3m
Copy link
Member Author

can you provide the errors please? @StRigaud

@StRigaud
Copy link
Member

StRigaud commented Sep 2, 2024

I have no issue with clesperantoj but this is the mvn output from trying to build clij3 (my branch) with clesperantoJ on my local machine:

error_mvn_build.txt

(the error msg is too long for github...)

@StRigaud
Copy link
Member

For now let's remove support of Icy (we keep it into a new branch and will come back to it later on)
ToDo for @carlosuc3m

@StRigaud StRigaud added the priority This is a priority label Sep 11, 2024
@StRigaud StRigaud assigned StRigaud and carlosuc3m and unassigned StRigaud Sep 11, 2024
@carlosuc3m carlosuc3m linked a pull request Sep 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority This is a priority
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants