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

Enable the new error messages for the libmamba solver option #102

Closed
2 tasks done
crogoz opened this issue Dec 12, 2022 · 4 comments · Fixed by #103
Closed
2 tasks done

Enable the new error messages for the libmamba solver option #102

crogoz opened this issue Dec 12, 2022 · 4 comments · Fixed by #103
Assignees
Labels
locked [bot] locked due to inactivity severity::4 low; functionality is inconvenient source::community catch-all for issues filed by community members type::feature request for a new feature or capability

Comments

@crogoz
Copy link

crogoz commented Dec 12, 2022

Checklist

  • I added a descriptive title
  • I searched open requests and couldn't find a duplicate

What is the idea?

Conda allows using the libmamba experimental solver. Mamba recently added new error messages (see https://medium.com/@AntoineProuvost/managing-conflicts-with-mamba-6a5fa10ed6a) that show the full dependency tree and the reason for failures.
The new error messages can be enabled setting the experimental_sat_error_message: true in .condarc.

Why is this needed?

The users will get an easy to understand error message - getting the full dependency tree and the reason for incompatibility.

What should happen?

A user will add experimental_sat_error_message: true in .condarc.
When they get an error message from conda, they will see both the old error message & the new experimental error message.

Additional Context

No response

@jaimergp jaimergp transferred this issue from conda/conda Dec 24, 2022
@jaimergp
Copy link
Contributor

While this is available on micromamba, the Python bindings for libmamba do not yet expose the Context.experimental_sat_error_message attribute, so we can't enable them yet :(

$ conda list libmambapy
libmambapy                1.1.0            py39hf0aba66_3    conda-forge
$ python -c "from libmambapy import Context; print(Context().experimental_sat_error_message)"
AttributeError: 'libmambapy.bindings.Context' object has no attribute 'experimental_sat_error_message'

Work being done on #103 if you want to track progress!

@jaimergp jaimergp linked a pull request Dec 28, 2022 that will close this issue
3 tasks
@crogoz
Copy link
Author

crogoz commented Jan 17, 2023

hey @jaimergp,
Thanks for looking into this! the newest version of libmambapy (1.2.0) should have the bindings for the experimental_sat_error_message attribute.
Also, I think .condarc might need to allow an experimental_sat_error_message attribute to be set.

$  python -c "from libmambapy import Context; print(Context().experimental_sat_error_message)"
False
$ conda list libmambapy
libmambapy                1.2.0           py311hf90d2e5_0    artifactory/external-conda-forge

@jezdez jezdez added type::feature request for a new feature or capability source::community catch-all for issues filed by community members severity::4 low; functionality is inconvenient labels Jan 24, 2023
@jezdez jezdez modified the milestones: TBD, 23.2.0 Jan 30, 2023
@costrouc costrouc self-assigned this Feb 13, 2023
@costrouc
Copy link
Contributor

Thanks for creating this issue @syslaila and letting us know that it is know available. I'm work on getting this added this week.

@jezdez
Copy link
Member

jezdez commented Feb 14, 2023

FWIW libmamba 1.3.1 and libmambapy 1.3.1 were released on defaults.

@jezdez jezdez modified the milestones: 23.2.0, 23.3.0 Feb 17, 2023
@github-actions github-actions bot added the locked [bot] locked due to inactivity label Mar 22, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
locked [bot] locked due to inactivity severity::4 low; functionality is inconvenient source::community catch-all for issues filed by community members type::feature request for a new feature or capability
Projects
Archived in project
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants