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

Update to Bevy 0.11 #4

Closed
wants to merge 8 commits into from
Closed

Update to Bevy 0.11 #4

wants to merge 8 commits into from

Conversation

ecrax
Copy link

@ecrax ecrax commented Nov 4, 2023

Hey there,
since I've wanted to use this plugin in one of my projects (which uses Bevy 0.11), I decided to give it a try and upgrade it. The only major changes I had to make were to the PipelineCache, which was upgraded in Bevy 0.11 (here).

Currently it does compile, but the examples do not work, because (for some reason) we cannot access the RenderDevice resource during plugin build (this line). I can't manage to fix this, maybe you have an idea why this is or how to fix it?

Thanks for your work on this! :)

@druskus20
Copy link

druskus20 commented Nov 12, 2023

I fixed by replacing Plugin::build with Plugin::finish, both for AppComputeWorkerPlugin and AppComputePlugin. Seems to work at first glance

@EngoDev EngoDev mentioned this pull request Nov 12, 2023
@ecrax
Copy link
Author

ecrax commented Nov 12, 2023

I fixed by replacing Plugin::build with Plugin::finish, both for AppComputeWorkerPlugin and AppComputePlugin. Seems to work at first glance

This worked wonderfully, I don't know how I could not have seen this :)

@Kjolnyr
Copy link
Owner

Kjolnyr commented Nov 23, 2023

Closing as you are in 0.12 now

@Kjolnyr Kjolnyr closed this Nov 23, 2023
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.

3 participants