search

SpoonLabs / npefix

Open-science code of the program repair tool described in "Dynamic Patch Generation for Null Pointer Exceptions Using Metaprogramming" (SANER'17)
https://hal.archives-ouvertes.fr/hal-01419861/document
14 stars 9 forks source link

How to set seed, m2, and server port? #18

Open jose opened 4 years ago

jose commented 4 years ago

Hi,

According to the README.md file

however, these options are no longer available in version 0.8-SNAPSHOT. Is there any way to define these options (using the command line options) so that I don't have to configure a new config.ini every single time I wanna run npefix with a different seed?

-- Best, Jose

tdurieux commented 4 years ago

Those variables are not used in the normal execution mode of NPEFix, you can simply ignore them.

jose commented 4 years ago

Thanks @tdurieux. Any reason to have them available in version 0.8-SNAPSHOT?

tdurieux commented 4 years ago

The default main class was different, we used: 'https://github.com/SpoonLabs/npefix/blob/master/src/evaluation/java/fr/inria/spirals/npefix/main/MainEvaluation.java' because it was the experiment for the lastest NPEFix paper. We changed the main class to be able to run NPEFix as an independent repair tool instead of being "hardcoded" for one of our benchmark. We are now using this main: https://github.com/SpoonLabs/npefix/blob/master/src/main/java/fr/inria/spirals/npefix/main/run/Main.java as the default main in the jar

jose commented 4 years ago

Not being able to define --m2 and --seed on the command line is a down for me. It would be great if you can make those available in the official main. Thanks in advance.

tdurieux commented 4 years ago

The variable m2 is never used in the code, it was only used the execution framework of a different experiment. The official execution mode of NPEFix does not have random. I will add the seed parameter but it will not be useful for the execution mode that you are using.

tdurieux commented 4 years ago

(I am waiting to receive the permission to push in this repo)

jose commented 4 years ago

The variable m2 is never used in the code, it was only used the execution framework of a different experiment. The official execution mode of NPEFix does not have random. I will add the seed parameter but it will not be useful for the execution mode that you are using.

Ok, I was not aware of that. In that case I guess we can close this issue.