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.

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

Remove filecache #389

Merged
merged 1 commit into from
Mar 21, 2023
Merged

Remove filecache #389

merged 1 commit into from
Mar 21, 2023

Conversation

jwang47
Copy link
Contributor

@jwang47 jwang47 commented Mar 21, 2023

Remove filecache since it's causing confusion and it's not necessary at the moment.

@jwang47 jwang47 requested review from andrew-openai and zhangmarvin and removed request for andrew-openai March 21, 2023 21:25
Copy link
Contributor

@andrew-openai andrew-openai left a comment

Choose a reason for hiding this comment

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

LGTM

@jwang47 jwang47 merged commit e83f0ba into main Mar 21, 2023
@jwang47 jwang47 deleted the alvin/rm-filecache branch June 2, 2023 23:04
Linmj-Judy pushed a commit to TablewareBox/evals that referenced this pull request Feb 27, 2024
Remove filecache since it's causing confusion and it's not necessary at
the moment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants