Bon j'ai passé mon code dans le code cleaning de CLion ...
Dans les lambdas, tu veux l'accolade à la ligne ? avec quelle indentation ?


не оставляй ссылки без классов либо помечай также классом что бы не было каскадов либо читай выше, и отказывайся от списка


I'd recommend this:

```python
```


Бред какой-то, подумай лучше!


We are registering the example with the group `examples` collection with the `example_group_class.examples << self` line in `Example#initialize` above.


Was the instruction to run `texhash` removed intentionally?


We're now "more" than close!


Тоже бред!


Also, if mob is already 0, you probably do not want to substract another S(46, 46)


Minor, but you could move this assignment to the `out` label.
```
out:
  if (out_changed)
    *out_changed = did_change
  return ret;
```


I found it @ManjaroCinnamonFan ! :stuck_out_tongue_winking_eye:


Yes, rubocop complained.  Fix in coming.


@Atcold You can write `THTensor_(abc)(...)` in any generic file, and it will expand to `THRealTensor_abc(...)` (with Real substituted with a currently generated type ofc).


I find `unless ... else` hard to read, so I would naturally invert this conditional to an `if ... else`.

I'm quite surprised the Rubocop doesn't have a rule for that.


Sorry, disregard the above, it was my very surfacial compile-fix-only commit..


fieldset внутри form , не лучшее семантическое решение, лучше какой-то измени, вообще fildset лучше применять реже и обудманно, это не кроссбраузерное решение


What errors do you run into with gsyslog on NetBSD?


Looks good. Eagerly awaiting v1.0.2.


TeamCity Kurma :: Integration Tests :: CentOS 7.2 [Build 1](http://ci.kurma.io/viewLog.html?buildId=87&buildTypeId=Kurma_IntegrationTests_CentOS72&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:02:55


Shouldn't this be "monografie" instead of "mografie"?


It seems like this isn't really related to the dateTimeStatic component at all. Did this commit come from another branch originally? If so, could you try to link the dependent PR whenever this occurs in the future?


@cb372 would be good to submit an issue on play bug tracker.


Опять же form вынеси как отдельный блок с нормальным названием, и от нее уже элементы и модификаторы, учись разбивать верстку на блоки


add only one line for
com.softserve.webtester


Actually perhaps it's OK in this case, because it more closely matches up with the related `unless ... end` blocks. Ignore my previous comment.


I'm happy with that, I'll update and test.


Guessing that should also say "TODO maybe should..."


What is the purpose of this? Were you planning to export it somewhere here? It seems like you shouldn't need to add imports for no reason.


Oy, @apcerabot... need to get you to only comment on PRs then, not on regular branch pushes.


Yay!


Indentation looks wrong, tabs please


y u do this
now go trigger a build for @minidude2012 , she's busy


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1023 of 1023 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-30.56%25-green.svg)


These tests could be broken down further for clarity. But whatev. Not a big deal either way.


I was expecting to see an equivalent line in the diff removing the rendering of this same partial, but there doesn't seem to be one in this case. I'm probably missing something obvious though - it's so hard to work out what's going on - I don't know how you did it!


Yes, I'd prefer it. If we accept it as is, it needs to changed just after this PR or we will forget. I can do a PR to you if you don't feel like doing it.


okay, thanks for the precision (it's unclear as an outsider)


Assignment posted for graduation


Please do not use import in such way.Please specify each class separately.


`implementation-defined` is supposed to be italicized (the text markup I use can't do that in code).


I got a NULL while using dbtext. Probably for MySQL an empty string is returned.
The issue with dbtext is that empty strings are not supported. Instead NULL is returned.
The b2b modules with dbtext on 1.11 and 2.1 will not work, unless the dbtext schema is manually altered.
We could backport, but the fix it's specific to dbtext, which is broken on old releases for this modules.


clearfix что б не видел такого класса вообще в разметке, если хочешь вставляй в стиле через миксин!


Hmm, no I'm not seeing any straightforward way to set a custom map implementation.

It'd be fine with me if you replace JsonDict's implementation (extends HashMap) with your implementation so it can wrap whatever kind of hashmap. It just seems redundant to have both classes.


This stuff is related to the message log changes. The two branches must have cross pollinated. :grin:


I think that's a bit optimistic. IME it's a lot easier to read over the code to check that `nil` is handled correctly than to track down any resulting bugs weeks later.


## Task 1

Write a program to count word usage in the given text.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.  You should use a *standard* dictionary as part of your function(s) to accomplish this task (see Task 2, where you'll use a slightly different type of dictionary).

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a dictionary to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 2

Using [Counter](https://docs.python.org/2/library/collections.html#collections.Counter) from the `collections` module, write a program to count word usage in the same text as above.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a `Counter` to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 3

Write a program that processess the given data to create a dictionary that maps the ExaML run name (e.g. ExaML_info.T6 as the `key`) to the ExaML log-likelihood value (e.g. -82924194.67 as the `value`). Use this function or a function that calls this function to prettily print the run name followed by the log-likelihood value in two column format to the screen.  Also include a function that sorts the log-likelihood values and helps us determine which run name minimizes the negative log likelihood value (e.g. the run name having the LL value closest to zero). Also output the name of this "best" run to the screen.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program prints run name/LL in two column format and correctly sorts and reports the "best" ML result (4 pts.)


Same italic.


TeamCity jonas-requests :: WindowsBuild [Build 19](https://build.neohq.net/viewLog.html?buildId=569151&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Why do we have to `eslint-disable-line` here?

Surely we can just not assign to a variable.  `new` side effects are allowed, so just change to

```js
new MprisServer();
```


这里应该还要判断 `x > y` 吧?不然上下滑动也会发日志的吧？


Update correct d.ts fiel for dataTabel and add more functionalities.
- DOM Settings
- Update function parameter types.
- Rename function type 'ButtonText' -> 'FunctionButtonText', to match other function names.
- Add some examples from project page for better testing.


Oh. Now I see that the `send_application_non_uk_visa_renew_new_colour` partial previously rendered the `book_appointment_by_email_with_3_dates` partial, but now we're not using that, we have to do it explicitly. Ignore my last comment.


I felt the original code was too hard to read -- one function call trying to do too much. By formatting the label separately I felt it increased readability. I feel the label formatting code is still too messy, but it was a small step towards improved readability.


> [Looks good. Eagerly awaiting v1.0.2.](http://media.tumblr.com/tumblr_m437t8pLx11r1802f.jpg)

FTFY.


task 1: program is  working fine when I run in command prompt, it does give desired output. excellent job
task2: program is  working fine when I run in command prompt, it does give desired output.great
task3: it works fine, great work

all the task has ifmain function.
some spacing issues are there between operators and equal sign.
As a whole its a awesome work.


Ditto.


tervel gives me this error. Tervel link : https://github.com/ucf-cs/Tervel/blob/master/tervel/util/memory/hp/hazard_pointer.h

tests: ../../Tervel/tervel/util/memory/hp/hazard_pointer.h:207: size_t tervel::util::memory::hp::HazardPointer::get_slot(tervel::util::memory::hp::HazardPointer::SlotID): Assertion `s < num_slots_' failed.


if you are testing this change it to like 32, this took several minutes to run


Now I'm just being a bit picky, but code style means a lot to me.  Can we have line breaks between class methods

so here it would be

```js
    ...
    this._listeners();
}

_listeners() {
    ...
```


I think having a separate function to compute it will do. We can stop passing it around and only compute it when we need it. That is a change I intend to make, partly in response to an earlier suggestion of yours.


TeamCity Kurma :: Integration Tests :: CentOS 7.2 [Build 2](http://ci.kurma.io/viewLog.html?buildId=88&buildTypeId=Kurma_IntegrationTests_CentOS72&guest=1) is now running


Please create validationMessages.properties file and move there all messages.


This pretty much kills the script..


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2169 of 2169 tests passed, 0 test warnings)

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


this is a shared thread data struct, used to share the tervel test. Each thread creates a new thread context with this tervel object. Datastore is also shared between threads


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Yep, I used js autoformat to fix a majority of the lint issues and didn't
know if that was one it needed or not. Will update when at PC.
On Mar 8, 2016 11:21 AM, "Samuel Attard" <notifications@github.com> wrote:

> Now I'm just being a bit picky, but code style means a lot to me. Can we
> have line breaks between class methods
>
> so here it would be
>
>     ...
>     this._listeners();
> }
> _listeners() {
>     ...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jostrander/Google-Play-Music-Desktop-Player-UNOFFICIAL-/commit/b7e0a0745652b615db8f0b3834521e44c064e6d0#commitcomment-16565855>
> .
>


是否应该测下 “所有的安卓手机浏览器都会触发`scroll`事件”，如果是这样的话可以判断如果是安卓则使用`scroll`事件发日志，否则使用`touch`事件，我感觉原生滚动事件性能要好点，这个点也可以测下～


How do feature test macros usually get added? I'm not sure from the rest of the Standard.


And now I see you've explained this at the bottom of the commit note. Double sorry!


copyright не надо заносить в section для этого используй либо div либо small !


I updated this with a lengthy comment explaining -- please take a look.


waiting for https://github.com/geoadmin/mf-chsdi3/pull/1966/files to change datasource from db to s3/json


`implementation-defined` italic here an below.


np: Maybe I would use a `let` here.


Is this loop necessary?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18641](https://teamcity.mesosphere.io/viewLog.html?buildId=156840&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 219 Build time: 00:03:57


в footer не выноси теги section потому что footer это в неком роде и есть секция


This is redundant, because this object is validated during creation/edition.

This is common for below Service, Label and etc which is created separately.


This is the project description.


TeamCity Kurma :: Integration Tests :: CentOS 7.2 [Build 2](http://ci.kurma.io/viewLog.html?buildId=88&buildTypeId=Kurma_IntegrationTests_CentOS72&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:02:16


Lol, sounds legit.  Link me that autoformat.  It sounds quite handy :+1:


np: Maybe "if not defined" instead of "otherwise" to make the context more clear: to find what `otherwise` means I've to read the previous test.


Excsue my ignorance, but what's the distinction between this new partial and the existing `contact_method` partial which seems to include a similar, but more extensive case statement?


Yes, it shouldn't be necessary to execute it on modern TeX distributions.

On Tue, Mar 8, 2016 at 5:13 PM, henczati <notifications@github.com> wrote:

> Was the instruction to run texhash removed intentionally?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/michal-h21/biblatex-iso690/commit/4da75b7aa399b0ed032e53f6056e2be8685bced7#commitcomment-16565606>
> .
>


Interface graphique du premier menu principal


в section не должно быть section , не плоди ты section по странице секциями ты разделяешь страницу на главные блоки со смысловой нагрузкой, если уж нравятся теги html5 то юзай article,aside и тд и тп


I think my HTML page had a unicode tick which made it here, instead of a regular apostrophe.


While creating a module is an appropriate way to address this problem, we could also consider writing some class, something like `MovieSearch` and inside of it, there can be a constructor method:

```
def initialize(search_string)
  @search_string = search_string
end
```

Then `MovieSearch#format` becomes an instance method called any `MovieSearch` object.


Fair enough. That commit message was too sparse.


Minor: I think it would've been slightly easier to see what was going on if this partial template had been inlined in a separate earlier commit.


if not defined is not clear, it require longer "if hidden timeout value is not defined" which is quite long to me


в figure заносить ссылку а потом еще и картинку это вообще бредово, ссылка должна быть родителем, а внутри размещай уже картинку


I goofed. https://github.com/lodash/babel-plugin-lodash/issues/37


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/57848bdc9c7c5fbd55a4336468434e00130efccc


merging basho/riak_kv/at-explain-debug = 57848bdc into borshop-integration-1367-at-explain-debug


basho/riak_kv/at-explain-debug = 57848bdc merged ok, testing candidate = fb91b982


I would probably make this element a `span` instead of `p` tag. The reason is `span`s display as inline by default, so that you wouldn't have to change the `display` to `inline` in your CSS later.


Rip


How do feature test macros usually get added? I'm not sure from the rest of the Standard.


I am wondering why I need to remove spaces, because I swear I'm just
copying what they tell me to do through the checkpoints.

On Tue, Mar 8, 2016 at 5:24 AM, Bobbilee <notifications@github.com> wrote:

> remove space
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/erfranson/bloccit/commit/b62ccc461f0e0e236308bc39ecd896ec4b6b38ba#commitcomment-16560151>
> .
>


that is actually my problem. I would like to use it in more describe....but maybe shared examples helps here?


I think this is a matter of taste and I happen to prefer my way.
To my eyes using parenthesis makes it look like I'm returning a tuple,
and I prefer the operators at the beginning, since it reads more naturally to me.


Code looks solid!


This cast will catch certain errors early.


Commit based on travis integration issue


заведи тег main внутри него уже размечай страницу, с начало будут идти секции, внутри которых dis,article и тд и тп!


Is this actually still needed in light of https://github.com/GNOME/ostree/pull/157.

I know we had to preserve this misbehavior in some parts of the API for backward-compat, but I forget which.


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2131 of 2131 tests passed, 0 test warnings)

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


:dancers:


ha yeah, but look at it, why would there be a space there.


Can't remove the .php from templates. Too major. It would break all existing third party themes, require modifying every file that calls a template, and would require modifying the core templating systems to accommodate. Removing the .php  will also make these files available as text in most cases, unless the users system is setup to process .inc files in a particular way.

In terms of the escape_super_globals() function, while I'd assume this will work, we'll need to see if there are any side effects. I;ve been testing with:

$_SERVER['PHP_SELF'] = htmlspecialchars($_SERVER['PHP_SELF']);

...and so far have not run into any problems.

Typo line 18.

I think also, only $_SERVER['PHP_SELF'] should be globally modified. Others would break in places where HTML is intended to be modified directly in a textarea, like the content editor, test item editor, banner editor, among others. htmlspecialchars_decode might work in such cases.


Once the number of conditionals gets beyond about two, I feel that it makes the code much clearer
to list each one on its own line. I don't like using parenthesis as it can mislead the reader into
believing a tuple is being returned.


Hello, Why did you do that?


The text around the link is slightly different, and `contact_method` is part of marriage abroad at the moment. I noticed that the GDS API now returns these links, but only as part of a blob of text, so we could even replace this content if parse those out, or maybe better - request they extend the schema to include a field with the link alone.


what? strange. to avoid this, it is better to rename field to just sql


ugggh... seems like the same stuff also got modified on TWN, and now we have a merge conflict on our hands that I don't know how to resolve. http://pastebin.com/9nwv5Jq9

Can I just pick one version over the other, or is a more intelligent merge required?


When opening a new branch, you have to start at jabref/master not at some other branch you are planning to get into jabref/master.


????? use loger instead of the Sys out prntln


Good suggestion. Will do.


Sure. I totally forgot about existence of this file, it seems like mapping between required fields of ISO 690 and biblatex fields. I don't think it is really correct anymore, as it reflects situation five years ago.


to compare deadline and Now(), could also say something like:
```go
time.Now().After(*deadline)
```


Не коммитилось, пересохронила JD01_15C в новом пакете


remove this js.

It is good practice to declare links for JS script at the top of the file


When combined with the triple-asterisk check, this double flags certain lines.


looks like you use tabs and not 4 spaces for formatting.
Could you please recap what we decided to use?


You can run sixad from command line without privileges.


...well, on Linux this was actually working almost correctly (probably the benefit of not changing desktop res); the only quirk was that ZDoom was getting minimized on restart (so you needed to Alt-Tab back to it)


I believe it is - if we didn't do this we'd iterate the NULL/default maincontext from a separate thread which is usually a bad idea.


We can change the file since its ours but the right fix is to add the missing cases. I'll look at it, thanks.


Block doesn't really seem like the right name for this to me. It's just another callable thing, but I realize that decision wasn't made in this commit.


It looks as if there are 2 lines here starting with `"? "` which seems wrong. Do you know where they came from? One of them is being deleted in this diff, but the other is being left in place.

It would be good to fix these lines in a separate PR before merging this one. It would also be good to work out how these lines got like this and/or how we might stop it happening again e.g. I think the checksum generating script probably ought to fail fast if it sees a line in the wrong format.


WTF xD


Still, I'm unsure how you would get around lack of `HsWithBndrs` in GHC 8 without employing  CPP to use the new `HsImplicitBndrs` that replaces it.


nice!!


@gabriel can you turn on eslint pre-commit hooks. you removed the call but left the import


Please move _all_ the test files. Right now this commit is just utterly incomplete with half of the tests in the java-check modules.

Please also rename the package, I don't think it should end up in a package called checks. (org.sonar.java.matcher ? )


Is that an image of your actual plugin?


Is this guaranteed to run fast?  I'm all for what the commit message says, but it looks like this operation will now block the daemon while it runs.


Code style: Spaces around operators.


for this purpose you can use good method from Appache Utils. CollectionUtils.isNotEmpty()


感谢作者


Same concern here.


Oh. I've just seen in the YAML docs that this is valid YAML - some kind of "complex mapping key". So I guess it's probably OK. I think it must've happened because the path is so long, the checksum has been pushed onto a separate line.


And here.


A couple notes:

- Your `placeholder` contents is missing quotes around it, so that for multi-word contents, the placeholder is only displaying the first word.
- Instead of a `placeholder`, I would probably set the `value` attribute of the `input`. That way, if the contents are pretty long and the user just wanted to make a minor correction, they don't have to write the whole thing over again.


Oh, yes - my bad - I didn't notice it was in marriage-abroad. It might be good to make a note in a Trello card about the points you've made in your comment.


why do you need this?


@JeroenDeDauw Cannot tell which lines create the conflict. I thought that I did not touch any message on twn. However, just move the twn stuff over our repo here. I will reapply the change. Was a two minute thing.


At this point, I would [__focus__](https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus) the text cursor in the input box so that users can just start typing right way, without having to click again.


otsukaresama


My EGO needs to be satisfied !!


I wonder why batch 32 half precision is almost 3x slower than single precision FP....at worst it should have been the same.


Looks like you also use tabs instead of 4 spaces for formatting. Please check you config in the eclipse


It's not a very nice diff to read, as you mentioned - the `unless .. else` is awkward, but used so both the conditionals related to `book_appointment_online_countries` are consistent.

With the deletion of the partial, I could of passed locals to `send_application_non_uk_visa_renew_new_colour` and added the same conditional branching in that partial, but given it was only used in two places in `making_your_application`, I felt deleting it and adding a little duplication in this file, was the lesser evil than spreading this ugly branching logic into another file.


This should most probably be called MethodMatcherFactoryTest.
The "raw" matching test are redundant with the ones that were not migrated.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18642](https://teamcity.mesosphere.io/viewLog.html?buildId=156843&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 219 Build time: 00:03:43


Misspelled.  Should be "canola".


Typo: "call" -> "calls"


+1. I think this is more interesting and practical for learners to think about.


Ah balls.  Sorry about that.


This is not an error, but repeating the test a few times, these numbers
change. I don't know why.


:smile:


The code was previously pushing 4 pixels at a time before, but this only pushes three pixels. Alpha appears to be missing.


Is it safe to dereference entry and call std::get on it in case the previous std::find_if returns backend_registry.end() ?


May I suggest `$sugar-$temperature` instead?


firstly update and then id in the URL


This is good. But one thing I tried when playing with the app is pressing escape to cancel the rename. Could we make that work as well?


the state of the block. usually just a convenience so you don't have to do world.getBlockState(pos).


add form:error for each input field and add validation


bad formatting


I think the issue is with relative paths? I wasn't able to find another plugin that uses requireIndex. I was wondering if using

```js
module.exports.rules = requireIndex(__dirname + "./rules");
```

would make more sense.


That is pretty weird, I'll have a look


remove commented code


js to the top of the file


Все верно сделал, но

1)В app.component забыл
mouseDown(event: MouseEvent) {
    if (this.tool) {
        this.tool.mouseDown();
    }
}

и аналогично с mouseMove() & mouseDown()

2)По стиля не забывай, что методы в TS лучше писать с маленькой буквы, а то они режут глаз и путаются с названиями классов

3)По структурированию кода - лучше вынести все классы Tool в отдельный файл tools.ts

3)drawerClass.ts - не хорошо, лучше drawerclass.ts (см https://google.github.io/styleguide/javascriptguide.xml, раздел Naming), а еще лучше что-то вроде elements.ts, т.к. слово class не несет ни какой смысловой нагрузки


So this one can now be factorized like the others...


Is this needed ?


Looks like you also use tabs instead of the 4 spaces for formatting jsps


This method and the two next one can be factorized.


js to the top


Now that this class is public, there is a bunch of duplicated documentation in subclasses that could be transferred here, where it naturally belongs. I will look into that.


Maybe. It's an awfully simple test so it's hard to argue that there's a code duplication issue. OTOH the compiler would probably inline a local call so there shouldn't be any cost for the extra function call. Anyway, there's still a lot of work to be done on this file.


saludo


Me gusta


As you mentioned previously, we could use `for-each` or enhanced `for` statement over here.


Perhaps add: "Please try again or manually pull $IMAGENAME using docker pull"


As with other commits, I think alphabetical here would be preferable, but we can cover that in a later tidy-up session.


Oh yes...I didn't mean to say it was a bug. I was just puzzled because at worst, half precision should always be equal to single precision unless some small overhead of converting single to half is present (which I think is not the case here). NV always has some surprises :)


"since both have now" should have been "since both have no"
Blurgh.


method should be private.


please change all the "P" to "p"


Just a note that newer versions of Node supports the `Array.from(arraylike object)` syntax, which is a lot easier to work with. Also, if you just need an array copy and `svgo.config.plugins` is already a real array, don't concat, just ask for a slice: `svgo.config.plugins.slice().sort()`. Note the lack of function into sort, since alphanumerical sort is already the default sorting function.


@Jord Please review my primitive code in the most primitive construct that I know in Java, I am sure things can be done in much nicer, beautiful way but unfortunately I am not up to date on those.

@Ankan @Amit @Sambit Please review it and ask as many questions as possible.


Also added proper recalculation of job durations.


magic number


Please write the Spring Controller in the way Zavibor did!


To the commit message, we introduced `--check` in the same release as the exit 77 so there's no chance of it breaking anything.


please describe the photon in the language we have developed.  Does it have mass?  Is it a boson?


logger!!!


Nice! Thanks for the trick! :)


With error I meant "transcription error" like last time. Of course it's
suspect, I noticed it immediately, but I didn't investigate.


Yeah, I can't think of a better name either.  o_O


looks like you want to use $(SDKROOT) instead (from our tools/depends/Makefile.include)


my redux frustration will forever live on in this commit


test


Good catch. I rebased as follows: edge change first. Then adding SRN limit change (squashing with making that limit optional, since it's all the same feature).


is this valid syntax?


__API Description file syntax error on line undefined__: ETIMEDOUT (Autogenerated message from apiary.io)


Please use NO SPACES in the file name and use latex to write τ.   Also do not use capital letters in the name.


Should we start writing test cases for our codes? We can start with JUnit test cases. We can follow TDD [Test Driven development]. First write a test-case and then write a code to pass the test case.

What do you say?


change $_POST to $_REQUEST


У меня к сожалению весь UI разсползся по экрану :(
В качестве картинок для иконок не стоит использовать такие большие по размеру файлы, потому что это очень сильно влияет на скорость закрузки страницы.

И все же, с 8 Марта Вас! Счастья, здоровья и успехов!


@vpratfr rename Userconfig.example to Userconfig and run ". do-update-vagrant.sh"

fix for W10 + vbox >v5.0.12 + vagrant > 1.8.1


## Task 1

Write a program to count word usage in the given text.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.  You should use a *standard* dictionary as part of your function(s) to accomplish this task (see Task 2, where you'll use a slightly different type of dictionary).

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a dictionary to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 2

Using [Counter](https://docs.python.org/2/library/collections.html#collections.Counter) from the `collections` module, write a program to count word usage in the same text as above.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a `Counter` to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 3

Write a program that processess the given data to create a dictionary that maps the ExaML run name (e.g. ExaML_info.T6 as the `key`) to the ExaML log-likelihood value (e.g. -82924194.67 as the `value`). Use this function or a function that calls this function to prettily print the run name followed by the log-likelihood value in two column format to the screen.  Also include a function that sorts the log-likelihood values and helps us determine which run name minimizes the negative log likelihood value (e.g. the run name having the LL value closest to zero). Also output the name of this "best" run to the screen.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program prints run name/LL in two column format and correctly sorts and reports the "best" ML result (4 pts.)


你这两个 commit 是怎么研究出来的?


Fair enough


@williewillus How about the duplicate `IBlockState` parameters here?


The code looks good to me. Pretty much descriptive and commented properly.


Muon12 is the file name not the glossary entry name or name.  Use lower case "muon".
Otherwise great


Same as [here] (https://github.com/MinecraftForge/MinecraftForge/commit/776eb00ed394f2e059f26127337ae2117d605c8b#commitcomment-16566674).


it looks you have forgotten to commit the Metadata class

see https://travis-ci.org/MetadataRegistry/ModelCataloguePlugin/jobs/114561603 failing as it cannot be found


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/3306


Omh, I hope you simplify this method


ah yes, I see the copy pasta typo, line 17. Yes, makes sence, but in those places I would take the approach of applying htmlspecialchars_decode() as you said. Its a white list approach which is much more secure.


I squashed this commit with the introduction of the SNR limit, so the change no longer exists as a change, but I'll explain the value in the commit msg.


@jordmoz Please review my primitive code in the most primitive construct that I know in Java, I am sure things can be done in much nicer, beautiful way but unfortunately I am not up to date on those.

@ankanm @amitbose327 @04msambit Please review it and ask as many questions as possible.


Suggestion: use composition over inheritance; by wrapping another `TimestampGenerator` this class would be immediately usable, no need to subclass it, and we wouldn't need to add warning thresholds everywhere in constructors of its subclasses.


You can use here CollectionUtils.IsNotEmpty() from apache library


i don't think so, but this does seem odd that the total number did not change. from this block:

https://github.com/dimagi/commcare-hq/blob/9bbbb29adc87a6740a0770799031e84ca62e9157/corehq/form_processor/tests/test_ledgers.py#L115
```
submit_case_blocks([
            CaseBlock(case_id=self.case.case_id, update={'a': "1"}).as_string(),
            BALANCE_BLOCK.format(
                case_id=self.case.case_id,
                product_id=self.product_a._id,
                quantity=100
            )],
            DOMAIN
        )
```
It seems that there is an update case block changing `a` to `1`, which accounts for one transaction. and there's a ledger blocker which accounts for the 2nd transaction. and then finally there's a case create transaction which accounts for the 3rd transaction


move to the separate css file


У хилов всегда мана отображается, а у дд и танков все абилки используют энергию, даже на отхил.


@borshop merge


Please use `element.getParentOfType()` instead of an explicit loop here and in the other macro.


Main change: manage history: undo and redo
Addition changes:
Fix rotation bug (90°, -90°)
Open a file given in command argument
Fix save file (with new data structure)
Change variable name: open_file
Update doc in create_tab
Improve create_pixbuf
Modify filter and create edit_image


I already switch to cm gps hal and update those blobs and have now issue of all varients.


move to the separate cs file


js to the top of the file


I'll solve that by providing default flag values below to support the AOSP/NDK build.  But I'd like this function to still do something sensible even when `args.cmake` is missing.


Great! Thank you!


no idea


Очень ждем новую версию в nuget.


> There's a difference between passing float, np.float32 and np.float64 as a stype to np.zeros.

s/stype/dtype. Fat-finger fail.


It might be better (and more consistent) to make this a separate TaskResource class in the applications package. On the other hand, this doesn't look like other Resource<...Entity> objects, so maybe not. Should we call it something else other than a Resource?


Good suggestion.


cf. #2
<!--
jenkins: report-done
-->


using style attribute is bad habit. The best solution is to create css class and use it. But I belive bootstrap has classes for different width.


Build failure reported at #3.

<!--
jenkins: report-done
-->


WELCOME TO BALRRHUB[](url)HTTP://WWW.BALRR.COM


Thanks for the suggestions, updated in b4023e15663f83a00870982ebf69309def6d6529.


I added another check `if typeof jQuery !== 'undefined'` here before checking if the template is an instance of jQuery


SourceForge is evil. Good riddance!

(Always nice to meet another OpenBSD user here, too; hello!)


Use log4j instead of the System.out.println


shoot. ordering is wrong.


I cannot understand this topic. I think we need example what are `Low Level`, `Mid Level`, `High Level`
Also 'unsafe' properties, 'unsafe' methods


Good catch. Probably not.


shouldn't it be:
ls990/proprietary/etc/sensors/sensor_def_variable.conf
instead:
ls990/sensor_def_variable.conf
?


does it make sense to use IPMI_CC_OK instead just in case ?


Good point


when i take out the additional case block, i only get 2 transactions. a create and a ledger


OK


It is not good idea to pass title from controller to the JSP. Better to set some flag ie. isUpdate and based on that flag use appropriate title on the JSP.

Need to separate presentation layer from controller layer.


Thank you for this fix.  The comment was not updated.  Also, the documentation for "2.2.4 Rendering an Arbitrary File" at http://guides.rubyonrails.org/v3.2.22/layouts_and_rendering.html#using-render
is no longer accurate.


Mmm, the separation of old_versions introduces another step. Nothing to worry about though.


it is better to use @Validated annotation


then it should be ran as sudo from the commandline rather than having sudo in the script really.


WTF..?


I disagree. I rarely provide module level doc strings when the module contains code that is thoroughly documented (and tasks tend to be very, very thoroughly documented). I think this is common practice in the project, especially for tasks. They even have their own page on Doxygen, which is how most folks (we hope) will find the task and learn how to use it.


This seems like a poor fix since now the error reporting will just provide unhelpful text for the scenario i described. Can we add another `except` block that catches `DisallowedHost` and tries to provide better information? Or can we at least send the `exc_info` in that case? Perhaps `render_to_string` something with the info?


fast-forwarding riak_ts-develop to borshop-integration-1367-at-explain-debug = fb91b982


Захардкодить переменную окружения это не очень правильно, надо бы от этого уйти. @gldmtr твое мнение?


plus you check for root anyway don't you ?


As I sad for Taras, please move all this messages to the ValidationMessage.properties


@freeman-lab thought I would get this rolling, curious what you think of the API laid out in this example.

This package is so general, that I'm tempted to say that it should be written against a `BoltArray` rather than `Thunder` objects .


task 2 returns the ten most common words in a nice format, but it returns the incorrect count for "the"


bad formatting. (tabs instead of the 4 spaces)


looks good.
this might intermittently fail on Travis.. For instance, the timer could start, and then `acquireWithTimeTaken()` isn't called until a couple seconds later. (sometimes things are strangely slow on Travis!)
But if it seems to work reliably, then good enough! I don't have a better suggestion yet.


bad formatting


Build failure reported at #4.

<!--
jenkins: report-done
-->


Sure enough there is! Thanks for the idea.

```
$ pip install git+https://github.com/numba/llvmlite@master
```


Pourquoi cette modification? Cet exemple est erroné.
Il faut y remettre les paramètres originaux `["key", "val"]`.


If the whitelist approach, we'll need  foreach($_REQUEST)  block, in addition to converting POST to REQUEST. There are places where POST data is access through REQUEST


haha, nice!


## Task 1

Write a program to count word usage in the given text.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.  You should use a *standard* dictionary as part of your function(s) to accomplish this task (see Task 2, where you'll use a slightly different type of dictionary).

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a dictionary to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 2

Using [Counter](https://docs.python.org/2/library/collections.html#collections.Counter) from the `collections` module, write a program to count word usage in the same text as above.  Have this program print out, in decreasing order (most common to least common) the 10 most common words and the count of each word used. As always, your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses a `Counter` to correctly count words in the text and returns correct words and their counts for the 10 most common words (4 pts.)

## Task 3

Write a program that processess the given data to create a dictionary that maps the ExaML run name (e.g. ExaML_info.T6 as the `key`) to the ExaML log-likelihood value (e.g. -82924194.67 as the `value`). Use this function or a function that calls this function to prettily print the run name followed by the log-likelihood value in two column format to the screen.  Also include a function that sorts the log-likelihood values and helps us determine which run name minimizes the negative log likelihood value (e.g. the run name having the LL value closest to zero). Also output the name of this "best" run to the screen.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program prints run name/LL in two column format and correctly sorts and reports the "best" ML result (4 pts.)


I am using Git and GitHub desktop, so with this I am syncing.


please move this css to some separate css file and load it. Maybe you have some application wide css file...


Build failure reported at #5.

<!--
jenkins: report-done
-->


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheonsite.io/docs/articles/sites/code/htaccess)
- [docs/articles/sites/code/redirect-incoming-requests](http://ttrowellht-static-docs.pantheonsite.io/docs/articles/sites/code/redirect-incoming-requests)


am i stupid or is that duplicated code?


Yeah! Logos in the header is actually my core use case :)

http://i.imgur.com/ufE6G1g.png


nothing, it was just a joke we made in the glua discord


Minor comment: I assume this file is auto-generated from sources. It doesn't have the header stating so, though. You know, the:
```
--DO NOT MODIFY THIS FILE, IT IS GENERATED AUTOMATICALLY FROM SOURCES
-- Complain if script is sourced in psql, rather than via CREATE EXTENSION
\echo Use "CREATE EXTENSION cdb_dataservices_server" to load this file. \quit
```

(not fully accurate for this file as it should say `ALTER EXTENSION cdb_dataservices_server UPDATE TO '0.5.1'`)


 aah you have sudo in that check too - just checked the script - it is really the wrong way to do it imho - with sudo everywhere - just do like in retropie_setup and prompt if not root

```
# check, if sudo is used
if [[ $(id -u) -ne 0 ]]; then
    echo "Script must be run as root. Try 'sudo $0'"
    exit 1
fi
```


{@link InProcessTransformResult} result


I realise this script is old and this may have been done before you did any changes :)


if *what* is a root transform?


Trying to learn how to use git…


todo: upload


!Iterables.isEmpty(committed)


I guess you took care on making this `kind` match the values from the editor, can you please confirm?


Move the comment up since it is explaining the if, not the completed.add(...)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18643](https://teamcity.mesosphere.io/viewLog.html?buildId=156859&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:05


Ah I see, of course.


task 1 not working. global vs. local variable issues. some mis-named variables.
task 3 returns nice two-column list, but the incorrect best run


Hi, are you sure about that? As far as I know, we should always return `this.ready()` instead of cursor.


But how would that work, given that `onDrift` is called from another instance method?


I cannot rule out the possibility that a little chmod may be required, though, since the device from which this setup is done does not support the "x" attribute. Mac adds this automatically. Linux does not (why?)


You don't really need this, since your config.yaml sets the default level to WARNING.

ctxt['log_level'] = config('log-level')


See my previous comment


iteration (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2363 of 2398 tests passed, 0 test warnings)

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


Do we actually need to be synchronized while executing the callback? If not, restructure so that we only synchronize around the code that pulls callbacks off -- which may not need to be synchronized:
1. If this is the only thread that removes from the pending transform callbacks, then we'll never see a transition from !isEmpty to isEmpty that is caused elsewhere.
2. If we see an  item on the callbacks that has a watermark that should fire, then even if the element that we get back from poll has displaced it, it should still fire (since it is less than the thing that was less than the output watermark).


@Prozite faut changer ça par <identity($usrFirstname)>.<identity($usrLastname)>@gmail.com


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1581 of 1581 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-70.86%25-green.svg)


Ah, you're saying I need [this](https://github.com/CMRemix/android_device_samsung_trlte-common/commit/a7281ba43e4bd0c8b16c2da1f50199fe0c4b91a1.patch)... gotcha.  @niv71 I'll try building w/that and see...


The workflow is/should be: ES2015 + babelify + uglify.


Consider

```
return input.isKeyed()
    ? InProcessBundle.keyed(output, input.getKey())
    : InProcessBundel.unkeyed(output);
```


This change should also change the default action otherwise it wont work.


```
#include <iostream>

int main() {

    std::string a("a");
    std::string b("b");

    // Binary operator '+' can't be applied to the expressions of type 'std::string' and 'std::string'
    a = a + b;

    // Binary operator '+' can't be applied to the expressions of type 'std::string' and 'const char[2]'
    a = a + "x";

    return 0;
}
```


I was surprised too. I kept it this way for consistency with the VB method. But I can change both to make more sense.


It's not completely clear how pendingCallbacks, pendingTransformCallbacks, and callbacks are all related. Is there a good spot to put some explanation of that?


Nice catch.


It should read "cardinality".
"cardinology" sounds like it's sth. to do with the Vatican ;)


Makes sense. I've tried to think of a better way than inlining the `EnsureAvailableType` code below in each method, but couldn't think of any. Any tips?


This class merits more comments on how the pieces fit together. Please add some that explain the responsibilities of this class, how the methods fit together, and how the fields are used, etc.


Yes I think you have got the restriction site thing.
On 8 Mar 2016 3:01 pm, "GabrieleNocchi" <notifications@github.com> wrote:

> p.s. I mite have just understood what you mean with the restriciton site
> reverse transcription. Basically if a motif is for example 5-attcg-3 i need
> to search for that motif and also for its complementary, which is
> 5-cgaat-3, still in the DNA same strand right?
>
> Date: Tue, 8 Mar 2016 03:57:32 -0800
> From: notifications@github.com
> To: Biocomp2Project@noreply.github.com
> CC: g.nocchi@hotmail.com
> Subject: Re: [Biocomp2Project] Removed another unused line. (e0a99ef)
>
> Shouldn't this return an array of start positions and an array of end
> positions in the gene sequence to be given to @DolapoA 's layer where he
> adds the HTML?
>
>
> —
> Reply to this email directly or view it on GitHub.
>
>
>
>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/RichardJActon/Biocomp2Project/commit/e0a99efcbc78bbd8af4ff1552ff23972e415b579#commitcomment-16563602>
> .
>


always 0 used_credits?


ok, there's no need to overcomplicate it.  Can you add a comment above the line explaining the "- 2" please?


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.73](http://build.clear-measure.com/viewLog.html?buildId=7599&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


I don't know @cedricpinson ?

In any way we need to dynamically swap between vr and non vr mode.


Thanks for contributing like this.  This makes it so much higher quality and nicer for everybody.  :+1:


I agree. I just made a test to get rid of some errors. There is already plenty of ES6 code.


@SnShine  I am using Jupyter, sorry. I'm used to calling it iPython. But I still get the error that relative pathing doesn't seem to work.

I tested it and if I changed the name to utils_aima.py, I run into not problems. This is just one solution, but perhaps a better one?


This method is getting a bit long. Consider adding some documented helper methods. The fact that this has two separate loops suggests you could pull each of those out, for instance.


I personally don't like that you increment weekStart property directly on locales object. My suggestion would be to create local variable like you do in renderMonth_() method. Could you fix that so i can merge PR.


I don't know the details you agreed with @santisaez regarding this format but:
- I'd make ES/kibana aggregate all geocodings (from editor and sql api)
- I'd add a field to distinguish between them. I don't know if the "event source" or the "program" fields are good enough for that.


I can put The if in The for  if you like to see that


Actually I have a better version of Riemann docker health which also collects the LSOF of every container. Would you like me to merge it?


changing a versioning name before youve done anything to a kernel to make it different, and putting the original person doing all the leg work as fifth down in your list of people to thank is still weak...Joe brought this a LONG way from stock CM kernel, and dumping 15 commits into it on top of his 100s doesnt really make it "yours"...A little more credit for the work done prior is always nice..and youd want it the same way, as would anyone...


Put these on separate lines. It makes it easier to verify that both are checked, and will more gracefully expand in the future.
```
return Objects.equals(...)
    && Objects.equals(...);
```


Even though this isn't public, please add Javadoc explaining what its role is.


ah ok, i am stupid :D the status is different ;)


@jameskoster IMO help tip icons are extra small. Need to increase the font-size :)


gracias


I had a different interpretation of that part of PEP-8 when I suggested it. What I got from it was that module level global variables should have its use limited to the module itself, be them imported directly or used from another module namespace. Anyway, this is more of a "desirable" than a "requirement" from PEP-8, and should be done project wise, so I'm fine with this.


Looks like more needed removal, not less...meaning the table data tags on either side of the sets of php conditionals removed. lines 652 and 653 start a new row/cell.  The next two conditionals are removed, leaving lines 653-664 wrapped in a tag that does nothing, right?


OK.


right, fixing...


This library supports discovery through three different mechanisms. There is pure WiFi and that should work on all versions but what's the point? If you just want WiFi then go grab mDNS or SSDP and call it a day. There isn't value here for that. The other mechanism is BLE/Bluetooth. But we use a version of BLE that is only supported on Lollipop and higher. Finally we have a legacy mechanism that doesn't work reliable and that is WiFi Direct. The good news on WiFi Direct is that it is supported on older platforms. The bad news on WiFi Direct is that it doesn't work. We have tested on various phones, software versions, etc. And it just won't work right in any kind of reliable way. So personally I wouldn't use this code on anything below lollipop but @tompaana probably has a different view. :)


an quitadola lista y no e podido verla


I didn't center it for a reason :)


Sure! A PR would be great.


@smcmahon Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/2/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/csrf.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/inline_validation.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/inline_validation.py:18:12: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:20:16: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:23:1: W293 blank line contains whitespace
plone/app/z3cform/inline_validation.py:24:26: Q000 Remove bad quotes.
plone/app/z3cform/inline_validation.py:29:9: E265 block comment should start with '# '
plone/app/z3cform/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/interfaces.py:3:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/layout.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/layout.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/layout.py:2:1: F401 'FormWrapper' imported but unused
plone/app/z3cform/layout.py:2:1: F401 'wrap_form' imported but unused
plone/app/z3cform/object.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/object.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/templates.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:16:1: E731 do not assign a lambda expression, use a def
plone/app/z3cform/templates.py:16:80: E501 line too long (90 > 79 characters)
plone/app/z3cform/templates.py:35:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/templates.py:36:1: W293 blank line contains whitespace
plone/app/z3cform/templates.py:41:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/templates.py:43:1: E302 expected 2 blank lines, found 0
plone/app/z3cform/utils.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/utils.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/utils.py:41:13: E129 visually indented line with same indent as next logical line
plone/app/z3cform/utils.py:41:13: W503 line break before binary operator
plone/app/z3cform/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:13:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:18:1: E303 too many blank lines (3)
plone/app/z3cform/widget.py:21:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:24:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:29:1: E303 too many blank lines (3)
plone/app/z3cform/widget.py:32:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:35:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:38:1: W391 blank line at end of file
plone/app/z3cform/select/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/select/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/select/__init__.py:1:1: F401 'SelectFieldWidget' imported but unused
plone/app/z3cform/select/__init__.py:1:1: F401 'SelectWidget' imported but unused
plone/app/z3cform/select/__init__.py:1:51: W292 no newline at end of file
plone/app/z3cform/select/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/select/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/select/select.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/select/select.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/select/select.py:43:1: E402 module level import not at top of file
plone/app/z3cform/select/select.py:106:17: Q000 Remove bad quotes.
plone/app/z3cform/select/select.py:107:17: Q000 Remove bad quotes.
plone/app/z3cform/select/select.py:109:17: Q000 Remove bad quotes.
plone/app/z3cform/select/select.py:124:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/select/select.py:133:80: E501 line too long (101 > 79 characters)
plone/app/z3cform/queryselect/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/queryselect/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/queryselect/__init__.py:2:1: F401 'schema' imported but unused
plone/app/z3cform/queryselect/__init__.py:12:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/queryselect/__init__.py:20:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:22:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:29:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:33:27: E701 multiple statements on one line (colon)
plone/app/z3cform/queryselect/__init__.py:39:27: E701 multiple statements on one line (colon)
plone/app/z3cform/queryselect/__init__.py:41:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:47:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/queryselect/__init__.py:49:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/queryselect/__init__.py:55:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/__init__.py:3:1: F401 'WysiwygFieldWidget' imported but unused
plone/app/z3cform/wysiwyg/__init__.py:4:1: W391 blank line at end of file
plone/app/z3cform/wysiwyg/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/widget.py:10:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/widget.py:13:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/widget.py:15:1: W293 blank line contains whitespace
plone/app/z3cform/wysiwyg/widget.py:32:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/example.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/example.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:13:25: E261 at least two spaces before inline comment
plone/app/z3cform/tests/example.py:32:1: W293 blank line contains whitespace
plone/app/z3cform/tests/example.py:36:25: E261 at least two spaces before inline comment
plone/app/z3cform/tests/example.py:47:1: W391 blank line at end of file
plone/app/z3cform/tests/layer.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/layer.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/layer.py:51:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/test_csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_objectsubform.py:34:35: S001 found %s formatter
plone/app/z3cform/tests/test_objectsubform.py:115:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/tests.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/tests.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/tests.py:4:1: F401 'z3c' imported but unused
plone/app/z3cform/tests/tests.py:13:1: F401 'IntegrationLayer' imported but unused
plone/app/z3cform/tests/tests.py:40:14: D001 found failUnless( replace it with assertTrue
plone/app/z3cform/tests/tests.py:46:14: D001 found failUnless( replace it with assertTrue
plone/app/z3cform/tests/tests.py:51:80: E501 line too long (86 > 79 characters)
plone/app/z3cform/tests/tests.py:52:13: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:53:13: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:53:80: E501 line too long (99 > 79 characters)
plone/app/z3cform/tests/tests.py:58:9: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:59:9: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:59:80: E501 line too long (94 > 79 characters)
plone/app/z3cform/tests/tests.py:60:9: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:66:80: E501 line too long (92 > 79 characters)
plone/app/z3cform/tests/tests.py:67:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:67:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:68:13: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:70:80: E501 line too long (96 > 79 characters)
plone/app/z3cform/tests/tests.py:71:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:71:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:72:13: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:74:80: E501 line too long (106 > 79 characters)
plone/app/z3cform/tests/tests.py:75:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:75:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:76:13: E124 closing bracket does not match visual indentation
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@esteele Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/3/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/interfaces.py:3:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/layout.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/layout.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/layout.py:2:1: F401 'FormWrapper' imported but unused
plone/app/z3cform/layout.py:2:1: F401 'wrap_form' imported but unused
plone/app/z3cform/templates.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/templates.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:16:1: E731 do not assign a lambda expression, use a def
plone/app/z3cform/templates.py:16:80: E501 line too long (90 > 79 characters)
plone/app/z3cform/templates.py:35:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/templates.py:36:1: W293 blank line contains whitespace
plone/app/z3cform/templates.py:41:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/templates.py:43:1: E302 expected 2 blank lines, found 0
plone/app/z3cform/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:13:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:18:1: E303 too many blank lines (3)
plone/app/z3cform/widget.py:21:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:24:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:29:1: E303 too many blank lines (3)
plone/app/z3cform/widget.py:32:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:35:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/widget.py:38:1: W391 blank line at end of file
plone/app/z3cform/queryselect/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/queryselect/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/queryselect/__init__.py:2:1: F401 'schema' imported but unused
plone/app/z3cform/queryselect/__init__.py:12:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/queryselect/__init__.py:20:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:22:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:29:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:33:27: E701 multiple statements on one line (colon)
plone/app/z3cform/queryselect/__init__.py:39:27: E701 multiple statements on one line (colon)
plone/app/z3cform/queryselect/__init__.py:41:1: W293 blank line contains whitespace
plone/app/z3cform/queryselect/__init__.py:47:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/queryselect/__init__.py:49:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/queryselect/__init__.py:55:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/__init__.py:3:1: F401 'WysiwygFieldWidget' imported but unused
plone/app/z3cform/wysiwyg/__init__.py:4:1: W391 blank line at end of file
plone/app/z3cform/wysiwyg/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/widget.py:10:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/widget.py:13:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/wysiwyg/widget.py:15:1: W293 blank line contains whitespace
plone/app/z3cform/wysiwyg/widget.py:32:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/kss/validation.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/kss/validation.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/kss/validation.py:52:1: W293 blank line contains whitespace
plone/app/z3cform/kss/validation.py:53:26: Q000 Remove bad quotes.
plone/app/z3cform/kss/validation.py:58:9: E265 block comment should start with '# '
plone/app/z3cform/kss/validation.py:86:30: S001 found %s formatter
plone/app/z3cform/kss/validation.py:88:30: S001 found %s formatter
plone/app/z3cform/kss/validation.py:89:50: E127 continuation line over-indented for visual indent
plone/app/z3cform/kss/validation.py:92:31: S001 found %s formatter
plone/app/z3cform/kss/validation.py:93:31: S001 found %s formatter
plone/app/z3cform/tests/example.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/example.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:13:25: E261 at least two spaces before inline comment
plone/app/z3cform/tests/example.py:32:1: W293 blank line contains whitespace
plone/app/z3cform/tests/example.py:36:25: E261 at least two spaces before inline comment
plone/app/z3cform/tests/example.py:47:1: W391 blank line at end of file
plone/app/z3cform/tests/layer.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/layer.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/layer.py:7:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/layer.py:8:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:11:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:15:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:18:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/layer.py:19:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:28:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:30:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:45:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:47:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:50:1: W293 blank line contains whitespace
plone/app/z3cform/tests/layer.py:51:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_kss_attr.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_kss_attr.py:12:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/test_kss_attr.py:13:1: W293 blank line contains whitespace
plone/app/z3cform/tests/test_kss_attr.py:22:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/test_kss_attr.py:25:40: S001 found %s formatter
plone/app/z3cform/tests/test_kss_attr.py:44:80: E501 line too long (117 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:48:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/test_kss_attr.py:51:40: S001 found %s formatter
plone/app/z3cform/tests/test_kss_attr.py:62:80: E501 line too long (85 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:63:1: W293 blank line contains whitespace
plone/app/z3cform/tests/test_kss_attr.py:67:1: W293 blank line contains whitespace
plone/app/z3cform/tests/test_kss_attr.py:77:80: E501 line too long (80 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:82:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:84:80: E501 line too long (85 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:92:1: W293 blank line contains whitespace
plone/app/z3cform/tests/test_kss_attr.py:97:80: E501 line too long (118 > 79 characters)
plone/app/z3cform/tests/test_kss_attr.py:101:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/tests.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/tests.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/tests.py:29:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:36:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:39:14: D001 found failUnless( replace it with assertTrue
plone/app/z3cform/tests/tests.py:40:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:43:14: D001 found failUnless( replace it with assertTrue
plone/app/z3cform/tests/tests.py:44:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:49:14: D001 found failUnless( replace it with assertTrue
plone/app/z3cform/tests/tests.py:51:1: E302 expected 2 blank lines, found 1
plone/app/z3cform/tests/tests.py:52:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:54:13: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:55:13: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:55:80: E501 line too long (99 > 79 characters)
plone/app/z3cform/tests/tests.py:58:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:61:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:62:80: E501 line too long (92 > 79 characters)
plone/app/z3cform/tests/tests.py:63:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:63:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:64:13: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:66:80: E501 line too long (96 > 79 characters)
plone/app/z3cform/tests/tests.py:67:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:67:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:68:13: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:70:80: E501 line too long (106 > 79 characters)
plone/app/z3cform/tests/tests.py:71:17: E128 continuation line under-indented for visual indent
plone/app/z3cform/tests/tests.py:71:80: E501 line too long (93 > 79 characters)
plone/app/z3cform/tests/tests.py:72:13: E124 closing bracket does not match visual indentation
plone/app/z3cform/tests/tests.py:73:1: W293 blank line contains whitespace
plone/app/z3cform/tests/tests.py:75:1: W293 blank line contains whitespace
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


You convenienced me :)


@cedricmessiant Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.relationfield/2/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/__init__.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/behavior.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/behavior.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/behavior.py:25:23: E128 continuation line under-indented for visual indent
plone/app/relationfield/behavior.py:29:80: E501 line too long (90 > 79 characters)
plone/app/relationfield/demo.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/demo.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/demo.py:18:28: E128 continuation line under-indented for visual indent
plone/app/relationfield/demo.py:19:28: E128 continuation line under-indented for visual indent
plone/app/relationfield/demo.py:20:22: E128 continuation line under-indented for visual indent
plone/app/relationfield/demo.py:20:33: Q000 Remove bad quotes.
plone/app/relationfield/demo.py:22:24: E128 continuation line under-indented for visual indent
plone/app/relationfield/demo.py:23:24: E128 continuation line under-indented for visual indent
plone/app/relationfield/event.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/event.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/event.py:10:1: C901 'extract_relations' is too complex (13)
plone/app/relationfield/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/marshaler.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/marshaler.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/marshaler.py:4:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/marshaler.py:8:1: W293 blank line contains whitespace
plone/app/relationfield/marshaler.py:10:1: W293 blank line contains whitespace
plone/app/relationfield/marshaler.py:15:1: W293 blank line contains whitespace
plone/app/relationfield/marshaler.py:16:80: E501 line too long (92 > 79 characters)
plone/app/relationfield/monkey.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/path.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/path.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/path.py:14:1: W293 blank line contains whitespace
plone/app/relationfield/path.py:16:1: W293 blank line contains whitespace
plone/app/relationfield/path.py:27:1: W293 blank line contains whitespace
plone/app/relationfield/path.py:32:1: W293 blank line contains whitespace
plone/app/relationfield/setuphandlers.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/setuphandlers.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/setuphandlers.py:8:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/setuphandlers.py:12:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/setuphandlers.py:20:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/setuphandlers.py:25:12: Q000 Remove bad quotes.
plone/app/relationfield/source.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/source.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/source.py:10:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/source.py:35:42: S001 found %s formatter
plone/app/relationfield/source.py:44:21: W292 no newline at end of file
plone/app/relationfield/tests.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/tests.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/tests.py:5:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/tests.py:6:1: W293 blank line contains whitespace
plone/app/relationfield/tests.py:11:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/tests.py:12:1: W293 blank line contains whitespace
plone/app/relationfield/tests.py:13:80: E501 line too long (83 > 79 characters)
plone/app/relationfield/tests.py:15:1: W293 blank line contains whitespace
plone/app/relationfield/widget.py:0:1: C101 Coding magic comment not found
plone/app/relationfield/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/relationfield/widget.py:15:1: E302 expected 2 blank lines, found 1
plone/app/relationfield/widget.py:137:1: W391 blank line at end of file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


But meh, it's fine


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.relationfield/3/violations
```
plone/app/relationfield/event.py:9:1: C901 'extract_relations' is too complex (13)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/6/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/converters.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:14:1: C901 'InlineValidationView.__call__' is too complex (12)
plone/app/z3cform/inline_validation.py:23:12: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:25:16: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:29:26: Q000 Remove bad quotes.
plone/app/z3cform/inline_validation.py:34:9: E265 block comment should start with '# '
plone/app/z3cform/inline_validation.py:41:17: E731 do not assign a lambda expression, use a def
plone/app/z3cform/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/interfaces.py:6:80: E501 line too long (85 > 79 characters)
plone/app/z3cform/layout.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/layout.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/layout.py:2:1: F401 'FormWrapper' imported but unused
plone/app/z3cform/layout.py:2:1: F401 'wrap_form' imported but unused
plone/app/z3cform/object.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/object.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/templates.py:13:1: E731 do not assign a lambda expression, use a def
plone/app/z3cform/utils.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/utils.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:5:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/widget.py:140:29: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:329:26: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:329:27: S001 found %s formatter
plone/app/z3cform/widget.py:396:30: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:396:31: S001 found %s formatter
plone/app/z3cform/widget.py:500:35: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:511:42: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:512:44: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:514:34: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:537:28: P101 format string does contain unindexed parameters
plone/app/z3cform/tests/example.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/example.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:16:16: Q000 Remove bad quotes.
plone/app/z3cform/tests/layer.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/layer.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/layer.py:33:10: Q000 Remove bad quotes.
plone/app/z3cform/tests/test_csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_objectsubform.py:34:35: S001 found %s formatter
plone/app/z3cform/tests/test_widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_widget.py:32:80: E501 line too long (126 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_widgets.py:547:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:550:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:766:36: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:768:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:957:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:997:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1020:80: E501 line too long (82 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1116:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:1124:39: Q000 Remove bad quotes.
plone/app/z3cform/tests/tests.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/tests.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/queryselect/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/queryselect/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/__init__.py:2:1: F401 'WysiwygFieldWidget' imported but unused
plone/app/z3cform/wysiwyg/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/widget.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@vgokhale As far as I remember TX1 is the only nVIDIA product that has a special hardware support for FP16, so it might indeed be faster. I think it's coming to Pascal GPUs too.


@vangheem Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/7/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/captcha.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/captcha.py:44:80: E501 line too long (86 > 79 characters)
plone/app/z3cform/converters.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:14:1: C901 'InlineValidationView.__call__' is too complex (12)
plone/app/z3cform/inline_validation.py:23:12: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:25:16: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:29:26: Q000 Remove bad quotes.
plone/app/z3cform/inline_validation.py:34:9: E265 block comment should start with '# '
plone/app/z3cform/inline_validation.py:41:17: E731 do not assign a lambda expression, use a def
plone/app/z3cform/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/interfaces.py:1:80: E501 line too long (85 > 79 characters)
plone/app/z3cform/interfaces.py:60:46: W292 no newline at end of file
plone/app/z3cform/layout.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/layout.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/layout.py:2:1: F401 'FormWrapper' imported but unused
plone/app/z3cform/layout.py:2:1: F401 'wrap_form' imported but unused
plone/app/z3cform/object.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/object.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/templates.py:13:1: E731 do not assign a lambda expression, use a def
plone/app/z3cform/utils.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/utils.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:5:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/widget.py:140:29: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:329:26: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:329:27: S001 found %s formatter
plone/app/z3cform/widget.py:396:30: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:396:31: S001 found %s formatter
plone/app/z3cform/widget.py:466:48: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:516:35: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:527:42: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:528:44: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:530:34: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:553:28: P101 format string does contain unindexed parameters
plone/app/z3cform/queryselect/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/queryselect/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/__init__.py:2:1: F401 'WysiwygFieldWidget' imported but unused
plone/app/z3cform/wysiwyg/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/example.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:16:16: Q000 Remove bad quotes.
plone/app/z3cform/tests/layer.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/layer.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/layer.py:33:10: Q000 Remove bad quotes.
plone/app/z3cform/tests/test_csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_objectsubform.py:34:35: S001 found %s formatter
plone/app/z3cform/tests/test_widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_widget.py:32:80: E501 line too long (126 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_widgets.py:548:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:551:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:767:36: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:769:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:961:18: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:961:80: E501 line too long (80 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:984:80: E501 line too long (82 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1003:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1026:80: E501 line too long (82 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1122:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:1130:39: Q000 Remove bad quotes.
plone/app/z3cform/tests/tests.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/tests.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


I actually did extract the commands myself before finding your repo. they are basically the same. one difference i think is that the heart-alive command times out on the AVR170 (i think in your unit sends a response?)

I was trying to write a [homebridge](https://github.com/nfarina/homebridge) plugin for the 170 ([here](https://github.com/andrei4002/homebridge-harman-kardon-avr)), but having all the commands timeout is basically making the development (at least for myself) kind of useless, since having one way triggers means that I will never actually have a working state of the unit (knowing if it's on or off or what source, volume etc).

hell, i can't even set the goddamn volume to a specific value. they did a terrible job implementing the software for these units


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.73](http://build.clear-measure.com/viewLog.html?buildId=7599&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 13 Build time: 00:05:53


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/8/violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
plone/app/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/converters.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/inline_validation.py:14:1: C901 'InlineValidationView.__call__' is too complex (12)
plone/app/z3cform/inline_validation.py:23:12: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:25:16: P002 found "hasattr", consider replacing it
plone/app/z3cform/inline_validation.py:29:26: Q000 Remove bad quotes.
plone/app/z3cform/inline_validation.py:34:9: E265 block comment should start with '# '
plone/app/z3cform/inline_validation.py:41:17: E731 do not assign a lambda expression, use a def
plone/app/z3cform/interfaces.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/interfaces.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/interfaces.py:6:80: E501 line too long (85 > 79 characters)
plone/app/z3cform/layout.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/layout.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/layout.py:2:1: F401 'FormWrapper' imported but unused
plone/app/z3cform/layout.py:2:1: F401 'wrap_form' imported but unused
plone/app/z3cform/object.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/object.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/templates.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/templates.py:13:1: E731 do not assign a lambda expression, use a def
plone/app/z3cform/utils.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/utils.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/widget.py:7:80: E501 line too long (83 > 79 characters)
plone/app/z3cform/widget.py:143:29: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:338:26: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:338:27: S001 found %s formatter
plone/app/z3cform/widget.py:419:30: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:419:31: S001 found %s formatter
plone/app/z3cform/widget.py:498:58: Q000 Remove bad quotes.
plone/app/z3cform/widget.py:545:35: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:556:42: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:557:44: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:559:34: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:582:28: P101 format string does contain unindexed parameters
plone/app/z3cform/queryselect/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/queryselect/__init__.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/queryselect/__init__.py:17:21: Q000 Remove bad quotes.
plone/app/z3cform/queryselect/__init__.py:17:80: E501 line too long (97 > 79 characters)
plone/app/z3cform/queryselect/__init__.py:18:21: Q000 Remove bad quotes.
plone/app/z3cform/wysiwyg/__init__.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/__init__.py:2:1: F401 'WysiwygFieldWidget' imported but unused
plone/app/z3cform/wysiwyg/widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/wysiwyg/widget.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/wysiwyg/widget.py:43:17: Q000 Remove bad quotes.
plone/app/z3cform/wysiwyg/widget.py:43:80: E501 line too long (84 > 79 characters)
plone/app/z3cform/wysiwyg/widget.py:44:17: Q000 Remove bad quotes.
plone/app/z3cform/tests/example.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/example.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/example.py:16:16: Q000 Remove bad quotes.
plone/app/z3cform/tests/layer.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/layer.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/layer.py:33:10: Q000 Remove bad quotes.
plone/app/z3cform/tests/test_csrf.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_objectsubform.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_objectsubform.py:34:35: S001 found %s formatter
plone/app/z3cform/tests/test_widget.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/test_widget.py:32:80: E501 line too long (126 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:0:1: I001 isort found changes, run it on the file
plone/app/z3cform/tests/test_widgets.py:549:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:552:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:769:36: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:771:9: D001 found directlyProvides replace it with zope.interface.provider
plone/app/z3cform/tests/test_widgets.py:919:80: E501 line too long (89 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:928:80: E501 line too long (90 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1008:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:1046:80: E501 line too long (81 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1069:80: E501 line too long (82 > 79 characters)
plone/app/z3cform/tests/test_widgets.py:1166:14: D001 found assertEquals replace it with assertEqual
plone/app/z3cform/tests/test_widgets.py:1174:39: Q000 Remove bad quotes.
plone/app/z3cform/tests/tests.py:0:1: C101 Coding magic comment not found
plone/app/z3cform/tests/tests.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.