Update tensorflow-haskell dependencies. #88604
Conversation
|
@GrahamcOfBorg build haskellPackages.tensorflow-proto |
|
Thanks for updating this. Can you mark the following Haskell packages as non-broken in pkgs/development/haskell-modules/configuration-hackage2nix.yaml around line 9900? (All you need to do is remove these lines from that file)
|
Updating to the current HEAD of the Tensorflow-Haskell bindings allows us to also update the dependencies, specifically proto-lens, and avoid having to retain their outdated versions.
mikesperber
force-pushed
the
update-tensorflow-haskell-dependencies
branch
from
b619b9b
to
d93aa81
Compare
|
@cdepillabout Done. |
|
Thanks, looks like these all compile! |
|
Oops, it looks like I'm not sure why this would be the case. Maybe you can try running If you can't figure it out, we can ask peti. |
|
@cdepillabout Good catch, thanks! I suspect that's because the I suggest just unbreaking it in |
Yeah, that is probably the easiest thing to do right now. Although, the better thing would be to actually get it on Hackage. I guess it doesn't make sense to have packages on Hackage if they have dependencies that are not on Hackage. Although I don't know why |
|
@cdepillabout I've asked: |
|
@cdepillabout So I've written to LeCun, but the whole process - if it's successful at all - will take a while. OK, in the meantime, to just unbreak tensorflow-mnist? |
|
@mikesperber It would probably be easier just to take out the |
Updating to the current HEAD of the Tensorflow-Haskell bindings allows
us to also update the dependencies, specifically proto-lens, and avoid
having to retain their outdated versions.
Motivation for this change
I finally got patches into the Tensorflow Haskell bindings to update the dependencies. This enables removing lots of cruft.
Things done
sandboxinnix.confon non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"./result/bin/)nix path-info -Sbefore and after)