-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Hey @Zha0q1 , Thanks for submitting the PR
CI supported jobs: [miscellaneous, windows-cpu, unix-cpu, windows-gpu, clang, centos-gpu, sanity, unix-gpu, centos-cpu, website, edge] Note: |
Why would we remove the |
The folder is invisible to setup.py. Let me see how I can keep the folder while still be able to pack the wheel correctly, I am guessing it's because the underscore? |
Yeah, maybe we can try changing the folder name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Move the _op_translations_opsetxx files outside the invisible _op_translation folder