Minor updates to CLIMB tutorial#2072
Conversation
Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>
Greptile SummaryThis PR makes three small corrections to the Nemotron-CLIMB tutorial: it fixes the broken
Confidence Score: 5/5All three changes are straightforward corrections with no functional risk — the import is now valid and points to the confirmed location of get_deterministic_hash, and the deferred imports in utils.py follow standard Python patterns. The changes fix a broken import, defer heavy optional imports to avoid eager loading, and correct a title typo. The function being imported exists at the referenced path with a compatible signature, and the TYPE_CHECKING pattern used in utils.py is well-understood by the relevant type checkers. No files require special attention. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[3_prune.py] -->|"from nemo_curator.utils.hash_utils import get_deterministic_hash"| B[hash_utils.py]
A -->|imports| C[utils.py]
C -->|TYPE_CHECKING only| D["nemo_curator.core.client.RayClient (type annotation)"]
C -->|lazy import inside attach_ray_client_args| E[nemo_curator.core.constants]
C -->|lazy import inside create_ray_client| F[nemo_curator.core.client.RayClient]
A -->|uses| G["get_deterministic_hash(source_files, task.task_id)"]
B --> G
Reviews (4): Last reviewed commit: "Merge branch 'main' into climb_nvbug" | Re-trigger Greptile |
Signed-off-by: Sarah Yurick <sarahyurick@gmail.com>
Fixes
get_deterministic_hashimport and editsutils.pyto not use top-level Curator imports.