Re: [Freeipa-devel] [PATCH 0087] Pylint: enable parallelism

2016-03-07 Thread Tomas Babej 


On 03/07/2016 10:58 AM, Petr Spacek wrote:
> On 4.3.2016 14:13, Tomas Babej wrote:
>> On 03/01/2016 03:46 PM, Petr Spacek wrote:
>>> Hello,
>>>
>>> Pylint: enable parallelism
>>>
>>> The config file specifies 8 cores but Pylint very quickly
>>> ends up with 3 cores so do not worry about overwhelming your system.
>>
>> I like the idea of the patch, however, on my single-CPU VM this causes
>> additional overhead and lint ends up taking more time (+15%).
>>
>> >From the pylint docs [1]:
>>
>>  If the provided number is 0 then the number of CPUs will be used.
>>
>> I'd suggest we use this value instead, to get the best of both worlds :)
> 
> Here you go.
> 
> Petr^2 Spacek
> 
> 
>> Tomas
>>
>> [1] https://docs.pylint.org/run.html
>>
>>

Thanks, ACK.

Pushed to master: 42c01eb3270d8c47c41f7f9b0da7064edb4b0e47

Tomas

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH 0087] Pylint: enable parallelism

2016-03-07 Thread Petr Spacek 
On 4.3.2016 14:13, Tomas Babej wrote:
> On 03/01/2016 03:46 PM, Petr Spacek wrote:
>> Hello,
>>
>> Pylint: enable parallelism
>>
>> The config file specifies 8 cores but Pylint very quickly
>> ends up with 3 cores so do not worry about overwhelming your system.
> 
> I like the idea of the patch, however, on my single-CPU VM this causes
> additional overhead and lint ends up taking more time (+15%).
> 
>>From the pylint docs [1]:
> 
>  If the provided number is 0 then the number of CPUs will be used.
> 
> I'd suggest we use this value instead, to get the best of both worlds :)

Here you go.

Petr^2 Spacek


> Tomas
> 
> [1] https://docs.pylint.org/run.html
From 700579ca97956e85c7501a96003627d301f5ebb8 Mon Sep 17 00:00:00 2001
From: Petr Spacek 
Date: Tue, 1 Mar 2016 15:42:48 +0100
Subject: [PATCH] Pylint: enable parallelism

The config file specifies 8 cores but Pylint very quickly
ends up with 3 cores so do not worry about overwhelming your system.
---
 pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pylintrc b/pylintrc
index 1003e49217ec9761ea5e17d3e965376a0cca1a32..2a4dd7b09ef8b7ab052fc12e382c88a7af2ed568 100644
--- a/pylintrc
+++ b/pylintrc
@@ -7,7 +7,7 @@ persistent=no
 load-plugins=pylint_plugins
 
 # Use multiple processes to speed up Pylint.
-jobs=1
+jobs=0
 
 [MESSAGES CONTROL]
 
-- 
2.5.0

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Re: [Freeipa-devel] [PATCH 0087] Pylint: enable parallelism

2016-03-04 Thread Tomas Babej 
On 03/01/2016 03:46 PM, Petr Spacek wrote:
> Hello,
> 
> Pylint: enable parallelism
> 
> The config file specifies 8 cores but Pylint very quickly
> ends up with 3 cores so do not worry about overwhelming your system.

I like the idea of the patch, however, on my single-CPU VM this causes
additional overhead and lint ends up taking more time (+15%).

>From the pylint docs [1]:

 If the provided number is 0 then the number of CPUs will be used.

I'd suggest we use this value instead, to get the best of both worlds :)

Tomas

[1] https://docs.pylint.org/run.html

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

[Freeipa-devel] [PATCH 0087] Pylint: enable parallelism

2016-03-01 Thread Petr Spacek 
Hello,

Pylint: enable parallelism

The config file specifies 8 cores but Pylint very quickly
ends up with 3 cores so do not worry about overwhelming your system.

-- 
Petr^2 Spacek
From cf0994b03016ca25ad1774f2b49266eb9abd5acb Mon Sep 17 00:00:00 2001
From: Petr Spacek 
Date: Tue, 1 Mar 2016 15:42:48 +0100
Subject: [PATCH] Pylint: enable parallelism

The config file specifies 8 cores but Pylint very quickly
ends up with 3 cores so do not worry about overwhelming your system.
---
 pylintrc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pylintrc b/pylintrc
index 1003e49217ec9761ea5e17d3e965376a0cca1a32..6717583690d2c05a5fdc61de003b1bc165538c82 100644
--- a/pylintrc
+++ b/pylintrc
@@ -7,7 +7,7 @@ persistent=no
 load-plugins=pylint_plugins
 
 # Use multiple processes to speed up Pylint.
-jobs=1
+jobs=8
 
 [MESSAGES CONTROL]
 
-- 
2.5.0

-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code