-
Notifications
You must be signed in to change notification settings - Fork 39
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
[Example] Update pytorch example #161
base: master
Are you sure you want to change the base?
Conversation
Hello, I am a code review agent on flows.network. Here are my reviews of changed source code files in this PR. .github/workflows/pytorch.ymlPotential issuesN/A Summary of changes
pytorch-mobilenet-image/gen_mobilenet_model.pyPotential issues
Summary of changes:
The code aims to convert the loaded PyTorch model into a TorchScript (a format used by Pytorch), which allows for saving and reusing models without needing PyTorch installed. However, the scripted model isn't actually being used in this sample - just saved if it doesn’t exist yet. This code could be improved by using or removing the sm variable altogether to ensure that all parts of a model are utilized during its lifecycle. pytorch-mobilenet-image/mobilenet.ptPotential issuesN/A Summary of changes
Note that this task requires a specific file to provide accurate details about the changes made in it. If you give me the patch file I will be able to parse through and provide more detailed feedback. pytorch-resnet18-image/gen_resnet18_aoti.pyPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/gen_resnet18_model.pyPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/gen_tensor.pyPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/image-1x3x224x224.rgbPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/input.jpgPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/resnet18.ptPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/resnet18_pt2.soPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/run.pyPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/rust/Cargo.tomlPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/rust/src/imagenet_classes.rsPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/rust/src/main.rsPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/rust/src/named_model.rsPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/wasmedge-wasinn-example-mobilenet-image.wasmPotential issuesN/A Summary of changesN/A pytorch-resnet18-image/wasmedge-wasinn-example-resnet18-image-named-model.wasmPotential issuesN/A Summary of changesN/A |
e2bc625
to
caaac54
Compare
Signed-off-by: Sylveon <[email protected]>
example for WasmEdge/WasmEdge#3724