Skip to content
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

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

Open
jose opened this issue Jan 25, 2020 · 7 comments
Open

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

jose opened this issue Jan 25, 2020 · 7 comments

Comments

@jose
Copy link

jose commented Jan 25, 2020

Hi,

According to the README.md file

  • (-k|--m2) <~/.m2> allows one to defined the path to the m2 directory
  • (-s|--seed) <randomSeed> allows one to define the seed of the random generator

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
Copy link
Contributor

tdurieux commented Apr 8, 2020

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

@jose
Copy link
Author

jose commented Apr 8, 2020

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

@tdurieux
Copy link
Contributor

tdurieux commented Apr 8, 2020

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
Copy link
Author

jose commented Apr 8, 2020

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
Copy link
Contributor

tdurieux commented Apr 8, 2020

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
Copy link
Contributor

tdurieux commented Apr 8, 2020

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

@jose
Copy link
Author

jose commented Apr 8, 2020

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants