Skip to content

Conversation

@wtmlon
Copy link
Contributor

@wtmlon wtmlon commented Dec 12, 2025

No description provided.

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 7.54717% with 49 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (develop@a666ddb). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...paddlefleet/pipeline_parallel/pipeline_parallel.py 4.08% 46 Missing and 1 partial ⚠️
src/paddlefleet/gpt_builders.py 50.00% 1 Missing and 1 partial ⚠️

❌ Your patch status has failed because the patch coverage (7.54%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

Impacted file tree graph

@@ Coverage Diff @@ ## develop #181 +/- ## ========================================= Coverage ? 7.54% ========================================= Files ? 2 Lines ? 53 Branches ? 14 ========================================= Hits ? 4 Misses ? 47 Partials ? 2 
Flag Coverage Δ
coverage_combine 7.54% <7.54%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/paddlefleet/gpt_builders.py 50.00% <50.00%> (ø)
...paddlefleet/pipeline_parallel/pipeline_parallel.py 4.08% <4.08%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
state_dict[self._pp_to_single_mapping[k]] = v

return state_dict

Copy link
Contributor

Choose a reason for hiding this comment

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

这里不能删,可能会影响权重名称的映射,dpo 如果set_state_dict 有问题,是不是提供一个做了名字映射的方法进行覆盖

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@@ -291,6 +296,9 @@ def __init__(self, layers, strategy):
# default loss function index
self.loss_fn_idx = 0

def is_pipeline_last_stage(self, ignore_virtual=False):
Copy link
Collaborator

Choose a reason for hiding this comment

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

这个接口在哪里使用

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants