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

Closes #17: Implementation of operations to support relational networks #23

Merged
merged 10 commits into from
Jan 31, 2022

Conversation

mathisrichter
Copy link
Contributor

This PR adds support to build relational networks:

  • new operations "ReduceAlongDiagonal", "ExpandAlongDiagonal", and "Flip"
  • Jupyter tutorial on relational networks

All unit tests should pass but the new Jupyter tutorial does not run through because of a bug in Lava 0.2.0 that has since been fixed.
@mgkwill Should I include the changes to make lava-dnf dependent on the Lava tip in this PR or should I do this in a separate PR?

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Also refactored a function in shape_handlers.py to reduce code duplication.

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Signed-off-by: Mathis Richter <mathis.richter@intel.com>
…ler.

Also renamed 'ReduceDiagonal' to 'ReduceAlongDiagonal' as well as renaming its shape handler.

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
…pe handler.

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
…lip', 'ReduceAlongDiagonal', and 'ExpandAlongDiagonal'.

Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Signed-off-by: Mathis Richter <mathis.richter@intel.com>
Copy link

@bamsumit bamsumit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't go through the notebooks in detail. Looks really good.

@joyeshmishra joyeshmishra self-requested a review January 31, 2022 03:43
@srrisbud srrisbud merged commit 126a435 into lava-nc:main Jan 31, 2022
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.

Relational networks require an operation that sums across a diagonal of a DNF
4 participants