Page MenuHomePhabricator

Hide "Herald added subscribers..." stories from feed completely
Closed, ResolvedPublic

Description

I noticed a weird looking feed item.

weird_feed.png (600×1 px, 163 KB)

This was the actual event(s).

actual.png (203×1 px, 28 KB)

Maybe this is some sort of batching/coalescing bug? Right now it looks like the comment and the herald triggering got jammed together.

Event Timeline

sshannin raised the priority of this task from to Needs Triage.
sshannin updated the task description. (Show Details)
sshannin added projects: Herald, Feed.
sshannin added a subscriber: sshannin.

This is displaying as intended. That is, if there is no image available, we still outline the actor space for consistency.

Oh, yes the "Herald" as the actor is incorrect.

This behavior is "correct" in some sense (Herald really did perform the "strongest" action in the transaction group) but the sorting on tiebreakers is likely wrong (see D14680 for a similar case) and we should sort actions by real users over actions by Herald at the same strength.

That is, there were three actions here:

  1. Chad added a subscriber.
  2. Chad added a comment.
  3. Herald added a subscriber,

I haven't actually looked at the values, but the result might be because of something like this:

(2) has the weakest strength (any other transaction is always more interesting than "added a comment", since we show the comment anyway), so we have a choice between (1) and (3) as the title of the feed story. We incorrectly select (3) as a stronger action, when a better choice would be (1).

However, it's actually possible that we're already doing strengths correctly -- I think we may have a special rule to make "user added <themselves> as a subscriber" into a very weak action. This legitimately makes the Herald action stronger. It should not be, so maybe we just add a rule to make "<any application> <did anything> <to subcribers>" extremely weak because that is soooo boring.

I'm not sure what we should do in this case:

  • Chad adds a subscriber.
  • In response, Herald closes the task.

In this case, "close" is legitimately very strong. What should we do?

  • Publish "Chad added a subscriber."
  • Publish "Herald closed the task."
  • (Publish both? Having Herald generate a second story is probably really bad in a lot of cases.)
  • (Something else?)

Offhand, I'm inclined to still publish "Herald closed the task." in the "subscribe + close" hypothetical, which I think is the current behavior.

Maybe slightly always ding a computer action vs. a human action. That is, if a human adds a subscriber, and herald adds a subscriber, that weight is (human action weight - 1), or something along those lines? Then if human adds a subscriber, but herald closes, the weight of the close action even though gets dinged, is likely still stronger.

From T10822, we currently choose [Changed Subscribers] as a stronger action than [Merged] for tasks in some cases in email subject lines. This is probably not the correct heuristic, [Merged] is normally a strong action.

There's some level of magical heuristic, so yell if you still see issues.

I think that story strength thing is non-retroactive? This may not really be fixed if new stories aren't better.

very nice

if it breaks again we can try adding some more 0.000000000's first and see if that helps

Deliberate Subscriber + Herald Subscriber = Herald Subscriber Story.

pasted_file (121×363 px, 18 KB)

That's because it was deliberate self-subscribe which is so completely boring that we normally wouldn't even show it. I think it's arguably correct to show the Herald story? Maybe?

I think if you add me and Herald adds eadler, it will show "chad added epriestley".

But if you add yourself (SO BORING) and Herald adds eadler, it shows "Herald added eadler".

If we don't do that, self-adds sometimes show up and sometimes don't which is a little weird maybe? I'm not sure there's any way to resolve this that's consistent/obvious/predictable/always makes sense.

maybe herald adding a subscriber is truly 0 then as a feed story.

Like just don't show it at all? That seems reasonable to me, although 0.00000000001 is very funny and we would be losing a truly great joke at such a young age.

Just reopening this so I don't forget about it since I agree about that last change.

epriestley renamed this task from herald showing as cause of feed item to Hide "Herald added subscribers..." stories from feed completely.Jan 26 2018, 5:32 PM

0.00000000001 is very funny and we would be losing a truly great joke at such a young age

F

This actually died earlier in D20540, but got to live for far longer than it probably should have.