search

unity-sds / unity-py

Apache License 2.0
2 stars 4 forks source link

roles is already an array, removed "array of array" #74

Closed mike-gangl closed 5 months ago

mike-gangl commented 5 months ago

Fix to the to_stac method where we take an existing array (df.roles) and wrapped it in another array([df.roles])

Our to_stac output looked like this:

 "/unity/ads/outputs/SBG-L2A-RESAMPLE/SISTER_AVNG_L2A_RSRFL_20220502T180901_001.bin": {
      "href": "/unity/ads/outputs/SBG-L2A-RESAMPLE/SISTER_AVNG_L2A_RSRFL_20220502T180901_001.bin",
      "title": "binary file",
      "description": "",
      "roles": [
        [
          "data"
        ]
      ]
    },

and it should look like this:

 "/unity/ads/outputs/SBG-L2A-RESAMPLE/SISTER_AVNG_L2A_RSRFL_20220502T180901_001.bin": {
      "href": "/unity/ads/outputs/SBG-L2A-RESAMPLE/SISTER_AVNG_L2A_RSRFL_20220502T180901_001.bin",
      "title": "binary file",
      "description": "",
      "roles": [
          "data"
      ]
    },
coveralls commented 5 months ago

Coverage Status

coverage: 85.912%. remained the same when pulling d8eaa147913c0b3166709e7f1c21a1f03153da3f on bug/roles-as-array into 746b930daaab4abfc61b09c31a177d685520e86a on develop.