Skip to content

Conversation

@DrownFish19
Copy link
Collaborator

PR types

Bug fixes

PR changes

Models

Description

  1. Fix import error for python3.8
  2. Remove output_router_logits for predictor, issue [Bug]: Qwen2ForCausalLM.forward() got an unexpected keyword argument 'output_router_logits' #8583
@paddle-bot
Copy link

paddle-bot bot commented Jun 12, 2024

Thanks for your contribution!

@codecov
Copy link

codecov bot commented Jun 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.42%. Comparing base (4609d07) to head (29af5ea).
Report is 241 commits behind head on develop.

Additional details and impacted files
@@ Coverage Diff @@ ## develop #8584 +/- ## ======================================== Coverage 54.42% 54.42% ======================================== Files 632 632 Lines 99450 99451 +1 ======================================== + Hits 54128 54129 +1  Misses 45322 45322 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@ZHUI ZHUI left a comment

Choose a reason for hiding this comment

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

LGTM

@ZHUI ZHUI merged commit 5bdf751 into PaddlePaddle:develop Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants