1
21
Fork
You've already forked big-brain
5

WIP: more detailed docs/guides #58

Draft
zkat wants to merge 1 commit from zkat/docs into main
pull from: zkat/docs
merge into: zkat:main
zkat:main
zkat:zkat/tutorial
zkat:zkat/dorfs-example
Owner
Copy link
No description provided.
cBournhonesque (Migrated from github.com) reviewed 2023年01月05日 08:49:06 +01:00
@ -31,2 +34,4 @@
*Newtypefor[`bevy::prelude::Entity`]sthatareActions.
*/
#[derive(Debug, Clone, Copy)]
pubstruct Action(pubEntity);
cBournhonesque (Migrated from github.com) commented 2023年01月05日 08:49:06 +01:00
Copy link

This is one of the more confusing lines of the docs, I think.

In my case, the entity performing the action IS the entity with a ThinkerBuilder component.

A new user doesn't understand the difference between Action and ActionBuilder, (same for scorer/thinker), so this line just confuses them more, I think

This is one of the more confusing lines of the docs, I think. In my case, the entity performing the action IS the entity with a ThinkerBuilder component. A new user doesn't understand the difference between Action and ActionBuilder, (same for scorer/thinker), so this line just confuses them more, I think
@ -31,2 +34,4 @@
*Newtypefor[`bevy::prelude::Entity`]sthatareActions.
*/
#[derive(Debug, Clone, Copy)]
pubstruct Action(pubEntity);
Author
Owner
Copy link

yes, I think your third paragraph is the issue here, really.

One of the things I've struggled with most w/ Big Brain ergonomics is how to have the Action/ActionBuilder (etc) distinction without overwhelming new users, but this might be because I've tried to hard to sweep it under the rug in "basic" cases...

yes, I think your third paragraph is the issue here, really. One of the things I've struggled with most w/ Big Brain ergonomics is how to have the Action/ActionBuilder (etc) distinction without overwhelming new users, but this might be because I've tried to hard to sweep it under the rug in "basic" cases...
This pull request has changes conflicting with the target branch.
  • src/measures.rs
  • src/thinker.rs
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin zkat/docs:zkat/docs
git switch zkat/docs

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff zkat/docs
git switch zkat/docs
git rebase main
git switch main
git merge --ff-only zkat/docs
git switch zkat/docs
git rebase main
git switch main
git merge --no-ff zkat/docs
git switch main
git merge --squash zkat/docs
git switch main
git merge --ff-only zkat/docs
git switch main
git merge zkat/docs
git push origin main
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
zkat/big-brain!58
Reference in a new issue
zkat/big-brain
No description provided.
Delete branch "zkat/docs"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?