TaskCluster, You didn't give the task-graph scopes allowing it define tasks on the queue. ```[
    {
        "message": "You do not have sufficient scopes. This request requires you\nto have one of the following sets of scopes:\n[\n  [\n    \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n  ]\n]\n\nYou only have the scopes:\n[\n  \"assume:repo:github.com/mozilla/balrog:pull-request\",\n  \"queue:create-task:aws-provisioner-v1/github-worker\",\n  \"secrets:get:garbage/*\",\n  \"secrets:set:garbage/*\"\n]\n\nIn other words you are missing scopes from one of the options:\n * Option 0:\n    - \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n----\nerrorCode:  InsufficientScopes\nstatusCode: 403\nrequestInfo:\n  method:   defineTask\n  params:   {\"taskId\":\"CD0kOc_ZQWO1vrBfc9uQrA\"}\n  payload:  {\n  \"provisionerId\": \"aws-provisioner-v1\",\n  \"workerType\": \"github-worker\",\n  \"scopes\": [\n    \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n  ],\n  \"payload\": {\n    \"maxRunTime\": 3600,\n    \"image\": \"taskcluster/image_builder:0.1.3\",\n    \"features\": {\n      \"dind\": true,\n      \"taskclusterProxy\": true\n    },\n    \"command\": [\n      \"/bin/bash\",\n      \"-c\",\n      \"git clone $GITHUB_HEAD_REPO_URL && cd balrog && git checkout $GITHUB_HEAD_BRANCH && scripts/push-dockerimage.sh\"\n    ],\n    \"env\": {\n      \"GITHUB_EVENT\": \"push\",\n      \"GITHUB_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_BASE_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_HEAD_REPO_URL\": \"https://github.com/mozilla/balrog.git\",\n      \"GITHUB_HEAD_USER\": \"bhearsum\",\n      \"GITHUB_HEAD_SHA\": \"41b55b2fcc7304465e4bc18154c15dff3476b519\",\n      \"GITHUB_HEAD_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_HEAD_USER_EMAIL\": \"bhearsum@noreply.github.com\"\n    }\n  },\n  \"extra\": {\n    \"github\": {\n      \"env\": true,\n      \"events\": [\n        \"push\"\n      ]\n    }\n  },\n  \"metadata\": {\n    \"name\": \"Balrog Docker Image Creation\",\n    \"description\": \"Balrog Docker Image Creation\",\n    \"owner\": \"bhearsum@noreply.github.com\",\n    \"source\": \"https://github.com/mozilla/balrog.git\"\n  },\n  \"created\": \"2016-03-02T14:40:38.173Z\",\n  \"deadline\": \"2016-03-03T14:40:38.173Z\",\n  \"schedulerId\": \"task-graph-scheduler\",\n  \"taskGroupId\": \"UfxN15vDRGGGo7Pep31eUg\",\n  \"routes\": [],\n  \"priority\": \"normal\",\n  \"retries\": 5,\n  \"tags\": {},\n  \"expires\": \"2017-03-03T14:40:38.173Z\"\n}\n  time:     2016-03-02T14:40:38.295Z\ndetails:\n{\n  \"scopesets\": [\n    [\n      \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n    ]\n  ],\n  \"scopes\": [\n    \"assume:repo:github.com/mozilla/balrog:pull-request\",\n    \"queue:create-task:aws-provisioner-v1/github-worker\",\n    \"secrets:get:garbage/*\",\n    \"secrets:set:garbage/*\"\n  ]\n}",
        "error": {
            "code": "InsufficientScopes",
            "message": "You do not have sufficient scopes. This request requires you\nto have one of the following sets of scopes:\n[\n  [\n    \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n  ]\n]\n\nYou only have the scopes:\n[\n  \"assume:repo:github.com/mozilla/balrog:pull-request\",\n  \"queue:create-task:aws-provisioner-v1/github-worker\",\n  \"secrets:get:garbage/*\",\n  \"secrets:set:garbage/*\"\n]\n\nIn other words you are missing scopes from one of the options:\n * Option 0:\n    - \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n----\nerrorCode:  InsufficientScopes\nstatusCode: 403\nrequestInfo:\n  method:   defineTask\n  params:   {\"taskId\":\"CD0kOc_ZQWO1vrBfc9uQrA\"}\n  payload:  {\n  \"provisionerId\": \"aws-provisioner-v1\",\n  \"workerType\": \"github-worker\",\n  \"scopes\": [\n    \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n  ],\n  \"payload\": {\n    \"maxRunTime\": 3600,\n    \"image\": \"taskcluster/image_builder:0.1.3\",\n    \"features\": {\n      \"dind\": true,\n      \"taskclusterProxy\": true\n    },\n    \"command\": [\n      \"/bin/bash\",\n      \"-c\",\n      \"git clone $GITHUB_HEAD_REPO_URL && cd balrog && git checkout $GITHUB_HEAD_BRANCH && scripts/push-dockerimage.sh\"\n    ],\n    \"env\": {\n      \"GITHUB_EVENT\": \"push\",\n      \"GITHUB_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_BASE_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_HEAD_REPO_URL\": \"https://github.com/mozilla/balrog.git\",\n      \"GITHUB_HEAD_USER\": \"bhearsum\",\n      \"GITHUB_HEAD_SHA\": \"41b55b2fcc7304465e4bc18154c15dff3476b519\",\n      \"GITHUB_HEAD_BRANCH\": \"dockerflow-endpoints\",\n      \"GITHUB_HEAD_USER_EMAIL\": \"bhearsum@noreply.github.com\"\n    }\n  },\n  \"extra\": {\n    \"github\": {\n      \"env\": true,\n      \"events\": [\n        \"push\"\n      ]\n    }\n  },\n  \"metadata\": {\n    \"name\": \"Balrog Docker Image Creation\",\n    \"description\": \"Balrog Docker Image Creation\",\n    \"owner\": \"bhearsum@noreply.github.com\",\n    \"source\": \"https://github.com/mozilla/balrog.git\"\n  },\n  \"created\": \"2016-03-02T14:40:38.173Z\",\n  \"deadline\": \"2016-03-03T14:40:38.173Z\",\n  \"schedulerId\": \"task-graph-scheduler\",\n  \"taskGroupId\": \"UfxN15vDRGGGo7Pep31eUg\",\n  \"routes\": [],\n  \"priority\": \"normal\",\n  \"retries\": 5,\n  \"tags\": {},\n  \"expires\": \"2017-03-03T14:40:38.173Z\"\n}\n  time:     2016-03-02T14:40:38.295Z\ndetails:\n{\n  \"scopesets\": [\n    [\n      \"secrets:get:repo:github.com/mozilla/balrog:dockerhub\"\n    ]\n  ],\n  \"scopes\": [\n    \"assume:repo:github.com/mozilla/balrog:pull-request\",\n    \"queue:create-task:aws-provisioner-v1/github-worker\",\n    \"secrets:get:garbage/*\",\n    \"secrets:set:garbage/*\"\n  ]\n}",
            "requestInfo": {
                "method": "defineTask",
                "params": {
                    "taskId": "CD0kOc_ZQWO1vrBfc9uQrA"
                },
                "payload": {
                    "provisionerId": "aws-provisioner-v1",
                    "workerType": "github-worker",
                    "scopes": [
                        "secrets:get:repo:github.com/mozilla/balrog:dockerhub"
                    ],
                    "payload": {
                        "maxRunTime": 3600,
                        "image": "taskcluster/image_builder:0.1.3",
                        "features": {
                            "dind": true,
                            "taskclusterProxy": true
                        },
                        "command": [
                            "/bin/bash",
                            "-c",
                            "git clone $GITHUB_HEAD_REPO_URL && cd balrog && git checkout $GITHUB_HEAD_BRANCH && scripts/push-dockerimage.sh"
                        ],
                        "env": {
                            "GITHUB_EVENT": "push",
                            "GITHUB_BRANCH": "dockerflow-endpoints",
                            "GITHUB_BASE_BRANCH": "dockerflow-endpoints",
                            "GITHUB_HEAD_REPO_URL": "https://github.com/mozilla/balrog.git",
                            "GITHUB_HEAD_USER": "bhearsum",
                            "GITHUB_HEAD_SHA": "41b55b2fcc7304465e4bc18154c15dff3476b519",
                            "GITHUB_HEAD_BRANCH": "dockerflow-endpoints",
                            "GITHUB_HEAD_USER_EMAIL": "bhearsum@noreply.github.com"
                        }
                    },
                    "extra": {
                        "github": {
                            "env": true,
                            "events": [
                                "push"
                            ]
                        }
                    },
                    "metadata": {
                        "name": "Balrog Docker Image Creation",
                        "description": "Balrog Docker Image Creation",
                        "owner": "bhearsum@noreply.github.com",
                        "source": "https://github.com/mozilla/balrog.git"
                    },
                    "created": "2016-03-02T14:40:38.173Z",
                    "deadline": "2016-03-03T14:40:38.173Z",
                    "schedulerId": "task-graph-scheduler",
                    "taskGroupId": "UfxN15vDRGGGo7Pep31eUg",
                    "routes": [],
                    "priority": "normal",
                    "retries": 5,
                    "tags": {},
                    "expires": "2017-03-03T14:40:38.173Z"
                },
                "time": "2016-03-02T14:40:38.295Z"
            },
            "details": {
                "scopesets": [
                    [
                        "secrets:get:repo:github.com/mozilla/balrog:dockerhub"
                    ]
                ],
                "scopes": [
                    "assume:repo:github.com/mozilla/balrog:pull-request",
                    "queue:create-task:aws-provisioner-v1/github-worker",
                    "secrets:get:garbage/*",
                    "secrets:set:garbage/*"
                ]
            }
        }
    }
]```


read


@georgecrawford do we want to have `min` and/or `max` for these?


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1570 of 1571 tests passed, 0 test warnings)

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


This comment doesn't seem to add much


Ah crap. I knew I forgot to change something back after testing.


Shift variable is useful for the calculation of conformation probabilities.
calc_conductance.f90, Now calculate the conductance modified by the model reported in "The diffusion of electrolytes in a cation-exchange resin membrane" Mackie & Meares (1955)


Has this been changed? When I wrote this, the module was installed as `cookie-monster` and, without a hyphen, pip failed


@lsj4401 이거 홀수 input.length() 가 `홀수` 일때는 오답나온다 ㅋㅋㅋㅋ 홀수 구분하려다가 실패해서 꼼수 썼음 ㅋㅋ


I've committed the change: 4a5adbe25855c356f309db3e9662a9846840b54b.


I have seen that elsewhere in the code, it is meant to tell you the following code implements the ISecManager overloads


this is my commint


I would also recommend reading my guide about when/how to write low-level code https://github.com/tdhock/when-c


method that accepts... and returns...


Is there any chance this project will be continued? Because AWE doesn't seem to be compatible with latest WorldEdit versions.


Why you have moved the signal into the lock?


Is "description" supposed to be a human readable overview of isecmanager implementation? If so could we expand the description?


to find packages that use regular expressions you can look at the Depends/Imports fields of the stringr/stringi packages on CRAN https://cran.r-project.org/web/packages/stringr/ https://cran.r-project.org/web/packages/stringi/


It looks nice :D

I have only two points before we go ahead:

* Fix build (maybe marking specs as pending, maybe);
* Merge these two commits;

I was thinking about doing another pair programming to fix all, but I think there are a lot of change… and this approach can be harder to follow :/


Why create a second socket vs using the existing socket but sending a different message? i.e. `timestamp`


so socket.on(`timeSentToDB`)


Auto formatted  all files to follow more closely to coding style conventions (curly braces, indentation (spacing), etc.). Also, for code readability.


Is there a possibilty to get access to? Getting the GA Tracking inside...


Hi @codingsharon, can you tell me what this is?


Is there an enumeration of these codes? if not, does it make sense to create one, or at least centralize a group of #defines?


nice boat!!


I don't think that you can even do this actually

Just use the same `socket` but a different message. So `socket.in('anothermessage', callback)


OK.


Why not just use `if(it->getNodeID() == curNode.getNodeID())` instead of adding this variable `left`?


...ha....


`it "should not be success?"` :P


don't need a second socket. use your existing one and just send and listen for different messages


@aaronkoz74 DWMR (Done with my review). Good here. Press ahead!


i would not recommend writing tests for internal functions and C++ functions.

The most important tests are input/output pairs for public R functions.


Not now. I still need to document code.

Nexus Razvan

On 03/02/2016 04:34 PM, Alexandru Onea wrote:
>
> Please create pull requests for merging
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thetopor/helix-drone/commit/895ba0388a95c2ff7ff119da994d9c38091a6c67#commitcomment-16446841>.
>

This message contains confidential information and is intended only for the individual named. If you are not the name addressee you should not disseminate, distribute or copy this e-mail. You cannot use or forward any attachments in theemail. Please notify the sender immediately by e-mail if you have received this e-mail by mistake and delete this e-mail from your system. Topor 2016. www.topor.io


final.


Hi @casielane! This is the file where the `MailchimpService` from the diagram will live. (No content just yet.)

You and Xin Tian will use this service in your controller to subscribe to the mailing list.

We decided to name it `MailchimpService` rather than `SubscriberService`, since it follows the convention Michael set forth already.


qwe


`Track` doesn't need to be called here; the `Stream` is entirely local to `IsFormatSupported`, and the callback function is never passed to C.


I prefer `not_to`, mate :P


> It can only work in unexpected ways for people who chose to ignore the warning which is not really our problem.

Fair enough i guess.. hmm. The only thing that bothers me is that it could be really hard to figure out why the expected data isn't served..


closing program with System.exit(0) does not completely fix window events and options in jmenu file tab, exit on close option.
Files associated:
preferences, exitDialog, window class of jframe (window events), app.java, appFrame.java this is a rooted problem that includes gui.


Appears to be missing Install notes


j


Hi,

My code already has an enum for different protocols ;)

IRQ driven RX is not mandatory for V202 protocol, rcReadRawData is called often enough to not desynch the channel hopping, but it will be required if we want to do rx+telemetry over nrf24, so we don't send telemetry packet while the TX is transmitting.

I know Hydra is interested in the nrf24 driver and protocols, how would you like to merge ?


d


Rewrite handlers of pressing as one functions with using [switch/if else] with actions in branches depending on transferred View parameter.


Ah I had checked it the night before and couldn't compile and only checked why the morning I wrote the comment. Nevermind!


this is a test


No bad :P


encore une petite accolade.. :-)


@mikecroucher they are already at the bottom, I just followed my own instructions ;)


Tu voulais vraiment commiter cette modif ?


`nextid` might be a more obvious name than `cid`?


Was this patch fixing a bug? Notice that inline_version is set in the body of all these constructors; it shouldn't be duplicated.


I thought your new SecurityManager framework provided 2 ptrees to isecmanager factory methods.
It might make sense to reflect that in the isecmanager interface (even if the implementations don't always make use of both trees).


I got a idea: we can use the first one to add changes at `spec/spec_helper.rb` and mark some specs as pending :P

So we will make changes as we can :)


heh, \_(ツ)_/¯


@atakacs5280 DWMR (Done with my review). Good here. Press ahead!


Its part of this message
     WWW-Authenticate: Basic realm="ESP (Authentication: HTPASSWD)"

I will add some additional wording


Encapsulate the three try-catches into methods to make it more readable.


Note you can add this kind of editor-specific pattern to `$HOME/.config/git/ignore` and avoid having to put it in all your git repos :)


thanks


@maxGimeno Those three data members no longer seem to be used anywhere, in today's `master`.


All these fields should be signed as private


I think the UNIMPLEMENTED macro throws... the return statement might be unreachable.


This starts work on #15


TaskCluster, Request payload must follow the schema: http://schemas.taskcluster.net/scheduler/v1/task-graph.json# ```{
    "validation": {
        "tasks": {
            "schema": {
                "1": {
                    "schema": {
                        "task": {
                            "schema": {
                                "scopes": {
                                    "type": "array"
                                }
                            }
                        }
                    }
                }
            }
        }
    }
}```


There is not, although a separate Jira should be added to create one


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#A8RW6s6VRGCnE8I6Psy3Ag


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1571 of 1571 tests passed, 0 test warnings)

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


use using instead


So the `close()` called on the page instance is propagated to the `close()` call on the resultSet (in case of SQL backend), right?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1571 of 1571 tests passed, 0 test warnings)

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


It does for instantiating pluggable security managers. This version has not been converted to a plugin yet


that is fine if you want to keep the part about trying a non-Rcpp implementation. if you do that it would be nice to create a blog post that highlights the differences between the two implementations, in terms of lines of code and execution time.

Very good that you added more details about the functions that have already been implemented.  Can you please also discuss the binary operators that you have implemented? I would further recommend adding details about the functions you would like to implement during GSOC. Which features from the stringr/stringi/namedCapture/RE2 libraries do you plan to implement?

personally I think the most important features to implement will be `re2_match` and `re2_match_all`, the functions that input (1) a character vector of subject strings and (2) a single regular expression, and output a character matrix (rows = matches, columns = capture groups).


You are correct--noroot, caps.drop.all, and seccomp all broke the chromium sandbox (in Vivaldi)--at least they did as of testing a few hours ago.


@casey-collab


REMOTE_DIRECTORY might me a too generic name. DEPLOYMENT_REMOTE_DIRECTORY might be more appropriate.
Also some documentation for it would be nice.


Correct, compiler forces it


useless return


I didn't see this used in your other changes.


appframe-Organized code by type & grouping of jmenu and components..


Create links to variables (soundPool, buttonPressSoundId, soundVolume etc) in order to simplify your code and do not call often getters. Only in this fragment.


New sentence/paragraph, start with uppercase (`Besides`)


I see, these are the dummy impls in case the inventory doesn't support big transactions..


is there also meant to be some mtext at the end here for a close bracket?


Picking the first available gateway, which one do we get? Is it the default route if it exists? I think that the sort order, which I don't know on top of my head, is important here. Please determine it and add a comment to the code at this point, thanks!


Yep, makes sense to me! Thanks a lot!


aSDF


@CharlesMosesMcMahon DWMR. Good here. Press ahead


Closed.

Didn't realize these comments were here. Some of these issues were addressed, but I'll check them off as I go along.


Ideally yes, but not in the near feature.


check values is not null; it can be despite being a vararg.


check matrix is not null.


Eu já tinha tirado. Faltou o `push`.


Ok gotcha. I guess this could be part of a future feature add if we combine both models into a single model. Will remove on the next commit.


I tried a similar idea in the past...but with no luck.

Hopefully you will have better luck than me...


Closed.

The DOM issues caused by the Frame changes should have been fixed when I reverted them. This includes all instances of "parent.left_nav"


I'm controlling the version. Would you please remove it.


Well, it is a Deque :)


Why not use `for(const &auto read : reference)`? You don't need the counter for anything, and you only use the `read` to construct a `KmerIt`, which takes a constant reference.
Why is the variable called `read`, it is not a read I think?


..a transactioN frame..


la compilation du project rpcs3-tests marche  plus, j'ai une erreur de links
https://gist.github.com/Zangetsu38/bac9168225c265be0337


this installs ts 1.6.2 which doesn't have 'parseConfigFileTextToJson' as required in https://github.com/SitePen/dts-generator/commit/a7a2482f0d2927879fb18a9fc597ef7b276aa3a6


What were you trying to do? What failed?


Why are you catching any exception? I think you must allow the test to fail in that case. You can add IOException to the test method declaration.


@CharlesMosesMcMahon DWMR. Good good.


Maybe I don't have the whole picture here, but why "StatusCommand" returns true?


extra blank line.


Strange, looks fine on my text editor.


I'm not quite sure what you're talking about...


self description on readme


Not wroking


Ah, of course! I had simply not noticed that it should be negated. Right, I'll get this merged in to master ASAP.


Closed.


how work this plugin?


Sure. Removed.

On Wed, Mar 2, 2016 at 8:35 PM, Ali Najafizadeh <notifications@github.com>
wrote:

> I'm controlling the version. Would you please remove it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BilalBudhani/react-native-webview-bridge/commit/add42e9cacf8f6854bb428db56a51280b30574b7#commitcomment-16447699>
> .
>

--
Best Regards,
Bilal Budhani
+91-98920-68490
bilalbudhani.me | LinkedIn <http://in.linkedin.com/in/bilalbudhani>


Could we use the PEP convention for defining module names (e.g., pvVerifier -> pvverifier): https://www.python.org/dev/peps/pep-0008/#package-and-module-names


Indeed -- I checked in a patch yesterday that uses message. The
error reported "text is read-only" through the handler was  misleading.
>>>>> "Michael" == Michael Heerdegen <notifications@github.com> writes:
    Michael> Hi, I followed the according thread in Helm.
    Michael>
    Michael> This commit looks like you should better use
    Michael>
    Michael> ``` (message "%s" ...) ```
    Michael>
    Michael> to avoid problems when the buffer contents contain
    Michael> format "%-sequences"? Sorry if I misunderstand what
    Michael> this function does, but this looks like a common
    Michael> error.
    Michael>
    Michael> --- Reply to this email directly or view it on
    Michael> GitHub:
    Michael> https://github.com/tvraman/emacspeak/commit/fab12eb7aad20f7a9a0134189149ed56ab08beed#commitcomment-16437387


Echipa :
1.Prunaru Cosmin-Nicolae
2.Mares Cristian-Ionut
3.Baronescu Andrei-Robert
4.Patru Andreea-Camelia


Balloon* :P


:o


We can get rid of this `else` altogether by `return`ing on the `if`.

The "right path" for this function is return the service, if the condition is not met then return the invalidApi.


need include guards _PG_CORE_UTILS_MAC_TABLE_H


In the end added an option to enable or disable this, thanks :)


lollo ci picchiamo?


/get LOLLO MUORI


Good question. Since we don't have a 1.2 release tagged, and the identifierType being too long for the field is an issue in the current version, I figured we wanted to fix the current releases sql. Do you think we just need to release a new version? Perhaps 1.1.1?


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


you should use pg_mac_table_full_mac instead of pg_full_mac (like other names in the file)


Please ignore this file, committed to Git by mistake as part of my test of what cachemeans does.


Resolves https://github.com/ArmorDarks/ekzo.sass/issues/41


Ah you're right :sob:


Poderia sim, mas qual seria a vantagem?


initial readme commit


This broke our reporting in Jenkins. Are we sure the official format is error?


ASSIGNMENT_10524825

[ID NUMBER-10524825.txt](https://github.com/ALI-SAMPSON/hello-world/files/154824/ID.NUMBER-10524825.txt)


@Shauren could you take a look at this commit and f4e1ee72f970b2 please?

I encountered the issue that if I shut down my MySQL server the worldserver doesn't terminate (not directly changed by this commit but accessing the deleted `MySQLConnection` in previous commits finally lead to abortion which is now impossible since the object stays valid until it's deleted in the WorkerPool).

Would it be useful inform the main loop to stop immediately on connection failure to prevent further data corruption?


//cc @rwjblue


I believe the status command was implemented in v1 (see `statusAction` of `DefaultController`) so that a user may send a `GET /status` request to check if Eratosthenes is online. If it is, then the response would be `true`.

Similarly we thought it might be good to have a status API in v2. Do you think this might be redundant?


Closed.


Yeah, bit of a shame about this. It certainly doesn't read as nicely as you would hope.


Shows your current git project and branch

![screen shot 2016-03-02 at 15 17 49](https://cloud.githubusercontent.com/assets/2115702/13464626/f68679d0-e089-11e5-9c73-340e53765a78.png)


Not sure if Transifex still auto-pulls, and if it does, someone should probably go there and make it pull from the proper URL now. :)


Change label from '* grant application' to '* grant application page'.


Looking at [this](https://svn.jenkins-ci.org/trunk/hudson/dtkit/dtkit-format/dtkit-junit-model/src/main/resources/com/thalesgroup/dtkit/junit/model/xsd/junit-4.xsd) and the formatter from above (failure and error are valid). I know that JUnit is looking for `failure` not `error`. Do we make this a config?


Looks great to me :smile:. What about your feelings @ryukinix? I don't know why there's a copyright on the top of the file and neither why your cpplinter requires that lol.


is there meant to be mtext for a close bracket at the end here?


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


I have activated the GitHub account, but I am not sure if I have done everything correctly


And now it works, int is only 16 bits width long. :disappointed:


Yes - but this can only happen if people ignored the warning. There is only so much we can do...There also is no actual way of fixing it for all people - some might have already adapted the new syntax and changing it would have unexpected results for them...


let's keep this variable


@VoyagerOne Thank you, I will take a look at your attempts. Did some of them pass STC?
Something which is strange with current implementation is that we apparently can afford to throw away order information when depth is > 17.
To me it appears that when it comes to a PV-change near the root (= the zero-window search after the TT-Move fails ) then in order to analyze the next potential best move first we rely on a history which is based on a much higher depth (which may be out of context).


inst.get_attribute('sourceDestCheck') fails on instances that are not in a VPC.


@JonathanRosado - after making sure it is working please create a PR for dev branch


@philipobenito this restriction on `nikic/fast-route` is disallowing v0.7.0 right now. Should this be switched to something like `~0.6`, or is it intentional?


2nd March


Ah, my bad. Yes, using a method is better than modifying the variable directly. Let's chat through that.


Oh yes, now I remembered what it is :sleepy:


Fixed all related tests, thank you for your assistance :)!
(though there are lost failing with difference of `.class`, which I presume is unrelated)

Will push a new patch soon :)


Erf!
broken!


Check if there is an issue to do that automatically. If not, make one and point to this commit as an example of when it would have been useful.


This does not even close to reallity:
__2ABJjxzNo:
```
f36: Duration: 00:04:24.94, start: 0.000000, bitrate: 105 kb/s
f43: Duration: 00:03:44.52, start: 0.000000, bitrate: 159 kb/s
```
BaW_jenozKc:
```
f36: Duration: 00:00:09.85, start: 0.000000, bitrate: 109 kb/s
f43: Duration: 00:00:09.93, start: 0.000000, bitrate: 149 kb/s
```
07FYdnEawAQ
```
f36: Duration: 00:06:58.98, start: 0.000000, bitrate: 226 kb/s
f43: Duration: 00:06:59.06, start: 0.000000, bitrate: 601 kb/s
```


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


http://stackoverflow.com/questions/2671496/java-when-to-use-static-methods

Por questões de OO.

Logar não é um comportamento de um objeto, já que não depende do seu estado interno e sim de características externas. A instância de RequestExceptionLogger não vai ser aproveitada em lugar nenhum, nem passada para outro objeto, então por quê guardar estado?

Faz sentido logar essa mensagem, mesmo que o seu Logger não tenha sido criado.


@nibra what is the plan to deal with changes in pages in between main versions?

I mean, what will be the namespace for a Page change in between Joomla 3.5 and Joomla 3.5.1


Thanks! this is fixed now


Let's use

```
const { source, ...rest } = this.props;

ref={RCT_WEBVIEWBRIDGE_REF}
key="webViewKey"
source={resolveAssetSource(source)}
{...rest}
```


那就乖乖用comandlone


The AH app has basically been my test bed to try out new features in the SDK when I implement them, and to ensure changes haven't broken anything. To move forward without keeping the AH app in sync, or at least in sync on a development branch, as proposed [here](https://github.com/cloudmine/AsthmaHealth/pull/26), we'll have to either get integration tests working, which will have to do eventually, or spin up another test/example app. The latter could be an opportunity to test Swift support in the SDK, which I think will be important before at Beta-quality release. It need not be something polished or customer facing, just a proof-of-concept where I can test out the SDK. What do you think?


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


I think this should fix #5466, am I right?


zdecyduj się czy boolean czy byte :D


Changing from a counted to an uncounted loop has negative performance implications and also adds no value, IMO.


Apparently, we were supposed to convert the body weight into grams. *facepalm*


@snopoke fyi on this change too. makes the `case_json` a dict instead of a json string


close #4


I inlined both this and `castExtraParametersFromLiterals` methods.


Realm導入してるね
素晴らしい
[![LGTM](http://lgtm.in/p/eLQVBNNSp)](http://lgtm.in/i/eLQVBNNSp)


thats right!!!!!


inlined instead


done


I've committed your unit test and a fix for the error.


This one's for you, @Beramos.


Yes, it's pretty easy and clean to remove. The only evidence that remains is the use of the static constructor, which is a fine design in its own right.

Let's say you have an existing code base though and you discover problems with the equals implementation, in order to make auto handle this you have to make changes outside the class wherever it's instantiated (not a huge deal, but a bit more refactoring) whereas with lombok you just add the annotation and it'll work without other code changes. Pros and cons either way, so It's a matter of preference, I lean towards Lombok and it sounds like you lean towards auto. Both are good :)


@alinz Sure. I will make those changes and I believe we should be discussing the further changes at https://github.com/alinz/react-native-webview-bridge/pull/68


Thanks for this @parkouss.  Merging now.


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


cool, in mine I just excluded the field completely


build in function(BIF)で解くと一瞬なので、pattern matchingを使った方が面白いかも、と思います。


`list`ではなく、`entities`のように複数形表現の方が良いそうです。引用元思い出したら共有します。


This test fails (I thinks it's related to the ucfg enhancement). `3` used to be converted to a string in accordance to the [docs](https://www.elastic.co/guide/en/beats/filebeat/current/configuration-filebeat-options.html#configuration-fields) for fields.

This conversion is/was handled by an `UnmarshalYAML()` callback to do custom unmarshalling of `MapStr`. The method is here: https://github.com/elastic/beats/blob/master/libbeat/common/mapstr.go#L99


Definitely agree that it is more clear when brackets wrap the action that follows the conditional expression.  Updated in commit [7acd282](https://github.com/jblakeman/memory-grid/commit/7acd2828e1f4a8784845a462a9fc1a7e0db15d39).


Is there any reason not to have the ng-click on the button itself?


確かに...！


remove function toDateTime


I thought GSON was available in CraftBukkit for those earlier versions?


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


@jreading - Can you open an issue?

@johanneswuerbach - I'm happy to revert the error/failure change while we figure out the right path forward (I'm also not sure how the config system works internally so would need guidance on the changes needed if we went that route).


Should probably remove this (it is never removed in any following commits).


player leave, because chunk no load norm...


che cazzo hai fatto? io me ne tiro fuori!!!!111!!!


True, I forgot to get rid of it.


This doesn't work. The JSON value is "browser_download_url".


it's the same for the information extracted from mpd:
l49mgGSsb-0
```
264          mp4        2560x1440  DASH video 10454k , avc1.640032, 24fps, video only, 63.18MiB
247          webm       1280x720   DASH video 1727k , vp9, 24fps, video only, 11.04MiB
```
```
f264: Duration: 00:01:37.56, start: 0.041711, bitrate: 5432 kb/s
f247: Duration: 00:01:37.51, start: 0.000000, bitrate: 949 kb/s
```
the information that i've add was extracted from watch_as3.swf.


I think we may need to put brackets around the `arg` in the last case to avoid operator precedence problems.

E.g. consider these RPN ops:
```
1
2
ADD
3
4
ADD
MULL
```
Ignoring brackets that would splice into something like: `1 + 2 * 3 + 4` but the original RPN ops should really translate to something like `(1 + 2) * (3 + 4)`


done


Actually i'm not sure it looks better, maybe underline or underline dotted? :flushed:


I wouldn't see why not is this case. The hit box should be the same.


`null`や`empty`判定もpattern matchingでかけると思います。

```scala
            acc match {
                case x :: xs if (elem == x(0)) => (elem::x) :: acc.drop(1)
                case _ => List(elem) :: acc
                Nil => acc
            }
        }
```

とかー。


ちょっと上の例は他の問題で使ったやつなのでそのまま使えませんがご参考にしていただければと。

また明日〜。


s/<=/==/


Working on setting things up for live shop testing...so far:
ALTER TABLE  `products` ADD  `products_gtin` CHAR( 14 ) NULL DEFAULT NULL ;
OK or??


Good point! Fixed in 27a610ad5d79824c4c83273008e6b06031ee7387.


@drewish @bfontaine
Since CGAL-4.7, [the support for Qt4 has been dropped in favor of Qt5][47]. Was that taken into account in the formula?

Not also that [CGAL-4.8 will drop the support for Qt3][48] completely.

[47]: http://www.cgal.org/2015/10/20/cgal-47/
[48]: http://www.cgal.org/2016/02/10/cgal48-beta1/


Hi,

I got an issue with this commit:
I tried to apply this fix on puppet 3.7.5 on Fedora 23 to add support to dnf, because my masters are stucked with puppet 3.8.4 on CentOS 7...

But using a simple package resource like:
```
package { "NetworkManager-config-server":
       ensure  => latest,
}
```

Gave me this debug output:
```
Debug: Executing '/usr/bin/dnf -d 0 -e 1 -y list NetworkManager-config-server'
Debug: Package[NetworkManager-config-server](provider=yum): Ensuring => latest
Debug: Executing '/usr/bin/dnf -d 0 -e 1 -y list NetworkManager-config-server install NetworkManager-config-server'
```
Which does not failed, but does not install the package either...

So, not being used to Ruby at all, I was wondering if the concat was not appending :install operation to the previous :list one...

I've got it working by using simple array addition rather than concat:
```
<       args = $yumopts + [install_options, :list, wanted].compact
<       yum *args
---
>       yum *['-d', '0', '-e', '0', '-y', install_options, :list, wanted].compact
===
<     args = $yumopts + [install_options, operation, wanted].compact
---
>     args = ["-d", "0", "-e", "0", "-y", install_options, operation, wanted].compact
```

Now my debug looks like this:
```
Debug: Executing '/usr/bin/dnf -d 0 -e 1 -y list NetworkManager-config-server'
Debug: Package[NetworkManager-config-server](provider=yum): Ensuring => latest
Debug: Executing '/usr/bin/dnf -d 0 -e 1 -y install NetworkManager-config-server'
```
And the package is installed.

Am I missing/misunderstanding something?


If I didn't do so yet, I am adding JUnit to the project to test classes like Updater. This way we are sure it will work before we push and we don't need to run a server (in most cases).


:+1: It seems like logically it would make more sense for the button to have the click handler


By the way, the function was returning a `edge_capacity_t` instead of a `edge_weight_t`. I fixed that using `Weight`.


I read somewhere that because conditional and loop blocks don't have their own scope in JavaScript (an unnecessarily strict warning in JSLint) as they do in other languages, it might be better to declare variables above their loops and conditional blocks, to avoid confusion.  I guess if you're coding in JavaScript, you should already be familiar with the scoping rules, but I guess it might help JavaScript newbies who are used to variables having local scope inside every block, not just functions.


Problem: underline = link.

I also tried putting caps and UGent yellow, both had too much emphasis. Since the text is already bold italic and Arial has no further weights, this was the best subtle thing I could do.

Another alternative is “open” with quotes.


@sarahn excellent!  Thank you for the update.
It's working perfectly now!!

I did have the save at the end of my commands file, but didn't know I needed an $args ie. /config/config.boot.

For reference of others.

This is a working example running on
```
$ ansible --version
ansible 1.9.4
```

Updates timezone and saves to /config/config.boot.test
```
set system time-zone America/New_York
save /config/config.boot.test
```

Updates timezone and saves to default /config/config.boot
```
set system time-zone America/New_York
save
```

My sample vyos.yml file looks like.  Runs vyos module and adds configuration from setcommands.txt.  Verifies changes where saved to /config/config.boot and displays output of 'cat /config/config.boot | grep -A1 time-zone '
```
---
- hosts: all
  tasks:
    - vyos: "{{ lookup('file', item)}}"
      with_items:
        - /etc/ansible/playbooks/my_modules/setcommands.txt
    - name: verify save configuration
      shell: 'cat /config/config.boot | grep -A1 time-zone '
      register: out

    - debug: var=out.stdout_lines
```

My setcommands.txt - this file has one set command per line
```
set system time-zone America/New_York
save
```


I like Roger Waters!


そうなんや...確かに、テストケース先に書いてメソッド自動生成したら、Intsとかsymbolsになってるね！


牛B啊，宏用的真不错。


Done. just minor stuff. Once test pass and you address the changes you can rebase and merge.


Потерян ньюлайн


Лишняя пустая строка


I did not. Now markDirtyAndUnlock() is invoked before emit(), always. The
diff is misleading ;)

On 03/02/2016 03:44 PM, Daniel Schürmann wrote:
>
> Why you have moved the signal into the lock?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/uklotzde/mixxx/commit/c58c8d2e8a43496996eca8ebe9663d6822acbd92#commitcomment-16447118>.
>


Сообщения к коммитам можно делать многострочными. Так будет аккуратнее выглядеть.


This is a huge class, and it seems like much of the code was copied from somewhere else. If this is true, the something should be refactored somewhere.


In the some cases, RADIO_CONTROL_LED is defined as "none" or as nothing. If I compile the ap side, my radio_control_led is defined as "none" instead of nothing...


TEST_RESULT() macro was in a non-utest test result counting fw defined sal/test/ctest_env.h. This is not used anymore. Now utest manages the test results.


OK please let me remove that change then.


Случайно отписал комменты к уже перепушенному коммиту:
https://github.com/timyrka67/raox/commit/5c5d1e7a2a93b104ca5b7ee4a10808d7ad6e3acb


You can see the drop shadow over the bubble head in compact, maybe have it as a further parameter into the mixin?


Класс переименован, а почему нет изменений в `RuntimeComponents`?


Awesome! Thanks!


Clarification"  Is it 40,000 characters or 40,000 bytes?


Ah I see. Would we just have the shadow bottom/right instead of top/left in compact? @bjedrzejewski


Thanks for the catch!  Fixed in commit [29876fe](https://github.com/jblakeman/memory-grid/commit/29876fe105f324035cfb7fcea2400d1f69504e47).


We should look into the standard [logging](https://docs.python.org/2/howto/logging.html) class to do this. I haven't used it but it is used to sort messages/warning/errors to stdout or file, depending on input.

I agree with you both, the if statement make the code hard to read. But sometimes it would be nice to get more information about the run. I use it self.verbose = True often.


.ts files is the source translation file in xml format.
.qm files is the compiled .ts files.
To compile .ts files you can use "Linguist" app from Qt installation or "lrelease" console command.
To translate program you should place appropriate .qm files to "translations" folder of "grblControl" app.


Closed.

Change made by previous developer.


This is just an overflow check (other methods use `overflows` which takes unscaled values as `long` or `BigInteger`).
In `doubleToXXXDecimal` we operate on `BigDecimal` values. That is the reason why instead using `overflows` we do simple test on `result.precision() > precision`.

It will not fail generally. Only if we encounter value which cast result precision after cast.

E.g `CAST(1.23  as decimal(2,1))` will work fine and return `1.2`

but `CAST(9.66 as decimal(2,1))` will fail as result `10.6` would not fit in result type.

I will extract `overflows` methods for BigDecimal for clarity.


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


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


mpd is unique for each video, these values are hardcodes for each video.


No reason not to merge imo


We take a different approach now.


space before the `#`


Issue #22 covers this.


パターンマッチ勉強します！


Can you move these into 1 line ? No reason to indent it like this


I'd rather this is done with two arguments:

    -top N
    -varExp V

and actual number of component subtracted is the minimum number based in either criteria. So If N = 30 but the first
20 components explain V variance, then is 20. This should be out-put in the logs as INFO.

The default value for these two parameters would be Integer.MAX_INTEGER and 1.0 respectively (so inactive).


Could we not keep immediately-called-functions in the AST, then optimize them in the backend (or use an helper for that ?). I would prefer not to remove the expressions from the AST itself


very minor issues, but everything else looks good to me


but I noticed that the ok handler did not execute when non-matching etag was provided. ie, got status 200, but content was empty.

Did not think too much about it and thought my tests were wrong, but on second thought I believe content should have been there.


Are you sure about this param? `Title::newFromText` signature is:
```
public static function newFromText( $text, $defaultNamespace = NS_MAIN )
```


in any case you can choose the name for 'varExpl'


Verify


Could you add a space between the cast and the variable name?


LGTM


Space between cast and variable name


please accept this a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s a u s s i e p a s t e b o y s


Yup, functionality I added in the `createNewElement` method.  Motivation was from having to repeat setting ids or class names every time I created an element.


Hey!
Why do you do check on (i % 6) condition? I think this case will be caught with (i % 3) condition.


Shouldn't this be for iphone, not for mac,. pixbuf_pool is only referenced in the iphone build..


Switched, still triggers correctly too.


Changed returned debug data from array to object and added response sample to log object.


Within SMP I believe we discovered this was too early to seek in some browsers and should actually defer until canplay or loadedmetadata... further fiddling required.


That's very odd, I'll take a look at that, a 200 except in the case of a HEAD request should return the relevant content.


Use int instead of auto since you know the type.  It's fewer characters and avoids bugs due to not knowing the available range.


auto -> int


Ideally yes. But this is written inside a class and we want to clean-up the object on test completion. Calling TEST_ASSERT_EQUAL_MESSAGE will abort the flow and leak the object. Though a tear down handler can be registered to cleanup the object. But historically the test was written around gracefully terminating the test.

A non class (callbacks ) based implementation is present in socket-test repo here https://github.com/ARMmbed/socket-test/blob/devel-transport/test/tcp_client_echo_v0/main.cpp


Will this method be called also on accessing `Special:InfoboxBuilder` without template name? If yes then we probably should test what will happen then.


@vlj I just noticed that DX12 is also affected on AMD in almost the same way, by PR #1510


@frasmage, did you want to remove these from the French version as well?


This may fail on if `fmt` has unexpected format. `spec[1]` may not exist and may not be `\d+x\d+`.


see https://github.com/basho/riak_core/commit/1b5081b45686676c7daf22b26628a0ac78b015d6


done :+1:


`_formats` may not contain `spec[0]` key.


@janko33bd MAX_MONEY is also used in a few other places


It should be called before doing resolve hence moved up.


I'd say so, we can leave the issue open for a longer-term fix


There is no reason to make a variable for 2 * NumDims.  Just use 2 * NumDims everywhere; it's clearer.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18032](https://teamcity.mesosphere.io/viewLog.html?buildId=151360&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 2 (new) Build time: 00:02:29


done


It typechecks but has not yet been refactored to be more boxen style


space before `#` ?


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


Sweet!


Here I would say what is the reason to choose the number of components to subtract... if it due to variance explain or to number of components requested.


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


Let me check with @20hertz, he is the one making the changes.


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


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


ouch, that's a sneaky one


Please don't change all the assignments to auto.  This one even changes the type from size_t to int.  That happens to be fine, but I'm guessing you changed the type by accident.  Please change it back.


Changed example variables to be more generic.


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


used chars.sort to get anagram


test


Good catch! Out of curiosity, have you tried the exploit? Does it return `true` without any other kind of failure?

I'm not sure if there is any "best practice" we want to show about checking this stuff. @meriac What do you think? Should one check that before a secure gateway? Something like:
```C
...
check_args(args_from_the_caller);
secure_gateway(..., args_from_the_caller);
...
```

Should uVisor provide an API to check if an object is fully contained between the active context?


a jest getBoolean()?


We want to when a sent packet is received back. Hence it should be initialized with no. of sent  bytes every time we send data. Alternatively as Brendan said it should have been initialized to 0 in constructor. But it wasn't.


Issue here:
https://github.com/testem/testem/issues/815


Indeed they need to be removed from french side too.  I will do so right now.


It actually was happening when I did not provide a direct location. If I have the recommended setup of aws creds file in ~/.aws/ but dont pass anything to rake terraform it seems that it cannot expand ~ into a viable path '/Users/USERNAMEHERE/.aws' with out using File.expand_path


Why not directly free(ma->macs[i * 64 + it]) ?


@raphaelpaiva A ideia é que existam outras classes em momento oportuno que possam estender RequestExceptionLogger e, por questões de OO, métodos estáticos não podem ser sobrescritos.


Use of string literal for Objective-C selectors is deprecated. It should be like this `#selector(MainViewController.dismissKeyboard(_:))` Check this, please: https://github.com/apple/swift-evolution/blob/master/proposals/0022-objc-selectors.md#proposed-solution


![screen shot 2016-03-02 at 9 55 23 am](https://cloud.githubusercontent.com/assets/3943219/13465841/f44ef638-e05c-11e5-8f9d-2aa1dc3e8c1e.png)


Capitalize "don't"


I would love it if we had that. We could make things so much more precise.


Indent break to match output->tensor...


...and put a dot at the end.


確かにfor文はeachに変換されていることが分かる。(「Rubyのしくみ」の第4章参照)


Consider using `->>`?


It looks nice! :+1:


In any case I would suggest fileing this (and the following commit) as PR for further discussions and removing it from master for now until this happened to make it into the next release.


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

My knowledge of ARM is very, very limited. Mostly what I know is the scraps I've read here and there (Wikipedia has a pretty good writeup), plus the efforts to get the thing building on a Raspberry Pi.

So, I don't know if there are any ARMv7 CPU's that don't support Thumb or if that's a requirement for the architecture. I also don't know what the pros and cons are of using one over the other.

I also don't know what other architectures might be added in the future, or what sort of variations we might see within those. A type parameter should cover a lot of ground, though.

Worst case scenario, it's an integer parameter that never gets used. I'll take that over another ABI revision later on.

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

For the x86-64 and pure C stubs, the configure script will use the --enable/disable-tls flag to set the gldispatch_use_tls variable. ARMv7 doesn't have TLS stubs yet, and the x86 ones are still broken, so in those cases it'll set gldispatch_use_tls to "no" unconditionally.


Also, `doall` shouldn't be necessary, as `every?` will force the sequence `take` returns.


`Initialize the challenge.`


Remove "Given `input`".


@juice500ml 광희야 이거 master에 merge해놓은거냐?


If `Diagonal` is in backticks indicating code, it should not be capitalized.


Rule number 1: One does NEVER EVER Store any content on the WINDOW reference. Keep a local variable in your current object for that purpose.

Putting data into the "window" object is the equivalent of using Global variables. This kind of coding style has been deprecated since the beginning of the century.

You have the "report-viewer" object sitting there. Why not use that?

The code, if run like this, will block the rendering of the report-viewer's frame until the server responds. That is really, really, really bad.

Please do that:
First, restore your original code for this feature from your branch BISERVER-13034. It was at the correct place. All you needed was to query and cache the server's response. For that, create a member variable in the report-viewer object to hold that boolean flag instead of having it as a method's variable.

Lets use the variable with 3 states. Null means: Unknown, and True or False are valid server responses.

Then you can go in and say:
if (_isAsync === null) {
   _isAsync = (pentahoGet(...) == "true")
}

which then guarantees that the pentahoGet method is called only once. After the first call, the _isAsync variable will no longer be <null>. It also means that this call is only executed when we try to communicate with the server for the very first time, instead of having to wait for the call to finish at construction time.

Remember: Servers can be far away and networks can be slow. You dont want to have users stare at a blank screen for 30 seconds while waiting for this call to return.

Then leave the code that queries the server into "reportViewer.js" at the end.


Could you also capitalize/punctuate this comment? We will be slightly converging to the format we have chosen :)


I don't think so. Because at least for me using @Setter (Which is embedded in @Data) never worked for overloading methods. That beeing said I also can't find that method in the bytecode/sourcecode, only the manually written deprecated method can be found.

I'll create a PullReq, one second.


`[D1, .., Dk]` should be in backticks.


затупил, про rootId было в прошлом комите.
Здесь про setRowOpened


Another way to install:
$uri = 'https://raw.githubusercontent.com/adbertram/PowerTrello/master/Trello.psm1'
Import-module psget
Install-module -ModuleUrl $uri


FWIW, I agree with this change and went ahead and made the same one.  If you're using NuGet you can update to version 1.1.1 to get this same behavior.


Also, perhaps it's better to isolate the "sampling" strategy for validating lazy sequences from the predicate for testing the joda DateTimes. I could see wanting to use it again.


yeah, it hasn't been added yet cause we weren't sure of all the properties to add. this is what it looks like in the form export schema: https://github.com/dimagi/commcare-hq/blob/index-together-domain-date-case-action/corehq/apps/export/models/new.py#L653


I preferred the origianl spelling


rank // 2 (spaces)


standard_report_area_check_tol (macumber) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Actually `evilBoundsError` is still used in a test case that verifies that `EvilCollection` traps on out-of-bounds indexing. I'm going to keep it.
However, `constructionMessage` below has no real purpose now, so I'm going to remove that.


![screen shot 2016-03-02 at 9 58 38 am](https://cloud.githubusercontent.com/assets/3943219/13465921/5e7c521c-e05d-11e5-88ae-cb2279b1e5b4.png)


> "Realm supports the following field types: boolean, byte, short, ìnt, long, float, double, String, Date and byte[]. The integer types byte, short, int, and long are all mapped to the same type (long actually) within Realm."

That's pretty shitty. Use `LocalDateTime` and convert it back down to a normal `Date` for Realm then. :\


テスト。


This was the error in the comparison.


Realm upsets me, @Brahmancer. It really does. :cry:


TaskCluster: @bhearsum does not have permission to trigger tasks.


:+1: was just thinking about that


I am not sure how this solves the problem?
If plugins dir is /var/lib/grafana/plugins
then this folder does not contain a dist folder, it is the walker function that walks into plugin directories


added customer data with a ÅÄÖ to verify umlauts support.


agreed, will remove update. doesn't seem like any harm in keeping index and attachment around


What does this mean


Since the feature flag was renamed to `Guest_ip_setting` we should probably change the exception as well.


LGTM


s/static/guest/ as in api_errors.ml


How is this working? getCount is the same as getViewCount?


I'll take care adding it back and will include the current year. It's not that tedious.

Moving forward, how do we list the copyright we're about to start having multiple authors contributing to them mix?


@ericzolf while the above works, I will not issue a PR for this as it is a dirty hack and not a proper fix.
Now that the issue is tracked here we can look at a proper fix later


standard_report_area_check_tol (macumber) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Instructions

second programming assignment will require you to write an R function is able to cache potentially time-consuming computations. For example, taking the mean of a numeric vector is typically a fast operation. However, for a very long vector, it may take too long to compute the mean, especially if it has to be computed repeatedly (e.g. in a loop). If the contents of a vector are not changing, it may make sense to cache the value of the mean so that when we need it again, it can be looked up in the cache rather than recomputed. In this Programming Assignment will take advantage of the scoping rules of the R language and how they can be manipulated to preserve state inside of an R object.
Review criterialess
This assignment will be graded via peer assessment. During the evaluation phase, you must evaluate and grade the submissions of at least 4 of your classmates. If you do not complete at least 4 evaluations, your own assignment grade will be reduced by 20%.
Example: Caching the Mean of a Vectorless
In this example we introduce the <<- operator which can be used to assign a value to an object in an environment that is different from the current environment. Below are two functions that are used to create a special object that stores a numeric vector and cache's its mean.

The first function, makeVector creates a special "vector", which is really a list containing a function to

set the value of the vector
get the value of the vector
set the value of the mean
get the value of the mean
makeVector <- function(x = numeric()) {
        m <- NULL
        set <- function(y) {
                x <<- y
                m <<- NULL
        }
        get <- function() x
        setmean <- function(mean) m <<- mean
        getmean <- function() m
        list(set = set, get = get,
             setmean = setmean,
             getmean = getmean)
}
The following function calculates the mean of the special "vector" created with the above function. However, it first checks to see if the mean has already been calculated. If so, it gets the mean from the cache and skips the computation. Otherwise, it calculates the mean of the data and sets the value of the mean in the cache via the setmean function.

cachemean <- function(x, ...) {
        m <- x$getmean()
        if(!is.null(m)) {
                message("getting cached data")
                return(m)
        }
        data <- x$get()
        m <- mean(data, ...)
        x$setmean(m)
        m
}
Assignment: Caching the Inverse of a Matrixless
Matrix inversion is usually a costly computation and there may be some benefit to caching the inverse of a matrix rather than compute it repeatedly (there are also alternatives to matrix inversion that we will not discuss here). Your assignment is to write a pair of functions that cache the inverse of a matrix.

Write the following functions:

makeCacheMatrix: This function creates a special "matrix" object that can cache its inverse.
cacheSolve: This function computes the inverse of the special "matrix" returned by makeCacheMatrix above. If the inverse has already been calculated (and the matrix has not changed), then the cachesolve should retrieve the inverse from the cache.
Computing the inverse of a square matrix can be done with the solve function in R. For example, if X is a square invertible matrix, then solve(X) returns its inverse.

For this assignment, assume that the matrix supplied is always invertible.

In order to complete this assignment, you must do the following:

Fork the GitHub repository containing the stub R files at https://github.com/rdpeng/ProgrammingAssignment2 to create a copy under your own account.
Clone your forked GitHub repository to your computer so that you can edit the files locally on your own machine.
Edit the R file contained in the git repository and place your solution in that file (please do not rename the file).
Commit your completed R file into YOUR git repository and push your git branch to the GitHub repository under your account.
Submit to Coursera the URL to your GitHub repository that contains the completed R code for the assignment.
In addition to submitting the URL for your GitHub repository, you will need to submit the 40 character SHA-1 hash (as string of numbers from 0-9 and letters from a-f) that identifies the repository commit that contains the version of the files you want to submit. You can do this in GitHub by doing the following

Going to your GitHub repository web page for this assignment
Click on the “?? commits” link where ?? is the number of commits you have in the repository. For example, if you made a total of 10 commits to this repository, the link should say “10 commits”.
You will see a list of commits that you have made to this repository. The most recent commit is at the very top. If this represents the version of the files you want to submit, then just click the “copy to clipboard” button on the right hand side that should appear when you hover over the SHA-1 hash. Paste this SHA-1 hash into the course web site when you submit your assignment. If you don't want to use the most recent commit, then go down and find the commit you want and copy the SHA-1 hash.
A valid submission will look something like (this is just an example!)


Oh, never mind I see how you worded it. Thanks.


thanks


Done


I generally favour this being the very first line of a function, so that it's obvious what's going on (because it's really compensating for the fact that default params are mutably stateful across invocations), and so that it doesn't matter what codepath you try to access/modify from


nit: maybe call this PythonExpression or EvalExpression or something?


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


gah, damn flu, this went to the wrong branch. sorry. head a bit woozy today.


Ohhh, ha i totally missed that method definition. Super smart! I really like that idea a lot


it makes sense to move it to session by now


@mcccs Different local or different global IP? You need to try another network...


same here, maybe `python-eval`?


Pour récupération correcte du CSS sous firefox


그건 버려진 commit이지롱 다시 만든 commit은 여기 591f86556d6ece04d01918efe20a72bf494b9f64
![screenshot from 2016-03-03 01 01 44](https://cloud.githubusercontent.com/assets/7028312/13466067/b8307b70-e0db-11e5-901a-cd5dafb945c6.png)


I think the `_expression` is a weird name. I'd propose `statement`


Please improve your GIT commit messages.


I'd name this `context_variables` and make a `DictProperty`


Remove the comment "TODO".


- This was already true prior to this PR.
- An outbox was also created, even though only a few tests were using it.
PR's goal is not to refactor the whole integration test fixtures, but only to add fixtures for testing sending emails feature.
You can log an issue to enhance the jmap method test if need be.


looks like a good start!


https://github.com/SpigotMC/BungeeCord/pull/1773 There you go.


why remove the comment, was a useful reminder how the connector gets used.


@zcorpan I don't think I understand this - why wouldn't it be `PutForwards=class`?


i mean it's the values reported by ffmpeg are almost half the values reported by youtube(for both the values that i added and the values reported by mpd).


makes sense


@j-friedrich  @agiovann

Hi Johannes,
It appears that the modification of of g in update_temporal_components creates the following error during merging:

```
  File "ca_source_extraction/merging.py", line 221, in merge_components
    g=np.vstack((np.vstack(g)[good_neurons],g_merged))

  File "/usr/local/lib/python2.7/site-packages/numpy/core/shape_base.py", line 230, in vstack
    return _nx.concatenate([atleast_2d(_m) for _m in tup], 0)

ValueError: all the input array dimensions except for the concatenation axis must match exactly

```

Can you please check?


Link for comms plan template https://docs.google.com/document/d/1xc7H6m7lfesCN-phJGvGSDPmtoinB5sM9KAA8deMNTQ/edit


Yes it was fixed in master a few hours earlier.


Could you describe this set up requirement a bit? You define `exposedProperties` but I'm not seeing anything done with it.


that's a mistake from my side. i took an example (http://shiny.rstudio.com/gallery/progress-bar-example.html) and i forgot the adapt it. i will correct it.


Should only refer to the team by one name. "outreach team" is preferred.


Didn't think i caught any changes in that revert, i accidentally overwrote several hours of changes and the revert should have taken it back to 9:30 A.M GMT thus fixing it, sorry if i caught your work in it.


Lets write some enzyme tests today.


since i'm moving here several utils that were disperse in bitmask_core, I think we should keep them in only one place. I'll check if it's possible that they might be used from more points (when simplifying the gui, no logic should need this stuff outside of bonafide, probably)


I think doing it in a single line would be better visually:
```php
if (!something) {
    return this->error;
}
return $this->success;
```
is slightly better than
```php
if (!something) {
   $error = this->error;
    return $error;
}
$success = this->success;
return $success;
```
(less lines, less things to remember from line to line, no difference in execution time, equally clear code)


true


100 here is not divisible by 1440, was that for testing purpose (see if you get black artifacts)?


I changed this method to make use of `DecimalType.parse()`. This one just constructs SqlDecimal object.
But I am not 100% that this is good move. This way we use product code to build expected value in tests. If that part of code has bug itself it may mask problems. Probably little chance.


First letter of each bullet should be capitalized. Change "comms team" to "outreach team"


"outreach team"


Great! That looks a lot better. :+1:


ceci est l'ajout de la 2eme ligne


revision


I'm aware of what you have mean. The main point is that mpd is **unique** for each format and actually gives pretty close numbers for some formats (several I've checked have deviation under 5%) while values from swf are hardcodes that are **the same** for each video. If we can't guess proper close hardcodes I see no point adding them at all.


in most of the extractors when the variable is meant to be constant it's name is in upper case.
https://github.com/rg3/youtube-dl/pull/6277#discussion_r36568145


I usually leave some dirty prints to remind myself this shouldn't stay beyond a reasonable debug phase, removing them on cleanup. this, for instance, needs to do the verification, I'm afraid that with another debug that'll be less obvious.


I feel these are too many methods :(


All the props go to Ezio and Griffin ;)


But not in this.


Iunno - maybe this question is already covered elsewhere, if so, I'll be happy to RTFM but why not leave the record field extraction in the function head for the fields you pull out in the function body?  Is that just your personal style preference?


@ericzolf now this dirty fix is in the correct branch, sorry for the noise.
As said in the wrong branch, as this is a dirty hack and not a proper fix I will not issue a PR for it. This can be fixed properly at a later stage.


Ha! Thanks! I'll correct that typo.. It was late last night :)


still ?


I like that, i'll also make that change for the sample.


characters


fixed - leftover from before we remove this syntax. I will rerun all the tests to make sure there are no more such things.


OK


fixed


I was quite experimental work indeed. My idea here was to use cred system to pass domain-specific avatars where needed, having bonafide hiding the cred mechanism from the rest of the system.
One of the places where it could be used is having a srp proxy authenticating only locally, if we want to do srp in places like the XUL.


Hi,

Is there any particular reason why this commit was reverted ? The min-height, min-width does no longer have influence on chrome 48 but firefox is still affected without these css properties.

Can you please collaborate ?

After upgrading to angular-material 1.0.6 our application which is in holly grail layout broke down.  This is the code we had to introduce to be able to run the application on Firefox and to support version 1.0.6


Hi @ThorbenLindhauer,

here a null pointer exception is thrown for tests:
- ProcessInstanceModificationEventTest#testStartBeforeTerminateEndEvent
- RuntimeServiceTest#testActivityInstanceForNonExistingProcessInstanceId

Are you okay with just adding a null check?
```java
if (processInstance != null) {
  processInstance.restoreProcessInstance(executions, null, null, null, null, null);
}
```


Nevermind, I think I get it now after diving into more examples.


saw approval from mrallen1
at https://github.com/basho/riak_api/commit/2cb64a2c5d114f4f15022091be0668d2a28de312


merging basho/riak_api/nem/webmachine-1.10.8-basho1 = 2cb64a2c into borshop-integration-107-nem/webmachine-1.10.8-basho1


basho/riak_api/nem/webmachine-1.10.8-basho1 = 2cb64a2c merged ok, testing candidate = 1c8e6888


We may probably want to introduce bitrate approx metafields similarly to filesize. But as I've already said this is the point for discussion in PR.


Corrected, I don't know what I was thinking. Will also verify in `juttle-engine` that this does indeed fix the bug that was manifesting there in the UI.


saw approval from mrallen1
at https://github.com/basho/riak_control/commit/bfe8a6ddcdd6b319d8953eb0351eebcd05a92cec


@abisker - @awfrancisco dropped some comments in here for #writing-lab edits. Can you do the edit reconciliation? I'm not totally sure how to do it, but Andre can help.


merging basho/riak_control/nem/webmachine-1.10.8-basho1 = bfe8a6dd into borshop-integration-192-nem/webmachine-1.10.8-basho1


basho/riak_control/nem/webmachine-1.10.8-basho1 = bfe8a6dd merged ok, testing candidate = 3bb9b788


I tried, but it doesn't work. metaclass stuff :/


Fixing this issue https://github.com/tangrams/tangram-play/issues/266


@Markenson Entendi. Achei que ele fosse genérico o suficiente pra não precisar de subclasses. Dese modo, seria interessante, então ter algum ponto de extensão dentro de logar(). Tipo um método logarDadosAdicionais(), por padrão vazio.

Senão a subclasse vai ter que reescrever o logar() inteiro.


good question. I think this regex is defined somewhere in the webapp... will check.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1353.87065455fb745c93c624445ef9f6941eb3e6d1ec](http://build.palaso.org/viewLog.html?buildId=69816&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Exit code 1 (new)) Build time: 00:00:25


"PofficialBuild" ?


Should not be better to use "None" as default, this should simplify also the following lines where you check for a pause.


For example this will become just "if start_pause:"


I didn't try the exploit, but it looked like it would work. I just fixed the FIXME.


Since you're using `return` there's no reason to use `elseif`, each conditional will only execute `if` it is met and stop code execution by returning.


I would put on a new line the two params... I've not checked but I feel that line is over 80 columns...


:arrow_up: @bterlson


sai loi chinh ta?


yes. currently I use the manhole to debug mail app, but I thought it could be useful to expose a pseudo-cli through ssh. It'd be nice where you run your core in a headless box and you have a prompt to control it. I also was thinking hidden service to host your mail.


Aula de DesignPatterns


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


Does this need to be a shell script or can it be dart?


I think we can spare the disk space for one more letter here  `Constrct` :smile:


shhh


That functionality got moved to the `spring-cloud-dataflow-server-kubernetes/pom.xml`, so the comment is there


it's a good candidate to me moved to common, but I woudln't right now, let's test it a bit more.


@ari-gold The first hop from the Site to the F5 LB Pair doesn't technically go through the Internet, it is all datacenter-local, so I would change the diagram to reflect that. Otherwise, looks great!


Woh, I love that these are doubles now.


nice


Modificado en github.com


hola github.com


@jabonzhang

请注意文件夹名字大小写要和你的用户名一致，否则这样的bug以后很麻烦。

要养成严谨的工作习惯。


I don't like having text that refers to a rule that no longer exists.


+1 LGTM


good


standard_report_area_check_tol (DavidGoldwasser) - i386-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


What happens if 24 hours passes and not everyone has voted?


@raphaelpaiva Pode ocorrer esse caso de reescrever o logar inteiro também. Mas se uma  subclasse sobrescrever o método logar() e precisar manter a lógica da classe superior, basta ele chamar super.logar() se quiser preservar o comportamento.


Hi David - Wouldn't it be better to use function grade_floats_different (from lib/gradelib.php) here?  That function looks very similar to your comparison except that it uses grade_floatval on both sides, which seems more correct.


I personally was tested and it is working with GHC 7.6.3 and LLVM 3.5.2


There's no reason it needs to be a shell script other than to follow the existing pattern for this part of the system.  I understand that you're unhappy that we're using shell scripting for this part of the system and I agree that we should consider changing that, but we can make that decision separately from this (already large and wide-ranging) patch.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1354.504b825efbddb8b1f35a5f20686e0c0e15f41fe3](http://build.palaso.org/viewLog.html?buildId=69817&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1028, ignored: 12 Build time: 00:06:33


:fire:


@martint what do you mean by plugins shouldn't be aware of `ShortDecimalType` and `LongDecimalType`. Even though those are implementation details, connectors must know internal representation of Presto decimals. They must know if they need to convert input format into `long` or `Slice`. They could also just copy bytes into `Slice` for long decimal with minimal conversion. I am not sure you can hide those details into core only. There are also utility methods in `LongDecimalType` which are used by connectors. We could move them into `DecimalType` but that wouldn't hide the fact that we deal with dual decimal representation.

Maybe you want to refactor this code into:
```
if (type instanceof DecimalType) {
  DecimalType decimalType = (DecimalType) type;
  if (type.isShortDecimal()) {
     ...
  } else {
     ...
  }
}
```
Is that what you mean?


We added features about connection introspection for SSL data, so when we released it we moved the point release up to 1.3.0 rather than 1.2.x


Hello

(sorry for my poor english...)
First, this is very useful tutorial and it helps me a lot ! Thank you
Then, as I've been blocked by a little mistake, I'd like notice you...
This little part of file where we add a subscribe to "parties" should have been done a little earlier in the tutorial...in fact, not at the point 9.9 but it should be at the point 9.6 just before trying to add "users". In my case, the details are not visible until I add this subscription.
More than that, this 9.9 appears after the 9.10... seems like it's not really the best place :)

Thanks again,

Antoine


@agherzan


standard_report_area_check_tol (DavidGoldwasser) - Win64-Windows-7-VisualStudio-12: Tests Failed (0 of 0 tests passed, 0 test warnings)

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


Not at all unhappy.  Just asking for context.  Thank you.


property official build


> (and the following commit)

what's wrong in the third commit?


This breaks some things for me... I really wonder what problem you had that required this.


this is rubbish!


all tests pass:
success: http://buildbot.bos1/builders/test-riak_api/builds/182


add buildrequires only when packages found:
```python
if sack.query().filter(provides=art.get_rpm_str(art.version))
    spec.BuildRequires.add(art.get_rpm_str(art.version))
```


(I also replaced deprecated evt iniztialization with new one, see: https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent/initMouseEvent


all tests pass:
success: http://buildbot.bos1/builders/test-riak_control/builds/28


While reviewing again with @book, we decided this conditional load should be moved to `PearlBee.pm`. I think after this we can merge it.


Maybe the variable naming is a big confusing.
pluginDir contains the path including the plugin itself. ex /var/lib/grafana/plugins/datasource-genericdatasource

The scan method is invoke on every plugin subfolder.


LGTM


thanks.  i hit up the #openldap channel on freenode and they informed me that the fix for 8336 was in 2.4.44 and the issue does come up more than originally though.


thanks so much @cindy-ckht :D
I'll put all of this into a .md so tht we can access it whenever we need


Similar tbr hardcodes.


ah, ok, sorry. Just i thought that this was small mistake.


Thanks for the input JD. Since I already had my `seeding data` branch open I went right ahead and integrated the default user code with my new seed code derived from the Faker gem


@Markenson Tem razão. Falei isso mais por que tem uma mensagem demarcando o fim dos detalhes de erro. Se escrever algo depois que chamar o super.logar() vai ficar estranho no log.


@swatijadhav @mcm-ham while I was fixing deprecation warning I noticed in this code we are using `pushPayload` as well as `push` however it looks like when we are using `pushPayload` that will automatically push all the objects so why we would need to have L#14? If you both agree then I will be removing line number 14.
http://emberjs.com/api/data/classes/DS.Store.html#method_pushPayload
http://emberjs.com/api/data/classes/DS.Store.html#method_push


So @Robotia, for how long did you have to twist Travis' arm to agree to building all your Thermos builds? xD


Just resubmitted. I was able to fix most of those things. Let me know what
you think.

Thanks!

On Tue, Mar 1, 2016 at 4:39 PM, Nick Becker <notifications@github.com>
wrote:

> In all 3 of your tests, you are purely checking that your functions are
> working as expected. For a complete test scenario, you should also look at
> where you code would fail. Try adding some more tests for what your code
> shouldn't do, or how it would react to things such as negative numbers,
> non-integer optional arguments, etc.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/DZwell/math-series/commit/6aa04d46d031b65be0eda278e3f6e1a0823248e5#commitcomment-16433907>
> .
>

--
Daniel Zwelling
Drummer | Bandleader | Educator
www.dzwellingmusic.com


Ahhh, it was a tough call and I thought it might be clearer if I gave a name to those objects returned :sweat_smile:
But seeing your examples I do think you have a point that it makes the code look cleaner :smile:

Fixed in de9a9fe782eb0028afb425ca9fdc84f92fb0a154.


@WinonaLyw related to IndexNode#insertPointer method.

The pointer is inserted to the right hand side the first time it meets 17 in the |13|17|17| node.

it gets the keyIndex = 1

and performs this.pointers.splice(keyIndex + 1, 0, pointer) => splice(2, 0, pointer)

current pointer is
[[2,10], [13,13], [17,17,17,17]]

so it inserts between index#2 and index#3 => [[2,10], [13,13], [17], [17,17,17,17]]


#coupon
Could you please use the "coupon" system instead of "sales"?It will improve the readability of your code,sir.


help file for the weave command


Hehe, I'll fix this comment up for ya.


hotovo


CC @nadako


Closed?


Good point! Fixed in 01c351e548dbefbfee662c0e68d46a3deac4e7e2.


rename this to tranData


świetny merdz


Hey!


We need to get multiple lua versions tested on Travis.


already exists :-P

https://github.com/znuny/Znuny4OTRS-Sublime/commit/196d044d2dc5491e3f1980130598c516bfb99512


Thouroughly tested version of the modified drawing and selection paradigm intended to fix bug #97 and end the interference between drawing activities of the same diagram in different contexts.


Why is this necessary? Because of this change, the same image that is not intended to have different versions for different screen scales appears differently. Basically the same image inappropriately smaller on devices with screen scale factor of 2 and 3.
Unfortunately this is taking effect after this commit: https://github.com/rs/SDWebImage/commit/95468315fc2892e3878a08c8f130276b47b2d408
In the time before the mentioned commit - this change was not affecting images at all.


Good call, i'm gonna change this up. Thank you


I think we should keep the same name


Updated packer URL


Well I was multitasking and doing it over the course of an hour...but now it should auto-build when a new release is made. All I have to do now...is add the proper release notes!


Well my hard disk was full so I had to make the hard choice of trimming one byte from the comment :sweat_smile:
After doing a disk cleanup, I realised there's now one byte more for a 'u' :smile:

Fixed in 4aa0ded49245f9e0d71bedd92d7d51c092ad4fbd.


Fixed #12


There are already safeguards in to make sure the same property isnt chosen.  Also, taking out those lines of code will totally screw up selecting properties like hit spell and hit area's.


Please observe the new logic. 'client' is used as a state to check if terminate has been called already. Some handlers especially onError can be called after deleting the 'client' object.


@raphaelpaiva blz!


Love life


Addressed by https://github.com/chaplinjs/chaplin/pull/889.


Yes, that motivation is outdated, but there is still a minor issue using enums and Google reccomend to avoid them (the annotations in my link are newer than 2011...
http://developer.android.com/training/articles/memory.html
It is not a Huge issue, but if used among all the library can affect perf and memory.
Here is also a video on this (2015):
https://www.youtube.com/watch?v=Hzs6OBcvNQE


Could you make a PR for main repo?


For obscure internal reasons, changes to the stream_executor are awkward to make.  Are these using declaractions necessary?  If so, could they be moved to one of the changed core tensorflow files?


This will crash if there is no `.` in the filename. Right now that doesn't seem to occur but it is not defensive programming.

I've debated this with Nallath and we decided that since this fix is a band-aid anyway, we're not going to fix it now.


The link in that commit message may have gotten munged.  See https://github.com/grails/grails-core/blob/758dbcc40112b19d4f5f8fbc7446ecdd4ebdc020/grails-validation/src/main/groovy/org/grails/validation/DefaultConstraintEvaluator.java#L359


one more thing:
after processCommand returns (with an error), the loop in processInputBuffer may continue processing the rest of the input buffer of that client.
it seems you need to break out of the loop if current_client==NULL


@petevine
Hi, did you modified src/sys/unix/uds.rs?

seen like your error message due to using unmodified version.


엥 그러면 브랜치 지워놓지 ㅠㅜ 내가 지울게


It's not that hard to define, modulo "implements" statements.  If we posit that "implements" should not affect branding (as in, the mixin is a mixin and you can't use it as an argument type, say), then things should be pretty straightforward...


is there a reason why you're not using [arity](https://docs.puppetlabs.com/puppet/3.8/reference/yard/Puppet/Parser/Functions.html#arity-class_method) instead of this?


cannot generalise this down to `:kernel => :linux`?


Revert rlempereur erroneous commit


It should never be defined as `none` in C defines (that goes for all LEDs).
You can disable a led with `<configure name="RADIO_CONTROL_LED" value="none"/>`, but that assigns `none` to the makefile variable and not the C define.
You probably accidentally used `<define name="RADIO_CONTROL_LED" value="none"/>` somewhere instead.
Please remove this and fix it in the airframe file.


As far as I know the only way is to do releases ;-)


I like this. Better than the fake-version.


We should clean this mess up. I don't like both of the "tabTriggers"...


But we don't need to use Moose for the application itself. It's just for maintenance.


organizing the directory


Doesn't this add a DB call per module per page load??


No. How would that look like?


i prefer to be explicit.


first try


Some of the this functions are duplicates or not correctly included. This depends if you are trying to compile on Windows or Linux. So if we apply this change compiling on Linux would not work anymore. I thought I had fixed it but you prove me wrong.


Maybe you should remove this --^
So it won't conflict with my PR. Or maybe that should be done in my branch and rebased after this one is merged?


LGTM!


112259371_UnitHeaterHWCoilUASizing (Nigusse) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


can you briefly explain this construct here?
it seems hard, especially if we're gonna use this as name/title


Travis fail because the new class should be added in test class


@Labrys can you check this?


What's the reasoning behind this change? It seems problematic for a couple reasons:

1. Opening a WebSocket connection is asynchronous so calling `open` and then immediately calling `send` will probably fail. `isOpen` will return false and the data won't be sent.
2. This requires sending data to open the connection so it's not possible to *receive* data without first sending some.


__API Description file syntax error on line 14__: the use of tab(s) '\t' in source data isn't currently supported, please contact makers (Autogenerated message from apiary.io)


looks good.


github.com Nasty14 psw nasty123


i don't think we need the backtrace here, i think --trace should already do that. No?


sudo apt-get update

sudo apt-get upgrade

sudo apt-get install libreadline-dev libconfig-dev libssl-dev lua5.2 liblua5.2-dev libevent-dev make unzip git redis-server g++ libjansson-dev libpython-dev expat libexpat1-dev

cd $HOME

git clone https://github.com/mohamadkings/ubiquitous-umbrella/


pls respond


'connected' and 'available' are moving to 'joined' and 'ready'.  no biggie.


also inconsistent here (no Backtrace here)


I don't know.


@irineau I didn't realize that had changed but my recollection is I tested building it with the various options. I'm happy to bump the QT version in a follow up if you think that's best.


See 'ShortFinMako'

http://tests.stockfishchess.org/tests/user/VoyagerOne?page=14


"key-<md5 of the value>"


fast-forwarding develop-2.2 to borshop-integration-107-nem/webmachine-1.10.8-basho1 = 1c8e6888


this is a single line, we can leave this as
```
desc 'The provider to use: `olc` or `augeas`'
```


fast-forwarding develop-2.2 to borshop-integration-192-nem/webmachine-1.10.8-basho1 = 3bb9b788


Yes... but is it worth to use Moo in this case? We would have a lower memory usage and faster startup, but we'd also have a slower runtime, wouldn't we?


Well, that's nice and all, but it does only if you can also pass --trace.


有没有最优答案或者起码是正确代码之类的？


Did you test this?
Episode.find({'characters.Character' : characterName}
Is that working?


Still DTM.


AntarticCod passed green at STC

http://tests.stockfishchess.org/tests/view/555f8b9c0ebc5940ca5d6cba


#include <Windows.h>
#pragma comment(lib,"winmm.lib")
int main()
{
	PlaySound(TEXT("d:\\map4-6.wav"), NULL, SND_FILENAME);
	Sleep(10000);
	return 0;
}


Dla comment on this commit


`path.join`'s output differs on win32 vs posix likely causing https://github.com/ember-cli/ember-cli/issues/5542


I think this is correct, actually.. Numbers are usually right aligned, and NULL isn't a number.


why not just include the class here?


Why not do this where `actualRun()` is called? That way, exceptions in non-token-tree checks would be caught as well and you don't have to remember to use try-catch every time / less redundancy.


My point is that the test won't detect certain classes of bugs in the `WSClient` implementation, regardless of whether it uses asio exceptions or not, except by manually examining the log output. Obviously it works now, but it's not very robust against bugs being introduced later. I'm not saying you need to dig deep into the message (which would be testing more of the rippled internals than may be appropriate here), just that you verify that there _was_ a message - that communication is happening.


What happens if reactor.resolve() fails?


Some white space here triggers pep8


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1355.74f850d2c0283fa99b1b7add79be443469650479](http://build.palaso.org/viewLog.html?buildId=69818&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1028, ignored: 12 Build time: 00:06:19


socialism


this is a very good test imo :O


Because it skips and does weird things then? :dancer:


I adhered to the style of the rest of the module. See https://github.com/JimPanic/puppet-openldap/blob/master/manifests/server/globalconf.pp


Great, thanks!


The Data Flecher.


I do not know exactly how homebrew works. But, if the Qt5 libraries are not installed during the build of the package, then the library CGAL_Qt5, and demos, cannot be built, even if `with-imaging` is true. This formula should probably depend on `qt5` if `with-imagine` is true.


this should be `if case.is_deleted`


I put a floating point zero so that you immediately that you need a number, but if None is better I'll change it.


hey


@senderista please scrutinize this line, too. I'm not sure if most unary aggregates were excluded by @7andrew7  simply because they had not been needed.


I am not a student, so ineligible. However for certain projects I might be
able to contribute in an advisory/mentorship role.

On Wed, Mar 2, 2016 at 2:58 AM, Bruno <notifications@github.com> wrote:

> @chris838 <https://github.com/chris838> @iurimatias
> <https://github.com/iurimatias> @masukomi <https://github.com/masukomi>
> @melink14 <https://github.com/melink14> @rjmarsan
> <https://github.com/rjmarsan> @octatone <https://github.com/octatone>
> @omesh <https://github.com/omesh> @ouiliam <https://github.com/ouiliam>
> @tonyolag <https://github.com/tonyolag> @shaolintl
> <https://github.com/shaolintl> @shubhampkj <https://github.com/shubhampkj>
> @Mobilpadde <https://github.com/Mobilpadde> @Vandan-V-Phadke
> <https://github.com/Vandan-V-Phadke>
>
> Mind The Word was accepted for Google Summer of Code in 2016. If you are
> student at any level (undergraduate, M.Sc. Ph.D.), you are eligible to
> apply.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OiWorld/MindTheWord/commit/8088133986563c8078685e689a7fbccc519fff31#commitcomment-16438538>
> .
>


These edits all look good to me. Yes, @awfrancisco if you can point me in the right direction for where/how to implement them that'd be great.


This help much? Any prereqs?


rất tốt, lỗi chính tả đã được sửa


never mind, saw you updated in a later commit


A bit difficult to distinguish code style changes (camelCasing) from the real change (course real_id)


TaskCluster: @bhearsum does not have permission to trigger tasks.


I think we might have things that implement EventTarget. We definitely also have purer mixins that do not want trait-like functionality. Hmm.


I think the path to `external_libraries_new` is not needed anymore after the new plan of progress we talked about.


@gregvanl you can go ahead and remove it, and I will test once that release comes out. If there are issues, I will add a pull request to update the documentation.


i think you need 2 newlines between imports and the rest of the code


Within the changeset, signature changes were made to the following objects:

- **Communications:**
	- Added a new dictionary to the constructor.
	- Added an index operator to simplify access to injected dictionary.
- **LitleRequest:**
	- Added a new dictionary to the constructor.
- **LitleFile:**
	- Added a new dictionary to the constructor.
	- Removed the "createDirectory" method.
- **BatchRequest:**
	- Added a new dictionary to the constructors.
	- Changed "addEcheckPreNoteCredit" to return the "tempBatchFilePath".
- **RFRRequest:**
	- Added a new dictionary to the constructors.
- **LitleOnline:**
	- Added a new dictionary to the constructors.
- **litleResponse:**
	- Changed the constructor to take in a "MemoryStream" instead of a file path.
	- Changed "DeserializeObjectFromFile" to take in "Communications".


also, we use one more newline here :)


Note that the areas for clouds will be pc^2 but the areas for the star formation rate are in kpc^2, so this should be multiplied by 1e6 to get this into years.


Now that we're backwards compatible, do we still need to delete this test code?


Amazing!


thanks @michaelmalonenz, i'm gonna make some changes and force push.

I'm also thinking about renaming the callback to something like `beforeDisplay` or something like that since you can do more than just position it.


i think the current functionality even lets you get and set values like name and owner_id with these methods, and i wouldn't be surprised if people have apps that get or set owner_id automatically in very specific workflows


Does this look better ?  #40 to #50 can be reduced to:
if(Objects.equals(mesosSlaveInfo.getLabelString(), assignedLabel)) {
               workspacePath = mesosSlaveInfo.getRemoteFSRoot()
 +                      + File.separator + WORKSPACE + File.separator + jobName;
}


Can we please put the `const` to the right of the thing it modifies? e.g.
```
is_call_possible_udt3 const, int(int)>::value, "");
```


Yes, I saw that it was failing on tests, it should be OK now. Travis is not yet finished but CA compilation is over without error.


Nice!  In terms of publishing on Packagist, how would I do that?


Actually this `WSClient_test` doesn't look finished or comprehensive. Note that this code is highly experimental.


Should be liter per minute


Would it be better to migrate all the old `external_libraries` to use `external_libraries_new` instead?


Makes sense...