Skip to content

CI/DOC: fix clipboard on travis #26103

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 15 commits into from
May 16, 2019
Merged

Conversation

saurav2608
Copy link

@saurav2608
Copy link
Author

this is more complex than what I thought. I will relook tomorrow.

@codecov
Copy link

codecov bot commented Apr 15, 2019

Codecov Report

Merging #26103 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #26103      +/-   ##
==========================================
- Coverage   91.96%   91.95%   -0.01%     
==========================================
  Files         175      175              
  Lines       52405    52405              
==========================================
- Hits        48193    48189       -4     
- Misses       4212     4216       +4
Flag Coverage Δ
#multiple 90.51% <ø> (ø) ⬆️
#single 40.72% <ø> (-0.14%) ⬇️
Impacted Files Coverage Δ
pandas/io/gbq.py 75% <0%> (-12.5%) ⬇️
pandas/core/frame.py 96.9% <0%> (-0.12%) ⬇️

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 b90f9db...3b6b879. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 15, 2019

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #26103      +/-   ##
==========================================
+ Coverage   91.69%   91.72%   +0.03%     
==========================================
  Files         174      174              
  Lines       50739    50741       +2     
==========================================
+ Hits        46523    46544      +21     
+ Misses       4216     4197      -19
Flag Coverage Δ
#multiple 90.23% <ø> (+0.03%) ⬆️
#single 41.68% <ø> (+0.36%) ⬆️
Impacted Files Coverage Δ
pandas/io/gbq.py 78.94% <0%> (-10.53%) ⬇️
pandas/core/frame.py 97.02% <0%> (-0.12%) ⬇️
pandas/util/testing.py 90.7% <0%> (+0.1%) ⬆️
pandas/core/generic.py 93.49% <0%> (+0.14%) ⬆️
pandas/io/clipboard/clipboards.py 31.64% <0%> (+12.65%) ⬆️
pandas/io/clipboard/__init__.py 56.86% <0%> (+17.64%) ⬆️

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 9c5165e...d8c6dc4. Read the comment docs.

@gfyoung gfyoung added CI Continuous Integration Docs labels Apr 18, 2019
@saurav2608 saurav2608 changed the title add install for xsel add install for xsel (Do not merge) Apr 19, 2019
@saurav2608
Copy link
Author

I am unable to replicate this in any of my environments. Thus I am create PRs to investigate.

@jreback
Copy link
Contributor

jreback commented Apr 20, 2019

@saurav2608 meaning the doc-build does not show any pyperclip error?

@jreback
Copy link
Contributor

jreback commented May 12, 2019

closing as stale

@jorisvandenbossche
Copy link
Member

(reopened to try something out, based on the diff I noticed the use of export DISPLAY, and that it was in a different place before. Who knows that makes a difference ..)

@jorisvandenbossche
Copy link
Member

@jreback OK?

(if this passes, will add a comment explaining why / referencing this PR)

@jreback
Copy link
Contributor

jreback commented May 15, 2019

so these are actually run? can u show that somewhere
maybe should make these tests highly visible somehow

@jorisvandenbossche
Copy link
Member

What do you mean with "so these are actually run?" ?
Or is it a comment meant for a different PR? (there are no tests involved here, only for doc building)

@jorisvandenbossche jorisvandenbossche changed the title add install for xsel (Do not merge) CI/DOC: fix clipboar in doc build on travis May 15, 2019
@jreback
Copy link
Contributor

jreback commented May 15, 2019

@jorisvandenbossche we didn't notice that we should be running these (and weren't); I dont' think this has changed. We need a highly visible test that we can simply look in the logs and inspect (maybe just add a print statement to it).

https://travis-ci.org/pandas-dev/pandas/jobs/532738283

-----------------------------------------------------------------------------------------------------------------------------
#5 pandas.tests.test_algos.TestRank.test_pct_max_many_rows[1d]: skipping high memory test since --run-high-memory was not set
#6 pandas.tests.test_algos.TestRank.test_pct_max_many_rows[2d]: skipping high memory test since --run-high-memory was not set
------------------------------------------------------------------------------------------------------------------------------
#7 pandas.tests.frame.test_rank.TestRank.test_pct_max_many_rows: skipping high memory test since --run-high-memory was not set
-----------------------------------------------------------------------------------------------------------------
#8 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[delims]: clipboard primitives not installed
#9 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[utf8]: clipboard primitives not installed
#10 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[utf16]: clipboard primitives not installed
#11 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[string]: clipboard primitives not installed
#12 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[long]: clipboard primitives not installed
#13 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[nonascii]: clipboard primitives not installed
#14 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[colwidth]: clipboard primitives not installed
#15 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[mixed]: clipboard primitives not installed
#16 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[float]: clipboard primitives not installed
#17 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame[int]: clipboard primitives not installed
#18 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[delims-\t]: clipboard primitives not installed
#19 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[delims-,]: clipboard primitives not installed
#20 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[delims-|]: clipboard primitives not installed
#21 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf8-\t]: clipboard primitives not installed
#22 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf8-,]: clipboard primitives not installed
#23 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf8-|]: clipboard primitives not installed
#24 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf16-\t]: clipboard primitives not installed
#25 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf16-,]: clipboard primitives not installed
#26 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[utf16-|]: clipboard primitives not installed
#27 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[string-\t]: clipboard primitives not installed
#28 pandas.tests.io.test_clipboard.TestClipboard.test_round_trip_frame_sep[string-,]: clipboard primitives not installed

@jorisvandenbossche
Copy link
Member

Ah, so those tests have not been running on master for a while? (since the CI env refactor PR #24632), ok, I see now.

But it seems that this PR actually solves that. They are running again (just not on the first travis build, but if you look eg at the second travis build: https://travis-ci.org/pandas-dev/pandas/jobs/532738284: no message about skipped tests anymore. On the first travis build, they have been skipped even before the CI env refactor).

But not sure I understand why the first travis build (in the build matrix) would skip those and the second not. I don't see anything in their definition that can explain that (the clipboard tests are marked as 'single').

@jorisvandenbossche
Copy link
Member

But not sure I understand why the first travis build (in the build matrix) would skip those and the second not.

Ah, maybe because the first build has no qt installed and the second does (through matplotlib), travis-37.yml vs travis-36-locale.yaml.

@jorisvandenbossche jorisvandenbossche changed the title CI/DOC: fix clipboar in doc build on travis CI/DOC: fix clipboard on travis May 15, 2019
@jreback
Copy link
Contributor

jreback commented May 15, 2019

right I don't care that they run on all builds, but at least one. I think just add a print statement in the basic clip test might be enough here to provide us an easy visual reference.

@jorisvandenbossche
Copy link
Member

@jreback I added some print statements, but they don't show up in the travis logs (although we have -s in the pytest command)

@jreback jreback added this to the 0.25.0 milestone May 16, 2019
@jreback
Copy link
Contributor

jreback commented May 16, 2019

@jorisvandenbossche then how can we be sure these are actually run? This is the entire problem.

@jorisvandenbossche
Copy link
Member

Well, you can say that for all other tests as well...
Unless we do -v to show each test that has been run.

@jreback
Copy link
Contributor

jreback commented May 16, 2019

Well, you can say that for all other tests as well...
Unless we do -v to show each test that has been run.

maybe, but this is a continual issue, and we need a foolproof way to say these tests are actually run

@jorisvandenbossche
Copy link
Member

Will open an issue about that, but merging this since it is actually solving a problem

@jorisvandenbossche
Copy link
Member

@saurav2608 thanks for putting me on the right track with the DISPLAY change !

@saurav2608
Copy link
Author

@saurav2608 thanks for putting me on the right track with the DISPLAY change !

@jorisvandenbossche Absolutely. I wish I could have contributed more. I am totally swamped at work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DOC: clipboard library missing in doc env on travis
5 participants