Page MenuHomePhabricator

Data Validation for iOS Image Recs
Closed, ResolvedPublic

Description

Once Image Recommendations is in Test Flight and available to beta testers, test and validate instrumentation.

Event Timeline

HNordeenWMF moved this task from Needs Triage to Up next on the Wikipedia-iOS-App-Backlog board.

Other external TestFlight beta builds: TestFlight 7.5.0 (3525) and TestFlight 7.5.0 (3511).

Tsevener raised the priority of this task from Low to High.Apr 30 2024, 8:10 PM

Validating we are seeing almost all events, full action_data for successful edits and edit events tracked in editattemptstep

Data Validation

I am not seeing the following events yet:

actionactive_interface

|view_edit_help| editsummary_dialog
|view_watchlist_help| editsummary_dialog
|impression|nosuggestions_dialog (I am seeing nosuggestions_back action, so this should be firing too)

@SNowick_WMF Thanks! I made a PR to fix the missing impression event. I was able to trigger view_edit_help and view_watchlist_help, so maybe beta testers aren't tapping those buttons?

Mazevedo subscribed.

PR with fixes merged, we'll validate it in the release

Hi @SNowick_WMF , could you please plan to validate that Watchlist Toggle instrumentation change as well in this task? https://github.com/wikimedia/wikipedia-ios/pull/4840
Thanks!

Validating we are seeing prior missing events from above:

actionactive_interface
view_edit_helpeditsummary_dialog
view_watchlist_helpeditsummary_dialog
impressionnosuggestions_dialog

@Tsevener I am re-opening this so we can investigate an issue I found in edit data while doing 15 Day Image Rec analysis - we can discuss in App Data meeting.

Details:
While querying for Image Rec edits I ran into 74 edits where we had revision_ids in our android_image_recommendation_event data that I couldn't match in data from my regular mediawiki_historyquery that looks for ios app edit in the revision_tags column.

I pulled the edit data by the revision_id from android_image_recommendation_event and found that for all of these edits there was no ios app edit value in the revision_tags - the column only showed ['mobile edit', 'mobile app edit'] - these were known iOS app edits (with Added image, etc. in event_comment for most of them) but for some reason they weren't properly attributed as ios app edit- this happened across multiple wikis:

wikirev_id (Count Unique)
enwiki63
dewiki5
cawiki2
frwiki2
itwiki1
trwiki1

Change #1039302 had a related patch set uploaded (by Dbrant; author: Dbrant):

[mediawiki/extensions/MobileApp@master] Look for iPadOS in user-agent, in addition to iOS.

https://gerrit.wikimedia.org/r/1039302

Patch looks good, I tested with both iPhone and iPad user agents. I tried to test a Vision Pro user agent, but I could not get my Vision Pro simulator to appear for testing.

Change #1039302 merged by jenkins-bot:

[mediawiki/extensions/MobileApp@master] Look for iPadOS in user-agent, in addition to iOS.

https://gerrit.wikimedia.org/r/1039302

Change #1043110 had a related patch set uploaded (by Dbrant; author: Dbrant):

[mediawiki/extensions/MobileApp@wmf/1.43.0-wmf.9] Look for iPadOS in user-agent, in addition to iOS.

https://gerrit.wikimedia.org/r/1043110

Change #1043110 merged by jenkins-bot:

[mediawiki/extensions/MobileApp@wmf/1.43.0-wmf.9] Look for iPadOS in user-agent, in addition to iOS.

https://gerrit.wikimedia.org/r/1043110

Mentioned in SAL (#wikimedia-operations) [2024-06-13T21:01:43Z] <jsn@deploy1002> Started scap: Backport for [[gerrit:1043110|Look for iPadOS in user-agent, in addition to iOS. (T362723)]]

Mentioned in SAL (#wikimedia-operations) [2024-06-13T21:04:09Z] <jsn@deploy1002> dbrant, jsn: Backport for [[gerrit:1043110|Look for iPadOS in user-agent, in addition to iOS. (T362723)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-06-13T21:15:54Z] <jsn@deploy1002> Finished scap: Backport for [[gerrit:1043110|Look for iPadOS in user-agent, in addition to iOS. (T362723)]] (duration: 14m 11s)

HNordeenWMF claimed this task.