Skip to content

[recipe, doc] feat: add Boogu-Image FlowGRPO v1 sync recipe - #30

Open
knlnguyen1802 wants to merge 1 commit into
mainfrom
feat/v1-sync-boogu-image-flowgrpo
Open

[recipe, doc] feat: add Boogu-Image FlowGRPO v1 sync recipe#30
knlnguyen1802 wants to merge 1 commit into
mainfrom
feat/v1-sync-boogu-image-flowgrpo

Conversation

@knlnguyen1802

Copy link
Copy Markdown
Owner

Summary

  • Add examples/flowgrpo_trainer/boogu_image/run_boogu_image_ocr_lora_v1.sh, a recipe-only v1 sync counterpart of the Boogu-Image OCR LoRA FlowGRPO script.
  • Mechanical v1 opt-in: main_diffusion_v1, trainer.use_v1=true, trainer.v1.trainer_mode=sync, and trainer.experiment_name=boogu_image_ocr_lora_v1.
  • Keeps the v0 Hydra overrides (LoRA targets, fsdp_layer_prefixes, request-level packing, 512 resolution, colocated 4-GPU actor/rollout/reward). Does not rewrite adapters or losses.
  • Documents the v1 launch in examples/flowgrpo_trainer/boogu_image/README.md only (docs/examples/boogu_image/flowgrpo_trainer_boogu_image.md is a symlink to that README).

Tracked by verl-project#389.

Why this is not duplicating an existing PR

Test plan

  • bash -n examples/flowgrpo_trainer/boogu_image/run_boogu_image_ocr_lora_v1.shpass
  • PR_TITLE='[recipe, doc] feat: add Boogu-Image FlowGRPO v1 sync recipe' python tests/special_sanity/check_pr_title.pypass (modules: ['recipe', 'doc'], type: feat)
  • Mechanical diff vs run_boogu_image_ocr_lora.sh: header + shebang + main_diffusion_v1 + experiment_name suffix + trainer.use_v1=true + trainer.v1.trainer_mode=sync only; LoRA, fsdp_layer_prefixes, packing, resolution, and colocated 4-GPU reward layout unchanged.
  • GPU training smoke (bash examples/flowgrpo_trainer/boogu_image/run_boogu_image_ocr_lora_v1.sh trainer.total_training_steps=1 trainer.logger='["console"]') — not run here (no GPU in this environment; recipe-only).
  • Shared examples/flowgrpo_trainer/README.md and docs/start/diffusion_v1.md were not edited.

AI assistance

AI assistance (Cursor) was used to draft the recipe, local README update, commit, and this PR. A human submitter has reviewed every changed line and is responsible for defending the change end-to-end. This is not a pure code-agent PR.

Made with Cursor

Port the existing OCR LoRA recipe onto the v1 sync trainer so this
verl-project#389 path can opt into TransferQueue + ReplayBuffer without changing
adapters, losses, or the colocated 4-GPU layout.

AI assistance (Cursor) was used for this change.

Co-authored-by: Cursor <cursoragent@cursor.com>
Signed-off-by: knlnguyen1802 <knlnguyen1802@gmail.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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