Skip to content
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

Making h5py an optional dependency, not installed by default #2171

Merged
merged 3 commits into from
Oct 25, 2023

Conversation

raph-luc
Copy link
Member

@raph-luc raph-luc commented Oct 24, 2023

Addresses remaining vulnerability in #1552

To be superseded by #2096 when the pyfluent-reader package is published

By default, pip install ansys-fluent-core or installing from a pre-built wheel file won't install h5py anymore.

To get data and case file reader functionality, users will need to:

  • pip install ansys-fluent-core[reader]
  • pip install -e .[reader] for development
  • pip install ansys_fluent_core-0.19.dev0-py3-none-any.whl[reader] from a wheel file

@raph-luc raph-luc enabled auto-merge (squash) October 24, 2023 20:57
@raph-luc raph-luc merged commit 23c2828 into main Oct 25, 2023
18 checks passed
@raph-luc raph-luc deleted the fix/h5py_optional branch October 25, 2023 02:39
@raph-luc raph-luc mentioned this pull request Oct 25, 2023
raph-luc added a commit that referenced this pull request Oct 26, 2023
* Fix vale warnings (#2139)

* fix tensor type for displacement variable (#2145)

* Fix set_state implementation for command argument instance. (#2147)

* Update flobject.py (#2148)

* SVAR Doc (#1635)

* Test to catch Watchdog launch errors, and improved Watchdog behavior on Windows (#2144)

* Cavitation Model Example And Example Warning Fix (#2102)

* Add type annotations for some modules under services (#2108)

* More robust Windows launch command for Watchdog (#2167)

* Making h5py an optional dependency, not installed by default (#2171)

* Expose settings root like in pyconsole. (#2149)

* Remove timeout loop in FluentConnection (#2126)

* Fix SVAR doc (#2172)

---------

Co-authored-by: Mainak Kundu <[email protected]>
Co-authored-by: Oleg Chernukhin <[email protected]>
Co-authored-by: Prithwish Mukherjee <[email protected]>
Co-authored-by: Harshal Pohekar <[email protected]>
Co-authored-by: Aseem Jain <[email protected]>
Co-authored-by: Prithwish Mukherjee <[email protected]>
Co-authored-by: Adam Boutin <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants