@Brahmancer **I am your messiah.** *Accept me and we shall transcend to our planar form as a collective whole conscience!*


Shouldn't this be `alljs`? Kids nowadays use `npm` for the browser with Browserify or whatever's trendy this week as well.


laughing uncontrollably at this commit


I think it makes sense to escape quotes there. We could even quote and escape unconditionally at that should be guaranteed to work across all platforms (IIUC).


@AlessandroA you could consider moving the static const g_driver into __box_debug_init - s it's only needed there.


line 29: can be run on your server -> can run on your server
31: Before we proceed, go ahead and copy over your ASP.NET 5 application to your server using whatever tool -> Remove "go ahead" and "over" (unless you are actually copying over existing files then disregard, but it doesn't appear that way). Also is there a specific location to copy files to?
44: as feature rich full-featured ->  as feature rich as full-featured
90: capitalize Unix
103: You might want to look at ``/etc/nginx/nginx.conf`` to make other necessary changes to your nginx environment to configure your nginx environment. -> You might want to look at ``/etc/nginx/nginx.conf`` to configure your nginx environment.
For the code blocks, I believe you need to add :dedent: 8 and :linenos:


Query execution is similar to upload operations, in that the duration of both depends on both (1) the operation and (2) the network speed. The default request config is used when the duration only depends on (2), whereas the upload request config is used when the duration depends on both (1) and (2). We can rename "upload" to something else if that makes it better.


I concede the point :)
On 3 Mar 2016 12:27 a.m., "Aldo Borrero" <notifications@github.com> wrote:

> Yup I agree with you @jose
>
> --
> Aldo Borrero
> Sent with Airmail
>
> From: Jose Luis Franconetti <notifications@github.com>
> Reply: mobilejazz/CacheIO <reply@reply.github.com>
> Date: 3 March 2016 at 00:07:53
> To: mobilejazz/CacheIO <cacheio@noreply.github.com>
> CC: Aldo Borrero <aldo@aldoborrero.com>
> Subject:  Re: [CacheIO] Removed implementations and fleshed out first
> ideas of cache apis (f220167)
>
> I don't like the name Blocking because for me it means that you are
> blocking something (that it's true).
>
> I prefer to use the name Sync that means the same and you know that it
> will block the thread where you are calling it.
>
> @aldoborrero What do you think?
>
> —
> Reply to this email directly or view it on GitHub.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mobilejazz/CacheIO/commit/f22016731de5f5427acdf8f7656bdb0f934a74f8#commitcomment-16460668>
> .
>


develop (rpg777) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


@Gheoan nothing very important, but the code using `for` looks nicer and much more concise.


we easily could - ron is always open to contributions - its more about triaging time


This should be `Vec3d`.


You should just put this in your own local gradle init file rather than in the project.


@jondlm latest change uses hash. currently master doesn't work


It seemed to work just fine in my testing. Why wouldn't it work, exactly?


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


Changed this to use resource stream


shouldn't the objective be `mu` in this case?


maybe a `test` method?


:+1: Nice work, @mchu0010!


I'm not 100% sold on this but if you like it... let's do this


True, I will make it `jsall`


@avaidyam forgive my sins, oh gracious one.


Ahh good catch, thanks for fixing it!


ready_aim_fire (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


@samalamb DWMR. :+1: Ship code. :articulated_lorry: :trophy:


nitpicky, but maybe make `T_S` and `C_1` (the floats, not the `Variable` objects) constants at the top of the file, since they're shared?


FixObjectCounts (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1967 of 2120 tests passed, 0 test warnings)

 * 153 tests had: Table big diffs.

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


Yes it will increase by approximately O(n), however as the memory footprint from substantial number of simulations (say 1000+) increases, R does a poor job of re-allocating memory so there can be some slow down such that in the real world it won't be say 1000x longer than 1 rep to simulate 1000 (even though that still falls under O(n) technically), and performance can exponentially degrade in certain scenarios, though I doubt that would arise)


Form::close should probably be here? This depends on how you want the table to work. If you want to approve 1 item at a time, the close should be here. If you want to approve many items at once, then the Form::open should go above the table (or whatever is repeating) and the checkbox should be an array:

`{!! Form::checkbox('approved[]', 1, null, ['class'=> 'form-control']) !!}`


LGTM


i support you <3


closes #621


quadcopter_pd_controller.mdl var inte alls irrelevant :)


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


The failing tests were failing before I made the changes:

  98% tests passed, 12 tests failed out of 612

  The following tests FAILED:
         51 - Belos_pseudo_ptfqmr_hb_1_MPI_4 (Failed)
         53 - Belos_pseudo_ptfqmr_hb_3_MPI_4 (Failed)
         90 - Anasazi_Epetra_BlockDavidson_test_0_MPI_4 (Timeout)
         91 - Anasazi_Epetra_BlockDavidson_test_1_MPI_4 (Timeout)
         92 - Anasazi_Epetra_BlockDavidson_test_2_MPI_4 (Timeout)
         93 - Anasazi_Epetra_BlockDavidson_test_3_MPI_4 (Timeout)
         94 - Anasazi_Epetra_BlockDavidson_test_4_MPI_4 (Timeout)
         99 - Anasazi_Epetra_BlockDavidson_auxtest_MPI_4 (Timeout)
        107 - Anasazi_Epetra_GeneralizedDavidson_solvertest_MPI_4 (Failed)
        151 - Anasazi_BlockDavidsonThyra_test_MPI_4 (Timeout)
        161 - Anasazi_Tpetra_BlockKrylovSchur_ComplexLap_test_MPI_4 (Timeout)
        310 - NOX_Thyra_Heq_MPI_1 (Failed)


ready_aim_fire (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


:+1: Sorry I forgot about you! Great work.


Definitely needs this feature so thanks for the patch!  If it works...

I'm getting error:
DBD::SQLite::db prepare failed: no such table: message at iOSMessages.pm line 91.

Having everything grouped would be awesome.  I'm definitely no expert when it comes to this sort of stuff.

Any ideas?

I'm running IOS 9.2.1 if that matters.

Thanks,


@Brahmancer you are forgiven, child. :raised_hands:


nooooo - no excel :-p - the key is to use additional functions built into R such as `expand.grid`


It is Java convention to name variables in `camelCase`, beginning with a __lower case__ letter. Only classes should be written in `CamelCase` beginning with an __upper case__ letter.


This foreach should go inside the table. In this implementation you are repeating multiple `<table>` elements on the page. Most likely you want to repeat `<tr>` elements inside the `<table>` tag.

Also, lose the capital letter on `$survey` and call it `$surveys` to have more consistent naming. You will need to change your controller to match also.

```
<table class="table">
@foreach( $surveys as $survey)
    <tr>  ... </tr>
@endforeach
</table>
```


I get this error when resetting the date, time and author name. We looked into debugging this but none of the tricks online seemed to help. Do you know how to fix it?

error: pathspec '2008-09-25 22:43:00' did not match any file(s) known to git.
error: pathspec 'priyam3nidhi' did not match any file(s) known to git.

Program runs successfully locally on the machine if the last three calls are created.


You don't pass `scenes` when calling `this.props.renderOverlay` in `_renderOverlay`.
Why?


The Form::submit and Form::close elements can stay here if you do one giant form with array approved checkbox elements, or needs to move to above


`newRecord` was only there because Dixon was using the `DRecord` class (which did absolutely nothing). It is no longer needed, so I removed it and just used `newKey` instead.


for example

```
library(dplyr)
expand.grid(ID = 1:3,
            Time = seq(0, 24, 6),
            EVID = 1) %>%
  as_data_frame %>%
  arrange(ID, Time)

```

will give you

```
      ID  Time  EVID
   (int) (dbl) (dbl)
1      1     0     1
2      1     6     1
3      1    12     1
4      1    18     1
5      1    24     1
6      2     0     1
7      2     6     1
8      2    12     1
9      2    18     1
10     2    24     1
11     3     0     1
12     3     6     1
13     3    12     1
14     3    18     1
15     3    24     1
```


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


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


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


I don't like gunships having a completely different icon to every other air unit.


The following is incorrect. It will penalize only if mob == 0
`else if (rank != RANK_1 && !(b & mobilityArea[Us] & file_bb(s) || b & mobilityArea[Us] & rank_bb(s)))`

But this is correct. It will penalize whenever the file mobility is 0 OR the rank mobility is 0

You will see, the bench is different.
`else if (rank != RANK_1 &&
(!(b & mobilityArea[Us] & file_bb(s)) || (!(b & mobilityArea[Us] & rank_bb(s))))`


Sorry! Kikade på den lite snabbt och trodde det var min gamla skruttiga PD-regulator :) Insåg först nu att jag måste raderat filen i fel repo... Ursäkta besväret!


Why not?


looks good to me--the kinds of things I would quibble about are more pervasive to the current internal APIs (which are noisy--but nicely explicit)


inga problem ;)


True, I was planning on removing it before I open a PR anyways.


Thanks!


s/NULL/nullptr/; cosmetic--may as well start switching these, tho


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


hungarian notation?  we don't use it elsewhere...sorry, I'll stop commenting on trivia


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2005 of 2158 tests passed, 0 test warnings)

 * 153 tests had: Table big diffs.

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


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


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


FixObjectCounts (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Ok

I set the priority to -1 to stop this test. Let me make the correction

Thanks

2016-03-02 17:45 GMT-06:00 Rocky640 <notifications@github.com>:

> The following is incorrect. It will penalize only if mob == 0
> else if (rank != RANK_1 && !(b & mobilityArea[Us] & file_bb(s) || b &
> mobilityArea[Us] & rank_bb(s)))
>
> But this is correct. It will penalize whenever the file mobility is 0 OR
> the rank mobility is 0
>
> You will see, the bench is different.
> else if (rank != RANK_1 &&
> (!(b & mobilityArea[Us] & file_bb(s)) || (!(b & mobilityArea[Us] &
> rank_bb(s))))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fbv-sf/Stockfish/commit/d8101b021b123a40793be49ad39b9f8257e93096#commitcomment-16461130>
> .
>


I'd like to make a few adjustments to this:

* Make the call to `struct_span_err` take the span of the duplicate def_id (the original value of `span` coming into this block), so it always points to the duplicate item.
* Make the `span_note!` different depending on whether `span_if_local` returns some or not:
  * If the original_def_id is *not* local, do what you are doing.
  * If the original_def_id *is* local, print "First defined here.", with the original span.

Add a second test case for the cross-crate case.

I think that will be clear. Does it make sense?


what is the preferred explicit name for "blind"?


@macool no es mejor hacer el required en el controlador?


very concise--looks good


:tada:


when installing cfitsio
./configure --prefix=/usr/local
make
sudo make install


is this just dead code?


oh, I see, it moved below


Test against oracle database with the command `npm run test-oracle`. You will have to have a running oracle db. See here for instructions on installing oracle.

Note the increased timeout. Running migrations on oracle is a lengthy process and tests will timeout and fail if we do not have a generous timeout (it could probably be much shorter than 150000 ms though).

Incidentally, I believe the sporadic timeout issue we have seen is due to the `beforeEach` hook and not the tests themselves, so I increased the timeout for the `test` command, as well. Since this is not an issue with the tests, but rather with circleci running the migrations, I think it's fine to have a longer timeout. I'm hoping we can finally resolve RILP-217 by increasing the timeout.


Is there are reason for having these gsettings in xfce, or is that a mistake? They are not even the actual background and gtk-theme of xfce! ;)


You're welcome. However, it has been a long time, so some docsting might not be correct. :laughing:


WOW!


wOWOW


s/proect/project/? probably fixed in some later commit


Seems all good for me. I'll improve the code and ping you back once done. Thanks for your clarification!


nit: Should be nuget v3


ROS_HOSTNAME should be set to your current machine hostname, not the target hostname


do we not want to visit all the fields? do we even HAVE a field named `substs`? WHAT IS GOING ON HERE?! :)


nit: nuget v3


This also outputs the x and y positions properly, I think.


Thanks, I just downloaded this to try out and it was driving me crazy!  I can confirm this solved the problem here as well.


Looks like babylon 6.6.4 has not been published to NPM yet?

![upload](https://s3.amazonaws.com/f.cl.ly/items/1o0i0l0S2e0L022L2w3z/Screen%20Shot%202016-03-02%20at%204.53.49%20PM.png?v=cb0ca0eb)

This is causing `npm install react-native` to fail currently.
https://www.npmjs.com/package/babylon


> But be ready to burn internet in some minutes... ;-)

Haha okay! I sense the hype man!


Nie możesz polegać na tym parametrze ponieważ nie jest on publiczny! Jeżeli Ci to działa, to tylko dlatego że jesteś zalogowany jako admin.


Is there a need for this? Is std::exception not enough?


@IggHub DWMR :+1: Ship code.


/me furiously refreshes browser


So, you're lucky that JavaScript figures out what's happening here. By not using `var` or setting up an empty tree object with `var tree = {}`, you're just making a global object all over again, but kind of worse because it's accidental. Just remedy the whole thing by doing `var tree = {}` on the first line of this function. Then you're good to go.


Add stingray to url


See [here](http://knexjs.org/#Builder-insert). This is to prevent `ORA-01008: not all variables bound` when inserting a knex model with not all fields populated. I'm not sure what happens to columns that should have a db trigger (like `created_at` and `updated_at`). We should double check that these are populated correctly, since oracle has given me problems with columns that should auto-populate.


typo? `2` should be `true`


Are you using a a proxy/mirror for npm?

Hopefully it's not just me... but I get
```js
npm view babylon dist-tags
{ latest: '6.6.4', old: '5.8.35', stable: '6.6.0' }
```

Check phabricator for issues with babel (https://phabricator.babeljs.io/T7181)


Let's define that 5 as a constant.


isn't that `https://img.shields.io/badge/devel-1.9.7-brightgreen.svg` a hardcode to always green?


Bus voltage works fine for me. Device crashes if I set it too low.


Let's define that 32 as a constant?


> /me furiously refreshes browser

Hahaha I got a refresh script constantly refreshing the tab


This line was removed during integration. It was a copy/paste error.


we executed a informal code review, confirmed that the application pulls default properties from this file if the user properties file does not exist. The is ready for pull.


OK, the bulk of this commit is the mechanic changes from view to log, and they look great. I just have some doubts about the 2 or 3 lines I commented before I ACK this commit.


Victoria University Melbourne Austria Harvard 2002 style


@jabonzhang 文件后缀请用小写


looks good... I don't feel like looking too thoroughly because we are building rough drafts id say at this point. we can refine it later.


I will remove these when they are built...


lol. The first original joke with one of my names in years. Fair enough. I deserved it.


@jabonzhang

文件夹名拼写请跟你的用户名大小写相符。

和windows不一样，GitHub和Jekyll系统中大小写是不一样的。以后要注意，否则这样的bug以后会很难发现。


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1013 of 1013 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-29.44%25-green.svg)


Sure, no problem. Looking forward to it and will be here to test if needed. :)


Why is this H3 instead of H2 ? There is no H2 to contain these.


OK, the best I could manage is:

```
  --show STREAM[:LVL]   List of comma separated builtin logs, or logging
                        streams optionally followed by LEVEL (DEBUG,INFO,...).
                        Builtin streams are: "test": test output; "debug":
                        tracebacks and other debugging info; "app":
                        application output; "early": early logging of other
                        streams (very verbose); "remote": fabric/paramiko
                        debug; "all": all builtin streams; "none": disable
                        console logging completely. By default: 'app'
```


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


I don't think this is necessary / desirable -- since the content is already in previous Docker image layers, it has to be downloaded anyway.


Why is `ha_pseudo_resource` in use here?
There is a real thing to check which I would have thought negates the need for a lock file.

Doing it this way prevents the use of a recurring monitor operation with role=Stopped which would seem useful in this case.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-MacOS-10.9-clang: OK (2121 of 2121 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)


Instead of overloading the single method to accept two different types of parametes, I'd name them differently (i.e add an "_int" in there somewhere).

Do we need to multiply the cmd.content.digicam_configure.exposure_type by 1.0e7f?  I think we need to do that for messages in which the x,y fields hold lat, lon values but not for the camera messages.  Doing it the way it's done here will, I think, mean that the digicam messages field values (shown in the mission command list) will be 1million times bigger if "int" style messages are used vs "non-int".  Also I think if the user changes the format of the message then an existing mission would need to be updated to reduce the values in those fields.  I haven't tested this but this is my guess from looking at the code.

the memcpy assumes the mavlink_mission_item_int_t and mavlink_mission_item_t structures are lined up exactly the same.  I wonder if that's valid on all platforms.


I mean we can use the RawTextHelpFormatter, but that's an overkill and we'd have to manually wrap all help texts...


Please fix the indentation


the pl.containers is already initiated in NewPodList()


Hi i9hdkill.

No I'm currently starting from scratch finding the cause of that error


Too many unrelated changed line. Please commit only the neccessary modified line


Taken almost directly from the README at https://github.com/gaearon/react-transform-catch-errors


then slowing build it back up to this... or better


** Please remove this file contents to dir: 3rd_party/apache-log4cxx-cmake/CMakeLists.txt
** Add necessary build command to generate libraries(.so, .a), refer to [https://github.com/smartdevicelink/sdl_core/blob/master/src/3rd_party/apache-log4cxx-cmake/CMakeLists.txt](url), for how to build you can reference this open source package official website.
** The Copyright now should be 2016


@renemilk, this has to be kept in order to ensure that the keys of ParameterType are ordered alphabetically, which in turn is needed for Parameter.from_parameter_type. (As I said, the whole implementation should probably be replaced by something more portable.)


A best practice is to not bind in render. This creates a new reference and triggers renders of children.


this sample code can be reused every time for console input.


NewPodList() has initiated the pl.pods, it is not possible to be nil


Can be implemented in fewer operations with an overflow check:

```ocaml
let add
    ({lo = this_low_; hi = this_high_} : t)
    ({lo = other_low_; hi = other_high_} : t) =
  let low = logand (add this_low_ other_low_) 0xFFFFFFFFn in
  let overflow = if other_low_ < 1n then sub Nativeint.min_int other_low_ < this_low_ else sub Nativeint.max_int other_low_ < this_low_ in
  let overflow_bit = if overflow then 1n else 0n in
  let high = logand (add overflow_bit (add this_high_ other_high_)) 0xFFFFFFFFn in
  { lo = low; hi = high }
```

Overflow check from here: https://www.fefe.de/intof.html


Thanks. You're the first person who's actually apologized for this (wrongly pulling me into a conversation like this), and I appreciate it. Thank you.


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


Usually you want to keep Dom event handling local to the component then pass the value to the prop function.


Hi Jake,

Thanks for the comment. I will correct this.

Also, your comments on Canvas mentioned that you were seeing linting errors
in the code. Can you elaborate on what errors you are seeing? I have flake8
installed for Atom, and it shows clean. It was throwing errors while we
were working this afternoon, and we addressed, so it appeared to be
working. I just want to make sure there isn't something I'm missing in my
setup.

Thanks,
Kevin

On Tue, Mar 1, 2016 at 5:19 PM, Jake Anderson <notifications@github.com>
wrote:

> The pythonic way to write this is actually with an else statement. You
> don't need to define the n > 1. If the first two haven't been triggered,
> the only other option is that n is greater than 1.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ksgifford/math-series/commit/d0f37a61b9784e0101f009e569096744d7753688#commitcomment-16434494>
> .
>


Apenas mudando o moverate das criaturas pra bater com a nova formula.
Algumas criaturas selvagens ja foram totalmente refeitas, em termos de status e etc.
Outras criaturas fora do grupo "selvagens" tiveram pequenas mudanças, mas nada radical. Ainda vao ser revistas.


Extra items in the top header，I can't find corresponding parameter to log from below.


For forms I typically use class since you need to wire up event handlers. Function components work best for rendering without interaction handling.


但实际上手札诱发的特殊性只是处理方式，形式上只要是在手牌发动的诱发效果都是“手札诱发”
完全不必在脚本做特殊标记


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2159 of 2159 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)


This solves #81


Abaixando o tag0.moddam, porque o dano de algumas criaturas ficou absurdo demais. Com 100 fica mais equilibrado.
Adicionando o cont.stam = <cont.dex>, pra evitar danos por stamina excessiva.


You don't seem to need to externally control value.


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


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


好的刘教授，我知道了，以后我会注意的


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


todos binarios :/


Anvils, furnaces, enchantment tables, and trapped chests?


Agreed.  Commit 63b3e4e840e363e0d6dee82fa11910f990990680 switches to that more compact expression using "count".


testing


are you interested in creating a PR? I will create a new snapshot of the compiler so it could compile this code


Javascript files


change to old


Arron Helped me finding the block of code.


remove line Game.permBoard, now in updateBoard()


this is a test for learnning git


Seems a javadoc is missing here which causes checkstyle problem


change to old since takeATurn() is being overridden now


I would recommend counting the lines in the file and using that number to create your array


may be we just need ngx_palloc here?


change to old, remove if statements since handled in Game now


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


my first pass did only send back _int, but accepted both as input. im happy to change back to only responding with _int


change to old, easier to see


much love


LGTM for the test changes.


add updateBoard() from old


Did this (`file >> *config)Array[i]`) work? I used `std::getline(<filename>, <variable>)`. I am not saying that my way is any better, just checking to see if it worked for you.


Are you outputting the data to verify that the array is populated correctly?


Hello, May I ask you a question about the percpu variable (exec_actions_level)?

We assume that the "this_cpu_inc" run in the thread A, and the thread A run on the cpu core 0.

After doing "do_execute_actions", the thread A has been scheduled to another cpu core 1, and "this_cpu_dec" will cause the cpu core1's copy of exec_actions_level plus one. Will this situation happen?


congrats on your first commit, Roger.


我意思是, 如果不使用 setTimeout, 有什么问题吗?


I don't think you need to manually run this, isn't it the default?


This is a clever way to manipulate process.argv when testing!

Now that you have learned a better way, be sure to manipulate things like process.argv using a beforeEach block.


As of now the default time of visibility for my push message on chrome
browser is 19 seconds which is too low, But Other push services like
OneSignal, Pushcrew, Let Reach are having visibility time of upto 3 minutes
for there messages. How can i do that, Need some help with this please

On Tue, Mar 1, 2016 at 9:41 PM, Brendan Ritchie <notifications@github.com>
wrote:

> I don't think you can control this. However, the service worker and
> notification specifications are still under development, and this might be
> possible eventually. See this stack overflow answer
> <http://stackoverflow.com/a/35724152/2993617> for a suggestion about what
> you might be able to do eventually. Or they might allow you to set a
> duration <https://github.com/whatwg/notifications/issues/11>.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bhritchie/web-push/commit/673011111c126a491b825635bd0c4eeb94c9ad74#commitcomment-16421998>
> .
>


If any of the players is a Druid, the choose one cards take multiple ID slots at the start of the game. Not sure if this changes anything for you, but wanted to mention it just in case.


I thought I put this in the base?


连续点击无效果。


Doesn't this fit React's unidirectional data-flow though? When I type anything in the input field, the value isn't changed unless I call setState above (part of the [controlled forms](https://facebook.github.io/react/docs/forms.html#controlled-components)).

Another reason I have it controlled is for form submission. When I submit the form I can quickly save the state. Otherwise I have to do weird things with uncontrolled forms.


Do you think we lose clarity by cutting out the suffix array name?


The path should come from the config_file_path parameter.


Ok for copy then delete, it will not loose messages upon errors. :+1:


You could have used i++ here too.


I agree. But in this case it's so simple and removes a couple lines. I think if I were doing validation or other checks on the form I'd definitely put event handling in. Good tip.


FILE PATH


Ack. Good to know!


I wonder if we can canonicalize image names such that each candidate has an image, and for every candidate, their image is named candidate_`{id}`.jpg, where `{id}` stands for their candidate ID in the database. That way we would not have to keep track of a separate list of candidate images.


http://www.sciencedirect.com/science/article/pii/S0167876096000530
This is a research article that’s mostly interested in the effect certain mental tasks have on (specifically) delta bands. The first experiment they did should be more applicable to us; the participants were shown a hard math question (in essence, what Paul suggested), and were told to solve it, then say the answer out loud. The type of questions the study used were like “(85/5)6”. This type of internal mental task was found to have increased delta activity. The study used a lot more electrodes, but they did include electrodes at location Fp1 and Fp2… Which is what I assume Muse measures…. (I am sorry if I’m wrong, everything I’m writing in this paragraph is based on this assumption). In the discussion, the researchers also mentioned in their discussion that activities that require “a state of internal concentration” also lead to a delta increase. This led me to think infer that we could use also meditation, which is the main purpose of Muse, no?
TLDR: hard mental math, yoga, meditation?

http://citeseerx.ist.psu.edu/viewdoc/download?doi=10.1.1.335.5231&rep=rep1&type=pdf
This is also a research article that’s not super applicable to us, since it deals with differentiating and comparing the left and right frontal cortices between pleasant/unpleasant and intense/calm stimuli. Also, the placements of the electrodes were not in the Fp1 and Fp2 locations; they were placed in the F3 and F4 regions. I just thought the article was interesting because of a point it brings up: different types of music induce “different affective states.” The study looks at alpha bands, and has 4 different classifications for music (intense-unpleasant [fear], intense-pleasant [joy], calm-unpleasant [sad], calm-pleasant [happy]). Since I have no clue what would happen if we did try this—or if we can even get some kind of measureable response from the Muse—I thought it would be cool to try it (if you guys want).
TLDR; maybe try listening to different types of music (that elicits different states of mind, like sadness or something that makes you want to dance) and see if there’s any appreciable change in brain activity that can be measured by our Muses… ensure that the songs suit the Muse wearer’s taste


Cassandra can not perform this operation in one time as we would be trying to update primary key. Such updates are not supported...


Ok, get it...


Note that `defer resp.Body.Close()` should be called even if you don't need to handle the response body, or it may lead to possible memory leak and the underlying tcp connection cannot be reused.

reference:
http://engineering.rainchasers.com/golang/2015/03/03/memory-leak-missing-body-close.html
https://github.com/robtuley/report/commit/dd379801bbed03881d6cf714639a56101261ef59
http://stackoverflow.com/questions/23928983/defer-body-close-after-receiving-response


There's also one other article I have yet to fully summarize, would you still accept it if I were to do it tomorrow? But still before the club thing, of course. Or would you just rather me post the link? I don't know how applicable it is to our research yet


hi


I don't think you need to specify `[*.md] trim_trailing_whitespace = false` since `trim_trailing_whitespace = true` is only for `[*.{js,jsx}]`

what I generally write is a bit stricter:

    trim_trailing_whitespace = true

    [*.md]
    trim_trailing_whitespace = false

so that it removes trailing whitespaces in all non-markdown files


Not sure this is needed either.


How do I test that without mocking my cassandra ? How do I test that at this level of abstraction ?

I don't remember having ever seen a test like this in James source code...


Are you still having problems with this? it looks like it should populate the array with the lines of the file.


I think a docstring like this would be ideal:
```
"Is VALUE an instance of a datetime class like `java.util.Date` or `org.joda.time.DateTime`?"
```
It's a bit shorter and has the added bonus of explaining what temporal means.


Change onCreateOptionsMenu inside DetailFragment, Why this?


@Arthur2e5 Requesting your help reviewing here...


Thanks for your review. I just updated this application.

Here is the list of updates:

1. This repo is now public.
1. Add contact and bio.
1. More info about what have been implemented before submitting this proposal.
1. New timeline according to [gsoc-timeline](https://developers.google.com/open-source/gsoc/timeline).
1. Remove the plan to add RE2 C++ API in the timeline.
1. Keep the plan to try to remove `Rcpp` dependency. <br/>`re2r` may not remove `Rcpp` dependency. I just want to gain some experience in writing some raw R-C API code, and get some info about the lines of code reduced by Rcpp and the overhead of Rcpp in a package.


Addresses https://github.com/alexandertuna/MMFE8Readout/issues/1


(General comments on `CrowdsourcingSubmitContentComponent`):

I wonder if the message on [L171](https://github.com/aaronnech/CheckYourBias/commit/3a6a5d2ddc2105dc10989da70cdea2c9a0626d5d#diff-690182c4f50a826b57f9ee954e84be7eR171) can be put into `Constants` file. Same applies for L164


This isn't a utility; it does not belong in the utilities file.


`nil` can't be an instance of anything so the outer `and` here is superfluous


Remained from a previous implementation I didn't clean. Nice spotted....


Where else would I bind the function? If I don't bind it, the incorrect `this` value is used.


You need to build that URL here for consistency. See CloneRepoAnon for an example.


Replies


It is ! Do you mean annotating it as Thread safe ?


I think (I am not positive) that this should be `string *file_contents = new string[line_count]`. the type that you are returning and the method type have to match.


So when one attempts to run Thumb "code" one a ARM CPU that lacks it can can see undefined behaviour, crashes and/or alike, right ? In that case your plan to have only non-thumb assembly sounds like the better solution imho. It allows flexibility, a single code path in both loader and vendor library, not to mention only a single set of binaries to distribute.

> It is a configure-time option, at least for platforms where both of them are implemented

Are you sure about this ? Afaics --enable-tls is a completely different thing from dispatch_use_tls


Why remove the testcase to test the InvalidoperationException path? if index is out of bounds Current should throw exception.


A few solutions work... You can use a decoratoring function to bind, bind in the constructor or bind componentWillMount.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2127 of 2127 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)


Ok ignore, i see this in the next test :)


有問題，你在check一下吧


Go Version:

```
func InContainer(pid string) bool {
	pidNameSpaceFile := fmt.Sprintf("/proc/%v/ns/pid", pid)
	if pidNameSpace, err := os.Readlink(pidNameSpaceFile); err == nil {
		if initNameSpace, err := os.Readlink("/proc/1/ns/pid"); err == nil {
			return initNameSpace != pidNameSpace
		}
	}
	return false
}
```

Tested on CentOS 7:
```
[root@ny-kbrandt02 ~]# ps aux | grep redis
polkitd  17780  0.0  0.1  37200  7712 ?        Ssl  03:36   0:00 redis-server *:6379
root     18289  0.0  0.1 142692  5668 pts/14   Sl   03:46   0:00 redis-server *:6379
root     18412  0.0  0.0 112640   960 pts/14   S+   03:50   0:00 grep --color=auto redis
[root@ny-kbrandt02 ~]# ~kbrandt/container 17780
true
[root@ny-kbrandt02 ~]# ~kbrandt/container 18289
false
[root@ny-kbrandt02 ~]# ~kbrandt/container 1
false
[root@ny-kbrandt02 ~]# ~kbrandt/container 30469 #rsyslogd
false
```


How about "Build your own type of documentation with custom plug-in"?


For sure.
But IMO :
 - Other test uses mocks in this part of the code base
 - It documents the backend calls done by this command. I find it interesting. Espacially if some day we end-up trying to improve IMAP performances...
 - ImMemory implementation wouldn't allow me to check explicitly error.


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1571 of 1571 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.81%25-green.svg)


Not a big difference but you could probably just define this as a partial function:

```clojure
(partial m/filter-vals (complement nil?))
```


We don't need Agenda with anchors here, affix is generated by default.


Ok


There's a pretty good chance you don't need this anymore now that you're checking target.isContentEditable:
```
	var selection = window.getSelection();
 	if (selection.focusNode) {
 		if ( ( selection.focusNode.nodeType != 3 && selection.focusNode.isContentEditable ) ||
 			( selection.focusNode.nodeType == 3 && selection.focusNode.parentNode.isContentEditable ))
 			return true;
 	}
```

But it probably won't hurt to leave it in.


TIght version


Nice Changelog bro


If you omit the value tag it should work. Forms have several solutions and your way is perfectly fine.


just a reminder armand that this should have 0.00 as the second argument rather than ''


:trollface: thanks for the compliments. Was trying to implement automated releases poorly.


Hahah great commit message of me trying to escape out of the required commit message form. Finally gutted this old repository and moved my new portfolio here - now to make some time to finish it between work, school, meetups and much needed naps.


Btw, this is now available in Vorpal `v1.10.2`. :+1:


Remind me to talk about inline styles and the `font`, `b`, `i` elements.


I think it's called `History.markdown` – will this matching fail?


[advanced tutorial](advanced_support_hyperlink.md) describes how to support hyperlinks in custom plugin


Jarb, many thanks!!


What does "metadata" mean?


C#


Fixes #2


This is not an indentation problem but a break-line in the interface you use to have a look at the source code (here github)

Have a look to line numbers, you will notice that their is no line number before "mailboxName) {"

Do you want me to split this line in two line to make it easier to read on github ?


"You can directly add `Microsoft.DocAsCode.Plugins` project reference to your project if you are building...


please where is the class named LinearEquation which used in problem 12 of chapter 9 .
or what do you mean about import ToolKit.LinearEquation in problem


with the same version as DocFX?


Option1
Option2


`app_logger.level = logging.DEBUG` probably yes, the rest is related to this patch. Let me move it there...


I don't get it... :disappointed:


<space>


Judging by the comment, should this be `is.null` not `!is.null`?


Refs issue #18


One downside of making DECIMAL a shorthand for DECIMAL(38) is that if we ever want to increase the maximum precision, we'd be stuck (or it'd be a lot harder).


Thanks, I mean it still works, so it's not essential to catch them all, but it's nice to do it so we  can benefit from the better error messages in case of failure.


Yay right :)
I probably will create a separate file for it when submitting it to the main repo, right now it's more like a piece of hacky code...


I'm not sure about this one, I'll double check...


run


Here are the rules for SQL Server: https://msdn.microsoft.com/en-us/library/ms190476.aspx


Refs #18


Fixed it in latest commit and updated hosted version. Should work now.


Git Cleanup


this fix is part of issue #1


cash


Don't abbreviate variable names


Nope, this is placed correctly.


rfun


Will the +1 cause problems at the upper end? I.e., when doing `a * aRescale / b` below, could it cause `a * aRescale` to overflow for some values of `a`?


Maybe a better name here is AnonymousPullNotInfluencedByPush. I can't seem to understand how the current name relates to the test. Sorry.


This doesn't seem to be used anywhere


Формат валидации лучше выносить в отдельный валидатор. Может потребоваться где-нибудь ещё.


The following URLs in the commit message appeared to have nothing to do with the changes:
- https://www.illumos.org/issues/510
- illumos/illumos-gate@5ead3ed


Done.


I really think this method should have a check for Guid.Empty and either throw on it or call CheckAnoymousPermission. This is a very important function and thus should realllllly avoid surprises (fail silently) if passed a Guid.Empty by mistake.

Another option would be to make this private or protected and only allow HasPermission to use this.


New version produces:

```
Failed to set log for --store-logging-stream asdf:Level FDSA: Unknown level: 'Level FDSA'.
JOB ID     : 18598f08e2bef4107120b2c3edb32fb46ee0e954
JOB LOG    : /home/medic/avocado/job-results/job-2016-03-02T05.15-18598f0/job.log
TESTS      : 1
 (1/1) passtest.py:PassTest.test: PASS
RESULTS    : PASS 1 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0
JOB HTML   : /home/medic/avocado/job-results/job-2016-03-02T05.15-18598f0/html/results.html
TIME       : 0.00 s
```


На самом деле password_confirmation никакого отношения к бизнес-логике модели не имеет.


Lines 97-104 should be deleted.


Missing ')' at the end


@DotHide 但我实际测下来看 ion-item 最后生成的

```html
<div class="item" href="#/topics/123>
  <a class="item-content">
  </a>
</div>
```

这样的结构无法实现 a:visited 样式，因为 `.item-content` 这个没有 href 属性


User DB 뿐만 아니라 Photo, Nickname, NickRecom DB 등 에서도 삭제해야해


maybe JSX?


Well, that was just bogus given that we catch the exception below and just return null.


@tin315 Is this something that needs to be updated at every API version bump? If so we should replace the string with @@CONVERSION_API_VERSION


LGTM. Is it based on upstream POT btw"


```onAnimationComplete = () => {```  and don't need ```.bind(this)```


It is. Straight from their Git tree.


Test


Kind of weird that they did not include a .pot in their tree.


This commit says "do not merge", was it left in purpose to allow developers to test?


Then JSX.Element


Not keeping this pull commit message as it is in the same repository


Removing the pull request message from this commit ~


New react-motion-ui-pack branch doesn't have this handler anyway.


Thanks, yeah I was trying to get this going on Docker Cloud Beta. Forgot to fix this.


All good. Creating a PR.


Do you want me to write these tests in a separated test class in the Cassandra project ?


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


Hey, thanks for the feedback. You're right and that's why we're closing the body anyway using `defer`.
By the by, this commit is one month old and things change quickly. The response is handled correctly though, even today in the current code.


Typo—did you mean to subtensor rather than divide?
If you did.. We shouldn't be changing `buffer_top_t` at all, right?
We just want to slice the actual TreeLSTM values.


System.out.println("I love my Dog");


Yep, I was surprised to find that lol


sorry! tiny thing English here, "has" not "have"


113107773-Improve-waste-heat-function-handling-of-AirloopHVACUnitarySystem-#5222 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2127 of 2127 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)


can we change this to, "does not have rounded corners"


the reason for the *1.0e7 is because otherwise its not like for like.
we are force converting a float to an int. atleast with the *1.0e7 the number will be maintained. asuming its a smaller number. also with the *1.0e7 approch it makes it easier, as there is no if cases regarding the value it may contain.


一般认为版本号 3.9.4b1 < 3.9.4 < 3.9.4.1b ，至于 3.9.4.b1 一般很少这么用


memcpy will line up, as they are packed structs. so long as the field definations dont change, it will be ok.


when can we expect this to land in the recent *-lts releases?  i'm hitting http://www.openldap.org/its/index.cgi/?findid=8336 which was fixed in that release.


Why slice `buffer_top_t`, which isn't a function of the TreeLSTM?


@NathanKell Yes I believe that was the case


While the older version may have had the wrong kind of endings, at least it *had* endings on all the lines.
The new version is missing the line ending on the final line, which is what that red symbol github is showing at the end of the green section means.


Why not different files for each tho?


I recompiled the pdf file and refreshed the master with that as well as your correction.  To update you might try git pull origin master. Or delete your branch and create a new one.  That will give you all the other readings as well.


グループ定義はlightbox[グループ名]を記述しました。


Looks like a typo from an earlier way of setting up the h_dim division. Eek.


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


Added feature to .css to allow css selectors by looking for ":" in the property, which is nifty for css selectors that can't be set via javascript


Er... no. The old state was a typo, but the buffer should be sliced. It *does* have a component that gets treated as a `c` state by the LSTM, and that component should be protected as such.


Change this to note that some work is licensed under CC non-commercial 2.0 and that a list is available at(blank) then create a file with the link to the licence and the paths of the file and their attributions.


Seems strange that the embeddings (or transformed versions of them) carry a `c` state.
But I won't object.


The phrasing seems confusing to me. Or like it'd be confusing on first read. 'Up and out to the caller', or 'Errors in the function body get propagated up and out to the caller'.


Please update `utils/line-directive` to match.


Thanks will try now.


I'm tired at the moment, so this is not a lucid response to the matter at hand. I'm writing to say that it is difficult to convey Cadence, so I'm open to hearing that there is a better way to think of things. Maybe we talk about it for a bit? Maybe we need to step back and work out an outline of points?


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#QnMbtix9RIqgXs-2Ds9wyw


Change body to bod


@neetVeritas <https://github.com/neetVeritas>  Sup, Laveer how you been? I
hope  you're doing  great 😀. Why did you remove the Finis post?

On Fri, Feb 26, 2016 at 5:35 PM, John Nolette <notifications@github.com>
wrote:

> @duke12468 <https://github.com/duke12468> Finis is in the works, and will
> be available as soon as possible. I do work 40 hours a week, and I'm a full
> time student -- so I can't fully dedicate myself to the platform, but I am
> making considerable progress.
>
> On another note, if anyone would like to live chat, I opened a discord
> server for Finis @
> https://discordapp.com/channels/152905170009718785/152905170009718785.
> I'll report updates as I develop.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/neetVeritas/pye-module/commit/0d64c4778e89a6519d6afd172827fa6c607e9ddf#commitcomment-16365590>
> .
>


These methods would read better if they were named "isXXX" (isResultEqual, isResultNotEqual, etc.)


This closes issue #11


``` onMenuItemClick , onClick, handleClickOutside ``` to?


Jjjj


@particlebanana thanks, meant to take care of this


solving #1 issue


Why `comparableWithVariadicBound`?


Would it be possible to use CSS to display the ellipsis? Take a look at the (text-overflow)[http://quirksmode.org/css/user-interface/textoverflow.html] property for inspiration.


(x is num_files % 20, so, modulo by zero exception here)


IMO installerIso should not be in nixpkgs at all, it should be in NixOS, near [iso_graphical](https://github.com/NixOS/nixpkgs/blob/master/nixos/release.nix#L133). Then you can disable it by using ``genAttrs []`` instead of ``forAllSystems``


doesn't the `Issue` object already have an `id` field? If so, then storing this as a separate state variable seems redundant.


Will do!


Render all time generates new components, then and ``` .bind ``` generates new function every time


How did you generate the keyword list? Is there a simple script we can run regularly to update the list?


this was being concatenated to `'Program Files/SteamSteamLibrary'` at compile time


@greeniceparadise The project has been forked and is now maintained at https://github.com/corejavascript/typeahead.js - you should update your dependencies to use the `corejavascript` project.


why _ syntax for identifiers here instead of camel case?


We should not be having the `Constants.CANDIDATES` object anymore, now that we can pull a list of candidates from the database.


[R/corrplot.R:310:12:](https://github.com/taiyun/corrplot/blob/333f115059704c683c77393270fcc4f60916d34f/R/corrplot.R#L310) *warning:* **no visible global function definition for ‘corrMatOrder’**
```r
    ord <- corrMatOrder(corr, order = order, hclust.method = hclust.method)
           ^~~~~~~~~~~~
```


Minified gzip'd bundle size: `153.13 kB` -> `82.84 kB`

```sh
> curl -s https://raw.githubusercontent.com/lukekarrys/insta.lukelov.es/77d523e03499692a4eb18fc946e9fb6909a26bd1/usi.b5427310394315ff3af6.js | gzip-size | pretty-bytes
153.13 kB
> curl -s https://raw.githubusercontent.com/lukekarrys/insta.lukelov.es/8d7f0cbbc84146142dbc4d82220225a2b9eb2e14/usi.13a978218bc29e3b7667.js | gzip-size | pretty-bytes
82.84 kB
```


Use `value`, `low` and `high` for the argument names.


One pattern i've seen with go tests, is to create a `testdata` folder along side the `_test.go` file to reference any files. Since you're moving the test.sh anyway - do you think that's a worthwhile pattern to follow?


好的，谢谢教授！


accidentally commented out?


psst... these need to get updated to reference version control/git instead of command line :-)


Can we use class ArrayList to store person information ?As ArrayList is dynamic, means it grow larger as we add data, no need to set a size of array.

here is line that I looked at http://stackoverflow.com/questions/3217989/how-to-store-information-about-many-people-in-java.


Git Bash - two words, GitHub - one word


GitHub - one word


commit to branch!


Nice edit!


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


After invoking on logging-configuration-rest its asking for dependency logging-configuration-core


We could also split this into categories so you'd do `Icons.action.accountBalance` or something. Not sure that's better though. It's not clear the categories matter for authoring (as opposed to for the presentation, where they are helpful).


Minecraft-styled running works by default successfully! The alias is created for new players and can be done just like in the aforementioned game, by pushing W twice!


(I'm fine with the _ syntax.)


This was a great idea, thanks!


We never handle ``malloc`` failure, so no need to do it here.


Instead of a for loop, can't you test for the presence of `issue.candidateRatings`?

E.g. `if (issue.candidateRatings !== undefined)`


2055_CommasInNames (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


LGTM


scheduleId_listがnullだった場合、scheduleId_list.size()でException吐いてしまわないですかね・・・。
Javaの場合左から評価されていくので、

`scheduleId_list != null && scheduleId_list.size() > 0`

の順のほうがいいかと思います。


test looks good


who cares


A basic weather app using UWP and the openweathermap currentweather api


Where does the 17 come from? The largest power of ten that can fit in a `long` is 10^18.


2055_CommasInNames (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


EipTScheduleMapのUSER_IDには設備のIDが入るケースもあるので、TYPEで絞り込んでおいたほうが確実かも？


変数名は可読性の高いものにしましょう。


Thanks for the update!
The official LastFM API docs list "Location" as one of the optional Params. I cannot seem to change the query results when adding it as a parameter.  For example:
```
lfm.geo.getTopTracks({
  country: 'Canada',
  limit: 3,
  location: 'Vancouver'
}, function (err, tracks) {
  if (err) {
    return console.log('OMG!', err);
  }
  console.log(tracks.track);
});
```
Would this be related to the issue I receive when running the getMetro method?  The following returns an error:

> error: 3,  message: 'Invalid Method - No method with that name in this package'

```
lfm.geo.getMetros(['Canada'], function(err, metros){
    if(err){
        return console.log('Get metros ', err)
    }
    console.log(metros)
})
```

Thank you for clarification.


I think we also need to write function for use case 5 that is start program and open a existing data file on disk


Yeah. It would probably make sense to make the buffer half-size and
initialize the TreeLSTM c states to zero when one of the children is a
word. But we made some decision early on to keep the buffer and the stack
the same size to make things simple with non-TreeLSTM TreeRNNs. And it's
perfectly well defined, just probably not necessary.

On Tue, Mar 1, 2016 at 8:44 PM Jon Gauthier <notifications@github.com>
wrote:

> Seems strange that the embeddings (or transformed versions of them) carry
> a c state.
> But I won't object.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/hans/rembed/commit/8f04d2666d36f3eac0cadc6a838bb90e8cbbc839#commitcomment-16436693>
> .
>


What seems less well defined is to have a backprop path into the initial c
state of a TreeLSTM that doesn't go through the TreeLSTM, which is what I
was trying to get rid of.

On Tue, Mar 1, 2016 at 9:15 PM Sam Bowman <sbowman@stanford.edu> wrote:

> Yeah. It would probably make sense to make the buffer half-size and
> initialize the TreeLSTM c states to zero when one of the children is a
> word. But we made some decision early on to keep the buffer and the stack
> the same size to make things simple with non-TreeLSTM TreeRNNs. And it's
> perfectly well defined, just probably not necessary.
>
> On Tue, Mar 1, 2016 at 8:44 PM Jon Gauthier <notifications@github.com>
> wrote:
>
>> Seems strange that the embeddings (or transformed versions of them) carry
>> a c state.
>> But I won't object.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/hans/rembed/commit/8f04d2666d36f3eac0cadc6a838bb90e8cbbc839#commitcomment-16436693>
>> .
>>
>


UYUI


Переводы выносить в locales


I just figured I'd put everything out there, but it shouldn't happen

> On Mar 1, 2016, at 5:04 PM, Neo Aero <notifications@github.com> wrote:
>
> Hey Matt, why did you add a red-rimmed fog tile? Shouldn't that never happen?
>
> —
> Reply to this email directly or view it on GitHub.
>


```C
for (int count  = 0; count < MAX_MSGS_IN_A_ROW; ++count)
```

We used to drain all the messages, but now we are only going to drain 5 messages every ``proxy.config.process_manager.timeout`` seconds (default is 5). That seems a lot less responsive.


This needs to be a bit more specific. For example, the spec says:

> a) If the operator is not division and the mathematical result of the operation is not exactly representable with the precision and scale of the result data type, then an exception condition is raised: data exception — numeric value out of range.
> b) If the operator is division and the approximate mathematical result of the operation represented with the precision and scale of the result data type loses one or more leading significant digits after rounding or truncating if necessary, then an exception condition is raised: data exception — numeric value out of range. The choice of whether to round or truncate is implementation-defined.


Perfect. Best Tagmanager to work with.


Well, nice catch :+1:


@yungchin I see two options to deal with this, perhaps a combination of the two.

1. Create a class which takes what it needs as parameters. Create a subclass which provides defaults (either as class attributes or super) to provide sensible defaults.
2. Take a config object for SSL parameters, such that there is only one parameter to pass in.

The combination of these would then be a class which takes a single config object, with a subclass which provides a default one.


Можно воспользоваться гемом simple_form. Он позволяет и ошибки, и labels выводить как надо и интеграцию с  i18n имеет


ログ出力は

`logger.error`

の方を使用しましょう。loggerはaipo.logにログが出力されますが、System.out.printlnの方はcatalina.outにログが出力されます。


Yeah, you're right about not needing that.

I only set it to js and jsx as I wasn't sure who was using it, I was just annoyed that it wasn't trimming my whitespace properly in the react component files.

Feel free to submit a PR with changes if you want to tidy it up / make it better


Oh heck yeah that would be great. I don't mind doing it by hand but
anything that can make it easier helps.

On Tue, Mar 1, 2016 at 11:26 AM, brine <notifications@github.com> wrote:

> hey, if its easier to input the card data into a spreadsheet, I can hook
> you up with my set xml compiler. When I typed card sets out by hand it as
> always easier putting them into a spreadsheet then doing it manually into
> the set xmls.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GeneralCake/FoWTCGOCTGN/commit/c500d98bffeb70dd48bbc54d1afaa2539577caef#commitcomment-16427075>
> .
>


@pombredanne – the unicode prefix is a Python 2 idiom; we've used the `@python_2_unicode_compatible` decorator so that this shouldn't be necessary.


It is not that important. But it look perfect if we write email.id first followed by phone no according to the sequence  given in contact list.


```
for (EipTTimeline e : list) {
}
```
内でリストの中身、要素数を上書きしてしまう処理は行わないほうがいいと思うのですが、大丈夫そうですかね・・・。


put вероятей всего


LGTM


Do we use tokenisation anywhere else? Or just in compute_fingerprint?
I'm not sure I see this as utility if we only use it in one place.


Why sqlOption here?


@pombredanne – is there a reason to prefer Python's identity checking, which means the doctest is less readable for people using `help()`? String identity should suffice (possibly with the addition of `+NORMALIZE_WHITESPACE`).


Release location has changed, but still no MSIs (just zipped exe files).  Is the plan to have installers?  If so, let's hope portable versions are still available.


Is the testing complete?


If add function 5 than need to call here


This is so, so much cleaner now :+1:


Do you


MIND


Going to use it to replace the CorpusReader in chunk.


行啊小伙子，速度还挺快。


I don't think plugins should be aware of ShortDecimalType vs LongDecimalType, as they are implementation details of DecimalType in the core (e.g., it makes this code sensitive to changes in what precision can be encoded by each implementation type).

Instead, I'd change this check to look at precision/scale of the DecimalType and making a decision to encode as long or as slice based on what Hive's encoding supports.


Will be useful later on for a set of opinionated defaults.


The list come from here:
https://github.com/google/material-design-icons/blob/master/iconfont/codepoints

We'll need to update it whenever we pull a new version of the ttf file.


離開場上進入手牌能觸發"此卡離開場上的誘發效果"
規則上也是在手牌發動效果
但這和手牌誘發有明確的不同


valid_physics is a constant, you should replace name to VALID_PHYSICS from valid_physics.

https://www.python.org/dev/peps/pep-0008/#constants
https://github.com/travnick/CryBlend/wiki/Code-rules


It appears to be the canonical way to name the icons.  Both the PNGs and the fonts use these names.


I really want the testing to be complete ;(


I don't think ShortDecimalType and LongDecimalType should be in the SPI. They are implementation details of DecimalType, and the distinction only exists for performance reasons. See my comment on https://github.com/Teradata/presto/commit/74f60676451605ec8923007963943d2dda604afe#commitcomment-16437020 for a related point.


Edit Review


develop (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


crashed on Windows without
if (Meteor.isServer) {
  Meteor.startup(function () {
    // code to run on server at startup
  });
}


Thank you. Never used python, I'm just learning.


@particlebanana -- It would be helpful to add the spacing rules around parentheses to the `.editorconfig`, but I'm not sure if it supports them.  We could also try the .jshintrc file -- It's the same as what's used in Sails core and a few other packages.  I updated it recently to include reasoning for each style choice and to be 100% sure all of the rules are still supported by jshint, so I'm confident it's up to date.  As for the .eslint file itself, I'm down to start using that directly during dev if we can find a plugin for Sublime Text 2.  I would want to make sure that everything in the jshintrc is covered as well (it looks like that's mostly the case already).  Re: the particular eslint rules here, I included some feedback below:

### Stuff I'm not sure about, but would be willing to try out:
+ re: [eliminating whitespace in conditionals](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-af2907760aec3c2e03f2c610ed013666L71)  (The funny thing here is that I've actually been making a conscious effort as of late to do the _opposite_-- that is, being sure and always include whitespace on the inside of either parentheses. I find that doing so makes it easier (a) to tell which closing paren goes with which opening one, (b) to detect "not" (`!`) symbols that might be missed if they were smashed up against parens and (c) .  That said, I don't think it's important that we all commit to going one way or the other for this particular style guideline.  Every conditional is different, and I can't confidently say one way is better than the other in every case yet.  But I know for sure that it would be annoying to go through all the work of purposely spacing out conditionals for better readability, but then being forced to change it by our build tools.  That said, if this is something you're really enjoying having be consistent, or if you're passionate about it, I'd be wiling to try it out)
+ re: [requiring newlines in dictionary literals](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-4fae11905e79b9064d18af6ddae9e252L61) and [preventing single-line blocks](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-77ea095c6f89aba19fcc340babf61685L177)   (this is one of those things that can vary dramatically based on context.  Here are a few examples: For dictionary literals with one or two keys, it can be much easier to read the code when they are expressed on one line.  When there is a good comment on the line above, it is often helpful to tuck away a short, unremarkable block of code that e.g. handles an error or does an unremarkable lookup onto a single line. This is so that you don't have to actually read the code-- you can just read the comment.  I've found that the benefit to the apparent simplicity of the containing function from keeping these blocks single line fare outweighs the risk of inconsistencies.  If this is something that's bitten you in the past when working with my code, I'm down to talk about it more to understand the context)

### Stuff I'm not cool with:
+ re: [preventing trailing commas](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-f21382d30ad2f3500dcdf8329777cb75L25) (over the years I've keeping trailing commas around makes it easier to edit code; i.e. swapping the order of array items / dictionary keys, duplicating lines, etc.)
+ re: [restricting the format of comments](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-af2907760aec3c2e03f2c610ed013666L23) (it is sometimes helpful/important to be able to use inline comments, and I don't think it's been a huge code consistency issue for us in the past.  But most importantly, inline comments on closing braces to keep track of the end of blocks has saved me a _huge_ amount of time in the last 12-18 months or so that I've used them.  I don't always do it, but when I do, I'm always glad they're there.  Here's [an example of what I mean](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-af2907760aec3c2e03f2c610ed013666L103))

### Stuff I like:
+ re: [requiring whitespace between function signatures and first curly brace](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-9ea3dc5522134c3b1302184660afedecL72)   (I think this enhances readability, and it's something I always forget to do)
+  re: [requiring whitespace after switch statements](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-af2907760aec3c2e03f2c610ed013666L78)  (same reason as above)
+ re: [forcing catch to appear next to the closing curly brace of `try`](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-4fae11905e79b9064d18af6ddae9e252L68)   (I wouldn't have thought of this, but it's a great idea-- at least as something to try out.  I like it because it looks similar to how asynchronous error handling looks-- i.e. callback function)
+ re: giving every function a name  (I'm normally all about this, but I'm not sure I feel comfortable enforcing it strictly.  For example, [here](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-77ea095c6f89aba19fcc340babf61685L195) it is good-- but [here](https://github.com/particlebanana/machinepack-postgresql/commit/fa01d8e9c19348c33d51bc5c8c696f8a6cb58475#diff-c67db75971673cb99b0083fd2676390dL57) it doesn't really do a whole lot of good, and kind of just creates stuff that has to be updated by hand.  But, after some careful thought, I think the good might outweigh the bad here.  I do wish there was a way to _not_ require names for functions which are the RHS properties associated with a key in a dictionary.  If we can think of a way to do that, I can get behind this 100%)


@valib did you save your attempt at tests? Maybe I can have a look, it might be faster than me writing them from nothing


just curious - why does 'yes |' need to be here?


Hum, definitely something we should fix soon, it's rather weird to have no output when connecting to a forbidden server...


@jdurgin If you were pointing to these lines, I was unsure of the third parameter to append_op_event, and if i was doing it right. So, I want to confirm and then work on it.


develop (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 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-NA-red.svg)


i believe the other docker images just use openjdk - any reason why you're using oracle java here?


@particlebanana This was done this way on purpose-- the error is handled below (this just attempts to release the connection to the pool first in case it was somehow connected).  If that fails, it doesn't mean anything.

We could also just remove that line of code, but since I didn't want to spend the time to get to the bottom of it, I wanted to go the safe route.


Unrelated changes?


Hi sid5491,

Please see our documentation here: https://documentation.onesignal.com/docs/customizing-features#section-how-can-i-automatically-dismiss-a-notification-after-some-time-. The notification should last indefinitely on Chrome by default, and on Safari and Firefox they are dismissed automatically.


Typo? "extenderprice"


@particlebanana We can address this differently (optional notifier function provided as an input), but we do need some way to provide info about these events back to the client.  I'll put something together to try that out.


LGTM


Same as above


start


`cst` likely stands for CMU Speech Technology (or something similar) -- it is used as a prefix throughout flite.


Test


It's no longer there in the current package so I don't see the relevance of this comment ?


Questo dovrebbe essere lo spazio per i commenti prima di approvare, se non sbaglio...


Call this "expression"


What does it mean for the expression to be `null`?


nice catch


are these paths right? seems weird that /usr/local/zookeeper was changed to /path/to/zookeeper?


formatting: put all arguments on the same line or each argument on a separate line.


Don't import Intellij's annotation.


Maybe we can change the properties namespace to `assertj.swagger. `?
The current one is the namespace of the swagger-parser project.


@yucaroll 관심있는 장소가 리스트에 쭉 뜨면, 사람들은 그 리스트중에 무의식적으로 해당 장소에대한 정보를 보기위해 그 아이템을 누를꺼에요.

그러면 그 유저의 관심있는 장소들을 나타내는 리스트뷰가 다른 ReviewsListFragment로 바뀌면서 해당 리스트뷰의 영역에 유저가 클릭한 장소에대한 Detail 정보가 나타나는 것이지요.

그 Detail Info 페이지에 해당 장소에대한 리뷰들 역시 나온다는 의미입니다.


I thought this got merged into master already.


@prakashsinhab Please review my file


And please consider the patch "pre approved" assuming it passes your local tests, thanks @dduan!


Here's what happened:
1. I write the initial pull
2. I write the little follow up that contain those lines in the web editor
3. Later, back in my local repo I branch off my local copy of feline_magnet into feline_magnet_2
4. That pull fails due to code I wrote in the web editor that I forgot to push back into my local.

In short, whoops


Лучше разделить получение на получение task сначала, потом на поиск attachment и destroy отдельно.
Такая запись создаёт неудобства при дебаге и усложняет понимание кода.


.... added
reviewed


the id of username input and password input changed, in last runs i wasnt able to reproduce this change of id


How about changing this to `a.properties |= is_zero ? DMTCP_ZERO_PAGE : 0;`? I'm assuming that the `properties` field is initialized to 0 initially.


Why fully-qualified class name? Is there another one by the same name? If so, we should fix the conflict, as it will only cause confusion.


Travis CI build fails at assignment_invitations_controller_spec.rb line 51. Maybe we can figure this out? @hackerkid


Nice！


Same as above: change this to: `area->properties |= DMTCP_SKIP_WRITING_TEXT_AREA;`.


OK. I'll try to add unit test to clarify you this problem.


可以用  sails 的 logger 機制


Call this "expression"


不用  res.serverError(e); 的考量是？


Why "LongVariableConstraint" instead of just "Constraint"?


@eoghanmcinerney Note the change I made to client/modules/people/who.import.less


@djalova Theoretically you are correct. But if you try this you would see that it's not working.
The reason is that Swiftauth and SoftLayer Object stores all expect X-Auth-User header to be tenant:username, while Joss creates it opposite, user:tenant. Therefore there is need to provide username as a tenant to Joss and tenant as a user. I don't know why Joss creates username:tenant and not tenant:username


test for duplicates


"SING"... typo?


El 01/03/2016 a les 22:38, Dominique Pellé ha escrit:
> Typo in comment: connextions → connections
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Softcatala/LanguageToolAndroidService/commit/ceafcecc13698a029d0340fcefd2f8b1c383403e#commitcomment-16430333>.

Fixed, thanks

Jordi


service 不要直接傳入 req，請在  controller 整理好相關資料後傳入


I'm having a problem using the BITFIELD command with "ordinal" offset.

For example I set the second field (#1) of type u5 to the number 5

```
> BITFIELD k SET u5 #1 5
1) (integer) 0

```
But if I do the get of that position I get a different number:

```
> BITFIELD k GET u5 #1
1) (integer) 4
```

If I set the value to a different value, I correctly get the previous 5 value:
```
> BITFIELD k SET u5 #1 6
1) (integer) 5
```


This commit should be squashed with the one that introduced the compareTo method, as it was buggy from the start.


I'm pretty sure the following line should restore buildability the old way:

`sed -i 's/+v6/+v7/;s/-march=armv6 -marm/-march=armv7-a/' mk/cfg/arm-unknown-linux-gnueabihf.mk`


These local variables are unnecessary


I'd rename the argument to just "value"


i d like to merge those very closly related loginmethods into one file but i was unsure about interfering in your "codebase".


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


bucket size can be O(n) here.
You can optimize this by changing the bucket_size and redo all calculations when size is too big.


{
success: false,
message: 'some error'
}


not working for me..


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


You'd get the journal_tid from m_journal->allocate_op_tid(), and use the Context ctx (that's the callback) created on line 151 as the third argument to append_op_event(). That way when the journal is finished adding the event, the callback will end up calling the next step, handle_add_journal_event().

The EventEntry can be constructed in-line as the 2nd argument in the append_op_event() call so you don't need to std::move() it.


self.find will return, index, key, and value.
if value == none, which means no (key,value) stored,
or value is return, then it will be overwrite, based on index.see line 33


@vyasrao can you give us a working demo for MVC? pls. thanks


@andela-bmakinwa I have added a new line to make it more readable


烦烦烦


I'm not sure I understand what this is doing, but is this predicate equivalent to:

```java
if ((leftInitialByte & SIGN_MASK) != (rightInitialByte & SIGN_MASK))
```


I'd call these variables just `left` and `right`


@andela-bmakinwa This was actually done by `rubocop` but I've fixed it and other occurrences.


remove this line and add a new line


I have added new lines to all blocks that require them


Thanks!


`leftInitialByte` and `righInitialByte` are going to be negative if the decimal is negative and positive if it's positive, no?

If so, isn't it sufficient to do `Byte.compare(rightInitialByte, leftInitialByte)` here?


@173210 this will work fine if you included sources in your local project once and for all. In case of manual update libriary sources you're to skip conf file or manually merge if it have new features added. In case of Github subproject, Iike mbedTLS (the PolarSSL successor), it will be the only way!


commit


真是太感謝您了


Again, why is this here? Just delete this line of code, its meaningless given the next line.


Yeah that is a bad idea.
Quitting should be done via ctrl+d and not an empty entry.


This will not work because if no limit is passed, `limit` will be `nil`, calling `to_i` on `nil` will give `0`. Hence, `@limit` will be `0` instead of `20`.


this ExclusiveLockRelease event is similar to Flatten in that it has no additional data to encode/decode/dump, so rather than creating those methods you can inherit from OpEventBase, add a constructor that takes a uint64_t op_id, and rather than defining encode/decode/dump, refer to the base class versions like: https://github.com/ceph/ceph/blob/master/src/librbd/journal/Types.h#L251-L253


@jebbdomingo, you also don't need this line of code anymore.


why are you duping? select returns a new hash.


This commit should be squashed with the one that introduced the changes for DECIMAL to `isTypeOnlyCoercion`


Hi,

I followed the according thread in Helm.

This commit looks like you should better use

```
(message "%s" ...)
```

to avoid problems when the buffer contents contain format "%-sequences"?  Sorry if I misunderstand what this function does, but this looks like a common error.


@andela-bmakinwa This will not work because if no limit is passed, `limit` will be `nil`, calling `to_i` on `nil` will give `0`. Hence, `@limit` will be `0` instead of `20`.


the needs to be
acc_exp = note->m_drawingAccid->GetAccid();


這裡看來你只要傳入  socketid 跟 roomId 即可，不需要  req


this is a great start and it would be great if we can keep this ideas of possible test cases up to date.


此处会产生如 `#00520` 的错误颜色码


這邊也是


아아아 db가 한개가 아니구나... 알겠슴당


ここの修正、これでよろしかったでしょうか？


I'll merge it now.


これでよろしいでしょうか


> acationheroTestProject
is that a typo?


@andela-bmakinwa The same issue arises here as well. If no `page` is passed `@page `will be `0`.This is because `page` is nil hence, `page.to_i` will be `0`.


It has been removed accordingly


@SeanF47 Notice how you have the same properties for the .a rule all the way through the .h rule.

There is a programming principle called D.R.Y. - Don't Repeat Yourself - and it applies here :)  When you find yourself doing the same thing over and over, how can you refactor (change/revise) your code to simplify it?


All instances have been fixed


Why would that be desirable?

Von meinem iPad gesendet

> Am 01.03.2016 um 19:08 schrieb Marc Philipp <notifications@github.com>:
>
> I think we should use Collectors.toCollection(LinkedHashSet::new) to preserve the order of tags.
>
> —
> Reply to this email directly or view it on GitHub.
>


Currently, you may want to make the squares a bit bigger (60x60?) than the pieces themselves.


You are passing an array (`self.filteredContentList`) into a method that expects a `NSPredicate` object. If you already have a filtered list, why don't you just return that one?


Deleted 'My first repository' and added spacing


@andela-bmakinwa I have replaced all instances with `find_by` which I think is neater by the way.


Good catch. It appears to be some kind of mistake.


In my opinion it is quite important that predicate is rewritten two times. If I hide in constructor of a PlanNode, then reader could be confused once he find it,

Additionally having it extracted as variable make it easy to see value of first and second rewrite in debug in a single step.

So I would leave it as it is, unless I didn't convince you.


The method call and other related calls have been fixed


test


done


@andela-bmakinwa This issue has been taken care of, all such instances have been edited accordingly


https://github.com/aipocom/aipo/blob/6ca592c20200f04ef92a4e01c45f0e3dc5be7f7b/portlets/schedule/src/main/java/com/aimluck/eip/schedule/util/ScheduleUtils.java#L399-L448

あたりのルールと合わせる必要がある？


@gMarx you have two migrations that add the email column to users, which is why you're getting this:
`SQLite3::SQLException: duplicate column name: email: ALTER TABLE "users" ADD "email" varchar DEFAULT '' NOT NULL/Users/swhopkins/.rvm/gems/ruby-2.0.0-p247/gems/sqlite3-1.3.11/lib/sqlite3/database.rb:91:in `initialize'`

Your `AddDeviseToUsers` is adding it after the previous migration already added the column


Ok, I see your point. I don't like the name of the variable with the "2" at the end, since it doesn't tell me anything about what the expression represents.


What WMS


Use uppercase "NULL" to be consistent with other place


Ini thread safe ngga ya..?


again


Make all these "try keyword" say "try function" instead.


Heh, we should fix this alignment...


Add space around `/`


h


Space around `/`


I kind of like it. Especially when running forward. Let's you know where
you are.

2016-03-01 17:39 GMT-08:00 Lindsey <notifications@github.com>:

> @rowanc1 <https://github.com/rowanc1> , @sgkang
> <https://github.com/sgkang> : What is your guys' take on the self.verbose?
> (personally, for reading the code, I don't like it... it doesn't actually
> do anything and sort of breaks up the chain of events but I can see where
> it might be handy for actually using the code, ha)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/simpeg/simpeg/commit/d5967d20b98d5be3c89b03d5c39bf7990851c916#commitcomment-16434724>
> .
>


Hi,

Please move to the new driver & ROS wrapper here: https://github.com/intel-ros/realsense
It should solve most of the issues and it is all open sourced :)

Thx,
Amit


remove the space before `*`


Consider installing the Jenkins slave agent on the libvirt host instead of doing everything over SSH, there are a few benefits to it:
- It would at least make logging easier
- It will give you the kind of isolation that Jenkins workspaces can
- It will let you track changes in Git repos used in the process with Jenkins
- It will allow you to scale by simply adding slaves
Also this will crash and burn if the 'sat-deploy' directory is not there, maybe add some command to create it?


kk, once you get it done I'll set up the compiler for it, that way it'll be compatible with future sets too


There are two ways to enumRPNs, one is first select operators, then permute; the other is first permute, then select operators. Either is ok, here uses the second one.
In L56-L61, two loops are used to add first two numbers, that helps performance but not necessary. As evalRPN excludes invalid RPNs.
In places like L64-65, you use cands.remove()/add() to remove used items. Notice that remove/add for ArrayList is O(n). So a better way is to use an bool[] used array to record used situation.
You first enumerates all RPNs, then eval all RPNs. It might take much memory. So one improvement can be eval immediately after enumerating one RPN.
Notice div 0 when eval / operation.


Hey @thomasdavis thanks, I see the fix now, maybe you can consider to mention this commit and the changes in your commit log next time, it'll be easier to trace :smile:


indexOf(Object o)
Returns the index of the first occurrence of the specified element in this list, or -1 if this list does not contain the element.


this is a abc.txt


can you hide this [here](https://github.com/MozVR/mozvr.com/blob/e86372f/assets/css/style.css#L792)?

```css
[data-supports-mobile="false"] [data-showcase-slug="spacelamb"] { display: none; }
```


MonthMudaViewControllerの43行目で落ちるから、mudaTotal = spTotal - (saveData.objectForKey("GOAL") as! Int)のところで、NSUserDefaultsのGOALのキーに対応してるところが、nilかどうかif文で分岐させたほうがいいよ


Should the Where clause be "units > 6"?


> I'll get a build together here. Just been swamped with my paying day job but being forced to take some time off before the end of the year.

This was two months ago… If you don't have time to maintain this package, will you agree with me moving it to python-modules team and preparing a new upload there (with new upstream release also included)?


あと、画面遷移のところで、戻るためにセグエをつなぐべきではないので、そこはボタンに対応してる@IBActionのなかで
dismissViewControllerAnimatedを実装したほうがいいよ
詳しくは、下記
https://github.com/honami725/Original/issues/4


"molto altro ancora", se non altro per spezzare di più rispetto alla frase seguente. Direi che possiamo lasciare "installato" sottointeso.


それ以外は
[![LGTM](http://lgtm.in/p/ier1)](http://lgtm.in/i/ier1)


looks like it! thanks!


for 迴圈少用，多用  each, map, reduce，除非有依序執行的必要


Sorry, I guess I misunderstood the description within test and the commit message.
The three examples @ljharb mentioned indeed throw in Chrome.

Thank you for the clarification.


Tasks are displayed in correct order.  ProcessTest has no failures.  Code is well-formatted.


When it comes to constructing good HTML, there is a guiding principle called "semantic markup," which basically means "Does this HTML represent what's actually being displayed on my page?"  That can be a little vague, so try to think about it more like, "Does the HTML I create communicate the meaning of what's on my page?"

When you have `<div class="board__square--light" id="a-1"></div>`, that tells me a lot.  When you have an empty div, that is a *code smell*, which is something you want to avoid.

So what are you trying to accomplish here?  My bet is that you were looking for a way to push the second rank to a new "row."

There are a lot of ways of accomplishing this, but let's use this as an opportunity to explore a new feature in CSS: the `:nth-child` **pseudo-class**.  Check this out for more info: [Pseudo Classes](https://developer.mozilla.org/en-US/docs/Web/CSS/Pseudo-classes)

One of the most common pseudo-classes is when you're trying to apply different states to a link (like hover, or visited) - like when someone hovers on a link, you want to add an underline and make it bold:

```
// Example:
a:hover {
    text-decoration: underline;
    font-weight: bold;
}
```

But [:nth-child](https://developer.mozilla.org/en-US/docs/Web/CSS/:nth-child) gives you some interesting options - such as only applying a rule to every **n**th (2nd, 3rd, 4th, 8th, etc) element.  Click on the nth-child link in the last sentence, and it will show you how to apply it in your CSS.

So, instead of making every square `display: inline-block;`, let's look at other options...

First, when you `float` a `display: block;` element, they will line up in a way that is similar to using inline-block.  Floating an element removes it from it's normal flow, and moves it to either the left or right side, depending on the direction you float (`float: left;` or `float: right;`).  Other content will "flow" around it if the floated element is less width than it's parent element.  That's how you can see text that will wrap around an image sometimes online.  [More info on float] (https://developer.mozilla.org/en-US/docs/Web/CSS/float)

Once you have all of the squares floated, you can then `clear` every 8th square, which is a way of saying, "Do not have this element wrap around previously floated elements."

All in all... this will help you start a new row, but without having to include an un-semantic `<div/>` that isn't really doing anything except something you can do with CSS.


Grazie *all’*account Firefox potrai *avere sempre a disposizione* segnalibri, password, schede aperte e molto altro *ancora* in qualunque dispositivo con Firefox. (niente installato)

Possibile alternativa (anche per la frase precedente): "con Firefox" -> "dove utilizzi Firefox"?


Why to force set the response buffer size to 256bytes? It should not be hardcode here.


connettiti -> accedi?


E se la girassimo completamente? "Accedi con Firefox da qualunque dispositivo"?


Decidi -> Scegli


You can use `a = mens.slice();` instead.  Also don't put a semicolon at the end of block statements such as for loops.


1.  Put your pieces on their squares, not listed below the board.
2.  Remove the ID's from the pieces.
3.  Give each piece a common class such as `piece`, before it's other classes.
4.  Think about how you can apply the B.E.M. method to naming your piece class.  What is the block? What is the element? What is the modifier?  Keep in mind that a single div can have multiple classes, so you can build up the classes until you have the correct setup.


good


Once you fix your failing tests you will need to change this back.


Uploaded HTML, CSS, img files which had been managed/ edited locally.


Same with the rest.


You can't just keep doing this forever. You have to do something on your side. Add a commit hook, use a checklist, whatever.


Small change of active link colour


I believe it is recommended to use `bundle exec sidekiq` to avoid issues and make sure that the right version of sidekiq is used.


@nk0leg, `installedServices`, `numServices` have not valid name style.


Add this afterwards:
```
See :doc:`/functions/conditional`.


"rinnovatissimo" suona fuori posto :-\

Benvenuto in Firefox Hello. A partire da questa versione, completamente rinnovata, ogni volta che inviti un amico in una conversazione verrà condivisa anche la pagina Web che stai visitando. (singolare, non plurale).


Ah, I see you did that here.


@nk0leg, better to write the whole word `error`


Heroku actually recommends puma now.


Good catch, thanks!


Indicato o spiegato? A me piace "indicato", nel caso è meglio adeguare la frase successiva.


@thomasdavis I add a new commit here: 2655722259198d069b68f9ab5f61d2649c54669e, should work now, I'll take a look at tutorial template and see what can we do.


@nk0leg, extra space at the beggining


This should be coming from figaro, not hard coded.


Gotta check the js console on the other pages so that JS doesn't stop executing site wide.

The reason it broke was that the scripts in layout.html were missing semicolons, thus the minify broke js. might be okay though


aaa


Doesn't `Redis.new` accept a url?


アドレスありがとうございます！
なおしますね！


Forget update version in this part TerrainBuilder.h,
`// see following files:
// contrib/extractor/system.cpp
// src/game/GridMap.cpp
static char const* MAP_VERSION_MAGIC = "w0.6";
`
if extract new mmaps got error
`maps/0002328.map is the wrong version, please extract new .map file`


If I remember right, this is for the hack. Heroku now gives a free worker dyno, no need to use the hack anymore.


特色团


I guess you can disregard my last comment. This is why a PR would be much better, only one changeset to comment on.


HTML is basically a structural representation of what we're building.  Would it be more accurate for the pieces to be outside of the board as they are now, or to place them inside the div of the board?


Need to fix indentation.


這邊 誇號應該是沒有意義的，請拿掉


This should come from an environment variable. No hard coded urls (which include passwords in this case). That is very bad to have in your repo, there are bots which mine repos for that kind of data.


no wtf


Thanks @thomasdavis, I'll check it.


Non suona granché come un titolo. Mi piace "Un solo Firefox." ma lascerei l'ordine originale. Per la prima frase.

Un'infinità di dispositivi.
Un mare di dispositivi.
Una marea di dispositivi.
Tanti dispositivi.
Tutti i tuoi dispositivi.
...


@PVince81 @schiesbn Do we need to clarify here that they can actually always be reshared. Just that it is currently not an option to disable it?


Turns out you didn't really need the add & remove class code with the way you implemented things :)


You are mixing what the output looks like (the view/UI) with the logic (this function).   For low coupling, you should separate the two.


My understanding was that everything is supposed to go through `FormProcessorInterface`, and then you use it's helper methods to get the other processors.

So for this you would use [this function](https://github.com/dimagi/commcare-hq/blob/master/corehq/form_processor/interfaces/processor.py#L70) and do:

```python
FormProcessorInterface(domain).ledger_processor.whatever()
```

Happy to change, but I think that's quite similar to how we do the case db and underlying form processor


Most browsers are pretty forgiving, but just as an FYI, you forgot the closing </html> tag. Not a big deal.


for 迴圈內避免使用 try catch


See vim-airline/vim-airline-themes #22
(use the default base16 theme, if you don't like the look)


You should not hardcode file sizes here, but rather should make it a configuration parameter of the software.


這邊也是 for 迴圈內不要使用 try catch


Just another repository


Why do the next 4 functions do the exact same thing? Why not have one general one that they all use?


Actually you can, you just have to open a separate window, my bad


`Un Internet dove le persone siano davvero protagoniste, in cui ogni individuo sia artefice della propria vita online, esercitando il pieno controllo in modo indipendente e sicuro.`

Un Internet in cui le persone possano essere protagoniste, in cui ciascuno di noi possa sentirsi artefice della propria esperienza online, sempre in controllo, sicuro e indipendente.


These invalid comments need fixed.


Here we may want to use do_umount() from test-lib.sh


test commit


It seems the code WAS correct before your commit. I found an article that says from Nelsen's book (1999) that the multivariate survival function S(x_i, i = 1, ..., n) = C_{survival}(S_i(x_i), i= 1, ..., n). So this should call the computeSurvivalFunction on both the copula and the margins... Maybe the problem comes from the copula::computeSurvivalFunction itself?


:disappointed:


Great! Thanks!


Отлично


Ok , Sekarang Gua Ngerti Kenapa lu ngoding lama. Bukan Gak bisa cuman tekniknya aja kurang di poles


Good idea but it will be an order of magnitude slower than raw operations, or worse. So it should really only be used when we have to.


We also want to have numbers, spaces, and /'s  in our List/Category/Item names


This is where I changed from recursive parsing to stack based parsing.  So elegant.