Skip to content

[Merged by Bors] - Update Query struct docs #5741

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
Closed
wants to merge 26 commits into from
Closed

[Merged by Bors] - Update Query struct docs #5741

wants to merge 26 commits into from

Conversation

Nilirad
Copy link
Contributor

@Nilirad Nilirad commented Aug 19, 2022

Objective

Solution

Nilirad and others added 25 commits August 19, 2022 12:33
@Nilirad
Squash and rebase
1f81535 
Co-authored-by: Alice Cecile <[email protected]>
Co-authored-by: MinerSebas <[email protected]>
@Nilirad
State that query creation has a time-constant cost
a04c319
@Nilirad
Table vs sparse set query iteratino performance
e403f9e
@Nilirad
Reference Query from WorldQuery
b0304e4
@Nilirad
Remove WorldQuery repetition in second paragraph
7be57c9
@Nilirad
Add note about for_each vs iter performance
8195f83
@Nilirad
Remove implementation detail from performance section
c7b2343
@Nilirad
Reword for_each performance
2a37c0e
@Nilirad
Remove for_each performance remarks from method descriptions
91c0b9d
@Nilirad
Add cross-links between iter and for_each methods
8ffe098
@Nilirad @james7132
Apply @james7132 suggestion on for_each performance
9166cd1 
Co-authored-by: James Liu <[email protected]>
@Nilirad
Break doc comments at clause or sentence boundaries
ea313b8
@Nilirad
Split long line
bbf8a1e
@Nilirad
Migrate many_for_each_mut → iter_many_mut
890624c
@Nilirad
Fix iter_many_mut description
68df415
@Nilirad
Update to_readonly description
4aa7e20 
This will reflect the added context in this PR
that is aware of what query items are
and that they can be read-only or not.
@Nilirad
Ref-style link revision
d5372e9 
Moved all links to the bottom of the doc comment.
Links have been alphabetically ordered.

Rebase notes:

- After bevyengine#5205 got merged,
  the “Safety” section of `WorldQuery` has changed.
  I listed the mentioned methods for a better layout.
  I also removed the `WorldQuery::` prefix for readability.
- I don't have enough knowledge of the previous and current state of `WorldQuery` types,
  so the documentation may be outdated somewhere.
  `WorldQuery` needs another review round.
@Nilirad
Move `WorldQuery safety section up
74cf66c
@Nilirad @rparrett
Apply suggestions from code review
01ff486 
Co-authored-by: Rob Parrett <[email protected]>
@Nilirad @BoxyUwU
Apply diff-based suggestions
fa78f7c 
Co-authored-by: Boxy <[email protected]>
@Nilirad
Remove duplicated and outdated “Safety” section
1ad2328
@Nilirad
Add link to State associated type
6a9c560
@Nilirad
Make clippy happy
684dff6
@Nilirad
Checkout other files from main
d18ea9b
@Nilirad
Remove changes to methods
f0874e4
@Nilirad Nilirad mentioned this pull request Aug 19, 2022
Closed
2 tasks
afonsolage
afonsolage approved these changes Aug 19, 2022
View reviewed changes
@Nilirad Nilirad added C-Docs An addition or correction to our documentation A-ECS Entities, components, systems, and events labels Aug 19, 2022
Weibye
Weibye approved these changes Aug 27, 2022
View reviewed changes
Copy link
Contributor

@Weibye Weibye left a comment

Choose a reason for hiding this comment

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

Looks good :)

@Weibye Weibye added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Aug 27, 2022
@Nilirad Nilirad mentioned this pull request Aug 27, 2022
Closed
@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Sep 2, 2022
@Nilirad
Update Query struct docs (#5741)
7511c9b 
# Objective

- Update `Query` docs with better terminology
- add some performance remarks (Fixes #4742)

## Solution

- See #4989. This PR is derived from it. It just includes changes to the `Query` struct docs.
@bors
Copy link
Contributor

bors bot commented Sep 2, 2022

Pull request successfully merged into main.

Build succeeded:

@bors bors bot changed the title Update Query struct docs [Merged by Bors] - Update Query struct docs Sep 2, 2022
@bors bors bot closed this Sep 2, 2022
james7132 pushed a commit to james7132/bevy that referenced this pull request Oct 28, 2022
@Nilirad @james7132
Update Query struct docs (bevyengine#5741)
d12db92 
# Objective

- Update `Query` docs with better terminology
- add some performance remarks (Fixes bevyengine#4742)

## Solution

- See bevyengine#4989. This PR is derived from it. It just includes changes to the `Query` struct docs.
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
@Nilirad @ItsDoot
Update Query struct docs (bevyengine#5741)
d952844 
# Objective

- Update `Query` docs with better terminology
- add some performance remarks (Fixes bevyengine#4742)

## Solution

- See bevyengine#4989. This PR is derived from it. It just includes changes to the `Query` struct docs.
@Nilirad Nilirad deleted the Query-body branch December 5, 2023 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alice-i-cecile alice-i-cecile alice-i-cecile left review comments

@Indeximal Indeximal Indeximal left review comments

@afonsolage afonsolage afonsolage approved these changes

@Weibye Weibye Weibye approved these changes

Assignees
No one assigned
Labels
A-ECS Entities, components, systems, and events C-Docs An addition or correction to our documentation S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Make a note of the algorithmic complexity of each of the query operations
5 participants
@Nilirad @alice-i-cecile @afonsolage @Weibye @Indeximal