tensorflow / tensorflow/tensorboard

Build: Remove WORKSPACE dependency on TensorFlow

Open
#2,227 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

stat:awaiting tensorflower type:feature
Dominant language
TypeScript
Stars
7.2k
Forks
1.7k
Avg merge
4d 22h
Merged PRs (30d)
1

Description

As discussed during the SIG-TensorBoard meeting, heres an issue with more detail. @nfelt

Currently TensorBoard has an http_archive rule in the WORKSPACE file depending on tf. It's been around for a long time and hopefully we can migrate away from it soon.
Eg: https://github.com/tensorflow/tensorboard/blob/cc01bc1eb57d123511dc0d8a659cb19a9c979976/WORKSPACE#L66-L74

Most other tensorflow-related packages are now based off how https://github.com/tensorflow/custom-op works, if possible TensorBoard should do the same. That way it'll build against the exact version installed (and the exact version that distro packagers ship) Uncoupling the exact version of tensorflow would also make the whole ecosystem easier to build and CI can build against TF master more frequently.

I realize that removing it may take a while, and from the SIG-Build side, I'd love to find out more about exactly what parts of the TF tarball you need to build. I want SIG-Build to put together a common set of build rules that all these related packages can use so if tensorboard needs things that custom-op does not yet provide then i'd love to hear more.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the TensorBoard WORKSPACE file, especially the http_archive rule linked in the issue, and compare it with the tensorflow/custom-op build approach. Identify which parts of the TensorFlow tarball TensorBoard needs for builds. Done means removing the WORKSPACE dependency while preserving builds against the exact installed TensorFlow version.

Written by the indexing model from the issue text.

Assessment

Tech stack
tensorflow
Domain
build-system
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.