-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: merge features back into remote branch #8
base: main
Are you sure you want to change the base?
Conversation
Thank you, @SamTV12345 , for Closing out the previous PR and taking a look at this one. This will take a while to review & test. I'll keep this thread updated. |
Sure :). If you have any questions feel free to ask. There is probably still a lot of improvement there especially on the resolving part. I tried to parallelize as much as possible to reduce latency between the pipes |
Hi @shahen94 did you have time for a review :) ? |
This pr adds the following features:
Sorry for the big pr. This branch also contains fixes for the mentioned compile errors @shahen94 on linux as I had your pipeline deactivated due to the fork.