Experimental - Initial commit for Kerngraph #81
Merged
christopherngutierrez merged 19 commits intomainfrom Jun 11, 2025
Merged
Conversation
…ed. Fixed a bug for generating output when loop_interchange is enabled.
…own before printing detailed messages
faberga
previously approved these changes
Jun 6, 2025
Contributor
faberga
left a comment
There was a problem hiding this comment.
LGTM as an experimental feature that we want in the codebase so we can start discussing optimizations and showing people the potential of the various intermediate manipulations/optimizations that can be done.
Contributor
|
Hi, @christopherngutierrez |
kylanerace
previously approved these changes
Jun 6, 2025
Collaborator
|
Only change I see would be to match the current file hierarchy on main (e.g. kerngen inside of p-isa_tools, same for functional_modeler, etc.). Just needs to be rebased with main |
Contributor
|
Hi @christopherngutierrez and @kylanerace |
Updates the requirements on [pydantic](https://github.com/pydantic/pydantic) to permit the latest version. - [Release notes](https://github.com/pydantic/pydantic/releases) - [Changelog](https://github.com/pydantic/pydantic/blob/v1.10.22/HISTORY.md) - [Commits](pydantic/pydantic@v1.10.13...v1.10.22) --- updated-dependencies: - dependency-name: pydantic dependency-version: 1.10.22 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update Documentation to match the change of repository name. Signed-off-by: Flavio Bergamaschi <flavio@intel.com>
* new repository structure * documentation update --------- Signed-off-by: Flavio Bergamaschi <flavio@intel.com>
* Reference implementation of the HERACLES Assembler tools --------- Signed-off-by: Flavio Bergamaschi <flavio@intel.com> Co-authored-by: Flavio Bergamaschi <faberga+github@users.noreply.github.com> Co-authored-by: Ernesto Zamora Ramos <sidezrw+github@users.noreply.github.com> Co-authored-by: Chris Wilkerson <ChrisWilkerson+github@users.noreply.github.com> Co-authored-by: Jose Rojas Chaves <joserochh+github@users.noreply.github.com> Co-authored-by: Kylan Race <kylanerace+github@users.noreply.github.com> Co-authored-by: Christopher Gutierrez <christopherngutierrez+github@users.noreply.github.com> Co-authored-by: Hamish Hunt <hamishun+github@users.noreply.github.com> Co-authored-by: Jack Crawford <jlhcrawford+github@users.noreply.github.com> Co-authored-by: Minxuan Zhou <minxuanzh+github@users.noreply.github.com> Co-authored-by: Wen Wang <wenwang3122+github@users.noreply.github.com> Co-authored-by: Adish Vartak <avartak+github@users.noreply.github.com> Co-authored-by: Michael Steiner <g2flyer+github@users.noreply.github.com> Co-authored-by: Avinash Alevoor <avinash-alevoor+github@users.noreply.github.com> Co-authored-by: Rashmi Agrawal <agrawalr-dev+github@users.noreply.github.com>
Bumps the pip group with 3 updates in the /assembler_tools/hec-assembler-tools directory: [fonttools](https://github.com/fonttools/fonttools), [pillow](https://github.com/python-pillow/Pillow) and [zipp](https://github.com/jaraco/zipp). Updates `fonttools` from 4.39.0 to 4.43.0 - [Release notes](https://github.com/fonttools/fonttools/releases) - [Changelog](https://github.com/fonttools/fonttools/blob/main/NEWS.rst) - [Commits](fonttools/fonttools@4.39.0...4.43.0) Updates `pillow` from 10.0.1 to 10.3.0 - [Release notes](https://github.com/python-pillow/Pillow/releases) - [Changelog](https://github.com/python-pillow/Pillow/blob/main/CHANGES.rst) - [Commits](python-pillow/Pillow@10.0.1...10.3.0) Updates `zipp` from 3.15.0 to 3.19.1 - [Release notes](https://github.com/jaraco/zipp/releases) - [Changelog](https://github.com/jaraco/zipp/blob/main/NEWS.rst) - [Commits](jaraco/zipp@v3.15.0...v3.19.1) --- updated-dependencies: - dependency-name: fonttools dependency-version: 4.43.0 dependency-type: direct:production dependency-group: pip - dependency-name: pillow dependency-version: 10.3.0 dependency-type: direct:production dependency-group: pip - dependency-name: zipp dependency-version: 3.19.1 dependency-type: direct:production dependency-group: pip ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Update dependabot rules for the weekly runs
…77) Fixing no hbm for new isa_spec
* Enabling configurable mem specifications * Adding documentation
Fixes two configuration files which contain the incorrect path to kerngen: .pre-commit-config.yaml and pytest.ini.
Fixes suggested bank error Co-authored-by: Kylan Race <kylan.e.race@intel.com> Co-authored-by: Flavio Bergamaschi <faberga@googlemail.com>
joserochh
approved these changes
Jun 11, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
Experimental
Kerngraph.pyis a command line tool that takes output fromkerngen.pythroughstdinand transform the Pisa for optimization.This PR is the starting point.
kerngen.pyTypes of changes
What types of changes does your code introduce to the Encrypted Computing SDK project?
Put an
xin the boxes that applyChecklist
Put an
xin the boxes that apply. You can also fill these out after creatingthe PR. If you are unsure about any of them, do not hesitate to ask. We are
here to help! This is simply a reminder of what we are going to look for before
merging your code.
Further comments
As an illustrate example, say that the contents of the file
bgv.add.highcontains:which produces the following output from kerngen:
With kerngraph, we can interchange the loop order to produce the following output:
Note the ordering of the generate pisa commands produced with kerngraph.