SEADAS: sentinel3 reader fixes (master) #117
Conversation
knowles
commented
Jan 13, 2023
knowles
commented
- Fixed regular expression in MerisLevel1ProductReader to match the corrected one in MerisLevel1ProductPlugIn.
- Fixed a broken target link in help toc.xml.
…corrected one in MerisLevel1ProductPlugIn 2. Fixed broken target line in toc.xml
| @@ -17,7 +17,7 @@ public MerisLevel1ProductReader(MerisLevel1ProductPlugIn readerPlugIn) { | |||
| @Override | |||
| protected Product readProductNodesImpl() throws IOException { | |||
| final String dirName = getInputFileParentDirectory().getName(); | |||
| if (dirName.matches("EN.*_(F|R)R(G|P|S).*")) { | |||
| if (dirName.matches("EN(1|V).*_ME(R|_)1?_(F|R)R(G|P|S|_)_(1P)?_.*")) { | |||
There was a problem hiding this comment.
I think you have extended the expression to match the zip file names like
EN1_MDSI_MER_RR__1P_20111230T102706_20111230T111042_051430_0123_20180901T181455_0100.ZIP
But actually, this is not the folder name according to the format.
https://earth.esa.int/eogateway/documents/20142/37627/MERIS-Sentinel-3-Like-L1-andL2-PFS.pdf (section 3.2 page 11).
However, probably it does not disturb the usual reading. But it might be good to split both cases and make the expressions more specific to make it clearer. Also a unit level test would be good.
knowles
changed the title
|
|
