Repository navigation
Retire the step-time chart; a .gitignore for this repository - #80
Merged
Merged
Conversation
AmitMY
force-pushed
the
simplify/bit-embedding
branch
from
October 10, 2026 09:12
b350f29 to
fb5beef
Compare
AmitMY
force-pushed
the
simplify/retire-step-time
branch
from
October 10, 2026 09:12
67defcd to
1cbb3ef
Compare
AmitMY
force-pushed
the
simplify/bit-embedding
branch
from
October 11, 2026 09:39
fb5beef to
bef353f
Compare
The optimization history (chart, per-attempt CSV and plot script) is done: its summary table and the list of what made training faster stay in the benchmarks README, the rest is in git history. The .gitignore was GitHub's 213-line Python template. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015Xm62qoeaTunFSU5GYCQ1f
AmitMY
force-pushed
the
simplify/retire-step-time
branch
from
October 11, 2026 09:39
1cbb3ef to
d0317db
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #79.
plot.py,step_time.csv,step_time.png, and the matplotlib dev dependency). The optimization history is done: the HF vs. Megatron-Bridge table and the list of what made training faster stay in the benchmarks README, and the per-attempt times and the HF harness are in git history (bc057a5)..gitignorewas GitHub's 213-line Python template. It is now the 17 lines this repository needs; the same files are ignored.6 files changed, +12 / −284.
🤖 Generated with Claude Code
https://claude.ai/code/session_015Xm62qoeaTunFSU5GYCQ1f
Note
Low Risk
Documentation and tooling cleanup only; no training or runtime code paths change.
Overview
Retires the per-iteration step-time chart workflow now that optimization history is frozen: removes
benchmarks/plot.py,benchmarks/step_time.csv, and the README image/plot instructions. The benchmarks README keeps the HF vs. Megatron-Bridge table and “what made it faster” narrative; reproduction is a singletorchruncommand, with the old HF harness and full attempt log pointed at git history (bc057a5).Drops
matplotlibfrom optional dev dependencies inpyproject.tomlsince nothing in-repo plots benchmarks anymore.Replaces the stock 213-line Python
.gitignorewith a short repo-specific list (core Python artifacts,output//wandb/, common editor folders) without changing what this project actually ignores in practice.Reviewed by Cursor Bugbot for commit d0317db. Bugbot is set up for automated code reviews on this repo. Configure here.