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

Neureka #1

Open
wants to merge 271 commits into
base: neureka
Choose a base branch
from
Open

Neureka #1

wants to merge 271 commits into from

Conversation

Scheremo
Copy link

This PR fixes various issues related to:

  1. ONNX spec compliance of DORY_node tensor name casting
    In the current version, tensor names are explicitly cast to an integer type. According to the ONNX spec, tensor names are strings and there is no guarantee for those strings to only contain numerics. This PR fixes issue by enumerating tensor names explicitly along the topologically sorted list representing the graph.

  2. Typing issues, Calling of undefined variables
    In the current version, some functions use bitwise operations intended to work on integers, on data that is not guaranteed to be integer type. This PR fixes this by checking if the data can be cast to an integer w/o loss of precision, and then casting to an integer.
    In the current version, some functions refer to variables that are only conditionally defined, with an implicit None value default. Since this is leads to Exceptions, this PR adds default definitions of such variables.

lukamac and others added 30 commits July 28, 2022 16:13
…EMO (Frontnet, ResidualNet, Dronet, MV1-128) and 4 networks from Quantlab (2 MV2 and 2 MV1, 4 bits and 8 bits). Only MV1-128 output is failing given the wrong export of last fully-connected layer weights
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.

8 participants