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

haskell-modules: update tensorflow for 1.14 #67952

Closed
wants to merge 2 commits into from

Conversation

yorickvP
Copy link
Contributor

@yorickvP yorickvP commented Sep 2, 2019

I've updated the tensorflow packages so they work with tensorflow 1.14.
Please let me know if there is a better way to put this in nixpkgs.

Notify maintainers

cc @basvandijk @peti

cabal2nix . --subpath $f
echo ') {};'
done
echo }) | sed 's1src = ./.;1src = tensorflow-haskell-src;1' | nixfmt
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great if there was a separate updater script that basically did the above.

A bunch of other places in nixpkgs have an actual updater script that bumps versions and re-runs commands like this.

Having a script like this would make it much easier for other maintainers to keep the tensorflow stuff updated.

@peti
Copy link
Member

peti commented Sep 2, 2019

Where do the packages listed in pkgs/development/haskell-modules/tensorflow-packages.nix come from? Are those packages that are on Hackage?

@@ -0,0 +1,310 @@
{ fetchFromGitHub }:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was thinking that these tensorflow packages could probably go in the non-hackage-packages.nix file (since that is what they are), but if you also add an update script, it might just be easier to keep them in a separate file (so tensorflow-packages.nix as you currently have it).

@cdepillabout
Copy link
Member

@GrahamcOfBorg build haskellPackages.tensorflow haskellPackages.tensorflow-core-ops haskellPackages.tensorflow-logging haskellPackages.tensorflow-opgen haskellPackages.tensorflow-ops haskellPackages.tensorflow-proto haskellPackages.tensorflow-records haskellPackages.tensorflow-records-conduit haskellPackages.tensorflow-test

Copy link
Member

@peti peti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this PR package non-Hackage versions of those packages?

Shouldn't we distribute released versions?

@cdepillabout
Copy link
Member

@peti It looks like the tensorflow libraries don't build with the latest proto-lens-protoc. yorickvP has apparently fixed them up and sent a PR upstream:

tensorflow/haskell#247

I have confirmed that all the packages from this PR build correctly, but I imagine it might just be easier for us to wait until upstream merges in the above PR and makes a new release. When that happens we will be able to get rid of the stuff from this PR.

@yorickvP
Copy link
Contributor Author

yorickvP commented Sep 2, 2019

I've asked upstream to do a release

@yorickvP yorickvP closed this Sep 2, 2019
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

3 participants