ideal :  1448.88778 +/-  83.22629    1401.86843 +/-  27.18782    1404.38681 +/-  25.35502
                  mb_hardcoded :  1558.87422 +/-  26.38986    1547.83999 +/-  29.43750    1585.06011 +/-  79.23123
              specialized mono :  1975.14865 +/-  60.61668    1975.08204 +/-  75.60153    1964.46347 +/-  45.05584
              specialized mega :  1957.83542 +/- 102.60030    1904.06425 +/-  12.10941    1908.42353 +/-  11.53038
                  generic mono :  1982.33960 +/-  65.19581    2235.39019 +/- 385.97410    2126.58638 +/- 287.39472
                  generic mega :  2003.73546 +/-  90.79997    1994.41181 +/-  70.34913    1967.10869 +/-  11.78608


im Library haben wir schon DefaultEdge und DefaultWeightedEdge Bro :)


Haha, ist das mies^^ Großartig :D


@DanielSperry thanks for the feedback! The StageInfo modifications were supposed to be immutable, I just didn't get around to doing the configuration parsing yet.

Using the inifispan cache seems like a great idea for checking the stage limits instead of using the StageInfo. The reservation requires a couple messages ("Hey! Reserve this activation for me!" and the responses  "okay, reserved!" / "sorry, couldn't!")  to avoid a race condition.

How should I implement that messaging without adding a custom message type? Singleton actors appear to be cluster wide and are unsuitable for this..


Yes of course. I'll do it. Just thought it is other issue.


@carlwatson this looks great, but let's try something else.


Yes, what I'm trying to say is that now you can remove "safety" check, as clipping already makes sure it's in the safe bounds.


Also, typically the user-facing implementation of node selection is referred to as a "router" so I'll go with that :)


doggg you gotta be more descriptive than just "updated it"


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2092 of 2201 tests passed)

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


dogggg


I would have put the condition down in :done before calling :write, but this works too.


@timrae   The layout-v14/ version of my_account_register.xml is not materially different from the default version of my_account_register.xml.

Any reason I shouldn't just delete ./AnkiDroid/src/main/res/layout-v14/my_account_register.xml ?

~/projects/Anki-Android$ diff ./AnkiDroid/src/main/res/layout-v14/my_account_register.xml  ./AnkiDroid/src/main/res/layout/my_account_register.xml
75c75
<             android:id="@+id/cancel_button"
---
>             android:id="@+id/register_button"
82c82
<             android:text="@string/cancel"
---
>             android:text="@string/sign_up"
86c86
<             android:id="@+id/register_button"
---
>             android:id="@+id/cancel_button"
93c93
<             android:text="@string/sign_up"
---
>             android:text="@string/cancel"
95d94
<


This solves only nonexistent file, what about other errors such as permission denied?


godddd


This is the only part I can really comment on. I presume pretty much anyone who is not using YouTube is using S3. My only suggestion might be to write down exact steps on how to do this in the AWS console.

But - actually - is this documentation for Open edX or edx.org partners?


@dubek That would be incredible if you were able to contribute something that could solve that! It's the only thing I'm unhappy with in the current implementation. We spent a good long while trying various ways to avoid that problem, but it gets really difficult due to restrictions in Redis/Lua scripting.

We have to block until at least the next millisecond as if you generate an ID with a sequence of 4095 and immediately after (and in the same millisecond) one with 0, the one with 0 would be ordered after the one with 4095 and that breaks our k-ordering guarantees (which basically say that we'll guarantee the IDs will be in exact order by time generated, and rough order across many nodes as the logical shard ID could be anything).

Ideally, you'd be able to just block for as long as it takes to make it until the next millisecond, but in Redis Lua scripts there's no way to tell when a millisecond comes (the purity restrictions of the Redis Lua scripts say that TIME, as a command that returns an unpredictable value, must be the last thing you call). So we have to block for at least 1ms even if we only need to block for, maybe, 100ns - which kind of sucks, but isn't the end of the world.

The first thought we had was that if we were to roll the sequence back to 0 on every millisecond that would at least allow us to generate a few more IDs per millisecond before blocking - but as it turns out, this is really difficult. The expiry of the keys in Redis isn't rounded to the millisecond like we thought - instead it can happen part of the way through a millisecond. Therefore, we will still end up wrapping part way through a millisecond and come up against the same ordering restrictions we had without the expiry (which is why we decided in the end to just drop it and keep it safe but slower for now).

Best case scenario would be that Redis added a new command that allowed us to set the expiry and check the TTL remaining for a key in nanoseconds instead of milliseconds - after (not much) pondering about it, I _think_ that  allow us to tell how far through the millisecond we are and would allow us to expire the sequence at exactly when a millisecond starts/ends.

I've created an issue at #2 for us to try and keep working on a way around this, so please keep putting any thoughts you have about this specifically in there as that would be so helpful!


Hello,

  The library is to be used to travsmit with RFM12B data toward a gateway node, see mysensors.org


Is ez_setup.py relevant?


esto es


I consent to the licensing transition


The styling of the (?) is configured for Firefox. This may not look so great on other browsers... I should optimize these styles for cross-browser compatibility.


Oops.


For qtoutv, you could probably put you and/or me.


Is it copied from my plugin?
https://github.com/szepeviktor/wordpress-plugin-construction/blob/master/mu-disable-updates/disable-updates.php#L201-L216


Facing the same issue as @calfzhou.  My swagger ui fails authentication for all requests when `description[:authorizations]` is added.  I would expect this to only be enabled with the `swagger` strategy and not the `default` strategy, that way my swagger ui can read the `access_token` from API Key field at the top of the swagger UI


I think there's a missing parameter for specular, at least most other materials have 4 parameters for ambient, diffuse and emissive, and 5 for specular (see also #606).


This is certainly faster, but this is also much less explicit than the older version. Since this model file is an example, I wonder if we should keep it as explicit as possible?


Cambiar color a tetera


Nice! I like this much more. I will pull down the changes soon and if all is good, will merge!

Thanks for holding on in there.


Intentionally package private?


null checks


rax retry tests


Hi @dbellettini, seen that you have no dick to do, help to understands events!!!


for more info:
https://developer.mozilla.org/en-US/docs/Web/HTML/Supported_media_formats#MP4_H.264_(AAC_or_MP3)


sounds right to me


also you can write this constructor in terms of the "this" constructor


Try `decimals : 1` in a series object to restrict to one decimal place when dragging.


Help ip adress


I guess we're carying this from some generated `.gitignore` for Python projects...


null check


Ok


As someone that's been saved from losing work by the `gco` commit, I'd like to advocate for its reinstatement. Won't someone think of the children?!


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1675 of 1675 tests passed)

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


CooldownMax twice?


Thanks for fixing this.
(note for myself: we should fix it in the backport PR)


Can you put some class description(javadoc)? Together with @since


hi, sorry for late response but my pc arrived at my cottage just now. first, my fix is not a proper fix, it just takes care of memory violations, and the commit message is wrong (that's not why it's wrong..)

1) that comment applies to direct calls to fft() fritsch, not twochanwithwindow. twochanwithwindow takes care of the offset'ing, https://github.com/xbmc/xbmc/blob/master/xbmc/utils/fft.cpp#L157
2) twochanwithwindow expands the data in the passed array,  https://github.com/xbmc/xbmc/blob/master/xbmc/utils/fft.cpp#L150. at the same time it applies the window to the data (that's the wn's). so yes, that ^^^ will do, but we'd only calc half the freq's we intended as you'd only use half the passed data.

what's going on in twochanwithwindow is called an even expansion. if you take the fft of an even signal, you get real coefficients. you can then reconstruct the rfft coefficients of the original signal afterwards, that's what is going on in the second for loop.


typo, thanks ;-)


Should this really return a concept uuid? :)


please, check #442

[this patch](https://github.com/balabit/syslog-ng/commit/335990dd1f3b32046fada18ed7dc9999e4d454ac)


you'll need to change the references to this file as well (I believe there's one in the getting started docs.) Would you mind moving this work to a feature branch?


Approved @maikany


What is the use of declaring a temporary uuid? Why not just pass serializedValue directly in the method call?
What is the use of declaring a temporary provider? Why not just return Context.getProviderService().getProviderByUuid()


Can you update http://collab.kpsn.org/display/tc/smart_scripts
with this?

What can you do with this action/event?


I switched it to use bootstraps alerts component.


actually, reading closer, that isn't the explanation. maybe you are correct, i'll try.


I see and concur.


Thanks


OK, the RHEL.7.1 anaconda works fine so I was probably just working around a pre-release anaconda bug of RHEL7. You can remove the commented part completely and merge the nicdelay if you want to.


88737258-EPW-Ruby-Access (jasondegraw) - x86_64-MacOS-10.9-clang: Tests Failed

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


This seems a bit odd to me. Why would a search need to be a POST? Shouldn't it be idempotent like a GET? Or are you creating something?


Can you include some class level javadoc? Together with @since


The repo reference is incorrect.

All patched jars should be in Payara_PatchedProjects and shouldn't need an additional repository referenced as the Payara_PatchedProjects repository is in nucleus/pom.xml


Wouldn't there ever be a case where you'd want the application to fail if it attempted to reconnect a bunch?


@chrisndodge, thanks, good question. This chapter is shared for both audiences. Partners who are using our new workflow for video encoding & hosting will most likely not encounter the S3 mime type problem as we handle the hosting for them. There is an additional guide on processing video files that is private for our partners only.
I have an aversion to documenting procedures for other companies' products, as they are difficult to keep up to date and to verify accuracy. I found pretty good instructions on the web here: https://foliovision.com/player/fix-amazon-mime-type, and on stackoverflow and several other sites, but I have no way to assess their accuracy. (I did not find a procedure in the Amazon doc set.) My thought was to give readers enough info to search on so that they could help themselves. Do you think I should provide a link to one of these other sites, or to a more thorough description of the problem like Adam supplied above?


No, majority of work is still database. My latest commits have been about refactoring out state, so you can use one instance of GER in a server, rather than initialising each request. And adding some functionality to recommend similar things to a thing, which uses all existing functionality.


hola comentado


Thanks @adampalay I will add this link. (It's written for a more tech-savvy audience than we typically aim at, but I think it's still a valuable resource!)


__API Blueprint syntax error on line 1551__: Undefined symbol Single Project Instances (Autogenerated message from apiary.io)


@hjmjohnson Needed to add a lock around this line because there are some race conditions happening inside this method. It appears the methods that cause these are dependent on the threadId so they aren't a problem between the higher level threads, only the tbb threads.


I can't update Wiki

You can use it for example for https://github.com/TrinityCore/TrinityCore/issues/6914
And many other cases, also it opens new possibilities how to script some things


This bit its not really necessary. We "shift()" because we only use the first item in the list, so there is no need to iterate since its just one object.


Can you also add some unit tests for this class?


@dgreensp over Slack:

> This particular destroyMembers is "extra", I think, in that it isn't required for proper clean-up, it just makes it happen sooner to prevent extra helper re-running (which can become a correctness issue in the presence of iron-router).  It's worth asking whether domrange is ever assigned *after* all the clean-up code is run, though.


you didn't use missing filter after all?


I'm here for this....to find some bugs ;)

Thanks


Perdon pero la verdad no sabia que solo las imagenes aceptaba. Coloque varias veces los archivos sin caso. Saludos Felices Pascuas!!!


Actually, @chrisndodge , what about this link to AWS concole user guide? http://docs.aws.amazon.com/AmazonS3/latest/UG/EditingtheMetadataofanObject.html


Thanks for bringing that up.  The stale entries are much smaller than the data itself, but they are memory wasted and should be cleared.  I'll look into wildfly-common and see if it's suitable for jberet standalone.


Very handy, I wrote alot of EVENTS and ACTIONS myself for cases, but this is also a good one.

Thanks.


Harihar,
  Sorry, I haven't posted the slg13 and akbc13 data yet. The code for both of these experiments is the same as iswc13. Let me explain the differences in data:
Our slg13 workshop paper used manually-defined weights on a different NELL dataset without training data. We manually labeled a small evaluation set for this problem.
Our akbc13 workshop paper partitions the iswc13 dataset into different slices, and we examine how partitioning affects running time and inference quality.

I'll try to post these datasets soon. Commenting on the commit is a bit awkward, so if you send me an e-mail, I'll let you know when they're on our server.


Should either run the BCC pipeline or return an error code (sys.exit(-1)  or something non-0) to let the user know that something went wrong


MaskImposter is an unfortunate name (but I feared that changing it would be too disruptive). It's an imposter for a non-existent plane; it returns the same value for any index. As such I believe it is clearly the right choice in this case, since we have no mask plane.


@khadijaibrahim


It should be OK to just leave those empty. https://www.python.org/dev/peps/pep-0376/#record


Теперь и после 5-и неправильных не показалась...


yeah, I'd be down.  I don't have a lot of time but I probably could give it a few hours on the weekend


@fdr the problem with volumes and bind mounting is that hsup may be running a different place (namespace, host) than the docker daemon, and volumes being bind mounted with `-v src:dst` are always relative to where the docker daemon runs.

The alternatives (that I can think of) don't look very good either:

1. instead of bind mounting the binary, send its bytes inside the container (stdin, pipes, sockets, etc) so that hsup can self-update. Need to still be careful to send the bytes for linux-amd64, in case the outer hsup is running on a different architecture (darwin)

2. make the docker image cache optional, and the default path is to always build a new image and bake a new hsup

3. make the outer hsup somehow discover the right path of the binary to be bind mounted where the docker daemon is. Maybe do it manually first, with a configurable (fixed) location, then try to auto-discover the proper hsup path where the docker daemon is. This still looks fragile, as there's no way to guarantee that an hsup binary will be present where the docker daemon is: just imagine hsup pointing to a remote docker daemon with `DOCKER_HOST=tcp://...`, or hsup running purely inside containers and not being installed directly on the host alongside the docker daemon.

Thoughts?


Careful.


Fair enough, thanks man!


And finally ported to TBC in cmangos/mangos-tbc@db363ecb8cb3cb0836be745b654e9aa877dda0a5 and cmangos/mangos-tbc@473aacb418d37a2a28d1c5b493ee8fa38ce98abb.


Я что-то пропустил? "maybeSome(['\s'])"?)))


У меня работает, например на странице http://linuxforum.ru/registration попробуйте шесть раз нажать на кнопку "Регистрация в сообществе", потом попробуйте шесть раз понажимать, например тут http://linuxforum.ru/forget


Well, wiki updated


Shouldn't "items" be spelled with a lowercase 'i' since %i is likely a number?


You no longer need the map if we change the get_levels to lower case


Good catch on escapes, a copy-paste relic. However the ."\n\n" where there for readability; looked better in that modal window.


Good catch!


o que talvez seja uma melhor solução para o middleware loca, é voltar  com a chave de configuração "local".

```php
'middlewares' => array(
     'global' => array(),
     'local' => array(
        'Application\Controller\IndexController' => array(
            'Application\Middleware\First',
            'Application\Middleware\Second',
        ),
    ),
),
```


I added ``content.history_content_type`` and tend use that - it feels more ducky than dispatching on type explicitly. It is a personal preference thing though.


I will actually enhance it on my local branch, so that I dont need to set the timer to a specific value. I want to increase or decrease it. (Every 5th Shadowbolt -> Corruption for example)

If I understand that action/event correctly.

thanks for the wiki update


Die tabel session weet ik niet of we die gaan nodig hebben. Maar da is een standaard tabel van codeigniter om sessies op te slaan in de database. Maar waarschijnlijk kunnen we het project ook wel maken zonder sessies op te slaan in de db.


Thanks @frewsxcv !


r+


тьфу, работает.. Просто разговор шел о трех...

> добавлена каптча (выводится с третей неверной попытки заполнения формы).


Add comment explaining what this does at a high level


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/7233aafb03da46078c8db3025987ed9d7071250a


merging Ms2ger/servo/MouseButton = 7233aafb into auto


Ms2ger/servo/MouseButton = 7233aafb merged ok, testing candidate = 4e89ec42


I think this needs to be vendored.


Awesome numbers! Looks very good Milos!


`w_p_fdn` corresponds to `foldnestmax`, which, AFAIK, should be between 0 and 20.
I don't understand, in fact, why the check about `flp->lvl` being negative after assignment. It shouldn't happen, IMO.
So, I'd agree to your proposed simplification using `MAX`.


This is perfect, exactly what we needed!


Изменил на 5, а то там такой бунт поднялся.


#include <cmath>


```
checking pkgs/top-level/all-packages.nix on i686-linux
error: while querying the derivation named ‘baloo-5.6.1’:
while evaluating the attribute ‘nativeBuildInputs’ of the derivation ‘baloo-5.6.1’ at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/lib/attrsets.nix":135:44:
while evaluating the attribute ‘nativeBuildInputs’ of the derivation ‘kfilemetadata-5.6.1’ at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/lib/attrsets.nix":135:44:
while evaluating ‘concatMap’ at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/lib/lists.nix":54:18, called from "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/lib/attrsets.nix":135:52:
while evaluating ‘resolveDeps_go’ at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/pkgs/build-support/autonix/default.nix":150:26, called from undefined position:
while evaluating the attribute ‘"${(dep)}"’ at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/pkgs/desktops/plasma-5.2/default.nix":41:7:
attribute ‘poppler_qt.override’ missing, at "/tmp/nix-build-nixpkgs-tarball-15.05pre1234.abcdef.drv-0/nixpkgs/pkgs/desktops/plasma-5.2/default.nix":41:20
```


form -> from


Just a convenience.  The group assignment is random so this lets us have say an "average" of 2.5 rows in each group.


Typo - it's the vision ZD2102


I think these two `when` calls could be collapsed into a single `if` (the latter `when` mapping to the "else" form).


one artsy coin for you sir

![artsycoin](https://cloud.githubusercontent.com/assets/49038/6972203/2c4642ea-d950-11e4-99a8-e60d933a12cb.png)


As always in your checkins, very helpful overview.


Ugh, really?  The whole point to this patch was to fix keyboard layout issues.  I tested it in Firefox 37 with various keyboard layouts.  Can you do me a favor and grab Firefox 37 and see if it corrects the issue?


Appending to the `List` in this way will result in a lot of `O(n)` operations as the list gets larger. It may be worth  prepending, then reversing the list before sending the batch out.


Doesn't // work for you? (I haven't tested gravatars yet)

It was done with intention to let the browser autoselect the correct http or https protocol for the external file link dependent of the parent page protocol.

You get a warning in some browsers if you mix https with http content.

Here is some explanation or google 'protocol relative link'

http://www.paulirish.com/2010/the-protocol-relative-url/


I agree calling `sortBy` should produce the same result as before, but I would feel much more comfortable to have the code written there. WDYT?


I do not see value for this method because the caller could the above statement before the loop and then just have the next line instead this method call.


why test both `borrow-*` functions?


We should leave one line of whitespace before the `import` s for consistency with the rest of the repo py files.


Why not just do this here? Set<FormField> formFields = form.getFormFields();


rax retry tests


The instead of the methods call, do formFields.contains(formField)


And even more, the function you're passing there becomes an `MbFunction`, so there's another layer of indirection. It would be much better to write the code yourself.


Well, locally it didn't work, so I suspected gravatars config to be missing, so var_dumped it, got undefined infex $fs, so I readded it, but it still didn't work. So I readded http and it worked. That is locally, dunno about the server. Will test now and if working remove it


Double buffer ?


smack_new_label_from_path may fail sometimes without setting errno e.g. when get_label fails inside it: Lets say we have set xattr "-" (which is forbidden smack label) on a file on a system without smack and then mount it on a system with smack. Reading this xattr will fail in libsmack, but errno will not be set. Would it?

I know this is quite strange case.
Btw: why should we avoid setting errno=0 inside a small binary?


Why don't you just return true without assigning the return value to a variable you will never use?


Why don't you just return false here?


Could you maybe put these in different files? Say `GenericTupleBench.scala`, `IdealTupleBench.scala`, `HardcodedMbTupleBench.scala`...? I wouldn't like these to interact with the other benchmarks (which are still the ones dating from 2012, when I first worked on miniboxing)


You are correct, it works online. My prod FS #1 is now using FS1.0 alpha 2


Same here, would be great to write the code, since the function becomes an `MbFunction` and all kinds of interactions may occur.


Changed the notes from A to Ad. The idea is Aa indicates Arduino Analog, and Ad indicates Arduino Digital. Or maybe it should be d for digital, a for analog?


Same here.


Cool :)

Ok, I will write code for quicksort by myself.

Also, what do you think about version with specialization, why is it that
slow? It is almost the same as for generics. Maybe I missed something there?

On Thu, Apr 2, 2015 at 9:53 PM, Vlad Ureche <notifications@github.com>
wrote:

> And even more, the function you're passing there becomes an MbFunction,
> so there's another layer of indirection. It would be much better to write
> the code yourself.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/milosstojanovic/miniboxing-plugin/commit/99c526b1ba275e3323f6a1208e84e0206bf20752#commitcomment-10553158>
> .
>


Which filter condition success? Can you make the method name more meaningful by stating explicitly what the filter condition is?


Looks like this is implemented but not released:

https://github.com/bbatsov/rubocop/pull/1475


(very minor) Does this print out like
```
WARNING:Some message
```
(i.e. with no space after the colon)?

Consider:

```print '%s: %s' % (STATUS_MESSAGES.get(status_code, 'UNKNOWN'), message)```

Because it also allows you to take the redundant colon out of all the status message strings.


Памылка `мучины`

:smile_cat:


`pass` to be consistent with my previous nitpick about empty classes


I spent some time poking at this, but compaction is not just a straightforward operation on a JSON-LD document; you have to provide a “context”, which we don’t have for this schema, and naturally [the spec](http://www.w3.org/TR/json-ld-api/#compaction) is unclear on how they work. At a bare minimum we’d have to write a context that gives a name to every property we’re interested in, which seems prohibitively complex at this stage.


my preciousssssss


config_file is always nil after this patch


So weird


Uhh... what's the idea of this commit, then? :|


Actually, this would not work. As the operations for opening the deeplink are async, this would always return true if you are on a mobile. I have planned to extend the api with the functionality you're looking for, please advice #6


saw approval from kuenishi
at https://github.com/basho/riak_cs/commit/a2c11f0cc21b7042bcf6e4bdaa8809043ccdda7a


merging basho/riak_cs/bugfix/known-issues = a2c11f0c into borshop-integration-1114-bugfix/known-issues


basho/riak_cs/bugfix/known-issues = a2c11f0c merged ok, testing candidate = 277e6d3a


The indentation changed due to the loop, so I can understand at first glance it appears I messed up the formatting.


Adjusted log message in new commit.


COMMIT提交过于草率，文件目录结构可以更新到READM中。

目前的代码结构过于复杂，且无相关注解，极不利于后期开发维护。

希望在下次更新重规划好目录结构，运行方法及相关注解也请写用MAKRDOWN语法写在README中

@chenqiuyuan


Not really. As the comment message mentions, it is intended to be used by
setting the `PIE` cmake variable to 1. We expect users who need that (e.g.:
Android 5.0 and up) to set it with `cmake -DPIE=1` when configuring.

On Wed, Apr 1, 2015 at 5:50 PM, strager <notifications@github.com> wrote:

> Seems unused?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sas/ds2/commit/05cda3e6a3f1a3b9649c5826f9cfb234ec1c13f7#commitcomment-10536729>
> .
>

--
Stephane Sezer


The `pass` statements aren't actually required. You'd normally only need these if you were doing nothing else: https://docs.python.org/2/tutorial/controlflow.html#pass-statements


Please update comment


Regarding the rename... I forgot to respond to this last night... I did that intentionally, for clarity. The variable name `platform` is used variously for the *name* of a platform and the `platform` object retrieved from `platforms`. I didn't want to add another usage, but use consistent naming for this new object throughout. It makes the diff more messy, but I think is clearer going forwards.


I learned something new.


this is old code


You actually need to call `fastLocalsToBoxedLocals` here rather than at the initialization of the frame object. You can see here that the dict gets updated with `f_locals` is accessed:

```
import sys

def f():
    d = locals()
    a = 0
    print 1, d
    b = sys._getframe(0)
    print 2, d
    b.f_locals
    print 3, d

f()
```

```
1 {}
2 {}
3 {'a': 0, 'b': <frame object at 0x7f9e62e296a0>, 'd': {...}}
```

This ensures that you always get the up-to-date values when you access `f_locals`.


iteration (kbenne) - i386-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

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


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1673 of 1675 tests passed)

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


Are you using DRAM to store BL30? If so, please use DRAM_BASE + your offset instead. And will this memory be used after initialization?


Good catch.  I forget to include *.pyc in my gitignore sometimes.


We are going to implement a change for this, however, the fix is going to be a little different than what is proposed here.   Expect to see a commit next week that addresses this.


PEP8 clearly states in the [Maximum Line Length](https://www.python.org/dev/peps/pep-0008/#maximum-line-length) section that:
```
The preferred way of wrapping long lines is by using Python's implied line continuation inside parentheses, brackets and braces. Long lines can be broken over multiple lines by wrapping expressions in parentheses. These should be used in preference to using a backslash for line continuation.

Backslashes may still be appropriate at times. For example, long, multiple with -statements cannot use implicit continuation, so backslashes are acceptable
```

Also, every syntax checker I have ever used with Python has presented an error whenever a line ends with a backslash.  Not to mention, they look absolutely hideous.

Then, there is the issue of accidentally having a space after the backslash that can easily go unnoticed to the naked eye and cause the code to break:
http://stackoverflow.com/questions/2070684/how-can-i-make-my-python-code-stay-under-80-characters-a-line

In this current situation, all we really needed to do was remove the backslash.  Unfortunately, adding a plus sign broke the code.  If there were no plus sign, since the string is contained inside of parentheses, it will be joined together as one string automatically:
```python
>>> one = 'ONE'
>>> two = 'TWO'
>>> print('"{0}" was not able to be precached due to the "{1}" table ' +
            'reaching its limit.'.format(one, two))
"{0}" was not able to be precached due to the "{1}" table reaching its limit.
>>> print('"{0}" was not able to be precached due to the "{1}" table '
            'reaching its limit.'.format(one, two))
"ONE" was not able to be precached due to the "TWO" table reaching its limit.
```

Notice how the second error shows "ONE" and "TWO" without the need of a backslash.


Page table will cost memory. If you only need to access SRAM in BL2, just increase the page table size in BL2.


good catch


all tests pass:
success: http://buildbot.bos1/builders/test-riak_cs/builds/1008


DesignDaysGridView (evanweaver) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


Necromancy!


Thanks!


The purpose of this program is to store data consisting of Apple computer models and parts. It will be created for the technicians at Macsuperstore. It's purpose is to make it easier on a technician when searching for the part number to a particular part of an Apple computer.


@HP-Test/test-python-proliant-sdk another test of this process


Good to go!


@HP-Test/test-python-proliant-sdk/owners Still another Test


All I can say is it is sooooooooo much easier to just drop in a file xD


@HP-Test/test-python-proliant-sdk ; @HP-Test/test-python-proliant-sdk/owners Final call


You might want to adjust the damage text levels for lost_limb/small, accordingly.


problems:
- undid diagram caching for PosNodes, so performance is worse
- TODO: figure out how to change cursor! another port in 0.15? make own port in short term; have JS?
- need function for determining whether port is a valid target for edge
  - and color-code this


python


@d4n13 did you get this


Use two spaces.


I think you’re a lot better at the whole “making it in code” thing. Can you do that? I really suck at it #dragndropfiles4dayz ;3.
> On Apr 1, 2015, at 8:17 PM, Joe <notifications@github.com> wrote:
>
> There seems to be code in there already, so it isn't too much work besides deleting this file.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/20zinnm/AntiBot/commit/96e60eba864e7b7b29631900e66beae912d64dca#commitcomment-10536990>.
>


@d4n13  you still there?


please review it https://github.com/infacq/export-csv/commit/cf6f8d234e2d2f2c29713465c48d30a0a05118a2


Wow! What a smart idea!


@WeetchKing

Would you prefer no one to maintain the project? Gorhill had made it clear the primary reason for not accepting donations was the commensurate expectation of future support.


fast-forwarding 2.0 to borshop-integration-1114-bugfix/known-issues = 277e6d3a


Bravo on the swing removal.


@d4n13 another test


> Are you saying that with this change the invocation of the date command is now clearly dependent on the HISTORY variable value?

I'm just saying that I've put the `DATE=` assignment down here, so that it's visible that it's dependent on the `HISTORY`. Purely stylistic, no change in functionality if I put it right after sourcing the config file.


$1, $2には、直前のmatchの結果が入るので、$1=darwin, $2=nullとなってしまって、
```
convert -background transparent -resize darwinx src/image/svg/read.crx.svg debug/img/read.crx_48x48.png
convert: invalid argument for option `-resize': darwinx @ error/convert.c/ConvertImageCommand/2555.
```
こんなことになります。

ビルド時にとりあえず出たのはこの部分ですが、他の部分にも影響するかと。


@mikeal can we sync tomorrow. I was reviewing @ingsings comment (https://github.com/joyent/nodejs-advisory-board/pull/33#issuecomment-88609886) and I think he's asking the same question I was probing on today when we spoke. The "one TSC" was meant to be the intersection of Project A, Project B and the Node Core projects. My intent was the one TSC would be a broader collaboration across all of the key projects. In my vision it was Project A, Project B, Node Subsystem A (Project C), Node Subsystem B (Project D) all on the broader TSC. Within each Project A-D, how the Project structures would be based on a classic Maintainer/Committer model (I believe this maps to the Collaborator role in io.js) or a model that the Project would define at Project creation. E.g. if an external project like Happy comes in, they already have a structure - we wouldn't require them to change (unless the structure was not acceptable to the TSC for a Foundation project - e.g. if they have no structure). How those Project teams operate would be up to each Project to define. I was not envisioning a TSC for every project and Working Groups for every project. The "one TSC" was just the intersection point for all of the Foundation projects to work on cross-Foundation issues/challenges. My assumption was new Projects that are external to the node core community would be at the Foundation for the benefit of staying in sync and collaborating with the others, and so my intention was this "one TSC" would be where that happens.

Let me be explicitly clear on my comments here. I'm absolutely not advocating for what I drafted as being appropriate in the context of the Foundation being setup; this is just my attempt to explain what I was thinking when I originally drafted the Lifecycle from our template. That does not mean it should be how it's done by any measure. I just want to offer this context if it helps as you and others look at ways to modify it.


Let's see if it's an everyone issue or just an isolated thing—then I'll merge. I'd like to avoid needing `sudo` if at all possible.


What are you deleting the schemes?


This does not look right.


Any reason why you left the beta config in the project?


Maybe include a comment with a link to vertica_python on pip?


91362860-Relax-fatal-error-for-DX-coil-zero-capacity-or-flow-rate-add-more-sizing-output-#4826 (rraustad) - x86_64-MacOS-10.9-clang: OK (1669 of 1671 tests passed)

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


More -1's for PR's


It didn't seem necessary for CI since the configurations themselves hold the settings. we can always add the schemes again if needed—I didn't really see the need for them.


may i know how to download to make to open vanilla input cin?


maybe not make clean this up now… that could affect other things - would rather not have it associated with this patch.


Boo PR's. I had a line comment earlier about a delete here causing a race with a FeedProcess job that had already loaded the object from sql, but not pulled the cached object from redis. 60 seconds delay to the real delete should eliminate that possibility.


switch the order of these above two statements so that they appear in the same order as the other clause


those are generated by CocoaPods so you can do stuff like:

``` ruby
pod 'HockeyApp', :configurations => ['Release', 'Beta', 'Alpha']
```

which then links HockeyApp in the specified `xcconfig` files.


@qzk, looks good to me.  Thank you for this!  Once John gives this a look, either of us can merge this.


oh wait, you mean the BetaConfig.xcconfig :point_up:? that must've been an oversight


あー、これtabとspaceが混在してるからレイアウト崩れているようになっているのかな


This all seems good except mostly the scheme stuff… still want those so we have the switch in the tool bar people can quickly change build settings. You can hide them in your UI if you don't want to see them.


looks like some accidentally committed git cruft


Why didn't you make a call for  def _assert_not_duplicate_name() method instead of using and if not in statement? Just for clarity i think it could be good to add an extra line to the _assert_not_duplicate_name() as a return which indicates that an exception was not raised, so you could just do a function call here (i'm not sure if you could just call  if(_assert_not_duplicate_name(event_name) and consider the occurrence of this call as a true to start a body statement after it.

So i could just imagine it would finally look something like this

  def _assert_not_duplicate_name(self, event_name):
        if event_name in self._event_lookup:
          raise self.DuplicateEventName()
       otherwise return something that tells me that no duplication occurred. i.e return 1;

 def subscribe(self, event_name, callback, *default_args):
       """Least safe, most convenient"""
       if (_assert_not_duplicate_name(event_name) == 1)

I'm not trying to complain about the way you did it like this, but i just had a concern if there is an objective of doing it that way or it was just a fetch of mind execution.


with the exception of the git stuff… this looks good to - maybe combine this with the Xcode stuff into a single patch. And we can push it through.


* パスの設定順に気をつければこの手順は不要になったりしませんか？
    * OSデフォルトのconvertコマンドより先にImageMagickのディレクトリが探索されるような順番

* また、特に分ける意味は無いので、README.mdに書いてもらってOKです。


Walk me through this stuff in the morning. Theres a lot here… I think best to separate this from the Xcode cleanup and such.


Also noticed the files are still on disk


:scroll:


Manually copied change into later version of code.


:rage1: this is what i get for stashing


Yes, at least once.  Will move comment.


Essa alteração aqui na verdade não muda nada. Só uma forma diferente de chamar o __invoke() mesmo. Mas quanto ao voltar o local eu acharia interessante também. Fica mais claro.


Want to check what faster on a lot of data, (next week)


I'm catching the thrown checked exceptions as before.


@milosstojanovic, very nice work. Overall, the two pieces of advice I can give are:
1. Write the code for the quicksort yourself, since using the generic version from the library produces many interactions, some of which I can't even reason about.
2. More a question of history -- separate the benchmarks completely, so the miniboxing benchmarks are separate from the tuple benchmarks (feel free to ignore this point) -- maybe it would be a good to merge them into `lib-bench`: https://github.com/miniboxing/miniboxing-plugin/tree/wip/tests/lib-bench


Prefer using `containsKey`.  Reads more clearly IMO.


No, I don't.  That was an oversight my me while testing, will revert.  This addresses the above comment too.


Looks good.  PR next time :P


Never mind, I see this changed in a later commit.


```c++
for(auto e : entities)
  _tiles[i].potential += e->getPotential();
```


symmetric case but different feature objects


@P-Kito: if you got improvements to SAI (new actions, new events, whatever) they are very welcome, do a PR


Please improve the description of GIT commit messages!


Great.
What I tried to do is to minimize the changes in the future. So this is not that easy but I tried to be sometimes less precise.
BTW why did you remove the +Preview.>file://figures/preview-1.png|width=70|label=preview-1+ and other?


is `db` a database object or a connection object? Can you be more precise in the docstring?


That's a good point and probably more flexible.


This update is a massive amount of work. Thanks for helping.


You can say a MongoClient object to be more precise.


Lines 5-7 and 17-19 are redundant: they are executed in both the if and else block. Take them out of the if-else block and put after it.


@ttuegel, the poppler re-factoring broke the build of haskell-poppler: http://hydra.cryp.to/build/718099/log/raw.


May be add a link to:
http://serverfault.com/questions/112795/how-can-i-run-a-server-on-linux-on-port-80-as-a-normal-user


cc @rsanheim


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1681 of 1681 tests passed)

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


I wish all fs devs a happy eastern. Will do more work after 12th april, now trip to croatia and 2 other projects press on my neck.. (sitzen mir im Nacken..)
Can listen a bit on camping wlan at evening, lol.


To set the Problem? Are you will this will be for only problems?


I don't understand the purpose of this attribute. How does the caller use a static `status_code`? I'm not sure how to read line 90 because I think `status_code` there refers to the parameter, but I'm wondering if it's interpreted as the static attribute.


Need a blank line before a member function.


@P-Kito yes, if you want to do some action after 5 spell hits its so easy with this


The same with the above. Will this be for only problems?


@ttuegel, this is what git bisect had to say:

    Bisecting: 0 revisions left to test after this (roughly 0 steps)
    [da74e8c317519059b62383a90f48d5403863038d] poppler: remove qtcairo patch cruft
    running nix-build -o /tmp/haskell --option extra-binary-caches http://hydra.nixos.org /home/simons/src/nixpkgs -A haskellngPackages.poppler
    /nix/store/hx38hgk01w0c5yfvmn7hjsi98pvpsmic-haskell-poppler-0.13
    c0d5cd0ff90cac2e04f6b436991baa4c4471150f is the first bad commit
    commit c0d5cd0ff90cac2e04f6b436991baa4c4471150f
    Author: Thomas Tuegel <ttuegel@gmail.com>
    Date:   Thu Apr 2 09:50:37 2015 -0500

        poppler: fix build of Qt 4 and Qt 5 wrappers

        The autoconf build system for poppler does not support building the
        wrappers separately, so this slightly enlarges the size of closures. To
        compensate, the command-line utilities have been separated into their
        own package.


I would not recommend doing that. That's silly when installing a proxy and using a real web server is trivial.


Can you also fix the spelling in the last word above?


I don't think so, the intent is to synchronize access to the closed flag...


Indentation for lines 90,91 seems off w.r.t the surrounding lines.


rax retry tests


Not sure how mvdstam authored this... must be the default git config email on the homestead vagrant box.


Copyright year = 2015?


But the watch does it waaaaayyyy too fast (faster than the harvest band, which breaks balance and is redundant)


Yay. When I wrote this register line in models.py, I was wishing for AppConfig stuff.


Fiz assim pra seguir essa padrão:
https://mwop.net/blog/2015-01-08-on-http-middleware-and-psr-7.html


Sorry about indentation


Why not use the chef file resource to create the link and let the bash resource just run build_bins.sh ?


this looks like it screwed up the nice fade effect on the home page


Enjoy your trip and happy easter time! Cya!


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - 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)


Should we also remove for creeps?


But it is const the map type is not const this is not accepted by clang and cxx11

(And not correct in general)


@mrunalp Pls, look at this https://github.com/avagin/libcontainer/commit/83e46951091db3372d90cbe29abfb9df9634082a. Is it what you mean?


To be clear, the problem is trusting the golang http server with the crazy internet. That's why another proxy is favorable instead. This is very normal practice. nginx and something like haproxy do a really really good job at this and are trusted to do so.

Most application level http servers are "good enough" to get their jobs done, but shouldn't be public facing. Also software like nginx or haproxy does not run as root either. They demote themselves to something with lesser privileges.

And for something as trivial as grafana, nobody should be adjusting iptable rules when you can `apt-get install nginx` and drop in a config. It's simple and the standard way of doing this stuff.


Nevermind:  I'll have a commit that fixes the problem pushed later today.


bare excepts are 99% of the time bad/evil (see https://docs.python.org/2/howto/doanddont.html#except).
please `except` at least on `Exception`.
Thx.


Nooo :D


tested and working


Use chef file resource to change file permissions instead. This block does nothing else.


Why is this a second migration, instead of being folded into 0001?


Ik denk dat dit niet juist is ... <% != {% ...


I think this comment can be dropped?


This "breaks" the behavior I liked earlier. Earlier, if I had a pattern matched by the `sml/replacer-regexp-list`, that took higher prority than the auto prefix for projectile: `P/projectname`.

After this commit, the projectile prefix takes the higher priority.


Does this mean that the following code raises an `AttributeError`?

```
err = NagiosStateError('Some sort of error')
print err.status_code
```
`self.status_code` doesn't get assigned if `status_code=STATE_OK` since `STATE_OK` is `0` which is fairly subtle and the code doesn't make it clear that this is deliberate.

Consider changing line 89 to `status_code=STATE_UNKNOWN` and dropping line 90.


Can this file have either a better filename, or a module level docstring talking about what it does?


This will be affected by (chef-bach issue #101)[https://github.com/bloomberg/chef-bach/issues/101]


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/442


Maybe `# Other architectures don't use assembly.`?


Similar to 88, I'm not clear on the purpose of this line.


Yeah, it's weird. That's how they got created when I ran the initial makemigrations. I guess we could merge/squash them.


Stylistic change.


This should have a better filename or a module docstring too.


r+


Great! Thanks! Just wanted to open an issue on it :+1:


Noted! Will fix that.

On Thu, Apr 2, 2015 at 3:46 PM Sylvain Utard <notifications@github.com>
wrote:

> Beware, you can use distinct=true if there are no facet filters, but some
> tag filters or some numeric filters or a query.
>
> So probably something like:
>
> if (!query && facetFilters.length === 0 && numericFilters.length === 0 && tagFilters.length === 0) {
>   distinct = false;
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/algolia/algoliasearch-helper-js/commit/ffd417451c5ddb1e9c84541af4e5ba847145d6c4#commitcomment-10546239>
> .
>


I'm confused by the 0002 migrations. They aren't data migrations, so what's going on?


please use `setattr(self, method, self._make_func(method))`


saw approval from Manishearth
at https://github.com/ChimeraCoder/servo/commit/a3eaacccf60fc05f43f353e859ae2807cec12f59


merging ChimeraCoder/servo/fix-issue-5466 = a3eaaccc into auto


ChimeraCoder/servo/fix-issue-5466 = a3eaaccc merged ok, testing candidate = e4da29b2


I was having errors with some pushes from a plugin in my IRC client. If I pushed the same text with chrome, it worked fine so it seems like it is an encoding issue with the plugin. In any case, I've found in the past that using format() on strings handles these sorts of situations better than the % method, and it does fix my issue.


@douglasgabriel @emanuelbatista
Ignorem o diff aqui desse arquivo, o que eu fiz foi adicionar o "snapshot" na versão do projeto e o plugin do Tomcat7, isso vai facilitar a vida de vocês. Deem uma lida http://tomcat.apache.org/maven-plugin-2.0/tomcat7-maven-plugin/plugin-info.html


I dont know if I want to PR all of them, quite alot of todo :-1:

My (custom) events or modified old events:
SMART_EVENT_NEAR_PLAYERS = 74 // min count, radius, first check, check period
SMART_EVENT_NEAR_PLAYERS_NEGATION = 75 // min count, radius, first check, check period
SMART_EVENT_EVADE = 7 // DespawnAtEvade_30s (1/0)?

My (custom) actions or modified old actions:
SMART_ACTION_SUMMON_CREATURE = 12, // CreatureID, summonType, duration in ms, storageID, attackInvoker, count
(added count)
SMART_ACTION_SET_ORIENTATION = 66, // random (added random)
SMART_ACTION_CONE_SUMMON = 112, // creatureid, duration, distRings, distRowSpawns, coneLength, coneDegree (+ target_0)
SMART_ACTION_SET_HP = 111, // flat, percentage (only player)
SMART_ACTION_TELEPORT = 62, // mapID, NearbyRandomTarget?, radius, noPos?
SMART_ACTION_RESET_THREATLIST = 113
SMART_ACTION_PLAYER_TALK = 114, // trinity_strings.id, flag
SMART_ACTION_PLAYER_MARK = 115, // Symbol 1, Symbol 2, ..., Symbol 5 (need to be in raid)
SMART_ACTION_PLAYER_SEND_BOSSWHISPER = 116, creature_text.groupid

My custom targets or modified targets:
SMART_TARGET_HOSTILE_RANDOM = 5, // added target_o for count of players
SMART_TARGET_HOSTILE_RANDOM_NOT_TOP = 6, // added target_o for count of players
SMART_TARGET_PLAYER_RANGE = 17, // added target_o for count of players
SMART_TARGET_NEAR_RANDOM_POSITION = 27, // radius, noPosition
SMART_TARGET_PLAYER_WITH_AURA = 28, // spellID, negatation?, distance, target_o=count


You pushed the application with an error. In the future make sure it runs before you commit and push it.


add: `orig_func = getattr(InfluxDbClient, func_name)`


replace with `for client in self.clients:`


Can these two blocks can be replaced by:
```ruby
[ "krb5-master", "krb5-admin" ]. each do |user|
   chef_vault_secret user do
      ...
   end
end
```


replace with `return orig_func(client, *a, **kw)`


Why do you make this a double?


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1681 of 1681 tests passed)

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


@gaoning777 None, actually. The biggest gains have been related to I/O so far (and pipelining for AppendEntries is the next one to do there). After that, there are probably some inefficiencies with respect to mutexes and condition variables, but I haven't pushed hard on finding them yet. I haven't been working on performance much lately in general, as I'm focusing on reaching a 1.0 release that can do rolling upgrades moving forward from there.


Good catch! So `pe_gem` would have always had the same issue?


Or remove the space before 'sha1', like the other arrays. :-)


We should only pull the annotations. Not the whole library - but then that wording of artifacts in Swagger is not always obvious. And it is quite possible that they have just merged annotations + code back together.
The older version of the annotations were standalone


reverted evernote specific text to original


That's perfect, thanks!


These are already defined in pkg/devicemapper/log.go


I do not know this code. Can you fix this by setting module? Or should we remove it so that we can improve it if we see it again?


Create any perfectly square placeholder image and name the portfolioPICone.png and portfolioPICtwo.png


Yes, good idea. I renamed this to chat-guidelines and edited it to be chat specific.


There's a pretty big gap between 32 kB and 1 MB.  Can I suggest the following values (in kilobytes): 1, 4, 16, 64, 256, 1024.


Not really important but for consistency please:
- replace var with `let`
- `window` with `$window`
- `==` with `===`
- `if(` with `if (`


add `bad_clients = []`


Haha sorry. That happened cuz I was just editing it in the GitHub editor and just making changes to that one line to try to get the mobile site to work properly


ho here you update `self.clients` ..  but with the `for client in self.clients` it's no more possible here.
So replace the line with `bad_clients.append(client)`


I was debugging this commit:

```el
(defun sml/replacer-raw (in)
  "Run on the string IN the replacements from `sml/replacer-regexp-list'.

If projectile is loaded, also performs replacements specified by
project name first."
  (let ((out in)
        proj)
    (when (and sml/projectile-loaded-p (eq sml/use-projectile-p 'before-prefixes))
      (setq out (sml/perform-projectile-replacement out)))
    (dolist (cur sml/replacer-regexp-list)
      (setq out (replace-regexp-in-string
                 (car cur) (car (cdr cur)) out)))
    (when (and sml/projectile-loaded-p (eq sml/use-projectile-p 'after-prefixes))
      (message "Before projectile %s" out)
      (setq out (sml/perform-projectile-replacement out))
      (message "After projectile %s" out)
      )
    out))
```
and that's giving me this:

> Before projectile :TB:
> After projectile [P/PRJ-NAME]digital/tb/:TB:

[Relevant portion of my `sml/replacer-regexp-list`](https://github.com/kaushalmodi/.emacs.d/blob/master/setup-files/setup-smart-mode-line.el#L35-51).


Yup


remove `tried += 1`
and add a, after the raise and properly aligned on the `try:` :
```
finally:
    if bad_clients:
        del self.clients[:len(bad_clients)]
        self.clients.extend(bad_clients)
```


On Thu, Apr 2, 2015 at 9:57 PM, Milos Stojanovic <notifications@github.com>
wrote:

>
> Ok, I will write code for quicksort by myself.
>
> Also, what do you think about version with specialization, why is it that
> slow? It is almost the same as for generics. Maybe I missed something
> there?
>

I guess the call to sortBy is generic, so there's no benefit to being
specialized. By the way, the comparator should also be specialized. I'd
feel much better if you also defined the ordering class yourself, with the
type parameter generic, specialized and miniboxed and, in the ideal case,
completely monomorphic. This way, we would avoid any source of noise. WDYT?

Cheers,
Vlad


sorryyyyy


soorrrryyy


s/werewolf/Werewolf/


Can you move these two outside the benchmark?  Or at least the `require('assert')` statement?

I'm not sure if it's great to add asserts in a micro-benchmark, by the way.  You'll be benchmarking more than just the operations you're interested in (`Buffer#equals()` in this case.)


lol just pulling your leg


We need the -2


@chrisndodge , @adampalay new commit: https://github.com/edx/edx-documentation/commit/a7a5a2005090376f3b8fd7465ce6b3a994de3f76


If this goes back to 1 we should bump the main version, otherwise it should stay at 2 i think.


rax retry tests


If the warning comes from inside vtk, it should be something like `module=r'vtk($|\.)'`


Can you correct the spelling of the last word above?


Style: space before {.  Also in the next file.


I think this can move out of the `try`.


Hopefully fixed in 7cc31b6b499193d095c3f82eecc82ab7cf0d7f10


Otherwise it's a good first version for a such a "cluster" client..


construct "an" instead of "a" active list.


ok so i honestly have no idea what im supposed to do about my issues? do i change this said files? do i wait for an update? idk


As loop is not mandatory, it shoulb be  better to add a test named like "test_xmlrpc_loop"
that add another loop that the selfloop and test it is passed to the transport. No ?


Should work fine.


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-MacOS-10.9-clang: OK (1675 of 1676 tests passed)

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


What would be incredible is a(some) test case(s), against a real cluster, where you "kill" (in a way or another) some of the cluster nodes and you assert that queries are always executed (and get the correct answer) despite some of the nodes being killed/shutdown..


First condition "expiry date more or equal current data", second one "and expiry date less or equal then 14 days from current date" (depends from modifier value)


While this code is correct, note that `ns_next_comma_list_entry()` tolerates NULL pointer. So it could be shortened to:

     const char *ext, *overrides = opts->custom_mime_types;
     ...
     while ((overrides = ns_next_comma_list_entry(overrides, &k, &v)) != NULL) {
        ....


This needs merging up to develop, right? Also, it looks like there was a typo on line 171.


Thanks for having my back on this Rimelek, totally forgot about double quotes... :+1:
Will try to do a more thorough check next time.


BSD style comments please

    /*
     * This is a multiline comment
     */


The url can also be reduced to:
http://redditbooru.com/images/?sources=1&limit=5&user=chiefnoah&q=alice

which removes the extra /api/ part of the route, but I figured this will likely be the convention for other API routes in the future.


This should really just fire a hook via `eventmgr_core` in my opinion.


I'm asking myself if it wouldn't be good to always randomly shuffle the `self.clients` before looping over it.. so that it's not always the same server that will get queries from this cluster-client.. (in case `self.clients[0]` is always up and running)
though it could be made optional also ..


[joke]Are you from the past ?[/joke]

This 80 characters limitation comes from punched cards, IBM card format, designed in 1928, 86 years ago: http://en.wikipedia.org/wiki/Punched_card#IBM_80-column_punched_card_formats_and_character_codes

With 80 characters, only 1/3 of my screen is filled.
At Eyepea/ALLOcloud, the standard is 160.
To be honest, 120 is the default value in PyCharm, I thought it was a good compromise.

Nevertheless, if this parameter gives you some insomnia, you can change that.


NEVER COMMIT TO MASTER!!!   Never Ever!


Hmm..., good point. Will fix it.


What do you think of this Click example where they use option counting for setting verbosity level?

http://click.pocoo.org/3/options/#counting


Why not extend `IClient.sendto_common_peers()` to make it optionally capability aware?


Hey Rodney- This looks really neat. Lets talk about your next project.
424.208.5411.

Tim


If this were a hook it could be properly isolated :)


For staff users sufficient_votes is always true. The message should show nevertheless when the results have not been published.


This is wrong, you will need to change this back.


This is evil but fine for now until messages are hooked in the same way I described above...


`AWAY` being an RFC1459 command, probably shouldn't be an extension.


Reloading OR Reading


isFirstOwner :)


add User profile, login, registration pages (only HTML)


isAbroadRegistration :)


The heck? the namespaces? Thats just a identifier in a form of a URI


Fixing so. :+1:


what about `1d6*100` ? if the answer is to use `roll('1d6')*100`, why isn't `1d6+1` done `roll('1d6')+1`?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1171 of 1171 tests passed)

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


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1681 of 1681 tests passed)

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


`sepcified`


(very optional) Consider using `contextlib.closing` to guarantee `client` gets closed. Doesn't make a big deal either way since this is just a test fake so even if we forget to close somewhere, doesn't hurt much, but just mentioning it in case the idea appeals to you.


I don't think this paragraph is needed, IMO a reference to `:h nvim` is enough.


Niiice. That's brilliant!


Recommend classic if statements here - unless we can be sure about the results of getMethod()

Using strings in a case statement is kind of whacky programming practice for java - if you must do it use method.intern(). String literals are always interned - so the pointer should be the same which is all the case statement checks.


This is where the Postgres JDBC driver ended up when I added it. Is there a better spot? Now that I've taken this pull request, it doesn't work anymore.


Well, that is the reason.


Those scripts are just a fix to get the serial console on Arista vEOS. It won't help you to run vEOS in the normal way (through Virtualbox or something like this)


Пока еще не прикрутил в свой форк .maybeWhitespaces - использовал maybeSome(['\s']) - но оно не верно работало, пришлось вернуть пока что .min(0).whitespace()


This seems to break the release process. I'm adding the note because, all releases in the history did not have the parent relation.


I found them using "Find usages" in PHP Storm and checked them one by one in the result list. Arguments can be variables, constants or static strings, expressions. I don't trust regexps unconditionally, but since you found and fixed most of them, I had easier job.


Dat spicy filename :D


how is the filename associated with a particular job maintained?

On 2 April 2015 at 17:24, tejasdp <notifications@github.com> wrote:

> client id is id of computer where the code runs
>
> D P Tejas,
> 4th Year Undergraduate Student,
> Department of Computer Sc and Engg,
> IIT Kharagpur
>
>
> On Thu, Apr 2, 2015 at 2:28 PM, dipayan92 <notifications@github.com>
> wrote:
>
> > What is client_id
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/samprit/Distributed-Systems-Project/commit/c454525050706afac8c7ff731a8ce16c8a6f6125#commitcomment-10541325
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/samprit/Distributed-Systems-Project/commit/c454525050706afac8c7ff731a8ce16c8a6f6125#commitcomment-10544204>
> .
>

--
*Subham Ghosh*
Fourth Year Undergraduate Student
Dept. of Computer Science and Engineering
Indian Institute of Technology Kharagpur


Would this work?

```el
(defun sml/perform-projectile-replacement (in)
  "If path IN is inside a project, use its name as a prefix."
  (let ((proj (projectile-project-p)))
    (if (stringp proj)
        (let* ((replacement
                (format sml/projectile-replacement-format
                        (projectile-project-name)))
               (short (replace-regexp-in-string
                       (concat "^" (regexp-quote (abbreviate-file-name proj)))
                       replacement
                       in)))
          (if (string= short in)
              (replace-regexp-in-string
               (concat "^" (regexp-quote (abbreviate-file-name (file-truename proj))))
               replacement
               (if (file-exists-p in)
                   (abbreviate-file-name (file-truename in))
                 in))
            in))
      in)))
```


`XCTAssertEqualObjects` is better for this kind of comparisons, it gives better output.

`XCTAssertEqual` is better to check NSIntegers, floats and so on.


Just hadn't got that far yet. I wanted to do some thinking about expanding the dice syntax, but didn't find anything yet. This will be useful for you, so I'll just punt on a better syntax and add support for `1d6*100`.


Nice prototype


Good. I'm thinking about applying this one separately.


Nice encapsulation


What does this mean?


I included this test in one of the methods, since is just another conversion.


No need for `instances.head` here if you bind in the unapply above as `h@Instances(host,port) :: tail`


and here


Why swallow the error here instead of just letting it go / wrapping it? The caller still has to explicitly check for `None` in this case, but the inner exception gets dropped.


Ah yeah, I just copypasted from modbutton and forgot to carry over the callback wrapper for it. Fixed. Also, unless something changed since I wrote it, there is never any need to check for self.setting('enabled') in a module like that, since the module's init() never gets called if it's not enabled. The event listener that calls self.run() is wrapped in self.init() so there's no path to execution if the module's init() isn't called.


1 april hehe


this is for the card facing up in my game


I consent to the licensing transition


thank you


Should this be `%>`?


not that it provides any performance improvements I don't think


tiens une nouvelle dépendance, faut que je mettre à jour le repo offline ;)


#25 @samaasallam @khadijaibrahim


@mrunalp It's strange that Jenkins doesn't worry about this. I've fixed this. Thanks.


add a test to vsync_highlighter_test too maybe?


Could we do a `:first-child` selector here instead?


Thanks @steveklabnik !


Whoops I forgot to remove the id from this element...


90179004-ConstrNumBackNotInitializedComplexGlazing (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (659 of 659 tests passed)

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


this needs to be updated :)


@maryam-osama
please delete line 6,10 and 23 in the applications controller


Updated develop branch with the fix.  This will be picked up in the next release.


String constructors might need more though testing though.


What it is it using as the secondary sort?


Consider doing this, which I think achieves the same effect in fewer lines.
```
buf = []
data = sock.recv(1)
while data and data != '\n':
  buf.append(data)
  data = sock.recv(1)
```


Similar to 117, seems strange to swallow the error.


@sdumitriu If PR #66 isn't merged before the next release to update PC, this commit should be cherry-picked, since Issue #63 is caused problems on PC right now.


This causes a deprecation warning with symfony/yaml ^2.2

    PHP Deprecated:  The ability to pass file names to the Symfony\Component\Yaml\Yaml::parse method is deprecated since version 2.2 and will be removed in 3.0. Pass the YAML contents of the file instead. in /vendor/symfony/yaml/Symfony/Component/Yaml/Yaml.php on line 58


Maybe just `See :help nvim for an introduction to Neovim.`, full stop?


hehe


it's the little things 😎


Thanks!


@dudekm jeśli repozytorium rzuci wyjątkiem, to `renderUser()` nawet się nie wykona – musisz wyjątek przechwycić w kontrolerze


The preview window has a bigger surface than the devices display. So that all text isn't shown on the pi_info page. This solves that until we find the correct sollution.


I’m using QT Creator. If you need help setting it up, let me know, and I’ll walk you through it. It should be pretty straight-forward, though.

http://wiki.qt.io/Category:Tools::QtCreator

Dave Pijuan-Nomura
1-416-857-4434

On Apr 2, 2015, at 11:50 AM, Damien Patou <notifications@github.com> wrote:

> I'm not sure this is the right place to ask but i'll try it anyways since I see a mention of it in this commit.
>
> I'm looking for heads up on how to get this compiled (and thus, running) under OSX. Any hint ?
>
> —
> Reply to this email directly or view it on GitHub.
>


Yeah sounds good buddy!


@dudekm Przypisujesz do zmiennej `$user` – a potem z niej nigdzie nie korzystasz – wywal


I'm sorry, I'm starting to learn about arduino, so I'm very noob yet.

I downloaded "Arduino-development" and catch "MySensor.h" and the other librarys:

![sem titulo](https://cloud.githubusercontent.com/assets/6225842/6973532/e0b12a7c-d961-11e4-8754-374aed20a678.png)

I put all the librarys on my library's folder, but when I compile a lot of erros appears.


lgtm


According to https://gnu.org/licenses/gpl-howto.html:

> For programs that are more than one file, it is better to replace “this program” with the name of the program, and begin the statement with a line saying “This file is part of NAME”. For instance,
>
> This file is part of Foobar.
>
> Foobar is free software: you can redistribute it and/or modify
> it under the terms of the GNU General Public License as published by
> the Free Software Foundation, either version 3 of the License, or
> (at your option) any later version.
>
>  Foobar is distributed in the hope that it will be useful,
> but WITHOUT ANY WARRANTY; without even the implied warranty of
> MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> GNU General Public License for more details.
>
> You should have received a copy of the GNU General Public License
> along with Foobar.  If not, see <http://www.gnu.org/licenses/>.

So I suggest putting these paragraphs in place of the "This program is..." section, substituting Gregorio for Foobar:

```
This file is part of Gregorio.

Gregorio is free software: you can redistribute it and/or modify
it under the terms of the GNU General Public License as published by
the Free Software Foundation, either version 3 of the License, or
(at your option) any later version.

Gregorio is distributed in the hope that it will be useful,
but WITHOUT ANY WARRANTY; without even the implied warranty of
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
GNU General Public License for more details.

You should have received a copy of the GNU General Public License
along with Gregorio.  If not, see <http://www.gnu.org/licenses/>.
```


@rafaeltorato bug solved! Can you test it for me?


Yeah, that got fixed real fast.  I fixed it locally within seconds, just forgot to push the commit that fixed it for a day or so.


@dudekm Tę metodę zostaw – odpal ją w kontrolerze jak przechwycisz brak użytkownika


This commit makes me really happy! Thank you for the hard work!


It may be tempting to `keepGoing = (daKey!=0);` here but there is small chance that another key on the queue will cause keepGoing to be reset to true.
Looking at your code, I don't think this can happen but why leave half a bug waiting around for the other half?


isn't there a ``squared`` argument to ``euclidean_distances``?


@SubJunk Building shortly


@dudekm Przechwytuj tylko `UserNotFoundException`, a nie wszystkie jakie się mogą wydarzyć.

Teraz nawet jakby był inny błąd, to pokażesz „brak użytkownika”.


Extraneous newline.


What the benefit of this changes?


Problème si le joueur est tué par plusieurs terroristes la même nuit


This should probably be ```unless (defined $metric_header_as_key)``` so the caller can specify 0 as a value.

Also, what do you think about throwing an exception below instead of printing a debug_message and using 0 as the default value?


There is a * in there that is breaking things.


Problème si le joueur est tué de plusieurs façons différentes la même nuit (un pop enlève une mort, est-ce la bonne ?)


Confirmed (Y)


1.  If the reservation checking is just like: "classType_slot00": true
You could use a distributed cache to represent them. And use the putIfAbsent() that is atomic.

2. You can install an IActorObserver in each server node and publish it's reference to a cache.
reserveationReferences.get(nodeAdress).reserve(...).then___(...)

3. Avoiding the cache, you could use a fixed Id for your IActorObserver like hosting does:
  `execution.createObjectReference(IReservatonSelectionPeer.class, this, "");`
 and then use
   `reservationPeer = execution.createReference(remoteAddress, IReservatonSelectionPeer.class, "")`

I'd recommend (2), since you could use the cache to publish the reference and other pertinent information at the same time, and the methods I've cited in (3) are private.

Observer references don't use the Hosting for routing, so the wouldn't
clash with your algorithm


@maikany


Done, thanks.


This is adding the option to pass a container name to the create container command.


Exciting yes! Size and shard-size work together to decide how many results to return from each shard and after combining shard results. Increasing shard size reduces the possible error in final results. In this fashion:

Hey ES - I'd like the top 2 (`size`) results please!
Master: Okay shards, let's get going: each of you give me 3 (`shardSize`) results!
Shards: Yay we're being useful!
Master: Great, now I'll just smash these results together and take the top 2 (`size`).
Master: FYI these counts are approximate if some shards didn't contribute to the sum of each document, so here's `doc count error upper bound` which is the maximum count of missing documents in any bucket.
```
shards                  combine   reduce
  2      2      2    ->   X    ->   2
+----+ +----+ +----+    +----+    +----+
|A(2)| |A(2)| |F(7)|    |A(4)|    |F(7)|
|B(2)| |C(2)| |D(6)|    |B(2)|    |D(6)|
                        |C(2)|
                        |D(6)|
                        |F(7)|

shards                  combine   reduce
  3      3      3    ->   X    ->   2
+----+ +----+ +----+    +----+    +----+
|A(2)| |A(2)| |F(7)|    |A(9)|    |A(9)|
|B(2)| |C(2)| |D(6)|    |B(2)|    |F(8)|
|C(1)| |F(1)| |A(5)|    |C(3)|
                        |D(6)|
                        |F(8)|
```

see also http://www.elastic.co/guide/en/elasticsearch/reference/1.4/search-aggregations-bucket-terms-aggregation.html#search-aggregations-bucket-terms-aggregation-approximate-counts


Looks like this commit broke build.
Please see http://ci.cgeo.org/job/c-geo/3585/console to remedy the problem.


BTW, IHosting is an IActorObserver.

      public interface IHosting extends IActorObserver


TIL python octal


remise dans Item


Pelo que vi por cima, isso seria outra forma de fazer um next();   a ideia do artigo e mostrar outra forma de chamar o próximo middleware.


The same can be written as below (with one more change):

```el
(defun sml/perform-projectile-replacement (in)
  "If path IN is inside a project, use its name as a prefix."
  (if (file-exists-p in) ; <-- Return `in' if `in' is already abbreviated string which is not a real file path e.g. :ED:
      (let ((proj (projectile-project-p)))
        (if (stringp proj)
            (let* ((replacement
                    (format sml/projectile-replacement-format
                            (projectile-project-name)))
                   (short (replace-regexp-in-string
                           (concat "^" (regexp-quote (abbreviate-file-name proj)))
                           replacement
                           in)))
              (if (string= short in)
                  (replace-regexp-in-string
                   (concat "^" (regexp-quote (abbreviate-file-name (file-truename proj))))
                   replacement
                   (abbreviate-file-name (file-truename in))
                   in)
                short)) ; <-- This probably needs to be `short' instead of `in'. Else the project abbreviation is lost (this fix works for me)
          in))
    in))
```


-If B<Timeout> is bigger than the B<Interval>, keep in mind that each slow
+If B<Timeout> is zero or bigger than the B<Interval>, keep in mind that each slow


The error messages in all these are a little strange in that it seems like you're referencing constants (e.g. `NOT_WRITEABLE`, `SENDCMD_FAILED`) but they don't seem to be defined as constants anywhere. `NOT_WRITEABLE` especially stood out to me because I usually see it written as "writable".

Are the constant-looking words just so we can look for specific things in the unit tests? The cleaner solution to me would be to subclass `CriticalError` and just check in the unit test that the particular subclass is what was raised. Because as it is now, if another developer comes along and says, "Oh, I don't like the way he spelled 'writable' in this error message. Let me fix it," that would break unit tests, which suggests brittle tests.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1264
success: http://build.servo.org/builders/linux2/builds/1262
success: http://build.servo.org/builders/android/builds/1298
success: http://build.servo.org/builders/mac1/builds/1477
success: http://build.servo.org/builders/mac2/builds/1480
success: http://build.servo.org/builders/gonk/builds/443


I think either something is broken, or the .apk intellij is sending to my phone is the wrong one.

```
04-02 16:57:05.759  27695-27695/com.frostwire.android E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.frostwire.android, PID: 27695
    java.lang.RuntimeException: Unable to create application com.frostwire.android.gui.MainApplication: com.frostwire.android.core.CoreRuntimeException: Unable to initialized main components
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4476)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
     Caused by: com.frostwire.android.core.CoreRuntimeException: Unable to initialized main components
            at com.frostwire.android.gui.MainApplication.onCreate(MainApplication.java:81)
            at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1007)
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4473)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
     Caused by: java.lang.UnsatisfiedLinkError: dlopen failed: cannot locate symbol "_ZNK10libtorrent14announce_entry12can_announceEN5boost6chrono10time_pointINS2_12steady_clockENS2_8durationIxNS1_5ratioILx1ELx1000000000EEEEEEEb" referenced by "libjlibtorrent.so"...
            at java.lang.Runtime.loadLibrary(Runtime.java:364)
            at java.lang.System.loadLibrary(System.java:526)
            at com.frostwire.jlibtorrent.swig.libtorrent_jni.<clinit>(libtorrent_jni.java:14)
            at com.frostwire.jlibtorrent.swig.torrent_alert.<clinit>(torrent_alert.java:59)
            at com.frostwire.jlibtorrent.alerts.AlertType.<clinit>(AlertType.java:11)
            at com.frostwire.bittorrent.BTEngine.<clinit>(BTEngine.java:49)
            at com.frostwire.android.gui.MainApplication.setupBTEngine(MainApplication.java:125)
            at com.frostwire.android.gui.MainApplication.onCreate(MainApplication.java:64)
            at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1007)
            at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4473)
            at android.app.ActivityThread.access$1500(ActivityThread.java:144)
            at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1265)
            at android.os.Handler.dispatchMessage(Handler.java:102)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5146)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:732)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:566)
            at dalvik.system.NativeStart.main(Native Method)
```


It bites me every time I try to manually make datetimes like `datetime(2015, 01, 01)`. :(


Seria interessante usar callable ao invés de Closure, isso deixaria as coisas mais flexíveis.

O tipo callable é PHP 5.4, se quiser usar, vai ter que ser sem forçar o tipo.


@donmccurdy


Same comment as @SaiedKazemi here. Do we want to keep the action script around or delete it?


Gaaah, such painful English. Why do I notice this only now? -.-


I was trying to include `PATH` variable and it wasn't able to handle it properly. Please check this if possible. Thanks.


fast-forwarding master to auto = e4da29b2


This event is not working in android app


```javascript
wa.connect(function connected() {
	console.log('Connected');
	wa.login(logged);
});

function logged() {
	console.log('Logged in');
	wa.sendIsOnline();

	wa.requestServerProperties(function(response) {
		console.log(response);
	});
};
```


looks neat, thanks!


the nose!!!


If N16 is no longer supported, would the check still need to be there?


Looks good


This is... acceptable.


90119792FourPipeInd_ErrMsg (wfbuhl) - i386-Windows-7-VisualStudio-12: OK (0 of 0 tests passed)

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


14.1.1 is already available https://confluence.jetbrains.com/display/IDEADEV/IntelliJ+IDEA+14.1.1+Release+Notes


Disregard, I see that this got replaced in a later commit.


@maryam-osama delete line 8, 13, 20, 25, 28


Yes, of course. Another `push -f`, or can I put that into the next change (I have more changes lined up)?


Do you have any tutorials for QT? would love to learn enough to help out.


done


Are we sure this source line will never change? Isn't this dynamically generated?


@bors: retry


The main benefit is that we forward all format calls to Trinity::StringFormat which uses the cppformat library. This results in much cleaner code and less clutter. Also we dont initialise static char arrays with  size of 1024 * 32 anymore for every format call which will result in faster speed.


typo: conatiner


interesting contribution


saw approval from Manishearth
at https://github.com/jgraham/servo/commit/d6a608806e84ea756eac171312adc2d08a0e30e8


merging jgraham/servo/css_tests_1 = d6a60880 into auto


Minor: maybe use criuClient since it isn't much longer?
Same for criuServer.


Cortex-A9 configuration in BLIS requires NEON, but not all Cortex-A9 processors support NEON


I prefer not to remove the existing N16 support, to make it easier for anyone else to keep supporting it (both @john9527 and @AndreyPopovNew do).  I just won't be testing, or adding new code related to it.


This is the most disgusting piece of code you've ever contributed to Nox.
I kinda like it.... :shipit:


si l'activité existe dans le club


Ааааа :joy:


Do we need to take out the commented code?


yayyy nice!


@maryam-osama  delete lines 21, 23, 32 in the requests controller
and make 'private' be right above the def


Well, this just worked brilliantly...


@stwirth What is up with the device_id_ -> device_id? Was this previously using an internal value rather than the param (incorrectly)?


`19 additions and 65 deletions.`


merging jgraham/servo/css_tests_1 = d6a60880 into auto failed


Thank you. I will fix it.


r=jdm


BLIS ARMv7 configuration doesn't use NEON, but it worth to do it on supporting CPUs (especially on Cortex-A8 with non-pipelined VFP unit).
@fgvanzee, maybe BLIS needs a generic ARMv7+NEON configuration?


@maryam-osama add a line between the first line and the second line in the welcome controller


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/af52ef2e665618a3e76fdcb0b5fb55407e5c0f66


merging Ms2ger/servo/MouseButton = af52ef2e into auto


Ms2ger/servo/MouseButton = af52ef2e merged ok, testing candidate = 8901bf93


MarkdownDocs (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1668 of 1670 tests passed)

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


ack - you're learning :)


Can you use and instead of &?


Since these registers are last, maybe we could only loop up to `STAR_REGISTER` instead. Although, if we do that, we should specify that no register should have a larger index than `STAR` or `PLUS`.


@bors: retry


We've put up a jenkins build server. it now runs pylint etc to do some code quality reporting. just needed it to trigger on builds


I prefer stable than space.


Only this line really needs to change:
```c
return &y_regs[*name == '*' ? STAR_REGISTER : PLUS_REGISTER];
```


don't you have important contributions to be making to our team other than
checking up on every character I type? :) This was genuinely useful - we're
setting up a test environment so we can say we were tested and reliable.
ᐧ

On 2 April 2015 at 22:21, svetlanaangelova <notifications@github.com> wrote:

> interesting contribution
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/samheather/iapt2group1/commit/4fb25f0132ace431f07b231f20268f8d2461ef88#commitcomment-10554611>
> .
>

--
*Sam Heather* | pingWHEN | m: +44 7585 441757 <00447585441757> | e:
sam@heather.sh

Website: www.pingwhenapp.com
Facebook: www.facebook.com/pingwhen
<https://www.facebook.com/pages/Pingwhen/292317570939511>
*a personal safety app for when you arrive (or don't) to a location. *


Don't you have some rakia to drink. Da?


What's the rationale behind `line-height-large` being smaller than `line-height-small`?
It seems counter-intuitive to me.


Is it the person to set this problem or the person that this problem is set for?


![screen shot 2015-03-20 at 17 44 52](https://cloud.githubusercontent.com/assets/3355771/6974099/a8116710-d987-11e4-8d50-11ef7e04b370.png)


Looks like my environment's code formatter went to battle with yours on whitespace ;)


ps. can you please bring me some rakia from bg when you come back. i like rakia


Add a triling slash.


Good spot, I'll be happy to merge a PR with an upgrade. :wink:


Same error here.


lol


Is it intended that the update of cmh2 in line 1336 is made unconditionnaly, without being protected by if (is_ok((ss-1)->currentMove)) ?


Spec?


A push server can **improve** the reliability ...


Looks like you need this line for a couple of tests. How about adding a `before_each()`?


@samheather - 1. Y MUST U DO DIS? 2. I found harold and kumar hilarious :)
@j6mes - I will bring you some of the liquid of the gods


Note that this commit also contains the addition of an event dispatcher class and some obvious (but critical) fixes to StateMachine.


Everything looks ok for me (Serial1 port and baudrate, 3.3V, etc).
I guess if there's no problems with the [mbili no.2 jumper](http://mbili.sodaq.net/features/) and Wee Work/FW Switch the board and wee should work well with this library.
Can you share the exact serial output while using kick() and restart() methods?


Changed to "an asterisk"


Yes, or more generally, all saved yankregs should be before all "volatile" yankregs.


What exactly did you change? I can't tell


Good catch.  I will fix that and resubmit.

Thanks.


@maikany


Называть коммиты "initial commit" уже не надо, да и на русский язык же перейти можно)


Please note: I made substantial changes to the input settings in this revision. I hope it doesn't break anything for anyone.


"...error queue instead."?


Жги еще)


... can only be used in a TypeScript file
or
... can only be used in a .ts file


This is the person that this problem is set for.
@dkayiwa Should it be  changed?


Are you sure?


Shouldn't it be checking IsIgnoringCooldowns too? in case spell is triggered this would prevent it from casting properly


This closes xwsxethan/MusicScoring#84


Hi Rob

Looking at the change diffs, I don't see anything related to ...

Added 'signalk.clock.source=system' to signalk.cfg. Set to 'rmc' to enable clock from RMC sentence.

What am I missing?

Thanks


LGTM. Good catch :+1:


Good point. it should not hurt to put this at top level as tests should not suppose the initial state of `"*` anyway.


Did you mean "after the caller" or simply "phone call"?


See #218


you probably want to add decorators now as well.


#25 @norine-badr


Will do.


Does this change only "delay" calls or actually reduce the amount of calls?


Just curious, but what does the offset do? My script I adapted from lean doesnt have one. sorry if its a dumb question.


Will do!


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (0 of 0 tests passed)

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


90119792FourPipeInd_ErrMsg (wfbuhl) - 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)


Once the pull request goes through, I'll add you on the BukkitDev page (besides, can't login nor reset my password right now)


Is there a default TTL? For example, the default is 4 weeks in GCM.


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


90119792FourPipeInd_ErrMsg (wfbuhl) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

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


Confirming it as working :)


I see it, over in core.


Forgot to mention this in commit message, tells people about how much spelling matters.


из таблицы


backward compatibility?


This background color shouldn't be here.


No but they make the JMH output more readable


I'd prefer a MUST. We're already assuming time "fudge" in the implementation so it's already imprecise.


Exactly.


88737258-EPW-Ruby-Access (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (2180 of 2201 tests passed)

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


Pinging @jrieken, would this cause issues for your js scenarios?


Recommend moving it up to Line 462 so that it's included in the section describing the value.


The libgdx word is probably not correct here.


Thanks, that PEP was the reference I was missing. I've had a go at implementing it: 1f4c9e463e91cd1fbdb249317184eec9c9d0644c


Nothing


@norine-badr


The error building `haskellngPackages.poppler` persists.


Hmm, that's a good idea. The question is whether to change the paths before testing on the regexes or change the regexes themselves. Either way seems like it has the potential to create bugs for *someone's* use case.


Is it problem's or problems?


I'll fix this :P.


what the benefit of this system compared to current one?


Again, you're mentioning libgdx project :).


Is it really the person to set this problem? Or the person that this problem is set for?


thanks a lot :)


According to a pending pull request, compiled binaries should exist as part of repo. Potentially including a script to generate the compiled binary for Windows or other platform would be useful instead.


Changed the function to return an error type.


100% agreed on it being verbose and against the Go idiom. I chose the long name to match all the existing code (nodecontroller.NewNodeController, resourcequota.NewResourceQuotaManager, namespace.NewNamespaceManager), but I'll change it to just New, and maybe swing back through here in a separate PR to shorten the others.


This and the previous test are a bit hard to follow. Are the `COLLECTD_PID` and `COLLECTD_UNIXSOCK` lines specific to each test or could they go into `setUp`?

I understand why you're setting the hostname, but it's a bit subtle. I think at minimum, we should document why we're setting the hostname to a weird value.

Since we control `FakeCollectd` we could alternatively do something like this:

```
self.fake_collectd.set_response_behavior(CLOSE_AFTER_REPLY)
```

which, to me, makes the intent of the test a bit more clear. This also simplifies the other tests because it means that the `check_collectd_mlab.HOSTNAME = 'my.hostname'` line only has to appear once in `setUp`


change != to < ?


Yay, thanks! Shame this is so clumsy in JAVA currently. But "glad" that War for the Overworld is still a little bit bugged that I had some time to work on this :)


can we just combine the two helpers.. there seem to be a log of duplication


A default of 0 is hopefully uncontroversial.  That is what Google recommend people do for maximum performance anyway.  The 4 week thing relates more to #2.


Order should be the other way around, this replaces `<quote>` with `<backslash><backslash><quote>` (instead of `<backslash><quote>`), see #18


Move this and the other log from j==1 to the end of the while loop? If the threshold is 100 pings, I don't think you want this line to be printed 100 times.


Ah, the distributed top K problem.  I don't think there's a certain # of
results you can deterministically get from each shard such that you get an
accurate top K result overall.  It's a hard problem.

On Thu, Apr 2, 2015 at 2:06 PM, John Krah <notifications@github.com> wrote:

> Exciting yes! Size and shard-size work together to decide how many results
> to return from each shard and after combining shard results. Increasing
> shard size reduces the possible error in final results. In this fashion:
>
> Hey ES - I'd like the top 2 (size) results please!
> Master: Okay shards, let's get going: each of you give me 3 (shardSize)
> results!
> Shards: Yay we're being useful!
> Master: Great, now I'll just smash these results together and take the top
> 2 (size).
> Master: FYI these counts are approximate if some shards didn't contribute
> to the sum of each document, so here's doc count error upper bound which
> is the maximum count of missing documents in any bucket.
>
> shards                  combine   reduce
>   2      2      2    ->   X    ->   2
> +----+ +----+ +----+    +----+    +----+
> |A(2)| |A(2)| |F(7)|    |A(4)|    |F(7)|
> |B(2)| |C(2)| |D(6)|    |B(2)|    |D(6)|
>                         |C(2)|
>                         |D(6)|
>                         |F(7)|
>
> shards                  combine   reduce
>   3      3      3    ->   X    ->   2
> +----+ +----+ +----+    +----+    +----+
> |A(2)| |A(2)| |F(7)|    |A(9)|    |A(9)|
> |B(2)| |C(2)| |D(6)|    |B(2)|    |F(8)|
> |C(1)| |F(1)| |A(5)|    |C(3)|
>                         |D(6)|
>                         |F(8)|
>
> see also
> http://www.elastic.co/guide/en/elasticsearch/reference/1.4/search-aggregations-bucket-terms-aggregation.html#search-aggregations-bucket-terms-aggregation-approximate-counts
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrata/spandex/commit/7b674835f7464b58bb423a1d9eb57f5d3be783fb#commitcomment-10554343>
> .
>

--
Evan Chan   |   Principal Engineer, Socrata     |    @evanfchan
*Socrata <http://www.socrata.com/> *| The Data Experience Company™
*Socrata isn't about worldly success, but about changing the world
successfully. This is why I'm excited to be here.*
*What does Socrata do? <https://www.youtube.com/watch?v=yyz86MlkriQ> *(watch
our 2 min video)


Thanks!