Skip to content
This repository has been archived by the owner on Aug 30, 2024. It is now read-only.

Fix warmup prompt length and add ns_log_level control #39

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Fix warmup prompt length and add ns_log_level control #39

merged 1 commit into from
Jan 10, 2024

Conversation

luoyu-intel
Copy link
Contributor

Type of Change

  1. fix the bug of default value -1 of n_predict.
  2. enable warmup when printing timings.

DDEle
DDEle approved these changes Jan 9, 2024
View reviewed changes
Copy link
Contributor

@DDEle DDEle left a comment

Choose a reason for hiding this comment

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

LGTM

a32543254
a32543254 approved these changes Jan 10, 2024
View reviewed changes
Copy link
Contributor

@a32543254 a32543254 left a comment

Choose a reason for hiding this comment

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

LGTM

@VincyZhang VincyZhang merged commit 070b6b9 into main Jan 10, 2024
10 checks passed
@luoyu-intel luoyu-intel deleted the fix_warmup branch January 10, 2024 07:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@zhenwei-intel zhenwei-intel zhenwei-intel approved these changes

@a32543254 a32543254 a32543254 approved these changes

@DDEle DDEle DDEle approved these changes

@airMeng airMeng Awaiting requested review from airMeng

@kevinintel kevinintel Awaiting requested review from kevinintel

Assignees
No one assigned
Labels
ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@luoyu-intel @DDEle @a32543254 @zhenwei-intel @airMeng @VincyZhang