Skip to content
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

Support simple trial x-axis histogram label #95

Closed
wants to merge 1 commit into from

Conversation

oeuftete
Copy link
Contributor

@oeuftete oeuftete commented Nov 21, 2017

Support simple trial x-axis histogram label, adding trial to the --name option. Without this, there appears to be no way to generate histograms for long test names without having the x-axis labels take up a large amount of available real estate.

TODO:

  • fix tests, maybe (problems locally, anyway)

@codecov-io
Copy link

codecov-io commented Nov 21, 2017

Codecov Report

Merging #95 into master will increase coverage by 0.34%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   78.05%   78.39%   +0.34%     
==========================================
  Files          16       16              
  Lines        1727     1722       -5     
  Branches      314      312       -2     
==========================================
+ Hits         1348     1350       +2     
+ Misses        323      311      -12     
- Partials       56       61       +5
Impacted Files Coverage Δ
src/pytest_benchmark/plugin.py 88.04% <ø> (ø) ⬆️
src/pytest_benchmark/utils.py 81.45% <50%> (+2.58%) ⬆️
src/pytest_benchmark/logger.py 82.35% <0%> (-4.92%) ⬇️
src/pytest_benchmark/stats.py 92.3% <0%> (-1.29%) ⬇️
src/pytest_benchmark/session.py 91.71% <0%> (-0.06%) ⬇️
src/pytest_benchmark/cli.py 34.95% <0%> (ø) ⬆️

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 2ed7063...4f5d8b4. Read the comment docs.

@oeuftete oeuftete changed the title WiP: Support simple trial x-axis histogram label Support simple trial x-axis histogram label Jan 9, 2018
@ionelmc
Copy link
Owner

ionelmc commented Jun 2, 2018

This seems quite fine. You usually only have a single benchmark in your suite right?

@oeuftete
Copy link
Contributor Author

oeuftete commented Jun 7, 2018

No, there were several benchmarks.

@ionelmc
Copy link
Owner

ionelmc commented Jun 7, 2018

Ah whoops, seems some conflicts appeared. Can you rebase over master?

@oeuftete
Copy link
Contributor Author

I've moved on from this. Feel free to bring these commits in through your own branches.

@oeuftete oeuftete closed this Aug 17, 2018
@ionelmc
Copy link
Owner

ionelmc commented Aug 17, 2018

Rebased in 5478cb0

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.

None yet

3 participants