Exit status of command 'git push origin origin/master:origin/master --tags' was 256 instead of 0.


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1639 of 1639 tests passed)

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


The room command doesn't *do* anything yet, because I just fixed errors and committed without finishing the feature ;)

I'm working on that now.


Hab nur gesehen, dass es von "The Watcher has died." zu "You have died." geändert wurde... dachte, irgenjemand dort wird sich schon was bei gedacht haben^^ Kann meinetwegen gerne rückgängig gemacht werden.


This is just temporary, until we fix this in the CCRE.


Ah. :) Sounds good lol


MarkdownDocs (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1639 of 1639 tests passed)

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


I already fixed it today on master.   Will back port to 3.0.x in a bit.


Did you change the field in DiagnosticSeverity from Strict to Information too? I can't seem to find the file on here.


Well, then it's *definitely* been the case since eec7c9e757ef6d74cc062caf5072505280ef8c3f.


the test fails when React.renderToString encounters the practice button...  the test needs to handle click events, not sure how


"consilidates" should be "consolidates" if you care about the commit message. ;)  rs=me


we need to rotate right the root if the root is left-left


Whoops. I forgot to upload the two files. Added back.


I agree that this is useful information to have. But I like keeping the readme short...if it's too long, people may not read it at all.


MarkdownDocs (Myoldmopar) - 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)


The rest of this test is essentially how I plan to watch for status.replicas=0


Oh @dashaluna, you are so right!


Great! Thanks


creo que el dueño tiene que hacer el pull


Actually Episode 4...


I'm going to set up a subdomain so we can do

    mix archive.install http://get.phoenixframework.org/v0.11.0

and for latest

    mix archive.install http://get.phoenixframework.org

When Elixir 1.1 lands, we'll just be able to do

    mix archive.install phoenix


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

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


But we don't use it anywhere.  I think it's implied that NULL is the initializer for atomic pointers.  That can't be wrong because we don't have a custom atomic_pointer type.


Very likely missing graveyards.


I don't know that language


Your code formation needs improvement but that's none of my business :P


I took it.  Just wanted the ATOMIC_INIT->ATOMIC_INT_INIT conversion and slightly cleaned up.

I didn't understand why you had the casts in get_unsafe() and set_unsafe() for example, so I removed to rely on the implicit casts.


Discussed this offline with Jean, and we agreed that "functionality" isn't completely precise. "Features" might be more accurate, but would conflict with Puppet terminology. Lacking any better ideas, we're sticking with "functionality" for now.


hpwh_residential (nmerket) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


@crawford having these as numeric UID/GID is probably inappropriate.

Is it reasonable to expect the configuration provider to know the numeric identifiers?

If we allocate the users and groups in the initramfs, we will have allocated the identifiers and will have a database available for name->id resolution.  So perhaps it makes more sense to have these be User/Group strings, create files after we do user/group allocation, and resolve the names at runtime.


:tada:


again testing


cheers mate, fixed in 378d036


is it `__checkValue` or `__check_value`?


Ditto as above.


Oops didn't even realized I added /idea :D


Spurious change, please remove


Where does `get_context()` come from if not from server?


That check is done inside a bad handle return. Not sure if "isInfo" optimization is really needed. If I were you,  I would just remove the check...


Changes are good


All of this commit seems OK.


LGTM


@cubiq Have you found any security issues of sending around the rp_cookie? I'm still getting the invalid key error despite having updated to wp 4.1.1 (someone said it was fixed in 4.1.1 ) . Sending the rp_cookie value still works but not sure if it's the right thing to do.


I just did apt-get update and then apt-get upgrade, same error, what version are you using? did you use any special upgrade instructions?


`metadata-notify` notifications do not use numeric 761 but instead a `METADATA` verb.


I just noticed your comment about being able to switch the `or` to an `and` as needed ... that makes this arel implementation very appealing.


hpwh_residential (nmerket) - i386-Windows-7-VisualStudio-12: Tests Failed

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


The casts were intentional to cover the case where hb_atomic_int_impl_t itself is a wrapper around int (i.e. QBasicAtomicInt).


Wouldn't we want to actually synthesize a metadata delete event for each of these and then handle distribution there?


This seems OK.


A couple things to consider: The .strip method is redundant now as you are removing all whitespace with .gsub. Line 2 could read "gets.chomp.downcase.gsub"

Best practice for the gsub syntax would be .gsub(/\s/, ''). /\s/ removes ALL whitespace characters which includes spaces, tabs, and newlines. Your current syntax only removes spaces. /\s/ is a "regular expression" abbreviated as "regex." You can read up on more regex for ruby here: http://ruby-doc.org/core-2.2.0/Regexp.html


No worries, there was already a bunch of Eclipse and other stuff, so it would not be at all remiss to add to the pile. But I'm always trying to eliminate this stuff when I find it, so in fact your commit was helpful in bringing to light the other stuff that needed to be removed.


Debian packages for `debian:jessie` are not available from ESL at this time, hence only updating Ubuntu.


done


True, doesn't cost that much to make all these tests even if not enabled for LogInfo


Wouldn't the implicit cast work?


I can put it back in if it helps.


Hey @yosiat - Apologies, I take it back :) I think my issues are elsewhere (iTerm).

Am I right in thinking this can't be managed via Vundle? I just have to drop this in .vim/colors manually?


Excellent, thanks for that :)


Thanks sbliven, traveling at the moment. Will remove as soon as I have access to a computer.


![screen shot 2015-04-08 at 5 49 46 pm](https://cloud.githubusercontent.com/assets/1529387/7057847/b0af7d32-de17-11e4-8a11-aefcfe991478.png)

:P


Hi @drlaj ,
This is alright :)

In iTerm the only problem is the background color - replacing the color palette to Solarized Dark might fix the problem.

I have dropped this to .vim/colors manually, I don't use (and don't know) Vundle, I am using Pathogen.
Please let me know if it worked for you in Vundle, if so I will add installation guide to the readme.


Another best practice in coding is DRY ("Don't repeat yourself"). You can apply this here by combining lines 7, 8 and 9 into "times = gets.to_i - 1" Chomp is redundant here as to_i gets rid of all whitespace for you : )


Навеяно возможностью [генерации кода в Slick](http://slick.typesafe.com/doc/2.1.0/code-generation.html) и вот этим [примером кастомизации](https://github.com/slick/slick-codegen-customization-example).

@v1pka @hodkoff2 @Argelein обратите внимание на файл с сгенерированным маппингом. Это еще неокончательная версия. Возможно и вовсе не стоит его оставлять, а всего лишь использовать для разумного копипаста.

@v1pka смущает то, что кейс классы не наследуются от одного предка, но это можно поправить. Что-то еще не так?

Пока две идеи:
* При необходимости изменить модель меняем ее в БД руками. Потом перегенерируем маппинг. Файл сгенерированного маппинга включаем в проект. При этом и файл с эволюциями сгенерируется, если это в конфиге включить.
* При необходимости изменить модель меняем ее в БД руками. Генерируем маппинг, но используем полученный файл только как средство для разумного копипаста кейс класса и описания таблицы.


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


WARNING: while all the functions are implemented, they don't yet protect against concurrent access! You may experience race-condition-related bugs. These will be fixed once I figure out java concurrency...


Yeah, they are definitely unnecessary. IIRC, at least one person (in addition to me) has mentioned they find them helpful for readability of this pattern, and I believe I actually went through and changed all the other places where we had `&foo->bar` to `&(foo->bar)` for consistency.


@omarwanis @AhmadHatem @karimFarid


please review


This doesn't work.  How did it even compile for you?  chars_len (which is used in the second ALLOCATE_ARRAY) is computed by the loop.  Now, we can just allocate buffer->len * 2 for that one as well.  I don't think there's any problem with doing that.


Are you profiling hb-coretext or suggesting just based on reading the code?


Is that Sebastiaan de With?


initial merge


this indentation got really gross.


const ref seems like it's unnecessary -- const should be sufficient. Ref removes the ability to call this with an inline call.. e.g.   method(StructName(), otherVar);


And you're scenario should not what it's doin but rather the steps of the action that will be taken.

You could say:

Scenario: Registrering a new account
Scenario: Logging in to the website
etc..


Why is this not Throwable? Seems possible to get a never ending rotation, right?


Could just say: return %s();


The commit message is missing.


@yjkim0
CDOMAIN을 사용하지 말고
$key, $value, $expire = 0, $path = '', $domain = '', $secure = FALSE, $httponly = FALSE
method arguments로 넘겨서 처리하는게 좋을것 같습니다.


This commit message is missing. And it does not follow the specs for the project - the test was supposed to be in its own commit before the implementation.


The car does not start out moving. Look at the code. The speed is 0 on initialization. Must accelerate first. Read the code and tests provided.


grpc supports a "stream" input/output type. This has three modes: bidi, client_stream and server_stream. All examples of bidi are one-to-one, but documentation about the spec is sparse. client_stream is a reduce method, server stream is a generator pattern.

Generally, method signatures look like:
void (in, out) for bidi
struct  {  void(in); out(out); } for client_stream
and
struct { bool (in, ref out); } for server_stream.

Still working on that prototype.


Same issue here - car is not accelerating first. So its speed will be zero.


Was this part of a PR? I've forgotten (again) how to jump from a commit to a PR in Github. The link in the email was to the commit.


> I still find it confusing that we define size in bits instead of bytes.

:+1:


Yep. Because you would reuse the steps. Although if you are familiar with nested steps or harnesses, these three lines would not be necessary and could be transformed to:

```
Then I should see the login form
```

But that's something you can do eventually.


Actually we can have the `var` and make it private and use `Meteor.call('fetchFeeds');` in `/telescope-post-by-feed/lib/server/cron.js` as proposed in https://github.com/TelescopeJS/Telescope/pull/894 and that works too.


LGTM


@bloodbare

Is there documentation anywhere that would help a user to know how to set up icons for a new content type?  I'm working on an add-on targeting Plone 5 and would like my content types to have nice icons.  Maybe I can use one of the icons provided here by the theme, but I can't find anywhere any documentation that shows what my choices are.

Alternatively, I could add icons of my own choosing, but how exactly do I do that without disrupting the existing icons registered here in the theme?

Any suggestions on where to look for this information?

Thanks!

c


dont I need to protect then?

```
this.currentUser && this.currentUser.get("trust_level")
```


You should add an observation to determine that the item that was once in the anon cart was carried over to the authenticated cart.


This is not right. This file is required.


Check [this answer in SE](http://unix.stackexchange.com/a/72092). Basically, you need to add `stty -ixon` to your `.bashrc` or `.zshrc`.


@yjkim0 이 코드의 목적은 무엇인가요?


The h is not silent in that word, regardless of accents.
It's like saying `give me an half gallon of milk`


@yjkim0 raw 는 말그대로 가공되지 않은 그대로를 전달하는것인데 어떤 경우에 false가 필요한가요?


$domain을 파라미터로 받는 다면
각각의 domain config에 cookie domain을 지정해두고 module에서는 해당 값만 사용한다면 문제없습니다.

그렇게 할까요?


Yeah gimme an 'alf gallon, wots ur trouble m8


넵 그렇게 하는것이 좋을것 같습니다~


@yjkim0 이부분은 제가 미리 수정해둔 코드가 있습니다. 그것을 커밋할께요~


HotFix-Regressions (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1173 of 1173 tests passed)

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


k switched it over to that syntax. this.get('currentUser.trust_level') I guess it is one case where get is handy.


@yjkim0 넵 제가 반영해서 커밋할께요~


array 형태의 data를 받기위해서 array.int 이런식으로 넘기도록 했거든요~

다른 좋은 방법을 추천해주세요 ㅠ.ㅠ


@yjkim0 넵 알겠습니다. 수정된 부분이 조금 있어서 반영해서 커밋할께요~


Seeing that this has a lot of scenarios it would be ideal if you split this feature into multiple features.

Feature: Edit Cart
Feature: Persist Anonymous Cart
Feature: Adding to Cart (not same as editing as editing implies that there already exists a cart and it's contents will be edited).

And what else you may think is necessary.


This looks like it needs some line breaks. Thoughts?


can we not go straight from our config to the etcd config?


Don't leave my config set to quiet


Also, Docker provision is now destructive to Docker containers!


Why is this sort() called twice? Is there any reason to do so?
It seems to me that we can move line 51 to here.
proxyModel_->sort(settings.sortColumn(), settings.sortOrder());


넵 커밋해주시면 받아서 코드 정리하겠습니다.


@Paxxi here GetCharWidth return different value when call _ChangeWordList from ChangeWordList and from OnMessage. don't know why? and this cause problems on page up and down.


Do you mean store a single height that is just the max of the two tree heights, in addition to the leftHeight and rightHeight?


It is best that the adding of an address is on a different feature and the shipping estimate is on another one as well.
```
Feature: Adding an Address
Feature: Updating an Address
Feature: Shipping estimate
  Scenario: Has an existing address
  Scenario: Has no existing address
  Scenario: User wants to ship to a new address
```

:dancers:


입력된 값이 없는(null) 경우 null이 그대로 넘어와서 default value set이 안되어서
null인 경우에는 false를 보내서 default value를 set 할 수 있도록 처리해둔거에요~


Instead of:

```
Then Item should be removed from cart
```

better would be:

```
Then I should no longer see item 'x' on cart
```

:dancer:


isDelete() 조건이면 충분함..
isDropIfEmpty()는 isDelete()가 true인 경우에 한하여 true가 될 수 있음.


넵 그렇게 변경할께요~


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1129 of 1129 tests passed)

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


Try at least a happy and sad path in all of your tests.

:dancers:

That is the Svajone way. :alien:

Because having a sad and happy path would at least tell you how your applicatoin or the system handles invalid entries, and having none can be proof of a lacking feature that needs to be added.


Bottom line (at the top!): I'm reminding myself that this comes back to "[never hesitate to sacrifice completeness for clarity](http://software-carpentry.org/blog/2015/03/teaching-tips.html)" and that we should just drop the aside about ON vs. WHERE, as you have already done.

My attempt to elucidate the points I was trying to get at below, FWIW...

> But as far as we're concerned with our inner-joins, ON *is* strictly an alias for WHERE.

Correct.  My thought was that hinting that there is a distinction between the two, even though we won't see it in the lesson, might be valuable when a student later learns about outer joins and wonders why ON and WHERE are acting strangely when they're supposed to be the same.  That said, see the first paragraph.  :)

> I think pre- vs. post-join filtering is mostly up to the query parser. Can you explain how I can see that difference in action?  I think the outer-join semantics are pretty clear, but I can't think of a way to see different results or timing with inner joins.

As I see it, the concept of when ON and WHERE are applied remains the same, it's instead that they're rendered exchangeable by the nature of an inner join.  The fact that some query plans will exchange ordering of ON/WHERE freely for inner joins is an implementation detail.  I think we're considering this from two different viewpoints: 1) as a conceptual framework for thinking of SQL's clauses, and 2) as how they actually work in practice.


sorry, my editor automatically deletes trailing whitespace...


one more error... Shop gold goes crazy


single quote '0' ?


I moved this into the function since I assumed you want to calculate the style based on each feature/resolution passed into the function


for reference: http://openlayers.org/en/v3.1.1/apidoc/ol.layer.Vector.html#setStyle


include


awkward


Removing FINALs


When in rails, use Object.blank? when checking for empty values. "" is not nil. I've fixed this in master http://api.rubyonrails.org/classes/Object.html#method-i-blank-3F


This terrible commit was due to a very simple error where I was making my changes on master instead of the dedicated branch I have for future changes, and I had to quickly resolve this issue.


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1129 of 1129 tests passed)

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


Comment L183 with the following plztks:

```
# RFC 6125, section 6.4.3, subitem 2.
# If the wildcard character is the only character of the left-most
# label in the presented identifier, the client SHOULD NOT compare
# against anything but the left-most label of the reference
# identifier (e.g., *.example.com would match foo.example.com but
# not bar.foo.example.com or example.com).
```


I think there is a `filter` because `parse("")` returns `List("")` and not `List()` and `""` is then failing `Parser[A]`. This should be handled differently.


shouldn't that dash be `\-`?


What is the Linux ABI?


After clarifying with Fabrice, gecko won't change the state of addons but this could be triggered in other apps (i.e. system app). I would revise the interface design to rely on the state change event to make it more robust. Thanks for pointing this out!


Out of curiosity what do you mean by "leakage of error suppression"?


rax retry tests


Deduped.


kapi.NamespaceDefault


Same


@andrewor14  @mengxr
This change seemed to be causing some bug:

```
org.apache.spark.SparkException: Job aborted due to stage failure: Task 1 in stage 200.0 failed 4 times, most recent failure: Lost task 1.3 in stage 200.0 (TID 2547, 10dian70.domain.test): java.lang.ArrayIndexOutOfBoundsException: -1
        at org.apache.spark.graphx.util.collection.GraphXPrimitiveKeyOpenHashMap$mcJI$sp.apply$mcJI$sp(GraphXPrimitiveKeyOpenHashMap.scala:64)
        at org.apache.spark.graphx.impl.EdgePartition.updateVertices(EdgePartition.scala:91)
        at org.apache.spark.graphx.impl.ReplicatedVertexView$$anonfun$2$$anonfun$apply$1.apply(ReplicatedVertexView.scala:75)
        at org.apache.spark.graphx.impl.ReplicatedVertexView$$anonfun$2$$anonfun$apply$1.apply(ReplicatedVertexView.scala:73)
        at scala.collection.Iterator$$anon$11.next(Iterator.scala:328)
        at scala.collection.Iterator$$anon$13.hasNext(Iterator.scala:371)
        at org.apache.spark.util.collection.ExternalSorter.insertAll(ExternalSorter.scala:210)
        at org.apache.spark.shuffle.sort.SortShuffleWriter.write(SortShuffleWriter.scala:63)
        at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:68)
        at org.apache.spark.scheduler.ShuffleMapTask.runTask(ShuffleMapTask.scala:41)
        at org.apache.spark.scheduler.Task.run(Task.scala:64)
        at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:203)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
        at java.lang.Thread.run(Thread.java:744)
```


this looks like a great contribution to the project


Sneaky! The commit message does not say anything about this case change ;-)


I had changed the package name already before releasing the app to play store. Just that, I pushed this commit yesterday. The one on the play store has this new package only.

Anyways, thanks for advice. :)


Synchronizing your submodules gave me this error:

````Synchronizing submodule url for 'modules/completion/external'
Synchronizing submodule url for 'modules/history-substring-search/external'
Synchronizing submodule url for 'modules/prompt/external/agnoster'
Synchronizing submodule url for 'modules/prompt/external/powerline'
Synchronizing submodule url for 'modules/prompt/external/pure'
Synchronizing submodule url for 'modules/syntax-highlighting/external'
fatal: reference is not a tree: d0381d859c0eb8d52a6211281e622e89153ef812
Unable to checkout 'd0381d859c0eb8d52a6211281e622e89153ef812' in submodule path 'modules/history-substring-search/external'
````

I needed to run this command to, manually, change the commit hash without the need to remove/ add again the submodule:

```# http://blog.evan.pro/update-a-git-submodule-to-a-new-commit-without-cloning-the-submodule
git update-index --cacheinfo 160000 6d7bc718aee81b9afa9e146b5567be014b3ff8f0 modules/syntax-highlighting/external```

Submodules are a mess. :(


how do i compile this ?

grtz Joey


test comment


The WRS service will overwrite the PREROUTING chain, which isn't used by Asus' NAT loopback.  That's why my loopback requires the firewall to be restarted after WRS is restarted.


These could all be `auto`.


We can use taskkill -pid


test


g


Actually, we're in Tcl land here and our `wish.exe` is a MinGW one, therefore I trust it not to be translated.

If in doubt, you can always add the line

```tcl
exec cmd /k
```
say, before line 646.

Note: you *cannot* test this using `echo.exe` because that is an MSys2 program and would perform that posix-to-windows path mangling.


Why
the
fascination
with
single
lines?

Heck, everywhere else I've ever worked, this would be written as so:

    LedgerEntrySet& view() { return mNodes; }

taking one line instead of six(!)


Yeah that would work as well, I don't think it makes a huge difference
either way. This is just shorter :)

On Wed, Apr 8, 2015 at 12:05 PM, Azizur Rahman <notifications@github.com>
wrote:

> Actually we can have the var and make it private and use
> Meteor.call('fetchFeeds'); in /telescope-post-by-feed/lib/server/cron.js
> as proposed in #894 <https://github.com/TelescopeJS/Telescope/pull/894>
> and that works too.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TelescopeJS/Telescope/commit/8f94eaa765435ed687660b1df58874b19605260e#commitcomment-10615821>
> .
>


This could just be `return {terResult, didApply};`


Why not make the log message the same as the exception message, to make later debugging easier?


Ditto the above.


Shouldn't we avoid using an unbounded buffer? Also, as I was concerned, `cx_id` was meant for object full names, am I wrong?


Looks reasonable to me! Thanks, @Freso.


Commit message got eaten, this was just run through the code formatter


@yjkim0 말씀주신부분 고민을 좀 해볼께요~

사용하는 예제 코드가 있다면 도움이 될것 같습니다.


@ndmitchell ah, yes. Oversight on my part. I was invoking:

```
$ dist/build/ghcid/ghcid -v -c "cabal exec ghci src/Ghcid.hs"
```

(that's almost inception-like) so didn't notice.


Oh man, I've been trying for so many hours to get `var_AddCallback` to work from within xdg.c on my own, and then I've decided to try your variant and for some reason it gives the same error (apparently there are no set variables anyway):

```
[00000000018de188] core playlist debug: keeping audio output
[00000000019157b8] core interface debug: looking for interface module matching "dpms,none": 14 candidates
[00000000018de188] core playlist error: cannot add callback 0x7f03b1cbd4c0 to nonexistent variable 'activity'
[00000000019157b8] core interface debug: using interface module "dpms"
[0000000001916f98] core interface debug: looking for interface module matching "hotkeys,none": 14 candidates
```

and I was sure yours would work thinking that it's just the xdg inhibit which was doing it somehow differently that I was getting that  "cannot add callback ... to nonexistent variable"

I am trying this on latest vlc-git commit https://github.com/videolan/vlc/commit/50accb82e6e8b10a3f3493091dfcd832405ba234

Please, if you have any ideas on how to get the callback to work, let me know.
What I initially wanted to do, is add a callback which would detect Pause(and Play) and execute the Inhibit function within xdg.c appropriately. Failed so hard at getting this callback to register ... I just can't believe it. And now I see that even your 'dpms' module isn't able to register any callbacks (tried 'activity' and 'intf-event')

Btw, they changed the Modules.am into Makefile.am
This is what I used for that: https://github.com/emanueLczirai/coostomhuston/blob/ae4897dc1e075203201d7357f749a2fab7c7ff16/system/lenovo%20z575/OS/manjaro/filesystem%20now/home/emacs/build/vlc-git/0001-Add-dpms-module-to-switch-it-off-while-playing.patch#L245

Cheers,
E


Hi Suresh,

We already have those configuration those values at clLog.xml. Why those are redefine and limit to 512?

Regards,
Hoang


remove this page and refer to login


Ugh, this commit was only supposed to have parallel installer changes, not the outdated changes too.


rax retry tests


10 years of awesomeness!


Sorry man, that sucks.


TeamCity ChameleonForms :: Continuous Integration [Build 229](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3543&buildTypeId=bt20) is now running


Are you sure the key exists? I don't think we changed the implementation in this PR.


Make this use `HDUtils.append`, and created a `HDUtils.concat` method used elsewhere.


:birthday::birthday::birthday:


そうそう、そういう手抜きあったよね。直すの忘れてた。


I would change this line to:
`animal = gets.chomp.downcase`
This way you don't have to call `.downcase` everytime you want to use `animal`. It's called sanitizing.


I would change this line to:
`sound = gets.chomp.downcase`
Same as line 2 above.


I would change this to
`puts "The " + animal + " goes " + (sound + ", ") * (multiplier - 1) + sound + "."`


This also needs mysql-server -> mysql-server | virtual-mysql-server to work with mysql-5.6


Oops, need to revert this commit


", when withDelete is false."


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0+009](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3543&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3004 Build time: 0:0:0


", when withDelete is true."


TeamCity ChameleonForms :: Continuous Integration [Build 230](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3544&buildTypeId=bt20) is now running


boolean result = decodeFile(file);


if (!result && file.exists()) { ...}


i prefer isOk, 'result' seems like a result code


* ExtendedBTreeGetOperationImpl (asyncBopGet), when withDelete is false.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1129 of 1129 tests passed)

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


ETC는 사용되지 않는다면, 제거하는 것이 좋지 않을까요 ??


위의 주석 대신에, NONE은 undefined 의미라고 하는 것이 좋을 것 같음.


OK, that makes sense.

Actually, using `swap` requires we have a minimum base version of [4.3.0.0](http://hackage.haskell.org/package/base-4.3.0.0/docs/Data-Tuple.html); this corresponds to GHC 7.0.1.

Whilst I don't think it's likely, it's possible that someone is still using fgl with older versions of GHC (and possibly other Haskell implementations); thus, I'm hesitant to use newer versions of base.  So maybe replace this with an internal definition (e.g. `uncurry (flip (,))` or just directly).


Was this resulting in double-parens?


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest114+011](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3544&buildTypeId=bt20) outcome was **SUCCESS**
Summary: Tests passed: 3004 Build time: 0:0:0


Making this a command line parameter means most users will be unaware of it. Also, it might be confusing since the deletion of the data is implicit in the save command. I've removed the parameter, and added a menu option to discard the count information instead (see b5d55742b0bea6d0b48cee9f98ba9fb90c7e808a).

Please note that already when generating the scan report in WhiteRabbit you can opt not to extract the count information, which will also make the scan a lot faster.


Do we actually need anything when the "down" function is called?


Oh right. I forget Heroku has that limitation.


Are you sure Compat is up to date?


should set it to null here as default as well and/or add comment using `#` to say that it is optional


ah ok... that is more simplified on one hand (more so than .split), but also a bit clunky


Default of DRIVE changed to STRAIGHT.


Duplication. Use the methods defined in SteerParam instead.


Yes, that looks OK then.

You could possibly make the definition of `es` cleaner by having it just return `[Edge]`, and then use the new `toLEdge` function (currently in the `more-tests` branch but I can put that over the master if you want access to it).


Changed the conversion to linear.


Snap to STRAIGHT.


So you mentioned a potential problem with this checkin.
Have you had a chance to investigate those issues?


These look good.


Angle is changed to int in order to simplify the process.


Look. You should not create another copy of the same file just because you modified it. Delete this directory and update the original file. Git preserves all the history of your changes and that's how you can track your changes.


@valib I'm not sure why this change was needed. Could you explain it?


Excellent. +1


Sure Professor. I will update the git.
I thought of removing it after verifying the new code.
Thank you,
Prasanth

PRASANTH VIVEKANANDAN
[KU Jayhawk]

On Apr 7, 2015, at 11:23 PM, Heechul Yun <notifications@github.com<mailto:notifications@github.com>> wrote:

Look. You should not create another copy of the same file just because you modified it. Delete this directory and update the original file. Git preserves all the history of your changes and that's how you can track your changes.

—
Reply to this email directly or view it on GitHub<https://github.com/tharunprasanth/DRONE/commit/98d86e53d0ecef1114a838db3d69a741046eaa99#commitcomment-10616287>.


Totally agree, but it's only like 30 characters longer than it was before ;-p


CAS operation에 대해 type 지정이 빠져있는 것 같음.


@fh1360 Stop spamming.


ここをいじらなかったのはこの方が早いためです。

値が小さい方から処理して行く方が早く条件に合致するので。


I hope you going to merge the rest of this branch soon as well ;-)


Would `match` not be faster here?


Also, custom message handlers on the python side must accept a new buffers argument, and custom fields that contain references to other models must have a custom JavaScript model class.  I think those are the two backwards incompatible changes.


Shouldn't this be self.Plugin, not self.Object?


You are right. When I added line 51, I probably thought that the settings should only be accessed from here, that it is cleaner not to introduce the settings into `chdir()` and that the settings are only updated before terminating the program, but the settings are updated after every change, so the suggested change should work fine (replacing line 332 by line 51 and copying line 45 before it).
Shall I add a commit and/or fixup the existing commit?


Right?! :)


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


@klocey @mmuscarella hey guys. I'm going to hold off on this. I ran into a conflict when trying to merge our older course repo; not sure where it's coming from. I realized Ken's changes are to the public QB repo. I didn't fork yet. Maybe we can discuss this and other issue tomorrow. Thanks for your help!


咦好多版本。。。


use
/*
*  This format
*  for multi-line
*  comments
*/


Why are we removing the logos?


LGTM


I understand boolean logic :) try to keep comments as succinct as possible, I'll let you know later if I don't follow


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1173 of 1173 tests passed)

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


Again, try to keep comments as professional/succinct as possible, I will be able to understand why you chose to do a task a certain way.


Thanks for the pull request! It was half of the fix, the other half was to fix the init.sh path!


@Zangetsu38 Proof? I doubt that DX 12 is gonna be much faster than DX 11 without changing anything yourself. I only found one benchmark that showed the difference between draw calls/s. As far as I know the big problem is right now that we still spend a big chunk of time interpreting/recompiling the instructions and that wont be probably getting much faster in future. I think what we need instead is something like Vulkan/DX12, but for the CPU (I don't think we can directly execute instructions on the CPU?).


it's not necessary to set to NULL since smartptr already have default NULL pointer


Done. Thanks!


I back in an AppVM on USB stick.  I will comment that when I get my system up and running.  I can get you qubes-builder tomorrow, just let me know how fast you need it; it won't take long to re-create the commits since most all the changes are in `nrgaway` branch already.  The only issue I have right now is I can not sign it until I get my system restored since I do not have access to my keys and I don't have enough space to restore a backup on USB stick... I want to try to recover my HD first since it been a few weeks since last backup.  If I can't get it recovered by tomorrow, I will just over-write it with restore image.

This sucks :)  It's the third time I lost my data; every time it has been when building.  The reason it happens is since I am using BTRFS and neglected to partition properly for VM images.  Number 1, each VM should have its own BTRFS share, and COW disabled.  What happens is because of COW, disk space runs out and when this happens during a write, the system locks up; and 1st superblock get corrupted along with other data.  It has pointers to data that exceeds the disk size, which is an error and reason for failure to boot and is very difficult to recover from.  With BTRFS, its hard to tell when you running out of space; a `df` told me I have over 500GB free, but I forgot I created a snapshot last week before updating to newest package releases and have been building qubes on and off for about a week testing all my changes.


Also, you could do a check
if (atLeastOne[index].checked == true) {
disabled = false;
break;
}

because we really only care if one or more is selected. so once we find at least one, we don't want to waste memory by iterating through the rest of what could be a very long loop, for no reason.


Why devDependencies and not dependencies?


This is the key line of the fix.


I appreciate this pull request


This should be:
 rw << next_check_time << interval;


It would be so nice to be able to skip that `dangerouslySetInnerHTML` part.


Editing Lua source code for warnings is really strange. I would compile Lua with no warnings turned on if thats bothersome.


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


@ashcoding Thank you so much for your help! I am downloading SDK 4.0.0 and hope it will work well.


변수명을 opType으로 짧게하여, OperationType 과 혼돈을 줄이는 것이 어떤지 ??


I am not sure about using `ty::deref`, as it returns an Option as well as mutability which we don't need here.


함수 위치를 handlingNode get/set 아래로 옮기면 좋을 것 같아요..


Example of Function usage and passing variables.


isDelete() 조건이면 충분함..
isDropIfEmpty()는 isDelete()가 true인 경우에 한하여 true가 될 수 있음.


This is incorrect and could easily break code. mkstemp does not do exactly what tmpnam does.


ETC 연산에 대해서도 operation type을 지정해 두는 것이 complete할 것 같음.


Why are we putting this test in the extras directory instead of making it part of the 1.0.4 conformance suite?


Lines 110-123


@yjkim0 run method에서 validate::getData의 결과가 null일 경우 arguments count를 세어 3개일때는 null, 4개이면 default지정의 처리를 하는것이 맞는데 수정해주신 코드가 흐름상 결국 같은 역할을 하는 더 심플한 코드네요. 반영하겠습니다~


cloned the repository so that the master branch can be viewed by all in real time :)


It is easily possible to have ratios greater than 5 on just horizontal position and velocity due to GPS glitches, so this should not be used as a fall-back metric. If all three are above 1, then the filter could be considered unhealthy as it means the height (which is normally very reliable) is in trouble which means the underlying inertial solution is likely unhealthy


@yjkim0 말씀주신부분 고민을 좀 해볼께요~

사용하는 예제 코드가 있다면 도움이 될것 같습니다.


Example of Function usage and passing variables
*See lines 110 — 123*


Calling this file GridFieldConfig.php doesn't make a lot of sense to me—it's a double up with the framework filename and doesn't communicate the most important information. Maybe GridFieldConfigEditor as the class name and GridFieldConfigEditor.php as the filename?


why you don't use so:
'''
def palindrome(x):
    return x[::-1] or len(x) < 2
'''
Just to know


The option part is pretty simple - if it returns None, then its not a pointer, so you can return false. You do throw away mutability, but you save code dup - you wouldn't have missed ty_uniq if you'd used it here and if we add another pointer type in the future, we might miss this function.


Теперь страница уведомлений в tango выглядят страшно.


Anything in particular you mean? I'm aware of the WorldEdit API one but I think I grabbed most of the changes. It's difficult to directly just merge upstream since I diverged too much, but I'm trying to bring in the fixes manually.


It could be used as a pre-arm check as is if && with !vehicleArmed, but we should really not be arming if any of the normalised  innovations are over 1. For a pre-arm health check I would do something like this that checks all of the observations.

if ((velTestRatio > 1 || posTestRatio > 1 || etc || etc) && !vehicleArmed)


아래와 같은 필드를 post로 넘기면
```html
<input type="checkbox" name="partner_id[]" value="3>
<input type="checkbox" name="partner_id[]" value="4>
<input type="checkbox" name="partner_id[]" value="5>
```

php 에서 post로 받으면 array data가 들어옵니다.
```php
$partnerId = $_POST['partner_id'];
pr($partnerId);
```
이런 데이터를 받는 방법이 필요해서 작성해둔 코드에요~


btw you can do as: :commentary


btw you can do polymorphic: true


Gah, I thought I fixed that. Thanks for catching it; I'll stack a commit on top.


I am sorry - but this method should not be necessary; as long as no default is specified in the Json configuration the `item->getRawDouble(0)` *will* throw. However - a test enforcing this behaviour should be added - see later comment.


This line can fail in case `sourceMap` is null.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (617 of 617 tests passed)

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


The commit has much unrelated white-space changes; try to avoid that if possible - it makes review more complicated.


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


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


              |--　_layouts
　　　　|　　　|--　default.html
　　　　|--　_posts
　　　　|　　　|--　2015-04-08-hello-world.html
　　　　|--　index.html


Evil.


This handles `POST` returning nil under certain circumstances.   `POST` itself should really never return nil IMO, but that seemed outside of the patch.


Fixed it in a new commit, because I'm a nice guy jaykul/choco@07bea8b
It'll show up in the last pull request :-P


Oooh.. https://github.com/mperham/connection_pool

:+1:


Hey, see -- I told you I switched it from package to pkg


在这个小型博文系列中我们想给有兴趣研究 Android Graphics Pipeline 内部结构的开发者带来一些启发

虽然这些资料肯定能帮助我们从宏观角度理解一个简单的view是如何显示在屏幕上的

而本系列博文将带你走进 Android Graphics Pipeline 这个有趣的世界。


FYI, these separate `RUN` statements won't actually remove anything from the image, they'll just mask it out so subsequent layers of the Docker image don't see it. If you want to actually keep the image size small, you'll need to chain those commands with `&&` as is done in the Docker official images.


@interactivellama I think this has had an unintended impact on the mctheme.  Checkboxes are now getting a glow when hovered.

![image](https://cloud.githubusercontent.com/assets/4172067/7039221/60cfa9f8-dd8c-11e4-9c59-61cba0fd9425.png)


As I understand it, `dependencies` are modules that your program **require** in order to run, `depDependencies


To check the behaviour when it comes to defaulted value item:

```C++
BOOST_AUTO_TEST_CASE(WELTARG_DEFAULT_VALUE_THROWS)
{
   std::string input = "\n"
   "WELTARG\n"
   "  OP_1  ORAT  /\n"
   "/\n"

   DeckPtr deck = parser.parseString(input);
   std::shared_ptr<const EclipseGrid> grid = std::make_shared<const EclipseGrid>( 10 , 10 , 10 );
   BOOST_CHECK_THROW( Schedule(grid, deck) , std::invalid_argument);
}
```


see line 29 :)


> Thus we want to warn the
programmer for now until we have the close invocations in place
everywhere.


from commit message


特别是你对 -》 就算你对


小演示app -》 小Demo


why? Now the variable is local it goes out of scope directly after anyway.


I think it would make sense to have these changes in the main `application.yml` with fallbacks to the default values. I also think that you may want to use `OHANA_API_ENDPOINT` here instead of the Docker link variables. You can use an entrypoint script to construct `OHANA_API_ENDPOINT` based on the Docker link variables if you like.

Here's an example of me deriving an environment variable from a Docker link variable: https://github.com/synctree/docker-mediawiki/blob/master/1.24/docker-entrypoint.sh#L57


iteratee won't be necessary there if you're testing it


理念 -》 概念


Windows uses %USERPROFILE%, not $HOME


We could have a lint rule for this.


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


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.173](http://teamcity.monogame.net/viewLog.html?buildId=6081&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 475, ignored: 6 Build time: 00:15:32


Maybe, not concerned with perf here. We use `replace` in other areas like `_.template` for simplicity and the same goes here.


calling set-env here is going to break things because it's set far too late. must be the first thing that is set


This still compounds specified value with fallback value while resolving the value. I can send a PR to completely bypass this step if you want.


テーブル(モデル)にできず、シリアライズで運用しなければならない(すべき)場面があるのか、どうか。


Load revised metadata layout related to issue #1


Ahh okay, thanks :D


disabled, rather


first practice repository


you need to integrate it here:

https://github.com/hzeller/rpi-rgb-led-matrix


Ahahhahhahaha


호출하는 php 예제를 주세요. default는 전체의 default인지 아니면 개개별 디폴트인지등등을 알고 싶습니다.


If you want to try this version, update django-restframework to the newest 3.x version (currently 3.1.1) and copy events/api.py.drf3 over events/api.py. Known problems: Keyword and Place serializers don't work properly when `include=location,keywords` is in query parameters, only link exists in the data and string fields are missing.

This kludge fixes PlaceSerializer:

```
class PlaceSerializer(LinkedEventsSerializer, GeoModelSerializer):
   view_name = 'place-detail'

   def __init__(self, *args,  **kwargs):
       args = list(args)
       print ("PLACE SER", args[0])
       args[0] = Place.objects.get(id=args[0].pk)
       super(PlaceSerializer, self).__init__(*args, **kwargs)
```


I'm not sure, but, Before this PR is merged.  The [lrGraphxSGD](https://github.com/witgo/spark/tree/lrGraphxSGD) can run normally.


Perhaps "which are not predictions or completions"?


Add some logic for a local data directory, as well as a directory that gets defined by an environment variable `%INEXOR_DATA%`. This way the data doesn't need to be stored in someone's user directory. I personally prefer to keep my projects self-contained, and I can't be the only one.

I just realized I'm having the batch file also check the default location for data. This is unnecessary, but it also shouldn't hurt.


I don't understand


were you able to fiddle with it so it's feasible with ipopt now?


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


Ok, let's leave it as is.


...As well as jquery


Interesting question. Can change to Throwable - I'm just a little worried about attempting to recover from Error's that are thrown in here.


It can't be a d because there is no d in the caption.


I'm a fan of `reduce` in general, but moreso when it's construction is lightweight, like in the case of [`fold (+) [1,2,3,4,5]`](https://wiki.haskell.org/Fold). I feel that it's harder to understand when the accumulator has to be explicitly constructed.

I believe in order to write that join with a SQL string instead of arel:
```
joins(:categories).where('categories.id IN (?)', category_ids).uniq
```

I discovered the `.uniq` part as I added more tests ... I wonder if we need it in other implementations?

I [started playing with a refactor](https://github.com/mtc2013/LocalSupport/compare/split-category-display...johnnymo87:split-category-display-refactor), but I ran into an issue where the `.uniq` clashes with the `.order`. I know I can get past this problem with raw SQL by explicitly choosing to add 'organisations.updated_at` in the `SELECT DISTINCT` clause that is currently being automatically generated by `.uniq`, but maybe arel can do better?


PR #274, IMHO raising a JIRA would be too much overhead :-)


Yes, and actually I started seeing this right after updating. Plus, if I change `using` to `import` it works fine (note: in the previous message I mistakenly wrote `include` rather than `import`)

(Actually, I'm not even sure what the syntax `using Modulename: function` means exactly in general, and how it differs from `import`.)


Perhaps; I just thought the formatting sets the appostive apart just enough.


That would be possible, although it would have larger memory implications and would require refactoring existing code to a much greater extent than I wanted to do first off. This fitted what we already do in the code, and turned out to be reasonably elegant (imho).


Done.


very sass, much insult 11/10 computer hurt my feels


From a support perspective, felt that it would be better to ensure it was flushed immediately, so that we can be sure the information is there in a crash. If it's a performance issue, we could remove it - although I haven't noticed it being so thus far.


To be more precise, and for posterity: I'm on Compat version 0.3.7.


Unnecessary line


lgtm


Found BROKEN by quick.
<casadi::MX, std::allocator<casadi::MX> >&, std::vector<casadi::MX, std::allocator<casadi::MX> >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:139:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Horzsplit::evalFwd(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:157:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Horzsplit::evalAdj(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:178:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Diagsplit::evalMX(const std::vector<casadi::MX, std::allocator<casadi::MX> >&, std::vector<casadi::MX, std::allocator<casadi::MX> >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:222:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Diagsplit::evalFwd(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:244:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Diagsplit::evalAdj(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:269:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Vertsplit::evalMX(const std::vector<casadi::MX, std::allocator<casadi::MX> >&, std::vector<casadi::MX, std::allocator<casadi::MX> >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:313:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Vertsplit::evalFwd(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:330:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp: In member function ‘virtual void casadi::Vertsplit::evalAdj(const std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&, std::vector<std::vector<casadi::MX, std::allocator<casadi::MX> >, std::allocator<std::vector<casadi::MX, std::allocator<casadi::MX> > > >&)’:
/home/casadibot/slaves/linuxbot/quick/source/casadi/core/mx/split.cpp:349:9: error: unused variable ‘nx’ [-Werror=unused-variable]
     int nx = offset_.size()-1;
         ^
cc1plus: all warnings being treated as errors
make[2]: *** [casadi/core/CMakeFiles/casadi.dir/mx/split.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[1]: *** [casadi/core/CMakeFiles/casadi.dir/all] Error 2
make: *** [all] Error 2

```


Not bad for a control that can only do so much, but it should check that the camera state in question even has a "height" value (for reference, Free camera does not, so the widget crashes if COFC is on).

Sadly it doesn't smoothly change the angle with the height, it seems like it changes height, then changes angle. Still, I appreciate the effort to make tiltzoom available to people who don't want to use COFC.


嗯，这句一定是测试忘记删了，merge 之后删掉就好。


This selector selects all input items inside radio control (not only input with specific value attribute!), because change event should be assigned to ALL radio items inside control to do the necessary action.


Original required.php of version 0.8.3 generates:

``` javascript
	jQuery(document).ready(function($) {
	"use strict";
	$("#customize-control-my_bg_color").hide();
	$("#customize-control-my_bg_mode input[value="colorful"]").change(function(){
		if ($("#customize-control-my_bg_mode input[value="colorful"]").val() == "colorful") {
			$("#customize-control-my_bg_color").show();
		} else {
			$("#customize-control-my_bg_color").hide();
			}
		});
		$("#customize-control-my_bg_mode input[value="colorful"]").trigger("change");
	});
```
which is not valid.
Please note that in lines 4, 5 & 11 of the above code, 2 double quotations are consecutive. One for jQuery $ selector and one for the value attribute that is inside $ selector, that It causes conflict & breaks the selector; therefore below JavaScript error will be displayed in browser console:
```
SyntaxError: missing ) after argument list on customize.php:885:170
```
As you know, when you use " (double quotation) for $ selector, you should use ' (single quotation) for things like value attr inside it, and when you use ' (single quotation) for $ selector, you should use " (double quotation) for things like value attr inside it to prevent selector break (incomplete selector).


Why not. :)


Why is that an issue? Just curious, I use `return` inside the `try` usually.


"Make build sur" in the commit message title?


Since PREFIX defaults to /usr if empty, you don't want e.g. "$$INSTALL_ROOT$$PREFIX/usr/lib" because that would install to /usr/usr/lib by default. So the /usr part needs to be removed from these the install paths.


Yeah...totally does.


The project won't build at the moment if this file is simply removed, as the file is still referred to in bluez-qt.pro and rpm/libbluez-qt5.spec. Those references need to be removed as well.


Hmm, should we make this configurable? Why does verify fail with e.g. Google as was the case in the forum discussion?


whaoh


we have a problem here - the cleanup will only look at iso and repo as the 'hdd' right now is used purely for user created assets.

So we need a way to mark these hdd as created by the system. Which brings me to another problem: you should hook the asset to the test it created, so it counts to the job group - in addition to the job using it of course.


If this needs to be sorted by fieldID (does it?), should ``concat()`` either here or when returning a cached array in ``Zotero.ItemFields.getItemTypeFields()``. Right now I believe this would change the order of the stored cached array.


Typo: convenience


These commits shouldn't actually include any of the about endpoint files
(I deleted all of those files). I couldn't stop the commit from saying
it had the new endpoint files though because I had committed them, and
then did a new commit that deleted them. This should only have changes
to the seed (some changes to the thing category in the seed, and changes
to the main html and the main scss.

On 2015-04-07 22:06, Aaron Gunderson wrote:
> Do we need this? You don't seem to be using the about endpoint at all.
>
>
> --
> Reply to this email directly or view it on GitHub [1].
>
> 	*
>
> Links:
> ------
> [1]
> https://github.com/Frannycesca/Observatory3/commit/d12a1d4e5af46cc20822993c43593c76558b4f2d#commitcomment-10615350


Why this line is needed?
This line prevents using any RequestParsers because of https://github.com/yiisoft/yii2/blob/master/framework/web/Request.php#L370

Without this line my test works anyway because https://github.com/Codeception/Codeception/commit/deb0e442260e99b588478e9f7399744911cfc8b6#diff-94abe3ee46e86aa1477f9565b6d2f9f5R65


rha yeah typos ... sur was meant to be "use" sorry ... that is gonna be tricky to update without redoing all the commits though ...


Yes! It's really annoying.


In theory the difference is the same as the standard `using`/`import` distinction: the former doesn't allow extending the bindings, whereas the latter does. But that distinction is not actually important in this case, and using `import` would be just fine.

I am kind of puzzled that `using` wouldn't work, though. I tried this on two systems (both 0.3 and 0.4 on OS X and Linux) and it loads on both. Also seems to work on Travis and AppVeyor.


I forget exactly where it happens, so this may be fine, but note that numbers are stored in SQLite as integers, not necessarily strings. (This might still get converted to an integer on insert, the same way that the textbox strings do.)


That's convincing. Done.


I like it a lot.  :)

Very minor things I noticed while skimming the commits:

If there is a delay argument maybe the delay could be soft-coded via

webgui restart 1

or

webgui restart -d 1

Sometimes "all" is used to denote all interfaces, which is more commonly
expressed by omitting the interface argument.

Otherwise using "all em0" makes little sense and it would remove an
exception handling from the interface parsing code.  Worst case, somebody
calls his interface "all".  :D

But none of this is important.  Thanks for cleaning this up!

> On 07 Apr 2015, at 21:08, Ad Schellevis <notifications@github.com> wrote:
>
> Commit 0adea63 contains the re factored code. I agree that service is spurious :-)
>
> All new actions should go into : /usr/local/opnsense/service/conf/actions.d/
> There still are some calls in our system which aren't used, so I'm going to cleanup the templates and move the action templates into the (new) directory.
>
> The new API for all configd calls would then be:
> [subsystem] [action] [parameters]
>
> Which makes a possible "delayed" parameter the responsibility of the executed script.
>
> agree?
>
> —
> Reply to this email directly or view it on GitHub.
>


It was highlited here https://auth0.com/blog/2015/03/31/critical-vulnerabilities-in-json-web-token-libraries/


There's a reason I have a branch for the deps... Would be better if you pinged me.


The library will not compile as of this commit.

bin/source/3ds.o
bin/source/fatfs/diskio.o
bin/source/fatfs/ff.o
bin/source/fatfs/option/cc932.o
bin/source/fatfs/option/unicode.o
bin/source/fs.o
source/fs.c:1:25: fatal error: fatfs/sdmmc.h: No such file or directory
 #include <fatfs/sdmmc.h>
                         ^
compilation terminated.
make: *** [bin/source/fs.o] Error 1


NB!
Also renamed allowAds to hideAds in the export XSD!
Missed that important information in the commit message .-(


That's okay, since the PR hasn't been merged to master yet, I'm fine if you want to just rebase and change the original commits.

I assume "build" shouldn't be there either. E.g. maybe "Make sure the right host binaries path is used when crossbuilding" instead.


Just a documentation note, but this can actually be a difference between different filesystems on the same platform; for example, most filesystems on Linux include the type in dirent, but XFS does not, so the `dirent.d_type` might be `DT_UKNOWN` and you have to do an extra `stat` syscall to answer these questions.

Since this can involve an extra syscall, I wonder if it's a good idea to have a series of `is_dir`, `is_file`, `is_soft_link`, rather than something that returns a `FileType` enum (or integer newtype, to make it non-exhaustive and leave room for expansion and platform differences). Having to make a syscall repeatedly for every directory entry if dispatching on file type can get quite expensive on large directories, while fetching a `FileType` and then matching on it involves just a single syscall.


`<%= render 'done_tasks', tasks: @tasks_done %>`


Works fine for me. I'll take a look at it when I get a chance.


I think it'd be better to just use separate assignments rather than indenting this all.


the same as above


Comment


The following PR on `roboptim-core` is related to the storage order: https://github.com/roboptim/roboptim-core/pull/89

Once it is merged to master, the default behavior should be back to `ColMajor`.


you need to use `tasks_done` instead of `@tasks_done`


It's an exciting new world.


Should have used this in the first place -- some of the text was ommited. http://pastebin.com/raw.php?i=4ZbyY8yW

btw - what devkitARM/Pro are you using? Should fatfs lib already be there?


Thanks... This is a debugging line I forgot to remove...


Can put this all between ```Zotero.DB.beginTransaction();`` and ``Zotero.DB.commitTransaction();`` to speed this up.


it goes in `doc/Makefile.am`


Hmm I think I got it. It's so that if `fn` was to throw, it would be caught by this handler and called again. Ouch.


The fatfs lib is included in ctr9's source. I probably just need to tweak the include directories.


this link should be http://cr.yp.to/critbit.html (with html extension, not htm)


thanks a lot! that might do the trick!

I'll have to check it out and add some tests before merging it in order to make sure it's not causing any unforseen issues, but that's great!


@tambry Eh.. Even from [MSDN blog](http://blogs.msdn.com/b/directx/archive/tags/directx12/) doesn't tell me DX12 released, still EA now. [DirectX 12 Early Access Program signup](http://1drv.ms/1dgelm6). It seems to be much better than DX11, but.. well.. DX11 is bundled in Win8.
"There will be DX 11.1 cards that take advantage of a lot of the driver and software tech that we're bringing in Windows 10, but if you want the full benefits of DX12, you're going to need a DX12 card." [Source](http://www.polygon.com/2015/1/22/7874793/directx-12-wont-require-a-new-graphics-card-after-all)


it might be better to make these two cumulative instead of replacing the current list of domain events


I apologize.


Ah of course, thanks.

I'll try to investigate a some more and see if I manage to get a small reproducible test case.


Great! Please move this change with a pull request to the bednee/cooluri master :)


Currently this throws a Nullpointer Exception on   uri.host.
Same goes with using get() instead of post(json).
JSON Object is tested to be consistent and the WS holder is okay with invoking url(), setHeader() and setContentType() and does return a null reference itself.
![screenshot 2015-04-07 23 30 49](https://cloud.githubusercontent.com/assets/5476826/7039870/9a45146c-ddc7-11e4-9fe2-b8b8ad111d57.png)
![screenshot 2015-04-07 23 30 49](https://cloud.githubusercontent.com/assets/5476826/7039874/a10f5a1e-ddc7-11e4-8b3e-0c2ba911e2e4.png)


This is likely no longer needed.


Yup, j'aurais opter pour un service également

Le mar. 7 avr. 2015 22:52, stephane <notifications@github.com> a écrit :

> J'ai fait le choix du mixin dans l'éventualité que l'appel à l'api sera
> fait par d'autres class routes ou controllers; il me suffit d'étendre la
> classe. Cependant j'ai "abusé" du mixin dans mon précédent projet et je me
> suis retrouvé avec des problèmes de code dupliqué.
>
> Le service semble plus adapté pour la gestion d'une api, je ne l'avais pas
> encore utilisé.
> donc:
> Un singleton injecté dans les instances des objets choisi pour le service
> VS étendre le prototype du constructeur pour le mixin
>
> Je passe sur un service!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stephane71/da-movie-quizz/commit/a77bef1055f10134a61f416bd7426d92525d9278#commitcomment-10611460>
> .
>


Opté*

Le mer. 8 avr. 2015 08:17, Nicolas Chenet <ironic51@gmail.com> a écrit :

> Yup, j'aurais opter pour un service également
>
> Le mar. 7 avr. 2015 22:52, stephane <notifications@github.com> a écrit :
>
>> J'ai fait le choix du mixin dans l'éventualité que l'appel à l'api sera
>> fait par d'autres class routes ou controllers; il me suffit d'étendre la
>> classe. Cependant j'ai "abusé" du mixin dans mon précédent projet et je me
>> suis retrouvé avec des problèmes de code dupliqué.
>>
>> Le service semble plus adapté pour la gestion d'une api, je ne l'avais
>> pas encore utilisé.
>> donc:
>> Un singleton injecté dans les instances des objets choisi pour le service
>> VS étendre le prototype du constructeur pour le mixin
>>
>> Je passe sur un service!
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/stephane71/da-movie-quizz/commit/a77bef1055f10134a61f416bd7426d92525d9278#commitcomment-10611460>
>> .
>>
>


I have no idea how much time I'll have to work on it, but I've gotten some semi-serious networking experience, so I want to try my hand at taking the current version and making it multi-player.


This is a bit of an unfortunate aspect of the persistence API: this match will issue a warning, and rightly so. OTOH we cannot even at runtime ascertain that the received value is of type `E` given Scala’s richness. A compromise would be to demand a `ClassTag[E]` when defining a PersistentFsmActor and to use its `.runtimeClass` information here.


please remove the round parentheses


Added so that the overlay doesn't get applied to the ticket page and user page. Also prevents edge case issues where if the same app is available in both the user and ticket sidebar, enabling it in the user or ticket sidebar will keep it enabled. W/o this if the app was hidden on the ticket sidebar, and if clicked show in the user sidebar, then the ticket would still be hidden as the hide button would be selected in the ticket sidebar overlay.


sneaky usage of a `var`, but this is the most obvious encoding


Nope, sorry.
Please open support ticket on PhoneGap Build community page.


ETC 연산에 대해서는 어떤 것들을 추가 할까요?
getStats 와 getVersions 에 해당하는 StatsOperationImpl 과 VersionOperationImpl 에만 연산을 추가 하면 될까요?


Yes, these icons are official. I have send email to Reshaper team about icon license. I hope we receive answer soon.


예, 두개 연산만 추가하면 되네요..

binary 연산 쪽은 operation type을 추가하지 않았는 데...
이것은 comment로만 남겨두면 좋을 것 같습니다.


@olegsobchuk Every time you do the post please make sure you have the correct tags and description


`helpers`


I don't think `>` should be here, just `&:before`?  You're already checking for `>` on span.


and here


"CiteProc"


Eventually we might, but for now this fix reverts the behaviour back to what it was using php5.5 (http://php.net/manual/en/migration56.openssl.php). Really fixing the notification code is not very high on my list for now.


CiteProc


An important piece that's missing here is a way to get the `d_ino` out of a `dirent` entry on Unix-like platforms. If you're iterating over directories that contain a large number of hardlinks, such as rsnapshot or Time Machine backups, or something where you use hardlinks to provide multiple different views of a particular directory hierarchy, it can be important to avoid an extra `stat` by caching inodes that you have already seen and just reusing the `stat` information you got the last time. A project I was working on in Python recently got bitten by this using `os.walk()`, where we had to do expensive `lstat` and `stat` system calls for each file, but saw a huge improvement in performance when using `ctypes` to call the libc API directly and were able to do the caching trick; see [this feature request](https://github.com/benhoyt/scandir/issues/44) (which [appears to have recently been implemented](https://github.com/benhoyt/scandir/commit/d2580cab2a0be790988be7e4f974cfc6ca278538)) on the new Python `scandir` function's `DirEntry` class for the same reason.

I don't mind whether this is exposed directly in the portable API (and thus would incur an extra syscall on Windows, since there is no inode equivalent in Windows' dirent equivalent as far as I can see; if you want performance on both platforms, you are going to have to follow different code paths), or as a way of lowering the `DirEntry` to a platform-specific version that contains more information; however, an API that simply returns a full `RawMetadata` object or requires you to stat it yourself wouldn't be sufficient, as you would then necessarily incur the extra syscall overhead for information that was already returned to you when traversing the directory.


Trailing comma


trailing comma


Too many spaces


Should have a newline between previous code and new comment


小组项目  4.8
1
2
3
4
5
6


Ok. :)


If we actually care about resetting the pref, it should be in a ``finally``. But I'm not sure these tests would ever actually be run in a non-test profile, so I'm not sure there's any reason to reset it — I think it can just be changed before the tests where it's relevant.


You're calling addEventListener directly on document.  This won't work in IE8. :(

Just call addEventListener that you required above.  The 3rd arg is the target element which defaults to `window` so you can leave this blank.  I think you want the click event on the window and not the document anyway.


Same as addEventListener comment above


Health + environmentally chisy product + joy of life + good mood + + love + happiness healthy and strong family + win!


@Natashfine Added this line as you requested.


@mesa57 I am finished, this was one-time only.


@quirkey Is sammy also published on npm? We need it at TodoMVC: https://github.com/tastejs/todomvc/issues/1110.


I was going for `toLower` in `| 'A' <= c && c <= 'Z' = toLower c` but sure this till do to ;)


Should config have defaults?

`config = _.defaults(config || {}, { width: 640, height: 320 });`


If I do a one-liner, any errors on that line won't appear
On Apr 8, 2015 04:35, "Matthieu Napoli" <notifications@github.com> wrote:

> Out of curiosity what do you mean by "leakage of error suppression"?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ocramius/cache/commit/5cf25179ffdf10c5f8c8180c0dce4fd9f05dc18b#commitcomment-10616004>
> .
>


The first massage


If you still have the issue, please send me a gs dump.
Hit shift f8 on a bad frame, gs file will be generated on the snapshot directory.


Hi aneillans,

Why do you prefer to use a new TfsConfigurationServer each time you use it instead of using one for the lifecycle of an TFAdminControllerBase instance ?

Regards,
François.


Hi, can you please fix methods name: ``prepareInSQLCaluse`` -> ``prepareInSQLClause``. Thanks


Hi i tried that.
Cann you schare youre makefile
Or an example on howto integrate
Sorry for my ignorance

Op 8 apr. 2015 07:27 schreef Sébastien Randoux <notifications@github.com>:you need to integrate it here:

https://github.com/hzeller/rpi-rgb-led-matrix

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


Hm... I remember doing this already once... I'll check.


Here I try to put my files from the first lab into a folder just to be more clear on which laboratory what we did.


What if person A invited person B to work on person A's song and person A has a paid plugin that creates some fancy-fun effect and person B attempts to copy this to their own project/song (or someone else's)?

What if person A doesn't want people to be able to copy things from their music into another project? I can see how it makes sense to say "nope, if you're using this software everything is open" and I can also see how it might make sense to say "you can only copy from your own projects to another project" or maybe even "you can only copy from your own projects and those you have a special permission granted with to another project"?


something I accidentally added during git add. should have put this in ignore.


Hi aneillans,

We understand that you prefer to use config files instead of environment  variables. However, using environment variables is also a good practice to avoid the mess of multiple environment config files.

We are keeping the 2 ways.

Regards,
François.


You could have asked :wink:


I'll try to take a look tomorrow


Exactly. This is super painful to debug as well as you're errors and stack traces from code that might not even be related to execution of the given function.


Ooops!


Thanks!! Finally I solved my app problem using this code!!


This is a temporary backward compatibility with error warning until we move 100% to the new API. Currently not even documentation is done so users might still use it.


OK


Best merged as (KeyError, AttributeError)?


In general, the answer to these types of questions is: the user will be able to do whatever they damn well please. :)


My idea is `metadata = {}`. The same in `test_factory`, instead of abusing `params` for our stuff let's call it `metadata` and freely abuse it for all purposes like `timeout` or dict vs. actual test URI.


I can close mine :)


咦咦。。。。。c++11 c++14 java7 java8...


Shouldn't fetch_harvard_oxford return a Bunch like all the other fetch_* functions?


Because at this point in my career, does anyone really care?


In future, use ruby 2.0 style hashes. Instead of:
```add_column :accounts, :experimental, :boolean, :default => false```
use
```add_column :accounts, :experimental, :boolean, default: false```


Feel also free to supersede mine, no problem at all ! :+1:


Also, when you run a migration then annotate. Only stage and commit the files you know have changed.


This code is :hankey:


缓存数据用组件做一定落地，这个定时器是在所有的组件停止后设置的，POMELO现在的确是有退出后LOG4JS的日志没写完整的问题，不过把这个定时器去掉，99%进程永不退出。这个并不好弄，并不好整理清楚是哪些事情还在事件循环中，不然可以不要这个timer


Error :sob:


Hey, how can this break build while mobile_static and monodroid built without problem?


Also in @throws InvalidArgumentException PHPDoc.


We have to check if this is causing #12 ...


코멘트 수정
When the type of an operation instance is undefined,
it has the operation type of UNDEFINED.


코멘트 수정
Arcus clients do not use


Probably better for the creator names not to all be the same? One could envision an export bug where the names got jumbled...

Also, we're envisioning different, more specific data for things like bibliographic output, right?


我在这写的是跑到下面么


Falto cambiar $skill_id, agregaré un fix


@aboch done, thanks for your input.


catchup


What the hell happened to the player?! The player is drawn at (0,0) again! xD


Your right that there are no rules against having Multiple "Half-" Templates. Using common sense and logic they are called "Half-"....Oh wait this is D&D there is no common sense or logic. LMAO; but seriously they are called "Half-" templates cause 2 halves make a whole not 3+ halves to make a whole. if that was the case they would be called "Third-", "Forths-", ect..


Well, im not sure about olive oil


Oh, that's useful! Thanks!


This should be « *appelez* vos députés ! » (imperative form)


Hi, we'll add it in the demos in the repository https://github.com/hzeller/rpi-rgb-led-matrix soon, you'll have to wait a little bit.


I guess if it's not already absolute, it must be relative to our working dir, so we could make it `filepath.Join(os.Getwd(), f.RawPath)` or so...


Problem solved. Thanks.


It seems I will have to remove my debug messages.


strtolower, strToLower 다 작동하나요?


Surprising that you changed the validation to allow numbers here.

But thanks a lot for fixing this and merging it :)


Thanks for the cleanup. IntelliJ isn't as good as eclipse was for automatically flagging and pruning imports as changes happen. Or more likely it's simply the driver behind the wheel not being as familiar with the car. :)

In any case, it's nice to have someone other than myself making changes and keeping the code base moving forward. Thanks.


Emaile rašei, kad neveikia tikrinimas. Netestavau kodo, bet manau, kad vietoje is_int turėtum naudoti is_numeric. $_POST'e visada ateina stringai, net jei tai ir skaičius. Funkcija var_dump() tau turėtų parodyti tipą.

Čia is_int() tikrina, ar kintamojo tipas yra integeris ir dėl to nepraeina. Tuo tarpu is_numeric() tikrina, ar kintamasis apskritai gali būti interpretuojamas kaip skaičius, nors jo tipas gali būti ir stringas. Tas parašyta ir dokumentacijoje: http://php.net/is_int (Note: To test if a variable is a number or a numeric string (such as form input, which is always a string), you must use is_numeric().).


TeamCity Toolbox :: Pull requests [Build 58](http://10.48.34.86:8111/viewLog.html?buildId=8146&buildTypeId=Toolbox_PullRequests) is now running


@BanzaiMan are there instructions somewhere describing how to use phantomjs 2.0 for testing?


TeamCity Toolbox :: Pull requests [Build 58](http://10.48.34.86:8111/viewLog.html?buildId=8146&buildTypeId=Toolbox_PullRequests) outcome was **SUCCESS**
Summary: Tests passed: 168 Build time: 00:00:27


-This is my first pull request. Joyce Otieno


@Aokromes is this right? Seems like you missed a month.


nice


I'd like to hear initial feedback from Apple and NVIDIA about how quickly this bug might be fixed before moving it into the conformance suite.


I think we might want to preserve the intent of "big bold", which is to make the text span stand out (with increased font height and weight). Can the issue be fixed by only replacing the last angle-bracket with `>`?


Yes, sorry, we discussed this internally yesterday and agree; we forgot to add a comment here.


@erictune could you please answer for @nzwulfin's question?


Heh, nice one! I think we should check for the situation where you're building for architecture i386 on a amd64 system though, since that doesn't require usage of qemu-debootstrap then, what do you think?


Huge thanks! Awesome work!


OK


My first attempt at pulling a request


Cambios incorporados, muchas gracias Fergu. Pasadlo muy bien en el congreso.

El 7 de abril de 2015, 23:55, Pablo García Sánchez <notifications@github.com
> escribió:

> Estoy en el tablet, que acabo de llegar de la cena de organización del
> evostar con los VIPS XD Siento no hacerlo en el .tex, que en el tablet es
> un cognaso. Cosicas:
> - El párrafo a pie de página mejor integrado con el texto, no me gustan
> mucho los pies de página salvo para poner urls.
> - No hace falta poner smartphone en cursiva
> - The risk of accident grow-> grows?
> - Cuando describas la sección sec:methodology di que también vas a
> presentar una metodología que permitirá tal y tal a partir de la
> generalización de un problema. Así, cuando escribas esa sección más tarde
> di que propones una metodología que incluye la definición del sistema, como
> parte de ella.
> - Al final de la sección, dí que también hablaras de trabajo futuro.
>
> Mañana estaré con el portátil en el congreso, podemos hablar por hangouts
> si quieres.
>
> Tahora!
>
> El martes, 7 de abril de 2015, Juan Julián Merelo Guervós <
> notifications@github.com> escribió:
>
> > Got it. Tomorrow.
> >
> > 2015-04-07 17:59 GMT+02:00 Paloma de las Cuevas Delgado <
> > notifications@github.com
> > <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>:
> >
> > > Writing introduction, to reviewed by @JJ and @fergunet
> > >
> > > ---
> > > Reply to this email directly or view it on GitHub:
> > >
> > >
> >
> https://github.com/geneura-papers/2015-GECCO/commit/819baf0b026a10ca84ca6d843475041d9a9d0eff
> > >
> >
> >
> >
> > --
> > JJ
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/geneura-papers/2015-GECCO/commit/819baf0b026a10ca84ca6d843475041d9a9d0eff#commitcomment-10609147
> >
> > .
> >
>
>
> --
> Dr. Pablo García Sánchez
> University of Granada
> http://www.ugr.es/~pablogarcia
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/geneura-papers/2015-GECCO/commit/819baf0b026a10ca84ca6d843475041d9a9d0eff#commitcomment-10612586>
> .
>


By your logic you can't have a half-dragon half-elf because half-elves are defined in the rules as being half-human and half-elf. Two halves make a whole right? And yet no where in the rules for templates say only one half-x template can be applied. And since this is a tool that should be as RAW as possible...

Besides, in a world with magic, who's to say a wizard can't make 2+2=fish?

----- Reply message -----
From: "TwistedFaith84" <notifications@github.com>
To: "Heliomance/HeroForge-Anew" <HeroForge-Anew@noreply.github.com>
Cc: "torrasque666" <dzsenak@yahoo.com>
Subject: [HeroForge-Anew] Unstable development version (5a44136)
Date: Wed, Apr 8, 2015 2:08 AM
Your right that there are no rules against having Multiple "Half-" Templates. Using common sense and logic they are called "Half-"....Oh wait this is D&D there is no common sense or logic. LMAO; but seriously they are called "Half-" templates cause 2 halves make a whole not 3+ halves to make a whole. if that was the case they would be called "Third-", "Forths-", ect..

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


This should be `except ImportError`
Is this needed anyway? This version of ATConentTypes works only with Plone5


删除


Thank you! This commit fixed the build. It was broken since http://ci.cgeo.org/job/c-geo/3609/


shouldn't this be 0x12, 0x09 ? :)


Ok. Fixed the @throws too :). Thanks!


Finally checked. It works, but I have another problem.. will issue a defect (or find one :) )


aap ne theik kaam neih kiya


Hm, I'm wondering if we couldn't just use $DEBBUILDOPTS for that to make it more flexible. I'm pretty sure some users don't use the labels (but e.g. a custom matrix setting instead) and then FORCE_BINARY_ONLY won't do anything for them. If we'd check for DEBBUILDOPTS being set and if so then not run any code from identify_build_type() at all your use case should be covered as well?


If date format is specified that should always win. Otherwise I guess we'll have to use a regex to determine which format to use (or just try and if it fails try the other one).

I think this is the only place that needs changing.


아래와 같은 형태로 사용했었어요~
```php
request\sanitize::get('property_contract_partner_id'     ,'array.int', 0);
```
default를 개개별 default로 했고
값이 없는 경우 [0] 이런식으로 들어가는걸로 했어요~
다른방식으로 바꾸면 거기에 맞춰서 수정할께요.


[HttpClient]


:beers:


TeamCity ChameleonForms :: Continuous Integration [Build 231](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3545&buildTypeId=bt20) is now running


Thanks for this PR! I need to verify though that this works as expected for existing setups where KEY_ID is already set and used for signing the reprepro repositories (only).


"tricky" was my maiden name ;)
check out my fix and let me know what you think.

On 04/07/2015 10:05 PM, jkowalleck wrote:
> to have this interact with main_keyboard and num bad, i guess this will be a tricky one :-P
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gpunktschmitz/nichtparasoup/commit/8470954e282da201b320e6e99f66dbdcac28b28c#commitcomment-10610610
>


Dans un repo public les gens peuvent modifier comme ça ?


kerner --> kernel


I can't migrate my db with the migrations in this commit. Is this just me or anybody else?


Hopefully that'll please people :)


drtynjryflu,fk cdtfjg


comentaaaaa


Uh... only Stylus file? What about other file case?


if (fileSysAddr[fileSysAddr.length()] != '/')  should be
if (fileSysAddr[fileSysAddr.length()-1] != '/')


This color for the operands will be suffuciant att this time.


TeamCity ChameleonForms :: Continuous Integration [Build 2.0.0-PullRequest114+011](http://ci.robdmoore.id.au:8010/viewLog.html?buildId=3545&buildTypeId=bt20) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3003 Build time: 0:0:0
### Failed tests
```
ChameleonForms.Tests.dll: ChameleonForms.Tests.Templates.TwitterBootstrap3.PrepareFieldConfigurationTests_TwitterBootstrapTemplateShould.Hide_label_and_set_as_checkbox_control_if_checkbox_and_in_section: Test(s) failed.   Expected string length 0 but was 5. Strings differ at index 0.
  Expected: <string.Empty>
  But was:  "label"

```


Ah yeah right I see it now ;)


I didn't check that. Maybe a solution could be add a boolean field inside RBinInfo to know when show that information and check it here. Which do you think would be the best way to handle this?


```if () {``` even if there is only one statement in if-block.


FAといいつつ、CMが
cyanogen_jf_ldefconfig となっていましたので、
cyanogenmod_jfltedcm_defconfig　→　cyanogen_jfltedcm_defconfigとします


Ditto.


what?


empty


The reason of this duplication is that `clone()` is protected in class `Object`. `clone()` is still part of the interface (it extends `Cloneable` for a reason), however protected. Overriding that visibility here is a redeclaration of the method, and leads to compatibility issues when implementing this interface. Consider the following:
```java
interface A extends Cloneable {
    // implicit: protected Object clone();
}
interface B extends Cloneable {
    public Object clone();
}
class X implements A, B {
    ??? Object clone() {};
}
```
Here, `X` must implement both `Object.clone()` and `B.clone()`. The compiler cannot infer which `clone()` method is implemented by `X.clone()` and consequently fails.

`Persistent` is also implementing `Cloneable`, so the problem is real: otherwise we cannot do what we do on `SubscriptionManager.java:733`:
```java
	public static abstract class ObjectChangedNotification<EventType extends Persistent & EventSource> extends Notification<EventType>
```
(notice the `extends Persistent & EventSource`)


zz


Blablabla


laaaaaaalalala


Added a PR with the C# implementation: #24


Should this be `int`?


Thanks @mcm-ham for refator code.


Sorry, I missed a "temps" there :(


Du coup tu peux virer ton mixin, et internaliser `buildUrl` dans la méthode `ajax` de ton service par exemple.
Ici, le `this.buildURL` fait encore référence au mixin ;)

Mais sinon, c'est l'idée. Well done !


no problem.


> there is an alternative: we could make the return statement mandatory and fix the ported comps. Happy to help with it - one time pain.

The ported icomps have already got return statements, see commit 8337623

I have been thinking about it too.  The added code would catch 98% of empty return statements and add a return to those without for the overwhelming majority.

It is very difficult however to cater for bizzare and downright faulty coding.
Putting one bracket inside comments is a possibilty, however remote, as are a miriad of other things
that I would not ever dream of doing or being done.

The ethos of comp, was that the writer was responsible for his own code and everything after the ';;' divider was just dumped 'as is' in the C file.

We need to add the struct definition so that the defined macros work and period, the user cannot be expected to know about that, but that is pretty well catered for by the initial changes.

It picked up the opening curly bracket after FUNCTION(), no matter whether it was on the same line or several lines down and inserted the declarations before dumping the rest of the users code.

So a note in the docs and all 'example' ported components having return values should suffice.

Trying to check the existence of return values is very hard work, but it will be easy to check if the code has 'return' in it and if not exit with an error message.


Hey -- I have that app on my phone


Nit: Don't need this empty line.


Good idea!


@uNiversaI  maybe you can use one of the ideas from samfisher (see http://forum.kodi.tv/showthread.php?tid=222444)


You can use the `%` operator


Can someone check it's correct to user the total number of transcriptions here? @samheather @j6mes


That is a very strange name for `stat` IMHO.


Could you leave a comment on what triggers this to be called?


I think we should keep all addrs. I would like to eventually expose READONLY functionality to users, so that they can perform readonly operations against all nodes. I was about to ask you for advice on the best approach, but I will submit a separate PR for this.


seems related https://issues.apache.org/jira/browse/FELIX-4258


But the Numpy convention puts a space between the input argument name and the colon?

"Enclose variables in single backticks. The colon must be preceded by a space, or omitted if the type is absent." https://github.com/numpy/numpy/blob/master/doc/HOWTO_DOCUMENT.rst.txt


Is this in pixels?


It does work, because we are changing the detection that tests if dnsmasq supports --bind-dynamic
http://code.metager.de/source/xref/lib/virt/src/util/virdnsmasq.c#669
and thus change how it is used only indirectly.
And looking at the libvirt code, this seems hard-coded otherwise with no option to change it.


Or just if !filepath.IsAbs() { filepath.Abs() }


Beware that File.exist? is used an awful lot in require and such (thousands of calls in typical apps). But while we have require in Java, we'll probably use the existing Java File.exists() or FIle.isFile(), which is likely more efficient.


Put ondelete="restrict"


dtyyujmng jkf cvhm


It should not happen, but i think twitch.tv's IRC is a little bit wierd sometimes.
When using Quassel IRC with twitch, Quassel picks up my name as "GLHF" instead of my correct name.


:+1: Relevant test


Closes #15


Version of DPU not updated, also readmefile shall be updated.
Feature branch not created for this issue.


neat-picking, but I think this would be clearer as `errno` or `error` (or `errorCode` maybe).


@gabrielasmeria fixed [one functional test](https://github.com/xwiki/xwiki-enterprise/pull/64/files) and I wrongly assumed she checked the rest of the tests from the same module.. my bad.


can't we reuse any of the 6 already exising "Off" labels, like f.e. 13551? And please add contextual help strings (I know this sucks) so that it's a) easier for translators to do a correct translation and b) easier for devs to find an already existing label that fits the usecase/context (like with this Off label). Thanks.

Also, could the "fps" part probably be appended by the settings system for value labels? Wouldn't this make sense for some of our settings? @Montellese


Closes #22


Having Ruby's `File::Stat` defined in Java could allow the Allocator trick, but difficult to tell if worth right now.


The link gives '404 not found' for me.


For consistency should we name all AudioDSP stuff like ADSP or only DSP?
E.g. `#include "cores/AudioEngine/DSPAddons/ActiveAEDSP.h"` would be `#include "cores/AudioEngine/ADSPAddons/ActiveAEADSP.h"`


Now it's totally optional to pass it to 'let'. Use Fetcher in specs and get rid of this let.


Did this for debug reasons, shouldn't be here (fix'd in local)


thank for your help


Maybe we could have a ReadObjectFieldNode<Type> ? It would avoid the cast and avoid all a dispatch chain when we know only one type is appropriate (but would mean we cannot have it nil, which is OK for internal structures IMO).


Move it to let! :)


OK.


Sweet, we'll be able to load more specs with this (as_superuser etc)


arg !


I had to drop, create and migrate the database. The problem I had was related to the `type` column and that it didn't exist.


Why not just replace this entire bizarre construct with
``` lua
local masterservers = {
  {"208.64.200.39", 27011 }
  {"208.64.200.52", 27011 }
  {"208.64.200.65", 27015 }
}
```


Lisp-style! (looks OK to me)


Trivial comment


@george08 I was out of office yesterday, but I'm back for today and tomorrow. If I understand what has been happening correctly, you are building custom JSON using custom scripts. Seems like a pragmatic thing to do, especially given the time frame. If you need any input, please let me know!


see https://github.com/odoo/odoo/issues/6189 about why this is not the correct way to fix this issue.


Should have been injected


commit


so is Netflix using rrd4j?


I'd added the link in prevision for the new build, but it looks like that hasn't uploaded correctly. Looking into it.


why not … as suggested here? http://stackoverflow.com/questions/8536375/android-replace-with-ellipsis-character


test


Closes #42


This comment is no longer accurate, since you don't use `debugMemSize` any more.


I cant see that forum.


The comment sounds as if the function were wrapping itself.


We don't want to return Optional.empty() in case of null input?


You don't call xmlMemRealloc and all those other xmlMem* functions any more.


The settings system can already do that. You can specify a list of possible values (in this case `0`, `5`, `10`, `15` and `24`), a special label for `0` to show the `Off` label and a generic formating label (in this case something like `%d fps`). It's already used for seconds and milliseconds values in other places.


a) IntelliJ auto complete suggestion b) the files are UTF-8 encoded anyway, so it should not cause any issues


You are querying database twice :) Cache it.


Oh. I've forgot to change this text. Please update it.


Why is there a space before Learn?
It might not align properly :)


![Uploading 500x500bb-80 (2).png . . .]()
![Uploading 500x500bb-80 (3).png . . .]()
![Uploading 500x500bb-80 (4).png . . .]()
![Uploading 500x500bb-80.png . . .]()


方便学习vc


You can save few lines using rates and uuid directly in hash below.


Allow youtube to show up inside sceditor on a https site, otherwise browser will just block youtube image and player with a warning:
> Mixed Content: The page at 'https://example.com' was loaded over HTTPS, but requested an insecure resource 'http://www.youtube.com/embed/XXX?wmode=opaque'. This request has been blocked; the content must be served over HTTPS.


Why you keep `completed`?


Maybe I'm a bit fussy and take it very exactly. I do not know now if that comes from the Gihub representation, but often the tabs after an if statement is not correct.

This:

if (ActiveAE::CActiveAEDSP::Get().IsProcessing())
  buttons.Add(CONTEXT_BUTTON_ACTIVE_ADSP_SETTINGS, 15047);

should be more like this:

if (ActiveAE::CActiveAEDSP::Get().IsProcessing())
    buttons.Add(CONTEXT_BUTTON_ACTIVE_ADSP_SETTINGS, 15047);


Be careful, this update breaks your templates !!

**You need to use template namespaces, from now** Eg,

```twig
{% extend '@MyTheme/base.html.twig' %}
```


link to acc.


I think it's cleaner to catch the `KeyError` exception and assign 'vi' to the editor variable there (leave the `or 'vi'` part in case `EDITOR` exists but is empty.


Please take a look on https://github.com/activeadmin/activeadmin/pull/3677#issuecomment-65922117


Reste de conflit ?


looks like they were afraid to use optional pre args: `(type = undefined, address)`


Can we do it later when we add that functionality? What is here seems correct and does exactly what Ruby Cluster client recommends. I don't exactly understand what you mean about "READONLY functionality", but there are chances that it will require separate `readOnlyAddrs []string` or something else.


Should this be OnContextButtonActiveAEDSPSettings(...)?


I see.  But there might be other cleaner options available which avoid monkey-patching, e.g. wrap dnsmasq.


Yeah, this isn't intended as an actual release. I uploaded it to GitHub as the easiest way to transfer it to a different computer. Nothing to see here, citizen, go back to your fun.


The purpose of task was to store only currency rates, not entire message :)


Loading a file with an unknown primitive is no longer a failure?


READONLY: http://redis.io/topics/cluster-spec#scaling-reads-using-slave-nodes

Storing `readOnlyAddrs` could make sense, but I had some sort of cluster client option `ReadOnlyMode off|slaves-only|all` in mind. Selecting a random item from two slices, e.g. `masterAddrs` and `readOnlyAddrs` is just less appealing.


@radarhere Probably yes, better to rename ``_`` to something more meaningful.


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


Егор, принято, что сообщение коммита отвечает на вопрос "Что было сделано?".
Например, для этого коммита я бы написал что-то в духе

Итерационный метод для нахождения корня был заменен на точный.
Удалил лишние файлы из репозитория


sua loi roi


All right, I added `swap = uncurry $ flip (,)` to `condensation`.


Nicely copied ;)


What if currency wasn't found ? You'll get an exception. In such case you want to reject job and requeue it.


Tests :)


Its like learning Wordpress again


Cheers! I'll fix that one, but is there a way to disable the memory debugging data structures?


Put into the pods please.


I forgot to delete this ;)


Yeah, using `toLEdge` would seem nice. Could you move the commit to `master` then?


thanks for the hint, fixed.


test successful


test


Should already be done.

Sent from a mobile device; please excuse any tpyos.
  Original Message
From: Sergiu Ivanov
Sent: Wednesday, 8 April 2015 6:30 PM
To: scolobb/fgl
Reply To: scolobb/fgl
Cc: Ivan Lazar Miljenovic
Subject: Re: [fgl] Add condensation. (a323bf3)

Yeah, using toLEdge would seem nice. Could you move the commit to master then?

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