Extended filenames as dict

        # Append filename extension to filenames
        if ext_fname:
            if not isinstance(ext_fname, (dict, str)):
                raise ValueError("Extended filename must be a str or a dict")
            if isinstance(ext_fname, str):
                splits = [pair.split("=") for pair in ext_fname.split("&")]
                ext_fname = dict(split for split in splits if len(split) == 2)

            logger.debug("%s: extended filename for all outputs:", self.name)
            for key, ext in ext_fname.items():
                logger.debug("%s: %s", key, ext)
             ext_fname_str = "&".join([
                 f"{key}={value}" for key, value in ext_fname.items()
             ])

            for key, filepath in kwargs.items():
                if self._out_param_types[key] == otb.ParameterType_OutputImage:
                    if "?&" in filepath:
                        logger.debug("Extended filename already provided for parameter %s", key)
                    else:
                        parameters[key] = f"{filepath}?&{ext_fname_str}"

added 1 commit

0c2bc219 - ENH: ext_fn tests with 2 outputs

Compare with previous version

added 1 commit

1e68f1e0 - ENH: ext_fn tests with 2 outputs

Compare with previous version

resolved all threads

added 1 commit

68e7fdb7 - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

resolved all threads

added 1 commit

5f837bef - Apply 1 suggestion(s) to 1 file(s)

Compare with previous version

approved this merge request

I think you should squash this kind of MR like !85 (merged) to avoid having commit history with duplicated messages, or things like "Apply 1 suggestion(s) to 1 file(s)" in the main branch

merged

mentioned in commit c4fe5746

Extended filenames as dict

Merged by Rémi Cresson 1 year ago (Jun 28, 2023 1:34pm UTC) 1 year ago

Activity

Extended filenames as dict

Merge request reports

Merged by Rémi Cresson 1 year ago (Jun 28, 2023 1:34pm UTC) 1 year ago

Activity