Skip to content

Commit

Permalink
Mitigate AI SDK's problem with numeric in properties (#2397)
Browse files Browse the repository at this point in the history
  • Loading branch information
troydai authored Mar 8, 2017
1 parent 2b77c3c commit a897493
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 2 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ __pycache__/
# Virtual environment
env/
env27/
.python-version

# PTVS analysis
.ptvs/
Expand Down
18 changes: 16 additions & 2 deletions src/azure-cli-core/azure/cli/core/telemetry_upload.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
import os
import sys
import json
import six
import azure.cli.core.decorators as decorators

DIAGNOSTICS_TELEMETRY_ENV_NAME = 'AZURE_CLI_DIAGNOSTICS_TELEMETRY'
Expand Down Expand Up @@ -39,12 +40,25 @@ def upload(data_to_save):
sys.stdout.write('Raw [{}]/n'.format(data_to_save))

for record in data_to_save:
client.track_event(record['name'], record['properties'])
name = record['name']
raw_properties = record['properties']
properties = {}
measurements = {}
for k in raw_properties:
v = raw_properties[k]
if isinstance(v, six.string_types):
properties[k] = v
else:
measurements[k] = v
client.track_event(record['name'], properties, measurements)

if in_diagnostic_mode():
sys.stdout.write('\nTrack Event: {}\nProperties: {}\nMeasurements: {}'.format(
name, json.dumps(properties), json.dumps(measurements)))

client.flush()

if in_diagnostic_mode():
json.dump(data_to_save, sys.stdout, indent=2, sort_keys=True)
sys.stdout.write('\nTelemetry upload completes\n')


Expand Down

0 comments on commit a897493

Please sign in to comment.