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

The #[derive(ProcBlock)] macro doesn't accept multiple inputs and outputs #311

Closed
Michael-F-Bryan opened this issue Sep 19, 2021 · 0 comments · Fixed by #321
Closed

The #[derive(ProcBlock)] macro doesn't accept multiple inputs and outputs #311

Michael-F-Bryan opened this issue Sep 19, 2021 · 0 comments · Fixed by #321

Comments

@Michael-F-Bryan
Copy link
Contributor

Someone was wanting to define a tokenizer proc block on Discord but didn't know how to tell the system they have multiple inputs and outputs. It turns out we've defined the proc-macro to only accept syntax for single inputs and outputs.

This is the error someone got when they tried what feels intuitive for the #[transform] attribute:

error: Unknown tensor element type
   --> /root/projects/hack/htn/bert/byte_input_conversion/src/lib.rs:205:39
    |
205 | #[transform(input = [u8; _], output = ([i32; 384], [i32; 384], [i32; 384]))]
    |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant