Skip to content

Refactor of _location data structure#11

Merged
marcoSanti merged 5 commits into
mainfrom
refactor-hash-map
Oct 11, 2025
Merged

Refactor of _location data structure#11
marcoSanti merged 5 commits into
mainfrom
refactor-hash-map

Conversation

@marcoSanti
Copy link
Copy Markdown
Member

This commit refactors the structure of the capiocl::Engine::_locations, by removing the entry tuple and replacing with a more understandable struct. It also updates the logic of the capiocl::Engine class to use the new format of the _location hashmap

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 94.41341% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 95.33%. Comparing base (c4cfb8e) to head (727258e).

Files with missing lines Patch % Lines
src/Engine.cpp 95.45% 0 Missing and 6 partials ⚠️
capiocl.hpp 66.66% 0 Missing and 3 partials ⚠️
src/Serializer.cpp 94.11% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
- Coverage   95.46%   95.33%   -0.13%     
==========================================
  Files           4        4              
  Lines         772      772              
  Branches      444      442       -2     
==========================================
- Hits          737      736       -1     
  Misses          2        2              
- Partials       33       34       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Change paths in capiocl::engine from std::string to
std::filesystem::paths.
Fixed python bindings.
@marcoSanti marcoSanti force-pushed the refactor-hash-map branch 2 times, most recently from df0e41e to 2082525 Compare October 11, 2025 11:08
@marcoSanti marcoSanti merged commit 8b3eee3 into main Oct 11, 2025
49 checks passed
@marcoSanti marcoSanti deleted the refactor-hash-map branch October 11, 2025 12:25
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.

2 participants