-
Notifications
You must be signed in to change notification settings - Fork 17
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
Update configs (default topo smoothing) + add docs #3416
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
akshaysridhar
force-pushed
the
as/update-configs
branch
from
October 31, 2024 22:01
dbbe946
to
5d70432
Compare
szy21
reviewed
Oct 31, 2024
szy21
reviewed
Oct 31, 2024
akshaysridhar
force-pushed
the
as/update-configs
branch
from
October 31, 2024 22:17
c63d392
to
be0f329
Compare
akshaysridhar
changed the title
Update configs (deepatmos, default topo smoothing) + add docs
Update configs (default topo smoothing) + add docs
Oct 31, 2024
akshaysridhar
force-pushed
the
as/update-configs
branch
from
October 31, 2024 22:32
c24f21c
to
e6d0937
Compare
I find it pretty hard to read the values from the plots with the colormap - is there a more quantitative colormap we can use? If it's hard then don't worry about it. |
Yes I can fix these up with cleaner plots with Makie. |
akshaysridhar
force-pushed
the
as/update-configs
branch
4 times, most recently
from
November 6, 2024 17:38
f2a8192
to
72e1d37
Compare
szy21
reviewed
Nov 6, 2024
szy21
approved these changes
Nov 6, 2024
akshaysridhar
force-pushed
the
as/update-configs
branch
from
November 6, 2024 18:30
b31019c
to
2287543
Compare
Update functions in topography_spectra example file Update docs + plots modified: examples/topography_spectra.jl Co-authored-by: Zhaoyi Shen <[email protected]>
akshaysridhar
force-pushed
the
as/update-configs
branch
from
November 6, 2024 22:09
2287543
to
b440db2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Update default topo smoothing
Add docs demonstrating topo smoothing.