Fix dictionary key error when benchmark plots are generated sequentially

[yantosca]

Name and Institution (Required)

Name: Bob Yantosca
Institution: Harvard + GCST

Confirm you have reviewed the following documentation

• Contributing guidelines

Describe the update

This is the companion PR to #285. This fixes the issue described in #285 by making sure that we append function output to the results variable instead of overwriting it. For example, in code blocks such as this:

    # --------------------------------------------
    # Create the plots in parallel
    # Turn off parallelization if n_job=1
    if n_job != 1:
        results = Parallel(n_jobs=n_job)(
            delayed(createplots)(filecat)
            for _, filecat in enumerate(catdict)
        )
    else:
        for _, filecat in enumerate(catdict):
            results = createplots(filecat)
    # --------------------------------------------

we have updated the else block so that the output from the function call when parallelization is turned off is appended to a list rather than ovewritten:

    else:
        results = []
        for _, filecat in enumerate(catdict):
            results.append(createplots(filecat))

Expected changes

This will prevent benchmark plots from failing when n_cores: 1 is specified in the YAML config file.

Related Github Issue(s)

• Closes Plot parallelization off failing in GCPy 1.4.1 #285

[lizziel]

Good to merge!

[yantosca]

Thanks @lizziel!