8000 Revert "fix(sdk): Fix type annotation for `train` method's `trainer` parameter" by Electronic-Waste · Pull Request #2651 · kubeflow/trainer · GitHub
[go: up one dir, main page]

Skip to content

Conversation

Electronic-Waste
Copy link
Member

Reverts #2646

/cc @eoinfennessy @kubeflow/wg-training-leads @astefanutti

8000
@google-oss-prow google-oss-prow bot requested a review from a team May 27, 2025 12:56
Copy link

@Electronic-Waste: GitHub didn't allow me to request PR reviews from the following users: eoinfennessy.

Note that only kubeflow members and repo collaborators can review this PR, and authors cannot review their own PRs.

In response to this:

Reverts #2646

/cc @eoinfennessy @kubeflow/wg-training-leads @astefanutti

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

…paramete…"

This reverts commit 7b1b39b.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
@Electronic-Waste Electronic-Waste force-pushed the revert-2646-fix-trainer-type-annotation branch from 82fa4a4 to 75929d2 Compare May 27, 2025 12:59
@coveralls
Copy link
coveralls commented May 27, 2025

Pull Request Test Coverage Report for Build 15275924720

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 28.807%

Totals Coverage Status
Change from base Build 15265251857: 0.0%
Covered Lines: 840
Relevant Lines: 2916

💛 - Coveralls

Copy link
Member
@eoinfennessy eoinfennessy left a comment

Choose a reason for hiding this comment

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

Thanks for reverting this, and apologies for making the change -- I didn't realise that this was intended to be kept as is until after LLM Trainer V2 was fully implemented.

Copy link

@eoinfennessy: changing LGTM is restricted to collaborators

In response to this:

Thanks for reverting this, and apologies for making the change -- I didn't realise that this was intended to be kept as is until after LLM Trainer V2 was fully implemented.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@Electronic-Waste
Copy link
Member Author

Never mind @eoinfennessy! Appreciate your passion to Kubeflow!

Copy link
Member
@andreyvelich andreyvelich left a comment

Choose a reason for hiding this comment

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

Thank you for this @Electronic-Waste @eoinfennessy!
As @Electronic-Waste mentioned, once we merge all of the PRs for LLM Trainer, we can support BuiltinTrainer trainer type.

/lgtm
/approve

Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andreyvelich

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit 385cc17 into kubeflow:master May 27, 2025
17 checks passed
@google-oss-prow google-oss-prow bot added this to the v2.0 milestone May 27, 2025
@Electronic-Waste Electronic-Waste deleted the revert-2646-fix-trainer-type-annotation branch May 28, 2025 06:06
szaher pushed a commit to szaher/sdk that referenced this pull request Jun 4, 2025
…parameter" (kubeflow/trainer#2651)

This reverts commit c926ec0.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
szaher pushed a commit to szaher/sdk that referenced this pull request Jun 4, 2025
…parameter" (kubeflow/trainer#2651)

This reverts commit c926ec0.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
szaher pushed a commit to szaher/sdk that referenced this pull request Jun 4, 2025
…parameter" (kubeflow/trainer#2651)

This reverts commit c926ec0.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
szaher pushed a commit to szaher/sdk that referenced this pull request Jun 5, 2025
…parameter" (kubeflow/trainer#2651)

This reverts commit c926ec0.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
akagami-harsh pushed a commit to akagami-harsh/training-operator that referenced this pull request Jul 17, 2025
…parameter" (kubeflow#2651)

This reverts commit 7b1b39b.

Signed-off-by: Electronic-Waste <2690692950@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0