Thanks @samdroid-apps , but this won't work because wrap_mode expects a Pango WrapMode [1] which differs from Gtk.WrapMode [2].

Gonzalo sent another PR which uses Pango WrapMode.

[1] https://developer.gnome.org/pango/stable/pango-Layout-Objects.html#PangoWrapMode
[2] https://people.gnome.org/~gcampagna/docs/Gtk-3.0/Gtk.WrapMode.html


this statement seems wrong. are you suggesting that default start date is always October 8, 2014?


Can you add javadocs what this represents?


That would be a nice-to-have, but the first order of business is getting top-level docstrings into the language.


You don't need to over-qualify this with the `footer` selector. Please remove it!

Also, if it's going to be for _all_ instances of the main content footer, then this should go in the [main content partial](https://github.com/IBM-Watson/design-library/blob/develop/library/sass/components/_main-content.scss), not the home partial


`When I click on the "Copy" button on the "Action Bar"`

But we must be careful with stuff such as `"Copy"`, for at least 2 reasons:
1. text may change
2. text will be translated

 Either "Copy" is used by the sentence implementation to find which button this matches within some list (using the dom ID maybe), or we need separate sentences (`When I click on the copy button on the action bar`). There aren't so many buttons in the action bar, maybe the parameter doesn't save that much work ?


This seems wrong; why are the tests changing?
tester:assert(randomkit.bytes(0.5)) should not pass.


`And I select the "Destination" folder...`


I exposed them on the others sensors (temperature and rotary) because, basically, they are simpler. There's a linear relation between those properties and the output. OTOH, the light sensor is not like this. I'm trying to find the source - and reasoning - of the formula to undertand it better. If I fail, I may drop this node altogether.


Actually you should be using maven place holders for module id and no @..@.


`And I click on the "Confirm selection" button`

(same remark than above about the button's text usage)


Maybe we should also change the order for moduleChooser and alternativeJre like in Java then?


Yes, yes, fully agree!


Add file


Can you remove this line from your PR please? the JMSJournalStorageManager doesn't deal with the bindingsLocation at all.. that was a typo from my PR.


You need to enable logd in jf-common or this will break logcat


See my comment in the trigger.rb file. It appears to be a bug.


@zuooooo ekhm? ;)


It seems [#update_column](http://apidock.com/rails/v3.1.0/ActiveRecord/Persistence/update_column) doesn't call save so I'm not sure this is working. Also it would be better to call [update_all](http://apidock.com/rails/ActiveRecord/Relation/update_all) on the relation as that creates a single sql query instead of loading everything into memory.

    previous.update_all(latest: false)


> " If we only ship python3 for Gramps 4.2 then we don't need to distinguish between the python versions."

OK. Have a patch:
```diff
diff --git a/debian/control b/debian/control
index 309977d..0476aed 100644
--- a/debian/control
+++ b/debian/control
@@ -10,6 +10,7 @@ Build-Depends-Indep:
  python3-all,
  python3-setuptools
 Build-Depends:
+ dh-python,
  debhelper (>= 9.0.0)
 Standards-Version: 3.9.5
 Vcs-Git: git://anonscm.debian.org/collab-maint/gramps.git
@@ -27,10 +28,11 @@ Depends:
  python3-bsddb3,
  xdg-utils,
  ${misc:Depends},
- ${python:Depends}
+ ${python3:Depends}
 Recommends:
  graphviz,
  libosmgpsmap-1.0-0-dev,
+ gir1.2-osmgpsmap-1.0,
  python3-pyicu
 Suggests:
  fonts-freefont-ttf,
```
Thanks!


An overflow calculating how many bytes to allocate causes a segmentation fault in our production system, both in the [0.5.10-mt](https://github.com/lh3/bwa/compare/0.5.10-mt) branch and this [0.6.1-mt] (https://github.com/lh3/bwa/compare/0.6.1-mt) branch.   In our case, ```sizeof(bwtint_t) * z->n``` returns a negative value, causing ```z->a``` to be null.  Defining ```z->n``` to be either ```int64_t``` or ```uint32_t``` should be a fix.
```
(gdb) (gdb) print sizeof(z->n)
$1 = 4
(gdb) print sizeof(size_t)
$2 = 8
(gdb) print sizeof(bwtint_t)
$3 = 4
(gdb) print z->n
$4 = 906435378
(gdb) print sizeof(bwtint_t) * z->n
$5 = -669225784
(gdb) print sizeof(bwtint_t) * (unsigned long)z->n
$5 = 3625741512
(gdb) print sizeof(bwtint_t) * (unsigned int)z->n
$6 = 3625741512
(gdb) print sizeof(bwtint_t) * (int)z->n
$7 = -669225784```


I don't think elixir would work without `gulp`, right?


If that work, good finding. Where did you get that information?


any reason why this was rolled back? mistake?


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

![Build Badge](http://img.shields.io/badge/build%20status-failing-red.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-88.12%25-red.svg)


Done


I just figured most people have gulp installed globally.

On Tue, Jun 23, 2015 at 11:02 AM, Mior Muhammad Zaki <
notifications@github.com> wrote:

> I don't think elixir would work without gulp, right?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/laravel/laravel/commit/04709cf0edd1f54889b4dd733a8d851b8de0b0e9#commitcomment-11819988>
> .
>


in JDK 8, the javadoc tool validates that the content of the documentation is HTML 4.01 compliant as explained in http://docs.oracle.com/javase/8/docs/technotes/tools/unix/javadoc.html.
The javadoc isn't compliant and therefore prevents mvn package from completing successfully :
![java 8 - javadoc error](https://cloud.githubusercontent.com/assets/6297886/8310710/9f485594-19c9-11e5-80cc-b8c7f577f691.png)


Seems like you have never addressed the review comments, can you please remove this? The test class can never be on the classpath at runtime unless you put it in a wrong place


The `reject` call loads the whole relation into memory. Perhaps instead’

    .where("certificate_generators.id not in (?)", id)


Je pense que cette modification est en tous points dramatique.


Added! Thanks for the comment!


rax retry tests


hehe :smile:


@ScottPJones, different Mike :)

> documenting type/immutable members

I've thought about adding that to Docile, shouldn't be too much trouble if I have some time soon.


TeamCity LabkeyTrunk :: CDS postgres 9.4 [Build 78](https://teamcity.labkey.org/viewLog.html?buildId=256494&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4, passed: 1 Build time: 00:17:55
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSUpgradeTest: <no details avaliable>

```


better not use `for .. in` with arrays.

http://bonsaiden.github.io/JavaScript-Garden/#array.general


Thanks for reminding me. Yeah, I should probably go ahead and make a release for the contributed preventmobspawn feature.


Can you open an Issue? Thanks! Issues are easier to track than commit messages.


Changed again, now it is `2207389747`. Maybe it's better to retrieve this at runtime.


Yeh, they probably do.


I think you got forgot the envar name in the example.


This is only a default though. If you don't, you can just change this in your app. :)


変更


Implement IC vs Q


TeamCity Colore :: Main Build [Build 58](http://tc.sharpblade.net/viewLog.html?buildId=579&buildTypeId=colore_mainbuild) is now running


This might mess up the theme for people using `smart-mode-line` or any other packages using the `custom-enabled-themes` to overlay a minor theme on top of the main theme.

So I have [this custom function][1] to disable themes.

May be you can add a regexp `defcustom` to mask disabling of themes matching that regexp.

[1]: https://github.com/kaushalmodi/.emacs.d/blob/b80883ca52fc4f6f474f94fd5c41cfc0fcd7ead8/setup-files/setup-visual.el#L42-L51


typo, "fis"


It probably could if I could figure out how. Also, I was trying to write code that I could reliably test, so that once I knew I was on the right track I could refactor the code to be better. A lot of this was way over my head, so I was trying to set up something that I knew the solution to in order to make my life a little easier.


what is this...????


This should be fixed in conjunction with #156; this seems like partially addressing #156 but in a different manner.


public class HelloWorld{

}


`filePath` should be `srcPath` here, that fixes the issue I described in https://github.com/sintaxi/terraform/pull/97#issuecomment-114245226


did you "adapt" this on purpose?  there was actually no need, the old function was general enough to cover all four cases.  or was there an error in it?


Hey @nicgrayson have you taken a look at https://github.com/mesos/kafka/tree/master/src/docker


TeamCity Colore :: Main Build [Build 58](http://tc.sharpblade.net/viewLog.html?buildId=579&buildTypeId=colore_mainbuild) outcome was **SUCCESS**
Summary: Tests passed: 113 Build time: 0:4:6


> Can you open an Issue?

Will do! Apologies for putting this in the wrong place.


Every single one of these element bindings MUST have an un-bind on destroy ... this will cause big memory leaks otherwise. Always remember, for every DOM event subscription (or any event apart from an angular event), if you don't un-bind, you cause a mem leak.


ok is part of chai's assertion, meaning that something is or is not ok, as in working based on the tests you write. To be fair, I really only have assumptions about how a lot of this is working (to be fair, most of it isn't working) and most of this was best guess, what I could cobble together from documentation, and some info gathered from stack overflow.


Ack, fixed in https://github.com/m-labs/artiq/commit/f0dddd9f391d1da7b660e85f3ff92d353f532508


this does'nt work
its escaping spaces and the servername is "a\ server\ with\ spaces" instead of " a server with spaces"


ok


Fixed, thanks.


ah...  performance.  good point.  :+1:


FacilityTabGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg)


> May be you can add a regexp defcustom to mask disabling of themes matching that regexp.

Sounds OK. Could you please PR to fix this? Just add a little `fboundp` thing to `counsel--load-theme-action`, like I do now for `powerline`.


@wilbert looks interesting, mind making a pull request? :sunglasses:


Could you try a few other small netlib problems e.g. blend, because it is possible the linear system formed for this problem is ill-conditioned (which may have a simple fix).


Should this be backported to 1.0?


Nice revisions lol


develop (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Hm, what about the [first line](https://github.com/timoxley/functional-javascript-workshop/blob/master/exercises/partial_application_without_bind/solution/solution.js#L1)?

```js
var slice = Array.prototype.slice
```


My mistake


This is an old comment. Now it is working for this problem and few other netlib problems I have tried. But when size is increased, it takes a lot longer time.


Removed unused lock strings


will be removed


I don't speak Chinese and google translation has no mean, please try re-ask your question in English or French.


We don't want this hunk in the patch


sorry, will be removed as well


Yeah, good spot.


@sedrubal Die Zeile war übrigens vorher nicht mit `+`, sondern mit `,`. In so einem PEP8-Refactoring commit wurde die dann kaputt gemacht :-P


gvir_sandbox_init_util_mkdir would be a better name (mount removed)


A (short) note in the commit message is fine with me.


sa


will be moved to another commit


why manual?


ok. have you set up a timer so you can figure out what is making it so slow?


I don't think so - a slightly inaccurate warning classname is trivial enough that it's not worth breaking the 1.0 RC code freeze over it, IMO.


can this be also merged to mms_sdp_dmgr ?


I suppose the factoring of this method is unrelated. Perhaps it should be in another commit.


Is that one really needed? There is no translatable string in that file


I'm assuming since we aren't even loading this module anymore we don't need blueimp as a dependency? we are still referencing this in bower, etc.. which means we'll still deploy it's files with the core.


I like this fix.


Good catch - thanks for fixing that - I missed it.


Ok means commits are ready to test now lol


check if this trigger is not done by reconcile_partial()


This message is already printed in the mount_mkdir function, would need to be removed


Message already in mount_mkdir, needs to be removed here


Hmmm. Intriguing. Yes, this could definitely help if I go this route for our actions.


Are you 100% sure about it?


As you may expect: Matrix decomposition. This was my initial observation, but haven't set timers yet. I will be on campus later today. Lets catch up.


Message already in mount_mkparent, needs to be removed here


Ok sounds good.


Formatting


This creates a new wrapper around the texture, not a new texture.


will be moved to another commit


How comes you remove that one?


Could the type of the `item` parameter of several functions and the return type of the get-function be a type parameter? That way, this stub would give some additional type safety. (A user can always use `Any` for the type parameter to get the behaviour of the current stub.)


Why is this one landing in this file?


right :)


The search box will fit on the right side allowing the table to move up to avoid a gap due to the search


I spent 60 secs searching for the ```payment_order_type``` variable before noticing ```self.``` was on the previous line.

So this is definitely bad identation.


headers are now found in the main table file and the type string will just be passed through


Super! Das ganze nimmt jetzt richtig Form an. Ich muss mich ranhalten.
Da fällt mir gerade ein: Wenn nun jemand DAI-Stil, aber ohne Autor-Jahr möchte, liesse sich dann die Option verwenden? Überschreibt Biblatex dann diese Voreinstellung mit der manuellen Option im TeX-Dokument?


I think you missed the `join` coroutine.


The functions you moved are not mount functions, rather mkdir ones.... the commit message needs to reflect that. Please add a short description about why you needed that change in the commit message description: this is not obvious when looking at this patch only


s/order_type/payment_order_type/ for consistency


fix if the third column is not an 'updated' column


Great! I'm still a bit confused as to how to solve this problem:

```
10/1/2007	If the targeted creature becomes an illegal target but the targeted Shapeshifter doesn't, the Shapeshifter will still become a copy of the object. The effect will use the illegal target's current copiable characteristics if it's still on the battlefield, or its last known copiable characteristics if it has left the battlefield.
```

It seems that I can't get the targeted creature/LKI for that creature from inside the effect because it will have been deleted from the target object during the `isLegal` check. Is there a way around this?


this file is not used


Ah yes, lazy copying! Will fix.


@alext left a comment about changing the interface for `PublishingAPINotifier`, once that's done I'll fix up these tests and avoid the internals testing


The `if` isn't necessary though.


Fix to match the strings found when getting the headers


Moved to table.php


moved to table.php


Perhaps this commit should go in another PR because it changes behaviour slightly, while the other 3 commits are just refactoring.


Em push lên thử để test chút ạ. :dancers:


Does this need a `query.carousel();` to init on document ready?

Right now, have to hit reload to get carousel to init again.


Hi Chris,
The setup is still not working on my Ubuntu 13.10 desktop environment. I'm getting the following when i ran setup.sh:
[*] Installing Ubuntu Dependencies
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package python-pip
setup.sh: 58: setup.sh: easy_install: not found
setup.sh: 59: setup.sh: pip: not found
setup.sh: 60: setup.sh: pip: not found
setup.sh: 61: setup.sh: pip: not found
setup.sh: 62: setup.sh: pip: not found
setup.sh: 63: setup.sh: pip: not found
setup.sh: 64: setup.sh: pip: not found

Any suggestions?


Commented out as it would be good to simply use the function in table.php


A personal style bug, I prefer

    query_fields.add(prop)

instead, because you don't have to instantiate a new set and (more importantly) I don't have to remember what the |= operator does.


Moved into table.php


My point is, on the thermal sensor, you expose a constant but not the other -- why? Either both or none IMHO.


`options.container` is already a proper selector object base on this line https://github.com/mobify/selector-utils/blob/master/src/selector/scrollTo.js#L31

There is no need to initialize again.


This TODO should be preserved in the refactored method


this is actually not true. IIUC when 2 packages have the same file (output of query), you will add them all to requires. Then the installation would fail because of conflicting files. You want to add just one package providing the file. Yield of `_resolve` should return one package name.


@Airse Why is this commented out?


Any idea where the extra spaces are indicated by the build failure?


The command and shift key needs to be pressed as well.
This is what happens given when user uses cmd+a:

```
[command] down
[a] down
[a] up
[command] up
```
Also the events and created source get released on the end to free memory


It is fixed but I suggest you to replaced strpos by preg_match or another due to some the php version are depreciated


OKAYYYY :D


I personally think this is not even relevant, but I have no time to check this.


Btw nice they are xD


Why you always do it differently then I want :). I don't know if `"dnf" in ["dnf"]` or `"dnf in "bladnfbla". Can you rewrite it to `self.assertEqual(["dnf"], self.spec.Requires)` to make it more clear?


this is the cause of #4 since this is not in scope of the callback, $.proxy should solve it


looks like you didn't effect this change to the schema. might need to `rake db:rollback` and then `rake db:migrate` again for these changes to take place. And if there's an error with the rollback, you might need to comment out these `change_column_null`s, then rollback, then uncomment, then migrate again. hope that all makes sense.


Passwords in a commit?


컴파일러 옵션으로 지정하는 것 뿐만 아니라 소스 내부에서 지정하는 것 등 모든 define류를 포함함.


FacilityTabGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2184 of 2213 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-98.69%25-yellow.svg)


You're going to just need to install pip on your system in a manner that works for you.

Unfortunately, there seems to be issues with pip, and the packages I'm installing on Ubuntu.  It doesn't happen on either Debian or Kali, so I'm not really certain about the problem with Ubuntu.  Also, I have been testing with 14.04, so I can't really speak to anything older.  But first step will be you needing to install pip.  Then try to see if the pip packages will install.  That's where I can had issue on Ubuntu.


We are in the in process of making these changes.


@addyosmani I used `^0.7.0` because it'll still grab 0.7.2, and because it's what I've seen around the Polymer elements (e.g. [iron-selector](https://github.com/PolymerElements/iron-selector/blob/master/bower.json)). If this element should require 0.7.2 at minimum, I can go ahead and change the version dependency to `^0.7.2`


Oops. Thanks for that clarification. I was reading the wrong line.

On Tue, Jun 23, 2015 at 8:55 AM, Ryan Batt <notifications@github.com> wrote:

> @mpinsky <https://github.com/mpinsky> They aren't the same. The deleted
> line 286 was the same is Spring. The added Lines 287-290 show the
> calculation for which strata were observed in all years. The Fall has a few
> less strata, but some of the strata in the fall were not used in the spring
> (i.e., the strata in either season are not all elements of the strata of
> the other season).
>
> Does this clarify things for you? They aren't the same, and I
> programmatically identified which strata to use, rather than listing them
> out manually.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mpinsky/OceanAdapt/commit/5a7b51f7d0a0071baf80812bc7494dd2f94a94b2#commitcomment-11815676>
> .
>


thanks for this


I think your on to something...


it comes with kconfig, but we can overwrite this help target.


Hi Chris,
Thank you so much for the quick response. Which version of the Debian or
Kali would you recommend? I've tried the Ubuntu 14.04 as well but I ran
into the same problem, couldn't install pip; so I'm thinking to try Debian
or Kali.

Best regards
Paul

On Tue, Jun 23, 2015 at 12:36 PM, ChrisTruncer <notifications@github.com>
wrote:

> You're going to just need to install pip on your system in a manner that
> works for you.
>
> Unfortunately, there seems to be issues with pip, and the packages I'm
> installing on Ubuntu. It doesn't happen on either Debian or Kali, so I'm
> not really certain about the problem with Ubuntu. Also, I have been testing
> with 14.04, so I can't really speak to anything older. But first step will
> be you needing to install pip. Then try to see if the pip packages will
> install. That's where I can had issue on Ubuntu.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChrisTruncer/Just-Metadata/commit/e79c18d3280bf020c69130ec7830afe69c109909#commitcomment-11820700>
> .
>


Ugh, that again >_< Yea, it's most likely in the `-'s`. I will patch it by the end of this week.


This block and `xinerama_index` should also be protected with `HAVE_LIBXINERAMA`, no?


Ah, I understand. Well, the reason why we built it that way was because we test drove parseKey - and then we extracted nospc. I would expect the public interface to be able to deal with spaces in hex strings, so that would be good to continue have - and have it tested somewhere.


Hey,

I've tested it on Debian Stable, so that would work.  I personally run it on the latest version of Kali, so either of those should work for you.


Can you move this chunk into the other commit?  It doesn't really belong here.


Good


Is it intended that this endpoint is not authenticated?  I can't see a reason why it needs to be - but just pointing out that it isn't by default if I remember correctly.


for styling?


I didn't have trouble with them not transmitting power...but your the one that knows how this works :P


yeah this isn't working... libevent is also calling SSL_Free unless you set a particular flag, I need to look into it some more!


File "config.exampe.js" needs to be renamed to "config.example.js".


Yeah! I thought is was pretty weird


So it's two Mikes doing all this neat stuff with documentation?  OK, I'll *try* not to get confused again! :grinning:


Looks good to me on the whole (how much my opinion matters, of course, is up to you =)

If you ever want to chat about implementation details or have any questions, I can be found as "cmyers" on irc.freenode.net very reliably, from 9am to 6pm.


You can make another pull request to add yourself to the contributors list to the readme.md

Thanks and keep adding awesomeness :+1:


`_shell_escape` isn't really part of the class's public interface. That's why it has the leading underscore. There is already another global shell_escape function that should probably be refactored and used everywhere instead. It's a larger change but I can do that if martine et al. think it's worth trying.


Please test your changes ALWAYS on your local machine. There is a comma missing after ```"sr" => "српски"```


right...


ok


:+1: for me


my bad... sorry.


@rottegift good point.  While "fairness" isn't always the best policy it's been my experience that it's what people expect to be the default.  Counter-intuitively this policy can also end up being best for real world performance because once one of the vdev's is near capacity it can cause significant performance penalties.  So it's a situation best avoided.  And for more advanced users there's always the `metaslab_bias_enabled=0` module option which @dweezil mentioned.  But thanks for bringing up that point and keeping us honest!


No prob. Thanx for bringing it in!


Sorry, what should I do here?


We either need to change these to package: urls or we need to add a baseUrl parameter here. The runtime relative url resolving is going to use mirrors and will not be available post-transform.


maybe also add a warning here for hash_admin_password(Scheme, _ClearPassword) for an invalid scheme? besides that, LGTM.


[Deployment available online](https://s.brightspace.com/lib/ifrau/0.2.0/ifrau.js)


Don't you mean to return the key's value instead of TRUE?


Hi!
Is it possible to calculated functional richness using species abundance (L59)? According to Villeger et al. (2008, Ecology) and Mouchet et al. (2010, Functional Ecology), abundance is not included in the functional richness index.
Thank you!
Cheers
Fábio


Ah, of course. I'll expose resistance too =D


So, `itemFlags`?


Thanks for attribution; by technicality do you not need to list me in the license now????

Don't really mind, as it's my GitHub contribution log that I care about; but just to fully meet the requirements of your BSD license (or as I call it the 'alternative' MIT license :laughing: )


omg zeach why you do this to me


nit: s/it's/its/


A suggestion: "The V8 library provides Node.js with a JavaScript engine, which Node.js controls via the V8 C++ API. V8 is maintained by Google, for use in Chrome."


I'm sure it's all related, but that gist is indicating a blow up trying to install swift - and the explosion looks exactly like you don't have the newest pip?  This only just got merged yesterday - if you pulled before you may not have theses fixes - can you pull again?


Just a comment about style (for future reference): You can put the comma at the end of each line, and an extra comment on the last line will not cause any problem. That way if someone inserts a line anywhere in the array, only that new line shows up in the diff.


On Android, SMS can be sent in two ways, one through using native SMS app or directly. The option values and comments are to show the capability to the user. The code is coming from the plugin's documentation: https://github.com/cordova-sms/cordova-sms-plugin


zlib is pretty standard for providing {de,}compression. It powers gzip, libpng, amongst others.


:thumbsup:


Oh I think this is an outdated commit! I add the validation code on Wednesday


By "color inheritance" I mean that the UI theme plays better with other syntax themes. Even those with light backgrounds, though it's not perfect, it looks pretty usable.


BTW, GitHub mangled my joke – I used many more `\` than GitHub decided to print. Yay, character escapes!


I think Neopixels are overkill (especially since they want ~5V) but I do like the idea of more LEDs :-)


I would say the extra entry is just confusing 'cause it's not a table name.... same for "orders" at the bottom.


Heads up, travis-ci test fails.


Made a couple of comments, otherwise looks good to me.


Keep using pathExist as and error flag. And check if entity['version'] equal 0; in this case build the scene.


1: M111 unrecognized component name: oauthclient


1: M111 unrecognized component name: legacy


1: M111 unrecognized component name: assets


1: M110 missing component name


1: M100 needs more reviewers
1: M101 signature is missing
1: M110 missing component name


1: M111 unrecognized component name: jasmine


Unfortunately not, Functional Richness is basically the volume of the multidimensional trait space, so only the species with the most extreme trait values are taken into account and their abundance cannot be used to compute Functional Richness. The other functional diversity indices can take into account abundance: Functional Divergence, Functional Dispersion and Functional Evenness. See the w.abun argument in ?dbFD.

Yours,
Lionel


jaa toch :smile:


I don't understand why you want some options before, some after the filename, or why you moved -E.

Wev.


Great, I'm going to pull down a copy of Debian and try it out. Thanks
Chris! Really appreciated!

On Tue, Jun 23, 2015 at 12:42 PM, ChrisTruncer <notifications@github.com>
wrote:

> Hey,
>
> I've tested it on Debian Stable, so that would work. I personally run it
> on the latest version of Kali, so either of those should work for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ChrisTruncer/Just-Metadata/commit/e79c18d3280bf020c69130ec7830afe69c109909#commitcomment-11820790>
> .
>


46:1: I100 Imports statements are in the wrong order. from MySQLdb should be before from invenio.utils.datastructures
48:1: I100 Imports statements are in the wrong order. from thread should be before from MySQLdb


35:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from wtforms.validators


1: D400 First line should end with a period (not 'e')
24:1: E402 module level import not at top of file
25:1: I101 Imported names are in the wrong order. Should be datetime_format, rlike, run_sql
25:1: I201 Missing newline before sections or imports.
25:1: E402 module level import not at top of file
26:1: I101 Imported names are in the wrong order. Should be CFG_WEBMESSAGE_DAYS_BEFORE_DELETE_ORPHANS, CFG_WEBMESSAGE_MAX_NB_OF_MESSAGES, CFG_WEBMESSAGE_ROLES_WITHOUT_QUOTA, CFG_WEBMESSAGE_STATUS_CODE
26:1: E402 module level import not at top of file
31:1: I101 Imported names are in the wrong order. Should be convert_datestruct_to_datetext, datetext_default
31:1: E402 module level import not at top of file
32:31: E126 continuation line over-indented for hanging indent
33:1: I100 Imports statements are in the wrong order. from invenio.legacy.websession.websession_config should be before from invenio.utils.date
33:1: E402 module level import not at top of file
35:1: E402 module level import not at top of file
38: D401 First line should be in imperative mood ('Check', not 'Checks')
38: D400 First line should end with a period (not 'e')
38: D205 1 blank line required between summary line and description (found 0)
45:10: E221 multiple spaces before operator
53:1: E302 expected 2 blank lines, found 1
53: D400 First line should end with a period (not 's')
53: D205 1 blank line required between summary line and description (found 0)
78:21: E127 continuation line over-indented for visual indent
79:21: E127 continuation line over-indented for visual indent
80:20: E127 continuation line over-indented for visual indent
96:1: E302 expected 2 blank lines, found 1
96: D202 No blank lines allowed after function docstring (found 1)
96: D205 1 blank line required between summary line and description (found 0)
117:1: E302 expected 2 blank lines, found 1
117: D210 No whitespaces allowed surrounding docstring text
117: D400 First line should end with a period (not 'r')
117: D205 1 blank line required between summary line and description (found 0)
133:1: E302 expected 2 blank lines, found 1
133: D210 No whitespaces allowed surrounding docstring text
133: D400 First line should end with a period (not 'd')
133: D205 1 blank line required between summary line and description (found 0)
150: D400 First line should end with a period (not 'l')
150: D205 1 blank line required between summary line and description (found 0)
169:26: E502 the backslash is redundant between brackets
170:21: E127 continuation line over-indented for visual indent
170:61: E502 the backslash is redundant between brackets
183:1: E302 expected 2 blank lines, found 1
183: D400 First line should end with a period (not 'd')
183: D205 1 blank line required between summary line and description (found 0)
199:1: E302 expected 2 blank lines, found 1
199: D400 First line should end with a period (not 'x')
199: D205 1 blank line required between summary line and description (found 0)
216:1: E302 expected 2 blank lines, found 1
216: D401 First line should be in imperative mood ('Check', not 'Checks')
216: D400 First line should end with a period (not ',')
216: D205 1 blank line required between summary line and description (found 0)
247:1: E302 expected 2 blank lines, found 1
247: D400 First line should end with a period (not ')')
247: D205 1 blank line required between summary line and description (found 0)
268:1: E302 expected 2 blank lines, found 1
268: D400 First line should end with a period (not 's')
268: D205 1 blank line required between summary line and description (found 0)
280:18: E225 missing whitespace around operator
290:9: E301 expected 1 blank line, found 0
290: D210 No whitespaces allowed surrounding docstring text
290: D401 First line should be in imperative mood ('take', not 'takes')
290: D400 First line should end with a period (not 's')
296:1: E302 expected 2 blank lines, found 1
296: D400 First line should end with a period (not 's')
296: D205 1 blank line required between summary line and description (found 0)
320:1: E302 expected 2 blank lines, found 1
320: D400 First line should end with a period (not 's')
320: D205 1 blank line required between summary line and description (found 0)
332:19: E225 missing whitespace around operator
342:9: E301 expected 1 blank line, found 0
342: D210 No whitespaces allowed surrounding docstring text
342: D401 First line should be in imperative mood ('take', not 'takes')
342: D400 First line should end with a period (not 's')
349: D400 First line should end with a period (not 's')
349: D205 1 blank line required between summary line and description (found 0)
371:9: E301 expected 1 blank line, found 0
371: D210 No whitespaces allowed surrounding docstring text
371: D400 First line should end with a period (not 's')
377:1: E302 expected 2 blank lines, found 1
377: D205 1 blank line required between summary line and description (found 0)
399:1: E302 expected 2 blank lines, found 1
399: D210 No whitespaces allowed surrounding docstring text
399: D401 First line should be in imperative mood ('check', not 'checks')
399: D400 First line should end with a period (not '1')
407:1: E302 expected 2 blank lines, found 1
407: D401 First line should be in imperative mood ('Create', not 'Creates')
407: D205 1 blank line required between summary line and description (found 0)
443:1: E302 expected 2 blank lines, found 1
443: D400 First line should end with a period (not 's')
443: D205 1 blank line required between summary line and description (found 0)
460:9: E301 expected 1 blank line, found 0
460: D210 No whitespaces allowed surrounding docstring text
460: D401 First line should be in imperative mood ('not(i', not 'not(is')
460: D400 First line should end with a period (not ')')
475: D400 First line should end with a period (not 'e')
475: D205 1 blank line required between summary line and description (found 0)
493:1: E302 expected 2 blank lines, found 1
493: D401 First line should be in imperative mood ('Update', not 'Updates')
493: D400 First line should end with a period (not 'd')
493: D205 1 blank line required between summary line and description (found 0)
499:10: E222 multiple spaces after operator
513:12: E225 missing whitespace around operator
526:1: E302 expected 2 blank lines, found 1
526: D400 First line should end with a period (not 'n')
532:37: E222 multiple spaces after operator
538:1: E302 expected 2 blank lines, found 1
538: D400 First line should end with a period (not 'e')
538: D200 One-line docstring should fit on one line with quotes (found 2)
545:75: E222 multiple spaces after operator
563:1: E302 expected 2 blank lines, found 1
563: D400 First line should end with a period (not 't')
563: D205 1 blank line required between summary line and description (found 0)
570:1: E302 expected 2 blank lines, found 1
570: D210 No whitespaces allowed surrounding docstring text
570: D401 First line should be in imperative mood ('Clean', not 'Cleans')
570: D400 First line should end with a period (not 'e')
595:5: E301 expected 1 blank line, found 0
626:1: W391 blank line at end of file


1: D400 First line should end with a period (not 'e')
23:1: I100 Imports statements are in the wrong order. from datetime should be before from time
25:1: I101 Imported names are in the wrong order. Should be rlike, run_sql
26:1: I101 Imported names are in the wrong order. Should be CFG_WEBMESSAGE_DAYS_BEFORE_DELETE_ORPHANS, CFG_WEBMESSAGE_MAX_NB_OF_MESSAGES, CFG_WEBMESSAGE_ROLES_WITHOUT_QUOTA, CFG_WEBMESSAGE_STATUS_CODE
32:1: I101 Imported names are in the wrong order. Should be convert_datestruct_to_datetext, datetext_default
33:31: E126 continuation line over-indented for hanging indent
34:1: I100 Imports statements are in the wrong order. from invenio.legacy.websession.websession_config should be before from invenio.utils.date
36:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy should be before from invenio.legacy.websession.websession_config
38:1: I100 Imports statements are in the wrong order. from invenio.modules.accounts.models should be before from invenio.modules.messages.models
38:1: F401 'User' imported but unused
43: D205 1 blank line required between summary line and description (found 0)
53:1: E302 expected 2 blank lines, found 1
53: D205 1 blank line required between summary line and description (found 0)
61:12: E127 continuation line over-indented for visual indent
66: D205 1 blank line required between summary line and description (found 0)
75:42: E225 missing whitespace around operator
77:5: B901 blind except: statement
78:42: E225 missing whitespace around operator
79:45: E225 missing whitespace around operator
81:1: E302 expected 2 blank lines, found 1
81: D401 First line should be in imperative mood ('Check', not 'Checks')
81: D400 First line should end with a period (not 'e')
81: D205 1 blank line required between summary line and description (found 0)
89:12: E127 continuation line over-indented for visual indent
90:12: E127 continuation line over-indented for visual indent
92:1: E302 expected 2 blank lines, found 1
92: D205 1 blank line required between summary line and description (found 0)
99:42: E502 the backslash is redundant between brackets
99:11: E271 multiple spaces after keyword
100:13: E126 continuation line over-indented for hanging indent
101:13: E127 continuation line over-indented for visual indent
102:13: E127 continuation line over-indented for visual indent
104:1: E302 expected 2 blank lines, found 1
104: D205 1 blank line required between summary line and description (found 0)
119:12: E127 continuation line over-indented for visual indent
120:12: E127 continuation line over-indented for visual indent
122:1: E302 expected 2 blank lines, found 1
122: D210 No whitespaces allowed surrounding docstring text
122: D400 First line should end with a period (not 'r')
122: D205 1 blank line required between summary line and description (found 0)
130:12: E127 continuation line over-indented for visual indent
131:12: E127 continuation line over-indented for visual indent
132:12: E127 continuation line over-indented for visual indent
134:1: E302 expected 2 blank lines, found 1
134: D210 No whitespaces allowed surrounding docstring text
134: D400 First line should end with a period (not 'd')
134: D205 1 blank line required between summary line and description (found 0)
140:12: E127 continuation line over-indented for visual indent
141:12: E127 continuation line over-indented for visual indent
142:12: E127 continuation line over-indented for visual indent
144:1: E302 expected 2 blank lines, found 1
144: D400 First line should end with a period (not 'l')
144: D205 1 blank line required between summary line and description (found 0)
159:12: E127 continuation line over-indented for visual indent
160:12: E127 continuation line over-indented for visual indent
161:12: E127 continuation line over-indented for visual indent
163:1: E302 expected 2 blank lines, found 1
163: D400 First line should end with a period (not 'd')
163: D205 1 blank line required between summary line and description (found 0)
170:12: E127 continuation line over-indented for visual indent
171:12: E127 continuation line over-indented for visual indent
171:44: E225 missing whitespace around operator
172:12: E127 continuation line over-indented for visual indent
174:1: E302 expected 2 blank lines, found 1
174: D400 First line should end with a period (not 'x')
174: D205 1 blank line required between summary line and description (found 0)
184:20: E127 continuation line over-indented for visual indent
188:1: E302 expected 2 blank lines, found 1
188: D401 First line should be in imperative mood ('Check', not 'Checks')
188: D400 First line should end with a period (not ',')
188: D205 1 blank line required between summary line and description (found 0)
199:16: E127 continuation line over-indented for visual indent
200:16: E127 continuation line over-indented for visual indent
201:16: E127 continuation line over-indented for visual indent
201:63: E225 missing whitespace around operator
202:16: E127 continuation line over-indented for visual indent
204:65: E502 the backslash is redundant between brackets
205:12: E128 continuation line under-indented for visual indent
206:12: E127 continuation line over-indented for visual indent
208:1: E302 expected 2 blank lines, found 1
208: D400 First line should end with a period (not ')')
208: D205 1 blank line required between summary line and description (found 0)
216:15: E128 continuation line under-indented for visual indent
216:62: E502 the backslash is redundant between brackets
217:15: E128 continuation line under-indented for visual indent
217:55: E225 missing whitespace around operator
218:51: E225 missing whitespace around operator
220:19: E127 continuation line over-indented for visual indent
220:59: E225 missing whitespace around operator
221:55: E225 missing whitespace around operator
222:19: E127 continuation line over-indented for visual indent
227:1: E302 expected 2 blank lines, found 1
227: D400 First line should end with a period (not 's')
227: D205 1 blank line required between summary line and description (found 0)
239:18: E225 missing whitespace around operator
249:9: E301 expected 1 blank line, found 0
249: D210 No whitespaces allowed surrounding docstring text
249: D401 First line should be in imperative mood ('take', not 'takes')
249: D400 First line should end with a period (not 's')
255:1: E302 expected 2 blank lines, found 1
255: D400 First line should end with a period (not 's')
255: D205 1 blank line required between summary line and description (found 0)
279:1: E302 expected 2 blank lines, found 1
279: D400 First line should end with a period (not 's')
279: D205 1 blank line required between summary line and description (found 0)
291:19: E225 missing whitespace around operator
301:9: E301 expected 1 blank line, found 0
301: D210 No whitespaces allowed surrounding docstring text
301: D401 First line should be in imperative mood ('take', not 'takes')
301: D400 First line should end with a period (not 's')
308: D400 First line should end with a period (not 's')
308: D205 1 blank line required between summary line and description (found 0)
330:9: E301 expected 1 blank line, found 0
330: D210 No whitespaces allowed surrounding docstring text
330: D400 First line should end with a period (not 's')
336:1: E302 expected 2 blank lines, found 1
336: D205 1 blank line required between summary line and description (found 0)
358:1: E302 expected 2 blank lines, found 1
358: D210 No whitespaces allowed surrounding docstring text
358: D401 First line should be in imperative mood ('check', not 'checks')
358: D400 First line should end with a period (not '1')
366:1: E302 expected 2 blank lines, found 1
366: D401 First line should be in imperative mood ('Create', not 'Creates')
366: D205 1 blank line required between summary line and description (found 0)
402:1: E302 expected 2 blank lines, found 1
402: D400 First line should end with a period (not 's')
402: D205 1 blank line required between summary line and description (found 0)
419:9: E301 expected 1 blank line, found 0
419: D210 No whitespaces allowed surrounding docstring text
419: D401 First line should be in imperative mood ('not(i', not 'not(is')
419: D400 First line should end with a period (not ')')
434: D400 First line should end with a period (not 'e')
434: D205 1 blank line required between summary line and description (found 0)
453: D401 First line should be in imperative mood ('Update', not 'Updates')
453: D400 First line should end with a period (not 'd')
453: D205 1 blank line required between summary line and description (found 0)
459:5: F841 local variable 'now' is assigned to but never used
459:10: E222 multiple spaces after operator
465:9: E122 continuation line missing indentation or outdented
465:34: E225 missing whitespace around operator
466:9: E122 continuation line missing indentation or outdented
467:9: E122 continuation line missing indentation or outdented
467:33: E225 missing whitespace around operator
468:9: E122 continuation line missing indentation or outdented
473:38: E225 missing whitespace around operator
482: D400 First line should end with a period (not 'n')
493:1: E302 expected 2 blank lines, found 1
493: D400 First line should end with a period (not 'e')
493: D200 One-line docstring should fit on one line with quotes (found 2)
518:1: E302 expected 2 blank lines, found 1
518: D400 First line should end with a period (not 't')
518: D205 1 blank line required between summary line and description (found 0)
525:1: E302 expected 2 blank lines, found 1
525: D210 No whitespaces allowed surrounding docstring text
525: D401 First line should be in imperative mood ('Clean', not 'Cleans')
525: D400 First line should end with a period (not 'e')
550:5: E301 expected 1 blank line, found 0
581:1: W391 blank line at end of file


1: M111 unrecognized component name: docker


1: M111 unrecognized component name: docs


33:1: I100 Imports statements are in the wrong order. from random should be before from invenio.utils.hash


1: D400 First line should end with a period (not 'I')
33:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcheck should be before from invenio.utils.deprecation
33:1: E402 module level import not at top of file
35:1: E302 expected 2 blank lines, found 1


95:1: E266 too many leading '#' for block comment
97:1: I100 Imports statements are in the wrong order. import re should be before from invenio.utils.deprecation
97:1: E402 module level import not at top of file
98:1: E402 module level import not at top of file
99:1: E402 module level import not at top of file
100:1: I201 Missing newline before sections or imports.
100:1: E402 module level import not at top of file
107:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from six
107:1: E402 module level import not at top of file
108:1: E402 module level import not at top of file
110:1: I101 Imported names are in the wrong order. Should be CFG_BIBRECORD_DEFAULT_CORRECT, CFG_BIBRECORD_DEFAULT_VERBOSE_LEVEL, CFG_BIBRECORD_PARSERS_AVAILABLE, CFG_BIBRECORD_WARNING_MSGS, CFG_MARC21_DTD, InvenioBibRecordFieldError, InvenioBibRecordParserError
110:1: E402 module level import not at top of file
114:1: E402 module level import not at top of file
118:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.utils.text
118:1: E402 module level import not at top of file
133:1: E402 module level import not at top of file
144:1: E266 too many leading '#' for block comment
1395:25: W503 line break before binary operator
1430: D401 First line should be in imperative mood ('Turn', not 'Turns')
1430: D400 First line should end with a period (not 'e')
1430: D200 One-line docstring should fit on one line with quotes (found 3)
1439:25: E128 continuation line under-indented for visual indent
1441:1: E302 expected 2 blank lines, found 1
1535:1: E266 too many leading '#' for block comment
1694:5: B901 blind except: statement
1760:9: E266 too many leading '#' for block comment
1761:9: E266 too many leading '#' for block comment
1762:9: E266 too many leading '#' for block comment
1765:9: E266 too many leading '#' for block comment


33:1: I100 Imports statements are in the wrong order. from invenio.base.factory should be before from invenio.utils.deprecation
33:1: E402 module level import not at top of file
35:1: E302 expected 2 blank lines, found 1
36: D103 Missing docstring in public function


38:1: I100 Imports statements are in the wrong order. import cgi should be before from invenio.utils.deprecation
38:1: E402 module level import not at top of file
39:1: E402 module level import not at top of file
40:1: E402 module level import not at top of file
41:1: E402 module level import not at top of file
42:1: E402 module level import not at top of file
45:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS, CFG_BASE_URL, CFG_BIBFORMAT_HIDDEN_TAGS, CFG_BIBINDEX_CHARS_PUNCTUATION, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS, CFG_BIBSORT_BUCKETS, CFG_BIBSORT_ENABLED, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE, CFG_CERN_SITE, CFG_INSPIRE_SITE, CFG_LOGDIR, CFG_LOGDIR, CFG_OAI_ID_FIELD, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_RECORD, CFG_SITE_URL, CFG_SOLR_URL, CFG_WEBSEARCH_DEF_RECORDS_IN_GROUPS, CFG_WEBSEARCH_DETAILED_META_FORMAT, CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS, CFG_WEBSEARCH_FIELDS_CONVERT, CFG_WEBSEARCH_FULLTEXT_SNIPPETS, CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH, CFG_WEBSEARCH_IDXPAIRS_FIELDS, CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT, CFG_WEBSEARCH_SYNONYM_KBRS, CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_WEBSEARCH_WILDCARD_LIMIT, CFG_XAPIAN_ENABLED
45:1: F999 redefinition of unused 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' from line 45
45:1: F999 redefinition of unused 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' from line 45
45:1: F999 redefinition of unused 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' from line 45
45:1: F999 redefinition of unused 'CFG_LOGDIR' from line 45
45:1: F999 redefinition of unused 'CFG_WEBSEARCH_WILDCARD_LIMIT' from line 45
45:1: F401 'CFG_XAPIAN_ENABLED' imported but unused
45:1: F401 'CFG_BIBINDEX_CHARS_PUNCTUATION' imported but unused
45:1: F401 'CFG_OAI_ID_FIELD' imported but unused
45:1: F401 'CFG_WEBSEARCH_VIEWRESTRCOLL_POLICY' imported but unused
45:1: F401 'CFG_WEBSEARCH_SYNONYM_KBRS' imported but unused
45:1: F401 'CFG_BIBUPLOAD_SERIALIZE_RECORD_STRUCTURE' imported but unused
45:1: F401 'CFG_WEBSEARCH_DISPLAY_NEAREST_TERMS' imported but unused
45:1: F401 'CFG_WEBSEARCH_FIELDS_CONVERT' imported but unused
45:1: F401 'CFG_ACCESS_CONTROL_LEVEL_ACCOUNTS' imported but unused
45:1: F401 'CFG_SITE_RECORD' imported but unused
45:1: F401 'CFG_BIBRANK_SHOW_DOWNLOAD_GRAPHS' imported but unused
45:1: F401 'CFG_INSPIRE_SITE' imported but unused
45:1: F401 'CFG_LOGDIR' imported but unused
45:1: F401 'CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG' imported but unused
45:1: F401 'CFG_WEBSEARCH_PREV_NEXT_HIT_LIMIT' imported but unused
45:1: F401 'CFG_BASE_URL' imported but unused
45:1: F401 'CFG_BIBFORMAT_HIDDEN_TAGS' imported but unused
45:1: F401 'CFG_WEBSEARCH_WILDCARD_LIMIT' imported but unused
45:1: F401 'CFG_WEBSEARCH_DETAILED_META_FORMAT' imported but unused
45:1: F401 'CFG_BIBSORT_BUCKETS' imported but unused
45:1: F401 'CFG_WEBSEARCH_FULLTEXT_SNIPPETS' imported but unused
45:1: F401 'CFG_SOLR_URL' imported but unused
45:1: F401 'CFG_WEBSEARCH_IDXPAIRS_EXACT_SEARCH' imported but unused
45:1: F401 'CFG_WEBSEARCH_IDXPAIRS_FIELDS' imported but unused
45:1: E402 module level import not at top of file
46:6: E126 continuation line over-indented for hanging indent
89:1: E402 module level import not at top of file
90:1: F401 'get_most_popular_field_values' imported but unused
90:1: E402 module level import not at top of file
91:1: I101 Imported names are in the wrong order. Should be InvenioWebSearchCitedbyLimitError, InvenioWebSearchReferstoLimitError, InvenioWebSearchUnknownCollectionError, InvenioWebSearchWildcardLimitError
91:1: I100 Imports statements are in the wrong order. from invenio.modules.search.errors should be before from invenio.modules.search.utils
91:1: F401 'InvenioWebSearchUnknownCollectionError' imported but unused
91:1: F401 'InvenioWebSearchReferstoLimitError' imported but unused
91:1: F401 'InvenioWebSearchWildcardLimitError' imported but unused
91:1: F401 'InvenioWebSearchCitedbyLimitError' imported but unused
91:1: E402 module level import not at top of file
92:6: E126 continuation line over-indented for hanging indent
96:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.modules.search.errors
96:1: F401 'get_fieldvalues_alephseq_like' imported but unused
96:1: E402 module level import not at top of file
98:1: E402 module level import not at top of file
99:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from .utils
99:1: I201 Missing newline before sections or imports.
99:1: F401 'create_record' imported but unused
99:1: F401 'record_xml_output' imported but unused
99:1: E402 module level import not at top of file
100:1: I101 Imported names are in the wrong order. Should be get_bibrank_methods, is_method_valid, rank_by_citations, rank_records
100:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.record_sorter should be before from invenio.legacy.bibrecord
100:1: F401 'rank_by_citations' imported but unused
100:1: F401 'get_bibrank_methods' imported but unused
100:1: E402 module level import not at top of file
105:1: I101 Imported names are in the wrong order. Should be calculate_reading_similarity_list, register_page_view_event
105:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.downloads_similarity should be before from invenio.legacy.bibrank.record_sorter
105:1: F401 'register_page_view_event' imported but unused
105:1: F401 'calculate_reading_similarity_list' imported but unused
105:1: E402 module level import not at top of file
106:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_stemmer should be before from invenio.legacy.bibrank.downloads_similarity
106:1: F401 'stem' imported but unused
106:1: E402 module level import not at top of file
107:1: F401 'BibIndexDefaultTokenizer' imported but unused
107:1: E402 module level import not at top of file
108:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_utils should be before from invenio.modules.indexer.tokenizers.BibIndexDefaultTokenizer
108:1: F401 'author_name_requires_phrase_search' imported but unused
108:1: F401 'get_field_tags' imported but unused
108:1: E402 module level import not at top of file
110:1: I101 Imported names are in the wrong order. Should be lower_index_term, wash_author_name, wash_index_term
110:1: F401 'wash_index_term' imported but unused
110:1: F401 'wash_author_name' imported but unused
110:1: F401 'lower_index_term' imported but unused
110:1: E402 module level import not at top of file
111:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibindex.engine_config should be before from invenio.legacy.bibindex.engine_washer
111:1: F401 'CFG_BIBINDEX_SYNONYM_MATCH_TYPE' imported but unused
111:1: E402 module level import not at top of file
112:1: F401 'create_download_history_graph_and_box' imported but unused
112:1: E402 module level import not at top of file
113:1: F401 'DataCacher' imported but unused
113:1: E402 module level import not at top of file
114:1: I101 Imported names are in the wrong order. Should be perform_external_collection_search, print_external_results_overview
114:1: F401 'perform_external_collection_search' imported but unused
114:1: F401 'print_external_results_overview' imported but unused
114:1: E402 module level import not at top of file
115:1: E402 module level import not at top of file
116:1: I101 Imported names are in the wrong order. Should be CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS, CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS, VIEWRESTRCOLL
116:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused
116:1: F401 'CFG_ACC_GRANT_AUTHOR_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused
116:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_USERIDS_IN_TAGS' imported but unused
116:1: F401 'CFG_ACC_GRANT_VIEWER_RIGHTS_TO_EMAILS_IN_TAGS' imported but unused
116:1: E402 module level import not at top of file
122:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch.adminlib should be before from invenio.modules.access.local_config
122:1: F401 'get_detailed_page_tabs' imported but unused
122:1: F401 'get_detailed_page_tabs_counts' imported but unused
122:1: E402 module level import not at top of file
123:1: I100 Imports statements are in the wrong order. from intbitset should be before from invenio.legacy.websearch.adminlib
123:1: I201 Missing newline before sections or imports.
123:1: E402 module level import not at top of file
124:1: I201 Missing newline before sections or imports.
124:1: F401 'InvenioDbQueryWildcardLimitError' imported but unused
124:1: E402 module level import not at top of file
125:1: F401 'deserialize_via_marshal' imported but unused
125:1: E402 module level import not at top of file
126:1: I100 Imports statements are in the wrong order. from invenio.modules.access.engine should be before from invenio.utils.serializers
126:1: F401 'acc_authorize_action' imported but unused
126:1: E402 module level import not at top of file
127:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules.access.engine
127:1: E402 module level import not at top of file
128:1: I101 Imported names are in the wrong order. Should be encode_for_xml, strip_accents, wash_for_utf8
128:1: F401 'encode_for_xml' imported but unused
128:1: F401 'wash_for_utf8' imported but unused
128:1: F401 'strip_accents' imported but unused
128:1: E402 module level import not at top of file
129:1: I100 Imports statements are in the wrong order. from invenio.legacy should be before from invenio.utils.text
129:1: E402 module level import not at top of file
131:1: E402 module level import not at top of file
134:1: I101 Imported names are in the wrong order. Should be calculate_cited_by_list, calculate_co_cited_with_list, get_cited_by_list, get_citedby_hitset, get_citers_log, get_records_with_num_cites, get_refers_to_list, get_refersto_hitset
134:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_searcher should be before import invenio.legacy.template
134:1: F401 'get_citedby_hitset' imported but unused
134:1: F401 'get_records_with_num_cites' imported but unused
134:1: F401 'calculate_cited_by_list' imported but unused
134:1: F401 'get_refersto_hitset' imported but unused
134:1: F401 'get_cited_by_list' imported but unused
134:1: F401 'get_citers_log' imported but unused
134:1: F401 'calculate_co_cited_with_list' imported but unused
134:1: F401 'get_refers_to_list' imported but unused
134:1: E402 module level import not at top of file
139:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrank.citation_grapher should be before from invenio.legacy.bibrank.citation_searcher
139:1: F401 'create_citation_history_graph_and_box' imported but unused
139:1: E402 module level import not at top of file
140:1: I101 Imported names are in the wrong order. Should be get_self_cited_by, get_self_cited_by_list, get_self_refers_to_list
140:1: F401 'get_self_refers_to_list' imported but unused
140:1: F401 'get_self_cited_by' imported but unused
140:1: F401 'get_self_cited_by_list' imported but unused
140:1: E402 module level import not at top of file
141:55: E126 continuation line over-indented for hanging indent
144:1: I101 Imported names are in the wrong order. Should be get_table_update_time, run_sql, run_sql_with_limit, wash_table_column_name
144:1: F401 'wash_table_column_name' imported but unused
144:1: F401 'get_table_update_time' imported but unused
144:1: F401 'run_sql_with_limit' imported but unused
144:1: E402 module level import not at top of file
146:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid
146:1: F401 'collect_user_info' imported but unused
146:1: E402 module level import not at top of file
147:1: I101 Imported names are in the wrong order. Should be create_error_box, pagefooteronly, pageheaderonly, write_warning
147:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.legacy.webuser
147:1: F401 'pagefooteronly' imported but unused
147:1: F401 'create_error_box' imported but unused
147:1: F401 'write_warning' imported but unused
147:1: F401 'pageheaderonly' imported but unused
147:1: E402 module level import not at top of file
148:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.legacy.webpage
148:1: E402 module level import not at top of file
150:1: F401 'apache' imported but unused
150:1: E402 module level import not at top of file
152:1: I100 Imports statements are in the wrong order. from invenio.legacy.websearch_external_collections should be before from invenio.utils
152:1: F401 'do_calculate_hosted_collections_results' imported but unused
152:1: F401 'calculate_hosted_collections_results' imported but unused
152:1: E402 module level import not at top of file
153:1: F401 'CFG_EXTERNAL_COLLECTION_MAXRESULTS' imported but unused
153:1: E402 module level import not at top of file
155:1: F401 'DatabaseError' imported but unused
155:1: E402 module level import not at top of file
160:22: E203 whitespace before ':'
160:14: E225 missing whitespace around operator
161:24: E203 whitespace before ':'
162:24: E203 whitespace before ':'
163:28: E203 whitespace before ':'
164:30: E203 whitespace before ':'
165:24: E203 whitespace before ':'
166:23: E203 whitespace before ':'
167:29: E203 whitespace before ':'
168:26: E203 whitespace before ':'
169:33: E203 whitespace before ':'
170:30: E203 whitespace before ':'
171:30: E203 whitespace before ':'
172:30: E203 whitespace before ':'
173:30: E203 whitespace before ':'
174:30: E203 whitespace before ':'
175:30: E203 whitespace before ':'
176:41: E703 statement ends with a semicolon
178:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from sqlalchemy.exc
178:1: E402 module level import not at top of file
179:1: F401 'collection_restricted_p' imported but unused
179:1: E402 module level import not at top of file
180:1: F401 'restricted_collection_cache' imported but unused
180:1: E402 module level import not at top of file
181:1: F401 'get_permitted_restricted_collections' imported but unused
181:1: E402 module level import not at top of file
182:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.utils
182:1: F401 'get_all_restricted_recids' imported but unused
182:1: E402 module level import not at top of file
185:1: F401 'check_user_can_view_record' imported but unused
185:1: E402 module level import not at top of file
187:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.records.access
187:1: E402 module level import not at top of file
188:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.collections.cache
188:1: E402 module level import not at top of file
189:1: F401 'get_field_i18nname' imported but unused
189:1: E402 module level import not at top of file
191:1: I100 Imports statements are in the wrong order. from invenio.modules.indexer.models should be before from invenio.modules.search.cache
191:1: E402 module level import not at top of file
197: D401 First line should be in imperative mood ('Create', not 'Creates')
197: D202 No blank lines allowed after function docstring (found 1)
197: D400 First line should end with a period (not 'n')
197: D205 1 blank line required between summary line and description (found 0)
205:33: E261 at least two spaces before inline comment
214:1: F401 'get_synonym_terms' imported but unused
214:1: E402 module level import not at top of file
216:1: F403 'from invenio.modules.search.washers import *' used; unable to detect undefined names
216:1: E402 module level import not at top of file
219: D401 First line should be in imperative mood ('Return', not 'Returns')
219: D300 Use """triple double quotes""" (found '-quotes)
240:1: I100 Imports statements are in the wrong order. from invenio.modules.collections.cache should be before from invenio.modules.search.washers
240:1: F401 'get_collection_allchildren' imported but unused
240:1: E402 module level import not at top of file
243: D401 First line should be in imperative mood ('Return', not 'Returns')
243: D202 No blank lines allowed after function docstring (found 1)
246:5: E266 too many leading '#' for block comment
250:5: E266 too many leading '#' for block comment
251:5: E266 too many leading '#' for block comment
253:34: E261 at least two spaces before inline comment
270:13: B901 blind except: statement
275:9: E266 too many leading '#' for block comment
287:9: E266 too many leading '#' for block comment
292:17: E266 too many leading '#' for block comment
301: D400 First line should end with a period (not 'o')
301: D205 1 blank line required between summary line and description (found 0)
301: D208 Docstring is over-indented
346:1: F403 'from invenio.modules.search.searchext.engines.native import *' used; unable to detect undefined names
346:1: E402 module level import not at top of file
349: D209 Multi-line docstring closing quotes should be on a separate line
349: D400 First line should end with a period (not 'y')
349: D205 1 blank line required between summary line and description (found 0)
349: D208 Docstring is over-indented
362:38: E203 whitespace before ','
393:1: E302 expected 2 blank lines, found 0
393: D209 Multi-line docstring closing quotes should be on a separate line
393: D400 First line should end with a period (not 'g')
393: D205 1 blank line required between summary line and description (found 0)
393: D208 Docstring is over-indented
422: D209 Multi-line docstring closing quotes should be on a separate line
422: D205 1 blank line required between summary line and description (found 0)
434:1: I100 Imports statements are in the wrong order. from invenio.modules.search.models should be before from invenio.modules.search.searchext.engines.native
434:1: E402 module level import not at top of file
439: D210 No whitespaces allowed surrounding docstring text
439: D400 First line should end with a period (not 'h')
439: D205 1 blank line required between summary line and description (found 0)
457: D401 First line should be in imperative mood ('I', not 'Is')
457: D400 First line should end with a period (not '?')
468:1: E302 expected 2 blank lines, found 1
468: D205 1 blank line required between summary line and description (found 0)
468: D208 Docstring is over-indented
475:1: I101 Imported names are in the wrong order. Should be CACHE_SORTED_DATA, SORTING_METHODS
475:1: F401 'CACHE_SORTED_DATA' imported but unused
475:1: E402 module level import not at top of file
478: D205 1 blank line required between summary line and description (found 0)
478: D208 Docstring is over-indented
492:20: E127 continuation line over-indented for visual indent
494:25: E126 continuation line over-indented for hanging indent
545: D103 Missing docstring in public function
555: D401 First line should be in imperative mood ('calculate', not 'calculates')
555: D209 Multi-line docstring closing quotes should be on a separate line
555: D202 No blank lines allowed after function docstring (found 1)
555: D400 First line should end with a period (not 'e')
555: D205 1 blank line required between summary line and description (found 0)
566:20: E261 at least two spaces before inline comment
570:17: E261 at least two spaces before inline comment
597:1: E302 expected 2 blank lines, found 1
615: D210 No whitespaces allowed surrounding docstring text
615: D400 First line should end with a period (not 'n')
621:25: E261 at least two spaces before inline comment
622:13: E222 multiple spaces after operator
645:10: E261 at least two spaces before inline comment
662:1: E266 too many leading '#' for block comment
664: D103 Missing docstring in public function
665:25: E128 continuation line under-indented for visual indent
666:25: E128 continuation line under-indented for visual indent
667:25: E128 continuation line under-indented for visual indent
668:25: E128 continuation line under-indented for visual indent
685: D401 First line should be in imperative mood ('Set', not 'Sets')
685: D202 No blank lines allowed after function docstring (found 1)
685: D400 First line should end with a period (not 'l')
685: D200 One-line docstring should fit on one line with quotes (found 3)
686:23: E128 continuation line under-indented for visual indent
687:23: E128 continuation line under-indented for visual indent
688:23: E128 continuation line under-indented for visual indent
689:23: E128 continuation line under-indented for visual indent
690:23: E128 continuation line under-indented for visual indent
728:9: B901 blind except: statement


34:1: I100 Imports statements are in the wrong order. from random should be before from invenio.utils.hash


1: M111 unrecognized component name: legacy


30:1: I201 Missing newline before sections or imports.
31:1: I100 Imports statements are in the wrong order. import logging should be before from six
31:1: I201 Missing newline before sections or imports.
33:1: I201 Missing newline before sections or imports.
34:1: I101 Imported names are in the wrong order. Should be current_app, has_app_context
34:1: I100 Imports statements are in the wrong order. from flask should be before import six
34:1: I201 Missing newline before sections or imports.
35:1: I100 Imports statements are in the wrong order. from functools should be before from flask
35:1: I201 Missing newline before sections or imports.
35:1: F401 'wraps' imported but unused
36:1: I101 Imported names are in the wrong order. Should be find_modules, import_string
36:1: I201 Missing newline before sections or imports.
111:13: B901 blind except: statement
224:1: E302 expected 2 blank lines, found 1
228: D102 Missing docstring in public method
231: D102 Missing docstring in public method
239: D102 Missing docstring in public method
242: D102 Missing docstring in public method
246: D401 First line should be in imperative mood ('Run', not 'Runs')
298:23: E222 multiple spaces after operator


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: legacy


26:1: F401 'CFG_DEVEL_SITE' imported but unused
80: D102 Missing docstring in public method
109: D209 Multi-line docstring closing quotes should be on a separate line
109: D203 1 blank line required before class docstring (found 0)
109: D400 First line should end with a period (not 'f')
109: D205 1 blank line required between summary line and description (found 0)
115: D102 Missing docstring in public method
169: D102 Missing docstring in public method
177: D210 No whitespaces allowed surrounding docstring text
177: D209 Multi-line docstring closing quotes should be on a separate line
177: D203 1 blank line required before class docstring (found 0)
177: D400 First line should end with a period (not 'l')
177: D205 1 blank line required between summary line and description (found 0)
205: D102 Missing docstring in public method
229: D102 Missing docstring in public method


1: M100 needs more reviewers
1: M101 signature is missing
1: M111 unrecognized component name: elasticsearch


Is `testKit` really the name we want to use for this? Seems so generic.


SortBy should work. Can you double check the lodash version on the server?


I might replace "Building" with "The build system." I might avoid mentioning `gn` here, because it does immediately make the reader ask "why aren't we using that?" (to which the answer is "they have no interest in making it available separate from the Chromium source" — and then we get down into a rabbit hole!)

A quick one-sentence "Node.js requires a build system because large parts of it — and its dependencies — are written in languages that require compilation."


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2013
23:1: I201 Missing newline before sections or imports.
28: D209 Multi-line docstring closing quotes should be on a separate line
28: D202 No blank lines allowed after function docstring (found 1)
37:5: B901 blind except: statement
38:9: F401 'pickle' imported but unused
83: D400 First line should end with a period (not 'y')
97: D103 Missing docstring in public function


30:1: I100 Imports statements are in the wrong order. from invenio.base.utils should be before import pkg_resources


52:1: I101 Imported names are in the wrong order. Should be current_app, jsonify, request
52:1: I100 Imports statements are in the wrong order. from flask should be before import requests
53:1: I201 Missing newline before sections or imports.
54:1: I201 Missing newline before sections or imports.
56:1: I100 Imports statements are in the wrong order. from invenio.utils.xmlhelpers should be before from lxml.etree


52:1: I100 Imports statements are in the wrong order. from urlparse should be before import requests
52:1: I201 Missing newline before sections or imports.
54:1: I101 Imported names are in the wrong order. Should be current_app, jsonify, request
55:1: I201 Missing newline before sections or imports.


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2014
23:1: I201 Missing newline before sections or imports.
26: D203 1 blank line required before class docstring (found 0)


33:1: I201 Missing newline before sections or imports.
37:1: I100 Imports statements are in the wrong order. from urlparse should be before from sqlalchemy.event


4: L102 copyright year is outdated, expected 2015 but got 2014
27:1: I100 Imports statements are in the wrong order. from datetime should be before from bagit
27:1: I201 Missing newline before sections or imports.
28:1: I201 Missing newline before sections or imports.
30:1: I101 Imported names are in the wrong order. Should be copydir, copyfile
32:1: I201 Missing newline before sections or imports.
34:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug.utils


4: L102 copyright year is outdated, expected 2015 but got 2014


4: L102 copyright year is outdated, expected 2015 but got 2014
24:1: I100 Imports statements are in the wrong order. from invenio.ext.sqlalchemy should be before from sqlalchemy.orm.exc
26:1: I201 Missing newline before sections or imports.
27:1: I100 Imports statements are in the wrong order. from invenio_records.api should be before from invenio_records.models


4: L102 copyright year is outdated, expected 2015 but got 2014
22:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, run_test_suite


4: L102 copyright year is outdated, expected 2015 but got 2014
21:1: I101 Imported names are in the wrong order. Should be InvenioTestCase, make_test_suite, run_test_suite
23:1: I101 Imported names are in the wrong order. Should be MultipleResultsFound, NoResultFound


1: D200 One-line docstring should fit on one line with quotes (found 3)
4: L102 copyright year is outdated, expected 2015 but got 2013
32: D203 1 blank line required before class docstring (found 0)
32: D204 1 blank line required after class docstring (found 0)
34: D102 Missing docstring in public method
38:17: E128 continuation line under-indented for visual indent
39:17: E128 continuation line under-indented for visual indent
41:17: E128 continuation line under-indented for visual indent
43:17: E128 continuation line under-indented for visual indent
45:17: E128 continuation line under-indented for visual indent
50:9: E128 continuation line under-indented for visual indent
52:9: E128 continuation line under-indented for visual indent
55:1: E302 expected 2 blank lines, found 1
55: D203 1 blank line required before class docstring (found 0)
55: D204 1 blank line required after class docstring (found 0)
57: D102 Missing docstring in public method
61:17: E128 continuation line under-indented for visual indent
62:17: E128 continuation line under-indented for visual indent
64:17: E128 continuation line under-indented for visual indent
66:17: E128 continuation line under-indented for visual indent
68:17: E128 continuation line under-indented for visual indent
70:17: E128 continuation line under-indented for visual indent
72:17: E128 continuation line under-indented for visual indent
75:1: E302 expected 2 blank lines, found 1
75: D203 1 blank line required before class docstring (found 0)
75: D204 1 blank line required after class docstring (found 0)
77: D102 Missing docstring in public method
81:17: E128 continuation line under-indented for visual indent
82:22: E128 continuation line under-indented for visual indent
84:17: E128 continuation line under-indented for visual indent
85:17: E128 continuation line under-indented for visual indent
87:17: E128 continuation line under-indented for visual indent
88:17: E128 continuation line under-indented for visual indent
93:17: E128 continuation line under-indented for visual indent
95:17: E128 continuation line under-indented for visual indent
97:17: E128 continuation line under-indented for visual indent
99:17: E128 continuation line under-indented for visual indent
101:17: E128 continuation line under-indented for visual indent
103:17: E128 continuation line under-indented for visual indent
103:31: E231 missing whitespace after ','
106: D203 1 blank line required before class docstring (found 0)
106: D204 1 blank line required after class docstring (found 0)
108: D102 Missing docstring in public method
112:17: E128 continuation line under-indented for visual indent
113:17: E128 continuation line under-indented for visual indent
115:17: E128 continuation line under-indented for visual indent
116:17: E126 continuation line over-indented for hanging indent
119:17: E128 continuation line under-indented for visual indent
120:17: E128 continuation line under-indented for visual indent
122:17: E128 continuation line under-indented for visual indent
123:17: E128 continuation line under-indented for visual indent
125:17: E128 continuation line under-indented for visual indent
126:17: E128 continuation line under-indented for visual indent
128:17: E128 continuation line under-indented for visual indent
129:17: E128 continuation line under-indented for visual indent
131:17: E128 continuation line under-indented for visual indent
133:17: E128 continuation line under-indented for visual indent
135:17: E128 continuation line under-indented for visual indent
137:17: E128 continuation line under-indented for visual indent
139:17: E128 continuation line under-indented for visual indent
141:17: E128 continuation line under-indented for visual indent
143:17: E128 continuation line under-indented for visual indent
145:17: E128 continuation line under-indented for visual indent
150:17: E128 continuation line under-indented for visual indent
161: D203 1 blank line required before class docstring (found 0)
161: D204 1 blank line required after class docstring (found 0)
163: D102 Missing docstring in public method
167:17: E128 continuation line under-indented for visual indent
168:17: E128 continuation line under-indented for visual indent
170:17: E128 continuation line under-indented for visual indent
172:17: E128 continuation line under-indented for visual indent
173:17: E128 continuation line under-indented for visual indent
175:17: E128 continuation line under-indented for visual indent
177:9: E128 continuation line under-indented for visual indent
180:17: E128 continuation line under-indented for visual indent
182:17: E128 continuation line under-indented for visual indent
184:17: E128 continuation line under-indented for visual indent
186:17: E128 continuation line under-indented for visual indent
187:17: E128 continuation line under-indented for visual indent
189:17: E128 continuation line under-indented for visual indent
191:17: E128 continuation line under-indented for visual indent
197:1: E302 expected 2 blank lines, found 1
197: D203 1 blank line required before class docstring (found 0)
197: D204 1 blank line required after class docstring (found 0)
199: D102 Missing docstring in public method
203:17: E128 continuation line under-indented for visual indent
204:17: E128 continuation line under-indented for visual indent
206:17: E128 continuation line under-indented for visual indent
208:17: E128 continuation line under-indented for visual indent
209:17: E128 continuation line under-indented for visual indent
211:17: E128 continuation line under-indented for visual indent
213:17: E128 continuation line under-indented for visual indent
214:17: E128 continuation line under-indented for visual indent
216:17: E128 continuation line under-indented for visual indent
217:17: E128 continuation line under-indented for visual indent
219:17: E128 continuation line under-indented for visual indent
222:17: E128 continuation line under-indented for visual indent
227:1: E302 expected 2 blank lines, found 1
227: D203 1 blank line required before class docstring (found 0)
227: D204 1 blank line required after class docstring (found 0)
229: D102 Missing docstring in public method
233:17: E128 continuation line under-indented for visual indent
234:17: E128 continuation line under-indented for visual indent
236:17: E128 continuation line under-indented for visual indent
238:17: E128 continuation line under-indented for visual indent
240:17: E128 continuation line under-indented for visual indent
242:17: E128 continuation line under-indented for visual indent
245:1: E302 expected 2 blank lines, found 1
245: D203 1 blank line required before class docstring (found 0)
245: D204 1 blank line required after class docstring (found 0)
247: D102 Missing docstring in public method
251:17: E128 continuation line under-indented for visual indent
252:17: E128 continuation line under-indented for visual indent
254:17: E128 continuation line under-indented for visual indent
255:17: E128 continuation line under-indented for visual indent
257:17: E128 continuation line under-indented for visual indent
259:17: E128 continuation line under-indented for visual indent
261:17: E128 continuation line under-indented for visual indent
263:17: E128 continuation line under-indented for visual indent
265:17: E128 continuation line under-indented for visual indent


1: D100 Missing docstring in public module
27: D103 Missing docstring in public function


24:1: F401 'intbitset' imported but unused
25:1: I201 Missing newline before sections or imports.
25:1: F401 'cached_property' imported but unused
27:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from werkzeug
75: D101 Missing docstring in public class
76: D102 Missing docstring in public method
103: D103 Missing docstring in public function
121: D203 1 blank line required before class docstring (found 0)
121: D204 1 blank line required after class docstring (found 0)
121: D400 First line should end with a period (not 'e')
121: D205 1 blank line required between summary line and description (found 0)
126: D102 Missing docstring in public method


23:1: I100 Imports statements are in the wrong order. from invenio.modules.search.api should be before from invenio_records.signals
23:1: I201 Missing newline before sections or imports.
44:32: E711 comparison to None should be 'if cond is not None:'
67: D103 Missing docstring in public function


26:1: I201 Missing newline before sections or imports.


23:1: I100 Imports statements are in the wrong order. import socket should be before from datetime
25:1: I101 Imported names are in the wrong order. Should be Blueprint, abort, current_app, flash, g, redirect, render_template, request, url_for
29:1: I201 Missing newline before sections or imports.
30:1: I201 Missing newline before sections or imports.
33:1: I100 Imports statements are in the wrong order. from invenio.base.decorators should be before from invenio.base.i18n
42:1: I100 Imports statements are in the wrong order. from invenio.utils.mail should be before from invenio_records.views
45:1: I101 Imported names are in the wrong order. Should be CmtACTIONHISTORY, CmtRECORDCOMMENT, CmtSUBSCRIPTION
54: D103 Missing docstring in public function
67: D101 Missing docstring in public class
69: D102 Missing docstring in public method
74: D102 Missing docstring in public method
78: D102 Missing docstring in public method
92: D102 Missing docstring in public method
101: D102 Missing docstring in public method
108: D102 Missing docstring in public method
121: D103 Missing docstring in public function
155:9: B901 blind except: statement
166: D103 Missing docstring in public function
180:9: B901 blind except: statement
255: D103 Missing docstring in public function
274: D103 Missing docstring in public function
294: D103 Missing docstring in public function
316: D103 Missing docstring in public function
324:5: B901 blind except: statement
332: D103 Missing docstring in public function
346:5: B901 blind except: statement
360: D103 Missing docstring in public function


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2014
23:1: I100 Imports statements are in the wrong order. from invenio_records.api should be before import six
24:1: I201 Missing newline before sections or imports.
27: D400 First line should end with a period (not 't')
27: D200 One-line docstring should fit on one line with quotes (found 3)
52: D400 First line should end with a period (not 's')
52: D200 One-line docstring should fit on one line with quotes (found 3)
64: D200 One-line docstring should fit on one line with quotes (found 3)
83: D400 First line should end with a period (not 's')
83: D200 One-line docstring should fit on one line with quotes (found 3)


1: D100 Missing docstring in public module
4: L102 copyright year is outdated, expected 2015 but got 2014
22:1: I100 Imports statements are in the wrong order. from invenio.utils.datacite should be before from werkzeug
22:1: I201 Missing newline before sections or imports.
24:1: I201 Missing newline before sections or imports.
25:1: I100 Imports statements are in the wrong order. from invenio.utils should be before from invenio_records.api
25:1: I201 Missing newline before sections or imports.
58: D102 Missing docstring in public method
61: D102 Missing docstring in public method
74: D102 Missing docstring in public method
78: D102 Missing docstring in public method
110: D102 Missing docstring in public method
122: D102 Missing docstring in public method
148: D103 Missing docstring in public function
177: D103 Missing docstring in public function
245: D103 Missing docstring in public function
280: D103 Missing docstring in public function


25:1: I201 Missing newline before sections or imports.
26:1: I100 Imports statements are in the wrong order. import dateutil.parser should be before import dictdiffer
26:1: I201 Missing newline before sections or imports.
28:1: I100 Imports statements are in the wrong order. from tempfile should be before import dateutil.parser
29:1: I100 Imports statements are in the wrong order. from functools should be before from tempfile
30:1: I101 Imported names are in the wrong order. Should be abort, current_app, request
30:1: I201 Missing newline before sections or imports.
31:1: I201 Missing newline before sections or imports.
32:1: I100 Imports statements are in the wrong order. from functools should be before from flask_login
32:1: I201 Missing newline before sections or imports.
35:1: I100 Imports statements are in the wrong order. from invenio.modules.editor.models should be before from invenio_records.api
35:1: I201 Missing newline before sections or imports.
36:1: I101 Imported names are in the wrong order. Should be Agent, Deposition, DepositionDraftCacheManager
36:1: I100 Imports statements are in the wrong order. from invenio.modules.deposit.models should be before from invenio.modules.editor.models
38:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.modules.deposit.models
41:1: I101 Imported names are in the wrong order. Should be deposition_record, make_record, record_to_draft
41:1: I201 Missing newline before sections or imports.
43:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibdocfile.api should be before from .helpers


28:1: I201 Missing newline before sections or imports.