-
Notifications
You must be signed in to change notification settings - Fork 292
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add custom scripts for arps sandbox data #307
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please have a look at the /contribute dir and follow the instruction there.
It is highly recommended to prepare visuaisation (script.js), raw (raw.js), and statistical feature enabled (eob.js) scripts to make it eaiser to be adopted by users. Please also make sure the formatting is consistent, e.g., the spaces and the indents.
You could also try to local build your changes following the instruction here. By doing this you can test if your changes work as your expectation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added some comments. I think we're almost there.
* remove redundant curly brackets in setup function * format color ramp of ndvi script * remove redundant if condition of true color cloud masked script
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I modified the return statement of true_color_cloud_masked/scripts.js
. Could you please give it a try and see if the modified script produces your expected result? If yes, we can merge the PR.
Thank you @chorng for the detailed review! I did not pay heed to the importance of formatting, so thank you for pointing multiple changes there. |
This pull request adds custom evalscripts for analysis ready planetscope's new sandbox dataset.