thought so, thanks. just wanted to make sure I do not break anything - build should be :green_heart: :tea:


Please user ! instead of not


Should be changed to
self.get_autoname(map.function, "f"))


prj_id is misleading since it's actually the project name


"e" missing in "requirements", also on line 908.


looks you put quite a bit of effort in your Python3 port but you might want to look at my soapbox fork ( https://github.com/FelixSchwarz/soapfish ) which I'm using with Python 3 for quite a while now.


it doesn't look like use of META_IMAGE is documented in Pelican, would this be a theme-related customization only?


Do we want this? By default the playback won't loop over the specified frame range anymore. I'm not against it, but it should be noted in the change log for example.


if you could push this into a separate pull request it'd be nice to merge it


"e" missing in "requirements", also on line 42.


``.update()`` je pense


``.update()`` ici aussi (peut etre)


Just a note to say that I've tested this particular change on my Debian 8 system where I'm running sentinel on a node without redis-server, and it fixed an issue where daemonize was incorrectly set to "no" which was causing systemd to constantly restart the service.


What the hell is this!!!


Start from python:2 or python:3?


Could you add a doxygen comment for  `UNLABELED_CONNECTION`?


separate pull request?


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


separate pull request? looks like this would benefit some documentation in the README as well so that people know how to use this?


@sttts , sure; i'll propose a new pull request here :).


c.Assert(len(rs.SortedColumns), Greater, 0)


ubuntu에 git이 기본 명령어로 내장되어 있다면, wget으로 zip파일을 받아오는 것보다는 git으로 받아오는 것이 깔끔해 보입니다.
wget으로 받아오면 zip파일이 남는데, 이를 별도로 압축해제하고 삭제하는 것 보다는 git으로 클론 하는 것이 간편하니까 말이죠. :D


That's just a cleanup of 5a18874875f22e37f3bc837, right? Up to you, but I would consider to squash both commits together


c.Assert(err, IsNil)


c.Assert(r, IsNil)


apologies, I should've asked the question with a little more detail about what I meant.
I was asking about the typecasting a zero to a pointer containing a dhcp_conn_t struct.
You could just assign *dhcpconn to NULL
Resulting in
struct dhcp_conn_t *dhcpconn = NULL;


@Jorch95 cambie para usar lo que pusiste en el comentario. Ahora se obtienen las imagenes del paquete `grafica.imagenes`. En caso de querer cargar una nueva, se puede hacer asi:
```java
File image = new File(imgPath + "HolaJorge.png");
img = ImageIO.read(image);
```
Fijate que `imgPath` guarda el path hacia el paquete.


My Android has 8 slots too lol - might be the gui size setting?


We usually set the media attributes inside the css files not here in the html


@ringods I see you've taken another approach already (one that came to my mind as well), so if you still want to discuss this give me a ping of IRC.


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


JavaScript code must be stored into an external file not inside the html file


I'd move the onClick logic to the js file that I've told you below


This broke nixpkgs tarball: http://hydra.nixos.org/build/28070722

```
checking pkgs/top-level/all-packages.nix on i686-linux
checking pkgs/top-level/all-packages.nix on x86_64-linux
checking pkgs/top-level/all-packages.nix on x86_64-darwin
error: while evaluating ‘override’ at /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/lib/customisation.nix:60:22, called from /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/pkgs/top-level/all-packages.nix:15147:21:
while evaluating ‘makeOverridable’ at /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/lib/customisation.nix:54:24, called from /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/lib/customisation.nix:60:31:
while evaluating anonymous function at /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/pkgs/misc/drivers/hplip/default.nix:1:1, called from /tmp/nix-build-nixpkgs-tarball-16.03pre72491.831aa51.drv-0/git-export/lib/customisation.nix:56:12:
evaluation aborted with the following error message: ‘HPLIP not supported on x86_64-darwin’
build time elapsed:  0m0.036s 0m0.027s 2m49.413s 0m6.659s
builder for ‘/nix/store/5qglq7ggbc460knzr5gvppz419ag1cq5-nixpkgs-tarball-16.03pre72491.831aa51.drv’ failed with exit code 1
```

That abort must be inside the derivation, not outside.


This is not right, you must not use a td to set : inside it


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


> Is the error, it's because it's passing in ModInfo objects rather than string objects, hence the change to make it x.name. x.name returns either a string or a GPath (I don't recall exactly which). Even so, GPath has a conversion to string object, so it handles it correctly whichever it is.

Thanks I know :D - a _Path_ ;)

> Invalid versions a file format thing. In the TES4 header, there's a version field, which really refers to the version of the file format. Bethesda sets this essentially. Anything not conforming to what was initially released (vanilla) or what the GECK outputs is indicative of a plugin that was not made with official tools, and hence the warning.

Excellent thanks - will merge this ASAP


can be `private`


ok


border is not supported by html5 please remove it and be careful with this. You must use css to style the table


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


You can group some of this css selectors since all of them are related to pictures


Why does registry avoid creating duplicate models? Surely a new model will be created and then used to update the entry in the registry?


Wieso castest du auf size_t?


2) В моей программе после создания переменной типа String, str будет в любом случае иметь значение, если не от пользователя, то значение по умолчанию "Unknown"
4)Мне удобнее когда конструктор и деструктор находятся рядом


Good job!

.attr('is-image', true) ==> plutôt .attr('data-is-image', true) non ? à voir les implications d'attributs personnalisés vs data.


Did you mean to `return render(cell)` here?


hi, now  nested sub-menu  cannot clickable on mobile device when  i tried clean theme


We probably will need these once I add the entity creation and update methods. Unless we move to the MB way of storing dates (which I don't mind us doing - I think it's a text field with a check?)


Looking good!


Absolut :) - Danke!


@martinyde: Can you tell me what the purpose of this is? /cc @cableman @kasperg
There is no real information or link to this PR in http://platform.dandigbib.org/issues/264.

(noted due to debug of http://platform.dandigbib.org/issues/752)


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


Please comment on https://tracker.moodle.org/browse/MDL-52249


> "Here we go again, another OS developer who thinks their code is too special for anyone else to work on..." ... Please don't take this personally.

@petermelias Taking a personal tone and then adding a throwaway "please don't take this personally." is rude, unproductive and unneccessary.

Let me reframe your comment for you:

"Hey folks. I need this package updated so I've decided to maintain a fork, at least for the time being. I plan to go through the outstanding PRs and issuing a PyPI release shortly. Other maintainers also welcome. {details, link}"

You're not entitled to ongoing unpaid support time from anyone else in OSS land. Nor are you entitled to decide how, when, and if someone should manage their own work.

I'd suggest trying to bring a less entitled and more positive attitude to OSS conversations in the future.


Pourquoi pas envoyer juste data:image... sans l'élément HTML au serveur ? Ca évite d'injecter de l'HTML d'autres personnes; suffit juste de $img.attr('src',bit.text)


ok sorry, i dont see it :)


**GenStackPush** : Da brauchst du kein assert auf die logLength und allocLength, weil die von deinem Programm selber gesetzt werden, du brauchst die asserts nur für user-Eingaben.

Aber sonst müsste deine lib zumindest so passen, also ich finde da kein fehler


почему все в h файле???


love it


я не поняла, зачем такой длинный main


mm shutil.


AllGroups -> AllGroupRepos


Ok


81,92. Остальное хорошо.


soooooooo, how *should* one camelize the data/props ?


Concordo e pensei em fazê-lo, mas se você checar a tradução do KR para EN "$You must make a doll by yourself. {nl}Check your Adventure Journal to see how you can make the doll." perceba que a frase em inglês também é redundante. Sendo assim preferi não tomar licença poética e alterá-la.


@aOJzQT XX-Net 并不是phuslu的作品 :)


Updated.


The error messages are also not translatable..


It would be good to have something where the order, filename, and links were all independent of each other. I seem to remember editing this section in the past and finding it a massive pain in the arse that changing one of them required manual changes of the others.


Brak dodanej  biblioteki itext, na razie  trzeba pobrać i dodać samemu  z http://sourceforge.net/projects/itext/files/5.5.7/ (najpierw wypakować i znaleźć w nim plik itextpdf-5.5.7.jar potem " Right-click onto the project and select “Properties”. Select “Java Build Path” on the left and click on the “Libraries” tab. Afterwards, click on “Add External JARs…” and select  the itextpdf-5.5.7.jar" )


Realmente, obrigado pela notificação.


as in, I generate a component like such:

`rails generate react:component ReaderTag id:number dist:number last_texted:string --es6`

everyone at this point is aware that `last_texted` is the key name for that property, not `lastTexted`. But here you magically make it camelizedCase.. and yet the data is still snake_case.


No problem.


issue: #410

I've done many a googles, cant seem to figure out an instance of this being a problem for anyone... somehow. Makes me feel like i'm missing something obvious.


Are you the bacon maintainer?


would something like `bindAll` be appropriate here?


Kudos for the great @mshibuya!!! :smile:


I feel bad for the nitpicking, but I suspect that inverting the branches with `if (!raise_ooM)` would give more readable code (and the compiler will optimize just fine).


@khalska - trochę nie za bardzo mi się to podoba, że do każdej metody trzeba dopisywać takie coś, żeby umożliwić korzystanie z niej stand_alone


> If you do not plan to use `caml_alloc_shr_no_raise` elsewhere than the one call in intern.c then it is useless to control this `raise_out_of_memory` because the size is tested before calling `caml_alloc_shr_no_raise`.

Yes, but the code of this particular function would still be easier to understand by a newcomer if `raise_oom` indeed governed all `caml_raise_out_of_memory` failures.


Just indenting this 2-space would be ok I reckon + it will probably look nicer because the `singleLineString` thing moves things out a lot.


Ok, will be renamed as "close_requests"


Hello Leonid, 2 things here:

(1) Your commit title says "lower opacity", but you actually increased it? By mistake? In my mind these colors only serve one single purpose: They mark certain regions. Once that is achieved, they should obstruct map viewing as little as possible, so I would say let us make them as transparent  as we can?

(2) I actually made a mistake in my original commit: I wanted 33% opacity (hex 55) for the status colors "uptodate", "updateable", and "downloading", and wanted to try only half hat (hex 2A) for "selected", because that is also visible on higher zoom levels much more often used for map browsing.  (I messed up and assigned 2A to "downloading" instead of "selected").

That was my original intent. So what did you wan to do? Should we maybe try my idea? Or make things even more transparent?

Thanks, Hardy


It is changed in other commit, the request part is not needed then


# To be done in the next update:
* Erase button should disconnect the device also
* PROGRESSS BARRRRRR!!!


Maybe these colors could be #defined?


sick commit message


@tampakrap, just search like this: https://github.com/saltstack/salt/search?utf8=%E2%9C%93&q=SUSE

then you can find something like:
https://github.com/saltstack/salt/blob/7c81188f675f88068b4b881c4b49b3b86159f0c3/salt/modules/service.py#L53

or

https://github.com/saltstack/salt/blob/b3f3275e0ae3db9a71114686d5ea266331a12513/salt/modules/localemod.py#L166

I can go on, with the salt source code, to look for modules that don't differentiate between SLES/SUSE/OPENSUSE....

@dmacvicar , I know you are right that SLES is a different distribution, so if you want to make the distinction, than all modules and states should be checked...not to mention all salt-formulas....

My solution would be to use 'Suse' as main distribution os and os_family grain for all SLES/SLED/Opensuse releases, and make the specific changes based on codename/release version...


What's the issue with Safari? We should probably just fix ffmpeg.to_wav to solve it.

Doing it this way is more brittle than specifying the flags explicitly because it will fall back to the settings used in the input file.


@dkayiwa, did you have a chance to test it out?


Excellent and SF.


Remove the lock and add it to gitignore.


This function is hidden by another function with the same signature on line 109. Hence this function is never called. Is this the result of a merge miss where both functions were kept?


Fair enough!


[RegXMM can also represent a double](https://github.com/PPC64/hhvm/blob/next/hphp/runtime/vm/jit/vasm-reg.h#L184). However I'm really confused whether to use single or double precision. I'm still checking that...


This isn't extra }), I checked with code folding and had @karthikbs351 look over as well.


Might be historic redundancy. Do tests pass without it?


I give the name of this commit Ben out of ten.


I have a work round to avoid this glitch
    Loader::model('package');
    $pkg = Package::getByHandle('the_package_handle');
Makes a database check if the package is installed first then only calls Loader::package if it is installed. So I can get rid of a cascade of warning messages at the expense of extra database calls.

For the future, I can think of 2 alternatives - add a quiet flag as an extra parameter to Loader::package, or a new method Loader::package_quietly


Thanks for this . However the inline.link  (line 270) is always blank.
{text:"google",link:"http://www.google.com"} didn't work.

Regards
Raja


This should be STABLE instead of VOLATILE. It will not be a performance problem for this specific function, but since the output is always the same for a input value it can be optimized.

http://www.postgresql.org/docs/9.4/static/xfunc-volatility.html


:smile:


updated


@manojo Amazing :+1:
Is it possible to use `Tuple2CPS` (do you have that?) for the `source` of the `zip` Stream?


How u do all this so quick
Do I need to doout to it all no


@lauraluiz please review


`abdicationCommand` is executed for its side effects so that I would prefer calling it with `()`


doc: fix message typos in systemd

Signed-off-by: Tobias Suckow <tobias@suckow.biz>


"wit" -> "with"


Not sure about a quiet flag, hardly anyone would use it and 5.6 is only under maintenance. Changing the API is imho a bad practice at this point.

I always check by using `Package::getByHandle`. What did you mean by "try and load it"? What other approach is there to check if a package is installed?


I don't know. I'll send a PR so that Travis will run the tests.


Agree


Thank You! I told you JS wan't my sepcialty - or I just didn't know where to look.


> そういうことでしょう。しかしその結果、初心者向けに便利な値の入力がボス戦に・・・。

入力や出力みたいな基本のこともボス戦になければ正確に理解されることができないのは、Cがプログラミング言語としてクソな１つです。

ちなみにこういうのは入力や出力だけではなく他もある。ポインタの真面目ユーティリティとか、配列の型と使い方とか。


Désolé Enzo, je pensais bien faire :x


This will throw a NullPointerException.


The functions returned by the model files are only ever called once per bookshelf object, in the init() function. Since we're using the registry, we don't need to include (and re-call) these functions in any other place. If duplicates were to occur, the registry plugin would throw an error that a model was already registered under that name, but this doesn't occur because of the structure of the code.


Also agree


Without this auto insert bullets feature will not work.
Disabling the feature can be a option.


* Описание коммита не соответствует содержимому. Нужно было сделать 2 отдельных коммита
* Чисто для выполнения задания, конечно, можно оставить стили внутри html файла, но вообще за такое отрывают руки. Хорошо хотя бы не inline-стили писал.


@sonora It is related to different understanding of "opacity" term. I meant that IMHO more visible fill is better but now I think that you are right.


it is actually complete unrelated to the rest of this pull request, the warning lines exists in current git master


First option , i mean


Seems pretty straightforward and readable to me.


I figured we may need them at some point, but for now they're untested and unused, and the parse function actually returns incorrect results. (Date zero-indexes months, even though year and day are one-indexed.) I figured that new implementations could go in if and when we needed them.


http://rghost.ru/6hscWh9K2 небольшая прикидочка по анимациям загрузочного экрана. Нужно будет попросить автора нарисовать еще по одному-два варианта для областей, чтобы вставить нормальные изображения.


... by making sure the datamanger (that centralizes user access rights checks) is used for all operations.


Hm... You're right... Ill #define the color later


Yes, I was thinking that users would define their own base components to save on boilerplate code. But that decision should be made independent of `htmltemplate-vdom` runtime.


`Content-Type: application/partial-update-json`, right?


Menu Swipe wurde eigentlich schon mit dem commit davor umgestellt.
Hier wird nur angepasst, dass section swipe nicht mehr abhängig vom angeklickten element ist.


@josevalim double checking here this is what we want


Test


just wanted to upload somewhere, it'll be it's own library I imagine


Final test Ref#67654751152349


minor: why these vals declared so far from the usage?


ok


Package::getByHandle makes a database query to find an installed package, then calls Loader::package to load the package if it is found. So just calling Loader::package is theoretically more efficient, especially if a package is installed. On the other hand, Package::getByHandle is more efficient if a package is not installed.


你拿掉 d3 這樣第14跟15行會出錯的啊啊啊啊啊啊


delete[] str; Игорь акцентировал внимание на этом пробеле, но не принципиально вроде, а так все пучком)


Это лишнее. **`<img>`** - это пример inline-block элемента, то есть элемент можно отцентрировать, использовав свойство `text-align: center` на родителе.


yes


Как уже сказал, а как же цвет текста?


yap, #492


Ну лишнее же свойство, в макете его как бы нет.


I'm start read code and found some mistake.

Before this commit it was:

    > gpx = data.get("gpx",None)
    > print not gpx
    True

And now:

    > gpx = data.get("gpx","").split(",")
    > print gpx
    ['']
    > print len(gpx)
    1
    > print (len(gpx) == 0)
    False

This used in "if statements" at lines 93, 159, 260. I think it mistake, but twms works, so may be you make  some comments about this?


Mere dokumentation


Działało, stąd moje było pytanie czy to zamierzone :)


Let's move this discussion to https://github.com/facebook/presto/issues/3938


er det meningen?


Done


thx for your suggestion .
Now that the code is in your hands , feel free to modify it at your convenience or make it better .


ok i will remove it thanks


dangling ;


println


Свойство `height` тут вообще лишнее. Оставь только вертикальный (горизонтальный не нужен) `padding` и этого будет достаточно, чтобы выровнять текст по вертикали в данном случае.
В целом, `height` тут не просто лишний, но еще и вредный и по-факту, текст не выровнен точно по середине.


Lorsqu'un utilisateur s'inscrit, il est inactif, et doit donc attendre qu'un responsable l'active pour pouvoir utiliser son compte.
+ Modification du design du formulaire d'enregistrement
En cours : envoi de mail au responsable lors d'une nouvelle inscription et envoi de mail à un utilisateur lorsqu'une action est faite sur son compte par un responsable (activation notamment)


Thanks, committed a version as you suggest (and also learned how to reply to a line comment).


done


Au passage, vu que la v1 a été tagguée, c'est pas mal de poser le tag sur le commit mettant a jour la version dans pom.xml et cadastrapp/pom.xml... et d'enlever le suffixe -SNAPSHOT :p


dashboarddb ,  i will fix it in the next correction, thanks for pointing out


Il doit y avoir un poblème dans cette fonction. Elle ne fait rien.


oh ok thanks alot


Agreed. I committed such a version.


I think you might want to `flush` here to guarantee that the write happens when you want it to. Alternatively you can open the file without buffering (`file = open(testfile, 'w', 0)`) so that you don't have to call `flush`.


extract some methods for artistPages etc


thanks. now it's worth fixing also the formatting issue on line 267


comments


Fichier.nomdelafonction: ...


Glande Giangi


ya my bad i forgot i had already added frandom


@andrewkroh Thanks, will try to see if it only breaks because of that.


Actually that is not true. If the event has unlimited seats but the ticket does, your code allows to order more seats than available on the ticket.


comment


@andrewshadura hey, looks like terrible performance bug! :)


Won't work well with the root world - it contains all other worlds inside it.


useless blank line


It doesn't look very useful, trying to add a number to undefined. It's probably not doing anything now. Let's keep this one on dev for a week or so and if no strange behaviour arises move it to stable.


確認しました！
現状のparamsはこのようになっています。
https://gyazo.com/bcd4a2971f99ed454bec76fd77e6d5af

listが複数ある場合はわかりませんが、同じtag_listで扱うタグ群なのでこれでできるみたいです。
https://gyazo.com/0f86cf506ec09ccb817fc98410a22487
参考URL:http://morizyun.github.io/blog/acts-as-taggable-on-gem-rails/


ikke unikke pakkenavne?


Please write the behavior of this function when outsize is `None` .


isSavedWithRelation


Done


I don't really like this implementation.  We shouldn't be treating `intptr_t` as a separate type for printf, we should be treating it as a width modifier along the lines of `h` and `l`.


I agree. We have to find the right balance between annotation bloat and DSL capabilities.


Totally cool git hash guys, where are the donuts?


The only explanation I can come up with is[ that a comment in the issue](http://platform.dandigbib.org/issues/264#note-12) states that unused fields and tabs have been removed.

I do not know what process lead to determining that these fields were not used.


It's possible, I was always "updating" via the github desktop and when it broke I had to manually download the previous build and overwrite the files, now I had to remove the whole repository and add it again because github desktop was giving me errors... so I possibly broke it myself sorry lol I will try to use the duckie executable to run it :)


Классов с такими именами быть не должно никогда! Даже в процессе **"просто понять, как делается"**. Эти блоки можно назвать `.floating`, например. Это хотя бы будет лучше отражать суть блоков.
Ну и еще лучше, если одинаковые стили (цвет, размер, граница и тп.) будут объеденины под одним классом (как раз упомянутый `.floating`), а также будут добавлены классы-модификаторы, которые эти самые блоки будут двигать куда нужно (например, в данном случае их можно назвать `.left`, `.center` и `.right`).


After this upload I tried to re commit my files for a further 3 hours and did not succeed. The local files worked at this point but this commit doesn't work. I aranged to see Chris McCready the following day (23/11/2015). We've fixed the local files and have moved them into a completely fresh repository, ElfPunchFinal. The first commit was the gitignore file, the second commit was the working local files I had at this point. If you want to see where I got up to on this date then you will have to go there for the files. This repository is now dead, there will be no more commits to here.


Any plans to add support into nanoVG?


aaaaaaaaaaaa


How about assigning `_pair(outsize)` when `outsize` is not `None` in order for the argument `outsize` to accept not only tuple but also int?


Why did you do this?


Please follow read me instruction


s/of this backend//


Читай комментарий для класса `.blue1` - тут все то же самое. Так как судя по заданию смысловая нагрузка этого блока в том, что он квадрат, класс можно назвать `.square-wrapper`, например.


Why do we need to truncate with zero size here?


@Komzpa thanks for quick answer. Ok, I fix it in my fork and a little bit later make pull request


dokumenter at der skal være en package.json i roden af nugetpakken


Just curious, why not shutil.copy2 rather than atomic_move?


na akkor meg is csak en basztam el, bocsi


I've been thinking about it. Splitting the stoke is slower than I'd like,
and I would like to find a way to do it on GPU. But that has its' own
interesting corner cases. I would like to implement [1], but the nanovg
shader is really heavy already.

[1] Shader-Based Antialiased Dashed Stroked Polylines -
https://hal.inria.fr/hal-00907326/file/paper.pdf

On Mon, Nov 23, 2015 at 3:02 PM, Murat Sari <notifications@github.com>
wrote:

> Any plans to add support into nanoVG?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/memononen/nanosvg/commit/dc75508682c0829a5dbd36912c8dc5721552f15c#commitcomment-14559460>
> .
>


s/Employeer/Employer/g


Firstly I considered about it, but I thought passing only height of outsize is rare...
What do you think about this?


lgtm


itt basztam el


Объедини одинаковые стили для классов `.one`, `.two`, `.three` и `.four` в один класс (внутренний квадрат - `.inner-square` или просто `.inner`), а на классы-модификаторы (как раз `.one`, `.two`, `.three` и `.four`) повесь стили, которые эти блоки будут двигать на те места, где они должны быть (фактически, все различающиеся свойства)


I know it's just a name, but don't say `gmysqlloader` here...


Please write the behavior of this function when outsize is None .


superflous


ditto (`_pair(outsize` ))


Para essas localidades favor se referir às convenções da Wiki: https://github.com/Treeofsavior/PortugueseTranslation/wiki/Tradu%C3%A7%C3%A3o-de-Lugares


Смысловая нагрузка - прямоугольник с прямоугольником внутри. `.rectangle-wrapper`


Mesmo caso aqui: https://github.com/Treeofsavior/PortugueseTranslation/wiki/Tradu%C3%A7%C3%A3o-de-Lugares


I thought the idea of nan boxing is that doubles can be used as is, why this subtraction is needed, it kills lot of speed ;-).


Смысловая нагрузка - внутренний прямоугольник. Имя класса сам придумай (пример написал для классов внутренних квадратов)


cellNeighborsAmount too?


indentation


say `Could not bind parameter.` instead?


Oh yyyyeaaaahhh!


Ah, ok, I guess doubles are munged to not munged pointers. Well, good tradeoff ;-).


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


This is not true. Please put your own name here.


\n


missing cr on 273 too :)


If it is duplicated delete it ;)


Some weird indentation here


This is incorrect. For the time being, we should probably not use Wilson coefficients to model NP effects in b->c l nu.


Is this check really necessary, isn't the `ree_fs_unlink` below sufficient? The only difference is that we will not see the error message in `tee_fs_common_write`.


See above.


It does make sense to check if the store is closed or not even in production.
Problem that this PR is fixing would be obvious right away if this check was there since the beginning.
`NeoStore` was used after it was closed and returned meaningful values for `getLastCommittedTransactionId`.


You can now get the latest nightly binaries here: http://flink.apache.org/contribute-code.html#snapshots-nightly-builds


make this an } else { ... and decide on one if brace placement ;)


Providing newDefaultTxn function may be better..


that's bad, there needs to be a return


Não deveria ser "Zumbi da Vela"?


should this line indeed be unreachable?


BTW, what is the purpose of `ChanClient`? Is it just for testing?


Isso deveria estar no singular.


J'ai fait un commit avec qques trucs par rapport à ton commit : https://github.com/EsupPortail/esup-smsu/commit/ba182364dadecb2713af87cfc450cae084712ffe

Ca t'irait ? Si ok je (ou tu) squash les 2 commits et je push


Tem espaços a mais antes do texto.


Updated.


Update


Better to cache `DependencyResolver` in field and still accept the `GraphDatabaseService`.
Such approach will allow us to isolate casts to `GraphDatabaseAPI` inside this class.


You sure you want to blindly change an API definition without thinking about the consequences ?

Hint, addons/library.xbmc.addon/libXBMC_addon.h is what binary addons use to call this.


"Minas Antipessoal" é o correto.


納得です。

なおC++17に統一関数呼び出し機構が提案されており、第一引数の変数名がthisの場合に限りドット演算子で(以下略)


The meta `keywords` is used also in the html provider so the opengraph provider is getting the same keywords, so there's a duplication. Am I wrong?


ff


kill those lines? :)


Changed here: https://github.com/ruflin/filebeat/commit/cd628bfec328c7a4c9cbae30b711155042ad74f2


dd


default args to SODBC constructor aren't used anywhere(?), remove them?


Well, you're the boss, you order and I do. :) jeje


Não acho que deva traduzir o original, já que Pavise é um nome próprio. Vide https://en.wikipedia.org/wiki/Pavise


Parfait pour moi, je te laisse faire :)
Merci !


我也是用的 IKVM 7.2.4360.5。但不同的是，不是直接用的IKVM，而是首先通过ikvmc将.jar转化为.dll。
执行命令为：
ikvmc -target:library hanlp.jar
然后就是编写C#程序，引用这个 hanlp.dll和IKVM.OpenJDK.Core.dll.
这个过程没有指定-cp参数的时机：（


d


Cの三大ボスは

- (多次元)配列
- 文字列操作
- 標準入出力

じゃないかと思う今日このごろ。


Hey Zephik, i think you resolve flashlight problem.

By the way, also i foun this to flashlight related and camera...

https://github.com/AOSPA/android_frameworks_av/commit/66296c4e99da5f2dd0a3dbf048be0d6ffc49a559

Thanks


Apparently the issue is in how defined `broken_output` but to me it looks identical to `file_input` a bit above...


@lauraluiz please review


why <=3.1 ? `wxSTC_INDIC_TEXTFORE` exists in wx3.1 so the old condition should remain
Maybe you have an outdated version of wxWidgets?


I think it should just be removed altogether.

If I hit enter after typing atomic scan, I have a good idea what is going on.


If performance matters I run `Package::getByHandle` during the installation process only and put the result somewhere. That way I've got a solid solution, by luck as it seems, and one that's fast too. I'm willing to merge a PR if you think you can improve something for your case, but please don't change the API.


Yes I already saw that paper earlier (last month) after I read you adding the dashed stroke to nanoSVG I thought I wait maybe you porting it over to nanoVG  :)
So If you do not plan to port it over in the near future I could to tackle soon :) please tell me what you think (Want to avoid too double the effort).
I am using nanoVG with custom SVG parser which works very well (First I tried nanoSVG but the requirement did not allow to use it).


TeamCity ReOpenLDAP :: devel [Build 204](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1211&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **FAILURE**
Summary: Tests passed Build time: 01:43:42
### Failed tests
```
all LDAP tests for bdb/DQD-KFA: test050-syncrepl-multimaster for bdb/DQD-KFA: <no details avaliable>

```


So, I downloaded the duckie tv lates mac release, when to applications and show package contents found the nwjs executable nwjs.app and moved it to the duckie tv repository folder, ran the app and now I see the nwjs icon on my tray but no duckie on the systray and no window open when you click on it.
I am attaching a screenshot that shows the app is running as it added a bunch of torrents to my utorrent queue (the onces that show active have just been added)

Maybe I am still doing something wrong?

![screen shot 2015-11-23 at 10 20 22](https://cloud.githubusercontent.com/assets/13501166/11337515/5bd70bea-91cc-11e5-8e36-b125abca22bb.png)


What if `facing == CVec(2, 1)` or larger?  You may want to change this to `Math.Abs(facing.X) + Math.Abs(facing.Y) == 1`.


@Jorch95 @santimarrone Seguramente hay una mejor manera de hacerlo, hoy lo voy a preguntar


Personally I think only joining the string is too trivial to abstract. The types, including the location, parameter are fairly common for the Google Places API, most calls support it. And I'm not aware of any other parameter that uses a pipe separator to join the arguments?

The unit-test setup isn't that complex also:

```go
func TestAddToQuery(t *testing.T) {
	query := make(url.Values)
	types := []FeatureType{"aap", "noot", "mies"}

	addTypesToQuery(query, types)

	if test := query.Get("types"); test == "" {
		t.Error("Expecting the query to have an element called 'types'.")
	}
}
```


I didn't like the idea of duplicating blob memory. Can't you use duk_push_external_buffer in this case?


Thanks for the patches.

Please fix this PEP8 error:
```
weather_api.py:30:9: E128 continuation line under-indented for visual indent
```


3) 의 unzip 명령어도 삭제해야됩니다 ㅎㅎ


@urso Nginx returns a huge JSON that holds its status and I just don't want to maintain a copy of that schema. So, I just pass the returned JSON into `Publish()`.


Oh, right! Missed that one! Will add once I get wifi. Thank you


Comment here.
I'm not in your organization.
I might have the same Gravatar, but I'm not I'm a different GitHub account


Should read as "meaningless" %)


I think it should be inlined or extracted as extension


toppie


I need to check DirectSound Sink and WASAPI again ... but iirc they don't care about the layout at all when doing passthrough - as also pulse does not - setting 2 channels by itself only.


Czy w tym miejscu wydarzy się efektywnie cokolwiek innego, niż w else? Mam na myśli, czy porównanie hashy przez == jest wydajniejsze. Jeżeli nie, to nie bardzo widzę sens tego rozgraniczenia.


No, those are the [default layouts](https://github.com/diaspora/diaspora/blob/0e0bd2719a26bb6c687d553da2c74a3ee944ada8/app/controllers/application_controller.rb#L29), you shouldn't need to specify them at all.


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

```


@mrkschan ok, for the json it seems to make sense then.

ChanClient is kind of a left-over from refactoring and partially used in testing I think. There is no real reason to use it.

Btw. since the publisher.Client is an interface, you can create a wrapper implementing PublishEvent, PublishEvents collecting all events to be published into a buffer (no limit in size) and have a 'flush' to push all collected events to the underlying client. The publisher and output plugins internally take care of splitting the batches into configured sizes. This way you don't have to change the logic in 'Publish', but get the buffering for doing bulk-inserts.


it seems like a duplicate. but not all adapters use html provider. so may be this duplication should be allowed?


Did you ever try not using this `request.format == :mobile` hack?


I'm sorry I did not get back to you with some comments last night. I was
going back and forth between finishing the revised introduction and
reviewing another paper that also has a short due date. I will finish up
that and send you some general comments later today. Then at some point
this afternoon or tomorrow morning, I'll do a more detailed edit.

We have a short week ahead of us because of Thanksgiving but I'll do my
best to get as much done on this as possible the next few days.

Lois


Add white space after the colon


I made META_IMAGE to handle meta images. You can read about it [here](https://moz.com/blog/meta-data-templates-123)


Sorry for the Ruby leak, Thanks


Suite à ba7af6c71ceb0c89a081032a4ce3a9c649d8f06e, ``/account/{idAccount}/transfer`` sera plus approprié


Errors should be handled here.


Ok, there are some buggs. :/

Arrows don't damage entities.

Snow balls barely go up...


Please fix them, thanks. :)


There's a number of build failures because of the helper rename:
- project-stories/_full_image.erb:1
- project-stories/_text_tile.erb:25

:scream:


Actually using stringext is a good idea; I think it's already a transitive dependency.


Good point. Runtime tested it but didn't use binary addons.


private


Bless this commit.


I disagree. On the other hand you are right that we should have failed even in production. Perhaps we are missing to nullify fields when closing NeoStore?


 #[zstack Test Bot Report] P A S S
 Integration Test Result:
Test Summary:

 TestSuite|TestCase | Pass | Fail | Timeout | Skip
 ---------|---------| ---- | ---- | ------- | ----
 Total|||||


明白，请参考：http://www.hankcs.com/nlp/call-hanlp-in-csharp.html


@nblumhardt Thanks! I had to figure out what to google before I could get the proper results on how to handle this with github. Sorry, I'm very new with contributing to a project via GitHub.


Site no ?


some weird typo here also :)


C'est terrible!

On 23 November 2015 at 13:57, Darafei Praliaskouski <
notifications@github.com> wrote:

> @andrewshadura <https://github.com/andrewshadura> hey, looks like
> terrible performance bug! :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Komzpa/twms/commit/54b06e3e444fc2eb99c149778445edcd48fe2ec6#commitcomment-14559361>
> .
>

--
Cheers,
  Andrew


just say `abi-version` here, to allow for renamed backend instances?


same


Ok , i work with fullscreen so i didn't see that point .

i use the funk i created :
void FxPanel::removePlugin(){
    QList<FxPanelItem*> items = findChildren<FxPanelItem*>();
    for(FxPanelItem* item : items){//find the first free slot to put the new plugin
        if(item->containPlugin()){
            item->unsetPlugin(); <<<<<< to use that funk that should close the editor too
            return;
        }
    }
}


to be honest, I think the BB\Fixtures annotation is useless (to be removed in BackBee too)


I'd extract it


You could output the download to a known filename, e.g. "wkhtmltopdf.tar.gz", that way you could just have `WKHTMLTOPDF_URL` (including file name) and skip `WKHTMLTOPDF_FILENAME` making config easier.


I believe the adding of luaMaterialData broke the CREG test


What adapters are you mean? I think the only adapter not using html provider is File.


I know everybody loves default args, but as there are no callers omitting timeout, I'd suggest removing the default values and stop providing new ones.
Also drop the `const` on `abiVersion`?


so? fuck


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1430 of 1430 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)


Needs a doc change, too?


This is bullshit. Reverting.


Enda - why has the file path changed for the images?


I meant 0.13.0-rc2 :-(


You can put the `&&` into the `[[ ]]` expression, so that only a single `[[ ]]` expression is used. Quotes are not be necessary within `[[ ]]` (they are only necessary within `[ ]`).

Other parts of this script also look a bit strange.


@mickaelandrieu don't review the code, it's not at that stage yet!


Fixtures will be loaded using fixtures bundle.


I've added a case that checks if a static app can be opened. I also check the redirectionDon't hesitate if you want me to do deeper tests.


Why do you disagree?
Having `IllegalStateException` with pretty message is more pleasant than generic `NullPointerException`.
Some fields in `NeoStore` simply can't be nullified because they are just `long`s.


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


The proxmox-plugin is fixed btw and compatible with Proxmox VE 4


fixed with that

void FxPanelItem::unsetPlugin(){
    this->plugin->closeEditor();<<<<<<<<<<<<<< i added that here
    this->plugin = nullptr;
    this->label->setText("");
    this->bypassButton->setVisible(false);

    updateStyleSheet();
}


Add STAR number to commmit title, please.


I amended this nested list and disabled tests on SunOS, as you claimed they were flaky.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1433 of 1433 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)


Please export `FACULTY_ID` in the export below.


asd


Another nested list I amended.


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


:+1: Awesome. I'm curious how did you came to this? Do you use the package with some issues?


phh.... :-) bash script :+1:


Also it would be perfect to include reference to where you got instructions for Opella-XD. So if it was https://github.com/foss-for-synopsys-dwc-arc-processors/toolchain/wiki/AXS10x-and-Ashling-GDB-Server then link should be in commit description.


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


develop (Myoldmopar) - 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)


I noticed that zenify/doctrine-migrations now has a dependency on this package so I wanted to look at it and found this when styding the source code.

By the way the tests did fail. Not sure if it's related to this change though.


Please use FQN for inheritance and rename the class to just Application.


@pluma please review.


I think with this method we can delete lines 49 and 50.
Also in the SearchQuery backtracking should be activated when the destination was found.


typo


found my mistake. will commit fix now.


Awesome bro
exp might confuse compiler to run into error.
Good job.


I see.

IIRC It is. Try reverting with some other change. Travis should be fine.


OK I'll play around with it. We started a Docker project at work so I'm going to be getting a lot of experience soon!


oh nice haha i upvoted you :smile: i think i'll change this back


Maybe do this:
```java
@Override
	public @TrackingEvent.Type int getType() {
		return TrackingEvent.EXPERIMENT_RUN;
}
```


If it is there should be some other way to solve it. This feels wrong.


Shouldn't we set the SearchQuery here at the other node?


Then, I respect your opinion. If there are such requests, we can reflect them later.


Do we always have to check all the visited nodes?


Actually not, I separated the conversions from rendering (which is called in line 25) to avoid impacting the cell width calculation by the additional html markup.


不错不错，java.lang.System.getProperties().setProperty 这个方法灵。
文档写的也清晰漂亮。


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


I understand the code, but I don't understand why do we need to use run_network_interacting_from_args to re-try. Is it that we just expect it to fail from time to time due to external issues? (network issues I would expect)


Danke dennis!


Use `R.propEq('faculty', facultyCode)` instead of `hasFacultyCode`.


This shouldn't be set here


A lot times the Xcode shows error of not loading the JVFloatedTextlabels, have you ever found that, kindly share your thoughts..

TIA
Vishwa


Yes, you're right. Just doing the string concatenation won't buy you much.

Regardless, I don't like pulling out that function because it adds another layer of indirection. If you want to find out how changing the call struct changes the query you have to jump to two places in the file.

If you want to reduce the complexity of the function maybe you could extract the common sections with NearbyCall as there's a lot of duplication there. Otherwise I think it would be simpler just to leave it inline.


Jimmy,

Can you give me specifics of what needs to be updated?

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Saturday, November 21, 2015 12:56 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Cleaned up and edited readme.pdf to make it more friendly. Added (27297cc)

the code layout picture needs update.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/27297cc4e74b193031b2fc009605142cc4e686de#commitcomment-14541398>.


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


Jimmy,

Good point.  I can do that.

Will Arthur
Intel Corporation
Server Security Firmware
803-216-2101

From: Gang(Jimmy) Wei [mailto:notifications@github.com]
Sent: Saturday, November 21, 2015 1:39 AM
To: wcarthur1/TPM2.0-TSS <TPM2.0-TSS@noreply.github.com>
Cc: Arthur, Will C <will.c.arthur@intel.com>
Subject: Re: [TPM2.0-TSS] Fixed bug with _Prepare tests. Needed to reset SYS_CONTEXT->rval at the right (d71ab7d)

Can we add "SYS_CONTEXT->rval = TSS2_RC_SUCCESS;" into beginning of function: CommonPreparePrologue()?
It can save a lot of size.

—
Reply to this email directly or view it on GitHub<https://github.com/wcarthur1/TPM2.0-TSS/commit/d71ab7d558b81a25f9410a8e9069771d5450f5af#commitcomment-14541486>.


Use `facultyId` instead of `facultyCode` to be consistent with config.


See [here](https://github.com/kennyk/RapidFTR---Android/blob/master/AndroidTests%2Fsrc%2Fcom%2Frapidftr%2Fdatabase%2FChildRecordStorageTest.java#L14)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6319](https://teamcity.mesosphere.io/viewLog.html?buildId=79409&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:13


All of them newlines! :stuck_out_tongue_closed_eyes:
```go
switch name{
case 1:
    something
case 2:
    something else
default:
    caribou
}
```


I'd be happy to see such a way in Nette. If you find some, I'd be love to merge it.


Cool!


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=10)


Hello,
When I do the same with code, I have the same bug. I find that is in getChildDrawingOrder().
I think the library doesn't allow the no-repeat coverflow...
I'm trying to edit the code for that... hope the lib owner can change it.


thanks


Is there a new issue for those TODOs?


恩，谢谢

2015-11-23 18:42 GMT+08:00 linhua55 <notifications@github.com>:

> 纠正错别字：
> peremater --> parameter
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/XX-net/XX-Net/commit/e79b15c90d65aa6a04c67d207067da1f6925005d#commitcomment-14556624>
> .
>


@guiocavalcanti, vc lembra o motivo da adição dessa linha ao Gemfile? Acho que esse serviço do GH não existe mais.

![gems_github_com](https://cloud.githubusercontent.com/assets/1200033/11338142/1d835fec-91d1-11e5-8457-8a675154d0d3.png)


Check out my implementation of bundling in the new v0.3.0 release of Pucauto: #f1c40f


Good catch!  I'll go through this and merge shortly.


im not sure that this handled atm - but im pretty sure we dont want to handle it here


nit: I think the whitespace/indentation is wrong here?

Also, since there is no third arg, may as well do a statement; while (condition) { instead of an incomplete for (...).


Мне кажется, некоторые слова могут быть написаны как с большой, так и с маленькой буквы


The point is nette/di calls prepareClassList sooner than beforeCompile so there should be no need to call it a seconf time.

https://github.com/nette/di/blob/master/src/DI/Compiler.php#L202-L207


Does the affect the mobile experience?


Niiice, didn't know that this could be used, thanks!


I would love to see some small tests to cover the new functions. (I'm not able to do that...)


No sweat, that is how you learn! :) Let me know if you have any questions.

Kind Regards,
Zachariah Kennedy

On Mon, Nov 23, 2015 at 6:56 AM, nikhil18 <notifications@github.com> wrote:

> ya my bad i forgot i had already added frandom
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nikhil18/lightning-kernel-OMNI/commit/9351c6c6038404d0f2ccdcc0d1381e940e9cf9c2#commitcomment-14559353>
> .
>


Since you're always passing this same value down to runKarma, it'd be best to remove this new extra argument from it and just make this change on line 135 directly.


Please add one or two sentences for each, describing them. Are there any differences in what they support and how you work with them?

Also, would you mind providing links to them?


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


the issue with your stock indexer is/was that you are only rewriting "Mage_CatalogInventory_Model_Resource_Indexer_Stock" not the 4 product type stock indexer. These 4 indexer do the main work and as you are not rewriting them they still use the default "getIdxTable" function. As result they are writing into the old magento temp table instead of your new one.

just to clear out the "strange behavior" statement ;)


Is it necessary to call path.resolve on this?


Насколько я понимаю, тут слово рандомного размера рисуется.
А в задании написано "размер слова должен быть тем больше, чем чаще встречается слово"


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


Acknowledged.

I'll revert that change and keep it inline. The feature in it's entirety is more the subject of this PR anyway. Perhaps we could take a look at refactoring in a different PR.


Collaborator:
A review for this commit was created. [Link to review](http://127.0.0.1:8080/ui#review:id=11)


See 9ed1afe


"__"-prefixed identifiers are normally reserved for the implementation... A single underscore prefix is normally used for internal/implementation only functions.

Besides, I'm not completely sure about the use cases for this ... :O


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


this has been added to openwrt r47588 , you can revert this file.


Good.

What can you do to improve the method such that it may scale?


good job!


@JustAkan 예 이번에 새로 구입을 하게 되었습니다.
옵지쪽은 관련 업데이트가 있을시 시간 되면 적용하도록 하겠습니다.


А еще мне кажется, так будут слова друг на друга наезжать. А этого не должно быть


Good summaries. I'd like to see even more of where you disagree with the authors, and also reason about what one might do to improve the methods to address your concerns.


this line should follow capitalization from the machine field in /proc/cpuinfo ... which is the same as in the WITI.dts file


Hey wait a second. How am I supposed to run the tests if composer doesn't install phpunit??


this line is good in lowercase as per convention.


I believe this change should be underlined in upgrade guides, etc. Many apps can be running with `Repo` as a worker, because of this generator.

Ecto had wrong hints about this too: https://github.com/elixir-lang/ecto/commit/a6c29df7c36d372a231564ecf01b16ede9bfcd82


I see. It might be due to unit test, since `beforeCompile()` has dependency on `ContainerBuilder`.

`EventDispatcherExtension` doesn't rely on `Compiler` life cycle, only on `ContainerBuilder`.

Saying that, test with `Compiler` that would test `EventDispatcherExtension::beforeCompile()` could pass even without `prepareClassList`.


Instead of repeating this code of concatenating the path prefix by hand it'd be cleaner to have an internal function, something like **prefix** that you could call instead. Then this line for example would be just:

```javascript
options.buildAmdDest = options.buildAmdDest || prefix('build/amd');
```

Also, you could have this **prefix** function use `path.join` if `options.pathPrefix` is set, instead of concatenating by hand and manually adding the backslash on line 11.


yup, it will definitely be in the 1.1 upgrade guides


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


should be "MQmaker WiTi" to match other dts files. .. this will match *"WiTi" in ramips.sh


This should read:

A collection of shell scripts that we use at DVELP for our making the everyday more efficient.


fix #1


you're right, sorry, I must have had a snafu in my wx versions cause I had no wxSTC_INDIC_TEXTFORE in my local wx3.1 rev.

I'll fix my patch asap.

thx,

-- p

>why <=3.1 ? `wxSTC_INDIC_TEXTFORE` exists in wx3.1 so the old condition should remain
>Maybe you have an outdated version of wxWidgets?
>
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/kluete/codelite/commit/c0ff3c420451e6290d7c6c43203559b7850738d5#commitcomment-14559880


In that case you can keep using ContainerBuilder in the test but you should call prepareClassList from the test, not here.


`lua_pushcclosure` can fail too.
You have an easy work around though: push *before* the `gumbo_parse_with_options` call! :)


For ease of use, we recommend first downloading the shell scripts, then adding an alias to them in your `~/.gitconfig`. For example:


s/dual License/dual license/


offset numberToReturn


@nirzar not a good idea to do anything less than a point, since it won't work on non-retina devices. if we need a 1px anything, it's best to do in code, but even then might not be feasible w/ newer devices:

http://stackoverflow.com/questions/21385159/retina-issue-how-to-make-uiview-to-have-width-exactly-1-of-pixel-instead-of-2-p


The prefix shouldn't be used here, `bower:` needs to be the prefix in this case, the path only comes up after it. Also, this is pointing to metal inside bower_components, so this prefix wouldn't make sense even after `bower:`, as the path to metal inside bower is always like this.


Good one.


Can you change the commit message to "bug 1206309 - Fix a typo in en-US" ?


can we get rid of param?


In theory yes. Could you confirm that.


This is fork of library I already deprecated.
You can look on new implementation here.. It does not have everything done, but you can move remaining things to this new implementation

https://github.com/applm/CarouselWidget


C99 has an explicit `printf`macro for printing an `int64_t` so I wouldn't use the cast here.

See http://stackoverflow.com/questions/9225567/how-to-print-a-int64-t-type-in-c


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1432 of 1432 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)


We need @nop33 to have a look at this. Just not sure it won't break the OAuth2 workflow.


Confirmed, preparing PR.


dup


This looks is great but the icon is missing from the git repository!


input type="file"


Why do we need such an abstraction for Responder functionality when only one implementation exists - ResponderImp? Trait is obsolete and *Imp calss's API actually just repeat  Responder's API.


Fixed


параграф убрать, а сам текст сделать лейблом для input type file


qué pena :crying_cat_face:


Was thankfully temporary only, on latest changes during development it is no more needed.


... the eye of the beholder :p https://i.warosu.org/data/tg/img/0378/09/1423000730947.jpg


Can you add this to `test/functional/webui/proxy_mode_test.rb` instead please?


That is on purpose, it shortens the output.


what a strange name


Looks like it's wrong fix. It's opposite to previous condition + plugin is not working with this fix .
Should be something like !(file && file.path && file.path.length > 0)


Update


Sounds good :)


I asked documentation for help on the WS domain description


Is it even possible to have a message not be a string?


Is this heroku specific? Should this be moved to the Heroku adapter?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1433 of 1433 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)


Looks good to me.


Why make the `port` separate from the configs? In my mind, if you seek to further control the `port` on which the integrator runs, a property in the `configs` object is preferable & continues in the existing pattern of configuration options.


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


I don't see the DoS vulnerability...


I've split the issue with function volatility into #44, since it affects more than just this one function. Please can you check the proposed script is sensible?


What I was going to say earlier (but thought it didn't matter that much) was that we should agree on a style for the comments: complete sentences starting with a capital letter and ending with a period, or always starting with a lowercase letter and not ending with a period.

It will improve the consistency of the codebase.


Should be "as *an* internal slot"


Why re-declare the `hookUrl` with each loop? It's my opinion that line 7 is a preferable place for this variable's creation.


What's the difference to `test/functional/webui/proxy_mode_test.rb`?


I'm fairly certain we don't want to change the global specs for hw_x86_64. Currently, we add `SPECS += acpi` to *$(BUILD_DIR)/etc/specs.conf* in *tool/create_builddir* for nova, sel4, foc_x86, and hw_x86_64. So, just recreating your build directory will fix your issue.


Done, let's wait for travis... https://github.com/Symnedi/EventDispatcher/pull/4


Perhaps the new factor is just a bit too high in midgame?


why do this when no networkPresets?


also fixed, will squash later. thanks


I just realized this might break your build script. I'll try to integrate it with the current main.cpp as soon as possible.


resultl?


thank you, @Dignifiedquire :)


I was trying to be like Andre

![](http://25.media.tumblr.com/tumblr_m76a6xFSiQ1qbx1zco1_500.jpg)


return this :)


Does this fix the camera?


For anyone interested in why this happened see:
http://atodorov.org/blog/2015/11/23/bad-stub-design-in-dnf/

The test code is updated as part of PR #113.


I agree. Let's pick a style and format (even for code?) and stick with it, because personally, I cannot guarantee consistency of my writings without one.


Prefer `$fz--xl`

`$fz--xl: fz(2);`


Mainly to facilitate testing. Having Responder as a trait allows us to easily generate mocks or test implementations in the projects that depend on this library.


TY -- good catch!

Note, however, this isn't JSON.


I think I don't understand this comment. Basically, all REST classes from pkg/registry/*/etcd/etcd.go are implementing those interfaces.
So you would like to extend those interfaces (which means basically extending pkg/registry/generic/etcd/etcd.go) with methods:
NewListOptions()
NewWatchOptions()
both returning *unversioned.ListOptions?

 What is the purpose of that? For me it's overcomplication that doesn't give much any value...

 Also note, that GracefulDeleter is taking explicitly DeleteOptions in its Delete method.


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


This means that calling setAttributeNode with an Attr with a prefix could remove an existing attribute without a prefix. Weird, but I think we should change this some more in https://github.com/whatwg/dom/issues/117


:gun:


Exposing FD Barrier Engine to SWIG/Python.


  File "ccTalk.py", line 394, in __len__
    repr(self.payload)+1)
TypeError: cannot concatenate 'str' and 'int' objects


test


This commit is being reviewed as a part of review issue #108.


Change `libopenzwave.so` to `libopenzwave.dylib` and this works for me on osx :) Good work!


I made the assumption that a <?php requires a closing ?>. Most of the
language files contain the closing ?>. The two files I edited did not.

-Matt
On Nov 23, 2015 2:51 AM, "gburton" <notifications@github.com> wrote:

> Why are you adding ?> endings to files that should not have them ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Matt-AF/Responsive-osCommerce/commit/d33cfee48b11a90002c0dd23841c603dbc2699ca#commitcomment-14555465>
> .
>


8 spaces !? Must be 4. Bad tab editor configuration ? Use "git ci -v" to review the code before committing...


trailing slash, we'll probably have stuff inside it


lgtm :shipit:


`old`


Oops, I'm pretty sure you didn't mean to return here, right?


Wow: the `ConsoleRunner` is becoming very feature rich.

Nice work! :+1:


Лучше бы все эти IParameter'ы объединить в один класс, назвать его например "настройки" или "конфигурация" или еще что-нибудь такое


The mobile experience should be more consistent. The "tap" event was a synthetic click to remove the 300 ms delay found in older mobile browsers. Since no solution will perfectly replicate the browser behavior, it would often lead to false "taps" when scrolling over an element. This simply allows the browser to capture and propagate user interactions, instead of trying to fake it.


That's what is used here: https://github.com/mapbox/mapbox-gl-js/blob/master/js/ui/handler/scroll_zoom.js#L40


Was issue #48


Why do we need this (and the corresponding one for Instruction)? Does the compiler not match [Foo *] to [const Foo *] automatically? If it doesn't, shouldn't this be a const_cast, rather than static_cast?


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


А зачем эти штуки вообще? И зачем две папки Client и Clients, почти с одинаковым содержимым?


initial commit


@PHPGangsta you are right, seems I missed the test script, thanks, here it is: https://github.com/php/php-src/commit/801e46d8c0a869d60e0af9aa0909d6df2cfc7958


Here you need to keep that part. Otherwise, the tests won't be run. You need to remove the else part, not the full if.


Linked to issue #47


Please follow proper camelCase even if it is helper function. Should be `logAllPosts`


@delta2323 Is this okay?


Please follow proper camelCase even if it is helper function. Should be `logLatestPostCreatedDate `


yes. if yoiu change the timer (counter).


popopo


we can also use "strpos()" here


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (895 of 895 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.33%25-green.svg)


快看看


This fix doesn't correctly mirror reflections. The shader fix #468 mirrors reflections and fixes the faces on the cubemap that needed fixing.


Well understood.


Uppercase g (for all) + function name


Pour le moment, j'ai juste un service generatePath et une publication du chemin sur un topic nav_msgs::Path, pour les tests. Je ne suis pas sur de ce qu'on a prévu de faire par la suite, dans mes schémas de conception j'ai mis une action, mais ça n'est pas définitif. Donc pour l'instant, le chemin est généré dans la callback du service sans thread, mais pareil, c'est prévu dans la conception, à voir. Je n'ai pas encore d'idée définitive parce que ça dépend un peu de la conception du manager de navigation.
Merci pour le code très lisible, cependant, tu changeras peut-être d'avis avec les mises à jour du dernier commit. Et il manque beaucoup de commentaires.
Mes objectifs principaux avant de faire une pull request sont :
- finaliser les liens ROS (services, actions, topics), et utilisation ou non de thread (classe PathPlanner)
- finaliser la classe Path qui n'est pas encore utilisée
- documenter les principale classes
- gérer les cas critiques (destination sur un obstacle fixe, destination hors map...)
La version actuelle est fonctionnelle et donne des chemins corrects, à priori. Les temps de calculs sont raisonnable (environ 1.5s pour un chemin traversant la map avec quelques obstacles) ce qui est une bonne surprise étant donné que j'ai codé sans chercher à optimiser quoique ce soit.


case class


Thanks!


thx a lot!


What happens if init is called twice?


I just did a quick test and you're right - we need this. I still think it should be a const_cast. What do you think?


Base - название для абстрактного класса. Лучше пусть какой-нибудь Simple. А еще, это одна из точек расширения - алгоритмы могут быть разными. Так что нужно наследовать его от какого-нибудь интерфейса


That is more out less the end goal, I'm just taking it one step at a time.

It can stay a struct in the C++ code though, no reason for it to change to
a class. It will be mapped to some properties of a Lua object though.

How about moving the one enum to two variables and transform the struct to
an class to be used in Lua?

—
Reply to this email directly or view it on GitHub
<https://github.com/Wohlhabend-Networks/LunaDLL/commit/6fb3f1ab07ebab54b5c7c12e859e59d8d2e4ddbf#commitcomment-14556293>
.


Thanks for the fast solution! I will change/retract this patch :) (once I find how)


I guess the registry plugin would complain about duplicates as you say, but do we want to pretext against that in any way?


Все что здесь происходит с Action'ами решительно непонятно. Нужно написать так, чтобы код можно было хотя бы примерно понять


Тут и во втором случае использования: в `return` происходит присваивание, это не хорошо. Тебе нужно чтобы `return` в `reduce` вернул новое значение. Еще вводит в заблуждение и заставляет задуматься переопределение параметра `sum`, который передается. Поэтому пусть будет проще: `return sum + item * b[i];`


Yes!  Also qstrs and small ints are easier to extract because their high bits are all zeros as well.  It makes a massive difference to code size (best to prefer ptrs, ints and qstrs over floats) and I also made similar change to 30-bit stuffing: 8b8d189bc09b050f9a66048736c7e990ac8c8335


И это тоже точка расширения. Форматы картинки могут быть разными. Так что тоже нужен интерфейс


`bookmarksMirror` is renamed `bookmarksBuffer` or `bookmarks` is renamed? We still seem to be using `bookmarksMirror`.


Имя файла лучше передавать извне или хотя бы в Program задавать


@ccordoba12, what about this one ?


I don't see the point in having a check always on for something that should not happen. For those check using `assert` seems more appropriate.

Having a nice message seems not so important to me, it doesn't give you any better info for debugging than a NPE.

About the long fields, we can have something like `FIELD_NOT_INITIALIZED` to mark them as unavailable.


where is the control_service used?


Where is the PR for it?


This change looks interesting to me: i need the response headers, to check if the resource returned ha some information data in the headers.

This because servers could return 200OK, but errors in http headers.

I do not want handle inside the library 200 ok -> logic error, outside is ok, but I need to access to the headers.


@ccordoba12, and this one ?


You should give some time for GC to do a cleanup (eg System.gc() &.Thread.sleep(1000) after warmup loop, and then one more parse)
Also maybe close FileInputStream - but not  really important

I noticed that mem usage is increased this way - as opposed to the original single parse


Should be repr(self.payload)+chr(1)


I would avoid using macros... Perhaps we should use a namespace "log", rather than prefixing the functions and enum with it. Although that might be a bad idea, not sure.


I think you managed to include a `pp hash` here, which isn't supposed to be here :)


I think you can base it on the max possible step size rather than having it be a preset thing.  Given the math for `stepSize` that follows a few lines below this, we get that this equation: `maxStepSize = (dist / thresh) * minStepSize`.  You  can rearrange to solve for `dist = maxStepSize / minStepSize * thresh`.  This value of `dist` makes our step size equal to the max - any dist bigger than this will keep the step size the same size because of our limit.

On another note, I think we only need one of `maxDist` and `searchRad` as parameters to the `nearestObstacleDist()` method, right?


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


lmao hate mail to steve


@tambry ? I suggested to put it into **separate** branch (e.g. _coverty_scan_), no the master as it is now.


Hehe for sure my friend we will not working during the two week of holiday!

Most hard done. I think only thing I don't view is the Capsule and the updated services with your fix :-D .

Can you make me a copy of your DB so I can test and tell you what we need to fix before Thursday meeting with the client to show her how to add stuff in her website? Own you 2 bottle of wine now :-P for sure!


Well, let's just bump up cvut/sirius#164


Good point.
To make the change incremental, see 90329ba: added a guard to only allow it explicitly.


:+1: 90329ba


``show_ips`` is now undefined at this point.


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


        if func.({key, filtered_value}) do
          Map.put(acc, key, filtered_value)
        else
          acc
        end


s/path/filepath/


Never mind, I found it.


Nope, defined in the page context now.  Never mind...


:+1: I just committed the first pass, don't look at the code! :wink: :see_no_evil:


Graphic of the log Total gas emissions vs log population density for each metropolitan statistical area. The size of the dots represent the total population of the Metropolitan Statistical Area.


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


@j-san The source code is using bower to create builds, I'm unclear on any problems with that.  If you a have an issue, please [file an issue](https://github.com/FezVrasta/bootstrap-material-design/blob/master/CONTRIBUTING.md#using-the-issue-tracker) and we can discuss there.


Во-первых название странное, во-вторых это опять же точка расширения, потому что могут по-разному подать текст. Тоже нужен интерфейс


I want to better understand Git Workflow


Did anything actually change in this file? Or is this just refactoring?


Hi, Cüneyt. Thank you for indent it.
In catalog/model/account/return.php, lines 11 and 61, I've leaved the call to trigger->fire without reference because  in PHP 5.4 it return fatal error:
"Fatal error: Cannot pass parameter 2 by reference in catalog/model/account/return.php on line 11".
"Fatal error: Cannot pass parameter 2 by reference in catalog/model/account/return.php on line 61"
According to php.net, the reference sign (&) should be used on function definition, instead function call (see note):
http://php.net/manual/en/language.references.pass.php


間違えた


绑核的目的为了获得更大的性能，减少多个线程跑到同1个核上造成性能瓶颈。


Here the error:
```c++
Scanning dependencies of target feelpp_test_interpolation_grad
[100%] Building CXX object testsuite/feelinterpolation/CMakeFiles/feelpp_test_interpolation_grad.dir/test_interpolation_grad.cpp.o
/Users/vhuber/PRO/IngeRecherche/feelpp/testsuite/feelinterpolation/test_interpolation_grad.cpp:233:13: error: call to implicitly-deleted default constructor of 'Test<3>'
    Test<3> test;
            ^
/Users/vhuber/PRO/IngeRecherche/feelpp/testsuite/feelinterpolation/test_interpolation_grad.cpp:126:13: note: default constructor of 'Test<3>' is implicitly deleted because field 'Igrad' has a deleted
      default constructor
  grad_type Igrad;
            ^
/Users/vhuber/PRO/IngeRecherche/feelpp/feel/feeldiscr/operatorinterpolation.hpp:390:5: note: explicitly defaulted function was implicitly deleted here
    OperatorInterpolation() = default;
    ^
/Users/vhuber/PRO/IngeRecherche/feelpp/feel/feeldiscr/operatorinterpolation.hpp:542:16: note: default constructor of 'OperatorInterpolation<Feel::FunctionSpace<Feel::Mesh<Feel::Simplex<3, 1, 3>, double,
      0>, Feel::bases<Feel::Lagrange<1, Scalar, Feel::Continuous, PointSetEquiSpaced, 0> >, boost::parameter::void_, boost::parameter::void_, boost::parameter::void_>,
      Feel::FunctionSpace<Feel::Mesh<Feel::Simplex<3, 1, 3>, double, 0>, Feel::bases<Feel::Nedelec<0, Feel::NedelecKind::NED1, 0> >, boost::parameter::void_, boost::parameter::void_,
      boost::parameter::void_>, boost::tuples::tuple<mpl_::size_t<0>,
      boost::multi_index::detail::bidir_node_iterator<boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::index_node_base<Feel::GeoElement3D<3, Feel::Simplex<3, 1, 3>, double>,
      std::__1::allocator<Feel::GeoElement3D<3, Feel::Simplex<3, 1, 3>, double> > > > > > > > > >,
      boost::multi_index::detail::bidir_node_iterator<boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy,
      boost::multi_index::detail::ordered_index_node<boost::multi_index::detail::null_augment_policy, boost::multi_index::detail::index_node_base<Feel::GeoElement3D<3, Feel::Simplex<3, 1, 3>, double>,
      std::__1::allocator<Feel::GeoElement3D<3, Feel::Simplex<3, 1, 3>, double> > > > > > > > > >, boost::tuples::null_type, boost::tuples::null_type, boost::tuples::null_type, boost::tuples::null_type,
      boost::tuples::null_type, boost::tuples::null_type, boost::tuples::null_type>, Feel::InterpolationGradient<mpl_::bool_<false> > >' is implicitly deleted because field 'M_interptype' has no default
      constructor
    InterpType M_interptype;
               ^
1 error generated.
make[3]: *** [testsuite/feelinterpolation/CMakeFiles/feelpp_test_interpolation_grad.dir/test_interpolation_grad.cpp.o] Error 1
make[2]: *** [testsuite/feelinterpolation/CMakeFiles/feelpp_test_interpolation_grad.dir/all] Error 2
make[1]: *** [testsuite/feelinterpolation/CMakeFiles/feelpp_test_interpolation_grad.dir/rule] Error 2
make: *** [testsuite/feelinterpolation/CMakeFiles/feelpp_test_interpolation_grad.dir/rule] Error 2
```


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


I was just looking over this...
Adding  'android:textColor="#000000"' to the EditText section in activity_rest_conversation.xml makes the RESTchat text (in the input boxes) black.

An issue I found with this solution is that for example when you click Profile all the fields' text is also white. And I think that these fields are added to the list view during run time .
(see ProfileActivity.getProfile() or MyProfileFragment.onCreateView().)
Also "Stats" are in white.

From what I can tell (from 15 min of looking at it) it seems reasonable that you have to define a theme EditText style, where the textColor is black. But I have no idea how, where/how...etc :) But it seems unreasonable that the default colors of background and foreground are the same.

So it is worth checking why the EditText  default textColor turned to white.
I believe that there are dark t and light android themes.
So maybe something to do with using dark theme parts on the light one ?
Or something missing in the  style file ?

Good luck with the rest :)


Wow, thanks a lot for this!

This proves my programming skill is waaaay to low. I could probably learn from this. By the way, I've read the codes, I'll merge them when I am sure that this would compile on Windows, or until I've made some dirty hacks (lol) if it fails to compile on Windows.


Ah, ok! I vote for "less visible" is better here (only as visible as needed), or else these shadows obstruct all effort of the two us to crate good map rendering ... :-)

And I do actually regularly use some of these zoom levels for map browsing on some occasions, especially for long-haul traveling.


This paragraph seems outdated: the ``astropy.table.jsviewer`` uses configuration options, so I guess that the goal is to avoid editing this file, but I don't how these configuration values can be changed ?


Main difference is that this test doesn't have to be enabled manually. It always runs and tests the proxy login.
It's also easier to see what's getting tested, and how the proxy login works


And it covers creation of new users


Why this commit?  I long made it.  Are you working a code base that is behind?


Por favor,  selecione um item.


Sorry, but this line breaks vertical aim at MAP02 DOOM2.WAD
When you shoot from top of the platform, a bullets hit the floor, not things
![underhalls 1](https://cloud.githubusercontent.com/assets/6560567/11339227/9c5490ce-9209-11e5-8ca8-058ebdb0ea65.jpg)

I made a save file where bug can be reproduced: https://db.tt/adXZCrUe


Why not Option.none() ?


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


You meant "melted"


Actually, Michael asked for these specific column names, so I changed it.


I propose to do it like that:
```
if (value.isEmpty()) {
            provider.defaultSerializeNull(gen);
        } else {
            provider.defaultSerializeValue(value.get(), gen);
        }
```


Changes here done as part of my comments to remove 'Model' from the class name.


Probably fixed in ec980c7.


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


Vielen Dank


I'm sorry I hadn't notice that there is a ValueSerializer


Awesome!


Would not be better to check if extension is enabled rather than loaded?

For my personal case, I would prefer to have xdebug loaded but disabled by default.


Joe, if we do a group by on nid, wouldn't that restrict the count for multiple webforms of the same nid submitted by different contacts?


Thanks!


We could do a COUNT(sid) instead of COUNT(nid), but it would essentially be the same thing.


Applied.


Unfortunately, Chrome and Firefox get the anchor positions all wrong when using windows > 1240px wide :(


Not as far as I can see, this setting was deprecated in df73fb3 and is effectively a noop here.


This was reverted because mac binaries are not available when using R-devel, so installation of dependencies was failing.


Yes, I think you can rephrase it to say to use the config options instead of the file. These are edited using for example:

In [1]: from astropy.table.jsviewer import conf

In [2]: conf.jquery_url
Out[2]: 'http://code.jquery.com/jquery-1.11.1.min.js'

In [3]: conf.jquery_url = '...'
```

Could you update this paragraph accordingly?


Well, actually, you need to doc the new parameter ;-)


I think you should set the filter to the queryset and then call and return super() as suggested in a previous comment. The reason for this is there's magic happening up the chain of objects, in GenericAPIView.get_queryset(), which I can't say I fully understand (calling _clone()) but it's probably safest to call it.


`docs/markdown/authoritative/settings.md` references `pipebackend-abi-version`; `docs/markdown/authoritative/backend-pipe.md` talks about `pipe-abi-version`, maybe check if that needs an update?


Should be fixed by ec980c7 (complained about at another commit as well).


This is already in master: https://github.com/PowerDNS/pdns/pull/2629/files#diff-9ddf9b1158555cabf17daf9f5118f614R33


It's a good idea to split commits to small logical changes. This commit removes comments, debugs and adds a missing check. If for example there was an error in the check and there was a need to revert this, the other two changes would also be reverted back.


@gilles-leblanc What does the `/\d{1,3}` part here mean? Is this specifying a "subnetwork" rather than a single address? The `replace_hosts` function doesn’t seem to support that, though.


I don't believe you're following the codestyle we agreed upon.


The message claims more than the actual commit. Did you forget to include something?


CC https://github.com/servo/servo/pull/8653


I don't agree with this change. It makes it confusing which namespace individual DSL elements come from.


Records the last valid position as computed by the pagination step. Is <null> to signal we have no such position. This way we can differentiate between not-known and zero.


@jerel I actually don't remember :( But I do know that `len(None)` raises `TypeError: object of type 'NoneType' has no len()`


Is BuildConfig.groovy still necessary?


Hm. I'm not sure how useful or sane it is to have WP-CLI wrap and execute Git commands.

What would be _more_ helpful is to have Git / Composer support for #2223. When an update is detected, provide some installation type specific direction on how to update (e.g. "Switch to git tag v0.21.1")

We might also want to create an abstracted utility for detecting WP-CLI installation type.


consider renaming this to `unwrap`, just to stay consistent with the rest.


Ok, also I just saw there is a section `[table.jsviewer]` in the default config file.


I will add specific testcases for every inference rule, this will be easier once we are finished with these so we can really reduce the tests to testing the rules (but we will keep the old testcases, since they also test something correct altough they do quite random testing of different functionality ^^). Especially NAL7 and NAL8 testcases need to be reconsidered, the NAL1-6 testcases tests are very good.


Added new flag to tell us whether we are inside a table-operation or not.


Hi Kevin,

und noch eine aktuelle Info. Mich hat ein Volksfreund Redakteur angerufen ob er mit mir und uns in Besslich kurzfristig eine Reportage über das Freifunk-Projekt machen dürfe. Ich hab zugesagt. Termin ist morgen um 9 Uhr.
Möchte/kann noch jemand aus der Gründerschaft der Community mitmachen/dabei sein?
Ansonsten werde ich mir Mühe geben, kräftig die Werbetrommel anzuwerfen.

Gruß
Heinz

  ----- Original Message -----
  From: Ranlvor
  To: freifunktrier/fftr-peers
  Cc: tackin
  Sent: Monday, November 23, 2015 3:45 PM
  Subject: Re: [fftr-peers] Delete 54310-Internet (38aef28)

  Vielen Dank

  —
  Reply to this email directly or view it on GitHub.


Perfeito tradução.


What's the point of this change? We've been using "connection" in the docs for a while. I think it's the most straightforward term. "Session" is confusing and makes it harder to notice what the difference is with `2.x`.

I suggest that this change reverted.


Within a table, the header is out of bounds. So we better not check that bands are aligned with their previous siblings.


That is pretty much the change I have just made to my UCP family addons which were the main ones affected by this, where lots of optional functionality can be provided from a plethora of other packages from myself and others.

In other places I already did that on the assumption it would be better for performance and as you say, by luck avoided the issues caused by the above change to Loader::package.

By fluke, I had unfortunately only noticed sporadic log messages in the past, then today I came across a site where circumstances exacerbated the problem and hence I tracked it down to here.

Its impossible to provide a clean solution that is backward compatible to the current core without at least adding something to the API. (But that won't solve the problem for existing sites that don't want/need to upgrade)

Some thoughts

- An optional parameter that defaults to the current behaviour, so is backward compatible with existing code, Loader::package('package_handle', $quiet=false);

- Add a new new method to load without warning, so is simply not used by existing code, Loader::package_quietly('package_handle')

- Add a new static method to test efficiently, Package::isInstalled('package_handle'), returning false or the version number.

Having published updates to solve my immediate problem, I don't have any urgency for any of the above. Maybe a similar issue is relevant to 5.7, I have not investigated.....


Har kollat igenom det, det ser bra ut.


.. and we forward that flag to the only place where we may shift whole boxes and all their children in one bulk operation.


This form is a little counter coding style... it's ok... but maybe better to drop this err check and after the err check?


The great thing about GIT is, you don't have to keep the things like this as comment in your code for later reference :wink:


@dafimor please note that you have deleted `src/files/img/tutorial_img/soomla_diagrams/LevelUp.png`
Please pay attention to the files you add/modify/delete.


Good to know, thanks!


<<4th Individual weekly report>>
Name: Sheethal Mathew
M: 11/16 :
T:11/17:
W: 11/18:  Hours: 0.5. Reviewed Code on command line written by Jeremy
Th:11/19: Hours: 1. Attempted the creation of own code. Not much happened
Fr:11/20:
Sa: 11/21


Please do not ignore the error returned by `runActionWithContext`


@michalmuskala this happens in a few other places (docs and examples), can you fix those too? @vadim-moz reported this on the mailing list the other day.


I split these up since we'll need to do them as separate DB calls eventually anyway (since forms and cases will be sharded by different ID's)


Inside a table, at table and section level, we have to revisit every node. Normally the layouter avoids recomputing nodes that have no changed (methods return true, and thus the child-nodes are not visited). But we have to adjust row positions, so within a table every box that leads to a row must be visited so that we never miss a row.


Needs docstring


there are two namespaces: one is client (explicit) and second one is `cql` (implicit).

We'd end up having DSL like

```
   (cql/create-table session :users
                 (cql/column-definitions {:name :varchar
                                      :age  :int
                                      :city :varchar
```

and `(cql/limit 10)` etc etc, which we've never head anyways because we had `query` as `:refer :all` all the time.

So if we want to rework it - we'd have to rework all docs anyways. But since we went from 3 keyspaces that are always needed to just two, this change appeared.


it does not matter, right?


Moved these to more SQL like queries to make it easier to move to SQL functions later


Test corrected and restored: #34


```
def validation_checks(result, borrower):
    try:
        if result:
            return result

        borrower_date = datetime.datetime.strptime(
            borrower['dob'], '%d/%m/%Y')

        if borrower_date > datetime.datetime.now():
            return True

        return False
    except:
        return True

```

This could be rewritten as a more generic date verification predicate using function chaining to pre-filter the borrower dob prior to calling this


No increase in the shift amount. Headers are outside of the normal flow and when they are repositioned, that should not affect other nodes.


added a variation without the &/


all of that is not necessary. We can just treat all boxes the same using our standard methods


Will work on it!


Rather long name, I'd suggest to simply call it `create_block_file`.


Yes, but looks clearly :)


I was working on the code at the same time you were, then you merged from your local copy of my pull request.  For some reason when I rebased on the upstream branch it didn't absorb this commit.  In the files changed on the current pull it looks like git correctly sees that you've already applied this change.


I think this commit is causing us some grief during builds - `libmariadbclient` doesn't seem to exist in the Ubuntu 12.04 repositories.


I'm not sure this is needed. If `window.codebender` is not defined compilerflasher should take care of that on its own. Maybe there was some other error you were running on. I'll be merging this change, but please check again to see if it works with out this commit.


questionable whether that was worth it but at least you can see what the dependencies are now.


`Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>`
`Tested-by: Jerome Forissier <jerome.forissier@linaro.org> (HiKey)`


We eagerly store the row's position as long as it is before the current page's end, but not when it is beyond that point. This allows us to recall it for all boxes that are printed, but keeps the future box position open.


the point of this change is that it's called `Session` everywhere in the driver, in the codebase, in tests and everywhere else. `Connection` is not entirely true for `Session` since `Session` has multiple connections.


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


Table/50024 Serviceleist. & Leihstellungen

Im Prinzip ist nur diese Zeile ausschlaggebend!


Restore a previous position if we have one. We only do that for past pages to keep the current page calculations open to changes from new boxes.


108453024-FixLeedNumberOfHoursNotMet (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1951 of 1963 tests passed)

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


document.styleSheets should never empty


In general, yes, but the SonaType repo doesn't allow the "+" semantic and rejects the POM. I've made that mistake before but forgot about it.

> On Nov 22, 2015, at 9:30 PM, Vijay Raghavan Aravamudhan <notifications@github.com> wrote:
>
> Isn't this duplicated from the above line?
>
> —
> Reply to this email directly or view it on GitHub.
>


`Reviewed-by: Jerome Forissier <jerome.forissier@linaro.org>`


This should probably go to stderr?


@Soullivaneuh there is a 4x slow-down of Composer just by loading XDebug, even if the profiling feature is not enabled.


should be ``% revision_ip.ip`` (the IP address), not ``% revision_ip`` (the ``__str__`` output)


Gotcha.  Truly parallel is an issue.  Something `git merge master` does the trick when you are in a branch, ditto with a fork relative to the other remote.

And most importantly the PR looked very clean in its diff.  Who care about a commit count :)


haha i can't believe i misspelled my own package


am still learning ;)


@borisspektor good catch. I missed it.


If in table, never call "BoxShifter.shiftBox" in this context. This would only work if we do not visit child nodes. But as we have to visit them to see all row-nodes, this would cause multiple shiftings of the same box, resulting in wrong positions.


I think it would be better to write down your full name ex. Betrand Lirette


Cette mise a jour est excellente!!!!


Please, add `gettext()` to the values.


Actually maybe this should be an exception?


that's true if only all the addresses get into page mode.

This could be a valid configuration for certain users who monitor outages..

You can't have more than memory-size / max-size consumers in page mode. So as long as you monitor outages that would be fine. So, I'm not sure this is a valid request.


White space is off for the opening curly brace. For better or worse, coffeescript is white space significant and this is causing an error. Could you correct the indent?


This new method allows me to extract the logical pages as produced by the pagination step. This way we have them unfiltered and can run better tests on the results.


1章で修正したプログラムと2章のプログラムです。手の空いた時間に見ていただけると幸いです。


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


Wouldn't this always be undefined being that there is no userId key on a conversation and the context here would be the conversation?


Very nice, great idea!  Thank you for sharing; I am glad you found this random plugin useful.


Does this import shop config?


:+1: I was about to report this for v3.0.0 :)


Good eye @XhmikosR! Took me a few weeks to spot, hah, Chrome doesn't complain about it...


Ah, great! That is a good note for a Shiboken FAQ


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


Python best practice is to use `except Exception:` and not a bare `except:` clause since a bare except clause will respond to any signal.

Also applies to line 38.


Fair enough.


That was just a bad <kbd>M--t</kbd> :)


miert nem es6-os szintaxist hasznalsz?


I use it in server side in a publication, so this.userId is defined but
Meteor.userId() isn't.

On Mon, Nov 23, 2015 at 9:08 AM, Kelly Copley <notifications@github.com>
wrote:

> Wouldn't this always be undefined being that there is no userId key on a
> conversation and the context here would be the conversation?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Neobii/socialize-messaging/commit/870504750041a89ff0f6f8a126dbb5497fe2ce76#commitcomment-14563053>
> .
>


hint: `max 0 1 == 1`


Not really... You wrote librenm on the link


dinamikus import megneheziti a debuggolast, hasznalj `import`-ot


@astrofrog Done !


The closeness of these buttons with opposite effect bugged me at first, but they go to the Admin where the step can be confirmed, so :+1:


lgtm :shipit:


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1433 of 1433 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)


In the function `out_of_place_write` we do ...
```
static in out_of_place_write(...)
...
b = block_cache_lookup(fdp, start_block_num);
```

But in fact there is no block cache in use when the `CFG_FS_BLOCK_CACHE` hasn't been set. Instead we are using the ree FS instead. To me it's a bit confusing to read `block_cache_lookup` in this case. I'd prefer having two different functions and separate with the `#ifdef` higher up in the call stack.


I think it is a mistake from a previous version