Skip to content

Network specification and start hops modeling#31

Merged
gilbertmike merged 31 commits into
mainfrom
hops
Feb 23, 2026
Merged

Network specification and start hops modeling#31
gilbertmike merged 31 commits into
mainfrom
hops

Conversation

@gilbertmike

Copy link
Copy Markdown
Contributor

Semantics for network spec:

  • Within a hierarchy, the fanout for a component should is still defined in the spatial of the component (or container). The network above the component is understood to be the one delivering to the array.
  • When the network nodes attribute is used, then the nodes are understood to be "flat".

Note that the network has an attribute shape for defining the shape of the mesh, which is useful in flat-mode. We cannot use spatial since that means an array of current components, which corresponds to instantiating separate networks instead.

@gilbertmike gilbertmike marked this pull request as ready for review February 23, 2026 16:43
@gilbertmike gilbertmike changed the title Network specification and WIP network hops modeling Network specification and start hops modeling Feb 23, 2026
@gilbertmike

Copy link
Copy Markdown
Contributor Author

Hops modeling isn't done yet. All diffs in model code is just preparing for the actual modeling code and cleaning up some modeling code.

The spec is done, however, and so I would like to pull into main at this state. There is one change in the Network spec since @tanner-andrulis last saw it:

  • The Network node itself is no longer a Flat node. Instead, a Flat node should be created under the Network node if flatness is desired. This is cleaner because a component is still always a Leaf, and "structural" information is always in an explicit Hierarchical, Flat, or Fork.

@gilbertmike gilbertmike merged commit 1f0d91a into main Feb 23, 2026
2 checks passed
@gilbertmike gilbertmike deleted the hops branch February 26, 2026 20:52
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.

2 participants