Opened 2 years ago

Closed 23 months ago

#934 closed enhancement (fixed)

wcst_import improvements

Reported by: dmisev Owned by: mdumitru
Priority: major Milestone: Future
Component: applications Version: development
Keywords: Cc: mdumitru, vmerticariu
Complexity: Medium

Description (last modified by dmisev)

This ticket collects suggestions for enhancements in wcst_import.

  1. automatic registration of recipes in a certain directory
    • recipe directory could be perhaps managed by an environment variable, or some python friendlier way if it exists
  2. support relative paths in ingredient files, so that they can be portable and easy to distribute along with the data
  3. support different time formats, AnsiDate, UnixTime, etc.
  4. allow setting null values

Change History (9)

comment:1 Changed 2 years ago by dmisev

  • Description modified (diff)

comment:2 Changed 2 years ago by dmisev

  • Description modified (diff)

comment:3 Changed 2 years ago by mdumitru

  • Cc vmerticariu added

1 & 2 have been fixed and commitred.

3 & 4 to go.
For 4 we will need Vlad's help in WCST so ccig

comment:4 Changed 2 years ago by dmisev

For 3 we can just parse the CRS code, it's enough if we support unix time in addition.

comment:5 Changed 2 years ago by dmisev

Submitted patch for 3.

I'm guessing 4 can be done by parsing the nilValues in the gml file passed to WCS-T?

comment:6 Changed 2 years ago by mdumitru

Did you try the patch with the current petascope? I'm not sure if petascope correctly parses non-ansi datetimes from the GML.
I'll try to setup #941 towards the end of the week so that we have a way of automated testing for this.

For issue 4, Vlad will include it in the patch tomorrow, the information is already present in the GML generated by the wcst_import

comment:7 Changed 2 years ago by dmisev

Yes unix timestamps work fine with petascope, but they have to be without quotes.

But is there an option in wcst_import already for setting the null values?

comment:8 Changed 2 years ago by mdumitru

Right now, it reads them from the gdal file, usually they have one there. I'll add an option in the config section for default null value to accommodate such use cases.

comment:9 Changed 23 months ago by dmisev

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.