Skip to content

Stop UserCred teleport from weighting likes by yesterday's mass - #174

Open
Pitchfork-and-Torch wants to merge 1 commit into
xai-org:mainfrom
Pitchfork-and-Torch:cursor/cred-teleport-circularity-1182
Open

Stop UserCred teleport from weighting likes by yesterday's mass#174
Pitchfork-and-Torch wants to merge 1 commit into
xai-org:mainfrom
Pitchfork-and-Torch:cursor/cred-teleport-circularity-1182

Conversation

@Pitchfork-and-Torch

Copy link
Copy Markdown

Bug

computeTeleportWeights built today's engagement teleport by inner-joining 7-day fav/RT edges onto yesterday's UserCred mass. An engager with no prior snapshot row contributed nothing. An engager with high prior mass donated that mass to every author they liked.

That is cred as an input to cred. Small and new accounts cannot confer authority. Elite like-rings reprint yesterday's ranking into today's teleport, then PageRank publishes tomorrow's prior.

Not the linked-pair filter. Not the premium uniform prior. Warm-start still reads the previous snapshot as the PageRank start vector only.

Proof

  1. Entry: computeTeleportWeights joined (engagerId, authorId) fav/RT counts onto priorMassPipe (previous UserCred snapshot, or uniform only on first_snapshot).
  2. Sink: engagerMass.mass * count / total became today's teleport. doPageRank plus writeResults wrote tomorrow's priorMassPipe. New engagers died on the inner join.
  3. Break: drop the prior-mass join. Each current valid engager donates count / total only. Invalid / deactivated rows no longer mint cred from leftover mass.
  4. Viewer effect: published UserCredV2.score / isHighPageRankV2 stop following last week's elite like-graph. Abuse skips and embedding user_credibility stop treating small-account endorsements as zero.
  5. Twin: same 7-day fav/RT graph, yesterday's masses swapped vs uniform. Old code moves teleport with the masses. New code does not.

Fix

One function: user-cred-v2/UserCredV2App.scala computeTeleportWeights. Call site drops the unused prior-mass argument.

Not a home-mixer / VF / Thunder change. Not a fork PR.

Engagement teleport joined 7-day fav/RT edges onto the previous
UserCred snapshot, so already-high-mass accounts minted more cred
and new or small engagers were dropped by the inner join.

Co-authored-by: Jon Bailey <Pitchfork-and-Torch@users.noreply.github.com>
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