Skip to content

Update to haddock files for tensorflow-0.3 package (TensorFlow 2.3.0). #269

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

Merged
merged 19 commits into from
Nov 13, 2020

Conversation

jcmartin
Copy link
Contributor

As promised, this pull request completes the last thing remaining (excluding feature requests) for TensorFlow 2.3.0: the haddock files. There was also a minor edit to refer to 2.3.0-gpu and not 1.14.0-gpu in the README.

It would be very nice if those with the right permissions could push this upstream to Hackage.

For someone who has the spare cycles, it would be nice to push this up into Stackage as well:
https://github.com/commercialhaskell/stackage/blob/master/MAINTAINERS.md#adding-a-package

At this point in time, to add to Stackage, one must ensure compatibility with GHC 8.10.2 since that is what version Nightly is currently using. Special attention should be given to editing debian-bootstrap.sh to ensure that libtensorflow-2.3.0 and protoc-3.13.0 is properly installed along with any other non-Haskell dependencies (see ci_build/Dockerfile for the complete list).

Copy link
Contributor

@blackgnezdo blackgnezdo left a comment

Choose a reason for hiding this comment

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

I there are some excess files, e.g. we have both
A docs/haddock/tensorflow-0.2.0.1/TensorFlow-Build.html (0)
...
A docs/haddock/tensorflow-0.3.0.0/TensorFlow-Build.html (0)

Maybe you meant to keep only the latter directory?

@jcmartin
Copy link
Contributor Author

Good catch. I had some issues with tools/haddock.sh, and it seems I didn't fully resolve it.

@jcmartin jcmartin requested a review from blackgnezdo November 13, 2020 05:21
@blackgnezdo blackgnezdo merged commit 6b19e54 into tensorflow:master Nov 13, 2020
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.

3 participants