Skip to content

remove duplicated outputs#1

Open
plight-cmu wants to merge 1 commit intomainfrom
duplicate-levels
Open

remove duplicated outputs#1
plight-cmu wants to merge 1 commit intomainfrom
duplicate-levels

Conversation

@plight-cmu
Copy link
Collaborator

Remove the duplicate printing of level 2, as well as duplicated optional fields.

Problem:
  1. When running part_3_convert_json.py, we saw that the contents of level 2 are printed out twice, instead of seeing level 1 and level 2 in order.
  2. Also, optional fields are printed twice in the second repetition of the field

Solution:
  In each iteration of the loop, create new objects for the level and fields read in. The
  duplication was caused by adding the exact same object to the level pack twice.

Remove the duplicate printing of level 2, as well as duplicated optional fields.

    Problem:
      1. When running part_3_convert_json.py, we saw that the contents of level 2 are printed out
         twice, instead of seeing level 1 and level 2 in order.
      2. Also, optional fields are printed twice in the second repetition of the field

    Solution:
      In each iteration of the loop, create new objects for the level and fields read in. The
      duplication was caused by adding the exact same object to the level pack twice.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant