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.

Sign up for GitHub

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

Jump to bottom

add some axes info in error message on invalid label #875

Closed
gdementen opened this issue Aug 3, 2020 · 0 comments
Closed

add some axes info in error message on invalid label #875

gdementen opened this issue Aug 3, 2020 · 0 comments
Assignees
@gdementen @alixdamman
Labels
difficulty: low enhancement priority: high size: small
Milestone
0.34

Comments

@gdementen
Copy link
Contributor

gdementen commented Aug 3, 2020 •
edited

Having something similar to AxisCollection.info added to the "%r is not a valid label for any axis" message would very helpful. I have actually hacked my version several times to do just that and found it very helpful. I would only avoid displaying the first line with the array shape.
@alixdamman alixdamman self-assigned this Aug 26, 2020
@gdementen gdementen added this to the 0.34 milestone Sep 29, 2022
@gdementen gdementen self-assigned this Sep 29, 2022
gdementen added a commit to gdementen/larray that referenced this issue Feb 14, 2023
@gdementen
FEAT: made Array[] error message more useful (closes larray-project#875)
78fa124 
We now have some axes info in the error messages

also factorized
* parts of _guess_axis and _translate_axis_key_chunk
* _display_name, _axis_summary and _axes_summary methods in AxisCollection
and this fixed two corner cases for ambiguous labels on anonymous axes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gdementen gdementen

@alixdamman alixdamman

Labels
difficulty: low enhancement priority: high size: small
Projects
None yet
Milestone
0.34
Development

No branches or pull requests
2 participants
@gdementen @alixdamman