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

keras: 1.2.2 -> 2.0.3 #25239

Merged
merged 1 commit into from May 1, 2017
Merged

keras: 1.2.2 -> 2.0.3 #25239

merged 1 commit into from May 1, 2017

Conversation

jyp
Copy link
Contributor

@jyp jyp commented Apr 26, 2017

Motivation for this change

Using current version

Things done
  • Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@FRidh
Copy link
Member

FRidh commented Apr 30, 2017

cc maintainer @NikolaMandic

@FRidh
Copy link
Member

FRidh commented Apr 30, 2017

@jyp could you in the future prepend pythonPackages. to the name in your commit? That makes it easier to find Python-related PR's.

@NikolaMandic
Copy link
Contributor

@jyp did you actually try to import in in python and run it. not just building but simple import and usage of anything
if you did then I'd say it seems legit

@jyp
Copy link
Contributor Author

jyp commented Apr 30, 2017

@FRidh Sure, I will.

@NikolaMandic Yes, I actually have a couple of projects already running on top of this PR.

@NikolaMandic
Copy link
Contributor

then as far as I'm concerned seems legit

@7c6f434c
Copy link
Member

7c6f434c commented May 1, 2017

@NikolaMandic for all the aquatic mammal cuteness, using GitHub approve button is sometimes convenient…

@7c6f434c 7c6f434c merged commit 8972abb into NixOS:master May 1, 2017
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