@rpatterson Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.indexer/1//violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/__init__.py Imports are incorrectly sorted.
plone/indexer/__init__.py:0:1: C101 Coding magic comment not found
plone/indexer/__init__.py:0:1: I001 found unsorted imports
plone/indexer/__init__.py:1:1: F401 'indexer' imported but unused
plone/indexer/__init__.py:1:44: W292 no newline at end of file
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/decorator.py Imports are incorrectly sorted.
plone/indexer/decorator.py:0:1: C101 Coding magic comment not found
plone/indexer/decorator.py:0:1: I001 found unsorted imports
plone/indexer/decorator.py:41:80: E501 line too long (97 > 79 characters)
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/delegate.py Imports are incorrectly sorted.
plone/indexer/delegate.py:0:1: C101 Coding magic comment not found
plone/indexer/delegate.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/interfaces.py Imports are incorrectly sorted.
plone/indexer/interfaces.py:0:1: C101 Coding magic comment not found
plone/indexer/interfaces.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/tests.py Imports are incorrectly sorted.
plone/indexer/tests.py:0:1: C101 Coding magic comment not found
plone/indexer/tests.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.indexer/plone/indexer/wrapper.py Imports are incorrectly sorted.
plone/indexer/wrapper.py:0:1: C101 Coding magic comment not found
plone/indexer/wrapper.py:0:1: I001 found unsorted imports
plone/indexer/wrapper.py:69:80: E501 line too long (89 > 79 characters)
```
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.indexer/2//violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
ERROR: /home/jenkins/workspace/package-plone.indexer@2/plone/indexer/__init__.py Imports are incorrectly sorted.
plone/indexer/__init__.py:0:1: C101 Coding magic comment not found
plone/indexer/__init__.py:0:1: I001 found unsorted imports
plone/indexer/__init__.py:1:1: F401 'indexer' imported but unused
plone/indexer/__init__.py:1:44: W292 no newline at end of file
plone/indexer/decorator.py:0:1: C101 Coding magic comment not found
plone/indexer/decorator.py:41:80: E501 line too long (97 > 79 characters)
plone/indexer/delegate.py:0:1: C101 Coding magic comment not found
ERROR: /home/jenkins/workspace/package-plone.indexer@2/plone/indexer/interfaces.py Imports are incorrectly sorted.
plone/indexer/interfaces.py:0:1: C101 Coding magic comment not found
plone/indexer/interfaces.py:0:1: I001 found unsorted imports
plone/indexer/tests.py:0:1: C101 Coding magic comment not found
plone/indexer/wrapper.py:0:1: C101 Coding magic comment not found
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


С другой стороны сначала настраиваем видимость, а потом логика. ХЗ, на вкус и цвет.


Note resetting is not only about propagating your own rank but also increasing the DTSN to have other nodes also reset their timer.


Looks good to me, kills the issue we've had with redis being unresponsive due to stunnel container update.


Yes, it is not send to the server. But If you want to use 'put' the implementation treats it as 'PUT' in that case. Well, in practice this probably doesn't happen but it is incorrect. I created a but report sometime ago.


you got a link to where you picked this up from.  It's going to deserve a comment.


I am not sure if it is just my browser, but the former is rendered as a single line for all three commands.


can reduce the spaces


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


multiply this out for us.  Looks like one day.  I'm assuming this doesn't do what I think it does or that seems crazy long.  I'll dig into a bit later.


>  Git help and documentation uses this functional naming convention

And that's why we should adhere to it here as well IMO. I'd argue that most of our collaborators use only two remotes, "upstream" and their own. Our contributors (and I myself) regularly struggle with Git and seeing conventions in our guidelines which differ from "conventional wisdom" and official docs won't make matters better.


I'll take a look and would be interested to weigh up the pros and cons (let me know if there's a particular reason you prefer that package, if it's just aesthetic that's fair too).

The main con I see is the extra work we'd need to keep the frontend and backend in sync. That mainly applies to things like working remotely, though given the very theoretical state of that feature, I wouldn't call that a complete deal breaker.


As far as I understand it, it is the `select` widget itself that broadcast the accessibility machinery.

Hmm, now that I think about it, perhaps `display:none` is an issue when Windows is set to high contrast mode – that suppresses most CSS styling (colours & backgrounds, including images). Not sure how `opacity` interacts with that, though. I’d need to ask someone who actually knows how to handle Windows…

IE 11+ has some Media Queries targeted at high contrast mode, I have the references somewhere on my iMac.


Replace with c.fail()


Oh... something's weird. Will try to fix it.


It was removed because it's not used anywhere in the ding2 code, and it does't keep with Drupal function naming (it should be ting_material_details_process_series_info() in the least).

If it's a formatting that's wanted, there should be a formatter that provides it. Apparently there's not.


OK, I'd be interested in seeing info on high contrast.


Just so you know, all this mess is because App review requires a menu press at home to go back to tvOS home. Pretty fast rejection if missing, so it is something they test right away.


Should be 0 then


Remove


This code doesn't work properly. I get an error when I try to save an object with validation error on sonata_type_admin field type.
> Impossible to access an attribute ("type") on a null variable in SonataAdminBundle:CRUD:base_edit_form_macro.html.twig at line 21


To determine ascending or not


Ah, então o exercício que eu fiz atenderia a isso perfeitamente.
Primeiro ele mandou fazer um dado (de seis lados), com uma propriedade (o valor que ele está mostrando) e um método (jogar do dado, que evidentemente muda o valor). Essa classe era usada para uma experiência básica de probabilidades: jogamos 2 dados, tipo, 1000 vezes, e contamos o número de ocorrências de cada valor combinado dos dados (que é de 2 a 12).
Depois o exercício fica mais interessante: ele manda a gente modificar a classe dado, para que ela possibilite criar dados de lados diferentes e manda o programa deixar a gente escolher quantos dados jogar (mais ou menos do que um par de dados).
E aí, aceita o desafio? rs. Esse exercício eu fiz em python, mas tô afim de praticar isso com php, ou outra coisa...
Enfim, alea jacta est... kkkk


Fixed, thanks!


Yes, exactly. But no need to propagate that anything updated if no one is on the need-to-know basis?! Assuming that there is no registered routes/children (in storing) and this node switch parent but not change rank - what is the change that others need to know?


this.cheProject - not used


For this repository (32 bit) I don't use prima drivers. I just patch [module.c](https://github.com/Mr-AW/Kernel_TeLo_LP_LenovoA6000/commit/ac78e28a44230d02b61072af55d3442799920d4d) file.
For 64 Bit I use prima drivers because I fork from flour_mo repo and it use prima drivers. You don't have to replace original wlan.ko from stock/custom ROM. But if you want to use newer prima drivers, just replace with your compiled prima drivers.


```
$modx_lang_attribute = 'uk';
setlocale (LC_ALL, 'uk_UA.UTF-8');
```
**Should this not be:**
```
$modx_lang_attribute = 'en';
setlocale (LC_ALL, 'en.UTF-8');
```
uk is Ukrainian


For some stupid reason, you can't have indent quotes in lists: https://gist.github.com/clintel/1155906

So you're right after all - apologies.


This file seems to be unused?? (same for "FragmentShader.glsl")


Is it necessary to get the blog object again since it's already retrieved [here](https://github.com/mzorz/WordPress-Android/blob/214708fad10f8396fb0cab82cc35e5aa9750a759/WordPress/src/main/java/org/wordpress/android/ui/comments/CommentActions.java#L442)?


I agree
Il 15/feb/2016 12:18, "Tobia Di Pisa" <notifications@github.com> ha scritto:

> @kappu72 <https://github.com/kappu72> @mbarto <https://github.com/mbarto>
> In my opinion it would be better to add an unique 'require' instead of
> adding all components here (i.e. an external index file with needed
> requires properly configured).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kappu72/csi-sira/commit/62a085ef522f68d4d999514f53a3caaff1b0a8e7#commitcomment-16092849>
> .
>


Similar comment about blog object already being retrieved.


why not optional?


seems debug info


Oh, I didn't saw the file, as it's in another commit.


Oh, I see. You're right. I'm afraid this upper-case thing is used in every python program, I remember having seen it in requests. I think the whole internet is broken on that point. I think (and hope) the IETF will never standardize new verbs being the existing one but with different casing. Unfortunately I think nobody in Python will want to fix it, and a lot of things will break if it's fixed :)


Change code


Hey @alexcrichton! Are you sure that this fixes everything? Because here is another one with uppercase letters still in the code: https://github.com/alexcrichton/gcc-rs/blob/dc3dd4aa169c8615834d84c3f699791c17ab9ca2/src/lib.rs#L411

Also: Our builds are now failing due to SAFESEH and I'm not sure why... :disappointed:
See here: https://ci.appveyor.com/project/zonyitoo/context-rs/build/1.0.78/job/0wgide4egb95i9bw


It's a proactive mechanism. There is something going on, let others know, have all re-disseminate their rank and refresh routes. I think that is the idea at least ;)


ну в целом неплохо


The function CfgRequestSymbolGroup, appears to always raise an error and never return. I have tried on different servers with different managers with different credentials. Any idea why that may be?


Java/Ch 04. Trees and Graphs/Q4_07_Build_Order/DFS/Question.java

... could be an `ArrayDeque` also


Удалить?


sscanf() with %llu was probably never the right call to use here, since usec_t is not necessarily unsigned long long. It is however equivalent to uint64_t, and hence we should use the safe_atou64() call to parse this here. (and if we do, we should also use log_debug_errno() and pass safe_atou64()'s return value).

[Note that we also have parse_usec(), which parses usec values, but that's explicitly not the right thing here, as that's for human-readable strings (e.g. `5min 2s`), and we don't care about those here. Hence, safe_atou64() is the way to go.]


Можно одной строкой


Done: a87aaee88340159509f8390198d4209111c6ae02


See PR #71.


why is it distributed? I don't see any submissions via servers, so I think it should also work for embedded mode


Yes, I agree - but we have had a too proactive reset which was not because we liked it but rather that the early drafts had a lot of trickle resets while there are much fewer specified resets now. And I think a few of the resets we still have can be removed without making RPL too static.


Merci :+1:


Oh no, bad autocomplete


Needs consistent values for the naming. Also maybe explain why 60 is sensible and 3600 isn't.


Я бы чутка покомментировал логику тут - чтобы было понятно, зачем оффсеты храним и что за deeper-upwards.


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


@joakimeriksson  In fact, a couple weeks ago, I was looking for definitions, when the DIO timer should be reset. And, I don't find any explicit and detailed description about that... Yeah, you may be right on the meaning of the sentence.

I'm thinking about phrases for rank which you used: "significantly changed" and "similar." The idea is, if  DAGRank(rank) changes, this is "significantly changed" and otherwise "similar"?


переименовала в val


Do we care if the line's 2+ logs have different values for `args` or is this more a bug that the check needs to fix? There doesn't seem to be any examples in our tests, but I created one in another PR.
Example:
````
              "137: " + getCheckMessage(MSG_ERROR_MULTI, "object def rcurly", 8, "10, 14"),
+             "137: " + getCheckMessage(MSG_ERROR, "}", 8, 10),
````
From https://github.com/checkstyle/checkstyle/commit/26022975bc27b9ee7432ad2912a287129511e311#diff-88d68aaf0963249939255684219c43b1R916


it was not introduced in your patch, but `res->href` will leak on further errors. You must release its memory in case of error.


That should happen once we add the mask and tune the effect the sky colour has


I agree @joakimeriksson, I was more trying to make sense of the standard than saying we should exactly implement this particular aspect. I agree we do trigger a reset too easily.


let's just write (unless I'm missing something):

```python
if isinstance(base_set, EmptySet):
    return base_set
if isinstance(base_set, FiniteSet):
    ...sifted, etc...
return Basic.__new__(cls, sym, condition, base_set)
```


The max light calculation is wrong at the moment


fixing the comment, +1


Fixes #2


исправила - все пишу теперь с маленькой буквы


You are quite right. Series are handled in the function process_series_description in the ting.entities.inc file.This is superfluous code.


thx.
Downloaded tails does not exist, means that the analysis of the data is not complete.
Therefore would like to have the conditions on the machine download TRAILS  (image docker), and then copy the image(docker) to the machine running on the no internet and No need to update.


ok


спасибо, отступ исправила


Thank you! 8-)


Ok:)


>Updated to 3.0.3103-3.

Вай-вай-вай!
Ты же не обновил до версии 3.0.3103-3, а просто  подкорректировал Spec.


we need to check, but it's not needed since, when dataType change, it also change the data


Ну, ок.

> On Feb 15, 2016, at 3:59 PM, Alexander Marchuk <notifications@github.com> wrote:
>
> С другой стороны сначала настраиваем видимость, а потом логика. ХЗ, на вкус и цвет.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/trashkalmar/omim/commit/ffd07e63adeec39e1ead5dee8496e21e8b9b5b86#commitcomment-16094803>.
>


Я бы добавил @nullable


не, не думаю. что еще может быть? Функция? HTML-элемент?
хотя... toString?


Метод бы ниже локального класса убрать


looks like debug code(?)


It does not.


----------------------------------------------------
2016-02-15
FREQUENCY CHECK

- updated interpreter for h1 and h4 (correct view of used symbols)
-first time -change the calculation procedure (decreases the pointer from last letter (regardless what is before the date and time)

FILTERED_PEAKFINDER -

- updated interpreter for h1 and h4 (correct view of used symbols)
-first time -change the calculation procedure (decreases the pointer from last letter (regardless what is before the date and time)


I'm not sure scandium will really like this, I think a `return null` is enough reading the scandium code :
```
String identity = pskStore.getIdentity(getPeerAddress());
if (identity == null) {
    AlertMessage alert = new AlertMessage(AlertLevel.FATAL, AlertDescription.HANDSHAKE_FAILURE, session.getPeer());
    throw new HandshakeException("No Identity found for peer: "	+ getPeerAddress(), alert);
}
```


Thanks. Done #2218


Yeah, I know...

This still exits though.. Checks to make sure you're on home with no dialog open


Да, -3. - это не часть версии программы, а лишь релиз версии пакета. Это не надо указывать в ченджлоге. Там надо было написать нечто вроде
`- Corrected files list owned by package`


Is this callback necessary? It seems like this could be done with simply:
```
CommentList comments = ApiHelper.refreshComments(blog, params);
CommentTable.deleteCommentsForBlogWithFilter(blog.getLocalTableBlogId(), mStatusFilter);
CommentTable.saveComments(blog.getLocalTableBlogId(), comments);
return comments;
```


да, спасибо, использую Object.prototype.toString
+ вынесу его в переменную


It is specific on what is inconsistencies (at MUST level):
-  When a node detects an inconsistency when forwarding a packet (e.g. loop)
-  When a node receives a multicast DIS message (two variants)
-  When a node joins a new DODAG Version (global repair or join new)

But it is also clear that there are other cases that needs (or should) reset DIO timer. I would say switch parent when children exists (that have sent DAO) and significant rank change (yet undefined ;-) - what more?


tabs instead of spaces


here as well


It does not, that just reads from the leader


oops sorry - good catch


`Zikula_Session` is deprecated


alright!


I'm not sure of it but it should release the memory of both memory banks so there would be no need of the two s5p_release_media_memory_bank


Fixed. Thanks.


Do not forget to add `@Configuiration` to enable swagger


Dude clean out all that useless imports...


@pynicolas I tried to improve this, see 54b2c7bd8e692d4c7d59e7a9fca10ed3ee258b0b


Тут и в других похожих местах можно сделать вот такое упрощение: `group_by(&:director)` — это абсолютно аналогичный код, `&:director` разворачивается в блок, который принимает один аргумент и вызывает у него метод `:director`.


А вот тут лишний символ.


картинку нужно класть в селл. в папку assets/images


@gaganawhad I _think_ that we want it in the gem.

1. The basic principle should be: keep as much wakes-related code in the gem as possible, and only put it in the app if truly necessary.

2. A second principle is: do we want to distribute this code if and when we publish the gem?

3. We would put it in the app if the job needs to be aware of things that only the app could know. Is the code specific to the DG app, or is it likely useful for all apps running wakes? If we had a second app that ran wakes, would we copy/paste this code snippet into that app as well?


@jsight, @mbriskar I didn't pay attention to this package name in PR review before, but isn't this a mistake of IDE to nest another org.jboss.windup... in org.jboss.windup.rules.files. ?


@rnveach
Good catch, i thought about it before sending fix and i hope proposed solution will be the beggining of the discussion about such issues


@jsight, @mbriskar again, isn't this a mistake of IDE to nest another org.jboss.windup... in org.jboss.windup.rules.files. ?


Yes, we will have to rewrite the ConfigurationPerformer behaviour, but may be in another issue.


lgtm


это элемент. в нем должно быть только расположение описываться


Ser en liten detalj her: Siden vi kjører `truncatechars:200` først, og `striptags` etterpå, er det fare for at man truncater midt i en HTML-tag som da ikke blir strippet. Bør nok bytte rekkefølge på det.


компонент


Хорошо бы кстати названия ключей вынести в константу, а здесь собирать хеш (и OpenStruct из него) с помощью zip


Ya. There is pootle server: http://i18n.brezblock.org.ua/projects/q4wine/
Any way, I am fine with any method you prefer (email/github/pootle), so I am not forcing ppl to use it.


I feel like this functionality should be included in object!:

    ENV['DEFAULT_HOST'].swap_value('default.host') do
      expect(location.url).to include('default.host')
    end

it executes the block with the new value swapped in, and then swaps back the old value before returning

anybody want to write a gem? @joeosburn-desiringgod @pjmelling @gaganawhad


can't you set `analysis.setQuery(explainAnalyze)` in `StatementAnalyzer` to avoid if here?


`1 line` - make a newcomer issue please


Oh wait - does this pass ? That would imply the bear also has a problem - mention in the newcomer issue :)


tree убрала


Hello @duluca,

This last 3 lines broke our angular app on browser. Because module support doesn't exist on browser. So module variable isn't defined.

And today, when I updated the package that are in bower.json, my app has crashed.

Therefore, I updated this condition on my fork. And I created a pull request for this update.


Hey there James! Sorry that it took me so long again, but those generators where something I've had no experience with.
My final replacement for the gwt-ent reflection library previously used, which lifts the requirement of having to use any extra dependency was to write a JDT AST generator:
https://github.com/fhcampuswien/atom/blob/master/atom-generator/src/main/java/at/ac/fhcampuswien/atom/ReflectionEmulatorGenerator.java
that adds the implementation for the 3 methods getAttributeValue, setAttributeValue und makeInstance of this ReflectionEmulator stub here:
https://github.com/fhcampuswien/atom/blob/master/atom-domain/src/main/java-stubs/DomainReflectionEmulator.java
I've adapted my project structure and maven build plugin usage to have this generated at every full build.


why? this button type refers only to `banner`.


@joakimeriksson I agree. And that is what I wanted to say! :-)
>  it is also clear that there are other cases that needs (or should) reset DIO timer.


да, действительно


Not sure but it looks like we were the only ones to use it.  The code was removed upstream as "dead" and was just a straight delete with not other changes.


Remove commented out code


Being on the topic, er også bedre å bruke `truncatewords` :)


Good cache! 🙃


i'm guessing, you didn't mean to commit those last two lines? :)


no variable when only a single usage


cool


I don't know what the original intend was of this code, but I was able to build the complete geoserver perfectly after the change.

Currently, the tests were using ISO character set, even in Linux. This made my tests break that write files in the default character set and then read them with a request. My change seems more logical.


test


@barinali very good call. I wasn't thinking about the browser usage, when I made the code changes in my fork.


Good catch, den første. Ja vurderte words, men tenkte at man har bedre kontroll på hvor langt det blir med chars. Ikke det at det har så fryktelig mye å si kanskje.


@AlfioEmanueleFresta in riferimento all'issue #155


@gaganawhad why add a `Wakes::SocialMetricsWrapper` class? Why not just go with your original code, using `ShareCounter` directly in the service? In a sense, `ShareCounter` is really just a wrapper class.

(I would also be very interested in keeping the `Wakes::SocialMetricsWrapper` class but _getting rid of_ `ShareCounter` and just ripping out the few lines of code that we need for FB and putting them directly in here. Using the `ShareCounter` gem made more sense when we also got twitter, but since we don't get it anymore, maybe the extra dependency isn't worth it anymore?


shouldn't this be combined with the resource_type_id of the item, you might get more than you bargained for otherwise
`AND item.item_id = metadatavalue.resource_id AND resource_type_id=" + Constants.ITEM + " AND metadata_field_id = ?;`


Du trenger ikke alltid bruke `with` for relasjoner. Her er det bedre å sørge for at `turledere` blir prefetched med `JOIN` i viewet, så kan du gjøre `.turledere.all` så mye du vil i templaten uten at det går ut over performance.


https://www.dropbox.com/sh/m5f9ke9c8ake7ll/AABuB-zmP9H_DTUwtpsJNd9fa?dl=0 bu haftanın videoları


Alright @andela-bmakinwa, I should do that in the current feature. What do you think?


yes, but owner(owner from ownership screen) is user and this is stored `evm_owner_id` column on VMs etc.
When we are building sql query for listing VMs etc. with ownership(set up on user's role called VM & Template Access Restriction), we are using for it always user's group_id(non tenant) and never tenant's group id. https://github.com/ManageIQ/manageiq/blob/master/app/models/mixins/ownership_mixin.rb#L43 We are not able to assign tenant's group to user as nontenant group. So due these facts I think that we should not store tenant's group id in VMs(etc.) `group_id` column and don't list tenants with non-groups here.


most of it is boring with maybe the exception of https://github.com/pweil-/origin/commit/184327199352d1bce085191e7a7803b1ffffee85#diff-62e1d6e1ea7f763bd41c44733aa2fba2R67


Discussion of the problem in issue https://github.com/gitpython-developers/GitPython/issues/382


Shouldn't we do **throw $e** instead ?


element is responsible for position. block( in our case is another component ) is responsible for appearance. And best of all to create `button` component, which will render us any button we want.


@AngeloLupo fai PR in master.


да, написала


No test, and just the assertion that you can't find a reason why @ndeloof added that a workspace is required for polling if the author in changelog extension is being used?

Why not just return to relying on the "false" which is returned by the requiresWorkspaceForPolling method which this is overriding?


OK


drop_item takes no parameter, it drops the active hand


isOn


True, this was mistake :)


This method has a lot in it. It should be broken down into methods.


`observes` 9 раз в коде, нужно подумать как сделать без него


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


need class I think


need class I think also


Going through all these now - structurally, I'm still going to keep them in readme here. Appreciate the suggestion though.


зачем это


Mja, man har jo fremdeles 3 referanser til objektet, og på denne måten slipper man å skrive `aktivitet_date.turledere.all` for alle disse?


I don't see how this changes the behavior of the code.  The only change was the inlining of the `->` value into the `get-in` call.


Yes, wise advice.  The primary difficulty with text is localization, as we have a lot of options for fudging it in English to ensure that it fits in a certain amount of space, but those options aren't always available in other locales.  These days, PD is relatively graceful about dealing with long text translations, but in a toolbar it could be tricky.

No way to know for sure without trying, though.


when scripting is part of alpha plot this part is useless ... so comment out this for now


Это необходимо, чтобы запомнить предыдущее текущее состояние полей блока рандомного диапазона. При изменении модели проверяется, больше ли первое значение, чем второе, и если да, то нужно вывести сообщение об ошибке и вернуть прежнее состояние измененного контрола


Almost 100% certain that this doesn't do anything in the case importer. But it probably should?


this button very specific for this block.


configurable in pillar = good. default to 60, dont know.
an hour seems too long, is 60 seconds good? probably, why would it not be....


Just testing a static analyzer to discover possible issues, duplicate code, etc.


Fixed in ecb7429733c3a. Thanks.


> It does not, that just reads from the leader

bummer.

@pweil- boring.


Are we using some different definition for "concurrency" (doing more than one thing at a time), because otherwise I cannot see your point? Because if node-bluebird is not using concurrency (I.E. it is only doing one thing at a time), making it concurrent would make the result even better!


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


keybase.io could be optional as well? at least keychain on ipfs is planned, could be good starter to use distributed identity as well. May use something from Bitcoin


May I ask what the intention behind this change is? Is this supposed to create overlapping, not disjoint ranges from the tripplets found in the configuration?


формат хороший, спасибо


this changes the meaning, illegal transformation


**Nitpick - Code Quality**
This method has a lot in it. It should be broken down into smaller chunks. We can have methods that will help in making this a lot cleaner.


Why only guard one of the two calls to rev-parse in CliGitAPIImpl?

If there are cases where rev-parse returns a leading empty line for the revParse(String) method, isn't it likely that there are cases where rev-parse will return a leading empty line for the validateRevision(String) method?


There should be a better way to know if you are at home and there is no model dialog up. Have to dig into the code.


This interface seems to be unnecessary


`var`?


it's found in the text once. I don't think so


apply (instead of applying)


Now that I think about it, wouldn't just checking for
    if (g_windowManager.GetFocusedWindow() == WINDOW_HOME)

Yield the desired outcome since that checks for dialogs first?


planning to create a scripting language as a part of alpha plot.. simple interpreter with basic scripting functionalities to replace muparser & python..


is this a reason to create element instead of button? `banner-button` as normal solution. `bunner__button` and then `button` inside as the best, I think


@pelwell OK, Phil. Just building again from the 4.4.y branch with the ATOMIC patch. I'll see if I can still re-produce.


Of course it uses concurrency... Just look at the code.


You can use `ctx.debug` as a shorthand


this commit broke build for debian package
make deb
return

<pre>
copying LICENSE -> /opt/diamond/build/diamond-4.0.380/debian/diamond/usr/share/diamond
error: can't copy 'README.md': doesn't exist or not a regular file
</pre>


I think this should be split into two separate comitts: one changing godeps should land inside k8s bump, vel as a separate repo commit, the second changing actual stuff inside origin.


Jo, sant nok det. Ser ut til at FB viser de 300 første tegnene. Fint med kontroll, kjør på med `chars`.


@zhuowei @jollaman999 @HashBang173 http://pastebin.com/TCgP1Raj here is my dmesg please see to it whts going wrong :(

and yes when i try to switch rom it shows error that data may corrupt so mrom is restarting :(


hmm instead of saying more presentable, let's focus on apply the same style to multiple graphs that the user creates, so we could say something like:
In this tutorial, we'll show you some tips to easily apply a consistent, customized style to your charts with Plotly's built-in themes.


This catch block should be indented one more level to match the corresponding try


Shouldn't this be defaulted to 5 and only force the use of -DDESIRED_QT_VERSION=4 if required?


ack


Attention, il y a d'autres éléments dans `www` qu'il vaut mieux éviter d'exploser. Les configurations de nginx, puma et leurs logs pour ne citer qu'eux.

https://github.com/HE-Arc/webapp-server/blob/master/files/templates/README-ror.md#creation-of-a-ruby-on-rails-web-application


Can we specify the function name? I can't understand what does it mean `textSelectionUtils.correctRectangle`. It was incorrect?)


I'd like to involve as little third parties as possible for now. I would love to add ipfs keychain support when it's ready for sure.


done


Geocoding a data needs this data to be a string.
That string is then converted to coordinates.

Some fields in Drupal are not composed of a single string. This plugins will preprocess those fields into a simple string that can be geocoded.


Changed in ecb7429733c3a. Marked it as 'Not recommended', same as some other documents. This kind of docs normally mentioned it as 'This approach is covered here due to it being mentioned in other guides'. Maybe we can also keep it this way here for user just in case.


My bad. I thought following part was belonging to the added code as well:

``` clojure
     (map :timestamp)
     tu/ordered-desc?
     is))
```

But this is not the case.


great so far! we should also add an example of how they can create their own theme by styling their chart then selecting save theme


Sorry, that was too long ago. I don't remember where I met this issue


maybe we should move out this two functions from this file?


I see what you mean, but it seems elsewhere we are relying on `ApiHelper` methods to _also_ save data to the database as they are fetched from the server. While it sounded weird to me at first (having ApiHelper fetch _and_ save, but that’s another discussion), we could just take that code out as you’re suggesting and leave the responsibility of saving/updating the database to the callers everywhere the method is called and a result is successfully retrieved, but it should be made clear with code comments (and we can only rely on the programmer reading those and remember they need to use it). OTOH having an interface as a needed parameter gives it more visibility - at least it is clear a listener is expected. Maybe I can enforce this by not allowing this parameter to be null: either by not checking for null and letting NPE happen, or throwing another custom exception by checking for null explicitly.

Furthermore, even in both options (having a listener or not having a listener) we need different implementations in at least 2 cases:

For Comments, when the filter is switched we need this (only delete the comments with the specified status):

                        int localBlogId = blog.getLocalTableBlogId();
                        CommentTable.deleteCommentsForBlogWithFilter(localBlogId, mStatusFilter);
                        CommentTable.saveComments(localBlogId, (CommentList)list);

But for the `ApiHelper.RefreshBlogContentTask` we need a slightly different one (delete all comments for a given blog):

                        int localBlogId = mBlog.getLocalTableBlogId();
                        CommentTable.deleteCommentsForBlog(localBlogId);
                        CommentTable.saveComments(localBlogId, (CommentList)list);

It seemed a candidate for using an interface so each one can have their own implementation. A third (non good) option would be to keep the responsibility as it *was* before my changes (that is, `ApiHelper.refreshComments` should also save to the database) but in that case the only way to retrieve the filter is to inspect the `Object[] params` parameter, which seems too general for such a specific purpose. So best way is to leave implementation on the side of who needs it (and knows about the implementation details, in this case, what a `filter` is or means) - just like you suggested. And if we’re going to leave implementation to the caller, but want to still make it visible to the programmer that data needs be saved so we can keep up with this  “caching” mechanism, then an interface sounded like the way to go.

Sounds good? LMK what you think - if you think this is good I also think we should continue down this path with the rest of the methods in that class to separate responsibility, on another PR of course.

All that said however - I'm happy with going with your suggestion and leaving any network/database/other refactoring efforts and energy to the flux-inspired network/db layer whenever that happens :+1: Just LMK


Ok, ja, da er det jo greit å øke til 300 iaf. :simple_smile:


Definitely, if we keep this.  I was hoping that someone knew how to get it in the generated pieces without another patch.


Just curious. Why remove this import?


Nice choice! **Deftones**


Oh, sneaky.  Let's go ahead and refactor our interface to return the error and try to keep our fatals contained to the `Run` method for now.


Is this really necessary (considering it just calls ```equals( o )```)?


+![att4 1.001 2 1515 840](results/att4_1.001_2_1515_840.jpeg)


should we add `startLine` and `endLine` fields?


I didn't notice it either, but yes, it definitely looks like a mistake.


afais, this should be
```sh
CXX=$(exhost --target)c++
```
because this package does not rely on gccisms


路过，弱势围观


Sant :) javisst, fint å gjøre det sånn da. Jeg tenkte det var gjort for performance, og det vil det også ha en effekt for. Likevel er det fint om du legger til `'turledere'` [her](https://github.com/Turistforeningen/sherpa/blob/aktiviteter-misc/apps/aktiviteter/util.py#L27) slik at prefetching optimiseringer tar effekt uavhengig av `with`.


двойные?


Thanks for updating your patch. I should've mentioned it in the first patch, but test-requirement.txt and tox.ini would be placed under masakari-* directory for every process. requirements.txt and test-requirements.txt has a different package list for controller and/or monitoring process.


The core uses this as well:
https://github.com/zikula/core/blob/1.4/src/system/BlocksModule/BlocksModuleInstaller.php#L103


@niallcreech queue_timeout refers to the internal queue (managed by beaver) as you can see [here](https://github.com/python-beaver/python-beaver/blob/645381f794999064c66145e968bd771bc49d9031/beaver/run_queue.py). The actual socket timeout is 10 seconds (hardcoded in beaver) [here](https://github.com/python-beaver/python-beaver/blob/645381f794999064c66145e968bd771bc49d9031/beaver/transports/redis_transport.py).

So, IMO the issues is more complicated since the redis server is down and there should have been at least some kind of handling between run_queue and the redis transport code.


Got a chance to test this out. Works fine windowed, but at fullscreen `Clone` throws an `OutOfMemoryException` [happens when the bounds are outside of the image](http://stackoverflow.com/a/4318723). Not sure what the calculation is doing exactly, but I get a rect `(0, 12, 1920, 1080)`, must be to do with the caption? Same exception in `FriendsListOpen` when fullscreen.


Hi. This modified line seems to add an extra '{' '}' to the bibtex fields. Is it necessary?


@curiosityflow Well done here. Press ahead!


This says `defaultNullable` is not supported yet.  Is that still correct?


Det var selvfølgelig UKMNorge/UKMvideo#4 det var snakk om...


Could we print the client version above `assertServerUp()`? That way, the client version will print even if the server is down.


Ja, det er vel a little bit of both som er årsaken. :-)


Also, `bmp` is null here if the `OutOfMemoryException` exception is thrown and caught above


linked in a comment (pulled from the command line flags where 14 was the default)


added in new version number


yes, it's one day.  changed to be multiplied out.


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


@curiosityflow Well done again. Good good.


Yes of course, or as I showed to make it work even if it's changed.


можно, конечно.
однако, фреймворк еще не совсем готов по части главного модуля (


Closes #11


I think where I get confused is I want it to work like this....


мне кажется что-то тут не так


Hi Brandon. I have tested your code and with the current kernel release, UART6 works out of the box.
Moreover, your patch is related to iMX6Q. That branch is never executed on the iMX6SX.


запишу идею в список улучшений


I'm trying to fix problem when updating the package fails to copy files, and this is the only solution I've found


Closes #19


Awesome, glad everything is working! Congratulations!

What's the issue with the map? Check the console (in Chrome it's View > Developer > JavaScript Console) for errors and check the [Leaflet docs](http://leafletjs.com/) for example code. Let me know if I can help.


Instead of "Documentation" could you name it "Install & Setup"?
(and maybe move "contributing" at the end)


Je suis absolument d'accord ! Cependant je n'arrive pas à comprendre comment ne **pas exploser** tout ça... J'ai compris la partie où on ajoute des infos dans *development:*, je vais faire ça dans la prochaine itération, cependant, je ne vois pas comment garantir son utilisation sur le serveur...


If the above was true then the queue_timeout should be maximum of (stunnel_image_max_pull_time + serice start)


I forgot I even made a commit :P


still no reason to use sudo, just use the line without sudo to remove the conflicting files.
Example:
https://github.com/KaOSx/apps/commit/b20f7852d69b547f87694b40df52fb2c57cb4915#diff-a463db3bfdfc4ec8bfa3b3fc331d1a36R40


Closes #18


Never mind, you already did that in a later commit! :100:


but if thats not what queue_timeout is for then what is it actually for?


Actually maybe this could be improved too.  Rather than the handsize crap each street should look up the hrange in the Card.games table.


согласно документации функция, передваемая в observes не принимает аргументов


this push is the GREATEST PUSH EVER


@stropitek beware ! `_.pluck` has been removed (replaced by `_.map`)


Fixes #14


@neobuddy89 @jesec
How can I calculate the addresses and holes for this patch here? If I would know how big this sizes here are I could do it myself, but like this I got no clue


TeamCity Civic Assembly :: Build all [Build 49](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1117&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


hahaha thank you!


@AbdealiJK  I know it's a grammatical mistake but the output contains "1 lines". :stuck_out_tongue:


Why modify the link?


that is not something we can change.
/opt/vc is where these libs are installed to on raspbian


xD you misunderstood :P
```php
if(count($args) === 0 && count($args) <= 2)
```


OK thanks!! I'll be testing


Install doc on https://bouquet.zendesk.com/hc/en-us is not the latest version - Latest is https://github.com/openbouquet/bouquet-parent/blob/master/setup.md


@shredtechular I see now there are two links to this blog post, one here on the state page, the other I commented on is at the bottom of the visualizations page.


Closes #21 #22


Fixes issue #4


Do we need to include the `NameType` in this table? It is a part of the `Name` attribute value struct.


@wangzeyi i have no idea what this line means!!!


I am not sure that this is the correct approach for determining if a node bundle has custom settings for the FIA custom view mode, but it is the only approach that I could find quickly


All these methods are called ONLY and ONLY if switch is toggled in settings, so this don't mean shit if you don't toggle the switch.. at least that is my thinking here

recreateStatusBar();
updateRowStates();
 updateSpeedbump();
 updateClearAll();
 updateEmptyShadeView();


![ios simulator screen shot 15-feb-2016 7 34 58 pm](https://cloud.githubusercontent.com/assets/1255516/13050966/a9c5bd64-d41c-11e5-84ac-1e21f571192b.png)
![ios simulator screen shot 15-feb-2016 7 39 30 pm](https://cloud.githubusercontent.com/assets/1255516/13050967/a9f1ded0-d41c-11e5-9e45-4509a40f4056.png)
![ios simulator screen shot 15-feb-2016 7 40 13 pm](https://cloud.githubusercontent.com/assets/1255516/13050968/a9f4ae3a-d41c-11e5-8c0e-e839c3f19e1b.png)

UI-Screenshots of App


"If you want, I can keep it a builder, build a "triplet" (sender, recipients, mime) and send it in the mailet."
That's what I meant :-)


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


Thats a nasty thing that happens during pushing names into storageContent, if it is fixed there we can remove it here ...


`notifyExtra.@each` это не работает, либо `notifyExtra.@each.someProperty` либо `notifyExtra.[]`


@amakropoulos I suggest you replace the above three lines by "Copyright 2016 Antonis Makropoulos".


TeamCity Civic Assembly :: Build all [Build 49](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1117&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 165 (1 new), passed: 7, ignored: 4 Build time: 00:06:39
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455545779826: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

##### there are 155 more failed tests, see build details
```


```
var contentValue = document.getElementById("content").value;
$scope.content = contentValue ? contentValue : '♪';
```
이거 전부 지우면 안되고, 음표 대신에 ''을 넣어주세요.
해당 코드는 필드 유효성 검사할때 사용되는 부분입니다.


In controllers you don't need leave variables with '@' in the end of method. They are available in the view by default


Посмею не согласиться:
http://emberjs.com/api/classes/Ember.Observable.html
```
Ember.Object.extend({
  valueObserver: Ember.observer('value', function(sender, key, value, rev) {
    // Executes whenever the "value" property changes
    // See the addObserver method for more information about the callback arguments
  })
});
```


Is that true? That was the case for GenericImage and BaseImage, but I don't think you have that problem. Can you try to just normally include the base class header after the include guard as usual and remove the comment?


@commonsguy Have you been able to find somedody else's work that is good for caching ?


Fixed.


So something I hadn't thought about: do these changes get triggered whenever I create an `OpaqueObject`, say in a testing script or when I'm hacking around in the interpreter? I'm not that knowledgeable about `SQLAlchemy` so maybe you can point me to the right docs here.


`if (queue_empty)` is equivalent to `if (queue_empty.load())`


sorry, will do that now


Good point. Made the change and squashed it into the same commit.


Please use mirtkUnorderedMap instead of std::unordered_map. It's just a type alias, though. But in MIRTK, we use those type aliases in the "mirtk" namespace instead of the "std" namespace types.


should be
```c
queue_empty = false;
```
?


Пофиксил


It is a bit curious that there are two functions to get the `root` element of window:

[sciter::host::root](https://github.com/c-smile/sciter-sdk/blob/master/include/sciter-x-host-callback.h#L175)
[sciter::host::get_root](https://github.com/c-smile/sciter-sdk/blob/master/include/sciter-x-host-callback.h#L222)


почему не через вызов селла?


really ?


Holasdasdasdasds


You don't need this. AND you should NEVER have such statement in a header file!


What is actually changed here? And also on the other two places...


I would actually think it can also be used for 3D+t / 4D images.


It's look like you don't use variable @events in view. You can use here simple variable without '@'


D'oh, there is a reason this was a grammar fix and not a spelling fix lol.


Awesome, sounds like a great plan!


why == if it works with one?


In MIRTK, we use camel case for member names. Hence, it should be rather ```DataIterator```.


this might crash? if `spdk_pci_device_get_device_name()` returns null?


interface refactor in last commit


You're absolutely right. :D I didn't find a fast idea to `define_method` on class. I'm working on it


Don't leave commented code )


Much better now, I can finally completely use curlconverter alone to make/edit my requests, instead of having to call system(). Kudos !


reack


the default_config_hash needs to be removed.  It came from a views export.


is this right? It works as expected but i'm not sure @giogonzo


parent element or modifiers should control demensions


I have hard time understand what the code does. Will usage of hashCode work with custom messages?


the fields in data have to support aggregation. they are consolidated/accumulated along the logging process. Adding a '_payer_user' as part of the operation name (category) is a way to support new operations by requester with minimal impact while keeping a similar performance


:tada:


Yes this remove a warning:
```java
Type safety: A generic array of Class<? extends Throwable> is created for a varargs parameter
```


@tomgross Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.transformchain/1//violations
```
plone/__init__.py:0:1: C101 Coding magic comment not found
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/interfaces.py Imports are incorrectly sorted.
plone/transformchain/interfaces.py:0:1: C101 Coding magic comment not found
plone/transformchain/interfaces.py:0:1: I001 found unsorted imports
plone/transformchain/interfaces.py:11:1: E302 expected 2 blank lines, found 1
plone/transformchain/interfaces.py:53:1: E302 expected 2 blank lines, found 1
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/tests.py Imports are incorrectly sorted.
plone/transformchain/tests.py:0:1: C101 Coding magic comment not found
plone/transformchain/tests.py:0:1: I001 found unsorted imports
plone/transformchain/tests.py:7:1: F401 'interfaces' imported but unused
plone/transformchain/tests.py:26:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:29:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:32:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:40:5: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:43:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:53:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:56:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:59:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:79:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:91:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:95:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:95:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:110:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:124:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:128:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:132:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:134:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:149:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:161:18: Q000 Remove bad quotes.
plone/transformchain/tests.py:165:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:165:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:195:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:215:34: Q000 Remove bad quotes.
plone/transformchain/tests.py:221:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:225:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:225:28: Q000 Remove bad quotes.
plone/transformchain/tests.py:225:36: Q000 Remove bad quotes.
plone/transformchain/tests.py:227:1: C901 'TestTransformChain.test_transform_mixed' is too complex (13)
plone/transformchain/tests.py:292:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:296:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:323:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:327:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:327:28: Q000 Remove bad quotes.
plone/transformchain/tests.py:329:1: C901 'TestTransformChain.test_abort_chain' is too complex (13)
plone/transformchain/tests.py:342:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:345:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:348:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:380:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:386:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:394:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:398:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:398:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:400:1: C901 'TestTransformChain.test_ordering' is too complex (13)
plone/transformchain/tests.py:413:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:419:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:432:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:438:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:451:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:457:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:465:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:469:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:469:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:471:1: C901 'TestTransformChain.test_request_marker' is too complex (13)
plone/transformchain/tests.py:484:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:490:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:503:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:509:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:522:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:528:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:536:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:540:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:540:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:545:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:549:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:549:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:551:1: C901 'TestTransformChain.test_published_marker' is too complex (13)
plone/transformchain/tests.py:564:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:570:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:583:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:589:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:602:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:608:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:616:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:620:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:620:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:625:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:629:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:629:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:631:1: E302 expected 2 blank lines, found 1
plone/transformchain/tests.py:642:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:657:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:672:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:692:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:702:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:708:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:719:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:719:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:725:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:736:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:736:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:741:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:751:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:756:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:767:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:772:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:782:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:789:13: T001 print statement found.
plone/transformchain/tests.py:789:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:794:17: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:804:18: D001 found failUnless( replace it with assertTrue
plone/transformchain/tests.py:804:80: E501 line too long (89 > 79 characters)
plone/transformchain/tests.py:811:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:821:34: Q000 Remove bad quotes.
plone/transformchain/tests.py:826:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:831:13: E301 expected 1 blank line, found 0
plone/transformchain/tests.py:846:14: D001 found assertEquals replace it with assertEqual
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/transformer.py Imports are incorrectly sorted.
plone/transformchain/transformer.py:0:1: C101 Coding magic comment not found
plone/transformchain/transformer.py:0:1: I001 found unsorted imports
plone/transformchain/transformer.py:12:1: E302 expected 2 blank lines, found 1
plone/transformchain/transformer.py:17:1: E302 expected 2 blank lines, found 1
plone/transformchain/transformer.py:27:9: E303 too many blank lines (2)
plone/transformchain/transformer.py:38:80: E501 line too long (85 > 79 characters)
plone/transformchain/transformer.py:56:27: F841 local variable 'e' is assigned to but never used
plone/transformchain/transformer.py:57:80: E501 line too long (88 > 79 characters)
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/zpublisher.py Imports are incorrectly sorted.
plone/transformchain/zpublisher.py:0:1: C101 Coding magic comment not found
plone/transformchain/zpublisher.py:0:1: I001 found unsorted imports
plone/transformchain/zpublisher.py:4:1: F401 'Interface' imported but unused
plone/transformchain/zpublisher.py:16:80: E501 line too long (118 > 79 characters)
plone/transformchain/zpublisher.py:18:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:29:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:41:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:44:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:71:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:79:80: E501 line too long (97 > 79 characters)
plone/transformchain/zpublisher.py:80:52: E272 multiple spaces before keyword
plone/transformchain/zpublisher.py:80:80: E501 line too long (100 > 79 characters)
plone/transformchain/zpublisher.py:81:10: E128 continuation line under-indented for visual indent
plone/transformchain/zpublisher.py:81:10: W503 line break before binary operator
plone/transformchain/zpublisher.py:81:80: E501 line too long (105 > 79 characters)
plone/transformchain/zpublisher.py:82:9: E124 closing bracket does not match visual indentation
plone/transformchain/zpublisher.py:84:80: E501 line too long (92 > 79 characters)
plone/transformchain/zpublisher.py:91:1: E302 expected 2 blank lines, found 1
plone/transformchain/zpublisher.py:99:42: E261 at least two spaces before inline comment
plone/transformchain/zpublisher.py:99:80: E501 line too long (90 > 79 characters)
plone/transformchain/zpublisher.py:106:80: E501 line too long (82 > 79 characters)
plone/transformchain/zpublisher.py:108:14: E261 at least two spaces before inline comment
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.transformchain/2//violations
```
ERROR: /home/jenkins/workspace/package-plone.transformchain@2/plone/transformchain/interfaces.py Imports are incorrectly sorted.
plone/transformchain/interfaces.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.transformchain@2/plone/transformchain/tests.py Imports are incorrectly sorted.
plone/transformchain/tests.py:0:1: I001 found unsorted imports
plone/transformchain/tests.py:87:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:99:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:103:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:103:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:118:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:132:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:136:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:140:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:142:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:157:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:169:18: Q000 Remove bad quotes.
plone/transformchain/tests.py:173:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:173:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:204:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:225:34: Q000 Remove bad quotes.
plone/transformchain/tests.py:231:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:235:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:235:28: Q000 Remove bad quotes.
plone/transformchain/tests.py:235:36: Q000 Remove bad quotes.
plone/transformchain/tests.py:237:1: C901 'TestTransformChain.test_transform_mixed' is too complex (13)
plone/transformchain/tests.py:302:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:306:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:333:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:337:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:337:28: Q000 Remove bad quotes.
plone/transformchain/tests.py:339:1: C901 'TestTransformChain.test_abort_chain' is too complex (13)
plone/transformchain/tests.py:352:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:355:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:358:24: Q000 Remove bad quotes.
plone/transformchain/tests.py:390:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:396:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:404:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:408:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:408:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:410:1: C901 'TestTransformChain.test_ordering' is too complex (13)
plone/transformchain/tests.py:423:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:429:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:442:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:448:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:461:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:467:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:475:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:479:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:479:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:481:1: C901 'TestTransformChain.test_request_marker' is too complex (13)
plone/transformchain/tests.py:494:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:500:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:513:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:519:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:532:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:538:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:546:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:550:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:550:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:555:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:559:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:559:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:561:1: C901 'TestTransformChain.test_published_marker' is too complex (13)
plone/transformchain/tests.py:574:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:580:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:593:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:599:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:612:33: Q000 Remove bad quotes.
plone/transformchain/tests.py:618:42: Q000 Remove bad quotes.
plone/transformchain/tests.py:626:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:630:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:630:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:635:19: Q000 Remove bad quotes.
plone/transformchain/tests.py:639:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:639:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:720:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:738:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:738:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:756:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:756:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:772:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:789:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:805:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:812:13: T001 print statement found.
plone/transformchain/tests.py:812:27: Q000 Remove bad quotes.
plone/transformchain/tests.py:827:18: D001 found failUnless( replace it with assertTrue
plone/transformchain/tests.py:850:34: Q000 Remove bad quotes.
plone/transformchain/tests.py:855:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:876:14: D001 found assertEquals replace it with assertEqual
ERROR: /home/jenkins/workspace/package-plone.transformchain@2/plone/transformchain/transformer.py Imports are incorrectly sorted.
plone/transformchain/transformer.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.transformchain@2/plone/transformchain/zpublisher.py Imports are incorrectly sorted.
plone/transformchain/zpublisher.py:0:1: I001 found unsorted imports
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Cela est peut-être indispensable, mais restreint considérablement l'ambition d'Open Data. Dommage que cela ne soit présenté que maintenant, plutôt qu'au moment du débat sur https://www.republique-numerique.fr/ .


I'd advise against using `map` and friends. It's not particularly Pythonic and may well break in Python 3 where `map` returns an iterator. List comprehensions are clearer and usually even faster:

```python
return [absolute_url(u) for u in asset_urls(webassets_env, bundle_name)]
```


Ma pregatesc pentru viata, boss! @BoldijarPaul


Changed approach and this code will be removed


The ability to pass a continuous data memory is mainly useful for usual images where the data is in a continuous memory. I wouldn't try to provide this "memory" wrapping for this type of HashImage. Then you don't need a pointer as member variable which isn't good. It's much better to have

```c++
typedef UnorderedMap<int, VoxelType> DataMap;
mirtkAttributeMacro(DataMap, Data);
```


it's a block. and it has only required property `height`. look this https://github.com/datarockets/datarockets.com/commit/fd02c40281920409559caadb298235921c3da9ce#diff-746d05feade62be2299ff867be1ec421R1


Also this should not be a pointer. I would use
```c++
mirtkPublicAttributeMacro(VoxelType, DefaultValue);
```
which is similar to the ```InterpolateImageFunction``` class.


@yehudasa at the first look it looks like a tightrope walker patch but it may make sense...


Стоп. Куда уехала логика эта? Она нужна для трека предустановок.


Hey, thanks for spotting this, i have updated the scope.sh file to not have this error.


An integration test! Great!


Send us PR when you have it working ;)


The ```data``` argument I would remove from any constructor and the Initialize function for reasons stated above.


It look better )) nice )


first commint


TeamCity Civic Assembly :: Build all [Build 50](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1122&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


lots of things in the core are wrong ;-)


Please add empty line in the end of file


I think it's fine if you just remove those functions from this class. I anyway don't like them too much...


I look foward to hypotermia, in game, but I am slightly worried on how it will work with the siemens coefficients, and how we will have to tune it down so people with certain clothes will be able to go out into the wilds... possibly to get syrups to stave off the cold? or make people do fires outside?


@vuntz.  OK, do we have a bug filed for that somewhere?  Or have things changed since then?


Please, use more informative names of variables


I'd like to avoid actually creating sockets and processes in the unit test suite. Instead, this should probably use `mock` to stub out the functionality to verify that the right calls are being made and that expected / unexpected return values are handled appropriately. For a reference, see ` kmip/tests/unit/services/server/test_session.py`.


See `mock` comment below.


Can we simplify `I*I` to `-1`?


Iits LA.BF.1.1.3_rb1.9


The official MS doc:
https://msdn.microsoft.com/en-us/library/windows/apps/hh465764.aspx

Bottom-line: background-images are not suppressed (my memory was wrong, blame that freaking snowstorm), but I’m not sure what happens with a black arrow when the Windows theme is black background/white text. I need to build a test case, and hoping G. is not lost in the Norwegian wilderness. Or maybe post it on the TXP-forums…

Thinking out loud, one could do something like:

```
@media not (-ms-high-contrast: active) {
   select::-ms-expand { display:none  }
}

@media (-ms-high-contrast: active) {
  select { background-image: none;  }
}
```

PS - You should check Hive after pressing Cmd-opt-control-8 on your Mac (if it does nothing: Keyboard  Preference Panel > Shortcuts > Accessibility - check Invert colors)


thx. i finally had an example request that really needed cookies to test with. this is near a CRAN release now!


That should go to the code of `imageset`, the current PR is good to go.


Please remove commented code that is not needed/useful for this special purpose image class.


it's private. But ok)


As I explained in class on Thursday, you shouldn't have two copies of the code that handles the mechanics of stepping to the next question. Instead, you should factor that out into a private method that is called no matter which of the two triggering events occurs.


it would be possible to display filtered positions (e.g. when we TableScanNode is translated into Scan, Filter and Project), by subtracting output rows - input rows


The += operator isn't necessary here.


Should this be dealt with by adding an abstract method to [manifold.py](https://github.com/j-towns/pymanopt/blob/master/pymanopt/manifolds/manifold.py) and hence all manifolds? Otherwise the respective item can be removed from [todo.md](https://github.com/j-towns/pymanopt/blob/master/todo.md).


Yep gd2 is not supported for XU3/XU4.


But what is the replacement then? Using strings instead of constants again?


anyway I'd like to know what does it do =)


Something like this (completely untested)

`        elif street in self.holeStreets:
            gamedef = Card.games[self.gametype['category']]
            hrange = gamedef[5][gamedef[3][street]]
            result = holecards[hrange[0]:hrange[1]]
`

probably hoik the gamedef up higher and use it to replace the other handsize bits, too.


According to the documentation:

```
# time in seconds from last command sent before a queue kills itself
'queue_timeout': '60',

# time in seconds to wait on queue.get() block before raising Queue.Empty exception
'wait_timeout': '5',
```
The crash when happened showed the following lines:

```
[2016-02-12 20:06:42,260] INFO    [/var/log/nginx/access.json] - watching logfile
[2016-02-12 20:06:42,262] INFO    [/var/log/auth.log] - watching logfile
[2016-02-12 20:06:42,262] INFO    [/var/log/syslog] - watching logfile
[2016-02-11 06:31:45,102] INFO    [/var/log/syslog] - file rotated
[2016-02-11 06:31:45,102] INFO    [/var/log/nginx/prod-pvb.dsd.io.access.json] - file rotated
[2016-02-11 06:31:48,017] INFO    [/var/log/nginx/prod-pvb.dsd.io.access.json] - watching logfile
[2016-02-11 06:31:48,018] INFO    [/var/log/syslog] - watching logfile
[2016-02-11 07:57:44,774] WARNING cannot parse as rawjson: message
[2016-02-11 09:12:36,564] WARNING cannot parse as rawjson: message
[2016-02-11 16:59:12,099] INFO    [/var/lib/docker/containers/4c8a31f349906aca8571ac1075fa82708f56077fcd0d59cf9ca3a61969df080e/4c8a31f349906aca8571ac1075fa82708f56077fcd0d59cf9ca3a61969df080e-json.log] - file removed
[2016-02-11 17:59:14,711] INFO    Queue timeout of "3600" seconds exceeded, stopping queue
Process Process-13:
Traceback (most recent call last):
  File "/usr/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/local/lib/python2.7/dist-packages/beaver/dispatcher/tail.py", line 83, in create_queue_producer
    manager.run()
  File "/usr/local/lib/python2.7/dist-packages/beaver/worker/tail_manager.py", line 76, in run
    self._tails[fid].run(once=True)
  File "/usr/local/lib/python2.7/dist-packages/beaver/worker/tail.py", line 118, in run
    self._ensure_file_is_good(current_time=current_time)
  File "/usr/local/lib/python2.7/dist-packages/beaver/worker/tail.py", line 211, in _ensure_file_is_good
    fid = self.get_file_id(st)
UnboundLocalError: local variable 'st' referenced before assignment
```

However a backend beaver issue should have been seen as the warning indicated [here](https://github.com/python-beaver/python-beaver/blob/645381f794999064c66145e968bd771bc49d9031/beaver/run_queue.py#L29).

So the issue is more complicated.


TeamCity Civic Assembly :: Build all [Build 50](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1122&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 165 (1 new), passed: 7, ignored: 4 Build time: 00:06:36
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1455546805448: <no details avaliable>

Blog categories tests: Should not create new blog category if cancelled: <no details avaliable>

Blog categories tests: Should add new blog category: <no details avaliable>

Blog categories tests: Should not change blog category: <no details avaliable>

Blog categories tests: Should change blog category: <no details avaliable>

Blog categories tests: Should not delete blog category if cancel pressed: <no details avaliable>

Blogs tests: Should not create new blog if Cancel button pressed: <no details avaliable>

Blogs tests: Should add new blog: <no details avaliable>

Blogs tests: Should not change blog if Cancel button clicked: <no details avaliable>

Blogs tests: Should change blog: <no details avaliable>

Blogs tests: Should not delete blog if Cancel button was pressed: <no details avaliable>

##### there are 155 more failed tests, see build details
```


I have (yet another :D) question : when straightening a request, the resulting object contains a url (`obj[[1]]$url`) but also a decomposed url (`obj1[[1]]$url_parts`).
This later is very practical, for example for a quick url edit without having to string-edit/regexp the whole url.

But when editing the `url_parts`, it's not taken into account when doing a `make_req()`, the base $url is used. Would it be possible to rebuild the url from `url_parts` instead ?


C'est normal que ça explose un peu au début. Si vous respectez la structure donnée, qui est la même que pour le projet Laravel, ça devrait le faire comme un charme.
```
# configuration de nginx (qui gère les fichiers statiques et fait office de proxy vers puma)
/var/www/config/nginx.conf
# configuration de puma (production|staging|etc...)
/var/www/config/puma.rb
# script qui lance puma
/etc/services/puma/run
```

Malheureusement, quand c'est puma qui part en vrille, il est difficile d'avoir accès depuis le conteneur. Un bon moyen de "tester" est de le lancer à la main.

```console
$ RAILS_ENV=production rails console (ou server)
```


Returning a mutable string like this isn't a good practice. The return type may be `NSString` but the debugger will say `NSMutableString` and future developers might exploit that.

We'll talk about `copy` in a later checkpoint.


Probably a silly thing, but could you order by `id` since it's already indexed?


GTNH #2


yes, exactly


:+1:  @hakanersu aslında bir VERSION.txt atılsa güzel olur. Kullanıcıdaki versiyonun hangisi olduğunu bilmek debug ve destek için iyi olur.


so what are you proposing @expez ?

as @Malabarba sez the `cider-jack-in-*` vars are not supposed to be changed by the user (they are not defcustoms, there are defcustoms to modify the CL params for the user)


I would include the `updateQuestion()` call in here, since it inevitably needs doing as part of moving to the next question, regardless the triggering circumstances.


Looks good now


why is this one required?


Then I don't understand why this was committed.


@mipreamble I haven't gone looking, but http://android-arsenal.com/search?q=cache turns up several possibilities.


Probably requires a parser for the floor temperature, but not just at the minute, use fixed values...


ScopedRef


Added in 1ac7fdcd136bbb7ae5476cb54f670be20178c4e2.


Мы это обсудили. Эта статистика больше оказалась не нужна. К тому же новый сторедж пока не умеет различать новую загрузку карты от обновления.


I would strongly oppose an automatic redirect. This just results in a self DOS.


That was my eclipse formatter.


Hej


I think something went wrong in that commit: you removed `maxbaroffsettextleft` and `maxbaroffsettextleft`, as well as the new comments


#
![img-20160211-wa0005](https://cloud.githubusercontent.com/assets/17186021/13051646/b1325a20-d3fa-11e5-87a9-dddf177d392c.jpg)


Yes, as long as all (new/custom) messages implement hashCode.
What I did here is similar to `DelegatingFactory#accessMessage`


we can put this line `Runnable a = () -> super.clone();` in any new method.


Kommentar


not sure where to store that missing image


isn't it possible for operators to have the same type?


This should support greater than or equal to 1000


>blessing you

Hm, should'nt it be more related to sleeping?


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


lets do this.


wut? ;P


Please, add newline here too )


Thank you so much :).


Got it. Made that change.


@hvelarde Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.api/42//violations
```
src/plone/api/content.py:268:1: C901 'delete' is too complex (15)
src/plone/api/content.py:360:1: C901 '_wf_transitions_for' is too complex (13)
src/plone/api/content.py:425:1: C901 'transition' is too complex (13)
src/plone/api/tests/test_doctests.py:121:13: T001 print statement found.
src/plone/api/tests/test_env.py:31:14: D001 found declarePublic replace it with AccessControl.ClassSecurityInfo.public
src/plone/api/tests/test_env.py:32:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:33:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:34:14: D001 found declareProtected replace it with AccessControl.ClassSecurityInfo.protected
src/plone/api/tests/test_env.py:35:14: D001 found declarePrivate replace it with AccessControl.ClassSecurityInfo.private
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Does your proposal really mute only CosyDVR's notification sound? The main idea was to hide notification beeps on restart and on every fragment split.


Damn son, where did u learn that?


ага, удалено


remove this code, completely incorrect.


卧槽，群主你是怎么发现的？


Thank you for your reply.
Yes i have other products with the same issue. I think it's related to declinaison.
I also try to delete recreate a product, but it's the same result :-(


```
$ ssh agent007@localhost -p 222
ssh_exchange_identification: Connection closed by remote host
```
Can u help me to fix it?


I think FlushChanges could be useful, but making it revert the actual documents would be tricky.  We'd have to save previous state on the whole document object tree as properties were changed.  And if you were swapping "branches" of the tree it could get really funny.

I think I'd recommend removing it for now, and we could add it in a future release if there's a need.

Brant


Yes please, I haven't managed to figure out the problem. I have also the following import problem:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "theano/__init__.py", line 39, in <module>
    from theano.configdefaults import config
  File "theano/configdefaults.py", line 1234, in <module>
    "theano_version": theano.__version__,
AttributeError: 'module' object has no attribute '__version__'


This can be done with one loop and some loop preparation. We'll discuss at our meeting.


isDefualtSchema or isDefaultSchema? Appears to be a typo.


Thanks for adding that "cd project". Took me a few minutes to figure out what I was doing wrong when I first went to compile, but figured it was just me.


Still, using hashCode to compare messages is probably not a good idea.


Removing this premature commit ~


Same comment as before on returning mutable types.


Perhaps some typos isDefualtSchema instead of isDefaultSchema?


smb tried to test usage of "( )" to wrap identifier before ".clone()".
Fix to any compilable code. If not possible - remove it.


Just tried Hive Classic Yellow in OSX inverted colours, and it looks fairly good - better than the GitHub site anyway. :)

Adding additional contrast features for 4.6 is probably out of scope now - but it is definitely a subject I'm interested in (as are all accessibility topics). I'll keep an issue open for 4.7 Hive theme's. Cheers Philippe!


Remove the second line 'This situation should be checked in Main' because it's confusing for the end user


This won't work on the first question.


Please change name to ```ToGenericImage```.

Also, the return type could be ```GenericImage<VoxelType>``` instead of a pointer. To save the copy of the data, you could have another overload such as ```void ToGenericImage(GenericImage<VoxelType> *)```. As this is kind of the other direction of ```CopyFrom```, I would call this one then ```CopyTo```.

This would allow the following use:

```c++
RealHashImage sparse;
RealImage dense;
sparse.CopyTo(&dense);
```
without any dynamic heap allocation of the image instances themselves.

And can be used to implement ```HashImage::ToGenericImage``` as follows:
```c++
template <class VoxelType>
GenericImage<VoxelType> HashImage<VoxelType>::ToGenericImage() const
{
  GenericImage<VoxelType> dense;
  this->CopyTo(&dense);
  return dense;
}
```


You're creating an array here and then overwriting it in the next line. You can combine the two and avoid the unnecessary assignment.


Same moment as above.


Эту строчку забыл упростить :)


Note, due to the way DM treats vars, anything unattached to an object or created outside of a proc is basically a global var. So these may potentially get changed and overwritten as multiple people deal with their characters. It might be an idea to adds them to the /datum/preferences object class, for safety's sake?


Nice use of `-firstObject`


[   64.418081] migrate_page_move_mapping page count(3) != expected_count(2)
[   64.418204] page:c16a8400 count:3 mapcount:0 mapping:e19b1144 index:0x158e
[   64.418353] page flags: 0x80010209(locked|uptodate|arch_1|mappedtodisk)
[   64.418755] private: 0  writepage=c085978c

[   64.418890] buffer_migrate_page has no buffer rc=-11
[   64.418980] move_to_new_page migratepage=c07c00c4 rc=-11
[   64.419598] migrate_page_move_mapping page count(3) != expected_count(2)
[   64.419755] page:c16a8400 count:3 mapcount:0 mapping:e19b1144 index:0x158e
[   64.419859] page flags: 0x80010209(locked|uptodate|arch_1|mappedtodisk)
[   64.420258] private: 0  writepage=c085978c


I'm not sure why I did this in two steps... but I think I used these 2 commands:
`for file in *\(*; do f=$(echo $file | perl -pe 's/\s*[\(\)]/_/g'); mv $file $f; done`
`for file in *_.gif; do f=$(echo $file | perl -pe 's/_\././'); mv $file $f; done`


а) константы называют капслоком (т.е. например ATTRS)
б) массив строк, в которых нет пробелов, аккуратнее записать так: `%w[refer name date country...]`


The generated method will look like this:

```
public CallTarget accessMessage(Message unknown) {
  if (unknown.hashCode() == 423430) {
    return Truffle.getRuntime().createCallTarget(MyExecute.createRoot(TestTruffleLanguage.class));
  }
  throw UnsupportedMessageException.raise(unknown);
}
```


ultima


Como primer aproach es mejor que nada! xDDD
Te lo acepto si quitas los ficheros ocultos .idea
Puedes configurarlo creando un fichero .gitignore en la raíz del proyecto que contenga:

.DS_Store
.idea
nbproject


I found a typo.

Open3.poepn3 ->
Open3.popen3


Nice use of `valueForKey:` here and `objectForKey:` above. We'll discuss the subtle differences today.


Так. Мы ж говорили про то что не используем `map!` (меня массив на лету) без самой крайней необходимости?.. Это — не она :)
А в идеале и вовсе ж можно вот так сделать: `films = CSV.read(...).map{...}`


a


b


c


Returning a mutable type, as mentioned before, isn't a best practice. We'll discuss.


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.transformchain/3//violations
```
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/interfaces.py Imports are incorrectly sorted.
plone/transformchain/interfaces.py:0:1: I001 found unsorted imports
plone/transformchain/tests.py:656:48: E901 SyntaxError: invalid syntax
plone/transformchain/tests.py:812:13: T001 print statement found.
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/transformer.py Imports are incorrectly sorted.
plone/transformchain/transformer.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/zpublisher.py Imports are incorrectly sorted.
plone/transformchain/zpublisher.py:0:1: I001 found unsorted imports
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Thank you @petermetz!


(на будущее) экономь слова в переменных. `longest` будет в самый раз — у нас вся программа с фильмами работает, не нужно к каждому массиву прибавлять слово films. Кроме того, если уж переменная из нескольких слов, то лучше вот так: `most_length_films`


Nice!


Строковая интерполяция, а не сложение!


Instead of giving access to the internal hash map (which isn't save and should be avoided), it would be better to just provide a ```Begin``` and ```End``` function that return a ```DataIterator```. These can then be used to efficiently iterate over the hash map entries.


In school


width="52" must be the default. No matter what I change it to, in the xml, I get the same sized switch. I copied this:
<Switch checked="true" color="red" backgroundColor="green" height="30" width="52" ios:borderWidth="2" ios:borderColor="blue" ios:borderRadius="15"
 +            verticalAlignment="center" horizontalAlignment="center" />


This seems to cause -ETIMEDOUT in case we're doing a pdata->cyclic DAC DMA.
Can you move this block?

-Michael


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


Thank you for catching that.  9ebb98e fixes this.


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.transformchain/4//violations
```
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/interfaces.py Imports are incorrectly sorted.
plone/transformchain/interfaces.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/tests.py Imports are incorrectly sorted.
plone/transformchain/tests.py:0:1: I001 found unsorted imports
plone/transformchain/tests.py:103:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:136:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:142:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:173:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:204:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:235:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:237:1: C901 'TestTransformChain.test_transform_mixed' is too complex (13)
plone/transformchain/tests.py:306:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:337:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:339:1: C901 'TestTransformChain.test_abort_chain' is too complex (13)
plone/transformchain/tests.py:408:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:410:1: C901 'TestTransformChain.test_ordering' is too complex (13)
plone/transformchain/tests.py:479:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:481:1: C901 'TestTransformChain.test_request_marker' is too complex (13)
plone/transformchain/tests.py:550:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:559:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:561:1: C901 'TestTransformChain.test_published_marker' is too complex (13)
plone/transformchain/tests.py:630:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:639:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:720:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:738:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:756:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:772:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:789:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:805:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:812:13: T001 print statement found.
plone/transformchain/tests.py:828:18: D001 found failUnless( replace it with assertTrue
plone/transformchain/tests.py:856:14: D001 found assertEquals replace it with assertEqual
plone/transformchain/tests.py:877:14: D001 found assertEquals replace it with assertEqual
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/transformer.py Imports are incorrectly sorted.
plone/transformchain/transformer.py:0:1: I001 found unsorted imports
ERROR: /home/jenkins/workspace/package-plone.transformchain/plone/transformchain/zpublisher.py Imports are incorrectly sorted.
plone/transformchain/zpublisher.py:0:1: I001 found unsorted imports
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Fuck her right in the pussy!

2016-02-15 15:52 GMT+01:00 Marco Koivisto <notifications@github.com>:

> In school
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SEMT2Group1/CASPER_Android_Client/commit/eb45fa33789903222336097cf5635b9d3d8f0deb#commitcomment-16097018>
> .
>


I strongly dislike c-style for-loops.

They are vastly different to the usual cases where for-loops are used:
```
for item in $(somehow produce a list of stuff); do foo $item; done
for name in "egon" "martin" "silvia"; do echo hello $name; done
for number in $(seq 99); do echo $number bottles of beer on the wall; done
```
All these loops are similar and easily to understand for the bash-novice.

This however:
```
for ((number=1; number<=99; number++)); do echo $number; done
```
Is not as accessible. It requires some knowledge of C-style for-loops.

Compare the usual way of doing this in Python as well:
```
for i in range(5):
    print(i)
```

Or Ruby:
```
for i in 0..5
   puts "#{i}"
end
```

ruby and python do not have c-style for-loop for a good reason.. We should avoid them as well.
The cost of calling any standard GNU-tool is trivial and if the readability can be improved by one smidgen we should use them gladly.


@AndreKlang What's your take on this? I think the best way to move forward if this is still an issue, is to:

1. Wait for @aait to complete the [PayEx_Payments branch](/PayEx/PayEx.Magento/tree/PayEx_Payments) and merge this to `master`.
2. @AndreKlang does a fresh clone of the PayEx.Magento repository
3. @AndreKlang creates a new branch called something like `feature/fix-rounding-errors` and within it do everything it takes to fix the rounding errors. Preferably, the existence of a problem and the proof of it being fixed should be done by creating an automated test in i.e. [PHPUnit](https://phpunit.de/).
4. Once the work on this branch is complete, @AndreKlang creates a new pull request in which we can discuss the proposed changes.
5. Once we agree on the changes, I or @aait will merge the pull request.

How does this sound?


First, thanks for the commit message kudos. I'm trying to get better with that.

>> Switched use of requests to urlfetch, which is the de facto method of getting remote resources via HTTP.

> Could you explain this bit?

The version of requests that is included does not appear to work within App Engine. I'm going to look into what's going on there, because I'd prefer to use requests myself, and I know that it _can_ work from other projects I've been a part of. Plus, then it's more portable code!


That won't work. brew's formula is pulling and old package (20150406) and we need 20150826. See https://github.com/Homebrew/homebrew/blob/8c6777b2b8e847d5f2b447eae532fa8ecffa6467/Library/Formula/libhdhomerun.rb


Очень нехорошо такие штуки через begin/rescue решать. rescue — для плохих, неожиданных ошибок. А тут у тебя ожидаемо, что часть данных некорректна — лучше их явно отбросить (чтобы код читался как «выбрасываем фильмы, у которых только год», а не как «товарищ! тут может произойти НЕОЖИДАННАЯ СТРАННАЯ ОШИБКА»)


не. элемент родителя должен ограничивать высоту.


Merci d'être revenu à la [version](http://www.republique-numerique.fr/projects/projet-de-loi-numerique/consultation/consultation/opinions/section-1-ouverture-des-donnees-publiques-1/article-1-elargissement-du-champ-de-diffusion-par-l-administration) précédant celle [établie en Conseil des ministres ](http://www.republique-numerique.fr/media/default/0001/02/c114b88595f6497aa82363b1653a588b0b298598.pdf)


As with `nextQuestion`, I would include the call to `updateQuestion()`. Also, I generally try to avoid `if` statements if I can find a general piece of code that would work equally well in both cases. That way, there are fewer paths to consider when understanding and testing. In this particular instance, you could use essentially the same formula as in `nextQuestion`, with `-1` instead of `+1`; the one little gotcha is that to ensure you get a nonnegative answer, you need to add the length before taking the remainder. If you choose to use that approach, then `nextQuestion` and `previousQuestion` would be so similar that it would be tempting to generalize them to a single method that can move in either direction depending on whether it is passed `-1` or `1`.


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


TeamCity Civic Assembly :: Build all [Build 51](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1129&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


@vuntz Why is this `clone` necessary?  I'm interpreting your "let's copy that" comment to refer to copying the behaviour, not cloning a `Hash`.


Actually I think that Ive found the problem.