)]}'
{"glanceclient/common/utils.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"9ecb5fb5ec2c83e581c689ee757ccef873f564ca","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    image \u003d dict([item for item in six.iteritems(image_obj)"},{"line_number":378,"context_line":"                  if item[0] not in ignore])"},{"line_number":379,"context_line":"    if human_readable:"},{"line_number":380,"context_line":"        image[\u0027size\u0027] \u003d make_size_human_readable(image[\u0027size\u0027])"},{"line_number":381,"context_line":"    if str(max_col_width).isdigit():"},{"line_number":382,"context_line":"        print_dict(image, max_column_width\u003dmax_col_width)"},{"line_number":383,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_585edfea","line":380,"updated":"2015-03-23 07:50:13.000000000","message":"Can this be the default behavior? I think we should *always* print the image size in a human readable format. Is it important for users to have the image size in bytes?","commit_id":"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"},{"author":{"_account_id":455,"name":"Stuart McLaren","email":"stuart.mclaren@hpe.com","username":"stuart-mclaren"},"change_message_id":"146e837aad7e2b25796812c6d308296f224cf153","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    image \u003d dict([item for item in six.iteritems(image_obj)"},{"line_number":378,"context_line":"                  if item[0] not in ignore])"},{"line_number":379,"context_line":"    if human_readable:"},{"line_number":380,"context_line":"        image[\u0027size\u0027] \u003d make_size_human_readable(image[\u0027size\u0027])"},{"line_number":381,"context_line":"    if str(max_col_width).isdigit():"},{"line_number":382,"context_line":"        print_dict(image, max_column_width\u003dmax_col_width)"},{"line_number":383,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_6173e34d","line":380,"in_reply_to":"1a6ced46_1eaa8f19","updated":"2015-03-23 11:03:10.000000000","message":"I think changing the default is problematic from a backwards compatibilty point of view. Any script expecting the current behaviour would fail.","commit_id":"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"482d3a14bd6c457dd388c3680ae15f43e47d34ed","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    image \u003d dict([item for item in six.iteritems(image_obj)"},{"line_number":378,"context_line":"                  if item[0] not in ignore])"},{"line_number":379,"context_line":"    if human_readable:"},{"line_number":380,"context_line":"        image[\u0027size\u0027] \u003d make_size_human_readable(image[\u0027size\u0027])"},{"line_number":381,"context_line":"    if str(max_col_width).isdigit():"},{"line_number":382,"context_line":"        print_dict(image, max_column_width\u003dmax_col_width)"},{"line_number":383,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_1eaa8f19","line":380,"in_reply_to":"1a6ced46_585edfea","updated":"2015-03-23 09:00:55.000000000","message":"Yes, this can be set as default, but i think we should provide both options. So that users have flexibility to choose from both the available options.","commit_id":"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"02d8b16d8a3c845e74c51f9620762b7f8dcc6ac9","unresolved":false,"context_lines":[{"line_number":377,"context_line":"    image \u003d dict([item for item in six.iteritems(image_obj)"},{"line_number":378,"context_line":"                  if item[0] not in ignore])"},{"line_number":379,"context_line":"    if human_readable:"},{"line_number":380,"context_line":"        image[\u0027size\u0027] \u003d make_size_human_readable(image[\u0027size\u0027])"},{"line_number":381,"context_line":"    if str(max_col_width).isdigit():"},{"line_number":382,"context_line":"        print_dict(image, max_column_width\u003dmax_col_width)"},{"line_number":383,"context_line":"    else:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a6ced46_878d4a7e","line":380,"in_reply_to":"1a6ced46_6173e34d","updated":"2015-03-23 13:42:53.000000000","message":"Agree with @Stuart, @Flavio what are your comments on this. I think we should go with the change.","commit_id":"c149a94ee1c8c5a04b984063cadf1dbc934eeb8b"}]}