#0: Add bc to docker image for pgm dispatch math (#16030)

### Ticket Blocking @jbaumanTT ### Problem description the container doesn't come with `bc` by default ### What's changed Added `bc` for math I'm wondering if we should take it out later and put it directly as a `sudo apt install` command in the workflow, so we don't set the precedent of chucking stuff into the image. ### Checklist - [ ] Post commit CI passes - [ ] Blackhole Post commit (if applicable) - [ ] Model regression CI testing passes (if applicable) - [ ] Device performance regression CI testing passes (if applicable) - [ ] **(For models and ops writers)** Full [new models](https://github.com/tenstorrent/tt-metal/actions/workflows/full-new-models-suite.yaml) tests passes - [ ] New/Existing tests provide coverage for changes
tenstorrent · Dec 13, 2024 · 7bd827c · 7bd827c
1 parent 27852c2
commit 7bd827c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/scripts/docker/requirements-20.04.txt b/scripts/docker/requirements-20.04.txt
@@ -22,3 +22,4 @@ cargo
 ninja-build
 patchelf
 graphviz
+bc
diff --git a/scripts/docker/requirements-22.04.txt b/scripts/docker/requirements-22.04.txt
@@ -20,3 +20,4 @@ cargo
 ninja-build
 libxml2-dev
 libxslt-dev
+bc