1: D400 First line should end with '.', not 'o'
1: D200 One-line docstring should not occupy 3 lines
2: L102 copyright year is outdated, expected 2015 but got 2014
24:1: I100 Imports statements are in the wrong order. import cgi should be before import time
27:1: I100 Imports statements are in the wrong order. import sys should be before import urllib
28:1: I100 Imports statements are in the wrong order. import string should be before import sys
31:1: I201 Missing newline before sections or imports.
32:1: I101 Imported names are in the wrong order. Should be CFG_BASE_URL, CFG_INSPIRE_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_VERSION, CFG_WEBLINKBACK_TRACKBACK_ENABLED, CFG_WEBSTYLE_TEMPLATE_SKIN
32:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.ext.template
32:1: F401 'CFG_SITE_SECURE_URL' imported but unused
32:1: F401 'CFG_SITE_RECORD' imported but unused
32:1: F401 'CFG_WEBLINKBACK_TRACKBACK_ENABLED' imported but unused
32:1: F401 'CFG_VERSION' imported but unused
32:1: F401 'CFG_WEBSTYLE_TEMPLATE_SKIN' imported but unused
33:6: E126 continuation line over-indented for hanging indent
46:1: I101 Imported names are in the wrong order. Should be gettext_set_language, is_language_rtl, language_list_long
46:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
46:1: F401 'is_language_rtl' imported but unused
47:1: I101 Imported names are in the wrong order. Should be create_html_link, get_canonical_and_alternates_urls, make_canonical_urlargd
47:1: F401 'get_canonical_and_alternates_urls' imported but unused
48:30: E126 continuation line over-indented for hanging indent
49:1: I100 Imports statements are in the wrong order. from invenio.utils.date should be before from invenio.utils.url
49:1: F401 'convert_datestruct_to_dategui' imported but unused
50:31: E126 continuation line over-indented for hanging indent
51:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.utils.date
53:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.html
57: D101 Docstring missing
59: D202 No blank lines allowed *after* method docstring, found 1
59: D400 First line should end with '.', not 'y'
59: D208 Docstring is over-indented
111: D401 First line should be imperative: 'Create', not 'Creates'
111: D400 First line should end with '.', not 'e'
111: D208 Docstring is over-indented
200: D401 First line should be imperative: 'Create', not 'Creates'
200: D202 No blank lines allowed *after* method docstring, found 1
200: D400 First line should end with '.', not 'r'
200: D208 Docstring is over-indented
261: D401 First line should be imperative: 'Create', not 'Creates'
261: D400 First line should end with '.', not 'r'
261: D208 Docstring is over-indented
290: D202 No blank lines allowed *after* method docstring, found 1
290: D400 First line should end with '.', not 'e'
290: D205 Blank line missing between one-line summary and description
290: D208 Docstring is over-indented
327:18: E126 continuation line over-indented for hanging indent
333: D401 First line should be imperative: 'Produce', not 'Produces'
333: D202 No blank lines allowed *after* method docstring, found 1
333: D208 Docstring is over-indented
369:13: B901 blind except: statement
369:20: E261 at least two spaces before inline comment
389:21: B901 blind except: statement
391:34: E261 at least two spaces before inline comment
441:17: E126 continuation line over-indented for hanging indent
441:28: E203 whitespace before ':'
444:37: E502 the backslash is redundant between brackets
445:24: E131 continuation line unaligned for hanging indent
447:24: E203 whitespace before ':'
448:24: E203 whitespace before ':'
449:31: E502 the backslash is redundant between brackets
449:28: E203 whitespace before ':'
450:21: E131 continuation line unaligned for hanging indent
451:90: E225 missing whitespace around operator
452:28: E203 whitespace before ':'
453:28: E203 whitespace before ':'
454:28: E203 whitespace before ':'
455:28: E203 whitespace before ':'
456:28: E203 whitespace before ':'
457:28: E203 whitespace before ':'
458:28: E203 whitespace before ':'
459:28: E203 whitespace before ':'
460:29: E203 whitespace before ':'
461:62: E502 the backslash is redundant between brackets
461:28: E203 whitespace before ':'
461:37: E225 missing whitespace around operator
462:34: E131 continuation line unaligned for hanging indent
462:67: E502 the backslash is redundant between brackets
463:34: E131 continuation line unaligned for hanging indent
464:15: E126 continuation line over-indented for hanging indent
467: D401 First line should be imperative: 'Print', not 'Prints'
472:53: E251 unexpected spaces around keyword / parameter equals
472:55: E251 unexpected spaces around keyword / parameter equals
472:78: E251 unexpected spaces around keyword / parameter equals
472:80: E251 unexpected spaces around keyword / parameter equals
532:41: E231 missing whitespace after ':'
533:41: E231 missing whitespace after ':'
534:42: E231 missing whitespace after ':'
537:41: E231 missing whitespace after ':'
538:39: E231 missing whitespace after ':'
539:41: E231 missing whitespace after ':'
540:42: E231 missing whitespace after ':'
549:9: E303 too many blank lines (2)
571:9: E303 too many blank lines (2)
583:30: E231 missing whitespace after ':'
584:38: E231 missing whitespace after ':'
589: D401 First line should be imperative: 'Print', not 'Prints'
619:69: E502 the backslash is redundant between brackets
623:41: E231 missing whitespace after ','
633:21: E203 whitespace before ':'
634:19: E203 whitespace before ':'
635:17: E126 continuation line over-indented for hanging indent
644:5: E303 too many blank lines (2)
644: D401 First line should be imperative: 'Display', not 'Displays'
644: D400 First line should end with '.', not 'd'
644: D205 Blank line missing between one-line summary and description
644: D208 Docstring is over-indented
662:9: F841 local variable '_' is assigned to but never used
685:9: E122 continuation line missing indentation or outdented
686:9: E122 continuation line missing indentation or outdented
686:13: E231 missing whitespace after ':'
687:9: E122 continuation line missing indentation or outdented
687:16: E231 missing whitespace after ':'
688:9: E122 continuation line missing indentation or outdented
689:9: E122 continuation line missing indentation or outdented
689:18: E231 missing whitespace after ':'
690:9: E122 continuation line missing indentation or outdented
705:21: E203 whitespace before ':'
706:22: E203 whitespace before ':'
707:22: E203 whitespace before ':'
708:21: E203 whitespace before ':'
709:93: E203 whitespace before ':'
713: D401 First line should be imperative: 'Produce', not 'Produces'
713: D202 No blank lines allowed *after* method docstring, found 1
713: D400 First line should end with '.', not 't'
725:9: F841 local variable '_' is assigned to but never used
729: D401 First line should be imperative: 'Return', not 'Returns'
729: D202 No blank lines allowed *after* method docstring, found 1


1: D100 Docstring missing
2: L102 copyright year is outdated, expected 2015 but got 2014
23:1: I100 Imports statements are in the wrong order. import errno should be before import os
25:1: I100 Imports statements are in the wrong order. import cgi should be before import time
27:1: I100 Imports statements are in the wrong order. import shutil should be before import sys
30:1: I100 Imports statements are in the wrong order. from collections should be before from urllib
33:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_CERN_SITE, CFG_PREFIX, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSUBMIT_STORAGEDIR
33:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibrecord
34:6: E126 continuation line over-indented for hanging indent
42:1: I101 Imported names are in the wrong order. Should be CrossrefError, get_marcxml_for_doi
43:1: I100 Imports statements are in the wrong order. from invenio.utils should be before from invenio.utils.crossref
44:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.utils
46:1: I100 Imports statements are in the wrong order. from invenio.modules.access.control should be before from invenio.modules.access.engine
47:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.modules.access.control
48:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid, isGuestUser, page_not_authorized
49:29: E126 continuation line over-indented for hanging indent
51:1: I101 Imported names are in the wrong order. Should be WebInterfaceDirectory, wash_urlargd
51:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.modules.records.access
53:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.url
54:1: I101 Imported names are in the wrong order. Should be decompose_file, propose_next_docname, stream_file
56:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.bibdocfile.api
58:1: I101 Imported names are in the wrong order. Should be InvenioWebSubmitIconCreatorError, create_icon
58:1: I100 Imports statements are in the wrong order. from invenio.legacy.websubmit.icon_creator should be before from invenio.utils.html
59:1: I100 Imports statements are in the wrong order. from invenio.legacy.ckeditor.connector should be before from invenio.legacy.websubmit.icon_creator
62:1: F401 'get_detailed_page_tabs' imported but unused
63:1: I101 Imported names are in the wrong order. Should be CFG_JSON_AVAILABLE, json
64:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.json
65:1: I100 Imports statements are in the wrong order. from flask should be before import invenio.legacy.template
65:1: I201 Missing newline before sections or imports.
70:1: I101 Imported names are in the wrong order. Should be action, endaction, home, interface, makeCataloguesTable
72:1: E302 expected 2 blank lines, found 1
72: D101 Docstring missing
74:77: E502 the backslash is redundant between brackets
75:79: E502 the backslash is redundant between brackets
78: D400 First line should end with '.', not 'r'
78: D205 Blank line missing between one-line summary and description
105:16: E127 continuation line over-indented for visual indent
127:13: B901 blind except: statement
134:9: B901 blind except: statement
141:61: E225 missing whitespace around operator
192:25: F841 local variable 'n' is assigned to but never used
211:34: E201 whitespace after '{'
211:57: E203 whitespace before ':'
212:57: E203 whitespace before ':'
212:69: E261 at least two spaces before inline comment
213:57: E203 whitespace before ':'
214:57: E203 whitespace before ':'
215:57: E203 whitespace before ':'
216:57: E203 whitespace before ':'
216:67: E261 at least two spaces before inline comment
217:57: E203 whitespace before ':'
251:5: E303 too many blank lines (2)
251: D202 No blank lines allowed *after* method docstring, found 1
251: D205 Blank line missing between one-line summary and description
258: D290 White spaces around docstring should be trimmed.
258: D400 First line should end with '.', not 'm'
281:16: E127 continuation line over-indented for visual indent
303:13: B901 blind except: statement
304:17: F841 local variable 'act' is assigned to but never used
373:63: E127 continuation line over-indented for visual indent
416:41: E128 continuation line under-indented for visual indent
417:41: E128 continuation line under-indented for visual indent
418:41: E128 continuation line under-indented for visual indent
422:25: B901 blind except: statement
456:26: E703 statement ends with a semicolon
535: D205 Blank line missing between one-line summary and description
547:16: E127 continuation line over-indented for visual indent
548:16: E127 continuation line over-indented for visual indent
556:9: E303 too many blank lines (2)
569:9: B901 blind except: statement
586:19: E126 continuation line over-indented for hanging indent
590:35: E231 missing whitespace after ','
593:30: E201 whitespace after '{'
593:53: E203 whitespace before ':'
594:53: E203 whitespace before ':'
595:53: E203 whitespace before ':'
596:53: E203 whitespace before ':'
597:53: E203 whitespace before ':'
598:53: E203 whitespace before ':'
598:63: E261 at least two spaces before inline comment
599:53: E203 whitespace before ':'
612:64: F841 local variable 'e' is assigned to but never used
625: D401 First line should be imperative: 'Thi', not 'This'
625: D290 White spaces around docstring should be trimmed.
625: D400 First line should end with '.', not 'g'
625: D205 Blank line missing between one-line summary and description
633:26: E261 at least two spaces before inline comment
634:32: E261 at least two spaces before inline comment
635:33: E113 unexpected indentation
638:47: E261 at least two spaces before inline comment
652: D401 First line should be imperative: 'Return', not 'Returns'
652: D400 First line should end with '.', not 'e'
652: D205 Blank line missing between one-line summary and description
663:9: F841 local variable 'uid' is assigned to but never used
664:9: F841 local variable 'user_info' is assigned to but never used
669:74: E502 the backslash is redundant between brackets
670:62: E502 the backslash is redundant between brackets
670:39: E221 multiple spaces before operator
681: D200 One-line docstring should not occupy 3 lines
709:44: E203 whitespace before ':'
724:9: F841 local variable 'myQuery' is assigned to but never used
755:36: E126 continuation line over-indented for hanging indent
765:22: E211 whitespace before '('
781: D400 First line should end with '.', not ')'
789:37: W602 deprecated form of raising exception
807:32: E203 whitespace before ':'
815: D102 Docstring missing
828:9: F841 local variable 't' is assigned to but never used
829:15: E221 multiple spaces before operator
840:9: F841 local variable 'nbFields' is assigned to but never used
846:24: E126 continuation line over-indented for hanging indent
846:35: E203 whitespace before ':'
847:31: E203 whitespace before ':'
848:30: E203 whitespace before ':'
849:30: E203 whitespace before ':'
850:23: E126 continuation line over-indented for hanging indent
852:59: E231 missing whitespace after ','
855:47: E231 missing whitespace after ','
856:52: E231 missing whitespace after ','
863:18: E126 continuation line over-indented for hanging indent
863:20: E251 unexpected spaces around keyword / parameter equals
863:22: E251 unexpected spaces around keyword / parameter equals
864:24: E251 unexpected spaces around keyword / parameter equals
864:26: E251 unexpected spaces around keyword / parameter equals
865:16: E126 continuation line over-indented for hanging indent
867: D401 First line should be imperative: 'Return', not 'Returns'
867: D200 One-line docstring should not occupy 3 lines
899: D102 Docstring missing
932:45: E127 continuation line over-indented for visual indent
941:62: E203 whitespace before ':'
941:109: E203 whitespace before ':'
941:129: E203 whitespace before ','
942:48: E126 continuation line over-indented for hanging indent
945:41: E128 continuation line under-indented for visual indent
950:62: E203 whitespace before ':'
950:109: E203 whitespace before ':'
950:129: E203 whitespace before ','
951:48: E126 continuation line over-indented for hanging indent
954:41: E128 continuation line under-indented for visual indent
964:62: E203 whitespace before ':'
964:109: E203 whitespace before ':'
964:129: E203 whitespace before ','
965:45: E126 continuation line over-indented for hanging indent
974:27: E225 missing whitespace around operator


I added this as another possible way of parsing the json, I haven't done any testing on it yet, but plan to do that with a unit test soon.


Indeed, much better.
Please review https://github.com/Geoportail-Luxembourg/geoportailv3/pull/192


Oops yeah that must go, thanks...

On Tue, Jan 27, 2015 at 5:37 AM, Akbar Gumbira <notifications@github.com>
wrote:

>  import pydevd
> pydevd.settrace('localhost', port=5678, stdoutToServer=True, stderrToServer=True)
>
> That will throw Python Error to end user (which we expect don't have
> pydevd on his/her QGIS) on loading QGIS that will result InaSAFE won't get
> loaded :)
>
> *I just tried it. For debugging purpose, I set PYTHONPATH to add
> pycharm-debug.egg on the script to load QGIS. I deleted it, and it indeed
> threw Python Error:
> [image: python error]
> <https://cloud.githubusercontent.com/assets/992519/5913087/696eee74-a610-11e4-9f34-abae44f1e43d.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/timlinux/inasafe/commit/7d338e46fefd4ba1efa7341a13d45edd9937c790#commitcomment-9452501>
> .
>

--
------------------------------------------------------------------------------------------
Tim Sutton
Visit http://kartoza.com to find out about open source:
 * Desktop GIS programming services
 * Geospatial web development
* GIS Training
* Consulting Services
Skype: timlinux Irc: timlinux on #qgis at freenode.net
Tim is a member of the QGIS Project Steering Committee
-------------------------------------------------------------------------------------------
Kartoza is a merger between Linfiniti and Afrispatial


@rgpass


This is no longer require for the Working with Custom Components demo?? Why?


Mostly I think these kinds of variable reductions are great.

But in cases like this, I'm not 100% sure it's a good idea. The variable names themselves serve as a form of documentation for what's going on. Even if it's redundant from the perspective of compilation, fully removing the distinction between `maskshift` and `pixelshiftdelta` here may not be wise, from the perspective of future maintenance. Who knows, maybe there is a variant in the spec we didn't support, where these two values are not always equal?


This commit breaks the unit tests. Particularly, ``LoggerMock``, ``LoggerMockTest``, ``AbstractARAClientLoggingTest``, and ``AbstractARAClientTest``. We have to evaluate if we want to test logging to a certain degree or not at all (since its now a third-party library).


Integer 20 is hard coded.


I guess it's not worth the pain of manually approving some whitelist of "acceptable violations"... compared to the huge benefit gained from the automated removal of such things.

It just makes me a tiny bit worried, that's all. It's probably fine.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.3.0.939](http://build.osmsharp.com:8080/viewLog.html?buildId=4190&buildTypeId=bt16) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 503 Build time: 0:0:0
### Failed tests
```
OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.Serialization.RoutingSerializationTests.RoutingSerializationFlatfileCHEdgeData: Test(s) failed.   Expected: True
  But was:  False
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

OsmSharp.Test.Unittests.dll: OsmSharp.Test.Unittests.Routing.Serialization.RoutingSerializationTests.RoutingSerializationFlatfileLiveEdge: Test(s) failed.   Expected: 2
  But was:  3
   at NUnit.Framework.Assert.That(Object actual, IResolveConstraint expression, String message, Object[] args)

```


@Deathamns

There is a problem in latest dev build of Firefox, and I suspect it's caused by a change somewhere in here.

Currently the core's onBeforeRequest() is being called with these details:

    details.frameId: 1
    details.parentFrameId: -1

Which should never happen: a sub-frame is always embedded in a parent frame. I found the problem because I was testing my fix to #614: <http://market.yandex.ru/> does not work on Firefox. It's because the filter works only for when the page URL is `market.yandex.ru`, but on Firefox it thinks the page URL is `yastatic.net` -- which really is a sub-frame of <http://market.yandex.ru/>


Here is another case where the removal of variables can actually be annoying. When you have the value in an intemediate variable, it makes using the debugger very simple. You break at the `return` statement and then inspect the value of `doc`. But as soon as there is no container variable, you lose debugging convenience. You can argue that you just make this change locally when debugging is needed, but sometimes you are debugging into a `.jar` file where changing the code is not trivial. And then you have to clone the project, link the versions with a SNAPSHOT so that Eclipse does a project coupling, etc... all because there is no variable.


typo: "Several jobs exist with the name"


This is inconsistent with modifications you did to other files where you changed it_IT to it.


@valib Thx :+1:
I suppose people are lazy or don't expect consequences when they copy some clips/movies from friends hdd or copy paste the movie name from IMDB etc. so names like "Father's kids" are not so rare.


thanks!


Syntax sugar: .delete(/^---\n/)


:+1: Awesome to bring the code into the brave new world of Java 5. :grin: (All these things are leftover from when Bio-Formats was Java 1.4...)


It's [here](https://github.com/manastech/crystal/blob/master/src/compiler/crystal/syntax/lexer.cr#L969). Basically, after a token is parsed the `@slash_is_regex` variable is set to false. Then it must explicitly be set to true by the parser before calling next_token. Is that your question?


Either line 254 or line 255 is probably wrong as both lines tries to copy a file to same destination ${TEMPFCFILE}.homedirs


@styx-hy  很久以前的patch  http://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/2013q2/007124.html

我把它用在最新版本里就失效了


Looks good :+1:


Probably [this](https://github.com/gorhill/uBlock/blob/263b756764829eebb877f1b5b92bba3da5cf28e0/platform/firefox/frameModule.js#L128), but I'm already working on implementing proper frameIds, so it will change anyway.


dbPassword / dbPwd


this was never even here in this branch to begin with


And it breaks ``OMNeTLogger`` in omnetpp.


< ahem >!... Garvin! :) Der clearfix geht um....


this is a problem with every language that has implicit calls, sadly.


Great.


@davidyen1124 I like this feature of get the package name from the directory name, you can make a pull request with this ?


rename b as bundle


Something else i missed, i think.
I think until about two years ago, dae/anki@a78c4d6c, Anki did not have xml.etree, so i did not use that in `downloader.py`. I think ElementTree is much nicer than BeautifulSoup. Now i can use ET as well. Nice to know.

Or i was thinking about [`lxml.`](https://github.com/ospalh/anki-addons/blob/master/lean_browser_qa.py#L10)[`html`](https://github.com/ospalh/anki-addons/blob/develop-ospalh-special/lean_browser_qa.py#L9), that is still not part of standard Anki.


Code looks good.  It might be too annoying for this feature, but is there a test?  Otherwise, ship her!


That can't happen in a snapshot: the dependencies defined in the DESCRIPTION field are fixed and can't change.

If the user re-runs a script with a new date as the argument for `checkpoint` then the dependency chain will be checked anew, as always.


check if parameter exists for required.
if userId in request.get_json():
    userId = request.get_json()
else
    abort 400


indent using tabs, not spaces.


Missing a closing brace here I believe.


This "show range" code can also be separated into its own component.


I'd call this method `list` instead. Since the collection you return has the `[]` operator, there is an order, so it is not a random collection. It is actually a list.


Take care... Not all resources that return collections in Abiquo have pagination. If the `totalSize` field is not present you should read the size of the `collection` element.


@brixen This defines X on Object, which I believe is unintended here.
I notably makes https://github.com/rubinius/rubinius/blob/master/spec/ruby/core/float/shared/equal.rb fail if that Float spec is run after this language lambda spec.
The Float spec should use Class.new with block, but this one (and below) should not define X on Object either.


this makes sense except for the line

var LocalStrategy = exports;

doesn't that confuse the exports global and shouldn't i use module.exports?


Its auto indented.


Thanks, forgot to test that!


The function constructMetric(..) no longer exists in the latest code. Everything was moved to more appropriate places.


Co robi ta metoda?


Also added prompts for debugging


I had to re-install Git once because I could not find the Git shell. Once I found the Git Shell I was able to follow the setup pretty well.


Is it possible to add a regression test?


@blisteringherb these changes I pushed last night aren't live on the site - can you see if wercker is wercking?


data["bundle"]
But I don't thin you need to create these structure "data"
URI can be added in the Bundle object to retrieve every times (on research, etc.)


Take care! limit=0 means no pagination. Make sure you properly handle this value in this inn the `[]` operator.


Why would you introduce this limitation for the 8byte nonce case? As far as I can tell boringssl is actually using 8-byte nonces and a 64-bit counter (https://boringssl.googlesource.com/boringssl/+/master/crypto/chacha/chacha_generic.c)


yes, or can you replace it all by call "spi_transfer_regs(dev, reg, out, in, 1)"?
I favor speed over code size :smile:.


В $aComponentParams будут попадать служебные параметры типа "_default_short" и "component"


zawsze "Done"? Niezależnie czy coś się wykona czy nie?


@czue


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10430/testReport/) for this commit**


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


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


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


remove print, use app.logger.debug("message") to do that.


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


This should be the first line of the method, shouldn't it? No need to do additional work if the input is null.


You removed all those fun and exciting tab characters!


Зачем дублировать самого себя?


Great! A few notes for improvement:

When you want to have a list, please use ```<ul>``` and ```<li>``` for items instead of the -.
The items seem to be sentences so start each on of the with capital letter and end with .

lingualeo.com  should be a real link ```<a href=""></a>```

I'll ask some English speaker to review the grammar so I won't correct you in the wrong way direction. :smile:


Hi


return the Bundle object updated.


final


Should this key change to blacklight.status.success?


return the Bundle object.


oh, I'm on 2.1.1.

I believe I fixed it now

On Tue, Jan 27, 2015 at 2:59 PM, tosiara <notifications@github.com> wrote:

> Ouch, git version 1.8.1.4
>
> Unknown option: -C
> usage: git............
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jonjonarnearne/smi2021/commit/83323df422e4b31b420692d0490214a5192a82b5#commitcomment-9459235>
> .
>

--
Jonarne
http://jonarne.no


Ok, I'll fix it now

2015-01-27 18:23 GMT+04:00 Gabor Szabo <notifications@github.com>:

> Great! A few notes for improvement:
>
> When you want to have a list, please use
 and
 for items instead
> of the -.
> The items seem to be sentences so start each on of the with capital letter
> and end with .
>
> lingualeo.com should be a real link
>
> I'll ask some English speaker to review the grammar so I won't correct you
> in the wrong way direction. [image: :smile:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/name2rnd/perlmaven.com/commit/02dcadb45c56c89fa22a5034543eec30bed33d79#commitcomment-9459700>
> .
>


use directly: requet.json() to get these data.


Hi,
In my opinion your example is too complex

first: You don't rely on the received-Hash- so disable it

    ib = IB::Connection.new client_id: 1112, ,  :received => false , : port => 7496

second: the block given to subscribe can be refactored to the following code – and contracts does not need to be an instance-variable

    # assume execution is complete once all contracts have received data.
    responded = []
    ib.subscribe(:Alert, IB::Messages::Incoming::HistoricalData) do |msg|
      case msg
       when IB::Messages::Incoming::HistoricalData
          puts " #{contracts[ msg.request_id ][:symbol]  } \t #{msg.to_human}"
       else
         puts msg.to_human
       end  # case
       responded << msg.request_id
     end # subscribe


LGTM


use jsonify (from Flask)


I'm a little concerned that session isn't being passed into the Ability initialization.  Maybe it makes the most sense to do the authorization check in the controller.


This doesn't include <head>, thus no CSS. It saved my file as a decapitated HTML ^^. I'm gonna look at that right now


Maybe these methods could return a tuple (error, success)?


use config parser to configure these parameters


doh


You are right. These are redundant.


Not as issue, but since it is a test could we use the assertEqual function call?


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10431/testReport/) for this commit**


Going forward we will be removing all notification msgs from within the individual resolvers and instead simply passing back return codes to allow the addons decide how to handle, can you remove this line?


commit on every new plugin or commit after extract all information on every plugins ?


It always exists if gradle is run. So I think it is not an issue.


Certainly this is more specific, but would the tests have passed by removing the logic in the test?  There are two job fixtures, both of different STI types.  I would expect both jobs to be returned in both tests rather than a single job.


Thank you mate! Also considered PowerMock. Only thing is that I think its bad design when you have to use this lib. And actually in this case the bad design is on Library side. Or I might geht things wrong... I guess, I'll stay with my work around and rather dont use PowerMock.

> Am 27.01.2015 um 10:33 schrieb Frank Rosner <notifications@github.com>:
>
> I think there is no way to do it with Mockito. You should try PowerMock in this case.
>
> —
> Reply to this email directly or view it on GitHub.
>


yes, will make the change.


Please remove this line, error logo's are being removed


It seems that the problems I had are summarized in the top two answers below, but apparently everything is fine here: http://stackoverflow.com/q/701313/288456


Fix #3


为什么要把本来遵守 PEP8 的排版改成不遵守的？


Also remove this line, we are moving to remove notification msgs from the resolvers and only passing back return codes for the addons to decide how to handle


@fffw Based on your comments in the PR, I made the entire configuration of the ConnPool immutable.


I'm don't entirely agree with this change.  Steps are ordered, not positioned.  How is position more consistent...and consistent with what?


Same notes for this resolver


This line as well


Remove


LINES 33-35
Are you using the maximize method to test desktop view only to navigate to the 'Find My Experience' front? Without maximize the window the tablet-icon may appear with nav-bar disabled. What is the goal of this test? Is the task to test the workflow through the nav-bar or just navigate to 'Find My Experience' front and ensure the correct front appears?

You could approach this test in a multiple ways
- Test the workflow from table-icon
- Test the workflow from nav-bar by maximize window
- Jump directly to the front by passing the href through the visit method, i.e. visit base_url("/beach/my-beach-experience/")
- You could add IF/ELSIF logic


1、那么 DEBUGINFO 这个字符串会解析成 logging.INFO ？
2、你这是一个时间复杂度为 O(n) 的方法，有一个时间复杂度为 O(1) 的方法，那就是：
```python
    return {
        'INFO': logging.INFO,
        'DEBUG': logging.DEBUG,
        'WARNING': logging.WARNING,
        'FATAL': logging.FATAL
    }.get(LOG_LEVEL)
```


I think we should probably try to keep the url requests out of the service classes. This can probably be done in `GmaApiClient`.


zupełnie niepotrzebne, w ramach tej samej klasy widzimy co tworzymy


Documentation is fixed now @lenart :smile_cat:


I have chose the "//" comment style everywhere, to use one unique style. Most were single-line comments only, so this is IMO a bit better. Since no real logic was touched in the file, I didn't add a specific second commit for this.


still ridiculously verbose


Strange, maybe I forgot to push the service configuration?


what is that even supposed to do?


Everytime you use `class_exists`, a kitten die.


Is it possible to add a regression test?


While I like the overall approach of having a default_type, it seems that with this hash constant, we are making more work for ourselves than necessary since the types are already defined in subclasses of JobTemplate.  Rather than simply extending JobTemplate in the future, we would also have to add each new template type to this hash.


GSON emitting out the this json: {"serverName":"github","repositories":[{"projectName":"dream","projectPath":"/Users/rvvaidya/"}]}


WoW yeah you are totally right !! Hmmm, sorry about that, that's what appends when we write code at 3AM :(

I just test with a micro html page so :s


Add a space at the end of this line


Znaczenia logicznego nie ma, ale dobre praktyki mówią, żeby nie robić takich składni, bo tego się czytać nie da.
Ładniej byłoby z "szybkiem returnem", czyli "else" jest niepotrzebny.


Sure, the code in the SO question contains obvious problem with returning char* from temp, but this is not the case here.


tester


An attempt to fix a different bug that ended up in the wrong place. Later reverted.


Delay the refresh because the processing usually takes longer than a page refresh.


Cleanup of accidental branches.


Write code that is as readable by adding meaningful comments is very important when someone else reviews the code for maintenance or handed off to someone else new to the project to enhance the tests.


@asterite Oh, I missed that. I was talking about [this]( https://github.com/manastech/crystal/blob/9787fefda15637c371d3f05e19d1ab9ae9065adb/src/compiler/crystal/syntax/parser.cr#L2851).


if (recMessage == null)
    return null;

Rozległych ifów (dłuższych niż 3-5 lini kodu nie da się czytać swobodnie)


I don't find the A, X, Y labels very useful here... I'd rather something like `You're trying to do %s += %s * %s`; e.g., `You're trying to do (1,) += (10, 2) * (10, 1)`.


Don't worry, I made a quick [fix](https://github.com/JbIPS/BackNode/commit/c592fda05ac4af59510b24c5c258740853867f11). I'll make a PR when I fix problem with clics inside the editor


update dis


does it work?


True, but the source repository doesn't have to be an MRAN snapshot.  It could be CRAN, or BioConductor.


Sorry about that, bad git branch hygiene, this was already fixed on master but not in draft-1 branch, I have brought it up to date now.


I agree with @mattfred

Activities should load data from the local database and trigger the sync service as appropriate. The sync service should be using the Api to fetch data and then the sync service should load it into the local database. Then the sync service should broadcast that the database has been updated. The Activity can listen for those broadcasts to refresh the locally loaded data.


Yea, probably not doing it right.


Hoe kan ik deze wijziging ongedaan maken? Zie nu pas dat er wordt gelinkt naar Nederlandse documentatie.... de commit is dus onnodig en kan ongedaan gemaakt worden.


Can't we remove this second check now, since the previous line/code already uses it/checks whether it compiles?


## This does not make any sense!!


awesome!


This is probably some debug output which is not complete and probably unnecessary


I would name it simple Widget::FilteringDuplicator to stay in line with all the other Widget methods.

@brendo ??


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3317/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3315/


Wcześniej pisałeś klase MessageFactory, tutaj ją można wykorzystać;),
zaś najlepiej to zastosować w niej "Static factory methods" skoro i tak wiemy jaka konkretna klasa się tutaj znajdzie.


This is incorrect. The only authorities the identity client should have is zones.create and whatever is needed to check for space developer at the CC. The identity client should NOT be able to create clients or IDPs using client_credentials


setTimeout(location.reload, 2500)

Now it works, but it's still a code smell.

Instead, use the $.post callback.


This would need proper escaping to prevent injections.

Maybe General::sanatize ??


Cambio de motor de mase de datos  a Innodb para establecer relaciones de algunas tablas


final


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1978](http://teamcity.monogame.net/viewLog.html?buildId=5099&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


Removed. Thx.


Lol I commented on the same issue. I vote for `Widget::FilteringDuplicator`


wtf?:D


LINE 35: Less code by using this find('.nav-find-experience a').click


@graemerocher Keep the secret key in the source code may not cause problems in the repository?


TeamCity xBIM :: XbimExchange :: Build [Build 57](http://ncl-xbim:8080/viewLog.html?buildId=766&buildTypeId=XBIM_XbimExchange_Build) is now running


You can use ```stage.getHtml()``` which does
```js
iframe.contentDocument.documentElement.outerHTML
```


idps.create should not be a client_credentials scope. You will lose audit history. These actions can only be taken by a user.


:grinning:


@hsmade shit, you are right. I am investigating right now.


you repet your code - this is bad habbit


Yeah it will be perfect ! As you want @JbIPS !!


Do you have a better solution?


TeamCity xBIM :: XbimExchange :: Build [Build 57](http://ncl-xbim:8080/viewLog.html?buildId=766&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:01:18


nightmare:

assume it will be later mandatory to change the 'ringlist' design;
then if code as above is not encapsulated, I predict that it will be a nightmare to change the 'ringlist' design.


I don't understand how the maximize window is executing the mouse hover. Please explain!


this is redundant code, better to use and leverage
https://github.com/cloudfoundry/uaa/blob/develop/uaa/src/test/java/org/cloudfoundry/identity/uaa/mock/util/MockMvcUtils.java


@F03SD: I checked with 3 characters and all were teleported to the home, however none was level 100.


TeamCity xBIM :: XbimWebUI :: Build [Build 25](http://ncl-xbim:8080/viewLog.html?buildId=767&buildTypeId=XBIM_XbimWebUI_Build) is now running


Experienceheader.feature file name, should use camel case or break up the two words with '-'


TeamCity xBIM :: XbimWebUI :: Build [Build 25](http://ncl-xbim:8080/viewLog.html?buildId=767&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:25


TeamCity xBIM :: XbimWebUI :: Build [Build 26](http://ncl-xbim:8080/viewLog.html?buildId=768&buildTypeId=XBIM_XbimWebUI_Build) is now running


TeamCity xBIM :: XbimWebUI :: Build [Build 26](http://ncl-xbim:8080/viewLog.html?buildId=768&buildTypeId=XBIM_XbimWebUI_Build) outcome was **FAILURE**
Summary: Cannot start build runner Build time: 00:00:15


good day
please, tell me, why it is time() ?? can this be changed to 0 or allow client to define this values ??
thank you very much


If your animation uses all of the frames in order, you can actually remove the frames directive for a smaller file. If you omit the frames directive, Minecraft checks the texture and counts the frames, then enumerates them in order.


@danilaml Definently not Alex's autotests. There's no chance we can have graphics running on the Coveralls (I think?). And for possibly making auto tests run we'd need a nice .elf of the cell tests.


mogę się mylić, ale nie ma pewnośći że przekażemy obiekt który bedzie immutable,
więc powinno być:
patientsInfo = new ArrayList(_patientsInfo)<Pacjent>


MessageBox.Show(" :) ");


Just tested, works perfectly, change done!


1: M111 unrecognized component name: authorids


not sure about using the favicon url directly, but couldn't make the previous "//images.duckduckgo.com/iu/?u=http%3A%2F%2Fdaringfireball.net%2Ffavicon.ico" work


Could you please add ,10 to the parseint call ? parseint without proper suffix can break things.

```js
parseInt(table.find('.order-entries-item').eq(0).text(), 10);
```


@IDragonfire That's why it returns `Optional`, it will be `Optional.absent()` in this case.
Read the Javadoc, it states
> If this item was not dropped from a player's inventory, then the thrower will not be available.


Hm, still does not work. Is `KBUILD_SRC` populated at that point?
I get "fatal: Not a git repository (or any of the parent directories): .git"

`$(info $$(KBUILD_SRC) is [$(KBUILD_SRC)])` in Makefile gives this output:
```
$(KBUILD_SRC) is []
```


**[unitTests_php_branches run failed](http://qa-s2.wikia-prod:8080/job/unitTests_php_branches/10433/testReport/) for this commit**


A simple `!filters` would be enough here.


Toto + (to iste v close) sa mi zda skarede, IMHO by transport mal mat prazdnu metodu flush a close, ktora nic nerobi...


My fault. Required Dispatcher's versions set to major version number in all packages (Bundle, lib and migrator). If you have any other question or proposals, i am glad to hear them in https://github.com/sokil/php-mongo/issues

Thank you for using PHPMongo


No need to change it if you think it's useless, but I think it makes more sense to add the the typecast as I suggested it. This would make it clearer that the typecast is required because of the generic `get` function. The `get` function's return type is `*` (anything) because it's generic and can be used to get any type of object. In contrast, `setOpacity` is a specific function, it requires a `number`.


Good build, will help me to change controls in some games.


Thanks. I've updated my tls branch to use the new API:
https://github.com/talex5/ocaml-conduit/commits/tls

(currently updating mirage-dev to use the new versions too)


Why create a new object and use param ? Creating an object can trigger GC. Why not just

```js
filters = '&filters=' + filters
```


TeamCity Toolbox :: Pull requests [Build 19](http://10.48.34.86:8111/viewLog.html?buildId=2207&buildTypeId=Toolbox_PullRequests) is now running


This needs proper escaping...


Only need to remove line 61 which pops up notification box, the logging of the error should stay


TeamCity Toolbox :: Pull requests [Build 19](http://10.48.34.86:8111/viewLog.html?buildId=2207&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 145 Build time: 00:00:17


jest jakiś sens na tworzenie tutaj instancji? Nigdy nie zostanie użyta, zawsze nadpisana.


@c9s Removing the php-version option makes it hard to install extension for multiple versions by script for a dev env, is there anything against it to bring it back?

@marcioAlmada What do you think?


Debianで`system()`は/bin/dashを使うだから、がふくいえさんのcorrectionは良いです。


refs #143


For the CreamOrGreater function, I don't think you need it on the Host, and it's easier to keep it on the connection: see pull request #301. But don't bother fixing it: we will fix up the merge as we now have two versions of it.


not ready yet... it introduced another bug :\


you don't need to keep the whole vertex here, you only need the key


The scenario name refers to verifying 'experience' logo and don't see any assertion that validates this logo. Also how are the best beaches, best photos, and free beach quotes front related to find my experience front? It seems that these front should be separated into different scenarios or write scenario outline.


I'll add it, even if it's only what has been mocked.


We just need to make sure that this line is only run once $('.toggle', ctx).click(function(event) {


I am planning on using it. It's a draft. It will be in a graph package needed for parsing trees, hence the name.


@Gooyeth let me guess, free trial ?


right, better create and emit a new Tuple here :)


Switch to the door http://localhost:3000 for operating the browser-sync :+1:


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1978](http://teamcity.monogame.net/viewLog.html?buildId=5099&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 368, ignored: 6 Build time: 00:16:03


@F03SD: No, Mists of Pandaria.


Sure :smile:


@qiuzi 我试了一下的确呢，不过这不是应该算是一个 bug 么


consistent with our codebase. See db/schema app_categories and service_categories for an example.


You should use static inline function instead. Type checking is important.


Ha! Code Review genius! (wanted to see how you fixed it and this jumped to my eye :D)

![](http://www.reactiongifs.com/r/dsgstd-nrd.gif)


here, you should check whether the vertex value has been updated. If not, there's no need emitting this vertex to the new workset (the vertex will become deactivated).


Can you explain further why you added this?


Minor detail: In wordpress shortcodes can be used for inserting php code into a page. Pages are generated from the db, so donation-form_sc.php is not actually a page. We could have used .inc (for include) to make this clearer but this is recommended against for security reasons in a PHP book i have read


Indenting lines of code introduce a hierarchy to the document, not only making it easier to scan, but also communicating information about the structure of the code visually.


Same as the comment above


According to Linux man page:
"The  pthread_exit()  function terminates the calling thread and returns a value via retval that (if the thread is joinable) is available to another thread in the same process that calls pthread_join(3)."

Do you think it is standard compliant?


Sorry for the delay i had the FFMPEG line too :
Copy https://github.com/frozeus/h5ai/blob/develop/src/_h5ai/server/php/inc/class-thumb.php#L7-L8
and
https://github.com/frozeus/h5ai/blob/develop/src/_h5ai/server/php/inc/class-thumb.php#L94-L118

Do you have a repo to check your code ?


Again please change "this page"


If you convert a class to spring service you have to remove the Service
interface as well as remove it from config-*.xml files.  Perhaps that was
forgotten?

On Tue, Jan 27, 2015 at 3:36 PM, María Arias de Reyna <
notifications@github.com> wrote:

> Strange, maybe I forgot to push the service configuration?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geonetwork/core-geonetwork/commit/46027b52535d5912e0759977e332ac16814143c6#commitcomment-9459966>
> .
>


Yes, they are removed now (commented).


Interesting... Does it also happen in the h5proxy branch?


I think I'd wrap some of the keychain stuff in try/except blocks so you can get nicer error messages in case something goes wrong, but otherwise looks good.


oh, I see, thanks for the comments! I think I understand how the worksets are updated now, I'll make the changes.


I am sorry you did not get my smile...


This new dev, allow get "via host IP" and push an alarm with destination IP of error reply request (this destination IP for reply request 4XX, 5XX, is source IP who is "issue" origin.
Kevin is going to change webhomer to consider this commit.


it does feel wrong, but why exactly is it better to emit a new Tuple? thanks :)


Thank you for going through the page. I think we need to make it clear when writing "tech design" documentation that wp does not use .php files directly, this will help avoid misunderstandings


> Are you thinking about defining different size-types for the different cases?

No, that's definitely pointless busy work. If anything, it makes the code harder to read. I just meant that we'll need to decide which of the built in types makes sense for each situation.


Thanks :)


Hmm, you change in `gulpfile.js` file?


@styx-hy  功能增加而已


Should also mention that the performance of these implementations is not optimised for production since the dates are generated using an algorithm, and providing a list of dates from an external vendor will be faster. This also makes it clear that we're not just shipping somebody else's holiday data.


Can @thread ever be nil? We were checking for that before.


thx to both of you @lexoyo & @JbIPS :+1:  :+1:


Hi,

Is there a working example of writing a file to sd card somewhere ? Do you have a link to an apk that I can test on my samsung kitkat device ?

Thanks

Charles


we had a meeting to brainstorm on the idea of whatt to do in our project


@pgiraud I've created a constant in index.html and I use it in the maincontroller. But I'm not sure if what I've done, is the best way to do that ?


Nice!


What would you actually want to set them to if it could be client configured? Do you know what the values are actually used for?


Is there any other place where the debugger client is closed (e.g. session end)? Otherwise we will leave the debugger client connected forever, which is IMO a bug.


Don't need this... It's one step too many. We should just grab the file... Look at using the file upload component in SA3.


head_js -> init_css


Found BROKEN by quick.
Here's the log file for step 'all':
```
[  0%] Built target casadi_tinyxml
[  3%] Built target casadi_qpoases
[ 16%] Built target casadi_csparse
[ 28%] Built target casadi_sundials
[ 28%] Built target casadi_runtime_embedded
[ 29%] Built target profilereport
[ 44%] Built target casadi_dsdp
Scanning dependencies of target casadi
[ 44%] [ 44%] [ 44%] [ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/generic_type.cpp.o
Building CXX object casadi/core/CMakeFiles/casadi.dir/casadi_instantiations.cpp.o
Building CXX object casadi/core/CMakeFiles/casadi.dir/__/__/casadi_meta.cpp.o
Building CXX object casadi/core/CMakeFiles/casadi.dir/options_functionality.cpp.o
[ 44%] Building CXX object casadi/core/CMakeFiles/casadi.dir/std_vector_tools.cpp.o
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/functor.cpp.o
In file included from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/generic_matrix.hpp:32:0,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix.hpp:35,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/sx_element.hpp:33,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = casadi::SXElement]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = casadi::SXElement]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:47:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
     const int nel = sz1*sz2;
               ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = long long unsigned int]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1046:5:   required from ‘void casadi::Matrix<T>::setBV(const casadi::Matrix<T>&) [with DataType = casadi::SXElement]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:47:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = double]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = double]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:60:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = int]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = int]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:61:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/functor_internal.cpp.o
cc1plus: all warnings being treated as errors
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/casadi_instantiations.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [casadi/core/CMakeFiles/casadi.dir/all] Error 2
make: *** [all] Error 2

```


Look at fileUpload xtype...


see: http://docs.sencha.com/extjs/4.2.3/#!/api/Ext.form.field.FileButton


I only now realized that rgbasm does have modulus abilities... I was looking in fixed point expressions instead of integer expressions.

Anyways, I rewrote the macros and tested it out. Adding
```asm
BCH2: MACRO
    dn ((\1) / 1000) % 10, ((\1) / 100) % 10
    dn ((\1) / 10) % 10, (\1) % 10
    ENDM

BCH3: MACRO
    dn ((\1) / 100000) % 10, ((\1) / 10000) % 10
    dn ((\1) / 1000) % 10, ((\1) / 100) % 10
    dn ((\1) / 10) % 10, (\1) % 10
    ENDM
```
and then puting
```asm
BCH2 180
```
on the
```asm
db $01,$80 ; 180 Coins
```
line made a make compare succeed.


You mean #171 :)


An idea to consider: Instead of modifying `front-end/inspector.json` that is a part of the front-end bundle fetched from Blink, implement an express route that will add this extra line dynamically.

Something like

```js
var inspectorJson = require.resolve('../front-end/inspector.json');
inspectorJson.unshift({ "name": "node", "type": "autostart" });
app.get('/inspector.json', function(req, res) {
  res.json(inspectorJson);
});
```

Thoughts?


Flink has two modes for objects, mutable (the same object is reused for several records and immutable (a new object created for every record). Immutable is used by default, so, in principle, the result should be correct. But it's better to use a new Tuple, in case at some point we use this code with mutable mode.
You can also take a look [here](https://issues.apache.org/jira/browse/FLINK-1005) for more details.


@waneck is this correct? :wink:


yes, hence the comment about the typo elsewhere. I hadn't finished my coffee...


s/accrues/accrued


Performance is not an issue. The algorithm is run once and the results cached.


No, mkstemp() creates the file (a tuple with the path as the second element), and `security` needs it to not exist when told to `create-keychain`.

So this is just a way to get a unique temporary path. Maybe there's a better way.. ?


Oh, unless this extra line is added by `npm generate-front-end`, in which case you can ignore my comment.


Oh, I see, thanks!


@vzvenyach - FBI files from their field office in Miss. investigating claims of local police violence. I think they could have done it in a few hours, but the backlog over there is enormous.


yeah, see https://github.com/bitvagas/bitvagas.github.io/blob/site/gulpfile.js#L13-L16


@jfischer-phytec-iot btw, could you write your code comments in the "Files Changed" tab of the PR https://github.com/jfischer-phytec-iot/RIOT/pull/11 instead?
It makes it much easier to get an overview of the comments and whether they have been addressed.


Warum sind eigentlich die BuildSettings im .gitignore? Das würde erklären, warum ich nie die Levels in den Build Settings hatte …


Sorry I can’t share our APKs that write out.

I can tell you that writing to SDs got more difficult with the newer version of File plugins.

When we get the file system, we reassign the “root” attribute to the actual root directory.  By default it comes through as an app directory and you can’t get out to the sd.

This function is how we get the filesystem in order to be able to write to the SDs.

              getFileSystem : function() {
                     var dfd = $.Deferred();

                     var gotRootFileEntry = function(fileSystem, dfd) {
                           return function(entry) {
                                  //console.log("GOT ROOT file system: "+entry.toURL());
                                  fileSystem.root = entry;
                                  //console.log("GOT ROOT file system: "+fileSystem.root.toURL());
                                  dfd.resolve(fileSystem);
                           };
                     }

                     var gotFS = function(dfd) {
                           return function(fileSystem) {
                                  simplec.api.file.hasFileSystem = true;

                                  if (window.device && window.device.platform && window.device.platform=="Android") {
                                         window.resolveLocalFileSystemURL('cdvfile://localhost/root/', gotRootFileEntry(fileSystem, dfd), fail(dfd));
                                  } else if (window.device && window.device.platform && window.device.platform=="iOS") {
                                         var s = cordova.file.documentsDirectory;
                                         //console.log("GETTING ROOT file system - on iOS "+s);
                                         console.log("file root "+fileSystem.root.toURL());
                                         console.log("doc dir "+cordova.file.documentsDirectory);
                                         window.resolveLocalFileSystemURL(s, gotRootFileEntry(fileSystem, dfd), fail(dfd));
                                  } else {
                                         //console.log("GOT ROOT file system - not on Android");
                                         dfd.resolve(fileSystem);
                                  }
                           };
                     };

                     var fail = function(dfd) {
                           return function(error) {
                                  //console.log(" FILE SYSTEM ERROR ");
                                  //console.log(JSON.stringify(error));
                                  //console.log(" FileError.QUOTA_EXCEEDED_ERR: "+FileError.QUOTA_EXCEEDED_ERR);
                                  //console.log(" quota: "+simplec.api.properties.fileSystemQuota);
                                  simplec.api.file.methods.fileError(dfd, error);
                           };
                     };

                     if (window.requestFileSystem!=null) {
                           //console.log("file system found in file.api");
                           var requestedSize = simplec.api.properties.fileSystemQuota;
                           window.requestFileSystem(LocalFileSystem.PERSISTENT, 0, gotFS(dfd), fail(dfd));// simplec.api.properties.fileSystemQuota, gotFS(dfd), fail(dfd));
                     } else {
                           var err = {code: 99};
                           dfd.reject(err);
                     }

                     return dfd.promise();
              },
[SimpleC Logo]

Douglas Nelson • VP of Technology
________________________________
300 Galleria Parkway • Suite 720 • Atlanta, Georgia 30339
Direct 404-966-1604 • Fax 888-829-9794 • Main 770-290-0020
dnelson@simplec.com<mailto:dnelson@simplec.com>  • www.SimpleC.com<http://www.simplec.com/>

                                [http://www.simplec.com/wp-content/uploads/2014/10/sboy.email_.png]

From: cblais2 [mailto:notifications@github.com]
Sent: Tuesday, January 27, 2015 10:11 AM
To: simplec-dev/KitKatExternalFiles
Subject: Re: [KitKatExternalFiles] . (fd91a33)

Hi,

Is there a working example of writing a file to sd card somewhere ? Do you have a link to an apk that I can test on my samsung kitkat device ?

Thanks

Charles

—
Reply to this email directly or view it on GitHub<https://github.com/simplec-dev/KitKatExternalFiles/commit/fd91a33bb50b3c60192f2f078eb921c157732060#commitcomment-9460627>.


This was one of the changes I was not really sure about myself. For the same reasons you mention. Undoing this wont do much harm.


Did you try `JSON.stringify(value, null, 2)`? In my experience, the output of that is readable enough, there is no need to  implement a custom formatter.


In this case, I cannot think of any other scenario in which we would need a public, user specific page in which would need a brand that points back to the user sign-in page. I'll make a note of this in confluence documentation though, since we would need to add those cases into this helper logic.


Thanks guys! Submitted a fix.

On Mon, Jan 26, 2015 at 8:18 PM, Pan Kim <notifications@github.com> wrote:

> errors
>
> /Plugins/org.apache.cordova.camera/CDVCamera.m:45:31: error: use of
> undeclared
> identifier 'command'
> pictureOptions.quality = [command argumentAtIndex:0 withDefault:@(50)];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:46:40: error: use of
> undeclared
> identifier 'command'
> pictureOptions.destinationType = [[command argumentAtIndex:1 withDefault:@(DestinationTypeFileUri)]
> unsignedIntegerValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:47:35: error: use of
> undeclared
> identifier 'command'
> pictureOptions.sourceType = [[command argumentAtIndex:2 withDefault:@(UIImagePickerControllerSourceTypeCamera)]
> unsignedIntegerValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:49:30: error: use of
> undeclared
> identifier 'command'
> NSNumber* targetWidth = [command argumentAtIndex:3 withDefault:nil];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:50:31: error: use of
> undeclared
> identifier 'command'
> NSNumber* targetHeight = [command argumentAtIndex:4 withDefault:nil];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:56:37: error: use of
> undeclared
> identifier 'command'
> pictureOptions.encodingType = [[command argumentAtIndex:5 withDefault:@(EncodingTypeJPEG)]
> unsignedIntegerValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:57:34: error: use of
> undeclared
> identifier 'command'
> pictureOptions.mediaType = [[command argumentAtIndex:6 withDefault:@(MediaTypePicture)]
> unsignedIntegerValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:58:38: error: use of
> undeclared
> identifier 'command'
> pictureOptions.allowsEditing = [[command argumentAtIndex:7 withDefault:@(NO)]
> boolValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:59:43: error: use of
> undeclared
> identifier 'command'
> pictureOptions.correctOrientation = [[command argumentAtIndex:8
> withDefault:@(NO)] boolValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:60:41: error: use of
> undeclared
> identifier 'command'
> pictureOptions.saveToPhotoAlbum = [[command argumentAtIndex:9 withDefault:@(NO)]
> boolValue];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:61:38: error: use of
> undeclared
> identifier 'command'
> pictureOptions.popoverOptions = [command argumentAtIndex:10
> withDefault:nil];
> ^
> /Plugins/org.apache.cordova.camera/CDVCamera.m:62:40: error: use of
> undeclared
> identifier 'command'
> pictureOptions.cameraDirection = [[command argumentAtIndex:11 withDefault:@(UIImagePickerControllerCameraDeviceRear)]
> unsignedInteg...
> ^
> 12 errors generated.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/cordova-plugin-camera/commit/4f056f0a73ff36bb34b172b0ba8bba16a4b06953#commitcomment-9451362>
> .
>


Good luck sir ;o)

On Tue, Jan 27, 2015 at 3:47 PM, Gabe Conradi <notifications@github.com>
wrote:

> @hsmade <https://github.com/hsmade> shit, you are right. I am
> investigating right now.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tumblr/collins/commit/f0b8be6e58974dda66d61306b6ef7a274c736757#commitcomment-9460163>
> .
>


@rwardenga  single method should do one major thing, therefore please move lines 30-39 to different method


I updated the README.md within the directory that shows how to run it with gradle. Maybe give that a try?


Found BROKEN by quick.
home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix.hpp:35,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/sx_element.hpp:33,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = casadi::SXElement]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = casadi::SXElement]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:47:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
     const int nel = sz1*sz2;
               ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = long long unsigned int]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1046:5:   required from ‘void casadi::Matrix<T>::setBV(const casadi::Matrix<T>&) [with DataType = casadi::SXElement]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:47:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = double]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = double]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:60:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = int]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = int]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/casadi_instantiations.cpp:61:18:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/sx/../matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
[ 46%] Building CXX object casadi/core/CMakeFiles/casadi.dir/matrix/matrix_tools.cpp.o
cc1plus: all warnings being treated as errors
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/casadi_instantiations.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
In file included from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/generic_matrix.hpp:32:0,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/matrix.hpp:35,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/matrix_tools.hpp:29,
                 from /home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/matrix_tools.cpp:26:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/sparsity.hpp: In instantiation of ‘void casadi::Sparsity::set(DataType*, const DataType*, const casadi::Sparsity&) const [with T = double]’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/matrix_impl.hpp:1039:5:   required from ‘void casadi::Matrix<T>::set(const casadi::Matrix<T>&, casadi::SparsityType) [with DataType = double]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/../function/io_interface.hpp:364:5:   required from ‘void casadi::IOInterface<Derived>::setInput(const casadi::Matrix<double>&, int) [with Derived = casadi::Function]’
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/matrix_tools.cpp:39:34:   required from here
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/matrix/sparsity.hpp:772:15: error: unused variable ‘nel’ [-Werror=unused-variable]
     const int nel = sz1*sz2;
               ^
cc1plus: all warnings being treated as errors
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/matrix/matrix_tools.cpp.o] Error 1
make[1]: *** [casadi/core/CMakeFiles/casadi.dir/all] Error 2
make: *** [all] Error 2

```


上课不能坐那么后面


it has been encrypted


Previously I'd had a try/except, but I wanted a way to be able to raise an error on either a nonzero return _or_ any stderr, and still be able to capture any stderr output to add to the ProcessorError string.

I could wrap a try/except around a `subprocess.check_call()` instead, but since that will just raise a `CalledProcessError` on a non-zero output, I figure I'm basically doing the same thing here with `if proc.returncode..`


I fixed that in  ac75259


Whie parallel assignment looks good and make you save a few extra lines of code you have to use it carefully(if any) since it less performant than sequential assignment. For more info refer to the @sferik talk [**Writing Fast Ruby in Baruco 2014**](https://speakerdeck.com/sferik/writing-fast-ruby).


2. I'm not sure. It does not deserve its own root directory in components.
So the two choices are ```patient-data``` and ```studies```.
```patient-data``` is good, but families aren't exactly patient data. It's a specific type of study - study of people related by blood.
But once again, this is not a clear choice for me.


application description


Yes this is a really good argument for not keeping this commit in.


thanks @seancribbs


This is an interesting one.  A couple revs ago, I had it this way.  However, the variable `jobs` should actually be an AREL thingy that can be filtered with a call to `#select` without reassignment.


1: M100 needs more reviewers
1: M111 unrecognized component name: authorids


28:1: I100 Imports statements are in the wrong order. from invenio.utils.shell should be before from invenio.version


4: L102 copyright year is outdated, expected 2015 but got 2014
29:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.ext.logging
31:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_DEVEL_SITE
31:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.utils
32:1: F401 'webinterface_proxy' imported but unused
59:9: B901 blind except: statement
69: D209 Put multi-line docstring closing quotes on separate line
69: D203 Expected 1 blank line *before* class docstring, found 0
69: D204 Expected 1 blank line *after* class docstring, found 0
69: D400 First line should end with '.', not 'e'
69: D205 Blank line missing between one-line summary and description
73:5: E301 expected 1 blank line, found 0
73: D102 Docstring missing
85:23: E127 continuation line over-indented for visual indent
92:9: B901 blind except: statement
100: D209 Put multi-line docstring closing quotes on separate line
100: D203 Expected 1 blank line *before* class docstring, found 0
100: D204 Expected 1 blank line *after* class docstring, found 0
100: D400 First line should end with '.', not 'f'
100: D205 Blank line missing between one-line summary and description
104:5: E301 expected 1 blank line, found 0
104: D102 Docstring missing
116:23: E127 continuation line over-indented for visual indent
123:9: B901 blind except: statement
132:1: B901 blind except: statement
138:1: B901 blind except: statement
144:10: E126 continuation line over-indented for hanging indent
145:1: B901 blind except: statement
151:10: E126 continuation line over-indented for hanging indent
152:1: B901 blind except: statement
162:1: E302 expected 2 blank lines, found 1
162: D203 Expected 1 blank line *before* class docstring, found 0
162: D204 Expected 1 blank line *after* class docstring, found 0
165:5: E301 expected 1 blank line, found 0
165: D102 Docstring missing
171:1: E302 expected 2 blank lines, found 1
171: D290 White spaces around docstring should be trimmed.
171: D209 Put multi-line docstring closing quotes on separate line
171: D203 Expected 1 blank line *before* class docstring, found 0
171: D400 First line should end with '.', not 'l'
171: D205 Blank line missing between one-line summary and description
176:16: E127 continuation line over-indented for visual indent
209: D102 Docstring missing
247: D102 Docstring missing


2: L102 copyright year is outdated, expected 2015 but got 2013
36:1: I100 Imports statements are in the wrong order. import tempfile should be before import zlib
37:1: I100 Imports statements are in the wrong order. import sys should be before import tempfile
38:1: F401 'traceback' imported but unused
40:1: I201 Missing newline before sections or imports.
47:1: I101 Imported names are in the wrong order. Should be cache_exists, delete_cache, delete_hp_change, get_cache, get_cache_post_date, get_record_last_modification_date, get_record_revision_author, uids_with_active_caches, update_cache, update_cache_post_date
48:5: E128 continuation line under-indented for visual indent
49:5: E128 continuation line under-indented for visual indent
50:5: E128 continuation line under-indented for visual indent
51:1: I101 Imported names are in the wrong order. Should be create_record, create_records, field_add_subfield, field_get_subfield_codes, field_get_subfield_values, print_rec, record_add_field, record_add_fields, record_delete_fields, record_get_field_instances, record_get_field_value, record_get_field_values, record_has_field, record_make_all_subfields_volatile, record_modify_controlfield, record_modify_subfield, record_order_subfields, record_strip_empty_fields, record_strip_empty_volatile_subfields, record_xml_output
60:1: I101 Imported names are in the wrong order. Should be CFG_BIBEDIT_EXTEND_RECORD_WITH_COLLECTION_TEMPLATE, CFG_BIBEDIT_LOCKLEVEL, CFG_BIBEDIT_QUEUE_CHECK_METHOD, CFG_BIBEDIT_TIMEOUT, CFG_BIBUPLOAD_EXTERNAL_OAIID_TAG, CFG_BIBUPLOAD_EXTERNAL_SYSNO_TAG, CFG_PYLIBDIR
60:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibsched.bibtask
60:1: F401 'CFG_PYLIBDIR' imported but unused
68:1: I101 Imported names are in the wrong order. Should be get_bibupload_task_opts, get_info_of_record_revision, get_marcxml_of_record_revision, get_record_revisions
68:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibedit.db_layer should be before from invenio.utils.text
71:1: I101 Imported names are in the wrong order. Should be get_all_collections_of_a_record, get_colID, get_record, guess_primary_collection_of_a_record, record_exists
72:6: E126 continuation line over-indented for hanging indent
74:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine
75:1: I101 Imported names are in the wrong order. Should be collect_user_info, get_email, get_user_info, get_user_preferences, list_registered_users
76:6: E126 continuation line over-indented for hanging indent
77:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.legacy.webuser
80:1: I100 Imports statements are in the wrong order. from invenio.legacy.refextract.api should be before from invenio.modules.access.engine
81:36: E126 continuation line over-indented for hanging indent
83:1: I101 Imported names are in the wrong order. Should be ParseError, transform_file
83:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord.textmarc2xmlmarc should be before from invenio.legacy.refextract.api
85:1: I100 Imports statements are in the wrong order. from invenio.modules.editor.registry should be before from invenio.modules.knowledge.api
88:1: I100 Imports statements are in the wrong order. from invenio.base.globals should be before from invenio.modules.editor.registry
94:5: F401 'loads' imported but unused
115: D101 Docstring missing
118:1: E302 expected 2 blank lines, found 1
118: D203 Expected 1 blank line *before* class docstring, found 0
118: D204 Expected 1 blank line *after* class docstring, found 0
118: D400 First line should end with '.', not 's'
124:1: E302 expected 2 blank lines, found 1
124: D290 White spaces around docstring should be trimmed.
124: D400 First line should end with '.', not 'n'
124: D205 Blank line missing between one-line summary and description
128: D290 White spaces around docstring should be trimmed.
128: D400 First line should end with '.', not 'e'
171:1: E302 expected 2 blank lines, found 1
171: D103 Docstring missing
177:1: E302 expected 2 blank lines, found 1
177: D401 First line should be imperative: 'Thi', not 'This'
177: D400 First line should end with '.', not 'l'
177: D205 Blank line missing between one-line summary and description
228:1: E302 expected 2 blank lines, found 1
228: D400 First line should end with '.', not 'e'
228: D205 Blank line missing between one-line summary and description
243:1: E302 expected 2 blank lines, found 1
243: D401 First line should be imperative: 'Ha', not 'Has'
243: D400 First line should end with '.', not 'y'
243: D205 Blank line missing between one-line summary and description
251:1: E302 expected 2 blank lines, found 1
251: D400 First line should end with '.', not 'l'
251: D205 Blank line missing between one-line summary and description
277:1: E302 expected 2 blank lines, found 1
277: D400 First line should end with '.', not 'e'
277: D205 Blank line missing between one-line summary and description
284:1: E302 expected 2 blank lines, found 1
293:1: E302 expected 2 blank lines, found 1
304:1: E302 expected 2 blank lines, found 1
304: D400 First line should end with '.', not 'n'
304: D205 Blank line missing between one-line summary and description
315:1: E302 expected 2 blank lines, found 1
319:1: E302 expected 2 blank lines, found 1
319: D205 Blank line missing between one-line summary and description
330:1: E302 expected 2 blank lines, found 1
330: D103 Docstring missing
334:1: E302 expected 2 blank lines, found 1
334: D400 First line should end with '.', not 'm'
334: D205 Blank line missing between one-line summary and description
402: D400 First line should end with '.', not 'd'
402: D205 Blank line missing between one-line summary and description
415: D290 White spaces around docstring should be trimmed.
415: D400 First line should end with '.', not 'd'
415: D200 One-line docstring should not occupy 2 lines
427: D290 White spaces around docstring should be trimmed.
427: D400 First line should end with '.', not 'e'
427: D205 Blank line missing between one-line summary and description
450: D400 First line should end with '.', not 'e'
450: D205 Blank line missing between one-line summary and description
480:24: E713 test for membership should be 'not in'
489:1: E302 expected 2 blank lines, found 1
489: D401 First line should be imperative: 'Convert', not 'Converts'
489: D400 First line should end with '.', not 'p'
489: D200 One-line docstring should not occupy 3 lines
496:1: E302 expected 2 blank lines, found 1
496: D401 First line should be imperative: 'Convert', not 'Converts'
496: D400 First line should end with '.', not 'e'
496: D200 One-line docstring should not occupy 3 lines
508:1: E302 expected 2 blank lines, found 1
508: D400 First line should end with '.', not 'd'
516:1: E302 expected 2 blank lines, found 1
516: D209 Put multi-line docstring closing quotes on separate line
516: D400 First line should end with '.', not 's'
516: D205 Blank line missing between one-line summary and description
516: D208 Docstring is over-indented
529:1: E302 expected 2 blank lines, found 1
529: D205 Blank line missing between one-line summary and description
539:1: E302 expected 2 blank lines, found 1
539: D205 Blank line missing between one-line summary and description
550:1: E302 expected 2 blank lines, found 1
550: D205 Blank line missing between one-line summary and description
557:1: E302 expected 2 blank lines, found 1
557: D205 Blank line missing between one-line summary and description
577:1: E302 expected 2 blank lines, found 1
583:1: E302 expected 2 blank lines, found 1
583: D103 Docstring missing
590:1: E302 expected 2 blank lines, found 1
590: D205 Blank line missing between one-line summary and description
606: D290 White spaces around docstring should be trimmed.
606: D205 Blank line missing between one-line summary and description
627:9: E128 continuation line under-indented for visual indent
630:1: E302 expected 2 blank lines, found 1
630: D209 Put multi-line docstring closing quotes on separate line
630: D400 First line should end with '.', not ']'
630: D205 Blank line missing between one-line summary and description
630: D208 Docstring is over-indented
669:1: E302 expected 2 blank lines, found 1
669: D401 First line should be imperative: 'Return', not 'Returns'
669: D400 First line should end with '.', not ']'
674:1: E302 expected 2 blank lines, found 1
692: D205 Blank line missing between one-line summary and description
700:1: E302 expected 2 blank lines, found 1
713:1: E302 expected 2 blank lines, found 1
753:1: E302 expected 2 blank lines, found 1
771: D400 First line should end with '.', not 's'
771: D205 Blank line missing between one-line summary and description
791: D401 First line should be imperative: 'Return', not 'Returns'
791: D290 White spaces around docstring should be trimmed.
791: D400 First line should end with '.', not '0'
791: D205 Blank line missing between one-line summary and description
802:61: E128 continuation line under-indented for visual indent
803:61: E128 continuation line under-indented for visual indent
804:61: E128 continuation line under-indented for visual indent
805:61: E128 continuation line under-indented for visual indent
808: D290 White spaces around docstring should be trimmed.
808: D400 First line should end with '.', not 't'
808: D205 Blank line missing between one-line summary and description
823: D290 White spaces around docstring should be trimmed.
823: D400 First line should end with '.', not 't'
846:48: E128 continuation line under-indented for visual indent
852:1: E302 expected 2 blank lines, found 1
852: D400 First line should end with '.', not 'd'
896:1: E302 expected 2 blank lines, found 1
896: D400 First line should end with '.', not 't'
896: D205 Blank line missing between one-line summary and description
914: D400 First line should end with '.', not 'e'
914: D205 Blank line missing between one-line summary and description
932:5: E122 continuation line missing indentation or outdented
948:36: E127 continuation line over-indented for visual indent
949:36: E127 continuation line over-indented for visual indent
950:36: E127 continuation line over-indented for visual indent
951:36: E127 continuation line over-indented for visual indent
955: D400 First line should end with '.', not 'n'
1013: D401 First line should be imperative: 'Create', not 'Creates'
1013: D400 First line should end with '.', not 'e'
1013: D205 Blank line missing between one-line summary and description
1028: D400 First line should end with '.', not 'n'
1028: D205 Blank line missing between one-line summary and description
1048: D401 First line should be imperative: 'Change', not 'Changes'
1048: D400 First line should end with '.', not ','
1048: D205 Blank line missing between one-line summary and description
1066:9: E128 continuation line under-indented for visual indent
1076:13: E128 continuation line under-indented for visual indent
1079: D401 First line should be imperative: 'Return', not 'Returns'
1079: D290 White spaces around docstring should be trimmed.
1079: D400 First line should end with '.', not 'e'
1107: D103 Docstring missing
1121:25: E128 continuation line under-indented for visual indent
1160: D401 First line should be imperative: 'Load', not 'Loads'
1160: D205 Blank line missing between one-line summary and description
1176:52: E201 whitespace after '{'
1176:62: E203 whitespace before ':'
1176:89: E203 whitespace before ':'
1176:105: E202 whitespace before '}'
1189: D401 First line should be imperative: 'Split', not 'Splits'
1189: D400 First line should end with '.', not ':'
1189: D205 Blank line missing between one-line summary and description
1189: D300 Expected """-quotes, got '''-quotes
1269: D401 First line should be imperative: 'Create', not 'Creates'
1269: D400 First line should end with '.', not 'e'
1269: D205 Blank line missing between one-line summary and description
1269: D300 Expected """-quotes, got '''-quotes
1304: D205 Blank line missing between one-line summary and description
1304: D300 Expected """-quotes, got '''-quotes