There are a few things you'll want to be careful of here:

1.  Don't use String(positions) to convert the object to a String.  You're just getting [Object object], which is not what you want.
2.  If you're going to return an object referred to in the memo table (and not a copy of it), you should be careful not to change it.
3.  For clarity (and easier parallelization), don't store output in the memo table until you've computed it.  It shouldn't matter for this implementation because none of the subproblems involve the original inputs.  However, imagine a different implementation in which problems are solved in parallel:

Suppose for example that you have two first basemen John Doe and Foo Bar, both making 1 million dollars and have a budget of 10 million.  So first you start running the recursive procedure for the team in which you chose John Doe so you're selecting the rest of the positions with 9 million left.  You make the recursive call and then immediately store the output:
{
          total_vorp: 0,
          money_spent: 0,
          players: {}
};

in the memo table with the pair of the remaining positions and 9 million dollars.  But before you actually change this to be the correct result, you also start computing the maximum VORP if you select Foo Bar, which also leaves you with 9 million left and the same positions.  If it looks in the memo table, it finds this key with the initial object having a total_vorp of 0 and uses it.  In the case that Foo Bar has a higher VORP than John Doe but not higher than John Doe + the VORPs of the remaining players that can be selected, the team with Foo Bar will be erroneously discounted.


this line will fail on hero killed, needs a valid check


Is this failing with older versions of debbuild? Is this breaking builds of old deb distros? Maybe /usr/src/debian should be set as fallback?


J'dois y réfléchir. Avec un CI comme Travis, je ne suis pas sûr qu'on s'expose à beaucoup de souci niveau dépendances (enfin, on s'expose autant que sans, mais on le détectera potentiellement beaucoup plus vite, tout en minimisant les problèmes de co-installabilité).


:metal:


能技略解:
0. 拥有领域能力,可以理解并提出目标对象擅长领域的 "好问题"
1. 对目标对象有能力完成自主挖掘,获得合理问题的释放渠道
2. 能技持续释放能力,能跟得上目标对象的反馈,形成高效率沟通,而不是简单的单轮灌输
3. 参考: [珠的自白:30 求求你,让我教你 | GDG Livin ZhuHai Life;-)](http://blog.zhgdg.org/2014-06/dm30-i-beg-u/)
4. ...


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


What a nice commit message!


:eyeglasses: fixing


fix_cppcheck_issues (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2083 of 2083 tests passed, 0 test warnings)

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


@nwellnhof let me know if you think this is a bad idea.

In using the API I found myself often repeating this pattern.


Agree. Python has many powerful features that help improve productivity and make code clean and elegant and **PYTHONIC**.


corebluetooth


Le reste défini en local


wiggidy


Index резолвер симпотишный


Did you hack my github son??? haha

I just signed up for codeschool.com and took the git course.  codeschool is
the sheeeit

On Sun, Jan 10, 2016 at 11:43 PM, Adrien Atallah <notifications@github.com>
wrote:

> wiggidy
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericatallah/practice/commit/752a986ea0be720a86124fe1e3041a5c4b800dad#commitcomment-15364984>
> .
>

--
Eric Atallah
Web Developer

Cell: (714)595-5111


fix_cppcheck_issues (mbadams5) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


The point for this was to exclude doxymentation in cpp files from the public API (since they are not public API) and from the documentation coverage threshold.


(minor) lets rename E and T to T1, T2 - sounds more general


Это да, удивился остутствию альтернативы url_for() (http://flask.pocoo.org/docs/0.10/quickstart/#url-building) из коробки


Maybe you can write up something about what makes it enjoyable and fulfilling for you, and if others can do anything to help. :-)


@m-kuhn yes, but isn't the public doxygen docs generated from the nightly builds? So they would still including any classes defined in a .cpp file. I'd prefer to keep checking them in Travis, so we know that they need to be @cond'ed out to avoid them appearing in any public docs...


Didn't realize there was a getUserMedia library already. sorry :)


`Fréquence de rafraîchissement des marqueurs ?`
serait plus clair.


Doh, it appears that the change has broken the build :( Informed dev@spark.

```
➜  spark git:(master) ✗ ./build/mvn -Pyarn -Phadoop-2.6
-Dhadoop.version=2.7.1 -Dscala-2.11 -Phive -Phive-thriftserver
-DskipTests clean install
...
[info] Compiling 8 Scala sources and 1 Java source to
/Users/jacek/dev/oss/spark/external/flume/target/scala-2.11/classes...
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:33:
object jboss is not a member of package org
[error] import org.jboss.netty.handler.codec.compression._
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:31:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.{ChannelPipeline,
ChannelPipelineFactory, Channels}
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:32:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.socket.nio.NioServerSocketChannelFactory
[error]            ^
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelPipelineFactory not found
- continuing with a stub.
[warn] Class org.jboss.netty.handler.execution.ExecutionHandler not
found - continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.handler.execution.ExecutionHandler not
found - continuing with a stub.
[warn] Class org.jboss.netty.channel.group.ChannelGroup not found -
continuing with a stub.
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:149:
not found: type NioServerSocketChannelFactory
[error]       val channelFactory = new
NioServerSocketChannelFactory(Executors.newCachedThreadPool(),
[error]                                ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:196:
not found: type ChannelPipelineFactory
[error]   class CompressionChannelPipelineFactory extends
ChannelPipelineFactory {
[error]                                                   ^
[error] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[error] Class org.jboss.netty.channel.ChannelPipelineFactory not found
- continuing with a stub.
[error] Class org.jboss.netty.handler.execution.ExecutionHandler not
found - continuing with a stub.
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:197:
not found: type ChannelPipeline
[error]     def getPipeline(): ChannelPipeline = {
[error]                        ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:198:
not found: value Channels
[error]       val pipeline = Channels.pipeline()
[error]                      ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeInputDStream.scala:199:
not found: type ZlibEncoder
[error]       val encoder = new ZlibEncoder(6)
[error]                         ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumePollingInputDStream.scala:29:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.socket.nio.NioClientSocketChannelFactory
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumePollingInputDStream.scala:73:
not found: type NioClientSocketChannelFactory
[error]     new NioClientSocketChannelFactory(channelFactoryExecutor,
channelFactoryExecutor)
[error]         ^
[warn] Class org.jboss.netty.channel.ChannelFuture not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[warn] Class org.jboss.netty.channel.ChannelUpstreamHandler not found
- continuing with a stub.
[error] Class org.jboss.netty.channel.ChannelFactory not found -
continuing with a stub.
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:33:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.ChannelPipeline
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:34:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.socket.SocketChannel
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:35:
object jboss is not a member of package org
[error] import org.jboss.netty.channel.socket.nio.NioClientSocketChannelFactory
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:36:
object jboss is not a member of package org
[error] import org.jboss.netty.handler.codec.compression.{ZlibDecoder,
ZlibEncoder}
[error]            ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:107:
not found: type NioClientSocketChannelFactory
[error]     extends NioClientSocketChannelFactory {
[error]             ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:79:
overloaded method constructor NettyTransceiver with alternatives:
[error]   (x$1: java.net.InetSocketAddress,x$2:
org.jboss.netty.channel.ChannelFactory)org.apache.avro.ipc.NettyTransceiver
<and>
[error]   (x$1: java.net.InetSocketAddress,x$2:
Long)org.apache.avro.ipc.NettyTransceiver
[error]  cannot be applied to (java.net.InetSocketAddress,
FlumeTestUtils.this.CompressionChannelFactory)
[error]         new NettyTransceiver(testAddress, new
CompressionChannelFactory(6))
[error]         ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:109:
not found: type SocketChannel
[error]     override def newChannel(pipeline: ChannelPipeline):
SocketChannel = {
[error]                                                         ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:109:
not found: type ChannelPipeline
[error]     override def newChannel(pipeline: ChannelPipeline):
SocketChannel = {
[error]                                       ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:110:
not found: type ZlibEncoder
[error]       val encoder = new ZlibEncoder(compressionLevel)
[error]                         ^
[error] /Users/jacek/dev/oss/spark/external/flume/src/main/scala/org/apache/spark/streaming/flume/FlumeTestUtils.scala:113:
value newChannel is not a member of AnyRef
[error]       super.newChannel(pipeline)
[error]             ^
[warn] 13 warnings found
[error] 24 errors found
[error] Compile failed at Jan 11, 2016 8:43:02 AM [0.602s]
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Spark Project Parent POM ........................... SUCCESS [  5.427 s]
[INFO] Spark Project Test Tags ............................ SUCCESS [  4.430 s]
[INFO] Spark Project Launcher ............................. SUCCESS [  8.698 s]
[INFO] Spark Project Networking ........................... SUCCESS [ 16.379 s]
[INFO] Spark Project Shuffle Streaming Service ............ SUCCESS [  8.800 s]
[INFO] Spark Project Unsafe ............................... SUCCESS [ 11.254 s]
[INFO] Spark Project Core ................................. SUCCESS [02:04 min]
[INFO] Spark Project GraphX ............................... SUCCESS [ 16.071 s]
[INFO] Spark Project Streaming ............................ SUCCESS [ 35.991 s]
[INFO] Spark Project Catalyst ............................. SUCCESS [01:34 min]
[INFO] Spark Project SQL .................................. SUCCESS [01:08 min]
[INFO] Spark Project ML Library ........................... SUCCESS [01:18 min]
[INFO] Spark Project Tools ................................ SUCCESS [  5.614 s]
[INFO] Spark Project Hive ................................. SUCCESS [ 40.699 s]
[INFO] Spark Project Docker Integration Tests ............. SUCCESS [  2.102 s]
[INFO] Spark Project REPL ................................. SUCCESS [  6.258 s]
[INFO] Spark Project YARN Shuffle Service ................. SUCCESS [  6.547 s]
[INFO] Spark Project YARN ................................. SUCCESS [ 12.898 s]
[INFO] Spark Project Hive Thrift Server ................... SUCCESS [  9.361 s]
[INFO] Spark Project Assembly ............................. SUCCESS [ 40.149 s]
[INFO] Spark Project External Twitter ..................... SUCCESS [  7.137 s]
[INFO] Spark Project External Flume Sink .................. SUCCESS [  6.203 s]
[INFO] Spark Project External Flume ....................... FAILURE [  1.010 s]
[INFO] Spark Project External Flume Assembly .............. SKIPPED
```


How about use pip to install such dependencies?


noice. yep


It is all fine - you moved the Builders already into Validation.
Renaming Validation.ValidationBuilder* to Validation.Builder would simplify the names.


No need for the `std::to_string` here. `cerr` (and `cout`) can happily print an integer with the left-shift operator.


If it's not needed it should be non-copyable, else everything sould be copied imo


fix_cppcheck_issues (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2051 of 2051 tests passed, 0 test warnings)

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


No need to bind the socket. The kernel will automatically assign a local IP and port number when you `connect` the socket to a peer address, or when you send a datagram with it.


In general, you probably don't want the `atoi` or `stoi` around port -- we generally want the user to be able to use "named" services (like "http") in addition to numerical ones (like "80"). And we probably want the user to be able to use named hosts (like "www.stanford.edu") instead of just numerical IP addresses. The Address constructor that resolves both is the one that takes two strings: https://github.com/cs344g/ps1/blob/master/src/address.hh#L33-L34

Implementation here: https://github.com/cs344g/ps1/blob/master/src/address.cc#L46-L83


Please, add documentation about additional parameter to the UltiSnips.txt.


I guess better name is `current_ulti_dict_info`.


git is the shit.  You're doing it all in the terminal right?


он на другой странице теперь


Maybe we should have a general exclude vector?


I simply like to write quality code. I do this for a living too (which is valued often more, both in terms of happy people and certainly financially).

You are an exception, but other people hardly help. It is consumerism all over the place. Look at XPrivacy, it was said that I dominated the development (not that exact wording), but leaving alone the XPrivacy source code for more than half a year resulted in exactly zero contributions from others.


Whoops I misread Imposter as Illusion, so there's no issue here


@jacobsa If there was: we'd post it. All the code is open-source and public so your help debugging would be welcome.


fix_cppcheck_issues (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1502 of 1502 tests passed, 0 test warnings)

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


I think we have to put a fixed jquery version in composer.json, to avoid changes in their repository.


この辺りの、必須項目の判定に関して。

フォームで必要な情報が、正しい形式で入力されているかチェックすることを、
「バリデーション」といいますが、Larvelにもその機能が備わっています。
http://readouble.com/laravel/5/0/0/ja/validation.html

読んでもらうとわかるかと思いますが、
フォームの項目ごとの条件（必須、情報の型など）を配列に設定しておき、
Validatorを呼んであげるだけで、情報のチェックをすることができます。

このメソッドに、Validationを実装してみてください。
わからなかったら聞いてください＾＾


The entire MELPA build usually takes 4 hours. And the next one is gonna start in 2 minutes: http://melpa.org/

In 4 hours we will see the result.


I'm really not sure on the root cause here, and being completely unable to reproduce locally isn't proving terribly useful. I'll try pushing a few test commits later, but can't promise one will magically work sadly.


Should fix #238


i think it's just endlines


if you use derive4J then the Either2 abstract class should have no fields at all.


@trueadm

I ran some mem benchmarks as you suggested. Here's what I found:

Inferno vs domvm
dbmon @ 100% mutation rate

with table display: none
120/s vs 60/s

while heap profiling
60/s vs 17/s

heap allocations:
100k vs 500k

with display: table
24/s vs 21/s

mem use (task manager)
~83MB for both libs with table both shown and hidden

From this, some things are clear:

- In synthetic tests (with the table hidden), Inferno kicks ass. Heap allocations are small as you mentioned probably from using objects rather than arrays, and the speed is fast since you just diff the values passed to the templates.
- Once the table is visible, many of the differences are dwarfed by DOM repaint/reflow and the libraries are very close. The 400k difference in heap allocations is also dwarfed with both apps flat-lining around 83MB mem usage (via Chrome's task manager).

Since they are in fact both view libs that render the DOM, the synthetic tests are not very meaningful in the bigger picture, IMO.

However, for 1% mutation rate and a visible table (which is by far more realistic than 100% mutation) the perf story is almost exactly aligned with the synthetic results above: 125/s vs 65/s. So the most truthful statement that can be said is:

> When mutation is low, Inferno offers 2x better performance than domvm and slightly better performance when mutation is high. While Inferno offers 5x better memory usage, this metric is practically insignificant relative to the task both libs perform.

We need to keep in mind, though, that your dbmon example uses the compiled syntax [1] (objects). While this is representative of expected use, it assumes a prior compilation step from either JSX or another more friendly template language. domvm offers a practical syntax (via arrays) but incurs the overhead of compiling it at runtime (to objects internally).

I think I better see the advantage of using functions to create the nodes to avoid intermediate array allocation and create the vnodes without an intermediate representation. But, my interest in doing this would not be primarily to reduce mem usage (since as seen above, it's not statistically significant) but to improve the vdom creation speed. I'm skeptical how much perf boost this will provide, but it can easily be offered as an alternative alongside the array syntax if it's anything to write home about. As far as diffing goes, domvm is a standard vdom lib, so it's stuck diffing everything from the `redraw()` root...but it ain't too shabby at it :)

BTW, the bug you found is the result of domvm expecting 2 text nodes as in the template, but since they're adjacent the setting of innerHTML merges them together. I will update domvm to merge them internally as well which should fix the issue. Good catch!

[1] https://raw.githubusercontent.com/trueadm/inferno/dev/examples/dbmonster/app.js


I actually had trouble migrating my own installations when I had two migrations, that's why I simply manually added the migration in my own apps and cleaned up the ones in the gem. If you want to upgrade, simply create a table SpinaRewriteRules with two string columns: old_path and new_path.


comment


An excellent suggestion, the build works just fine now.


Not working


Well done!


This is not working .


*Fix crash when no chat found


there is an R class on codeschool.com looks legit, but probably short


*NOT very familiar


(#1)

This line is causing your issue. You overwrite your `names` variable with the result of stringifying `names`. This changes `names` data-type from an `Array` to a `String`.

Jump to my other comment (#2).


It doesnt exist anymore in this version.
dterm_cut_hz has now substitute, which is hardcoded. However this all is
not "officially" released. It is still in testing stage
Op 11 jan. 2016 07:02 schreef "Larry" <notifications@github.com>:

> may i ask you why you delete this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/borisbstyle/betaflight/commit/7ef593c54f879cc68843d556e14cbddabc7ebede#commitcomment-15363792>
> .
>


(#2)
So now, the **next** time someone connects and registers, this line of code tries to invoke the `push` method stored on `names` but this causes an error because `names` is now of type `String` and **does not** have the method `push` available to it.

Go back to (#1 and continue).


@quicquid Any ideas? Or is this actually correct?


@s7design: There was a typo here causing failure on postgresql. Please watch out for this when updating more tests.


Please note: Some of the things you changed are not actually mistakes. For example, you changed `analyse` to `analyze` even though `analyse` is the correct spelling in British English, and you changed `"Creating a Backup,"` to `"Creating a Backup",` even though commas are supposed to go inside closing parentheses in American English. Nonetheless, the *majority* of the changes in this commit are fixes of actual mistakes, so I'm accepting this commit as-is. Thank you for doing this!


brandon.  The site looks good. gliven


The first version of my fix was to make it non-copyable, but that conflicts with the interface requiring a Clone() implementation, which is never used at runtime. I don't like this interface and the concept of a Clone() method, and probably you aren't aware of how many implicit temporary copies of UI objects there are, but that's how it is.

What I won't do is change this commit to copy everything. I consider that a waste of my time. If this commit can't be merged because it's considered incomplete, I'll drop it and let somebody else care for this. I don't care.


lol Maribelle, dafuq?


Um. https://github.com/PocketServer/PocketServer/commits/chunk-data-saving


What we usually do is add a link to the upstream bug with the word hack like this:

```
HACK: http://...
```


can you replace it with bstWrite8(0) and bstRead8(), because the other side need to support all version.


ok thanks


업데이트 릴리즈는 언제쯤!


May be you need this
```PHP
$mac = $this->format_mac($mac, $device_vendor);
```


this will not work :(


In this case I did just that, but for something more complicated or with many future migrations it could become more difficult.

From what I've noticed from other rails engines, migrations are added to the engine and then rails apps can run `rake engine:install:migrations` to copy any new migrations which you can then run as normal

I think the engine migrations are prefixed with a order number rather than the date, which when copied is given a datestamp


아 맞다


ところで、ここなのですが、文字化けしていませんか？


the reason we can't have nice things


No use


Ya, all on terminal.  Are you studying R?  What did you use the raspberry
pi for?

On Mon, Jan 11, 2016 at 12:05 AM, Adrien Atallah <notifications@github.com>
wrote:

> there is an R class on codeschool.com looks legit, but probably short
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ericatallah/practice/commit/752a986ea0be720a86124fe1e3041a5c4b800dad#commitcomment-15365178>
> .
>

--
Eric Atallah
Web Developer

Cell: (714)595-5111


細かいようですが、後ろの括弧「{」は改行です。


> On 10 Jan 2016, at 23:24, Sebastian <notifications@github.com> wrote:
>
> Why the additional variable a?

feel free to change - as written in a private mail - this wins no prize in code quality. I see it only as a proposal.
However, my question is: is the reasoning OK in your opinion?

a.


Maybe one can add an update button. This loop may not be good in some cases.
The message should be "Could not update the version of the game. Maybe you have no internet connection. We will use the older version."


Just better error handling. When running the cmake without any options, the script didn't work (which forced me to dive into the cmake to see why). This check simply prints out an error message which should be enough info to resolve the issue.


This line is causing #13763


Purpose? Why not use t() directly, without format_string() call above.


改行して、

```php
class Tavern extends Model
{
   private ...
}
```

です。クラスとメソッドを定義するときは注意しましょう。


MyPlot1
plot1 <- function() {
        hist(df$Global_active_power, main = paste("Global Active Power"), col="red", xlab="Global Active Power (kilowatts)")
        dev.copy(png, file="plot1.png", width=480, height=480)
        dev.off()
        cat("Plot1.png has been saved in", getwd())
}
plot1()


Do not manipulate dom in JS unless it's impossible to achieve same effect in module (PHP) code.


I will do that. Thanks.


Wow, your code is awesome!  I want to be like you when I grow up.


This will be used for issues #48 (Creating a report) and #46 (The accuracy of historical address data)


Nice, thanks. ;)


fix_cppcheck_issues (mbadams5) - Win64-Windows-7-VisualStudio-12: OK (2051 of 2051 tests passed, 0 test warnings)

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


"In order to update the game, be connected to the internet when the game starts. You can find the update button in the menu." - Maybe these long messages are ok. I do not know.


Nice, thanks! ;)


Thanks Martin. Yes, I see now, it is our mistake. We will take care about that in further update tests.
Regards
Zoran


@genesixx build completed successfully, now it works with BlissPop!
my local_manifests is here https://gist.github.com/lindwurm/4c86bb2a5387d0a41d10
![screenshot](https://pbs.twimg.com/media/CYbCjfeVAAA7toO.png)


good idea!


This is a comment.


It looks good


I am curious to know if this is the correct way to do it.
We add a similar macro for testing assert//abort and we found out that if you assert from a noexcept user functions, remplacing the assert by a throw make the program std::terminate.

Do I understand correctly that by hooking abort and not assert itself, we're passed the noexcept wall of user functons ?


fix_cppcheck_issues (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (951 of 951 tests passed, 0 test warnings)

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


Care to elaborate? ;)


agree with what you say about Vim distributions,when I start to us vim,I use the default vimrc,then add something line by line


mark


@JustAkan arter97 님 레포지토있는걸 fork 했습니다.
https://github.com/jollaman999/aarch64-none-elf-5.3


能技略解:
0. 拥有领域能力,可以理解并提出目标对象擅长领域的 "好问题"
1. 对目标对象有能力完成自主挖掘,获得合理问题的释放渠道
2. 能技持续释放能力,能跟得上目标对象的反馈,形成高效率沟通,而不是简单的单轮灌输
3. 参考: 珠的自白:30 求求你,让我教你 | GDG Livin ZhuHai Life;-)
4. ...

@ZoomQuiet
看了之后, 又觉自己还是能技不足,,,对大妈的挖掘不够,,,,而且目前绝对是单轮灌输状态, 跟不上反馈.... :-(


I think this commit is obsoleted by 0baa109bd15f3b2fc7ca96b5e8e07e024c31c377


i don t think this compiles, at least on arduino 1.6.5
what arduino ide are you using?


validatiorですね！やってみます！


the second


I think you can just use "while True:" and the add break after the winning guess message prints


the


これは文字化けではなくて、漢字チェックの正規表現です。一から龠までのマルチバイトが漢字の範囲になるようです！


more nested model will require different properties e.g data object will require class name and instance name, so this approach with hardcoded  `{ instance: instance.name }` will not work for other models, second thing is that in different models those properties will have different names e.g: for instance u have `name` but in class model u will have `instance` or `instanceName`


Hi mohan, in here the backslashes are used to escape the double quotes within a double quote string


Imho his should have gone to a separate sub class as it changes behavior of the class quite a bit: 1) serialization works differently 2) you can't use late template initialization anymore


How to short map object


@trociny
Eeeghhh, stupid.misser... Yes you are right.

Sorry about the alarm


Done


Why not use "while True:" and add return True in the function?

Looks good


Yep
make it device specific


We could probably lose the "and t.humidity < 2.6".


@chrisze Sorry. I forgot what you said. This addresses the memory problem.


shhhh..... I messed up OK :(


Then --list=encodings should also list utf8 and use utf-8 as a fallback. Whatever you prefer.


Interestingly: https://github.com/Homebrew/homebrew/issues/47953


I just had the same issue - downloaded the latest master, had to manually patch with this to workaround it.


+    # Enable writing to led blink node from userspace
+    chown system system /sys/class/leds/led:rgb_red/blink
+    chown system system /sys/class/leds/led:rgb_red/duty_pcts
+    chown system system /sys/class/leds/led:rgb_red/ramp_step_ms
+    chown system system /sys/class/leds/led:rgb_green/blink
+    chown system system /sys/class/leds/led:rgb_green/duty_pcts
+    chown system system /sys/class/leds/led:rgb_green/ramp_step_ms
+    chown system system /sys/class/leds/led:rgb_blue/blink
+    chown system system /sys/class/leds/led:rgb_blue/duty_pcts
+    chown system system /sys/class/leds/led:rgb_blue/ramp_step_ms

please add them PROPERLY! to ueventd


The `http://doc.wakanda.org/home2.en.html/Wakanda/0.v10/Datastore-Model.200-1102147.en.html#/Wakanda/0.v10/Datastore-Model.200-1102147.en.html` link:

- points to a what look like to be a broken page.
- contains v10 in the url

shouldn't the url be `http://doc.wakanda.org/home2.en.html#/User-Guide/Datastore-Model.200-1102147.en.html` or equivalent?


Is creative.. Why the ?:
It returns true or false


+    # Enable writing to led blink node from userspace
+    chown system system /sys/class/leds/led:rgb_red/blink
+    chown system system /sys/class/leds/led:rgb_red/duty_pcts
+    chown system system /sys/class/leds/led:rgb_red/ramp_step_ms
+    chown system system /sys/class/leds/led:rgb_green/blink
+    chown system system /sys/class/leds/led:rgb_green/duty_pcts
+    chown system system /sys/class/leds/led:rgb_green/ramp_step_ms
+    chown system system /sys/class/leds/led:rgb_blue/blink
+    chown system system /sys/class/leds/led:rgb_blue/duty_pcts
+    chown system system /sys/class/leds/led:rgb_blue/ramp_step_ms

please move them to ueventd


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 929](https://teamcity.labkey.org/viewLog.html?buildId=296155&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 249 Build time: 00:38:28
### Failed tests
```
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>

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

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

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

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

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

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

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


The Permalinks are no longer working for some reason. That's fine. You can use the URL you posted. It's just better to go to the User Guide than the link you had before.


这是我的测试


I would like to keep the following to key points about the **prototyper** regarding the data model:

- **validate** the data model
- **add some data** in it


rajan


@varadharajan I test the code with some data, but find that after every iteration in pagerank, the ranks.count() become smaller, and at some point it stop decreasing.
Befor going to iteration in pagerank, ranks.count() is 60509, but after iteration , ranks.count become 906. ..... I don't what happens.


lalalaa


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


This branch Onderscheid-bet-nietbet-best was old and was already merged with design. So was not up to date


It would be nice if you didn't have to specify the formula for the logical axiom rule explicitly.


is it ok to not check `props.selectedValue` here? It seems component will not update if selected value changed.


@liyi193328 I'm not sure if this has anything to do with this commit. I think its better if you could send out a mail to Spark users mailing list (http://spark.apache.org/community.html) with details so that others can chip in with their thoughts as well.


I agree, that's how it should be.


There's already a `rename` function, which does precisely this:
```scala
rename(quantifiedVariable, freeVariables(subGoal.conclusion))
```


cc @AlbertoFuente


`Nitpick` `<hr>` tag should also be close like this `<hr/>`


Done. v1.0.3.

Regards,
Jadran

On Sun, Jan 10, 2016 at 10:18 AM Peter Dajka <notifications@github.com>
wrote:

> I think you can allow 0 too and set default to ~0.2 sec for better UX.
> You can check the VS Code solution (~0.2 sec) or Sublime Text (~0sec).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/yaddran/brackets-occurrences-marker/commit/9b6a38ba45b0ec8d7d4d7b55f894e4e920afefda#commitcomment-15359132>
> .
>


Actually it was an old request of mmoy http://piwigo.org/bugs/view.php?id=3116


なるほど！ありがとう！


Note that because of the `using namespace std` at the top of this file, you don't need the `std::` before `cerr`, `cout`, and `endl`.


Good idea, I added a log warning.


**Intel Core i3-2100** result is better than **Intel Core i7-3615QM** - **Intel Core i5-3210M** - **Intel Core i5-3427U**  :+1:  It's mean maybe cpu freq is main in this test. But what about **Intel Core i5-3427U (1.8GHz)** and **Intel Core i7-3615QM (2.3GHz)**


А как на неё попасть? На сайте ссылку не нашёл


initial


Should be levitate and feather fall, hover is the same as levitate :P


you should create a global gitignore to prevent this: https://help.github.com/articles/ignoring-files/#create-a-global-gitignore


IDK , I will check these later , It just take from ImagicalMine


There is an `orElse` method on `Option` as well:
```scala
t1(proofState) orElse t2(proofState)
```
This is why I suggested the name `orElse` in the first place. :smile:


Just to be sure, you checked that the parentheses in the string are processed properly? If not, I'd probably go with ```resources/bundle_deps_for_optical``` which IMO stills gives enough info and is shorter.


See sample code in the jQuery.ajax() docs: https://api.jquery.com/jquery.ajax/

This JavaScript snippet is intended to be inserted into customers web sites by the customers themselves. The snippet is currently relying on plain JavaScript. What problems might be caused by changing the snippet to rely on jQuery?


I'm not sure that you can see this comment in the generated scaladocs, as it is inside an anonymous class.  This comment should probably go to the `orElse` method instead.


This breaks the logic only to go to "now" (https://github.com/MaxKellermann/xbmc/commit/e6f2bfb447507559dad76dac80a67cf9f287d7ce#diff-d2e33589075d13d688b1a200a39e6652R1064) as fallback case. If prevTag == SelectedTag we must not do anything else than reinit m_item, especially not "go to now".


You must specify the years explicitly not as a range, hence: `2013, 2014, 2015, 2016`.


Done


You're welcome :+1:  :wink:


I suggest:

After designing your data model, you can prototype your app to make sure that the data model corresponds to your needs.


Same here.


`Nitpick` Remove space between `=` and `"project 1 image"` this should be `alt="project 1 image"`
not `alt= "project 1 image"`


What about:

> After designing your data model, you can prototype your app and _start adding data_ to make sure that the data model corresponds to your needs.


Is this MSMQ specific? If it isn't, then maybe the snippets could use the dummy `MyTransport` instead of `MsmqTransport` to indicate this is applicable across different transports. This would be consistent with transaction scope options snippets for v6.


Check if `@event` is being populated


i added it in testing because @TheRingMaster wanted to test, it's not merged into mm6.0 yet.
Keep calm


This would be a lot nicer if `LogicalAxiomTactic` didn't require you to specify the formula explicitly.


@ccx-testerbot runtests master


This breaks code logic. Please see my comments for the related master PR => https://github.com/MaxKellermann/xbmc/commit/e6f2bfb447507559dad76dac80a67cf9f287d7ce#commitcomment-15365621


OK (without italics)


Thank you for the request. It has been added to the queue (id: 143). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/143) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


@wjwithagen you can disable this by passing `--disable-gitversion` to configure. see `configure --help` for more info.


Conventional to use lowercase attribute names for acronyms `:URL` -> `:url`


Currently, Kodi crashes. Which is clearly worse than going to "now". What this commit does is have a non-optimal fallback instead of crashing. If you have a better fallback, submit another commit on top of mine, after mine has been merged.


Sorry, my NP++ does it automatically for some reason. How do I disable it?


What's more conventional ruby for looping through a collection?


Branch


> I'll work on tests more once we're sure of things design-wise

Probably. Sometimes I'm too "conservative" about tests. You're 100% right that the logic is currently quite straightforward and there is no need to unit test that.


@remicollet : :+1:


历史会记住它的。


I'd expect Strife to match Doom 2 1.666 with respect to OPL behavior, since it otherwise seems to be based on that version. Maybe the DMX version is different though?


:bow:


@wjwithagen imho, "updates to several files" does not help to the readers of your commit message. could you be more specific on why you removed  "set -xv"?


This is one of the cases where recursion is probably a lot nicer than iteration.  Using recursion, this whole function collapses to:
```scala
tact(proofState) match {
  case Some(newState) => apply(newState)
  case None => Some(proofState)
}
```

BTW, the scala compiler will produce almost exactly the same code for the recursive version (the recursive call occurs in "tail position", and this tail-recursion is automatically converted to a loop).


Not exactly correct. This breaks nothing that wasn't already broken before. The behavior you demand has not existed, because Kodi has crashed.


Reintroduce the `if` block wrapping these configs: `if Rails.env.development? || Rails.env.production?`

As this is no longer limited to the development and production environment, currently running the tests (which uses the test environment) will try to send real emails through SendGrid. This could cause various problems, including SendGrid flagging the account for review.


abc какое-то ужасно название


On a second thought I think I'd prefer a slightly less hard-coded solution here. How about just using a different select statement (old line 727) when the state is crowbar_upgrade (leaving out the ```v["states"].include?("all")```)?

It seems a bit cleaner than just hard coding a single role here. And that way it would be easily possible to add upgrade specific roles to other barclamps if we ever need that.


ところで、このgetCommentのメソッド、他で呼ばれていないような気がするのですが、どうでしょうか。
あと、先日お伝えした、ModelとRelationを実装すると、
このメソッドが不要になるので、コードがすっきりしますよ＾＾


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


Sure :)


This turned out really great, @litenjacob! Well done! :D


а зачем удалил?


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 62](https://teamcity.labkey.org/viewLog.html?buildId=296152&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 14, passed: 34 Build time: 01:06:32
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

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

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

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

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

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

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

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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


This introduces a BC break, current documentation has "collection_upload" as configuration parameter in example.

I do not see the reason to rename parameter, it could stay with same key in order to keep BC.

If you keep this renamed parameter, maybe you should correct the documentation as well?


So... good idea @moy !


Minor nit: Can you undo these unrelated changes? Looks good otherwise!


I meant on this documentation: https://github.com/symfony2admingenerator/FormExtensionsBundle/blob/master/Resources/doc/documentation.md


@andresgutierrez this fixes
* https://github.com/phalcon/cphalcon/issues/11026
* https://github.com/phalcon/cphalcon/issues/11131
* https://github.com/phalcon/cphalcon/issues/2399

issues?


How i do can u saject me


These files are used in Windows version only. Will it be better to move then to the corresponding list of platform-specific source files in `src/CMakeLists.txt`?


\o/


Yes, good idea.


Doesn't appear to actually redirect for me.

Adding 'permalink: /404.html' in the YAML fixes this (https://help.github.com/articles/custom-404-pages/)


Fixed


</configuration>


Plagiat !!!


da ce'am facut sefu


OK！ :+1:


Text could be centred on screen? Slightly left-aligned on my screen

<img width="1425" alt="screen shot 2016-01-11 at 09 12 40" src="https://cloud.githubusercontent.com/assets/1264223/12229875/9056f662-b843-11e5-9298-7826a119115d.png">


On the long polling chat app, people (and me) are experiencing random loss of connectivity. Common denominator is that you have to close the tab and open a new one to restore connection.

@icing I think you'd be able to reproduce this yourself if you joined my chat again and just idled there for a while.


Monstre ! J'avais mis DES HEURES a faire cette mod !


なるほど、承知いたしました！


no, in this case selenium will get element twice


@graingert the problem now seems that all tests use python 3.5. IMO line nr. 2 should be removed. do you agree?


Runtime


:warning: Don't merge. This doesn't work. No idea why.


No. It is only reference. Type doesn't metter.


That's not the case: https://travis-ci.org/sehmaschine/django-grappelli/jobs/101538004#L123

We're simply using Tox to manage the build env


hmmm, why not put everything in the loop.

Your while condition makes it a little hard to break out of.


As an abbreviation, "ui" should be uppercased. @ptihic


Yes, for CouchDB. Without them, a "change" will be detected and we run into the INSERT+UPDATE issue there.


Done.


Так там нет этого списка.


so this doesn't matter? https://travis-ci.org/sehmaschine/django-grappelli/jobs/101538004#L89

why do we need the python versions in that case anyway?


@ardobirk Ole hea tee repositooriumis pull request, siis saan arvestatuks lugeda.


Sorted


Wie is dit?


А как попасть на эту страницу с сайта? Где ссылка?


Something like

```ruby
map = crowbar["run_list_map"].select { |k,v|
    (self.crowbar["state"] != "crowbar_upgrade" && v["states"].include?("all"))
    || v["states"].include?(self.crowbar["state"])
}


You mean "Stop people challenging multiple people at once"


My file seems fine, not sure how can I sort it as I guess the problem is with the previously existing spaces


we need "Python 3.5" in the travis.yml so that travis will pull in Python 3.5 for tox at https://travis-ci.org/sehmaschine/django-grappelli/jobs/101538004#L68


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | **12/15** |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | **18/24** |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **5/21** |
| _WTV | 13/13 |
| _XDS | **33/34** |
| _Options | **82/87** |
| _DTVCC | 14/14 |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/143) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


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


@Raffo

looks good so far.

There's one thing we've discussed here, could you please add the additional event type "app_terminated_event"

...
switch eventType {
 	case "app_terminated_event":

And we need to add in types.go

type AppTerminatedEvent
  Event
  Appid      string `json:"appId"`
}


I see ... so 2.7 and 3.4 are already installed and 3.5 needs to be downloaded. right?


Done


ok


Sure, I'll add it.


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


@sehmaschine yup


layout


thanks for clarifying!


close #75


测试


fix #75


(I removed the note).


Only to track it

methods should be renamed to
* handleCreate(e event)
* handleUpdate(e event)
* handleDestroy(e event)


прости. на какую?


> Not exactly correct. This breaks nothing that wasn't already broken before.

 Really? It changes the logic. Not initializing m_item is ofc a bug, but fixing this like you did introduces another bug.


It's time to stop.


done.


**Bravo**


Usage: mvn clean install -Pwas85x -Dwebsphere.home=<path to websphere home>


@vitaly-krugl I've pretty explicitly stated that pipelining of synchronous methods without `nowait` set **is not guaranteed to be safe** by the server. If you need a definition of pipelining, it is sending an Nth+1 request before receiving a response for the Nth request one. I don't know what exactly "back-to-back" means to you. The above is the clearest I can put it.

In several clients I maintain this translates into waiting on a continuation before a synchronous method without `nowait` set returns.


aha ok to ja pa ql


If you let me know when its done that'd be great as I'm looking forward to pulling this into my fork. To be honest wouldnt even be that bothered if it was just a config option, its such a massively missing feature!

Nice work though! :D


No view for these fields?


@56 self


这个文件可以在.gitignore中忽略


For me, Kodi does (by luck) not crash. I'm using the broken code you fixed on a daily basis for months. Your change breaks behavior 100% (no luck to get it correct) and makes EPG actually unusable. Fixing this "jumping" EPG grid took me quite some time and I will really take care that this logic gets not broken. Thus, I do very sorry, -1 for this commit.

But it is actually trivial to do it right, isn't it. ;-)


Just a pedantic style thing, generally in HTML you'd use double quotes on the class attribute, so in JS you'll see us doing strings in single quotes a lot:

$(this).append('
' + display + '%
');


LGTM


на первую можно попасть из меню. на вторую никак, она лично присылается автору


Ну тогда ваш новый автор об этом не знал, раз в чате спросил.


agree, the code ahs been changed
now we have
mail = super(MailMail, self).create(vals)

so returning mail is correct


@AlekSi в каком чате?

@dcromster, поможешь?


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


Not sure why you've added downloader here; I've removed the dependency on that package in favour of using curl+httr (all of these dependencies should be retrieved automatically so should be removable now).


Rather than:
  "Benchmark was executed with attached debugger."
this reads better as:
  "Benchmark was executed with the debugger attached."


@akhedrane what do you think about listing all Arabic speaking listings at the end with some title Arabic? or Middle East ?

I'm just suggesting something here, eventually it's your decision :smile:


To avoid creating a new delegate instance for every dispatch.


Он у вас один, https://gitter.im/backendsecret/global_backend


@malirod as I can see from current implementation we set deregister callbacks for libusb and after that perform libusb shutdown. Seems like it's correct behavior


fix_cppcheck_issues (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1502 of 1502 tests passed, 0 test warnings)

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


actually, the commit still has to be reverted


у меня его нет.


I hate this solution but in order to do this in a cleaner way I would need to refactor too much.


huge amount of space indents replaced with tab indent for consistency


@dcromster должен помочь в будущем


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


You can safely remove setTimeout and replace them with offsetHeight request to trigger repaint. The popover take an initial position without animation and is after animate with a new position.

Replace the old hack with transition-delay: 0.1s


Why force: true ?

Does other drivers will be ok with this option?


Feel free to make any changes in my English. =)


This also fixes building 5.5 series on my FreeBSD 9.3 box on sparc64, please backport it there as well.


:+1: . I've changed that as part of https://github.com/Particular/docs.particular.net/pull/1036


Le controlleur ne permet pas des ajouts donc j'ai supprimer certaines fonctions afin de mieux les re-ecrire


I do.


I did a new PR : https://github.com/OCA/social/pull/36


:)

-rosh

> On 11 Jan 2016, at 6:29 PM, Craig Barraclough <notifications@github.com> wrote:
>
> Update the README - got to keep @rvilat happy!
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cbarrac/jokes/commit/a2da43ec69abe8517f342bd0eaf1ef283122d109


Nice investigation @leeoniya.

I love the dbmonster benchmark as it's actually a very realistic real-world for the line of work I'm involved in (financial trading applications). In terms of the Inferno benchmark, none of that is actually compiled, that's all hand written templates (I actually love writing them explicitly like that). In terms of JSX, some of the things it can do that you can't really do at runtime is determine things `val1 + val2`. At runtime you don't have access to that, you simply get a value, but because we know that there are two references at play we can do all sorts of cool things. It's actually a very complex problem – and lots of very smart people are trying to solve these problems, thus why Babel has taken off so much.

Interestingly enough, where Inferno really shines compared to vdom implementations is on mobile. Everyone always benchmarks their frameworks and libraries on desktop – but really, there's not much of a need as no one is really having performance issues on desktop platforms. Inferno literally destroys the other implementations when I tested it a few days ago on Safari/iOS 9 with an iPhone 6 – which is very important!

Great work with domvm though. Keep me updated on it! Hopefully my critique has helped/motivated you and helped with some ideas. :)


Я имел ввиду у "команды backendsecret" :)


port?.. Add arg to class?


count(*) != 0 is not SQL standard


shiyao


@Krukov, он не до конца рабочий)
по задумке: в таблице выведутся все студенты из модели Студент, но чекбоксы селектятся если записи из журнала по дате равно дате в календаре.
Я пробовал объект Журнала итерировать for j in journal и в теле цикла играться с days[], students[]
Но тогда таблица выводит все чекбоксы выбранные, где надо, но тогда список студентов сокращается до тех, где есть записи в журнале.
Может даты из журнала в отедльный массив загнать, а потом в days.present проверять равно ли дата из массива дате календаря?
Вся проблема с вот этим пониманием: как правильно зациклить, чтобы сохранить всех студентов для видимости( в будущем же по клику на чекбокс нужно будет insert ить записи аяксом)


What's the point of adding `scriptCode` here? `txTo.vin[nIn].prevout` already identifies the output and we do not really need to keep compatibility with useless OP_CODESEPARATOR behavior here.


It's time to stop.


в 1С раскраска внутри строкового литерала применяется только к строкам комментариев, которые начинаются с новой строки


In README.md `revision range` is used. I think it would be good to keep the docs similar. Same difference is on line 17 (revisions vs commits).

!mustfix unify doc.go and README.md


use="required"


tadyto je sranda.


@nk0leg please resolve conflicts


"xsd:string" msdata:Ordinal="1" />


homework


This is a bit of an odd one. All drawing is done to `double_buffer` which is blitted or stretch-blitted to the screen. So I would expect `double_buffer->w` would be the correct value to use. However it isn't, AFAICS `double_buffer` is set to be `SCREEN_W2` * `SCREEN_H2`, and these are defined in `kq.h` as:
~~~~
#define SCREEN_W2 ((320*2) + 16 + 16)
#define SCREEN_H2 (240 + 16 + 16)
~~~~
I don't know why.


Any plan to remove that ?


This commit introduces a major undocumented BC break.


refs #64


Well, it basically says the current taken drawn from battery during sleep in microamps. I have not measured, I have no means to do that. In sony brooks architecture, with screen off it is almost the same as in hawaii, that is 1450 uA. With ambient display on, the sleep current in sony brooks is 4500 uA, see https://android.googlesource.com/kernel/bcm/+/android-bcm-tetra-3.10-lollipop-mr1-wear-release/arch/arm/mach-java/sony/brooks/board-bcm59xxx_brooks.c. I have noticed in my tests that after a long sleep, the lost charge estimates of coulomb counters where like three times too small when compared against ocv estimates. In my opinion, it suggests that the orignal sleep current value was too small. I increased it.

If calibration works well then this value does not really matter. It can be compensated by the adaptive algorithm.


/resheader> <resheader name="writer">- <value>System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089</value>


I think testScheduleWithDelayObserveOnReturnsOnTheCorrectThread is not specific enough. I.e. it should be x thread not a general vertx thread. I say this because this is what the change it supposed to fix.

 line 118: I would suggest that result -> {} is a final static object to reduce object churn.

I like the cleanup of the tests but it is less obvious that the subscribe thread is calling though the observable. but it is less code and looks simpler.


@bambr1975 спасибо, поправил!


Hi Claudio, sorry, I was not online Sunday afternoon...


Sonandan


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 192](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=654&buildTypeId=XbimEssentials_XbimEssentials) is now running


#6 imageモデルは複数名詞として認定されずダメ。


i need someone have meter to measure it too, look like on android they have this list
https://github.com/SandPox/android_device_samsung_kylepro/blob/cm-12.1/overlay/frameworks/base/core/res/res/xml/power_profile.xml for system to calculate how much time left using battery


@bambr1975 хотя, наверное, поторопился я с ^ в начале регулярки. должно быть `^/s+` если я не ошибаюсь, да?


@fishcakez this commit makes ownership checkout manual. Right now we have about 8 failing tests.

I have excluded only the reconnection tests because I don't think I can make them work. If something checks out the connection and crashes, the whole ownership thing is gone, so we can't reconnect. The other tests need checking on a case by case basis and I will need your help to decide if it is a test issue or an ownership issue.


added plugin clean


if nesting could be avoided by making this a guard that returns false


Not yet, but I'm on it.


Good job!. Great use of the AND statement


I'll have to set one up.  It looks like there are at least two variables at work here:

- Python earlier than 2.7.9 vs 2.7.9+
- pyVmomi 5.5 vs 6

As you know 2.7.9 replaced the entire ssl stack, introducing SSLContext.  It looks like pyVmomi added SSLcontext in version 6.  So we need the Salt vmware modules to support both pythons earlier than 2.7.9 (including 2.6) as well as later, and both pyVmomi 5.5 and 6 (assuming that 6 cannot talk to vSphere/ESXi 5.5 and 6), in any combination.

I have code in my pyvmomi-fix branch (https://github.com/cro/salt) that was a start, but it looks like pyVmomi carries the context all the way into httplib, and httplib earlier than Python 2.7.9 doesn't recognize the context kwarg.

Feel free to poke at this some more, let me know what you find.


Great job. Another alternative here was to use an AND statement in the condition.
`if (i % 3 === 0 && i % 5 === 0)`


`return nil, err` here too, or are you assuming that something later is going to fail anyway? (if the latter, a comment saying so would make it easier for future code-readers)


I suggest the following instead. Simpler, more precise, and not relying on the start/end points to be in Intersector::WINDOW coordinates.

<pre>
    osg::Vec3d eye, center, up;
    mView->getCamera()->getViewMatrixAsLookAt(eye, center, up);
    osg::Vecd direction = center - eye;
    direction.normalize();
    return eye + direction * 50;
</pre>


Please include the contents of `err` in the error message, so there's some hope of debugging the issue from logs (pet peeve ;)


Thanks for this Fernando this is very helpful!  I tried to add comments to
my project when I saw that it was in the rubric, however, had trouble
trying to push it back to my github. Would it be possible to go back and
add to the project and re submit for a higher score?

If not no worries I just figured I'd ask.

Thanks Fernando!

On Monday, January 11, 2016, Fernando Jose Delgado-Quintana <
notifications@github.com> wrote:

> Great job. Another alternative here was to use an AND statement in the
> condition.
> if (i % 3 === 0 && i % 5 === 0)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TravisSalad/wats1020-fizzbuzz/commit/d67339a8ccd56d2ef17eda2f61295b207d6a1e2c#commitcomment-15384738>
> .
>


mode-mouse and mouse-* doesn't exist anymore with tmux 2.1. You must remove all these entries and add only "set -g mouse on" to enable mouse mode.


Great job. Good use of the AND statement


This is not a breaking change, right? It looks like it's adding an option to turn off the strict floats behavior. Otherwise looks good to me.


Note: The ```<< 1``` conflicts with PR #872. I refactored the mask setting so that shifts are no longer necessary.


Sure!


The spacing alignment here is a little strange.


綾瀬駅の路線で、人が暴れてるため遅れます。
2015/12/25 16:28 "gg-soeda" <notifications@github.com>:

> 添田です。
> 良いですね！こうやって自分で改造する事は、どんどんやってください。
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/masaki4680/camp-challenge/commit/9c690fc0dd6d14c58971e7db96954bfb124ab33e#commitcomment-15159285>
> .
>


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


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


@cepages I hope this is useful!


Why was paint creation removed from drawImage? It's possible to draw an image with a drop shadow, and I thought that paint object is required for shadows.


There needs to be a better way to resolve this - most likely with a prompt to the user, but this is sort of deep in the code and far from a user.

A user-side workaround for this is to supply multiple *default* credentials, making sure that any one cloud has only one *default* set checked.


Why do we send a 200 code for all batched results regardless of condition?


@caioeps você tem que colocar o valor onde tem esse "nil".
Olha aqui: http://api.rubyonrails.org/classes/ActionView/Helpers/FormTagHelper.html#method-i-text_field_tag
Desse jeito da erro


In files that only contain classes, all of these checks become redundant - there's no side-effects if someone takes the trouble to access the file directly, but there is an extra task to see whether the constant is defined on every file load. If the constant isn't defined, the page will exit. If the constant is defined, the interpreter will read the class, do nothing, and exit.


I realize I'm an outsider and I'm sure I don't have any sense of the issues @3rd-Eden went through, but as a consumer, big +1 that this change broke us and has been a frustrating ongoing source of issues.

It doesn't seem to be as simple as "ignore" if you're using Webpack, but I'm admittedly new to Webpack. (I never had to think about it before, as our simple bundle "just worked" everywhere.)


Nevermind, that wouldn't account for the mouse coordinates.


Awesome thank you!!!

On Monday, January 11, 2016, Fernando Jose Delgado-Quintana <
notifications@github.com> wrote:

> Sure!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TravisSalad/wats1020-fizzbuzz/commit/d67339a8ccd56d2ef17eda2f61295b207d6a1e2c#commitcomment-15384939>
> .
>


It is just a tempolary jeklly blog. I'm just too lazy to add it. :bomb:


I've put this into a PR, along with some retry logic that we have been using anyway.

https://github.com/appium/appium-uiauto/pull/96


@aseemk, are you using socket.io by any chance? you may only need to upgrade it.


Oh yeah I realised afterwards and actually edited a comment into your wiki for anyone else with the same issue with scrot :)


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


This is not a breaking change since the flag is set to the old default of `strict_floats = True`.


Awesome use of conditionals, loops, functions, events, DOM manipulation. Great work. Your code looks clean and efficient. One recommendation, from my point of view: you can add a `<br />` at the end of all your `innerHTML` lines so instead of creating a paragraph you have a list to the bottom. Just my opinion everything looks great.


No, we use `ws` directly. Thanks though. =)


thoughts on submitting a PR to the gem for this? I need the same change


This was our solution:
https://github.com/socketio/engine.io-client/blob/master/package.json#L56


this.userId?


contriollare


Outra coisa, como tu fez se o user não tem esses campos?


Whats the expected output? Can uou write a test?

> On 11 Jan 2016, at 23:36, pagabuc <notifications@github.com> wrote:
>
> Thank for your super prompt reply, but it's still not working as expected.. it doesn't print "invalid" :-(
>
>
> —
> Reply to this email directly or view it on GitHub.
>


Cambierei `il` con `lo` (suona meglio no? Lo zero-value)


Hrm, moving this code here makes neutron required to get any functionality from this cloud provider.  Previously we avoided looking for / requiring the neutron client until the actual networking interfaces were asked for, so you could get (eg) `Instances()` but not `LoadBalancer()` if you weren't using neutron, and (in theory) we could test for and provide one of several implementations depending on what we discovered (eg: I intended to write a nova-network implementation, and a rackspace implementation - but someone made the latter as a fork of openstack.go and I haven't refactored it back again yet).

I agree the increasing complexity demanded of cloud providers is making it harder to keep the sub-interfaces logically separate, but I still kind of like that original goal... If it gives a substantial code cleanup then it'll be worthwhile - let me chew over this for a bit.

If anyone out there needs the openstack cloud provider and is *not* using Neutron, now is the time to shout.


See Issue #70. Basically, after some discussion we decided that there are two many combinations of choices which come up if there are partial failures. The best choice was to treat a batch like a success and allow the user to determine failures out of the batch.


Cambierei `e` con `ed` (iniziallizzare una...)


@rauchg: That doesn't seem to do it for Webpack, but there are other ways (with various pros/cons). I'll post here once I nail down what we land on.


This is just looking at the success field. Why bang bang it? Or we specifically want to coerce a string into boolean?


The indenting is off in this file. It makes it pretty hard to read. I'd use a beautifier or clean it up by hand.


Your original approach was not wrong. It was simply missing the `create_collection_of_integers` functionality that it depended upon.

In this new approach, it looks like you are expecting `max_by` to do more than it's capable of. Would it help if we discussed what `max_by` does?


Oddly enough, I was just reading about the benefits of env this past weekend.


Looks like @TeoTwawki didn't change the var name when he changed from mobid check to skill list check though. Should be skillList var not mobID var.


Hey Rodrigo, I was testing the Shortcake integrations, and I'm finding that it doesn't make sense for any of the components that wrap content or other components. Could you kindly remove the Shortcake integration from Mm Expandable content? Thanks!


I should have made this commit in smaller pieces, one with the playGame method and one with the main method.


Same note as the bang bang status.


Cambierei con `è l'unico costrutto per eseguire cicli in Go`


What's 'online'?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - Win64-Windows-7-VisualStudio-12: Build Failed

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


You actually don't need this method. If you have a template and the route, Rails will fill in the gaps.


Indentation issue.


As discussed this not needed, its a bug in our library.


Indentation issue on this and the next line.


Qui mi discosterei dall'originale e cambierei `il tipo` con `il ciclo`. Siccome si parla di tipi anche prima non vorrei creare confusione.


Can we change "Inform" to "Chose", and can we explain what needs to be used here? I tried using the class for FontAwesome (fa-fort-awesome), and it didn't work. :-/


Can we please use a more *testy* value?


This is a bad idea to name arguments, name it exactly as documented.


We should probably add `null: false` to the migration for this column.


Possiamo cambiare con `inizializzazione/test/incremento` (da [Wikipedia](https://it.wikipedia.org/wiki/Ciclo_for) )


Probably going to remove this.... that's a link to an unsupported UI for it, not the actual builtin.


Why did you drop this (out of curiosity)?  I agree it's a micro-optimisation, but we know here that we're expecting exactly one element in serverList in the vast majority of cases.


Is it really copied if it isn't used?


typo: `tiime`


Did you write any specs for this controller? I think it would be a good exercise.


I think you should write some tests for this controller.


This should still be `release`. Once 0.5 comes out we cannot add 0.4 to this list.


Is there a way to upload multiple images here? In a real-world scenario, we'd have a number of logos in displaying when this element is used. [10up](http://10up.com/) has a good example of logos on their home page, and while it's not a strip as such, it's still a good example of what we're aiming for with multiple logos.


This is a random token, right?


Need more than this - would you include email address?


Cambierei `agli range statement` con una delle due:
* `allo statement range` mantenendo l'inglese.
* `al comando range` che fa sembrare il discorso un po' più lineare.

Francamente non ricordo se prima abbiamo usato statement o comando, comunque è abbastanza indifferente.

Inoltre ti volevo avvisare che io in seguito ho usato `channel` invece di canali (seguendo il principio di mantenere in inglese tutti i nomi di comandi/strutture dati di Go, vedi slice, map, array, etc.). Che famo?


Please make this a json hash. We'll have a much harder time dealing with the RPC layer when there are all of these special cases.


This probably contains data that is temporal.I will see if I we can download the Intel implementation.


Can temporary creds not be created from temporary creds?


I use the keys to create new values, the `@autorelease` block ensures that they are autoreleased before the end of the runloop.  Using string constants or autoreleased strings means the values are not deallocated by the time we test to see if values ares still present in the cache.


You can probably delete this file and the welcome_helper file.


We probably should add OSX for testing.


Mmmmmm... Pie.

Is this a random, test token?


You could test the validation we wrote last time here.


Not much to test in the Item model. Go ahead and remove this file.


Nah. There is no noticeable difference to the user whether the failure callback happens asynchronously or synchronously relative the the core data save.  The optimization is just that the save completion block is already executed on the main thread.


You can remove this file.


Any reason why we use the master branch of OpenCL?


Should we be returning `serverFileUri` instead? Looks like we do the remote-to-local mapping to that variable but still return the local path. I noticed this after testing this change with a different project than the testproject included.


Go ahead and remove this file as well.


Hm?


Just a bunch of snippets stored in the cloud


This can be `list.items.new(item_params)` and then you can remove this next line.


This was my first mockup as well, but I'm not sure it's correct. According to [the pg spec](http://www.postgresql.org/docs/9.3/static/datatype-numeric.html) there are three float/double types. Does the pg gem map them to `:decimal` and `:float` or do they have to be covered as well?

![skarmavbild 2016-01-12 kl 01 21 33](https://cloud.githubusercontent.com/assets/220827/12250914/f3026652-b8ca-11e5-9564-77b9b292a278.png)


This shortcake element seems to be missing URL fields for our default social networks, so there is no way for this to actually output on the page.


I think this will only find the first address on each network (thanks to `list[0]`) - am I misremembering the expected datastructure here?


First PR! Looks good, great start to the permutations!


what are you doing
array len works fine

it's that args are passed as ^str, not []str


If your method is a big `begin .. rescue` block it can be written more concisely like so:

```
def destroy
  item = Item.find(params[:id])
  item.destroy

  render json: {}, status: :no_content
rescue ActiveRecord::RecordNotFound
  render :json => {}, :status => :not_found
end
```


Ah, I understand now.


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


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


You could scope items to current_user like we did in our last session with the lists controller. I know you were concerned about security.


This effectively duplicates https://github.com/kubernetes/kubernetes/pull/18829 - which I see hasn't been submitted yet (it's ready to go afaics).


http://www.material-ui.com/#/components/svg-icon

https://design.google.com/icons/


@rhc54 if ```OPAL_ENABLE_DEBUG``` is 0, then the global variable num_event_users is not incremented.
i was conservative and considered this as a bug when i fixed this in open-mpi/ompi@0fb7b07a719a4dbcdd1cfaefc220f84edb4ef9a3

can you please double check that and confirmed that was a bug and not an optimization ?


Check out my comment on destroy in `app/controllers/api/items_controller.rb`


Thanks for spotting this. Hiwever, the protocol scheme should be "git+https". "git" will also work in most cases.


same comment here, ```num_event_users``` is not decremented if ```OPAL_ENABLE_DEBUG``` is 0


pbi?


Derp.


Oh shit yeah we fixed, lol


This should be a noop if `tags` is an empty array (I think). Was it causing problems?


_buffered_state_diff is used to keep track of what should be synchronized from the frontend to the backend.  Did you find that synchronization was echoing?  Alternatively, you could iterate of the state and call

```js
WidgetModel.__super__.set.apply(this, arguments);
```

Either way, could you add a comment explaining why this (or that) is needed?  This is the kind of line that would be real easy to forget about.


Oh, I assumed there was a reason it wasn't included before


Old habit, and you're definitely correct.

I err on always including it vs. mistakenly omitting it, and there are likely to be much more worthwhile optimizations to be had elsewhere, but I'm okay with a PR to clean up files where this check is functionally useless.


You're right!


Add `null: false` here.


@gavinking Is this on some branch, or was it reverted? I still see `value` on the String constructor in my (up to date) working copy...


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 119](https://teamcity.labkey.org/viewLog.html?buildId=296277&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 21, passed: 267 Build time: 01:36:32
### Failed tests
```
org.labkey.test.tests.DataFinderTest.testStickyFinderFilterOnDataset: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


I was being lazy before and didn't check if `val == null`, in which case the result should be `[]`.


This isn't rendering correctly when I choose all of the options (on either the TinyMCE preview, or on the published page). Can you make sure when boxes are checked that they are saving correctly and outputting correctly.


you are right my bad. It is fixed now.


Wait no, when I run I get this:
```
felix@felix:~/dev/ark$ ark build test
error: [test:10:10] Expected `:` (operator), got `::` (operator)
        C::printf("%s\n", args[i]);
         ~~
```
Which I'm 99% sure is due to the array len being in that for loop condition


this line has to be deleted in next commit


not if you use julia 0.4.0


Woops, sorry, I thought I'd gotten all of these, but it looks like I missed the rename step on that.


Woo! There are a few things I noticed... I'm working on and testing a few changes right now related to this and will submit an issue to explain them (and hopefully a PR) if they all work out. I learned and figured out a lot while continuing to look into the lowercase mapping issue.


In webpack I usually create a stub.js that just exports an empty function,
then alias the offending package to it.
On Jan 11, 2016 6:14 PM, "Aseem Kishore" <notifications@github.com> wrote:

> @rauchg <https://github.com/rauchg>: That doesn't seem to do it for
> Webpack, but there are other ways (with various pros/cons). I'll post here
> once I nail down what we land on.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/websockets/ws/commit/e54d45fbab3b19c2940e9057ce1e7b8f105873e0#commitcomment-15385122>
> .
>


Pressurestat (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1450 of 1450 tests passed, 0 test warnings)

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


my bad! I forgot to change a line of code that I knew was not correct at
runtime.
The fixed code is now on github

On 11 January 2016 at 20:08, Federico <notifications@github.com> wrote:

> I've integrated the new code, however I get runtime error:
>
> Optimisation::find(): iter=0 params[l,s]=[1.000000, 1.000000]
> gaussian_process: /usr/include/eigen3/Eigen/src/Core/ProductBase.h:95: Eigen::ProductBase<Derived, Lhs, Rhs>::ProductBase(const Lhs&, const Rhs&) [with Derived = Eigen::GeneralProduct<Eigen::ReturnByValue<Eigen::internal::inverse_impl<Eigen::Matrix<double, -1, -1> > >, Eigen::Matrix<double, -1, 1>, 4>; Lhs = Eigen::ReturnByValue<Eigen::internal::inverse_impl<Eigen::Matrix<double, -1, -1> > >; Rhs = Eigen::Matrix<double, -1, 1>]: Assertion `a_lhs.cols() == a_rhs.rows() && "invalid matrix product" && "if you wanted a coeff-wise or a dot product use the respective explicit functions"' failed.
>
> I get there with sphere and with mug. Not sure if i missed some
> parameters, but i copypasted test_gp.cpp
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pacman-project/gaussian-object-modelling/commit/53c05d80e5a80affc88cfcdc4db24ec42b2797a2#commitcomment-15379626>
> .
>

--
Claudio

Curiously enough, the only thing that went through the mind of the bowl of
petunias as it fell was "Oh no, not again".
Many people have speculated that if we knew exactly why the bowl of
petunias had thought that we would know a lot more about the nature of the
Universe than we do now.
D. Adams - Hitchhiker's guide to the galaxy


Awesome work! You've really done well on these projects. I have a couple of things I'd like you to fix, and some suggestions (up to you if you want to change the code).


Is this correct? I'm pretty sure it is the number of elements created in the map.


Why the hell, bro?


Code looks well formed. Does the check @ line #722 want to be `>= disarm_time`? LGTM as-is. :+1:


Thanks! @imurchie


Hmm, you're right, but, I thought I fixed that...

My other point still stands though.


Why is `setHasOptionsMenu(true)` called in the constructor of `DetailFragment` here whereas it is called in `onCreate()` for `ForecastFragment`?


ack
Didn't know we even had that file...


ack


Could you add a newline at the end of the file?


That's a kind of magic.


Wish I'd noticed that space after before the closing parenthesis sooner. Hate that.


It's a data format actually, as opposed to the binary format. When I talk about ASCII art, I mean the symbols that can make up a word or phrase.


It seems we have the same interesting behaviour in this fixed version.

Iter: 431, A: 5000, alpha 0.602000, gamma 0.101000
param: PW[1], best: 209.50, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[2], best: 208.89, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[3], best: 205.29, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[4], best: 203.30, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[5], best: 205.98, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[6], best: 201.39, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[7], best: 201.39, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[8], best: 204.72, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865


remove... we are now es5 compliant.  I changed all the for..of to for..in.  Unless... you've added new es6 stuff?  I'd recommend against that, as it seems to cause trouble for the Android people.


AArch64 supports PIC fine, it is only the x86_64 recompiler which doesn't.


static int pull_period()


unack, wait, did you copy the commit message from the commit in pyprint? Since the file already existed, "add manifest" is not fitting^^


Haters nothing but fans ;) GIVE IT UP cause even this one man show does more work then your team of retards that have to beg for members and go as far as tricking them .... bitch please you just a new koolaid flavour thats gonna loose its taste fast!


Since this overrides you can make it just always set s_opengl_mode to be MODE_OPENGLES3 instead of overriding in Create


Do you think they should be defined in GLFW.jl?

Sidenote: the `isequal` definition may not be needed since [the documentation says it'll use `==` by default](http://docs.julialang.org/en/release-0.4/stdlib/base/#Base.isequal).


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


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1202.f6cb7481d70cb500c37d1cd2bf3d43cd9f666047](http://build.palaso.org/viewLog.html?buildId=67839&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 683, ignored: 6) Build time: 00:08:17
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


This makes good sense. I was already planning to do something like this because two lists shouldn't be maintained. I'm also planning on adding a cli option along the lines of `--sites=christianity,hermeneutics`


:+1: :100:


Thanks!


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Build Failed

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


Like I commented [here](https://github.com/apple/swift/pull/435#issuecomment-170743357), this issue should probably be fixed in the swift cmake files, so it may be more correct to create a patch and try to get that merged upstream.


OK I am stumped.

The class SecureHash provides a method to calculate a secure hash, that as an input, takes a secure hash?
And then we are rejecting the SecureHash keys in it.

Can you please clarify how this is supposed to work?


OKです、ではまたpushをお願いします！


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1204.f6cb7481d70cb500c37d1cd2bf3d43cd9f666047](http://build.palaso.org/viewLog.html?buildId=67842&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 623, ignored: 4) Build time: 00:04:38
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


@pikajude Mind explaining why this was reverted?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1203.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67841&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 629, ignored: 4) Build time: 00:07:20
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


this commit breaks all ldap groups in our configuration :/, i'll add more information tomorrow


775-CurveAsResource (kbenne) - i386-Windows-7-VisualStudio-12: OK (2328 of 2373 tests passed, 0 test warnings)

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


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


i think this commit belongs in another PR, since i don't think it's directly related to the bug at hand (any more than it is any other open bug or enhancement).

also, it appears that coveralls no longer reports status on the PR (on this branch), which is not the case on other open PRs.  i don't have any real coveralls knowledge, but maybe removing the `.wear!` call is causing that trouble?

at any rate, i'd vote for giving these changes review of their own, separate from issue #76.


@echel0n Nobody is tricking anyone, and I'm not part of the SR team, so it's not "your team". And if your claim of "one man show does more work than your team" was valid, I would still be using this version. From what I see though that version is where all the development is going on, issues are actually able to be reported, and stuff is handled within hours instead of weeks.

Can you please respond to all the PR's you have been closing where people ask you why with no response? You are doing nothing but hurting yourself there.


Nice


Yeah, for now it's more of an idea of how the language should look... so I'll leave it as is for now


I created a [new Browserify + React example app in raven-js-examples](https://github.com/getsentry/raven-js-examples/tree/master/commonjs-react) to verify this works.


I think there is a typo:
on BALL_PLACEMENT_YELLOW a lower case 's' should be sent


Or straight out ignore it and then just in Create set it to ES3.
Since the EAGL context won't ever actually be supporting GL.


I agree. I just need to _something_ to get this building again. Do you even know where to begin?


It won't do anything to websocket handler as every message will be processed in different thread (websocket handler will consider the message is processed). The thread will probably stop when a RuntimeException raised here. Because there shouldn't have anything really cause the InterruptedException, so I think stop the working thread is the right thing if an interruptedException happens.


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


:+1: to @jmartin-sul comment


rebase avec la master stp j'ai des conflit ^^


@ragesoss Try now?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Build Failed

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


It looks like these were the only places where ManualDiscountForm was used; could you double-check and then purge it if that's the case?


Testing a method should have its own describe block.
```ruby
describe "#score" do
before do
# setup here
end
...
tests here
end
```

Additionally, in a seperate commit we should fix the other test, the it block and let values for that test should be in another describe block.

```ruby
describe "validation" do
end
```

~senjai


@nickyoung87 Looks great.


s/blackhole /has no endpoints/


Ouch. Indeed!

But I'd really prefer if we could leave this as constant string instead of invoking snprintf() each time. snprintf() is very slow after all, and we do this for each line.

Can you rework this patch to just use the string literal "SYSLOG_FACILITY=4" instead, and then maybe add a compile-time assertion that this is indeed 4? (`assert_cc(LOG_FAC(SYSLOG_FACILITY) == 4);`)

That way the string remains constant, and we make sure it actually matches the header's definitions. (Not that the value could ever change, after all it's part of various syslog RFCs)


remove?


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


get-pip add to installer


Why pass any options at all?


match -> matching


Is this really required?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1206.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67845&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 2 (1 new), passed: 673, ignored: 6) Build time: 00:11:13
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1205.f6cb7481d70cb500c37d1cd2bf3d43cd9f666047](http://build.palaso.org/viewLog.html?buildId=67843&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 2 (1 new), passed: 686, ignored: 6) Build time: 00:11:20
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData: <no details avaliable>

GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


```ruby
def create
  if current_quote.votes.create vote_params.merge(user: current_user)
# do things
end
```

You dont necessarily have to do it this way, but in my opinion it's a cleaner approach.


dictionaries are cool too.


Updated with another commit that tracks per-diagnostic state.


@johanjanssens don't we want multiple tables to share the same tag and alias? Otherwise we would end up with {id:1, tag:coffee, table:people} and {id:2, tag:coffee, table:todos}. In that case tags will be unique per table and not per system as a whole.


It's all Taylor :)


> I can't seem to find any CMake documentation on this idea of namespaces for targets

@LegalizeAdulthood It's described in https://cmake.org/cmake/help/v3.3/manual/cmake-packages.7.html, this example is ready-to-experiment with such approach: https://github.com/forexample/package-example


Will fix


Not exactly since I want to enable multiple calls to `configure` for different adapters.

Nevertheless I think this should be `_.extend(adapterConfig, _.clone(config));`


This URL works fine for me.


You'll want to `EnsureChain(utiliptables.TableFilter, iptablesServicesChain)` and `EnsureRule(utiliptables.Prepend, utiliptables.TableFilter, ...` which jumps to iptablesServicesChain.  See the block around line 464 and replicate that, I think.  Then you can insert this rule into iptablesServicesChain


We should define subject explicitly here.


Not sure this has any noticeable effect.


saw approval from javajolt
at https://github.com/basho/riak_kv/commit/8e0f837344142057314e64cd475421a8ab441ac0


merging basho/riak_kv/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 8e0f8373 into borshop-integration-1323-perf/ttb_newmsg_ts1.0_release_candidate_v2


basho/riak_kv/perf/ttb_newmsg_ts1.0_release_candidate_v2 = 8e0f8373 merged ok, testing candidate = 448e4c0f


wtf haha
On Jan 11, 2016 17:20, "Gigatek1" <notifications@github.com> wrote:

> Very good. Looks like everything is in place for the new pizza cannon
> addon.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/A3Armory/ArmA3_Wasteland.Altis/commit/47c791187f60fa585d61f948a3da23db18420ae2#commitcomment-15384135>
> .
>


Subject should be defined only once per describe and should be consistent throughout your test.

`subject { post :create, vote_params }` where you can define vote_params with let() in each subcontext you require the params to be different.


That will do it. :facepalm:


some server does not -> some servers do not


I checked VAImage has to be hold before releasing VA buffer handle.


then instead of forcing cheats to be enabled, just KISS and restrict the functions to LuaRules...


@fw42 can we get rid of this hack now that golang seems to support handling SIGCONT now (https://github.com/golang/go/issues/8953)


```ruby
vote = current_quote.votes.find_by(user: current_user)
vote.update(vote_params)
```


update commit per last review comments. will push to 01org as well.


Angularjs-Bootstrap-Firebase singlepage App for the artist's paintings' collection


is_systemd cannot be used for this purpose. the `log_job_error_with_service_result()` needs a new parameter.


sorry sr. way sulod akong solar system sr?


Same comment as before w/ respect to subject


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Build Failed

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


witsh -> with
work -> return true


I think this one snuck in during some of my commit shuffling. I'll split it out before final rebase


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1207.af507631ecf13b1efa8f9a014ba5525241f6eec8](http://build.palaso.org/viewLog.html?buildId=67847&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1, passed: 630, ignored: 4) Build time: 00:05:57
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Rather than doing two "for each service" loops, why not stick this in at the place where we loop "for each endpoint".


the attribute `distinguishedName` is specific to active directory, we're using openldap, and since this can not be configured it breaks our configuration, and probably all other ldap servers as well, since those attributes are not standardized


Fixes #11.


@rmdstudio Nope, we don't want system unique tags. We consider that a special case and not the default case. We will also take the 'table' column out as we no longer require that. Tags will be per component, not per system. Which makes sense as we are building a component driven architecture.