passportxyz / passport

Passport allows users to prove their identity through a secure, decentralized UI
Other
941 stars 451 forks source link

Score Weight Update Documentation #2333

Open erichfi opened 5 months ago

erichfi commented 5 months ago

User Story:

As a member of the Passport project team, I aim to efficiently implement and communicate the recent score weight changes within our platform. This will ensure that all stakeholders, including our users and integration partners, are well-informed and can smoothly transition to these updates.

Acceptance Criteria

GIVEN the introduction of planned score weight changes, WHEN these updates are deployed in the forthcoming platform update, THEN the adjustments are precisely incorporated into the scoring model. Additionally, extensive support documentation and resources are made available to all parties impacted by these changes.

Product & Design Links:

[Placeholder for links to documentation, FAQs, and change logs detailing the score weight changes]

Tech Details:

The upcoming update will introduce several adjustments to the weight of credentials available in Passport. These changes aim to refine the scoring model, enhancing its effectiveness and equity. The detailed list of score weight changes includes both increments and reductions, underpinned by thorough analysis and deliberation. Below is the updated list of credentials and their new weights:

Open Questions:

Notes/Assumptions:

erichfi commented 5 months ago

Pessimistic 2 Points

lucianHymer commented 5 months ago

Hey @erichfi ! What exactly does this ticket include from an engineering perspective? Should I just be updating the default weights and then running the rescore after merging? The UI updates have been addressed here https://github.com/gitcoinco/passport/issues/2330

lucianHymer commented 5 months ago

These new scores add to 125.913 Is that fine? Are some of these mutually exclusive now?

It doesn't really matter from a scoring logic perspective. But it does make the progress ring in the UI potentially a poor representation of the score.

erichfi commented 5 months ago

Hey @erichfi ! What exactly does this ticket include from an engineering perspective? Should I just be updating the default weights and then running the rescore after merging? The UI updates have been addressed here https://github.com/gitcoinco/passport/issues/2330

@lucianHymer, yes, making sure everything gets scored as planned.

These new scores add to 125.913 Is that fine? Are some of these mutually exclusive now?

It doesn't really matter from a scoring logic perspective. But it does make the progress ring in the UI potentially a poor representation of the score.

Yes the total >100 is fine

lucianHymer commented 4 months ago

Will handle re-scoring after deployment