configure.pyをwindowsに対応 exampleをctypes化 #80
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
PRありがとうございます！ 結構な変更量ですね･･･！ @Yosshi999 さん、一緒にレビューお願いします 🙏 |
Yosshi999
requested changes
Feb 20, 2022
| core_dll_path = Path(os.path.dirname(__file__) + f"/lib/{lib_file}") | ||
| if not os.path.exists(core_dll_path): | ||
| raise Exception(f"coreライブラリファイルが{core_dll_path}に存在しません") | ||
| lib = cdll.LoadLibrary(str(core_dll_path)) |
There was a problem hiding this comment.
[QUESTION]
この件は他の方(@Hiroshiba etc.)にも試してほしいんですが、Python 3.7.11ではここで
OSError: [WinError 193] %1 は有効な Win32 アプリケーションではありません。
と表示されてdllをロードできませんでした。何か心当たりはありますか？
ちなみに3.8.12では動きました。
どうも3.8でwindowsのdllロードがいろいろ変わったっぽいんですがそれが原因かもしれません。
3.7対応を切るか、何らかのトリックを加える必要がありそう。
https://docs.python.org/ja/3/library/os.html#os.add_dll_directory
https://docs.python.org/ja/3/library/ctypes.html#ctypes.WinDLL
There was a problem hiding this comment.
レビューしていただき大変ありがとうございます
こちらでも、エラーの種類はアクセス不可領域へのアクセスのようで異なるものの、エラーを確認しました
以前の__init__.pyにあったdll読み込みのコードを用いることでこちらでは解決できましたが、そちらではどうでしょうか？
Co-authored-by: Yosshi999 <[email protected]>
…_core into windows_config
Yosshi999
approved these changes
Feb 20, 2022
Hiroshiba
approved these changes
Feb 20, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
configure.pyをwindowsに対応します
その際、.libファイルを.dllファイルから生成するために、PATHを通さない限りVS開発者コンソールでしか開けないdumpbinを使用する必要があったため、configure.pyのみで完結することができませんでした。
これを解決するため .libを必要としているcythonを排除し、ctypesを用いた実装に変更しました
configure.pyのwindows対応に合わせてREADMEを大幅に改稿しましたが、問題があるようであれば元に戻します
その他
Windows x64, WSL2でのみ動作検証しているので、動作検証にご協力いただければ幸いです
このPRがマージされ次第、DirectML版をPRする予定です