It was, but the behavior occurs when using mutable types (like a dict) as
default values.

From
http://effbot.org/pyfaq/why-are-default-values-shared-between-objects.htm:

"Because of this feature, it is good programming practice to not use
mutable objects as default values. Instead, use
*None*<http://effbot.org/pyref/None.htm> as
the default value and inside the function, check if the parameter is
*None*<http://effbot.org/pyref/None.htm> and
create a new list/dictionary/whatever if it is."



On Wed, Oct 9, 2013 at 5:21 PM, Guillermo Carrasco <notifications@github.com
> wrote:

> In scilifelab/report/gdocs_report.py:
>
> > @@ -102,7 +102,7 @@ def upload_to_gdocs(fcdir, credentials_file=None, gdocs_folder=None):
> >          write_flowcell_metrics(project_samples, ssheet, wsheet_name)
> >
> >          # Create the summary over all worksheets in the project
> > -        summary_samples = summarize_project(ssheet,{})
> > +        summary_samples = summarize_project(ssheet)
>
> @b97pla <https://github.com/b97pla> wasn't this a problem with previous
> calls with default parameters?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/SciLifeLab/scilifelab/pull/307/files#r6859033>
> .
>



-- 
--------------------------------------------------------
Pontus Larsson, PhD
Science For Life Laboratory
www.scilifelab.se

Email: Pontus.Larsson@scilifelab.se
Phone: +46 8 5248 1440
Mobile: +46 76 946 9788
--------------------------------------------------------