Skip to content

Conversation

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe:

  • It looks like the 'tensorflow-base' output doesn't have any tests.

@xhochy
Copy link
Member

xhochy commented May 18, 2022

Looks good. I can kick off builds for this tomorrow!

@h-vetinari
Copy link
Member Author

Looks good. I can kick off builds for this tomorrow!

That would be awesome! 🤩 ... and might be a record turn-around for a PR on this feedstock 😅

(though of course, I'm only riding on the coattails of previous work by @ngam & @hmaarrfk 😇)

@ngam
Copy link
Contributor

ngam commented May 18, 2022

All good with me, thanks for pushing along. Hopefully we will do the 2.9 soon too (I will start poking with it early next week)

@ngam
Copy link
Contributor

ngam commented May 19, 2022

@xhochy only potential issue to pay closer attention to here is XLA on osx. I believe if it doesn't work, it will just fail back safely (i.e. no clustering), but it is worth testing it. Everything else should be straightforward

@xhochy
Copy link
Member

xhochy commented May 19, 2022

XLA is heavily used in jaxlib so I would guess that it will also work here.

@xhochy
Copy link
Member

xhochy commented May 23, 2022

@hmaarrfk Builds are in https://anaconda.org/uwe.korn/repo and in https://anaconda.org/uwe.korn-tf-gpu/repo Can you do the usual review, copy, merge, please?

Logs:

@ngam
Copy link
Contributor

ngam commented May 23, 2022

🚨 https://github.com/tensorflow/tensorflow/releases/tag/v2.9.1
https://github.com/tensorflow/tensorflow/releases

Looks like tensorflow isn't supposed to work with protobuf 3.20... but it is passing our tests somehow??

    'protobuf >= 3.9.2',
    # TODO(b/182876485): Protobuf 3.20 results in linker errors on Windows
    # Protobuf 4.0 is binary incompatible with what C++ TF uses.
    # We need ~1 quarter to update properly.
    # See also: https://github.com/tensorflow/tensorflow/issues/53234
    # See also: https://github.com/protocolbuffers/protobuf/issues/9954
    # See also: https://github.com/tensorflow/tensorflow/issues/56077
    # This is a temporary patch for now, to patch previous TF releases.
    'protobuf >= 3.9.2, < 3.20',

but it seems only applicable to windows...? just flagging this here for your consideration before finalizing this

@hmaarrfk
Copy link
Contributor

"Luckily" for us we don't even package windows!

@njzjz
Copy link
Member

njzjz commented May 24, 2022

3.20 is OK for us, but per tensorflow/tensorflow#56077, protobuf 4.0 (not released yet) will break tensorflow. We may add <4.0:

    'protobuf >= 3.9.2, < 4.0',

@hmaarrfk
Copy link
Contributor

This isn't necessary as per protobuf and libprotobuf are pinned together
https://github.com/conda-forge/protobuf-feedstock/blob/main/recipe/meta.yaml#L29

@hmaarrfk hmaarrfk merged commit 139cd32 into conda-forge:main May 24, 2022
@hmaarrfk
Copy link
Contributor

thank you @xhochy

@ngam
Copy link
Contributor

ngam commented May 24, 2022

perfect 😅

thanks all, I was quite confused by their decision to release a patch for all releases but the note indicated it was only windows, and so I wasn't sure if this was needed beyond windows

@ngam
Copy link
Contributor

ngam commented May 24, 2022

we march forward, our linux (including cuda) builds are ready-ish, but not sure about the osx ones yet #240

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

6 participants