Are you just making fun of me with this?

> On Aug 31, 2015, at 4:39 PM, Eric <notifications@github.com> wrote:
>
> The only change between these two files was removal of blank lines on the bottom.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/johns996/NMU-Zen/commit/596a0e0a95880cd61a893ecd446bd0283c4af77d#commitcomment-12984938>.
>


shouldn't this be its own edition?


Apparently the capitalization matters.


Haha yes.


Hello,

Is there possibility to update code ?

There is an issue to commpile even build-gmp. I ve tried with xcode 6 and SDK 8.4.

configure: error: could not find a working compiler, see config.log for details
Build...
make: *** No targets specified and no makefile found.  Stop.
make: *** No rule to make target `install'.  Stop.
Build library...

but of course commpiler is present

checking whether /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc is gcc... yes

checking compiler /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -arch i386 -pipe -no-cpp-precomp -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.4.sdk -I/Users/miracleboss/Downloads/GnuTLS-GMP-Nettle-for-iOS-master/usr/iPhoneSimulator8.4-i386.sdk/include -arch i386 -pipe -no-cpp-precomp -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator8.4.sdk -I/Users/miracleboss/Downloads/GnuTLS-GMP-Nettle-for-iOS-master/usr/iPhoneSimulator8.4-i386.sdk/include... no


This is clearly for our alpha users! Thanks for thinking about them.


This could be done at configure but would be overkill... your call.


Is the text-slicer usable as a `tiddlywiki --command` ?


Yes, sorry for not noticing that. Hive needs to have permission to access files created by the Hadoop user, but that can be different depending on how you run hive. I'll clarify.


Will do.


Neat! Thank you :)


Yep, added in 1.8 when different template backends was introduced.


eh?


It will appear in f5470840


I'd rather not use epoll.  Use select or poll instead.


Oops?


What's this?


Probably not, unless someone has a huge cluster and does lots and lots of metadata reads that hurt the primary?


We will use java 8 here I guess.

We can put here a profile to disable build with older JDK, and disable animal sniffer.


why not just `items`?


Version in the parent ?


why a string and not an enum?


It did. Other things didn't.


by "connect to Presto as the 'hive' user" I meant sudo -u hive ./presto-cli (I think that'd be the Presto username).  The issue is with the temp directory that Presto creates in HDFS: it has the permissions of the Presto user, unless you specify the Hadoop user via the JVM config, in which case it's the Hadoop user. How should permissions for this be handled?


Maybe too generic?


tbh, these days I'd have used `xss-lock` instead. It even handles sleep via systemd automatically.


i'm talking about the module name ;) easy to rename later when it needs to be more specific


Names of class & benchmark are out of sync here. Could you fix the naming?


should be plural here


Was this workflow used for the parameter scan?


rename it to CERT.at


The goal is to retrieve the author, but it's not really easy with the GitHub plugin.
The simple way to achieve that is to give the author repo URL (https://github.com/aduprat/james-jdkim.git), then to split in order to retrieve the author.

I need this for fetching his james-parent repo.


'registration' schema? i'm thinking about moving surveys to their own schema, too, since they do have a few tables.


hasta ahora no ha causado ningún problema, pero hay que monitorearlo con el uso, lo unico que hace es que ahorra tiempo por que no hace operaciones extra, pero no se mueve ni afecta


Right, done


Done.


We recommend in the doc that people use something like:

```
test -n "$TEST_LONG" && test_set_prereq EXPENSIVE
```


Done


missing relationships (`registration_form`, `user`)


changed the initial height


CERTatContactExpertBot


Jackson version we use in mailbox is 2.3.3.

Things may brake...


You probably don't want to give me too much credit here.


You may not want to give me credit.


お疲れ様です。


>Обрати внимание на название метода -- messageSource, если метод переименовать, то локализация должна сломаться.

В твоем случае это не так. Только что проверил -- не сломается. А знаешь почему? Потому что Spring нигде не использует этот `messageSource`. :) Его использует hibernate-validator и делает это он сам, без помощи Spring-а.

Поэтому -- можешь смело этот бин удалить.


@ralphbean for ZeroMQ discovery on Linux you can use abstract IPC endpoints. So, e.g. "icp://@/service/path/whatever". So e.g. event publishers can bind to well known endpoints, and subscribers then connect to those.

You could also run a broker for specific kinds of message flow, and we have one (Malamute) that can run as a thread.

Feel free to ping me if I can help in any way.


these should be removed


LGTM


So nice of you to think about me! /play ohmy


I think "Hook" is the best because hooks are way faster than listeners. /s


"Erstelle neuen Druckauftrag..."


final ?


Giving me too much credit?


@amdj15 - with the 5.0 release of _react-validation-mixin_ I've extracted the joiValidationStrategy into a [separate repo](https://github.com/jurassix/joi-validation-strategy). With these changes I now expose options when creating the strategy. This should cover you use case. If you are still interested, try it out and let me know.

```javascript
export validation(strategy({...options})(component);
```


Yes, there is already some Java 8 code committed.


Why is that necessary?


Read the documentation [here](http://jurassix.gitbooks.io/docs-react-validation-mixin/content/overview/index.html) for reference.


Cool that this library handles everything for the screenshot.


Looking at the test case, shouldn't we have a count of 5, not 4?

One on line 2, one on line 6, two on line 11, and one on line 14?


Use java 8 optionals ?


Do you know what the problem with varintgb.h is?


Tengo la sensación de que con el Oculus ya puesto, va a obligar a mover mucho la cabeza, o a dejar letras muy alejadas. Lo dejaremos por ahora, hasta probar con el Oculus.


Why not provide static constructor ? Or a builder ? And make attributes final ?


cuando regreso de un menú y lo paso por aquí para enviar, la linea anterior le agrega un espacio extra, esto hace que GenCodeCommand no encuentre parecidos


@claudehohl

Can you add me to authors? I forgotten to add myself some time ago and i doesnt want to create PR only for this.


Just 'CERT.at'? I would prefer some more specific like 'CERT.at Contacts' to avoid possible future renaming because of ambivalent names (happened with Spamhaus).


you don't need to call this because the default finder allready calls it


I am not fan of this naming convention...


don't need a leading slasl


wrong


no, sorry. My proposal:

'CERT.at Contact Database'


done in a later commit


you need to add a newline before you write `The array has the following items:`.


idem, done later


Errors!!!

frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:713: error: illegal start of expression
    public void setHideQsTilesWithSensitiveData(boolean value) {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:713: error: illegal start of expression
    public void setHideQsTilesWithSensitiveData(boolean value) {
           ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:713: error: ';' expected
    public void setHideQsTilesWithSensitiveData(boolean value) {
                                               ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:713: error: ';' expected
    public void setHideQsTilesWithSensitiveData(boolean value) {
                                                             ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:717: error: illegal start of expression
    private class H extends Handler {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:733: error: illegal start of expression
    private static class Record {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:733: error: illegal start of expression
    private static class Record {
            ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:733: error: ';' expected
    private static class Record {
                  ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:733: error: not a statement
    private static class Record {
                         ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:733: error: ';' expected
    private static class Record {
                               ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:738: error: illegal start of expression
    private static final class TileRecord extends Record {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:738: error: illegal start of expression
    private static final class TileRecord extends Record {
            ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:738: error: ';' expected
    private static final class TileRecord extends Record {
                  ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:747: error: illegal start of expression
    private final AnimatorListenerAdapter mTeardownDetailWhenDone = new AnimatorListenerAdapter() {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:758: error: illegal start of expression
    private final AnimatorListenerAdapter mHideGridContentWhenDone = new AnimatorListenerAdapter() {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:774: error: illegal start of expression
    public interface DetailCallback {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:778: error: illegal start of expression
    public interface Callback {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:784: error: illegal start of expression
    private class SettingsObserver extends ContentObserver {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/QSPanel.java:838: error: reached end of file while parsing
}
 ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:49: error: illegal start of expression
    public boolean hasSensitiveData() {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:49: error: ';' expected
    public boolean hasSensitiveData() {
                                   ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:54: error: ';' expected
    protected BooleanState newTileState() {
                                       ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:59: error: ';' expected
    public void handleClick() {
                           ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:70: error: ';' expected
    protected void handleSecondaryClick() {
                                       ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:77: error: ';' expected
    public void handleLongClick() {
                               ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:84: error: ';' expected
    protected void handleUpdateState(BooleanState state, Object arg) {
                                    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:84: error: ';' expected
    protected void handleUpdateState(BooleanState state, Object arg) {
                                                               ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:84: error: not a statement
    protected void handleUpdateState(BooleanState state, Object arg) {
                                                                ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:84: error: ';' expected
    protected void handleUpdateState(BooleanState state, Object arg) {
                                                                   ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:95: error: illegal start of expression
    private boolean isSyncEnabled() {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:95: error: ';' expected
    private boolean isSyncEnabled() {
                                 ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:100: error: ';' expected
    public void setListening(boolean listening) {
                            ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:100: error: ';' expected
    public void setListening(boolean listening) {
                                              ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:105: error: illegal start of expression
    private SyncStatusObserver mSyncObserver = new SyncStatusObserver() {
    ^
frameworks/base/packages/SystemUI/src/com/android/systemui/qs/tiles/SyncTile.java:116: error: reached end of file while parsing
}
 ^
frameworks/base/packages/SystemUI/src/com/android/systemui/statusbar/phone/PhoneStatusBar.java:580: error: ')' expected
                    false, this, UserHandle.USER_ALL);
                                                     ^
36 errors


> "<Provider> does not support changing `store` on the fly. It is most likely that you see this error because you updated to Redux 2.x and React Redux 2.x which no longer hot reload reducers automatically. See https://github.com/rackt/react-redux/releases/tag/v2.0.0 for the migration instructions."

I got this error from the start, so probably with this updates sources also should be updated.


Like this:

```php
     /**
      * Returns the attributes of the method under the given index.
      *
      * The array has the following items:
```


I should clarify that this just moves these functions and the custom properties on which they rely into the revAnimation stack, the only place they are used.


But there's only one `0` with a unit? The other is already unitless.


yep, good catch


related to #7


In other part of james, this is a test scope.


That's strange seems to have think that we want to delete the file.

Just updated it with new changes and committed that.


composable なら、僕は合成可能と訳します。Applicative をつかってエラーの累積やるよ、とかいう話なら合成可能で通じるんじゃないでしょうか。Error handling は「エラー処理」と僕はよく訳しますが。


That's the example given by Jackson. This is fixed in a later commit.


Надо еще поработать над атомарностью коммитов -- если ты здесь поменял, то и изменение во вью тоже должно быть в этом коммите, потому что оно является частью этого изменения. У тебя сейчас атомарность на уровне файлов, а надо стремиться на уровень логического изменения.


of course :)


Yes. :)


@ipernet - thanks.
Can you try: http://flowplayer.blacktrash.org/fp6/dash.html - uses an additional load() call, I'd like to make do without any of the ugly seeking in paused state workaround.


metoda asta se apeleaza in json si imi intoarce role, pentru ca cu jquery iau subscriberii, iar fieldul de role e in WebsiteMember


What were your contributions? just that i know what to write in the authors.md


Ok... It outdates some of my previous comments


oh WOW. I'm bad at reading.


De acuerdo, pero creo que podemos dejarlo así:

(newCommand + " " + command).Trim();


closes #7


Perfect, fame +10 ;)

Several fixes (QR code, hardcoded paths etc., geshi update)
https://github.com/claudehohl/Stikked/commits?author=PJanisio


Fixed


very nice


And thanks


I think we should use C++ style casts throughout.

i.e.
```C++
VectorXd nullvec(static_cast<Eigen::DenseIndex>(0));
```


great! thx
one sec


P.S.

>В твоем случае это не так. Только что проверил -- не сломается. А знаешь почему? Потому что Spring нигде не использует этот messageSource. :)

Это не совсем так. На самом деле Spring оттуда все-таки кое-что использует, а именно `typeMismatch` и он, после удаления сломается.

Тогда давай так сделаем -- не будем смешивать разные штуки вместе. Вынеси `typeMismatch` в другой properties-файл и его уже используй как `MessageSource`.


? tu veux que vrmt que l'on mette en prod du code commenté ?


The CLI defaults to the system property `user.name` (in `ClientOptions`) to the server (via `X-Presto-User`). This can be specified directly via the `--user` property.

This username is passed to the connector via `ConnectorSession`. It is used for exactly two things in the Hive connector:
# The "owner" of created views
# As a lame security check that the owner matches when dropping a table

Additionally, the username is passed as the `owner` in the `ConnectorTableMetadata` object passed to `ConnectorMetadata.createTable()`, which the Hive connector then uses as the table owner (in the table object passed to the Hive metastore).

Thus, I do not think this should have any impact on HDFS.


Maybe considering changing this to Log.debug?  I still see the stack traces in my logs b/c of the `info` level and at first it's always very alarming and I think there's a regression. :)

Or maybe something like this, if you still want to log the full stack trace:

```
Log.info("Ignoring exception; verified object {} already exists with desired state.", resourceId);
Log.debug("Ignoring exception; verified object already exists with desired state.", ioe);
```


wtf, why y do dis?


See note above about Log.info and Log.debug.

Thx again for jumping on this and getting a fix out.  Life saver :)


This line is strangely indented.


原文書いた人は意識的に "side-" 抜いて "effect" と書いてると思うので、「作用」で ok だと思います。


Só estranhei a falta de um verbo ou algo que remeta uma ação no nome do método.


+1 . I added it mostly because there are now two places that would use it, but it's not part of the context for this PR. Should I add a javadoc with the expected behavior once it's implemented, or add a javadoc saying that it currently just returns true?


Indentation is a strange above too.


daca fac asta before_create token, nu pot sa il trimit ca parametru mai departe in email, pentru ca e  posibil sa nu existe userul.. si nu stim la ce WebsiteMember sa adaugam website_id si member_id


Pah, *listener*, *handler*. Nonsense! "Sink" is clearly the word you're looking for.
\*_runs away_\*.


Having the `return` keyword is pointless :P


可以留一下您的邮箱吗 我正在使用您这个库，但是发现不太会用，可以请教一下吗？我的邮箱是jp_athena@163.com


Good point, this do not have sense anymore, who is working with MRI 1.8 in these days?


Этот файл не совсем корректно назван. Если ты внимательно прочитал доку, то знаешь, что валидатор ищет файлы в некотором порядке -- `ValidationMessages_ru_RU`, `ValidationMessages_ru`, `ValidationMessages`. Если у тебя сообщения на русском то место им в `ValidationMessages_ru`, а не в дефолтном, который будет использоваться, например, для английской локали.


@jeremy-asher, I'm trying to understand this it looks to me like every resource is added to the filter here.
I have the Problem that the base class is falsely shown as "not covered", and without this line everything seems to work just fine.


@JanPeCol  how this exactly works?


Options to contain "file": Fixed in commit 18ac177 .

I decided to use the default mangling - key information about the command is at the beginning of filename (so worth for sorting/searching file in `sos_commands/qpid` dir), calling individual`add_cmd_output()` would make the plugin code less radable/understandable, due to the comments and `pep8` rules.


Actually, there were already a couple of NPEs on that. I'm kinda thinking that a NPE is indeed the proper exception here, as we don't expect it to be null. If it's null, then there's a problem in the setup of the consuming module.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 409](http://192.168.30.225:8111/viewLog.html?buildId=2311&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


private ?


@claudehohl
thx :)


なるほど、ありがとうございます。


@iamstarkov I'll take a look. Tracking it here: https://github.com/davezuko/react-redux-starter-kit/issues/42


了解です。


.. all of its texts,


thanks. sorry I just delete comment, because failed to find it. so I moved it into #40


TeamCity YaccConstructor :: YC.Core General Build [Build 490](http://teamcity.codebetter.com/viewLog.html?buildId=198374&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


ma sergiule nu intelegi ca nu e vre-o legatura intre subscriber si membru?? Tu chiar nu pricepi chestia asta? ... De cate ori trebuie sa iti zic. Subscriberii sunt aia care folosest un website si primesc eroare si membrii sunt aia care vad erorile adica developerii siteului. ... Ce treaba are subscriberu cu un membru?


its :)


@stephenharris There's a missing space before `to`.


Could be a good thing, but I think the costs are too high for the benefits. I mean, we would need documentation, maintenance, and so on for a configuration change that would probably never be changed. But if you still think it's good to externalize it on some configuration property, then I'll do it.

By the way: there's no Accounts configuration as of now, only a couple of system properties on the standalone.xml.


extract a method "extractUsername"


ti-am zis in mail cum sa faci. Asa ca fai cum t-am zis. Trimite mai jos website_member.id ca parametru si apoi in UserMailer faci WebsiteMember.find()


Reference application module is what pulls in all the global css from uicommons module, so you need to run the latest version of it that has been built off the uicommons module for this to work.


this sentence seems to long for me


Ever since this merge, I can no longer get "make dist" to minify initjs, it uses the un-minified version even though I still have Java installed and it successfully minified it before.


loooong sentence


I think I recall `hasDatepicker` was added automatically by the library and it was causing some issues, but it was too long ago to actually remember any details. It's possible that  the library has been updated though and it doesn't cause issues anymore.

Looks good to me though


Remember to move the css to the columns instead so that the icons in the action column don't get a strike through


Yep, this is okay!

Really, migrant is awesome gem tho :)


Fixed, thanks.


You sure? I remember reading on the javadoc some time ago that the try-with-resources in a reader closes all the underlying resources. I think I added this explicit close() there because I was having another issue and wasn't trusting that the javadoc was telling the truth (but the problem was somewhere else).


This looks *much* less like deleting the file! Yay!

Is the $ on 172 some remnant jquery in the code?


Hello,

I'm new to github and would like to include these changes to my next openwrt build.
Can you please help me merging your changes locally?

Thanks,
cchhat01


As for HDFS, there is this code in `HiveMetadata`:
```java
// use a per-user temporary directory to avoid permission problems
// TODO: this should use Hadoop UserGroupInformation
String temporaryPrefix = "/tmp/presto-" + StandardSystemProperty.USER_NAME.value();
```
This uses a temporary directory based on the operating system user that is running the Presto server, since this is what the HDFS client uses by default.

If we need to be able to override the username chosen by the HDFS client, we should provide a first class way to do it via `HiveClientConfig` (getting this into the HDFS client might require some hacking).


`then` and `do` should be indented as the `if` and `for` are on the previous line.


I think it's super good! :+1:


I cannot find the ending `dot` in the PSR5, nor the line break.


Yes this is the correct file.

Sure the last remnant jquery code, as you put it, is for the event handling on line 172.

After that the aim, is to see what else needs achieving so the widget works on IE8 too.


DashboardFixes (macumber) - Win64-Windows-7-VisualStudio-12: OK (2307 of 2343 tests passed)

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


Well, it calls `ps` directly so all that needs to be done is substitute in
the pure-darwin equivalent.

On Tue, Sep 1, 2015, 04:12 Christian Theune <notifications@github.com>
wrote:

> Or make it optional?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/4121c5064aea6a9a755a767f3ac4288e997498a8#commitcomment-12996132>
> .
>


単体テストとい意味不明な日本語がまかり通っているわけだから、「性質テスト」と誰かが訳し始めてもおかしくないはずだけど、まだメインストリームじゃないから訳語が求めれれていない? とりあえず「プロパティベースのテスト」で ok じゃないでしょうか。


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 474](https://teamcity.labkey.org/viewLog.html?buildId=269444&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:36
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

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

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

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

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

```


@jvazquez-r7 in this line, i noticed it's better to match the resp in lowercase:

if resp.downcase.include?('busybox') && resp.downcase.include?('built-in shell')

The reason for this is that i got another router recently, NuCom R5500UN. This router uses a newer busybox version. The jailbreak and modules work fine for this router but the busybox ash shell presentation text is in lowercase (the router answer is "built-in shell" instead of "Built-in shell").


it’s funny — i actually thought about this one.  scott meyers actually says that the one place he still uses c-style casts is this case — where you’re simply telling the compiler which method to call.

> On Sep 1, 2015, at 11:17 AM, John Carter <notifications@github.com> wrote:
>
> I think we should use C++ style casts throughout.
>
> i.e.
>
> VectorXd nullvec(static_cast<Eigen::DenseIndex>(0));
> —
> Reply to this email directly or view it on GitHub <https://github.com/RobotLocomotion/drake/commit/152b62249be35d3902de536c10a2bddf73043075#commitcomment-13001871>.
>


Sup I wanna help


I'm not sure exactly why the MSVC2015 compiler has issue with equal(datas[k].begin(),datas[k].end(), recov), but in any case it can be beneficial to know exactly which element differs between the two data sets. Not essential by any means, same as with the pretty print change above, please feel free to drop it if you have to, it's there just so people using ms compiler can proceed.


Guys seriously stop the port this I gonna be bad


This will break everything


DashboardFixes (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2294 of 2343 tests passed)

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


Looks like there are some extra spaces


twi header example, twi.h is under Wire\utility\twi.h


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 409](http://192.168.30.225:8111/viewLog.html?buildId=2311&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:12:08


I think that's the timeout set by `exwm--make-emacs-idle-for` hitting at just the right time. I really think we should remove `exwm--make-emacs-idle-for`, then remove any remaining problems that doesn't fix.

However, there might be other code using timeouts, and right now that would leave us in an inconsistent state, because we don't use `unwind-protect` when we should, so until that's fixed, it's probably best to do it this way.


A blank line after the header is correct


Different messages for each impl is better.  If that causes test problems then let's modify the tests.


Hey Justin,

These files were deleted because they are duplicates of photos that are still in the image dump folder. I have original copies of all files offline though, if you need them.

Cheers,

Amy


Yeah, maybe having a static build flag apart from the special mac logic
would make sense. Adding a new environment variable would probably be
easiest, something like

ifdef STATIC
STATICFLAGS=-static -static-libstdc++ -static-libgcc
endif

My only request would be that static somehow stays off by default, at least
on Mac...

On Tue, Sep 1, 2015 at 9:00 AM, Erik Garrison <notifications@github.com>
wrote:

> Gotcha. How would you enable this using a command-line flag or env
> variable to make?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/glennhickey/vg/commit/ac63771775d795055ace0de4ce76ebce9dec2163#commitcomment-12998267>
> .
>


This particular test doesn't have anything to do with system status though.


This should be "Allow" rather than "Allows"


Do you accept a promise, instead of a code? :-) I'd like to add that properly, instead of hacking something that wouldn't scale (I would have to have 10 properties on the standalone.xml for accounts... )


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1814 of 1814 tests passed)

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


I don't know why I assumed that :)


@tedhabermann this needs to be sync'd with the online version at http://standards.iso.org


Consider `maxGeolocationAge`.  It's a little more clear what it does since it's declared closer to where it's used, but still took me a minute to figure out what it was used for.


LGTM, but: "maitained" -> "maintained". Should also mention that `eventd_plugin_id` is a `const char*`.


^
Perhaps we can use this as an excuse to start that alumni page?
Also, [holy line lengths batman](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#1-overview)


If you want to split this commit into a separate PR, we can merge it now


[#102474398] test commit comment


Got you. This is a very good point.

The thing that sucks about auto-generated doc sets, such as the Go doc set, is that it's so impersonal. It's the entire payload of that library. I really don't want `wat` to be some official documentation source as that's already covered, rather 20 lines or less usage snippits for people already familiar with a lib and just want syntax. So I have to figure out how to automatically design these "partial" docs that emphasize usage examples over full descriptions, if you see what I mean, and it should apply to more than Node.

I was thinking of making a smart markdown to syntax parser that scaffolds out the docs, as READMEs are the closest thing to what I am trying to approximate with Wat: they're human-to-human. But that would obviously need to be pruned after generation as Qix said: there's no standard for markdown layout.

Wat will also have a command, btw, such as `github chalk`, which will pull the Chalk readme from Github and render it in the CLI., just as it currently supports `stackoverflow js splice an array`, using the Stack Exchange API.

Perhaps I can also check updated dates on docs, and for older docs make a warning that `wat` is not an official doc source, and on discepancies refer to the repo `here` kind of thing.


@djberg96
why do you need the "/"?


原文: 配備
訳案: デプロイ


//here we finish define variables


These are cut down from a more extensive test case that also checks returning structures from functions and it looks like I cut out too much.  I will fix it up.


Yeah, you're right, some code paths do accidentally call ItemLoader.UseItemFrame. At the moment the only useStyle at risk of doing this is useStyle 5 since UseItemFrame is in a bunch of else statements; I guess all we'd have to do is add an else before if(UseItemFrame).


:+1:  The keyword dynamic is a very good find.


had never seen that class, wow.


Ah, now I see that `inputStream` is used in `try()` to create the `InputStreamReader`. That makes the `inputStream.close()` redundant and `inputStream` can never leak.


101884090--NewVenting-Availability-Schedule (lgu1234) - 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)


www いいですね。


I think I'm now I just want to move the auxiliary files and macros to a subdir.

No, automake is overkill ... considering I want to manage only one makefile, so as long as I can avoid using it the better


I pushed the fix as a follow up commit, let me know if you'd like me to squash them.

Tested with an invalid log, daemon-reload produced:

```
systemd[1]: [/etc/systemd/system.conf:24] Invalid value for JoinControllers: "invalid,quoting
systemd[1]: Failed to parse file '/etc/systemd/system.conf': Invalid argument
```


#git clone https://android.googlesource.com/device/samsung/manta


Addressed in a follow up commit.


Creating Repository has been succesfull.


TeamCity YaccConstructor :: YC.Core General Build [Build 490](http://teamcity.codebetter.com/viewLog.html?buildId=198374&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 236 Build time: 00:16:43


同僚の名前を借りて煽り芸をやってる。


Git pull https://android.googlesource.com/device/samsung/manta


@NathanW2 I guess that is related to your recent theming work? Didn't look into it but I disabled it for now since it tags pull requests as bad and hides "real" issues. This message is just to notify you ;-)


Since we're effectively looping over `LAYERS` twice, consider pruning layer keys and forming layer URLs in a single unified loop.


Could we simplify the code and just have the list of differences? My main concern is that having too many "#if ..." makes the code harder to test and maintain. So if we can avoid them, it is better. Don't you agree?


原文: crafting real microservices
訳案: 本物のマイクロサービスを作る際の


This leaves RelStorage users with (cosmetic?) tracebacks on instance shutdown, always.  Would it be reasonable to wrap this in a try/except AttributeError?

Ref:
http://stackoverflow.com/questions/32326932/when-i-shutdown-the-zope-server-it-shows-an-attributeerror
https://mail.zope.org/pipermail/zodb-dev/2013-August/015119.html


@m-mcgowan Is this supposed to be the same as the User Module version?  If so it's currently set at 3, and by this change should be incremented to 4.  Are these here just for a compiler test, vs. letting Travis do it?
https://github.com/spark/firmware/blob/develop/modules/photon/user-part/import.mk#L1

Also we should duplicate this change in the APP=tinker app, or maybe make the APP=tinker app the default and remove this one so we don't have duplicate apps?  It's nice to be able to specifically say APP=tinker, and it's explicitly labeled.


When looking at the resulting file, for example here:

https://github.com/moy/sharness/blob/56e3872c3e3bcab620740ba313a56aa791dd7a47/README.md

it looks like `<dir>` from the above line is not displayed properly.


Actually, not sure that's appropriate here. As much as I think that we should use UTF-8 everywhere, this specific code is being used only to write the url parameters to Keycloak, and that should be properly encoded before calling this code. Perhaps the appropriate solution would be to add a javadoc mention setting this expectation.


@sagarlekar This is already inherited... Not sure what kind of weird issues this may or may not cause with mongoid


Ack. Sorry about that, not sure how I didn't notice it. I'll get this and other indentation fixups. Thanks!


I press enter key in a text field (eg. in Creating New Item) and get called a bitch. OFFENDED!


OK for me


Make clean removes compiler.jar and "make dist" doesn't copy it back which is most likely the issue.

I basically moved the compiler.jar dependency to "make dist-src-official" only because minification shouldn't matter for non-official builds.

Maybe this should be smarter.


Not at all, giving credit where credit is due, as it is mostly rehashing of your code base.


What is wrong with 10 properties for Accounts? Accounts is as important that 100 would still be OK ;) The number does not matter. Or you mean scalability in sense of runtime performance - i.e. that `System.getPropperty()` is slow?


> I cannot find the ending dot in the PSR5, nor the line break.

it's not psr5, it's symfony


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


@blacktrash Same results than https://github.com/flowplayer/flowplayer-mpegdash/commit/1de538ea6cbeeec1be0ec448af4b976e1520a292#commitcomment-12997774 on this page


@bronaghs This way I don't have to worry about capitalization issues or any minor string differences in powerstatus from MS. I had trouble with literal strings.


I believe the definition of "persist" should go into a separate subsection, parallel to 3.1 defining "ma".


Input is a built-in ..


Same here (input)


do we need both?


And actually, I'd find a System.getPropperty("accounts.mykey", 500) here in code enough. We do not need to put the default into `standalone.xml`


Perhaps the rule should detect the presence of closure-compiler/build.compiler.jar and copy it or use it instead.

Or alternatively, don't remove compiler.jar in make clean (only make cleanall).


原文: The Reactive Manifesto is applied along with Scala, Akka and Play.
訳案: コードは Scala、Akka、Play で書かれており、リアクティブ宣言の設計原則を適用する。


I wasn't actually exporting Intl properly indeed. I'm doing the fix right now.
I'll try to deploy it as a beta4 in the next minutes.
I might not have time though to test it properly. It's quite simple, but let me know if there is anything


Additionally - DRMHandler should be excluded from source files

s.exclude_files = "PlayerSDK/DRMHandler.*"

As it doesn't appear to be referenced, and is not protected from being compiled in a simulator build (thus getting link errors due to DRM library not being available in simulator builds)


Question to the reviewer: are such asserts acceptable?


@duglin Please ask on the mailing list. I don't want to split the thread. :)


Good catch.


Sorry about that.


Added the link


Opened a new PR to fix this at #11449


it's just hard to read in the diff, the sentence continues on line 95


I really would like to keep them separate.  There is no meaning behind their being the same really and I'm trying to get api/checkouts to use that module less, not more.


Eh?


Not sure exactly, here is the error:
Error C2475 'std::get': redefinition; 'constexpr' specifier mismatch	inmemorybenchmark
	C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\include\array	501
The thing is, FastPFor project builds fine with it, it's just inmemorybenchmark that has some header snafu going on with varintgb.h.


Done


Looks good


Consider a list comprehension over a `filter`
```python
[layer for layer in layers if layer.get('code') == table_code]
```
I think we can also lose the default `None` in the `get`


Testar lägga till bilden


Also, it's super-straightforward code.  Keeping it here makes it more readable about what's going on and hopefully keeping them separate will make it more immediately obvious that they can be independently changed.


Christian Couder <notifications@github.com> writes:

> When looking at the resulting file, for example here:
>
> https://github.com/moy/sharness/blob/56e3872c3e3bcab620740ba313a56aa791dd7a47/README.md
>
> it looks like <dir> from the above line is not displayed properly.

Well spoted. Fixed.

--
Matthieu Moy
http://www-verimag.imag.fr/~moy/


我无法反抗墙，
只有反抗的愿望。
我首先必须反抗的是：
我对墙的妥协，
和对这个世界的不安全感。
—— 舒婷：《墙》，1980年


I am closing issue #4 thanks to the available workaround.


Consider converting this to more idiomatic python string interpolation instead of concatenation .


What?


You're welcome. ;3


Oops I was on the wrong pull request.


ok thx


@4d47 This has been removed from the master branch – I will personally go through the entire codebase with a regex search and ensure there are no more offensive words/alerts. My sincere apologies... I can only assume this came from a late night debug session.


But I do, and it also makes for a simpler and more uniform license header, if you're ok with it.


For a start you don't even format the code... use a PHP ide. Second, half this code is wrong.


Quick list of the issues with writing data for Hive tables:
* Where to write final data for new tables: under the `Database.locationUri` from the metastore
* Where to write temporary data: assume `/tmp` on the same file system as the final location
* Avoid writing everything directly in `/tmp`: use a subdirectory `/tmp/presto`
* Avoid multiple HDFS users sharing same temporary directory: use a per-user directory `/tmp/presto-$USER`
* Can the user write to `/tmp`? It should be created in HDFS with mode `1777` (i.e. world writable with sticky bit -- standard UNIX / POSIX for a temporary directory)
* Can the user write to the database location (default `/user/hive/warehouse`)? This seems to be the tricky part and I'm not sure how it is "supposed" to work, but it seems that it also needs to be mode `1777`, or at least group writable by any user that needs to create tables.


Is there a race condition here? What if another worker writes to this key at the same time as the current worker?


I get the following console spam when executing upload_sources.sh (on the latest arm64 GMSCore v8.1.04)
```
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/1/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/2/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/3/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/4/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/5/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/6/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/7/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/8/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/9/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/10/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/11/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/12/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/13/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/14/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/15/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/16/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/17/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/18/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/19/': No such file or directory
fatal: Could not switch to '/media/git/opengapps/sources/arm64/priv-app/com.google.android.gms/20/': No such file or directory
```


Great refactoring in this commit.  I realize that my comments referred to existing code, but I figure if we're sprucing it up, we might as well.


"F help 1"? They invented ($args) for a reason


Ah so sorry I made you do this! Quoting JSON is horrible. I'd envisioned writing Python (dicts) then encoding that to JSON then URL encode it, to save you some quotes :) But this is definitely good!


Look at tntrun and see how "bad" are our ports


Ok, just seen that file reported as missing by grep, so thought you should know.


Hey luca28


No sync of addons ! to be added later


this is **very important**


`const This &other`


Will do. I just added them for consistency wrt. ``__TIMESTAMP__``, which I will also remove.


That would have been fun to engage one of the creature to see the linked ones coming from another room with all the "friends" they would have gathered will coming!


Tnt run isint that bad


Apparently I pushed the same fix from different computers. Doh.


Ouch, good catch.


btw...who are you?


But this is absolutely fucking shit


FYI I deployed the current SNAPSHOT on top of a 1.609.1 and it seems to work correctly... and before all it fixes the SOAP/REST switch transparently


I am a coder who is 12 years old and has like 250 twitter followers


basta, sto qui ha scassato


`getActualClass()` will return null unless the class is already compiled on the Classpath. This isn't the case for any of the API classes.


To have full speed operation (with SSE acceleration), you can consider using Eigen::Map and use operator on the resulting objects.


Awesome.


closes #900


In fact it is not bad at all
And this is not gonna be bad too


Databases are per-worker so that's not a risk, however if you've badly instrumented your code there's potentially a race with threading for the same worker so a global lock is needed on this __init__.


There is no formatting at all


They need to learn how to use
```php
if($args[0] ==


There will be


I'm gonna rewrite this and brb in 1 minute it'd gonna be way better


I believe this change is result of bad merge and should be reverted. There are more unrelated changes in this commit.


piaoguo


There are already other team mates who can do this


Thanks for the tip on "make dist-src-official", that'll help me for now.  I sometimes need to make modifications to Duktape itself (for example the error intercept behavior from #286 is a key feature in minisphere now) and being able to make an unofficial "official" build is useful. :-)


@philips will do - didn't originally because that thread has diverted from just "do we need a name field?".


We wan't your rewrite, go away


Should be fixed via 66563862ec3e0ac22a0cc6369eec587fa6dd0fec

On Tue, Sep 1, 2015 at 2:15 AM Eelco Dolstra <notifications@github.com>
wrote:

> This broke iso_minimal_new_kernel: http://hydra.nixos.org/build/25314303
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/8e26a55dc4e91b6590b6f616c0db1f22bef88b89#commitcomment-12994004>
> .
>


`school` doesn't have a `geom_detailed` column, you can ditch this line.  Currently, only `huc12` has it defined.


check https://github.com/locationtech/spatial4j/commit/106b861519c9d23d30d9db881d6ace93c4522db8


Should it be the same for "FAST_RUN"? Like this:
```
if theano.config.mode == theano.compile.get_mode("FAST_COMPILE"):
    mode = theano.compile.get_mode("FAST_RUN")
```


Actually I was wondering if this test is a good idea. I just wrote it to test that SWIG will not warn about those macros, even though the date-time warning option is set. OTOH it would even make sense to get a warning from SWIG when it ever parses these.
What would be the preferred way to test only if an option is accepted? Just replace the wdate_time.i with a file containing only a comment and set ``SWIGOPT += -Wdate-time`` for that file?


I'll try to figure out a cleaner solution - my bad.


Yes, because you don't pass the name into the stringWithFormat method but just the POICategory object. You want to call "name" on the object you get out of the array.


add newline


add pdb


@fycarman comunque non si sa perché abbia messo questa classe qui
occorre spostarla tra i sub command


wwwwwwwwwwwwwwwwwww


Good catch, and thanks for changing it in #3617


It won't return error, because we always declare a "help" flag if you don't.  Now if you declare a non-bool help flag it could also return an error.

My goal is to follow the same pattern as the "help" subcommand. We'll add a help thingie if you don't. But we also provide as few 'knobs' as possible to support everything users need.

I'm still not sure I understand your second question about contextual information in the help output. Looking at https://gist.github.com/eparis/5d0c25f605209cf2c1c0 I'm able to do what it seems you were wanting. What am I missing? (given https://github.com/spf13/cobra/pull/136)


Did you step into the poiCategoryWithName method and debug each line? It seems as if the creation of the entity fails. You have an error log inside of that method. Does it output anything?


cancella il progetto almeno la smette di rompere


va bon


MGNREGA


Too much talk and not enough X. という言い回しなので、「あまりにも数多くの講演があったが、」はいらないと思います。


For user documentation, I think just `ibm1.translation_table['buch']['book']` would be preferable, since this is closer to what they would be entering. We can use ellipsis on the output, i.e. `0.889...`


[The tool "Automake"](https://www.gnu.org/software/automake/manual/html_node/Use-Cases.html "Use cases for the GNU build system") can help you to handle more challenges around [make script portability](https://www.gnu.org/software/make/manual/html_node/Missing.html "Incompatibilities and missing features"), can't it?


Added the retired class from rows to columns at https://github.com/openmrs/openmrs-module-adminui/commit/d7a8c72f71ee50554daedc6d69b2d9902e75a4f7


null reference


`path: path.join(__dirname, 'file.ext'),`


You may want to split the body of this clause to a separate function.


etc


+1 , simple is good. Added :)


Is the tick tuple off by default unless the user sets it via `withTickTupleIntervalSeconds` ? Should we set some reasonable defaults here so that the tuples are always acked before they expire?


Because this is a moving target, the core signature checking is not dependent on this change, and the performance hit of checking the transaction signature before the SusPay digest is only an issue with transactions first seen during consensus, I would be a lot more comfortable removing this entire commit from this PR, and either giving it to you for #1255 or submitting it as another PR later. Do you agree?


Lol deleted here is what it could be like

```php
class Factions extends PluginBase{

public function onCommand(CommandSender $sender, Command $command, $label, array $args){
     if(!$sender->hasPermission("blahblah.f")){
		 return false;
	 }
	 if($command->getName("f")){
		 if(count($args) == 2){
			 if($args[0] == "help"){
				 if($args[1] == "1"){
					 $this->showHelpPage($sender);
				 }
				 elseif($args[1] == "2"){
					 $this->showSecondHelp($sender);
				 }
			 }
		 }
	}

     public function showSecondHelp(Player $player){
		 $sender = $player;
		 $sender->sendMessage(TextFormat::GREEN . "-=-=-Factions Help-=-=-");
		 $sender->sendMessage(TextFormat::GREEN . "/f ally <name> = send an ally");
		 $sender->sendMessage(TextFormat::GREEN . "/f neutral <name> = make a Faction neutral");
		 $sender->sendMessage(TextFormat::GREEN . "/f claim = claim a land");
		 $sender->sendMessage(TextFormat::GREEN . "/f unclaim = unclaim a land");
	 }

	 public function showHelpPage(Player $player){
		 $sender = $player;
		 $sender->sendMessage(TextFormat::GREEN . "-=-=-Factions Help-=-=-");
		 $sender->sendMessage(TextFoemat::GREEN . "/f list = show the list of all Factions");
		 $sender->sendMessage(TextFormat::GREEN . "/f create <name> = create your Faction");
		 $sender->sendMessage(TextFormat::GREEN . "/f invite <player> = invite a player to your Faction");
     	 $sender->sendMessage(TextFormat::GREEN . "/f kick <player> = kick a player from your Faction");
		 $sender->sendMessage(TextFormat::GREEN . "/f join <name> = join a Faction");
		 $sender->sendMessage(TextFormat::GREEN . "Do /f help 2 for second page");
	 }

}


Nope, this one: 95c2def7ee40c7a48c3868443387039f8d7247c3


Just fyi, there's about 80 warnings with this commit missing XML comments.


What about the case where the PID is the same? We are using the gevent gunicorn worker, which doesn't appear to spawn OS threads per worker.


This comment does not make sense given the context where it's used. (No mention of exports.) Also, can you explain what needs to be done in that case?


Thanks @hoontw


`@KitchenSink`

On Tuesday, September 1, 2015, Mumfrey <notifications@github.com> wrote:

> Pah, *listener*, *handler*. Nonsense! "Sink" is clearly the word you're
> looking for.
> **runs away**.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SpongePowered/SpongeAPI/commit/64a72376a3d6f761c417d7a01fa4d58067f66391#commitcomment-13001939>
> .
>


Best layout yet here.  I love it!

You can visually see the cases where you increment and where you don't.  In the previous layout, it took me a second reading to understand that you didn't increment the variable in each case.


Is there a specific version we can add


ああ、こういう成句なんですね。勉強になります。


seems ok


return**s** :)


Sure thing, thanks for the quick response.


In that case you don't need the multi-process collector, as the usual way will work fine.


wooo you can use sendMessage(), you're a god


Or maybe:
```
#pragma omp simd
for(size_type i = 0; i < size(); ++i)
{
    data[i] += other[i];
}
```


Let me check by removing the include


With %s substitutions, the message should still only be evaluated when the message is emitted, whereas "".format() would be regardless. The {} syntax could be used if the logging [formatter](https://docs.python.org/3/howto/logging.html#formatters) is changed, but we should spick {} or %s and use it throughout the project.


Ah! Thanks.


Please note I did not write the appropriate test for this as I do not have a dev environment for GS.


What?


I fixed formatting (There was none) and I made commands better... just use that code above and use pocketmine\Player


Very good thinking, it should be.


But  - isn't this returning an iterator?  What sort of iterator isn't movable?  Heck, aren't iterators even supposed to be copyable?


Short comment to explain this bit at a high level?


Whoops, yes, that was committed in error, thank you


You can drop both `pass` here.


Hi @eparis ,

See this [line](https://github.com/spf13/cobra/pull/136/files#diff-9c42b524cb14ca001c61267826cbefb1R575). Note that when you create the help flag you're doing so in the context of a function that is given a reference to a command. That command's name is therefore available when initializing the help flag. This could be possible if I wanted to traverse my command heirarchy myself, but that's cumbersome.

Perhaps a `(c *cobra.Command) VisitAll(func(currCmd *cobra.Command))` method for a command to visit itself and all of a command's sub-commands? That way we could easily do global initialization for all commands at once.

FWIW, the HelpTemplate, HelpFunction, and HelpCommand are all IMO broken. I simply cannot get them to work properly. I'm not sure if this is related to anything else, but it is the case nonetheless. What I want from those three things is something tantamount to `SetUsageFunc` where I give a function that is called for generating the Help string. Instead it's a combination of three things that just is not clear on how it all works. Again, this could be an aside, but I'm not sure.

Let me itemize my three asks:

 1. Change the shorthand value for the help flag.
 2. Change the help flag's usage string so it's cased the same way as the other flags (cased as a proper sentence without trailing punctuation)
 3. Have the help flag's usage string reflect the name of the command for which it's being printed

Clear? As mud most likely :) I do try though!


Please don't reply again. Our team have big developer like @xionbig , @luca28pet and @EvolSoft, we can fix it


I'm friends with EvolSoft online kinda


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (760 of 760 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-9.67%25-green.svg)


@scbunn I let myself remove the float() call here as it was a no-op, added decimal point to 1000 to make it clear though


I asked to join I was told yes.


@scbunn See my comment on line 53


Maybe yes in EvolSoft organization,but not in ItalianDevs4PM


No need to use AM_CONDITIONAL here. Simply check for non-empty PGREP in makefile.am. Make sure to replace the [no] from AC_PATH_PROG to nothing.

You can see all output variable in config.log in the output variable section.


Absolutely.


Using that markup isn't very convenient. I was thinking of mapping formatting like bold, italics to classes. But maybe we just need a better syntax for specifying CSS classes on paragraphs.


I'm with the other big developers like @byteandahalf @RedstoneGunMade @TrinityDevelopers (MrAndyPE) @DefinitlyEvil @NiclasOlofsson @jython234 @TekkitCommando

Beat that


none of them does pm plugins


I'm not Italian and I was giving you code don't get so cranky about it we all know it's your period but seriously.


Oh well I do pm plugins that all that matters :P


@zidein07 https://fortawesome.github.io/Font-Awesome/icons/


This was just on my mind given @mourner's blog post, but I don't think we've heavily tested recent versions (i.e. @kkaefer's improvements beyond my initial port) inside of GL proper yet.


I see, thanks :+1:


Take a look at this commit.
https://github.com/losipiuk/presto/commit/5e29651a3a189ab6a5cb614a1ea97aeac8dbbf14
Does this look better to you?


From EntityAIControledByPlayer


good catch -- check: https://github.com/locationtech/spatial4j/commit/c6bd836091ff27f8d641c659a757a1f8cdc16682

"The spec" :)  we are making it up here -- the standard really only supports *polyline* we are adding some sugar to that to represent the other Shape objects


and the dot is not special rule, that's just a rule of English, lol


Are you on IRC or somewhere we can chat a little faster?   eparis on freenode....


fixed


Order that `Listenerd` methods may be registered at?


Ah, disregard. Yes, we want https://github.com/mapbox/geojson-vt-cpp/commits/995ffc72c556da4b4880a6036fbcf7159fc5ecce for now, but need still to integrate https://github.com/mapbox/geojson-vt-cpp/compare/995ffc72c556da4b4880a6036fbcf7159fc5ecce...master into GL. I will ticket.


Indeed! 16e2851


`std::unique_ptr<sles_type::iter_base>` is not an iterator.  It is the guts of an `iterator` that is involved in this particular bit the API.  And again, this PR is not going to seriously alter the design of this API.  If desired, that is a separate task.  This PR is focused on optimizing that one line you noted yesterday.


JavaDocs missing


Since there was no response on this I will revert this change: #9352


Updating Colloquy right now :)


Sorry, I'm just testing my bot. Ignore this.


Yes, that's a good observation. The "/v1/query" interface is not meant for public consumption, yet (there are no guarantees for backward compatibility, etc), so let's not move it to presto-client at this time, then.


if "exponentiate" refers to back-(inverse-link)-transforming the parameters/confidence intervals, might I suggest that a more neutral/general argument like `scales="exp"` would be appropriate? Otherwise I'm afraid we might end up with a wide variety of transforming/scale options across different models (cf. `scales` argument in lme4 tidiers; `unconstrained` argument in the `coef()` methods for `nlme` ...)


How about "Some configuration options may require turning in specific situations:"


Oops, I guess I had the opposite impression of which way allowed for lazy eval, but that makes sense. I'm not sure that I care that it has to be absolutely consistent throughout. I wonder why pylint prefers the non-lazy expression.


The loading functions is in this file because of the boost dependencies. I will move it in [`descriptors.hpp`](https://github.com/poparteu/openMVG/blob/voctree/src/openMVG/features/descriptor.hpp) without the dependency.


@biyeun doesn't this *only* let them see "unsafe" reports?


como implementar 1.8 sem perdermos o calculo de raiz? Sugestão, vamos criar uma nova improve, digamos improve3 e manter a improve ou mesmo renomear para improve2.  estou rejeitando este pull request para receber nova sugestão. sqrt-iter e sqrt também podemos duplicar para ter uma para sqrt2 e sqrt3.


I'd rather diagnose the issue:  how is it that only RelStorage leaves the connection in the state where it has `None` as its `_storage` attribute?


stop flaimg.


Add a TODO indicating that this needs to be a parametric function in the future (once the type inference engine and system are fixed)


well, as the variable says, it's a CONN/B1 PDU, internal commands are not explicit but I think we can live with it


Hi @eparis,

I just sent my phone # to your RedHat e-mail address if you want to shortcut IRC altogether...


Don't we need intlTelInput.min.js ...


英語だと引用符が必要かもしれませんが、日本語だと英語書きにしただけで他から浮くので引用符抜いたほうがいいと思います。


Isso é lindo!


Also adjust the names in the tests please.


What is `0x3f`?


All this shuffling around between representations (I remember seeing some earlier too) makes me wonder whether you picked the best data representations.


"Use hash distributed joins instead of broadcast joins."


that was fast!


What is `10`?


Thanks.


frontend treba koristiti nasu fju knockDown samo da bi nam javio koliko cunjeva da srusimo radi odrzavanja score-a


Hi Simon,

My name is Vinay a PhD candidate from WMU. I am trying to use your library in my dissertation and was having problems while running on large data sample of 100x9. I modified example code to take distance from my distance matrix. For some reason algorithm ran all night and was still computing. Is it suppose to take that long, and you please suggest what I am doing wrong. My email is vinay.b.gavirangaswamy@wmich.edu.
Thank you,

ps: sorry, I couldn't find your contact so leaving message here.


Hi Anthony I've tried many many variations on this one. I know that tag is
the issue, but from what I understand and have seen through my testing is
that I need to do something like that. The reason is because when I do it
without 'nesting' it doesn't work the way I've intended with the re-prompt
and while this way doesn't work at all, it's the only strategy that's of
significant difference that I feel I'm on the right track, but I can't
quite figure out how I need to change it to make it work.

On Tue, Sep 1, 2015 at 8:08 AM, Anthony Sopkow <notifications@github.com>
wrote:

> Is this where you actually want the "}" to be?
>
> What are you actually trying to close off.
>
> Hint: Chrome's error is "Unexpected token else"
> What this means is that when the javascript is running, it comes across
> one of your "else" statements when it's not expecting to.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/pythonlabsstudent/js1/commit/61eca6f3fcbb74f2b0fd83951813234e9a29aeaa#commitcomment-13000008>
> .
>


So systems where getenv() makes no sense will still get all this block before falling into their own implementation?


Hadopo typo


`sles_type::iter_base` is not an iterator.  It is the abstract base class for a family of polymorphic iterator implementations that are subsequently wrapped by a handle that is an iterator: https://github.com/ripple/rippled/blob/develop/src/ripple/ledger/detail/ReadViewFwdRange.h#L36-L75


I noticed that, but [the test is short and sweet.](https://github.com/groupserver/gs.dmarc/commit/851c7b604ef3eb429e77e209510eeeb4c8f2cc0a)

The gs.dmarc library is so low-level that it requires almost nothing from the rest of GroupServer. It is even [on PyPI.](https://pypi.python.org/pypi/gs.dmarc/) :)


`If not already backed up`


Interesting!  How long has that abstract IPC stuff been available?  I considered pgm early on but ran into compilation issues -- didn't spend much time on it.

We do have a couple of our own little broker-like daemons for this project.  ``fedmsg-relay`` rebroadcasts messages from ephemeral processes (like shell scripts) and ``fedmsg-gateway`` rebroadcasts the activity of the whole network for subscribers outside the firewall.

FWIW, fedmsg is pretty stable and working well, so we're not looking to make major changes unless there's a big win in there.  zeromq's been good to us.  :)


原文: Functional programming is the source of the flexible abstractions we need, because working with data is doing Mathematics, the basis of FP.
訳案: Big Data において必要とされる柔軟な抽象化は関数型プログラミングから得ることができる。なぜなら、データ処理とは「数学をする」ことに他ならず、これは関数型プログラミングと基礎を共にするからだ。


Move the `new Cast` into the `createFailureFunction()` method. It's an implementation detail of how the failure function is realized.


I don't see any uses of this. Is it dead code?


<ShadowAce> Paleocrafter is a spell checker xD
<PaleoCrafter> Ahem, it also is 'capitalization' :P
<jakimfett> ...gawddamnit this is why I can't have nice things


Why the substitution of HashMap for Hashtable?


@skeptical @taconaut Shouldn't the button just be hidden (.setVisible(false)) for non-Windows OS'es?


Use the code I gave you don't be a bitch


Thanks @quazzie, i will push a new version today or tomorrow. i will merge this one then


Why change from Hashtable to HashMap?


l guess you need to test if RID doesn't exist and not wireless[RID], else it could abort with a missing table index error


I think this should be `type(expression).getTypeSignature().toString()`.


```go
else {
  self.DeleteStateObject(stateObject)
}
```


Extraneous blank line?


It would be nice if you could have two DataBoundConstructors, one with the old signature and one with the new signature. Is this possible? The passing in nulls to get a default value for a parameter, such as the templates, is not ideal.


If we're going to do this for jira parameters, presumably we're going to do it for every variable in these config files?


This breaks toolbars that use `ng-include` or `ng-controller`. Of course the developer could wrap there toolbar in an extra `div` and place their `ng-include` or `ng-controller` attributes on that element but then you end up with two extra unnecessary `divs`

```html
<md-toolbar>
  <div ng-transclude>
    <div ng-include="my-toolbar-template.html">
      <!-- actual toolbar content -->
    </div>
  </div>
</md-toolbar>
```

I think most developers would expect to be able to put `ng-include` or `ng-controller` attributes directly on the `md-toolbar` element.

Maybe instead of using transclusion you can manually wrap the contents of the `md-toolbar` element in an extra `div` if necessary. In addition, the isolated scope is going to break putting `ng-controller` directly on the `md-toolbar` element as well. Maybe you can use `bindToController` instead and create a controller to handle scroll shrink?


If `Signature`s are not types, then you should add them to the list of excluded values in `Type/c?` and `Values/c?`.


you could use `const`, right? any reason not to? Or does it not work if it's a `Type`


TeamCity FluentMigrator :: FluentMigrator [Build 340](http://teamcity.codebetter.com/viewLog.html?buildId=198376&buildTypeId=bt82) is now running


Valid concern. I'd still like to see the regex appear so that end users would know what standard we'e using to declare the input invalid.


You guess?


:( damn. I guess I'll replicate this on linux-micro and systemd, then.


@dsanders11 was there a noticeable improvement?


hmm this variable line and 31 should be the same.


I don't know. @samth, @takikawa?


indent 2 spaces


Ok, that looks reasonable. I'll try again.


Makes sense to me. There's already a few methods supported by `broom` that could take advantage of one or more generic method(s) to transform the coefficients.


Idem.


@rafaelfranca https://github.com/rails/rails/pull/21453


Pull request?

On 9/1/15, Joesalmon1985 <notifications@github.com> wrote:
> So I want to accept these changes, but don't seem able to do this.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Joesalmon1985/Canvas/commit/fe92c34fe9999c8a77baa57abf7c1f7d072da377#commitcomment-12999706


Or ifdef it, better.


@dsanders11 was there a noticeable improvement with a larger buffer size?


In the future with the region concept the loading can be done according this load function: https://github.com/openMVG/openMVG/blob/develop/src/openMVG/sfm/pipelines/sfm_regions_provider.hpp#L60


2 space indentation, same as others below. you might want to consider changing the indentation value in sublime before things really get out of hand.


原文: Because of Scala's basis in FP, its general flexibility, and its JVM roots, it has become a standard for Big Data programming.
訳案: Scala の 関数型という基礎、多岐にわたる柔軟性、JVM という出自を考えると、Scala が Big Data プログラミングの標準として台頭してきていることはある意味当然の帰結とも言える。


TeamCity FluentMigrator :: FluentMigrator [Build 340](http://teamcity.codebetter.com/viewLog.html?buildId=198376&buildTypeId=bt82) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 3113, ignored: 60 Build time: 00:02:09
### Failed tests
```
FluentMigrator.Tests.dll: FluentMigrator.Tests.Unit.Generators.Oracle.OracleTypeMapTests.CurrencyIsNumber: <no details avaliable>

```


Actually, might as well be explicit. Do you like tabs or 4 space indents?


By definition in the constructor the field __templates__ cannot be __null__ and yet there are multiple checks for it being non-null in other parts of the code.


I don't think you can use `const`. I prefer the use of `Type` to prevent magic strings from being sprinkled around to the mutability possible with this approach.


For the vocabulary tree creation we don't need the features, only the descriptors. So maybe it's not necessary to use the regions ?


what is it


general note, no negative commit messages please :smile:


Hey @temasek ,

Does this actually work for you? I picked it from Slim's gerrit a few days ago and I couldn't get it to work on my build.


@scottpurdy this test is heavily dependant on nupic.vision, so I've removed it. If you want, I can create an issue in vision where I'd paste the code and leave it to be implemented sometime?


If a lot of tests are broken then it means we have a lot of code that depends on these attributes that should never be used directly but via the 'xwiki-meta' service instead!


@4d47 I sincerely apologies, This probably was some late night coding frustration. These doesn't get into the public. Was trying to say to myself. No offense to anyone Intended.


Its not clear to me how you can avoid using a smart pointer container to hold a polymorphic iterator implementation.


Looks good


Thanks for some feedback this is good.  The first caption issue: the button should be disabled or removed in the case where no tracks are added. Will fix today.   The second one Is only in IE.   Not really a control bar issue but rather the ability for dash.js to clean up tracks in IE.  I should be able to patch for IE not to show tracks marked disabled.....

Regarding the MP issue couple of questions. Did the native controls only show the current period's duration or the entire sum of all periods.

Did the native controls allow seeking into a future period and back into periods after they have past.   I need to look at this but really it should be simple as the seek bar is a reflection of duration and time so must not be getting right values set in duration for period content.  Lets talk on the call about this and Ill fix today and try to get fullscreen done.


@djberg96
is the idea here that the VM power status is set to "VM Starting" until the next EMS refresh updates the power status value to "On"?


@ipernet - one more for the road http://flowplayer.blacktrash.org/fp6/dash.html - and if it works, can you try pause early on, and then seek to an unbuffered position, to check whether it stays in paused state after seeking.


It was a suggestion... that show how it was done in other place in OpenMVG.
You can take a look at the code and only load the descriptors that will be limited to one type...
You can imagine make a struct Regions_Descriptor_Provider that does not load the features...


I just added. Yet to test. Will do it first thing tomorrow.


yeah


hehehe


Git clone


In this context, you automatically have an instance variable called @video that has what you would expect in it.


Git init


Can you change the comment


Thanks


Sure, let's fix them before breaking them all! I'm not fond of the approach of committing something and breaking everything and then fixing them one by one because this introduces instability for everyone and slows down everyone in their work.

Also, don't forget about existing extensions that may use these.


add comment explaining that this is to limit what can be done via the central HTTP API when Heroku is being used, for security


Maybe horizontal instead of left to right, because he is also going from right to left.


Haven't tried it, but couldn't we just compare `stateObject` to `coinbase` here?


You are commenting in a PR


Yes.


Here too


comment on why this is necessary for heroku


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


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


Also I think there might be issues if we create a clean state. Maybe stateObject.Copy() would be a better option?


And by PR you mean commit. ;-) Yes, sorry, that's the problem with these stacked PRs.


:point_right: https://github.com/mapbox/mapbox-gl-native/issues/2226


Take a look at this..


@idgserpro se você executa localmente não vai ter problema: eles bloquearam o Travis


The abstract IPC functionality came in 4.0.1. It makes IPC a lot more usable. If you liked ZeroMQ so far then Malamute could be a fun future addition. It does pubsub but the main advantage is mailboxes, so you can do remote service calls and replies.


Ah no, that copies code and everything, and we want a fresh account. Though it would be nice to know what the correct values for the nonce and others are after a suicide.


Add something like: The game shall end if all the enemies are dead


Can you actualize the apidoc documentation with the new parameter in the routes that return a session/s? Also remember change de SuccesExample in "/games/:gameId/versions/:versionsId/sessions" (delete de start field and add the new).


Why do you want to revert this one? pulling ion @writeline


Top left?


Hello @Andrei-Stepanov, I see your point, but this does not work nicely with paginator. (I know, in this case it's intended, but we should keep to the logging mechanism used withing the project).

I can imagine several solutions, the simplest by using dedicated logging stream (we'd like to introduce some debug streams tunable from cmdline, we're just not there yet). Alternatively you can use a `View` to display minor message.

My preferred solution would be to execute `discover` as separate thread and periodically (eg. per second) check the status and display a message. This should be even faster, than printing each line :wink:.


@dcs619 Do you know where I can toggle a setting to get warnings for missing XML comments?

I've installed GhostDoc and gone through and cleaned up the missing XML I can spot but I want to make sure I don't miss any before I pull.


And top rigt for lives


where you can training data...


power-mup xp?


Although it might look like the VM is hanging, its probably better than being in an "Off" State until the next refresh.


Any reason we need the sqsp identifier? So long as we use this forked version we should be alright or is the concern that the non-forked version may end up being installed / used?


Is the guest groupId always going to be 0? Seems like we use the
```
Group guestGroup = GroupLocalServiceUtil.getGroup(getCompanyId(), GroupConstants.GUEST);
```
elsewhere too.


Very cool.  Thanks!


Updated as suggested.


MSM8226 :p


Yup! I've tested it. What are you looking for?

> On Sep 1, 2015, at 1:46 AM, asdfsx <notifications@github.com> wrote:
>
> Execuse me. Does this project support mesos 0.23.0?
>
> —
> Reply to this email directly or view it on GitHub.
>


You misspelled canola, idiot.


Here is a comment


add link to the PR that made nestRemoting?


My mistake. It should be

```

if theano.config.mode == "FAST_COMPILE":
    mode = theano.compile.get_mode("FAST_RUN")
```


we have login such account.uuid/login and after this showing only login


Good idea. I hope they will update this service to some newer Ubuntu.
By the way, there is also a problem with libgsl.

g++ -c -m64 -pipe -msse2 -ffast-math -fopenmp -O3 -D_REENTRANT -Wall
-W -DQT_WEBKIT -DUSE_EXR -DQT_NO_DEBUG -DQT_GUI_LIB -DQT_NETWORK_LIB
-DQT_CORE_LIB -DQT_SHARED -I/usr/share/qt4/mkspecs/linux-g++-64 -I.
-I/usr/include/qt4/QtCore -I/usr/include/qt4/QtNetwork
-I/usr/include/qt4/QtGui -I/usr/include/qt4 -I. -I/usr/include/OpenEXR
-I../qt -o animation_flight.o ../src/animation_flight.cpp

In file included from ../src/keyframes.hpp:28:0,

                 from ../src/animation_flight.hpp:26,

                 from ../src/animation_flight.cpp:23:

../src/morph.hpp:28:28: fatal error: gsl/gsl_interp.h: No such file or directory

2015-09-01 14:37 GMT+02:00 Sebastian Jennen <notifications@github.com>:

> i added libqt5gui5, qt5-default as a issue to the approval of travis ci,
> lets see how this works out...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/b22ec57194fa1e98ffa91aba97bc5dee2e449236#commitcomment-12997822>
> .
>


use array literal notation and camelCase: boxPolys = [];


All looks good to me - and a much better GetMapFromAttribute!


Sure


It's to ensure we pull our own version from devpi. Since devpi is a caching proxy, it will go fetch the real one if ours isn't there for some reason.


This isn't quite a _range_, but it will do as a first approximation. Unfortunately, this pattern is not accepted by some of the supported compilers (at least VS2013).

Also, formatting looks weird.


You seem to be relying on the fact that all reasonable encodings have the first 127 positions identical with ASCII? I have not studied the practices in the wild in detail, but such encodings are clearly possible. Googling for 'ascii 7-bit "incompatible" encoding' brings this [1] page that names a couple of examples: UTF-7, UCS-2 and UTF-16. UTF-16 used to be the default on some Windows versions, IIRC.

[1] http://perlgeek.de/en/article/encodings-and-unicode


This line gives me an error, you need to index it by doing .get()


Here you have command line only with yum, but later "dnf install scap-security-guide" you have yum/dnf variant. Do you have any special reason for that?


You should get the max-let-depth and the dummy too


Same thing here with .get()


It's still there, dummy


Why we are comment this thing?


I'm not sure _when_ it calls that. But it doesn't for me. Everything is fine when I remove that dependency. :)


Picky but could you do with a filter or a comprehension?


Seems to me ```super.isValid(value)``` will be always ```true```. Should we omit this call?

I believe that ```(required && checked || !required)``` can be simplified to ```!required || checked```.


**markdown**


Some comment


Why this change? The convention is to use `typename`, and it introduces inconsistencies even within this file.


Either this or m should record the phase for the higher phase errors.

Can you post the error you get from the phase=2 test?


I might also be misunderstanding - you're saying we're calling the "impure" version of ps otherwise?


could be a lambda?


probably capabilities => Capabilities


@AkamaiDASH, native controls show the sum of all periods and they allow to seek into a future period and back, so in general it looks like we have a single period (and I believe it should be like this)


remove tests for '3p4...' because they have several structures in pdb. (and too lazy to count).


I don't know anything about that branch.

On 9/1/15, Imnotgivingmyname toamachine <ben.greencanvas@gmail.com> wrote:
> Pull request?
>
> On 9/1/15, Joesalmon1985 <notifications@github.com> wrote:
>> So I want to accept these changes, but don't seem able to do this.
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/Joesalmon1985/Canvas/commit/fe92c34fe9999c8a77baa57abf7c1f7d072da377#commitcomment-12999706
>


s/delet/delete


s/butons/buttons/


I believe this might fail with gcc4.6, be sure to make a run for it on buildbot.


s/hte/the


Is this better than `np = __import__("numpy")` ?


test file to intro class


Hi, is there a reason why onDraw is called with setTimeout?

This makes it impossible to update the DOM styling before browser rendering. So if you change the background of specific cell in `ondraw` event, it will cause a flicker. (because changing of the background will happen after a full render)


LGTM


Thanks for reviewing Diane.  I will make those changes and re-commit.

On 2015-08-31 01:28 PM, Diane Srivastava wrote:
>
> @adletaw <https://github.com/adletaw> Thanks for making these commit
> notes so easy to follow! Everything looks really good, except I would
> like to make these two changes:
>
> K547, K560: FD5 -> 1, we know little about Brachypremna apart from it
> being a Tipulid genus, so there is little indication that it consumes
> more "living leaf tissue" (FD5) than the average tipulid. Keep at
> tipulid average of 1 for this trait therefore.
>
> K433,K506, K544, K434: FD8-> 1. For species identified only to
> subfamily Limoniinae, I set FD8 (living macroinverts in diet) to 1
> because FG6 (engulfing predators) for this subfamily is 1. Although
> some genera include predatory as one of several trophic modes (e.g.
> Hexatomini, Rhabdomastrix) the others do not (Ormosia, Trentepohlia,
> Limonia sp). (DS notes). Of these genera, Trentepohlia is far and away
> the most common in our dataset.
>
> Either you can make these changes if you are online, or I will see
> that they are done.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SrivastavaLab/bwg_names/commit/e3da5739da5b21d42aa8eed49c29e2c32ca2bc82#commitcomment-12984669>.
>

--
Alathea D. L.
PhD Candidate, Srivastava Lab
www.zoology.ubc.ca/~srivast


EDAPT_Issues (macumber) - i386-Windows-7-VisualStudio-12: OK (2302 of 2343 tests passed)

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


@lgravley need help contacting WhiteRose


One thing. All SAI comments should be "Creature - Event - Action" where all yours is "Action".


You simply never know what will be the given platform's default. I strongly vote for explicitly putting "UTF-8" or similar here.


The name of species.java needs to be capitalized since the class is in this file. I get an error otherwise.


原文: Sane Sharding with Akka Cluster
訳案:  Akka Cluster によるシャーディング
"Sane" は正気、つまり狂気の沙汰じゃないシャーディングという意味ですが、抜きましょう。この界隈で「健全」と言うと、hygienic の意味だと読まれる気がしますが杞憂でしょうか https://ja.wikipedia.org/wiki/%E5%81%A5%E5%85%A8%E3%81%AA%E3%83%9E%E3%82%AF%E3%83%AD


Hello, I cant ANY support for the weather module not working correctly to my site?


@jushe
It's click effect or **feedback** not an animation


Well I haven't broken out the tunneling electron microscope to check the individual atoms on the chip as it steps through the instructions yet, so it's _just_ a hypothesis. :)


When I build this reaction from parsing, I will only have the k. I don't think the other two arguments are going to be relevant for the constructor.


Added valuable new code


Thanks - I will fix it in the Clang repo


Thanks for finding that!


it can indeed, but I think I'll add automake to the mix only when strictly necessary


原文: If you are interested in scalability, this talk is for you.
訳案: このセッションはスケーラビリティに関心を持つ人に是非来てほしい。


確かに。あと、型理論方面でも「健全性」とか言いますもんね。


EDAPT_Issues (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2279 of 2343 tests passed)

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


`__import__` is a low level function, usually using `import` is prefered.


101884090--NewVenting-Availability-Schedule (lgu1234) - i386-Windows-7-VisualStudio-12: OK (1814 of 1814 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)


@Nadahar I didn't want to change the behaviour when doing the pull request, but it would definitely make sense :)


right
I don't like that magic number
you can use split


You carried out that merge on July 29th. What is wrong?

On 9/1/15, Imnotgivingmyname toamachine <ben.greencanvas@gmail.com> wrote:
> I don't know anything about that branch.
>
> On 9/1/15, Imnotgivingmyname toamachine <ben.greencanvas@gmail.com> wrote:
>> Pull request?
>>
>> On 9/1/15, Joesalmon1985 <notifications@github.com> wrote:
>>> So I want to accept these changes, but don't seem able to do this.
>>>
>>> ---
>>> Reply to this email directly or view it on GitHub:
>>> https://github.com/Joesalmon1985/Canvas/commit/fe92c34fe9999c8a77baa57abf7c1f7d072da377#commitcomment-12999706
>>
>


Ops, thanks


Yes, that's exactly what's happening. If you read the source you can see it
executes PS. Inside of a pure Linux build this naturally fails as that
utility is not available. On darwin it may be part of stdenv or impure.

On Tue, Sep 1, 2015, 09:52 Christian Theune <notifications@github.com>
wrote:

> I might also be misunderstanding - you're saying we're calling the
> "impure" version of ps otherwise?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NixOS/nixpkgs/commit/4121c5064aea6a9a755a767f3ac4288e997498a8#commitcomment-13004191>
> .
>


There's something disturbingly regular about these tests....Would it hurt to have one entrypoint test_compare and then:

map(['allergies','care_plan'....],self.python_outpuit_is_same_as_javascript)

Or would that make the tests more complicated?


reminder:
* re-add the doxygen comment and
* move it in (temporary) file inside voctree so it can be used elsewhere


closes #901


原文: First, I will create a single-noded app. Then we will talk about scalability and availability problems with this approach and introduce sharding as potential solution and transform our app into more scalable one.
訳案: まず、単一ノードのアプリを作った後、この方法の問題点であるスケーラビリティや可用性について解説する。解決策の一つとしてシャーディングの手法を紹介し、アプリをよりスケーラブルなものへ改良する。

「改良」を採用しました。


So I just went through a whole PEP8 clean up and opted to use the double brackets instead of the offset statements. Is there a preference for this? I personally find this less readable.


reminder:
* add back the `.txt` case, we still want to load feature from a simple and plain text file


thanks!
updated in 33c84febf81f490238979c46aa06a9fc5a285b1c


Clang is a new compiler compiler language.
http://clang.llvm.org/comparison.html

I successfully compiled the android kernel with it but the kernel does not boot at all. I think on android some board specific patches are necessary but i must investigate that. The kernel now can be compiled with clang and gcc with these patches.


I need to look at the PDF output, but a_{'a,a'} looks surprising.


So what's the general motivation of this diff? Was there an alternate code path if pustules was not present?


OK, I'll see if I can get a build env up on a Windows box.  VS 2013 with CUDA Toolkit 6.5?


ok great, it make sense to me !


why ?


It means {\y a}_{a,a'}.


I don't know. @samth, @takikawa?


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


Don't do this here. The message will be shown on every invocation of `api`.


I think this is not working.
In case basepath is a file we will end up with a "trace_txt" local which has that string appended to the file name.
Otherwise, in that case, we should strip the file name from the basepath...

Actually, I've tried to factorize even more the generation of the locals "trace_xxx" vars but the final code was not so easy to read. That's the reason why I've re-posted the patch version of the initial pull request.


ref name can be something like refs/tags/v1 so you need to check to replace above line by:
&& refNameFromEvent.startsWith("refs/heads/")) {


The point is it won't show up in the log.  Someone who opens the commit to
look at it will probably have little trouble figuring out the purpose of
the one-character change to the value of INDEX_UTILS_VERSION.

-- Dan.

On Tue, Sep 1, 2015 at 9:26 AM, Werner Dietl <notifications@github.com>
wrote:

> But your comment here is visible, so maybe simply comment what a better
> commit comment would have been?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/typetools/annotation-tools/commit/ce78fd6afcdb92edcb8af0e54eb9a8c64991f57b#commitcomment-12994202>
> .
>


Needs this as I missed one part
https://github.com/temasek/android_frameworks_base/commit/28b98b559cfd3a2e35e02dcd6352a61e42b6c2fa


I see... I've not considered these WA related use-cases.
Will update the patch to keep this generation.


Line endings are messed up. Only changed line 307.


The encoding to use here is should actually be given by the HTTP standard. I could not find the definitive answer quickly, it looks like one can either specify an explicit `charset` in `content-type` header or one can perhaps simply use (in this case were we can safely assume that there are no chars > 127 in the string) some 8-bit ASCII-compatible encoding like UTF-8, ISO Latin1, etc.


[optional] I would put "p.getCompareType().equals(CompareType.PLAIN)" first


going to use `#` moving forward but not going to go back and change `.` unless you think i should. let me know.


Hi!
I'll try to create a new entity and practice * @ORM lesson.
I must type all this * @ORM you included? Or did you order a comand? I ask you because I inserted the type of the field (ex: nomautor) and length.

Respect src/TestBundle/Resources/config/doctrine/Proyecto.orm.php, is it a buckup? I dont' understant so much this file.


better to explicitly list the strictly needed openMVG libraries, like `openMVG_features` and `openMVG_sfm`


:ok: looks like this is shaping up nicely.


I think this needs to be more explicit: it should say that this only applies to unit files that are included in the initrd.


That code causes 27 assertions to fail from the libs unit tests because it only verifies the version is greater than the **first** min version found in the array.

For example this test which is expected to fail will in fact pass because 1.8.0 is greater than the very minimum version found for doJo in `check-libs.js:90` which is 1.5.3.

Test:
    'dojo - v1.8.0': function (test) {
        checkPage('4.html', test, {
            passed: false,
            data: [
                {
                    lineNumber: 7,
                    version: "1.8.0",
                    minVersion: "1.8.5"
                }
            ]
        });

If this is the expected behavior then the tests will have to be updated to reflect this.


TeamCity Chorus :: chorus-precise64-master Continuous [Build 1.5.459.de275939eed03371b4e82440214b2abb96f02201](http://build.palaso.org/viewLog.html?buildId=62482&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:17:49


you are right @mbr0wn
I was following the guide on http://gnuradio.org/redmine/projects/gnuradio/wiki/Guided_Tutorial_GNU_Radio_in_Python .
The result wasn't what I was expecting, so I jumped into the code :)


fixed.


should be

```
[HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\VisualStudio\12.0\Generators\{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}\ProcessNodeFile]
```


Fixed e17e759


Remover esse comentário


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


Ugh, did it unintentionally. Do we really have convention to use typename keyword for type parameters (apart from this file)? I think I've already introduced quite a few "template<class..."


lol


Error is line 7, indentation error: unident does not match any outer indentation level


I guess that this statement can be deleted now because the needed M4 scripts should be automatically included from the macro directory by the autoconf software.


Other than my comment at line 845, I think the logic is fine but I am not sure the code belongs here, let me know when you have time, I can pass by your desk and explain my point.


hmmm ... might want to rethink this. code looks sloppy.


Apenas 1 teste?

- E se houver mais de uma anotação de parâmetro?
- E se não houver nenhuma?
- E se houver duas do mesmo tipo?
- E se houver muitas?
- E se houver a mesma anotação para uma parâmetro e para o método ao mesmo tempo?


wait, the spacing looks completely different on here...?


TeamCity pull requests :: netty [Build 161](https://garage.netty.io/teamcity/viewLog.html?buildId=1048&buildTypeId=pulls_netty&guest=1) outcome was **FAILURE**
Summary: Tests passed: 5593, ignored: 21; resource leak detected (new) Build time: 00:35:54


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


do you need a `return` here?


Oi @Gustavohqo , coloquei alguns comentários para melhorar o commit. Por favor, atualize o commit baseado nas minhas sugestões.


Now done as a second pass after validating all inputs.


I mean  **Client** now allows for a reconnect to server without restarting the app


either `throw` something (to be decide if/how to throw exceptions) or `return numDescriptors;` (for now let's go with that)


ok, moved to `check_app`


Hi Chris,
Thank you for your reply. I am on Version 6.4 (6E35b) of Xcode. After my initial post, I had a dig around some of the existing 'Issues', and found that a few other people had the same issue. In particular [issue 70](https://github.com/objcio/functional-programming-in-swift/issues/70), which showed the same error message and also pointed out that the code still runs correctly (despite the error). After checking my local playground, this transpired to be true for me also.

Thanks for your time, I'm looking forward to working through the rest of the book :)


http://alexvermeer.com/why-your-mindset-important/

https://www4.esu.edu/academics/enrichment_learning/documents/pdf/developing_growth_mindset.pdf

Steps to Developing a Growth Mindset:
Step 1: Learn, Learn, Learn
Step 2: Realize hard work is key
Step 3: Face Setbacks

Remember to
*Focus in effort, struggle, persistence despite setbacks
*choose difficult tasks
*Focus on Strategies
*Reflect on different strategies that work and don’t work
*focus on learning and improving
*Seek challenges
Work hard


101884090--NewVenting-Availability-Schedule (lgu1234) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1288 of 1288 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-70.02%25-green.svg)


The groupId doesn't matter as long as defaultUserId is a default user. I'm passing zero because it won't be used at all.  The default user is not a member of any site - which is awkward to the point that if we called user.getGroupId() it would throw a NPE.


It seemed more consistent with other code in this area to qualify access of a fields without '_' prefixes.


Okay I pushed another commit which will limit to 9999 for now so that the failsafe is working. I still would like to add more formatting options at some point but for now, I just want to make sure that counting loop isn't going haywire.


- Use `string.Equals`
- Is it always lowercase "neutral" or should it be case a insensitive compare?


Okay but can we use test data that has no more precision than this?  I don't *think* that would be a PITA but if you've tried I'll take your word for it.


[source](http://play.golang.org/p/sFin2N8WeN)
$ go test -bench=.
testing: warning: no tests to run
PASS
BenchmarkPrintf-16       1000000              1572 ns/op
BenchmarkHex-16  5000000               379 ns/op
ok      _/tmp/test      3.871s


Can you think of a better commit message for this? It would be good to include a bit on why this change is happening.


Thanks @leeola


Thanks!


:+1: will fix :)


@arranf , by default, the Release configuration in VS will give compiler warnings for any missing xml comments (the Debug configuration does not)...

![image](https://cloud.githubusercontent.com/assets/548180/9611202/33345230-5093-11e5-8c21-6064aeb75905.png)

Just change to Release and do a rebuild, and you should get all the warnings.


I've tried to consistently use "syntaxAndDeclarations" to refer to the "SyntaxAndDeclarationManager" instance (I felt the name was just too long, and "syntaxManager" doesn't really reflect everything being managed).


@stulees "than" should be "then" :wink:


Fixed following error: make: *** No rule to make target '/home/patrick/android/CM/cm12.1/out/target/product/degaswifi/obj/SHARED_LIBRARIES/libMarvellWireless_intermediates/export_includes', needed by '/home/patrick/android/CM/cm12.1/out/target/product/degaswifi/obj/SHARED_LIBRARIES/libbt-vendor_intermediates/import_includes'.  Stop.

Thanks Sub77