Ok, just wanted to make sure I wasn't missing something.  I agree that the block move is inert as the code is right now.


> does auto-selecting cases still work for user cases and submodules?

Nothing that used to work should stop working. But I'm realizing I'm not 100% clear on how user as a case should behave. Will offline with you.


ここまだ治ってないです


Hehe, good eye :)


`watchtag` ?


Not seeing IT tests results still.

http://sonar.pentaho.com:9000/dashboard/index/47855


I suggest that you develop features on a different git branch than master. Now, when there are updates by upstream, then you will likely experience merge conflicts.


ok,this time is realy success to pull the request!


renaming of `RangeOffsetHelper` to `RangeOffsetConverter` occured in previous commit. I wonder whether the previous commit actually compiles...


File-Namen bitte ohne Leerzeichen und Sonderzeichen inkl. ü,ö,ä,ß.


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


Should it be "memory_size / m_cost == 0" ?


when we need to convert an array into another array, instead of using the `each` method and pushing elements into a new array, we usually use the `map` method which will do that for us:

```
if name
  new_array = name.split.map do |name|
    name.capitalize
  end
  self.name = name_array.join(' ')
end
```

**OR** we can do this all in 1 line:

```
if name
  self.name = name.split.map(&:capitalize).join(' ')
end
```

**OR** we can use the `each` method and the `capitalize!` method which will change the string instead of just returning a capitalized copy of it:

```
if name
  self.name = name.split.each(&:capitalize!).join(' ')
end
```

Anyway...lots of other ways to do this, but mostly I wanted to point out that if you're using `each` to change elements and push them into a new array, we usually use `map` instead.


OK. The change from `message` to `reason` aligns more closely with the HTTP spec that calls the text after the code the "reason phrase". And having just one `message` that contains text puts less burden on the exception handling mechanism and code (no need to track and bubble up a `title` and `detail`).
:+1:


agree with this proposal ...

and i think in the release notes we should say something like ... :
"php 7 is coming out in January, at which time we will move to php7. we will support php 5.x for 6mo(?), and then as of June 2016, we will do the following ..."

that is, give advance warning for our plans.


Додати тінь для цього блоку


This whole commit is so wrong....
server should never return anything else than 200 or 503, where 200 means you got new api key and 503 means-> you already had your new api key in the last 24hours, any other response code means internal server error. 4/7 of this code will **never** be executed.


well your methods should be able to extract any documentation, including python. So if you can make your class do the same thing it's proof that it works well.


Isn't this an unrelated change which was already merged?


a message such as "Inconsistency detected in Hightlighting data. Highlighting will be ignored for file XXX" followed by stacktrace would be more useful to end user in case of actual error (and would please SonarTech)


@ljharb still not fixed? O_o Ok, I will open issue (and some other). It should be fixed, overwise need replace `object.assign` to `core-js` or something else for prevent regression %)


EXTERMINATE semicolons!


didn't reallize that these are actually arrays... I'll make another commit on that


1907-TerminalOASpec (kbenne) - i386-Windows-7-VisualStudio-12: OK (2338 of 2365 tests passed)

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


Yeah, working on this - just typing another email in this regard. You can merge, however - I'll send another commit later implementing the modified model.


I'm nervous about `split.first` returning `nil` and then: `!params[k].last.key?(first_key)` behaving oddly.


I have been meaning to email you, was in La this weekend and my code was on
the desktop at home.  I tried to add a Hello.java file to git but I don't
think I committed it yet.  just some code and wanted to do a test to see if
I can upload it to the repository.  Also I wrote my code in eclipse but
when I saved the file it showed it as a jgrasp file.  weird

On Sun, Oct 18, 2015 at 3:26 AM, Ben <notifications@github.com> wrote:

> Looks Awesome.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qWeX23/BlackJackSEM/commit/e267cbc5a85eaf1ec98e235a34c16b93199f7de2#commitcomment-13834822>
> .
>


@GregoireH @ndousse
Here I did not know what to put because I do not really understand.

in bmp085.cpp:
```
bool Bmp085::update(void)
{
[...]
			altitude = 44330.0f * (1.0f - pow(pressure_ /sea_level_pressure,0.190295f)) + altitude_offset_;

[...]
	return res;
}
```
```
bool Bmp085::reset_origin_altitude(float origin_altitude)
{
	altitude_offset_ = - (altitude_ - altitude_offset_ - origin_altitude );
	return true;
}
```

- what is the sign of Bmp085::altitude() ?
- what is Bmp085::altitude_offset_ ?
- what is the sign of Bmp085::altitude_offset_ ?
- When is Bmp085::altitude() equal to 0 ? When the UAV is at altitude_offset, or when it is at the origin_altitude ?


I curious as to what problem this fixed.  It seems we've gone from at least a semi-legitimate default address to a completely bogus one.


Yep, indeed. Good catch.


Just to be sure:
- Do you want to assign the opposite signed value of `relTime` to `pTimerInfo->dueTime.QuadPart`? So, if relTime is e.g. -100 you assign +100.

- Do you want to assign always the negative value of `relTime` to `pTimerInfo->dueTime.QuadPart`? Then you would need to obtain the absolute value of `relTime` first.

- Do you want to substract `relTime` from `pTimerInfo->dueTime.QuadPart`? Also consider the `relTime` 's sign then!


Why we remove these all (except show_tag_list)?


nice!


Add i18n support here


Please add i18n support here.


I'll take a look when I get to class, will you be able to stay around for
the GUI building ?
On Oct 20, 2015 10:01 AM, "Jennifer" <notifications@github.com> wrote:

> I have been meaning to email you, was in La this weekend and my code was on
> the desktop at home. I tried to add a Hello.java file to git but I don't
> think I committed it yet. just some code and wanted to do a test to see if
> I can upload it to the repository. Also I wrote my code in eclipse but
> when I saved the file it showed it as a jgrasp file. weird
>
> On Sun, Oct 18, 2015 at 3:26 AM, Ben <notifications@github.com> wrote:
>
> > Looks Awesome.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/qWeX23/BlackJackSEM/commit/e267cbc5a85eaf1ec98e235a34c16b93199f7de2#commitcomment-13834822
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/qWeX23/BlackJackSEM/commit/e267cbc5a85eaf1ec98e235a34c16b93199f7de2#commitcomment-13877654>
> .
>


@pekohl DWMR. No worries that things don't show up in the drop-down in bloc. Pasting a link is perfect. Good work here. You are shipping code :truck:


I was thinking that you had created a new extension and it was a typo and you created something with exceptions & tracy :)


@vilchik-elena perfect!


Sorry, I don't get the meaning of the sentence *(The function is called mutually exclusive with the Edrv callback functions (Rx and Tx)).*


Doesn't look like its an override either, so :+1: https://github.com/rails/rails/blob/3-2-stable/activesupport/lib/active_support/cache/file_store.rb


Ok, I have changed it. Does it look better now?


Doesn't dzil generate this anyway?


At least some of this code seems to be copied from GPL2 code (libnetlink.c). Not only was the copyright removed, I believe the license is incompatible with the license of runc.


using LogTester to verify flow is a code smell, imho
in addition, RangeOffsetConverter is testing and the raised exception is tested twice

I suggest you refactor so that `RangeOffsetConverter` is an injected dependency, which behavior you can directory control in unit test to raise an exception.

extra benefit: you get rid of static code


Ok. Done in 4cfe7534593693320d9a40b4a31171a947f5773c

In wp-config.php you can do

```
global $pb_minimum_php;
$pb_minimum_php = '9000.0.0'
```

And it will override.


```
$ rm -rf node_modules/
npm installjohnkim:xpush-link johnkim$ npm install
npm WARN engine karma@0.12.37: wanted: {"node":">=0.8 <=0.12 || >=1 <=2"} (current: {"node":"4.1.2","npm":"2.14.4"})
npm WARN deprecated find-file@0.1.4: Use the globby package instead
npm WARN deprecated CSSselect@0.7.0: the module is now available as 'css-select'
npm WARN deprecated CSSwhat@0.4.7: the module is now available as 'css-what'

> phantomjs@1.9.18 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma-phantomjs-launcher/node_modules/phantomjs
> node install.js

PhantomJS detected, but wrong version 1.9.7 @ /usr/local/bin/phantomjs.
Download already available at /var/folders/3l/35cgx8r13y50nxq68_1d1kbh0000gn/T/phantomjs/phantomjs-1.9.8-macosx.zip
Extracting zip contents
Removing /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma-phantomjs-launcher/node_modules/phantomjs/lib/phantom
Copying extracted folder /var/folders/3l/35cgx8r13y50nxq68_1d1kbh0000gn/T/phantomjs/phantomjs-1.9.8-macosx.zip-extract-1445353330493/phantomjs-1.9.8-macosx -> /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma-phantomjs-launcher/node_modules/phantomjs/lib/phantom
Writing location.js file
Done. Phantomjs binary available at /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma-phantomjs-launcher/node_modules/phantomjs/lib/phantom/bin/phantomjs

> kerberos@0.0.15 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/connect-mongo/node_modules/mongodb/node_modules/mongodb-core/node_modules/kerberos
> (node-gyp rebuild) || (exit 0)

  CXX(target) Release/obj.target/kerberos/lib/kerberos.o
  CXX(target) Release/obj.target/kerberos/lib/worker.o
  CC(target) Release/obj.target/kerberos/lib/kerberosgss.o
../lib/kerberosgss.c:509:13: warning: implicit declaration of function 'gss_acquire_cred_impersonate_name' is invalid
      in C99 [-Wimplicit-function-declaration]
        maj_stat = gss_acquire_cred_impersonate_name(&min_stat,
                   ^
1 warning generated.
  CC(target) Release/obj.target/kerberos/lib/base64.o
  CXX(target) Release/obj.target/kerberos/lib/kerberos_context.o
  SOLINK_MODULE(target) Release/kerberos.node

> bson-ext@0.1.13 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/mongoose/node_modules/bson/node_modules/bson-ext
> (node-gyp rebuild 2> builderror.log) || (exit 0)

  CXX(target) Release/obj.target/bson/ext/bson.o
  SOLINK_MODULE(target) Release/bson.node

> gifsicle@0.1.7 postinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-gifsicle/node_modules/gifsicle
> node index.js

✔ pre-build test passed successfully!

> pngquant-bin@0.3.5 postinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-pngquant/node_modules/pngquant-bin
> node index.js

✔ pre-build test passed successfully!

> jpegtran-bin@0.2.8 postinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-jpegtran/node_modules/jpegtran-bin
> node index.js

> optipng-bin@0.3.11 postinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-contrib-imagemin/node_modules/imagemin/node_modules/imagemin-optipng/node_modules/optipng-bin
> node index.js

✔ pre-build test passed successfully!
npm WARN engine xmlbuilder@2.2.1: wanted: {"node":"0.8.x || 0.10.x"} (current: {"node":"4.1.2","npm":"2.14.4"})

> kerberos@0.0.15 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/mongoose/node_modules/mongodb/node_modules/mongodb-core/node_modules/kerberos
> (node-gyp rebuild) || (exit 0)

  CXX(target) Release/obj.target/kerberos/lib/kerberos.o
  CXX(target) Release/obj.target/kerberos/lib/worker.o
  CC(target) Release/obj.target/kerberos/lib/kerberosgss.o
../lib/kerberosgss.c:509:13: warning: implicit declaration of function 'gss_acquire_cred_impersonate_name' is invalid
      in C99 [-Wimplicit-function-declaration]
        maj_stat = gss_acquire_cred_impersonate_name(&min_stat,
                   ^
1 warning generated.
  CC(target) Release/obj.target/kerberos/lib/base64.o
  CXX(target) Release/obj.target/kerberos/lib/kerberos_context.o
  SOLINK_MODULE(target) Release/kerberos.node

> ws@0.4.32 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma/node_modules/socket.io/node_modules/socket.io-client/node_modules/ws
> (node-gyp rebuild 2> builderror.log) || (exit 0)

  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o

> v8-debug@0.5.4 preinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-debug
>

> v8-profiler@5.3.2 preinstall /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-profiler
>

> utf-8-validate@1.2.1 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/ws/node_modules/utf-8-validate
> node-gyp rebuild

  CXX(target) Release/obj.target/validation/src/validation.o
  SOLINK_MODULE(target) Release/validation.node

> bufferutil@1.2.1 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/ws/node_modules/bufferutil
> node-gyp rebuild

  CXX(target) Release/obj.target/bufferutil/src/bufferutil.o
  SOLINK_MODULE(target) Release/bufferutil.node

> fsevents@1.0.2 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/karma/node_modules/chokidar/node_modules/fsevents
> node-pre-gyp install --fallback-to-build

[fsevents] Success: "/Users/johnkim/IdeaProjects/xpush-link/node_modules/karma/node_modules/chokidar/node_modules/fsevents/lib/binding/Release/node-v46-darwin-x64/fse.node" is installed via remote

> v8-profiler@5.3.2 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-profiler
> node-pre-gyp install --fallback-to-build

[v8-profiler] Success: "/Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-profiler/build/profiler/v5.3.2/node-v46-darwin-x64/profiler.node" is installed via remote

> v8-debug@0.5.4 install /Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-debug
> node-pre-gyp install --fallback-to-build

[v8-debug] Success: "/Users/johnkim/IdeaProjects/xpush-link/node_modules/grunt-node-inspector/node_modules/node-inspector/node_modules/v8-debug/build/debug/v0.5.4/node-v46-darwin-x64/debug.node" is installed via remote
karma-script-launcher@0.1.0 node_modules/karma-script-launcher

karma-firefox-launcher@0.1.6 node_modules/karma-firefox-launcher

jit-grunt@0.5.0 node_modules/jit-grunt

grunt-contrib-copy@0.5.0 node_modules/grunt-contrib-copy

karma-html2js-preprocessor@0.1.0 node_modules/karma-html2js-preprocessor

karma-ng-html2js-preprocessor@0.1.2 node_modules/karma-ng-html2js-preprocessor

karma-requirejs@0.2.2 node_modules/karma-requirejs

connect-livereload@0.4.1 node_modules/connect-livereload

errorhandler@1.0.2 node_modules/errorhandler

karma-jasmine@0.1.6 node_modules/karma-jasmine

open@0.0.5 node_modules/open

composable-middleware@0.3.0 node_modules/composable-middleware

grunt-karma@0.8.3 node_modules/grunt-karma

grunt-rev@0.1.0 node_modules/grunt-rev

grunt-open@0.2.3 node_modules/grunt-open

node-uuid@1.4.3 node_modules/node-uuid

karma-ng-scenario@0.1.0 node_modules/karma-ng-scenario

requirejs@2.1.20 node_modules/requirejs

lodash@2.4.2 node_modules/lodash

grunt-express-server@0.4.19 node_modules/grunt-express-server

morgan@1.0.1 node_modules/morgan
└── bytes@0.3.0

method-override@1.0.2 node_modules/method-override
└── methods@1.0.0

serve-favicon@2.0.1 node_modules/serve-favicon
└── fresh@0.2.2

grunt-env@0.4.4 node_modules/grunt-env
└── ini@1.3.4

cookie-parser@1.0.1 node_modules/cookie-parser
├── cookie-signature@1.0.3
└── cookie@0.1.0

express-session@1.0.4 node_modules/express-session
├── uid2@0.0.3
├── debug@0.8.1
├── utils-merge@1.0.0
├── cookie@0.1.2
├── cookie-signature@1.0.3
└── buffer-crc32@0.2.1

passport@0.2.2 node_modules/passport
├── pause@0.0.1
└── passport-strategy@1.0.0

express-jwt@3.1.0 node_modules/express-jwt
├── express-unless@0.3.0
└── async@0.9.2

grunt-concurrent@0.5.0 node_modules/grunt-concurrent
├── async@0.2.10
└── pad-stdio@0.1.1 (lpad@0.2.1)

time-grunt@0.3.2 node_modules/time-grunt
├── date-time@0.1.1
├── pretty-ms@0.1.0
├── hooker@0.2.3
├── text-table@0.2.0
└── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)

jshint-stylish@0.1.5 node_modules/jshint-stylish
├── text-table@0.2.0
└── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)

grunt-contrib-concat@0.4.0 node_modules/grunt-contrib-concat
└── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)

grunt-usemin@2.1.1 node_modules/grunt-usemin
├── debug@0.7.4
└── lodash@1.0.2

grunt-contrib-htmlmin@0.2.0 node_modules/grunt-contrib-htmlmin
├── each-async@0.1.3
├── pretty-bytes@0.1.2
├── html-minifier@0.5.6
└── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)

grunt-contrib-clean@0.5.0 node_modules/grunt-contrib-clean
└── rimraf@2.2.8

should@3.3.2 node_modules/should

ejs@0.8.8 node_modules/ejs

compression@1.0.11 node_modules/compression
├── vary@1.0.1
├── on-headers@1.0.1
├── compressible@1.1.1
├── bytes@1.0.0
├── debug@1.0.4 (ms@0.6.2)
└── accepts@1.0.7 (negotiator@0.4.7, mime-types@1.0.2)

passport-local@0.1.6 node_modules/passport-local
├── pkginfo@0.2.3
└── passport@0.1.18 (pause@0.0.1)

body-parser@1.5.2 node_modules/body-parser
├── qs@0.6.6
├── media-typer@0.2.0
├── raw-body@1.3.0
├── depd@0.4.4
├── bytes@1.0.0
├── type-is@1.3.2 (mime-types@1.0.2)
└── iconv-lite@0.4.4

karma-chrome-launcher@0.1.12 node_modules/karma-chrome-launcher
└── which@1.2.0 (is-absolute@0.1.7)

grunt-newer@0.7.0 node_modules/grunt-newer
├── async@0.2.10
└── rimraf@2.2.6

express@4.9.8 node_modules/express
├── cookie@0.1.2
├── utils-merge@1.0.0
├── merge-descriptors@0.0.2
├── fresh@0.2.4
├── media-typer@0.3.0
├── vary@1.0.1
├── serve-static@1.6.5
├── finalhandler@0.2.0
├── range-parser@1.0.2
├── escape-html@1.0.1
├── cookie-signature@1.0.5
├── parseurl@1.3.0
├── path-to-regexp@0.1.3
├── methods@1.1.0
├── depd@0.4.5
├── qs@2.2.4
├── on-finished@2.1.1 (ee-first@1.1.0)
├── debug@2.0.0 (ms@0.6.2)
├── proxy-addr@1.0.8 (forwarded@0.1.0, ipaddr.js@1.0.1)
├── etag@1.4.0 (crc@3.0.0)
├── send@0.9.3 (destroy@1.0.3, ms@0.6.2, mime@1.2.11, on-finished@2.1.0)
├── type-is@1.5.7 (mime-types@2.0.14)
└── accepts@1.1.4 (negotiator@0.4.9, mime-types@2.0.14)

karma-coffee-preprocessor@0.2.1 node_modules/karma-coffee-preprocessor
└── coffee-script@1.7.1 (mkdirp@0.3.5)

passport-twitter@1.0.3 node_modules/passport-twitter
├── xtraverse@0.1.0 (xmldom@0.1.19)
└── passport-oauth1@1.0.1 (utils-merge@1.0.0, passport-strategy@1.0.0, oauth@0.9.14)

passport-facebook@2.0.0 node_modules/passport-facebook
└── passport-oauth2@1.1.2 (uid2@0.0.3, passport-strategy@1.0.0, oauth@0.9.14)

passport-google-oauth@0.2.0 node_modules/passport-google-oauth
├── pkginfo@0.3.1
└── passport-oauth@1.0.0 (passport-oauth1@1.0.1, passport-oauth2@1.1.2)

grunt-contrib-watch@0.6.1 node_modules/grunt-contrib-watch
├── async@0.2.10
├── tiny-lr-fork@0.0.5 (debug@0.7.4, qs@0.5.6, faye-websocket@0.4.4, noptify@0.0.3)
└── gaze@0.5.2 (globule@0.1.0)

supertest@0.11.0 node_modules/supertest
├── methods@0.1.0
└── superagent@0.17.0 (methods@0.0.1, cookiejar@1.3.0, qs@0.6.5, extend@1.2.1, debug@0.7.4, emitter-component@1.0.0, mime@1.2.5, reduce-component@1.0.1, formidable@1.0.14)

grunt-contrib-uglify@0.4.1 node_modules/grunt-contrib-uglify
├── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)
├── uglify-js@2.5.0 (async@0.2.10, uglify-to-browserify@1.0.2, source-map@0.5.1, yargs@3.5.4)
└── maxmin@0.1.0 (pretty-bytes@0.1.2, gzip-size@0.1.1)

grunt-contrib-cssmin@0.9.0 node_modules/grunt-contrib-cssmin
├── chalk@0.4.0 (has-color@0.1.7, ansi-styles@1.0.0, strip-ansi@0.1.1)
├── clean-css@2.1.8 (commander@2.1.0)
└── maxmin@0.1.0 (pretty-bytes@0.1.2, gzip-size@0.1.1)

grunt@0.4.5 node_modules/grunt
├── eventemitter2@0.4.14
├── dateformat@1.0.2-1.2.3
├── which@1.0.9
├── async@0.1.22
├── colors@0.6.2
├── getobject@0.1.0
├── lodash@0.9.2
├── rimraf@2.2.8
├── hooker@0.2.3
├── grunt-legacy-util@0.2.0
├── exit@0.1.2
├── coffee-script@1.3.3
├── nopt@1.0.10 (abbrev@1.0.7)
├── iconv-lite@0.2.11
├── underscore.string@2.2.1
├── minimatch@0.2.14 (sigmund@1.0.1, lru-cache@2.7.0)
├── glob@3.1.21 (inherits@1.0.2, graceful-fs@1.2.3)
├── grunt-legacy-log@0.1.2 (grunt-legacy-log-utils@0.1.1, lodash@2.4.2, underscore.string@2.3.3)
├── findup-sync@0.1.3 (lodash@2.4.2, glob@3.2.11)
└── js-yaml@2.0.5 (esprima@1.0.4, argparse@0.1.16)

grunt-ng-annotate@0.2.3 node_modules/grunt-ng-annotate
└── ng-annotate@0.9.11 (tryor@0.1.2, stringset@0.2.1, stringmap@0.2.2, simple-is@0.2.0, simple-fmt@0.1.0, alter@0.2.0, stable@0.1.5, convert-source-map@0.4.1, ordered-ast-traverse@0.1.1, esprima@1.2.5, source-map@0.1.43, optimist@0.6.1)

grunt-mocha-test@0.10.2 node_modules/grunt-mocha-test
├── hooker@0.2.3
├── fs-extra@0.8.1 (jsonfile@1.1.1, rimraf@2.2.8, ncp@0.4.2, mkdirp@0.3.5)
└── mocha@1.18.2 (diff@1.0.7, commander@2.0.0, growl@1.7.0, mkdirp@0.3.5, debug@2.2.0, glob@3.2.3, jade@0.26.3)

grunt-angular-templates@0.5.7 node_modules/grunt-angular-templates
└── html-minifier@0.6.9 (relateurl@0.2.6, change-case@2.1.6, clean-css@2.2.23, uglify-js@2.4.24, cli@0.6.6)

grunt-dom-munger@3.4.0 node_modules/grunt-dom-munger
└── cheerio@0.12.4 (entities@0.5.0, underscore@1.4.4, htmlparser2@3.1.4, cheerio-select@0.0.3)

jsonwebtoken@5.4.0 node_modules/jsonwebtoken
├── ms@0.7.1
└── jws@3.1.0 (jwa@1.1.0, base64url@1.0.4)

karma-phantomjs-launcher@0.1.4 node_modules/karma-phantomjs-launcher
└── phantomjs@1.9.18 (which@1.0.9, progress@1.1.8, kew@0.4.0, request-progress@0.3.1, adm-zip@0.4.4, request@2.42.0, npmconf@2.1.1, fs-extra@0.23.1)

grunt-build-control@0.4.0 node_modules/grunt-build-control
└── shelljs@0.2.6

grunt-svgmin@0.4.0 node_modules/grunt-svgmin
├── each-async@0.1.3
├── pretty-bytes@0.1.2
├── chalk@0.4.0 (has-color@0.1.7, ansi-styles@1.0.0, strip-ansi@0.1.1)
└── svgo@0.4.5 (colors@0.6.2, whet.extend@0.9.9, coa@0.4.1, sax@0.6.1, js-yaml@2.1.3)

grunt-contrib-jshint@0.10.0 node_modules/grunt-contrib-jshint
├── hooker@0.2.3
└── jshint@2.5.11 (underscore@1.6.0, strip-json-comments@1.0.4, exit@0.1.2, console-browserify@1.1.0, minimatch@1.0.0, shelljs@0.3.0, cli@0.6.6, htmlparser2@3.8.3)

karma-ng-jade2js-preprocessor@0.1.5 node_modules/karma-ng-jade2js-preprocessor
└── jade@1.3.1 (commander@2.1.0, character-parser@1.2.0, mkdirp@0.3.5, constantinople@2.0.1, with@3.0.1, transformers@2.1.0, monocle@1.1.51)

connect-mongo@0.8.2 node_modules/connect-mongo
├── depd@1.1.0
├── debug@2.2.0 (ms@0.7.1)
└── mongodb@2.0.46 (readable-stream@1.0.31, es6-promise@2.1.1, mongodb-core@1.2.19)

karma-jade-preprocessor@0.0.11 node_modules/karma-jade-preprocessor
└── jade@0.33.0 (character-parser@1.0.2, mkdirp@0.3.5, commander@1.2.0, constantinople@1.0.2, with@1.1.1, monocle@0.1.48, transformers@2.0.1)

grunt-wiredep@1.8.0 node_modules/grunt-wiredep
└── wiredep@1.8.6 (propprop@0.3.0, minimist@1.2.0, chalk@0.5.1, through2@0.6.5, glob@4.5.3, bower-config@0.5.2)

grunt-injector@0.5.4 node_modules/grunt-injector
└── wiredep@1.7.3 (minimist@0.1.0, chalk@0.1.1, lodash@1.3.1, through2@0.4.2, glob@3.2.11, bower-config@0.5.2)

grunt-nodemon@0.2.1 node_modules/grunt-nodemon
└── nodemon@1.0.20 (minimatch@0.2.14, ps-tree@0.0.3, update-notifier@0.1.10)

nodemailer@0.7.1 node_modules/nodemailer
├── public-address@0.1.1
├── directmail@0.1.8
├── he@0.3.6
├── readable-stream@1.1.13 (string_decoder@0.10.31, isarray@0.0.1, core-util-is@1.0.1, inherits@2.0.1)
├── simplesmtp@0.3.35 (xoauth2@0.1.8, rai@0.1.12)
├── mailcomposer@0.2.12 (mime@1.2.11, dkim-signer@0.1.2, follow-redirects@0.0.3, mimelib@0.2.19)
└── aws-sdk@2.0.5 (xmlbuilder@0.4.2, aws-sdk-apis@3.1.10, xml2js@0.2.6)

grunt-autoprefixer@0.7.6 node_modules/grunt-autoprefixer
├── diff@1.0.8
├── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)
└── autoprefixer@1.3.1 (postcss@0.3.5, fs-extra@0.9.1, caniuse-db@1.0.30000348)

grunt-babel@5.0.3 node_modules/grunt-babel
└── babel-core@5.8.25 (slash@1.0.0, try-resolve@1.0.1, babel-plugin-remove-console@1.0.1, babel-plugin-remove-debugger@1.0.1, babel-plugin-react-display-name@1.0.3, babel-plugin-constant-folding@1.0.1, babel-plugin-eval@1.0.1, babel-plugin-react-constant-elements@1.0.3, babel-plugin-dead-code-elimination@1.0.2, babel-plugin-jscript@1.0.4, babel-plugin-inline-environment-variables@1.0.1, babel-plugin-property-literals@1.0.1, path-exists@1.0.0, trim-right@1.0.1, shebang-regex@1.0.0, babel-plugin-member-expression-literals@1.0.1, babel-plugin-undefined-to-void@1.1.6, to-fast-properties@1.0.1, path-is-absolute@1.0.0, babel-plugin-proto-to-assign@1.0.4, fs-readdir-recursive@0.1.2, babel-plugin-runtime@1.0.7, globals@6.4.1, private@0.1.6, esutils@2.0.2, convert-source-map@1.1.1, js-tokens@1.0.1, home-or-tmp@1.0.0, babel-plugin-undeclared-variables-check@1.0.2, babylon@5.8.23, debug@2.2.0, line-numbers@0.2.0, source-map@0.4.4, detect-indent@3.0.1, is-integer@1.0.6, repeating@1.1.3, bluebird@2.10.2, chalk@1.1.1, minimatch@2.0.10, output-file-sync@1.1.1, resolve@1.1.6, source-map-support@0.2.10, regexpu@1.3.0, json5@0.4.0, lodash@3.10.1, regenerator@0.8.35, core-js@1.2.2)

karma-babel-preprocessor@5.2.2 node_modules/karma-babel-preprocessor
└── babel-core@5.8.25 (slash@1.0.0, try-resolve@1.0.1, babel-plugin-remove-console@1.0.1, babel-plugin-remove-debugger@1.0.1, babel-plugin-react-display-name@1.0.3, babel-plugin-eval@1.0.1, babel-plugin-jscript@1.0.4, babel-plugin-react-constant-elements@1.0.3, babel-plugin-constant-folding@1.0.1, babel-plugin-inline-environment-variables@1.0.1, babel-plugin-dead-code-elimination@1.0.2, babel-plugin-property-literals@1.0.1, path-exists@1.0.0, shebang-regex@1.0.0, trim-right@1.0.1, to-fast-properties@1.0.1, babel-plugin-member-expression-literals@1.0.1, babel-plugin-undefined-to-void@1.1.6, babel-plugin-proto-to-assign@1.0.4, path-is-absolute@1.0.0, fs-readdir-recursive@0.1.2, babel-plugin-runtime@1.0.7, globals@6.4.1, private@0.1.6, esutils@2.0.2, convert-source-map@1.1.1, js-tokens@1.0.1, home-or-tmp@1.0.0, babel-plugin-undeclared-variables-check@1.0.2, babylon@5.8.23, debug@2.2.0, line-numbers@0.2.0, source-map@0.4.4, detect-indent@3.0.1, is-integer@1.0.6, repeating@1.1.3, bluebird@2.10.2, chalk@1.1.1, minimatch@2.0.10, output-file-sync@1.1.1, resolve@1.1.6, source-map-support@0.2.10, regexpu@1.3.0, json5@0.4.0, lodash@3.10.1, regenerator@0.8.35, core-js@1.2.2)

grunt-protractor-runner@1.2.1 node_modules/grunt-protractor-runner
├── split@0.3.3 (through@2.3.8)
├── through2@0.5.1 (xtend@3.0.0, readable-stream@1.0.33)
└── protractor@1.8.0 (jasminewd@1.1.0, jasminewd2@0.0.2, html-entities@1.1.3, q@1.0.0, saucelabs@0.1.1, minijasminenode@1.1.1, adm-zip@0.4.4, optimist@0.6.1, accessibility-developer-tools@2.6.0, glob@3.2.11, request@2.36.0, source-map-support@0.2.10, jasmine@2.1.1, selenium-webdriver@2.44.0)

grunt-contrib-imagemin@0.7.2 node_modules/grunt-contrib-imagemin
├── pretty-bytes@0.1.2
├── async@0.7.0
├── chalk@0.4.0 (ansi-styles@1.0.0, has-color@0.1.7, strip-ansi@0.1.1)
└── imagemin@0.4.9 (stat-mode@0.2.1, ware@0.3.0, image-type@0.1.4, nopt@3.0.4, tempfile@0.1.3, rimraf@2.4.3, fs-extra@0.10.0, imagemin-svgo@0.1.1, imagemin-gifsicle@0.1.1, imagemin-pngquant@0.1.3, imagemin-jpegtran@0.1.0, imagemin-optipng@0.1.0)

grunt-google-cdn@0.4.3 node_modules/grunt-google-cdn
├── chalk@0.5.1 (ansi-styles@1.1.0, escape-string-regexp@1.0.3, supports-color@0.2.0, strip-ansi@0.3.0, has-ansi@0.1.0)
├── google-cdn@0.7.0 (regexp-quote@0.0.0, google-cdn-data@0.1.22, async@0.9.2, semver@2.3.2, debug@1.0.4, cdnjs-cdn-data@0.1.2, bower@1.3.12)
└── bower@1.6.3

mongoose@4.0.8 node_modules/mongoose
├── hooks-fixed@1.0.2
├── async@0.9.0
├── regexp-clone@0.0.1
├── mpromise@0.5.4
├── mpath@0.1.1
├── muri@1.0.0
├── ms@0.1.0
├── sliced@0.0.5
├── kareem@1.0.1
├── mquery@1.6.1 (debug@2.2.0, bluebird@2.9.26)
├── bson@0.3.2 (bson-ext@0.1.13)
└── mongodb@2.0.34 (readable-stream@1.0.31, mongodb-core@1.2.0)

karma@0.12.37 node_modules/karma
├── rimraf@2.4.3
├── di@0.0.1
├── q@1.4.1
├── mime@1.3.4
├── graceful-fs@3.0.8
├── colors@1.1.2
├── useragent@2.1.7 (lru-cache@2.2.4)
├── glob@5.0.15 (path-is-absolute@1.0.0, inherits@2.0.1, once@1.3.2, inflight@1.0.4)
├── source-map@0.4.4 (amdefine@1.0.0)
├── minimatch@2.0.10 (brace-expansion@1.1.1)
├── optimist@0.6.1 (wordwrap@0.0.3, minimist@0.0.10)
├── log4js@0.6.27 (async@0.2.10, underscore@1.8.2, semver@4.3.6, readable-stream@1.0.33)
├── http-proxy@0.10.4 (colors@0.6.2, pkginfo@0.3.1, utile@0.2.1)
├── connect@2.30.2 (cookie@0.1.3, bytes@2.1.0, cookie-signature@1.0.6, utils-merge@1.0.0, content-type@1.0.1, pause@0.1.0, on-headers@1.0.1, fresh@0.3.0, response-time@2.3.1, vhost@3.0.2, basic-auth-connect@1.0.0, parseurl@1.3.0, cookie-parser@1.3.5, depd@1.0.1, qs@4.0.0, method-override@2.3.5, http-errors@1.3.1, serve-favicon@2.3.0, connect-timeout@1.6.2, debug@2.2.0, morgan@1.6.1, finalhandler@0.4.0, type-is@1.6.9, express-session@1.11.3, csurf@1.8.3, multiparty@3.3.2, serve-static@1.10.0, errorhandler@1.4.2, compression@1.5.2, body-parser@1.13.3, serve-index@1.7.2)
├── lodash@3.10.1
├── socket.io@0.9.16 (base64id@0.1.0, policyfile@0.0.4, redis@0.7.3, socket.io-client@0.9.16)
└── chokidar@1.2.0 (arrify@1.0.0, path-is-absolute@1.0.0, async-each@0.1.6, glob-parent@2.0.0, is-glob@2.0.1, is-binary-path@1.0.1, lodash.flatten@3.0.2, readdirp@2.0.0, anymatch@1.3.0, fsevents@1.0.2)

grunt-node-inspector@0.4.1 node_modules/grunt-node-inspector
└── node-inspector@0.12.3 (path-is-absolute@1.0.0, async@0.9.2, semver@4.3.6, debug@2.2.0, rc@1.1.2, which@1.2.0, strong-data-uri@1.0.3, serve-favicon@2.3.0, glob@5.0.15, yargs@3.27.0, express@4.13.3, biased-opener@0.2.6, ws@0.8.0, v8-profiler@5.3.2, v8-debug@0.5.4)
```


Thanks.

about.me/squeedee

On Tue, Oct 20, 2015 at 9:34 AM, Glyn Normington <notifications@github.com>
wrote:

> I was trying to avoid polluting the users machine, but I don't feel
> strongly, so I've reverted this.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bunniesandbeatings/go-flavor-parser/commit/7d51f49c52cf771dafd3dd98f166fd71358aba53#commitcomment-13874800>
> .
>


Just wanted to let you know that I'm really looking forward to merge your changes =)


Yep, they broke out addons into individual packages with this release instead of bundling them. :+1:


ah, I thought this was solving the intermediate problem where the sign-in page shows:

![screen shot 2015-10-20 at 16 07 12](https://cloud.githubusercontent.com/assets/27760/10611582/aebab8d6-7744-11e5-8f74-b2fecc693f19.png)

back to the drawing board!


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


logging is really not what is expected: what is expected is not failing on error, not saving Highlighting and keep on reading

logging is a side effect


yes I will be here.


GHX_Vert_IDD (mmatts) - x86_64-MacOS-10.9-clang: OK (1951 of 1951 tests passed)

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


PullLogicFromPlantComponents (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1360 of 1360 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.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.37%25-green.svg)


most methods of this test are really about testing `RangeOffsetConverter`, not the responsibilities of HighlightingLineReader


Yeah, I'll mention this. It will go away once we have Wheel based build system.


If you remove the goto done here, you can move the delete [] tempfile_name out of the else branch, remove the else branch and remove the delete [] tempfile_name at line 3720 above, resulting in single place of deletion, which seems better from the code maintenance point of view.
Also, there is no need to se the tempfile_name to NULL.


@koying fixed
https://github.com/xbmc/xbmc/commit/02c72cd6f59f5160537fcef2832bef9264a58c59


so, either tests are duplicated or tests of `RangeOffsetConverter` are incomplete


You could use a constant for FILE1 and FILE2


Doesn't solve that problem in that screenshot for me.


Sir. Plz.


okay.... So this function is always called by the timers when expired and they call the callbacks to the stack modules, right?


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


In fact it should be memory_size / m_cost != sizeof(block). Fixed with commit 6f70354331cbb4b7ad2294b64f7e2a904257b0d5.


I guess this would pass at 40+0.4, but I wanted to confirm my fear that this depends on (total time)/increment and that better formula is required.


TeamCity Xbim :: Xbim Essentials :: XbimEssentials [Build 30](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=347&buildTypeId=XbimEssentials_XbimEssentials) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:04


[WiMAX-WiFi Handover Tutorial.docx](https://github.com/kkchenk/geni-openflow-vertical-handover/files/16899/WiMAX-WiFi.Handover.Tutorial.docx)


Prefer
```
from Exceptions import (
    CommandNotFound, CommandTimeout, DataCollectorError,
    LoginFailed, NoServerFound, NoValidConnection,
    ObjectCreationError, StateTimeout, ZentinelException
)
```


I think that's a good idea -- I'll add touch() to the FedoraResource interface.


please use "p" prefix


This is known to degrade performance, [requiring two memory accesses instead of one](http://stackoverflow.com/questions/7793511/are-there-performance-issues-when-using-pragma-pack1) Shouldn't we put perfomance over memory usage?
I didn't make a comming so as to hear your thoughts on that of course.


looking at the tests, I sense this log is here to cope with statically bound code and bad separation of concerns


then this might be a Python 3 issue.


Je pense que c'est bon !


what if instance_l is NULL?


Обёртка не нужна, можно просто возвращать `this.filterIn(field, filter)`.


Sort output boards alphabetically (std::sort)?
On Oct 19, 2015 8:33 PM, "arcshock" <notifications@github.com> wrote:

> I think that this might be an issue to be used for general testing. There
> might be implementations that does not match orderings, e.g.
>
> Expected = {0, 1, 2,..., *n*}
> Actual = {2, 0, 1,...,*n*}
> *Where the integers represent different boards.*
>
> Need to see about finding a way to do comparisons of board lists without
> preference to a particular ordering of boards.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nekram/AITestBed/commit/94e652f262ecd3d71a04535c9c8feb88af85eb53#commitcomment-13865022>
> .
>


Looking into the error you are describing. Have found the cause and will work on fixing it.


same comment here as for highlighting reader, log is not useful for end user


Thanks


Dzil won't update the .mailmap file.


`string` can also read via pipe, so this would probably be nicer (and possibly faster) as `rustc | string match | string trim`.


Have I missed anything!? Why is this function empty?


Why use a copy-paste script instead of an npm package? Lets try to avoid static dependencies!


we can rename it to condional_attr, conditional_value, because we have studio support, and module is not pure xml!)


good catch! (unfortunately didn't help)


good idea!


Ok. Merged.


I know that it was not internationalised, but please add internationalisation, similar to any other xmodule.


Always put one space on each side of an operator, they are introverts and need more space (see [guidelines](https://github.com/cvut/javascript#spacing)) :cat:. I’ve fixed it for you.


Это строчка не понадобится, если цикл начинать с нуля.


Yeah, I was pointing at the README


please use **PLK** instead of EPL!


where the logs are written.


thanks m'friend


@spMohanty Sorry, there was no button to begin with. I was contemplating adding one but since you found it odd that one was missing, I think it would better to have it in there.


Just curious,  what is root_xblock?


Please explain why we need context.get('is_unit_page') ?


please prefer to use `UINT8*` instead


    self.make_graph({name: [(x // xmax) * 100 for x in xdata]
                     for name, xdata in self._avgPerformance.iteritems()},

Similar below


Thanks!

> And I guess, we just should move #if MICROPY_* to above the includes

Yes, but of course after mpconfig.h, or equivalent.


I would allow setting figure size through `kwargs`.


If the other end never sends us any Syn packet, this will block indefinitely.


Should probably be stored as `hudson.util.Secret` so it's encrypted on disk.


typo "run*s*ning"


Fix #4


There's some repetition in this method, specifically:
`content_tag :div, capture(&block), class: 'form-group'` appears twice.

Not a big deal, but since you're grasping most of the foundational concepts, I thought I'd point out 1 way to *DRY* up your code here (there are several ways):
```
extra_class = 'has-error' if errors.any?
content_tag :div, capture(&block), class: "form-group #{extra_class}"
```


Looks good, have you checked this case manually? And other cases?


Checkmarx scan results 20 Oct 2015 11:20 UTC -04:00

High
Medium
Low
Info

New
1
1
7
0

Recurrent
0
0
0
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

a4a0bdc6fc9ab3780c81de1c65fe70a137c596af	@djdrisco


Is it required assigning zero to the 64 bit padding variable before?

````
frameInfo.frame.padding2 = 0;
frameInfo.frame.pBuffer = (tPlkFrame*)pVEthTxData_p;
````

(maybe not because it is not used cross platform, right?)


This is likely the cause of the performance degradation. https://github.com/enketo/enketo-core/issues/337


@Pike this have been reverted in a followup commit https://github.com/c9/core/commit/b6d0c1cdb6cff3cf94c0e88172f4c0a9bb8ed9ae. We now have a ci test for issues like this which runs before pushing to this repository so if config is inconsistent it won't push


Thanks for finding and suggesting the fix, Daryl.  Feel free to issue pull requests through github in the future to get your changes into the baseline in an easier way.


HIer war mir nicht ganz klar, in welchem Zusammenhang der Text genutzt wird, ist "Öffne Feedback" hier richtig?


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 126](http://teamcity/viewLog.html?buildId=1547&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:01:56
### Failed tests
```
SmokeTests.dll: SmokeTests.AdminTests.Login: <no details avaliable>

SmokeTests.dll: SmokeTests.HomePageTests.LoadBlogListingPage: <no details avaliable>

SmokeTests.dll: SmokeTests.HomePageTests.LoadHomePage: <no details avaliable>

SmokeTests.dll: SmokeTests.HomePageTests.ValidateBlogListing: <no details avaliable>

SmokeTests.dll: SmokeTests.HomePageTests.ValidateHomePage: <no details avaliable>

```


čo si pridal??


These validations are great, but 1 thing I want to point out is that when you validate the minimum length of an attribute, you don't need to also add the `presence` validation. The minimum length validation ensures that the attribute must be present because it has to be at least x characters long.

That said, if you wanted to validate the minimum length of something *only* if it is present but ignore the length if it is nil, you can add an option `allow_nil: true`. That way, the attribute must be nil *or* at least x characters long.


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


If you return here, the `*pReleaseRxBuffer_p` is not assigned to any value. Does it matter?


Does the `.highlighted-box` class need to be used on both the wrapping div and the h2 inside it?


We need to add this functionality to Studio as well (https://github.com/raccoongang/edx-platform/commit/fa266be1736d1c92585bc663a2b57473c207e482#diff-9b06cf689e0418560b71d971161b6531R277)
We now check for sources in studio, but not for children.
And it would be nice in both cases to display error in Studio. Thanks.


I wonder why have you changed the code below to use length variable instead of using the MAX_LONGPATH constant as it was before. The effect is the same, since the length is assigned MAX_LONGPATH and never changes.


I think this needs a closing `<h2>`.


Still testing this part - can see AppEnvironmentExtra being set in the Registry ok, but PATH for running process is not updated.


This is a comment on a commit.


Ca me semble super ! Beau travail bébé !


@neutt22 thanks for pointing that out!


Should be just

      if yerr and xerr:


Dice ori que sucumbiste al ```while``` pawah


With the addition of this validation, the `save!` call in [`track_set`](https://github.com/mysociety/alaveteli/blob/develop/app/controllers/track_controller.rb#L137) now may raise an error, so I think we probably want to convert it to a `save` and handle the returned value being `true` or `false` - I suggest it should probably set the error notice and return false. This is going to necessitate having all the calling bits of code do something sensible on a `false` value being returned from `track_set` - my immediate thought is that they should probably just redirect to wherever they redirect to on success.


thanks for this info. i have no time right now... pls feel free to correct it.


What is wrong here??? Nothing is changed in the commit.


Missing space


Why is that needed? Can we avoid this?


@polesye please review if you want.


Ack


there is


> On Oct 20, 2015, at 3:58 AM, jzmaddock <notifications@github.com> wrote:
>
> Reverting the changes here, break MSVC linking of the lib again:
>
> msvc.link.dll ......\bin.v2\libs\serialization\build\msvc-14.0\debug\threading
> -multi\boost_serialization-vc140-mt-gd-1_59.dll
> Creating library ......\bin.v2\libs\serialization\build\msvc-14.0\debug\thr
> eading-multi\boost_serialization-vc140-mt-gd-1_59.lib and object ......\bin.v2
> \libs\serialization\build\msvc-14.0\debug\threading-multi\boost_serialization-vc
> 140-mt-gd-1_59.exp
> xml_oarchive.obj : error LNK2001: unresolved external symbol "protected: virtual int __thiscall boost::archive::detail::utf8_codecvt_facet::do_in(struct _Mbstatet &,char const *,char const *,char const * &,wchar_t *,wchar_t *,wchar_t * &)const " (?do_in@utf8_codecvt_facet@detail@archive@boost@@MBEHAAU_Mbstatet@@PBD1AAPBDPA_W3AAPA_W@Z)
> xml_oarchive.obj : error LNK2001: unresolved external symbol "protected: virtual int __thiscall boost::archive::detail::utf8_codecvt_facet::do_out(struct _Mbstatet &,wchar_t const *,wchar_t const *,wchar_t const * &,char *,char *,char * &)const " (?do_out@utf8_codecvt_facet@detail@archive@boost@@MBEHAAU_Mbstatet@@PB_W1AAPB_WPAD3AAPAD@Z)
> xml_oarchive.obj : error LNK2001: unresolved external symbol "protected: virtual int __thiscall boost::archive::detail::utf8_codecvt_facet::do_length(struct _Mbstatet const &,char const *,char const *,unsigned int)const " (?do_length@utf8_codecvt_facet@detail@archive@boost@@MBEHABU_Mbstatet@@PBD1I@Z)
>
> Just so you know.
>
Thanks for letting me know.

I’ve got two things going on here:

a) I didn’t realize that utf8 codecvt facet was used in the serialization.dll - I thought it was just the wserialization.dll.  Turns out that wchar_from_mb dataflow iterator users it.  So I guess that the utf8-iterator should be in the serializaton.dll.  I think there’s still something slightly out of whack with the export/visibility of it.

b) A more interesting aspect of this is why it even appears here at all.  Every place it’s used, the code looks like this:

#ifdef BOOST_NO_CXX11_HDR_CODECVT
    #include <boost/archive/detail/utf8_codecvt_facet.hpp>
#else
    #include <codecvt>
    namespace boost { namespace archive { namespace detail {
        typedef std::codecvt_utf8<wchar_t> utf8_codecvt_facet;
    } } }
#endif

I read this as only using our utf8 codecvt facet only the standard one std::codecvt_utf8<wchar_t> does not exist.
I believe that recent versions of MSVC do include this standard header so this issue should never arise in any case.

Clearly I”m missing something here.  Since I’ve already got you sucked into this, I’d like to hear your comments

Robert Ramey
Robert Ramey Software Development
(805)569-3793
ramey@rrsd.com
www.rrsd.com


Thanks.


lol, sorry, I did not test it


I guess this is a mistake??


Finde ich sehr gut!


@tancredi I'll defer to you as you are the expert here, and I'm not really good with NPM so this wasn't something I thought of. We can find a proper replacement Friday?


We moved the API tests?


no prob. we’ll go though it together

On Tuesday, 20 October 2015 at 16:27, Alec Molloy wrote:

> @tancredi (https://github.com/tancredi) I'll defer to you as you are the expert here, and I'm not really good with NPM so this wasn't something I thought of. We can find a proper replacement Friday?
>
> —
> Reply to this email directly or view it on GitHub (https://github.com/KanoComputing/make-art/commit/7960f3462d70dafa11391c17e19f19fcaf8648cf#commitcomment-13878488).
>
>
>
>
>


Hasn't this been tested before the move was made?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.539.3144afed434d0771ba857e491ff8b8b4802d5173](http://build.palaso.org/viewLog.html?buildId=64607&buildTypeId=bt223) outcome was **FAILURE**
Summary: Tests failed: 2 (2 new), passed: 3323, ignored: 140 Build time: 00:07:21
### Failed tests
```
PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.WinKeyboardDescriptionIClonableGenericTests.IClonableGenericTests`2.Equal_OneFieldDiffers_ReturnsFalse: <no details avaliable>

PalasoUIWindowsForms.Tests.dll: PalasoUIWindowsForms.Tests.Keyboarding.WinKeyboardDescriptionIClonableGenericTests.IClonableGenericTests`2.Equal_SecondFieldIsNull_ReturnsFalse: <no details avaliable>

```


no newline at end of file


closes #16


@plugz see this: ca3ca186cd6f7b138d10f87bdf764063bd6f0061
I benchmarked the whole init function on the Raspberry Pi and it took 1ms or even less.
I should do the same on Windows and OSX to be sure, but for now this is the solution.

I tested the network bearer example and it could work for us.
http://doc.qt.io/qt-4.8/qt-network-bearermonitor-example.html
However I wouldn't use it inside plugins (lot of redundant code otherwise) but inside the IOPluginCache class, by adding a new capability to plugins like "Network".
So one single bearer that informs the "network" plugins when the system interfaces change.
I don't really have the energies to do it now, but let's keep this in mind for the future


'Any failure occurring at this stage will cause the operation to fail'
Same below


Hi, you have to remove a reference reg_lcd0_backlight from other files or just remove only gpio = <&gpio1 4 0>;


I think the API requests were failing and we were trying to roll out the profiles quickly. I can throw that stuff back in.


Look up, its still there line 40 in green.


We might not need the `.text` wrapper as well as the 2/3 column. GOV.UK elements uses `.text` on its full width pages to prevent lines of text which aren't inside a grid column from getting too long.


Hey, remind me, what's the value of boolean args?


Have you considered using [send_multipart](https://zeromq.github.io/pyzmq/api/zmq.html#zmq.Socket.send_multipart)?  You will still need to use the `array_split`.


leave it--ill put it back in..focus on the recipients!


Thanks! Really like these changes :+1:

One more question - do you think the hash maps ought to have some kind of human readable field that explains **why** a submap exists?  I mean, computers won't really care why there's a submap but it seems like a human operator or support person might care about that.

Maybe that's an implementation detail that doesn't need to go into this document though.


Could you change this to "Node Core Contributor at NodeSource"? Thanks. :)


fixed that, thanks


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


I think for simulate, generating many samples simultaneously is more natural than evaluating the pdf at many samples simultaneously.


Ok, I understand, thing is `isImplemented()` really only makes sense for functions and values, for classes it would depend recursively on all of *its* members, etc. But right now at least you're saying that an inner native class can never be implemented and that's not correct at least.
I'll think about it some more and hopefully come up with a better answer :)


čo ????


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.516.3144afed434d0771ba857e491ff8b8b4802d5173](http://build.palaso.org/viewLog.html?buildId=64608&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3365, ignored: 134 Build time: 00:10:57


I think that this one is your first float that was throwing you off. I don't believe that you actually need this one. And when you delete it, you can delete most of the ones following it and you get to keep your margins :)


@agilob: From where do you have this information?
These return codes are from the OCID Wiki and can all be returned.
Please read the comments.


Fixes https://github.com/ARMmbed/uvisor/issues/65


你index.html文件里好像有php代码！


OK, pushed another fix - please give it a try and let me know if it worked or not.


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 127](http://teamcity/viewLog.html?buildId=1553&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 5 Build time: 00:01:02


Meo, tá show. Podia usar as funções do array em algumas partes, concorda?


Suprise!


Can be replaced with simple if else statement.


Every attribute which has getters/setters can be made private


There is no need for this statement actually. If `elements.length` is zero, the for loop below would not execute. Or you should change this to
```java
       String[] elements = cp.split(File.pathSeparator);
        if (elements.length == 0) {
            elements = new String[] { "" };
        }
       URL[] urls = new URL[elements.length];
```


Are you using two properties for ParticipantObjectID ? I dont think it is required as both values are same in this testcase.


dddd


**M**ain file ....


what a trick. lol


The class name should be `Animals`, as each instance models one single animal, not more.


@Thibaut Thanks! The `id` attribute works for me now, I didn't test the `<a name=>` anchor yet.


Ето това означава да имаш едно на ум!


known issue after this: bots repeatedly go back to last corpse until corpse disappears or player force them to another action.


My first pull request!

/cracks champagne


```erb
<li><%= link_to 'FOI Awareness', help_general_path(:action => 'foi_awareness') %></li>
<!-- Assuming file a complaint should go to /help/the_iic? -->
<li><%= link_to 'File a Complaint', help_general_path(:action => 'the_iic') %></li>
<li><%= link_to 'Partners', help_general_path(:action => 'partners') %></li>
```


Instead of passing them as parameters, you could have them as fields, instantiated in the constructor of this class.


I'd strongly recommend a getter/setter and making them private


Punctuation would be appreciated on this list. We should get another WG member's +/- on this if you don't agree.


Good work so far


I guess the ilab site is http://ilabliberia.org/?


Bullet?


No it will not work yet, I am still working on it, I have to do stuff on the bot too.


Why set to `FALSE`?


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 128](http://teamcity/viewLog.html?buildId=1556&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 5 Build time: 00:01:00


now only for admins... without that link you have to click twice to see userdetails and smurfs


Dont worry i just note that by accident while playing arround;) nevermind


What did removing this line of code do?


@vcunat I can't really remember why it was `hydraPlatforms = []` but it's unfree nevertheless (see the line.
The output is not really very large -- ~110MB.


I'm wondering if it's ok to put the global variable in the last line...


@gergoerdosi I don't how you run tests, so i used console.log, to test final value.
how you execute test suite ?


See i told you @ratexindex he was still working on it.


Can? I'd reword this to "Usually," as major releases are, at least in my experience, almost always breaking.


commiting


@vincentchabannes Just to inform you that I added the `install` command for the feelpp_modelalg target. the library was missing when I tried to compile quickstart after installing feelpp.


You should avoid identifying the implementations as "version 2" or "version 3".


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


Susu bon release mdr


Instead, keep single functional change as possible.


What does this mean? Through what channels is the API identified?


Okay I see, here it is initialized...


holy! do we really use all of these defines?


Looks like seven ate nine.


@louy, my philosophy behind the theming stuff was that themes should always exist in `this.state`. That is, if a theme is passed down to the component using `context`, that theme should be assigned to `state` in `getInitialState()`. Otherwise, a default theme should be assigned to the state in `getInitialState()`. Based on this, do you think it would be better to frame this line as:

```js
return Styles.prepareStyles.apply(Styles, (this.state && this.state.muiTheme).concat([].slice.apply(arguments)));
```

If we do this, #1922 would have to change to use `getInitialState()` inside `clock.jsx`.

To me, this is more of a style convention -- if any component doesn't expect a theme to be received, make it independent by defining a default theme and storing it inside `state`. Even if a theme is passed down, store it inside `state` and always access it from there (instead of `context`). More precisely, it is okay for `state` to be initially populated using `context` for the theme, but after that, `state` is the single source of truth for the theme.

This is what the `getInitialState()` looks like:

```js
 getInitialState () {
    return {
      muiTheme: this.context.muiTheme ? this.context.muiTheme : ThemeManager.getMuiTheme(DefaultRawTheme),
    };
  },
```

Following this approach should also eradicate future errors of `context.muiTheme` not being defined. Thoughts? Also, @oliviertassinari


I see that's what it said in the original version - I would have noted it there as well.


Oh, I think I see why you used it. So that the `stop` function would not close our UdpSocket. But should setting the state to `Closed` be enough? [This line](https://github.com/meqif/rust-utp/blob/88a3aa6410f52eac1a228790c4297b650c1d3274/src/socket.rs#L325) suggests that if the state is `Closed`, nothing happens to the socket.


Which team? TSC? WGs? Individual members?


Nope, should we get rid of it? does kind of suck that it would fail hard if it were to ever break


ok


indeed !


What is this suppose to fix?  see belledonne fixes on my branch.
Shareef


Students/Others/Pop I. Andrei/Assignment 3 - Java/src/javasmmr/zoowsome/services/factories/AnimalFactory.java

Hah!! This actually made me laugh :laughing:

So, if you want to simply throw an exception, just.. throw it! Don't..catch it while you throw it :)))

Simply writing `throw new Exception("Invaild species exception");` will be enough.
Yes, you will have to add `throws Exception` to the function declaration, but this way, you are not really doing anything with the exception except, well, catching it yourself :smile:

I'd strongly suggest you take a quick look over some exception tutorials


Is the modification of the `instanceCount` done atomically? Otherwise it could be corrupted?


It looks like the FullPathLength computation above is wrong, It doesn't account for the `/` we are adding between those two.


Students/Others/Pop I. Andrei/Assignment 3 - Java/src/javasmmr/zoowsome/services/factories/AquaticFactory.java

bad formatting


Looks like the other "/" should be removed as well..


No, I just put defines for all the registers in the QuarkX1000 datasheet. But I can remove those which I'm not using. What do you think?


Students/Others/Pop I. Andrei/Assignment 3 - Java/src/javasmmr/zoowsome/services/factories/InsectFactory.java

same as the above :laughing:


Sorry @gaearon for I'm late.
Unfortunately I started this project but it is unfinished and abandoned because I had to change focus.
The point is to modify `gitbook-plugin-anchors` to wrap the whole title in an anchor, not just the anchor icon when the mouse is over.
FYI I may come back to this project within a week or two.


So you didn't notice his question can't be taken seriously?


Thanks for this code! I'm trying to use it to create and validate hashed passwords. Everything works great except that I can't use a header to send the user to the next page after a successful login because I get the message "Cannot modify header information - headers already sent by (output started at /usr/home/asphn/public_html/include/passwordLib.php:1) in /usr/home/asphn/public_html/check_login.php on line 156"  PasswordLib.php requires one time passwordLibClass.php. When I look at these, I can't figure out why they are sending headers. Any ideas? I'd love to have this work on my site but can't if it prevents me from using headers to redirect visitors. I'd really appreciate any help.


with the __exception__ (pun intended) of the small suggestion on.. exceptions, good work


fixt


Can you please check your checkstyle/formatting settings?  It's strange that all of your commits and contributions contain spacing changes.

< review


Does order matter here? Like if InternalUrlModule gets configured before the SLF4JLoggingModule is that a problem?


This sentence has problems, I think this would be better: "If an error occurs, the operation will still be treated as a success but some garbage files may be left over in the Linux file system".


A new commit, updated this commit with new code. Took some inspiration from FlexiblePublish.
Added endBuild and endRun. Called ProxyPublisher perform within endRun. I have not got the foggiest what to do with startBuild and endBuild.

The MatrixAggregator should call all the publishers from the defined project in ProxyPublisher. Since I do not want to duplicate its own perform, I call the ProxyPublisher perform on the MatrixRun within the aggregator.


Lovely work, nothing to object


fixed? :open_mouth:


Should the stuff below be done only once, e.g. only if `instanceCount == 0`??


`--verbose` shortcut doesn't clash with that of `--version` anymore


> does not take the decision lightly

That's weird wording. Maybe "Adding a new API to Node.js is an important decision, and is considered deeply by the core team"?

Also, what is the core team?


Students/30421/Goia I. Paula Ioana/Zoowsome/src/controllers/Constants.java

Aha. Having these type of curly brackets - not Java standards!


Students/30421/Goia I. Paula Ioana/Zoowsome/src/controllers/MainController.java

Also, apparently you're not being consistent with those curly brackets


Are there possible race conditions where we could get an orphaned thread pool?
```java
private transient Object poolLock = new Object();

private static Future startCallable(Callable callable){
    if(cachedPool == null) {
        synchronized (poolLock) {
            if(cachedPool == null) { // make sure we got the lock in time
                cachedPool = Executors.newCachedThreadPool();
            }
        }
    }

    return cachedPool.submit(callable);
}
```


Yea, so, except for the curly brackets, nothing to object. Good work


Sure, some extra info to help human managers makes sense.  we can rub a
proplist on it, if nothing else.


If you are targeting the first letter of running text, you could add a `<span>` tag (for example about the initial letter. Then you could style this letter individually and make it stand out.


@joergoster but if so, how come bench is different?


"Some questions they ask are:" makes more sense to me.


I really like this format as well. I plan on formatting my site as one page with jump links to specific portions of the page. Maybe I'm just boring--but I prefer really straight forward layouts.


I'd like to see all the imports kept explicit.


Hmm... we put the `import_readline` in the end of file. So ... `readline` must run after the function declaration.


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1893 of 1897 tests passed)

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


Hum, indeed, it's strange


Ah, I commented on the version with `mit` in it :-) I'm not certain whether Hydra.nixos.org policy is to only allow this for firmware/drivers, but we'll see with `hydraPlatforms` not set.


"If this API is overwhelmingly in use" is, again, a bit of an odd way to word it. "If this API is widely used" would be better.


futilityValue is tested at pseudo moves.


How did you pick this version?


**\**param


Awesome specificity!


Ah crap... Sorry, seems i was in the wrong tab when digging through the
code... Ill cut-paste it later when im home ;)
Am 19.10.2015 10:00 nachm. schrieb "Malware" <notifications@github.com>:
>
> @ChrisonSimtian I think you would be better served commenting on the
official branch rather than in my fork... I doubt they'll see this. What
you're looking at here isn't my code, it's theirs.
>
> But yes, it looks like a duplicate.
>
> —
> Reply to this email directly or view it on GitHub.


Yes. I have writing problems, because I have had a brain damage


Very nice job overall! I really like how clean and easy to read your code is.


After further consideration, I think probably all of the above unit types should be dropped, except power_unit_t, since they don't have any specific rules yet. Thoughts?


Reverse "adding" and "deprecating". The rest of this section is about deprecating, and adding is previously addressed, so they should be in the order of "deprecating" -> "adding".


Maybe set `plkDriverInstance_l.pDrvDeviceHandle` to NULL after dereg?


This will be the final state at the end of my series.
On Oct 20, 2015 4:25 PM, "u2yg" <notifications@github.com> wrote:

> This line would look nicer over here: log_pipe_unref(&pclone->super);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balabit/syslog-ng/commit/5a959720a8ac00fa5834b4da11bc6794b0bbb678#commitcomment-13876367>
> .
>


Alterei o código usando sort, filter e join.

segue:

function queroCafe(n, array){
  function ordenaDec(a, b){return a > b};
  function filtraMenorIgual(a){return a <= n};
  return array.filter(filtraMenorIgual).sort(ordenaDec).join(',');
};


Whoops, missed it :) Thanks.


Congrats! - hopefully the first of many :-)


o Git não pegou a indentação, mas o código está indentado


Also here, what value should be set depending on `timeInMs_p` 's sign??


copy_deep() is only available since glib 2.38. Are you sure we should
introduce that dependency?
On Oct 19, 2015 6:42 PM, "Budai Laszlo" <notifications@github.com> wrote:

> What do you think about using g_list_copy_deep ?
>
> In that case the implementation could be a single line:
> g_list_copy_deep (string_list, (GCopyFunc) g_strdup, NULL);
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/balabit/syslog-ng/commit/a6c9b87cc659d510e57008c22510928879411260#commitcomment-13853400>
> .
>


Your fork is behind master quite a bit. The new code doesn't do that at all anymore.


The fact is that I didn't want to send the stacktrace, because it won't contains any interesting information.


Se eu não der o crédito para o Tefferson ele vai me bater.


Please provide a "user has provided 1 arg or arg 1 isNaN" execution path (in other words, this "else"), otherwise users will not get any feedback when they use "!raid <NaN>" or "!raid <NaN> <something>".


I have no idea I just stole this from @pbrisbin


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1416 of 1416 tests passed)

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


I don't understand your second comment. The goal of the goal is to display useful information to the user, in order for him to understand why highlighting won't be displayed.


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


Endorsement or not from the Python community, I like @luac's suggestion.


alterei o nome para  "textos"


corrigido.


Hueheu, capazz. Conhecimento é pra ser compartilhado hehe


Should we move this to tabletserver?


Usei if(petalas.constructor === Array) para iniciar a função, caso não seja um Array a função não irá fazer nada. Essa verificação é uma boa saída?


TeamCity libpalaso :: palaso-win32-master Continuous [Build 2.6.540.199167cea54ae7082b639a98c3eabc2c8e1901ee](http://build.palaso.org/viewLog.html?buildId=64609&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3325, ignored: 140 Build time: 00:04:33


Cala - I can help describe the logic that is causing the issue.  Assume: 3 toon group - player, bot1, and bot 2.

Case 1: Log in as player, group  bot1 and bot 2.  Kill one mob (and hope there is more than money on the mob).  Do not loot it -- bots will loot the money, which gets distributed ... and then will sit there and wait until you loot it or it despawns.

Case 2: Log in as above - kill fast without looting so there are many corpses.  What can happen is, based upon the round robin logic, a deadlock will form -- where bot2 is standing on your or bot1's allocated corpse, while bot1 is standing on your or bot2's corpse.

So - logic may need to ignore a corpse until a) has been unlocked (look or partial loot) or b) fully looted and ready for skinning et al.


"Conhecimento é para ser compartilhado, créditos não." Linspector, Tefferson.


is there anything else you can do beside 'sleep' to fix the race? :)  What are you waiting for here?


looks good


empd (nmerket) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I think this should be a boolean flag to the `initialize()` method instead of a setter. You're setting state of a service here…


I was uncertain about this; the R-devel version of WRE is silent on this new issue but contains the usual statements that all packages that you import from need to be in Imports or Depends. I went with belt-and-braces, but can get rid of this `Imports` tag.


This was pushed directly to master? =/


Thanks for the message.  I am new to DFS, never played a game but think I
mentally have come up with a decent model base on portfolio theory.  Here
is a summary of what I am trying to do for 50/50 games.  Let me know how
feasable you think this is in R

Step 1- Generate Lineups of all possible teams above a point threshold
(this is where I was going to use your model)
Step 2- Simulate games based on expected points, as well as expected SD and
player Covariances (This will allow me to calculate standard deviation and
covariances of teams)
Step 3- Maxamize a portfolio of a specified number of teams ( for this I
was going to use
sum(TeamWinnings*probablity of win)/portfolioSD

Like I said I am new to DFS and never played a game yet.  Thought this
model would be a good one and possibly produce consistent winnings.  I am
also fairly new to R, used it in college but haven't use it in 10 years,
but through it would be very helpful in the data analytics and webscraping
of data.  I would love to hear your thoughts on my system and any help you
are willing to provide would be greatly appreciate.  I also have other
models I have thought of for GPP games but wanted to get the 50/50 & Double
up model going.  Was also going to extend this model to Basketball and
Baseball once I got it up and running.  Thanks again for sharing your work,
it helpful to go through the code and see how it work.

On Mon, Oct 19, 2015 at 2:46 PM, geekman1 <notifications@github.com> wrote:

> Thanks for the compliment! If some players are undervalued, it is likely
> they will be in many of the top lineups. It depends what your idea of
> variation is. You can force it to remove a player after they have been in a
> certain number of lineups. See the edited function below. Now, if a
> player/team has been in two lineups already, they are removed. A simple
> alternative is to remove players manually from the 'train' file and then
> re-run the function. I'm not sure if there is a best solution to this issue.
>
> top_teams <- function(train, num_top, cap, constraint, setplayers = NULL,
> pointmax = NULL){
> result <- find_teams(train, cap, constraint = constraint, setplayers =
> setplayers, pointmax = pointmax)
> j <- 1
> while(j < num_top){
> result <- rbind(result, find_teams(train, cap, constraint = constraint,
> setplayers = setplayers,
> pointmax = (result$TotalPoints[nrow(result)] - .001)))
> duplicates <- unique(result$Id[duplicated(result$Id)])
> train <- subset(train, !(train$Id %in% duplicates))
> j <- j+1
> }
> result
> }
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geekman1/fantasylineup/commit/e1d644fe967c5ab18ba7a1a086ac91bacc1fa912#commitcomment-13856308>
> .
>


Added note on Jupiter's moons


opa, acho que fica mais simples usar o filter() nesse caso.


Nesse commit eu havia usado o sort no exercício queroCafe().. Porém em um novo commit utilizei o sort com filter e join para melhorar a performance.


Does electron support in-place updating?


no need for a build_depend on romeo_description


I would put the registration/session routes in a different file. Generally reserve the index.js for just setting up your app. When signup/sign in becomes more complicated, it is nice to isolate that stuff elsewhere.


Use `%w` or `%W` for array of words.


Thanks. Fixed.


Extra four spaces -- otherwise this looks like the start of the body.


probably best to just squash this commit into the last one (for upstream merge at least).

I haven't tried the launchpad git integration yet.  We could use this as a test of that, but the simplest way to get this into lp:vmbuilder would be for you to git format-patch here, then apply each patch as a separate bzr commit to a bzr checkout, push and propose that for merge.


run_depend on romeo_description is already included below


Yeaaah, I think we should decide on this one https://github.com/Wikia/guidelines/pull/99/files#diff-085461842ea3b6fedd3df3f589e07715R485 ;)


I'm not happy with the solution at all. See https://github.com/contao/core/issues/8080 and my comment at https://github.com/contao/core/issues/8080#issuecomment-149573030. IMHO we should add a configuration option for this just like `prepend_locale`.


How about , while you are adding functions, one to give the next (X) appointments. So I can list the next 5 appointments. These may or may not be recurring appointments.


Conventionally this would probably be `/students`, but the code the implementation is :+1:


This received from Mike Hulsman @ nluug:

I got some errors when syncing from ibiblio.
receiving incremental file list
rsync: send_files failed to open "/pet_packages-4/nohup.out" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_agrmodem-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_brcm-20131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_carl9170-4.1plus_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_dgcmodem-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_ess-20131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_hcfpcimodem-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_hsfmodem-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_intel536-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_intel537-20131209_CE.pet"
(in puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_martian-20131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_slmodem-20131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_linux_module_wl-20131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-common/firmware_module_b43-131209_CE.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-slacko/PupSnap-1.6.2_Scrot-0.8_32Bit.pet" (in
puppylinux): Permission denied (13)
rsync: send_files failed to open
"/pet_packages-slacko/zz_shutdownconfig-20110926.pet" (in puppylinux):
Permission denied (13)

Seems that permissions on ibiblio are not right.
rw-r-----          3,931 2013/12/12 09:36:09
firmware_linux_module_agrmodem-20131209_CE.pet
-rw-r-----        773,360 2013/12/12 09:36:13
firmware_linux_module_brcm-20131209_CE.pet
-rw-r-----         10,020 2013/12/12 09:36:14
firmware_linux_module_carl9170-4.1plus_CE.pet
-rw-r-----         30,450 2013/12/12 09:36:14
firmware_linux_module_dgcmodem-20131209_CE.pet
-rw-r-----          2,130 2013/12/12 09:36:15
firmware_linux_module_ess-20131209_CE.pet
-rw-r--r--        128,309 2012/10/15 03:42:34
firmware_linux_module_hcfpcimodem-20121004.pet
-rw-r-----        176,154 2013/12/12 09:36:16
firmware_linux_module_hcfpcimodem-20131209_CE.pet
-rw-r-----        101,669 2013/12/12 09:36:17
firmware_linux_module_hsfmodem-20131209_CE.pet
-rw-r--r--          6,356 2012/10/07 18:55:30
firmware_linux_module_intel536-20121007.pet
-rw-r-----          6,424 2013/12/12 09:36:18
firmware_linux_module_intel536-20131209_CE.pet
-rw-r--r--          6,715 2012/10/07 18:55:52
firmware_linux_module_intel537-20121007.pet
-rw-r-----          6,790 2013/12/12 09:36:19
firmware_linux_module_intel537-20131209_CE.pet
-rw-r--r--        190,323 2012/10/22 10:09:48
firmware_linux_module_martian-20121021.pet
-rw-r--r--        194,224 2013/05/16 01:15:08
firmware_linux_module_martian-20130514.pet
-rw-r-----        194,225 2013/12/12 09:36:20
firmware_linux_module_martian-20131209_CE.pet
-rw-r--r--        867,756 2012/10/06 23:20:11
firmware_linux_module_mwave-20121004.pet
-rw-r--r--        613,047 2012/10/14 01:44:10
firmware_linux_module_slmodem-20120930.pet
-rw-r--r--        612,335 2012/10/06 23:43:28
firmware_linux_module_slmodem-20121004.pet
-rw-r--r--        611,268 2013/02/11 01:28:12
firmware_linux_module_slmodem-20130210.pet
-rw-r--r--        620,635 2013/05/20 12:28:20
firmware_linux_module_slmodem-20130519.pet
-rw-r-----        620,642 2013/12/12 09:36:23
firmware_linux_module_slmodem-20131209_CE.pet
-rw-r-----          1,413 2013/12/12 09:36:23
firmware_linux_module_wl-20131209_CE.pet
-rw-r-----        119,549 2013/12/12 09:36:36
firmware_module_b43-131209_CE.pet

Regards.
Mike Hulsman


why do we still have ```Pr2MoveItSensorManager``` ?


Perfect!


@jacekjursza @fraszczakszymon I almost forgot about this >.<


I would probably keep these (although i havent really looked at their accompanying spec)

type power_unit_t;
This is needed by various processes to shutdown

type systemduser_unit_t;
if this is "/usr/lib/systemd/system/user@.service"  then this is started by logind, so this should have a private type


Could you please put `MIN_SHOULD_LENGTH = 5` at the top of the file, since this is a constant and a magic number.


Please just add a comment on top that describes what this file is doing, thx


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 2.6.517.199167cea54ae7082b639a98c3eabc2c8e1901ee](http://build.palaso.org/viewLog.html?buildId=64610&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3365, ignored: 134 Build time: 00:09:39


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

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


1. use underscores to transform into _italics_

2. use asterisks to emphasize a portion of a w*or*d

3. headings
 # the largest heading
 ## the second largest one
 ###### the 6th largest one

4.
	> Blockquotes


Remove closing parenthesis (no matching opening one).


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


Why did the trace in the tests change?


Could clean this whitespace up a bit


What was wrong with the previous pattern-matching?


why is robot_state_publisher removed from anywhere ? (maybe it's right, I just don't know)


Oops, I thought we are at step 7. Sorry.


Let me guess. Since nonexistent const gets imported as undefined and the reducer handles only one type of action, it is a way for this to pass.


ü


> I read this as only using our utf8 codecvt facet only the standard one
> std::codecvt_utf8<wchar_t> does not exist.
> I believe that recent versions of MSVC do include this standard header
> so this issue should never arise in any case.
>

It appears it's buggy in VC14, we have:

// Bug specific to VC14,
// See
https://connect.microsoft.com/VisualStudio/feedback/details/1348277/link-error-when-using-std-codecvt-utf8-utf16-char16-t
// and discussion here:
http://blogs.msdn.com/b/vcblog/archive/2014/11/12/visual-studio-2015-preview-now-available.aspx?PageIndex=2
#if defined(_CPPLIB_VER) && (_CPPLIB_VER == 650)
#  define BOOST_NO_CXX11_HDR_CODECVT
#endif

In Boost.Config.


А не лучше сделать .pro файл для этой XML библиотеки в 3party?


The remainder could probably, at least for now, be removed. also for simplicity.


These files are stubs that has to be there because npm complains when creating symlinks if there aren't any files there yet. Once the module is installed completely the placeholders will be overwritten when extracting the .tar.gz stream.


Sorry, this line should read `properties (Constant = true)`


I don't think so, serving update streams is not really related to query service...


I get the same error on Py2.7 / Win7 / virtualenv / pylint 1.4.4 / logilab.common 1.1.0
@bjones1 Do you use virtualenv? Could you try this example that it freezes and runs successfully?
```python
import pkg_resources
print(pkg_resources.get_distribution('logilab-common').version)


Why would a user need to update in place?


Boa!!


So once again, why the heck we use constants in a dynamic language? As we see here, it doesn’t prevent errors at all…


I see, ugh.  I'd rather split this into two different functions.


s/venu/venue/


I created a branch for batch downloading, because I thought you may not need it.
I can start a pull request, and you can check if you want all the functions I added.


Пустая строка. Перед SaveToFile. Реализацию Instance() тоже лучше в .cpp.


This does not look correct.  ImplementationClassUID is File Meta Information. I don't think, that we should add it to the normal dataset.
When writing such a decompressed dataset, we should re-write the ImplementationClassUID in the FMI anyways.


Please, sort lexicographically.


Ok, after reading your other comments I see what you mean. I don't like the fact to mock the behavior of the converter as it's very close to the business of the reader, but I agree that it will simplify the test.
I will give a try.


Log message about error, please.


> TO-DO:
> Use the local storage cache for data state preservation and export the file instead of text file to CSV file.
Could you please explain the difference between a text file and a CSV file to me?


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


That's one way of sugar-coating that defect!  :-)


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


opa, acho que tu consegue simplificar o código da 3 usando só o filter()


oh yes;)


You interrupted the inline env vars (that don't need &&) with a logic block. They aren't export lines, they only apply to the command that follows them.


This would lead to a race (which I think go race detector should catch). You need to RLock() to read the value


Oops, not used to the web editor, didn't mean to commit directly :(


czemu tak?


opa, tu acabou fazendo a mesma coisa que eu, não precisa somar por mundiais, e sim por todos os titulos.


`Info_gen`, not `Config_info_gen`.


Thanks; I've committed this to trunk.


What do we gain at making this parametrized ? Do you have a use case ?


Hi, Please add ``CONFIG_BACKLIGHT_GPIO=y`` to config file.
Without gpio-backlight driver you will have blank screen.


"contains"


This is bad ... not what you did, but, that the serverside needs all 4 fields. We only have account number and BLZ for German bank accounts.

Other countries most work with IBAN/BIC only. Its no blocker for the moment since we rarely get donations from other countries bank accounts. But we need to fix that anyhow in the next iteration.

e.g. BE68539007547034 / CITIBEBXXXX should work


Closes #4


sss


Good ,Let me have a try too !


I see :-) Go ahead then :+1:


def piece exists?  looks like when we replaced the x it put ROW in here.


cara, tenho quase certeza que não precisa ordenar por nome e sim por quantidade de títulos.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.693](http://teamcity.monogame.net/viewLog.html?buildId=10729&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 541, ignored: 6 Build time: 00:19:42


TeamCity pull requests :: netty [Build 444](https://garage.netty.io/teamcity/viewLog.html?buildId=2534&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 4785, ignored: 21 Build time: 00:31:55


Ok, thanks, I'll make that the default.


excellent work whoever managed to edit this file and save this function!


In the Palm of the Hand

Kann man das nicht auch: "In der Hand"
übersetzen?
Habs mal durch paar übersetzer gezogen und bei einigen kommt: "In der Handinnenfläche der Hand" herraus. Da ich diese schreibweise persönlich etwas komisch finde, würde ich für die Kurze form tendieren.
oder liege ich da nun irgendwo falsch?^^


It would make more sense to do this before calling ListFolderEntries I think - i.e. make the list start with "..", and then ignore any entries coming from the callback which are ".."


> [returning an error if the corresponding channel has hung up](https://doc.rust-lang.org/stable/std/sync/mpsc/struct.Receiver.html#method.recv)

The channel hungs up when the other end is closed. The other end is closed when the thread finishes. Therefore, it does not block indefinitely. Rust abstractions are safer than you think, I guess.
=P


Actually I thought that by using CSV files I can filter the data later by using dataminer. Last time me n saimadhav and min thought we can use a dataminer tool ....previously i was making a file with .txt extension and same problem was there ......to be honest i wanted csv so that i can use it later for data ....... Am i doing something wrong? I am confused becoz I am almost there...


I've run into the error as well, we should probably add this.


I did put a htaccess that disallowed access to the config.ini file for security purposes but I don't think having it as a config.php is more secure. Unless you can elaborate?


If we're going to mark these lines as `html_safe` in order to display the `<br>` tags correctly, then I think we do need to have an `h` call on the value before we call `gsub` to make sure that it doesn't contain any dangerous markup.


For the `typ_modgen` yes: I don't want Mirage users to call functions from functoria (so they should use `Mirage_info.pp Info_gen.info` instead of `Functoria_info Info_gen.info`. For the `Info_gen` I'm not sure anymore...


When the other end finishes, `packet_tx` gets dropped and this `recv()` here will return an `Err`. It's that simple.


Just leaving a comment for now, will doublecheck this in the morning: server-side validation does not require to specify the "Kontonummer" and BLZ for non-German donations. That would make no sense, as you say. In most countries there is even no such thing as BLZ, I believe.
At least I recall having it in mind and writing some tests related to this thing. If it does not working it'd mean that server-side validation is broken in this part.


Same comment as above.


嗯，需要php环境。


So if this replaces `PATH`, I'm not entirely sure this value is adequate.  I would think that we also need `C:\Windows\system32` and `c:\windows` at the very least (so that `cmd.exe` is in `PATH`).

When starting the Puppet command prompt, the following paths are in `PATH` that *could* be relevant.  I think we should shoot for the bare minimum, but I'm not entirely sure what is / is not necessary here for pxp-agent.  When the `pxp-module-puppet` plugin calls `environment.bat`, it will get most of these additions from Puppet anyway.

```
C:\Program Files\Puppet Labs\Puppet\puppet\bin
C:\Program Files\Puppet Labs\Puppet\facter\bin
C:\Program Files\Puppet Labs\Puppet\hiera\bin
C:\Program Files\Puppet Labs\Puppet\mcollective\bin
C:\Program Files\Puppet Labs\Puppet\bin
C:\Program Files\Puppet Labs\Puppet\sys\ruby\bin
C:\Program Files\Puppet Labs\Puppet\sys\tools\bin
C:\Windows\system32
C:\Windows
C:\Windows\System32\Wbem
C:\Windows\System32\WindowsPowerShell\v1.0\
```

So my vote would be to also include:

```
C:\Windows\system32
C:\Windows
C:\Windows\System32\WindowsPowerShell\v1.0\
```


Gotta love Aeson and Haskell typeclasses...


Looks like that's fixed. Okay, these are now activated on EOL:
http://purl.obolibrary.org/obo/RO_0002627
http://purl.obolibrary.org/obo/RO_0002626

Thanks!


Off by one? This should be 0.20, right?


1907-TerminalOASpec (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Two business days? That will kill 90% of the bugs in the repository, and will probably make a lot of users angry.


Just in case.
=P


I rarely use it, but ghci has an `:edit` command. This setting might even be superfluous if ghci respects `$EDITOR` or `$VISUAL` for this anyway.


In fact no, as I've replaced this Exception by a custom one in order for the caller to catch only this specific error.


This isn't a bad idea but I don't think it is really useful. From a user's point of view having another related list on a module that just shows internal counting information is not useful. I would eliminate the whole section and not add related list to the other modules.


If we added a timeout, we would also need to figure out some reasonable response to failure.  By the way, I have not yet observed the Ethernet MAC fail to process a buffer.


It's available since version 2.34 (https://developer.gnome.org/glib/stable/api-index-2-34.html#api-index-2.34) (I remembered 2.32, but I was wrong... and no, we shouldn't depend on that version) .


:sunflower: :sunflower: :sunflower:


What's a reasonable number?


The usual comparison semantics don't apply to NULL values. E.g. `"Attribute" = NULL` never yields TRUE, even if the attribute is NULL. It always yields NULL.

So at least [here](https://github.com/qgis/QGIS/blob/441f6f8cf1c5a500c3e898bfbcdff6757e6d8d73/src/providers/postgres/qgspostgresprovider.cpp#L399-L410) would be a problem. Maybe somewhere else, too.

See also my newly filed [bug 13641](http://hub.qgis.org/issues/13641).


shouldn't it be 0.20? it is deprecated in 0.18 and therefore removed in 0.20, right? two versions?


should update the demo project as well, or add it to readme


I am sorry, I have missed the fact that the OpenStringBuffer adds one to the FullPathLength. So that's correct.
But the second parameter to the strcpy_s is off by one, instead of subtracting `PathLength`, you need to subtract `PathLength + 1` to account for the `/`


Indeed `HandlerResult` was meant as a container for the return value plus any other additional information such as a ResolvableType for the handler return type. `HttpHandlerAdapter` can easily supply that (it's fixed actually) and the `RequestMappingHandlerAdapter` already deals with the return type too. Any other HandlerAdapter's can provide it too if applicable, or if not available then the `SimpleHandlerResultHandler` can pass and let something else resolve it.


Either I'm making off-by one errors in both directions or I'm not the only one making them ^^


I think there should be a best effort in trying to find someone to procure the hardware, since users can be unreliable, and if you can just update the phone to fix the bug, then why are we bothering to fix it?  The user should update their device anyway to prevent their device from being vulnerable.


I think the below deleted method has something to do with bonemealing - don't really remember.


This is incorrect, you cannot insert directly into the table, you must to full "save"s for each one, like you do in CRMEntity


Can you change the "<" symbol in the javadoc another description (maybe in words)? This causes an error when running a Maven build (Maven thinks it's a HTML tag).  < documentation, configuration


http://qiita.com/kkabetani/items/e2f2675dd6e3c2212c1b
これ参考にしてみてください


Why don't you guys add this to the CoreFx repo?


Why, why... WHAT IS THE REASON TO DO THIS...


Why? This can be done without a read lock & a check in line 38 after getting a lock. Just a small optimisation but should work.


My homeworks


Does not compile on Visual Studio 2013, does not know "_beginthreadex_proc_type"


I see. Alternatively, we could re-export the functions (and the type) directly inside `Info_gen`. If we re-export in mirage all the thing that are generic in functoria, it's going to be annoying if we add new ones.


@Mpdreamz thoughts on this?  Also, I think this could replace ReadAsTypeConverter entirely?


Blockers right now mean that the project doesn't build, or the failure is catastrophic (i.e. We broke Hello World).  At a bare minimum, the project has to build and we have to be able to run Hello World and MobileSpec for it to not be a blocker.


As a library user, I wouldn't expect the length to be randomized. My assumption would be that the field is just masked, like it is whenever you type a password field on pretty much any site/app/UI. If we're doing something other that that normal "password" behavior we should probably be sure to document it.


Beautiful


ACK. Feel free to merge this PR (:


Why do you delete this comment?


MINOR: although this gets fixed in a later [commit](https://github.com/kkaempf/nailed/commit/b6c1e521a99ab36f7a96b86c995b30e0e14c79a3#diff-31b83c06ad82c5fa304c8499c7ec2524R11), it could be changed here in this commit directly


We need to garden these.  We have people who tag their fixes as easyfix when they're actually not, and easyfix is often used by users who are trying to be passive aggressive about their issues.


That's not the right level to do this modification at all. You should implement `Functoria_graph.pp` instead.


I think something similar to L93:
```
raise ParameterTypeError, "expected Array (got #{params[k].class.name}) for param `#{k}'" unless params[k].is_a?(Array)
```
would help.


remove this


Question: Why both MainActivity and DangerousPermissionsChecker implement ActivityCompat.OnRequestPermissionsResultCallback?


How many digits is that, precisely? My understanding is that the number of useful digits in the string representation is dependent on the algorithm used to convert the string to a float. The way this comment is written, blame for loss of precision is placed solely on the storage capabilities of `Float`.

Granted, the loss of precision is quite small.


@ragesoss This resolves a React error on dashboard overview that sounded something like this:

"Warning: Each Child in an Array or Iterator Should Have a Unique “Key” Prop"

I searched around for an open issue, but didn't see one. Let me know if this rings a bell :)


Yes it does. Overriding it isn't necessary though and the way it was the tree would grow instantly with only one item being used up.
Not sure if the saplings were working before this commit, but when I was playing they just never grew. They definitely do now though :P


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


GHX_Vert_IDD (mmatts) - Win64-Windows-7-VisualStudio-12: OK (1957 of 1957 tests passed)

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


The day falco's already damaged brain became completely dead.


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 78](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=348&buildTypeId=Xbim_XbimExchange_XbimExchange) is now running


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


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


TeamCity Xbim :: Xbim Exchange :: XbimExchange [Build 78](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=348&buildTypeId=Xbim_XbimExchange_XbimExchange) outcome was **SUCCESS**
Summary: Running Build time: 00:00:39


вместо точки давай поставим слеш: `img/logo.png`


Uh, I'd say this was a pretty smart move. I do love me some spaces. Don't know what the delaio with you is.


We already agreed to put occupied? on the game.rb with Aaron. This is the same as occupied? The missing x on the piece_exist? is not a big deal. I understood what this is for. Why are you repeating same methods with different def names on two different classes? @davidslee101 @dvanetten03


@pataccini thanks for the notification. Next time we pull the icub-main
repository I'll take care of the changes.

On 10/18/2015 02:25 PM, Ugo Pattacini wrote:
>
> @plinioMoreno <https://github.com/plinioMoreno> @gsaponaro
> <https://github.com/gsaponaro>
> People in Lisbon who're maintaining the |Vizzy| gaze controller should
> be aware of this latest change!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robotology/icub-main/commit/3e28aa5a08c243f5b64b6470491dab3d50c08834#commitcomment-13835708>.
>

--
___________________

Plinio Moreno López
http://www.isr.ist.utl.pt/~plinio
plinio@isr.tecnico.ulisboa.pt
___________________


Tweet (copy-paste I assume).


Hey Alex,
Looks like your editor is removing trailing spaces. This was happening to me too. If you're using Atom, take a look at this to turn it off: https://github.com/atom/language-python/issues/27.


Ah, you're right; that definition looks VS2015-specific. Fixed.


Thank you for your comments!


BC break in 2.3?


head 应该不用标记 permanent 吧。


Isn't object_validator_int ^ ?


It should default to live stream *on*. But there seems to be an issue sometimes where the widget is instantiated with it *off* and then clicking the button does nothing. Refreshing the page "fixes" that issue.


That could work, we would need some sort of lookup for the op though, there is already a bunch of defines for when the recompiled codes are disabled, maybe we could hook in to that some how, or make use of the opcode tables.


Это пока пробный вариант. Для реюзания библиотеки надо будет либо вынести ее в 3party, либо положить в coding.

> On Oct 20, 2015, at 09:01, ygorshenin <notifications@github.com> wrote:
>
> А не лучше сделать .pro файл для этой XML библиотеки в 3party?
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/deathbaba/omim/commit/7e175b4c9ef5bef4f701dc15b455405e075eb11a#commitcomment-13879501>.
>


Yeah, they were based on world light value which is pretty pointless in the dark BL ;)