-
Notifications
You must be signed in to change notification settings - Fork 57
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
changed clap to arghs in examples #277
changed clap to arghs in examples #277
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please be careful how do you use copilot
@@ -12,34 +12,32 @@ use ort::session::Session; | |||
/// Represents a detected object in an image. | |||
#[derive(Debug)] | |||
pub struct Detection { | |||
/// The class label of the detected object. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't remove this comments
struct Args { | ||
#[arg(short, long)] | ||
#[argh(option, short = 'i', long = "image_path")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#[argh(option, short = 'i', long = "image_path")] | |
#[argh(option, short = 'i')] |
no sense to add this if it's the same as the member itself
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same for the others
@@ -13,6 +13,6 @@ version.workspace = true | |||
publish = false | |||
|
|||
[dependencies] | |||
clap = { version = "4.5.9", features = ["derive"] } | |||
arghs = "0.1.7" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add once in the root workspace
@@ -123,8 +119,6 @@ fn main() -> Result<(), Box<dyn std::error::Error>> { | |||
println!("out_tensor: {:?}", out_tensor.shape); | |||
|
|||
// parse the output tensor | |||
// we expect the output tensor to be a tensor of shape [1, 300, 6] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't remove comments
@@ -7,7 +7,7 @@ edition = "2021" | |||
publish = false | |||
|
|||
[dependencies] | |||
clap = { version = "4.5.4", features = ["derive"] } | |||
args = "1.0.0" # Replace with the appropriate version for args |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's this ?
@@ -13,7 +13,7 @@ version.workspace = true | |||
publish = false | |||
|
|||
[dependencies] | |||
clap = { version = "4.5.9", features = ["derive"] } | |||
args = "1.0" # Replace with the appropriate version for args |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what's this?
Updated the files in the "examples" directory to"argh" library