ACCESS-NRI / accessdev-Trac-archive

Archive accessdev Trac contents as issues
Apache License 2.0
0 stars 0 forks source link

Update "ScpBufrToRaijin" to allow addition of Bufr files to tarballs #306

Closed penguian closed 7 years ago

penguian commented 7 years ago

keyword_APS3_Bufr_remote_copy_sam_MARS resolution_fixed | by jtl548@nci.org.au


Currently if Bufr files for a new observation type are to be included in tarballs the tarballs need to be recreated. It would be convenient to simply add new Bufr files to existing tarballs. This ticket describes the work for adding this functionality to the package as well as some minor improvements.


Issue migrated from trac:306 at 2024-01-31 18:27:51 +1100

penguian commented 7 years ago

@jin.lee@bom.gov.au commented


My development branch is,

https://access-svn.nci.org.au/svn/nwp/branches/dev/jtl548/da/utilities/odb/scripts/r597_ScpBufrToRaijin_added_option

penguian commented 7 years ago

@jin.lee@bom.gov.au changed status from new to assigned

penguian commented 7 years ago

@jin.lee@bom.gov.au set owner to cit548

penguian commented 7 years ago

@jin.lee@bom.gov.au commented


Hi Chris,

Following changes were made to the package,

There is still an aspect of the package that needs to be improved but was not done in this ticket: when one or more Bufr tarballs are to be concatenated the ECMA archive from the relevant cycle is retrieved from sam regardless of whether the additional Bufr files come from MARS or not. Due to time constraint this defect is not tackled in this ticket. However this should be revisited later.

If you can review the code changes I would be grateful.

Cheers,

Jin

penguian commented 7 years ago

cit548@nci.org.au commented


Hi Jin,

I've reviewed the revised script and tested the new functionality that allows concatenation of a Bufr tarball to a pre-existing archive on Raijin. The concatenation works properly.

However, there's still a problem with the specification of RemHostTwo: at the very least, it needs the nci.org.au domain added. Susan has also pointed out that it should be "r-dm", rather than "raijin".

One other suggestion: given that you've added variables to the input file, the short description of its contents in the command line help message is incomplete.

Cheers, Chris.

penguian commented 7 years ago

cit548@nci.org.au changed _comment0 which not transferred by tractive

penguian commented 7 years ago

cit548@nci.org.au changed owner from cit548 to jtl548

penguian commented 7 years ago

@jin.lee@bom.gov.au commented


Replying to ACCESS-NRI/accessdev-Trac-archive#306 (comment:3):

Hi Jin,

I've reviewed the revised script and tested the new functionality that allows concatenation of a Bufr tarball to a pre-existing archive on Raijin. The concatenation works properly.

However, there's still a problem with the specification of RemHostTwo: at the very least, it needs the nci.org.au domain added. Susan has also pointed out that it should be "r-dm", rather than "raijin".

One other suggestion: given that you've added variables to the input file, the short description of its contents in the command line help message is incomplete.

Cheers, Chris.

penguian commented 7 years ago

@jin.lee@bom.gov.au changed status from assigned to closed

penguian commented 7 years ago

@jin.lee@bom.gov.au set resolution to fixed

penguian commented 7 years ago

@jin.lee@bom.gov.au commented


My development branch,

https://access-svn.nci.org.au/svn/nwp/branches/dev/jtl548/da/utilities/odb/scripts/r597_ScpBufrToRaijin_added_option@603

was merged into the "trunk",

https://access-svn.nci.org.au/svn/nwp/da/utilities/odb/scripts/ScpBufrToRaijin@604

The trunk version is installed in the following location on embery,

embery:/data/access/da/utilities/odb/scripts/ScpBufrToRaijin