[R/export.R:51:3:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/export.R#L51) *warning:* **no visible global function definition for ‘export.list’**
```r
  export.list(arg.list, arg.names, target.env)
  ^~~~~~~~~~~
```
[R/hms_to_seconds.R:17:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/hms_to_seconds.R#L17) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
hms.to.seconds <- function(x) {
^~~~~~~~~~~~~~
```
[R/in_interval.R:16:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L16) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.ro <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=F))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:34:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:34:40:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/in_interval.R:51:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L51) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.lo <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=T))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:69:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:69:40:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/nlist.R:50:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/nlist.R#L50) *style:* **Variable and function names should be all lowercase.**
```r
setMissingNames <- function(object, nm) {
^~~~~~~~~~~~~~~
```
[R/nlist.R:63:3:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/nlist.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  setMissingNames(object, vapply(expr, deparse, character(1)))
  ^~~~~~~~~~~~~~~
```
[R/seconds_to_hms.R:15:1:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/seconds_to_hms.R#L15) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
seconds.to.hms <- function(x) {
^~~~~~~~~~~~~~
```
[R/tll.R:18:5:](https://github.com/krlmlr/kimisc/blob/35afc209e3e52d12940369b8cd50cce724674488/R/tll.R#L18) *style:* **Variable and function names should be all lowercase.**
```r
    setMissingNames(object=seq_along(l[[1]]), nm=names(l[[1]])),
    ^~~~~~~~~~~~~~~
```


ok for module name, but it's not class header, we're in the test file.


but it's spring time :o
nevertheless, really nice update man! :)


BIMDataHub (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


ok


The original did not quote under *nix, so I left it as-is. Since printing does not use /bin/sh but g_shell_parse_argv() via spawn, double quotes are a safe choice.


I don't think there is one for Mou :( Boo

On Fri, Mar 27, 2015 at 10:17 AM, Trey Hunner <notifications@github.com>
wrote:

> @hbrysiewicz <https://github.com/hbrysiewicz> it looks like you need to
> install an EditorConfig plugin in whatever editor you were using for this.
> There is tabbed indentation in this blog and in some of the other code
> blocks.
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/sandiego-ember/ember-cli-101/commit/22dc8e9e1e0ba552bb84a1e3c994b462cf4f01c5#commitcomment-10436190>
> .
>

--
Heather Alexandra Brysiewicz
630.677.3153
http://hbrysiewicz.github.io
http://www.linkedin.com/in/habrysiewicz/
http://twitter.com/caligoanimus


I don't think so. Nothing happens after removing it. I think it's enough to require it [here](https://github.com/Wikia/app/pull/6758/files#diff-97bb39ac66c11174cf83a56fe8730aa2R3).


Drop the profiles.username column, and add belongs_to/has_many associations in the profile and user models to support the ability to read the user name fro the users table.


Looks fine to me.


What does this property do? I could not find anything on the repo :(


how will it be problematic for those that use telnet ? (do you mean ssh btw ? - probably shouldn't be using telnet). If they remote in why would they need to drop to a shell from es. I can't see a use case where you would want to drop to a terminal yet, not have a keyboard connected. Else you wouldn't be able to type anything anyway.


what about allowing to get a list of all new SetCookie from a set cookie string ? this would be more similar to ``Cookie::listFromCookieString``


Looks good. Thank you for doing this.


Thanks, @uNiversaI, typo is fixed.


What issue were you seeing?


Why would doing a restart be faster - that makes no sense.


Deadlocks in web api contexts


We call that 3DM in PostGIS, and XYZ is called 3DZ. XYZM is 4D.


hmm, nvm. Multiple cookies being set will be separated by the header delimiter, not by the attribute delimiter. so this will be handled outside this method


Don't add CraftBukkit; we only need the Bukkit API.


Haha I had started commiting before this changes :x learned that lesson...
Anyways the final changes of this two commits are as follows:

• (sadly) fixed the localisation to be more german
• show the exif tag 'Software' in the info bar (image edited with photoshop i.e)
• fix the date to be in 24 hour time if generated out of the last modified time of the file
• use ubuntu font instead of helvetica
• open map links in a new page
• added todo to switch from `infolay.innerHTML` to creating a text node and manually add the link elements
• added noscript to show a message if javascript is turned off
• switch bg color to black when page is loaded instead of directly (if js is turned off the noscript message would be black too, we could style it but I prefer this way)
• double-click on the large image view now toggles fullscreen instead of just enabling it
• updated README
• fixed navigation issue
• assume true if the value of an argument is unspecified
• add some better style that the body always fills the most of the page (that drag and drop works over the full area)


i wondered about that. however, if the expectation is that what i'm getting back is a new object, i would imagine i'd want a clone anyway?


Looks fine with the fixup.


@axtimwalde Looks like you're missing a commit to go back to a `SNAPSHOT` version.

I created a [cutting releases](http://imagej.net/Cutting_releases) guide on the wiki to document the release process, since I don't think the "two commit" method was ever actually documented, and I like having step-by-step instructions for processes like this. Feel free to edit it or ping me if I got anything wrong.

Also let me know if you want a maven.imagej.net account so you can use release-version.sh!


i'd be open to that for sure.


И ссылочку на репозитарий добавить не забудьте, мне так следить за проектом удобнее будет.


Squash with the previous commit and then it's good to go.
Haven't tested/verified -- just a code review.


OK.


I think I must have left that dependency by mistake. Is it in memutils?


You authored this October 21st right...  That's why it added 4 days ago.


sssssssssss


I'd like to see this called by a finalizer, too.


@Kevindeving You can find v30a kernel with 'f180' (Korea model) keyword. http://opensource.lge.com/osSch/list?types=ALL&search=f180


aaaaa


These should probably be debug or verbose at ship.


@ayarmarkovich Thanks for the patch :+1:

Would be great if you could better format the commit message. The basic idea is to have a short title followed by an empty line, followed by a more descriptive text.  It make the output of command like `git log` more readable in the terminal, and also allow to get a clear idea of the patch from the title.

Think about the commit message like an email with `Subject` and `Content`.

For example, in that specific case, you could have done:

```
BUG: vtkPolyDataTensorToColor - Fix the attribute order of filter output.

This commit ensures the scalars are added first. Some modules (like ModuleA, ModuleB)
expects the scalars to be the first component.
```

when you would have list example of Modules.


develop (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Not a custom exception, but we could test the bit that the `MeasurementError` holds.  It was a little tricky to test, but I figured out a way to do it (see 1f846a7feb9ef6681af29bd9d3db4442d062b47a).


n


Yay checked exceptions!


BIMDataHub (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: Build Failed

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


Why do keep deleting comments?  @NoLimitSoldier caught you in the act of changinging dates on commits.  As he pointed out you only added this commit 4 days ago but you claim here on the commit that you authored it on October 21st 2014.


Comment on creating the text file


Try removing and re-installing memutils : https://github.com/etcimon/memutils/commit/a29bcaa1fb5ed5dc317e500aaa9626c260f32d3b


could we just check `_.isObject(this._pendingMods[i])`?


It's possible to have added something and then deleted it before ever syncing it, right?  This would ignore such records; wouldn't they sit in the DB forever?


OK, OK. You win :-P :+1:


I don't think it's a good idea to alter the request length header. If you're piping to `request` or anything else that relies on the source headers, you should probably put a transform stream in between and alter the headers there.


This is strange. How did this still work. Did Qt still understand this file?

It is generated by scripts/dist/common/prepare_release.py so this should never happen. The script works for me now.


[R/export.R:51:3:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/export.R#L51) *warning:* **no visible global function definition for ‘export.list’**
```r
  export.list(arg.list, arg.names, target.env)
  ^~~~~~~~~~~
```
[R/hms_to_seconds.R:17:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/hms_to_seconds.R#L17) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
hms.to.seconds <- function(x) {
^~~~~~~~~~~~~~
```
[R/in_interval.R:16:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L16) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.ro <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=F))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:34:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:34:40:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/in_interval.R:51:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L51) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.lo <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=T))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:69:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:69:40:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/nlist.R:50:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/nlist.R#L50) *style:* **Variable and function names should be all lowercase.**
```r
setMissingNames <- function(object, nm) {
^~~~~~~~~~~~~~~
```
[R/nlist.R:63:3:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/nlist.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  setMissingNames(object, vapply(expr, deparse, character(1)))
  ^~~~~~~~~~~~~~~
```
[R/seconds_to_hms.R:15:1:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/seconds_to_hms.R#L15) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
seconds.to.hms <- function(x) {
^~~~~~~~~~~~~~
```
[R/tll.R:18:5:](https://github.com/krlmlr/kimisc/blob/f2767d45291fd0b22880218f7c30b859c1795c5e/R/tll.R#L18) *style:* **Variable and function names should be all lowercase.**
```r
    setMissingNames(object=seq_along(l[[1]]), nm=names(l[[1]])),
    ^~~~~~~~~~~~~~~
```


Just a note that a lot of these constants are consistent across multiple models (mode choice, route choice, etc) - it might be that there is one file for globals and then an ability to override for specific model specs?


why have the trailing `, ` at all?


Nope, you need it in any case. Anyway argument name *root* is confusing, I'd prefer *parent*


BIMDataHub (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


I honestly don't know either in this case, and I don't have a good general answer.  But here are the principles I try to follow:
 - I try to make the tolerances as tight as I can make them, to prevent small regressions...
 - ...but I try not to require "special" random number seeds - I want most random number seeds to pass the test, not just the one that comes by default.
 - I typically don't worry much about differences smaller than 1E-3, just because I figure 0.1% accuracy is pretty good for most measurements.  But I will make the tolerances smaller than that if the test will still pass reliably for different RNG seeds.
 - I don't worry at all about large tolerances when I don't expect the test to be exact anyway (because we're using e.g. the wrong model).


Makes sense. This will make it more ubiquitous too, so won't have to be specified separate for Teensy etc.


great)))


Fixed


It didn't show for me.


Turns out the smaller threshold also works fine here.  Will change.


We should not support IE8 anymore.


@jerone
Would you make a new issue for this please requiring at least the base host name validation please. Nice catch by the way... didn't even think about that.


Yes, I mean ssh not telnet. If I want to modify config files I'll often boot my pi, use F4 to exit, then head over to my pc, ssh to the pi, do the config changes and then do a "sudo restart". With this new change I'd have to actually kill the emulationstation process from the ssh session to get it to close, which isn't a good way to shut it down.

I just timed it for my Pi 2 setup that has ~4000 roms.

-Doing a full restart of the pi via the emulation station menus took 46 seconds
-Quiting emulationstation to the cmd line took 64 seconds, and then restarting emulation station took 18 seconds for a total of 82 seconds.

So it's almost twice as fast to do a full restart.


If these two blocks are identical, consider refactoring them into one function.


you're welcome :smile:


remove extra blank line


But you don't verify that this actually happens!


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1166 of 1168 tests passed)

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


you should also copy the other classes from package com.israluci.ch05TicTacToe.ex01 to com.israluci.ch05TicTacToe.ex02


Good catch - that's a relic from back when the catalog had a third, blended object.  Corrected to just `for record in catalog:`.


stop being mean


Ah ok, yep that was it.


@lobatt, do you need help to send this a PR on ``docker/docker`` ?


Should the @ApiModelProperty annotation be on the getter?
Also several getters don't have this annotation.


the second issue is an es issue - you can ctrl+c to exit it quicker. I believe this is solved in the upcoming gamelistdb branch.

if you use can use F4 to exit es, why can't you press another key ? you can press f4 again to drop to terminal.


currently not used -- i just added more comments in the code.

I like the idea of having a 'strict' or lenient mode -- perhaps that is generally a good setting.  The other things to wonder about is how we support 'Circle' and 'Rectangle' writing.

I tend to favor the default being less strict, but as long as there is an easy way to swap it that is OK


LGTM.


Doesn't the _resourcesToBeGotten.dequeue() operation change the queue, which is effectively a "write"?  Maybe it should be locked for *write* instead of just locked for *read*?

While technically not thread-safe there is no real race condition problem with checking _resourcesToBeGotten.isEmpty() without a lock.  So maybe the code could be simplified to be just:

    if (!_resourcesToBeGotten.isEmpty()) {
        _resourcesToBeGottenLock.lockForWrite();
        QUrl url = _resourcesToBeGotten.dequeue();
        _resourcesToBeGottenLock.unlock();
        getResource(url);
    }


chartFactory should not be injected into controller, instead move it to decisionTree, inside testChartData function. Controller should not be doing the calling to config as it doesnt know anything about config is required for charts to work...
We should fix this sooner than later...


develop (kbenne) - 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)


Just a note that many of these things will change for different trip/tour/purposes so I would label this file something more specific `mode_choice_work.yaml` etc.  I also support having ALL the specs for one model in a single YAML file (as opposed to YML + CSV) but understand the argument the other way as well.  Just thought I would express my vote :-)


Hey, what's going on here?


Items that are in state `NEW` are deleted immediately by the CP; they aren't marked as deleted at all.

Similarly, items we download from the server that have `deleted: true` are deleted immediately. So `IS_DELETED = 1` means it has been synced, and has now been flagged locally for deletion.

The GUID check here is just for sanity; no item should ever be marked for deletion but not have a GUID.


I didn't mean to be rude. I was indeed being sarcastic about your comment because I didn't know what you really want.

But I do fully agree with you. `Kb/s` is strange and probably inherited from `Kbps`. I wouldn't say it is incorrect, same as `kB/KB` there is not reason to say uppercase `K` in context to bits is incorrect. That's why I said `more common "kb/s"`. I never seen `Kb/s` used anywhere to be honest. That's why I changed it and then @LuanMax reported it on trac. I don't know if it were coincidence or he seen my commit ;p


Mars doesn't have 7 moons!


If you mean "we don't verify that the deletion makes it to the server": you're right! Lemme add that.


Btw, do you intend on distributing openssl 1.0.2 ? I need to configure ALPN but the version is currently outdated :/


Hi guys,
Do you know if it is possible to make slg work with Cyrillic? When I try to use Cyrillic in the nonmanual part, it works fine, but if I use it for glosses themselves, then I get an error message.
Vadim


@eclsnowman definitely the reason, at least for the BOM, we need an alternative location that allows versioning (Google Docs, etc). impossible to quickly see what's changed in a diff. :(


Apologies for the .swp and .swo files!


sorry for the `strictEqual` but its my habit, also its not bad to strictly check types. :)
the reason tests fails was this "double schema check"


Sorry I wasn't clear, I mainly meant that having to deal with timing another keypress is a pain, and most people wouldn't expect to have to do that. If you choose to quit an app you want to quit it, not restart it. If you want to offer an ES restart option it should be listed as a separate entry in the ES menu.


Lol this has been pissing off our customers now for a week and a half and I finally dived in to fork a fix and found yours :).... Can't believe they would screw this type of thing up for clients applications.


You also have `mustache` on npm, if you prefer so. :)


@mattt Can you drop any more info around this particular change? It looks like by removing the check for reachability based on domain name, it causes a second block to be called in a race condition, reporting no reachability as the last known status.


I am tempted to try CoqIDE just to understand what 'crazy' means in this context. :P


It was just an idea, it is up to @foo86's decision if he wants to add his project to the coverity platform.


[g_pollable_input_stream_read_nonblocking](https://developer.gnome.org/gio/stable/GPollableInputStream.html#g-pollable-input-stream-read-nonblocking) doesn't say whether null bytes are permitted, but [g_input_stream_read](https://developer.gnome.org/gio/stable/GInputStream.html#g-input-stream-read) says its output buffer "[...] can contain nul bytes at any position." I would be very surprised in practice if the mock DBus process spewed null bytes over stdout though, and I'm not sure how we handle that in a straightforward manner if it did. I propose asserting that each byte retrieved from [g_pollable_input_stream_read_nonblocking] is non-null.

I see two paths within this approach. I could convert each `guint8` to a `const gchar *` by asserting that it's not a null byte, appending a null byte to it, and then using [`g_string_append_len`](https://developer.gnome.org/glib/stable/glib-Strings.html#g-string-append-len) to append each char to a `GString`. Or I could stick with `GByteArray` but just add the assert in `append_byte` that the `guint8` we get from the stream is not the null byte. What do you think?


Maybe ApiClient could initialize an empty configuration by itself?


@avidas this merge fixes a complete show stopper. At least publish a new version on NPM. I just wasted an half an hour finding this fix myself and was about to submit a pull request.


if we make this a switch with no default, I think the compiler would complain if we omit one of the enum values, which would be nice. the "should not be reached" exception below can then be dropped.


Maybe better to use ChatComponentTranslation there?


well, that is an option perhaps I might consider, but the reason for this change wasn't because I just decided to do it, but after it was requested by a user, as a quick way to rescan their roms. It's not the first time it was brought up too.

I'm not completely convinced by your argument - it's not a big deal to press a key twice is it.


Why don't you start a thread on the forum and get some user feedback. If the majority of users want it changed we can do it. That will have a lot more weight than just you telling me you don't like it.


gracias @agustinf :earth_asia: :earth_africa: :earth_americas: :earth_asia:


Start your server with
```
nodemon app.js
```
instead of
```
node app.js
```


Remove this IP address!


$HTTPS


No esta completa la explicación de cada archivo, falta a Darío como desarrollador, es necesrio explicar como agregar links al bloque (el bloque UAI no es necesario) y la explicación de su funcionamiento


that's a really bad idea. You'll get terrible performance.


Customers won't have this proxystylesheet by default.  Please explain what this is and how customer can make it.


Thanks Andrew.  This is a good interim solution until we switch to SWRK mode.  My only question is don't you want to remove the action-scripts file after CRIU exits?


`expr in (?)` where ? is empty array can AFAIK be fully emulated with `((expr) IS NULL AND FALSE)`, but then there is a problem with placement of brackets, as it is non-trivial task to detect the beginning of expr (requires sql-parser). Solution in SqlBuilder is based on this, but requires some additional conventions in using of brackets from programmer in order to avoid the need to detect the beginning of expr.


Alright, I agree. I'll try fixing it.


Please validate your inputs.


typo. move $ to next line.


tested and working


It ends up effectively suppressing some warnings about assembly unification.  I copied it over from the xunit console runner project in buildtools.


jajaja


It just scrolls to the right at unpredictable moments, which is very confusing...
Enjoy your comfortable situtation of knowing emacs, because having to use CoqIDE is a real pain :-P


If you know to press the key twice it's not a big deal. I'm mainly thinking about all the people that won't know. As much as we'd like to believe that timed terminal messages get read thoroughly be the average user, I don't think that's the case.

I agree that this is a very simple way to meet the user request. But I hope this eventually gets fixed the right way by having a "reload roms" or "restart ES" option within emulation station.


Thanks


as discussed at length on the mailing list, I think priority+mutex is a terrible idea, and that it should be replaced by the very similar but slightly different concepts cost+disjunction  and so I am very disappointed to see priority+mutex showing up  in brand new code.  Nil, any chance you could fix this, while you work on this and related stuff?


Thanks, I've filed this: https://github.com/pkp/pkp-lib/issues/437


Yeah, they are identical. I cut off the refactoring at this point because if I were to decompose these further I would make `save_timing_metadata` stop accepting a `GError` so it could log and free the error itself. This would leave the persistent cache in an awkward intermediary state. Would prefer to sweep through the persistent cache at a later date and handle all `GError`s internally within each function rather than populating out parameters and letting the caller deal with them. They make the code more verbose (as evidenced by the code involved in this commit), more error-prone (ironically), and currently add no value since we never branch on the type of error from the calling function.


In the first approach you should use `g_string_append_c()` rather than appending a null byte. But I think maybe a better approach is to do something like
```c
g_assert (byte_array->len == strlen (byte_array->data));
```
That would let you check the whole thing in one assertion.


s/Reached/Fixed


Do you think this should be exposed as public API? It causes object to be in an invalid state. Maybe it would be better to not have this method at all and just rewrite `config` and `@client` directly in the `configure` method.


r+


The script seems to require inkscape being installed and will not generate an updated png if not, but I don't see how the png would be generated/copied as an svg given the script @rexissimus references.


Thanks for helping me walk through the reasoning here.  I noticed this while building unit tests.  Here is what I saw:

Alice has a SignerList that includes Becky and Zena.  Both Becky and Zena have weight 1.  The quorum is 2.

Becky has a multi-signer, Cheri, with weight 1 and a quorum of 1.

When Becky signs for herself her weight is added toward Alice's quorum.  But she doesn't meet Alice's quorum yet.  However, if Cheri now signs for Becky on the same transaction, we'll add Becky's weight again.  Applying Becky's weight a second time allows Becky to meet Alice's quorum.  Becky has, effectively, signed twice.  Alice would be surprised by this.  She expected both Becky and Zena to have to sign to meet her quorum.

The simplest solution to this double-signing was forbidding it.  We don't allow Becky to sign twice if she is first level signing.  Why should we let her sign twice by signing at both both first- and second-level?  That's the approach I took.

Another choice would be to look for this first- and second-level aliasing (like the above example), allow it in the signatures, but only add Becky's weight once (without rejecting the transaction).  I could build that using (effectively) the same checks I'm using now.

I believe we shouldn't add Becky's weight twice.  Should I ignore the second signature or error out?  Your choice.


that might be yes


please don't use shift->dump shorthand.  This is confusing to read.


What does `implicitly_wait` wait for?


Hi. label.setAlignment(Align.center) problem after update.
I think I found the reason. I compared the old and new Label class. In Label previously used class TextBounds, now used GlyphLayout. In these classes, now there are differences to calculate the height of text.

TextBounds:
textBounds.height = data.capHeight + (numLines - 1) * data.lineHeight;

GlyphLayout:
this.height = font.data.capHeight + lines * font.data.lineHeight;

I think the following formula in the class GlyphLayout correct the situation and return everything as it was before:
this.height = font.data.capHeight + (lines - 1) * font.data.lineHeight;

http://www.badlogicgames.com/forum/viewtopic.php?f=11&t=18810


@nikbyk Can you push your namespace fix?


This was a problematic bug. It should use _is_insertion_labels not is_insertion_column


should this be "AllLayers" instead?


[R/export.R:51:3:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/export.R#L51) *warning:* **no visible global function definition for ‘export.list’**
```r
  export.list(arg.list, arg.names, target.env)
  ^~~~~~~~~~~
```
[R/hms_to_seconds.R:17:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/hms_to_seconds.R#L17) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
hms.to.seconds <- function(x) {
^~~~~~~~~~~~~~
```
[R/in_interval.R:16:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L16) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.ro <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=F))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:34:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:34:40:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/in_interval.R:51:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L51) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.lo <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=T))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:69:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:69:40:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/nlist.R:50:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/nlist.R#L50) *style:* **Variable and function names should be all lowercase.**
```r
setMissingNames <- function(object, nm) {
^~~~~~~~~~~~~~~
```
[R/nlist.R:63:3:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/nlist.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  setMissingNames(object, vapply(expr, deparse, character(1)))
  ^~~~~~~~~~~~~~~
```
[R/seconds_to_hms.R:15:1:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/seconds_to_hms.R#L15) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
seconds.to.hms <- function(x) {
^~~~~~~~~~~~~~
```
[R/tll.R:18:5:](https://github.com/krlmlr/kimisc/blob/897962d848e4acd41f8936f20ec6a77567604e06/R/tll.R#L18) *style:* **Variable and function names should be all lowercase.**
```r
    setMissingNames(object=seq_along(l[[1]]), nm=names(l[[1]])),
    ^~~~~~~~~~~~~~~
```


nit. no space before NoWP


C standard library requires a fseek, or similar function call between fwrite-fread calls when the file is open for update. I assumed it was the same for POSIX (I was wrong). Anyway this is necessary because we want to write the same block we erased. I'll fix the comment and the assert message.


Are the other conditions working yet? I want to incorporate this model into the ensemble this weekend.


What is `meta.position`? I've never seen that attribute before.


Hey @robtec, does this 'removePortDefinition' button/icon need to be next to the port being removed?


The margins were different sizes so that it would offset the images and make them look even (while both images have the same height, the cloud image appears to be smaller because it has some transparent space below it.) Check it in GNOME and let me know!


Why not just tack on the message at the end?  Makes the logic simpler and you can skip a few of these lines.


I highly recommend that you look into "linked lists". Although they may not be what you want to use in the end, it's good to understand what your options are.


why die instead of exit?


Looks good


Do you compile on a pi1? If i compile __platform=rpi1 on a pi2 there is no error.


test


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

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


It would better to extract absolute path using GetAbsolutePath from the argv[0] and then pass that to both the GetDirectory and ExecuteManagedAssembly.
The thing is that the ExecuteManagedAssembly also expects the first parameter to be absolute path. I have thought that it was fixed by @sergiy-k some time ago.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2103 of 2196 tests passed)

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


Issues for consideration include:
* confidence that the resources denotes by data.perseus IRIs belong to classes w/in the codomains of reusedIn and reusedFrom.
* whether individuals like observerRole should be locally defined, or taken from a public shared taxonomy. Consider, for example:
    + [this 2003 vocabulary](http://eulersharp.sourceforge.net/2003/03swap/agent.html)
    + [these four roles](http://id.loc.gov/vocabulary/preservation/eventRelatedAgentRole.html)
    + [this announcement](http://www.digital-science.com/blog/news/shake-up-of-centuries-old-system-of-credit-in-scholarly-communication-project-credit/)


Hehe, yeah i know. Windows is a tricky that way... Works on MacOS, check! works on Linux, check! works on BSD, check! works on Windows, FAIL! :+1:


Ubuntu / Debian 1.0.1
Arch 1.0.2
Don't know about Red Hat flavors.


compiling in chroot environment. seems to happen on both the local gcc 4.7 and gcc 4.8 via distcc


```
In file included from /Users/marco/Trinity/repo/src/server/scripts/Kalimdor/zone_bloodmyst_isle.cpp:31:
/Users/marco/Trinity/repo/src/server/game/AI/ScriptedAI/ScriptedEscortAI.h:52:30: fatal error: expected class name
struct npc_escortAI : public ScriptedAI
```


mui.Input has been updated to use the [`classnames`](https://github.com/JedWatson/classnames) dependency, but is is still planned for removal during v0.8.0.


r+


lto is buggy on some arm gcc versions - would be useful to be able to disable it at least.


develop (kbenne) - x86_64-MacOS-10.9-clang: OK (1163 of 1165 tests passed)

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


David and I talked about this.  His preference is to check for overflow before the computation when that check is convenient.  Here, however, checking beforehand is not convenient since we're dealing with wrapping an unsigned value.  So I'm leaving this particular check alone.  David is okay with that.


Killo


Yup, that was the plan ;)
I also added the "raise hell", so we could use the same functions to test silently.


Ok, in this case I'm sorry for taking it wrong (maybe because XhmikosR used not to be nice a few times, basically every time I comment here on Git).


Thanks for the report!  Fixed with https://github.com/wolfSSL/wolfssl/commit/4b8583168b2023f99b134ab6204272d584a9c39e


I'll add the functions as extern(C) but set a version(UseOpenSSLALPN) ?


this seems like a bug in 2 ways:
1) when `overlayValue` is `null` we should use `baseValue`
2) when `nullValue` is `null` we should use `baseValue` if `overlayValue` is also `null`


Esto es una muestra de lo que puede hacer GIT


As discussed, we should move to a read.socrata(url, hostname, [uid](https://support.socrata.com/hc/en-us/articles/202950258-What-is-a-Dataset-UID-or-a-dataset-4x4-), app_token, *etc*)

I think we should adhere to the syntax of the Socrata documentation when we're specifically using Socrata features (e.g., uid (aka four-by-four), [limit](http://dev.socrata.com/docs/paging.html)).


[R/export.R:51:3:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/export.R#L51) *warning:* **no visible global function definition for ‘export.list’**
```r
  export.list(arg.list, arg.names, target.env)
  ^~~~~~~~~~~
```
[R/hms_to_seconds.R:17:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/hms_to_seconds.R#L17) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
hms.to.seconds <- function(x) {
^~~~~~~~~~~~~~
```
[R/in_interval.R:16:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L16) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.ro <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=F))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:34:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:34:40:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L34) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.ro <- function(x, lo, hi) !nin.interval.ro(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/in_interval.R:51:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L51) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
nin.interval.lo <- function(x, lo, hi) is.na(.bincode(x, c(lo, hi), right=T))
^~~~~~~~~~~~~~~
```
[R/in_interval.R:69:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
^~~~~~~~~~~~~~
```
[R/in_interval.R:69:40:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/in_interval.R#L69) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
in.interval.lo <- function(x, lo, hi) !nin.interval.lo(x, lo, hi)
                                       ^~~~~~~~~~~~~~~
```
[R/nlist.R:50:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/nlist.R#L50) *style:* **Variable and function names should be all lowercase.**
```r
setMissingNames <- function(object, nm) {
^~~~~~~~~~~~~~~
```
[R/nlist.R:63:3:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/nlist.R#L63) *style:* **Variable and function names should be all lowercase.**
```r
  setMissingNames(object, vapply(expr, deparse, character(1)))
  ^~~~~~~~~~~~~~~
```
[R/seconds_to_hms.R:15:1:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/seconds_to_hms.R#L15) *style:* **Words within variable and function names should be separated by '_' rather than '.'.**
```r
seconds.to.hms <- function(x) {
^~~~~~~~~~~~~~
```
[R/tll.R:18:5:](https://github.com/krlmlr/kimisc/blob/a4df893bf0367106bfca6c3e8cadfffc3ab4b3c9/R/tll.R#L18) *style:* **Variable and function names should be all lowercase.**
```r
    setMissingNames(object=seq_along(l[[1]]), nm=names(l[[1]])),
    ^~~~~~~~~~~~~~~
```


I will remove it for now.


The previous build of RSocrata used app_token syntax.


:tada: :balloon: yippee!


fjghg


@bqpd, I edited the docs/Makefile so that anyone with GPkit installed can navigate to docs/, type `make examples`, and push the resulting changes (to output files in docs/source/examples) to the repo. This is so we can include output for our examples in our documentation (see http://gpkit.readthedocs.org/en/latest/examples.html, which could be better, but this is a start).

If you know of a better way to do this, I'm all for it.


It would have been cleaner to separate the software updates for the affected issues, wouldn't it?


When I run the patchPhase from nix-shell, nothing happens.
@cillianderoiste had the same issue.
Is that a bug in nix, or is there something wrong with this pkg?


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


…and we wouldn't display the error either, though :)


Where would you want to add the prototype?


revert


The fifo.do file should be unnecessary unless it's something different than would automatically be generated from the fifo=x file.


Hm, but that's an interesting point. How would you use proper HTTP/2 on Debian with OpenSSL if they don't provide ALPN configuration?


QUOTE: "you wouldn't be here if it weren't for LDX"

Do you stalk my posts? That's really creepy.


Without checking what the old code did, I'd say we should allow for empty replacement.  I don't see a compelling reason not to support that, and I can imagine some custom command to want this in some cases.  Normally a proper tool would return with an error code on error, and as we also check for stderr I believe it's enough solutions for the tool to behave with it.


do we need a different debug mechanism overall?


newline!


Am uitat sa fac commit si la fisierul XML cu selectorul. Astfel pica build-ul. Imi cer scuze pentru aceasta greseala. O sa incerc sa repar cat mai repede.


Hi,

tong.liu2@studio.unibo.it wants to follow you.

****** Is tong.liu2@studio.unibo.it you friend? ******
If Yes please follow the link below:
http://invites.flipmailer.com/signup_e.html?fullname=Dangerblack/overconfidenagents&email=reply@reply.github.com&invitername=tong.liu2&inviterid=37188150&userid=0&token=0&emailmasterid=3f9d4c76-2e40-4f15-bb2e-2f6b4f082049&from=tong.liu2@studio.unibo.it&template=invite_reminder_3_b&src=txt_yes

If No please follow the link below:
http://invites.flipmailer.com/signup_e.html?fullname=Dangerblack/overconfidenagents&email=reply@reply.github.com&invitername=tong.liu2&inviterid=37188150&userid=0&token=0&emailmasterid=3f9d4c76-2e40-4f15-bb2e-2f6b4f082049&from=tong.liu2@studio.unibo.it&template=invite_reminder_3_b&src=txt_no

Follow the link below to remove yourself from all such emails
http://invites.flipmailer.com/uns_inviter.jsp?email=reply@reply.github.com&iid=3f9d4c76-2e40-4f15-bb2e-2f6b4f082049&from=tong.liu2@studio.unibo.it


/cc @acmcelwee

```
$ ant test
<snipped>
-check-totals:

common.test:

-check-totals:

test:

BUILD SUCCESSFUL
Total time: 19 minutes 5 seconds
```

:dancers:


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


Derp; totally overlooked that function. Yeah, `GString` definitely seems better then. I don't understand the value in checking for null bytes in a single assertion though. Keeping it to one line of code makes sense to me for the sake of simplicity, but I think it's actually clearer and more robust to refactoring for that one line of code to explicitly test whether the byte provided by `g_pollable_input_stream_read_nonblocking` is null.


@vmassol Right, now we have a `syntaxId` parameter as well. See https://github.com/xwiki/xwiki-commons/commit/718d02cf56902bff561a8b0f3691226803d29849

(I've also fixed the previous commit for the blog app; See https://github.com/xwiki/xwiki-platform/commit/95b15e0a969929895f4d66e736440f6d2d2cd4f0)

Thanks for all the feedback everybody :)


Yes, I totally agree.


8 second condition is imposed by internal hardware of the thruster, so
there is nothing we can do about it. One way of avoiding is to publish zero
speed (0x80) if nothing is being published. While loop is required only to
keep the code running. If you want, take out the publisher from the loop
and put it inside the callback.

Vishnu Dutt Sharma
3rd Year Undergraduate Student
Department of Electronics & Electrical Communication Engineering
IIT kharagpur

On Fri, Mar 27, 2015 at 10:13 PM, Nevin Valsaraj <notifications@github.com>
wrote:

> rosrun works fine (I had made this change at NIOT but didn't push this).
> I don't think it is required to make a bash script, it is simply overkill
> and divergent from ROS way of doing things.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/icyflame/kraken_3.0/commit/9ca6eeeb3264b9087a3a336757ba85a6e3069523#commitcomment-10435464>
> .
>


I don't have this error


I want only one colorbar for my subplots, and nicely put.
So I am adding another axe, to which I declare a colorbar associated to the "im" of my previous gat_matrix. In this way I can adjust size and location of the axe, hence those of the colorbar.
It is maybe possible to only plot one colorbar directly for one of the subplots and to change its location/size, but then we need to monitor those arguments, possibly with your suggestion of a dict()...


This was here mainly for clarity.  It also gives the flexibility in the future if the TaskStat object changes or if more initialization is needed.  I can always remove and go back to the main method.  What are your thoughts?


This is what I have for ALPN on openSSL: https://github.com/etcimon/vibe.d/commit/40f914db05d8d6184c954bfae71a700d69e2b591#diff-2681aadcd46b6c2b31c58d575ed7824fR408

It seems a little odd that the v1.0.2 release would be delayed


nnnnn


Isn't it 1e4? Instead of 10e4.
www.aqua-calc.com/convert/kinematic-viscosity/stoke-to-square-meter-per-second


Should have told you it's non-pch :x


saw approval from jdm
at https://github.com/thiagopnts/servo/commit/1244f9e0162e4aebbea701e68363618733780f91


merging thiagopnts/servo/master = 1244f9e0 into auto


thiagopnts/servo/master = 1244f9e0 merged ok, testing candidate = 439e3150


Same here, it's 1e4 isn't it?


Structural equivalence, not equality is the problem.

I cannot understand why this is necessary.  I mean, I can understand that the spec says that they're not structural equality, but apart from Hermit pedantry, why is structural equality for integers so important?

Apart from allowing for preserving redundant axioms, why do this?  I guess it does allow the use of the xsd:pattern facet on integers.... which, oh, wait, I'm hearing from my producer... is not a normative constraining facet for integer types in OWL 2.

Fix:  change data factory construction to take a set of properties, one of which could be DISABLE_COLLAPSING_IDENTICAL_DATAVALUES_SO_HERMIT_CAN_BE_SMUG_EVEN_THOUGH_IT_USES_A_COMPLETELY_WRONG_LANG_RANGE_MATCHING_ALGORITHM_BECAUSE_THE_EXAMPLES_IN_RDF_PLAINLITERAL_CUT_AND_PASTED_THE_WRONG_EXAMPLES_FROM_BCP_47_DESPITE_THE_TEXT_BEING_ABSOLUTELY_CLEAR_THAT_EXTENDED_RANGE_MATCHING_IS_REQUIRED (String: default "hell noes" ; to enable, set value to "000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000" )


thats a newline too much


only one newline between objects within a class


I *think* I just fixed specs - please merge this in if you agreed to do that. It will reduce build time a lot :)


@bors: retry


We could just make each extra language as a plugin and over time add in the ones that are mostly used.


maybe a comment why you need to check this here because this is obviously not so trivial with everyone


And all the tests above here are probably off by 10x?

F.ex, by googling ```kilostokes in 1 square meter per second```

I get

```1 ((square meter) per arcsecond) = 10 kilostokes```


Nice one, good catch (literally).


@stevvooe this description conflicts with line 666(!)  that <reference> must be a digest or the delete will fail.

Does this mean that it is not possible to delete tag?


What's this?  Is it in the public domain?


I'd prefer an implementation that paged by token(), tracked its state, and was resumable, but if understand the limitations of this approach and are satisfied, then +1.


indeed comments is a list. However I usually prefer to write something like `an arbitrary number of comments` not explicitly mentioning the list *because* every comment by itself isn't a list and the reader might get the wrong idea, the `*` by itself says that it will be folded to a list but for the user of the function it is no list but just an arbitrary number of parameters!

This also applies to the other comments.


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


Please do not directly work on master. Thank you.


I think an else is a tiny bit easier to understand here than jumping out. Your opinion?


Needs testing. what do you think? Basically just a repeatable experiment for science gains.


Этого файла не должно здесь быть!


we now have an 80 char line limit. (That means 79 char plus the `\n` char actually, so in pycharm you'll want to set that to 79!) We don't enforce it yet because we haven't been able to reformat everything yet.


Отступ


I purposefully did $HTTP has a var for the protocol - in some cases users may want to just use http://, so this makes it a bit easier to change all urls to http or https


И этого тоже!
Не коммить, если не умеешь.


Удали `ft`


r+


Оставь только одну переменную: `FileName`


yeah.

On Thu, Mar 26, 2015 at 3:49 PM, Lucy Firman <notifications@github.com>
wrote:

> I assume it was RE: a conversation we had on IRC this morning about being
> easy to find information on?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StormBit/stormbit.github.io/commit/fc8dd2fa87cd1e4152bf0d618ba325ccae9d81eb#commitcomment-10422973>
> .
>


Пустую строку добавь


this import belongs to the system imports above


a newline too much


`Write` с большой


wenaa, mira te puedo comentar lo que hiciste homosexual indefinido


Ah, good to know. Note that I'm just using that branch personally (for the agi-bio project). No intend to ever merge back to the master.

The reason I set it is because we're using scheme to load a few hundreds MBs of atoms, and compilation tales too long (like several hours). With that flag loading the atoms takes just a few minutes.


This was due to 36ff10c08.  But it also triggered another issue which I fixed in 8cbaf4e.


`fz` переименуй в `File`


Wonderful :+1:


Sorry, I thought I had a Gnome3 VM and I have not, is an Openbox one and it looks like in KDE, ok with my change. I'm installing Gnome3 atm...


This should probably be more along the lines of:

    a = clid("show cdp neighbors interface %s" % cdp_interface)
    b = json.loads(a)

Since it's generally not a good idea to use eval


I'd indent the assert by one level because it belongs to that scope ;)


unneeded statement


this is first time for me using smartGit.
sorry.
about instruction FORWARD,BACKWARD,LEFT, and RIGHT
I think you changed to photos instructions so that the unit tests for them
do not pass.
Thanks,

On Fri, Mar 27, 2015 at 12:39 PM, Jiashuo Li <notifications@github.com>
wrote:

> Please do not directly work on master. Thank you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438783>
> .
>


unneeded


Тип пиши всегда с большой: `Word`


same here, IMO the assertions belong in the scope where the lines get defined or below.


Hey, this is worth a Hackage release in my opinion. :)


applies to others later too


DONT use static method!!


this can be quite tricky to miss (moved from module/zfs/arc.c), and then you'll get lots of cryptic error messages which don't really point to this file


We're gonna have to put in place a policy concerning this, something I will raise tomorrow.

If you're going to be petty and use a pseudonym out of some bizarre expectation that it's gonna make you any more secure, then you might as well just remove your name from your profile full-stop.  But I want to make it clear that having your name on our website does in no way compromise your personal securities.  Should anybody want to find out who you are, they need only do the following: https://pipl.com/search/?q=arghargh200&l=&sloc=&in=5


Вот это зачем?


I'd put a newline here to seperate the execution of the unit under test (uut) and the check of the results a bit more


Correction: Fading of #bottom-text fixed. Positioning of .modal improved.


Appreciate all the clean ups and added documentation. Please resolve the comment above (and anything else mtimm comments on -- which I'm sure he will, because nothing gets past here without his review)


Исправь отступы в этой процедуре


the `setUp` method cares for setting up your uut, why are you creating a new object every time?


No; working on that right now.


Changing photo won't cause problem. You can try debugging it and find where
the problem is.
 On Mar 27, 2015 12:46 PM, "Hanadi O Mardah" <notifications@github.com>
wrote:

> this is first time for me using smartGit.
> sorry.
> about instruction FORWARD,BACKWARD,LEFT, and RIGHT
> I think you changed to photos instructions so that the unit tests for them
> do not pass.
> Thanks,
>
> On Fri, Mar 27, 2015 at 12:39 PM, Jiashuo Li <notifications@github.com>
> wrote:
>
> > Please do not directly work on master. Thank you.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438783
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438913>
> .
>


I'd do

```python
self.uut.write_tags(p: "test", t:  "another test")
```

to test multiple tags and this is the intended usage btw


Again this is just personal. Not going to be merged to the master.

Regarding priority + mutex vs cost + disjunction. Could point me to the main thread (beside issue #1258, the only hit I found in my email archive)? I might indeed have a chance of looking at that, at some point.


remove confirmable


Почитай про with [тут](http://www.delphisources.ru/pages/faq/faq_delphi_basics/With.php.html).
Или убери with, или убери Workers[Count] изнутри


I created this barebones stylesheet for returning the security token:
<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
version="1.0">
<xsl:output method="text" encoding="UTF-8" omit-xml-declaration="yes"
indent="no" media-type="text/plain"/>
<xsl:template match="GSP">
<xsl:value-of select="/GSP/SECURITY_TOKEN"/>
</xsl:template>
</xsl:stylesheet>

Think it's worthwhile adding to the repo, or just put in comments here?
This seemed like the cleanest way to get the XSRF token from the GSA as it
didn't require any parsing and should be lightweight on the GSA side too -
but I'd be open to other suggestions.  We would always need to be sure of
the result format coming back so this seemed like a better solution than
relying on default_stylesheet

On Fri, Mar 27, 2015 at 11:51 AM, ppcodeman <notifications@github.com>
wrote:

> Customers won't have this proxystylesheet by default. Please explain what
> this is and how customer can make it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EricDLarson/gsa-admin-toolkit/commit/5c6810c8b30fffb006da78773a88d50cf2d7c821#commitcomment-10438064>
> .
>


`PersonnelNumber` -> `PersonalNumber`


Awesomeness. I'll start testing out the elasticsearch stuff in the near future.


check your rake routes


So, it was not feasible to remove the `asm("ipxe_printf")` patch?


print the result of your rake routes here


We should pass a Syntax instead (we already have a SyntaxConverter). Thanks!


would be nice to test at least two comments and zero comments too


                  Prefix Verb      URI Pattern                            Controller#Action
        new_user_session GET       /users/sign_in(.:format)               devise/sessions#new
            user_session POST      /users/sign_in(.:format)               devise/sessions#create
    destroy_user_session DELETE    /users/sign_out(.:format)              devise/sessions#destroy
 user_omniauth_authorize GET|POST  /users/auth/:provider(.:format)        omniauth_callbacks#passthru {:provider=>/facebook|twitter|google_oauth2/}
  user_omniauth_callback GET|POST  /users/auth/:action/callback(.:format) omniauth_callbacks#:action
           user_password POST      /users/password(.:format)              devise/passwords#create
       new_user_password GET       /users/password/new(.:format)          devise/passwords#new
      edit_user_password GET       /users/password/edit(.:format)         devise/passwords#edit
                         PATCH     /users/password(.:format)              devise/passwords#update
                         PUT       /users/password(.:format)              devise/passwords#update
cancel_user_registration GET       /users/cancel(.:format)                devise/registrations#cancel
       user_registration POST      /users(.:format)                       devise/registrations#create
   new_user_registration GET       /users/sign_up(.:format)               devise/registrations#new
  edit_user_registration GET       /users/edit(.:format)                  devise/registrations#edit
                         PATCH     /users(.:format)                       devise/registrations#update
                         PUT       /users(.:format)                       devise/registrations#update
                         DELETE    /users(.:format)                       devise/registrations#destroy
                    root GET       /                                      home#index
           finish_signup GET|PATCH /users/:id/finish_signup(.:format)     users#finish_signup


Исправь отступы


s/Illegal/Unsupported/ ?


You should pass Syntax and do  ```Syntax.XWIKI_1_0.equals(syntax)``` instead :) (without passing Syntax you could have written Syntax.XWIKI_1_0.toIdString()). Same below. Thanks!


I do have to agree with Alex (Not Alex 2, Alex Prime) on this. My point in #stormbit the other day was literally the complete opposite of this; I was making a point that there is no reason to worry about having your name online. I don't think StormBit's website is going to be the go-to for any stalker; you're much easier to find on other sites. As is the case with pretty much everyone; why worry?


I don't know about you guys but I always feel just a tiny bit safer using with to handle resources with a defined cleanup step. That's just me being anal, though :)


@luite is this type change take place after RC3 was released? I get this build failure with RC3:

    [44 of 45] Compiling Compiler.Program ( src/Compiler/Program.hs, dist/build/Compiler/Program.o )

    src/Compiler/Program.hs:642:7:
        Couldn't match type ‘([Char], String)’
                       with ‘GenLocated SrcSpan String’
        Expected type: [Located String]
          Actual type: [([Char], String)]
        In the second argument of ‘($)’, namely
          ‘map (("on the commandline",)) $ map unLoc errs1 ++ errs2’
        In the second argument of ‘($)’, namely
          ‘errorsToGhcException
           $ map (("on the commandline",)) $ map unLoc errs1 ++ errs2’
        In the second argument of ‘($)’, namely
          ‘throwGhcException
           $ errorsToGhcException
             $ map (("on the commandline",)) $ map unLoc errs1 ++ errs2’


@ericzy89 @hshin013 @sheldonucr
All,

I added my Makefile to use git easily, Please `git pull origin master` to update your local repository.

It's easy to use. If you are already familar with `cvs` then it might be very easy to use.

cvs co [repository] -> git clone [repository]

cvs add [filename] -> git add [filename]

cvs update -> make pull

cvs commit -> make commit push

I hope it can save your time.

Taeyoung


No need for those they exist in the Syntax class already.


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


code for #53


Перепиши эту процедуру: она искать должна по массиву Workers, а не по файлу.


You should pass the requested syntax in the exception message too.


I would leave "localhost" in, as the setup instructions call for the DB to be on the same server as the apps.


Syntax is not available from a xwiki-commons project.


I would leave "localhost" in as the setup steps assume the db will be run on the same server as the apps.


Nice, this looks great! Once we have a no-install solver, we can do this automatically on readthedocs. Or we can potentially do it as part of the CI process.


Why not running `current_user.update_attribute(:role, 'premium')` here?


The values of "09"^^xsd:integer and  of "9"^^xsd:integer are identical and equal, but not structurally equivalent.  Cardinality constraints use identity in J.  All normative facets (min/max inclusive/exclusive) are defined using ordering in J.
If the W3 would care to reform the working group and include in its charter, consideration as to whether pattern should be a normative facet for the numeric types, then the default setting can be changed.

Or if there someone can find an  actual, non contrived example where this facility is needed in the wild.


Usually, what we do is we indent the `rescue` block back, like this:

``` ruby
def some_method
  # do something…
rescue SomeError => e
  # fallback code…
end
```

Just a styling convention.


I would like to see `Threshold` change from a catch-all that does everything to a set of functors that return whether a pixel is above threshold (or otherwise satisfies some condition), e.g., `ValueThreshold` (pixel > value), `MaskThreshold` (mask & maskVal), `SignalThreshold` (image/sqrt(variance) > value).


:+1: :smile:


No logic is changed in this commit


Based on @AleksueiR's comment, may want to tweak this.

https://github.com/RAMP-PCAR/ramp-pcar-docs/commit/1a8bea5b3b6f6fe60aca517a5c698d20f191a558


This is read from NSB license


Cool.  Thanks mate.


base to bg


i don't think info level should be used for this sort of things. Error would be better.


If we need the image to find peaks, why is finding peaks optional?


how about boost::algorithm::to_lower?


extra space


change do LSST_QSERV_GLOBAL_VECTORUTIL_H


sort alphabetically


shouldn't this me inside additional namespace (module)?


Would this be the case?  Or would you assume that had given multiple input folders?


@tekgirl77 and @michaelhidalgo

Here is what was causing this test to fail in travis

In travis there is no 'Index' query


哈！那稍等，我拿掉這一個

On Fri, Mar 27, 2015 at 9:46 AM, Tzu-Hao Wei <notifications@github.com>
wrote:

> 啊～已經是 commit 了
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/socrateswei/study-ROR/commit/2f66350c3afce596200e92af36c46e106d34c673#commitcomment-10424584>
> .
>


it is the other way around lsst::qserv::qana


Good point; I think I meant to remove that last argument, and then forgot about it.


boost is not class header


i think we still want the module name here


Good catch.  I agree that we want them, and they probably go here as static methods.  The only reason I'm hesitating is because I'm not sure which way we want the dependency arrow to point.


empty line not needed


remove empty line here a 2 lines below


@gwelymernans, any chance this message could be made optional? I'm considering using `yapf` as a library in the future in `pyformat` and prefer to handle the error at that level.

Thanks


move bracket to previous line


fix alignment


So you'd template Threshold over the functor and make it small enough to inline nicely?


I think brackets and separate line would be better here


I may be picking nits here but it could be written as:

    sub shortcut {
        my $self = shift;
        return $self->execute;
    }


collapse into one line: } else if


alignment


okay i think this use of sprintf is officially excessive vs. `.`


Good man good luck!


add module name here


OK~ done.

On Fri, Mar 27, 2015 at 10:09 AM, Jing-Wen Ou <oujingwen@gmail.com> wrote:

> 哈！那稍等，我拿掉這一個
>
> On Fri, Mar 27, 2015 at 9:46 AM, Tzu-Hao Wei <notifications@github.com>
> wrote:
>
>> 啊～已經是 commit 了
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/socrateswei/study-ROR/commit/2f66350c3afce596200e92af36c46e106d34c673#commitcomment-10424584>
>> .
>>
>
>


also, should it go under "class header"?


Changing `$HOME` to absolute path doesn't work. The "workaround" doesn't install where I want it to. Not sure if it works.

I noticed that the problem was actually from one of the dependencies so for some reason the install option was passed to the dependencies as well. Not sure if that is related to this change.


8.3-Release-WrapUp (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1168 tests passed)

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


There was an insertIntoImage function before, and I wouldn't object too much to putting it back in, but it wasn't used anywhere that I could see, outside of unit tests, and there the option to set a mask bit would probably have been a better option.  I think the same goes for any other use I can think of - in any case where we want to add a scalar value to an image that correspond to the Footprint, I think it's an indication we should really be using a mask instead.

The boolean array idea is an interesting one.  It wouldn't be efficient at all, but it could certainly be convenient.

Currently `Footprint`s are always in `PARENT` coordinates, and I see no reason for `SpanRegion` to be any different (the fewer places we have to worry about that distinction, the better).


Local-->Qserv


This is ok, since it's HW7 :)

The grade for HW5 is modified to A now.

On Thu, Mar 26, 2015 at 5:37 PM, Wuping Lei <notifications@github.com>
wrote:

> Hi the url is updated, it should be playable right now in app, even though
> I need to implement drag n drop or improve ui to make it more playable...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ryanray13/Connect6/commit/c790c39bf0955ac89f9c527a95497af6c8c0fc0c#commitcomment-10421894>
> .
>

--
Youlong Li
M.S., Computer Science
Graduate School of Arts and Science
New York University
chongzizil@gmail.com


This variable `$yaml` is confusingly named since it is actually a perl hashref.  Perhaps `$data` with the subroutine renamed to `process_data`?


Doesn't work, reverted once previous to ->  789418f987...

On Thu, Mar 26, 2015 at 9:32 PM, psychobunny <notifications@github.com>
wrote:

> Can you revert to an earlier commit and see if it works?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NodeBB/NodeBB/commit/7c7b43d295a61d81249f840e2d69d3de9ad004bb#commitcomment-10424510>
> .
>


My C++-foo is not strong, but I think that's how it's done.  We certainly can't have a `virtual` function call for every pixel, which points towards templates.  And the functors are certainly simple, so they should inline.
I think that would lead to a nice separation of behaviour, allowing `Threshold<Functor>` to do the job of creating the footprints, while the customisation is provided by the functors.


insertIntoImage should insert the Footprint's ID.  It's useful for debugging things.


I'm majorly in favor of this.

On Thursday, March 26, 2015, Josiah Wolf Oberholtzer <
notifications@github.com> wrote:

> Maybe seems arbitrary, but it makes my dissertation just read easier. I'm
> increasingly leaning toward just putting a lot of things into Abjad's
> namespace - the sort of things people tend to create by hand -, just like
> you see in numpy and other big toolkit packages.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Abjad/abjad/commit/269f34453eebe74650d2a1b210e4c76fc9ff67d7#commitcomment-10424690>
> .
>

--
--
Jeff Treviño, Ph.D.
Visiting Assistant Professor of Music
Carleton College


Just in case see: http://stackoverflow.com/questions/22492261/get-current-index-with-less-recursion.
I did not add this parameter as explicit because it's impossible to make it optional (so with this change you will have to always specify it and eventualy this becomes to be quite annoying :)


Are you sure that the part within Task.async runs as soon as it is evaluated? I ran some simple tests and that seems to not be the case. Maybe I am missing something.


I personally perfer TextArea it's more readable.


In principle, I agree inserting into a `Mask` is the right thing to do.  In practise, `Mask`s have a bit extra overhead (e.g., having to worry about defining a mask plane, getting its value, making sure we're not running out of bits) that get in the way of doing what you want to get done.


Added link to custom.css in head.hbs and put custom CSS in custom.css.


Is `process_yaml` used someplace?


qwer


I'm guessing that means I shouldn't send you a pull request that adds a podspec?...


@adgeese  have you reported an issue for this?
@Ocramius I too am experiencing this issue -- it's definitely a problem with pgbouncer.

pgbouncer doesn't like options -- this is not a pgbouncer bug but indeed a dbal issue.

Here is a very similar issue/pull request from node.js's pgsql driver resolving the exact same issue:
https://github.com/brianc/node-postgres/pull/356


@joeshaw @jobi do you have any sugguetion to change/fix?


Fix indent.


This renders DBAL unusable with pgbouncer -- I am forced to downgrade.


There are two remote names here


Cleanup.


@Ocramius this is the exact same change that needs to be made:

https://github.com/Hebo/node-postgres/commit/323a2f9f49faf8ea183b256a74dace0e8332304a

I'll see if I can find a bug report -- if not maybe I can create a pull request.


pr/sockets almost works.  The only remaining issue is that the call to `fi_eq_sread` is returning `-FI_EAGAIN`, but it should return `-FI_EAVAIL` to indicate that an error is available.


There is a possibility of a panic when the MAC is invalid


package quadro.de.horarios;

import java.awt.GridLayout;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTable;

public class Janela extends JFrame {
	 JPanel painelFundo;
	 JTable tabela;
	 JScrollPane barraRolagem;

	 String [] semana =  {"Segunda","Terça", "Quarta", "Quinta", "Sexta"};

	 Object [][] disciplinas =
	  {
		{"Programação Orientada a Objetos I",
		 "Projeto Integrador I",
		 "Análise e desenvolvimento de sistemas",
		 "Sistemas Operacionais",
		 "Análise e Desenvolvimento de Sistemas"
		},
		 {"Programação Orientada a Objetos I",
		  "Interação humano-computador",
		  "Interação humano-computador",
		  "Projeto Integrador I",
		  "Sistemas Operacionais"
		 }
	 };

	public Janela() {
		super("Quadro de horários acadêmico");

   }

	public void criaJanela(){

		painelFundo = new JPanel();
		painelFundo.setLayout(new GridLayout(1, 1));

		tabela = new JTable(disciplinas, semana);

		barraRolagem = new JScrollPane(tabela);
		painelFundo.add(barraRolagem);

		getContentPane().add(painelFundo);

		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setSize(500, 120);
		setVisible(true);

	}

	public static void main(String[] args) {

		 Janela janela = new Janela();
		 janela.criaJanela();

	}

}


Transaction bug still not replacing SignUpFragment with KingdomFragment


My name is xiaojinhua, major in computer, like programming. I hope that I will be a excellent programmer.


Transaction bug stil not fixed...


double with


var Calculate = {};

แบบนี้ได้ไหมครับ


se crea README.txt


bump @tridge


90946736_WSHP_#4567 (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1671 of 1676 tests passed)

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


@Shougo
Please elaborate.  This change was necessary, otherwise it failed for UltiSnips.
If I remember correctly the code (`script`) was from https://github.com/SirVer/ultisnips/blob/master/autoload/UltiSnips/bootstrap.vim#L31-39.


Right. it's already been set by the open ledger code.


@bors: retry
* infrastructure problems


@valib thx :)


I just confirmed the fix works locally... IE, passing the client_encoding parameter directly rather than through the 'options' parameter


90946736_WSHP_#4567 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1670 of 1676 tests passed)

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


saw approval from jdm
at https://github.com/Manishearth/servo/commit/1a952b935b11dfbf6523da447444fe5b862384fd


merging Manishearth/servo/trace_raw = 1a952b93 into auto


Manishearth/servo/trace_raw = 1a952b93 merged ok, testing candidate = 5dd43bf8


英文我觉得没必要这样格式化。处理中文情况就好了。


这里 `\` 我觉得用 `、` 更合适。


90946736_WSHP_#4567 (Nigusse) - 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)


@gwelymernans, should this be `79` instead of `80`?

https://www.python.org/dev/peps/pep-0008/#maximum-line-length

Thanks


Just tried it again, with your current branch, and added prints:

    b'import vim'
    b'import vim, os, sys'
    b"sourced_file = vim.eval('s:SourcedFile')"
    b"while not os.path.exists(os.path.join(sourced_file, 'pythonx')): sourced_file = os.path.dirname(sourced_file)"
    b"module_path = os.path.join(sourced_file, 'pythonx')"
    b'vim.command("let g:UltiSnipsPythonPath = \'%s\'" % module_path)'
    b"if not hasattr(vim, 'VIM_SPECIAL_PATH'): sys.path.append(module_path)"
    b'from UltiSnips import UltiSnips_Manager'
    Error detected while processing function UltiSnips#bootstrap#Bootstrap..provider#python3#Call:
    line    1:
    TypeError("Can't mix strings and bytes in path components",)
    Traceback (most recent call last):
      File "…/nvim-python-client/neovim/msgpack_rpc/session.py", line 168, in handler
        rv = self._request_cb(name, args)
      File "…/nvim-python-client/neovim/api/common.py", line 229, in filter_request_cb
        walk(self._in, args, self, name, 'request'))
      File "…/nvim-python-client/neovim/plugin/host.py", line 65, in _on_request
        rv = handler(*args)
      File "/usr/local/share/nvim/runtime/autoload/provider/script_host.py", line 78, in python_execute
        exec(script, self.module.__dict__)
      File "<string>", line 1, in <module>
    TypeError: Can't mix strings and bytes in path components


汉字和全角标点之间，不建议增加空格。


You could instead conditionally build opengl only when you are on a supported arch.


这个专有名词不是这样大小写的吧。。。


The url helper should be `topic_post_summary_path(@topic, @post)`.


nitpick: could move this class into TrackedOp.cc.


`<%= form_for @summary, url: topic_post_summary_path(@topic, @post) do |f| %>`


`@summary = @post.summary`


Yeah I added Beam, Crescendo, Decrescendo, Glissando and the like for similar reasons.


This caused #1130


That will be fantastic.

By the way, one funny issue that comes up is that some of the outputs currently include solve times, which of course differ from run to run. This unfortunately leads to the output files changing despite no changes to the example scripts.


@chibill That was my original idea.


I'd just have it check my condition, if that one doesn't hold, we go out and about for all others.


One of causes of root element missing error.  I will fix with my next commit.


It ain't HermiT being smug. There is a W3C test stating that 1 and 01 are distinct as literals, although they have a coincident meaning. I.e., the literals "1"^^xsd:int and "01"^^xsd:int mean the same thing, but need to be distinct - and the OWL API is not allowed to do normalization. Why this is considered a good idea is past my ability to comprehend, but HermiT is only complying with the powers that be.

There's another test where two XML literals have to be considered equivalent if there is extra whitespace, e.g., a carriage return in the middle of the XML string. Which is ironic, since an XML parser would report that there is a text node there - so the enforced meaning is /not/ the meaning determined by the XML spec.

I feel the need to flog this horse, even though the smell tells me it's dead. I shall try and resist and do something constructive instead.


We could just add "printing=False" to the solve call?


I guess this is a typo.


```other == this``` calls the ```operator ==``` method a few lines up - it's just a regular function call with fancy syntax. The overhead of the function call is slowing down the check, so I've just put in the logic directly.

The sets added in the earlier commit call the ```Equals(CPos)``` method a ton when determining if cells are duplicates or not, so speeding this up helps us a fair amount.


ok
you modify the instruction lists.
now all unit tests work.

On Fri, Mar 27, 2015 at 12:48 PM, Jiashuo Li <notifications@github.com>
wrote:

> Changing photo won't cause problem. You can try debugging it and find where
> the problem is.
> On Mar 27, 2015 12:46 PM, "Hanadi O Mardah" <notifications@github.com>
> wrote:
>
>
> > this is first time for me using smartGit.
> > sorry.
> > about instruction FORWARD,BACKWARD,LEFT, and RIGHT
> > I think you changed to photos instructions so that the unit tests for
> them
> > do not pass.
> > Thanks,
> >
> > On Fri, Mar 27, 2015 at 12:39 PM, Jiashuo Li <notifications@github.com>
> > wrote:
> >
> > > Please do not directly work on master. Thank you.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438783
> > >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438913
> >
>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438957>
> .
>


Thanks Mark.  I know it's good practice but Is it required to commit a SNAPSHOT?


I don't think "narcotics' is in the bootstrap data for this test, is it? Ideally this test would loop over all the field values in the bootstrap data and ensure they aren't included..


hlavně, abych dosáhli oddělení funkční části od gui, potřebuji i parametr výstupní složky, který tam z gui pošlu, test, který jsem vytvořil zkouší kopírování, ale nepřekopíruje data, protože nemá vstup.


"an actual, non contrived example where this facility is needed in the wild" - by more than a handful of people, I'd add. You and me too, mate. <shakes cane/>


Ideally I would like to be able:
1) Send the command
2) Update data
3) Re-run the JS function draw(data, option) and have the charts update


Here's a typo. Sorry, was the wrong line before.


What do you mean?

On Fri, Mar 27, 2015 at 3:38 PM, Dave Loyall <notifications@github.com>
wrote:

> What's this? Is it in the public domain?
>
> --
> Reply to this email directly or view it on GitHub
> <https://github.com/sekritskwurl/crypto_fun/commit/378d8bcc60d862af01af6356351d7c73b478a1d2#commitcomment-10438756>
> .
>


Aici ar fi misto sa poti adauga la subtitlu (sumar) si dimensiunea elementelor selectate)


That's probably the best option.


Merged to release, thanks.
On Mar 27, 2015 1:03 PM, "Hanadi O Mardah" <notifications@github.com> wrote:

> ok
> you modify the instruction lists.
> now all unit tests work.
>
>
> On Fri, Mar 27, 2015 at 12:48 PM, Jiashuo Li <notifications@github.com>
> wrote:
>
> > Changing photo won't cause problem. You can try debugging it and find
> where
> > the problem is.
> > On Mar 27, 2015 12:46 PM, "Hanadi O Mardah" <notifications@github.com>
> > wrote:
> >
> >
> > > this is first time for me using smartGit.
> > > sorry.
> > > about instruction FORWARD,BACKWARD,LEFT, and RIGHT
> > > I think you changed to photos instructions so that the unit tests for
> > them
> > > do not pass.
> > > Thanks,
> > >
> > > On Fri, Mar 27, 2015 at 12:39 PM, Jiashuo Li <notifications@github.com
> >
> > > wrote:
> > >
> > > > Please do not directly work on master. Thank you.
> > > >
> > > > —
> > > > Reply to this email directly or view it on GitHub
> > > > <
> > >
> >
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438783
> > > >
> > > > .
> > > >
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <
> >
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438913
> > >
> >
> > > .
> > >
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10438957
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ZijinZheng/iRobotGUI/commit/1d63d449a10c8d3839241169b7e09cdc8d6d298c#commitcomment-10439149>
> .
>


this is a simple RSA implementation in python3 with support for pycrypto for random number generation and 8192 bit key size.


sample keys included.


pdf output for #3


Somehow the node process is still blocked if I call `.stop()`. Even If I change this line.


Привет от Знатоков!


then thats your style and therefore ok


I mean, it's a simple option, maybe a first step. But it's not actually *safe*, because the `Charge.create` method actually fires a POST request, and that request could fail. So it'd be better to check the result of that request, and because a request like that is asynchronous, you cannot just wait there for the result (it would block your entire application!). In order to do so, you'd need to use the infamous Webhooks. It's basically an endpoint in your application Stripe will POST to, to notify you about something (like, a successful charge).


Toppers! God påske!


Especially since this is 100% pointless anyway since his name is in the Git
history anyway.
On Mar 27, 2015 2:54 PM, "Lucy Firman" <notifications@github.com> wrote:

> I do have to agree with Alex (Not Alex 2, Alex Prime) on this. My point in
> #stormbit the other day was literally the complete opposite of this; I was
> making a point that there is no reason to worry about having your name
> online. I don't think StormBit's website is going to be the go-to for any
> stalker; you're much easier to find on other sites. As is the case with
> pretty much everyone; why worry?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StormBit/stormbit.github.io/commit/fc8dd2fa87cd1e4152bf0d618ba325ccae9d81eb#commitcomment-10439046>
> .
>


:+1:

The `objCType` was a fun find though :-)


what's the difference between those 2 lines exactly?


that make noise on ctrll+c when running

	March 27, 2015 - 14:06:02
	Django version 1.7.7, using settings 'importd'
	Starting development server at http://0.0.0.0:8080/
	Quit the server with CONTROL-C.
	^CException AttributeError: Exception AttributeError"'None: Typ"e' 'NobonjeeTyctpe h' asobj noect a httasri nobut ate tr'nibowute'" ' in now'" in <bound method D.__del__ of <importd.D object at 0x7ff420b71790>> ignored
	<bound method D.__del__ of <importd.D object at 0x7fc470b996d0>> ignored


this is no longer needed?


But but buuuuuuut! From reading [the documentation](https://stripe.com/docs/api/ruby#create_charge), it seems like Stripe is smart enough to actually handle the asynchronous aspect of the charging process for you. If the POST request fails, you'll get a `Stripe::CardError`, so you could actually "cancel" (as in, revert) the role update in the `rescue` block, to set it back to the default value.


Oh totally!  Have to file that one away.


Also relevant: http://whois.domaintools.com/arghlex.net
On Mar 27, 2015 3:07 PM, "Alex Ingram" <alex@alexingram.net> wrote:

> Especially since this is 100% pointless anyway since his name is in the
> Git history anyway.
> On Mar 27, 2015 2:54 PM, "Lucy Firman" <notifications@github.com> wrote:
>
>> I do have to agree with Alex (Not Alex 2, Alex Prime) on this. My point
>> in #stormbit the other day was literally the complete opposite of this; I
>> was making a point that there is no reason to worry about having your name
>> online. I don't think StormBit's website is going to be the go-to for any
>> stalker; you're much easier to find on other sites. As is the case with
>> pretty much everyone; why worry?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/StormBit/stormbit.github.io/commit/fc8dd2fa87cd1e4152bf0d618ba325ccae9d81eb#commitcomment-10439046>
>> .
>>
>


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.90](http://teamcity.monogame.net/viewLog.html?buildId=5891&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 462, ignored: 6 Build time: 00:14:25


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


please help me.i cant ubderstand wht u mean. if u know please send me an email.im always online                            mahmoudie9@gmail.com


No version annotations for these new functions


please help meeeeeeeeeeee         mahmoudie9@gmail.com


So, I am simply checking for the lines to be empty, and the scope here is that of ```test_constructor``` method, so essentially it's in the right local scope . Could you elaborate what you mean by "belongs to that scope" please :)


Found an interesting post that might help with JUnit testing:
http://andrewtill.blogspot.nl/2012/10/junit-rule-for-javafx-controller-testing.html


player.addChatComponentMessage(new ChatComponentTranslation("message.eureka"));
player.addChatComponentMessage(new ChatComponentTranslation("message.researchFinsished", TextGetter.getTitle(key)));


You are right actually, should fix this :)


for the record.... I am deleting all linked copies of the craftbukkit source in this thread


You need that pointer by reference only for line #172. You can change root's left/right links without changing root itself. So my suggestion is to prepopulate root with empty TreeNode object without any value in it. Also *root* is perfectly fine name because we're working with subtrees not nodes.


If `JSON` isn't available, I guess you could just wrap the string in quotes and do a `.replace(/"/g, '\\"')`.

Seems like an academic discussion though. Who's actually still using such an old version of FP?

Also, if you want to enforce a version, shouldn't you set `target-player` and `swf-version` in `build.xml`? It seems like you're just building against whatever Flex SDK version is available on the system. Has the SWC actually been tested with old versions of Flash?


The XML attributes of the `WeekView` will readily be available when you add the library as a dependency. Currently you have to copy the contents of the `attrs.xml` file to support the attributes.


//file open/close


So isnt this below where the lines get defined? IMHO it should not be treated out of scope.


Right. Will do.


Fixed in c0e334a


Well its a good one. As it keeps the actual api nice and clean.


// March 2015
// Kyle Bull
//  This file is used to make a Map and be able to change the set up for the map

using UnityEngine;
using System.Collections;

public class TileMap : MonoBehaviour {
    // This needs improvement
    // Gameobject is which target you select, atm its only that target
	public GameObject selectedUnit;

	public TileType[] tileTypes;

	int[,] tiles;

	int mapSizeX = 10;
	int mapSizeY = 10;

	void Start() {
		GenerateMapData();
		GenerateMapVisual();
	}

	void GenerateMapData() {
		// Allocate our map tiles
		tiles = new int[mapSizeX,mapSizeY];

		int x,y;

		// Initialize our map tiles to be grass
		for(x=0; x < mapSizeX; x++) {
			for(y=0; y < mapSizeX; y++) {
				tiles[x,y] = 0;
			}
		}

		 //Make a big swamp area
        for(x=3; x <= 5; x++) {
            for(y=0; y < 4; y++) {
                tiles[x,y] = 1;
            }
        }

        // Let's make a u-shaped mountain range
        // Hard coded in so we can change later
        tiles[4, 4] = 2;
        tiles[5, 4] = 2;
        tiles[6, 4] = 2;
        tiles[7, 4] = 2;
        tiles[8, 4] = 2;

        tiles[4, 5] = 2;
        tiles[4, 6] = 2;
        tiles[8, 5] = 2;
        tiles[8, 6] = 2;

	}

	void GenerateMapVisual() {
		for(int x=0; x < mapSizeX; x++) {
			for(int y=0; y < mapSizeX; y++) {
				TileType tt = tileTypes[ tiles[x,y] ];
				GameObject go = (GameObject)Instantiate( tt.tileVisualPrefab, new Vector3(x, y, 0), Quaternion.identity );

                ClickableTile ct = go.GetComponent<ClickableTile>();
                ct.tileX = x;
                ct.tileY = y;
                ct.map = this;
			}
		}
	}

    public Vector3 TileCoordToWorldCoord(int x, int y)
    {
        return new Vector3(x, y, 0);
    }

    public void MoveSelectedUnitTo(int x, int y)
    {
        selectedUnit.GetComponent<Unit>().tileX = x;
        selectedUnit.GetComponent<Unit>().tileY = y;
        selectedUnit.transform.position = TileCoordToWorldCoord(x, y);
    }

}


It has. We are able to compile the SWC with a Flash9-compatible SWF.

From: Tim De Pauw [mailto:notifications@github.com]
Sent: Friday, March 27, 2015 4:13 PM
To: openvv/openvv
Cc: iRoll-Apis-Announcements
Subject: Re: [openvv] Escaping injected script tag URI with JSON.stringify (0bfaed9)

If JSON isn't available, I guess you could just wrap the string in quotes and do a .replace(/"/g, '\\"').

Seems like an academic discussion though. Who's actually still using such an old version of FP?

Also, if you want to enforce a version, shouldn't you set target-player and swf-version in build.xml? It seems like you're just building against whatever Flex SDK version is available on the system. Has the SWC actually been tested with old versions of Flash?

—
Reply to this email directly or view it on GitHub<https://github.com/openvv/openvv/commit/0bfaed9503fe1f3cfac3227a7d653732e58d666d#commitcomment-10439294>.


It's technically a breaking change that we do this.  Except that we're only expanding the alignment requirements and you never should have made one of these this way anyway (you always allocated via new function that gave you a more aligned pointer).  If a user was making one of these on their own before it was completely broken anyways


Another option is to rely on the fact that `Charge.create` is actually returning a value only if the POST request to Stripe's API is successful, which means you *could* rely on the value of `charge` (as set line 14) to conditionally update the user's `role`.

But my understanding is that it's not the way you should do it, becase if `Charge.create` fails, then an error will be raised. Meaning `charge` can only be set to an actual value, otherwise you'd go straight for the `rescue` block, bypassing any code starting on line 20. It means that you can safely run `user.update_attribute(…)` on line 20, and you need to cancel/revert it in the `rescue` block.


they need to fix their makefile really - it shouldn't even do any building at this stage, but seems to want to relink or so.


@gandy92,

Please don't forget to use this field when library has own home page. I will show it on library page later. Thanks! :+1: :blush:


This might be more an issue for us than fedmsg


Confirmed by [that post](http://stackoverflow.com/questions/26985956/checking-for-a-successful-charge-using-stripe-for-rails).


Why would TSP require asymmetric matrixes?


this file uses 4 spaces :laughing:


One of causes of root element missing error.  Disable for now in next commit.


@daviferreira @phiggins42 : I modified the test case to not include spans, but also include some other inline elements.

Interestingly, I found that if you have a `<span>` that has styles that may have come from the browser, all browsers will remove it except IE (`<span style="color:red;">` will get removed by everyone but IE).

However, if you just put a regular `<span>`, it won't be removed at all by our headless tests...


@rhb-anme GA is the easiest I think. Please go ahead and add GA to blocstagram as a 30 minute exercise. You'll need a GA account - simple. You'll need to download the GA iOS SDK - simple. You'll need to import some GA SDK files into your project - yeah that's simple too. You'll need to add some #imports to .h files and modify some declarations and add some code - simple for you now! Then run it and then, later - like tomorrow, look at GA for the data. You'll want to do this every single time you build an app - no exceptions.


bad bad bad


aws-sdk gets it's credentials from the environment variables AWS_ACCESS_KEY_ID, AWS_SESSION_TOKEN and AWS_SECRET_ACCESS_KEY in development. When deployed, it resolves it's credentials using IAM roles automatically


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1168 tests passed)

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


@typemytype: here and couple of lines before, you reverted d737cfc7c9c40d967ab17facda6695f66c193025 (unintentionally?)


@axtimwalde Well, the goal is to ensure only one commit has a given non-SNAPSHOT version. Changing it immediately is just a convenience to prevent future mistakes - in general the longer people wait the more likely they are to forget, or if someone else is working on the project they may not notice the version is not a SNAPSHOT, and do some work without changing the version.

If another commit is made on accident with the same release version specified, the following will happen:

  1. Jenkins sees a new commit on GitHub and builds the project.
  2. Jenkins tries to deploy the latest commit to Maven and fails because it already exists.
  3. Anyone who wants to look at that release in the code now cannot differentiate between the two commits, and we have lost reproducibility.

If two commits do exist on master with the same release version, the options would be:

* call that release corrupt/invalid and bump to the next logical version as soon as possible.
or
* force push master to remove/fix the second commit with the non-SNAPSHOT version

Neither of which is fantastic.


Wow.