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

Updated bracket-bevy to Bevy 0.11.0 #342

Open
wants to merge 49 commits into
base: bevy
Choose a base branch
from

Conversation

IAmSegfault
Copy link

@IAmSegfault IAmSegfault commented Jun 2, 2023

As mentioned in this issue, I updated bracket-bevy to use Bevy 0.10.1.

I Replaced stages with stageless sets. I also Replaced the function body of update_mouse_position to handle Window entities, with a workaround for the panic I mentioned. I also edited the Msaa resource so it reflects that it's an enum. I'm not sure how this branch is being used so if you would prefer the merge go into master as not to deal with merge conflicts let me know.

EDIT:

I added some changes to this pull request so that bracket-bevy now depends on Bevy 0.11.0. Changes were made to use the new scheduling system in 0.11.0. More testing is likely going to be needed however before merging this pull request.

I did notice two different bugs that occur rarely that both seem to happen once per 70-100 application launches. The first one appears to happen no matter what and results in a window with a gray screen and none of the systems appear to run. I'm not exactly sure what causes it because I can only reproduce the issue when running the application from a terminal, running from Visual Studio Code cannot seem to trigger the issue.

The second appears to happen if there are two or more different systems in the application that call the set() method on BracketContext. Changing the schedules of the systems and forcing them to execute in a specific order using .after() on the systems did not seem to resolve the issue. In this case it will occasionally cause this line in Bevy's render function to panic.

AndreasHempel and others added 30 commits February 10, 2022 14:26
This allows using these types in structs that want to derive
Debug themselves
* export colors mods separately to aid precise imports

* use updated bracket-color
thebracket and others added 19 commits October 4, 2022 09:50
…culations, the astar example actually finds a path now (updates logic from amethyst#268)
…hed testing and some more issues), relax dependency versions to make releases easier.
…on Bevy 0.9. There are some issues left to resolve, particularly why is it spawning continuous Resize messages on Windows?
Replaced stages with stageless sets. Replaced the function body of update_mouse_position to handle Window entities.
Updated systems initialized by bterm_builder to use the new simplified system scheduler in Bevy 0.11.0. Bumped up Bevy dependencies in bracket-color and bracket-geometry to Bevy 0.11.0. Fixed bug with inverted y-coordinates for the mouse's world space position in fn update_mouse_position.
@IAmSegfault IAmSegfault changed the title Updated bracket-bevy to Bevy 0.10.1 Updated bracket-bevy to Bevy 0.11.0 Aug 9, 2023
@sparr
Copy link

sparr commented Oct 22, 2023

Is this intended to support GUI pseudo-terminals only? I'm trying to port my existing project using bevy + bracketlib over to using your updated bracket-bevy but disabling default features on bracketlib and enabling curses still leads to bevy creating a GUI terminal

@IAmSegfault
Copy link
Author

Yes, bracket-bevy utilizes Bevy's renderer to provide a gui based pseudo terminal. I'm not sure if the curses backend is currently in a working state.

@sparr
Copy link

sparr commented Dec 30, 2023

Thanks for the update. I'll keep using my own much less capable hacked-together bracket+bevy stack (https://github.com/sparr/ascii-factory/blob/main/src/bevy_bracket.rs) for now. I'd love to contribute here, but my Rust isn't sharp enough yet to dive into a codebase as complex as yours.

@thebracket
Copy link
Collaborator

thebracket commented Dec 30, 2023 via email

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.