Skip to content

Improve performance of get_vector and get_coordinates in some cases o…#779

Merged
mateuszbaran merged 1 commit intomasterfrom
mbaran/perf-euclidean-complex-vec
Jan 10, 2025
Merged

Improve performance of get_vector and get_coordinates in some cases o…#779
mateuszbaran merged 1 commit intomasterfrom
mbaran/perf-euclidean-complex-vec

Conversation

@mateuszbaran
Copy link
Copy Markdown
Member

…n complex Euclidean

@mateuszbaran mateuszbaran self-assigned this Jan 7, 2025
@mateuszbaran mateuszbaran added the Ready-for-Review A label for pull requests that are feature-ready label Jan 7, 2025
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.44%. Comparing base (2b13e39) to head (38d21bf).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #779      +/-   ##
==========================================
- Coverage   96.44%   96.44%   -0.01%     
==========================================
  Files         127      127              
  Lines       11845    11843       -2     
==========================================
- Hits        11424    11422       -2     
  Misses        421      421              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Member

@kellertuer kellertuer left a comment

Choose a reason for hiding this comment

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

lgtm

@mateuszbaran mateuszbaran merged commit ca9e634 into master Jan 10, 2025
@kellertuer kellertuer deleted the mbaran/perf-euclidean-complex-vec branch July 5, 2025 13:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready-for-Review A label for pull requests that are feature-ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants