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

docs: add entry to FAQ about broken package issue #531

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

tnoczyns-volue
Copy link
Collaborator

Description of ways to address #526 issue.

@tnoczyns-volue tnoczyns-volue added this to the Version 1.11 milestone Jan 8, 2025
@tnoczyns-volue tnoczyns-volue added the Kind: Documentation Improvements or additions to documentation label Jan 8, 2025
.. code-block:: bash

# Generally we recommend to run this in virtual environment
pip install poetry-core==1.9.1
Copy link
Collaborator

Choose a reason for hiding this comment

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

I believe the "recommended" way to run pip is by doing python -m pip instead of invoking it directly.

Comment on lines 42 to 45
This issue is fixed in Mesh Python SDK 1.11.0. All users with Mesh server
version starting from 2.15.0 should upgrade Mesh Python SDK. Users with Mesh
server version 2.14 and below need to reinstall Mesh Python SDK the following
way:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
This issue is fixed in Mesh Python SDK 1.11.0. All users with Mesh server
version starting from 2.15.0 should upgrade Mesh Python SDK. Users with Mesh
server version 2.14 and below need to reinstall Mesh Python SDK the following
way:
This issue is fixed in Mesh Python SDK 1.11.0. All users with a Mesh server
version starting from 2.15.0 should upgrade the Mesh Python SDK. Users with Mesh
server version 2.14 and below need to reinstall the Mesh Python SDK by running
the following:

pip install poetry-core==1.9.1
pip install grpcio-tools==1.66.1
# Set your Mesh Python SDK version in the next line
pip install --no-build-isolation git+https://github.com/Volue-Public/energy-mesh-python@v1.8.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't this be 1.11.0 instead of 1.8.0?

Comment on lines 56 to 59
The reason is that we need poetry-core version below 2.0.0. To do this we need
to install Mesh Python SDK with `--no-build-isolation` flag. This flag requires
all the build dependencies to be already installed on the system. That is why
we install poetry-core and grpcio-tools before installing Mesh Python SDK.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
The reason is that we need poetry-core version below 2.0.0. To do this we need
to install Mesh Python SDK with `--no-build-isolation` flag. This flag requires
all the build dependencies to be already installed on the system. That is why
we install poetry-core and grpcio-tools before installing Mesh Python SDK.
The reason for this is that we need poetry-core to be older than version 2.0.0. Here we use the `--no-build-isolation` flag to prevent pip from automatically installing any dependencies, and we install them manually instead.

@tnoczyns-volue
Copy link
Collaborator Author

Review suggestions applied in 2373e66. @martingalvan-volue could you re-review?

@tnoczyns-volue tnoczyns-volue merged commit 2b74ad1 into master Jan 8, 2025
8 of 9 checks passed
@tnoczyns-volue tnoczyns-volue deleted the tnoczyns-2025-01-08-faq-broken-package branch January 8, 2025 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Kind: Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants