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

rename kornia-core to kornia-tensor #195

Merged
merged 1 commit into from
Dec 25, 2024
Merged

rename kornia-core to kornia-tensor #195

merged 1 commit into from
Dec 25, 2024

Conversation

edgarriba
Copy link
Member

@edgarriba edgarriba commented Dec 25, 2024

This pull request includes significant changes to the kornia project, particularly focusing on renaming and restructuring core components. The most important changes involve renaming the kornia-core and kornia-core-ops crates to kornia-tensor and kornia-tensor-ops, respectively, and updating dependencies and imports accordingly.

Renaming and restructuring:

  • Cargo.toml: Updated workspace members to use wildcard notation for crates and added kornia-tensor and kornia-tensor-ops while removing kornia-core and kornia-core-ops. [1] [2]
  • Renamed crates/kornia-core to crates/kornia-tensor and updated the package name in Cargo.toml.
  • Renamed crates/kornia-core-ops to crates/kornia-tensor-ops and updated the package name in Cargo.toml. [1] [2]

Dependency and import updates:

  • Updated dependencies in various Cargo.toml files to replace kornia-core with kornia-tensor. [1] [2] [3]
  • Updated imports in source files to replace kornia-core with kornia-tensor. [1] [2] [3] [4] [5] [6] [7]

Error handling and test updates:

  • Updated error handling in crates/kornia-image/src/error.rs to use kornia_tensor::TensorError instead of kornia_core::TensorError.
  • Updated tests in various source files to use kornia_tensor instead of kornia_core. [1] [2]

@edgarriba edgarriba merged commit e8bbc0e into main Dec 25, 2024
9 checks passed
@edgarriba edgarriba deleted the rename-core-tensor branch January 2, 2025 06:52
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.

1 participant