)]}'
{"zun/objects/inventory.py":[{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"3e66805577cddd560ae539ee65a4f796565456f1","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    VERSION \u003d \u00271.0\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    fields \u003d {"},{"line_number":29,"context_line":"        \u0027id\u0027: fields.IntegerField(read_only\u003dTrue),"},{"line_number":30,"context_line":"        \u0027resource_provider\u0027: fields.ObjectField(\u0027ResourceProvider\u0027),"},{"line_number":31,"context_line":"        \u0027resource_class\u0027: z_fields.ResourceClassField(read_only\u003dTrue),"},{"line_number":32,"context_line":"        \u0027total\u0027: fields.NonNegativeIntegerField(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_33666c2a","line":29,"updated":"2017-02-15 10:50:41.000000000","message":"don\u0027t need a uuid?","commit_id":"139f7318d496fe9e4cff4f6e57d033231f3e0834"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"020fd4b44fb339a269da7c8e9f935339f144e56f","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    VERSION \u003d \u00271.0\u0027"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"    fields \u003d {"},{"line_number":29,"context_line":"        \u0027id\u0027: fields.IntegerField(read_only\u003dTrue),"},{"line_number":30,"context_line":"        \u0027resource_provider\u0027: fields.ObjectField(\u0027ResourceProvider\u0027),"},{"line_number":31,"context_line":"        \u0027resource_class\u0027: z_fields.ResourceClassField(read_only\u003dTrue),"},{"line_number":32,"context_line":"        \u0027total\u0027: fields.NonNegativeIntegerField(),"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_89eb40db","line":29,"in_reply_to":"fa31d9ce_33666c2a","updated":"2017-02-16 04:38:06.000000000","message":"Perhaps, it needs. I think it depends on whether we want to expose Inventory via a REST API, which is not decided yet. I am ok to add it if you want, but it is better to do it in a separated patch since it requires a db migration.","commit_id":"139f7318d496fe9e4cff4f6e57d033231f3e0834"},{"author":{"_account_id":12175,"name":"Eli Qiao","email":"qiaoliyong@gmail.com","username":"Eli"},"change_message_id":"3e66805577cddd560ae539ee65a4f796565456f1","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \u0027max_unit\u0027: fields.NonNegativeIntegerField(default\u003d1),"},{"line_number":36,"context_line":"        \u0027step_size\u0027: fields.NonNegativeIntegerField(default\u003d1),"},{"line_number":37,"context_line":"        \u0027allocation_ratio\u0027: fields.NonNegativeFloatField(default\u003d1.0),"},{"line_number":38,"context_line":"        \u0027is_nested\u0027: fields.IntegerField(nullable\u003dFalse),"},{"line_number":39,"context_line":"        \u0027blob\u0027: fields.ListOfStringsField(nullable\u003dTrue),"},{"line_number":40,"context_line":"    }"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_137c08b4","line":38,"updated":"2017-02-15 10:50:41.000000000","message":"An integer? if nullable\u003dFalse, should have a default value.","commit_id":"139f7318d496fe9e4cff4f6e57d033231f3e0834"},{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"020fd4b44fb339a269da7c8e9f935339f144e56f","unresolved":false,"context_lines":[{"line_number":35,"context_line":"        \u0027max_unit\u0027: fields.NonNegativeIntegerField(default\u003d1),"},{"line_number":36,"context_line":"        \u0027step_size\u0027: fields.NonNegativeIntegerField(default\u003d1),"},{"line_number":37,"context_line":"        \u0027allocation_ratio\u0027: fields.NonNegativeFloatField(default\u003d1.0),"},{"line_number":38,"context_line":"        \u0027is_nested\u0027: fields.IntegerField(nullable\u003dFalse),"},{"line_number":39,"context_line":"        \u0027blob\u0027: fields.ListOfStringsField(nullable\u003dTrue),"},{"line_number":40,"context_line":"    }"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa31d9ce_89c26051","line":38,"in_reply_to":"fa31d9ce_137c08b4","updated":"2017-02-16 04:38:06.000000000","message":"I can give a default value. Will address it in a revision.","commit_id":"139f7318d496fe9e4cff4f6e57d033231f3e0834"}]}