Fix CLI file argument deprecations#134
Fix CLI file argument deprecations#134neuralsorcerer wants to merge 1 commit intofacebookresearch:mainfrom
Conversation
|
Thanks @neuralsorcerer I see this adds dependency on Questions
Thanks. |
Since |
|
Thanks for teaching me this. I'll need another person to accept the diff, I believe it should land within 24 hours. Thanks for the contribution. |
|
@talgalili has imported this pull request. If you are a Meta employee, you can view this in D86629212. |
|
@neuralsorcerer this should land in the next ~1 hour. |
|
@talgalili merged this pull request in 5714d1c. |
Replaced deprecated argparse
FileTypeusage in the CLI withpathlib.Patharguments and relied on pandas to handle file opening, eliminating the need for manual handle cleanup and resolving thePendingDeprecationWarningnoise.