Skip to content

Commit

Permalink
#11854: Move .umd that houses cluster descriptor to TT_METAL_HOME (#1…
Browse files Browse the repository at this point in the history
…2113)

#11854: Move .umd that houses cluster descriptor to TT_METAL_HOME, as in production Python environments with the wheel, we don't want .umd to be generated in the package directories for Python. One problem is sometimes the packages directory is owned by root, but users shouldn't have to be root to use packages
  • Loading branch information
tt-rkim authored Aug 30, 2024
1 parent db3a60f commit 2e14e61
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tt_metal/llrt/tt_cluster.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -96,10 +96,10 @@ void Cluster::detect_arch_and_target() {
std::filesystem::path get_cluster_desc_yaml() {
namespace fs = std::filesystem;

// RK: We eventually need to take out the create-ethernet-map binary and use it
// as a binary in the environment
// TODO: The interface into create-ethernet-map should be through an API rather a
// subprocess call to the binary
const fs::path tt_metal_dir = fs::path(tt::llrt::OptionsG.get_root_dir()) / "tt_metal";
const fs::path umd_path = tt_metal_dir / ".umd";
const fs::path umd_path = fs::path(tt::llrt::OptionsG.get_root_dir()) / ".umd";
fs::create_directory(umd_path);
const fs::path cluster_desc_path = umd_path / "cluster_desc.yaml";
if (!fs::exists(cluster_desc_path)) {
Expand Down

0 comments on commit 2e14e61

Please sign in to comment.