coming icon indicating copy to clipboard operation
coming copied to clipboard

New input

Open martinezmatias opened this issue 6 years ago • 6 comments

New kind of input: to have two arguments corresponding to two files. Then, coming executes the analyzers over the pair.

martinezmatias avatar May 08 '19 15:05 martinezmatias

Doesn't -input files do kind of the same thing? Though, it needs to be encapsulated within three folders with a particular folder hierarchy.

What args are -input and -location supposed to have in the given case?

sedflix avatar May 29 '19 11:05 sedflix

Hi @sedflix

Doesn't -input files do kind of the same thing?

This new input will target to have two arguments e.g., - file_left and -file_right. The goal will be to analyze only one pair of files, given by the previous two arguments.

What args are -input and -location supposed to have in the given case?

In fact, maybe we should rename 'input' by 'inputtype' (I will do it after the conference). So I imagine that for this new input location will not be required. WDYT?

martinezmatias avatar May 29 '19 13:05 martinezmatias

In fact, maybe we should rename 'input' by 'inputtype' (I will do it after the conference).

Yeah. I think that same I made an issue for the same #64 earlier.

So I imagine that for this new input location will not be required. WDYT?

One of the choices could be to use -location in the form of <file_left>:<file_right> instead of introducing a new option. Adding two new arguments is more expressive but it also increases the number of options.

sedflix avatar May 29 '19 13:05 sedflix

the choices could be to use -location in the form of <file_left>:<file_right>

It's a good idea. Let's go for that option. (As separator we use File.pathSeparator)

martinezmatias avatar May 29 '19 14:05 martinezmatias

Cool! What would be the name of this input?

sedflix avatar May 29 '19 14:05 sedflix

What would be the name of this input?

Let's say "filepair", something like that

martinezmatias avatar May 29 '19 14:05 martinezmatias

Done in "filespair" format

monperrus avatar Mar 25 '24 21:03 monperrus