-
Notifications
You must be signed in to change notification settings - Fork 60
feat: Initial bulk of type checking + setup #4761
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.
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
Open
Gobot1234
wants to merge
72
commits into
main
Choose a base branch
from
jhilton/typing-improvements
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 67 commits
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
4f2c990
Add type level support for passing Path instances to download_file
Gobot1234 1ddf8e8
Add typing_extensions
Gobot1234 7825ff4
Fix inheritance for Meshing
Gobot1234 90d69a0
Fix create_launcher allowing None
Gobot1234 654deb4
Update launchers typing to be more specific
Gobot1234 ec258c3
Fix ci
Gobot1234 00c3a85
Fix ci again
Gobot1234 b6d2e3e
remove accidental commits
Gobot1234 0a48f90
run pyupgrade for moving typing to builtins and a couple of other things
Gobot1234 7b36a1c
fix some np.arrays that are meant to be ndarrays in types
Gobot1234 c04b413
pre-fix the conflict that will happen
Gobot1234 08e5525
add types for services
Gobot1234 7c2e9c3
Merge branch 'jhilton/typing-improvements' of https://github.com/ansy…
Gobot1234 0acf4ea
fix some low hanging fruit
Gobot1234 e6a7bf9
Merge remote-tracking branch 'origin/main' into jhilton/typing-improv…
Gobot1234 2ea0029
fix broken imports
Gobot1234 f28ce78
chore: adding changelog file 4761.added.md [dependabot-skip]
pyansys-ci-bot 316128d
make the other imported modules not show up in __all__
Gobot1234 79956f3
pass bool
Gobot1234 4984170
respond to some review
Gobot1234 e130f13
add project infra for typechecking
Gobot1234 ee72ce6
test
Gobot1234 6e30e69
Merge branch 'main' into jhilton/typing-improvements
Gobot1234 0c7fb4b
Merge branch 'jhilton/typing-improvements' of https://github.com/ansy…
Gobot1234 f67ef50
gitattributes: enforce UTF-8 (no BOM) for patch and diff files
Gobot1234 e0ce93b
Add: public API regression test and fix type annotation issues
mayankansys 912a036
Merge branch 'main' into jhilton/typing-improvements
Gobot1234 dc85158
Update ci.yml
Gobot1234 9a58a1e
Revert ci stuff
Gobot1234 e5309b7
Merge branch 'jhilton/typing-improvements' of https://github.com/ansy…
Gobot1234 3d37b81
no idea if this fixes ci
Gobot1234 6c5e692
Apply suggestions from code review
Gobot1234 b8a4a85
Update src/ansys/fluent/core/launcher/watchdog.py
Gobot1234 b38db19
fix ci up slightly
Gobot1234 5395956
fix launching and make it more type safe
Gobot1234 eae0f5b
misc changes
Gobot1234 7338cf9
Fix ci issues
Gobot1234 5bdade1
fix more stuff?
Gobot1234 ab34346
more typing fixes ci is still confusing me
Gobot1234 c4f0732
far too conservative
Gobot1234 a82d391
fix object installer types
Gobot1234 535c6b7
fix ci and mode related bug
Gobot1234 f6feab8
more ci work (api codegen check types)
Gobot1234 77e9854
fix ci
Gobot1234 f9b639a
fix not passing down all params in from_connection
Gobot1234 d1c6f53
run pyright
Gobot1234 dc6ec0b
Merge branch 'main' into jhilton/typing-improvements
Gobot1234 feac0d2
Update src/ansys/fluent/core/launcher/standalone_launcher.py
Gobot1234 e8cb20f
downgrade to 261 type checking for now
Gobot1234 57bd15c
Try logging in first
Gobot1234 58aefc4
cache for code style changes
Gobot1234 1809f84
fix some implicit optional parameters & other bits and bobs
Gobot1234 e050944
fix invocation of python
Gobot1234 dedba6c
gen certs
Gobot1234 13c8692
Merge branch 'main' into jhilton/typing-improvements
Gobot1234 3decd50
bump pytest in ci_types
Gobot1234 96e56a8
typing ci fixes
Gobot1234 1bb1408
add more deps?
Gobot1234 1f07cca
idk what else is missing here
Gobot1234 120f42d
Fix certificate generation script for Windows
Gobot1234 61b6806
chore: adding changelog file 4761.added.md [dependabot-skip]
pyansys-ci-bot 4c1cb41
hopefully fix all the other issues
Gobot1234 3ac6141
Merge branch 'jhilton/typing-improvements' of https://github.com/ansy…
Gobot1234 d5596c6
Merge branch 'main' into jhilton/typing-improvements
Gobot1234 ba011d9
fix ci
Gobot1234 6e8cd22
okay for real this time
Gobot1234 b0aa392
I hope this fixes it
Gobot1234 83c443d
more errors
Gobot1234 bc787f1
fix the typing only symbol
Gobot1234 2de9fc2
Update slightly silly test
Gobot1234 c8e8999
Update examples/00-fluent/DOE_ML.py
Gobot1234 9517350
Update examples/00-fluent/DOE_ML.py
Gobot1234 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Empty file.
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| Initial bulk of type checking + setup |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.