Current State
test_converter_roundtrip.py already covers the core conversion logic, but it has meaningful gaps:
What is covered
merge_lora_to_hf() from convert_lora_to_hf.py (called directly via importlib)
convert_dcp_to_hf() from nemo_rl.utils.native_checkpoint (the underlying library function)
export_model_from_megatron() from nemo_rl.models.megatron.community_import (the underlying library function)
What is NOT covered
- The CLI entry points of
convert_dcp_to_hf.py and convert_megatron_to_hf.py — their parse_args() + main() paths are never exercised
convert_lora_to_hf.py's own main() is also not invoked — only the inner function is imported
Current State
test_converter_roundtrip.pyalready covers the core conversion logic, but it has meaningful gaps:What is covered
merge_lora_to_hf()fromconvert_lora_to_hf.py(called directly viaimportlib)convert_dcp_to_hf()fromnemo_rl.utils.native_checkpoint(the underlying library function)export_model_from_megatron()fromnemo_rl.models.megatron.community_import(the underlying library function)What is NOT covered
convert_dcp_to_hf.pyandconvert_megatron_to_hf.py— theirparse_args()+main()paths are never exercisedconvert_lora_to_hf.py's ownmain()is also not invoked — only the inner function is imported