Skip to content
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

Syntax Error in B2_Interpreting_Images.qmd #1

Open
frasercrichton opened this issue Jul 12, 2023 · 1 comment
Open

Syntax Error in B2_Interpreting_Images.qmd #1

frasercrichton opened this issue Jul 12, 2023 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@frasercrichton
Copy link

Hi folks,

with line 427 in B2_Interpreting_Images.qmd:

var seaRemap = seaWhere.remap([0, 1, 2], // Existing values.   [9, 11, 10]); // Remapped values.  

It causes a syntax error when you copy and paste it into the Earth Engine because of the comments.

Could it be rewritten? Maybe as:

var existingValues = [0, 1, 2]
var remappedValues = [9, 11, 10]
var seaRemap = seaWhere.remap(existingValues, remappedValues);   

Happy to PR.

A minor niggle as the course is great!

@frasercrichton
Copy link
Author

Looking a bit more at this and in the generated script checkpoints it works fine as the parser generates code on new lines but I've been working through cut and pasting from the docs to understand the code a bit more.

@GalenReich GalenReich added the bug Something isn't working label Mar 14, 2024
@GalenReich GalenReich self-assigned this Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants