Skip to content

GEOPY-2387: force use of MKL for Blas implementation#250

Merged
sebhmg merged 1 commit intodevelopfrom
GEOPY-2387
Aug 14, 2025
Merged

GEOPY-2387: force use of MKL for Blas implementation#250
sebhmg merged 1 commit intodevelopfrom
GEOPY-2387

Conversation

@sebhmg
Copy link
Contributor

@sebhmg sebhmg commented Aug 14, 2025

GEOPY-2387 - use again MKL for Blas implementation on Windows instead of OpenBlas

@sebhmg sebhmg requested review from Copilot and domfournier August 14, 2025 15:33
@github-actions github-actions bot changed the title [GEOPY-2387] force use of MKL for Blas implementation GEOPY-2387: force use of MKL for Blas implementation Aug 14, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR forces the use of MKL for BLAS implementation across all platforms by uncommenting the libblas = "*=*mkl" specification in pyproject.toml. The change consolidates BLAS implementation to use Intel MKL consistently, which is already brought in by the simpeg dependency.

  • Uncomments and enables MKL BLAS specification in conda-lock dependencies
  • Removes platform-specific MKL configurations from individual Python environment files
  • Updates all conda lock files with new dependency hashes and package versions

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.

File Description
pyproject.toml Enables MKL BLAS dependency specification
py-3.*.conda-lock.yml Updates lock files with new dependency hashes and package versions
environments/*.conda.lock.yml Updates environment-specific lock files
environments/env-python-*.yml Removes redundant MKL specifications

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sebhmg sebhmg merged commit 86bffc3 into develop Aug 14, 2025
15 checks passed
@sebhmg sebhmg deleted the GEOPY-2387 branch August 14, 2025 16:24
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