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

Add instructions for manual installation of ansys-api-fluent #508

Merged
merged 1 commit into from
Jun 16, 2022

Conversation

mkundu1
Copy link
Contributor

@mkundu1 mkundu1 commented Jun 15, 2022

Having ansys-api-fluent as a git dependency is problematic as a C++ compiler is required to build its .whl file. Adding an instruction to manually install the package till we have it in PyPI.

@mkundu1 mkundu1 closed this Jun 15, 2022
@mkundu1 mkundu1 reopened this Jun 15, 2022
Copy link
Collaborator

@dnwillia-work dnwillia-work left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting, why is a C++ compiler required.

@mkundu1
Copy link
Contributor Author

mkundu1 commented Jun 15, 2022

Interesting, why is a C++ compiler required.

What I've found - ansys-api-fluent has a build-time dependency on ansys_tools_protoc_helper which depends on an older version of grpcio-tools (1.17.0) which doesn't have a .whl file for python 3.9 in the PyPI. So the .whl is compiled for grpcio-tools and that has a C++ dependency as it uses python extensions.

@dnwillia-work
Copy link
Collaborator

Interesting, why is a C++ compiler required.

What I've found - ansys-api-fluent has a build-time dependency on ansys_tools_protoc_helper which depends on an older version of grpcio-tools (1.17.0) which doesn't have a .whl file for python 3.9 in the PyPI. So the .whl is compiled for grpcio-tools and that has a C++ dependency as it uses python extensions.

In the readme:

https://github.521000.best.mcas.ms/ansys/ansys-tools-protoc-helper

it says you can use a newer version of grpcio-tools right? Does that matter / help?

@mkundu1
Copy link
Contributor Author

mkundu1 commented Jun 15, 2022

Interesting, why is a C++ compiler required.

What I've found - ansys-api-fluent has a build-time dependency on ansys_tools_protoc_helper which depends on an older version of grpcio-tools (1.17.0) which doesn't have a .whl file for python 3.9 in the PyPI. So the .whl is compiled for grpcio-tools and that has a C++ dependency as it uses python extensions.

In the readme:

https://github.521000.best.mcas.ms/ansys/ansys-tools-protoc-helper

it says you can use a newer version of grpcio-tools right? Does that matter / help?

ansys-api-fluent has a runtime/install-time dependency on grpcio~=1.30 (which is installed during the installation of .whl file). But it has a build-time dependency on grpcio-tools==1.17.0 (which is installed during the compilation of .whl file) through the ansys_tools_protoc_helper package. By adding ansys-api-fluent as a git dependency of pyfluent, ansys-api-fluent is built from its source during the pyfluent installation. So, the pyfluent installation now requires installation of grpcio-tools==1.17.0.

@mkundu1 mkundu1 merged commit ec1624f into main Jun 16, 2022
@mkundu1 mkundu1 deleted the maint/api-install branch June 16, 2022 06:42
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