Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make prompt shorter to fit 2048 window size #78

Merged
merged 4 commits into from
Oct 26, 2023
Merged

Make prompt shorter to fit 2048 window size #78

merged 4 commits into from
Oct 26, 2023

Conversation

sharonwx54
Copy link
Collaborator

@sharonwx54 sharonwx54 commented Oct 25, 2023 •
edited by lwaekfjlk
Loading

Closes #80

📑 Description

Removing format string from prompt engineering as we don't need to include it for FT.

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • Branch name follows type/descript (e.g. feature/add-llm-agents)
  • Ready for code review

ℹ Additional Information

@sharonwx54 sharonwx54 requested a review from lwaekfjlk October 25, 2023 23:24
@sharonwx54 sharonwx54 self-assigned this Oct 25, 2023
@lwaekfjlk lwaekfjlk changed the title Feature/remove prompt format Make prompt shorter to fit 2048 window size Oct 26, 2023
lwaekfjlk
lwaekfjlk approved these changes Oct 26, 2023
View reviewed changes
Copy link
Member

@lwaekfjlk lwaekfjlk left a comment

Choose a reason for hiding this comment

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

Nice!

@lwaekfjlk lwaekfjlk merged commit 0b5d982 into main Oct 26, 2023
3 checks passed
lwaekfjlk pushed a commit that referenced this pull request Nov 16, 2023
@sharonwx54 @lwaekfjlk
Make prompt shorter to fit 2048 window size (#78)
6fbaa75 
* remove format in prompt

* small bug in filtering

* remove one line of format

* adding sliding window to remove dialogue but keep context when surpassing 2048 tokens
lwaekfjlk pushed a commit that referenced this pull request Nov 17, 2023
@zhisbug
update nginx deployment HTTPS etc. (#78)
0934f63
lwaekfjlk pushed a commit that referenced this pull request Nov 17, 2023
@sharonwx54 @lwaekfjlk
Make prompt shorter to fit 2048 window size (#78)
1c715b6 
* remove format in prompt

* small bug in filtering

* remove one line of format

* adding sliding window to remove dialogue but keep context when surpassing 2048 tokens
lwaekfjlk pushed a commit that referenced this pull request Mar 14, 2024
@zhisbug
update nginx deployment HTTPS etc. (#78)
d02445d
lwaekfjlk pushed a commit that referenced this pull request Mar 14, 2024
@sharonwx54 @lwaekfjlk
Make prompt shorter to fit 2048 window size (#78)
cbbc1e6 
* remove format in prompt

* small bug in filtering

* remove one line of format

* adding sliding window to remove dialogue but keep context when surpassing 2048 tokens
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lwaekfjlk lwaekfjlk lwaekfjlk approved these changes

Assignees

@sharonwx54 sharonwx54

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT]: Make the prompt shorter to fit in 2048 window size
2 participants
@sharonwx54 @lwaekfjlk