-
Notifications
You must be signed in to change notification settings - Fork 56
ci: Install dependencies first [skip tests] #4344
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
Closed
Closed
Conversation
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
prmukherj
approved these changes
Aug 4, 2025
seanpearsonuk
approved these changes
Aug 6, 2025
mkundu1
approved these changes
Aug 8, 2025
Added End to End PyFluent example on Single Cell Battery Simulation based on [Ansys Fluent documentation]( https://ansyshelp.ansys.com/public/account/secured?returnurl=/Views/Secured/corp/v242/en/flu_tg/flu_bat_tutorial_cell.html). Please review the same! Thanks! --------- Co-authored-by: pyansys-ci-bot <[email protected]> Co-authored-by: Abhishek Chitwar <[email protected]> Co-authored-by: achitwar <[email protected]>
useful for triaging --------- Co-authored-by: pyansys-ci-bot <[email protected]>
) While downloading the example file, if there is an `urllib.error.URLError` due to an `SSLError`, raise the underlying `SSLError`. For all other cases raise `RemoteFileNotFoundError` as before. Example SSLError: ``` ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1007) ``` Showing the underlying error will help users to triage connection issues in their system, e.g. the above error is due to CA certificates are not installed on the system. Fluent bug - 1320755 --------- Co-authored-by: pyansys-ci-bot <[email protected]>
Bypassing the server-info file-permission restriction for docker compose as a quick-fix. Added #4400 for triaging the root cause. --------- Co-authored-by: pyansys-ci-bot <[email protected]>
… in builtin_settingsgen (#4404) Co-authored-by: pyansys-ci-bot <[email protected]>
This pull request focuses on improving maintainability by extracting and explicitly defining allowed values for settings classes. The main change ensures that allowed values are clearly listed and type-annotated, which will help with code clarity and future maintenance. Settings class allowed values extraction: * Added the `_allowed_values` attribute to settings classes in both implementation and stub files, making allowed values explicit and type-annotated (`src/ansys/fluent/core/codegen/settingsgen.py`). * Documented the extraction of allowed values in the changelog (`doc/changelog.d/4342.maintenance.md`). --------- Co-authored-by: pyansys-ci-bot <[email protected]>
Update CHANGELOG for v0.35.dev0 and remove .md files in doc/changelog.d. > [!CAUTION] > Do not merge this pull request until the release has been successfully made.
…#4411) Potential fix for Fluent bug 1318884 --------- Co-authored-by: pyansys-ci-bot <[email protected]>
This pull request enhances the API data generation in `src/ansys/fluent/core/search.py` by introducing a mapping of object names specifically for meshing and solver sessions. The changes help organize API object names by session type, making them easier to access and utilize downstream. **API Data Structure Improvements:** * Added an `api_object_name_map` dictionary to categorize API object names under `meshing_session` and `solver_session` during data generation. * Updated the recursive tree traversal to populate `api_object_name_map` with relevant object names based on their session context. * Sorted and included the session-specific object name lists in the final `api_tree_data` output, making them available for further processing. --------- Co-authored-by: pyansys-ci-bot <[email protected]>
Create parent directory for server-info file if it does not exist. --------- Co-authored-by: pyansys-ci-bot <[email protected]>
) Co-authored-by: pyansys-ci-bot <[email protected]>
Release CI is broken - https://github.com/ansys/pyfluent/actions/runs/17295108997/job/49117558886 ``` Run actions/[email protected] Downloading single artifact Error: Unable to download artifact(s): Artifact not found for name: ansys-fluent-core-artifacts Please ensure that your artifact is not expired and the artifact was uploaded using a compatible version of toolkit/upload-artifact. For more information, visit the GitHub Artifacts FAQ: https://github.com/actions/toolkit/blob/main/packages/artifact/docs/faq.md ``` --------- Co-authored-by: pyansys-ci-bot <[email protected]>
Update CHANGELOG for v0.35.dev1 and remove .md files in doc/changelog.d. > [!CAUTION] > Do not merge this pull request until the release has been successfully made.
1. Remove the config equivalent to `PYFLUENT_FLUENT_ROOT` environment variable. That environment variable is only for development use. 2. Remove the mentions of the environment `PYFLUENT_LAUNCH_CONTAINER` from user documentation. That environment variable is only for CI use. 3. Fix a typo in slurm launcher. --------- Co-authored-by: pyansys-ci-bot <[email protected]>
…ests] (#4403) Updated the ``CONTRIBUTORS.md`` file with the latest contributors. --- This pull request was created by a bot. If you have any questions, please ping the PyAnsys Core team for assistance. To ping the team write a comment in this issue with the following mention: `@ansys/pyansys-core`. If you want this repository to be excluded from this automated maintenance process, please let us know by [filling in the opt-out request form](https://github.com/ansys-internal/pyansys-maintenance-automation/issues/new?labels=&projects=&template=opt-out.yml&title=Request+to+opt-out+from+automated+maintenance+for+ansys%2Fpyfluent). --------- Co-authored-by: pyansys-ci-bot <[email protected]>
Corresponding Fluent PR - 615951 When `_has_migration_adapter` is true for a setting object, we shall skip type-checking within set-state (not added in this PR). --------- Co-authored-by: pyansys-ci-bot <[email protected]>
|
The following people have not signed the Contributors License Agreement (CLA):
Read the CLA in the link above and sign it by clicking the link below: You will receive a confirmation as soon as your signature is captured. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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.