This is the major culprit


Thanks for the reply. I'd already removed the `.otf` file. Tried a restart but same result. I'll have a look again another day.


On Dec 14, 2015 10:28 AM, "Tamas Nagy" <notifications@github.com> wrote:
>
> In let's say 95% of the commits I've checked, those eligible for
copyright did not extend the respective declarations or the AUTHORS file.
In general this should be interpreted so that they would like to release
their changes under the public domain and/or would like to donate the code
to us.

This is not true. AUTHORS is not more authentic than git commits. It's
merely a list of credits, but not submitting a change to the AUTHORS file
doesn't mean anything.

Mentioning someone in AUTHORS is an appreciation of their work.

The only legal declaration is the signed-off-by line in the submitted patch.

This could be either because they don't think the contribution was major,
or they would like to remain anonymous. I have no problem with the current
practice how it is done here, I'm only documenting what I see.
>
> Your comment seems to suggest that after literally any change that is
made, we must record the identification of the author so we could waive
some of the issues we could face in a later legal incident. Am I reading
you correctly?
>

No. But a signoff cannot be made by someone anonymous.

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


> I think we should add a big fat warning that this is exponential, maybe
> even rename it to cnf-exponential?

I agree on that. Additionally I plan to implement a cnf-lightweight option which does not generate a full cnf, but instead uses some heuristics, and lets deeper stacked ANDs stay where they are.

> I don't think the renaming makes sense, by default everyone expects
> CNF-ying to be polynomial in input formula size.

I don't think that it's that misleading. Even the former naming, where an only equisatisfiable formula was returned by using CNF Tactic had the same problem.


Why do you need this?


Nice ASCII :-)


Added a space after the "//", and capitalized the "Forcing".


fixed setup.py, almost no cpu load anymore. Instead a snmp walk of 2000 oids  from 30 seconds it needs around 3 seconds. With pySnmp it creates a 100% load.


Didn't mean to put that there, fixed now.


Assets/Scripts/Scripts that go on everything/Movement_Speed.cs

If you change the starting speed or the current speed, gameObject.GetComponent<NavMeshAgent>().speed is going to be set each frame. Are you sure this is right? (ok, I know, maybe I should wait untill you commit this, and then review your code, but maybe I would have forgetten this then) (only trying to help)


Why do you need separate types for constructors and destructors? They're just functions.


please reduce number of "poison free" functions to 1, they all the same


Not very well, but this is how everything else in this class works. But it will always be zero when called.


1.
```
e anal.recont=true
af
afb
s 0x401012
afb
s 0x40dfd0
afb
```
2.
```
e anal.recont=true
af
afb @ 0x401012
afb @ 0x40dfd0
```
These tests are doing the same. The one difference is that the first test prints additionally blocks at 0x401000.


Indeed, I was just talking of setting back the iteration number correctly (and you think the same: https://github.com/M2-AAIS/BAD/commit/e88f3c6c152be3fb0ed28d115f0f7080b6941a8d). ;)


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 846](https://teamcity.labkey.org/viewLog.html?buildId=291400&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24, passed: 223 Build time: 00:29:24
### Failed tests
```
org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

org.labkey.test.tests.RedCapTest.testLongitudinal: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.tests.RedCapTest.testSeparateDemographicProject: <no details avaliable>

org.labkey.test.tests.RedCapTest.testVisitBased: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

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


here you need to call "poison" version


We don't need this if we go the init.sh route


ログイン回数の表示についてです。(index.phpの56行目から)

ポイントは、次の点です。

「PDOを利用して取得した結果はどういうデータ構造になるか。」
言い換えると、PDOStatement::fetchとPDOStatement::fetchAllの理解ですね。
($stmt->fetch()や$stmt->fetchAll()するとどうなるか)

さて、今回の場合だと
"select sum(login_count) from users" という SQLを実行しています。
これはターミナルで実行してもらうと分かる通り、
sum(login_count)というカラムに全てのlogin_countの総和が返ってきます。
(この `sum(login_count)` というのは一時的なテーブルのイメージです。「エイリアス」で調べてもらうと良いかと思います)

この(一時的ではあるが)新しいカラム`sum(login_count)`がミソです。

今まで勉強してきたパターンだと、
"select * from users"というようなSQLを実行していました。
そして、 $stmt->fetchAll(PDO::FETCH_ASSOC)とすると

usersテーブルのデータの行数だけ配列ができて、その配列の中にそれぞれのカラム名をキーに持つ要素が入っていましたよね？
(あまり意識していなかったとは思います)

ということは、 $stmt->fetchAll(PDO::FETCH_ASSOC) は
n行分の配列の中に指定したカラムがキーとなる連想配列を返す訳です。

つまり、今回の場合だと、
array(1) { [0]=> array(1) { ["sum(login_count)"]=> string(1) "2" } }
こうなります。
よって、結果を表示したいのであれば、

echo $sum[0]['sum(login_count)']

などとすればとりあえず解決します。

ただ、この場合だと、コチラがほしい結果(=login_countの総和)は、1行しかありません。
だから、fetchAll()ではなくfetch()で十分です。

なので、

こうすると[0]をつけなくてよくできます。
また、fetch()が呼ばれているので、特定の1行だけが興味の対象であることがソースコードから伝わります。
なので、

$sum = $stmt->fetch(PDO::FETCH_ASSOC);

としてあげれば、echo $sum['sum(login_count)'] でいけます。

ここからは補足です。
変数名とキーでsumがかぶっていて若干気持ち悪いのは、

$sql_sum = "select sum(login_count) as hoge from users";
$login_count = $stmt->fetchAll(PDO::ASSOC);

としてあげれば、
echo $login_count['sum']
で総和が出せます。
この方が変数名を見ただけで何が表示されるかが伝わりますよね？

fetch() と fetchAll() の違いは公式のリファレンスを見てみてください。
とくに、オプション(PDO::FETCH_ASSOC)とかをいろいろ変えて試してみましょう。


все супер!


Is there a reason why this is merged now? It was moved to a branch because it breaks Vigilante8. The game crashes with this commit. I assume this bug still exists.

I think there is a wrong assumption in the copy from RDRAM code. It assumes that the buffer's width and height match VI.width and VI.height. The plugin copies from the RDRAM area corresponding to a buffer to this buffer. VI values should not matter (unless no buffer is found?)


ユーザーネームもpasswordにすれば、確かにセキュリティは向上しますが、
1) ユーザーネームは実質的に公開されている
2) ユーザーネームの誤入力の可能性が高まる
という点からあんまりよろしくはないですね。


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


this was "1. " in previous version


Nice! It'd be nice if we had some dynamic lint in cider, which alerted us for similar shadowings.


Вот только здесь это  не сработает и надо руками вызывть метод ```comedy?```.
Дело в том, что этот метод **rspec** пытается вызвать динамически и не может его найти, ведь ```comedy?``` работает через *method_missing*.


Ah you are right we can provide a full URL . Let me handle that.


Great!


@izinin solved.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.936](http://teamcity.monogame.net/viewLog.html?buildId=12189&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:29:26


It is not minio XL


TeamCity TripleA :: UnitTest [Build 471](http://home.k-moeller.dk/viewLog.html?buildId=1110&buildTypeId=TripleA_UnitTest&guest=1) is now running


I think it seems reasonable. It might actually be worthwhile to consider
also adding Zero_init as a special case since that's the default
initialization for scalars.

Andrew Sutton

On Sun, Dec 13, 2015 at 7:50 PM, Wyatt Childers <notifications@github.com>
wrote:

> @asutton <https://github.com/asutton> I'm thinking the most logical way
> to implement initialization for user defined types is always to call out to
> some user defined, or compiler generated (during elaboration) function,
> with all the information stored in the call expr itself.
>
> How does this look thus far?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Beaker-Compiler/beaker/commit/7756fda83c507a88760f7789d4dff7a90929c2a8#commitcomment-14949408>
> .
>


OK sure :)


@bberenz I know you're rewriting much of this logic, so I won't port to 2.0 just yet, but the non-printing dialogs didn't have the checkbox logic.  I just combined them.


You got this wrong, this dict contains the physical disk path for each disk, identified by 'serial'


TeamCity TripleA :: UnitTest [Build 471](http://home.k-moeller.dk/viewLog.html?buildId=1110&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:40


*not tested*
i i guess you need some variant of https://github.com/notspiff/kodi-cmake/commit/25fe7fb44acc9466d2ddde2e5f84b272680f0770


:ok:


Nit: the "I" in "if" (near the end of the line) is a "T" here.


oops... thanks


Why not simply mock_disk.path.return_value.RelPath = mock.sentinel.rel_path


Oh, yeah. We really needed this, but weren't sure how to do it in Angular. Thanks!


readelf gives the same info. Remember that this memory is not mapped inside io logic/magic is just showed with `iS` command


As mentioned in the above comment, this is not what we expect this method to retrieve.


git should add "fb like" for comment !


Are you sure this comment can be here? Because JSON doesn't do comments...


If wonder if this one should also go into the experimental package. To keep backward compatibility, one could re-alias it in dom/package.scala from `type MediaStream = raw.MediaStream` to `type MediaStream = experimental.mediastream.MediaStream`


这个和最后 ClosedFlag 没有用到？Go 可以直接写 `type isClosed bool`，然后对 isClosed 类型定义方法，参考 log.go。


这个和最后 ClosedFlag 没有用到？Go 可以直接写 type isClosed bool，然后对 isClosed 类型定义方法，参考 log.go。


yep, that did fix the problem at first sight.
I will try longer tests tomorrow. thanks! :)


It would be wise to test on something that has more `read` wise impact than `write`.


This leaks on the error path. {{mgmt_message_parse}} might fail on the third field in which case the first two would already be unmarshalled. Output pointers are guaranteed to be NULL or valid.


Although I agree with you, I prefer to not change the other config options to avoid further conflicts.


:+1: now it's about lowering ram usage ;) Perhaps we can think of backporting to 15.09 too if it works fine.


Ops, noticed this isn't the nixpkgs repo yet... sorry :P


no need for std::


As a suggestion, this test heleper could accept an argument specifying whether multiple planned vms will exist. This will make this test easier to follow and modify in case there are other cases when an exception may be thrown.


std::


rotated plants, see Splizard's snow mod's shrubs


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


same here


Хахаха. Не всё так просто. RSpec кидает тебе исключение типа `expected ... to respond to comedy?`, правда?
Это потому, что у объектов Ruby есть такой метод `respond_to?(methodname)` — который возвращает true/false в зависимости от того, есть ли у объекта такой метод.
И поэтому хороший разработчик, когда перекрывает `method_missing`, вместе с ним перекрывает и `respond_to?` — и тогда весь код, зависящий от концепции «есть ли у твоего объекта такой метод», работает как должно.


TeamCity TripleA :: UnitTest [Build 472](http://home.k-moeller.dk/viewLog.html?buildId=1114&buildTypeId=TripleA_UnitTest&guest=1) is now running


Close #162
Close #181


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


I would prefer this to be a flags field (unsigned int), with values defined in  {{SSLConfigParams}}. It makes it more obvious to the reader when they see {{load_ssl_file_func(foo, SSLConfigParams::UNVERSIONED)}} compared to {{false}}.


>
> @asutton <https://github.com/asutton> I'm pretty sure that this needs to
> modify the call expr, probably by doing some look up to change the call
> expr's underlying function which will be invoked to perform the record
> type's defined reference initializer.
>
Do you have any suggestions about how to perform the look up and/or store
> the different variations of constructors for look up (trivial, reference,
> copy).
>
Trivial initialization is valid only if the class can be trivially
initialized. So: scalars and aggregates (arrays, structs) of scalars.

For default and copy initialization of a record, you just need to search
the constructor list for an appropriate constructor. If found, call that.

It might be worthwhile to analyze record definitions after all members have
been analyzed and then generate default and copy constructors as needed.
Then resolution always selects one of those functions to call.

Reference initialization is special and not something that can be
overloaded. There's only one way to do it and it only happens when the
object has reference type (i.e., it's not a class object).

> Also, if we choose to go with this methodology should we have a "trivial
> constructor" syntax, so a user can specific a constructor explicitly as the
> trivial constructor?
>

No, that would defeat the purpose.


fixes #3


TeamCity TripleA :: UnitTest [Build 472](http://home.k-moeller.dk/viewLog.html?buildId=1114&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


fixes #2


simpler to sudo apt-get install phablet-tools :P


Not sure, it was in the example and I left it. Lets remove it and see if it works. Thanks for the catch.


I just thought of another potential issue with this cop. `hash.merge(k: v)` is often used as parameter to a method call.
Changing
```ruby
def my_method(options = {})
  other_method(options.merge(c: 3))
end
```
to
```ruby
def my_method(options = {})
  other_method(options[:c] = 3)
end
```
would not be equivalent

We may need to change the cop to check context of the call to merge and not warn when it is used as an argument to another method.


`visitExpression()` actually enforces it is zero. But I will change it to just call `formatExpression`, then it will work correctly.


not supported anymore XD


That check is valid. This a bug somewhere else. Are you sure you used make your compile this?


This is wrong on Mac OS X where you get this from the libc.
My vague recollection from autotools is that you rather had to set libdispatch_la_LIBADD = $(KQUEUE_LIBS) in the Makefile.am instead no?


sorry bit out of context, meant http://www.stroustrup.com/C++11FAQ.html in-static member initialization. justa  fyi


Now I'm actually testing this, fencing.py.py is also missing an update for in-line help in the action block e.g.:
`        "help" : "-o, --action=[action]          Action: status, reboot (default), off or on",`
should probably list diag too so people know they can use it?

Also, I finally took some time to test this, and... this does not behave as expected at all when I tried this?
I had to specify -m cycle to get the expected behavior, otherwise it does ipmitool chassis power off then waits till off then turns it back on (default -m offon)
e.g. `fence_ipmilan -v -a IP -o diag -m cycle -P` worked but `fence_ipmilan -v -a IP -o diag -P` did exactly the same as `fence_ipmilan -v -a IP -o reboot -P`

It's really easy to test so please do.


(drive by commenter here!

If this doesn't work, consider adding:
```
addons:
  firefox: "latest"
```

per https://docs.travis-ci.com/user/firefox/

Good luck!


Fix me


Me either. Research! :)


That's not the right way to mock a class. Use a decorator instead.


wow y u so much in love with lp ;-;


How come you decided to add a new `dEidxj[3][3]` array rather than extend the `Ei[3]` array to be a `Ei[4][4]` like the B field?


I'm waiting to decide if I like it more than Coveralls before swapping the badge out :-).


Atributo ne mayúsculas!


Please don't forget to create a git tag for this change


I'm actually going to fix this up and clean up other PRs to other roles, this can be done like:

```
action: "{{ ansible_pkg_mgr}} package=foo state=installed"
```


i guess mainly a style choice but i'd stick this in the destructor and delete the struct here. kills the if ;)


I guess you are testing here the behavior of the _update_planned_vm_disk_resources, right? Well, this should be in a different test.


Mejor iniciailizar a null


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


This doesn't make much sense, use different mock objects for the conn objects, otherwise we could have bugs if those get swapped by mistake.


I'm hoping to implement something similar to flux-stateful. I'm curious why you got rid of this? I thought this was a strong solution, did you run into issues?


Yes - I also though about it. Will do.


Testy jsem nespouštěl :-( sorry :-(


Will be done - makes sense. Thank you.


Atributo en mayusculas!!


many thanks, Chirs
some typo in the new terms.
  from the loest portion

name: esophagogastric junction muscularis muscularis propria

to replace by
esophagogastric junction muscularis propria

Thank you again


No reemplaza todas las apariciones


I am still dubious about passing pure nil here, could you maybe add some failing test (nil is returned by read-file-fn) and see if the result map is validated?


>
> On Dec 14, 2015 10:28 AM, "Tamas Nagy" <notifications@github.com> wrote:
> >
> > In let's say 95% of the commits I've checked, those eligible for
> copyright did not extend the respective declarations or the AUTHORS file.
> In general this should be interpreted so that they would like to release
> their changes under the public domain and/or would like to donate the code
> to us.
>
> This is not true. AUTHORS is not more authentic than git commits. It's
> merely a list of credits, but not submitting a change to the AUTHORS file
> doesn't mean anything.
>
> I would like to clarify that the above sentence contained a construct
which you might have interpreted as a typo, while it was not.

According to my proposal, those who would not like to waive their copyright
for the benefit of the other contributors, should extend the respective
declarations at the top of the given files and/or they could add themselves
to the AUTHORS file in (TODO: policy?).

Of course, we have the opportunity to question either one of their actions
when we are reviewing a pull request, just as we do with the coding part.

This may seem odd to you, but it's best to catch things at the source and
preempt any misunderstanding in the future.

Mentioning someone in AUTHORS is an appreciation of their work.
>
> This direction sounds right. But then, the work done by those who don't
mention there are less valuable?

The only legal declaration is the signed-off-by line in the submitted patch.
>
> We should not expect people to distribute a whole .git tree when they are
distributing open source software. All copyrights and attributions should
be contained in the respective files themselves. The top of each file is
the best place for this in my opinion, although an AUTHORS file has its
uses as well, only in much more rare occasions. I do not recommend stuffing
the AUTHORS file, but in its current state, it's not doing the best it
could.

On another note, neither are the copyright declarations used at the top of
our files legally binding. I could go into details if you are interested,
however basically once you create and publish your lines of code, it is
already protected. Actually, if we went ahead and stripped away all such
notices from every file, we would be still good to go (this should not to
be taken as professional legal advice). This is not the point. We are doing
a favor for both ourselves and the community by adhering to these standards.

This could be either because they don't think the contribution was major,
> or they would like to remain anonymous. I have no problem with the current
> practice how it is done here, I'm only documenting what I see.
> >
> > Your comment seems to suggest that after literally any change that is
> made, we must record the identification of the author so we could waive
> some of the issues we could face in a later legal incident. Am I reading
> you correctly?
> >
>
> No. But a signoff cannot be made by someone anonymous

> An anonymous GitHub e-mail can be repurposed for this task:

https://help.github.com/articles/keeping-your-email-address-private/

Anyway, when distributing a source tarball, the recipients will not see the
signoff's. Neither do we after a few rounds of PE/OSE/GitHub/incubator/etc.
porting and copy&pasting.

And anyway, every time I would like to reuse an open source module for my
project from someone else, do you expect me to download all git trees of
the given project, scan them for self-plagiarism, trace back edit and
section origin, double check aggregated licensing, reassemble attribution
and cross reference all these to possible mailing list announcements
regarding community decisions?

You might not have seen such written policies before, however I guess most
project silently apply these principles already anyway.

If you have ideas about the wording, terminology or the philosophy in
general, I'm open to concrete suggestions and elaboration. Unfortunately, I
can't draw a conclusion about next steps from your comment yet.


Hi @disconn3ct,

Please respect the license and my copyright....
My code is  easily recognizable because of its distributed architecture, so If you "hack" it, I will know it very easily...

Regards.


@ThomasAdam
I missed. and I'm troubled...
Could you give me some advice?


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


latest mc:
```
mc: <ERROR> Unable to stat ‘https://aas3bucket.s3.amazonaws.com’. Bucket name cannot be smaller than 3 characters.
 (2) ls-main.go:103 main.checkListSyntax(..) Tags: [https://aas3bucket.s3.amazonaws.com]
 (1) url.go:76 main.url2Stat(..) Tags: [https://aas3bucket.s3.amazonaws.com]
 (0) pkg/client/s3/s3.go:326 s3.(*s3Client).Stat(..)
 Release-Tag:UNOFFICIAL.2015-12-14T13-25-45Z | Commit:3301c6263eda | Host:escape | OS:linux | Arch:amd64 | Lang:go1.5.1 | Mem:2.5MB/6.4MB | Heap:2.5MB/3.8MB
```


And, others too...


J'avais simplement des bugs de compilation si il était pas là. J'ai pas trop cherché à résoudre autrement honnêtement vu que le download faisait la job. Si on trouve la source du problème, c'est mieux.


Esto no está bien!


wtf :-)


It's a common idiom to leave out parens on zero-args functions if they're not side-effecting. That makes it easier to think of them/make them act like vals, which they share the same namespace as. So I'd call this def 'input'.


Este método no tiene sentido!


For now I just added this in my script:
function brew(){ echo ; }
right before the call to
source ./dev.sh
It's a dirty quick workaround, if someone esle will need it.
I'll try to return with PR in a couple of days.


Este método no esta bien!


This doesn't work because `process()` appends directly to the output builder.


причем тут носачфилд?


Ditto not using parens to call a method that is pure, and I'd tend not to name things that I'm tail-returning.


We have Chrome working. :)


It is not "hacking" to use it in keeping with the license. The license prohibits commercialization, but does not say it cannot be forked and extended so long as I do not charge.


### Debug this simple game program. The game should generate a random number between 0 and 100
### when the user hits Enter. The user can stop the game by typing "Stop".
### A score over 50 should result in a win.

import random

roll == raw_input("Press Enter to Spin the Wheel or Type ""Stop""!")

while roll == " ":
    prize = random.randint(0, 100)
    print "You're number is " str(prize) "!"
        if prize < 50
        print "Sorry, you didn't win. Try again"
        if prize > 50
        print "Congratulations! You're a winner!"

    roll = raw_input("Press Enter to Spin the Wheel or type 'stop'!")

if roll == "Stop":
    prnt "Thank you for playing"
else:
    print "Well, so long!"


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


I think this check and the next should be `frozen_<:<`


space


By duplicating a line and changing the solver to set a "total" value of 10 I get the solver to work more like I would expect it to.
solver.addEditVar(total).suggestValue(total, 10).resolve();

We now have 10  dogs
Plus 0  cats
For a total of 10  animals


Pretty cool. Part of my thinking when I switched it on by default was that your Vagrant work would make a Redis dependency no big deal -- is that wrong? If so, this looks perfect, but I was thinking that if Redis is in the Vagrant box, then they'd be all set.


Falta código para ver si es spam o no!


@alexsmithfanning I added it because the activity fades in from the right and fades out to the left if i close it, so it makes sense that a left facing arrow is closing it. It also better fits with the other look of the app, in the settings for example.


La detección del spam debería ir en el otro método!


add
ErrorDocument 404 "
Page not found
"
for stalkers


El commit no se corresponde con el código!


城会玩


This breaks Alloy's selective compilation feature. Sending a PR in some minutes.


ок


`col.filter(...).size` might be clearer as `col.count(...)`

I go back and forth on making anonymous functions more succinct (here's definitely a cutoff) but you might think of changing `c => c == '('` to `_ == '('`


Thanks @hjanetzek  :D.


Fine with me ... does it work well with multiple packages (`with_items:`), though? I think ansible has some kind of hack for package management modules and `with_items`.


Este método no está bien!


Updates to tests should always go together with the code changes they support.


Commit inicial das telas


Este método no está bien tampoco!


Hi, do you to consult a lawyer?  I have some good friends..


But that means you're having to tell the silver the solution. We want it to
find it by itself.

What we will do in the next commit is tell the silver that the variables
all need to be non-negative.

On Monday, December 14, 2015, Mara Alexa <notifications@github.com> wrote:

> By duplicating a line and changing the solver to set a "total" value of 10
> I get the solver to work more like I would expect it to.
> solver.addEditVar(total).suggestValue(total, 10).resolve();
>
> We now have 10 dogs
> Plus 0 cats
> For a total of 10 animals
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MaraAlexa/constraintsProgramming/commit/962df7140a03a4cd95f8263fd88ba59ba9a833dc#commitcomment-14964554>
> .
>


Este método no va en esta clase!


你这个。。我就commit了一行代码＝ ＝。。。 现在在纠结是vim＋eclim还是eclipse＋vrapper。。感觉java没有ide和auto complete写起来要吃屎的。。


já je pak opravím


Esto no es necesario aqui!


reack


Since you also changed `typeRef` to always do symbolic typerefs I think this comment is out of date.


Este método no funciona bien!


Don't need comma at the end.


тоже стрим, не?


This is general comment


Needed rebase and signature.


Chance isn't utilized in crop processing at all, or in nerfed comb processing. (Not sure if that's intentional)


extra line


uncorrect? :-)


good spot, thanks!


Esto no funciona bien!


Yeah I double checked, the point of using pkg-config is that if libkqueue is installed in a weird place, and needs extra arguments past -lkqueue it still works. it will define KQUEUE_LIBS KQUEUE_CFLAGS and friends, and you should use KQUEUE_LIBS and not AC_SEARCH_LIBS. libpthread_workqueue doesn't seem to have a .pc (according to https://packages.debian.org/sid/amd64/libpthread-workqueue-dev/filelist) so its case is slightly different, even if it would likely be cleaner to when you find the headers, define PTHREAD_WORKQUEUE_LIBS too.

please use KQUEUE_LIBS in src/Makefile.am instead for that one.


найди нормальную версию гуавы


Added conf.php to the script and updated rar.


i think you misunderstood me here. the neat thing with c++11 is that you can initialize the member directly.

```
struct foo
{
  int A = 3;
  std::string B = "bar";
}
```

this allows you to kill the constructor completely.


Estas contando dos veces por cada mensaje recibido!!


Looks great man.


两个对象动画，分别会弹出两alert是正确的   this指向的是动画对象，alert(this.id)一目了然，并没有写错


I don't think you need async here (sorry for not catching that the first time)


вроде тебе не нужен декларед, просто конструкторс.


Yay, veiled threats!

> Developer grants a non
>  exclusive perpetual license to use the Background technology in the Software developed for and delivered to Customer under this
>  Agreement. However, the Customer shall make no commercial use of the Background technology without
>  Developer's written consent.

In lovely recursive fashion, I have cut and pasted the relevant part of the license, which itself appears to be a fragment cut and pasted from a web application license.


TeamCity pull requests :: netty [Build 862](https://garage.netty.io/teamcity/viewLog.html?buildId=4071&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6229, ignored: 21 Build time: 00:44:53


Don't commit directly to the `modules/pmi` directory in IMP, please. Make these changes in the PMI repository and then run `tools/git/update-pmi.sh` in IMP.

I will revert these commits.


.. and this was placed on the wrong line ;)


ну и название :-)


Este metodo no devuelve null si el correo es spam!


You could mock the class itself.


Can we remove the extra comment stuff if not being used?


@disconn3ct, I will delete your posts and this is not relevant to the work I'm trying to do here.


Actually.. everything before https://github.com/yracine/device-type.myecobee/commit/22f52f3edafed30d59d5df347aceaa60aff6073a was licensed under Apache.

So no worries. I'll revert my fork before that and work from there, under the apache license.


Con tu código aqui necesitarias comprobar si es spam!


reack


No, it should.


调用 initEncrypt 的地方都有检查 enc 是否初始化，这里不用判断了吧。


тут что-то не так, должен быть способ проще.


That is fine, this is your playground and you are taking your ball and going home... but just out of curiosity what is the work you are doing here?


@Stasik0 Are you sure about this line?

a) We init a bytestring without freeing first, possibly leaking data
b) We copy UA_STRING_NULL into the request. But the nonce in the request has already be initialized to UA_STRING_NULL

And yeah, that was bad already in the existing code.


OK!


Please unstage this file from the commit. Nothing meaningful happened here.


I like this implementation. You could consider it as looking a bit like List.scanLeft, where we abstract away from keeping track of the index and the recursion:

`input.toIterator.scanLeft(0) { case (floor, char) =>
  case ')' => floor +1
  case '(' => floor - 1
}`

That gives us a running track of what floor the lift is on. To get the first time it goes into the basement:

`.takeWhile(_ > 0).size`

Without using a scan, I'd probably look at inlining the if-else into the pattern match, but that's a tiny nit.

+1 for making it tail recursive.


Luckily, we don't need those. I'm removing those from the tested patch as well.


I'll try it... https://developer.mozilla.org/en-US/docs/Web/API/Blob/Blob


Not needed.


Yeah, I see what you're saying. I wouldn't have known that. KitKat animations aren't really a thing.


Remove parenthesis, and remove whitespace between if and else causes, as well as at the end of functions. There is no reason to have an extra space between the last line of code and the closing brace.


Faltan los mensajes enviados!


TeamCity TripleA :: UnitTest [Build 473](http://home.k-moeller.dk/viewLog.html?buildId=1118&buildTypeId=TripleA_UnitTest&guest=1) is now running


Ahhh. Ok. So we give it general boundaries like (not negative) and then let it figure it out. That's even better. :)


А, вот оно как все устроено. RSpec умнее, чем я думал. Буду знать, спасибо)


You should actually ensure that the _run_and_check_output method of this class is used, no matter how you call it.


Not a huge thing, but it's good to be aware of the App trait, which defines a main for you and lets you just run code in the body of your class.


I don't have xelatex to compile the different files I added so there are some links (chapters/equations) I can't do..
I will try to find a solution tonight


TeamCity TripleA :: UnitTest [Build 473](http://home.k-moeller.dk/viewLog.html?buildId=1118&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


Debería de poder volver a false no solo con este if!


I guess the only change that fixed the bug for @NetoLe is setting
`opnSecRq.securityMode = UA_MESSAGESECURITYMODE_NONE /* == 1 */ `
also when renewing the channel. The securityMode is initially 0, i.e. `UA_MESSAGESECURITYMODE_INVALID`.


shapefile.py has been removed from the development version of basemap.  It is now in the [pyshp projection](https://github.com/GeospatialPython/pyshp).


Overall I thought this was really well-structured and readable.


May need to check the return value of `_trigger` instead of checking `event.isDefaultPrevented()`. Something like [this in draggable.js](https://github.com/jquery/jquery-ui/blob/ffcfb85c9818954adda69e73cf9ba76ea07b554c/ui/widgets/draggable.js#L291-L293) or [this in spinner.js](https://github.com/jquery/jquery-ui/blob/master/ui/widgets/spinner.js#L331-L334).


Ah, sorry, I think I misread what the string you try to find in metadata.values() represents. It still doesn’t look like you’re getting the data type from the value of a specific field in the meta file though


The chances are only for direct processing into metal dusts that can be enabled per config.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.937](http://teamcity.monogame.net/viewLog.html?buildId=12213&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:23:02


ack


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


Esta condicion no esta bien!


Really like this pattern-matching assignment.


Use `getInferredClassType`.


Hay que inicilializar los atributos!!


it's fine I got it - those in-member init makes sense if there are different ctors, so I will use them - cause someone might forget to init those in the future.

Thanks again.


Not sure if the extra line in here is on purpose?


heh, bake-a-cake. - ack


Falta lo del mensaje más largo!


Fixed in 44f2f51.


I'd avoid the type ascription here, with split being so close and obviously returning ints.


Rename `selector` argument to `state`? Since its used both as selector and class.


You should test that this method is called with the expected args.


ah, no problem


For this class in general, if you're not going to use the AWBanner open source project, don't call it AWBanner. BannerView will be sufficient. If you're simply reusing the 3rd party component then we should simply add it as a submodule and reuse the code from them instead. Let me know your decision on that.


good. i just saw you changing to an init list and thus i wanted to make sure.


Que ha pasado con este commit??


Many tests are failing:
```
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/imap/tests/utils.py", line 87, in setup_server
    soledad=self._soledad)
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/imap/server.py", line 76, in __init__
    imap4.IMAP4Server.__init__(self, *args, **kwargs)
exceptions.TypeError: __init__() got an unexpected keyword argument 'userid'

leap.mail.imap.tests.test_imap.AccountTestCase.test_illegalMailboxCreate
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testCapability
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testCapabilityWithAuth
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testCheck
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testCreate
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testDelete
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testExamine
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testExpunge
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testFailedLogin
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testFailedStatus
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testFullAppend
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testHierarchicalRename
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testIllegalDelete
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testIllegalInboxDelete
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testIllegalInboxRename
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testLSub
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testList
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testLogin
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testLoginRequiringQuoting
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testLogout
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testNamespace
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testNonExistentDelete
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testNoop
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testPartialAppend
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testRename
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testSelect
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testStatus
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testSubscribe
leap.mail.imap.tests.test_imap.LEAPIMAP4ServerTestCase.testUnsubscribe
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/imap/tests/test_imap_store_fetch.py", line 60, in testMultiBody

exceptions.AttributeError: 'StoreAndFetchTestCase' object has no attribute 'client'

leap.mail.imap.tests.test_imap_store_fetch.StoreAndFetchTestCase.testMultiBody
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/imap/tests/utils.py", line 118, in tearDown
    del self.client
exceptions.AttributeError: client

leap.mail.imap.tests.test_imap_store_fetch.StoreAndFetchTestCase.testMultiBody
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/trial/runner.py", line 497, in loadPackage
    module = modinfo.load()
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/modules.py", line 392, in load
    return self.pathEntry.pythonPath.moduleLoader(self.name)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 303, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 250, in _importAndCheckStack
    reraise(excValue, excTraceback)
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/imap/tests/test_incoming_mail.py", line 33, in <module>

exceptions.ImportError: cannot import name SoledadBackedAccount

leap.mail.imap.tests.test_incoming_mail
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/trial/runner.py", line 497, in loadPackage
    module = modinfo.load()
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/modules.py", line 392, in load
    return self.pathEntry.pythonPath.moduleLoader(self.name)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 303, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 242, in _importAndCheckStack
    return __import__(importName)
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/tests/test_mailbox_uid.py", line 34, in <module>

exceptions.TypeError: Error when calling the metaclass bases
    Cannot create a consistent method resolution
order (MRO) for bases SoledadTestMixin, TestCase

leap.mail.tests.test_mailbox_uid
===============================================================================
[ERROR]
Traceback (most recent call last):
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/trial/runner.py", line 497, in loadPackage
    module = modinfo.load()
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/modules.py", line 392, in load
    return self.pathEntry.pythonPath.moduleLoader(self.name)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 303, in namedAny
    topLevelPackage = _importAndCheckStack(trialname)
  File "/home/meskio/dev/leap/env/local/lib/python2.7/site-packages/twisted/python/reflect.py", line 250, in _importAndCheckStack
    reraise(excValue, excTraceback)
  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/tests/test_service.py", line 31, in <module>

  File "/home/meskio/dev/leap/leap_mail/src/leap/mail/service.py", line 32, in <module>

exceptions.ImportError: cannot import name proto

leap.mail.tests.test_service
-------------------------------------------------------------------------------
```


that's ugly, why use an exception for logic like that?


Switch to mock.sentinel.adapter_handle


Excellent. Think this was better than my solution.


Run multiple forks to account for run-to-run. For reference, the default is 10


Claro que "XML" era "XLS" :-P


unack, come on, didn't we talk about this function being *way* too long?


No recibes el email del servidor!!


`Review.reviews` is a function so you need `()` since this isn't Ruby :)


So combs aren't supposed to use it in "nerfed" mode, but what about crops? They don't use it in either mode.


Este método no funciona!


Muito bem, sim senhor! :+1: :smile: :100:


Closes #16.


I think I prefer the previous solution!


No need to mock this.


In the logic that handles `new` you also check whether the symbol is a `SymbolFlags.Function`. If you need that there then you also need it here, no?


:+1: for shortening this mega long type name


No lo actualizas en ninguna parte!!


please make a wrapper like `extract_documentation_style_from_string`, define the interface clearly! You want a list of strings! You don't want this or that or that. We're designing by contract here, that should be clear.


What handler?


And pushed again.


Esto no tiene sentido!!


I think trading a MatchError for a RuntimeException isn't necessarily a great exchange.


Thanks for catching it!


or don't do the wrapper at all, we need this function *nowhere* for strings and it's a oneliner to do if it's ever needed. I'm strongly against including those lines.


You can determine the raised exception (if any) based on the other two arguments.


Or `this.copy(y = y+1)` (with or without the `this`).


No inicializas el atributo!!


please put this stuff into a function with a well described name and put your comment in the documentation comment, not a bunch of statements with a huge ugly doc comment


s/test_open_adapter_with_name/test_adapter_by_name


this loop is unreviewable, please split out parts into functions.


There is a field, `testFile`.  I have updated the Javadoc.


I will try to make the rapport “compilable” with pdflatex.


Well done!


@nesl247

I was thinking about using [`Emitter::ensureListener($listener)`](https://github.com/thephpleague/event/blob/master/src/Emitter.php#L106-L117) here maybe, but not entirely sure:

```php
/**
 * @return ListenerInterface
 */
public function getListener()
{
    if ($this->listener === null) {
        try {
            $listener = $this->container->get($this->alias);
        } catch (ContainerException $exception) {
            throw new \BadMethodCallException(sprintf(
                'Unable to fetch a service for alias "%s" from the container',
                $this->alias
            ));
        }

        $this->listener = $this->ensureListener($listener);
    }

    return $this->listener;
}

/**
 * @param mixed $listener
 *
 * @return ListenerInterface
 */
private function ensureListener($listener)
{
    if ($listener instanceof ListenerInterface) {
        return $listener;
    }

    if (is_callable($listener)) {
        return CallbackListener::fromCallable($listener);
    }

    throw new \BadMethodCallException('Fetched listener neither implements ListenerInterface nor is a callable');
}


```
local function check_output_alias(ent, portname)
    if ent.OutputAliases then
        return ent.OutputAliases[portname] or portname
    end
end
```
is in my opinion easier and shorter than your version


Pides otro email???


You should move this within the "with" block.


I'm totally against you using an exception to break out of loops, do you know the `break` statement? If your stuff is too complicated for that, just introduce a state variable, like `breaking = False` you set to `True` if you want to break out. This is highly unusual


Fixed in cfb6e63.


@suunrider, still wrong.

```Lua
local function check_output_alias(ent, portname)
  return ent.OutputAliases and ent.OutputAliases[portname] or portname
end
```


done.

On Sat, Dec 12, 2015 at 3:51 PM, Cosmin Radoi <notifications@github.com>
wrote:

> @msaxena2 <https://github.com/msaxena2>, did you get this? Please delete
> the branch once you use it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kframework/k/commit/622b5d7df322c48382d395d98d1b5e2d56b6406b#commitcomment-14943341>
> .
>


Arkadaşlar merhaba. Demo öncesi son kontrollerimi yaparken sizin siteden çok bir şey anlamadığımı farkettim. Yarın öğleden sonra herhangi bir grup üyesi Lab 2'ye gelip site üzerinde örneklerle ne olup bittiğini gösterebilir mi? Ayrıca sürekli internal server error da alıyorum. egazioglu@itu adresi üzerinden öğleden sonrası saat için randevulaşabiliriz. Kolay gelsin.


Ok, nevermind my other comment. But I still think you need to check `SymbolFlags.Function`.


that'd be a simple `break` wouldn't it?


this isn't working on bash shells. $list probably needs to be quoted.


It's definitely possible to lean on Scala collections more here:

```scala
input.scanLeft(startingPoint) { case(last, elem) => last.move(elem) }.distinct.size
```


En más sitios debes actualizar `recibidos`!!


yup


The name of the field is currently not translated. Could I simply add every field to [templates/i18n.html](https://github.com/c2corg/v6_ui/blob/master/c2corg_ui/templates/i18n.html) and then provide a translation for each field label?


the params here:
```
     r.get_unread(unset_has_mail=True,update_user=True)
```
already does this


`Insert` is a suboptimal name as STL's one does not overwrite.


I'm having someone here at utah running: "python gemini_install /usr/local /usr/local/share/" and still getting an error with annotation_dir. Is this due to them getting an older version from conda?


Mine is easier to understand :(


Although some would say that no comment is a bad comment, I prefer to have comments explain what code does instead of how it does it.

In this case we `preventDefault`ed then told the reader that we "prevented" something?


I didn't know that sliding can take a stride argument. This is nice.


Hi,

You should check if the ngModelCtrl.$$parentForm is defined before accessing .$submitted

This throws an error when I have a input inside a directive with an isolated scope and the form is injected


Reemplaza todas las apariciones??


I think I saw it on Hacker News? It's pretty fun -- me and a coworker are both working our way through it right now. Let me know if you start, would be cool to see your solutions


What do you think of `s/FieldsByReference/Data/g`?


And the pressures can't be placed on the fence gate xD


TeamCity TripleA :: UnitTest [Build 474](http://home.k-moeller.dk/viewLog.html?buildId=1122&buildTypeId=TripleA_UnitTest&guest=1) is now running


`EXPECT_FALSE(Exists(fields.d["three"]));` ?


Te falta marcar spam a false


Except it doesn't even work


Fixed in 24a2530.


TeamCity TripleA :: UnitTest [Build 474](http://home.k-moeller.dk/viewLog.html?buildId=1122&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:43


Abigails does work, tricky lua shit


What if we would not mock ctypes.c_char? This would be tested better.


added


Just out of curiosity what is `winfo` referring to?


Why append " Accessory"? I made a similar change locally and it works fine without any suffix. Does Siri ignore "Accessory"? I wouldn't want to say "Accessory" all the time.


No necesariamente es spam!


Qué paso con este commit??


This is great, very clean.


Good point.

There are a lot of places in RC where we want to know whether the retval of an expression "matters" or not. Adding a general-purpose method which can traverse the tree and figure this out will be very worthwhile.


at the moment, you're calling this on a mock object, so the 'strip' argument is ineffective unless you do something like mock_buff[:].strip.assert_called_once_with('\x00')


No hace falta!!


Couldn't `_SessionCoordinates` normalize this one way to another so clients have a definite expectation?


This is typical thing where one would really want to expose the special case on the AST level as specialization. (only if it is really worthwhile to optimize anyway, because this is BigInteger arithmetics, which ideally is not necessary at all)


how ready is android 6.0 for the sm-p600?


Yup, thanks


How can you add AND fix score at the same time?


why not mock_mapping = mock_get_target_mapping_struct.return_value


Why remove it? Now it sounds like it's speaking in general, while I believe this is referring to Rexipher which is the evil presence roaming around in the mausoleum.


Is there a ticket for this? If so, you should add it to all the commit messages.


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


The first sentence requires attention. 'just under attacked' ?


This should be part of a different test.. I'd remove it.


This should actually be a side effect, testing the case when there are more elements then requested, ensuring that this case is covered.


When I tried renaming the services through the iOS API, it seems HomeKit would not allow naming the service the same as the accessory. Plus, Siri uses the name of the service (not the accessory) for control since the accessory could have multiple services (Even when the accessory has only one service, from my findings).


Reemplaza todas las apariciones??


The "expected value" should always be the first - switch the parameters please.


That works nicely. I'd also change the `@param mixed` to `@param ListenerInterface|callable`.


Remove the period.

Also, 'Stone Lantern'?


This still needs fixing


Avoid calling mocks like this.


or just `input.count(part1)`


This function is used to simplify the BigInteger results. I mean for instance, ((1 bsl 20000) & 16#10000000000000000000000001) will result 0 as BigInteger (but both arguments are BigIntegers). But to keep it as BigInteger is crazy, better to simplify it to long.


@no1dor для интеграции добавить новый класс на контейнер `.navbar--submenu-tiles-container` и класс-модификатор `.menutiles-large` на сами `.menutiles`


Reemplaza todas las apariciones?


Good stuff.


So that's it..., it was -> So that's it... It was


Для мобильного меню не надо никаких изменений


second symbol of ordeal


Nitpick: I think we can do more cleaning here.

These methods do not require the parameter `event` as available via `self.event`. Instead we can maybe redefined them as Python `property`.


TeamCity TripleA :: UnitTest [Build 475](http://home.k-moeller.dk/viewLog.html?buildId=1126&buildTypeId=TripleA_UnitTest&guest=1) is now running


suggest:
```
self._app_path = kwargs.get('app_path', '/')
if self._app_path is None:
     raise ValueError("app_path cannot be None")
if not self._app_path.startswith("/"):
    raise ValueError("app_path should start with a '/' character")
```


not prepared -> unprepared


Begin to study Python


I think this is ok. I think originally win_multiarch was to be set by a checkbox in the UI or by some property in the message received to kick off a build. But that clearly never happened to it makes sense to remove conditionals that are not really conditional.


ОК )


Did you investigate using angular's $cachefactory? https://docs.angularjs.org/api/ng/service/$cacheFactory


Nope, the additional problem was that the server issued a new token, bit the client was using the old one which eventually timed out. It happend because the nodeid of the node was not copied over in case of review.

Im on iphone now, cannot see the marked line atm.

> Am 14.12.2015 um 18:21 schrieb Julius Pfrommer <notifications@github.com>:
>
> I guess the only change that fixed the bug for @NetoLe is setting
> opnSecRq.securityMode = UA_MESSAGESECURITYMODE_NONE /* == 1 */
> also when renewing the channel. The securityMode is initially 0, i.e. UA_MESSAGESECURITYMODE_INVALID.
>
> —
> Reply to this email directly or view it on GitHub.
>


TeamCity TripleA :: UnitTest [Build 475](http://home.k-moeller.dk/viewLog.html?buildId=1126&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:45


That's a bit ugly and not necessary since the names of the fields are already extracted from the "view" and "edit" templates.


done to done


Qué ha pasado con este commit????


Simple, clean and gets the job done :) Don't forget to use your custom print method in your for loop.


Unpopular opinion here: I find regexes pretty unreadable except when they're very simple and would avoid using them (especially if I were learning Scala). Specifically, I'd find it very hard to be confident that I'd see bugs in the implementation of `doubleLetter`, `hasMultiplePairs` and `hasRepeatWithDistinctMiddle`.


We don't need set it as uncapturing widget anymore, see https://github.com/rsredsq/AtomicGameEngine/commit/5d4cb694642ca6368ec26a5f575604cea8404e07#diff-fde1469d69986e75881b81a15dd1f531R247


Late to the party, but this seems wrong. Setting `fieldLength` to 4 should produce 4 characters from the actual input, *This* in the case of your test, the `...` being added shouldn't be part of that total character count.


No es necesario!!


suggest:
```
self._base_url = kwargs.get('url', DEFAULT_SERVER_HTTP_URL)
if self._base_url is None:
    raise ValueError("url cannot be None")
if self._base_url == 'default':
    self._base_url = DEFAULT_SERVER_HTTP_URL
if self._base_url.startswith("ws"):
    raise ValueError("url should be the http or https URL for the server, not the websocket URL")
```


I'm uncertain about this change. This type of messages appear in a black background with beigeish color. It's very plausible that this is in the I-form.


Don't need anymore


We'll get you using `dict.get` yet :)


Yes, but if this is on the fast path, so, if this is ever performance critical, this should be turned into a node that has `bi.bitLength() <= 63` as a guard and to the `longValue`, and fall back for the 'just return' case. That way, the Graal compiler will see that the operation _always_ results in a long, and never in an object (but perhaps a deoptimization, i.e., going back to the interpreter).

I am just saying this, because these are Truffle/Graal basics. Since but since BigInteger is the input type, it ideally is in this specific case never on the fast path anyway. Still, the same principle applies to 'all the other things' in the interpreter, too.


Phew. Let's close this ticket and pretend the madness never happened.


Les attributs "color" dans les fichiers JSON peuvent être retirés (leur couleur est déterminée aléatoirement parmi celles disponibles)


Explanatory attachment

![fixup](https://cloud.githubusercontent.com/assets/318549/11788842/20dceda2-a27a-11e5-9e9e-ee9b6ca73b83.png)


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


Can you fix this too? Clearly I wasn't paying enough attention when I worked on this file. :)


`` size of the input ``


Looks good!


Thank you for all comments


commit comment'i eksik olmuş tabii.. :)


Even better ```Received excess data on input reader. Expected only ‘%d’ bytes, but received ‘%d’ bytes.```


Oh, I see. How do you write a guard which depends on a property of the result? In this case `bi.bitLength() <= 63`.


One other change if you don't mind:

```ruby
defaultfor operatingsystem: [:ios_xr, :nexus]
```


Hi Mohsen,
Thank you for your pull request.  I don't think we need to add an import of DiagnosticSeverity to the client code.
It does look like the walkthrough is stale compared to the current version of the language-server-node-example code (https://github.com/Microsoft/vscode-languageserver-node-example).   We'll update the walkthrough code extracts in our next release.


Cool :)


Regexes are extractors in Scala, so you can match with them a bit more fluidly:

```
input match {
  case regex(op, fromX, fromY, toX, toY) => Command(getState(op), fromX.toInt, fromY.toInt, toX.toInt, toY.toInt)
}
```


This seems large, but it is only temporary until we have a better test suite


Would like to merge this one change back into master


Ugh. In that case I should probably just move service into it's own provider.


Wow do I hate where this conversation keeps going...

@jaypeecee, firstly I totally agree. This project clearly took a lot of time and efforts to bring it to where it is now, and we are very grateful for all of the hard work that's been put into this project. I wish end users would see the time it takes to do something like this. If you aren't helping develop these programs it would be reasonable to assume that a small donation here and there should be made. Personally I'm in the same boat as @disconn3ct, I just bought my thermostat and smartthings hub recently and hadn't even gotten the apps fully working before this code was taken down. I was hoping to contribute, or donate, but I'm a bit concerned about how all this is going down.

I was also like @disconn3ct and I bought my ecobee because I saw, what I thought, was an open community of support. I was quite surprised when I suggested a change in the file structure that @yracine suddenly became overly concerned about people touching his code. (@StrykerSKS offered to re-arrange the file structure to allow it to sync with the IDE more easily and was told this was in violation of the copyright). Within a few days the code was removed off Github and became pay-only.

I'm new to all of this, but it was my understanding that a community of developers help each other out by cleaning each other's code and adding fixes and features etc. It seems that @yracine is not interested in this. And I guess that's what surprised me most. Developing something like this is a lot of work, why do it alone?

@jaypeecee : I like your thought about having a 'donate for support' option. @yracine, if you are spending a ton of time troubleshooting people's code then it would make sense for you to get paid for your time. I would like to add that you don't have to be the only developer working on this code. I'd suggest that if you open the code up then other developers can jump in to help you work on this, and in so doing have less burden on you. However, from your most recent comments I'm not sure that this is something you would consider. I'd love to be wrong.

Again, I want to re-iterate that I respect your decision @yracine to make this a pay for use, but at the same time I'm obviously sad to see that.


I am not sure what it going on here, but it looks like you thought it through :)


Ok. Im just worried about the uppercase names and such. We can review/check later at night

> On 14 Dec 2015, at 17:48, Álvaro Felipe Melchor <notifications@github.com> wrote:
>
> readelf gives the same info. Remember that this memory is not mapped inside io logic/magic is just showed with iS command
>
> —
> Reply to this email directly or view it on GitHub.
>


@yagebu The filters are now applied for every request the web server gets, correct? The filters should only applied if a filter has changed.


@azturner We're seeing a timeout trying to perform this query when it includes the ".Account" on the Queryable. I temporarily removed it from the block on our prod-test system and it's working (very quickly).  @mikepetersonccv are you guys having any issues with this viewing a batch details?


Probably the node/specialization would look something like this `@Specialization(guards = "bi.bitLength() <= 63") public long toLong(BigInteger bi) {...}`


Nice cleanup, thanks, this made me feel a little dirty :+1:


can you move this over to /tests/utils/testUtils.js? looks like a handy function that could be resused


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.938](http://teamcity.monogame.net/viewLog.html?buildId=12217&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 552, ignored: 6 Build time: 00:31:46


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


jmptbl option passes all tests now


TeamCity TripleA :: UnitTest [Build 476](http://home.k-moeller.dk/viewLog.html?buildId=1130&buildTypeId=TripleA_UnitTest&guest=1) is now running


Began work on writing the ROCParameter Object in R. Did some file clean up with ROCParameter (header and cpp).


Instead of here I would put all of them in `require-node-test` so that we know everything is working with `IO`


I am aware that we are talking about message digests here. However the initial failure output I kept seeing said "Bad md type" which lead me to believe we did not support a sha256, sha384, or sha512 message digests. This made no sense to me and led me down a rabbit trail before realizing that it was supported, just not accounted for. That is why I changed the debug message to say "not yet implemented" as in it may be supported and is just not implemented in our compatibility layer. It is more then likely not a "Bad md type".


I tried to replace `symbolicTypeRef` by `typeRef` in the expression below and `Ycheck` failed again for `t3452h.scala`, so something weird is going on, probably related to the caching done in `typeRef`


TeamCity TripleA :: UnitTest [Build 476](http://home.k-moeller.dk/viewLog.html?buildId=1130&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:44


In practice I don't use any data that requires `waitFor`. Also, some time ago I've got an issue with my current project that also uses [retransmitter](https:/github.com/alexeyraspopov/retransmitter). I wasn't able to fix the actual problem so this change was the only one solution.


Ah, okay, got it. So leave the `BigInteger` part as is (whatever the actual result is), and insert a new node that may convert back to `long`. Right?


TeamCity TripleA :: UnitTest [Build 477](http://home.k-moeller.dk/viewLog.html?buildId=1132&buildTypeId=TripleA_UnitTest&guest=1) is now running


Right, testing the current release version of foundation, both methods actually include files prefixed with `._`. I'm updating this PR to reflect that.


Bit of a nit, but I'd prefer something like:

```scala
 = {
  for {
    x <- lights.indices
    y <- lights(0).indices
  } yield lights(x)(y)
}.sum
```


TeamCity TripleA :: UnitTest [Build 477](http://home.k-moeller.dk/viewLog.html?buildId=1132&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


should be "reasonably" I guess


Please move these declarations above where they are first used, i.e. above `getImage()`.  Also, now that`validImageExtentions` is declared outside the scope of `getImage` and `remoteHandler`, you shouldn't need to pass it around.


This is in master, add it to the OCD list because no one is going to fix this straight onto master.


don't forget to purge `goog.string` and `goog.string.StringBuffer`, see `require-node-test`.


That there is even the chance variable in crops is only because of C&P. With disabled config bees and crops are meant to work the old way and that is bees using chances to reduce output of higher tiers, crops using different growth times.


This could be simplified to `/src="([^"]+)"/`.


Sorry; these Fortran things slip by me.  :-(

I see you doing some work on this in open-mpi/ompi#1217.  If that split-into-three-files thing works, then let's go with it.


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


maybe swich the if with else: `if not begin_match: do_not_so_much; continue; else: do_a_lot`


ADS1115_REG_CONFIG_DR_8SPS is zero, !0 is true and you get an error if you try use sps=8.


TeamCity TripleA :: UnitTest [Build 478](http://home.k-moeller.dk/viewLog.html?buildId=1139&buildTypeId=TripleA_UnitTest&guest=1) is now running


Don't we need to validate the extension on `imgurl`?  Could it be something we don't support, .tif, .svg, etc.? (Genuine question, I don't know).


TeamCity TripleA :: UnitTest [Build 478](http://home.k-moeller.dk/viewLog.html?buildId=1139&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:46


TeamCity TripleA :: UnitTest [Build 479](http://home.k-moeller.dk/viewLog.html?buildId=1140&buildTypeId=TripleA_UnitTest&guest=1) is now running


Please please do not push commits without my review.  All pull requests need to be approved by me and I have not (intentionally) given commit access to anyone as of yet. These are unreviewed changes which may have unintended consequences on the code, and may need to be reverted.

I'm fairly new to GitHub--I apologize if I haven't set things up properly to make this clear.

Can you give me examples where an accidental following a root indicates a change of chord root--a Neapolitan notated, say, as IIf rather than fII?  If that is so, then I'll assess this in that light.  Otherwise, alteration belong with figures, or perhaps with quality.


@TheSnoozer what do you think - is it possible to parse this data from font awesome instead of merge all future updates here?


Cool, just wanted to make sure the chance variable wasn't accidentally unused. I'm enjoying Experimental so far, keep it up.


TeamCity TripleA :: UnitTest [Build 479](http://home.k-moeller.dk/viewLog.html?buildId=1140&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


Yes, I assumed that was the case however I wasn't sure how to handle it from a manifest perspective. Should the user specify a list of supported cards in the manifest?
How do we know the order of the card types? (`limit-resource module-type m1 m1xl m2xl f2e `


I ended up pushing this commit (and a few others) to a new PR: https://github.com/trotzig/import-js/pull/71 . I wanted to keep the discussion here intact, so I created a new branch.


TeamCity TripleA :: UnitTest [Build 480](http://home.k-moeller.dk/viewLog.html?buildId=1144&buildTypeId=TripleA_UnitTest&guest=1) is now running


The second line leaks `this` while the object is being constructed.


> Just set your umask.

I just realized this is not a umask thing, in unix. My umask (0022) only filters the write bit for "group,other". Still if I `touch base` the file "base" will be rw-r--r--. But you might be able to get around this behavior (I presume it's a Cygwin thing) using `umask 133`.


TeamCity TripleA :: UnitTest [Build 480](http://home.k-moeller.dk/viewLog.html?buildId=1144&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:41


Hi, It's a personal decision based on what's the best for my family and myself. I think that I'm entitled to be rewarded for the work I've been doing for the past 18 months.

It's started to be just a hobby, but now it's getting more serious considering the work I've done since the beginning of SmartThings.

Architects/Developers like myself don't work for free forever.

And like my dad used to say 'there is no free lunch'...

I'm just amazed that some people do not want to contribute after using the device and the smartapps for  a while....It takes a lot of development and testing to build such an ecosystem.

I do not think that I'm asking that much...But,eh this is my point of view (just ask my wife and she will answer with more sarcasm).

Regards.


Is it worth turning this into an Error struct so we can include information like the username of the account that failed?


Oh, thanks for getting to this faster @sgomes. My fault on this one, git only showed me the second method as a conflict area.


Why are we modifying the site name ?


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


@ask in debugging session I see the socket error is because kombu trying to connect rabbitmq on localhost, but actually I have configured `BROKER_URL` to point rabbitmq on another machine in django settings.py and celery worker load it correctly; why in django runserver it's not reading that?

http://docs.celeryproject.org/en/master/django/first-steps-with-django.html

```console
.../lib/python2.7/site-packages/kombu/connection.py in _ensured

<Connection: amqp://guest:**@127.0.0.1:5672// at 0x7f4ce9f96110>
```


This change is to prevent rescanning the entire chunkmap every run. There's no point in scanning chunks that we know from previous runs aren't jumbo. This drastically improves hunter performance while trying to split down a 900 million doc chunk.

You'd be expected to drop the output collection and start from scratch after a compaction or running the balancer. Readme should be updated to reflect this.


LGTM!


Is there an easy way we can differentiate between the two error cases for Two Factor auth here?


The site tag should be normalized here no ?


@lucasart @mcostalba @burak13 : I do appreciate your comments, but let me explain few things:
1) This small change moved from +2.8 ELO to +6 ELO on STC. But OK, I'll test it together with some other
    improvement.
2) The main problem with reduction formula is the same move-ordering subformula for all depths.
    I guess this is +10 ELO if somebody picks the right thing. But OK, I'll test my ideas locally and submit
    only if there will be a sign for such big progress.
3) I was going to the trip and submitted those two tests before my travel, knowing that I'll maybe not
    submit for a longer time.
4) There are problems with SPSA tuner and even Lyudmil admits that. If we use too big c_k values it's
    possible to get wrong results because of non-convexity. If we use small c_k values, we'll get only
    small change with reasonable number of games. I'll use SPSA only occasionally (or with small
    number of games) to get the direction of the change and then try to improve it locally.
5) I have nothing against good psychotherapist, but that is not developed science. It's better to use all
    other people for that purpose.


closes #193


I think I finally understand this function.  Could you please rename it to `extractValidImageUrl` to better indicate what it's doing?


Ha! Yes, I'm sure she would!

Best of luck to you @yracine.


@gEndelf - sure this is possible and this was one of the thinks I kind of cherry picked as well. It's kind of a parse but out can check it out here:
https://github.com/TheSnoozer/font-awesome-to-png/blob/master/update-font-awsome-icons.py

Haven't tested with the current version of font awesome but could check on that :-)
Not sure if a runtime parser is a waste of time since the icons does not update that frequent....rather prefer an offline solution that needs to be maintained though...


Seems like a good start, but I wonder if there is a more comprehensive list of things we should look for somewhere?


why `expt` instead of `experiment`?


Hay que comprobar que no sea null!!


And, I'm not the only one that wants to be paid as a developer (just look at the ST community thread).

https://community.smartthings.com/t/developer-run-smartapp-device-store/23080/7
https://community.smartthings.com/t/build-an-paid-app-on-app-store/6935
Regards.


This stuff? Yea, it should be done for now. Rewriting it with the new and improved (tm) timing information can be postponed.
Disclaimer: I do hope this is bug-free. I've run it through the debugger and all the timing information is sensible. But I still want to do a manual comparison of this code with the last (nonsensical) stuff. Will do that when I have the time.


My bad! But, I think you can use `%Q` instead of `%q`, which might be a bit nicer than escaping all those double quotes.


Sugiero que no se cambien las modificaciones de la BD, ya que al hacer el pull-request estos cambios se integraran tambien. agrega estos campos a tu .gitignore o bien no los agregues en tu commit...


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


with the change to getCommand above, this could turn into `input.map(getCommand).foreach(update(lights, _))`. If you curry `update` it's even nicer.


Is it idiomatic to do the _ with a  separate getter? Seems like python...


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


Nicely written, easy to follow code. I like your getWord function. Everything looks good here :)


Oh, sorry.  I think I just misread everything.  I told you I'm fairly new to this :(

I see that this hasn't been pushed to master; I didn't notice that I was looking at your _fork_

Please forgive any offense, just a momentary confusion on my part!  I will look through what you've suggested.  It would be great if you have an example of this usage, b/c I've always seen the alteration before the RN.

Best,
David


This is only a direction of the change. For serious tuning someone needs small c_k values and extreme number of games. Higher c_k values will give wrong results if they catch non-convexity. I'll refine this idea with local testing before submitting a test.


This would be a little better if you used backreferences. Something like this maybe:
```
/.*('|")([^\1]+)\1 is not defined/
```


Isn't this part of `babel-preset-es2015` [which lists using this plugin already](http://babeljs.io/docs/plugins/preset-es2015/)?


If you go this route, you'll obviously need to update the next line as well.


@TheSnoozer ok, thx )


Same comment here about backreferences.


Mmm, ¿leíste la cita de nuestro paper que puse en la introducción? ¿Qué es lo que no queda claro?


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


I definitely felt the same temptation to make this mutable that you did. I think for a 1Kx1K matrix I'd probably still refuse it. If you wanted to take it, exploring the structural similarities between parts 1 and 2 would (I think) be a good way to introduce yourself to typeclasses.


Does this require an update to Regent?


why keep the setter here instead of allowing direct access to `currentBlock`?


Yeah, maybe. This will at least catch the bug we had.


색 추가가 아니라 에딧 텍스트;;


Se añadió el logo del parque y del ministerio como uno solo, ya no estan por separado


Thanks for the quick response. What did the issue entail?


Also, I wonder--do jshint/eslint use different exit codes when they fail in these ways vs failing the rules? That would be convenient.


fixed


Y si es null??


Nope, unfortunately not. (Always 1 for lint errors, including parse errors). Note that jshint support is removed in a subsequent commit.


:+1: I approve of this. If there is demand for this someday, we can consider bringing it back. But until then let's keep moving forward.


clear this in PacketContext.clear()


It's not about preferences, I just always forgot that the interpolation exists. Sorry about that. However, fixed.


TeamCity LabKey :: Sprint :: Premium :: Git postgres 9.4 [Build 11](https://teamcity.labkey.org/viewLog.html?buildId=291404&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitPostgres94) outcome was **FAILURE**
Summary: Tests failed: 26 (3 new), passed: 284 Build time: 01:55:41
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyBoxPlots: <no details avaliable>

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

org.labkey.test.Runner: org.labkey.test.tests.JUnitTest$RemoteTest.org.labkey.core.admin.test.SchemaXMLTestCase: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyColorStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyStudyAndTreatmentVars: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnStudyNavigator: <no details avaliable>

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


Did you mean a different @jetpack? :)


TeamCity TripleA :: UnitTest [Build 481](http://home.k-moeller.dk/viewLog.html?buildId=1150&buildTypeId=TripleA_UnitTest&guest=1) is now running


*laugh* this is a nice demonstration of StringContexts, but you might want to look into parser combinators for something of this complexity.


No es correcto!


haha. github is lame sometimes. carry on


Is this `TODO` still valid ?


TeamCity TripleA :: UnitTest [Build 481](http://home.k-moeller.dk/viewLog.html?buildId=1150&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:40


this is wrong!


done


@nbougalis So when this is called `config.RUN_STANDALONE` will always be false (to match the old code?). Looking....


Darn, now the pull request shows up in my inbox... Again, so sorry!


Este metodo no va aqui!!


Will the test stop working after :santa: :christmas_tree: ? :stuck_out_tongue:


[nit] `Map.empty[String,String]` means you don't make N copies of the empty map.


Este método no esta bien!


Modified this only because *test* release was having problems adding junit to jar.


Really great organization and commenting throughout. It made it very easy to follow through your program


Wouldn't this be relying on the fact that the `fpgas` collection doesn't change? I guess this is guaranteed by `toSet` returning an immutable set?


This took me a minute to understand - had no idea you could do this.  Do we gain something for this bit of cleverness?


Esto no está correcto!


mishatronic.ucoz.ua
olimpvin.at.ua
olimpinvin.esy.es


So remove this, and we'll keep it in KitchenSink. (You'll want to re-add to the interface in KitchenSink)


I don't remember adding this line; must've come through in the merge


Looks great!  Only issue:  rename "testUri" -> "actual", "goal" -> "expected".  Those are the standard names used testing equality in unit tests.


I *think* this can be removed once it's in KitchenSink.  (verify @ojkwon? TypeScript type things get my head in a bind at times)


Can you add a comment why this is necessary, or what is happening here, this is good to do in general with the UI stuff since it can be hard to figure out weeks later :)


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `/tmp/tmps_kd_x93/test.py`, line 2.*

The issue can be fixed by applying the following patch:

```diff
--- a/tmp/tmps_kd_x93/test.py
+++ b/tmp/tmps_kd_x93/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `python`.* *Related to file `/tmp/tmps_kd_x93/test.py`, line 4.*

The issue can be fixed by applying the following patch:

```diff
--- a/tmp/tmps_kd_x93/test.py
+++ b/tmp/tmps_kd_x93/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


Thanks for the feedback!


This header is imported both by C++ code and C code.  IIUC, removing this will break C99 packages.  I think what we want instead to to only set C99_INLINE_SEMANTICS if we're compiling C code and not C++.


[this invariant violation](https://github.com/facebook/flux/blob/master/src/Dispatcher.js#L162-L163)


Nice and clean. I like it.


TeamCity pull requests :: netty [Build 863](https://garage.netty.io/teamcity/viewLog.html?buildId=4072&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6228, ignored: 21 Build time: 01:24:15


Damn it, I forgot about that trait.  I knew about it, but it's been a while since I've build standalone apps like these.  :-)


Thank you. I know we need to get this version discrepancy fixed, but I think the utility of getting everything passing under continuous-integration outweighs having the test suite fail constantly as a reminder.


Y si es null??


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


This feels like the wrong fix to me.  Surely pcre must build on osx.  Either we need a pcre update or something's misconfigured.


Atributo en mayusculas!


Don't forget that a `Map[String, Int]` is a function `String => Int`! With an `eval` that doesn't memoize:

```scala
Map.empty[String, Int].withDefault{(s: String) =>
  val rv = eval(s)
  this += s -> rv
  rv
}
```


I'm not sure it's because of this commit but I got a random one letter first name, "I", which was kinda weird.


Hay que inicializar a null!!


Thank you for the compliment.  :-)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.939](http://teamcity.monogame.net/viewLog.html?buildId=12221&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 553, ignored: 6 Build time: 00:24:57


command #5


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


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


The mobile application targets end users like a DevOps engineer. The user who is monitoring services on cloud can login to the mobile app and can do a health check of the projects. The health check here refers to know the status of HTTP server, database server and some other metrics related to the project. Detailed graphs are provided for the analysis.


ConvertFromRoot changes x and y directly, first it converts it for captured_widget local coordinates, then to get down_widget we need to have the old x and y coordinates(global coordinates)


@madeleinefletcher @AlexBulankou in https://github.com/Microsoft/ApplicationInsights-JS/commit/e0ea11d7d9286dfefee278e87de1fd4bff597ea4#diff-a15f3fef07295091d48670d0feda1e4eR204 - do we not want to send client perf in case custom duration is given? If this is intentional then it's not obvious enough for users


Very nice work!
Have you looked into the Windows piping (if it exists) by chance?


I think this should work:
```bash
HOMEBREW_PREFIX=$(type brew >/dev/null 2>&1 && brew --prefix 2>/dev/null)
```
`type` is a shell builtin for bash, so it's guaranteed to be available, while which is an external program.


MCrafters


@maaarghk, it doesn't seem to - if I run steamiebot twice, it's still picking up the day's songs. See the test posts I made in /r/steamiebottest today.


this method does not do input validation. For example, "FOO" is not going to trigger an error.


Good point, I haven't used scanLeft much in the past, but I've seen other implementations use it for some solutions.  I'll keep it in mind for future solutions.


so awesome


No pide descargar uno nuevo!!


please rename genInfo -> _moveGenInfo, keep it as member of MoveGenerator class, and work with this pointer later instead of coping and keeping other members (_availableMoves, _currentMovePos, etc ...)
also put assertion here for _moveGenInfo to be valid pointer


@kasparsd Could you change to LF lineends?


I learned about the stride length from another puzzle I worked on a while back, it's definitely handy.


No es correcto!


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


looks like you did a solid job of dealing with slim, ruby, and coffee-script all at the same time.  very easy to understand!  :+1:


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


Thank you!


you might make this not captial letters for csv.  you may just make the route: '/csv' or something. but up to you!


D'oh!  Good point.


I've never actually seen this line in any of our Gemfiles fwiw. Is it actually needed?


Could it be unused?


Why do you inherit the Bear class ?

If I'm not mistaken, the Lint class can just have the `check_prerequisites` function - and when the `Lint` and `LocalBear` are inherited by `XYZLinterBear`the function will be available - making it use the `check_prerequisite`. So, I think the only issue that comes up is which `check_prerequisite` is given higher priority in `XYZLinterBear`


Y si es null?


Fair enough.  I fell back on the regexes because I'm a bit more comfortable for them with certain things like these.  Crafting Nginx rules tends to do that to you...


Wow.. I guess there's no going back to /structures/stool/ now :D


Next fixing the other APIs.


has_many references always in plural
```ruby
has_many :carts
```


El correo anterior me reemplazaba todo menos las mayúsculas, ya está
solucionado, y este lo modifiqué esta mañana en clase, si quieres vuelvo a
hacer un commit y lo subo a github

2015-12-14 19:04 GMT+01:00 Miguel Bayón <notifications@github.com>:

> Hay que comprobar que no sea null!!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jesus1310/mi-mail-system/commit/4fe0e17c58d645db32992ffda6add3ad8d176f55#commitcomment-14965865>
> .
>


Faltan cosas!


This is the trickiest puzzle we've seen to date, I think, but this is clear and well-structured. One thing I'd suggest is to be a bit tighter with the return types - there are a lot of Option[T]s that could be Ts by construction (maybe all of them? I'm not sure).


I see, thanks @dtenenba .  I figured this made the code more clear & maintainable.


Please also add TODO here to assign from, to, ... directly to _availableMoves[_availableMovesSize++] object, in lieu of creating tmp obj and assinging it
(that is the very intention of using pre-allocated mem pool)


Falta incluir su mensaje!


Seeing them side-by-side I tend to agree!  Not sure what made me want to go with the more archaic syntax.  Personally I hate the symbol operators in Scala since I tend to feel that they are overused.  Oh well, must've been a late-night hack where I wasn't thinking of things long-term.


Casing matters here? Very strange if that's the case


Could it be that this belongs to `admin_init()`?
A plugin should include admin code on the front-end.


This is definitely somewhere I'd turn to parser combinators for. :-D


Esto no va aquí!!


Never commit database changes. Actually it's my fault, I'll put it in the .gitignore. Nothing to do here.


Y si es null?


I didn't see at first that this commit is in a pull request and has the buildtools update (with validation disabled). I'll queue up a pull request that I can submit once this goes through.


Thanks for the tip!  I didn't know about that.  I'll definitely be filing that away...


Este método no es correcto!


wrong name. quantiy == 'quantity'


Fair point.  I think I found this one in a SO answer when I was looking for Regex's in case statements for matching...  ;-)


If the keys are a Set rather than a tuple, you get the swapping for free.


f


Are you aware that there is a threading issue if you don't use the `onComplete()` callback?  As the implementation stands now, `consume()` is synchronous.  Before we go to `M1` however, that won't be true.


I would suggest changing to isset($event['event']) as it's a little more clear. Would you mind making this agains develop branch? I will cut a release with update today.


wrong indent: should be 2 spaces.


No, good point.  This could come and bite you later if this was done in production code that is called n^m times....


nevěděl jsem, že je loadsh zakázanej, pamatuju si, že ho tam byla spousta
:-) fixnu

2015-12-11 18:14 GMT+01:00 Martin Halamíček <notifications@github.com>:

> fuj co ten lodash tady? koukam za ho zaroven ani nemame v zavislostech
> takze tam musi byt jenom diky nejake knihovne ktera ho pouziva. Stejne se
> snazim zbavit underscore.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/keboola/kbc-ui/commit/bf7615bcbc7c47715fc2aae40e1d73c59d6024ba#commitcomment-14930485>
> .
>

--
Ondrej Hlavacek
Developer, Keboola
+1 (778) 551 2270
@keboola


First, thank you for fixing this so soon after I reported it! For those still running into errors after this fix, try removing the one "&" before "||" on line 81.


Seems weird to me to that the spec compatibility would be dependent on the jvm version. May be split the spec from the tooling?


Remove tests files. For now, we don't have tests


Thanks.  I was planning on coming back to this one actually but I wanted to get some distance from it since it caused me no end of frustration in the past.  :-)  Nonetheless, I appreciate the feedback.


No. Pybind11 is pretty recent. I know the pypy crowd has been actively pushing CFFI in every way possible, including huge performance improvements. I'm still keeping an eye out for their response to Pybind11.


There is also grpperm.gi:1592. (This particular code probably should be looked over a bit more carefully, because it uses the Size command already, so maybe an IsFinite check should be done for this?


git add <directory>


On 11/12/15 02:48, Asim RP wrote:
> Could we also wrap all the alter and drop DDLs into one big transaction to reduce 2PC overhead?

Maybe. I wasn't 100% sure if it makes a difference that the tests happen
in separate transactions, though. Also, if one of the statements fail,
it will cause all the subsequent statements to fail too. That doesn't
matter much for statements that do preliminary setup before the actual
statements we're trying to test, but it can be annoying when debugging
the test statements themselves.

The speedup from using a single transaction isn't actually that big
anyway. I'm not sure this patch was really worth the effort, from a
performance point of view, but it was nice from a readability point of
view too. And you never know whether a performance improvement will pan
out before trying it out, and after you've written the patch, you might
as well commit it if there's any benefit at all.

- Heikki


Well, I did a VS2015 fix for the piping in 6f4ba08d4adca28390218a9557c793f666a7c48d, though the commit message for that is partly wrong, it does seems to work in debug mode. Or is there a similar redirection thread deadlock issue on Windows?


@szepeviktor That is a very good point. I switched to Atom for this project and haven't yet configured all the details.


This is beautiful.


Interesting!  It's been a while since I've dealt with for comprehensions, but I appreciate the examples!


This is the icon of the menu, 'fa-product' doesn't exist, put 'shopping-cart' back


tab


Oh, currying...  :-)


I logged this as issue #47010 but perhaps it's better to carry on the conversation here.  What is the purpose for adding a dependency on boost?


Actually, it requires the most recent snapshot of Bishop that uses 'mapper'
as a starting keyword.

Wonchan Lee

On Mon, Dec 14, 2015 at 10:08 AM, Elliott Slaughter <
notifications@github.com> wrote:

> Does this require an update to Regent?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StanfordLegion/bootcamp-2015/commit/cecd7db2c84f64d8ff5ae4c6b563adde7053ef65#commitcomment-14965944>
> .
>


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


Yes. Upstream decided to use Boost [much more heavily](https://dev.mysql.com/doc/relnotes/mysql/5.7/en/news-5-7-5.html) in 5.7.x and compile seemingly falls apart if you don't have boost available.

> The Boost.Geometry library now is required to build MySQL. Two new CMake options enable control over the library source location, and whether to download it automatically:


If I had modified the parameter I would have to change every single call of this function. Additionally, calculating derivatives is a computational overhead if you do not need them. This way they are optional.

Am 14.12.2015 17:56 schrieb Sanmeet Chahal <notifications@github.com>:

How come you decided to add a new dEidxj[3][3] array rather than extend the Ei[3] array to be a Ei[4][4] like the B field?

—
Reply to this email directly or view it on GitHub<https://github.com/wschreyer/PENTrack/commit/af5f0acc1facd71008e4c6fe42a96ec8c37b9b99#commitcomment-14964283>.


Also can't call `getClass()` from a static :(


Looks good! :bear:


Good point.

I've just changed this to 'along with native Android and iOS sender apps that use the native SDKs'.

Is that OK?


Cool, I'll take a look at that.


BRUTE FORCE FTW!!!!111!!!1!!!  ;-)


"**Meme**bers". (fixed in 53fcc92fd1a0453d4f3b52efc4965b1d12083849)


TeamCity TripleA :: UnitTest [Build 482](http://home.k-moeller.dk/viewLog.html?buildId=1154&buildTypeId=TripleA_UnitTest&guest=1) is now running


Ah - that makes sense...but in that case, shouldn't it be a build time dependency rather than a runtime dependency?


@elliotmoore Comment


What kind of stuff?


Dead code...


Maybe add plot.getOwner() != null back in


This'll work, just one minor knit I'm happy to fix in the merge.  This should be.

```
       AS_IF([test "x$cross_compiling" != xyes], [ ... ])
```


TeamCity TripleA :: UnitTest [Build 482](http://home.k-moeller.dk/viewLog.html?buildId=1154&buildTypeId=TripleA_UnitTest&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 205 Build time: 00:00:42


As a matter of fact, opencaching.jp existed for only 1 year, 2010-2011.


No es correcto!


I just picked Django < 1.9. I'm not wildly excited about adding yet another place to update versions.

Can we get away with just doing "Django"?


Can you be more specific? Swallowing any bugs here too isn't great.


Nice mate!


I like this one because of its simplicity.


Yes, it was meant to be used for cache busting during plugin updates but I haven't implemented it yet. I'll add that to the to-do of the pull request.


who's first?


I should have made the commit title better!


@swiftix I think this change deserves a new test that doesn't involve super_method, since it is not limited to devirtualizing super_method.


Good call — done.


That's files created by emacs :p Every time you modify a file in emacs, it creates the same filename with ~ at the end which contains the original file before changes, so if you ever need to revert all the changes you did, you still have a backup. It's stupid to be honest and I've never found a use for it, but it's what emacs does.


Possibly. The documentation wasn't hugely clear on whether or not if you decided to use a local already-installed Boost rather than a downloaded-by-cmake version there was an expectation it would remain available.

If you wanted to check that out and file a PR to change the dependency if it proves to not be runtime that'd be welcome.