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

[Upgrade] 'dgstrf info' in stdout, update scipy #2442

Open
6 tasks done
DEUCE1957 opened this issue Nov 7, 2024 · 1 comment
Open
6 tasks done

[Upgrade] 'dgstrf info' in stdout, update scipy #2442

DEUCE1957 opened this issue Nov 7, 2024 · 1 comment
Labels
help wanted! We are looking for contributions from the community to help with this issue maintenance miscellaneous tasks for housekeeping and maintenance

Comments

@DEUCE1957
Copy link

DEUCE1957 commented Nov 7, 2024

Bug report checklis

  • Searched the issues page for similar reports

  • Read the relevant sections of the documentation

  • Browse the tutorials and tests for usefull code snippets and examples of use

  • Reproduced the issue after updating with pip install --upgrade pandapower (or git pull)

  • Tried basic troubleshooting (if a bug/error) like restarting the interpreter and checking the pythonpath

Reproducible Example

In Scipy <1.14.1 (Pandapower currently uses 1.13.1) whenever we solve a Sparse matrix that is singular the following text is sent to stdout:

dgstrf info 1
dgstrf info 1
dgstrf info 1
dgstrf info 1
zgstrf info 2
dgstrf info 1
dgstrf info 1
dgstrf info 1
dgstrf info 1
dgstrf info 1

Since this comes from Scipy internally (compiled in C) there is no way to suppress it.

Solution

Thankfully this has been fixed in Scipy 1.14.1, see Issue 20933
If possible, can PandaPower upgrade to using Scipy 1.14.1?

Expected Behavior

Expect no messages to be written directly to stdout when encountering a singular sparse matrix.

Installed Versions

  • Python: 3.12.7
  • PandaPower: 2.14.11

Label

  • Relevant labels are selected
@DEUCE1957 DEUCE1957 added the bug label Nov 7, 2024
@vogt31337
Copy link
Contributor

Good point, I'll add it to the list, but I don't consider this a bug, merely a nuissance.

@vogt31337 vogt31337 added help wanted! We are looking for contributions from the community to help with this issue maintenance miscellaneous tasks for housekeeping and maintenance and removed bug labels Nov 21, 2024
@vogt31337 vogt31337 changed the title [bug] 'dgstrf info' in stdout [Upgrade] 'dgstrf info' in stdout Nov 21, 2024
@vogt31337 vogt31337 changed the title [Upgrade] 'dgstrf info' in stdout [Upgrade] 'dgstrf info' in stdout, update scipy Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted! We are looking for contributions from the community to help with this issue maintenance miscellaneous tasks for housekeeping and maintenance
Projects
None yet
Development

No branches or pull requests

2 participants