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\":\"aiqLWb0UTHeU1tMzNebzgg\"}\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\": \"b4de303ad889f7aa23f96def2d0cb6364d99cb2a\",\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      \"branches\": [\n        \"master\",\n        \"dockerflow-endpoints\"\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-10T15:09:45.373Z\",\n  \"deadline\": \"2016-03-11T15:09:45.373Z\",\n  \"schedulerId\": \"task-graph-scheduler\",\n  \"taskGroupId\": \"N1IHpAgbRTK_nWJuk37Dtg\",\n  \"routes\": [],\n  \"priority\": \"normal\",\n  \"retries\": 5,\n  \"tags\": {},\n  \"expires\": \"2017-03-11T15:09:45.373Z\"\n}\n  time:     2016-03-10T15:09:45.686Z\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\":\"aiqLWb0UTHeU1tMzNebzgg\"}\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\": \"b4de303ad889f7aa23f96def2d0cb6364d99cb2a\",\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      \"branches\": [\n        \"master\",\n        \"dockerflow-endpoints\"\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-10T15:09:45.373Z\",\n  \"deadline\": \"2016-03-11T15:09:45.373Z\",\n  \"schedulerId\": \"task-graph-scheduler\",\n  \"taskGroupId\": \"N1IHpAgbRTK_nWJuk37Dtg\",\n  \"routes\": [],\n  \"priority\": \"normal\",\n  \"retries\": 5,\n  \"tags\": {},\n  \"expires\": \"2017-03-11T15:09:45.373Z\"\n}\n  time:     2016-03-10T15:09:45.686Z\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": "aiqLWb0UTHeU1tMzNebzgg"
                },
                "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": "b4de303ad889f7aa23f96def2d0cb6364d99cb2a",
                            "GITHUB_HEAD_BRANCH": "dockerflow-endpoints",
                            "GITHUB_HEAD_USER_EMAIL": "bhearsum@noreply.github.com"
                        }
                    },
                    "extra": {
                        "github": {
                            "env": true,
                            "events": [
                                "push"
                            ],
                            "branches": [
                                "master",
                                "dockerflow-endpoints"
                            ]
                        }
                    },
                    "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-10T15:09:45.373Z",
                    "deadline": "2016-03-11T15:09:45.373Z",
                    "schedulerId": "task-graph-scheduler",
                    "taskGroupId": "N1IHpAgbRTK_nWJuk37Dtg",
                    "routes": [],
                    "priority": "normal",
                    "retries": 5,
                    "tags": {},
                    "expires": "2017-03-11T15:09:45.373Z"
                },
                "time": "2016-03-10T15:09:45.686Z"
            },
            "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/*"
                ]
            }
        }
    }
]```


Test


Could you put _send_email_in_transaction() just above this last line? It seems slightly odd to separate them by a bunch of stuff.


Я не просил излишне оформлять статью


I see it the other way round. The order is an additional contract which I'd only guarantee if there's a compelling reason that it's useful.

> Am 10.03.2016 um 15:26 schrieb Sam Brannen <notifications@github.com>:
>
> I would expect the order to be retained unless there is a compelling reason not to.
>
> —
> Reply to this email directly or view it on GitHub.
>


We have a very strong preference to keep the same parameter interface as numpy/scipy. Why did you revert that order?


Full disk, should not be connected to this


**SUCCESS** - TeamCity HotRod C++ Client :: Pull requests monitor - WIN_S2012 [Build 358](http://ci.infinispan.org/viewLog.html?buildId=37276&buildTypeId=HotRodCpp_PullRequestsMonitorWinS2012&guest=1) for 6396809633bf909a9a6f7540994af9804d44f89f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:40


Два закрывающих тега body?


嗯，上面的问题清楚了，这里就可以调用`Handler#removeCallbacks()`传入成员`Runnable`，避免内存泄露了。


Ah ok, pefect; yeah, definitely, that seems like a good idea and queries will load faster.
Anyway, I think we organized this thing quite well by writing that API spreadsheet, now I have a concrete idea about what to do and how to do it, which is great; I believe I am be able to write all functions, only one it will probably dealay me a bit is finding a way to build that array with introns and exons alternating for Dolapo to higlight exons.

If you are in earlier I will be in the class around 5.30.
See you later
Date: Thu, 10 Mar 2016 07:01:55 -0800
From: notifications@github.com
To: Biocomp2Project@noreply.github.com
CC: g.nocchi@hotmail.com
Subject: Re: [Biocomp2Project] updates to database code and API spreadsheet (7310d9f)

I was planning on using part of the function you write to get the untranslated cds for the purpose of highlighting the exons to populate this feild. In order to make the other queries that need the untranslated cds load faster.

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


The code does not comply to PEP8.

*PEP8Bear, severity NORMAL, section `autopep8`.*


Shortlog of HEAD commit is too long.

*GitCommitBear, severity NORMAL, section `commit`.*


Line is longer than allowed. (85 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


Line is longer than allowed. (91 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


У вас есть проблемы с выравниванием кода, а вы стараетесь ещё и сделать больше. Попробуйте сделать меньше, но абсолютно правильно.


恩。其实这里的复杂度搞得这么搞，就要考虑下是否有过渡设计，失去灵活性的同时，感觉对于架构并不讨好，你决定下是否这里需要同步做，让接口实现的地方自己去考虑是否需要抛ui线程，我们只要在这里注释清楚，给出建议就好了。这个你决定下。其实很抱歉，现在看到复杂度提高这么多，站在这个角度，感觉直接同步会更好了。抱歉了。


И здесь


И здесь и так далее


:smile:


@dcesari I have integrated your script.


gl hf


the generator does not actually need to return anything


develop (energy-plus) - Win64-Windows-7-VisualStudio-14: OK (1579 of 1579 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)


Ug, import `LogFactory`.


Couldn't you just do `return Contains(item)`?


This is already done at line 276 in the subclass. Why do you need that here? Could you try to fix something that is already fixed?


Этот require не нужен — у тебя же этот файл всё равно из movie require'ится.


Wow Gold. :D


What's that?:-) Isn't it copied twice?


Такие файлы как этот обычно называют `lib/core_ext.rb` (и складывают в них все твои расширения для классов Руби)


ty gg
10.3.2016 17.13 "Ari-Pekka Jääskeläinen" <notifications@github.com>
kirjoitti:

> gl hf
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/samigrano/Ristinolla/commit/35d1f2d60825885cf402d06a025c3ab6ce9f5990#commitcomment-16619142>
> .
>


Ok, sounds fine.


:question: :facepunch:


TeamCity Hid Library :: Deploy [Build 3.2.32.0](http://build.mikeobrien.net/viewLog.html?buildId=2541&buildTypeId=hidlibrary&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:47


Thanks. I am still learning JS :)

On Thu, Mar 10, 2016 at 10:08 AM, Wez Pyke <notifications@github.com> wrote:

> It's best practise to put a semicolon after a function expression -
> https://developer.mozilla.org/en/docs/web/JavaScript/Reference/Operators/function
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/wezpyke/turtleblocksjs/commit/e58bcfb668211f87990b33687c56bb9e2cf34e47#commitcomment-16619004>
> .
>

--
Walter Bender
Sugar Labs
http://www.sugarlabs.org
<http://www.sugarlabs.org>


Is there a reason relation needs to be sent in with the request? Could you just set the correct relation in the code?


This change creates an incompatibility with the debug toolbar...
with the debug toolbar the `path_info` starts with: `/_debug_toolbar/` ...


Task 1: just a few strange formatting things. You have spaces where there shouldn't be any and also make sure you leave a space on either side of an equals sign when defining variables and that you put a space after each comma. Just small stuff, but it could matter in longer more important codes.

Task2: Again with strange formatting...extra spaces in functions.  I can't get this to work. It makes the file, but nothing is in it.  Same thing for task 3.

If you think I'm doing something wrong you can let me know exactly what you put into the command line to make it work and I'll try again. Here's a picture of what I was using, and like I said it created the output file as I expected, but the output file was empty.
![capture](https://cloud.githubusercontent.com/assets/16652287/13673893/c5c6e600-e6a0-11e5-905f-3a472a20073a.PNG)


Přidat někde sem ekvivalent argumentu `` --hostname=`sed 's/:/-/g' /sys/class/net/eth0/address` ``


زرت#


@atd, This is currently not working, not number is shown when I use it in firefox an chromium.

I checked out this commit and it also seams not to work.

I tried to fix it, but the only way I managed to do it is to come back to how it was before this commit.


Why is it a `BucketGrid2D` that maps locations to line segments, rather than a normal `map`?


This same line needs to be added to Amazon Fire OS platform definition.


lol oops. this was meant to read "Move `clear` around", but it looks like bash expanded the `clear`.


@atd, should we revert this change?

I can fix it that way if you are ok with that


could we make bash and regular coreutils available?

the busybox/toybox toolset sucks for interactive usage... there is nothing more annoying than using a shell with them and barebones "vi" editor (instead of vim/emacs/nano)


Done.


lel


Ty


Fixed problem with these script.I've tried to install owtf and the scripts were not creating the .owtf folder properly. That wase because the function get_config_value function (included in both scripts) was returning a bad value because the colours were making a bad path to the desired path.

And usage added to the /install/kali/install.sh script were teh OWTF project folder is required


[R/impute_glmerMod.R:69:16:](https://github.com/ThierryO/multimput/blob/c9ef04a9ec184402e1628fdb26624491c818d936/R/impute_glmerMod.R#L69) *warning:* **Do not use absolute paths.**
```r
        paste("~0 + ", hash) %>% #no lint
               ^~
```


This should be updated to Map<String, Donor> (I know it wasn't necessary for this task, but it will be helpful for the tasks coming)


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


LGTM


what is `six`?


develop (energy-plus) - Win64-Windows-7-VisualStudio-12: Build Failed

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


Cool beans. :+1:


Ok


Sorry... I did the split in later commits...


По общей структуре папок в lib: смотри, логика обычно такая:
```
lib
  my_library_name.rb # главный файл, который подключает остальные
  my_library_name/
    my_library_part.rb
    my_large_part.rb
    my_large_part/
      my_library_subpart.rb
```

При этом обычно ещё и делают так, чтобы структура модулей соответствовало структуре папок. То есть вся твоя библиотека — в модуле `MyLibraryHame` (чтобы твой класс с названием просто `Movie`, когда библиотеку подключат в другой большой проект, не путался с другими Movie), а в файле `my_library_name/my_large_part/my_library_subpart.rb` лежит обычно класс или модуль `MyLibraryName::MyLargePart::MyLibrarySubpart`. Это не догма, но привычка. Нарушать её можно (например, складывая маленькие классы в один файл), но начинать лучше именно с такой структуры.


TeamCity Civic Assembly :: Build backend [Build 7](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1782&buildTypeId=CivicAssembly_BuildBackend&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1194, ignored: 3 Build time: 00:15:08


Gemfile should call `gemspec`, and the dependencies should go in the gemspec file.  This article explains the details
http://yehudakatz.com/2010/12/16/clarifying-the-roles-of-the-gemspec-and-gemfile/

Also, it's better to use `https` in the `source`


Mmmm @DomT4 are you sure it is what it's supposed to be ? :-)
Actually, the current home for librsync is https://github.com/librsync/librsync

See screenshot below:
----

![image](https://cloud.githubusercontent.com/assets/270528/13674097/7d41b766-e6dc-11e5-9382-cd9133e2687a.png)


Add Django 1.9, and DRF 3.3


Very good, but this patch is subject to security issues, and will explode when fuzzed.

- [ ]  `*_serialize()` should return the number of bytes written and -1 on failure, optimally enforced via `__attribute__((warn_unused_result))`. A `int32_t` should suffice, since containers are small by design.
- [ ] `*_deserialze()` should have the exact number of bytes it should read, typed as `size_t`. This should be enforced with checks and return codes. A failure to deserialize should not segfault/assert/etc.
- [ ] make sure to follow the code style guidelines, there's tool for this in tools/. I can help with reformatting if clang-format is not available on your hosts.
- [ ] We carefully allocate on aligned boundary for AVX instructions (see `posix_memalign`), this code does not align allocated containers.


My experience is that CMake transforms that path to canonical form. Originally I used lower case drive letter, but in the MITK_BINARY_DIR it become upper case and therefore the the two paths did not match. It might be done by the ExternalProject_Add call, not sure.


can't we simply always use full path here?


@jorenberg:
Thanks **Emma**.


Eeeh...no idea, probably yes :)
I must have accidentally  copied it here, thanks for pointing out.


Because I needed all the Rules in the same class... I had some hard time with this...
(That is of course what I tried to do first ... )


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1021 of 1021 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-30.56%25-green.svg)


:100: I skipped that, relying on the fact that the front end will not make that request. I have fixed that. :raised_hands:


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Ignorem esse commit, o código inteiro foi descartado.


Learn about exceptions and `rescue` first (so Ruby error handling). Rails adds a bit of framework-specific methods on top of that, like `rescue_from` in controllers: http://api.rubyonrails.org/classes/ActiveSupport/Rescuable/ClassMethods.html


Isn't this fix applied to the wrong branch? I think the chrome48 one (which is actually chrome49) is the one you want.


It was a copy and paste gone wrong which I thought I fixed but must not have committed. Looks like there are a few more that will need fixed as well...


Uh, yes. You see?


TeamCity Civic Assembly :: Build frontend [Build default-frontend-9](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1785&buildTypeId=CivicAssembly_BuildFrontend&guest=1) is now running


indeed we could


$message should be $data


try to merge and bump number in same commit please!


My fault...

IntelliJ integration with maven is suffering a lot from project reload between branches were the sub-projects exists / don't exists.
I got a pretty akward bug, were the test resources were not located, so the wrong configuration files were loaded... That was an intent to understand what was going on...
Sorry for having missed it on the way to commits


Good idea. both changes accepted.


On my way to becoming a GITHUB MASTER.  I love Ruby and JavaScript.


Jeśli to jest po zapisie nowego druku potwierdzenia to może hook nazwać analogicznie jak w innych tego typu sytuacjach?


I'm wondering if it wouldn't make sense to include `otl_parse` into `otl_POST`/`otl_GET` so we can directly do at the end of the functions:

```r
...
res <- otl_POST(...)
res
}
```


Name/ID alignment in left_nav closed by 7951d5bf2f:

> Provide backwards compatibility for deprecated TD align attribute


When you don't create things, you become defined by your tastes rather than ability. Your tastes only narrow & exclude people. So create!
-Why The Lucky Stiff


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


This is pointing to the chrome49 branch which I think is wrong and it should be the chrome48 branch on libchromiumcontent which is the more updated 49 port.


That's the thing about people who think they hate computers. What they really hate is lousy programmers.
-Larry Niven


@kares it has exposed problems in generating bytecode so I think it can have value in development.  It also happens off-thread (compilation) so it is less critical and hopefully really uncommon.


@vilchik-elena Suggested factorization: b33114543a0b59aa2fa0461fd9f0726307b6d2a9


Hi,

AssetHelperHelper.php should be AssetHelper.php ? Couldn't get it working otherwise.

Thanks, Christopher.


They make a fair point I guess.


@sorinsrn7, Doing the sort here means that we are sorting the results every time a file is parsed. We should likely use the post-parsing hook we have in `pad.rules.ruleset` (`post_parsing`). This way the results will only be sorted once.

We could even likely just insert the rule in the right place in `add_rule` but I think that might just over complicate the logic with very little benefit.


It would be logically correct, but I think the current code is fine, because it will easier to add something if it will be needed. It is a personal preference tho.


Are you sure ProGuard is needed here? I don't feel the app is large enough to warrant it, and ProGuard is just unnecessary headache.


This works on my live shop..android smartphone. @Tsimi78 @gburton


[2 * 3 = 6](http://pythonhosted.org/six/)


So one of the things that we should be avoiding in the future is the var type.

var should be reserved for anonymous types, not anything that has a known type in advance.

Its far too easy for a var type to be used in appropriately at a later date without the compiler giving any warning that you've drastically changed the resulting behavior by plugging in a completely different type that happens to have prototype that matches 'close enough' for the code to compile.

In this example, assembly should be an Assembly type, assemblyName should be an AssemblyName type, ect.


I removed unused code


Pycharm package sorting


interesting, i think that would be a good route to go. the past logic used to use time to break off the task. see:
```
if now > cutoff:
   break
```
where cutoff is now + the interval of the task haha


It's not doen for all subclasses.  Specifically it's not done for the GPU versions.


yeah, but just realized this is broken on pagination


About the BatchCom page, can I simply remove that light-blue BatchCom title that opens up the frame in a new tab?


will up the chunk size (accidentally committed `10`)


Yes, I saw.


Esto tendriamos que mirarlo con calma, ya que lo que hace la taglib es ir a la pestaña anterior del navegador, pero parece que no es posible por el momento hacerlo usando el back superior.

Lo miramos con detalle si quieres cuando lo estes revisando


wait nvm it's not broken


this is just not used


The latest version of roxygen (> 5.0.0) doesn't write the version number on each page, just in DESCRIPTION. So if you update it won't happen again :)

I don't see what has changed in the docs though.


Did you check that this works well for extended partitions vs. their first logical one? This might or might not work: They probably both start at the same cylinder, yet the extended should come first. Maybe add more sort criteria, e.g. type primary before type logical if the region start is the same?


That makes sense. I'm now thinking where I should put that. Given that the methods are passing the values by the current implementation I have.


Fixed in de17573


TeamCity Civic Assembly :: Build frontend [Build default-frontend-9](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1785&buildTypeId=CivicAssembly_BuildFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 76 (20 new), passed: 148 Build time: 00:08:11
### Failed tests
```
Blog categories tests: Should delete blog category new blog category 1457623741014: <no details avaliable>

Blogs tests: Should Delete an Blog: <no details avaliable>

Basic USER CRUD Actions: Should Delete CONTACT: <no details avaliable>

BasicMedia Types CRUD Actions: Should Delete an Media Types: <no details avaliable>

Basic Subscription List CRUD Actions: Should Delete Subscription List: <no details avaliable>

Basic USER CRUD Actions: Should Delete USER: <no details avaliable>

Venue tests: Stadium type: Should Not Add Stadium if Cancel button pressed: <no details avaliable>

Venue tests: Stadium type: Should Add Stadium: <no details avaliable>

Venue tests: Stadium type: Should Add an Stadium and CONTINUE: <no details avaliable>

Venue tests: Stadium type: Should Not Edit a Stadium: <no details avaliable>

Venue tests: Stadium type: Should Edit a Stadium: <no details avaliable>

##### there are 66 more failed tests, see build details
```


I always make sure this does not happen... failed this time.

good call, thanks!


protect_from_forgery with: :exceptionのところのインデント下げすぎてる。
階層レベルは  def after_sign_out_path_for(resource)と同じにすべき


Why are you invoking an explicit GC here?


i looked this up and it doesn't appear to be possible. we'd have to make our own function to do this. do you think it'd be better to just make a key based on the ID and have a key for each record? arguably this is pretty close to the locking functionality


Esto era un error. cuando creamos "normalmente" un wiki page, se crea el WikiPageImpl y no es nulo, pero si new.

Asi que es necesaria esta segunda clausula para que se muester el permissions


Shouldn't this list levelSliderControl and not colorTempSliderControl?


Careful of your formatting. There should be a space on either side of equals signs when defining variables and there should be a space after each comma. Also, in task 2 there are extra spaces between functions that aren't necessary.

Otherwise it looks great! I love looking at your code, you write your functions to be so efficient. Its a great learning experience for me.


@pynicolas fine for me


Hello, I am working on some changes to this code but I do not have the slightest clue on how to build this. Can someone help me with building this project?


Me lo he podido cargar y en mis pruebas no he encontrado ningun error.

La logica por tanto a partir de ahora es:

newPage = (wikiPage == null) || wikiPage.isNew()

y

!newPage = (wikiPage != null) && !wikiPage.isNew()


@mattgarrish: Looking at the above commit and/or https://rawgit.com/w3c/aria/master/aria/dpub.html#doc-endnote, it appears that you forgot to change `doc-biblioentry` to `doc-endnote`.


thanks for the hint. I switched the order, but left 48000, because it's the current default rate in v0.5


It returns either `a number` a `false`, and I'm calling the method if what was returned is `a number` and not `false`.


@ccx-testerbot runtests master


The admin removed your request (id 167) from the queue. Please get in touch to know why.


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


so that utility library is helping to make sure this library works in Python 2 and Python 3?


It appears that you get a prompt when installing the plugin via `cf install-plugin` (hence the Travis failure).  A `-f` flag with the command should do the trick:

`travis_retry cf install-plugin autopilot -f -r CF-Community`


改行！


There was a bit of confusion with the title-custom/title-background class in the previous reviews. I made a refactor commit that was supposed to fix this issue by renaming title-custom to title-background and replacing instances of "title title-custom" with "title-background". I can revert this commit. I need to run a Git Blame and have a look at these extra <p> and <b> tags that Kevin mentioned. If the previous developer did in fact add those in, then it's not acceptable and I will remove them and refactor the title-custom class to work properly.


@vilchik-elena I think the new sub-condition is useless: if `className == null`, then `methodName.equalsIgnoreCase(className)` is false.


"I 've" should have the space removed from it


If color temperature is being removed, shouldn't the capability be removed as well?


Nevermind! I got it working. I was just using the wrong input file for tasks 2 and 3. I forgot that you need to use the output from task 1 for the input instead of c-darwin.


Good catch, but I don't think error messages should have the same treatment, I.e those within the checkArguments.


听你这么一说，确实感觉有点把简单问题复杂化了，我觉得要不还是改成第一个commit https://github.com/szitguy/JKeyboardPanelSwitch/commit/a75880de4e15e476d0055c43a75b0d28fe19203c#diff-cfbe49b259e3abd81b1c57014a299fd5R178
这样的同步调用吧？你怎么看


Always use unique_ptr.  Always.


translation, ~~go~~ +visit+ the pull request and ~~there~~ +find+ the tab "Files changed" +right under the PR's title. There+ you will be able


This would be a good candidate for either singleton pattern or a DI framework


Even for FILE*


Check if that condition does what supposed.


This observable never gets closed -- potential memory leak


You're using indents of 4 characters, right ?


Yes, 4 spaces.


any possibility you want some help? It might be more fun to work on it yourself and my time is limited but I am willing to sling some code your way if you are interested.


Good candidate for Singleton/DI


Observable never closed


there is a change in the english version of the challenge.


TeamCity Civic Assembly :: Build frontend [Build default-frontend-10](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1789&buildTypeId=CivicAssembly_BuildFrontend&guest=1) is now running


Fixed in 043d4c7.


Just tried these instructions and got
```
$ mirage configure --unix --net=socket
mirage: unknown option `--net', did you mean `--xen' ?
Usage: mirage configure [OPTION]... [FILE]
Try `mirage configure --help' or `mirage --help' for more information.
```
I'm using mirage 2.6.1. - debugging now. But thought i would let you know either way.


Nit: identifiers starting with a capital letter.


need a more descriptive name here


does this function belong in 'create_database.py'?


same with this one, probably doesn't belong in create_database.py


reverting in master


Observable never gets closed


This has been changed in the other branch, so I decided not to cause a conflict here


typo: Clent => Client


Maybe an alternate modification ?

```diff
diff --git a/gramps/gui/selectors/baseselector.py b/gramps/gui/selectors/baseselector.py
index 7ddaac3..fa558a7 100644
--- a/gramps/gui/selectors/baseselector.py
+++ b/gramps/gui/selectors/baseselector.py
@@ -273,7 +273,11 @@ class BaseSelector(ManagedWindow):
         """
         Builds the selection people see in the Selector
         """
-        filter_info = (False, self.search_bar.get_value(), False)
+
+        if not self.filter[1]:
+            filter_info = (False, self.search_bar.get_value(), False)
+        else:
+            filter_info = self.filter

         #set up cols the first time
         if self.setupcols :```

we need filter set on Selectors for reports and Family Editor.

```
File "Gramps/gramps/gui/selectors/baseselector.py", line 292, in build_tree
    search=filter_info)
  File "Gramps/gramps/gui/views/treemodels/peoplemodel.py", line 598, in __init__
    order=order, sort_map=sort_map)
  File "Gramps/gramps/gui/views/treemodels/treebasemodel.py", line 340, in __init__
    self.rebuild_data(self.current_filter, skip=skip)
  File "Gramps/gramps/gui/views/treemodels/treebasemodel.py", line 497, in rebuild_data
    self._build_data(self.current_filter, None, skip)
  File "Gramps/gramps/gui/views/treemodels/treebasemodel.py", line 566, in _rebuild_filter
    self.gen_cursor, self.map, self.add_row)
  File "Gramps/gramps/gui/views/treemodels/treebasemodel.py", line 599, in __rebuild_filter
    if not dfilter or dfilter.match(handle, self.db):
```

 ```
  File "Gramps/gramps/gui/plug/_guioptions.py", line 847, in __get_family_clicked
    filter=rfilter )
  File "Gramps/gramps/gui/selectors/baseselector.py", line 110, in __init__
    self.build_tree()
  File "Gramps/gramps/gui/selectors/baseselector.py", line 292, in build_tree
    search=filter_info)
  File "Gramps/gramps/gui/views/treemodels/familymodel.py", line 86, in __init__
    sort_map=sort_map)
  File "Gramps/gramps/gui/views/treemodels/flatbasemodel.py", line 488, in __init__
    self.rebuild_data()
  File "Gramps/gramps/gui/views/treemodels/flatbasemodel.py", line 618, in _rebuild_filter
    dlist = self.search.apply(self.db, allkeys, tupleind)
```


Task 2 returns 'none' but the file still appeared, appropriately re-named and containing t-words.
Task 3 only produces the z file, not others.


All look good, minor comments.

What value does filenameAtZeroIndex have over filename. In that block there is only one input and this only one filename.

Good catch on the mapper. I think a few other util classes exist that could use a similar exception in their private constructor. FfmpegUtils for example.


OK, I did not look for any other occurrences. Just noticed a few days ago that the FR24 stats link was not working and only found time today to try and find out why.


I'd add another function (with different name) to sure that its usage not marked


Any chance you could use the JsonBuilder here?


creating a https://pymotw.com/2/collections/namedtuple.html named tuple for User makes sense. These are simple objects that allow you to group data but not behavior (i.e., not full fledged objects).

Go ahead and refactor the parsing of the chunks out of this function, and have this function just take in an array of User named tuple objects. This function can then be decouple from the parsing.


This should be a regular member variable, not a pointer.


this is nice!


@ccx-testerbot runtests master


Thank you for the request. It has been added to the queue (id: 168). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/168) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


**FAILURE** - TeamCity HotRod C# Client :: Release Windows 2 [Build 13](http://ci.infinispan.org/viewLog.html?buildId=37281&buildTypeId=HotRodCSharp_ReleaseWindows2&guest=1) for 6ee9b91732e2f108349ff583e30b5c3bbfbad717
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:13:24

Process exited with code 1


Please re-add guessing of DOIs. This was wished by a user long time ago. I can search for the link if you want.


same here, pull parsing of these rows out into its own function that would return a named tuple called something like NationalityDateFormat with the two fields you need, nationality and date_format. Pass an array of that into this function to pull parsing out of here.


Maybe prefix the error string by `DocSearch`?


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


Same


need a better name here. and 3 is a magic number so you might want to use keyword parameters to make this code more readable, like

```
get_data(USERS_FILENAME, chunksize=3)
```


`saveRequired`?


恩。写好注释说明，让用户能够深谙: 这个是在View绘制的生命周期的layout上捕获同步调用的，为了避免掉帧，更何况layout的目的只应该是为了计算left、top、right、bottom，因此如果有任何耗时操作(i/o，复杂运算(cpu高消耗)，alloc object等) 建议抛出去做。


good job updating the docs!


interesting bug catch!


You are parsing the entire URL to then, maybe, retrieve the locale, why not check if the locale is in the URL before parsing potentially saving time. Or is the locale always there ? In which case you could go get that specific value rather than parsing everything.


Why this attribute has been removed ?


it doesn't create B :)


# 使用方式

> 布局：
****
`<com.pulltorefreshlibrary.FacePullToRefreshLayout
        android:id="@+id/face_pull_to_refresh"
        android:layout_width="match_parent"
        android:layout_height="match_parent">
        <TextView
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="Hello World!"/>
    </com.pulltorefreshlibrary.FacePullToRefreshLayout>`
代码：
****
`facePullToRefreshLayout.setRefreshListener(new FacePullToRefreshLayout.RefreshListener() {
            @Override
            public void onRefresh(PtrFrameLayout frame) {
                facePullToRefreshLayout.postDelayed(new Runnable() {
                    @Override
                    public void run() {
                        Log.e(Tag, "刷新...");
                        facePullToRefreshLayout.refreshComplete();
                    }
                }, 3000);
            }
        });`


use more keyword args to make these numbers less magical and this code more readable


Beautiful mess


What does that mean? - Maybe "the input paragraphs. Split based on paragraph separators"?


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/11/violations
```
plone/app/z3cform/inline_validation.py:15:1: C901 'InlineValidationView.__call__' is too complex (15)
plone/app/z3cform/widget.py:355:26: P301 format call provides unused index (1)
plone/app/z3cform/widget.py:604:35: P101 format string does contain unindexed parameters
plone/app/z3cform/widget.py:619:21: P201 format call uses to large index (1)
plone/app/z3cform/widget.py:619:21: P301 format call provides unused index (0)
plone/app/z3cform/wysiwyg/__init__.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Hello this could be reverted back when my pull request https://github.com/edjafarov/node-webkit-updater/pull/105 gets accepted as I already included this to default options. I think including this into default options can cause no harm just benefit.


This is 'logically' cohesive, they all 'get' something, but there are probably better ways to group functions. Also, this definitely needs a new name,

```
get.get_x(SOMETHING, 1, 2)
```
is super hard to read


TeamCity Civic Assembly :: Build frontend [Build default-frontend-10](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1789&buildTypeId=CivicAssembly_BuildFrontend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 76 (4 new), passed: 148 Build time: 00:08:43
### Failed tests
```
Basic Message Recipients CRUD Actions: Should Delete Message Recipient: <no details avaliable>

Basic Event Categories CRUD Actions: Should Delete an Event: <no details avaliable>

Basic Page Categories CRUD Actions: Should Delete an Page category: <no details avaliable>

Basic Static Content CRUD Actions: Should Delete an Page category: <no details avaliable>

Basic Subscription List CRUD Actions: Should Delete Subscription List: <no details avaliable>

Basic USER CRUD Actions: Should Delete USER: <no details avaliable>

Venue tests: Venue type: Should Not Add Venue if Cancel button pressed: <no details avaliable>

Venue tests: Venue type: Should Add Venue: <no details avaliable>

Venue tests: Venue type: Should Add an Venue and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Edit a Venue: <no details avaliable>

##### there are 66 more failed tests, see build details
```


Done :+1:


The content is split based on newlines not on paragraph separtors. Thefore, the variable "lines" would be more fitting, wouldn't it?


This shouldn't be in source, it's private information


Sneaky. It this something that needs to be forward ported?


We need to add an additional 60 minutes just as we do in the if case. For example, London is currently in UTC time zone and timeZoneMinutes = 0. However, 100000001 days from 1970 Jaunuary would be in BST time zone, thereby loosing one hour. Hence, this test would not cause an overflow. I think no special case is required based on timezones.


已实现大多数功能，还差用户购买功能的实现


感谢，最近改动比较大，已经发现了，还没来得及更新


## Task 1

Similar to your previous assignment, write a program to count word usage in Chapter 1 of The Origin of Species (you are free to use most/all of that your code to do similar tasks).  Have this program print out, in decreasing order (most common to least common) the 20 most common words and the count of each word used. Your program should contain a `main` loop and an `ifmain` statement, it should be formatted correctly, and you should **use `argparse`** to get the name of the input and output files from the user.  You may use any type of dictionary object in your code.  You should print your Top 20 results to the screen using the `.format()` string method, and your results should be prettily spaced (again, using `.format()`) so that the column of words and word counts are justified left, like so:

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get input and output files (1.0 pt)
- [ ] Program uses any sort of dictionary to correctly count words in the text and returns the correct words and their counts for the 20 most common words (3 pts.)

## Task 2

Write a program that uses `argparse` to get the name of the tab-delimited file you created above from the user.  Read that file into your program, and determine count of all words that start with the letter "t" (or "T").  Using the `os.path` module, write these t-starting words to a new tab-delimited file that has the same name as the input file, but with "T-words" prepended to the input filename.  For example, if the input filename is "all-word-counts.txt", then your output file of t-words should be "T-words-all-word-counts.txt".  You can accomplish these filename manipulations using the `os.path` module. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested file containing words that start with "T" and the count of these words (3 pts.)

## Task 3

Getting the t-words is all good, but your boss wants you to output a separate file for words that start with all letters of the alphabet.  So, modify your program from Task 2 to output 26 files, the names of which should begin with the appropriate letter of the alphabet (e.g. `A-words-all-word-counts.txt`, `B-words-all-word-counts.txt`, `C-words-all-word-counts.txt`) and the contents of which should be those words that start with the corresponding letter and the counts of those words in Darwin's text.  Format each file, as above, in tab-delimited format. Your program should contain a `main` loop and an `ifmain` statement, and it should be formatted correctly.

- [ ] Program is correctly formatted (0.5 pt)
- [ ] Program includes `main()` function and "ifmain" statement (0.5 pt)
- [ ] Program uses `argparse` to get the input filename (1.0 pt)
- [ ] Program correctly writes the requested files, names according to letters of the alphabet, where each file contains words that start with the corresponding alphabet letter, and the count of these words (3 pts.)


Side note, you should get a new key and refactor this out of the code


develop (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2330 of 2397 tests passed, 0 test warnings)

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


Refactor the username, nationality and birthdate validation/parse functions into a single function that returns one of your User named tuples. In this way, you should end up having *two* factory functions, one that generates User named tuples from a row in a file, and another that generates User named tuples from a row in a database.

User becomes your domain object, rather than having this concept 'smeared' across multiple parsing and validation functions for both files and databases.


Thank you man. Will check that out :)


I might phrase this differently...

"Using only the `http` module, create a server that listens on any port of your choosing that responds with "Hello, World!".


as always good assertions.


develop (energy-plus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1579 of 1579 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.85%25-green.svg)


[R/parse_data.R:6:1:](https://github.com/jimhester/covr/blob/bf38a63511bbe0ff0ebb689253b09ede78be02eb/R/parse_data.R#L6) *style:* **lines should not be more than 120 characters.**
```r
    warning("Parse data not found, coverage may be inaccurate. Try declaring a function in the last file of your R package.",
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
```
[R/parse_data.R:22:17:](https://github.com/jimhester/covr/blob/bf38a63511bbe0ff0ebb689253b09ede78be02eb/R/parse_data.R#L22) *style:* **Variable and function names should be all lowercase.**
```r
  original[[1]]$parseData <- parse_data[[1L]]
                ^~~~~~~~~
```


Maybe you already found out since it is fixed/changed in later commits but this commit really made my gyro flip out. Pulled up until this commit yesterday and unfortunately wasn't unable to test it properly but the gyro values was really all over the place.

Rolled back one commit and it worked OK, also pulled latest commit now and it works there.

Tested with a spracing board.


Nice!


AKA:

```python
for i, dump in enumerate(self.dumplist):
    if dump:
        ...
```


This commit will be made into the main branch to allow for readability and to conform to the text in the official PDF. Thank you for catching this for us!


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.2.18.9da1461a283ac460c84ca12bfe913cd8c8dfca18](http://build.palaso.org/viewLog.html?buildId=70199&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1045, ignored: 12 Build time: 00:06:24


how about:
* add default value to parameter in ctor
* add  parameter with default value in ctor


:ship:


Ah, thanks, rushing and copy/paste never end well...


I have changed delete implementation in my pull request https://github.com/edjafarov/node-webkit-updater/pull/105 as there were issue https://github.com/edjafarov/node-webkit-updater/issues/90 when user opens folder, run nw.exe and keep folder open as it will remain locked. Therefore implementation was changed to rmDir which recursively removes files and folders keeping root folder untouched.


@chdiza Could you please add this comment in the [“Files changed” tab](https://github.com/Homebrew/homebrew/pull/49905/files) of the PR instead? Comments that are attached to specific commits are unfortunately not as well integrated with the pull requests and are easily lost, once I push a code update to the PR. Thanks!


@liorkesos @rjVapes FYI this to npm 3 change breaks the postinstall process for packages, resulting in errors like:

```
Error: npm install failed { [Error: Refusing to install swagger as a dependency of itself] code: 'ENOSELF' }
```

It seems this error is due to packages being named the same as their folder name.


i'm not sure i understand why it's necessary to read RC from the pixhawk - is there hw flow control in place or something?


In the end I decided to undo this change. Instead, the parser will now return FALSE if the RTF couldn't be parsed due to it being malformed. Some people will run the RTF parser over normal text, and then - with this change - it would go into an infinite loop.


this probably belongs in create_database, and instead of data_object, you'd want to call this 'command line parameter'.

You may still have a use for this general 'get' pattern underneath some more clearly written parse/validation functions, or you may not, I'm not sure. But as written, having the name 'data_object' and 'position' everywhere makes the code look like copy-pasta, and these variable names aren't very informative.

I think your intuition is correct that there's probably some common pattern here that can be refactored, but it would take even me a few design approaches to get this right. Best for now to just let each of these functions be one offs that themselves are more clearly described than try and make it fit a pattern with data_object, position, assertions, re search, etc...


# Анатолій Мусь


wenn von addons rechte hinzukommen müssen die dann wieder den admin zugewiesen werden.
würde direkt die funktion hasParam ändern: https://github.com/cmsone/cmsone/blob/master/redaxo/include/classes/class.rex_login.inc.php#L49


http://localhost:8080/func?table=customers
http://localhost:8080/func?table=taxis
http://localhost:8080/func?table=orders

http://localhost:63342/etaxi/templates/index.html


Overall notes:

Dislike:

- New iconography doesn't match Material spec
- DI framework removed
- Sliding panels don't have the anchor state or icon in [the spec](https://www.google.com/design/spec/components/bottom-sheets.html#bottom-sheets-persistent-bottom-sheets)
- Duplicate API objects created everywhere, should be centralised into either a DI framework or singleton
- YouTube handling should probably be done through the [native player api](https://developers.google.com/youtube/android/player/#How_It_Works)

Like:

- Service looks good :+1:
- Based on media transports
- Totally fragment based


Yup! file-tree is the only provider to the side-bar currently, so if you disable the 'nuclide-file-tree' feature the side-bar will now collapse.


Figures: dimension et polygone(s)


command line args are already a list. The lower makes sense to just make your assumptions easier, and i do like the assertions, but this probably needs to either be scrapped and the resultant logic be placed in the three files, or better named and put in its own file in terms of handling command line args. It doesn't really belong here and may be 'too abstract'.


I'm not sure about it, but would it be better to have a way to run a Callable within "greedy" mode and remove this setter? Perhaps it would be possible to solve possible concurrency issues by synchronizing calls to that method? I guess it's ok to have the "isGreedy" method on the interface to find out the state.


you repeat this helper in multiple places, refactor it out so that each of these guys can import it and use the same one


develop (macumber) - 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-warnings-yellow.svg)


Technically speaking, you can't just reassign copyright, it must be passed from owner to grantee


Initial commit


Das stimmt. Ich würde jetzt direkt über all im Code auf isAdmin() prüfen wo das Perm auftaucht.


Task 1 is great.
Task 2 returns a file with only 'to' and 'that', both with the count of one.
![screen shot 2016-03-10 at 9 57 52 am](https://cloud.githubusercontent.com/assets/16668795/13675303/956347b4-e6a6-11e5-9116-365f2eaa333c.png)


This could be a directory, so you'd need to scan it recursively for the latest modification stamp


Congrats Stefan ! Another very strong and original contribution !

One day everything will be "deep-learning".
Meanwhile, there seems to be plenty of information and patterns in the search tree
which can be extracted on the fly and used for some kind of "position specific auto-learning"
as the game progress, and you found a new useful one.

It makes me think of another idea.
Maybe there is a way to feed this "learned history information" back to the evaluation functions.
For example, search "knows" what are the hot squares in the current position
and eval could use this to make psqt adjustments.


Yes. How do you check it? With

```
import pymysql
print(pymysql.__version__)
```
?


Hey. Your formula is doing pretty well so far


It would be good to also preserve the name of let bindings ([line 274 in Compile.hs](https://github.com/emilaxelsson/raw-feldspar/blob/master/src/Feldspar/Hardware/Compile.hs#L274). You can see how it's done in [Feldspar/Run/Compile.hs](https://github.com/emilaxelsson/raw-feldspar/blob/master/src/Feldspar/Run/Compile.hs)


Oder was hat es mit hasPerm genau aufsich?


**SUCCESS** - TeamCity HotRod C# Client :: Release Windows 2 [Build 14](http://ci.infinispan.org/viewLog.html?buildId=37285&buildTypeId=HotRodCSharp_ReleaseWindows2&guest=1) for 6ee9b91732e2f108349ff583e30b5c3bbfbad717
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:49


Merged, you can close this.


I don't think you need to reference this by `prototype` as the function can be pulled down directly.

You should only need prototype when working on an instantiated object ... I think :)


I like it. Compact and efficient :+1:


@mikeweaver Brilliant! I was wondering how you go ahold of that element in the first place.
I did a similar thing back in 2002 with ads on Geocities. I :monkey: 'ed with CSS to move them like 5000 pixels to the right.

So... was it a new bit of CSS they started injecting?
Or was some sneaky javascript building up parts of the page dynamically and you figured out where that was happening? I wasn't able to click "inspect element" on that stupid SIGNAL twilio ad.


True and in that case I should just create a Jar since it already does that for me.


Yes, I can totally replace this with a callable, or `Action`. It would introduce the creation of an anonymous inner class, but that's far from being the price of exceptions, so acceptable. I couldn't find any better name than `greedy` now. As for concurrency, I didn't care much about this since nothing in the project is thread-safe anyway.


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


Oops, when you explained the bug to me I didn't realize it was a site setting based thing. I reviewed the pre-vdom code and found it's actually just enabled if there is a summary, so I fixed it to work that way: 5bfc9cdf50fb08ff4b6aaafc10fc1d27e29851d7


@tumbili please remove the untested RC in part


left unattended for over 2 weeks (ask


It is on Aurora. But I'd say we treat only Firefox stable as the final product? (We don't have Firefox Developer Edition on mobile.)


Can’t handle the redirect in the same place [where we’re doing the rest of the routing](https://github.com/opengovfoundation/madison/commit/0b56dd6b15a7aff8178424de308ba94db8c14df0) because `location.href` in IE9 is asynchronous, so the redirect event happens *anyway* and you end up on the 404 page.  Just throwing in an alias for now.


TeamCity Glyssen :: Glyssen-master Continuous [Build 0.2.19.cf3ce239e033f3a65e275d114fa51e3ed9675b2e](http://build.palaso.org/viewLog.html?buildId=70200&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1045, ignored: 12 Build time: 00:06:18


'-m' message comes from a bad script (sigh). Actual text should have been:" VEGA uses the cmake VEGA_EXTRA_VERSION parameter in its version number."


@johnrees I definitely recommend trying out writing a Foxx backend for your Rails App :smile: Do all the graph stuff there.


PURE EPICNESS!

The scoring of moves is probably the most underrated and under-looked feature in modern Chess Engine.
There is surely going to be more ELO coming from this patch.  Just like we did with CMH.

Congrats!


wow. Congratulations!


Its work well now. Tks.


Awesome!


Yes you need mirage 2.7.0


task 3 works great. To comment on the above review, I think that you have to use the whole 'c-darwin-chapter-1.txt' as an input. if you use the output for task 1, then many of the files are empty bc there are no words that begin with those letters in the top 10!
![screen shot 2016-03-10 at 10 04 21 am](https://cloud.githubusercontent.com/assets/16668795/13675483/7aa4abe2-e6a7-11e5-8ad7-69e2535005e6.png)


This will also will give a very strong boost for Lazy-SMP.  As it is highly dependent on these stats.


Nice patch. :) Congratulations!


What is this your terminal history  ? Please Delete it.


Why is this called C-playground if it is C++?!!!


Task 1 is great.
Task 2 is good, but it only produced 'to' and 'that'. 'the' was not in the list, though it was in the output file from task 1.
![screen shot 2016-03-10 at 9 57 52 am](https://cloud.githubusercontent.com/assets/16668795/13675530/aa88e95e-e6a7-11e5-81b2-6609a235d5af.png)


Ratan, acho que o legal era mover o .js para esse repo, assim podemos apagar o app antigo, tem como você fazer isso e colocar o form para o final da pagina logo depois do texto "Se você está lendo esse texto..."


What is the different between this function and the `==` [operator for arrays](http://php.net/manual/en/language.operators.array.php)?


Fixes preview not working on Arch Linux, Atom 1.5.4


$canActivate and $routeConfig are perfectly valid properties for components. See the new [Component Router Developer Guide](https://docs.angularjs.org/guide/component-router#router-lifecycle-hooks). Is there any reason why you removed them?


Just noticed, you have to explicitly set the name to be an array for now.


"filename" which hides the field declared at line 34.
FfmpegUtils  already with private constructor


Chatted to Sheel. It's no longer in use. Feel free to delete custom/dhis2/views.py and the URL that gets here, @benrudolph, and I'll clean up the rest in a separate PR.


Now that being optional is gaining a bit of traction upstream, should we look at getting rid of _true_ (merge to forceoptional)?


So a lot of great work here, and I really appreciate the fast turn around. We're nearly to the finish line, and so you must be pretty excited and proud.

A few minor things to fix, but the more major one would be the get module. I am really happy you're experimenting with design, it *takes* experimentation to learn. But this module has a confusing name, the functions in it rely a lot on magic numbers, and it's not very cohesive to group all the 'I find the X in this list and validate it' functions together. Do you see why?

The magic numbers and the names of the functions themselves are particularly hard to understand - for instance, in get_birthdate is only used once with the magic numbers 2 and 3. Can't these be encoded into get_birthdate, simplifying its signature to only take a single object? I think this is a case of over abstraction, perhaps you saw a pattern with nationality and you thought birthdate might need to be parsed in multiple ways. But it doesn't, at least not right now, and adding that ability makes this a little hard to read.

Get_nationality is another case - here you use the logic twice. But what you *really* want to use twice is the validation, right? The rest, the actual position in the array of paramters, is what's abstracted, so think about this:

```
get_nationality(COMMAND_LINE, 2)
```

to this

```
make_nationality(COMMAND_LINE[2])
```

Those are more or less equivalent right? Only now the nationality generating function doesn't need to know/support taking in an array of paramters, and be passed in a magic number to help it out.

Check out this:
http://silkandspinach.net/2004/07/15/avoid-boolean-parameters/

They talk about booleans but the same is true of integers.

I'd like to see your validation for nationalities, dates, etc... all be in purely validation functions, and your actual array 'picking' logic outside of that.

Interestingly, in your first implementation, the use of objects seemed a little unneeded, but now we at least need named tuple objects to make the code clearer, both User and NationalityDateFormat named tuples would help a lot. This is usually a good sign - the more you implement good design, the more other good design approaches make sense.

Again wanted to thank you for getting this turned around so quick, we're really close to the finish line now so I don't want you at all to be discouraged. I appreciate your experimentation here and would never want to make you feel bad for trying. You'll need to experiment again and see if we can't get this a bit more readable :)

Let me know if you have any questions.


`message` -> `body` and there's a missing comma at the end of line 55


Why did you make this `auto` instead of just specifying the type? It was a very simple type.


Yes you are right, but single place for change is always better than change into multiple places.


Yes. The asset categories selector extends the asset tags selector, so it inherits the asset tags selector's _getPopup() function, which calls _initSearch(). However in the case of the asset categories selector that first call doesn't really do anything, it's only the second call that successfully adds the results tree to the entries node _if_ the nodes are finished expanding.

So to skip that first call I emptied out the asset categories selector's _initSearch() and moved it to the _initSearchAfterLoading() function so it could only call it once like it should. That was my strategy, anyway.


@jensens Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/12/violations
```
plone/app/z3cform/inline_validation.py:15:1: C901 'InlineValidationView.__call__' is too complex (15)
plone/app/z3cform/wysiwyg/__init__.py:0:1: I001 isort found changes, run it on the file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


My repo, my life :)


awesome, thanks for looking into it


知道了，感谢Jacks哥的耐心啊，这两天打扰你了，这两天跟着你学到了很多，虽然只是一些简单的代码，但是你提到了很多优化和细节方面的东西，受益匪浅！
写程序容易，写好程序不容易，以后要多花些功夫在把程序写好这方面上了😄


Yes, URL isn't a defined field. My mistake.


Removed from the current draft.


I created an issue #74 for this.


closes #16745


@heplesser I guess it's a matter of taste. I took it as an indication that it was an input, when it's only an output variable, and hunted for where the initialization was necessary. If the following chunks are wrapped up into a convenience function, it goes away altogether.


thx for the contribution


"and with disabled interrupts".


Creator methods for exceptions... beautiful! You got this from @ocramius, right?


Hm... Ok, sorry about the noise. I've been looking at [this](http://clang.llvm.org/docs/CrossCompilation.html), which is supposed to be for version 3.9. It uses `-target` there...


I know :) The focus of this spec has change to an API for creating views similar to ResourceViews, so extending dataset like this is no longer relevant.


Yes - it should be resources.


So, according to my experience, it is not necessary, but you can of course convert both, just to be on the safe side. I do not mind if my name is not on the commit, feel free to change it.


There is no index file available. Please check the [status](http://ccextractor.canihavesome.coffee/test/168) page, and get in touch with @canihavesomecoffee in case of an error!


Agreed. For now, I think this is sysadmin only, but could be expanded in the future to include Organization admins, for example.


this morning powered by not enough coffee :coffee:


thanks @vcunat, i didn't get it to compile yet.


I might be wrong but I had to change ".add_tos" to ".add_to" or i would get an exception


The problem is that spaces are allowed inside a DOI, so we cannot really guess a DOI...


dude?


why changing this?


TeamCity jonas-requests :: WindowsBuild [Build 25](https://build.neohq.net/viewLog.html?buildId=573873&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Happy Mult?


hey, your commit message needs a little work, see http://coala.readthedocs.org/en/latest/Getting_Involved/Writing_Good_Commits.html for more info


needs work


Shouldn't this be `<`, because thread_t::priority is actually the nice of the thread?


@bashor I don't get what you mean


Because deployment doesn't work properly in atlas-admin, this broke the styling


note to self: this is to be tested in CW13


Комментить - это просто!


Invariants: simtime & res are floats (then it works correctly and the same for py2 & py3)


Thanks! Just did an update on the other Pi and the links (FA,FR24, PF) work fine now. I wonder should we open them in new tab (target='_blank')? Otherwise you are navigating away from the portal and don't have the menu on top.


*yes* ._.


Workaround for starting Zeppelin


I'm confused ...
According to [1] zmq_ctx_destroy() is deprecated and should be replaced by zmq_ctx_shutdown() and zmq_ctx_term().

[1] http://api.zeromq.org/4-2:zmq-ctx-term


It was documented by @rosstuck in http://rosstuck.com/formatting-exception-messages/


:+1:

Seems like something we should add to https://github.com/loc-rdc/bagit-conformance-suite?


Possibly `getParam()`, but `getQueryVar()` is a bit long to me.


I'll see if I can get you something for the parallax scrolling on Android. Do you already have something?


Perfect


Did you read my comment on the last pull request? I think this package needs further fixing: https://github.com/Alexpux/MINGW-packages/pull/1108


very perfect


very very perfect


so perfect the perfect is very very very real


TeamCity jonas-requests :: WindowsBuild [Build 25](https://build.neohq.net/viewLog.html?buildId=573873&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **UNKNOWN**
Summary: Canceled Build time: 0:3:2


TeamCity jonas-requests :: WindowsBuild [Build 26](https://build.neohq.net/viewLog.html?buildId=573874&buildTypeId=JonasHaRequests_WindowsBuild) is now running


Too much perfectness, stop or you will get killed by a black hole


Can't we re-use the ESTree.SourceLocation? It saves code and we also don't have to convert each time (if it matches exactly).


*dies*


nice


probably. but how do you specify that the end result should have two entries in the metadata with different values?


yes


TeamCity jonas-requests :: WindowsBuild [Build 26](https://build.neohq.net/viewLog.html?buildId=573874&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 0:1:18


you should be dead


please put the `}` into the previous line :)
--> `"esversion": es_version}`


ikr


you're alive, you're immortal, you're


nobody


Please use the Open Source Registry instead of Docker Hub, see e.g. https://github.com/zalando/planb-provider/blob/master/Dockerfile


This implementation appears to also include some aspects of the "Tariff 2.0" method, dveloped in
Serina P, Riley I, Stewart A, James SL, et al. Improving performance of the Tariff Method for assigning causes of death to verbal autopsies. BMC Medicine. 2015 Dec 8; 13:291. http://www.biomedcentral.com/1741-7015/13/291


Now stop


TeamCity jonas-requests :: WindowsBuild [Build 27](https://build.neohq.net/viewLog.html?buildId=573880&buildTypeId=JonasHaRequests_WindowsBuild) is now running


remove return statement


add getter methods


Changing the check in `pytypes.h` seems to cause some problems with pure virtual functions (example12 fails). So that one seems important


@jamesgk could you please explain a bit why you needed to add this `componentFlags` argument?
I don't see it used in [ufo2ft/outlineOTF.py](https://github.com/googlei18n/ufo2ft/blob/223747585ae68570226aa199410cb525e3a2b0db/Lib/ufo2ft/outlineOTF.py#L805), for example.

Also, why you choose `0x4` as default value?

The specs say this value corresponds to bit number 2 (the third counting from 0) in the a GlyphComponent `flags`:  `ROUND_XY_TO_GRID`. This is described like this: "For the xy values if the preceding is true".
And by "preceding", they mean the bit 1, i.e. `ARGS_ARE_XY_VALUES`, which is defined like this: "If this is set, the arguments are xy values; otherwise, they are points."

So, as far as I understand, bit 2 (`ROUND_XY_TO_GRID`) is only meaningful if the preceding bit 1 (`ARGS_ARE_XY_VALUES`) is also set.

https://www.microsoft.com/typography/otspec/glyf.htm

/cc @behdad


I think the first step would simply be ensuring that the bag is correctly treated as valid. This seems like part of https://github.com/loc-rdc/bagit-conformance-suite/issues/4 where in addition to testing different implementations we need a way to compare correct metadata across implementations.


Doge is love. Doge is life.


@KatjaSchulz thanks for your input - for now, these interactions are not included in GloBI. This is easily changed using the configuration file in this repository.


`e.backtrace.split` looks suspicious. `Array.split` comes from ActiveSupport and there's no ActiveSupport in straight-server.


I tried this but doesn't worked.


pass these as params


Is this necessary? I was thinking of a config on Heroku where you can define the number of workers, and they'll take care of everything else.


lines 87-90 and lines 102-107 are identical, i would refactor them in a method like `void verifyEdgeUsage()`


Superclass introduced in 28082e3.


Code style: variable names use lower case with underscores.


okay


Yes, already fixed that :)


Congratulations.


You're right! My bad.


always return null password.


Might want to do that for sure.


TeamCity Todo :: Build [Build 60](http://109.103.226.38:8512/viewLog.html?buildId=38282&buildTypeId=Todo_Build) is now running


lgtm r+


TeamCity Todo :: Build [Build 60](http://109.103.226.38:8512/viewLog.html?buildId=38282&buildTypeId=Todo_Build) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:00:11


this is what we test on cowfish https://ci.openmicroscopy.org/view/DEV/job/OMERO-DEV-merge-integration-Python27/155/consoleFull


I see the instructions


布局：

       <com.pulltorefreshlibrary.FacePullToRefreshLayout
        android:id="@+id/face_pull_to_refresh"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <TextView
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="Hello World!"/>
    </com.pulltorefreshlibrary.FacePullToRefreshLayout>

代码：

        facePullToRefreshLayout = (FacePullToRefreshLayout) findViewById(R.id.face_pull_to_refresh);
        facePullToRefreshLayout.setLastUpdateTimeRelateObject(this);
        facePullToRefreshLayout.setRefreshListener(new FacePullToRefreshLayout.RefreshListener() {
            @Override
            public void onRefresh(PtrFrameLayout frame) {
                facePullToRefreshLayout.postDelayed(new Runnable() {
                    @Override
                    public void run() {
                        Log.e(Tag, "刷新...");
                        facePullToRefreshLayout.refreshComplete();
                    }
                }, 3000);
            }
        });


It is not just one file that was ignored. It is a whole folder.


Well, which one did you change? The one which I was referring to is on line 431, not 434


Not Pythonic, should be:
```python
if ip.dns_settings:
```


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


Perhaps the simplest (best?) way I can think of is to read the valid bag
and then write it somewhere else and make sure they are exactly the same?

On Thu, Mar 10, 2016 at 11:24 AM, Chris Adams <notifications@github.com>
wrote:

> I think the first step would simply be ensuring that the bag is correctly
> treated as valid. This seems like part of
> loc-rdc/bagit-conformance-suite#4
> <https://github.com/loc-rdc/bagit-conformance-suite/issues/4> where in
> addition to testing different implementations we need a way to compare
> correct metadata across implementations.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LibraryOfCongress/bagit-java/commit/474d511adfac9e05f3e37cdab5544e3236bf5500#commitcomment-16621398>
> .
>

--
~John Scancella


hello


delegate mapping of client entity to another method


review


please do not hurt the "open web" by adding non conventional tags like "  target='_top'" !!!


I see. It's a bit confusing as I would use an `init()` function for that case, but I see that such a function already exists. maybe just rename it to `setup(..)` or `initState(...)`?


TeamCity Selenium POM :: CI [Build 72](https://teamcity.sponte.uk/viewLog.html?buildId=1086&buildTypeId=SeleniumPom_Ci) is now running


change to all()


add in comment(constructor) all possible values of grantType


Forgot to add new files - doesn't work after the commit. Files are located on another machine, will add them in next commit.


TeamCity Selenium POM :: CI [Build 72](https://teamcity.sponte.uk/viewLog.html?buildId=1086&buildTypeId=SeleniumPom_Ci) outcome was **SUCCESS**
Summary: Tests passed: 46 Build time: 00:01:01


@ccx-testerbot runtests master


They're not valid properties for components in pure AngularJS. They're an extension added by the Component Router. People who don't use the Component Router will surely find these properties confusing (I do). This should go to a separate typings file for the Component Router.


HEAD commit has no message.

*GitCommitBear, severity NORMAL, section `commit`.*


This will error out if its unset which will break all existing data. Should I make unset mean version 1?


This broke things, badly. Due to the semicolon nothing actually gets written to the config file. This means that supervisor starts nginx once, and it goes into daemon mode and exits. Then supervisor tries to restart nginx FOREVER, but fails because the original daemon is still running. The site doesn't break because the first instance still keeps running, and if it dies then supervisor will work again once. However it makes the error log completely useless since it's just filled with nginx startup failures.

Also, the nginx user is incorrect (nobody), which makes rails logs unwritable (because they belong to www-data).


Do we want to use `String.format()` instead of `getString()` due to [this](https://github.com/wordpress-mobile/WordPress-Android/pull/2955)?


That could work - does the spec authoritatively state that the bag-info and manifest ordering should be preserved across that operation? That's only thing which I'd expect to get false-positives on for the cases where e.g. people read bag-info into a hash and the results are equivalent but ordered differently.


Thank you for the request. It has been added to the queue (id: 169). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/169) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


r


Thanks you!

Sent from my iPhone

> On Mar 10, 2016, at 1:37 AM, brunosmash <notifications@github.com> wrote:
>
> Done
>
> —
> Reply to this email directly or view it on GitHub.
>


Ah :hankey: - you're right


I recommend adding a comment stating what each of these params is.


Actually, even better, after you move this sorting to `post_parsing` you will have to sort the `checked` ordered dict. This  will map the name of the rule to the actual Rule. So we can actually implement the magic method required to compare them (`__cmp__` IIRC) in the base rule class:

```python
def __cmp__(self, other):
     return -cmp(self.priority, other.priority)
```

The the sorting can be done much simpler:

```python
from operator import itemgetter
sorted(self.checked.items(), key=itemgetter(1))
```


we want to keep this for students but add an extra check (should only be visible when `current_season.started`)


I think this is not `inception-v2` but `GoogLeNet` without the factorisation of the first layer, which was not even published.


HMM, @rafaelcn, you doesn't already change this at #11?


ok, I would vote for a renaming of all of those as I don't see the point in shortened (user-facing) method names


The `deployment_result = result.result()` is already a blocking call which wait for completion.
Actually, `create_or_update` returns a AzureOperationPoller instance https://github.com/Azure/autorest/blob/master/ClientRuntimes/Python/msrestazure/msrestazure/azure_operation.py#L474
Which contains three methods:
- wait : which wait for completion only
- result : which wait for completion + do a last get to return the expected object
- is_done : Is the operation done or not?

The two first can take a `timeout` parameter if you don't want to be blocked:


Ahh that explains it. Yes, changing on 431 seems ok


same here, `if current_season.applications_open?` should be changed to `if current_season.started?`


Opa, legal incluir o css no cabeçalho e o js ao fina na pagina (junto com os outros), ainda acho legal mover o form para o fim da página.

@Angelo, acho legal abrir um branch para ir integrando aos pouco.


Is idea utilities for the same issue available here? e.g. com.intellij.execution.util.ExecUtil


sorry, i miss


Ah, okay!


In the Java class GeographyPointValue, I defined EPSILON to be 1.0e-12.  This is an upper bound on how much a value might change after a round trip from (lat, lng) to (x, y, z) and back.

Is the value we're using in Java reasonable in your opinion?  Whatever we do here, we should be doing the same in Java.


Double checked, all includes are flat and in quoted form, which means the search starts from current directory; all should work fine as it is!


I am a nice comment


Does this need to be removed as it's covered by NispFrontendController?


Hi @mikecmpbll, I just stumbled upon this patch via https://github.com/influitive/apartment/issues/180 , and it seems to solve the issue we have using apartment with PostgreSQL database strategy together with sidekiq. Do you plan to get this merged into official apartment or why not?
Cheers, Frank


@pynicolas thanks! 8852875604f91db96d7cf8939870af3f0f5dc818


Seems weird to pass the same string resource for both zero and one. Perhaps we need a separate `getQuantityString()` that doesn't have a zero param?


Yes, order does need to be preserved as it is specified in the spec.


Incorrect package name when looking at directory structure?


@vilchik-elena Do we need that line and the last one of the method? Removing them has no impact on unit tests.


I could see the manifest files getting messed up that way since the order does not have to be preserved. But I would rather deal with that then try and specify metadata in some format for all test drivers


Got it!


Why don't we want to log in this case?


why extracted?


@gavinking I've been testing for and arguing against compatibility breaks, based on the idea that the universe of Ceylon modules shouldn't have to be recompiled every time there is a patch release.

Should I just stop? Is the plan that module compatibility will be invariant wrt Ceylon releases?

The specifics of any given change seem less important: very useful new features make compatibility breaks more severe, because... the new features will be used. Useless features OTOH break compatibility without reason.


shouldn't mirage check the ocaml version and complain if out to date?


Ah yeah, read that too. But saw it in your talk at php antwerpen


Ah i see, because we already call this several lines above.


потоки ввода-вывода должны возвращаться по ссылке - на уже существующие объекты - здесь вы возвращаете по значени (создаете копию). А куда будет вестись чтение-запись в дальнейшем?


Shortlog of HEAD commit is too long.

*GitCommitBear, severity NORMAL, section `commit`.*


[R/parse_data.R:33:31:](https://github.com/jimhester/covr/blob/67dce64b28b6a3b5a65bb70808a44bea676bd31e/R/parse_data.R#L33) *style:* **Variable and function names should be all lowercase.**
```r
  else if (!is.null(data <- x$parseData)) {
                              ^~~~~~~~~
```


@vIiRuS I suppose this is a typo, `remiDNers`


плохое название для геттеров-сеттеров. Есть такое соглашение (convention) - именовать геттеры-сеттеры для поля m_field так:

getField(), setField()

это называется модель Java Beans


。。最好是翻译好了再commit，这些历史价值不大，最好不要入库


Let me make it exactly the form as we want so that it is easier to read once we need to come back


Just use std::tolower, it would be ascii unless setlocale was called in the code, which would have broken stricmp as well.
If you want to avoid potential locale dependent issues, use std::tolower(char c, std::locale::classic) - but we never had an issue with stricmp so that's not an issue at this time.


Does the commit message has been written quickly too ? :trollface:


shouldn't these be slf4j?


Все поля должны быть только private


много лишних пустых строк


Não é estranho ter escrito 'item' quando é 1, e não ter quando tem mais que 1?


сокращение в названии приводит к недоумению, что это за поле

нужно m_mode


Please modify Copyright years 2013-2016


I have some parallax going on for the login screen. Are you referring to the list screen?


Agreed - and it'd be pretty easy to have a recommended comparison tool which knows which things should be treated as unordered sets.


Do you want `mirage` to do an `opam pull` and check if a newer version is available? That could be a good idea, I know `brew` is doing similar things. Or we can bake a mininal mirage version in the `config.ml`.


AFAIK it's not enough and for right solution we should read in loop


Not related to this line but to a previous line. Is there a way (if make sense, I don't have all the context) to know if pluginClassLoaderManager is related to the server plugin management system or the extensions plugins?


for the time being i'd recommend jspm@0.16.15


@madalinoprea how did this turn out. works well?


Fair enough


ok, I am updating the valid bags now. Should see a push in a few minutes


Hum, apparently there was a bug here - 0 can be selected, but I'm not sure what that means in the settings. I must replace this zero string id by something else.


Well, this is a backward-binary-compatible change.


iteration (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2328 of 2397 tests passed, 0 test warnings)

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


add missing inline here


Библиотека не используется.


Should be: `return singular if n == 1 else plural`


Thank you for the tests! so could this be joined in one single function, insertBinary(rootDir string, name string) ? and use it for both fleetd and fleetctl, yes it's minor thing but perhaps better ?


Thanks! I will read this new paper soon. I think the current implementation includes some of the updates at http://www.biomedcentral.com/content/supplementary/1741-7015-12-5-S7.pdf, which seems to be officially described as Tariff 2.0 here?


идиома RAII предполагает, что ресурс получаем в конструкторе класса-обертки (над ресурсом), а освобождаем в деструкторе класса-обертки. Ресурсом в данном случае является файл. Открывать его нужно в конструкторе, а не в методе write(). Подумайте, какие должны быть флаги mode.


M.getModuleIdentifier() returns the name of the module.
What happen when there is multiple private or internal values that must be renamed in the same module. Do they get the same identifier ? Does setName handle the collision ?

This seem either a bug or something that should be explained with a comment.


@bscharm - i see what you're doing here.  this approach might be useful in a handful of ways, what do you think if we completely hijack: `options.churros` namespace and anything under there is used for something and then removed?  thoughts?


Choice 1 is 1, 2, or 3: although this accepts anything other than 1 or 2 as if it's a three.
Choice 2 and 3 should be within the range of their hand. I'll add a check here on a later commit to return failure on bad choices and card choices.


Дважды описана функция, но аргументы не отличаются => не переопределение => нельзя.


These unilateral reverts are horribly demotivating to a variety of people. If you want to impose standards or guidelines, send out a notice to the group of committers about your expectations, but letting people think they have authority they don't with vague and then reverting their stuff just annoys people. You can see the negative reaction with the other couple of commits you reverted last week.


в вашей реализации файл не открывается в конструкторе. Не понятно, как может работать метод read()


A few things in this block.

1. we need to handle sets of intervals, not just numerics
2. we should handle single-element sets, which aren’t wrapped in Data.Set
3. the catchall case should _probably_ do `success(t.dataType)` (I _think_ that’s the name), so we don’t propagate a Const that really isn’t a const.


@pynicolas I've added unit test


Ruby uses snake notation in naming things. I.e. not `yearPasses`, but `year_passes`.


If I'm not wrong, the `==` operator checks for the order of elements as well (e.g. see http://sandbox.onlinephpfunctions.com/code/79dcedc4ee22ac8f9d413fcb4995d3c2cd796c86). In this function, we only check that the two arrays have the same elements, but they need not be in the same order.


Oh I see the descriptions have more details in the 2015 paper; that's very helpful, thanks!


Issues of how the commit got in aside, the text needs some revision.

"Endpoint auditing is configured using the registry on this machine. See the Particular documentation for details on how to address this with your version of NServiceBus."

Do we have any sort of standard around how we refer to the documentation? I'd be inclined to remove "Particular" from that sentence also, if it's not required.


if и прочие операторы всегда должны открывать фигурные скобки. Недопускается писать блоки без скобок, даже если они однострочные


It's probably worth making sure this is synced when people use the colour pickers above.


Yep. It's perfect now. Thanks a lot for the help.


Right now `lec` changes visibility of private or internal variables to global so the extracted code can still find them.  This change of visibility is done through ugly `sed` hacks.
I would be nice if the change of visibility could happen in this module after the rename. What do you think ?


все большие буквы - это нарушает общую стилистику


cc @cstrahan


I think you should keep the coding standard:

"if( !override_verbose )" should be "if (!override_verbose)" as the code un final line 1664 "if (dry_run)"


All `List`s will have to be recompiled against `1.2.2`.


Ok to make it better, I suggest you use a table where you have { filename, content, mode } then just loop and passe it to ioutil.WriteFile()


@nfeske did I get it right that this file is bogus?


Please update the date


не берите чужое решение, всегда пишите только собственное


Changed this to be false per @kennyp.
I liked keeping the way the query param is set though (query[:snapshot] = true if snapshot)


It's not oddit :stuck_out_tongue_closed_eyes: It's a bug https://github.com/dotnet/cli/issues/1764

The latest dotnet restores relative to the project rather than relative to the current directory


Use std:: prefix on tolower, C functions without the std:: prefix are deprecated in C++.


You probably also want planner and integration tests for this. That’ll help ensure that you handle the cases that we actually see, where Sets are not always treated as they should be.


cc @zimbatm


@goldsky very good, can you please update the Create meeting unit test to cover your changes please?


iteration (macumber) - 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-warnings-yellow.svg)


> All `List`s will have to be recompiled against `1.2.2`.

That is simply *not* correct.


Passing the absolute path should work correctly with old + new CLI. Does that sound right?


I'm asking why we set any directory at all?


I think it would be nice to have a minimal version encoded somewhere and also test it with travis. That way would make sure it does get updated or the build will fail.


You should really look at spacing. Use additional spaces before and after symbols: `puts 'I really love ' + aGoodThing + '!'`


Thanks, not sure how I missed this one.


Не совсем понятно, для чего к одному и тому же значению прибавлять дважды разные. Вероятно должно было быть так:
_x += a._x;
_iy += a._iy;


Because we use it to locate Sake \ xunit etc


How else will they get a bridge method for the new default method? Calling `mapElements()` will result in a no such method error.


Oh, because we those packages are only used by KoreBuild. Also, we want them in a known location and not be in the business of figuring out where the global packages are


Use newline to divide methods from each other.


Должно быть реализовано через +=, нельзя сложить два объекта класса не имея переопределенного оператора +. А в переопределении его, следовательно, никак нельзя этим пользоваться.


Та же ошибка, что и со сложением, но только тут даже нет переопределения -= в классе.


Та же ошибка, что и с вычитанием.


Before saving? I didn't hook that because doing so could generate way too much calls in the front (in case there were several lines, we'd need to read and repaint the input for each change). What do you think? Maybe adding an 'autofill' option at the end of the columns that sync this at once?


unnecessary new line


I was thinking something like the Google Play store is what you were after.