-
Notifications
You must be signed in to change notification settings - Fork 16
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] Implement new API PluginManager::nn_preload
#69
Conversation
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR. Overall Summary:
In addition, there are some less critical findings such as updates to crate versions and the Rust SDK version bump, which could require further clarification or justification. Overall, it is recommended to address these potential issues and provide more comprehensive documentation and explanation for the changes before merging the pull request. DetailsCommit aa9b0ec1d06d0d998a620cf27a3094d9be4e734dKey Changes:
Potential Problems:
Commit 126297b24659ce24812569bc7f4413d40fdf90fbKey changes:
Potential problems:
Recommendations:
Commit 54990d67e36e8b17cb68d3b44190063fc8e67ae6Key changes:
Potential problems:
Commit 0b86bc9344d9a22db004c80d709142a431907cfeKey changes:
Potential problems:
Commit 2030bbb5ea8130e02c0da2024c516adc3bb028ffKey changes:
Potential problems:
Commit 83906b12cbb3340c3c828f7da670dcdb8f963c7aKey changes:
Potential problems:
Overall, it seems there may be some missing context and documentation in the pull request that should be addressed before merging. Commit 723ec2e51eed2da0e6d6973c029fc071bcb33f61Summary of key changes:
Potential problems:
|
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
@L-jasmine Could you please help review this PR? Thanks a lot! |
@L-jasmine Thanks for the review! |
In this PR, add new API
nn_repload
forPluginManager
to support new C-APIWasmEdge_PluginInitWASINN
.