ansys/pyfluent

some optional dependencies are not real project dependencies

seanpearsonuk opened this issue · 4 comments

Some of these are only used in documentation, not in library code:

openpyxl = { version = ">=3.1.5", optional = true }
plotly = { version = ">=5.22.0", optional = true }
python-pptx = { version = ">=0.6.23", optional = true }
seaborn = { version = ">=0.13.2", optional = true }
tensorflow = { version = ">=2.17.0", optional = true }

See pyproject.toml

Some of these are only used in documentation, not in library code:

openpyxl = { version = ">=3.1.5", optional = true } plotly = { version = ">=5.22.0", optional = true } python-pptx = { version = ">=0.6.23", optional = true } seaborn = { version = ">=0.13.2", optional = true } tensorflow = { version = ">=2.17.0", optional = true }

See pyproject.toml

@seanpearsonuk These dependencies are required for example scripts therefore we have made them optional.

@hpohekar optional mitigates the issue but the dependendnecies are nevertheless in the wrong place. They should be expressed as documentation dependencies only.

@seanpearsonuk We have already added them in [tool.poetry.extras]

@hpohekar optional mitigates the issue but the dependendnecies are nevertheless in the wrong place. They should be expressed as documentation dependencies only.

OK.