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

SBFL_improvemment #76

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

mguerrinha
Copy link

@mguerrinha mguerrinha commented Aug 8, 2024

Context

Implementation of a formula that improves the final ranking

Check lists

  • Unit tests
  • Test pass
  • Coding style (indentation, etc)

Additional comments

According to the paper "Enhancing Spectrum Based Fault Localization Via Emphasizing
Its Formulas With Importance Weight" it is possible to improve the ranking of the tested methods by multiplying the result given by the SBFL formulas by the formula ef/(ef + nf). This will emphasize methods that are executed by more failing
tests and lower the rank of methods that are executed by less failing tests.

  • ef: represents the number of failed test cases covering the
    program element 𝑒.
  • np: represents the number of passed test cases not covering
    the program element 𝑒.
  • nf: represents the number of failed test cases not covering
    the program element 𝑒.

@jose
Copy link
Member

jose commented Aug 13, 2024

Thanks @mguerrinha for contributing to GZoltar. Could you please provide more context/info of these changes?

@jose
Copy link
Member

jose commented Aug 14, 2024

Thanks @mguerrinha for the additional comments. Any chance you could add a new option/parameter to enable such multiplication? I would say that by default, GZoltar should not compute such multiplication; only when enabled.

You can see the current set of GZoltar's options/parameters in here (for the CLI module) and here (for the Maven module).

@mguerrinha
Copy link
Author

Thanks @jose for the reviews. In the last commit the multiplication is called only when it's enabled. However I'm not sure if the organization and changes I had to make are the most correct.

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

Successfully merging this pull request may close these issues.

2 participants