some flake8 details in backend normaliz

[fchapoton]

and also fix two lgtm warnings

nothing serious, mostly cosmetic changes

CC: @jplab @LaisRast @kliem @sophiasage

Component: geometry

Author: Frédéric Chapoton

Branch/Commit: 310903a

Reviewer: Jonathan Kliem

Issue created by migration from https://trac.sagemath.org/ticket/30878

[fchapoton]

Branch: u/chapoton/30878

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

310903a

some details in backend pynormaliz

[sagetrac-git]

Commit: 310903a

[kliem]

comment:4

This seems inconsistent to me:

-            h_vertices = [ list(v) + [1] for v in vertices ]
+            h_vertices = [list(v) + [1] for v in vertices]

-                dieq = [ ZZ(d*ieq_i) for ieq_i in ieq ]
+                dieq = [ ZZ(d * ieq_i) for ieq_i in ieq ]

[fchapoton]

comment:5

well, I was tempted to remove (as suggested by flake8) all superfluous whitespace before or after brackets. But then I decide to try to keep the diff small, so I did that only once.

[kliem]

comment:6

I think for i,j in enumerate... is better to read than for i, j in enumerate,
but this might also be because I'm used to it and it seems to be against pep8.

[kliem]

Reviewer: Jonathan Kliem

[vbraun]

Changed branch from u/chapoton/30878 to 310903a