-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
CI: remove failing line #24092
CI: remove failing line #24092
Changes from 5 commits
a0a37cc
1f15d10
e79d52e
1d0a775
d3df86f
378515a
092320a
e20850e
ce0e25c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -161,15 +161,7 @@ def write_result(self, buf): | |
_classes.extend(self.classes) | ||
|
||
if self.notebook: | ||
div_style = '' | ||
try: | ||
import IPython | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
if IPython.__version__ < LooseVersion('3.0.0'): | ||
div_style = ' style="max-width:1500px;overflow:auto;"' | ||
except (ImportError, AttributeError): | ||
pass | ||
|
||
self.write('<div{style}>'.format(style=div_style)) | ||
self.write('<div>') | ||
|
||
self.write_style() | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,6 +37,8 @@ def s3_resource(tips_file, jsonl_file): | |
""" | ||
pytest.importorskip('s3fs') | ||
boto3 = pytest.importorskip('boto3') | ||
# GH-24092. See if boto.plugin skips the test or fails. | ||
pytest.importorskip("boto.plugin") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. For some reason, this line fails instead of skipping. https://travis-ci.org/pandas-dev/pandas/jobs/463496968 That job is using the newest There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Going to try to avoid pinning pytest for now. |
||
moto = pytest.importorskip('moto') | ||
|
||
test_s3_files = [ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is OK to do...
_sparse_series_to_coo
is the only caller, and it uses_get_level_number(x)
to go from maybe labels to numbers.