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

New release request: Flexibility on qutip + scipy extensions (Forschungszentrum Jülich/JSC) #775

Closed
AlvaroHuanay opened this issue Dec 5, 2024 · 6 comments
Milestone

Comments

@AlvaroHuanay
Copy link

Dear Pulser team,

We are contacting you from Forschungszentrum Jülich. Developing Pulser 1.1.1 for our Stage 2025, we came accross with an incompatibility issue. I would like to ask if it would be possible to make a new Pulser release compatible with the following libraries:

qutip==5.0.4
scipy==1.13.1

The other dependencies you have for Pulser fit perfectly for us. Those are the only extensions we have issues with. The requirements you have in your last release for these extensions are:

qutip~=4.7.5
scipy<1.13

Could you release a new version with these compatibilities?

Any additional information you may need I will be glad to provide you so.

Kind regards.

@HGSilveri
Copy link
Collaborator

Hello @AlvaroHuanay , thanks for reaching out!

I'm afraid this will not be straigthforward because we encountered a severe performance degradation when we first tried to upgrade to Qutip 5 (see #730 for details). There has been no Qutip release since we last tried so I don't expect things to have changed on that front.

If I may ask, do you happen to know if you actually need to run emulations on this environment with restricted dependencies? I ask because these conflicts come from the pulser-simulation package, which is only needed for emulations. All sequence creation functionality is in pulser-core, so you could eventually just install pulser-core instead of pulser - which includes both packages.

@AlvaroHuanay
Copy link
Author

Hi @HGSilveri

Thank you very much for your quick update. We are trying to get Qutip 4.7.5 on our systems. Nonetheless, we would really appreciate if the extension "scipy==1.13.1" could be added at least to be able to integrate it.

Our intention is to have the complete pulser module as we will use both tools. Do you have in mind upgrading to Qutip 5.X for later releases? I ask this based on the coments on #730, #757.

Kind regards.

@awennersteen
Copy link
Member

Hi @AlvaroHuanay,

Unfortunately that wouldn't help you much as QuTiP is also incompatible with Scipy>=1.13.
Hence that would involve patching QuTiP as well. I know and understand the JSCs desire to avoid another older SciPy install, but in this case in order to ensure the stability and quality of the software I would ask that you consider it.

QuTiP PR for the restriction on their side:
qutip/qutip#2383

As mentioned in the issues you linked we will have to resolve this, but I don't think we can commit to a timeline immediately. Henrique will know better.

Best,
Aleksander

@HGSilveri
Copy link
Collaborator

HGSilveri commented Dec 5, 2024

Hi again @AlvaroHuanay

Thank you very much for your quick update. We are trying to get Qutip 4.7.5 on our systems. Nonetheless, we would really appreciate if the extension "scipy==1.13.1" could be added at least to be able to integrate it.

I'm afraid this won't be possible because Qutip 4.7.6 (the latest and probably last release of Qutip 4) enforces scipy<1.13; there is some issue between Qutip 4 and scipy>=1.13, so they enforced this restriction. It is the only reason we have it as well, actually (as @awennersteen just pointed out 😅 ).

Our intention is to have the complete pulser module as we will use both tools. Do you have in mind upgrading to Qutip 5.X for later releases? I ask this based on the coments on #730, #757.

We will take another crack at this very soon. We can keep you up to date on how it goes but it is indeed not possible to commit to a timeline at the moment.

@AlvaroHuanay
Copy link
Author

Greetings,

Thank you both (@awennersteen and @HGSilveri) for the explanation. I contacted our team and we are going to use pulser-core in the meantime. Regarding @HGSilveri last comment:

We will take another crack at this very soon. We can keep you up to date on how it goes but it is indeed not possible to commit to a timeline at the moment.

could you please reach to us when you are able to solve this issue with Qutip 5.X and Scipy 1.13.1? I will stay tunned on your repository as well as Qutip's repository to be aware of it just in case because our intention is being able to use the complete pulser module in our systems for the future.

Thank you very much, your comments were very helpful.

Kind Regards.

@HGSilveri HGSilveri added this to the v1.2 milestone Dec 17, 2024
@HGSilveri
Copy link
Collaborator

Hi @AlvaroHuanay !
We just released Pulser v1.2.0, which upgrades pulser-simulation to Qutip 5 and drops the Scipy restriction.
In principle it should solve your issue but feel free to reopen this if that's not the case.
Happy holidays!

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

3 participants