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.

Sign up for GitHub

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

Jump to bottom

Mapping to cgra update reduce #101

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Mapping to cgra update reduce #101

merged 3 commits into from
Nov 15, 2023

Conversation

bobcheng15
Copy link
Collaborator

@bobcheng15 bobcheng15 commented Nov 15, 2023 •
edited
Loading

  1. Update compute_node.py to add the required new configurations to configure the PE sitting inside the reduce_pe_cluster
  2. Update reduce_node.py to add the required new configurations to configure the Reduce sitting inside the reduce_pe_cluster
  3. Update all nodes that connects to/from reduce node to update the connection port name required by the change of port name in the updated reduce_pe_cluster

@bobcheng15
update configuration code for pe and reduce to account for the new co…
ef31fa3 
…nfig register heirarchy in reduce_pe_cluster. also modified the signal name for connection from crddrop to reduce and reduce to arrayval to match the new signal name of reduce in cluster, matmul_ijk_crddrop passed
@bobcheng15
update port name for connection from reduce to reduce, also updated t…
d0793dd 
…he reduce node configuration to use dense mode
@bobcheng15
fix code style
b39b2b4
@bobcheng15 bobcheng15 self-assigned this Nov 15, 2023
kalhankoul96
kalhankoul96 approved these changes Nov 15, 2023
View reviewed changes
Copy link
Collaborator

@kalhankoul96 kalhankoul96 left a comment

Choose a reason for hiding this comment

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

LGTM

@kalhankoul96 kalhankoul96 marked this pull request as ready for review November 15, 2023 23:19
@bobcheng15 bobcheng15 merged commit cade108 into mapping_to_cgra Nov 15, 2023
1 check passed
@bobcheng15 bobcheng15 deleted the mapping_to_cgra_update_reduce branch March 12, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kalhankoul96 kalhankoul96 kalhankoul96 approved these changes

Assignees

@bobcheng15 bobcheng15

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bobcheng15 @kalhankoul96