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

maint: changes for 2025.0 compiler #2868

Merged

Conversation

Alexandr-Solovev
Copy link
Contributor

@Alexandr-Solovev Alexandr-Solovev commented Aug 5, 2024

Description:

Maint: Changes for 2025.0 Compiler Compatibility

Summary:

This PR updates the oneDAL codebase for compatibility with the upcoming 2025.0 compiler version. It includes fixes and modifications across multiple files to address issues related to the new compiler requirements.

Key Changes:

  1. Compute Kernel CSR Implementation:

    • Updated compute_kernel_csr_impl_dpc.cpp for conditional compilation of sycl::fmin and sycl::fmax.
  2. Compute Kernel Dense Implementation:

    • Modified compute_kernel_dense_impl_dpc.cpp for proper handling of sycl::sqrt.
  3. Test Adjustments:

    • Updated test files (badargs.cpp, overflow.cpp) for algorithms like covariance, dbscan, kmeans, linear_kernel, and pca.
  4. Backend and Primitives:

    • Adjusted backend and primitive test files for consistency with new macro definitions and enhanced stability.

Copy link
Contributor

@ethanglaser ethanglaser left a comment

Choose a reason for hiding this comment

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

Nice work on this

cpp/oneapi/dal/test/archives.cpp Outdated Show resolved Hide resolved
cpp/oneapi/dal/backend/primitives/test/ndarray.cpp Outdated Show resolved Hide resolved
@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

1 similar comment
@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev
Copy link
Contributor Author

Copy link
Contributor

@icfaust icfaust left a comment

Choose a reason for hiding this comment

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

Please make a ticket for the observed windows sycl issue, otherwise CIs are green, and the changed to tests/ basic_statistics / macro.hpp are straightforward. Just make sure someone else just glances at it other than me (verbal approval sufficient).

@ethanglaser
Copy link
Contributor

Looking good - can we ensure than CI job with nightly (not CEV) is running smoothly?

@Alexandr-Solovev
Copy link
Contributor Author

Hi @ethanglaser , nope, with this pr CEV/MEV will be red, but due to it necessary intel/scikit-learn-intelex#2000, I guess we can merge it in the same time as we merge your prs with cl removals

@ethanglaser
Copy link
Contributor

Hi @ethanglaser , nope, with this pr CEV/MEV will be red, but due to it necessary intel/scikit-learn-intelex#2000, I guess we can merge it in the same time as we merge your prs with cl removals

Yes I mean regular nightly scope (not CEV)

@Alexandr-Solovev Alexandr-Solovev changed the title maint: changes for 2025.0 compiler maint: changes for 2025.0 compiler (DO NOT MERGE) Aug 13, 2024
@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev Alexandr-Solovev changed the title maint: changes for 2025.0 compiler (DO NOT MERGE) maint: changes for 2025.0 compiler Aug 14, 2024
@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

1 similar comment
@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev
Copy link
Contributor Author

/intelci: run

@Alexandr-Solovev Alexandr-Solovev merged commit 2a9918c into oneapi-src:main Aug 30, 2024
16 checks passed
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.

None yet

5 participants