Where is more documentation for the file listed here that was added along with two others. I don't question your selections, I simply wonder what the reasoning is for choosing these particular ones. I have had I issues with bad help in the past, so you'll havE to forgive me if I offend. Also just a simple honest answer would be more than sufficient because in my opinion, by going ahead and keeping these files and not replace with more will leave an opening for unwanted guests that will probably screw things up more than help. I do not mind fixing any issues, but I do find it rather rude to set up someone's account and know that it is vulnerable but choose not to do proper fix as if it were your own. Anyways, thank you.


please remove the printing ;)


Well, as I said, I know nothing about ASM. It still looks like that instruction can easily just be added back in, and I think having it in there makes sense in any case. If there is nothing speaking against it, I suggest we add it back in here, and I'd also add it back in in my BrahmaLoader branch.


```@Ignore```


not super relevant right now but eventually I would change the permalink to something that looks a bit better – maybe like `hall-of-fame-2015`


I'd add the schema to the logging statement


Since partner and platinum are a "subset" it would be great to have them as a "smaller" headings such as h4


не так :) этой аннотацией помечают конфигурацию контейнера, в твоем случае - сам контейнер. В момент его иннициализации сканируешь класспас на предмет eager singleton'ов и инстанцируешь их


@jebbdomingo @raeldc @ercanozkaya  We are backporting the assets filter to framework 2.x so this change will no longer be needed. See : https://timble.assembla.com/spaces/nooku-framework/tickets/275 We will merge this to master once tested.


I like the idea of pointing this out but there are two issues to consider.

One is that the term "semigroupoid" is apparently not used in a very consistent fashion in the literature. Some people seem to use that for what I would call a semicategory (i.e., category sans identity and inverse). On the other hand, if you are thinking about groupoids in the usual categorical sense it seems pretty clear to me. Frankly I think a big part of this inconsistency can be blamed on the mathematicians for de-emphasizing these structures so much. I view situations like this as an opportunity to push back against that trend at least a little bit :)

Another issue is that the notion of 0-semigroupoid as a PER (and setoid as 0-groupoid) is apparently somewhat controversial. The nlab of course calls 0-groupoids sets and doesn't include setoids at all in the periodic tables. Then again, the HoTT folks are happy with doing formal mathematics in a fictional setting anyway so I don't think that should be taken too seriously…


That's not exactly what the code does. It adds output render target to the texture pool if it's not already added. The existing code already adds output RT to the pool when input and output RTs aren't the same.


I'd change all of the white headings (sponsors, partners, etc.) to a more readable color – perhaps the red?


On 11/26/2015 08:05 PM, Zbigniew Jędrzejewski-Szmek wrote:
> So... this looks like it could work.
>
> Why do you use the timeout of 1s? If you want the timeout to be as short
> as possible, 1 µs should be used. For some systems, 1 s is very long.
>

Agreed. I was fooled by the current code because it converts device
timeout into a num of seconds before writing them as new value for
JobTimeoutSec and also by the name "JobTimeoutSec" which has "Sec" prefix.

Since JobTimeoutSec uses the same format as x-systemd.device-timeout one
there's no point into doing the conversion and actually it's boggus
because if a passed device timeout is less than 1s, the computed  value
of JobTimeoutSec will be 0 which means wait forever.

> After your changes, |systemd.device-timeout=| setting is ignored for
> nofail units, unless parsing fails, in which case nothing is written
> out! The logic should be reversed:
>
>   * if |systemd.device-timeout=| is specified, honour it
>   * if timeout parsing fails, warn
>   * if |systemd.device-timeout=| was parsed write it out, otherwise for
>     |nofail| units write out 1 µs.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fbuihuu/systemd/commit/3b9ddcee778c0b604683e21f2e2b678eb9ff9269#commitcomment-14645401>.
>


@freebroccolo Sounds reasonable; do you want to try composing a remark to put in here?


Right, so we could change it so that it doesn't add the Ouput RT to the pool at all, and let the last filter render directly to the Output RT.


you'd probably need to make an own layout for this page – otherwise it shows all of the partners at the bottom again like in the landing page


Sure, I'll give it a shot.


On 11/26/2015 08:05 PM, Zbigniew Jędrzejewski-Szmek wrote:
>
> After your changes, |systemd.device-timeout=| setting is ignored for
> nofail units, unless parsing fails, in which case nothing is written
> out! The logic should be reversed:
>
>   * if |systemd.device-timeout=| is specified, honour it
>   * if timeout parsing fails, warn
>   * if |systemd.device-timeout=| was parsed write it out, otherwise for
>     |nofail| units write out 1 µs.

OK.


@finnq Great work!


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


As you pointed out - that you are in the progress of learning git :-) mind adding a "space" behind the if? rebase, squash and force push?


Do dummy files work somewhat like a non-dummy file? My first reaction would to guess no, but I don't know a whole lot about using dummy files because anything that has the word dummy in it sort of makes me want to pick a different one because most dummies are used for show or just simply to fool someone who has the potential to believe what is displayed on his/her  computer screen. I will look for an answer as well so that I have a little more knowledge of dummy files.


yeah, I start recalling)


Alrighty, I'm good to go with the basics of git!


Don't get me wrong. I understand your point. I just see this option for phase 1 everywhere! (Attached screenshot of Apple Configurator 2 – VPN Profile)
![Apple Configurator 2](https://cloud.githubusercontent.com/assets/83901/11430861/263c1278-9474-11e5-936f-630f81d76703.png)


The Jacoco exception is an expected exception part of testing.
When you say it hangs, this test is a longer running test in the suite, can take upwards of 20-30 secs, how long did you wait for it?  Also, you can try and change lines 57 and 58 on that test to:
    int ttl_jobs = 100;
    int per_thread = 3;

Let me know if it was just a wait issue, and how it ran after lowering those numbers.

Thank you,

Osman


@bryk I have no idea how to declare in JSDoc type of this.label. It gives an error that property label on this controller is not defined. Any idea?


You were searching for something to end in `.core`, but I'm guessing your system is using another file name - the default on Linux is often just `core`. You can check what your kernel uses by running a command like this: `cat /proc/sys/kernel/core_pattern` (see `man core` for more details). This can include a path where core files are saved.

As an alternative, just run the genrev binary in your debugger.
* Run `gdb Build/genrev` (as you did before).
* In gdb, run genrev and pass it the parameter it expects: the path where your repository is located. For example, type `r /home/mangos/mangos`. I'm guessing this is the path in your case based on the fact that I see you're using a user 'mangos' and that you ran the command 'cd ../mangos', but adjust the path as needed.
* This will probably crash too and gdb will tell you about the segmentation fault. If it does, run `bt` to get a call stack as suggested by namreeb.

FYI: the last rev builds fine on my old Linux box, but I'm still using old tools (git 1.7.x for example).


:+1: thanks


@bryk Moved to directive and created different controller for labels management. Please take a look.

Don't mind deletions that's because i've committed from windows and formatter did not change line endings. I'll fix that tomorrow at work. :)


Python has ruined me, sorry :(


@BeerfishWoW No, it does not help.  Try what @evil-at-wow suggested.  Also, please do report still on your git version.


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


@namreeb I whispered you on IRC, but here it goes. My Git version: 1.9.1.
@evil-at-wow Thanks, I'l try that.


I removed what I have done before and add a similar verification in usersInGroup() method. I know... it's not without additional server request but it will produce less request especially if nested group is enable.

I tried other approach but each time it was breaking one of my ldap testing config.


@BeerfishWoW I don't have IRC open right now.  Let us keep the conversation in one place so as not to confuse things.  Thank you.


No problem, namreeb.


Not exactly. Third party modules should not be in the repository. I'm not exactly sure as I didn't try it, but I assume Atom checks the theme's (or any kind of package for that matter) `package.json` and installs any and all dependencies defined in there. So yeah, don't worry about it.


Right, thanks!!! I added this to horst


Yes that would work I suppose. The only thing you have to make sure is input RT doesn't render straight to output RT because that's not permitted if they are the same instance.


Output of cat /proc/sys/kernel/core_pattern:
mangos@SNOWFALL:~$ cat /proc/sys/kernel/core_pattern
|/usr/share/apport/apport %p %s %c %P


Do these logging statements still need to be here, @ZachOhara ? As far as I can remember, no glitches have happened in the last few tests.


Feel free to do so, but why timeout 500 and buffer size 8<<20?


@BeerfishWoW Try what @evil-at-wow said in terms of running gdb:

gdb path/to/genrev
(gdb) run /path/to/mangos/folder
** it should crash here **
(gdb) bt

Copy results, paste here.


Yeah thats what I meant. Best check atom recognises it and installs it otherwise people are gonna be complaining the theme doesn't work.


Done.


My understanding is that having a running `rqworker` or using `--no-rq` makes importing to be run in async or sync mode. The choice doesn't make anything *faster* though.


You don't need the if clause - if you check sizeof the buffer itself which is either float or simple char, same above


Col should be already set right above. What's the issue you are trying to solve here?


Unused


generation values


Then I'm probably less cautious then you. ;-)


L'API dédiée a été supprimée du texte, sera-t-elle tout de même mise à disposition ou pas ?


(me is PosixCompatible on XDA)
That patch alone does nothing, just allows changing the current limit via a sysfs setting. An App for that will follow shortly :-)


The only way to check that is to publish the theme and install it inside Atom or using `apm` in the CLI.


It's df.item_type, not dfhack.item_type.


You're right regarding to the interface but BackBee User still have it. If you use a Security provider that not fill an id for BackBee User, the User management part will broke for sure :(


get weird data in sniffs...
ServerToClient: 3680121 (0x382779) Length: 14734 ConnIdx: 0 Time: 11/26/2015 21:31:18.531 Number: 3


sorry - i wasnt implying that it was - but the point is now you *do* have to have one of those options for the command to work - whereas previously you didnt


`btn-lg` powinno byc tez w class attribute


1) mangos@SNOWFALL:~$ gdb build/genrev
GNU gdb (Ubuntu 7.7.1-0ubuntu5~14.04.2) 7.7.1
Copyright (C) 2014 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i686-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from build/genrev...(no debugging symbols found)...done.

2) (gdb) run /home/mangos/mangos
Starting program: /home/mangos/build/genrev /home/mangos/mangos
Cannot exec /home/mangos/build/genrev -c exec /home/mangos/build/genrev /home/mangos/mangos.
Error: No such file or directory
During startup program exited with code 127.

3) (gdb) bt
No stack.

I know I'm obviously doing something wrong, but I am following up on the instructions posted by you guys.


The info here in bands.max_chan_width should show what the wifi HW supports. If it supports 20MHz, 40MHz (HT40) or even 80MHz or 160 MHz (802.11AC) channel width. Then it is set to the maximum when setting the channel. In Linux if you just set a channel, it is set to 20MHz width. The problem is, that if 20MHz is set, horst can not see packets from a station which sends at 40MHz channel width. So if the HW supports it we want 40MHz or even 80MHz width. No idea how this is handled in OSX...


But from looking at below it does have this information


It looks like die() and exit() cause PHP to prematurely terminate the HTTP response, producing an error or an invalid response. I think you have to let the code run through to the end of the script.


```
 constituent.setParent(parent);
```

This should be called before addToIndexes().


lepiej stosowac `!==` i `===` przy porownaniach
```
http://stackoverflow.com/questions/359494/does-it-matter-which-equals-operator-vs-i-use-in-javascript-comparisons#answer-359509
```


Forget that ... it still is tunneled in bytes ... the actual format size could be saved somewhere in class then that if case could be removed ... but there are worse things on this planet.


primeiro commit


TeamCity ReOpenLDAP :: devel [Build 208](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1248&buildTypeId=ReOpenLDAP_Devel&guest=1) outcome was **SUCCESS**
Summary: Tests passed Build time: 01:43:02


typo: Add bowti**e**2 batch script bt2_batch.sh


Performance improvements are great!!


Bad commit it's http://www.wowhead.com/spell=5143/arcane-missiles no't workd on proc http://merlin.org.ua/wowhead/?spell=44401


@Jorch95 el nivel va a venir desde la grafica


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


@Jorch95 Modelo no hace falta, fijate lo que les mande que vanessa me respondio, modelo ya esta todo implementado. Es todo el codigo dentro del paquete `modelo`


It was more a general remark, because on a quick look it seemed that we can organize a lot of code in a better way than right now.


index dinamico para mostrar las fichas. Hace mucho que no me costaba …
…tanto una tarea. Pero lo logré :+1:  :+1:  :rocket: Luego de un dia de quemar la cabeza leyendo angular y prbando.. con directivas (link, controller, compile, etc). En un principio era un ng-repeat de una directive armada dinamicamente. El tema es que se armaba en cada iteracion en lugar de ser un molde , clonar y popular. Tuve que llevar el armado del molde dentro de la directive y compilarlo.. muy complicadooo !!!

Recompensa: lo probé con 1000 registros el tiempo paso de 5-7 segundos a menos de 3.  Por cada regsitro hace una tarjeta.. por elemplo contactos o empresas. De todas maneras voy limitar a 50 0 100 por un tema de performance!!


This could be encapsulated in the entity stores (eg `getSecondaryFields()`) where typed "models" should be instantiated for every entity-type to allow us abstract from the actual "db" representation. Until we do that I would be happy to keep it in the print-store as you have done it. Well done!

Any thoughts on managing the page numbers for *long* actions and sections?


ainda não testei, mas irei fazer isso ... qualquer coisa eu testo com outro char ;) obrigado por responder


The problem was the response format. I needed to have a json encoded response (in this case a simple "done") to make it work. Pushed the change.


@namreeb mangos@SNOWFALL:~$ gdb build/genrev /home/mangos/build/core
GNU gdb (Ubuntu 7.7.1-0ubuntu5~14.04.2) 7.7.1
Copyright (C) 2014 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i686-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from build/genrev...(no debugging symbols found)...done.
[New LWP 4194]
Core was generated by `/home/mangos/build/genrev /home/mangos/mangos'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x401a63b0 in _IO_new_fclose (fp=0x98d50f0) at iofclose.c:57
57	iofclose.c: No such file or directory.
Traceback (most recent call last):
  File "/usr/share/gdb/auto-load/usr/lib/i386-linux-gnu/libstdc++.so.6.0.19-gdb.py", line 63, in <module>
    from libstdcxx.v6.printers import register_libstdcxx_printers
ImportError: No module named 'libstdcxx'
(gdb) bt
#0  0x401a63b0 in _IO_new_fclose (fp=0x98d50f0) at iofclose.c:57
#1  0x0804a5d1 in extractDataFromGit(std::string, std::string, bool, RawData&) ()
#2  0x080498b9 in main ()


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 55](https://teamcity.labkey.org/viewLog.html?buildId=287814&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 22 (8 new), passed: 268, muted: 2 Build time: 01:30:08
### Failed tests
```
org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.ScrumtimeTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.util.JUnitHeader: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.scrumtime.harvest.HarvestPayRateService$TestCase: <no details avaliable>

org.labkey.test.util.JUnitFooter.afterJUnit: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

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


Maybe it's easier to move the `i` to the first entry that shoudl be kept and do a `g_strjoin("\n", lines[i]);`.


[Like this](https://github.com/odlgroup/odl/commit/7bc3bd273c64ebc02f94768d8fbfc34f195d11c9#diff-f36fc6a61d27372c9894e2c690269d28R50)


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


sure


nossa, realmente facil de mudar o storage, bom, tenho 3 chars no banco de dados, fica realmente fácil, agora vou partir pros testes.


Good fix with! Thanks! But there is one small problem: the
 should to be removed from output. If - for instance - you like to render the errors as unorderd list the
 could break the layout.


Or, to include the scalar-only case,
```python
arr = np.array(array, copy=False, dtype=dtype, ndmin=1).ravel()
```
Up to the `dtype`, this is equivalent to `atleast_1d` to my knowledge.


@XuetaoLiu-CLS, this is what was causing the crash on SXRMB.


Where the hell were you when this was in PR state for months?


Ah. Okay, that's my doing. That's so the proxy can provide a meaningful error message to the end user to report to me, should there be a problem. Well done!


"yanlış karar"


Thanks, mate. Nice nickname ;-)


You mean the 2-dimensional guy, right? In some way, it's a bit arbitrary to flatten in C order or to flatten at all, to begin with, I agree. I considered raising an exception, but then I decided to flatten for the sake of usability. Maybe it's not so useful or hard to understand.


Haha, nice plug-in. Will install it on my colleague's laptop when he is not looking. :)


comment


@kriox26  ahi leí lo de Vanessa que no lo habia leido jaj. Joya entonces. La clase modelo la saco entonces, pero igual hay que poner la clase Main() en Controlador no ?


To reduce merge conflicts, I would change this once the branch is merged into the trunk.


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


This should probably always be checked, now that I look at the code again.


Ah, thanks for the clarification. I agree, organization is not optimal right now.


@Jorch95 esastamente monsignor :+1:


This lloks similar to the queue logic. I think you should try to extract this to util.c. I thing of functions like `void util_file_push(const char *file, const char *line, unsigned int maxitems)`, `char *util_file_pop(const char *file, int *items_count)` and possibly `void util_file_unshift(const char *file, const char *line, unsigned int maxitems)`. There could be used for the closed pages file as well as the current `bookmark_queue_*` stuff.


need commas before "first create" and "then push"


You can actually avoid `if not CUDA_AVAILABLE`, but I suppose its good for the informative error message.


Is this needed?


This unfortunately works, but the officially sanctioned API is to pass it into the super constructor call 4 lines above, as [`persistentOutput: true`](https://github.com/broccolijs/broccoli-plugin#new-plugininputnodes-options).


I don't think you did something wrong. It's one of those horrible gdb errors that give you no clue about what's wrong. I think gdb uses a shell to start applications (so you could pass arguments that require shell expansion) and this seems to fail. I'm guessing that your SHELL variable is not set or set to a file that doesn't exist.

I don't know what shell Ubuntu uses, but let's try to find one that works. Run `which bash`. If that returns something - says /bin/bash - then we can use that. Otherwise try to run `which sh` and see if that returns something non empty. Now, do something like this:

`echo $SHELL` (I'm expecting this to be empty)
``export SHELL=`which bash` `` (or basically any shell that works)
Then run gdb again as above.

And I think we'll have another problem eventually. Your gdb reports 'no debugging symbols found' for genrev because you ran cmake with -DDEBUG=0. So any stack dump we'll get will not show readable function names and will be pretty useless. You'll have to run cmake again with -DDEBUG=1 for this.


> The new Params feature is just great!

Glad you like that, it's a pretty handy feature (although I'm biased as I wrote it)

> The only thing that is missing are plugins for the most common Continuous Integration systems like Team City etc. I wish I had more time to write one.

@adamsitnik could you [post an issue](https://github.com/PerfDotNet/BenchmarkDotNet/issues) about this? I think it's a really good idea and I'd like to get a few of the other devs involved in a discussion about how best we can do it.

Having said that, take a look at [this code snippet first](https://github.com/PerfDotNet/BenchmarkDotNet/blob/master/BenchmarkDotNet.IntegrationTests/PerformanceUnitTest.cs#L24-L31), it shows you how you can run BenchmarkDotNet benchmarks inside an XUnit test and have the test fail if one benchmark is slower than another on or slow than X ms.


@kingofgotham @Jorch95 No creo necesario implementarle un back end a la grua....... Que interaccion tiene con felix, niceland y todo eso?? No es solo grafica la interaccion??


no - will remove - that was from attempt 1!


should just define this once..


There is a bug with the footer style. When there is not enough content above, it is displayed just after them and not in in the bottom of the window.


I might be able to without the JSON on success, but it will be there on failure, so it's probably better to just require it.


I was more thinking of the situation that `dtype` is a floating point type but still not supported. On the other hand, the exact same error will be raised in the next step when the space is created.

Regarding `CUDA_AVAILABLE`, are you sure? Looking at this isolated code, I'd rather say that you get the less informative
```
TypeError: cannot create 'NoneType' instances
```


Anyway, I've succeeded in rewriting the YuvSourceFilter part. It's now implemented in a somewhat more sensible manner. At the bottom the chain now looks like:

TrueSourceFilter -> RgbFilter -> VideoSourceFilter

Where the RgbFilter is skipped if possible.

I don't think we'll need to worry too much about the situation with multiple SourceFilters, because I don't think it's a good idea to allow that to happen in the first place. To get the class to function properly you need to change the RenderScript Descriptor, so only RenderScripts should be allowed to create SourceFilters.

Anyway, I fear I won't be able to add resource management for the Input RT today. But I'm also not too sure anymore what our goal is. Do we want to reuse as much memory as possible? Or do we want to prevent this (possible) copy to the OutputRenderTarget?


Don't forget to change STR_2367 as well.


@evil-at-wow
mangos@SNOWFALL:~/build$ export SHELL=`which bash`
mangos@SNOWFALL:~/build$ echo $SHELL
/bin/bash
mangos@SNOWFALL:~/build$ cd ~/
mangos@SNOWFALL:~$ gdb build/genrev /home/mangos/build/core
GNU gdb (Ubuntu 7.7.1-0ubuntu5~14.04.2) 7.7.1
Copyright (C) 2014 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i686-linux-gnu".
Type "show configuration" for configuration details.
For bug reporting instructions, please see:
<http://www.gnu.org/software/gdb/bugs/>.
Find the GDB manual and other documentation resources online at:
<http://www.gnu.org/software/gdb/documentation/>.
For help, type "help".
Type "apropos word" to search for commands related to "word"...
Reading symbols from build/genrev...done.
[New LWP 4572]
Core was generated by `/home/mangos/build/genrev /home/mangos/mangos'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x401a63b0 in _IO_new_fclose (fp=0x94ce0f0) at iofclose.c:57
57	iofclose.c: No such file or directory.
Traceback (most recent call last):
  File "/usr/share/gdb/auto-load/usr/lib/i386-linux-gnu/libstdc++.so.6.0.19-gdb.py", line 63, in <module>
    from libstdcxx.v6.printers import register_libstdcxx_printers
ImportError: No module named 'libstdcxx'
(gdb) bt
#0  0x401a63b0 in _IO_new_fclose (fp=0x94ce0f0) at iofclose.c:57
#1  0x08049d6f in extractDataFromGit (filename=..., path=..., url=false, data=...)
    at /home/mangos/mangos/src/tools/genrevision/genrevision.cpp:224
#2  0x0804a51d in main (argc=2, argv=0xbf9ede54) at /home/mangos/mangos/src/tools/genrevision/genrevision.cpp:326


TeamCity ReOpenLDAP :: next [Build 123](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1249&buildTypeId=ReOpenLDAP_Next&guest=1) is now running


Thanks Bert! This made it possible for me to serve the GUI through an nginx-based reverse proxy from a subdirectory. For those interested in doing the same, this is the relevant minimal nginx config:

    location /dnsdist/ {
        proxy_pass http://my.dnsdisthost.goes.here/;
        proxy_set_header Authorization "Basic YWRtaW46YWRtaW4=";
    }

My nginx server handles authentication for all content globally using LDAP. The authentication as required by the dnsdist GUI has therefore been silenced in this config by feeding the Authorization header statically from the config. In the above config, the password that is used is "admin". So this would work for a dnsdist instance that is setup using something like:

    webserver("0.0.0.0:80", "admin")

When the authentication must not be silenced, simply leave out the proxy_set_header line.


NaDownload mo ba yung database


nice one, I hate comma first and I'm learning a lot with the express code XD


We may need to wrap this in an object, or at least have a pre-created RVG. At the moment, I think it may return the same sp every time BitEncrypter is run since we are creating it with the same seed each time.


Sorry for changing CharSequence to String. CharSequence is indeed nicer


Missed some refactors... forgot to check for value in strings


Hm that is interesting. Well it still compiles with VC++, so if this makes the difference of compiling or not, then I'll just accept the pull request


Needs a "return;" to avoid fallthrough


jajaj yo no se que es esa grua ni si quiera jaj @kingofgotham @kriox26


I will miss efs :< @Kureev


@grillbiff

I have asked several times to correct your code, but you don't reply it.
            dispatch_async(dispatch_get_main_queue(), ^{
                reloadIndexPaths();
            });
==> reloadIndexPaths();

Why do you call reloadIndexPaths in async, not call directly?
Is it impossible, difficult or incorrect?(direct call)
And when will you merge it to master branch?

Anyway, your code is excellent.

Thank you for your help.


@kingofgotham @Jorch95 para mi que se drogo con los medicamentos para la gripe y necesitaba una grua que lo levante de la cama entonces la programó


Thanks
Please check the owner of package contents after installation as well
ls -l /usr/share/starcal2


Doesn’t work because because of the username in some results


i think the problem is with passing variadic arguments to macros. I'll check how VCC does it.


@caiocutrim me too ;)


you could use `group:tree` instead of hard coding every single "tree" node


If we fail in the middle of `this._applyPatch`, then the current build fails (and that's OK), but also all subsequent builds will fail, because `this._currentTree` no longer matches the actual directory contents. In other words, the instance ends up in a "wedged" state. It seems we should be able to recover from this without restarting the server.

How should we handle this case?


@mkmcc You may want to have a look to the `xml` library anyway, it could make it easier to parse HTML code.


Changed IntersectionConicConic back to IntersectConicConic in six places and instead added a geoMacros.IntersectionConicConic function for that purpose.


buena!!


If I remove it, I get a syntax error..


Hi @justinedelson! Came across this project while researching and was wondering if there was a cleaner way for our config to take precedence over this config (deployed by default if the publish instance was started with sample-content).


didn't know about ```expect_output()```, that's gonna be a tad more elegant ;)


I know, but I want birch and oak to burn longer than normal wood, and the same with almost everything else; they're harder, too.


Если не-`void` функция завершается без `return`, возвращаемое значение не определено. Поэтому, если `convert()` отработала без ошибок, то есть не было выполнено ни `return err_from`, ни `return err_to`, ни `return freeze_temrp`, её результат неизвестен. Он может случайно оказаться равен `0`, `err_from` или любому числу. Таким образом, проверка через `switch` в `main()` теряет смысл.

Нужно явно возвращать `good` при успешном преобразовании температуры.


Перечислимый тип (`enum`) --- это не то же самое, что `int`. Если вы используете его для кодов ошибок, нужны `Error_funct last_error` и `Error_funct get_last_error()`.


Hmm, I think this file was just an example for how something like the sdetools_connector could be implemented, and isn't actually used. A couple of months ago before the sdetools_connector for apiv2 was implemented in sde we decided to leave this file in it's current state because we didn't want to get rid of it, but we also couldn't really do anything with it at the time. I guess now that we have implemented the sdetools_connector in sde we could technically update this file. But I think we should think about what the future of this file is before putting man hours into updating it. We should think about if it's clear that this file is meant as a sort of documentation to begin with, and if we want to spend future time maintaining it after every api change.


try/finally and make an empty current tree


Не ст*о*ит смешивать `enum` и целые числа-значения его элементов, хотя это и корректно. Одна из задач перечислимого типа --- как раз отличаться от просто числа (быть отдельным типом, быть различимым в коде).


Agreed, we should fix this but we'll have to coordinate with @ralphbean for this as it will break fedora-infra/fedora-packages otherwise


This might return a list of errors, not just one; they have to be separated with something. Whether comma or line. If <br> will break your layout then an output modifier can be used. eg.:[[!+error.message:replace=`<br>==, `]]


Nice :)


Вы упустили одно из условий задания:


@dreispt
i changed of `True` logic to `False` logic


El nombre debería ser getRenglonesUnificadosPorIdPedido


Во-первых, код после `return` не выполнится, во-вторых, зачем здесь возбуждение исключения?


В заголовочные файлы (почти) всегда необходимо добавлять стражи включения, например `#pragma once`.


This was me testing something. I'll delete it.


Thanks, I can see what's wrong now.

```
bool extractDataFromGit(std::string filename, std::string path, bool url, RawData& data)
{
    FILE* EntriesFile = fopen(filename.c_str(), "r");
    ...
    if (EntriesFile)
    {
        ...
    }
    else
    {
        EntriesFile = fopen((path + ".git/HEAD").c_str(), "r");
        ...
        fclose(EntriesFile);    /* LINE 154 */
        ...
    }
    ...
    fclose(EntriesFile);
    return true;
}
```

If the first fopen returns a null pointer, the code path takes the else branch and does a double fclose if the second fopen is successful. That's a very bad idea.

I don't know if it's expected to take the else branch in normal circumstances, but as a (temporary?) fix, try to change src/tools/genrevision/genrevision.cpp and remove line 154 (see code above). I'll let @namreeb commit a good fix :smile:


I know that. haha
Damn scala ;)


На будущее: заносить под контроль версий файлы, которые генерирует программа, не нужно (но файлы с исходными данными --- бывает необходимо).


when testing -> when using


A possible suggestion for fs-tree-diff: Rename the `'linkdir'` operation to `'symlinkdir'`, and similarly `entry.linkDir` to `entry.symlinkDir`. `'link'` evokes hardlinks to me, as in link(2).


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


Shall we just make this a typdef and perhaps a very light-weight demo_face_t?


This should also use lmNew and friends.


lmNew


@myfluxi please add

https://github.com/alviteri/platform-hardware-qcom-audio/commit/414ccb4b10e3a174456cd8c1ce5a62dff1def31a

I mean

LOCAL_CFLAGS := -DPLATFORM_MSM8974


Should this be set before you call runAll?


<3 for commit msg


Whoops again on #29 --- my understanding of JS scoping rules is shaky.

As far as I can tell, issue here is that after `var parseFile = require('../file').parse;` in exec.js, the `this` inside of `parseFile` becomes the `this` that's the top-level object of exec.js, right? Maybe if I said `var file = require('../file');` then later `file.parse(foo)` that would work?

I should have written a test to catch this before pushing. I'll look into the right way to write integration tests for express.js (which is the library you're using, right?) servers so that this kind of thing gets caught.


:heart:


Double underscores.


The other texture looked good but it was 64x. It would work well in the 64x version


Бинарные операторы имеют вид `T opeprator+(const T& right, const T& left)`. Как видите, они возвращают не ссылку, а новое значение, что логично: результат (1+2) никак не связан ни с 1, ни с 2. Вы же получаете ссылку на текущий объект `*this`, создаете его копию приведением к `Temperature`, изменяете копию и возвращаете ссылку на нее. Но копия будет разрушена по выходе из функции, и получится т. н. "висячая ссылка" (dangling reference) на уничтоженный объект.

Замечание относится и к оператору ниже.


Это сравнение некорректно, если различны шкалы, то есть функция не универсальна. К счастью, у вас как раз есть реализованный оператор сравнения температур в любых шкалах.


I tried finding the implementation for `'update'` in the fs-tree-diff package. Grep shows that it's mentioned in the README, but it doesn't look like `'update'` ever occurs in the module code.


Easier to use GGO_BEZIER?


First change


Perhaps name this glyphy_gdi32 or glyphy_gdi?  I suppose DirectWrite has different API.


Not sure why github for windows adds this.


Use Markdown formatting instead of HTML.


Se podría cambiar el parámetro, por el nro de Pedido.


Stray variable?


English nitpick: "Script Error" is probably a better title.


По заданию требовались беззнаковые двухбайтовые (16-битные) числа. В частности, потому, что представление в памяти знаковых (отрицательных) чисел неочевидно.


good


Код функций преобразования систем счисления вам не принадлежит (кто первый сдал, тот и автор). Перепишите его с использованием шестнадцатеричных констант вместо двоичных. Не забывайте применять *Format use AStyle* перед commit-ом.


thanks )


@vlj j'ai fait un peut comme toi pour Motorstorm, ça marche comme ça, mais je sais pas du tout si c'est le bon return que j'ai mis


I'm more scared of people doing this by misstake and then not getting an error where they would expect one.


Hey Chuka, try and personalise your README so its not just the generic weekend challenge one. As you've done with previous challenges, give a basic summary of the challenge - i.e. premise, the tech used and instructions for how someone can download and run it.


На будущее (и для практики): если вы только переформатируете код, особенно автоматически, выносите это в отдельный commit. Иначе полезные изменения (смену `int` на `uitn16_t`) трудно найти.


No you could do:

```python
if arr.dtype not in CUDA_DTYPES:
    raise
```

instead of

```python
if not in CUDA_AVAILABLE:
    raise
if arr.dtype not in CUDA:DTYPES:
    raise
```

the only difference is that you can give better errors in the second case. In the end, providing good error messages inside `CudaFn` would solve all of this.


Anyway, I'll add a pull request with what I have so far.

In that pull request I partially revert the changes here, so I could implement some fundamental changes to the way the SourceFilter works. I think the new way will make it easier to achieve whatever it is we want to do with the InputRenderTarget, but you might want to have a look first.


Did we agree this is the best approach?  Is that version even available on those old distros?


There are may ways to allocate memory. HeapAlloc is the Windows API function.

The preferred way in the Ironfist codebase is to either use C++'s "new," for objects, or to use the ALLOC macro defined in base.h . The ALLOC macro is good because it will store data about what line of code allocated that memory, which helps in debugging memory leaks.

Also, C++ is not garbage collected. Whatever choice you use, you will need to free the memory afterwards so that it can be reused in further allocations. The FREE macro is the one that corresponds to ALLOC.


@tammoippen I think "labels" here should be "labeled".


Yes


@elbosio do we have a test for this method? I'm not 100% sure which comparator is used to test the "in" if it's not explicitly defined on the class (inheritance from Comparable). If this checks by the "id" should work, but if this checks by the instance identity, if might return repeated measures. Just double check.


Seems cleaner than dealing with `nil` later on in the code.


Indeed.  @BeerfishWoW, try commit https://github.com/cmangos/mangos-wotlk/commit/b4b1dc22d6f1c6d58766221b0b0160af4a342474


Generally in software engineering, you want all code to be written in the same style. Local variables in the Ironfist codebase are lowercased so they can be readily distinguished from functions; I recommend following suit.


На будущее: в Git есть возможность поправить предыдущий (на самом деле, любой) commit перед отправкой на сервер. В TortoiseGit в диалоге commit-а нужно для этого поставить флажок *Amend last commit* под полем ввода комментария --- и commit будет не новым, а правкой старого. Это удобно, если вот так забыть добавить новый или изменившийся файл.


Check the date of the latest (official) commit.

On that fork, the last commit by dz0ny was in July 2013 and the last commit on Oct 2014. If you cloned a month ago, I'm sure yours would be more up to date - especially since you seem to have the `devel` branch.


I'm starting to believe cookies are the reason of all evil


Удалять `proparray` всё равно нужно, но после того, как удалены его элементы (освобождена память, на которую они указывают).


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


@elbosio for stuff like this, I would prefer to define a method boolean isActive(), so we don't need to do the comparison outside the Goal instance, and we keep the behavior inside the Goal class. What do you think?


It's okay


i don't think its better to convert it into List.


this is really old haha


amazing much wow


fabulous mhmhmhmhm


yeysysysy YO HI MOM


I was not aware of the `patch.js`, but somehow I ran into this problem even with the `patch.js`, which is why I think it may not cover all the cases. So I think we should use my fix as it is simpler and seems to work better.


LMAOLMAOLMAOLMAO i am really sorry hahah um????


I think we're missing a `'mkdir'` patch here. Here's a failing test case to demonstrate that we're dropping empty merged directories:

```patch
diff --git a/test.js b/test.js
index dc37a3f..3aba045 100644
--- a/test.js
+++ b/test.js
@@ -47,6 +47,22 @@ describe('MergeTrees', function() {
     })
   })

+  it('merges empty directories', function() {
+    return expect(mergeFixtures([
+      {
+        foo: {},
+        bar: {}
+      }, {
+        bar: {},
+        baz: {}
+      }
+    ])).to.eventually.deep.equal({
+      foo: {},
+      bar: {},
+      baz: {}
+    })
+  })
+
   it('refuses to overwrite files by default', function() {
     return expect(mergeFixtures([
       {
```


OHWAIT HI GUYS :'''''') what i dont even know
if you're reading this say mom


i am very sorry guys it's not my fault i was dared to spam your emails and i don't know your emails so n i c e


> Do we want to reuse as much memory as possible? Or do we want to prevent this (possible) copy operation to the OutputRenderTarget?

Ideally both of course. But I'd imagine speed over memory reuse would be sensible seeing as even low end iGPUs have access to 1GB of RAM these days.

> I think the new way will make it easier to achieve whatever it is we want to do with the InputRenderTarget, but you might want to have a look first.

I'll take a look.


I'm surprised that we could eliminate the conditional (L148 ff.) without breaking things. Aren't we pushing duplicate entries into the `result` array whenever a file overwrites another file?


Was not aware. Rewerting.


When I change the user to root, it works fine. (yes, I was able to run
./bin/kibana, but it is nice to have it running as a service)

On Thu, Nov 26, 2015 at 3:29 PM, akabdog <notifications@github.com> wrote:

> Can you do me a favor and try changing
> USER=nobody to USER=root ?
>
> Also, does kibana start fine when you run it via /bin/kibana ?
>
> I'm running 4.3 myself, so it should work fine.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/akabdog/scripts/commit/ea13f095d224db6f9466d15b9fd6e03c185f80bf#commitcomment-14646147>
> .
>


c'est l'image qu'on affiche quand on a gagné !


G'day @rmflight - thanks so much for this great series of blog posts & data. So you don't include the code for processing the raw .txt files into .RData binaries? Or am I missing something? I have an application where processing the raw .txt files is actually something I want to share in a vignette & write functions for. Would I be able to use `read.delim()` (with appropriately abstracted filepathing) as normal in a vignette?


Will revert and add deprecation warning.


I chatted with deadwolf again. A scope like this one would make stripper clips incompatible. If we want a scope, we would need to either forego clip-based reloading and go for manual feed (which we can fake by lowering the gun as we do for so many guns already), find a different scope with an offset mount, or scrap the scope entirely. It's your call.


I now removed the `CUDA_DTYPES` check since the `CudaFn` already prints a good error message.


+1 for `force`, in particular if it only ignored non-existing files, and would still throw if the file exists and cannot be deleted.


[birch, oak and normal wood burn for the same amount of time](https://github.com/kaadmy/pixture/blob/f602cb022fef8e76b86bae78f68c60e94ca87973/mods/default/crafting.lua#L789) or did you mean take longer to "cook"?

(they are gonna turn into the same kind of "lump_coal" so i don't see how "lump_coal"s made using birch or oak would burn longer than "lump_coal"s made using normal wood)

tbh, it would be much more accurate to turn them into charcoal than coal

but it's your game so do whatever you like.


Does this need to be reverted, as with technic/technic_worldgen?


Thanks @Rog360 . Fixed in https://github.com/ACID-Scripts/WOTLK/commit/bb7a7a5590b03b83b39aa338e0d96c39268365f8


I still have 4 lines, but that's due to the fact that NumPy interprets explicit `dtype=None` as `dtype=float`, obviously :headbang:


Indentation de mes couilles


Oops, I've just proposed an alternative [on the forum](http://forums.ppsspp.org/showthread.php?tid=16348&pid=113007#pid113007) :-/
I guess I'm too late ?


Note that we should use ES6 module importing.


Thanks! Can you tell me how to reproduce the problem that this change resolves?

It looks like your change on line 5084 initializes outMatrix with zero columns, but then line 5180 resizes it to the right number of columns. Is that right? I have two concerns: (1) Line 5122 assumes outMatrix has already been resized, and (2) Line 5180 is inside a loop that begins on line 5088, so it looks like it will end up resizing the matrix multiple times.


good catch, fixed.


`dis` reverted


Drive by code review!


I'm surprised this didn't get caught by fs-tree-diff. For non-empty merged directories, it seems we're missing an entry for the directory itself. That is, we get an entry for `'bar/somefile.txt'`, but not for `'bar'` itself. Shouldn't the diffing algorithm throw an error in this case?


Issue #26 also fixed.


Ok, I'm convinced.


This is related to issue #156


@cy6erGn0m Copy-paste detected :eyes:


uploads ディレクトリには何が含まれていますか？


@ccrs do you have that bug ?


Yep you are totally right!


ファイルがアップロードされた際のテンポラリフォルダになります。


26/11/2015
Primeira carga


accidental `npm publish`


O modal é adicionado em index, não em main.


Nice and clean


Something weird happened with the formatting a few days ago. Not sure what to do about it, changing it all back by hand seems annoying but is maybe what I'll have to do.


Is there a reason for this change? I'm not sure this ends up properly partitioning the segments. For example, if we have two sections (num_sect = 2), the for loop here never executes (range(1,1) is empty, since the upper bound in python's range() function is exclusive), leaving us with the partitions (0,0) and (0, num_segs).


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


work with new kiosk questioanre


Thx, this is fixed in 2b8c51a484de756a560e434607398bafcaed14cb


I actually wonder if the patch application, along with this kind of recovery logic, could be pushed up into fs-tree-diff. It seems completely generic.


Awesome! But may I ask the choice of `#define` instead of a `typedef`? And the commit adds lots of whitespace, do you care to remove it or should I do it?

Cheers!


"there must be a better way!" - well there is ... a friend recently told me about /usr/bin/watch to replace my whiletrue loop.


Hmm, right so if we prefer speed over memory then we should try to make the last filter render to the OutputRenderTarget. Which is probably best done by extending the ITextureFilter interface.

It is in theory possible to allow the OutputRenderTarget to be used as an intermediate render target as well, but that's considerably more complicated so it's probably best to ignore that for now.

And to be safe the InputRenderTarget should probably not be used as an intermediate render target if it's equal to the OutputRenderTarget. This would more or less avoid a situation where the InputRenderTarget is rendered directly to the OutputRenderTarget we can simply do nothing, except if the input is unchanged, in which case we don't need to do anything.


@garbas  This should be `self` instead of `pkgs.self`. PR for the fix: #11293

See Travis fails like these: https://travis-ci.org/NixOS/nixpkgs/builds/93419025#L173


Anyway I need to be up early tomorrow, so you're on your own for now.

Cheers.


I think we should avoid reusing Input/Output RTs for now as it is hardly worth the effort.

Anyway, see you tomorrow.


Does typedef work as platform dependent code? But to answer your question, I was just following Apple's example of using different, but similar classes, dependent on platform
`#if TARGET_OS_IPHONE`
`#define SKColor UIColor`
`#else`
`#define SKColor NSColor`
`#endif`

As for whitespace, not really sure why Xcode added that, but I'll try to clean it up quick.


Fixes issue #153


You know it better than I do, you can fix my rough debugging job. That line was failing on an instance between 1200-1300, so you could edit add_to_ARFF.py to skip to those hundred files or so and debug


## Just a tip

...for writing commit messages in public repositories.

Try to capture what concrete change you actually made. "Stablized algorithm." says pretty much nothing. "Today I wrote some code" would been almost as helpful. I am not trying to be rude, but I'm just trying to make the point that it's difficult to understand what you did.

I have no idea what the changes improved upon, apart from "Stabilizing" the algorithm. Of course I don't expect that "the algorithm" was shaking, but perhaps you meant something similar to:

## Objects wrapping around screen no longer shake

There was a bug where the ship and all asteroids were shaking across the screen.
I chose to stabilize this shaking buggy behavior by rewriting large parts of the algorithm.


TeamCity ReOpenLDAP :: next [Build 123](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1249&buildTypeId=ReOpenLDAP_Next&guest=1) outcome was **FAILURE**
Summary: Tests passed Build time: 01:32:42
### Failed tests
```
all LDAP tests for mdb/DQD-KFA: test063-delta-multimaster for mdb/DQD-KFA: <no details avaliable>

all LDAP tests for bdb/DQD-KFA: test063-delta-multimaster for bdb/DQD-KFA: <no details avaliable>

all LDAP tests for hdb/DQD-KFA: test063-delta-multimaster for hdb/DQD-KFA: <no details avaliable>

all LDAP tests for mdb/KFA: test063-delta-multimaster for mdb/KFA: <no details avaliable>

all LDAP tests for bdb/KFA: test063-delta-multimaster for bdb/KFA: <no details avaliable>

all LDAP tests for hdb/KFA: test063-delta-multimaster for hdb/KFA: <no details avaliable>

```


TeamCity ReOpenLDAP :: master [Build 35](https://binbug.bigbrother-matrix.ru/teamcity/viewLog.html?buildId=1250&buildTypeId=ReOpenLDAP_Master&guest=1) is now running


I got a question.. don't you find this strange that if consumer connection gets lost and AbstractSession.close() is called then it first calls ctx.close() and only after that connection.close()?
What I am after is that during ctx.close() it changes state and calls listeners, and one of them (from Camel-Smpp) tries to reconnect and only after it gets reconnected the control is returned back to AbstractSession.close(), which then calls connection.close().. And so it closes the connection which has just been re-established...  Sounds strange to me..


Need this decl to be in the supply block, or it will be shared between tappings.


Sounds good to me


I'll try to write something in the readme for it later tonight. Thanks for the suggestion.


Nice job Berkut. This file was totally not necessary.


Oh.... did this fixz the bug?
No wonder sound did not play.


Needed for side-effects.


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


why do we keep _export_path if it's only needed in the constructor?


This would basically cancel the effect of the last commit 1fbf4c4e337e2242b3e1d266edd6ec99b699e163

May we could build 4.5 extension only once or twice a week ?

Cc: @pieper


@LorenzMeier , @mcharleb : This problem with `updated` being passed un-initialized to `orb_check` which then doesn't explicitly set or clear it is pervasive throughout the code.

The current type of `orb_check` is:

```
int uORB::Manager::orb_check(int handle, bool *updated);
```

but of the 200+ uses of this method, I can only find about 10 instances where the return value is actually assigned to anything or checked.

I therefore wonder if it might not make sense to change that function to signature to:

```
bool uORB::Manager::orb_check(int handle);
```

to make that method much harder to use incorrectly.


Yes, I do realize this.

Lacking a proper automated mechanism, how about I take on the responsibility to manually update this script every time there is an update to the ExtensionsIndex 4.5 branch to build 4.5 branch first?

Basically I just want to get a working package of the Reporting extension on Windows to make sure it compiles and is working. I do not know a better practical way to achieve that goal. Once we have that package, I can take on responsibility to rever this commit.


Nice, thanks!

The versions we run the test suite against are 2.6, 2.7, 3.3, and 3.4, which you can see here: https://travis-ci.org/rollbar/pyrollbar/builds/93288659 . That's probably the right list to use here, and in the readme as well I suppose.


Why use a symbol vs bool?


if `_period_path == NULL` you are returning garbage from stack.


You actually don't need the extra parenthesis in `_period_path != NULL`. If you use it, then the alignment here is wrong... you should align the line break with the first parenthesis.


No need to go to such extreme. Your analysis makes sense. Given the fact RSNA is coming up and you need the nightly extension build on windows, I will simply revert 1fbf4c4 integrating your comment.

For now, would it work if on windows 4.5 build take the priority only during the weekend  ?


Исправляет ошибку
```
XrayWrapper denied access to property "then" (reason: object is not safely Xrayable). See https://developer.mozilla.org/en-US/docs/Xray_vision for more information. Note that only the first denied property access from a given global object will be reported.1 Dollchan_Extension_Tools.es6.user.js:7881:26
```
которая была замечена в том числе в #867
В чём суть: при вызове `resolve` тот проверяет наличие метода `then` в объекте (чтобы понять промис это или нет), но в лисе из-за давнего бага это делать нельзя, пока не обернёшь объект в специальный типизированный массив (при помощи функций `nav.getUnsafeUint8Array` или `nav.getUnsafeDataView`). Чтобы избежать этого результат чтения файла оборачивается в дополнительный объект, в котором уже и проверяется наличие `then` и ошибка не возникает.


Please add comments for why you didn't do this in libhardware.


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


I think it deserves a comment about ownership passing, since the constructor sets this variable and then the destructor is responsible by freeing it.


This would be great, thank you JC! Weekend only for 4.5 makes sense to me.


this comment is now out of place


why this change?
what's the value stored in NFC_NCI_NXP_PN54X_HARDWARE_MODULE_ID?


@tchaikov . I think i make a mistake.
-  if (r < 0)
-    return r;
+  if (r < 0 || exist == 0)
+    return !exist ? -ENOENT : r;
This should: return r < 0 ? r : -ENOENT;(if get_mangled_name return error, it don't set exist. But the default exist is zero) .This can pass test for no-root user.
But i still want to remove permission check in test because permission use for osd not for localfs.


oredered -> ordered


And according to our coding style `char *export_path` is preferred over `char* export_path`.


I like the way you did it better as i'd like to "assume" that -> construct implies closure while function may not (not that Julia makes such a distinction but who knows)


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


本仓库会陆续上传我的一些Python小程序，如下：

NPM： ping检测设备的存活状态，并邮件短信报警。
NCM：自动备份交换机、路由器的配置
UDT： 程序自动抓取交换机的mac表和arp表并进行匹配，通过web页面清晰查询IP-MAC-PORT之间的关系。
alllogscan：自动检测log文件中的关键字，发送邮件短信报警。
thresholdWarning：自动登录交换机、路由器、防火墙，抓取cpu，端口数值，并实现邮件短信报警。
ChangeVlanbyself：实现自助划分vlan的功能，让非网络人员也可以划分交换机端口，操作简单，内涵4层安全检查，杜绝误操作引起的网络故障。


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


@pydsigner this is NOT the major version, it's the protocol version, which has nothing to do with the manaplus version. see https://github.com/themanaworld/tmwa/blob/master/src/mmo/version.hpp#L43

bots just need to support the `MIN_CLIENT_VERSION` and tmwa will be retro-compatible with them


I will update the script accordingly then. Working on it now.

In parallel, I also spotted an other issue with the upload from the windows VM ... see screenshot below:

![slicer-windows-extension-build-are-not-published](https://cloud.githubusercontent.com/assets/219043/11414345/4b520a16-93c2-11e5-9549-c3280ea13995.png)


CORRECTION: The correct demonym for the Bahamas is "Bahamian".


Muito bom kleiton. HSUAHSUAHSUASHU


Vocês não acham melhor utilizar "resources :comments" no lugar de especificar cada rota? ao utilizar o resources ele faz isso automaticamente, e se você perceber, vocês estão especificando um comentário especifico para ser, editado, deletado, mostrado e atualizado quando se especifica o ID, quando fui rodar os testes de BDD do projeto de vocês eu consegui pegar todos os comentários mas a pagina dava erro pq as rotas de Editar, ver e excluir estavam com erro, pq vcs especificaram essas rotas para o ID: 1 só.


do you still have the one you coded ? - -


According to the constructors the only path that could be nullptr is the polarity path.  I'd prefer that all the paths are checked in the constructors if they are !NULL, and then we panic() here rather than when set/get_xxxx() is called (or just failing silently as is happening now).


Thank you for your service to our nation.


you are not printing anything here so you could just strdup()


test


Too ugly. Too many variables


Thank you for your contribution to our nation.


test inline comment


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.555.afa14067527f2c2615d99fcb65a224d59219b657](http://build.palaso.org/viewLog.html?buildId=66604&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:22:27


leave a message to test


Could you add this to the diamond.conf.example with the appropriate documentation


how does it work?


I think it would be better to rename the current class to to PWM_Sysfs_Base. And then name the "concrete" class as PWM_Sysfs.

Also, make the constructor of PWM_Sysfs_Base to protected since it's not intended for public usage... it needs a concrete implementation to figure out the paths (and due to it receiving char* it's a recipe to disaster to be public since if the user calls with a stack variable it would segfault when you call free()).


is it the same?


In the previous commit, if you follow what I said, renaming the PWM_Sysfs to PWM_Sysfs_Base and then adding the concrete implementation as PWM_Sysfs, then you don't need this commit.


Travis supports installing apt packages in non-sudo mode, which will be the default in the near future (if it isn't already) for free projects. It's already been whitelisted at https://github.com/travis-ci/apt-package-whitelist/issues/440.


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-MacOS-10.9-clang: OK (1968 of 1969 tests passed)

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


This is one example where splitting the commit is terrible for review :-(. You remove it here and add in the other making it much harder for review.


Hey David,
Please update your source to lasted revision and then apply your fix since doing rebase failed


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.142.afa14067527f2c2615d99fcb65a224d59219b657](http://build.palaso.org/viewLog.html?buildId=66606&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:33:27


## UX Text (v1.6.7)
We're turning off google search suggestions on the main file search field for **FrostWire Basic** users only.
The rationale comes from a lot of one-star reviews on Google Play, basic users complain about suggestions that cannot be found.

Hopefully this will not create an even bigger number of complaints from those users that rather had those suggestions as optimistic as they were.


@TrySound do you have any good ideas


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6882](https://teamcity.mesosphere.io/viewLog.html?buildId=81486&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:57


Note that the payload of an _Inert_INTNEG is actually a *positive* integer, so that you really ought to negate the integer.


Thing I just don't get is: why installing ZF 2.3.* if we already have a 2.4.* available for production?


You are apparently missing PXF and erlebrain, the ones that were in fact using the GPIOs under "linux ifdef" in the previous way. Maybe bbbmini is also using it, but I'm not sure.


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1974 of 1975 tests passed)

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


NFC_NCI_NXP_PN54X_HARDWARE_MODULE_ID is just a prefix of lib name so that system could load the correct library. (nfc_nci.pn54x.so in this case).

It is only used here when HAL pn54x source code is using. I think put here make it easier to find and trace source code


NFC_NCI_NXP_PN54X_HARDWARE_MODULE_ID has not been defined. Should be defined somewhere.
The id should be prefix of library name being built (library name is defined in Android.mk: LOCAL_MODULE := nfc-nci.pn54x.default).


草が生えるようになりました


another nice thing would be to call the pins by name rather than using numbers.


Seems @kmod suggest to use underscore_case for C function instead of camelCase.


Actually this is not correct!  This is a bigger conversation about references vs. objects but for now you should know that 'is' should only be used for True, False and None.

I would revert this change.


This not doing anything since the address is in each message.  It only makes sense when you are using the smbus IOCTLs.


101162956-IncorrectIDDError-4647 (rglazer2) - 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)


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


坑我，我还以为是汉语讲解架构呢


Can you move this before the ep.Gateway() check ?


We could delete this file?


So, is in this line can be changed to ==?
`if device is 'DHT11' or 'DHT22' or 'AM2302' or 'AM2315':`
to
`if device == 'DHT11' or 'DHT22' or 'AM2302' or 'AM2315':`


There are also some mistakes in indentation. I guess this was copied from elsewhere?

I prefer a slightly different implementation that I'm doing on the threaded I2C: https://github.com/lucasdemarchi/ardupilot/commit/60fca6fe387dfd48cb3d80b15b44f5d7b7a56cc7


Done.


Дублирует строку 46


Not related with this PR, but we probably should write `thrift-maven-plugin` sooner or later.


Will try in my free time and open source it.


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


I don't mind reviewing & merging your PRs, but for the private vars, you'll just have to push directly to the private repo.


This actually fixes a bug where *sometimes* the tempdir would be deleted when it shouldn't be, causing much havoc (i.e. abnormal exits)


Could we use the same package with the ttext implementation? i.e. src/test/resources/com/linecorp/armeria/common/thrift/text


Nice! Another board using the sysfs implementation.


@TheNikiz https://github.com/TheNikiz/android_kernel_samsung_hawaii/tree/master/arch/arm/mach-hawaii 3.10.x does not have my kyleproxx board source, it would take a lot of time to implement it to the new source and I doubt it will work


seems like an accident


I wonder if this is for debugging purpose only. What prevents people from using it as an alternative to REST and TJSON?


wow.. so many boards with inverted polarity for on/off :-o


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


才开始，慢慢来


We could make this `final` and set its value only once.


from, for or of?


完全看不进去，看到英文 代码 就抵触


Any reason `addBrowserPrefixedStyles` needs to be promisified?  What happens too quickly?  At minimum I'd hope you could get by with `setTimeout(fn, 0)`, which is also a macrotask.


I added second language to config/cockpit/i18n/xx.php
Switch in user account, and it doesn't work.


Isn't it more traditional to use platform-newlines for this?


When the value is null, it will also throw NumberFormatException and the
warning log will tell us that (value is null)

On Thu, Nov 26, 2015 at 12:58 PM, Sean Flanigan <notifications@github.com>
wrote:

> It would be good to avoid a warning for the common (non-exceptional) case
> where the system property hasn't been set.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zanata/zanata-server/commit/45be747238b7b9fcf4300ce5ea7206b4cac6375f#commitcomment-14631385>
> .
>

--
Patrick Huang
Senior Software Engineer
Engineering - Internationalisation
Red Hat, Asia-Pacific Pty Ltd
Level 1, 193 North Quay
Brisbane 4000
Office: +61 7 3514 8278
Fax: +61 7 3514 8199
IRC: pahuang
github: github.com/huangp
Website: www.redhat.com


When the initial viewDate is an invalid moment, it breaks the datepicker. Taking the invalid viewDate's date and converting it back into a moment fixes this.

This commit also fixes the failing DateTimeField test. It now reflects the changes in commit 064e18d4242729186aadfe95a18fa4a9b3fdf6ba where the defaultText is shown as a placeholder rather than the input's value.


@jonas should this be moved to a makefile target as well? or should this recipe only ever be run in the context of travis (in its entirety) and never anywhere else?


:+1:
This actually fixes #2968 on first installs.
Let's do the same for upgrades ;)


Where does NFC_NCI_NXP_PN54X_HARDWARE_MODULE_ID come from?


for (WebElement currHotel : hotelList) {
-------》》change currHotel to currentHotel


That should actually probably read:
```python
if device in ['DHT11', 'DHT22', 'AM2302', 'AM2315']:
```
----
Here's why:

```python
if device is 'DHT11' or 'DHT22' or 'AM2302' or 'AM2315':
```
Truthiness for strings in python is pretty standard:  ```bool('')``` is False, every other string is True so we get:

``` python
if device is True or True or True or True:
```

This always evaluates to True

and as long as ```device``` isn't an empty string it will look like this:
``` python
if True is True or True or True or True:
```
This also always evaluates to True

The ```is``` vs ```==``` distinction can be subtler.  Check these out:

http://stackoverflow.com/questions/132988/is-there-a-difference-between-and-is-in-python
http://stackoverflow.com/questions/26595/is-there-any-difference-between-foo-is-none-and-foo-none


done


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (900 of 901 tests passed)

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


private InternationalTravelPageObj travelPO;
----- 》 》 change travelPO to travelPageObject


`losely` :arrow_right: `loosely`


I believe that either start and end will both be None or neither will, so if you want the whole span in the None case you can change the 0 in the unwrap_or case to input.len() - 1. Or you could match on the input.find results.


よろしくお願いします


@jonas where is this `GH_TOKEN` coming from?


SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
----》》change sdf to simpleDateFormat


@jonas good stuff. gives me ideas on how to tweak my doc push script for personal project [here](https://github.com/suhailshergill/research/blob/master/travis/travis-doc-upload.sh)


It's a little strange that these rejection notices are set and retrieved through the helper class.  Maybe the rejection notices could be pulled out into constants,
```js
return reject(errors.NOT_COMPATIBLE)
```

Or the helper functions could simply return the appropriate error, and you could do an `instanceof Error` check around here.


That makes sense, and interestingly I should have realized I was testing for truth. I tend to make mistakes late at night when I find something I've been working on finally runs. I should really test conditions more before considering the job finished.


This seems incorrect, shouldn't this be `Meteor.settings.public.intercom.apikey` as mentioned in the README above? I'm about to push a fix for this, but thought I should mention here.


What's the reasoning behind animo extending its helpers rather than just using the helper methods?  Currently methods like `generateRandomCharacters` appear to be in animo's public API.  Maybe some of the helper methods should be an animo base class, and others (i.e. `generateRandomCharacters`) should go into a totally separate module.


*too


アクセスの仕方むずくない？


Is `AnimationEnd` guaranteed to eventually be called at this point?  Is there anything that could prevent the animation from occurring or could prevent the end event from firing?  Just making sure there isn't a missing case that could cause a forever-pending promise.


@WDLmaster #121


Unintentional extra whitespace?


@jlduran I'm guessing it's not really using AES-GCM in phase 1, at least I've never seen that work when configured. RFC 4106 does not specify use of AES-GCM for phase 1 and recommends use of AES-CBC with the same key size as used for AES-GCM. Not finding anything newer than that which specifies otherwise.

What's your 'ipsec statusall' look like? If there is actually some circumstance where that works, I'll bring it back.


Fixed several indentation issues, removed a few lines of non-functioning code, renamed css classes to start with oppia-, alphabetized css atrributes, etc.


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


Consider using `document.createElementNS()`.  Might not have to worry about `xmlns` below that way.


Cheery picked yours! Thanks for reporting this and sorry that I'm very slow.


I guess we just needs one... Let's use InvalidInput for Txn too.


写得好啊！


Sorry for the weird commit message, I was in the dark that we need to get the latest commits from the server before being able to push ours.


Why this is an input error?!

This should be a bad connection.


This should not be an invalidInput error.


我也是


ah ok


this error message is unclear. `txn command does not accept arguments `


This might not remove the event listener as desired since `super.once(onComplete)` (the added listener) and `onComplete` (the attempted-to-be-removed listener) are technically two different functions.


use exitWithError?


As an alternative to changing the return type, it might be preferable to change `orb_check` to explicitly set `*updated` to `false` before calling `px4_ioctl`.

```
int uORB::Manager::orb_check(int handle, bool *updated)
{
	*updated = false;
	return px4_ioctl(handle, ORBIOCUPDATED, (unsigned long)(uintptr_t)updated);
}
```


Is this `map` really being used as a `forEach`?


Test


@Shiandow
I think this works well now.


Rejections should be error objects rather than strings.  This helps you get useful stack traces, among having other benefits.


```
$ ipsec statusall
...
con6[110]: IKE proposal: AES_GCM_16_256/PRF_HMAC_SHA2_384/MODP_3072
...
```

```
Router# show crypto ikev2 sa detailed
...
Encr: AES-GCM, keysize: 256, PRF: SHA384, Hash: None, DH Grp:15, Auth sign: PSK, Auth verify: PSK
...
```


Thanks


What does "sed -i.bak <> <>" do? Just curious.


Do we need to worry about the permissions/chmod of plist file?


Can any of `degrees`, `duration`, or `blurAmount` be a falsey value like `0`?


hh


Curious again: You are starting the echo with ${red} but closing with ${NC}. Why so?


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


buck - the installer breaks here in the recovery log


you can remove the "-r" option since it's not recursive anyways.


Hi @twiecki , would you please fix cumulative.py LIne 393, in which sharpe_ratio is called.


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


I guess unrar for OSX hasn't been updated to 5.30 as beta3 still has 5.21


This also secretly fixes https://github.com/levlaz/braindump/issues/6


This fixes https://github.com/levlaz/braindump/issues/1


A couple of hard restarts and everything's fine now. Don't know why it happened though.


I think I understand– if you change styles multiple times in a row then you'll only get the last set style rendered.  I still _think_ using `setTimeout(fn, 0)` should break this up appropriately.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 6890](https://teamcity.mesosphere.io/viewLog.html?buildId=81498&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Execution timeout (new) Build time: 01:00:10


How does this work?  Is it better than testing `element.localName === 'img'`?


I used to get some dependency resolution bugs without clearing .SUFFIXES. The makefile was probably just broken, but something to keep an eye out for.


Problème: imports relatifs à com.asystan.common.cayenne.* introuvables.
Solution appliquée: redéfinition d'une librairie com.asystan.common.cayenne_new, identique à la précédente.
Réimplementation des méthodes "RedirectCallBack()" et "CompositeComparator".


Really?


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


Fooey. I've no idea how I managed to get this commit in here.


Problème: imports relatifs à la librairie com.common.cache.* introuvables.
Solution appliquée: redéfinition d'une librairie com.common.cache_new, identique à la précédente.


just nicely formatted output with colors. Liked how it looked in my terminal


@Pokechu22 if your local install is working, do you mind confirming / denying the note I left for me?

Just report a subreddit via the browser, then in `reddit-shell` get the subreddit object by it's name, `_load()` it if it's not `loaded` and then get the `reported` attribute. If it's 1 (or greater if you reported the sub before) that means I had an issue on my install, if it's 0 that means the note is valid and I'm very confused as to how what I did fixed the issue.


@mekolat Ah, that's good to know. Will fix the comment.


:eyeglasses:


I don't think we should change the interface from the jdk interface, since part of the point of re2j is to be able to just replace the regex library you're using from java.util.regex to this and for things to just work.  We can have methods that take slices directly, but we should also keep the other version and convert the input and output as needed.


thanks~


Is this because of having multi byte characters?


I want to trigger shutdown of application based on some condition. I don't want to abruptly stop the thread which does file discovery part, instead in "scanIterationComplete" I would like to check for trigger and then stop the thread from processing and then throw shutdown exception to terminate application.


why did you get rid of the end parameter?


You can move this to a _samlconfig.php file in the realme module, and use http://stackoverflow.com/questions/18670767/php-composer-to-always-load-a-class to load it on every request (even requests that go to the vendor/simplesamlphp/simplesamlphp/www dir).


101162956-IncorrectIDDError-4647 (rglazer2) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1438 of 1439 tests passed)

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


For those wondering, this api key has since been deactivated :P


这个 include 有点奇怪啊，这个例子演示的是从 fetchedComment 上获取 post 的完整数据，和 author 没什么关系呀。


spaces after commas

what would roger say :(

@mogest


Might as well do `m.groups(0)` here?


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.556.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66607&buildTypeId=bt323) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 930, ignored: 56 Build time: 00:26:08
### Failed tests
```
Chorus.Tests.dll: Chorus.Tests.ChorusSystemTests.CanMakeNotesBarWithOtherFiles: <no details avaliable>

```


It looks to me like the caller of mergeInfillLines, GCodePlanner::writeGCode gcodePlanner.cpp#467 could pass the travel speed as the speed parameter.  Then instead of maintaining the flow by doing the speed_mod, it just hits the cap of 150.

With my current settings, I believe http://www.thingiverse.com/thing:603412/#files hits this case using 0.4mm width lines shell thickness set to 1.2mm.


[Magic numbers](https://en.wikipedia.org/wiki/Magic_number_(programming)) make code much harder to read, and make refactoring a lot harder.

I'm assuming that if a card has a value of `1`, it is an Ace - consider having a `Card::VALUE_ACE = 1` constant, or a `Card.ace?` function.


* ignored so many data
* any wildcard?
* etc


Too bad that github's review supports only line level note. Gerrit supports start/end line/character.


This is just a test.


Quotes unnecessay


It could visually be more clear if there was a new line between setup and your assertions


A quick code review - mostly nitpicks on whitespace. There are a lot of curly braces without spaces in-between their inner expressions, which makes it a little less readable.


b1 and b2


Just to be clear, it's the case here that the catch block has no real protected region, right?

As an aside, what possible reason could a code generator have to emit such code?


This makes me wonder if we are actually handling this case sanely in the intrinsics. It seems that we currently rely on the arch's implementation of the `OP_ATOMIC_EXCHANGE_...` opcodes to do any null checking, which in all cases seems to translate to just doing the operation and letting it fault if the location is invalid. This isn't gonna work on targets that need explicit null checks, is it?


In my earlier comment, I believe I mentioned that you don't need to have these extra `expect(compileAndDigest).to.not.throw()` assertions. Please remove this line.


Also remove this line please.


hao to use


minor: spelling of handlebar


you got rid of rune_any and rune_any_not_nl.  Are those just special cases of rune/byte that are no longer useful?


apart from one comment +1


related to #16


No new line at the end of file


:heart:


Don't use acme in our project


101162956-IncorrectIDDError-4647 (rglazer2) - Win64-Windows-7-VisualStudio-12: OK (1974 of 1975 tests passed)

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


Excellent use case for a [Driver](https://github.com/ReactiveX/RxSwift/blob/master/Documentation/Units.md#why) instead of an observable :wink:. It fulfills both [catching errors and returning something](https://github.com/ReactiveX/RxSwift/blob/master/RxExample/RxExample/Examples/WikipediaImageSearch/ViewModels/SearchViewModel.swift#L39) and delivered on the main thread.


`convertBase10()` can convert base 10 numbers to anything up to base 16.


ok i'am fine with this approach, but please change it like this:
```
+ ifnames="$ifnames \"unresolved:$interface\""
```
(so its clear while reading the generated config file what happend)


Shouldn't it be: preg_replace('/rename_|remove_|modify_|delete_|add_|_table|_tbl/gi', '', $action);?
...And... where is $action defined?...


typo errors. this is the correct commit name: "scripts: No more dirty"


You changed this from HANDLE* to void*, when it should have been to void**.
https://github.com/project64/project64/commit/3024d7c524fccb06a5cef15fbc54476f4dcc60da#diff-b42cbfe6842f2af5a46ee723220ae2b1L40

Maybe you did this on purpose not accidentally, as you also changed this:
https://github.com/project64/project64/commit/3024d7c524fccb06a5cef15fbc54476f4dcc60da#diff-edc655eec1617ecce451fcaa21ef17f9R371

Either way accident or no accident it causes Frank74's #776, so I think I will propose offering to revert this part.


Thanks!


revert or test per #22416


Oops, I accidentally typed an "L"

Sorry, I was on my iPad XD


Oops... Thanks for noticing :+1:


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.143.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66610&buildTypeId=bt2) outcome was **FAILURE**
Summary: Execution timeout (new); tests passed: 797, ignored: 45 Build time: 01:01:09


test commit.


continue.


@RobertCNelson I've just had a thought. For some reason the UARTs are being allocated very high IRQ numbers and so are bottom priority. Is there a way of setting these higher at all?


1 minute is probably too often for ~O(GB) storages.


@AndrKlesh
Сюда же можно добавить файлы .idea
# IDEA project files, because
# the project can be imported from settings.gradle
.idea


As discussed, this step is probably no longer required.


Can you verify the expected outcome instead of catching and ignoring the exception, this is not a useful test if it just calls all posiible cases without defining the expected results.


As discussed, it's probably better for Gradle to place the files into the desired locations.


As above.


Sigo buscando la mejor manera de almacenar y gestionar la fecha en base de datos.... :(


MEant to say added ackLoss and ackErrors


Primed endpoint should now stream back the actual `install.sh` script text.


Primed endpoint should now stream back the zip file bytes.


Can you add a few more cases of non-ascii sheetnames which you would expect to work after these changes are applied?


I think that is different topic about precedence of properties file


untuk database sudah ganti dari dbhtpro jadi dbphpro


371 MB will be downloaded!.


please note that releasing package with pip depends is very difficult, so when ever you add pip based depends, please consider deb package based solution, specially repository like jsk_common where we assume most of users use this with deb install.

 https://github.com/ros-infrastructure/jenkins_scripts/issues/51


Вместо *fill_parent* лучше использовать *match_parent*. *fill_parent* - это устаревший вариант.


I like for


TeamCity pull requests :: netty [Build 756](https://garage.netty.io/teamcity/viewLog.html?buildId=3624&buildTypeId=pulls_netty&guest=1) is now running


it wasn't used either, so i've removed it


This empties your data


noted, and done. thanks


The line 11


That was an insightful comment. I found the article you linked to interesting and thought provoking. Thanks for sharing that.


The problem was not the missing Dom


@AndrKlesh @Omennight
Также как можно чаще применяйте автоформатирование кода, это позволяет улучшить читаемость кода
Комбинация клавиш кажется **Ctrl+Alt+L**.


As discussed, a wicked little `wireMockServer ?: createWireMockServer()` would work nicely here ;-)


Thanks


Worst commit ever! I demand vengeance! Return the ninja!


Классы-наследники компонентов в Андроиде таких, как activity, fragment, view и тд. должны иметь суффикс соответствующий компоненту. Иными словами, в вашем случае, класс лучше переименовать в *RadioSetWiFiActivity*.


You're passing None into rundata, so make it an optional argument instead:
Record('Host', ['ports'], {'rundata': None})


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


Should be updated so that it does not respond from the hard-coded `__files` folder but explicitly with primed responses.


Thanks for the information.
With no `build_depends`, there is no problem, isn't it? (I set python packages only as run_depends)


why is this necessary? Just call .start() on it elsewhere, it's not like you're adding functionality here anyway.


Should be a commit in libhardware.
I skipped this commit, then use "nfc_nci.pn54x" directly later


Should have a counterpart method that does not respond from the hard-coded __files folder but explicitly with primed responses.


As the number of containers increase you need to increase this memory


Should have a counterpart method that does not respond from the hard-coded `__files` folder but explicitly with primed responses.


hello


@AndrKlesh @Omennight
Открывающую скобку любого блока кода принято помещать на предыдущей строке, где находится соответствующая инструкция. При автоформатировании это исправится само.


_live and _sock to signify 'private/protected' variables that are not part of the class's API


first use github


to see if it remains stable once... jajaja.
@Zealar +1
@Sacrifice78 -1jajaja


全角。。


no, deb packaging system try to add all run_depend to it's depends.
so everytime I run bloom release, I had to stopped the process and manually
fix them

--
◉ Kei Okada

On Thu, Nov 26, 2015 at 3:09 PM, Kentaro Wada <notifications@github.com>
wrote:

> Thanks for the information.
> With no build_depends, there is no problem, isn't it? (I set python
> packages only as run_depends)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsk-ros-pkg/jsk_common/commit/560d701f62eddb2b914b4fe42d5d78e593f0aad9#commitcomment-14632618>
> .
>


不用第二个参数


It is intentional just to tell user what tags need to be used


默认就是 false


分号


as I understand, here the for loop should be interrupted but function should still run rest of the code ... after debugging for me it seems with 'return' the whole function is finished without restoring zway stuff ... with 'break' it doesn't


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


不建议把 var 放到 try 里面


101162956-IncorrectIDDError-4647 (rglazer2) - i386-Windows-7-VisualStudio-12: OK (1974 of 1975 tests passed)

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


wtf?


建议 MutationObserver 里面的处理函数定义到到 try 外面，否则可能造成误解


How about RundataCache or something other than just Cache?


'__ZWay' that you entered above insn't an array. As I know forEach could only arrays - so that ZWAY Restore has failed every time


The depth buffer format enums appear to be correct, but the color buffer enums are not. Of the defined values, bits 0-1 represent the pixel size (0 = 16-bit, 2 = 32-bit), and bits 16-18 represent the particular format (0 = RGBA8, 2 = RGB5_A1, 3 = RGB565, 4 = RGBA4). As far as I can tell, there is no RGB8 format for the color buffer (probably due to it being 24-bit, which does not appear to be a valid pixel size for color buffers). So the values should be assigned as follows

GPU_RB_RGB8    = 0x00002,
GPU_RB_RGB5_A1 = 0x20000,
GPU_RB_RGB565 = 0x30000,
GPU_RB_RGBA4 = 0x40000


For clarity, consider storing a number of valid types found, and breaking here instead.


Just one comment, otherwise looking pretty good.


Actually, for this use-case, this would be even better:
Record('Host', [], {'ports': set, 'rundata': None})

Then below:
self.hosts = collections.defaultdict(Host)
And then Track becomes:
self.hosts[host].ports.add(port)


Hmm, I'm afraid that I can do nothing for this.
I use the package released on deb (if it exists) and use that on pypi unless there is no released package on deb..


@AndrKlesh @Omennight
Идентификаторы для view и контейнеров следует писать в нижнем регистре, разделяя слова нижнем подчеркиванием. Кроме этого, лучше добавлять в начале имя файла разметки без префиксов activity_, view_ , а в конце имя класса. В данном случае, это будет выглядеть так: *main_discover_button*


This is copy pasted. You have no idea what you are doing.


I've changed type of '__ZWay' to array. For me that has made sense because, below your using _.forEach() that needs an array and of course there could be more than one 'zway' name ... if you still want to use object, we could also map the keys of '__ZWay' and do _.forEach() on them below


try to use deb package as much as possible, specially when you think your
package is really useful.
https://github.com/jsk-ros-pkg/jsk_common/pull/543
https://github.com/ros/rosdistro/pull/5824#issuecomment-56717671

--
◉ Kei Okada

On Thu, Nov 26, 2015 at 3:22 PM, Kentaro Wada <notifications@github.com>
wrote:

> Hmm, I'm afraid that I can do nothing for this.
> I use the package released on deb (if it exists) and use that on pypi
> unless there is no released package on deb..
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsk-ros-pkg/jsk_common/commit/560d701f62eddb2b914b4fe42d5d78e593f0aad9#commitcomment-14632704>
> .
>


I've changed the order:
first stop all modules, but ZWave at least, clear loadedSingletons, Remove API web server, send emit


rubocop would probably tell you that `return` is not needed here.


This is not fixing anything. Also, this is not related to


@AndrKlesh @Omennight
В имени файла разметки лучше указывать префикс, поясняющий к какому компоненту относится эта разметка. Например, *activity_main* (*fragment_name*, *view_name*, *adapter_view_name*)


I suggest to follow to AirBnb styleguide (https://github.com/airbnb/javascript) and don't mix few classes in one file


@josevalim Looks good?


I'm not sure which call of 'saveObject' is causing this but after some time ["data","code","message","error"] are also pushed into '__storageContent' array and then they will produce unneccessary entries during backup ... If I next time find out why, I'll try to change this.


On the other hand, if both arrays are supposed to have the same set of keys, the whole method can be implemented just as `new != orig`


It is infinite loop because there's literally nowhere to put the string. There will be undefined behavior.


i was thinking about it:
It's better if we cannot resolv *any* interface, to autoset the uci var olsrd.@olsrd[0].AllowNoInt='yes'

what do you think about it?


Ok, I created an issue. https://github.com/jsk-ros-pkg/jsk_common/issues/1246
Please list python dependencies concretely in there which cause problems.
maybe below ones?

- https://github.com/jsk-ros-pkg/jsk_common/blob/master/jsk_data/package.xml#L11
- https://github.com/jsk-ros-pkg/jsk_common/blob/master/jsk_data/package.xml#L12


are you looking for `strcmp`?


Also, your part is compensated already with couple of lines further down: `current = len(result) - len(kb.chars.stop)`


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


Hmm, merge commit, did you use `git pull --rebase origin master` ?
@itsanna


Mention or check that method is POST if there's a message, since you GET with a message.


These two lines are repeated quite a few times, can you refactor it?


NP: Now that the method does not know what the buttons are for (passed by parameter), I would call it just ```enable_buttons``` (although I would prefer something without "enable", like `set_buttons_status`)


wrong indentation


What the point in having `buttons = []` here? Does it make any sense to call the function without passing buttons?


Once a fin packet is received, we should not send more data using this socket, as the socket is closed. Changing the state to closed will make rust-utp reject requests from the user on this closed socket.


You rock!!! :+1:


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


What is this? Problem with tabs and spaces?


ok. will do.


Same than above


@AndrKlesh @Omennight
Следует указывать модификаторы доступа с минимальным возможными правами (областью видимости). В данном случае, для полей класса следует указать private. Если вы ничего не указали, это значит *default* ( *package-private*).

Поле класса следует именовать, используя принцип *lowerCamelCase*. Также лучше в конце указать имя класса view т.е. *discoverButton*


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.144.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=66614&buildTypeId=bt2) outcome was **FAILURE**
Summary: Execution timeout; tests passed: 806, ignored: 45 Build time: 00:55:43


I think this isn't a problem, as a read might just consume the internal buffer of already received messages. As such, there isn't really an invalid state, as it should be valid to have a state different than `Connected` right after a successful read.


TeamCity pull requests :: netty [Build 756](https://garage.netty.io/teamcity/viewLog.html?buildId=3624&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5713, ignored: 21; resource leak detected (new) Build time: 00:42:06


good catch


Indentation


Merge conflict accidentally checked in.


@AndrKlesh @Omennight
Инициализацию следует производить по возможности в одном месте кода. Как правило, для этого используют конструкторы класса, но в даном случае, нужно использовать метод жизненного цикла *onCreate()*