Skip to content

Support vanilla CI - #1

Merged
fzyzcjy merged 105 commits into
mainfrom
feat/simple_ci
Oct 9, 2025
Merged

Support vanilla CI#1
fzyzcjy merged 105 commits into
mainfrom
feat/simple_ci

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator

p.s. wandb can be seen here: https://wandb.ai/ch271828n-team/projects

@fzyzcjy fzyzcjy closed this Oct 9, 2025
@fzyzcjy fzyzcjy reopened this Oct 9, 2025
@fzyzcjy

fzyzcjy commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator Author
image

@fzyzcjy
fzyzcjy merged commit ca8a419 into main Oct 9, 2025
2 checks passed
fzyzcjy added a commit that referenced this pull request Mar 1, 2026
- #1: Move argparse import to top-level in parallel_utils.py
- #2: resolve_megatron_path returns Path instead of str
- #3: Log non-zero comparator exit code in compare.py
- #4: Use `is not None` instead of truthiness for consistency in compare.py
- #5: Replace dataclasses.asdict with explicit kwargs, remove type: ignore
- #6: Add PreTrainedTokenizerBase type annotation in prompt_utils.py
- #7: Replace assert with ValueError in _resolve_raw_text
- #8: Handle default_factory in DataclassArgparseBridge
- #9: Document bool | None as unsupported in argparse_utils docstring
- #10-#12: Add missing type annotations (model_provider, forward_backward_func,
  losses, records)
fzyzcjy added a commit that referenced this pull request Mar 19, 2026
GuanxingLu pushed a commit to GuanxingLu/miles that referenced this pull request Apr 21, 2026
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.

1 participant