unable to annotate tuple return type of extern function #81

New Issue

sb10q · 2021-11-10T13:47:19+08:00

sb10q commented

2021-11-10 13:47:19 +08:00

@extern
def rtio_input_timestamped_data(timeout_mu: int64,
                                channel: int32) -> Tuple[int64, int32]:
    """Wait for an input event up to timeout_mu on the given channel, and
    return a tuple of timestamp and attached data, or (-1, 0) if the timeout is
    reached."""
    raise NotImplementedError("syscall not simulated")

thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: "unknown class name"', nac3artiq/src/lib.rs:378:44

Also we should have a better error message for this.

``` @extern def rtio_input_timestamped_data(timeout_mu: int64, channel: int32) -> Tuple[int64, int32]: """Wait for an input event up to timeout_mu on the given channel, and return a tuple of timestamp and attached data, or (-1, 0) if the timeout is reached.""" raise NotImplementedError("syscall not simulated") ``` ``` thread '<unnamed>' panicked at 'called `Result::unwrap()` on an `Err` value: "unknown class name"', nac3artiq/src/lib.rs:378:44 ``` Also we should have a better error message for this.

pca006132 commented

2021-11-10 15:28:14 +08:00

No, the problem is that we do not support Tuple, use tuple instead. Error message is indeed a problem.

No, the problem is that we do not support `Tuple`, use `tuple` instead. Error message is indeed a problem.

pca006132 closed this issue

2021-11-10 15:28:14 +08:00

sb10q commented

2021-11-10 15:30:41 +08:00

Poster

OK. I see typing.Tuple is also deprecated.

OK. I see ``typing.Tuple`` is also deprecated.

sb10q referenced this issue