Skip to content

Comments

Github Logs in WANDB Notes Section#312

Merged
mpragnay merged 3 commits into3.0_betafrom
Add-Github-branch-info-to-Wandb-Logs
Feb 23, 2026
Merged

Github Logs in WANDB Notes Section#312
mpragnay merged 3 commits into3.0_betafrom
Add-Github-branch-info-to-Wandb-Logs

Conversation

@mpragnay
Copy link

Added branch name, latest commit hash, latest commit message, for better tracking of wandb runs.
The details appear in the wandb notes section:
Screenshot 2026-02-23 at 2 38 45 PM

Copilot AI review requested due to automatic review settings February 23, 2026 19:38
@greptile-apps
Copy link

greptile-apps bot commented Feb 23, 2026

Greptile Summary

Added git repository metadata (branch name, commit hash, commit message) to WandB run notes for better tracking of experimental runs.

  • Captures git state when initializing WandbLogger using subprocess commands
  • Falls back to error message if git commands fail (not in repo, git not installed, etc.)
  • Minor issue: redundant pass statement in exception handler

Confidence Score: 4/5

  • This PR is safe to merge with minimal risk
  • Simple, well-contained addition that captures git metadata for WandB tracking. Proper error handling prevents failures in non-git environments. Only minor style issue with redundant pass statement.
  • No files require special attention

Important Files Changed

Filename Overview
pufferlib/pufferl.py Added git repository metadata to WandB notes; includes redundant pass statement in exception handler

Last reviewed commit: dde8710

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

git_notes = (
"Error fetching git info. Make sure you're running this in a git repository and have git installed."
)
pass
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

redundant pass statement - exception block already handles assignment

Suggested change
pass

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds Git repository metadata to the WANDB notes section to improve tracking of experiment runs. The implementation captures the current branch name, latest commit hash, and commit message, then displays them in the WANDB notes field.

Changes:

  • Added Git metadata extraction using subprocess commands in the WandbLogger initialization
  • Implemented error handling for cases where Git is not available or the code is not in a Git repository
  • Formatted the Git information as markdown for display in WANDB's notes section

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

mpragnay and others added 2 commits February 23, 2026 14:56
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@mpragnay mpragnay merged commit a38327b into 3.0_beta Feb 23, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants