AI-Hypercomputer/maxtext

Support nsys profiler upload in all cases

Open

#911 aberto em 24 de set. de 2024

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)Python (527 forks)github user discovery
buggood first issue

Métricas do repositório

Stars
 (2.311 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

For both jax.profiler (profiler=xplane in maxtext) and a GPU nsys profiler (profiler=nsys in maxtext) we upload the profile to the base_output_directory (source)

Typically this directory is GCS, it can also be local. However for the nsys profiler we hardcode the uploader to use gsutil source, which has two problems

  1. Output directory may not be GCS, so gsutil is not applicable
  2. Hosts may not have gsutil installed, since gsutil is not in requirements.txt

We should modify the nsys profile upload to work in all cases.

Additional context - https://github.com/AI-Hypercomputer/maxtext/pull/909 was added as a temporary fix for 2 - we won't upload the profile when gsutil is missing, so training may continue

Guia do colaborador