Skip to content

Revised documentation, whatsnew and deprecations.#31

Merged
pp-mo merged 1 commit intostructured_load_apifrom
structured_load_api_whatsnew
Nov 14, 2016
Merged

Revised documentation, whatsnew and deprecations.#31
pp-mo merged 1 commit intostructured_load_apifrom
structured_load_api_whatsnew

Conversation

@pp-mo
Copy link
Owner

@pp-mo pp-mo commented Nov 9, 2016

Attempt at whatsnew + detail documentation.

For now at least, I've put all the details docs into the docstring of the public context-manager.
Note that in documenting callback requirements, I had to make 'FieldCollation' a public class.

More work wanted before I'm ready with this...
DO NOT MERGE

@pp-mo pp-mo force-pushed the structured_load_api_whatsnew branch from 19e38d6 to 73b290b Compare November 10, 2016 10:24
@@ -0,0 +1,10 @@
* Support for "fast" loading of UM files has been added.
This can dramatically accelerate loading speeds of fieldsfiles and PP files,
if they meet certain common structural requirements.
Copy link

@marqh marqh Nov 11, 2016

Choose a reason for hiding this comment

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

I wonder whether a little more on the common structural requirement would help here.

Perhaps something along the lines of

common structural requirements.  The expected structure is the one that Fields Files conform to
 as they are written out by the Met Office Unified model, where fields are ordered by vertical level, 
then by time, and finally by STASH code, within the file

I also think a glib statement about results being unpredictable in cases where this structure does not exist within the file would be useful

Copy link
Owner Author

Choose a reason for hiding this comment

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

You're right, we really do need a specific explanation somewhere that "normal" UM outputs will definitely meet the "structured criteria".

However, my key goal right here was to keep this as short as possible,
as I don't think anything lengthy belongs in the initial 'whatsnew' list.
so I'm thinking we should link to the main docs for all the details.

However, it's just struck me that it would be very natural to have a discussion of this as a "featured item" at the bottom of the whatsnew page, as we have done before.
So, that would be a good home for some introductory non-reference-documentation-like notes and explanations.
I'm not quite sure how we can prototype that approach, but it can be stuck in this file for now
-- the contributions files are really only source material for someone finalising a release, after all.
Does that sound appropriate to you @marqh ?
I think that approach should let us to incorporate all your suggestions here so far (for more detail in the whatsnew) without breaking up the flow.
I'll have a go ...

#
# Also in our case, we need to apply the basic single-field filtering
# operation that speeds up phenomenon selection.
# operation which speeds up phenomenon selection.
Copy link

Choose a reason for hiding this comment

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

s/which/, which

'structured' UM files, providing much faster load times.
"Structured" loading is a streamlined, fast load operation, to be used
**only** on fieldsfiles or PP files whose fields repeat regularly over
the same vertical levels and times (see full details below).
Copy link

Choose a reason for hiding this comment

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

perhaps useful working to repeat in the what's new entry

vertical levels and time points found in the file.
When a user callback function is used with structured-loading, it is
called in a somewhat different way than in a 'normal' load :
The callback is called once for each basic *structured* cube loaded,
Copy link

Choose a reason for hiding this comment

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

should the need to alter encoding of call back functions be explicitly referenced in the what's new?

@pp-mo pp-mo force-pushed the structured_load_api_whatsnew branch from 73b290b to c97a57d Compare November 14, 2016 14:59
@pp-mo
Copy link
Owner Author

pp-mo commented Nov 14, 2016

Rebased.
Hopefully addressed the existing comments ?

@pp-mo
Copy link
Owner Author

pp-mo commented Nov 14, 2016

I'm going to merge this back onto the main branch (cf #27), and make a full Iris PR out of it ...
Everything else there is already 'agreed', at least up to a point, but it isn't acceptable without the extra documentation included.
So I think we can happily discuss what remains to be done to improve the docs, within the context of a single PR.

@pp-mo pp-mo merged commit c641653 into structured_load_api Nov 14, 2016
@pp-mo pp-mo mentioned this pull request Nov 14, 2016
@pp-mo pp-mo deleted the structured_load_api_whatsnew branch January 3, 2017 13:13
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