iris-hep / idap-200gbps-atlas

benchmarking throughput with PHYSLITE
6 stars 1 forks source link

`OutputFormat` and `Delivery` enums do not make logical sense #139

Open gordonwatts opened 2 months ago

gordonwatts commented 2 months ago

In the following code:

    spec = sx.ServiceXSpec(
        General=sx.General(
            ServiceX="atlasr22",
            Codegen=query[1],
            OutputFormat=sx.ResultFormat.root,  # type: ignore
            Delivery=("LocalCache" if download else "SignedURLs"),  # type: ignore
        ),

The OutputFormat and Delivery can be enums. However, they need to be "normalized":

It would be good if all of these enum's could be normalized, and importing them could be done in a straight forward way (currently deliver is at the top level import in servicex, but these enums are very deep)