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

DELF: Create the output directory if it doesn't already exist #11248

Open

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

Contributor

khatchad commented Aug 7, 2024

Otherwise, the script fails if it doesn't exist.

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.


          Create the output directory if it doesn't already exist.

596084b

Otherwise, the script fails if it doesn't exist.

khatchad requested a review from andrefaraujo as a code owner

August 7, 2024 16:45

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