Skip to content

⬆️ Resolve networkx python package 3.6 version issue.#312

Open
BBBela wants to merge 1 commit intofacebookresearch:mainfrom
intel-staging:networkx-bugfix
Open

⬆️ Resolve networkx python package 3.6 version issue.#312
BBBela wants to merge 1 commit intofacebookresearch:mainfrom
intel-staging:networkx-bugfix

Conversation

@BBBela
Copy link

@BBBela BBBela commented Dec 17, 2025

What does this PR do?

HTA has the requirement in requirements.txt:

networkx>=3.1

The 3.6 version of networkx package has a bug described in this issue: networkx ISSUE

That causes the Critical Path Analysis to fail when networkx package version is 3.6.
The next version fixed that bug, so the networkx==3.6.1 works fine.

This PR adds excluding networkx version 3.6 from requirements.

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
    • [] N/A
  • Did you write any new necessary tests?
    • [x ] N/A
  • Did you make sure to update the docs?
    • [ x] N/A
  • Did you update the changelog?
    • [x ] N/A

…ch#16)

* Resolve networkx python package 3.6 version issue.

HTA has the requirement that networkx package should be greater than or
equal to 3.1 version. For some reason, the version 3.6 of networkx
package causes HTA to fails during critical path analysis.
What is interesting, the version 3.6.1 of networkx package works fine
with HTA. Therefore, to avoid the installation of the problematic
version 3.6, we exclude it in the requirements.txt file.
Issue in networkx repository: networkx/networkx#8367

Signed-off-by: Benedykt Bela <[email protected]>

* Change comment.

Signed-off-by: Benedykt Bela <[email protected]>

---------

Signed-off-by: Benedykt Bela <[email protected]>
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 17, 2025
@ZhaoqiongZ ZhaoqiongZ moved this to Aged Pending Review in PyTorch Intel Jan 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants