-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Unresolved and unexplained bug in timed events $duration property #149
Comments
hey @martins-jansevskis , just reached out to you in the support ticket. Mind following up there? It looks like $duration should still be in seconds (even if the timestamp of the event itself has millisecond precision): https://github.com/mixpanel/mixpanel-android/blob/153674249b3f588dcb597318d850a76460e7725a/src/main/java/com/mixpanel/android/mpmetrics/MixpanelAPI.java#L800 |
The following two lines of code are in milliseconds and not seconds:
|
You are correct, the reference I had above was a confusion I had with the Android library. I will create an internal bug report and review it with the engineering team. |
@martins-jansevskis @argenisf thanks for the report. this issue should be fixed in |
The commit: #146
The mixpanel documentation are describing that $duration is in 'seconds' for every timed event int Unity, Android and iOS SDKs.
The '$duration' property in 3.3.1 Unity is a timestamp in milliseconds and seconds from other SDKs and older Unity SDK versions.
The clients who have production applications with some users on 3.0.1 and others in 3.3.1 have corrupt '$duration' data.
The text was updated successfully, but these errors were encountered: