Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: Pavlos Parissis <pavlos.parissis@gmail.com>
Newsgroups: comp.lang.python
Subject: list slice and generators
Date: Tue, 24 Nov 2015 22:38:22 +0100
Lines: 72
Message-ID: <mailman.50.1448431814.20593.python-list@python.org>
Mime-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="bjEfDi2Lv6fo1KgvlgmH80UHfkgbdSWs3"
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.8.0
Precedence: list
Xref: csiph.com comp.lang.python:99413

This is an OpenPGP/MIME signed message (RFC 4880 and 3156)
--bjEfDi2Lv6fo1KgvlgmH80UHfkgbdSWs3
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable

Hi,

Do you see any possible dangerous hidden bug in the below code(using
python2.7 and python3.4)?

My goal is to avoid go through the metrics list twice. But, I don't
know if there will be a problem with doing in place replace of list
elements using 2 generators.

# metrics =3D ['', '0', '10'....]
metrics =3D [x.metric(name) for x in self._server_per_proc]
metrics[:] =3D (converter(x) for x in metrics)
metrics[:] =3D (x for x in metrics if x is not None)

return calculate(name, metrics)


def calculate(name, metrics):
     if not metrics:
        return None

    if name in METRICS_SUM:
        return sum(metrics)
    elif name in METRICS_AVG:
        return int(sum(metrics)/len(metrics))
    else:
        raise ValueError("Unknown type of calculation for {}".format(name=
))


def converter(value):
    try:
        return int(float(value))
    except ValueError:
        return value.strip() or None
    except TypeError:
        return None


Cheers,
Pavlos


--bjEfDi2Lv6fo1KgvlgmH80UHfkgbdSWs3
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: OpenPGP digital signature
Content-Disposition: attachment; filename="signature.asc"

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAEBCAAGBQJWVNjOAAoJEIP8ktofcXa5L0AQAIQIp9/xXZEc9v8omNH94rCL
dSq1wE7qItFMSLivRgCNnj0201WRTG/pyVPvYgqZA8gh6TBDEaOKD0IhtCyEU56t
qnqnESGylUZh6D1F0pQgN6Dvm3XZGwCygVICks64oXRpm30+m4gmx494TBXLm1ln
8KvYRO9MNvaQzpD0nn66pwaQhsulv1WVGMpQZJUzgSkcj7JBa+1xz1ddf2sfGjwV
xpj4sKr06pKyR/uxfbkelNbYCr5WM7H/K9MLim/rld6Twd/QW1KCsUz+QGLTa3+P
TSYB3YE5k4LKTlGSC84dT+twHQq0wPVxMAPsGwADoVOC2KwJCYeFh+v1A28SJJZw
8P8yW2sZbNSMHU7ngmVWFsWpUYZvg/uuzC0HRJPWCpiJ9fc9SK/aUIAaee/X9AlV
hsobCC6HDB1RWC+kBVY0ROCZT2uvUFhB2vETLSn2HOzTcZcST/1JQPmuEooHJNTZ
YQfJCqScFkxrjnHiJJ5eWYFVJpJFJmKjimF2Yu2dznLplF8LHTizHnd1GcZh+ESL
bcew9zV/cjGYVF/gvefgFjPMWpXOnI7YXbLGVA+LP4Hik2N78fJhflkmCIQHS7pW
e54yCeN800Yq2fL/1JuEsJUD7XQGf/taMrdOOSumfJlCbxPEVZ9flaboQi0JXfx9
FoOBdf/rlId+BfPp9ufn
=teLb
-----END PGP SIGNATURE-----

--bjEfDi2Lv6fo1KgvlgmH80UHfkgbdSWs3--