iiitv / ChefLib

CodeChef Rated Contest Solutions
https://iiitv.github.io/ChefLib/
62 stars 62 forks source link

EXPTREE (JULY17) [PYTHON] #191

Closed sirjan13 closed 6 years ago

sirjan13 commented 6 years ago

Fixes #186

aashutoshrathi commented 6 years ago

Hi! Thanks for PR. Make sure you have checked points in PR Template correctly.

aashutoshrathi commented 6 years ago

Comment on 30700639812adfd6031c0b9a0322112eed019238, file 2017/JULY/JULY17/EXPTREE/EXPTREE.py, line 28.

Line contains following spacing inconsistencies:

Origin: SpaceConsistencyBear, Section: python spacing.

The issue can be fixed by applying the following patch:

--- a/tmp/tmprcdfe371/2017/JULY/JULY17/EXPTREE/EXPTREE.py
+++ b/tmp/tmprcdfe371/2017/JULY/JULY17/EXPTREE/EXPTREE.py
@@ -25,7 +25,7 @@
     ans = (g[1] % m + m) % m
     return ans

- 
+
 mod1 = 10**9+7
 mod2 = mod1 + 2
 for cppp in range(int(raw_input())):
aashutoshrathi commented 6 years ago

Comment on 30700639812adfd6031c0b9a0322112eed019238, file 2017/JULY/JULY17/EXPTREE/EXPTREE.py, line 28.

The code does not comply to PEP8.

Origin: PEP8Bear, Section: pep8.

The issue can be fixed by applying the following patch:

--- a/tmp/tmprcdfe371/2017/JULY/JULY17/EXPTREE/EXPTREE.py
+++ b/tmp/tmprcdfe371/2017/JULY/JULY17/EXPTREE/EXPTREE.py
@@ -25,7 +25,7 @@
     ans = (g[1] % m + m) % m
     return ans

- 
+
 mod1 = 10**9+7
 mod2 = mod1 + 2
 for cppp in range(int(raw_input())):
Monal5031 commented 6 years ago

@sirjan13 There are a lot of pep8 issues, try testing the python code locally using pep8 it reduces extra work. 😉

sirjan13 commented 6 years ago

True just realised this will take care next time

sirjan13 commented 6 years ago

now that i have worked on 2 issues can I work on more or its restricted now?

Monal5031 commented 6 years ago

@sirjan13 You cannot work on more than 2 issues at a time, there is no restriction over total issues. Feel free to open more issues.