This repository has been archived by the owner on Apr 4, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 3
make tensorflow-addons dependency optional #9
Comments
Correction: in fact it was quite simple.
So the Github release artifacts seem to be enough. |
Hmm, perhaps this recipe does not work, after all. When TF is loaded, it immediately complains…
Then later-on when starting Calamari training, TF fails with:
(Before you ask: no, unfortunately I cannot install TF 2.6 here.) |
No, that's actually not related (I managed to get rid of the warning by using TF addons v. 0.12.1 built against TF 2.4.1, but the exception remains). |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
In an HPC environment (where I want to
calamari-train
), with Power9 CPU (ppc64le
architecture), I getJudging by this issue I guess it will be quite painful to build this from source in the HPC environment (
bazel
etc).Since this seems to be needed only for under certain conditions (when the MovingAverageOptimizer gets used), I wonder if this could be made optional here.
The text was updated successfully, but these errors were encountered: