Skip to content

Add Jaccard as a fourth SBFL ranking metric - #2

Merged
mohrez86 merged 1 commit into
atom-sw:mainfrom
Sonzakir:main
Jul 6, 2026
Merged

Add Jaccard as a fourth SBFL ranking metric#2
mohrez86 merged 1 commit into
atom-sw:mainfrom
Sonzakir:main

Conversation

@Sonzakir

@Sonzakir Sonzakir commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Implements the standard Jaccard similarity coefficient, ef / (ef + nf + ep),
alongside Tarantula, Ochiai, and DStar.

  • New MetricJaccard class following the existing metric interface, with the
    shared epsilon guard against division by zero
  • Registered in RankingMetricManager; added Jaccard column + index to the
    Score table and included it in both ranked-entity queries

No downstream changes needed, reporting iterates the metrics generically, so
Jaccard gets its own score table and report file automatically.

@mohrez86

mohrez86 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Hi @Sonzakir ,

Thank you for your contribution. Adding Jaccard to SBFL is actually a nice idea, and I really appreciate how you cleanly adapted the codebase's style. I am a bit tied up at the moment, but I will run some end-to-end tests asap and merge the PR once everything looks good.

@Sonzakir

Sonzakir commented Jul 5, 2026

Copy link
Copy Markdown
Contributor Author

Hi @mohrez86 ,

Thank you so much for the quick response and for this excellent tool. FauxPy has been incredibly useful for the Repair Framework I’m currently working on.

In addition, I ran into a few bugs when trying to run FauxPy in bug-isolated containers. I seem to have resolved them for now, but I haven’t tested the framework thoroughly enough yet. Once I’ve finished reviewing everything on my end and the tests pass, I’ll open a pull request for that as well.

@mohrez86
mohrez86 merged commit 46a06fc into atom-sw:main Jul 6, 2026
4 checks passed
@mohrez86

mohrez86 commented Jul 6, 2026

Copy link
Copy Markdown
Collaborator

@Sonzakir ,

Thank you for the feature! Merged.
I am glad to hear that FauxPy has been useful for the repair framework you are working on.
Please do open that next PR once you have finished testing.

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