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

Wetsnow notebook #19

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Wetsnow notebook #19

wants to merge 5 commits into from

Conversation

ValentinaHutter
Copy link
Contributor

No description provided.

@ValentinaHutter
Copy link
Contributor Author

not sure about the colormap - if required, update as you prefer!

Copy link
Contributor

@sreimond-eodc sreimond-eodc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice thanks.
maybe add a few comments about what's happening? E.g., here:

unique, counts = np.unique(wsm[0, 4450:6800, 0:4000], return_counts=True)

or in general a very short summary after the title?

@ValentinaHutter
Copy link
Contributor Author

nice thanks. maybe add a few comments about what's happening? E.g., here:

unique, counts = np.unique(wsm[0, 4450:6800, 0:4000], return_counts=True)

or in general a very short summary after the title?

I agree, I asked Senmao about it, as I do not have detailed information about the values or their meaning.

@ValentinaHutter
Copy link
Contributor Author

nice thanks. maybe add a few comments about what's happening? E.g., here:
unique, counts = np.unique(wsm[0, 4450:6800, 0:4000], return_counts=True)
or in general a very short summary after the title?

I agree, I asked Senmao about it, as I do not have detailed information about the values or their meaning.

the np.unique is used to get an overview about the values that are actually in the dataset - we might want to remove it, if we can explain the meaning of the values in the introduction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants