ee-rgb-timeseries icon indicating copy to clipboard operation
ee-rgb-timeseries copied to clipboard

Start year and end year parameters not being respected

Open jdbcode opened this issue 4 years ago • 5 comments

[via email to @jdbcode]

I have been using this module to look at the NDVI time series of some fire sites I studied in the Northwest Territories. I am trying to run the Landsat example, but I seem to be having some problems setting the ee-lcb parameters. I am interested in extracting only the data from 2010-2020. For some reason, regardless of how I change the start/end year, I am still always getting 1984-2018. I have included a screenshot below. The code is line-for-line the same as your example.

image

jdbcode avatar Mar 28 '21 17:03 jdbcode

I can't reproduce the problem. If I change the start and end year, the chart reflects the change (see image). I know you said the script is line-for-line the same, but can you send me the script link so I can try to reproduce the issue.

image

jdbcode avatar Mar 28 '21 17:03 jdbcode

Hi Justin, here's the script link. GEE_RGB_TS

Edit: Updated link to public repository

arangech avatar Mar 29 '21 00:03 arangech

This is strange. The fix is to define a props variable resulting from the lcb.setProps call and when lcb props are needed, retrieve them with props.propertyName instead of lcb.props.propertyName. I can't figure out I could not reproduce. Anyway here is your script with the changes:

https://code.earthengine.google.com/037107c9d1d958f59da5ce591e6f2df6

I'm going to keep this open for a while until I can figure out why I could not reproduce.

I have fixed the example on the readme page.

jdbcode avatar Mar 29 '21 17:03 jdbcode

Note that the root of the issue is with the required ee-lcb module: https://github.com/jdbcode/EE-LCB, not the this rgb-timeseries module

jdbcode avatar Mar 29 '21 17:03 jdbcode

This is strange. The fix is to define a props variable resulting from the lcb.setProps call and when lcb props are needed, retrieve them with props.propertyName instead of lcb.props.propertyName. I can't figure out I could not reproduce. Anyway here is your script with the changes:

https://code.earthengine.google.com/037107c9d1d958f59da5ce591e6f2df6

I'm going to keep this open for a while until I can figure out why I could not reproduce.

I have fixed the example on the readme page.

Thanks. This works for me now with the props variable defined.

arangech avatar Mar 31 '21 21:03 arangech

Not using the ee-lcb module anymore - this particular issue is not relevant anymore.

jdbcode avatar Feb 28 '24 22:02 jdbcode