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.

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

Animation graph example: simplify dragging with RelativeCursorPosition #2

Conversation

rparrett
Copy link

@rparrett rparrett commented Mar 2, 2024

Just sharing a few ideas for the animation graph example:

  • Using bevy_ui's built-in RelativeCursorPosition and Interaction components to simplify the drag functionality.
  • ClipNodeText was being added to a non-text ui node which was a bit confusing.
    • Renamed it to ClipNode and added it only to the container.
    • Added the "node background" to the hierarchy
  • Update UI only when weights change
  • Show weights with two decimal places

Feel free to take some/all/none of this.

Copy link

github-actions bot commented Mar 2, 2024

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@pcwalton
Copy link
Owner

pcwalton commented Mar 3, 2024

Merged, thanks!

@pcwalton pcwalton closed this Mar 3, 2024
pcwalton pushed a commit that referenced this pull request Jun 26, 2024
…3906)

# Objective

- Second part of bevyengine#13900 
- based on bevyengine#13905 

## Solution

- check_dir_light_mesh_visibility defers setting the entity's
`ViewVisibility `so that Bevy can schedule it to run in parallel with
`check_point_light_mesh_visibility`.

- Reduce HashMap lookups for directional light checking as much as
possible

- Use `par_iter `to parallelize the checking process within each system.

---------

Co-authored-by: Kristoffer Søholm <k.soeholm@gmail.com>
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