Closed sumit0190 closed 2 years ago
We will also need to update tensorflow-estimator.
@tobijk Added an update for tensorflow-estimator
. I had to remove its test
section because I wasn't sure what we are supposed to test, and a "real" test would require importing tf
and verifying that the estimator methods exist. We could add a pip check
like I did with keras
.
I'm still not so happy about the run_constrained
part. Putting it there would mean that installing TF becomes optional, but AFAIK keras does not work without TF?
EDIT: Never mind, one can use keras without TF :)
Upgrading
tensorflow
to2.9.1
requires version2.9.0
forkeras
,tensorflow-estimator
andtensorboard
, which this PR tries to address.Concourse for
keras
: https://concourse.build.corp.continuum.io/teams/main/pipelines/keras (NOTE: This fails on certain platforms because of the issue described here).Concourse for
tensorboard
: https://concourse.build.corp.continuum.io/teams/main/pipelines/tensorboardConcourse for
tensorflow-estimator
: https://concourse.build.corp.continuum.io/teams/main/pipelines/tensorflow-estimatorkeras
andtensorflow-estimator
needtensorflow
as arun_constrained
dependency sincetensorflow
doesn't have a build with that version number (because it requireskeras
...a cyclic dependency of sorts).(NOTE: Keeping the base branch
master-2.8.2
for now, as it is the closest to where we want to be when we are ready to upgrade to2.9.1
.)