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

BLD: Add line displaying CPU info to CI #5059

Merged
merged 1 commit into from Sep 7, 2018

Conversation

bashtage
Copy link
Member

Show CPU type in CI to help understand random failures

@coveralls
Copy link

coveralls commented Aug 30, 2018

Coverage Status

Coverage remained the same at 83.975% when pulling 90a4d12 on bashtage:get-cpu-info-appveyor into a9bb76a on statsmodels:master.

@bashtage
Copy link
Member Author

Coveralls is really bugged -- 0 changes to code and a decrease in coverage.

@jbrockmendel
Copy link
Contributor

Travis log suggests running with sudo.

+1 on this.

@jbrockmendel
Copy link
Contributor

Looking at the logs this works as advertised. @josef-pkt should be merged

@codecov
Copy link

codecov bot commented Aug 30, 2018

Codecov Report

Merging #5059 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5059   +/-   ##
=======================================
  Coverage   81.45%   81.45%           
=======================================
  Files         582      582           
  Lines       90116    90116           
  Branches    10093    10093           
=======================================
  Hits        73401    73401           
  Misses      14416    14416           
  Partials     2299     2299

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 a9bb76a...90a4d12. Read the comment docs.

@jbrockmendel
Copy link
Contributor

@josef-pkt we’ve got a fellow trying to improve the CI here. Maybe let him?

@jbrockmendel
Copy link
Contributor

@josef-pkt please empower someone to help with this kind of thing.

@josef-pkt josef-pkt merged commit 25ebae9 into statsmodels:master Sep 7, 2018
@josef-pkt josef-pkt added this to the 0.10 milestone Sep 7, 2018
@bashtage bashtage deleted the get-cpu-info-appveyor branch September 9, 2018 07:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants