search

TelemetryDeck / SwiftSDK

Swift SDK for TelemetryDeck, a privacy-conscious analytics service for apps and websites.
https://telemetrydeck.com/
Other
149 stars 30 forks source link

Architecture reported incorrectly #87

Closed biocross closed 1 year ago

biocross commented 1 year ago

Hello,

I'm using the SDK in a Mac app, and lately I've noticed:

  1. The architecture graph has automatically switched to modelName attribute.
  2. The architecture data is mixed with model name (see attached image)
Screenshot 2023-01-25 at 2 56 34 PM

This was an important data metric for me to track how many of my users were using Apple Silicon vs Intel. Is there something wrong with my query / graph configuration, or is this a bug in the SDK?

I'm using the library version 1.4.0

winsmith commented 1 year ago

Thanks for reporting this! I have some follow-up questions:

(Please note that dashboard and API are having some issues right now, caused by an outage at our hosting provider. They are aware of the problem and working diligently on it, but it might prevent you from double checking the data in the next hour or so)

biocross commented 1 year ago
  1. App ID: <hidden> (unsure if this should be public 😅)
  2. Yes, same issue on dashboard.telemetrydeck.com. Slick dashboard btw!
winsmith commented 1 year ago

Thanks, I'm investigating!

The implementation for getting the build architecture hasn't changed, so I assume this is a server issue.

The dashboard is the canonical way for TelemetryDeck now -- right now we don't have the bandwidth to work on the desktop app, so new features only land on dashboard.telemetrydeck.com

The app ID is not a strong secret, but if you'd like, edit your post to remove it, now that I've copied it. Can't hurt :)

winsmith commented 1 year ago

Okay so the insight had modelname as the breakdown property, so it's not surprising that it did show model names. What is surprising is that it did mix model names and architectures in your screenshot, so there's clearly something going on here.

I switched the insight back to using architecture as its breakdown key and it should work as expected again. In the meantime I'll see if I can find a bug that could have caused this. Let me know if you have further troubles with this please! 😊