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

Fix tests on big-endian systems #3125

Merged
merged 2 commits into from
Jul 15, 2019
Merged

Fix tests on big-endian systems #3125

merged 2 commits into from
Jul 15, 2019

Conversation

ginggs
Copy link
Contributor

@ginggs ginggs commented Jul 14, 2019

  • Tests added
  • Fully documented, including whats-new.rst for all changes and api.rst for new API

@pep8speaks
Copy link

pep8speaks commented Jul 14, 2019

Hello @ginggs! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2019-07-14 16:19:09 UTC

@codecov
Copy link

codecov bot commented Jul 14, 2019

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3125   +/-   ##
=======================================
  Coverage   95.98%   95.98%           
=======================================
  Files          63       63           
  Lines       12777    12777           
=======================================
  Hits        12264    12264           
  Misses        513      513

@max-sixty
Copy link
Collaborator

Thanks @ginggs

Is your PR here still relevant? #2334. I'm sorry no one reviewed that.
This isn't my area of expertise; @pydata/xarray is there another member of the team who would be able to help?

@ginggs
Copy link
Contributor Author

ginggs commented Jul 15, 2019

@max-sixty

Yes, #2334 is still relevant, I recently rebased and added another XFAIL for a new test.

@shoyer
Copy link
Member

shoyer commented Jul 15, 2019

I don't suppose any of platforms Azure pipelines offers for tests are big-endian? https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/hosted?view=azure-devops#software

I'm guessing not, which is a bit of a shame, but that shouldn't stop us from merging this.

@shoyer shoyer merged commit 3c05a20 into pydata:master Jul 15, 2019
@shoyer
Copy link
Member

shoyer commented Jul 15, 2019

Thanks @ginggs !

@max-sixty
Copy link
Collaborator

@ginggs if you're up for adding a whatsnew (and give yourself credit), please do!

@ginggs ginggs deleted the big-endian branch July 15, 2019 19:19
@ginggs
Copy link
Contributor Author

ginggs commented Jul 15, 2019

@max-sixty done in #3135

max-sixty pushed a commit that referenced this pull request Jul 15, 2019
dcherian added a commit to yohai/xarray that referenced this pull request Aug 3, 2019
* master: (68 commits)
  enable sphinx.ext.napoleon (pydata#3180)
  remove type annotations from autodoc method signatures (pydata#3179)
  Fix regression: IndexVariable.copy(deep=True) casts dtype=U to object (pydata#3095)
  Fix distributed.Client.compute applied to DataArray (pydata#3173)
  More annotations in Dataset (pydata#3112)
  Hotfix for case of combining identical non-monotonic coords (pydata#3151)
  changed url for rasterio network test (pydata#3162)
  to_zarr(append_dim='dim0') doesn't need mode='a' (pydata#3123)
  BUG: fix+test groupby on empty DataArray raises StopIteration (pydata#3156)
  Temporarily remove pynio from py36 CI build (pydata#3157)
  missing 'about' field (pydata#3146)
  Fix h5py version printing (pydata#3145)
  Remove the matplotlib=3.0 constraint from py36.yml (pydata#3143)
  disable codecov comments (pydata#3140)
  Merge broadcast_like docstrings, analyze implementation problem (pydata#3130)
  Update whats-new for pydata#3125 and pydata#2334 (pydata#3135)
  Fix tests on big-endian systems (pydata#3125)
  XFAIL tests failing on ARM (pydata#2334)
  Add broadcast_like. (pydata#3086)
  Better docs and errors about expand_dims() view (pydata#3114)
  ...
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.

4 participants