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.

Sign up for GitHub

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

Jump to bottom

python37Packages.opt-einsum: 3.2.1 -> 3.3.0 #93907

Merged
merged 1 commit into from Jul 31, 2020
Merged

python37Packages.opt-einsum: 3.2.1 -> 3.3.0 #93907

merged 1 commit into from Jul 31, 2020

Conversation

r-ryantm
Copy link
Contributor

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python3.7-opt_einsum/versions.

meta.description for python37Packages.opt-einsum is: "Optimizing NumPy's einsum function with order optimization and GPU support."

meta.homepage for python37Packages.opt-einsum is: "https://github.com/dgasmith/opt_einsum"

meta.changelog for python37Packages.opt-einsum is: ""

Updates performed
  • Version update
To inspect upstream changes
Impact
Checks done (click to expand)
Rebuild report (if merged into master) (click to expand) 
90 total rebuild path(s)

24 package rebuild(s)

24 x86_64-linux rebuild(s)
22 i686-linux rebuild(s)
22 x86_64-darwin rebuild(s)
22 aarch64-linux rebuild(s)


First fifty rebuilds by attrpath
python37Packages.arviz
python37Packages.baselines
python37Packages.dm-sonnet
python37Packages.edward
python37Packages.graph_nets
python37Packages.mask-rcnn
python37Packages.opt-einsum
python37Packages.optuna
python37Packages.pymc3
python37Packages.pyro-ppl
python37Packages.rl-coach
python37Packages.tensorflow
python37Packages.tensorflow-bin_2
python37Packages.tensorflow-build
python37Packages.tensorflow-build_1
python37Packages.tensorflow-build_2
python37Packages.tensorflow-probability
python37Packages.tensorflowWithCuda
python37Packages.tensorflowWithoutCuda
python37Packages.tensorflow_1
python37Packages.tensorflow_2
python37Packages.tflearn
python38Packages.opt-einsum
python38Packages.pyro-ppl
Instructions to test this update (click to expand)

Either download from Cachix:

nix-store -r /nix/store/1qzfx99isaj1qpd076i0c1rfh3bdz64n-python3.7-opt_einsum-3.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.

Or, build yourself:

nix-build -A python37Packages.opt-einsum https://github.com/r-ryantm/nixpkgs/archive/6e12b2f89cd47e1d8cb9ca64e23431a93e5f7bc2.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:

ls -la /nix/store/1qzfx99isaj1qpd076i0c1rfh3bdz64n-python3.7-opt_einsum-3.3.0
ls -la /nix/store/1qzfx99isaj1qpd076i0c1rfh3bdz64n-python3.7-opt_einsum-3.3.0/bin

Pre-merge build results

We have automatically built all packages that will get rebuilt due to
this change.

This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.

Result of nixpkgs-review 1

17 packages failed to build:
  • python37Packages.arviz
  • python37Packages.baselines
  • python37Packages.dm-sonnet
  • python37Packages.edward
  • python37Packages.graph_nets
  • python37Packages.mask-rcnn
  • python37Packages.optuna
  • python37Packages.pymc3
  • python37Packages.pyro-ppl
  • python37Packages.rl-coach
  • python37Packages.tensorflow (python37Packages.tensorflowWithoutCuda)
  • python37Packages.tensorflow-bin_2
  • python37Packages.tensorflow-build_2
  • python37Packages.tensorflow-probability
  • python37Packages.tensorflowWithCuda
  • python37Packages.tflearn
  • python38Packages.pyro-ppl
2 packages built:
  • python37Packages.opt-einsum
  • python38Packages.opt-einsum
Maintainer pings

cc @teh for testing.

@ofborg ofborg bot requested a review from teh July 26, 2020 12:44
@FRidh
Copy link
Member

FRidh commented Jul 26, 2020

...all reverse dependencies are failing?

@teh teh self-requested a review July 27, 2020 08:35
@ruuda
Copy link
Contributor

ruuda commented Jul 31, 2020

It looks like 3.2.1 is broken in the latest nixpkgs-unstable, but I can build 3.3.0.

builder for '/nix/store/xmhxh0hrf1qgznmvav8m59dalj7a2zra-python3.7-opt_einsum-3.2.1.drv' failed with exit code 1; last 10 log lines:
      
          # If no optimization, run pure einsum
          if optimize is False:
              if specified_out:
                  kwargs['out'] = out
  >           return c_einsum(*operands, **kwargs)
  E           ValueError: order must be one of 'C', 'F', 'A', or 'K' (got 'W')
  
  /nix/store/1ddc4m9alb8av6mba0vhn8yv62z9w5si-python3.7-numpy-1.19.0/lib/python3.7/site-packages/numpy/core/einsumfunc.py:1350: ValueError
  ============= 1 failed, 5660 passed, 155 skipped in 22.75 seconds ==============

@raboof raboof mentioned this pull request Jul 31, 2020
Closed
10 tasks
@jonringer
Copy link
Contributor

this seems to fix the build of a broken package. Merging.

closes: #93107

@jonringer jonringer merged commit 21b77ce into NixOS:master Jul 31, 2020
@r-ryantm r-ryantm deleted the auto-update/python3.7-opt_einsum branch August 1, 2020 14:13
@ajs124 ajs124 mentioned this pull request Aug 18, 2020
Closed
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FRidh FRidh Awaiting requested review from FRidh

@jonringer jonringer Awaiting requested review from jonringer

@teh teh Awaiting requested review from teh

Assignees
No one assigned
Labels
6.topic: python 10.rebuild-darwin: 11-100 10.rebuild-linux: 11-100
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@r-ryantm @FRidh @ruuda @jonringer @teh