Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Adds performance tests, that compare to published NuGet #975

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Oct 23, 2019

Conversation

lostmsu
Copy link
Member

@lostmsu lostmsu commented Oct 22, 2019

A new project src\perf_tests\Python.PerformanceTests.csproj, that uses BenchmarkDotNet to benchmark compare current version with the latest version published on NuGet (2.3 as of now).

During build the baseline and new version are copied into separate directories. Benchmark code then loads new or baseline based on an environment variable.

In the end, an "optimistic performance ratio" is computed, that has to be less than some target (usually close to 1.0) for the performance test to pass.

@lostmsu
Copy link
Member Author

lostmsu commented Oct 22, 2019

Not sure I made the CI/build script changes correctly. Plus we might have to restrict testing to Python 3.5 for now, as it is the only version supported by a published NuGet.

@lostmsu lostmsu force-pushed the features/PerfTests branch from ad92f96 to 2e28711 Compare October 22, 2019 20:08
@lostmsu
Copy link
Member Author

lostmsu commented Oct 22, 2019

Excluded CI changes so that this can be merged independently.

@lostmsu lostmsu force-pushed the features/PerfTests branch from 327c311 to 6a9f75e Compare October 22, 2019 21:09
@lostmsu lostmsu force-pushed the features/PerfTests branch from 6a9f75e to 250a490 Compare October 22, 2019 21:11
@codecov-io
Copy link

codecov-io commented Oct 22, 2019

Codecov Report

Merging #975 into master will increase coverage by 21.26%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #975       +/-   ##
===========================================
+ Coverage   65.44%   86.71%   +21.26%     
===========================================
  Files           1        1               
  Lines         301      301               
===========================================
+ Hits          197      261       +64     
+ Misses        104       40       -64
Flag Coverage Δ
#setup_linux 65.44% <ø> (ø) ⬆️
#setup_windows 71.42% <ø> (?)
Impacted Files Coverage Δ
setup.py 86.71% <0%> (+21.26%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f2e6f6f...3372416. Read the comment docs.

@lostmsu
Copy link
Member Author

lostmsu commented Oct 22, 2019

@filmor ready to review
AppVeyor result: https://ci.appveyor.com/project/pythonnet/pythonnet/builds/28304985

@filmor filmor merged commit 1463538 into pythonnet:master Oct 23, 2019
@lostmsu lostmsu deleted the features/PerfTests branch November 21, 2019 00:02
AlexCatarino pushed a commit to QuantConnect/pythonnet that referenced this pull request Jun 27, 2020
Add performance tests that compare to published NuGet
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.

3 participants