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

Fix issues in perplexity.py script #110

Merged
merged 4 commits into from
Feb 20, 2024
Merged

Conversation

aahouzi
Copy link
Member

@aahouzi aahouzi commented Feb 2, 2024

Type of Change

Introduces a fix to some issues when measuring perplexity with your script:

  • Add missing package to requirements.txt
  • n_threads default value to support Windows & Linux platforms.

Description

  • Same as above

Expected Behavior & Potential Risk

  • N/A

How has this PR been tested?

  • N/A

Dependency Change?

  • Add matplotlib package.

@aahouzi aahouzi requested a review from DDEle February 5, 2024 09:52
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

@airMeng airMeng merged commit 0bca577 into intel:main Feb 20, 2024
6 checks passed
@aahouzi aahouzi deleted the fix-perplexity branch February 21, 2024 08:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants