hellas-ai / hellas-ai/catgrad

Interpreter `Backend` trait should have an Error type, and every op should yield a Result type

Open
#189 0 comments 0 reactions 1 assignee Claimed by @statusfailed View on GitHub
enhancement
Dominant language
Rust
Stars
34
Forks
7
PR merge metrics
No merged PRs in 30d

Description

An example of this happening: the `add` kernel receives a (1,2) tensor and a (2,2) tensor. These are broadcastable, but catgrad convention is that broadcasting must be explicit.

Proposed fix:

1. Add an associated Error type to `Backend`
2. Return `Result` from each backend kernel
3. Modify interpreter to handle kernel error cases (propagate up to interpreter error type)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.