search

r-earthengine / ee_extra

A ninja python package that unifies the Google Earth Engine ecosystem.
https://ee-extra.readthedocs.io/
Other
58 stars 10 forks source link

test_06.js fail #4

Closed csaybar closed 2 years ago

csaybar commented 2 years ago

JS file :

  var levelsDic = ee.Dictionary({
        'level0': {'zoom': 0, 'scale': 157000},
        'level1': {'zoom': 1, 'scale': 78000},
        'level2': {'zoom': 2, 'scale': 39000},
        'level3': {'zoom': 3, 'scale': 20000},
        'level4': {'zoom': 4, 'scale': 10000},
        'level5': {'zoom': 5, 'scale': 5000},
        'level6': {'zoom': 6, 'scale': 2000},
        'level7': {'zoom': 7, 'scale': 1000},
        'level8': {'zoom': 8, 'scale': 611},
        'level9': {'zoom': 9, 'scale': 306},
        'level10': {'zoom': 10, 'scale': 153},
        'level11': {'zoom': 11, 'scale': 76},
        'level12': {'zoom': 12, 'scale': 38},
        'level13': {'zoom': 13, 'scale': 19},
        'level14': {'zoom': 14, 'scale': 10},
        'level15': {'zoom': 15, 'scale': 5},
  });

GOT:

import ee
import math
from collections import UserDict

exports = UserDict()

levelsDic = ee.Dictionary({
'level0:{zoom': 0,'scale': 157000},
      'level1': {'zoom': 1,'scale': 78000},
      'level2': {'zoom': 2,'scale': 39000},
      'level3': {'zoom': 3,'scale': 20000},
      'level4': {'zoom': 4,'scale': 10000},
      'level5': {'zoom': 5,'scale': 5000},
      'level6': {'zoom': 6,'scale': 2000},
      'level7': {'zoom': 7,'scale': 1000},
      'level8': {'zoom': 8,'scale': 611},
      'level9': {'zoom': 9,'scale': 306},
      'level10': {'zoom': 10,'scale': 153},
      'level11': {'zoom': 11,'scale': 76},
      'level12': {'zoom': 12,'scale': 38},
      'level13': {'zoom': 13,'scale': 19},
      'level14': {'zoom': 14,'scale': 10},
      'level15': {'zoom': 15,'scale': 5},
})

EXPECTED:

import ee
import math
from collections import UserDict

exports = UserDict()

levelsDic = ee.Dictionary({
      'level0': {'zoom': 0,'scale': 157000},
      'level1': {'zoom': 1,'scale': 78000},
      'level2': {'zoom': 2,'scale': 39000},
      'level3': {'zoom': 3,'scale': 20000},
      'level4': {'zoom': 4,'scale': 10000},
      'level5': {'zoom': 5,'scale': 5000},
      'level6': {'zoom': 6,'scale': 2000},
      'level7': {'zoom': 7,'scale': 1000},
      'level8': {'zoom': 8,'scale': 611},
      'level9': {'zoom': 9,'scale': 306},
      'level10': {'zoom': 10,'scale': 153},
      'level11': {'zoom': 11,'scale': 76},
      'level12': {'zoom': 12,'scale': 38},
      'level13': {'zoom': 13,'scale': 19},
      'level14': {'zoom': 14,'scale': 10},
      'level15': {'zoom': 15,'scale': 5},
})
davemlz commented 2 years ago

This one should work. For me it is working well :) The fixing was done in https://github.com/r-earthengine/ee_extra/commit/9d554dc39a2aad5ec0cfef8392457b76e57ee3eb

davemlz commented 2 years ago

@csaybar try it again, make a pull, it should work! :)

csaybar commented 2 years ago

ty!