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

"no collapse operator, using sesolve" message #1047

Closed
goerz opened this issue Jul 16, 2019 · 1 comment
Closed

"no collapse operator, using sesolve" message #1047

goerz opened this issue Jul 16, 2019 · 1 comment

Comments

@goerz
Copy link
Member

goerz commented Jul 16, 2019

Since the update to version 4.4, calling mesolve without collapse operators prints a message "no collapse operator, using sesolve" to stdout. Is printing this message really necessary? It breaks our examples for the krotov package, see e.g. the notebook https://krotov.readthedocs.io/en/latest/notebooks/01_example_simple_state_to_state.html, where these messages show up in the notebook when it runs with qutip 4.4. Of course I understand that the message is technically correct, and we could work around the problem in krotov, but it was nice to be able to use mesolve as a catch-all for either a Schrödinger equation or a master equation. We'd have to start routing different cases to mesolve and sesolve manually, otherwise. Personally, I would argue that this kind of message should be emitted as logger.debug instead of print, if at all.

@goerz
Copy link
Member Author

goerz commented Jul 23, 2019

As #1049 has been merged, I’m going to assume this will be fixed in the next release. Closing.

@goerz goerz closed this as completed Jul 23, 2019
goerz added a commit to qucontrol/krotov that referenced this issue Aug 3, 2019
All problems introduced in QuTiP 4.4 were worked around, except for
qutip/qutip#1047

A fix to qutip/qutip#1047 was merged into
QuTiP's master, so presumably the problem will disappear with the next
release of QuTiP.

This commit specifically excludes QuTiP 4.4.0 from being installed on
Travis and AppVeyor. We do not exclude that version in the list of
dependencies in setupy.py, because the issue is mostly cosmetic
(although it will fail tests). The average user should be OK using
krotov with QuTiP 4.4.0

Closes #49
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

No branches or pull requests

1 participant