Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Platform-specific config in .bazelrc is not picked up by bazel build #7186

Closed
hujiajie opened this issue Dec 19, 2022 · 1 comment · Fixed by #7192
Closed

Platform-specific config in .bazelrc is not picked up by bazel build #7186

hujiajie opened this issue Dec 19, 2022 · 1 comment · Fixed by #7192
Assignees

Comments

@hujiajie
Copy link
Contributor

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template

System information

  • OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
  • Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
  • TensorFlow.js installed from (npm or script link):
  • TensorFlow.js version:
  • CUDA/cuDNN version:

Describe the problem
It seems build:linux --spawn_strategy=dynamic,remote,sandboxed,worker,local --internal_spawn_scheduler just has no effect. I noticed this because sandboxed build of WASM is painfully slow, so I tried to remove dynamic and sandboxed from this line, but the build time didn't boost after that. I suspect a build --enable_platform_specific_config directive is missing there.

Provide the exact sequence of commands / steps that you executed before running into the problem

Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@google-ml-butler
Copy link

Are you satisfied with the resolution of your issue?
Yes
No

mattsoulanille added a commit that referenced this issue Dec 21, 2022
Fix #7186

Co-authored-by: Ping Yu <4018+pyu10055@users.noreply.github.com>
Linchenn pushed a commit to Linchenn/tfjs that referenced this issue Jan 9, 2023
Fix tensorflow#7186

Co-authored-by: Ping Yu <4018+pyu10055@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants