[#944] Change to originality score and new threshold value #2072

SkyBlaise99 · 2023-12-09T02:46:43Z

Part of #944

Proposed commit message

Currently, lowering the similarity threshold increases the algorithm’s
stringency. Typically, threshold functions in a way where meeting or
surpassing it results in a better score. However, in this case, full
credit is awarded when the similarity threshold is not reached.

Also, it is unclear why the similarity threshold was set to 0.8, which
means that the last author only need to change roughly 20% of the line
to be awarded full credit.

Let's use a new metric called the originality score, where exceeding the
originality threshold will then be awarded a full credit. Furthermore,
let's update the originality threshold to to 0.51, where the last author
needs to change at least half the line to be considered as making a
significant change and obtain full credit for it.

Other information

Basically, originalityScore = 1 - similarityScore.

Currently, users are unable to select a zoom range that includes the until date. This results in misleading data being presented to users.

…ense#2041) Chrome bug is causing cypress to fail to open a browser on Github Actions, causing frontend tests and CI to fail. Upgrading cypress to greater than 12.15.0 will fix this issue. Let's upgrade cypress to fix the failing CI.

Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate the rest of the JavaScript code to TypeScript code to facilitate future changes to the code.

…posense#2040) Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate the rest of the JavaScript code to TypeScript code to facilitate future changes to the code.

Currently, Cypress zoom feature tests are failing due to a recent change in behavior caused by a bug fix. With the tests failing, we are unable to detect any future regressions. Let's update the Cypress tests to test for the new intended behavior.

…#2043) Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate random-color-generator.js JavaScript code to TypeScript code to facilitate future changes to the code.

…sense#2036) Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate the rest of the JavaScript code to TypeScript code to facilitate future changes to the code.

Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate the rest of the JavaScript code to TypeScript code to facilitate future changes to the code.

Bumps [zod](https://github.com/colinhacks/zod) from 3.20.6 to 3.22.3. - [Release notes](https://github.com/colinhacks/zod/releases) - [Changelog](https://github.com/colinhacks/zod/blob/master/CHANGELOG.md) - [Commits](colinhacks/zod@v3.20.6...v3.22.3) --- updated-dependencies: - dependency-name: zod dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [@cypress/request](https://github.com/cypress-io/request) to 3.0.1 and updates ancestor dependency [cypress](https://github.com/cypress-io/cypress). These dependencies need to be updated together. Updates `@cypress/request` from 2.88.12 to 3.0.1 - [Release notes](https://github.com/cypress-io/request/releases) - [Changelog](https://github.com/cypress-io/request/blob/master/CHANGELOG.md) - [Commits](cypress-io/request@v2.88.12...v3.0.1) Updates `cypress` from 12.17.4 to 13.3.0 - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v12.17.4...v13.3.0) --- updated-dependencies: - dependency-name: "@cypress/request" dependency-type: indirect - dependency-name: cypress dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate the rest of the JavaScript code to TypeScript code to facilitate future changes to the code.

author

Currently, when granularity is set to day or week, clicking on a ramp will open up a zoom view where commit messages are not being displayed and sorting by insertions does not result in any sorting. Let's fix the unintended behaviour of the zoom view.

Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate repo-sorter.js to TypeScript code to facilitate future changes to the code.

Currently, there is still some JavaScript code which remains unmigrated. This allows for type unsafe code to be written, potentially resulting in unintended behavior. Let's migrate safari_date.js to TypeScript code to facilitate future changes to the code.

Currently, frontend linter is failing due to lint scripts checking javascript files, the last of which has been removed in PR reposense#2053. Lets update the lint command to exclude javascript files front the check.

…nse#2056) Currently, most tooltips are shown above buttons and text. When these tooltips appear at the top of the viewport, part of the tooltips will not be rendered. Let's implement changes such that these tooltips appear below the text or button, when appearing at the top of the viewport.

… file title (reposense#2057) Currently, if one hovers over a tooltip of the pinned title of a file whose content is scrolled almost completely, such that the title of the next file is just below the pinned title, the tooltip is not displayed appropriately, as the title of the next file obstructs it. Let's fix this issue.

…cs (reposense#2050) There are still leftover references specific to GitHub on parts of the codebase and docs that have been generalized to accept other remote git hosts. Let's update these GitHub references to use more general language.

# Conflicts: # frontend/cypress/tests/chartView/chartView_zoomFeature.cy.js # frontend/src/views/c-authorship.vue

This reverts commit 950c912, reversing changes made to 4bd05a7.

…orship

ckcherry23

LGTM

github-actions · 2024-01-08T06:26:10Z

The following links are for previewing this pull request:

Dashboard Preview: https://dashboard-2072-pr-reposense-reposense.surge.sh
Docs Preview: https://docs-2072-pr-reposense-reposense.surge.sh

jq1836 and others added 30 commits September 25, 2023 01:37

[reposense#2027] Fix date range bug (reposense#2034)

075d34d

Currently, users are unable to select a zoom range that includes the until date. This results in misleading data being presented to users.

Merge branch 'master' into 944-analyze-authorship

174ecc5

Give partial credit if annotated author is not the same as the blame

e200e5e

author

Remove frontend JS lint (reposense#2063)

056fa5f

Currently, frontend linter is failing due to lint scripts checking javascript files, the last of which has been removed in PR reposense#2053. Lets update the lint command to exclude javascript files front the check.

use full and partial credit color

bdeb15a

Add test cases for annotated author overriding last author's credit

a187d9c

Merge branch 'master' into 944-analyze-authorship

58b7002

revert merge from master

b296b83

revert merge from master 58b7002

4ce6545

Trigger workflow

4bd05a7

Merge branch 'master' into 944-analyze-authorship

950c912

# Conflicts: # frontend/cypress/tests/chartView/chartView_zoomFeature.cy.js # frontend/src/views/c-authorship.vue

Revert "Merge branch 'master' into 944-analyze-authorship"

a46d423

This reverts commit 950c912, reversing changes made to 4bd05a7.

fix frontend test failing

bba556d

Merge branch '944-analyze-authorship' into 944-analyze-authorship

4d7d3aa

SkyBlaise99 and others added 5 commits November 12, 2023 15:10

Merge branch '944-swap-color' into 944-analyze-authorship

1b25572

Merge branch 'reposense:944-analyze-authorship' into 944-analyze-auth…

9e93961

…orship

switch to originality score and threshold

7908253

update originality threshold

d086da2

revert frontend code changes

6c8a051

SkyBlaise99 marked this pull request as ready for review December 9, 2023 11:17

ckcherry23 approved these changes Jan 8, 2024

View reviewed changes

ckcherry23 merged commit c85423e into reposense:944-analyze-authorship Jan 8, 2024
9 of 10 checks passed

SkyBlaise99 deleted the 944-originality-score branch February 6, 2024 01:33

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#944] Change to originality score and new threshold value #2072

[#944] Change to originality score and new threshold value #2072

SkyBlaise99 commented Dec 9, 2023 •

edited

Loading

ckcherry23 left a comment

github-actions bot commented Jan 8, 2024

[#944] Change to originality score and new threshold value #2072

[#944] Change to originality score and new threshold value #2072

Conversation

SkyBlaise99 commented Dec 9, 2023 • edited Loading

Proposed commit message

Other information

ckcherry23 left a comment

Choose a reason for hiding this comment

github-actions bot commented Jan 8, 2024

SkyBlaise99 commented Dec 9, 2023 •

edited

Loading