Don't know if you guys caught this one. This was a relatively classic Djanog update from 1.8.6 to 1.9. Now after installing and having Gunicorn restarted (``pkg_resources.get_distribution('django').version`` in the shell also shows 1.9 as version number) I am not seeing the new admin layout in the browser (Chrome), even when deleting cache and reloading the page,

Cheers
Holger

-----
(venv)ubuntu@[MYIP]:~/fs-api-admin$ pip install -U Django
Downloading/unpacking Django from https://pypi.python.org/packages/source/D/Django/Django-1.9.tar.gz#md5=110389cf89196334182295165852e082
  Downloading Django-1.9.tar.gz (7.4Mb): 7.4Mb downloaded
  Running setup.py egg_info for package Django

    no previously-included directories found matching 'django/contrib/admin/bin'
    warning: no previously-included files matching '__pycache__' found under directory '*'
Installing collected packages: Django
  Found existing installation: Django 1.8.6
    Uninstalling Django:
      Successfully uninstalled Django
  Running setup.py install for Django

    no previously-included directories found matching 'django/contrib/admin/bin'
    warning: no previously-included files matching '__pycache__' found under directory '*'
    changing mode of build/scripts-2.7/django-admin.py from 664 to 775
      File "/home/ubuntu/venv/lib/python2.7/site-packages/django/conf/app_template/models.py", line 1
        {{ unicode_literals }}from django.db import models
                                 ^
    SyntaxError: invalid syntax

      File "/home/ubuntu/venv/lib/python2.7/site-packages/django/conf/app_template/apps.py", line 4
        class {{ camel_case_app_name }}Config(AppConfig):
              ^
    SyntaxError: invalid syntax

    changing mode of /home/ubuntu/venv/bin/django-admin.py to 775
    Installing django-admin script to /home/ubuntu/venv/bin
Successfully installed Django
Cleaning up...
(venv)ubuntu@[MYIP]:~/fs-api-admin$


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


I have an compile error here: ```request``` passed in ```willSendRequest``` should conforms ```requestType``` protocol, right now the request which you passed in is a ```NSURLRequest``` :p


What is the purpose of returning an int? Will you use this integer for another purpose than comparing with 0? If no, I think you could put the `0 !== $diff` in your method and maybe rename to `matchingSeparators()` to return a bool.

But I'm maybe out of context! :)


Debug (see also below).


LGTM


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.863](http://teamcity.monogame.net/viewLog.html?buildId=11837&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:04:52


ok, update on PR Issue55-try-3


_problem resolved
-needs save button functionality


Good job! :beers:


Thanks! Here is the algorithm. If it is not easy to understand, I can send diagrams instead.

One-by-one radial algorithm

Find coordinates of centre by finding the mean of all data points
Order list of points by distance from centre, furthest first
Plot data point 1
Plot data point 2
Connect with line
Plot data point 3
Connect with lines to data point 1 and data point 2
Plot data point 4
Connect with line to nearest data point
If data point 4 is within the polygon, this data point will already be connected to 2 others; connect data point 4 to both of these points; remove the longest line of the smallest triangle and the line which formed the second larger triangle.
If data point 4 is outside the polygon, this data point will already be connected to 2 others; connect data point 4 to both of these points; remove the line which formed the smallest triangle, then remove the line separating the triangle from the rest of the polygon.
Continue for all further data points as data point 4.


You need to update the CoreBundle: https://sonata-project.org/bundles/core/master/doc/reference/form_types.html#symfony3-supports. But please in mind the dev branch is a work in progress.


Extra line. Is it required by some convention?


Sao nội dung ko có gì vậy?


Adding outcommented code doesn't really make sense. It would be better to at it later, if it will be needed at some point.


Thanks!


No, it should be removed


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


ok


With this all the modify events go through all the elifs even if they're being ignored.


Yes, it's on purpose, see commit title, the feature needs more work and will be introduced in next sprint


We should meet and discuss if you want to format this code in a different way.


>  2) thats why i'm checking zk.exists first ...
I meant: node may exists when you call zk.exists, but it might disappear when you will call zk.get_children


The Turtle passed through the data parameter in both registerTransformer() and registerTransformerFactory() specifies both registrations are containers. I've not changed the data parameter; just aligned the headers as needed. If Transformers and TransformerFactories need not be containers, then I think the code in master needs changing and this patch be discarded. The container issue aside, a slug should be supplied to give the resource a readable name rather than a generated ID.


Hm, I don't see any exception in this method. And wouldn't it be better in any case to create the diff first and then check the result? And then throw exceptions or what not?


@swatijadhav just reviewing this again. Definitely correct to always send id through and look it up again in the DB when the ActiveJob runs.

I think now we can remove the `if(token)` code, unless you think it is possible there is a case where the job would start to run before the authtoken has been saved?


Since you're adding a note about snappy, can you also fix the sentence:

"""pghoard compresses the received WAL logs and basebackups with LZMA (level 0)
in order to ensure good compression speed and relatively small backup size."""


```
root@vagrant-ubuntu-vivid-64:/home/vagrant/Selenium-Setup# wget https://getcomposer.org/composer.phar
--2015-12-02 10:05:14--  https://getcomposer.org/composer.phar
Resolving proxy.avangate.local (proxy.avangate.local)... 10.5.5.254, 10.5.5.254
Connecting to proxy.avangate.local (proxy.avangate.local)|10.5.5.254|:8080... connected.
Proxy request sent, awaiting response... 200 OK
Length: 1200480 (1.1M) [application/octet-stream]
Saving to: ‘composer.phar’
```


dsa


What happens when the spawned thread fails during the process of updating, but then the search index property has been set to indicate that index is properly built? :)


There are some weird (wrong) dates in the `CoreData_1Dec2015.xlsx` file, e.g. "21/1/2014" in row 1155. I have corrected these to 12/1/2015 based on the Picarro timestamps.


typo: identified


true


@rande we should make a doc description to this method IMO,that this works with a little magic


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


The addr:port type is required by l2tp as well. I provided two new types, `hostport` and `ipaddrport` which you can use. See 9b30357454ea53a9f1d7e82d8f894db5f409de98.


intentionally public? I guess it should be protected.

maybe we should rename it to `getDeserializerForElementName` to be more explicit?


@OskarStark my current focus is to make Sonata works with SF3.0. We can add the missing sugar ASAP the migration is done ;)


Hi @m-kuhn and @volaya,

To be sure, I'd like to resume the comportement if value is set to None:
* For non-optionnal parameter without default value (layer, table, etc): throw an exception
* For non-optionnal parameter with default value (String, Number, etc): set value to default ? or throw an error ?
* For optionnal parameter with default value: set value to default ? or to None ?
* For optionnal parameter without default value: set value to None

I'm not sure to have well understand what you should when a parameter has a default value.


What if somebody add two SciptjsLoaders as sibling? Will both load google maps api? Will it not break the app?


The `Ban after:` comment is not more up to date, isn't it?


let me try if this works out for me..


@simongoffin @Whisno

The problem is from the updateBalance function which resets the partial_reconcile field, so in the js

https://github.com/odoo/odoo/blob/c02d0287d80ab6ca693f3d6543ff8e94f221d434/addons/account/static/src/js/account_widgets.js#L1560-L1567

we should remove the "Update partial reconciliation if necessary" tab. Should i create a new PR for this?


So my concern with the current design is that there are lots of different methods that can post events to the `EventBus`, which makes for a complicated interface - when you're trying to maintain this code now, you have to check carefully to see whether the event gets fired by tracing through multiple calls.

Also, symmetry in the interface is important - if `sendOdkResultToServer` is the only thing that can fire a `SubmitXformSuceededEvent`, it should also be the only thing that can fire a `SubmitXformFailedEvent`. This makes it easier to understand how the code fits together.

For these reasons, I think a much cleaner design is to ensure that all posting to the `EventBus` happens from the one method. To do this, you would change the call site to something like:

```java
Uri uri = data.getData();
if(!validateContentUriType(uri)) {
    EventBus.getDefault().post(new SubmitXformFailedEvent(Reason.CLIENT_ERROR));
    return;
}
```

What do you think?


At least you tried... ammout


uhhhhh whatt?


There has to be a better way


I see. :ok_hand:


C in english.


i2d_OCSP_RESPONSE() накручивает указатель на всю длину данных


I don't know why but with this version store.dispatch no longer returns the dispatched action as it is described in redux documentation (http://rackt.org/redux/docs/api/Store.html#dispatch).


That page is out of date, but hopefully more modern browsers act **less** strange.

For example, on desktop Chrome this turns normal `textarea` into effectively read-only:

``` javascript
$0.onkeydown = function(e) {
  if (e.preventDefault) e.preventDefault();
  if (e.stopPropagation) e.stopPropagation();
  return false;
}
```

Here's a demo to try:

https://gist.githack.com/mihailik/69e762837a4dc8276a2e/raw/d395d5f63f326f0c0d9d5f0f99f1db8f3b8a05d7/index.html

I haven't got iOS devices, but it indeed works as expected on Chrome desktop/mobile, FireFox, IE9+.

Note that FireFox/mobile this keydown.preventDefault trick doesn't work. Perhaps detecting Gecko and enabling readonly. Unfortunately, setting readonly on textarea in FireFox also disables touch keyboard.


$this->api again, shoud be app :)


@steveyken yes, it happens, sometimes job starts before the authToken is saved. Hence just for safer side kept that condition.


dd


지우는 법을 모르겠어요 이거


Looks good. Please PR this to v2.x.


Als nog 1 iemand dit aanpast wordt ik boos!!!!


Resources.Load  will be failed cause `string path = "Assets/Slua/setting.asset";` is not under `Resources`


standards


`:type fill: bool`


oh, that's why there was this conflict.


4 space indent


Space before brackets


We'll need to start making use of this tag then :P


I see. TBH I would prefer that who- / whatever uses this takes care of setting User.current. But I am ok with this solution too given this is only used internally.


ammo and shield* floor units


resolved in #f8e1136


I'd leave it as "Signal A and B can both have undefined pivots".  We should get rid of `None` in pivots, I'll send a patch for that later today.


This seems weird to limit here after all the data is fetched...wouldn't it be better to send the limit via the API?


Duplicate implementation for searching a file.


PHP Are cool. It's very simple.
I begining on PHP. :smile:


Ça me parait être dans l'esprit de la classe, elle est prévue pour être étendue en fonction des besoins, d'où le template.


Early morning commits are a good idea. It is just me. Me migrating unit/integration tests.


Nit: typo


I'm very much for removing. Will do that in this PR.


Does it bother you so much? I like to have an example here so the user directly gets what's meant. Your suggestion is just literal restructurization^^


This is really great! So happy to see my little mockup spark something awesome.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 240](http://ci.infinispan.org/viewLog.html?buildId=32354&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for f8e336b281588a18b8c8f02baffca124b5660785
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 1


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 242](http://ci.infinispan.org/viewLog.html?buildId=32405&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for f8e336b281588a18b8c8f02baffca124b5660785
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:00:02

Process exited with code 1


101162510B-SubsurfaceOutwardAngleWarning-5016 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1974 of 1976 tests passed)

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


in sätz muesch das ez sicher nöd überall gross schribä, das isch ja kein name womer gross schribt. please revert
in titel und labels ischs scho ok wennd capital letters bruchsch


Warum haste das denn rausgenommen?


I don't like the idea to have mutable devicesType  in the header and init it the ctor. I propose to keep the anonymous namespace and, again, check for c++11, and if available then use pretty brace map init, in the worse case use something like this http://stackoverflow.com/questions/2636303/how-to-initialize-a-private-static-const-map-in-c . So everything will be in the cc.


HBIRE_vector_take2 (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Pretty hardcore here... how about making templates for getters or setters? Or maybe consider using luabind :P


这个方法是没法调用的，因为pollDesc是没有导出的struct。


Since new release, I have a bug when the program start with Lyra2v2 algo :
````
G:\Miners\ccminers>cd release74
G:\Miners\ccminers\release74>ccminer.exe -a lyra2v2 -o stratum+tcp://pool.verters.com:4444 -u xxxx -p xxxx
*** ccminer 1.5.74-git(SP-MOD) for nVidia GPUs by sp-hash@github ***
        Built with VC++ 2013 and nVidia CUDA SDK 6.5

  Based on pooler cpuminer 2.3.2 and the tpruvot@github fork
  CUDA support by Christian Buchner, Christian H. and DJM34
  Includes optimizations implemented by sp, klaust, tpruvot, tsiv and pallas.

[2015-12-02 11:13:17] NVAPI GPU monitoring enabled.
[2015-12-02 11:13:17] Starting Stratum on stratum+tcp://pool.verters.com:4444
[2015-12-02 11:13:17] 1 miner thread started, using 'lyra2v2' algorithm.
[2015-12-02 11:13:17] Binding thread 0 to cpu 0 (mask 1)
[2015-12-02 11:13:17] Stratum difficulty set to 0.4
[2015-12-02 11:13:17] pool.verters.com:4444 lyra2v2 block 412683
Cuda error in func 'scanhash_lyra2v2' at line 141 : out of memory.
````
My config : EVGA GTX 960 2 Go GDDR5, driver v359.06 (latest), OS Windows 10 Pro 64-bit, 8 Go RAM. Previous version (release72) was ok.


We need a proper API for this


Thank you so much for this!

I even posted a bug report back then, because I have mistaken this "special effect" for a graphical glitch:
http://savannah.nongnu.org/bugs/?37659


Please refactor by adding hybris-common target with all common parts from both lines, thanks!


ok


I think we should remove the composer install completely. The documented method to install the pjax js it is by using `bower-asset/yii2-pjax`.

if you use `yiisoft/yii2-pjax` it will install the project without problems but on the 'wrong' folder since the pjax assets will try to load it from `@bower/yii2-pjax` https://github.com/yiisoft/yii2/blob/master/framework/widgets/PjaxAsset.php#L20


Please, add tag for this commit, because bower install this plugin 1.5 version without this fix.


Seems odd to have this data exported.  Previously it was only included in tests.  If you need it for the evaluation code too, maybe put it in a testdata package instead?


`alpha_t` should be `2. / 7`, the condition is valid for `T=3` and `T=4`


URL parameters are processed by the key name instead of the index, allowing the parameter to be in any order on the URL.
getParamValueOf(requestData[0]) -> requestData.get("kpiId")


:+1: yay


Escaping.

(Yes, I realize this is test code, but might as well do it right.)


Nice. Great update.


Add a comment with the graph here to make it clear where the `3/2` and `2/7` come from


change canols for canola.


Wieso hast du das alles entfernt? :open_mouth:


Modified steps section as per the review comments.


Try to add 16GIG of virtual ram reeboot and try again.
If it doesn't work, reduce the intensity with the -i parameter.  (ccminer -a lyra2v2 -i 13 ...

try -i 13 , then -i 14, then  -i 15.... until it crashes.


I know right ?? well as of now little smart dot is at v1.2 and I think soon enough I'll make that appear online, stay tooned ;)

just optimizing the aim-to-kill system and the split-to-evade system these days


Thanks


Should really be 1.10 instead of 2.0, 1.9 instead of 1.9rc2


You welcome


We should log an error here


onDrawItem in not called


LuaTypes sa mi zdá trochu pritiahnuté za vlasy, radšej asi pridám 3 wrappery na LuaValue, LuaValueMap, LuaValueList


Another drawback of this approach. If TransportAdapterImpl is not singleton, then this map will be created for each instance.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.605.58002108455f5f96d13e8f6b84314997f0c65a00](http://build.palaso.org/viewLog.html?buildId=66757&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:05:29


No, fortunately it doesn't. Because `sortedBy` is stable in the sense of not changing the order if the keys are identical we actually only need two levels in sort key, not three.
With this change things become so much simpler it's almost embarrassing!


Thank you for the tip, I pushed a commit with these changes. The tests are looking much better now!


@boffbowsh Agreed and addressed here.


yeah but it's more of a general thing, keep it short :)

2015-12-02 11:28 GMT+01:00 Makman2 <notifications@github.com>:

> Does it bother you so much? I like to have an example here so the user
> directly gets what's meant. Your suggestion is just literal
> restructurization^^
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/coala-analyzer/coala/commit/febe0fcb28c66f94f7826223d9c910a9763effca#commitcomment-14731880>
> .
>


It was in the task description:
ReceiveFromANode () : Int //блокирует поток до получения сообщения
I was wondered a bit, but made as Dmitry ask.


Is there any convention regarding names for the bool members & methods? I mean e.g. "is" prefix.


why you made this


@jiachenwang  :'( why?! check git output before commiting! [resolve](http://githowto.com/resolving_conflicts) conflicts after [merge](https://git-scm.com/book/en/v2/Git-Branching-Basic-Branching-and-Merging)


Il est trop bow


pep8 recommends "Use the function naming rules: lowercase with words separated by underscores as necessary to improve readability", so `get_overshoot()`


@jiachenwang ...


After all, "turning things around" doesn't really buy much, since the heavy lifting is actually done around the sorting of the request headers, which can't be pulled out. So, in order to keep things consistent between all "negotiator" I structurely left things as they are (headers as members, alternatives as parameters).


`get_undershoot(self, **kwargs):`


@jiachenwang it hurts...


@jiachenwang eh....


Can we reuse this test for getOvershoot():

    self.assertEqual(s.get_overshoot(method="rect"), expected)


fontHandle = cc.path.join(cc.loader.resPath, fontHandle);


Это неправильный код по следующим причниам:
1. Он заставит картинку загрузиться два раза
2. Это не соответсвует тексту задания


satisfies issue #12


Je propose plutôt de créer un _middleware_ réutilisable, avec une véritable gestion des erreurs.

```js
function extractInsee(req, res, next) {
    doSomething()
    if (/* erreur *) {
        return res.status(400).send({ code: 400, message: 'blabla' });
    } else {
       req.codeInseePropre = 12345;
       next(); // On continue
}

Derrière tu l'utilises dans les routes :
```js
 router.get('/localisant', extractInsee, function (req, res) {
    /* Logique du contrôleur */
});

Pour le cas spécial des __communes__, gères le cas directement dans le _middleware_, en testant sur les clés présentes dans `req.query`.

+ des tests sur le middleware ;)


Cependant, j'ai plusieurs remarques sur l'implémentation :
- Tout d'abord, je n'ai pas mis de documentation, mais dans mon esprit duration doit retourner un temps en secondes, pour garder le généricité de la classe chrono. Dans les évolutions futures je prévoie, peut-être, l'implémentation d'une classe Benchmark qui prend en template un chrono pour faire des mesures multiples de temps d'une fonction et de générer des statistiques de temps d’exécutions (min, max, moyenne, écart type...).
- Je pense que si tu veux mesurer des ticks c'est pour plus de précisions, cependant je ne pense pas que la fonction clock() corresponde à ce que tu as besoin voire : http://en.cppreference.com/w/cpp/chrono/c/clock par exemple
- Premier problème, elle semble mesurer des temps pour touts les threads du programme, or ros fait tourner des thread en fond pour les communications TCP, donc tes mesures sur une fonction seront faussée.
- clock() est une des fonctions de mesures de temps les moins précises, je te conseille plutôt de regarder du coté de clock_gettime avec CLOCK_REALTIME en paramètre qui donne une précision jusqu'à la nanoseconde. C'est que j'utilise quand j'ai besoin de précision.


I used template like [this](https://github.com/biancama/akka-docker-cluster) and unfortunately I didn't have enough time time to clean up the code.


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1991 of 1991 tests passed)

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


Hi Israel, yes this is a GPL project, thanks for reminding me to add a license to it. But I see that you already cloned the repository yesterday and and it has been since offered at FGMEMBERS: look, it's none of my business how you deal with your repository, but cloning something and offering it _before_ you know which license a project is licensed under seems quite fishy to me. If this turned out to be a non-GPL project for a number of reasons (let's say we had some textures or sounds that we really wanted to include, but they weren't GPL-friendly), you'd have been releasing non-free material. Well, you know all this better than me, so take this as a friendly suggestion only. Cheers!


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.606.58002108455f5f96d13e8f6b84314997f0c65a00](http://build.palaso.org/viewLog.html?buildId=66758&buildTypeId=bt322) outcome was **FAILURE**
Summary: Tests passed: 2160, ignored: 78; exit code 1 Build time: 00:14:31


:+1: WOW


I am quite not sure with this import, I will investigate it later.


why?


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


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


Sorry, somehow I missed that yesterday.  Fixed in 8758e4314a0d31f6ad68e7d03fbdec7791122224.


That's already seconds isn't it? Why the / 1000?


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


What about nested plugins?


还是先把writev去掉了，EAGAIN时不能WaitWrite，以及没有writeLock，必然会导致各种问题。
还是等golang自己支持writev吧。


Where is it documented?


Well, that was just an abstract description of the task; naturally, if you use actors, you don't need to block any threads.


1. Copieu `burcat.py`  al directory on teniu el `thermopy-0.3`.

2. Al nivell de thermopy-0.3 feu:
```
   python setup.py install
```


if proxy.config.matchContext is a function it will determine the final value returned so I'd rather do:

```
if(typeof proxy.config.matchContext === 'function'){
    return proxy.config.matchContext(req);
}

return req && utils.matchContext(proxy.config.context, req.url);
```


are you sure about using `interval_sum` here?  The way it's defined is different from the way we define `alpha_t` above.  In the description, we say that the condition is true between T=3 and T=5, and the graph shows that. However, `interval_sum` only considers the signal valid if it was asserted at the specific time.  Either fix the graph in the example above or fix `interval_sum`.  To me it sounds like `interval_sum` is not properly defined.


is unmarshalling from String gone?


Maybe throw an error otherwise? Because this would mean a conflict, which propably is a problem


I would put this in it's own file; `index.config.js`.


I don't think we should disable debug in development… its messages can *sometimes* be useful.


Is it possible to move error text to constants?


@dkayiwa True. The scenario is that, the user changes the value of the global property in the admin UI and redeploys the app. In that situation, the older blocking method gets called (https://github.com/openmrs/openmrs-core/blob/652ecf2ba4642df56f41249fe86381c3bf863d3a/api/src/main/java/org/openmrs/api/db/hibernate/HibernateContextDAO.java#L461).

The async method is targeted for people wanting to use the API directly. They will get a Future object, using which they can determine the status of the spawned thread.


I have the same problem than MaxBoltik with Symfony 2.7.
Can you fix this please (Symfony 2.7 is a LTS)


oh sorry I misread it, forget about that :)


после нужно вставить контейнер и далее по аналогии  в отдельных секциях


внутри ссылка, а  внутри нее img


.menu-toggle в твоем случае ты написал просто кнопка в хедере


The reason for this is that you cannot create an `HttpEntity` from a `String` if you use a binary `ContentType` (which is here implicitly created from the given binary `MediaType`). Therefore the entity is created from a `ByteString` here.
This makes tests a small bit more verbose but helps prevent users from doing the wrong thing (e.g. giving text-based content a binary `ContentType`).


нав входит в верхний хедер, а h1 находится уже вне его, h1 дать классы
main-header__title main-header__title--form
main назначить класс page-main после него открыть контейнер
Поделитесь своей историей - p класс  page-main__description


у подобных оберток первый класс будет form__wrapper  и 2 доп. класса с модификатором один от смысла группы инпутов другой от фонового цвета


просто form-block во всех подобных местах это обертка на лейбл и инпут, указание на стилистику и содержимое блока мы дадим модификатором выше к общей обертке каждого филдсета пример form__fieldset form__fieldset--app в блоке апликейшн


form__label в дальнейшем будут еще модификаторы


Actually, the difference is between the step-post and step-pre, the graph
that you are seeing is for step-post (the graph steps in the next index)
and interval_sum is implemented for step-pre. I will modify interval sum to
work for step post and pre both

On Wed, Dec 2, 2015 at 11:01 AM, Javi Merino <notifications@github.com>
wrote:

> are you sure about using interval_sum here? The way it's defined is
> different from the way we define alpha_t above. In the description, we
> say that the condition is true between T=3 and T=5, and the graph shows
> that. However, interval_sum only considers the signal valid if it was
> asserted at the specific time. Either fix the graph in the example above or
> fix interval_sum. To me it sounds like interval_sum is not properly
> defined.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sinkap/bart/commit/fc465d9e482382a598135cc389e1b2025909e797#commitcomment-14732620>
> .
>

--
Regards,
Kapileshwar Singh(KP)


form__fieldset  с нужным модификатором


дать классы элементам


лейблам сделать привязку через for и id


upload-form__remove + в этих блоках должна быть подпись <figcaption class="photo-item__caption">IMG-1.JPG</figcaption> внутри figure


дать класс


просто download  как navigation


доступно для ... -  download__description


advantages


можно убрать эту обертку там как внутри 2 блочных элемента


убрать бр'ы


advantages-item


после section закрываем advantages


reviews  и далее отталкиваясь от этого в этом блоке


дать классы .__list, .__item , первые li это заголовки - .__item .__item--header


I could instead make SystickHandler non-static. And cast the pointer as the function SysTickIntRegister expects it.
    void SysTickIntRegister(void (*pfnHandler)(void))

This compiles with a pedantic warning, but works:
    SysTickIntRegister((void (*)())&TivaCHardware::SystickIntHandler);

This compiles with a pedantic warning, but works:
    SysTickIntRegister(reinterpret_cast<void (*)()>(&TivaCHardware::SystickIntHandler));


i think, here is language mismatch - mix of english and russian


You're right, I'll fix that.


Ah great. Didn't see the types when reviewing here, but now that you mention it that's really nice.


contacts избегай по возможности слова в классы box, block  и т.п.где возможно, хотя порой бывают исключения т.к. они по сути не несут в себе смысла  по слову контакты понятно что это раздел с контактами, потому эти слова лишь лишние символы в коде


contacts-wrapper


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


дать класс


When do we have `pluginSettings[key][key]`?


No need  for for loop:
if you have a vector and paste that together with a single word then the single word is repeated.
However, be careful if you use this on two vectors: if the vectors are not of equal length, the values from the shortest vector are recycled.
Compare:
test<-letters[1:4]
 paste(test, "_test")
[1] "a _test" "b _test" "c _test" "d _test"

another_shorter_vector <- c("_1", "_2")
paste(test, another_shorter_vector)
[1] "a _1" "b _2" "c _1" "d _2"
another_shorter_vector2 <- c("_1", "_2", "_3")
paste(test, another_shorter_vector2)
[1] "a _1" "b _2" "c _3" "d _1"


What if we're overriding something here? Shouldn't we raise an error or at least warn?


same here


Hi @zuccs, would you mind to rollback this change. Given the `.DS_Store` is a system file, I suggest you to add it to your [global gitignore](https://help.github.com/articles/ignoring-files/#create-a-global-gitignore).


i mean, placeholder for examle in english, and button values in russian


It's valid, OK, but it made no sense here since it wasn't mutating anything, right? That's why I was expecting the compiler to throw error that I think I'm mutating the instance, while I actually don't


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


print?


That's what you meant by external!
Maybe it would be better to make `rel=external` instead of `external=true` ?
Other HTML5 valid `rel` values are:
- next
- help
- search

And I can imagine this beeing actually useful - help for example


Note that the AVR is a little ended device, which means that the low byte in messageIndex gets put on the wire first. This is the same as the x86 machine (also little ended) so that if this structure is reconstructed at the other end, there is nothing to do.

struct {
 uint8_t Type;
 uint16_t messageIndex;
 uint8_t bytes[DR3];
} payloadStructure;


It seems that each single parameter is depth dependent, and that we'll be solving chess for the next 1000000 years.


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


@anosach-luxoft, jfyi, it's better to write `using std::swap; swap();`. This will be handy in case if we will create custom swap for this class. No need to fix.


That one was made specially for getting `_id`, anyway we could add ability to customize that too.


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


I think this is the copy&paste issue. Here should be `close` function call


NOTE: As a result of these changes the .map format has changed - This means all maps will need to be reextracted


Looks like the `break` is missing here?


Dope


dope


А вот что значило вернуть стили, я подумала, что надо вернуть строку со style. Удивилась. Сейчас все будет


You have gone in the wrong direction
All you needed to do is to mock the metric factory in the constructor with something like that
    MetricFactory m = mock(MetricFactory.class);
    when(m.createCounter(any(),any())).thenReturn(mock(Counter.class));
   .....


fine by me


it's giving me compile error
http://www.hastebin.com/rapaxebigo.vbs

which kernel are u using ??

and can we talk on gtalk : aapav01@gmail.com


Already fixed through pulling in your commit: https://github.com/derickr/mongo-php-driver/commit/35e81e12b29dd3120611ffc6cbf64af142f489d4#diff-3f13a4fd6822b3a6bb69c71be3c9ea9dR491


STATUS doesn't need to be committed (FYI)


Does it make sense to extract this case into one or more methods? It is a little bit difficult to understand because of the deep nesting.
Also I would suggest to apply the default formatting to all new additions/changes.


Thanks, though I realise that I've hardcoded the fish path, where I should use the `$SHELL` env I think.


Hey!

Why did you removed legacy's changes ?

[code] && !Player.Spellbook.IsCastingSpell && !Player.IsDashing() && !Player.IsStunned;[/code]

Just curiosity, bb~


Note that the AVR is a little ended device, which means that the low byte in messageIndex gets put on the wire first. This is the same as the x86 machine (also little ended) so that if this structure is reconstructed at the other end, there is nothing to do.

struct {
uint8_t Type;
uint16_t messageIndex;
uint8_t bytes[DR3];
} payloadStructure;


Sometimes yoneda sticker!

(https://twitter.com/larsr_h/status/660800178350374912)


test


It's a `DFrame`, it has its own close button.


what is this


Hi @weppos, sorry, good idea. Have done that now.


Lokaverkefni í vsh2 er hér byrjum nú vinnuna


I don't understand. Error message mentions radiator and wall thermostat but if condition only checks for wall thermostat. Which one is correct?


#1032

> Indentation - Indenting often uses two spaces. This is followed in Programming in Lua, the Lua Reference Manual, Beginning Lua Programming, and the Lua users wiki. - http://lua-users.org/wiki/LuaStyleGuide


`interval_sum` is defined step-post for transitions from 0 to 1 and step-pre for transitions from 1 to 0.

Are you sure step-pre make sense for signals that are not binary? For signals that are only 1 or 0 you can have different behaviours for the 1 and the 0, like we do in `interval_sum`.  But if it can be any number, why are some values treated different than others?

Put it another way, shouldn't [3, 3, 3, 0, 3, 3, 3] have the opposite shape of [0, 0, 0, 3, 0, 0, 0] or [1, 1, 1, 4, 1, 1, 1]?


I think you mean "accepted"


Callback parameter is used on first initialization;


detail.value !='' should be like !detail.value


```ruby
expect do
  service.call
end.to change{ assignment.reload.deadline }.from(nil).to("2016-01-01".to_date)
```

this way you can be sure that the `service.call` was what actually changed the value to what you expect it to be


should be !==


add comment " // namespace".


All expression should be more simple


The placeholder will not be replaced here because String concatenation is used.


The Linux one was fixed by pulling in your PR's commits: https://github.com/derickr/mongo-php-driver/commit/35e81e12b29dd3120611ffc6cbf64af142f489d4#diff-788d457a20b110cc38e571dec9ddc68cL430

Windows (hopefully) fixed in https://github.com/derickr/mongo-php-driver/commit/548894d4ae6204aa2e04b82985e2e0cd7e7961ee


Sounds good. I won't tell anyone. ;)


@syl20bnr It must be really tough, when you genuinely care about your userbase, make a decision after considering all pros and cons, and then somebody just comes at you and start talking to you like you're some random nobody and do not know what you're doing at all. :( That's physically painful to see. Personally, I fully support this patch if it'll actually prevent the 2 minutes waiting before Spacemacs start.


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (917 of 917 tests passed)

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


almost the same as above, but

```ruby
expect do
  service.call
end.to_not change{ assignment.reload.deadline }.from(nil)
```


no longer used.


Already done for all of them by pulling in the commits from your PR: https://github.com/derickr/mongo-php-driver/commit/35e81e12b29dd3120611ffc6cbf64af142f489d4#diff-31bcbfc15964e93321b82f3a78c7606dL40 (and further)


What is "x"? please let me know what kind of fields you are iterating for


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


https://jakubmarian.com/in-the-future-vs-in-future-in-british-and-american-english/

> However, when it means “from now on”, there is a divide between American English and British English. An American would still say “in the future”, as in the previous case, whereas a Brit would likely say “in future” (with no article).

Still, will change as I don't want Americans complaining at me.


I personally feel that in the case of Utils libs, it's prettier to do

```
import Utils from '../core/Utils'
```

```
and then later in the code just say `Utils.toggleClass()`
```

e.g. if we end up with 20 functions it's going to be a big import, and a big claim on the namespace of this file. And also might not be clear where addListenerMulti is coming from.


Thanks! :)


Sweet stuff!


Feature 'reset'? Copy&Paste error?


if i put it in a testdata package, i would still need to export it to be able to access it. i wanted to keep the eval program in a separate location (package) to keep it separate from the tsz library


seems clang-format use another rule for the "*". Seems in the rest of the code pointer is near type.


This indent look very bad. I assume that this is some auto-format problem...


'CONTENT_BLOCKER_UPDATED' event still needed.


akhil we dont need this!


Should be "Feature 'contact' can only be a Contact");" now I assume?


Veel succes mannen!


Bu it is stell according to the google style


Perfect, thanks a lot!


Is this block correct? shouldn't it be `#if WTF_COMPILER_GCC` instead of `#if defined (__GNUC__)`?


yep, we can hide the option


Excellent commit message lol, although I do prefer `3.4` over `2.7`!


[clipsToBounds](https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIView_Class/#//apple_ref/occ/instp/UIView/clipsToBounds): *Setting this value to YES causes subviews to be clipped to the bounds of the receiver.*

So what I did was setting clipToBounds on the parent view.
`self.clipToBounds = YES;` should work.


The INT_TYPE_GXX #defines are poorly named. They should be named INTR_GROUPXX instead. Could you prepend this patchstack with another patch to rename these before they are too widely used?


Something


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.864](http://teamcity.monogame.net/viewLog.html?buildId=11841&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:34


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


Yeah, that's a bit fishy, but it's like that in the original `assert` because `(1L << 31) - 1 == std::numeric_limits<int32_t>::max()`. I wasn't sure if it was done intentionally in the original code? I meant to ask earlier but got sidetracked by the clang crash.


nit: Conditionally including a header is unnecessary.


Btnはボタンだよね。
でも呼び出すのはボタンを呼び出すのかな？
呼び出すのはジャンプだよね、メソッド名が正しいかどうかもう一度考えよう。


Fixed


@rmcc

build/core/base_rules.mk:157: *** packages/apps/FMRadio/jni/fmr: MODULE.TARGET.SHARED_LIBRARIES.libfmjni already defined by hardware/qcom/fm/libfm_jni. Stop.

device Sony z3 (aosp)


yeap, that's true


途中まではMoveBy。途中からはSequence。
こういう事柄が違う間には改行をいれよう。


Possibly yes... I don't insist on fix, if this was made by clang-format.. but this looks strange :(


Could hoist tab to top of function.


同じだね。これもメソッド名が悪くなってしまったな。
あくまでリセットする事とボタンが動作する事は別と考えよう。

例えば。
void resetSetPosition(...);
とかかな。


I don't mind it being exported from a test data package. I mind it being exported from the main tsz package.


Could possibly save some lines here:

activeTabLink = tab || $firstTab;


Should we be catching SubscriptionReminderError specifically, or are we expecting other exceptions too?


Ygb


Good catch that userAgent is the most sensible detection.
I'll expand on that and will extract the logic into a module.


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


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.866](http://teamcity.monogame.net/viewLog.html?buildId=11849&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependency failed: MonoGame :: Develop (Win10) Build time: 00:00:00


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.865](http://teamcity.monogame.net/viewLog.html?buildId=11845&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:41


Can this be static?
Also suggest to start with the private keyword for consistency.


Add a trailing new-line


That could be shortened to
`proposition = 'worldwide' if proposition = 'world-location-news'`


according to this link both styles are valid. but in the code I see at the moment another style. I vote for consistency. I don't that any style is better. I think that we can adjust the clang-format config. as far as I remember this rule is configurable.


$tab should be declared at beginning of function.


### ohk cool


Header is missing


Shouldn't this be an equality check, not an assignment?


As we have learned in the past, there is no single libpn544_fw.so that works for all versions of the M8. This commit breaks NFC on my GPE-converted Nordic M8 for instance. The built-in firmware works fine in 6.0 too so there is no reason to include this file.


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


Potentially returning undefined, just checking, are you handling this?


#srikang


suggest to start with the private keyword for consistency.


ah yes! That too :-)


don't forget to get back this before merging this PR


Erm... What is serializing JSON?

There's no refs or test case.


Would also be nice to have a ref to the commit making these changes.


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


setback to debug


you'll hate me, but delibrately -> deliberately


`C-x 8 RET runs the command insert-char`: I bound it to 'counsel-unicode-char, since they make the exact same thing: insert an unicode char at point.


You probably only need a create now that we have test isolation


Can you please add the new classes to the sample HTML that appears in the JS module itself? Just for clarity of how they're used, and to let people know they're available.


What do you want to say with this comment?


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


This is not valid for most of the CPUs out there. It's nothing but an STM32-specific hack. It can be put into generic modmachine source code if you want, but should be clearly #ifdef'ed as STM32-specific hack (which has limits of its usage even there, like not being able to access standard Cortex-M control block at 0xexxxxxxx).


Why do we need to copy this string? Pointing to it would also work.


Testing github filter


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.867](http://teamcity.monogame.net/viewLog.html?buildId=11853&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:23


The conformance to `RequestType` is at the bottom of this file. And in the demo, it do works without any compile error. 0_0


create should be enough


This commit disables the toolbar altogether, as it's commented out from the middlewares. Was that intended?
FWIW, I'm happy to drop all references to the template timings panel.


@kmonson Can you explain how this fixes the issue?  Is this just not a proper place for this code?  Is this code somewhere else?


Is this an actual todo?


سلام من از با یکم تغیر راش انداختم
exec --no-startup-id "setxkbmap -model pc105 -layout us,ir -option grp:alt_shift_toggle"


you can also drop this, newer stacks seem to work fine without it


`Accept-Encoding`


2x "most"


let's drop it completely?


there's an issue open for it already, along with a patch.


این اما کار نمیکردکه درستش کردم
exec --no-startup-id "setxkbmap -model pc105 -layout us,ir -option grp:alt_shift_toggle"


That wasn't my intention, but it'd be nice, and seems harmless.


test


@rochesterj Thank you!


@3nids unfortunately not :(


Sure. The issue was that httpretty's setup.py included some non-ASCII characters that then failed to decode as us-ascii characters (implied by `LANG=C`).

But since httpretty 0.8.10 is already released, even if condition was specified as `httpretty>=0.8.6`, `pip` (and company) would still prefer the newest released version, i.e. 0.8.10. Except when httpretty were already installed (when even 0.8.6 satisfied the dependency), and #40 is a non-issue. :smiley:


Alright; if this builds I'll drop 7.8, and bump Travis' stack.


I am not sure, that separation to 2 different files is good idea. More files - slower the work.


Is this just laying around here?


1. Why did you use 'self' in function context? simply this.
2. You can use Utils.debounce


Looks good to me :+1:


How will the user know that it failed and hence they need to reset the global property?
Since the caller gets a Future object to determine the status of the spawned thread, can't you use the same to determine when to reset the property, on successful completion?


Decard cain not recognizing swagger properly


Hi @jcfr,

sorry for not having explained extensively the signals in https://github.com/Slicer/Slicer/pull/377 .
It is true that the signals are not used in the VR widget module. However, I thought they could be very handy, when any another module has to synch it's current Scalar and Display MRML nodes to the Volume rendering ones. Ones the MRML are in synch this can be used to update the GUI of both modules, whatever the user click in one or the other. I found that this can speed up a lot the navigation through the modules.
Here examples:

https://github.com/Punzo/SlicerAstro/blob/master/AstroVolume/qSlicerAstroVolumeModuleWidget.cxx#L98-L101

https://github.com/Punzo/SlicerAstro/blob/master/AstroVolume/qSlicerAstroVolumeModuleWidget.cxx#L437-L480


Its the same value, ```(1L << 31)``` is the first value which does not fit into 31 bit so we subtract 1 to get to the largest one which fits...
And sorry for not continuing to fix your problem but I got too a little bit demotivated after seeing the clang crash and the additional warnings...


This also means bumping GHC.


I should have seen this when I reviewed it :disappointed:


sounds good :shipit:


that's great


Ya kanger


As stated, build the kernel with the following config enabled, and then please paste the compile errors here.


Oh no you didn't


No, it's for example purposes.


Declare at top of function.


Instead of exports.MKM_PRODUCT_ID_CORRECT, shouldn't you use the replacement vector that already exists?


Just saw that was corrected separately: https://github.com/alphagov/government-frontend/commit/d4242b7147c126931d855b20bcf76d762abf3fa8


This check is redundant.


Could be cleaner, less var declarations.

var activeTabLink = '',
      tab;


asasa


Derper


No


Personally I hate that syntax because it obfuscates what really happens.  Mixing natural language with programming language may make code easier to read for non-programmers and superficial programmers who tend to trust what they read, but it just makes the code more vague and harder to reason about.

Just take "is".  What does a is() unary function really do?

Since I care about understanding code I read I can't simply assume that it does "something sensible".  This forces me to read the source code for the testing framework and ensure I understand the implementation.  Not just for the sake of understanding the precise semantics, but in order to understand if that line of code is reasonable.

Don't mix natural languages into programming languiages.  It doesn't work.  It just makes stuff much harder to reason about.  There's a reason we have programming languages.  They are semantically simple so you don't need an advanced degree in semiotics to use them.


Why you kang


Thank you for your comments, @capnfabs.
I see `sendOdkResultToServer ` as still having symmetry even it doing internal calls to different validation methods.
Trigging `SubmitXformSuceededEvent` and `SubmitXformFailedEvent` is an Android-approach to propagate that somethings goes wrong inside a method. Thinking as a plain java though, i don't see as design responsibility leak having a outer method which delegates validation responsibilities to other methods.

But i totally agree with you that using this current approach you need to check carefully each method to verifies if they are trigging the event ... and it may difficult maintenance, though i think it improves readability.
what about this following design ?
```
try {
   Uri uri = data.getData();
   validateContentUriType(uri);
   // do the same for other datas
   // do whatever needs with the data
}  catch(SomeValidationException ex) {
  //Triggers failed event
}
```
The trigger event is now centralized in `sendOdkResultToServer` and near success event trigger.
Do you think it sounds better or you still prefer the original manner ?


As this is an FVP option, not an ARM standard platform option, the values should all be of the form FVP_GICXX_XXX, not ARM_GICXX_XXX.


Лучше перевести это как Украшение для волос. Потому что на голове есть 2 слота под украшения.


Links were left to do at the end, since there's the possibility of still changing the structure.

Disclaimer I'm not sure if guides are each going to have one, or if it will be left for the devguide intro that Ward did.


Remove DOCUMENT and SUBDOCUMENT types.


Since FVP_GIC_SOURCES is local to this file, would be better to use := instead of +=.


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


The rest of the document uses \emptyset.


Yes, I am working to see if we can support 2.7, 2.8 and 3.0 in one release. Otherwise we will support 2.7 in 1.0.* and move 2.8+ compat to 1.1.*


こんにちは, Tiryohさん,

Sadly I don't speak Japanese, but your changes look good, so I'm planning to integrate them in Rufus - thank you very much for reviewing the existing Japanese translation!

One thing I would like to also ask is: is there a chance you could also translate the new messages I added/changed in the most recent versions of Rufus?

These changes are described [HERE](https://github.com/pbatard/rufus/blob/master/res/localization/ChangeLog.txt#L18-L52).

The Japanese translator for Rufus has to tend to other things, so I haven't been able to get the Japanese translation updated for a few months now. This is becoming an issue, as most Linux ISOs will produce the following __untranslated__ dialog, which may be confusing for native speakers:

![untitled](https://cloud.githubusercontent.com/assets/1206968/11530194/98ae5f30-98ea-11e5-8370-71c806077799.png)

Thus, I would really appreciate to have the Japanese translation for this new message and others if you think you can do it.

Note that, if you need more help on how to proceed, you can have a look at [the guide](https://github.com/pbatard/rufus/wiki/Localization#Editing_an_existing_translation).

Thanks,

/Pete


Logical OR needed.


Why two casts are needed here?


That would do the same thing as using a `Where` but be more code to write.


You may want to add a retry logic for this task


`done` again :)
but nevermind, it's not critical


"Simple MVC Framework" ;)


Nova Alt


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.868](http://teamcity.monogame.net/viewLog.html?buildId=11857&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:05:31


The original implement is <code>[str drawInRect: rect withFont:font lineBreakMode:NSLineBreakByWordWrapping alignment:nsAlign];</code> in iOS, and it's fine.
But the method is not available on tvOS, so I guess you changed it before.


Looks OK, but needs a fix for the cgls, too, i.e. the recently added e97a84757f76b9c4c401bbc6fa147e35204b2a1b cares for ENOEXEC, and needs to be fixed too.


I've used this in my framework, I've amended the name now.


In the *General Layout* section, we use **header** to denote the entire text before `--BODY--`.  The different items that make up the header are then called **header items** in the rest of the document.


With the changed method signature of arrayMinusObject, I get this:

> The method arrayMinusObject(Collection<T>, T) in the type ERXArrayUtilities is not applicable for the arguments (NSArray\<Person\>, NSKeyValueCoding.Null)

Generics tend to make my head hurt, so I'll just ask: Is there a way to make this work?


whoops. lemme fix


> The latter because surveying the entire repo for each core.Resolve is too expensive.

It should definitely happen on writes.

Also, we should have a way of _not_ running GC again in the case that:
- above gc watermark
- runs gc once
- gc does not drop storage below watermark
- another write happens (no pinset removes)
- running gc again will waste a ton of read time
- another write should not trigger gc

MaybeGC could track when new unpinned things could happen:
- set a dirty bit when things removed from pinset (things become cached)
- set a dirty bit when things new writes without adding pins (new things cached)
- maybeGC skips GC if dirty bit not set


@lgierth sorry to crowd your commit with this


nit: Better to use := instead of += for local vars.


Add *.iml to .gitignore;


goeie code man


cover sublime config files.


faka maaaaaan


In theory you can remove stuff below here...


I guess, it depends on the purpose of the test. With `iso_c_binding` in the tests, we can prove internal consistency: the Fortran-interface of the library works well on a given platform, provided you use it with precisions, which are compatible with `C_FLOAT` and `C_DOUBLE` on that platform.  It would not warranty though, that it is compatible with the precision the user uses in his Fortran programs. (But since the user could also decide to use quadruple precision, even explicit testing with `real` and `double precision` would not ensure, that the library is compatible with the users program.)


you're right!


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


Hi,
so the error we encountered while making caffe in an conda environment created form **-c ilastik everything-but-tracking** was:
![upgrade_solver_proto_text bin_2](https://cloud.githubusercontent.com/assets/15635315/11530677/f1c527d2-98f6-11e5-823f-da41316a08cd.png)
we could solve the issue by typing **conda remove xz**


Ups, you just found a bug :)
``` javascript
positiveMatch = positiveMatch && proxy.config.matchContext(req);
```


Done for next PR.


Ref: http://www.sebastien-han.fr/blog/2012/12/18/noac-performance-impact-on-web-applications/


Actually User might have their own config (txt,json,xml) that serialize from disk (even web). Slua just provide a way to configure options should be fine.


JoseAntonio MedinillaCruces


@ernestfegarido
that's because kRO updated their criatura map and quest
use this one and put into your custom grf :
http://herc.ws/board/topic/10579-2015-09-05-142-criatura-academy/?p=66294


@shrutih-cuelogic Be consistent in naming. Change "Floatmethod" to "floatMethod", proper casing is important.


cant wait for ppl to copy paste our code and see it doesnt work because i used ATs


toxml reduced to a single static method of ErrorLogger, now called from `Tokenizer::dump`, `Tokenizer::dump`, and `Preprocessor::dump`.


No, unfortunately, not even gcc 4.8 as shipped with Ubuntu 14.4 LTS. It works for gcc 4.9 and  5.2 (current stable). Unfortunately, gfortran hangs way behind the commercial alternatives in implementing Fortran 2003 OOP features.


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


Hi, commenting on commits isn't the place to report bugs nor to get help. Please use https://code.djangoproject.com/wiki/TicketClosingReasons/UseSupportChannels. Thanks!


Shouldn't that be `STRING+`, since empty alphabets are disallowed?


oh sorry


Ok, je viens de jeter un rapide coup d’œil au man de clock_gettime et du coup ce que je rechercherai c'est plutôt CLOCK_PROCESS_CPUTIME_ID et CLOCK_THREAD_CPUTIME_ID, l'idée étant de mesurer le "temps" d’exécution d'un bout de code un peu plus indépendamment de la charge du CPU.

C'était aussi pour ça que j'affichais les cycles CPU, pour être aussi indépendant du CLOCKS_PER_SEC.


hey wtf here ? :D


MArray_Elim (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1455 of 1455 tests passed)

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


SSL_dh_get_param_from_file() не документирован в манах, потому решил его не юзать


Should be named FVP_GICXX


I have noticed this kind of solution on StackOverflow too. But ended up just making README.md a symlink of README instead because the project's structure is not "foreign".


Hi! This patch is not needed anymore since this commit https://github.com/rncbc/qtractor/commit/9d09922551eab1aaa30d9f14caafe82b5a3050c2


@shrutih-cuelogic It's good to write the **for** loop as follows. This will avoid calculating of length in every loop.
`for (var i = 0, len = ele_list.length; i < len; i++)`


В модели не должно храниться два пароля, сохраняется только один и в хешированном виде.
Два пароля мы отлавливаем в форме.


I think we really do need numpy on appveyor, so that the wheels (binary packages) that I'll publish to PyPI support the primesieve.numpy module you've added


Видимо на самом деле хотел создать форму.


noticed an error in log file.
think it pertains to this file of the commit:
https://github.com/marcelveldt/skin.titan/blob/87406c9ae3ceab22a7c8cd921d3c134f518c1ac5/skin.titan/1080i/IncludesViews.xml

error:

	293	04:19:18 T:7328   ERROR: Missing operand
	294	04:19:18 T:7328   ERROR: Error parsing boolean expression [Control.IsVisible(51) + Skin.HasSetting(View51_UseSlimList)] |
	295
	296	                                                        [Control.IsVisible(504) + Skin.HasSetting(View504_UseSlimList)] |


ТАкой модели не должно быть.


@shrutih-cuelogic Be more descriptive with your naming conventions.


`gulp make`? :)


QT5 is currently the default (I'm using the QT5 version). Please, analyse it.


Good, I will merge.


I don't know why numpy should be so slow to install on Appveyor. Let me do some research.


start a discussion about commits


В модели должен быть один пароль, как говорил раньше, а второй пароль нужно явно задать полем в форме. Я недаром предлагал сначала сделать версию с одним полем для пароля, а потом ее двоработать.


Mam problem z modyfikacja karuzeli. Posłużyłam się gotowym szablonem z http://www.w3schools.com/bootstrap/bootstrap_carousel.asp. Niestety u mnie na blogu wyświtelne obrazki są niewielkich rozmiarów za to cześć z szarym tłem na której są strzałki jest bardzo duża i za nic nie mogę tego zmienić.


Why don't you move this guy to the `init` method as `this.server` so you can call `this.server.start()`?

About the promise implementation, I don't know if we have much more to do, the only thing that I would change is moving the `this.server.start()` to inside the promise and return it.

```js
this.server = new this.karma.Server({ configFile: this.karmaConfig });

var promise = new Promise(function() {
  return this.server.start();
}.bind(this));
```


Awesome


nie potafię zrobić za pomocą HTML warunku, żeby wpisana wiadomość w textarea miała min. 10 znaków


Хех.


@pallavjosh would you be willing to throw together a quick PR? Would be very helpful.


`contr`?! why not `contribution` like for the other attributes?


pullcontent "https://github.com/nhsevidence/ld-content-test"


Что-то меня эти квадратные скобки напрягают.
21 век, все-таки.
Вангую недовольных. Щито поделать.


If you use the object itself as the key, you can simply initialize this in `__init__`. If you prefer to keep it local to the field, better define it locally and pass it to the two methods below as an argument.


int(float()) seems superfluous. Wouldn't it be possible to just write `int(val)` here?


really, we have 'summary' with a special key?!


@Zuchens A co to?


Zep, where's the authorship??  You're better than this?  I don't have the original commit offhand, but I know that this wasn't yours. https://github.com/PureNexus/android_device_huawei_angler/commit/788a20bf3f36dc305412ca3cdaa217bc72d2561c


По моему кнопка совсем не в тему. Особенно этот градиент. Да и маленькая она по-моему.


I wanted to avoid passing it as an argument, it's seems quite ugly (and long). I want this dict to be cleared in every loop...


same code all the time besides the `field_type` - why not keep it outside the ifs?


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


Oh, so I migrate it normally?


I don't think that name is good. Maybe it'd actually be better to use `max_length` and `max_words` and only set whatever it used in the old field


I will miss RXSwizzling :) It had such a nice ring to it. :)  Keep up the great work, Mr. Zaher!


Please use `_.forEach` instead of `for in` to be consistent with the rest of the code base.


Right


It's a bit messy to have function-scoped state living on the class scope..


All doubts about this can be cast aside when replacing the `styleEngineLoaded` signal with a unified mechanism: Just invalidate the `nullStyleSetProps` like done in https://github.com/mli-ableton/aqt-stylesheets/commit/4a5bc40285f972e18e0dad32449f4689360001b1. We'd like to do this on a later cleanup branch though.


why not handle the single-line text field here, too?


`old_data` might be less confusing


I **do** need to use the ID though.. so we either stay as I have it, or pass the dict as an argument in the 2 methods. Preferences?


I like this approach


On Fri, Nov 27, 2015 at 02:56:57PM -0800, psyhomb wrote:
> Actually it is not very reliable method

So, your testing has shown it not to be reliable? Odd, isn't
it?

> and above all it's not compatible,
> not usable in all environments, so after all i really don't have better
> solution at the moment. But like i mentioned before sleep is quite reliable
> and functional, we're using it in our prod environment w/o any issues,
> everything is working like a charm. Any thoughts, suggestions? Thanks

Do we know what can actually go wrong? The "virsh
migrate-setmaxdowntime" is going to fail, right? How bad
is that, i.e. what would be the consequences?

There are a number of patches/merges in this pull request. Can
you please rebase to a single patch and then "git push -f".


remove


Вчера были внесены исправления, попробуйте с последними исходниками. Должно работать.


Yup.


This is what I mean :
`( is_enabled($Config->{'proxy'}{'verify_ssl'}) ) ? $TRUE : $FALSE;`


Why did you remove "creates=..."?


@bump :+1:


yes


Nope, response size is nothing to do with slowloris. (Also, please watch out for request/response terminology, as your code doesn't do anything with the request size, only the response size.)

I can start an E2 HTTP request, make my client never return any of the net messages (or return them with a big delay between them), and then eventually the server will start handing out UIDs that are currently in use. Now, given that your code never checks that it doesn't hand out UIDs twice, I'm not sure this is actually such a problem - if a request is around for too long it'll just be dropped without notifying either the client that's sending it or the E2 that requested it. However, it will mean that some HTTP requests will never have either their success or failure callbacks called, which might cause problems elsewhere.


I don't get it. Why do you duplicating the application?


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


That's something we should change everywhere, probably to `2016` right away.
@ktoso ?


`Unrecognized`, and I'd use `print_error` for this or even raise an exception. It's something we don't expect to happen and that needs to be checked further if it does happen


I don't like how we've "mostly" removed the runtime configurability of the GIC init data except for this bit, and we've introduced a new way of mananging it that is separate from `arm_config_t`. I think `arm_config_t` needs a `gicv2_driver_data_t` field to cope with this case (which is initialized here), rather than allowing `plat_arm_gic_data` to be modifiable.


No need to make www-data the owner of the directory, I think just giving it to group www-data and making it group-writable is a better solution.


Test


Closes #4


cambios


Les lignes vides que tu as ajoutées.


creates was wrong there because the certificate might have changed, but it wont install it since the file already exists. (AFAIK `creates` checks for existence, not the actual content).

Thats why I used the `when: private_key.changed or certificate.changed or ca_certificate.changed`. If one of these changed, then we need to create the combined file again.


Oops, I must have read wrongly: I understand now.


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


Please use `const`


hi


I think this is missing a suitable default value if munin_node_specific_plugins has not been defined.


That is a bit of a problem. Even the TestNG documentation contains only vague documentation of these options and no examples at all.

For groupByInstances I could reuse the scenario from the integration test.

For preserveOrder it is even more complicated to give a meaningful example, since specifying an order of the test classes can only be done by using suiteXmlBuilder(). But in this case I already specify the the complete XML configuration of the test suite and can easily add the preserve-order attribute directly instead of setting the preserveOrder option. So why supporting the preserveOrder option at all? That's because of a implementation changes between TestNG 6.1.1 and 6.2 that changed the execution order of test. While before that change a test having a @BeforeClass, a @Test and a @AfterClass method was completely executed by the test thread before the next test was started, that changed that behavior in a way that the @BeforeClass methods of different tests could be executed before the @Test and @AfterClass methods of that tests are run. If the @BeforeClass methods allocate a unique resource and the @AfterClass methods free that resource again, this change will get you into big trouble. This behavior of TestNG 6.2 and following versions can only be prevented by setting the preserveOrder or groupByInstances option.

That's the background for these changes but I do not think it is a good example for illustrating the option.


Yandex and bingbot is good. Please remove it.


@GastonTS Ahí está. Juan conocía el tipo (pasaje o encomiendas)desde la app, así que vivan los typetest(?.
No sé si será lo mismo pero así la db se encarga y no es la app la que tiene que estar viendo si el tipo es uno o el otro. Prefiero delegar el if a la base en vez de hacer 2 sp. ¿Qué opinás?
Ignora el 'ambas' eso esta mal porque no hay correlación entre ambos cuando tienen el mismo número, lo sacó en el próx commit


thanks !


`const`


Don't like there being a modifiable `plat_arm_gic_data` being referenced that may be of variable type. This should be const and named something like `arm_gicv2_data`. If it needs to be referenced elsewhere then the extern declaration should be in `plat_arm.h`.


$SU isn't referenced anywhere.


Hmmmm, I think I misused the /**< thing. I'll recheck that. thanks


ok thanks for clarifying


still wander that ES6 is real )


That's good but also need a test that it's actually grabbing sth from an environment variable


Thanks :-)


yes


@toast38coza when you see this if you see this :)
Is this the correct way to test if the user can edit an entry? What i want to test is:
1. The user can edit an entry

Then:
2. The user cannot edit an entry once its status is not open.


I think this is a critical error and should throw an exception instead of using `console.error`.

I'm sorry that I didn't notice this point.


Ow, true


site loves fancy quotes ‘’“” but assumes UTF-8. Whatever you pushed got �-replaced by git first :)


It's probably good to also export a utility function called arm_gicv2_init() which does the same thing, in case any specific platforms need to re-use this.


:laughing:


Not correct to use OCF_RESKEY_* here, because the value can change. You really need to put the defaults in other variables, then reference those both here and up in lines 19-26.


Right, please make sure the files here are `2009-2015`.
Once new year hits, we'll mass `sed -i` to 2016.


Are the extended (i.e. longer than the default of 20s) timeouts actually needed?


Should be const. If we need runtime configurability, add it to arm_config_t.


These timeouts should still be 20s.


As for GICv2 driver, may need utility function that specific platforms can use.


`^2.3.5|^3.0`.

BTW, the `^` is preferred instead of `~` now (more semver compliant).


Может просто вернуть как было только без квадратных скобок?


Not necessarily ERR_CONFIGURED, it could also be something else, and only the validate procedure would know. The best way is to save the return code, then return that.


https://github.com/akka/akka/issues/19068 :wink:


It would be good to ocf_log info "already running".


I really should learn to review my commits before pushing them


Cause then I would need to check
`if field_type in ('AbstractTextAreaField', 'AbstractInputField', 'AbstractSelectionField')` before creating the new `ContributionField` object.


T_s is a vector of state variables. Why make it a parameter?


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


Should not be removed


I agree we don't need more details on TestNG itself. It is already covered in the documentation that you provided links to. I meant an example of using Gradle API to enable them. That would be just a short snippet of build code like  in other sections of the documentation.


This is something nagios specific, but looks a bit strange to me. Is the "command" file generated elsewhere and at what times?


Revert that change


Hmm I don't think it makes a huge difference, because in the end the user will only recognize the error by looking at the his console and at the moment that's exactly what the Jupyter extension loading core is doing too.

However I guess you are right that in the future they might have some more sophisticated behavior and then it's nice to send them a real error :)


Ah, nvm, found another solution.


adding subset label can be done in one or two lines
something like:
colnames(df) <- paste(colnames(df), labels[1])


@teohhanhui the same for me here...


Why initial algorithm and not initial equations?


Not if you omit the `event_new=self.event` here. Instead, use `self.event.contribution_fields.append(field) after `if not new_field: continue` (or anywhere else where you only have a valid field)


Pls take this back.


keep old "-o" handling


This patch is off-topic to the rest of the commits. Please submit as a separate PR.


You forgot to remove this case.


I realize this is just a comment, however one of the goals of this project is *not* giving your data to google. There should be a warning here as well.


Because it works this way w/o warnings.


:-1:

Care to give more details in the commit message?


If you are using SonataCoreBundle (2.3.10) you don't need to do that. Form alias can be used with a small set of configuration: https://sonata-project.org/bundles/core/master/doc/reference/form_types.html#symfony3-supports


This and the 2 earlier commits to the TSP/TSPD are off-topic. Please submit the 3 of them as a separate PR.


Although this is already fine, what do you think of using a `finally` block to perform the `self._shutdown_socket` call and other cleanup tasks?


Pls take this back and add additional check when this condition makes sense `&& rerunCount >= failureCount`.


well, then is a MyUPB plugin for connectivity to OC DB pretty well :) .. imho ;)
best regards
Blacky


Maybe we should rollback the vector append operation.


xx


info severity level is better, either here, or after the monitor succeeds below. In both places would be fine too, as it can sometimes help to figure out what happened in case of a failure.


Same as above.


A mon avis c'est plus CLOCK_THREAD_CPUTIME_ID qui t’intéresse, du coup le mieux serait de construire une classe similaire aux classe std::chrono::*_clock pour pouvoir l'utiliser dans ma classe Chrono, en s'assurant que c'est bien disponible sur nos versions (de noyau, compilateur, libc...?). Avec CLOCK_PROCESS_CPUTIME_ID, tu vas mesurer pour tous les threads donc même souci avec ROS, que clock().

C'est le problème de clock, la doc indique qu'elle renvoie un nombre de ticks, ça n'est pas directement lié au nombre de cycles CPU, c'est une valeur définie par le noyau par rapport à une seconde d'où la constante  CLOCKS_PER_SEC.


Where's the commit that fixed it? Btw thanks


Ща попробую еще вариант. Удрать ссылку вообще и сделать bold текст.


Still doesn't help in the situation where my E2 makes an HTTP request and then my client doesn't send *any* messages, because then `handleIncomingRequest` will never be called.


Should be OCF_ERR_INSTALLED.


example of use:
$xmlGenerator = new XmlGenerator();
            $xmlGenerator->setPiNode(
                array(
                    'xml-stylesheet' => 'type="text/xsl" href="http://w3c/styl.xsl"'
                )
            );


Ditto.


revisão ok. PR aceito.


Let's skip this one IMO. These are not part of the application, this will just add another copy of all the (large) PNG images to the repository without changing anything substantial.


Coding style: Lua allows trailing commas in tables, and they're recommended when having one entry per line (so the diff doesn't show a line being changed just to add a comma). This'll be in the coding style guide.


Usually the cmake style is:
~~~cpp
target_include_directories(introspection-subsystem
                           PUBLIC "include" "${CMAKE_CURRENT_BINARY_DIR}")
~~~


It's not actually a ComponentType but a Component (instance). It could also be mapped to a ParameterBlock. Please remove this unnecessary detail.


new line


No that wasn't intended. I'll fix :-)


LGTM


I don't understand: what is the downside in that? Once the UID counter goes from 0 to 254 and begins to cycle back to 0, if a slowpoke client is using UID 0, assuming that more than lib.sendTimeout seconds had passed, a new request would simply overwrite index 0


I always wanted to do this `&&` thing on functions, so for the sake of it!


Not sure this base is of any use.


> Hmm I don't think it makes a huge difference, because in the end the user will only recognize the error by looking at the his console and at the moment that's exactly what the Jupyter extension loading core is doing too.

> However I guess you are right that in the future they might have some more sophisticated behavior and then it's nice to send them a real error :)

Yes. Actually you are right but as you mentioned as well, I think we should follow *correct* behaviour of javascript :-) Thanks for understanding :)


That's better as it makes the hack easier to see, but that's still a huge hack :D
Why is this ```line``` at an offset of +2?


In openSUSE the package is called `python-Cython` - will need to be conditionalized.


Hello Dan,
I wonder if this commit fix some memory leaks during the fusion step ?
Thanks.


Because T_s[i+1] depends on T_s[i].


style: please don't break the line after `PUBLIC` but rather *before*. Also, you may simply specify `PUBLIC` once for both paths.


While I see it, I think we can/should hard-code the protocol (i.e. `https`) here instead using a wildcard. It's unlikely that we will ever allow plain HTTP on `issues.adblockplus.org`. Plus, the webRequest handler explicitly specifies `https://` as well.


missing newline at the end of the file (did you write this on Windows?).


Good change, but this wrapping if statement is no longer needed. Before, it served as a check to avoid doing an unnecessary lookup. Since we now need to do the project lookup every time this function is called, and since the two nested if statements contain all the appropriate checks for the code within them, the wrapping if statement can be deleted. Please delete that as well as the comment.


Because T_s only is state variable of the initial system only.


Done.


style: we align the asterisks together everywhere else.


As I said on ad722f6e610461ac48213b33173809b9ace32d96, given that your code will happily stomp over currently-in-use IDs, I'm not so sure there is a problem - except it means you can make HTTP on which neither the success or the failure callbacks will ever be called, which might introduce nasty behaviour elsewhere (as it's usually a reasonable assumption that one will be called, even on timeout).


ditto


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


Hi! Can you describe a bit at what memory leaks are you referring?


saw approval from andytill
at https://github.com/basho/riak_kv/commit/e6b4009c225ed74d37f57b509d65f83721ccacba


merging basho/riak_kv/bugfix/jrd/ddl-siblings = e6b4009c into borshop-integration-1212-bugfix/jrd/ddl-siblings


basho/riak_kv/bugfix/jrd/ddl-siblings = e6b4009c merged ok, testing candidate = 5f8cb21e


A nicer solution might just be to check when hanging out a new UID, that if it's already in use then call the failure callback on the HTTP request associated with it.


Also rollback vector append operation


Ditto for all checks


this is not in my pull request.

I added:
MJ12bot|MegaIndex|OpenLinkProfiler|spbot


Would it not be better, if this commit was reverted and `(set_color normal)` was removed from those lines? [Line 726](https://github.com/oh-my-fish/theme-budspencer/blob/0cbc06b57b21f1f722bf4c0f7719e0105f277bc4/fish_prompt.fish#L726)  ensures, that colors are reset correctly. This fixed #14 for me.


for data types such as proteomics and metabolomics this is expected to happen (as the amount of detected molecules is likely to vary between MS runs), but then again: probably these data types shouldn't be merged anyway if the data from the different nodes was not derived from the same Mass Spectrometry run.


Ditto.


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


I am not sure if the line is at an offset of +2 or if it's just an alignment for visual enhancement.


Hi! I happen to stumble across your Greek translation of Baka MPlayer. We would be honored to have a Greek translation to allow Greek speaking people to use Baka MPlayer. Do you have plans to create a pull request? If not, do you mind we use this translation?
Thanks


For a test code, you may very well use ALWAYS_ASSERT. We want test of release build also to find bugs.


I did originally hard code `https` but for some reason that broke the cookie change listener.


It would be wise to provide alternatives (best based on examples from current nixpkgs usages)


This line was removed in f4dfb01.


Why not just return none here so that empty `alternatives` don't need to be special cased above (and below)?


new line


Hi Tibor,

Sorry, but I don't understand exactly what you're asking for here. But
regardless, isn't rerunCount counting *runs* (eg run each test an extra 3
times if it is failing), whereas failureCount is counting *tests* (eg if 10
tests have failed all their runs - however many reruns that is - then skip
the rest of the tests)? Unless I've misunderstood failureCount? Is
failureCount counting runs or tests?

I don't think it even makes sense to compare rerunCount with failureCount,
because the units are not the same. Are they?

On 2 December 2015 at 23:03, Tibor Digana <notifications@github.com> wrote:

> Pls take this back and add additional check when this condition makes
> sense && rerunCount >= failureCount.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/seanf/maven-surefire/commit/f1518a1b4f8ce87cc6f3884550af24594cd98c59#commitcomment-14734966>
> .
>


:ok:


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


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 7640](https://teamcity.mesosphere.io/viewLog.html?buildId=84509&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 205 Build time: 00:04:48
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


That's one of the cases that make me wish for a few more comments in Godot's code base :)


I have a problem with the title of the PR and the reality of the code. One reads in the title ```900px``` then in the code we have ```1106px```. Up to which width shall the table page be responsive?


AND the kangbang continues!!! hahaha


remove the space after `0`.


Ладно. Щас @VisualKei сказет своё "буэ", и я откатываю.
Дело в том, что я все на сосаче смотрел. С фотоном очень классно смотрится. А на футабе может быть и не очень.


Might be worth checking but I think coding style requires:
```cpp
void setConfigurationParameter(const std::string &strDomain, const std::string &strConfiguration,
                               const std::string& path, std::string& value)
```


Strange. It works on my javascript. What is your browser ?
Le 30 nov. 2015 11:27 PM, "Francis Appels" <notifications@github.com> a
écrit :

> As said in
> https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math/round
>
> The Math.round() function returns the value of a number rounded to the
> nearest integer. But we need a float.
>
> Math.float() doesn't exist. You give 2 params to Math.round(x), javascript
> ingnores second param.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fappels/dolibarr/commit/4a0ea9777239108a32bf6aad2f4a1b513f3b612f#commitcomment-14695069>
> .
>


Sorry for spreding FUD, I'm not referring to any particular memory leak. But yesterday I have found my computer crashing on a "big" scene at the fusion step because it was running out of memory. So to be honest, my question should have been formulated like this : "Is this commit optimize memory management during the fusion step" ?


idem align `(` and `const`


For reference: https://github.com/mli-ableton/aqt-stylesheets/commit/334a21f1bf3c4063f45d4da64e85546f407384ef doesn't compile with Visual Studio 2013 :disappointed:. I'll drop it...


I don't understand what you mean. The equation section contains:
```
  for i in 1:N loop
    tau_sh/N * der(T_s[i]) = w_s*(T_s_b[i] - T_s_b[i+1]) + NTU/N*(T_source - T_s_b[i+1]);
  end for;
```
If you make T_s a parameter, this dynamics is removed from the model, which is obviously not intended.

The initial equations just set its initial value to the equilibrium point


I have reverted it and made a 1.0.4 release. Can you please test my dev branch on 2.7: https://github.com/gfreeau/GfreeauGetJWTBundle/tree/dev. I think this version will work on all symfony releases.


Of course, but I prefer to stay declarative and let the compiler figure it out


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


255 requests cannot occur at once. I can't see how it can. Checking whether a request is occupied by a UID would only add extra unnecessary overhead


no, but a next commit will, working on it


I counted and
```cpp
        mayFailCall(&PF::accessConfigurationValue, strDomain, strConfiguration, path, value, false);
```
is just 100 char long.


Maybe rollback the vector additions here too.


nice ! thanks a lot.


And here.


#KangAllTheThings


yes, it should be moved to constants, baby :+1:


Having this as a mixin is more flexible for now. See [below](https://github.com/joakin/loot-ui/commit/b3eb60f996ac4b8bc9fd8237ec8ac6b606c0667b#commitcomment-14716327).


Interesting that you ran into this, just yesterday we found a bug in HuBoard related to URI escaping the labels url.

I'll get you more information related to the issue, but I think there is a better way to tackle this issue than hiding a key in the params.

You're good for now but we will need to revisit this, don't let me hold you up, we can refactor this later


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


MobileFrontend has a full-width 1px solid line separating the chrome from the content. `.Chrome` isn't responsive whereas `.Chrome-content` is.


It depends on how specific it will be. If it turns out being just a regular commonjs build, then it could be useful to have it on gulp-metal. Otherwise having it here makes more sense.


Thanks for the error message.  I don't mean to reward your good deed with a complaint, but in the future, please copy+paste **text** instead of screenshots.  It makes searching on google much easier.


Would it perhaps be a good idea to change `?o`, `?p` and `<s>` into something more descriptive, too? Should I create an issue (or would you prefer a PR)?


Consider `std::string subsystemType = "Virtual";` to avoid considering `""` as `"Virtual"` in ConfigFiles.


None of my tests failed with this implementation. However you 're right, the definition of REPO_ROOT is a bit obscure. I pushed another commit which fixes the issue.


Done.


move is useless, `createConfig` returns by value thus it is an rvalue, thus move is useless.


Can't this be private?


Отлично!


%{_bindir}/cloudkitty
%{python2_sitelib}/cloudkittyclient
%{python2_sitelib}/python_cloudkittyclient-%{version}-py?.?.egg-info


I've removed unnecessary wrapping braces.


tabs ? Your alignment is odd. At least on github.


It should be replaced with %{__python2} setup.py install -O1 --skip-build --root %{buildroot}


Это, наверное, от части и моя ошибка была я недостаточно точно выразился. Entschuldigung!


neat approach, I like that the translation method lives in the same proxy class as the dsl method

ie; it took me a bit to understand why you were passing `self` in Proxy.new


Okay, but please leave the trailing '\' chars near the line endings, because we already have different alignment for them anyway.


I am going to finish my part in this week and once we finish all the requirements, we are done for this project!


Already in namespace `parameterFramework`


You could do this assignment in an else block after line 105 as well.


You can use `tessedit_char_whitelist` variable.


I see now. Sorry.


thx!
In Russian:
Авторизация тесно связана с синхронизацией времени, которую так же пришлось исправлять (баг). А синхронизацию времени надо вставлять в connect(). Коннект так же пришлось изменить...
В этом commit не вся работа проделана до конца. В последнем commit слишком много изменений.


This pull request has just changed the `<s>` into `<entity>`. I'll also expand the others (no need to make an issue).


To clarify: this file was created by the Anaconda installer (2.1.0), the file '.bashrc' is exactly the same, except that it adds the 'cl' alias that I created for me.


Thanks @fantastic91 I note the related changement in Crop API were introduced this modification.

https://www.drupal.org/node/2625244#comment-10621496


What about using ERXArrayUtilities.removeNullValues(NSArray)?


:-D


You have create an D.O issue to take credits in D.O commit ;)


Or you could use `CHECK(parameterFramework::introspectionSubsystem::getParameterValue() == false);`. That is completely equivalent. I prefer this form though because you can see what is expected and it is similar to how you would test it if is was an other type. Nevertheless if you prefer your form, keep it.


prpr


`== true` for the same reason as previously.


Approvata


@dcousens ok, thanks!


Since this is new code, you can ditch the hungarian style.


new line


Done in 8171f951254d7507c876d259be307ce2f5312fb9.


If the input rate is high; Because the FD does not block on any OS synchronization mechanisms the true portion of this ``if``` will run without giving up the cpu at all.

Because the FD is tied to a queue, you might consider adding :usleep and using 200 us on a successful read


commit message: add a context information in the headline. As it stands, the headline suggests that the Parameter Framework as a whole was unable to set a configuration value.


Ahh, of course! Danke Johann!


Could also just pass in the avatar manager here, but I guess this also works and is easier to use


btw, I'd say this is the kind of change very fitting to be `scalaTest`ed.
what do you think?


Hi,

an DB-Connector it's a really well thing, and the approach with it it's also really petty well to make it possible like in OC, to selecting the one of the available DB's.
Changing later, by add an convert-routine in the plug in then, in the specific database-connector-plug in,
or, creating an specific database-connector-convert-plug in.
(dcc-SQLite-MySQL/dcc-MySQL-SQLite/dcc-SQLite-TextDB/dcc-SQLite-OC)
(dcc-SQLite-OC for load up an MyUPB on ftp with users and create an User-db in the OC and create a OC-Outside-connectivity into the OC as suggestion (the plugin need only the login to the OC-DB))

And later can comes many other plug in's through others, because in this direction it's able
to create an TextDB also, but as plugin ;) and if want have Clark an TextDB can he create and
maintain the plugin and so :)


That banner is awful!


The reason why in my implementation, I wrapped both values (original and lower cased) in an object, is integrity and consistency. So that code that there is only one variable and if it is set user data are available.


Well, this regexp does pretty much the same as the other one, just that the delimiter is different and that we match it against a different string. So it makes sense if it's only for consistency to use the optimized version here as well. Plus, I'm sure once the othere regexp is gone this will become a bottleneck as well at least in cases where we have a lot of matches to calculate a score for.


Note that the third clause is executed **after** each iteration. So if I don't miss anything your logic will always yield a match in the first line.


`Math.max` doesn't do anything here as you only pass one argument.


It's still on my to-do list, along with a few other things for BoA (and a couple of other projects I've offered to help on for other folks). Sadly, with my terrible job, I've had very little energy to work on these projects...


255 requests can occur at once under malicious circumstances, because a request will never end if my client never sends back the relevant net messages at all.


private


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 10 to 10.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 43 to 43.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 60 to 60.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 69 to 69.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


What version of `liblzma.so` do you have on your system?


Line is longer than allowed. (81 > 79)

*Detected by LineLengthBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the file `coalib/results/ResultFilter.py`, lines 86 to 86.*

```
                                                                               v
    :param original_file_dict: Dict of lists of file contents before the changes
```


Line contains following spacing inconsistencies:
- Trailing whitespaces.

*Detected by SpaceConsistencyBear with severity 1 while running section `pythoncheck`.*

*The issue occurred in the lines 64 to 64.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/results/SourceRange.py
+++ b/coalib/results/SourceRange.py
@@ -61,5 +61,5 @@
     def overlaps(self, other):
         if self.file != other.file:
             return False
-
+
         return self.start <= other.end and self.end >= other.start
```


final


Requests automatically time out after five seconds


override