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

Fix build #38

Merged

KOVALW merged 1 commit into wtk_facemask from ekr_wtk_facemask

Jan 8, 2025

Collaborator

ekr-cfa commented Jan 8, 2025

@KOVALW I fixed this build for you. There were several distinct problems:

Rust was having trouble doing the type inference on instance_map, so I told it specifically what the type was.
You need to insert a boxed closure because that's what's stored in instance_map
InterventionFn takes &Context, not &mut Context.
I iterated over &instance_dict rather than instance_dict to avoid a move on iter.

I haven't reviewed the code or even run it, but this should fix the build.


          Fix build

a3cd56a

KOVALW merged commit af5569c into wtk_facemask

2 of 3 checks passed

KOVALW deleted the ekr_wtk_facemask branch

January 8, 2025 14:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet