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 stray quotation marks in question_full.jsonll #12

Merged

Conversation

cfiken
Copy link

@cfiken cfiken commented Jan 31, 2024

Why are these changes needed?

There is a stray quotation mark in JSON format in question_full.jsonl so that one raises error on json.load()

Related issue number (if applicable)

none

Checks

  • I've run format.sh to lint the changes in this PR.
  • I've included any doc changes needed.
  • I've made sure the relevant tests are passing (if applicable).

polm-stability pushed a commit to polm-stability/FastChat that referenced this pull request Mar 1, 2024
@kaisugi
Copy link

kaisugi commented Mar 4, 2024

@polm-stability @leemengtw
Could you review this PR? 🙏

@polm-stability
Copy link

Sorry for never replying to this! Not sure how I missed either the initial issue with the quote or this PR. It looks like a good fix to me 👍

@polm-stability polm-stability merged commit a07fec9 into Stability-AI:jp-stable Apr 12, 2024
3 checks passed
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