Re: [BangPypers] Fwd: Need a help on reviewing a python code. Message-ID: cacimrglh7bharoas710ofojd8g2i_liynbywka1640dvk5r...@mail.gmail.com

2015-07-17 Thread Senthil Kumaran
Very minor comments. 1. planet.py: from math import pi as PI Unless there is a name conflict with the identifier being used. You could avoid using as in the import. Here I guess, you wanted constants in capitals (perhaps a linter tool required). In that case, you could write it as import math;

[BangPypers] Fwd: Need a help on reviewing a python code. Message-ID: cacimrglh7bharoas710ofojd8g2i_liynbywka1640dvk5r...@mail.gmail.com

2015-07-17 Thread Ramesh Rajagopal
With regards to the below thread-id, I have posted my code into github account. Please have a look at and review it. https://github.com/rameshrajagopal/alignment Regards, Ramesh Message: 1 Date: Fri, 17 Jul 2015 01:13:13 +0530 From: Gora Mohanty g...@mimirtech.com To: Bangalore Python Users