-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Open
Labels
error messagesBetter, more actionable error messagesBetter, more actionable error messages
Description
Currently, it seems error hints have to be defined for a concrete exception type. Indeed, this line
Line 318 in 24d2f4a
hinters = get(_hint_handlers, typeof(ex), nothing) |
only gets exception handlers for typeof(ex)
, and not for its supertypes. Would a PR be welcome to change that?
Concrete use case: I have recently added a common error supertype to Enzyme, in order to be able to define hints based on it inside DifferentiationInterface. However, no exception has the type Enzyme.Compiler.EnzymeError
, they have concrete subtypes of that which may change between versions since they are internal details.
Metadata
Metadata
Assignees
Labels
error messagesBetter, more actionable error messagesBetter, more actionable error messages