Aggregator does not unpack .zip results correctly.

[Jammy2211]

Lets say I have results at:

/workspace/output/pipeline_name/pipeline_tag/phase_name/phase_tag/something.zip

This should unpack the results at

/workspace/output/pipeline_name/pipeline_tag/phase_name/phase_tag/

Instead it unpacks them at

/workspace/output/pipeline_name/pipeline_tag/phase_name/

Duplicating results.

We need the phase tag too be included in the unpack.

[Jammy2211]

The problem was that the phase tag was used for the .zip file name, but not used when unpacking the .zip:

        for root, _, filenames in os.walk(directory):
            for filename in filenames:
                if filename.endswith(".zip"):
                    with zipfile.ZipFile(f"{root}/{filename}", "r") as f:
                        f.extractall(root)

I have fixed this by inserting the filename:

        for root, _, filenames in os.walk(directory):
            for filename in filenames:
                if filename.endswith(".zip"):
                    with zipfile.ZipFile(f"{root}/{filename}", "r") as f:
                        f.extractall(f"{root}/{filename[:-4]}")