-
Notifications
You must be signed in to change notification settings - Fork 189
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
Add SendDataToElement action for nodegroup elements #6041
Add SendDataToElement action for nodegroup elements #6041
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can squash any changes
// and the nodegroup by using `db::get_mutable_reference`. If/when we | ||
// start dynamically inserting and removing elements, we'll need to update | ||
// how we handle this. For example, we might need the containers to have | ||
// strongly stability guarantees. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
strong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep! Done!
// exact minimum number of sends we need to do, but gets us close in most | ||
// cases. If we really wanted to we could also add the number of | ||
// directions that don't have external boundaries in our neighbors block. | ||
// if (count >= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
did you mean to leave this if commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, because this is the bookkeeping code I talked with Will about to minimize communication. I want to get the rest in before I get the counting sorted out completely :)
722524c
to
67e2b1a
Compare
Rebased and squashed. Thanks for the review! |
Proposed changes
Upgrade instructions
Code review checklist
make doc
to generate the documentation locally intoBUILD_DIR/docs/html
.Then open
index.html
.code review guide.
bugfix
ornew feature
if appropriate.Further comments