Skip to content

fix: update to RustNN with allowed type validation - #31

Open
theHamsta wants to merge 1 commit into
rustnn:mainfrom
theHamsta:update-rustnn
Open

theHamsta wants to merge 1 commit into
rustnn:mainfrom
theHamsta:update-rustnn

Conversation

@theHamsta

@theHamsta theHamsta commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

Update to RustNN with generated type validation. This reveals a few buggy tests.

This PR fixes the following error (error was in test)

context = <builtins.MLContext object at 0x74017dbf2e70>, builder = <builtins.MLGraphBuilder object at 0x74017d6326b0>

    @requires_execution_backend
    def test_logical_not_computation(context, builder):
        """Test element-wise logical NOT"""
        x = builder.input("x", [2, 3], "float32")
>       y = builder.logical_not(x)
            ^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalNot.a allows [Uint8], but uses Float32

tests/test_python_api.py:673: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                24% ██▌

――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_and_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74016ed02510>, builder = <builtins.MLGraphBuilder object at 0x74017d4549d0>

    @requires_execution_backend
    def test_logical_and_computation(context, builder):
        """Test element-wise logical AND"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_and(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalAnd.a allows [Uint8], but uses Float32

tests/test_python_api.py:692: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                25% ██▌

―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_or_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74017d635980>, builder = <builtins.MLGraphBuilder object at 0x74017d454d50>

    @requires_execution_backend
    def test_logical_or_computation(context, builder):
        """Test element-wise logical OR"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_or(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalOr.a allows [Uint8], but uses Float32

tests/test_python_api.py:712: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                25% ██▌

――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_xor_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74017d409200>, builder = <builtins.MLGraphBuilder object at 0x74017d454f10>

    @requires_execution_backend
    def test_logical_xor_computation(context, builder):
        """Test element-wise logical XOR"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_xor(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalXor.a allows [Uint8], but uses Float32

tests/test_python_api.py:732: ValueError

Fixes the following error (error was in test)
```
context = <builtins.MLContext object at 0x74017dbf2e70>, builder = <builtins.MLGraphBuilder object at 0x74017d6326b0>

    @requires_execution_backend
    def test_logical_not_computation(context, builder):
        """Test element-wise logical NOT"""
        x = builder.input("x", [2, 3], "float32")
>       y = builder.logical_not(x)
            ^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalNot.a allows [Uint8], but uses Float32

tests/test_python_api.py:673: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                24% ██▌

――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_and_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74016ed02510>, builder = <builtins.MLGraphBuilder object at 0x74017d4549d0>

    @requires_execution_backend
    def test_logical_and_computation(context, builder):
        """Test element-wise logical AND"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_and(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalAnd.a allows [Uint8], but uses Float32

tests/test_python_api.py:692: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                25% ██▌

―――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_or_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74017d635980>, builder = <builtins.MLGraphBuilder object at 0x74017d454d50>

    @requires_execution_backend
    def test_logical_or_computation(context, builder):
        """Test element-wise logical OR"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_or(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalOr.a allows [Uint8], but uses Float32

tests/test_python_api.py:712: ValueError

 tests/test_python_api.py ⨯                                                                                                                                                                                                                                                                                                                                25% ██▌

――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――― test_logical_xor_computation ――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――――

context = <builtins.MLContext object at 0x74017d409200>, builder = <builtins.MLGraphBuilder object at 0x74017d454f10>

    @requires_execution_backend
    def test_logical_xor_computation(context, builder):
        """Test element-wise logical XOR"""
        a = builder.input("a", [2, 3], "float32")
        b = builder.input("b", [2, 3], "float32")
>       y = builder.logical_xor(a, b)
            ^^^^^^^^^^^^^^^^^^^^^^^^^
E       ValueError: logicalXor.a allows [Uint8], but uses Float32

tests/test_python_api.py:732: ValueError

```
@theHamsta theHamsta changed the title fix: for updated RustNN with allowed type validation fix: update to RustNN with allowed type validation Nov 6, 2026

This branch has not been deployed

No deployments
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 this pull request may close these issues.

1 participant