From 7bd827cbc6e3119be89d85e4f708f728c7bd8784 Mon Sep 17 00:00:00 2001
From: Raymond Kim <109366641+tt-rkim@users.noreply.github.com>
Date: Fri, 13 Dec 2024 18:22:02 -0500
Subject: [PATCH] #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
---
 scripts/docker/requirements-20.04.txt | 1 +
 scripts/docker/requirements-22.04.txt | 1 +
 2 files changed, 2 insertions(+)

diff --git a/scripts/docker/requirements-20.04.txt b/scripts/docker/requirements-20.04.txt
index fa95ad6aa74..d5401cc5bfe 100644
--- 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
index 64f463dfe27..3800f75f030 100644
--- 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