memray icon indicating copy to clipboard operation
memray copied to clipboard

Use scikit-build

Open pablogsal opened this issue 1 year ago • 2 comments

  • Try to respect RPATHS of calling dlopen modules with dlinfo
  • Use scikit-build-core

Issue number of the reported bug or feature request: #

Describe your changes A clear and concise description of the changes you have made.

Testing performed Describe the testing you have performed to ensure that the bug has been addressed, or that the new feature works as planned.

Additional context Add any other context about your contribution here.

pablogsal avatar Sep 03 '24 12:09 pablogsal

Codecov Report

:x: Patch coverage is 93.75000% with 1 line in your changes missing coverage. Please review. :white_check_mark: Project coverage is 93.54%. Comparing base (457365a) to head (44ec99f). :warning: Report is 67 commits behind head on main.

Files with missing lines Patch % Lines
tests/integration/test_tracking.py 80.00% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #673      +/-   ##
==========================================
+ Coverage   92.91%   93.54%   +0.63%     
==========================================
  Files          95       97       +2     
  Lines       11475    11558      +83     
  Branches     2114     2114              
==========================================
+ Hits        10662    10812     +150     
+ Misses        813      746      -67     
Flag Coverage Δ
cpp 93.54% <93.75%> (+0.63%) :arrow_up:
python_and_cython 93.54% <93.75%> (+0.63%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Sep 18 '24 23:09 codecov-commenter

This is missing the npm install and npm run-script build steps, I think

godlygeek avatar Apr 29 '25 19:04 godlygeek