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

[OTHER] Improve Error Message Clarity #202

Open
aturker-synnada opened this issue Feb 18, 2025 · 0 comments
Open

[OTHER] Improve Error Message Clarity #202

aturker-synnada opened this issue Feb 18, 2025 · 0 comments
Labels
good first issue Good for newcomers

Comments

@aturker-synnada
Copy link
Collaborator

Summary

Some error messages in the library are insufficient and do not provide enough context for users to diagnose issues effectively. Enhancing these messages will improve debuggability and usability.

Context

Currently, certain error messages lack key information:

  • set_shapes Function: When a non-Tensor error occurs, the message should include the name of the corresponding key. Additionally, if multiple errors exist, they should be aggregated and displayed together rather than stopping at the first encountered issue.

  • unnamed_data_keys Error: The current error message prints connection dollar names ($ prefixed identifiers), which can be unclear. Instead, the full connection name should be displayed for better readability.

Additional Details

N/a

@aturker-synnada aturker-synnada added the good first issue Good for newcomers label Feb 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant