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

Setup CSCS CI #54

Merged
merged 18 commits into from
May 12, 2024
Merged

Setup CSCS CI #54

merged 18 commits into from
May 12, 2024

Conversation

kanduri
Copy link
Collaborator

@kanduri kanduri commented Apr 29, 2024

Added config files and tweaking configurations in the backend

@kanduri
Copy link
Collaborator Author

kanduri commented Apr 29, 2024

cscs-ci run

1 similar comment
@kanduri
Copy link
Collaborator Author

kanduri commented May 3, 2024

cscs-ci run

@frostedoyster
Copy link
Collaborator

Very weird @kanduri. It seems that compilation is failing because the compiler can't understand atomicAdd. Do you have any ideas?

@kanduri
Copy link
Collaborator Author

kanduri commented May 6, 2024

@frostedoyster
I made some attempts to resolve the compilation errors.
Regarding HPE, the polynomial order 0 (zero) implementation cannot work. We should remove it from the switch case ladder. In the file hpe.cu, we see multiple places where a divide-by-zero error can occur. For example:

int32_t i_monomial = threadIdx.x % polynomial_order;

Copy link
Collaborator

@frostedoyster frostedoyster left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks everyone!

@frostedoyster frostedoyster merged commit 52e39ee into main May 12, 2024
5 checks passed
@frostedoyster frostedoyster deleted the cscs_ci branch May 12, 2024 11:22
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