Closed ashoka1234 closed 5 years ago
That was a nice one liner re.findall. Thanks Cheers UNOFFICIAL Classified by ashoka.jayawardena@ga.gov.au on 19/12/2018 11:52:42 AM From: Damien Ayers notifications@github.com Sent: Wednesday, 19 December 2018 11:39 AM To: GeoscienceAustralia/COG-Conversion COG-Conversion@noreply.github.com Cc: Jayawardena Ashoka Ashoka.Jayawardena@ga.gov.au; Author author@noreply.github.com Subject: Re: [GeoscienceAustralia/COG-Conversion] Cog auto ashoka (#14)
@omad approved this pull request.
Looks great Ashoka.
I've made a few little tidy ups, see my last commit for details.
Let me know if you'd like to discuss anything, or if you'd prefer that I left PR review comments instead of just changing the code!
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/GeoscienceAustralia/COG-Conversion/pull/14#pullrequestreview-186345846, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHhwa7rWL-VqJqvscL4sWVqpeEK9fktFks5u6YqUgaJpZM4ZXj5T.
Request for this pull request
Work list generation for products
Proposed solution
aws_products_config.yaml
defines theprefix
andname_template
for each product. Theprefix
is a top-level fixed part of the product file keys whilename_template
defines variable part to be used for each dataset. File extension and band names are to be appended to the combination ofprefix
andname_template
in order to compute file names corresponding to datasets.work_list.py
does the following: 1) Extract datasets from datacube: It uses productname_template
to identifyfield_names
required for datacube query. 2) Construct/fill the correspondingname_template
using the information supplied by the datacube query and generate a list of expected.yaml
files. 3) FilterAWS inventory list
to extract existing.yaml
files corresponding to datasets for the given product. 4) Subtract (3) from (2) and write the list of files tofile_list
inoutput_dir
specifiedproduction inventory lists
andproduction database
. More targeted tests are yet to do.