Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Adding Benchmarks#165

Open
ms2892 wants to merge 9 commits into
bloomberg:main from
ms2892:main
Open

Adding Benchmarks #165
ms2892 wants to merge 9 commits into
bloomberg:main from
ms2892:main

Conversation

@ms2892

@ms2892 ms2892 commented Jan 15, 2024

Copy link
Copy Markdown

#61 (comment)

Added Benchmarks
This pull request introduces the Benchmarks using airspeed velocity - (asv)

Testing performed
Locally ran the following commands

  • asv run
  • asv show
  • asv publish

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ae14c86) 89.80% compared to head (8606548) 89.80%.

Additional details and impacted files
@@ Coverage Diff @@
## main #165 +/- ##
=======================================
 Coverage 89.80% 89.80% 
=======================================
 Files 48 48 
 Lines 5524 5524 
 Branches 873 873 
=======================================
 Hits 4961 4961 
 Misses 563 563 
Flag Coverage Δ
cpp 75.20% <ø> (ø)
python_and_cython 99.93% <ø> (ø)

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

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

pablogsal and others added 8 commits January 25, 2024 19:53
See changelog for more details.
Signed-off-by: Pablo Galindo <pablogsal@gmail.com>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v5)
---
updated-dependencies:
- dependency-name: actions/setup-python
 dependency-type: direct:production
 update-type: version-update:semver-major
...
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Both need to be updated at once, and also the uploaded artifacts now
must be each be named something different and are made immutable. This
means that we cannot keep appending to the same archive called
"artifact", and that we instead must create one artifact for the sdist
and one for each architecture.
Note that the `upload_pypi` step also needs to be changed to ensure it
downloads all of the artifacts and extracts them as before.
Signed-off-by: Gus Monod <gmonod1@bloomberg.net>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Signed-off-by: Matt Wozniski <mwozniski@bloomberg.net>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Signed-off-by: ms2892 <msadiq074@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /