Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

replace element_neighbors_locks_ lockes with read locks where possible #401

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

meiravgri
Copy link
Collaborator

Describe the changes in the pull request

A clear and concise description of what the PR is solving.

Which issues this PR fixes

  1. #...
  2. MOD...

Main objects this PR modified

  1. ...
  2. ...

Mark if applicable

  • This PR introduces API changes
  • This PR introduces serialization changes

@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Attention: Patch coverage is 96.22642% with 2 lines in your changes missing coverage. Please review.

Project coverage is 95.52%. Comparing base (fccd74b) to head (f038427).
Report is 60 commits behind head on main.

Files with missing lines Patch % Lines
src/VecSim/algorithms/hnsw/hnsw.h 96.07% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #401      +/-   ##
==========================================
- Coverage   95.63%   95.52%   -0.12%     
==========================================
  Files          66       66              
  Lines        4173     4182       +9     
==========================================
+ Hits         3991     3995       +4     
- Misses        182      187       +5     

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants