From 2e14e610caee7bb411c23aeddc66fe6bf447a520 Mon Sep 17 00:00:00 2001 From: Raymond Kim <109366641+tt-rkim@users.noreply.github.com> Date: Fri, 30 Aug 2024 18:34:13 -0400 Subject: [PATCH] #11854: Move .umd that houses cluster descriptor to TT_METAL_HOME (#12113) #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 --- tt_metal/llrt/tt_cluster.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tt_metal/llrt/tt_cluster.cpp b/tt_metal/llrt/tt_cluster.cpp index 2cb94178022..0ee7ee7c5df 100644 --- a/tt_metal/llrt/tt_cluster.cpp +++ b/tt_metal/llrt/tt_cluster.cpp @@ -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)) {