DELF: Fix CSV read error #11249

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

Open

khatchad wants to merge 1 commit into tensorflow:master from ponder-lab:contrib_delf_fix_csv

Contributor

khatchad commented Aug 7, 2024 •

edited

Loading

Fix #10474. CSVs should be text files.

Type of change

Bug fix (non-breaking change which fixes an issue)

Tests

I followed the instructions at https://github.com/tensorflow/models/blob/b4c7ea90e8e9209c568d1b59f71432cfd69bb688/research/delf/delf/python/training/README.md#prepare-the-data-for-training.

Checklist

I have signed the Contributor License Agreement.
I have read guidelines for pull request.
My code follows the coding guidelines.
I have performed a self code review of my own code.
I have commented my code, particularly in hard-to-understand areas.
My changes generate no new warnings.


          Fix tensorflow#10474.

03aa7c9

CSVs should be text files.

khatchad requested a review from andrefaraujo as a code owner

August 7, 2024 16:48

laxmareddyp added the models:research label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

models:research