If you add it here, you might as well use it in the class instead of the literal undef there.


@jquadrin You will also need those changes. :D


> We've both broken stuff along the way.

I prefer the term "introduced undocumented features" myself.


Ah bah bravo Morray!


Also, everything should be docblocked, with

```php
/**
 *
 */
```

not

```php
/**
 *
 **/
```


I'm guessing this result is deliberate:

WARNING: Warning: Unable to find file: /icons/units/opc2001_icon.dds
WARNING: stack traceback:
WARNING:         c:\github\fa\lua\ui\uiutil.lua(345): in function `UIFile'
WARNING:         ...iance\gamedata\lua.scd\lua\ui\dialogs\createunit.lua(451): in function <...iance\gamedata\lua.scd\lua\ui\dialogs\createunit.lua:442>
WARNING:         ...iance\gamedata\lua.scd\lua\ui\dialogs\createunit.lua(499): in function <...iance\gamedata\lua.scd\lua\ui\dialogs\createunit.lua:497>

But my god it's annoying. What do we do to solve it?


TeamCity Ybackend :: Feodorov/ybackend [Build 75](http://192.168.30.225:8111/viewLog.html?buildId=636&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6; exit code 1 Build time: 00:01:49


LOL, yes! That's in the same spirit as the term [rapid unscheduled disassembly](http://www.theguardian.com/technology/2015/jan/16/elon-musk-falcon-9-rapid-unscheduled-disassembly), which I very much enjoy.


TeamCity Ybackend :: Feodorov/ybackend [Build 76](http://192.168.30.225:8111/viewLog.html?buildId=638&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


That's right, something like

    case "$prev" in
    	--timeout|-t)
    		return
    		;;

should do.


Test the getPath method


Personally I prefer using pick: ```$real_enabled = pick($enabled, getvar("foreman_proxy::${title}"))```


Bon fixed, il faut bien activer l'header.
> This server supports HTTP Strict Transport Security with long duration. Grade set to A+


What does that do?

> On Jan 22, 2015, at 11:12 PM, hakim89 <notifications@github.com> wrote:
>
> When debugging, you can pass the -profile extendedDesktop argument to ADL to enable the NativeProcess functionality
>
> —
> Reply to this email directly or view it on GitHub.
>


See 7269758 :)


用sqlx吧，不然查询的时候for，next，scan这种的好蛋疼，尤其是字段多了之后


TeamCity Ybackend :: Feodorov/ybackend [Build 76](http://192.168.30.225:8111/viewLog.html?buildId=638&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6; exit code 1 Build time: 00:01:50


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 144](http://192.168.30.225:8111/viewLog.html?buildId=639&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Thanks!


@asmecher, these should be the context or journal equivalents.  Let me know if you want me to do this for you.


good point, i'll fix it soon


The margins added cause a regression here:

![selection_001](https://cloud.githubusercontent.com/assets/923011/5875764/a2a987ba-a2de-11e4-9917-573e6b3c7407.png)

Currently overriding with custom CSS, but I'd like to know why this was added?


Sérieusement?


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=36f1053015f845bab60eb73b4b7ff189
 plone-5.0-python-2.7 [SUCCESS]


No need to add .install and .module


m !


Hi,

do we really need this  ?
I thought the setIntervall call to workersCheck would be enough.
Is there some cases where it is not good enough ?


m toi meme !


@Urigo Why did you remove the dependency? It means that ionic can get loaded before angular and that's bad... That happens right now at our project...


@edolstra ouch. Should I revert or is there a simple fix?


In https://github.com/prdn/pigato/blob/10afbef87aa5beaa9022d930a9ab1cd36dbaea38/lib/Client.js#L41L42 , we call 2 times putils.args. One should be enough


And if you really want to remove the dependency than maybe a weak dependency would be better?


`<<version>>`

^^


> This could probably also be in the abstract VolumeDriver ;)
Right :smile:

`dimensions` calls asynchronous. It's when the info dialog was called, when opening the file.
When cache results, I think the usability improves.


Please add comment like
/**
 * Implement hook_install().
 */

Indent code (https://www.drupal.org/coding-standards#indenting)

Refer any contib module from drupal.org (https://www.drupal.org/node/1354)


Oops, that works only for language statistics ...


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 143](http://192.168.30.225:8111/viewLog.html?buildId=637&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6; exit code 1 Build time: 00:11:28


Please use db_select() instead of db_query()


field_purge_batch(500);

why this code used ???


Est-ce qu'il ne serait pas plus logique d'inverser ces deux lignes?
Histoire de n'appeler `getPierre` que lorsque l'on a vraiment besoin de la pierre...


J'ai pas compris. Comme ca ca sera fini rapide non?


Became far more organized.


It should [floor](http://en.wikipedia.org/wiki/Floor_and_ceiling_functions) the doubles passed in, so this is expected behaviour.
An example is that `(-2, 1.9, -5.1)` will go to `(-2, 1, -6)` which is the correct location since anything between -5 and -6 will be -6 for the z axis. Simply casting the double to an int will mean that the position is 1 block out for any negative value.


I was trying to do it but ran into some problems.

 * setFilterExpression is not working properly because it does not update the subsetOfAttributes (i.e. only works when that's not set to something that prevents it from working)
 * the expression filtering code is virtual in the abstractFeatureIterator (as fallback to allow overriding it in the provider iterator)
 * so there should be another virtual "setupExpressionFilter()" method that adds missing fields to the request that can be overridden if a provider implements a native translation
 * adding missing fields in QgsAbstractFeatureIterator is not possible because the expression only has field names, subsetOfAttributes is a list of indices and the mapping from names to indices is missing

I would propose to add knowledge about the fields it operates on to the QgsFeatureIterator, but I'd like to get your opinion about this @wonder-sk


No need to add .install and .module


I suppose https://github.com/matejc/nixui/blob/master/default.nix and https://github.com/matejc/nixui/blob/master/node.nix should be copied into nixpkgs?


Indent code (https://www.drupal.org/coding-standards#indenting)


What if opt.from is something like `src/**/*.tag`? Watching supports globbing - but compiling does not?


Is it really the sorting? It sorts to a named pipe, which does not create a file. Unless you are so low on
memory that it has to resort to merge sort?

If this genomic_context.bed.gz, it is sorted. Re-sorting it should not take much extra time, though it would be nice to turn this off via command line options.

You are right, the output of the statement could be gzipped - or read directly from the stream.


Yes, we can.


我一時間還不太會怎麼在已經提交的Commit里再修改。
下一次更新的時候我會刪除的。多謝提醒。


Hi,

IMHO, it could be relevant to define polish translations file between 'nl' and 'zh' ones:

```javascript
define([
     './locales/de.js',
     './locales/nl.js',
     './locales/pl.js',
     './locales/zh.js'
 ], function() {
     var locales = {};
     for(index in arguments) {
```

because it is better to find alphabeticaly a file in the defined list and it will ease the code merging with further PR addition about translations like mine #191.

What do you think about it ?

Vincent


Ok, fixed. Now this makes a lot more sense, and is thus consistent with the JS (which was a bit of a struggle before).

One of the things that may have contributed to this misunderstanding is that "higher" and "lower" levels terminology.

The *higher* level is commonly understood as that with the *lower* ordinal (i.e. L0 is /higher/ than L1 despite 0 < 1) - but some of the original comments turned it around and spoke of the higher ordinal as the higher level (e.g. in the script during a backwards-iteration, which is even more confusing then :D).

In any case, the required-parameter would not have been needed - and I wouldn't have implemented it, but now it's de facto already there, it's just to change the name and the "and not" into an "or", so backing out is actually more effort ;)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 144](http://192.168.30.225:8111/viewLog.html?buildId=639&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Tests passed: 6; exit code 1 Build time: 00:11:18


Yeah, that's basically the only solution at the moment.


I could probably send in some PR's but I don't want to go down that rabbit hole as I'll send in a million and not get any work done.

But, this would be better:

> Security wise, eval is a minefield. If you are evaluating a string that comes from user input (forms, the query string, etc) then your server and all your data could be accessed or manipulated. Some developers try to sanitize the functions which are allowed in the evaluated PHP, but it is easy to bypass this. Never ever use eval with user inputs.

Or something like that.


does
`#!/usr/bin/env python3` work for you? This should be better according to some sources.


@cpina, can you look into this? It looks like these journals are still active and published by Elsevier, e.g.:

http://www.elsevier.com/journals/acta-astronautica/0094-5765/guide-for-authors#68000
http://www.elsevier.com/journals/accident-analysis-and-prevention/0001-4575/guide-for-authors#68000


At some point we may need to change this to save the data in the tables before dropping.


Is this what you mean?

```
$version = $::foreman_proxy::plugin::abrt::params::version,
```

It's in the classes.  I just saw it was the only unparameterized thing and figured I'd fix it while I was in there.


Not at all. I understand. But I tested this on my site and what I saw? All items are displayed in News section :-) . The better way is to fix by myself.


why are you trying to write to a null file?  The default output goes to results.txt


This method already exists some lines above...


Selon wikipdedia ([jeu de go](http://fr.wikipedia.org/wiki/R%C3%A8gles_du_jeu_de_go#Fin_de_partie)), la partie se termine lorsque les deux joueurs ont passé leur tour.


Yes, we caught this yesterday and just hadn't fixed it yet, but it is 4.1.3

Our next update brings a Gulp build process to the project. This will forever squash any issues with the minified version. I hope to push this update shortly.


D'ailleurs, il y a un bug, il faut remettre à zéro le compte de tours passés lorsque l'on joue.


This is a good change. I don't know if it really makes sense though this way with Zookeeper as a whole. Maybe we should just have an admin port that works like Mesos executor protobuf. We can then have this entire code runnable on mesos. If someone wasn't using mesos then we could have another module that would do this async listen with zk notification and talk to the consumer about what is going on.


reference `this.dialect` instead of this hack


I only try to write to /dev/null if fopen fails trying to open outputName.  Right now we just proceed on even if trace is NULL (fopen fails) which causes a seg fault.


override doTearDown instead. Remove the @After


See:
https://github.com/flavour/eden/blob/b4a32594b415e540c7b24a83cb3436ef72a1326e/modules/s3/s3widgets.py#L4427

Iterating forward through the levels and determining whether the current level is the highest by testing for a lower one instead of a higher one - it should have been "- 1" here not "+ 1".


It was incorrect code at all. It's already removed.


Ahh, I looked for a few minutes to find a decent indicator & then just settled to this.
Will update ofc


AS I understand, you added this to only match spaces that are not escaped by an anti-slash. But (1) look-ahead references are not supported by earlier versions of ruby supported by xcpretty, and (2) I tested it with a path containing spaces and it does not seem to work.

* Please add specs for this case of path containing spaces
* I suggest using this regex instead: `/^CompileC\s.*?\s(.+\/(.+\.(?:m|mm|c|cc|cpp|cxx)))\s.*/`, that is:
  * use `+` instead of `*` if you expect at least one repetition, meaning that your match cannot be empty
  * reduce the greediness of the first repetition operator (`.+?` instead of just `.+`) to allow characters being part of the path to be eaten by the rest of the regex

```
/^CompileC\s.*?\s(.+\/(.+\.(?:m|mm|c|cc|cpp|cxx)))\s.*/.match(SAMPLE)
=> #<MatchData
 "CompileC /Users/musalj/Library/Developer/Xcode/DerivedData/ObjectiveSugar-ayzdhqmmwtqgysdpznmovjlupqjy/Build/Intermediates/Pods.build/Debug-iphonesimulator/Pods-ObjectiveSugar.build/Objects-normal/i386/NSMutableArray+ObjectiveSugar.o /Users/musalj/code/OSS/ObjectiveSugar/My Classes/NSMutableArray+ObjectiveSugar.m normal i386 objective-c com.apple.compilers.llvm.clang.1_0.compiler"
 1:"/Users/musalj/code/OSS/ObjectiveSugar/My Classes/NSMutableArray+ObjectiveSugar.m"
 2:"NSMutableArray+ObjectiveSugar.m">
```

Note that I haven't tested this regex intensively, so it has to be validated that it works for all cases, I may probably have missed some cases.

You may also consider replacing `.+` (in the part of the RegEx matching the path) by something like `(?:\\.|[^ ])+`, to tell it to match "any character other than a space, except if it's a character escaped by an anti-slash".


TESTS FAILED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=52338ac402bd4a799189b5a3e15aa1eb
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


`code-block-with-bs-panel.patch` now has same content as `include-code-with-bs-panel.patch`, there must be an error. Don't forget to regenerate the blog after fixing that so [this](http://kaworu.github.io/octostrap3/setup/octopatch/) page will get correct content.


You mean to the server?


no prob. your comment in that issue reminded me that this needs merging.

https://github.com/tgriesser/knex/blob/master/test/integration/suite.js#L10


w0w.  so s0lid


TeamCity Ylab :: synteny/ylab [Build 38](http://192.168.30.225:8111/viewLog.html?buildId=640&buildTypeId=Ylab_SyntenyYlab&guest=1) is now running


Ok, got it :) The current method does not support native Object constructor right now.
This way:
```javascript
is.object(Object);
```
will return true. Do you have any idea to return false for any ordinary function?


Sure, I'll have a look next week


看body不爽


Well I am wondering what would be the best option. I guess, since we are syncing the data to the server anyways, maybe we don't need to worry about saving it when the table drops. Nevermind.


Maybe, there are already a few links to composer already.


As if there were any project standard ;)


Hello!


You removed string and no one will receive exception object??? At least IDEA must tell you about this.
As we told we have 2 types of Disconnect:
- successful
- with error

I this case you must pass `e` to `handler.onDisconnect();`


蛮过瘾哦


we should not exit if we can't write the results.  We should continue to execute and output to the screen.  A printf about not being able to write the file should be ok, but I wouldn't exit


Was duplicated, see getDefinition method


As a school project, there were other concepts that we were concerned about
besides other people wanting to corrupt, troll, or modify our images.
Should we be worried about you?

On Thu, Jan 22, 2015 at 5:23 PM, Walt Schlender <notifications@github.com>
wrote:

> Did you mean to expose your credentials on github?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BobGu/dinner_dash/commit/cc38d1d8daa8d5958bb173df0454a0f55eab7b9a#commitcomment-9407979>
> .
>


Done.


You're right! I confused with `run` method that is used in controllers.


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=8b46cc8d256c41619cba51f58a2b7384
 plone-5.0-python-2.7 [SUCCESS]


First, if you're going to remove the icon, also remove the template IF/END block.

Second, this icon was meant to identify *local* users that have their password managed by PF (in the lovely table `temporary_password`).

What I would like to see eventually in the users list is an identifier (icon or string) for every type of user we can have:
- created users from PF and sponsored users (`temporary_password` table)
- users activated by email
- users activated by sms
- users activated by oauth
- paying users (billing engine)


Aha I'm being a muppet - the problem is simply the lack of compression - I'll submit a pull request! thanks.


srg


awfdcv


check on null is not needed here: (null instanceof class) == false


@Kflash great job, it seems to be a really interesting service. Need to dig into it more.


I would view any data stored in the database on the device a temporary cache. Lots of things can happen that would cause the data to be lost.

So, with that mentality just dropping the table without attempting to save the data should be fine. The data should be able to be rebuilt from the server


You fixed my mistake. I did not write the code is not better than was after my modification. I used DB_NEWS as key for $contents incorrectly. I just wrote that you can simple initialize the array (with a correct name of course) instead of conditionally unset it. My first mistake was perfectly fixed by you but my second mistake was using a wrong array in a loop after I renamed it where I initialized it.


It's what I would prefer, but I originally had some issues with reddit's text format. I've since solved them, I'll consider using real json.


Can you remove the added extra white spaces after '\t'? It would add a weird indentation to the generated code.


TeamCity Ybackend :: Feodorov/ybackend [Build 77](http://192.168.30.225:8111/viewLog.html?buildId=641&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


TeamCity Ybackend :: Feodorov/ybackend [Build 77](http://192.168.30.225:8111/viewLog.html?buildId=641&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:09


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 145](http://192.168.30.225:8111/viewLog.html?buildId=642&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


In the original code, the first sample was at old gain and the last sample was at new_gain - gain_delta.  now the first sample is old_gain + gain_delta and the last one is at new_gain -- I'm not sure that's really an off-by-one error.  It would be nice to have a unit test that confirms that the ramping code works the way we expect (extremely short buffers of 10 samples, for instance, should be easy to set up and test)


Hot pink theme please!


`text_type` is needed in this context because `normalize` requires a unicode value. `to_string` doesn't modify values if they are already string or unicode so in Python 2 a string will stay a string and not be converted to unicode. If you're going to add `to_string` then it needs to be `text_type(pyd.to_string(text))` which will convert string to unicode in Python 2.

I have considered modifying `to_string` to convert non-unicode strings to unicode though. But I'm not sure whether that could have some unintended side-effects? Do you having any thoughts on this?


you're right, I forgot the IF block. I'll remove it.

Good idea for the icon representing the user. I'd see it also as a separate column (that we could make searchable)


This doesn't support all the possible syntax angularjs supports. Maybe change it to use the attribute.$attr array values instead and just ignore if attribute was defined as just target? Check this article for more details:
http://www.bennadel.com/blog/2648-inspecting-attribute-normalization-within-directives-in-angularjs.htm


yeah good point.  my immediate problem is that I couldn't troubleshoot the
issue with the PG deploy error on heroku because of all the bots creating
log spam.  But I'll write up a ticket to create a better, more permanent
solution

On Thu, Jan 22, 2015 at 10:49 PM, Ben Taitelbaum <notifications@github.com>
wrote:

> This could get messy quickly if we start adding all of the attempts du
> jour here. Is there a gem that already has a list of them that we could
> just add?
>
> I'm also not sure it's even worth rendering a nice 404 page for things
> like this. If we can let the rack middleware handle it to just return an
> empty 404, that would suffice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coshx/coshx/commit/e122d1300031aeb2f4b5dd8e87db1722a9f9afcc#commitcomment-9409935>
> .
>


Should be fixed by next io.js release: https://github.com/iojs/io.js/issues/536


Same as above, look-ahead patterns are not compatible with every version of Ruby, sadly.


Ok. @matejc sorry I marked the package as broken, can you copy those files in a new PR?


# This function creates a special "matrix" object that can cache its inverse
makeCacheMatrix <- function(x = matrix()) {
    m <- NULL
    set <- function(y){
        x <<- y
        m <<- NULL
    }
    get<-function() x
    setmatrix<-function(solve) m<<- solve
    getmatrix<-function() m
    list(set=set, get=get,
         setmatrix=setmatrix,
         getmatrix=getmatrix)
}

#This function computes the inverse of the special "matrix" returned by makeCacheMatrix above.
#If the inverse has already been calculated (and the matrix has not changed),
#then the cachesolve should retrieve the inverse from the cache.
cacheSolve <- function(x=matrix(), ...) {
    m<-x$getmatrix()
    if(!is.null(m)){
        message("getting cached data")
        return(m)
    }
    matrix<-x$get()
    m<-solve(matrix, ...)
    x$setmatrix(m)
    m
}


TeamCity Ylab :: synteny/ylab [Build 38](http://192.168.30.225:8111/viewLog.html?buildId=640&buildTypeId=Ylab_SyntenyYlab&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 129 Build time: 00:12:16


This change isn't required. Snapshotting is enabled by default.


TeamCity Ybackend :: zmactep/ybackend [Build 48](http://192.168.30.225:8111/viewLog.html?buildId=643&buildTypeId=Ybackend_ZmactepYbackend&guest=1) is now running


In the whole readme or just the last instance?


TeamCity Ybackend :: zmactep/ybackend [Build 49](http://192.168.30.225:8111/viewLog.html?buildId=644&buildTypeId=Ybackend_ZmactepYbackend&guest=1) is now running


TeamCity Ybackend :: zmactep/ybackend [Build 48](http://192.168.30.225:8111/viewLog.html?buildId=643&buildTypeId=Ybackend_ZmactepYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:10


'k


Looks good


TeamCity Ybackend :: zmactep/ybackend [Build 49](http://192.168.30.225:8111/viewLog.html?buildId=644&buildTypeId=Ybackend_ZmactepYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:00:09


Ser bra ut


Yeah though I think most links there don't have multiple parameters. I'll grep the thing to make sure.


On uninstall this module got following errors.
Also node/add shows product : this should be removed on uninstall

Please fix these.

Notice: Undefined index: products in node_add() (line 77 of D:\xampp\htdocs\testsites\drupal\testsite1\modules\node\node.pages.inc).
Notice: Trying to get property of non-object in node_add() (line 77 of D:\xampp\htdocs\testsites\drupal\testsite1\modules\node\node.pages.inc).
Notice: Undefined index: products_node_form in drupal_retrieve_form() (line 806 of D:\xampp\htdocs\testsites\drupal\testsite1\includes\form.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'products_node_form' not found or invalid function name in drupal_retrieve_form() (line 841 of D:\xampp\htdocs\testsites\drupal\testsite1\includes\form.inc).


Ended up using `knex.client.dialect` as `this.dialect` is obviously not accessible in nested functions.

(btw: funny thing, I was exploring your blog due to that issue, while you were remembered on this PR)


Au lieu de faire des commentaires corrige les problème sonarqube restants ^^ j'en ai enlevé la moitié déjà =)


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=5a85766bf97e44a78e5c95dc44d8d551
 plone-5.0-python-2.7 [SUCCESS]


@randsonjs I think we can keep the installation instructions.


Thanks! I had forgotten only emcntr is embedded.


Can I help with translaction?


Thank you.


Quel est l'intérêt de `(couleur == Couleur.NOIR ? "0" : "1") )` par rapport à `couleur.toSTring()` ?


咋可以编辑别人的注释哦


Will do. Just thought I would let you know :). It happens to me all the time as well when I'm hand editing these files, but fortunately generate_styles_from_data.rb checks.


I'm curious, what's the warning here? `Not initialized @banner`?


Yes. It gives compiler error otherwise.
It doesn't generate default constructors implicitly because I have defined the other constructor.


## This is a markdown file


能收到吗


Thank you for the double putils.args just fixed.
Multiple workersCheck are needed for the reason that if you try to send 100000 concurrent requests the broker enqueues that setTimeout for the workerksCheck in the event loop after all the I/O.
This causes workers to not receive heartbeats so they disconnect and reconnect.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 145](http://192.168.30.225:8111/viewLog.html?buildId=642&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:24


This unlocks the lock too early which makes `base_gid_` vulnerable to races.


最后评价


看到了哦


OK.


Nie wystarczyłyby ostatnie 2 case'y?


Thanks for clarification

On Fri, Jan 23, 2015 at 3:43 PM, Paolo Ardoino <notifications@github.com>
wrote:

> Thank you for the double putils.args just fixed.
> Multiple workersCheck are needed for the reason that if you try to send
> 100000 concurrent requests the broker enqueues that setTimeout for the
> workerksCheck in the event loop after all the I/O.
> This causes workers to not receive heartbeats so they disconnect and
> reconnect.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/prdn/pigato/commit/10afbef87aa5beaa9022d930a9ab1cd36dbaea38#commitcomment-9416825>
> .
>


You changed the branch alias to 1.1, than tagged 1.0.1?


A nie, ok, widzę.


No such key exists. The correct key is `w00t_no_ungrafted_taxa`.


@kevgathuku Thanks!


Good one. I don't know why we need to get this repo from bloomonkey rather than the Python Package Index. Is it somehow optimized for Cheshire?


It's more for aesthetics than performance :) it's very fast either way
On Jan 23, 2015 6:45 AM, "battaile" <notifications@github.com> wrote:

> Ah my bad, didn't even think about the overhead of that call.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dumbmatter/basketball-gm/commit/e0659c9323ad3341697048ab8fe226e8dcd00781#commitcomment-9414462>
> .
>


We probably shouldn't be using the implemented code to define the expected result in the tests?


Oh darn! Sorry Joscha, this file needs the same fix: ocb_cleartmp_navigation.php


changed to descending sort (and also changed < to > in lines 560 and 561), so that the it works when there are more than 2 clusters. (cluster_ratio = max_cluster_count / second_best_cluster_count).


@nandy-andy - Slash is missing

```
PHP Warning: include(/extensions/wikia/ArticlesAsResourcesArticlesAsResources.i18n.php): failed to open stream: No such file or directory in /includes/LocalisationCache.php on line 461
```


It's ok, it doesn't even work yet.


> The worst that would happen is if one thread thinks a block is APPENDABLE, but it's actually not.

That's if someone deletes the flag explicitly?

> I don't see there being an issue with this if a block is unshared. The array append code already assumes it's thread local based on the type.

Right, that was my thought too.


At this time, the project does not compile or run.  There is no current timeline for
getting it working.

> On Jan 22, 2015, at 9:00 PM, KimMinhoh <notifications@github.com> wrote:
>
> Is it possible to build armv7? ex) iPhone4s , iPad mini
> I have iPhone 4s, iPad mini. It's architecture with armv7.
>
> I want to build this program on iPhone4s. please give me advise.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/ggeorgaklis/FaceTracker_iOS/commit/f2131966235576ffd8924e5921c6d855d482a5de#commitcomment-9408971>.
>


happy to help

On Fri Jan 23 2015 at 2:56:45 PM Ethan <notifications@github.com> wrote:

> Began working on lost form.
>
> Going to work on getting all the information onto the page and
> manageable first, then I'll worry about making it pretty. perhaps
> enlisting the help of @the-hobbes
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/EdgeCaseBerg/col-con-lost-n-found/commit/
> 6bbe709ec3d8e641939bf896ac229d5d1a472348
>


I'm not sure about this part. Could you, @r20, check if it's ok?


@hakanensari only issue here is that this would now require MWS_AUTH_TOKEN on all request except this https://github.com/pmdeazeta/peddler/commit/823296f2f7f which would still succeed even MWSAuthToken parameter is empty. Which means we should not execute `params('MWSAuthToken' => -> { auth_token })` if ENV['MWS_AUTH_TOKEN'] is nil

Also you need to use a separate account to use MWS_AUTH_TOKEN, since you cannot request for a token on your own account


Are you bypassing the LDP client here and hitting Fedora directly?


Is this an attr_accessor from another class?


I'm not sure I follow what's going on here. Is this borrowed from Rails?


Should remove the token parameter from the constructor of RankedVote()?


This part is a bit hard to read. I assume that you just changed the indentation, didn't you?


to check the indentation of consensus_seq_lookup return in get_consensus_seqs_lookup function.
(raises excpetion in case of early return of consensus_seq_lookup)


I was just hooking this repo up to styleci. :)


Ah nice :)


这里生成的sql 语句 也不是初衷了吧，本来我group by uid 只有一个字段做group by 但是这里就会对columStr 做group by了


`G` was returning additional wrong solutions when one of the equation was not polynomial in nature. Hence added an extra check after the final solutions have been calculated.
Also added formatting for `test_issue_8828`.

So now, both F and G return the right solutions without returning any extra solutions.

@smichr please review.


What's the warning here? I failed to reporduce.


Also you don't have to redefine "default:glass" since the "default" mod already has this drawtype.

And when you add the "_detail.png" textures to the tiles table the inventory image will be messed up and that's the reason you'll have to use minetest.inventorycube() for the inventory_image. You'll see what I'm talking about.


I'm seeing a lot of these TODOs for breaking the cache. Could you put in an issue for that so we don't forget about it?


different sample ids and random barcodes are included in the data to make sure that the defaultdict consensus_seq_lookup works as expected - for different random bcs, sample ids, etc.


More indentation changes, I guess?..


I lost my mac build environment in a hard drive crash.  Can someone with a working mac build try moving the line and commit it if it does no harm?


Missing capital "T". I would lower-case the "d" in documentation, though.

Also, we could be more consistent and call it a "ZIP file" in every logged message.


No need for "return 0" any more.


Maybe we could say that this is only needed if using temp credentials.  Thanks for adding this!


hello, i got your repo (fix) but the problem persists:

$ perl vcxproj2cmake.pl aa.vcxproj Release
Not a HASH reference at vcxproj2cmake.pl line 150.


Oh I see. I realised we did that in notifyme too. Makes more sense that way.


We need to put the loop in a `try:` clause and the "rmdir" line in a `finally:` clause, otherwise the temporary directory is not deleted if we raise an exception during the loop.


What the fucking fuck is this?


Hello, This does the inverse. If you empty the string. an empty string is added to your translation. So inheritence from the default inside the module is gone


aka 0.7.7


If `request.authorization` is None, this line and the next one will raise an error.


Sure - fine with me!

Jonny

Sendt fra min iPad

> Den 23. jan. 2015 kl. 13.35 skrev Martin Connell <notifications@github.com>:
>
> Okay, except I was thinking isVirtualPath would be equivalent to your previous isRelativePath, i.e. "NOT isAbsolutePath".
>
> Therefore, isVirtualPath defaults to true, etc.
>
> I'm happy to do the changes if that makes sense.
>
> —
> Reply to this email directly or view it on GitHub.
>


Note that it might be redundant now, but it wasn't in the past. I can't remember the details but I remember it failed.

So, make sure everything builds fine and rebase too


this is fixed in 1.6.0.12, more details:
http://www.prestahome.com/prestashop-tips-tricks/empty-modules-translations-in-prestashop-back-office/


Oh that's nice :+1:


Merged this to upstream 0.11 branch. Thanks.


looks good


Oh yes, I see you already did what I meant and I failed to see it :)


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=a1d6a66c5ebd40659972ab85c886c6a5
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


I think `COV_PATH` is better; we already use `CURL_PATH`.


What is the 3rd coordinate used for? Dimension number?


Also merging this .gitignore changes.


Do we need this as part of the interface? On the rdkafka side of things this is not part of the public api, so I'd have to make this a meaningless "return True" (is that ok?).


Is that file actually written `cov_upload.log`?


No, mi ricordo che la prof ha detto che in questi casi è meglio mettere le dimensioni delle immagini nell'html
E' più immediato per il browser capire le dimensioni, così


...if so, we might need to add it in .gitignore too.


Sorry for not responding sooner. Because of health issues, I've stopped in the middle of a big refactoring in preparation for the next version, and let things in a bad shape.
The main repo is bosonic. Platform and runtime will be closed. bosonic.github.io is the website repo. Elements that currently have their own repo will be grouped in *-elements repos. Expect a website update with the new version in 2-3 weeks.


Hand-rolled JS slide-show? Nice and minimalist!


 [TestMethod]
        public void ElementAt0()
        {
            SinglyLinkedList list = new SinglyLinkedList();
            list.AddLast("foo");
            list.AddLast("bar");
            Assert.AreEqual("foo", list.ElementAt(0));
        }


Is it not possible to make a multi-sig QR.... or is this just a to-do?


Since the CMake change wouldn't happen until 3.2 or 3.3, mind making a Gerrit branch for this? @jcfr


So, the answer to my question about using an instance method as a callback in C++ is a no - or at least not as simply as I can in either C (by calling a function pointer and passing a pointer to the instance) or Python (in which a bound instance method inherently knows which instance it's associated with no matter how it is called).

Looks like there are several ways to achieve essentially the same thing in C++, but I'll need to do some more reading first.


@sgrshah @caseypugh @chhhris @charlietran any API additions, please keep this repo up-to-date.


That merging didn't work out. :)

I am getting Errors since you did that:

WARNING	 Download failed: FileXY.rar | ReCaptcha instance has no attribute 'old'


That's an issue.

We would be fine if `params` discarded keys with nil values, no?


What if `values` is empty?


Please add comment like
/**

Implement hook_install(). */
Indent code (https://www.drupal.org/coding-standards#indenting)

Refer any contib module from drupal.org (https://www.drupal.org/node/1354)


update openrecovery-twrp-2.8.3.0-titan.img and SuperSU-v2.40.zip


is a todo..


I'm sorry, but that begs the question. By what philosophical-legal framework do you gain legitimate ownership of these kittens, and what is its basis and justification? And what epistemological methodology do you use to "know" something to be "yours," according to that framework?


~~commited~~ committed


On uninstall this module got following errors.
Also node/add shows product : this should be removed on uninstall

Please fix errors after uninstall a module


I'd be in favor of dropping the extra "Odoo" wherever possible in the whole readme, so e.g. "local users" rather than "local Odoo users", "database" instead of "local Odoo database", unless you think it would be very confusing in this context. Even "System usernames need to be unique..." seems to work instead of "Odoo usernames...".
It's tempting to reword/simplify those awkward sentences, but it's probably best to avoid too much rewording, if only for the sake of the fuzzy matching later.


I would pass options as object, not as argument (gives you more flexibility)

```coffeescript
new CsvMapping {excludeEmptyStocks: argv.excludeEmptyStocks}
```


:+1: if it works; the logic seems fine


##set the value of the matrix
  ##get the value of the matrix
  ##set the value of the inverse
  ##get the value of the inverse


What do you need this for?


Hmm, still not loading it in.


Is it possible to blacklist some Environments/ManagedPeripherals from this?


##
##set the value of the matrix
  ##get the value of the matrix
  ##set the value of the inverse
  ##get the value of the inverse


@Kelketek Oh snaps, you're totally right. Adding and pushing now.


Yeah I also noticed this change by a raised error. Also this change has resulted in helpers not being available via Rails: ActionController::Base.helpers.cms_block_content. I don't see why though. Is that a deliberate change?

For reference: I ended up including Comfy::CmsHelper in my application controller.


this is programmatic switch control?


## x is the makeCacheMatrix function to initialize the cache
  ## First get the Inverse


[STABLE][2015.01.20] SuperSU v2.45
http://forum.xda-developers.com/showthread.php?t=1538053


التحالف واعتلاء الأمريكية ذلك كل, الدول اتفاق عالمية بحق ٣٠. ٣٠ المحيط الحيلولة لها. جيوب أثره، لمّ أن, عن جهة كانتا إيطاليا مواقعها, وفي وقوعها، الدّفاع أن. لهذه المعاهدات قام مع, بعض المتحدة واستمرت عل. الشمال الوزراء الحيلولة ان جعل, مرجع واستمر الإثنان ومن و, تم الخاسر بالرغم شموليةً الا. الأمور المحيط وقد أم.شدّت تطوير الشتاء، أسر تم, أما أي شمال الهادي. قبل ان اعلان بالفشل وهولندا،. كرسي الشمال الثقيلة عل ذات. اليها الواقعة تلك أم, الا ان بالسيطرة التبرعات. كلّ بـ بولندا، الأمريكية, تونس بتخصيص والكساد بل به،.


ВСЁ ШАМПАНСКОЕ,ВСЁ В АРТСТАЙЛА МИНУС АРТСТАЙЛ,КУРОНИТИ СПИВАЕТ АРТСТАЙЛА, ГОБЛИН РАЗЛИВАЕТ,СПИТЬ НАДО ХОТЬ КОГО-ТО,ОЛИВЬЕ СТАВИТ,ОЙОЙОЙ КАКОЕ ХОРОШЕЕ ОЛИВЬЕ У ТОЛСТОЙ СКАТИНЫ,ПЫТАЕТСЯ ЧТО-ТО СДЕЛАТЬ НЕ СПАИВАЕТ НИКОГО,ФОБОС БЛЮЕТ НИКОГО НЕ ЗАДЕВАЕТ ЗДЕСЬ ЗАКУСКИ ЛОСТА ПЫТАЮТСЯ ЧТО-ТО СДЕЛАТЬ ИХ ТУТЖЕСЪЕДАЮТ.ПУТИН!НА ИТОГИ ГОДА ОПЯТЬ ВСЕ,ВСЕ ЧЕТВЕРО ПОПАДАЮТ НА ИТОГИ,ДЕНДИ УЕЗЖАЕТ НА ТАКСИ,СПИЛИ ЛОСТА, СПИЛИ ФОБОСА,СПИЛИ ГОБЛАКА,МОЖНО ЕХАТЬ В БОЛЬНИЦУ,ЭТО БЛЯТЬ НЕ НГ, ЭТО ПРОСТО ПОШЛИ ОНИ ♥♥♥♥♥♥♥♥♥♥ БЛЯТЬ


##
##  x is the makeCacheMatrix function to initialize the cache
##  First get the Inverse


The test failure is due to this mis-replace.


## If it is not NULL, it means it has been calculated before
  ## we can retrieve it from the cache, so we can RETURN


```
The licence and distribution terms for any publically available version or
derivative of this code cannot be changed.  i.e. this code cannot simply be
copied and put under another distribution licence
[including the GNU Public Licence.]
```


 ## Otherwise, we need to calculate the inverse using SOLVE
  ## We will set it to the cache to store it for future use


`$prefix` should be wrapped in `preg_quote($prefix, '#')`


ack


Not sure if you still wanted this or not...


  ## If it is not NULL, it means it has been calculated before
  ## we can retrieve it from the cache, so we can RETURN


Oh, if you're just doing string matching, use `substr` and `===`. No need to involve regular expressions :)


  ##  x is the makeCacheMatrix function to initialize the cache
  ##  First get the Inverse


Ah, thank you, I was looking for the cause of the error. Wonder how I did that.


Any way to automate?


nice!


Thanks. Yeah, will do.


Yes, exactly this method is used for switch/switch.go utility


Why not use the colors right away? Suppose I want to integrate the chat in a project: I already have a color scheme, an maybe I already decided how the chat should look like:
If I want to change the rooster backround - I find `$rooster_bg`, and then I have to go and modify $color_2, or replace it directly, and I'll have the variable just...sitting around, adding room for confusion.


Herro i am Li Wangchang, i gro up in china wher i wurk 2 gro rice. sry fo bad engrish i no speaky engrish. une dy fata say "u nu gru rices u gu2city n u gu pwy pufesuino doto" su den i gu pwey pwefusuionl doto leik fartr sai n i get rekt bai sum fagt naem dondo. plz nu copy pasterino dis is lif stury


Yes, it can use an interface.


asdasdasda


Yes, `instance variable @banner not initialized`.


Até o momento não identificamos necessidade de ter.

2015-01-23 10:36 GMT-02:00 Eduardo Oliveira <notifications@github.com>:

> Pessoal, o hotsite vai ter back-end?
> Em 23/01/2015 08:07, "vieiragabriel" <notifications@github.com> escreveu:
>
> > Vou ajustar no próximo commit. Faltou css específico, ai ele pegou do
> > "body > *"
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/agile-alliance-brazil/agile-brazil-2015-hotsite/commit/32d02ad43d3064d3356b78243b9040fc3c0124e5#commitcomment-9413216>
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agile-alliance-brazil/agile-brazil-2015-hotsite/commit/32d02ad43d3064d3356b78243b9040fc3c0124e5#commitcomment-9415072>
> .
>

--
Att,

Gabriel Vieira


want to count number of exported inventories.


Why set timestamp here? Maybe it is better so:
```c++
Msg::Msg(int id, MsgType type, const QString &message)
    : id(id)
    , timestamp(QDateTime::currentMSecsSinceEpoch())
    , type(type)
    , message(message)
{
}
```


That should work and i think that's how it should be,  we should not be passing a parameter with an empty value to Amazon. Should we update Jeff for this?


The only thing that was causing me trouble was the last test using SQL, then I followed along as Tom debugged it to get it to work. Need to brush up on SQL since I find it a little odd just writing it out as a string in Ruby.


What would that mean? Where could I use this flexibility?


:) nice

fyi, mocha passes the same context (`this`) to describe and it blocks. So in this case you could always close over the context (`var self = this`) or `bind`. But using the dialect from the `knex` object works better.


That is, the removed key is `w00t_no_unfragted_taxa` https://github.com/inaturalist/inaturalist/blob/1c6770c979904b3b894a930d0fe60383122dcb65/config/locales/eu.yml#L2965 which was incorrect


You've listed 1.3 twice. Did you want 1.4 here, instead?


id**0**? timestamp**0**? ...?


done


Not necessarily. It's the request to the origin service. It might fail during making the request or processing the response.

On January 23, 2015 7:43:38 AM CST, Adrian J George III <notifications@github.com> wrote:
>response?
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/rackerlabs/repose/commit/736c3cd2e3019dfef21d34eecdf12ad373f3230b#commitcomment-9415942

--
Sent from my Android device with K-9 Mail. Please excuse my brevity.


ack


Yea. I'll fix that up.


Blz.
Em 23/01/2015 13:18, "vieiragabriel" <notifications@github.com> escreveu:

> Até o momento não identificamos necessidade de ter.
>
> 2015-01-23 10:36 GMT-02:00 Eduardo Oliveira <notifications@github.com>:
>
> > Pessoal, o hotsite vai ter back-end?
> > Em 23/01/2015 08:07, "vieiragabriel" <notifications@github.com>
> escreveu:
> >
> > > Vou ajustar no próximo commit. Faltou css específico, ai ele pegou do
> > > "body > *"
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/agile-alliance-brazil/agile-brazil-2015-hotsite/commit/32d02ad43d3064d3356b78243b9040fc3c0124e5#commitcomment-9413216>
>
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/agile-alliance-brazil/agile-brazil-2015-hotsite/commit/32d02ad43d3064d3356b78243b9040fc3c0124e5#commitcomment-9415072>
>
> > .
> >
>
>
>
> --
> Att,
>
> Gabriel Vieira
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/agile-alliance-brazil/agile-brazil-2015-hotsite/commit/32d02ad43d3064d3356b78243b9040fc3c0124e5#commitcomment-9417419>
> .
>


What was the problem with the range/xrange solution I suggested?


Wouldn't be an easy task to automate documentation. Not too much effort to manually update this on the occasional API change.


Then simply do `console.log "Export #{_.size(rows)} stock entries."`


@brverbur I have just realized that storing in the session is also vulnerable to all the attacks we have mentioned. The user just needs to access the website *once* and collect the session cookie and the value shown in the form. Because now he has the session and form values, he can still attack. We are just making it slightly harder but it is still an attack easy to automate and perform.

So far then, the only advantage of a secret is to at least allow previous tokens to be expired, but that can also be easily done with a version. Thoughts?


And here you have to filter out null values


Yes, I just write in another method. For a while I removed this.


What about for non continent maps? Like ICC gunships.


> > The worst that would happen is if one thread thinks a block is APPENDABLE, but it's actually not.
>
> That's if someone deletes the flag explicitly?

If someone deletes the flag explicitly, bad things can happen. The flag really should not be publicly available, but it has to be for certain runtime-aware functionality (e.g. Appender). I would say the setting of the APPENDABLE flag should happen ONLY on block allocation, and it should remain that way until the block is freed.

It should be visible to other threads if the block is shared. That's the only time.


Thank you so much for this.


Okay, thank you. I guess it could be like this now, for the sake of consistency with the ETL procedure, but in the future I think we have to seriously consider how to deal with this (either when calculating on the fly or during ETL (although I think pre-calculating the z-scores on the whole set of data is not preferred). I would suggest to not trim the z-scores in the future, and also to do use the mean for z-score calculation (unless there is a valid reason to choose the median).
Also I think probe aggregation should take place *before* z-score calculation. In the case of pre-calculated z-scores during the ETL this is not possible, but that would be possible if you calculate z-scores "on the fly" while running the analysis.


@jcfr Yep, that looks like a good fix to VTK that should get in the main repo. Do you want to push a patch to gerrit that I can review, or should I? A new test or modifying the existing test for vtkColorTransferFunction would also be good.


Correct. It fixes 'mismatched indentation' warnings.


```coffeescript
rows = _.chain(stocks.body.results)
.map (stock) => @_mapStocks(stock)
.filter (stock) -> not _.isNull(stock)
.value()
```


TESTS PASSED
 Mr.Roboto url : http://jenkins.plone.org/roboto/get_info?push=6cb44d8d0eee4a08a0099718d04ec5a7
 plone-5.0-python-2.7 [SUCCESS]


Ohh, I think I figure what you mean... sorry

Commit in https://github.com/antoniopaisfernandes/traccar-web/commit/9a7d758761f311b9873a62dac0c41eec8d86abcd

Better? :)


Merged slider fix to upstream. Also fixed export facts in upstream, but with a different approach (reusing the buildBrowserArgs method trying to be more consistent). Thanks.


Why do I have to filter them? They will be just ignored when creating the csv.


See comments below (you can get rid of `@i` then)


ack


TESTS FAILED
 Mr.roboto url : http://jenkins.plone.org/roboto/get_info?push=828d0c4f3ea44f948370ce1cfc2adb3a
 plone-4.3-python-2.6 [FAILURE]
plone-4.3-python-2.7 [SUCCESS]


I would recommend simplifying ``yaslfree()``:

```c
void
yaslfree (yastr s) {
    if (s) { free(yaslheader(s)); }
}
```

Else, :+1:


Nah, just a holdover from old code. We can lose it.


1: M100 needs more reviewers


1: M100 needs more reviewers
1: M101 signature is missing
1: M190 line is too long (55 > 50)


1: D100 Docstring missing
4: L102 copyright year is outdated, expected 2015 but got 2013
28:1: I100 Imports statements are in the wrong order. from invenio.bibrecord should be before from invenio.search_engine
29:31: E126 continuation line over-indented for hanging indent
32: D103 Docstring missing
59: D103 Docstring missing
65:49: E127 continuation line over-indented for visual indent
78: D400 First line should end with '.', not 'd'
84: D103 Docstring missing
89: D103 Docstring missing
93: D103 Docstring missing
103: D101 Docstring missing
104: D400 First line should end with '.', not 't'
114: D102 Docstring missing
117: D102 Docstring missing
120: D102 Docstring missing
149:5: E303 too many blank lines (2)
149: D102 Docstring missing
152: D102 Docstring missing
159:5: F999 redefinition of unused 'get' from line 152
159: D102 Docstring missing
166: D102 Docstring missing
176: D102 Docstring missing
179: D102 Docstring missing
186: D102 Docstring missing
202: D102 Docstring missing
216: D102 Docstring missing
222: D102 Docstring missing
250: D102 Docstring missing
254: D101 Docstring missing
255: D102 Docstring missing
258: D102 Docstring missing
261: D102 Docstring missing
265: D101 Docstring missing
266: D102 Docstring missing
273: D102 Docstring missing
275:22: E126 continuation line over-indented for hanging indent
277: D102 Docstring missing
279:21: E126 continuation line over-indented for hanging indent
281: D102 Docstring missing
284: D102 Docstring missing
287: D401 First line should be imperative: 'Return', not 'Returns'
287: D400 First line should end with '.', not 'e'
294: D401 First line should be imperative: 'Replace', not 'Replaces'
294: D400 First line should end with '.', not 't'
306: D102 Docstring missing
309: D401 First line should be imperative: 'Return', not 'Returns'
309: D400 First line should end with '.', not 'e'
319: D102 Docstring missing
322: D102 Docstring missing
328: D101 Docstring missing
329: D102 Docstring missing
333: D102 Docstring missing
335:43: E126 continuation line over-indented for hanging indent
337: D102 Docstring missing
340: D102 Docstring missing


1: M100 needs more reviewers
1: M101 signature is missing


1: D205 Blank line missing between one-line summary and description
1: D208 Docstring is over-indented
4: L102 copyright year is outdated, expected 2015 but got 2012
29:1: I100 Imports statements are in the wrong order. from invenio.bibindex_tokenizers.BibIndexMultiFieldTokenizer should be before from invenio.dbquery
33:1: I100 Imports statements are in the wrong order. from invenio.bibindex_engine_utils should be before from invenio.config
34:1: I100 Imports statements are in the wrong order. from invenio.bibfield should be before from invenio.bibindex_engine_utils
54:1: E303 too many blank lines (3)
54: D203 Expected 1 blank line *before* class docstring, found 0
54: D205 Blank line missing between one-line summary and description
54: D208 Docstring is over-indented
64:41: E251 unexpected spaces around keyword / parameter equals
64:43: E251 unexpected spaces around keyword / parameter equals
64:66: E251 unexpected spaces around keyword / parameter equals
64:68: E251 unexpected spaces around keyword / parameter equals
64:94: E251 unexpected spaces around keyword / parameter equals
64:96: E251 unexpected spaces around keyword / parameter equals
64:123: E251 unexpected spaces around keyword / parameter equals
64:125: E251 unexpected spaces around keyword / parameter equals
64: D102 Docstring missing
72:5: E303 too many blank lines (2)
72: D400 First line should end with '.', not 's'
72: D205 Blank line missing between one-line summary and description
100:9: E303 too many blank lines (2)
131: D401 First line should be imperative: 'Tokenize', not 'Tokenizes'
131: D205 Blank line missing between one-line summary and description
147: D102 Docstring missing
150: D102 Docstring missing
153: D102 Docstring missing
156: D102 Docstring missing
159: D102 Docstring missing


1: D290 White spaces around docstring should be trimmed.
1: D400 First line should end with '.', not ')'
4: L102 copyright year is outdated, expected 2015 but got 2013
24:1: I100 Imports statements are in the wrong order. from invenio.bibupload should be before from invenio.crossrefutils
27: D400 First line should end with '.', not 'd'
27: D205 Blank line missing between one-line summary and description
53:17: E128 continuation line under-indented for visual indent


1: D290 White spaces around docstring should be trimmed.
1: D400 First line should end with '.', not 'I'
4: L102 copyright year is outdated, expected 2015 but got 2012
25:1: I100 Imports statements are in the wrong order. import datetime should be before import urllib2
27:1: I100 Imports statements are in the wrong order. from time should be before from xml.dom.minidom
29:1: I101 Imported names are in the wrong order. Should be CFG_CROSSREF_EMAIL, CFG_CROSSREF_PASSWORD, CFG_CROSSREF_USERNAME, CFG_ETCDIR
30:2: E121 continuation line under-indented for hanging indent
31:1: I100 Imports statements are in the wrong order. from invenio.bibconvert_xslt_engine should be before from invenio.config
34:1: I100 Imports statements are in the wrong order. from invenio.jsonutils should be before from invenio.urlutils
43:1: E302 expected 2 blank lines, found 1
43: D203 Expected 1 blank line *before* class docstring, found 0
43: D204 Expected 1 blank line *after* class docstring, found 0
43: D400 First line should end with '.', not 's'
45: D400 First line should end with '.', not 'n'
49: D401 First line should be imperative: 'Return', not 'Returns'
49: D400 First line should end with '.', not 'e'
53:1: E302 expected 2 blank lines, found 1
53: D205 Blank line missing between one-line summary and description
68:52: E222 multiple spaces after operator
85:5: E122 continuation line missing indentation or outdented
87:39: E502 the backslash is redundant between brackets
88:21: E128 continuation line under-indented for visual indent
91:1: E302 expected 2 blank lines, found 1
91: D400 First line should end with '.', not 's'
109:16: E261 at least two spaces before inline comment
110:16: E261 at least two spaces before inline comment
111:16: E261 at least two spaces before inline comment
112:16: E261 at least two spaces before inline comment
113:16: E261 at least two spaces before inline comment
114:16: E261 at least two spaces before inline comment
115:16: E261 at least two spaces before inline comment
116:16: E261 at least two spaces before inline comment
117:16: E261 at least two spaces before inline comment
178: D400 First line should end with '.', not 'h'
178: D200 One-line docstring should not occupy 3 lines
214: D400 First line should end with '.', not '6'
214: D205 Blank line missing between one-line summary and description
239:1: E302 expected 2 blank lines, found 1
239: D200 One-line docstring should not occupy 3 lines


1: D290 White spaces around docstring should be trimmed.
4: L102 copyright year is outdated, expected 2015 but got 2012
28:1: I100 Imports statements are in the wrong order. from operator should be before from pprint
29:1: I100 Imports statements are in the wrong order. import re should be before from operator
38:1: I101 Imported names are in the wrong order. Should be CFG_BASE_URL, CFG_SITE_URL
39:1: I101 Imported names are in the wrong order. Should be AID_ENABLED, BIBAUTHORID_UI_SKIP_ARXIV_STUB_PAGE, MAX_NUM_SHOW_PAPERS, PERSONS_PER_PAGE, PERSON_SEARCH_RESULTS_SHOW_PAPERS_PERSON_LIMIT, VALID_EXPORT_FILTERS
39:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_config should be before from invenio.config
39:1: F401 'PERSON_SEARCH_RESULTS_SHOW_PAPERS_PERSON_LIMIT' imported but unused
39:1: F401 'BIBAUTHORID_UI_SKIP_ARXIV_STUB_PAGE' imported but unused
39:1: F401 'PERSONS_PER_PAGE' imported but unused
40:29: E126 continuation line over-indented for hanging indent
43:1: I101 Imported names are in the wrong order. Should be CFG_INSPIRE_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_SECURE_URL, CFG_SITE_URL
43:1: F999 redefinition of unused 'CFG_SITE_URL' from line 38
45:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_name_utils should be before from invenio.config
46:1: I101 Imported names are in the wrong order. Should be page, pagefooteronly, pageheaderonly
46:1: F401 'pageheaderonly' imported but unused
46:1: F401 'pagefooteronly' imported but unused
47:1: I100 Imports statements are in the wrong order. from invenio.messages should be before from invenio.webpage
49:1: I101 Imported names are in the wrong order. Should be WebInterfaceDirectory, wash_urlargd
50:1: I100 Imports statements are in the wrong order. from invenio.session should be before from invenio.webinterface_handler
51:1: I101 Imported names are in the wrong order. Should be get_canonical_and_alternates_urls, redirect_to_url
51:1: F401 'get_canonical_and_alternates_urls' imported but unused
52:1: I101 Imported names are in the wrong order. Should be collect_user_info, emailUnique, email_valid_p, getUid, get_email_from_username, get_uid_from_email, get_user_preferences, isGuestUser, page_not_authorized, set_user_preferences
52:1: F401 'get_user_preferences' imported but unused
52:1: F401 'set_user_preferences' imported but unused
62:1: I100 Imports statements are in the wrong order. from invenio.access_control_admin should be before from invenio.webuser
62:1: F401 'acc_get_user_roles' imported but unused
65:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_config should be before from invenio.search_engine_utils
67:1: I100 Imports statements are in the wrong order. import invenio.webauthorprofile_interface should be before import invenio.webinterface_handler_config
68:1: I100 Imports statements are in the wrong order. import invenio.bibauthorid_webapi should be before import invenio.webauthorprofile_interface
69:1: I101 Imported names are in the wrong order. Should be get_title_of_arxiv_pubid, get_title_of_doi, is_valid_orcid
69:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_general_utils should be before import invenio.bibauthorid_webapi
69:1: F401 'get_title_of_doi' imported but unused
69:1: F401 'get_title_of_arxiv_pubid' imported but unused
70:1: I101 Imported names are in the wrong order. Should be get_claimed_papers_of_author, get_orcid_id_of_author, get_title_of_paper, get_validated_request_tickets_for_author, update_external_ids_of_authors
70:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_backinterface should be before from invenio.bibauthorid_general_utils
71:13: E126 continuation line over-indented for hanging indent
75:1: I101 Imported names are in the wrong order. Should be author_has_papers, get_person_id_from_canonical_id, get_person_redirect_link, is_valid_canonical_id
75:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_webauthorprofileinterface should be before from invenio.webauthorprofile_orcidutils
78:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_templates should be before from invenio.bibauthorid_webauthorprofileinterface
82:1: I101 Imported names are in the wrong order. Should be field_get_subfield_values, record_get_field_instances, record_get_field_value, record_get_field_values
83:31: E126 continuation line over-indented for hanging indent
88:1: E302 expected 2 blank lines, found 1
88: D203 Expected 1 blank line *before* class docstring, found 0
88: D204 Expected 1 blank line *after* class docstring, found 0
88: D300 Expected """-quotes, got '''-quotes
114: D401 First line should be imperative: 'Thi', not 'This'
114: D400 First line should end with '.', not ':'
114: D205 Blank line missing between one-line summary and description
114: D300 Expected """-quotes, got '''-quotes
121:12: E713 test for membership should be 'not in'
130: D300 Expected """-quotes, got '''-quotes
167: D205 Blank line missing between one-line summary and description
167: D300 Expected """-quotes, got '''-quotes
260:5: E303 too many blank lines (2)
260: D300 Expected """-quotes, got '''-quotes
297:19: E127 continuation line over-indented for visual indent
299:13: E129 visually indented line with same indent as next logical line
313:21: E713 test for membership should be 'not in'
315:5: E129 visually indented line with same indent as next logical line
330: D401 First line should be imperative: 'Generate', not 'Generates'
330: D300 Expected """-quotes, got '''-quotes
359:9: E303 too many blank lines (2)
366:5: E303 too many blank lines (2)
366: D401 First line should be imperative: 'Generate', not 'Generates'
366: D300 Expected """-quotes, got '''-quotes
417:9: E303 too many blank lines (2)
424:5: E303 too many blank lines (2)
424: D401 First line should be imperative: 'Generate', not 'Generates'
424: D400 First line should end with '.', not 'n'
424: D205 Blank line missing between one-line summary and description
424: D300 Expected """-quotes, got '''-quotes
472:9: E303 too many blank lines (2)
479:5: E303 too many blank lines (2)
479: D401 First line should be imperative: 'Generate', not 'Generates'
479: D300 Expected """-quotes, got '''-quotes
504:9: E303 too many blank lines (2)
511:5: E303 too many blank lines (2)
511: D401 First line should be imperative: 'Generate', not 'Generates'
511: D300 Expected """-quotes, got '''-quotes
595:29: E711 comparison to None should be 'if cond is None:'
640:9: E303 too many blank lines (2)
647:5: E303 too many blank lines (2)
647: D401 First line should be imperative: 'Generate', not 'Generates'
647: D300 Expected """-quotes, got '''-quotes
673:5: E303 too many blank lines (2)
673: D400 First line should end with '.', not 'f'
673: D205 Blank line missing between one-line summary and description
673: D300 Expected """-quotes, got '''-quotes
698:73: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
706:77: E712 comparison to True should be 'if cond is True:' or 'if cond:'
707:71: E221 multiple spaces before operator
710:18: E713 test for membership should be 'not in'
712:55: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
715:69: E221 multiple spaces before operator
718:55: E712 comparison to False should be 'if cond is False:' or 'if not cond:'
727:5: E303 too many blank lines (2)
728: D401 First line should be imperative: 'Determine', not 'Determines'
728: D400 First line should end with '.', not 'n'
728: D300 Expected """-quotes, got '''-quotes
756:36: F821 undefined name 'acc_find_user_role_actions'
765:5: E303 too many blank lines (2)
766: D401 First line should be imperative: 'Return', not 'Returns'
766: D205 Blank line missing between one-line summary and description
766: D300 Expected """-quotes, got '''-quotes
777:5: E303 too many blank lines (2)
815:21: E129 visually indented line with same indent as next logical line
829:5: E303 too many blank lines (2)
829: D205 Blank line missing between one-line summary and description
829: D300 Expected """-quotes, got '''-quotes
867:53: E231 missing whitespace after ':'
875:47: E231 missing whitespace after ':'
908:9: F841 local variable 'ulevel' is assigned to but never used
952:14: E127 continuation line over-indented for visual indent
953:14: E127 continuation line over-indented for visual indent
959: D401 First line should be imperative: 'associate', not 'associates'
959: D400 First line should end with '.', not 'd'
959: D300 Expected """-quotes, got '''-quotes
959: D200 One-line docstring should not occupy 3 lines
967:29: E128 continuation line under-indented for visual indent
973:29: E128 continuation line under-indented for visual indent
979:29: E128 continuation line under-indented for visual indent
986: D401 First line should be imperative: 'associate', not 'associates'
986: D400 First line should end with '.', not 'd'
986: D300 Expected """-quotes, got '''-quotes
986: D200 One-line docstring should not occupy 3 lines
994:29: E128 continuation line under-indented for visual indent
1000:29: E128 continuation line under-indented for visual indent
1020:29: E128 continuation line under-indented for visual indent
1026: D401 First line should be imperative: 'associate', not 'associates'
1026: D400 First line should end with '.', not 'd'
1026: D300 Expected """-quotes, got '''-quotes
1026: D200 One-line docstring should not occupy 3 lines
1034:25: E128 continuation line under-indented for visual indent
1050:22: E225 missing whitespace around operator
1060:18: E127 continuation line over-indented for visual indent
1064:22: E127 continuation line over-indented for visual indent
1068:22: E127 continuation line over-indented for visual indent
1093:46: E127 continuation line over-indented for visual indent
1102: D401 First line should be imperative: 'emptie', not 'empties'
1102: D400 First line should end with '.', not 'e'
1102: D205 Blank line missing between one-line summary and description
1102: D300 Expected """-quotes, got '''-quotes
1130:29: E128 continuation line under-indented for visual indent
1221:29: E128 continuation line under-indented for visual indent
1226:29: E128 continuation line under-indented for visual indent
1256:29: E128 continuation line under-indented for visual indent
1265:29: E128 continuation line under-indented for visual indent
1271:29: E128 continuation line under-indented for visual indent
1278:52: E251 unexpected spaces around keyword / parameter equals
1278:54: E251 unexpected spaces around keyword / parameter equals
1305: D401 First line should be imperative: 'delete', not 'deletes'
1305: D400 First line should end with '.', not 'd'
1305: D300 Expected """-quotes, got '''-quotes
1305: D200 One-line docstring should not occupy 3 lines
1313:29: E128 continuation line under-indented for visual indent
1319:29: E128 continuation line under-indented for visual indent
1328:25: E128 continuation line under-indented for visual indent
1330: D401 First line should be imperative: 'perform', not 'performs'
1330: D400 First line should end with '.', not 'e'
1330: D300 Expected """-quotes, got '''-quotes
1330: D200 One-line docstring should not occupy 3 lines
1405: D401 First line should be imperative: 'send', not 'sends'
1405: D400 First line should end with '.', not 'n'
1405: D300 Expected """-quotes, got '''-quotes
1405: D200 One-line docstring should not occupy 3 lines
1456:13: E303 too many blank lines (2)
1480:29: E128 continuation line under-indented for visual indent
1486:29: E128 continuation line under-indented for visual indent
1526:5: E303 too many blank lines (2)
1526: D400 First line should end with '.', not 'd'
1526: D300 Expected """-quotes, got '''-quotes
1562:13: E129 visually indented line with same indent as next logical line
1581:13: E128 continuation line under-indented for visual indent
1582:13: E128 continuation line under-indented for visual indent
1583:13: E128 continuation line under-indented for visual indent
1584:13: E128 continuation line under-indented for visual indent
1586: D401 First line should be imperative: 'Initialize', not 'Initializes'
1586: D400 First line should end with '.', not 'o'
1586: D300 Expected """-quotes, got '''-quotes
1607:5: E303 too many blank lines (2)
1607: D401 First line should be imperative: 'delete', not 'deletes'
1607: D400 First line should end with '.', not 't'
1607: D300 Expected """-quotes, got '''-quotes
1607: D200 One-line docstring should not occupy 3 lines
1616:5: E303 too many blank lines (2)
1616: D401 First line should be imperative: 'delete', not 'deletes'
1616: D400 First line should end with '.', not 't'
1616: D300 Expected """-quotes, got '''-quotes
1616: D200 One-line docstring should not occupy 3 lines
1623:5: E303 too many blank lines (2)
1623: D300 Expected """-quotes, got '''-quotes
1623: D200 One-line docstring should not occupy 3 lines
1649:5: E303 too many blank lines (2)
1649: D300 Expected """-quotes, got '''-quotes
1670:25: E128 continuation line under-indented for visual indent
1674:17: E128 continuation line under-indented for visual indent
1675:17: E128 continuation line under-indented for visual indent
1676:17: E128 continuation line under-indented for visual indent
1677:17: E128 continuation line under-indented for visual indent
1678:17: E128 continuation line under-indented for visual indent
1681:5: E303 too many blank lines (2)
1681: D401 First line should be imperative: 'Clean', not 'Cleans'
1681: D400 First line should end with '.', not 'd'
1681: D205 Blank line missing between one-line summary and description
1681: D300 Expected """-quotes, got '''-quotes
1702:13: E129 visually indented line with same indent as next logical line
1722: D400 First line should end with '.', not 'n'
1722: D205 Blank line missing between one-line summary and description
1722: D300 Expected """-quotes, got '''-quotes
1741: D400 First line should end with '.', not 'd'
1741: D300 Expected """-quotes, got '''-quotes
1776: D400 First line should end with '.', not 'e'
1776: D205 Blank line missing between one-line summary and description
1776: D300 Expected """-quotes, got '''-quotes
1792:9: F841 local variable 'person_id' is assigned to but never used
1793:9: F841 local variable 'uid' is assigned to but never used
1837:9: E303 too many blank lines (2)
1863:9: E303 too many blank lines (2)
1875:55: E251 unexpected spaces around keyword / parameter equals
1875:57: E251 unexpected spaces around keyword / parameter equals
1878:9: E303 too many blank lines (2)
1885: D400 First line should end with '.', not 's'
1885: D300 Expected """-quotes, got '''-quotes
1980: D401 First line should be imperative: 'call', not 'calls'
1980: D400 First line should end with '.', not 's'
1980: D300 Expected """-quotes, got '''-quotes
1993:9: F841 local variable 'nquery' is assigned to but never used
2016: D400 First line should end with '.', not 's'
2016: D205 Blank line missing between one-line summary and description
2016: D300 Expected """-quotes, got '''-quotes
2035:16: W601 .has_key() is deprecated, use 'in'
2047:29: W601 .has_key() is deprecated, use 'in'
2073:29: W601 .has_key() is deprecated, use 'in'
2095:29: W601 .has_key() is deprecated, use 'in'
2127: D300 Expected """-quotes, got '''-quotes
2145:16: W601 .has_key() is deprecated, use 'in'
2157:29: W601 .has_key() is deprecated, use 'in'
2160:43: E127 continuation line over-indented for visual indent
2169:29: W601 .has_key() is deprecated, use 'in'
2177:29: W601 .has_key() is deprecated, use 'in'
2185:29: W601 .has_key() is deprecated, use 'in'
2196:5: E303 too many blank lines (2)
2196: D400 First line should end with '.', not 'e'
2196: D300 Expected """-quotes, got '''-quotes
2286:112: E202 whitespace before ')'
2316: D400 First line should end with '.', not 'x'
2316: D300 Expected """-quotes, got '''-quotes
2336:9: F841 local variable 'pinfo' is assigned to but never used
2354: D400 First line should end with '.', not 'x'
2354: D300 Expected """-quotes, got '''-quotes
2398: D400 First line should end with '.', not 'x'
2398: D300 Expected """-quotes, got '''-quotes
2435: D102 Docstring missing
2439:20: W601 .has_key() is deprecated, use 'in'
2447:9: B901 blind except: statement
2517:5: E303 too many blank lines (2)
2518: D102 Docstring missing
2531:13: B901 blind except: statement
2548:5: E303 too many blank lines (2)
2549: D400 First line should end with '.', not 'x'
2549: D300 Expected """-quotes, got '''-quotes
2564: D400 First line should end with '.', not 'x'
2564: D300 Expected """-quotes, got '''-quotes
2575: D400 First line should end with '.', not 'x'
2575: D300 Expected """-quotes, got '''-quotes
2598: D400 First line should end with '.', not 'x'
2598: D300 Expected """-quotes, got '''-quotes
2615:33: E231 missing whitespace after ','
2623: D400 First line should end with '.', not 'x'
2623: D300 Expected """-quotes, got '''-quotes
2651: D400 First line should end with '.', not 'e'
2651: D300 Expected """-quotes, got '''-quotes
2690:26: E128 continuation line under-indented for visual indent
2704: D102 Docstring missing
2741:5: E303 too many blank lines (2)
2741: D400 First line should end with '.', not 'a'
2741: D300 Expected """-quotes, got '''-quotes
2787:12: E713 test for membership should be 'not in'
2824:5: E303 too many blank lines (2)
2827: D101 Docstring missing
2835: D401 First line should be imperative: 'Thi', not 'This'
2835: D400 First line should end with '.', not ':'
2835: D205 Blank line missing between one-line summary and description
2835: D300 Expected """-quotes, got '''-quotes
2842:12: E713 test for membership should be 'not in'
2851: D300 Expected """-quotes, got '''-quotes
2895: D400 First line should end with '.', not 'e'
2895: D208 Docstring is over-indented
2895: D300 Expected """-quotes, got '''-quotes
2952:13: F841 local variable 'ssl_param' is assigned to but never used
2955:9: E303 too many blank lines (2)
3002:9: E303 too many blank lines (2)
3027:5: E303 too many blank lines (2)
3027: D102 Docstring missing
3052:5: E303 too many blank lines (2)
3052: D102 Docstring missing
3053:43: E231 missing whitespace after ':'
3076:5: E303 too many blank lines (2)
3076: D300 Expected """-quotes, got '''-quotes
3091:20: W601 .has_key() is deprecated, use 'in'
3100:9: B901 blind except: statement
3113:5: E303 too many blank lines (2)
3113: D102 Docstring missing
3114:43: E231 missing whitespace after ':'
3132:5: E303 too many blank lines (2)
3132: D300 Expected """-quotes, got '''-quotes
3147:20: W601 .has_key() is deprecated, use 'in'
3156:9: B901 blind except: statement
3165:5: E303 too many blank lines (2)
3169: D300 Expected """-quotes, got '''-quotes
3190:25: E128 continuation line under-indented for visual indent
3194:17: E128 continuation line under-indented for visual indent
3195:17: E128 continuation line under-indented for visual indent
3196:17: E128 continuation line under-indented for visual indent
3197:17: E128 continuation line under-indented for visual indent
3198:17: E128 continuation line under-indented for visual indent
3201:5: E303 too many blank lines (2)
3204: D101 Docstring missing
3215: D300 Expected """-quotes, got '''-quotes
3240: D300 Expected """-quotes, got '''-quotes
3255:20: W601 .has_key() is deprecated, use 'in'
3263:9: B901 blind except: statement
3282:5: E303 too many blank lines (2)
3282: D300 Expected """-quotes, got '''-quotes
3297:20: W601 .has_key() is deprecated, use 'in'
3305:9: B901 blind except: statement
3321:5: E303 too many blank lines (2)
3321: D300 Expected """-quotes, got '''-quotes
3336:20: W601 .has_key() is deprecated, use 'in'
3347:9: B901 blind except: statement
3368:5: E303 too many blank lines (2)
3368: D300 Expected """-quotes, got '''-quotes
3383:20: W601 .has_key() is deprecated, use 'in'
3394:9: B901 blind except: statement
3419:5: E303 too many blank lines (2)
3419: D300 Expected """-quotes, got '''-quotes
3434:20: W601 .has_key() is deprecated, use 'in'
3445:9: B901 blind except: statement
3470:5: E303 too many blank lines (2)
3470: D300 Expected """-quotes, got '''-quotes
3485:20: W601 .has_key() is deprecated, use 'in'
3492:72: E231 missing whitespace after ','
3493:70: E231 missing whitespace after ','
3494:62: E231 missing whitespace after ','
3497:9: B901 blind except: statement
3532:5: E303 too many blank lines (2)
3532: D300 Expected """-quotes, got '''-quotes
3547:20: W601 .has_key() is deprecated, use 'in'
3555:9: B901 blind except: statement
3574:5: E303 too many blank lines (2)
3584:5: E303 too many blank lines (2)
3589: D203 Expected 1 blank line *before* class docstring, found 0
3589: D204 Expected 1 blank line *after* class docstring, found 0
3589: D200 One-line docstring should not occupy 3 lines
3650: D102 Docstring missing
3670: D203 Expected 1 blank line *before* class docstring, found 0
3670: D204 Expected 1 blank line *after* class docstring, found 0
3670: D300 Expected """-quotes, got '''-quotes
3711: D102 Docstring missing
3714: D102 Docstring missing
3747: D203 Expected 1 blank line *before* class docstring, found 0
3747: D204 Expected 1 blank line *after* class docstring, found 0
3747: D300 Expected """-quotes, got '''-quotes
3754:26: E231 missing whitespace after ','
3756: D102 Docstring missing
3759: D102 Docstring missing
3762: D400 First line should end with '.', not 'm'
3762: D200 One-line docstring should not occupy 3 lines
3770:29: E124 closing bracket does not match visual indentation
3822:34: E251 unexpected spaces around keyword / parameter equals
3822:36: E251 unexpected spaces around keyword / parameter equals


1: D400 First line should end with '.', not 's'
1: D200 One-line docstring should not occupy 2 lines
4: L102 copyright year is outdated, expected 2015 but got 2011
22:1: F401 'escape' imported but unused
23:1: I201 Missing newline before sections or imports.
26: D200 One-line docstring should not occupy 3 lines
36:13: W601 .has_key() is deprecated, use 'in'
65:1: E302 expected 2 blank lines, found 1
65: D400 First line should end with '.', not 't'
65: D205 Blank line missing between one-line summary and description


1: D400 First line should end with '.', not 'r'
1: D200 One-line docstring should not occupy 3 lines
4: L102 copyright year is outdated, expected 2015 but got 2012
28:1: I101 Imported names are in the wrong order. Should be sleep, time
29:1: I101 Imported names are in the wrong order. Should be datetime, timedelta
29:1: I100 Imports statements are in the wrong order. from datetime should be before from time
31:1: I100 Imports statements are in the wrong order. from re should be before from re
32:1: F401 'urlopen' imported but unused
33:1: I100 Imports statements are in the wrong order. from collections should be before from urllib
33:1: F401 'deque' imported but unused
40:5: F401 'ET' imported but unused
42:1: I101 Imported names are in the wrong order. Should be deserialize, serialize
43:1: I101 Imported names are in the wrong order. Should be CFG_BIBRANK_SHOW_DOWNLOAD_STATS, CFG_WEBAUTHORPROFILE_ALLOWED_FIELDCODES, CFG_WEBAUTHORPROFILE_CACHE_EXPIRED_DELAY_LIVE, CFG_WEBAUTHORPROFILE_COLLABORATION_TAG, CFG_WEBAUTHORPROFILE_FIELDCODE_TAG, CFG_WEBAUTHORPROFILE_FKEYWORD_TAG, CFG_WEBAUTHORPROFILE_KEYWORD_TAG, CFG_WEBAUTHORPROFILE_USE_ALLOWED_FIELDCODES, CFG_WEBAUTHORPROFILE_USE_BIBAUTHORID
43:1: I100 Imports statements are in the wrong order. from invenio.webauthorprofile_config should be before from invenio.webauthorprofile_config
49:1: I101 Imported names are in the wrong order. Should be create_normalized_name, gathered_names_by_personid, get_arxiv_papers_of_author, get_canonical_name_of_author, get_coauthors_of_author, get_confirmed_papers_of_author, get_existing_authors, get_hepnames, get_names_count_of_author, get_names_of_author, get_orcid_id_of_author, get_papers_by_person_id, get_person_redirect_link, get_title_of_paper, is_valid_canonical_id, remove_empty_authors, split_name_parts
49:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_webauthorprofileinterface should be before from invenio.webauthorprofile_config
49:1: F401 'gathered_names_by_personid' imported but unused
56:1: I101 Imported names are in the wrong order. Should be get_title_of_arxiv_pubid, get_title_of_doi
56:1: I100 Imports statements are in the wrong order. from invenio.bibauthorid_general_utils should be before from invenio.bibauthorid_webauthorprofileinterface
57:1: I101 Imported names are in the wrong order. Should be cache_element, expire_all_cache_for_person, get_cache_oldest_date, get_cached_element, get_expired_person_ids, precache_element
59:1: I100 Imports statements are in the wrong order. from invenio.search_engine_summarizer should be before from invenio.webauthorprofile_dbapi
59:1: F401 'summarize_records' imported but unused
60:1: I100 Imports statements are in the wrong order. from invenio.search_engine should be before from invenio.search_engine_summarizer
63:1: I100 Imports statements are in the wrong order. from invenio.bibrank_downloads_indexer should be before from invenio.search_engine_summarizer
65:1: I100 Imports statements are in the wrong order. from invenio.bibformat should be before from invenio.intbitset
66:1: I101 Imported names are in the wrong order. Should be CrossrefError, get_marcxml_for_doi
66:1: F401 'get_marcxml_for_doi' imported but unused
66:1: F401 'CrossrefError' imported but unused
83:1: E302 expected 2 blank lines, found 1
83: D103 Docstring missing
91: D205 Blank line missing between one-line summary and description
91: D300 Expected """-quotes, got '''-quotes
110:1: E302 expected 2 blank lines, found 1
110: D401 First line should be imperative: 'Retrieve', not 'Retrieves'
110: D205 Blank line missing between one-line summary and description
110: D300 Expected """-quotes, got '''-quotes
129:1: E302 expected 2 blank lines, found 1
129: D290 White spaces around docstring should be trimmed.
129: D300 Expected """-quotes, got '''-quotes
133:1: E302 expected 2 blank lines, found 1
133: D290 White spaces around docstring should be trimmed.
133: D300 Expected """-quotes, got '''-quotes
138:1: E302 expected 2 blank lines, found 1
138: D401 First line should be imperative: 'Return', not 'Returns'
138: D290 White spaces around docstring should be trimmed.
138: D300 Expected """-quotes, got '''-quotes
146:1: E302 expected 2 blank lines, found 1
146: D401 First line should be imperative: 'Expire', not 'Expires'
146: D290 White spaces around docstring should be trimmed.
146: D300 Expected """-quotes, got '''-quotes
150:1: E302 expected 2 blank lines, found 1
150: D401 First line should be imperative: 'Return', not 'Returns'
150: D205 Blank line missing between one-line summary and description
150: D300 Expected """-quotes, got '''-quotes
158:1: E302 expected 2 blank lines, found 1
158: D401 First line should be imperative: 'Return', not 'Returns'
158: D205 Blank line missing between one-line summary and description
158: D300 Expected """-quotes, got '''-quotes
166:1: E302 expected 2 blank lines, found 1
166: D401 First line should be imperative: 'Return', not 'Returns'
166: D205 Blank line missing between one-line summary and description
166: D300 Expected """-quotes, got '''-quotes
179:1: E302 expected 2 blank lines, found 1
179: D401 First line should be imperative: 'Return', not 'Returns'
179: D205 Blank line missing between one-line summary and description
179: D300 Expected """-quotes, got '''-quotes
187:1: E302 expected 2 blank lines, found 1
187: D401 First line should be imperative: 'Return', not 'Returns'
187: D205 Blank line missing between one-line summary and description
187: D300 Expected """-quotes, got '''-quotes
198:1: E302 expected 2 blank lines, found 1
198: D401 First line should be imperative: 'Return', not 'Returns'
198: D205 Blank line missing between one-line summary and description
198: D300 Expected """-quotes, got '''-quotes
206:27: E128 continuation line under-indented for visual indent
208:1: E302 expected 2 blank lines, found 1
208: D401 First line should be imperative: 'Return', not 'Returns'
208: D400 First line should end with '.', not 'e'
208: D205 Blank line missing between one-line summary and description
208: D300 Expected """-quotes, got '''-quotes
215:27: E128 continuation line under-indented for visual indent
217:1: E302 expected 2 blank lines, found 1
217: D401 First line should be imperative: 'Return', not 'Returns'
217: D205 Blank line missing between one-line summary and description
217: D300 Expected """-quotes, got '''-quotes
228:28: E128 continuation line under-indented for visual indent
230:1: E302 expected 2 blank lines, found 1
230: D401 First line should be imperative: 'Return', not 'Returns'
230: D205 Blank line missing between one-line summary and description
230: D300 Expected """-quotes, got '''-quotes
241:28: E128 continuation line under-indented for visual indent
243:1: E302 expected 2 blank lines, found 1
243: D401 First line should be imperative: 'Return', not 'Returns'
243: D205 Blank line missing between one-line summary and description
243: D300 Expected """-quotes, got '''-quotes
254:28: E128 continuation line under-indented for visual indent
256:1: E302 expected 2 blank lines, found 1
256: D401 First line should be imperative: 'Return', not 'Returns'
256: D205 Blank line missing between one-line summary and description
256: D300 Expected """-quotes, got '''-quotes
265:1: E302 expected 2 blank lines, found 1
265: D401 First line should be imperative: 'Return', not 'Returns'
265: D205 Blank line missing between one-line summary and description
265: D300 Expected """-quotes, got '''-quotes
279:47: E231 missing whitespace after ':'
279:64: E231 missing whitespace after ':'
281:27: E128 continuation line under-indented for visual indent
283:1: E302 expected 2 blank lines, found 1
283: D401 First line should be imperative: 'Return', not 'Returns'
283: D205 Blank line missing between one-line summary and description
283: D300 Expected """-quotes, got '''-quotes
292:47: E231 missing whitespace after ':'
292:64: E231 missing whitespace after ':'
294:27: E128 continuation line under-indented for visual indent
296:1: E302 expected 2 blank lines, found 1
299:1: E302 expected 2 blank lines, found 1
299: D401 First line should be imperative: 'Return', not 'Returns'
299: D205 Blank line missing between one-line summary and description
299: D300 Expected """-quotes, got '''-quotes
314:27: E128 continuation line under-indented for visual indent
316:1: E302 expected 2 blank lines, found 1
316: D401 First line should be imperative: 'Return', not 'Returns'
316: D205 Blank line missing between one-line summary and description
316: D300 Expected """-quotes, got '''-quotes
331:1: E302 expected 2 blank lines, found 1
331: D401 First line should be imperative: 'Return', not 'Returns'
331: D205 Blank line missing between one-line summary and description
331: D300 Expected """-quotes, got '''-quotes
339:1: E302 expected 2 blank lines, found 1
339: D401 First line should be imperative: 'Return', not 'Returns'
339: D205 Blank line missing between one-line summary and description
339: D300 Expected """-quotes, got '''-quotes
353:1: E302 expected 2 blank lines, found 1
353: D401 First line should be imperative: 'Return', not 'Returns'
353: D205 Blank line missing between one-line summary and description
353: D300 Expected """-quotes, got '''-quotes
361:1: E302 expected 2 blank lines, found 1
362:11: E222 multiple spaces after operator
375:1: E302 expected 2 blank lines, found 1
375: D401 First line should be imperative: 'Return', not 'Returns'
375: D205 Blank line missing between one-line summary and description
375: D300 Expected """-quotes, got '''-quotes
383:1: E302 expected 2 blank lines, found 1
383: D401 First line should be imperative: 'Return', not 'Returns'
383: D205 Blank line missing between one-line summary and description
383: D300 Expected """-quotes, got '''-quotes
437:1: E302 expected 2 blank lines, found 1
442:16: E126 continuation line over-indented for hanging indent
459:17: E126 continuation line over-indented for hanging indent
478:25: E203 whitespace before ','
480:1: E302 expected 2 blank lines, found 1
480: D103 Docstring missing
497:1: E112 expected an indented block
498:26: E231 missing whitespace after ','
504:1: E302 expected 2 blank lines, found 1
504: D103 Docstring missing
518: D401 First line should be imperative: "Person'", not "Person's"
518: D205 Blank line missing between one-line summary and description
518: D300 Expected """-quotes, got '''-quotes
527:1: E302 expected 2 blank lines, found 1
527: D401 First line should be imperative: "Person'", not "Person's"
527: D205 Blank line missing between one-line summary and description
527: D300 Expected """-quotes, got '''-quotes
535:1: E302 expected 2 blank lines, found 1
535: D103 Docstring missing
541:1: E302 expected 2 blank lines, found 1
541: D401 First line should be imperative: 'Return', not 'Returns'
541: D290 White spaces around docstring should be trimmed.
541: D300 Expected """-quotes, got '''-quotes
547:1: E302 expected 2 blank lines, found 1
565:1: E302 expected 2 blank lines, found 1
565: D401 First line should be imperative: 'Return', not 'Returns'
565: D205 Blank line missing between one-line summary and description
565: D300 Expected """-quotes, got '''-quotes
577:1: E302 expected 2 blank lines, found 1
577: D401 First line should be imperative: 'Return', not 'Returns'
577: D205 Blank line missing between one-line summary and description
577: D300 Expected """-quotes, got '''-quotes
601:1: E302 expected 2 blank lines, found 1
601: D401 First line should be imperative: 'Return', not 'Returns'
601: D205 Blank line missing between one-line summary and description
601: D300 Expected """-quotes, got '''-quotes
628: D401 First line should be imperative: 'Return', not 'Returns'
628: D400 First line should end with '.', not 's'
628: D205 Blank line missing between one-line summary and description
628: D300 Expected """-quotes, got '''-quotes
636:1: E302 expected 2 blank lines, found 1
646: D401 First line should be imperative: 'Return', not 'Returns'
646: D290 White spaces around docstring should be trimmed.
646: D300 Expected """-quotes, got '''-quotes
656: D401 First line should be imperative: 'Return', not 'Returns'
656: D205 Blank line missing between one-line summary and description
656: D300 Expected """-quotes, got '''-quotes
662:29: E128 continuation line under-indented for visual indent
665:1: E302 expected 2 blank lines, found 1
668:1: E302 expected 2 blank lines, found 1
668: D401 First line should be imperative: 'Return', not 'Returns'
668: D205 Blank line missing between one-line summary and description
668: D300 Expected """-quotes, got '''-quotes
674:29: E128 continuation line under-indented for visual indent
680: D401 First line should be imperative: 'Return', not 'Returns'
680: D205 Blank line missing between one-line summary and description
680: D300 Expected """-quotes, got '''-quotes
686:29: E128 continuation line under-indented for visual indent
691:1: E302 expected 2 blank lines, found 1
691: D103 Docstring missing
701:1: E302 expected 2 blank lines, found 1
725:1: E302 expected 2 blank lines, found 1
725:4: E271 multiple spaces after keyword
725: D401 First line should be imperative: 'Return', not 'Returns'
725: D290 White spaces around docstring should be trimmed.
725: D300 Expected """-quotes, got '''-quotes
758:1: E302 expected 2 blank lines, found 1
758: D401 First line should be imperative: 'Return', not 'Returns'
758: D205 Blank line missing between one-line summary and description
758: D300 Expected """-quotes, got '''-quotes
766:1: E302 expected 2 blank lines, found 1
766: D401 First line should be imperative: 'Return', not 'Returns'
766: D205 Blank line missing between one-line summary and description
766: D300 Expected """-quotes, got '''-quotes
773:1: E302 expected 2 blank lines, found 1
773: D401 First line should be imperative: 'Return', not 'Returns'
773: D290 White spaces around docstring should be trimmed.
773: D300 Expected """-quotes, got '''-quotes
778:1: E302 expected 2 blank lines, found 1
778: D401 First line should be imperative: 'Return', not 'Returns'
778: D205 Blank line missing between one-line summary and description
778: D300 Expected """-quotes, got '''-quotes
789:1: E302 expected 2 blank lines, found 1
789: D401 First line should be imperative: 'Return', not 'Returns'
789: D205 Blank line missing between one-line summary and description
789: D300 Expected """-quotes, got '''-quotes
804:22: E231 missing whitespace after ':'
804:46: E231 missing whitespace after ':'
804:57: E231 missing whitespace after ':'
804:82: E231 missing whitespace after ':'
804:93: E231 missing whitespace after ':'
806:1: E302 expected 2 blank lines, found 1
806: D401 First line should be imperative: 'Return', not 'Returns'
806: D205 Blank line missing between one-line summary and description
806: D300 Expected """-quotes, got '''-quotes
814:1: E302 expected 2 blank lines, found 1
814: D401 First line should be imperative: 'Return', not 'Returns'
814: D290 White spaces around docstring should be trimmed.
814: D300 Expected """-quotes, got '''-quotes
819: D401 First line should be imperative: 'Return', not 'Returns'
819: D202 No blank lines allowed *after* function docstring, found 1
819: D205 Blank line missing between one-line summary and description
819: D300 Expected """-quotes, got '''-quotes
826:29: E128 continuation line under-indented for visual indent
829:1: E302 expected 2 blank lines, found 1
832:1: E302 expected 2 blank lines, found 1
832: D401 First line should be imperative: 'Return', not 'Returns'
832: D205 Blank line missing between one-line summary and description
832: D300 Expected """-quotes, got '''-quotes
838:29: E128 continuation line under-indented for visual indent
841:1: E302 expected 2 blank lines, found 1
862:1: E302 expected 2 blank lines, found 1
862:4: E271 multiple spaces after keyword
862: D401 First line should be imperative: 'Return', not 'Returns'
862: D290 White spaces around docstring should be trimmed.
862: D300 Expected """-quotes, got '''-quotes
864:19: E711 comparison to None should be 'if cond is None:'