Open adam-dziedzic opened 2 years ago
Hi @adam-dziedzic,
Thanks a lot for your interest! I just tried locally using the repo I cloned when writing https://github.com/google-research/federated/issues/57#issuecomment-1062468566 and it worked, but when I try re-cloning the repo it produced the error you shared.
It seems like this commit (https://github.com/google-research/federated/commit/f80ba310e6dbb7bd5ea524330c660595e36363b9) introduced the symbol change; for workaround, maybe try using an earlier commit (e.g. 92e87a3) or just change the symbol manually to TensorboardReleaseManager
(lower case b
)?
This looks like a TFF change so I'll let others comment on whether a different TFF version should be used.
@adam-dziedzic @kenziyuliu This is coming up due to https://github.com/tensorflow/federated/commit/7fa77074dd0f799f0a2fdcea07b131041283b612. I believe that TFF is moving to a more regular release cadence, instead of relying on a nightly package, so the code in federated_research
has been changed to match that internally but obviously this does not work without a new TFF release.
The error above should be fixed simply by changing to TensorboardReleaseManager
manually. We are currently evaluating how to make sure that this repo is useful to external users even in the face of the TFF changes, so feedback is obviously welcome.
Hi All,
Thank you for publishing the code.
I installed the packages in the same way as recommended here: https://github.com/google-research/federated/issues/57#issuecomment-1062468566
However, I run in the following problem: