-
Notifications
You must be signed in to change notification settings - Fork 286
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
Sampledata #19
Sampledata #19
Changes from all commits
a79f9dc
3553e52
bf24e7b
985909c
a7147e0
117164b
b355368
015d8a8
434ca2e
a357345
d905142
e221d38
5d14e8f
c2f1ca7
8f64187
f166844
3d2c4e7
d4b39f7
75d6a79
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -211,8 +211,8 @@ def NAME_to_cube(filenames, callback): | |
# --------------------------------------------- | ||
|
||
def main(): | ||
fname = iris.sample_data_path('ascii', 'NAME', '20100509_18Z_variablesource_12Z_VAAC', 'Fields_grid1_201005110600.txt') | ||
|
||
fname = iris.sample_data_path('ascii', 'NAME', '20100509_18Z_variablesource_12Z_VAAC_subset', 'Fields_grid1_201005110600.txt') | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I know you've inherited them, but the file names in the Iris examples are crazy! The complicated directory structure stuff is just an artefact of the original data zoo. We need much more user-friendly names. E.g. perhaps just "VAAC.txt" for this one, or "NAME.txt". And certainly, there's no need for any of them to have a "_subset" suffix. |
||
boundary_volc_ash_constraint = iris.Constraint('VOLCANIC_ASH_AIR_CONCENTRATION', flight_level='Boundary layer') | ||
|
||
# Callback shown as None to illustrate where a cube-level callback function would be used if required | ||
|
@@ -237,4 +237,3 @@ def main(): | |
|
||
if __name__ == '__main__': | ||
main() | ||
|
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.
This plot looks far less interesting now. Can the number of levels be increased slightly to see any improvement?
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.
Instead of chopping it down to (2, 8, 50, 50), why not leave the displayed dimensions intact and the chop the others? e.g. (2, 70, 2, 100). That would look as good as the original but be even smaller than your current file.
Or you could go even further, and pre-trim the vertical data as well (ditching the comment about emphasising the terrain) -> (2, 15, 2, 100).