Another suggested refactoring:

```swift
public func <(lhs: Version, rhs: Version) -> Bool {
    let lhsComparators = [lhs.major, lhs.minor, lhs.patch]
    let rhsComparators = [rhs.major, rhs.minor, rhs.patch]

    guard lhsComparators == rhsComparators else {
        return lhsComparators.lexicographicalCompare(rhsComparators)
    }

    guard let lhsPrereleaseIdentifiers = lhs.prereleaseIdentifiers else {
        return false // non-prerelease lhs >= potentially prerelease rhs
    }

    guard let rhsPrereleaseIdentifiers = rhs.prereleaseIdentifiers else {
        return true // prerelease lhs < non-prerelease rhs
    }

    for (lhsPrereleaseIdentifier, rhsPrereleaseIdentifier) in zip(lhsPrereleaseIdentifiers, rhsPrereleaseIdentifiers) {
        guard lhsPrereleaseIdentifier != rhsPrereleaseIdentifier else {
            continue
        }

        switch (Int(lhsPrereleaseIdentifier), Int(rhsPrereleaseIdentifier)) {
        case let (lhsPrereleaseNumericIdentifier?, rhsPrereleaseNumericIdentifier?):
            // compare numeric prereleases
            return lhsPrereleaseNumericIdentifier < rhsPrereleaseNumericIdentifier
        case (nil, nil):
            // compare non-numeric prereleases
            return lhsPrereleaseIdentifier < rhsPrereleaseIdentifier
        case (_?, nil):
            // numeric prereleases < non-numeric prereleases
            return true
        default:
            return false
        }
    }

    return lhsPrereleaseIdentifiers.count < rhsPrereleaseIdentifiers.count
}
```


@sstephenson a Bats test file is not a Bash script. It is a Bats script. If you ran it through Bash, you would get this as expected:

    bash: @test: not found


LGTM


TeamCity LabKey :: LabkeyModules15.3 :: Git sqlserver 2014 [Build 66](https://teamcity.labkey.org/viewLog.html?buildId=291061&buildTypeId=LabkeyModules153_GitSqlserver2014) outcome was **SUCCESS**
Summary: Tests passed: 218 Build time: 00:24:19


Issue: https://github.com/Azure/azure-webjobs-sdk/issues/170


TeamCity pull requests :: netty [Build 845](https://garage.netty.io/teamcity/viewLog.html?buildId=3986&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 6230, ignored: 21 Build time: 00:46:55


Thanks for your thoughts. I disagree; I think `replaceObjectsInRange:withObjectsFromArray:` is less specific, as it allows for fewer parameters - you can't specify the range of the other array. To me, it makes more sense to keep it this way, but I'm open to hearing others' thoughts as well.


You are right. Feel free to change it for maybe I will.


It's not that it's a breaking change which is the issue for me. The issue really lies in that the correct version constraint was not set from the start, and this change is not really a a fix for it.


Router.route("/members", {
  name: "members",
  waitOn: function () {
    return [
      Meteor.subscribe("members", 50), //limit=50 corresponds to rowsPerPage: 50
      Meteor.subscribe("invitation"),
    ];
  }
});


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


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


FWIW, my change did allow the build report to be generated successfully. Though I haven't confirmed that testthat packages with failures have the logs included correctly.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9145](https://teamcity.mesosphere.io/viewLog.html?buildId=91073&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0, ignored: 205 Build time: 00:04:27
### Failed tests
```
null: Kubernetes e2e suite.Kubectl client Guestbook application should create and stop a working application [Conformance]: <no details avaliable>

```


Can you place anything behind manual.groundalgae.description= in the language file? I won't be able to fix this until late tomorrow


Hello, could you please clarify exactly which line you are saying contains an error?  When I check line 19 of domainchecker.tpl here all I'm seeing is `</div>`.  Thanks!


Test Autoincrement Max Values


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#KASm21wEQwWaZ0irmd-1OA


Test Table without Primary Key


s/which doesn't/that don't


I admit I got a bit carried away with the naming. Mostly just getting started with some cleanups while exploring usages of Constant / Literal.


#5174-Zone-Name-for-Condenser-Placement-in-DX-Coils-is-misusing-the-zone-name-as-a-node-name (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1983 of 1984 tests passed)

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


Hey this worked really seamlessly! Nice job!
One comment - when setting a brightness level,Echo seems to send an 'on' then a 'bright/dim level' judging by the debug output. Is that expected / modifiable? For Insteon at least it would be great if they were actually combined commands to only send the 'dim level' onto Indigo since otherwise it goes from off to full to dim.

Not a huge issue given the otherwise useful functionality but that would be the only downside I see so far.

Again, great job and thanks!


Not at all! I was just passing around on your code because i'm interested
in this deserialization thing
Thanks

Le ven. 11 déc. 2015 19:28, Chris Frohoff <notifications@github.com> a
écrit :

> Good catch, though out of curiosity does this actually cause an issue or
> error under certain conditions?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ltearno/ysoserial/commit/5b4b8a3320a934aa08e84554f2d841c3658c3292#commitcomment-14931843>
> .
>


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


Thanks for uploading it.  I was going to do it yesterday but I forgot.


I didn't remove any working feature.  I added debug output when the program uses unsupported commands instead of silently dropping them.  Anyway, I removed the unrelated changes from this branch since it's confusing the basic issue.


Add a comment about where these numbers come from.  It looks like it's flag+anchor combos, right?


Here the bug is the if clause. It should be ```
    if(request->getMegaTextChatList())```
I've fixed that in my code


I would recommend following any tips you find for speeding up the startup
process, then make the first frame of your animation match the splash
screen. Also make this full screen animation the first screen in your .kv
file. This way, it should be a seamless transition from the static splash
screen, such that it appears to suddenly start moving once the initial
loading is complete. Make sense?
On Thu, Dec 10, 2015 at 12:16 PM admica <notifications@github.com> wrote:

> Thank you. That worked like a charm.
>
> Can I animate the presplash or get python access to the presplash widget?
> I just want to take the static image and bounce it around or grow/shrink it
> while it's loading to show the user something while it's loading.
>
> Here's the app i'm working on for reference:
> https://play.google.com/store/apps/details?id=org.missingclues.missingclues
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/j-santander/python-for-android/commit/b394bb3ae13f874c8e7ab01c3668ee732eb8f146#commitcomment-14910180>
> .
>


Ah, clever.  That is much nicer


Should this be explicitly coerced to `boolean`? (Ie, is it possible that this option is unset / `undefined` on the Ace side?)


Victor, this is so awesome!


This one is needed to avoid 404 errors on pages that existed previously. I believe we should also have one for the missing query now. Can you confirm @clintongormley ?


BTW:

foam.build.BuildApp can generate an appcache manifest for you.


When does this actually get run?


缓存机如果空闲，则要检查上位机是否需要送板
@YoungZiyi


start


Poor Brian !!!
Now who is going to call him !!! ???


Upgrading to 3.1.0-dev, this is no longer an issue. Thank you for taking the time to address this!


This doesn’t say it’s a Bash script, full stop. It says it’s a Bash script with special syntax for defining test cases.


![](http://rghost.net/private/7hlJhw6vb/b3f891a04f778bf36c9b51265d70e84b/image.png)
А пустота справа это так и должно быть? Везде разной ширины, выглядит хреново. Может лучше туда перевод строки вставить? Или в одну строку как было.


Why did you add public???


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


Curious, Why the spaces?

Currently generates something like this:

    curl --compressed -H "Accept-Encoding: ' gzip, deflate '" -H "Connection: ' close '" ...

VS:

        StringBuilder sb = new StringBuilder();
        for (String h : request.getHeaders(header)) {
            if (sb.length() > 0) {
                sb.append(", ");
            }
            sb.append(h);
        }

Which generates:

    curl --compressed -H "Accept-Encoding: gzip, deflate" -H "Connection: close" ...


Awesome, thanks @curthard89 . This is rendering better than SVGKit now, I had some test SVGs that weren't correct there but they look spot-on in IJSVG now.


Thanks, good to know.


@Giuseppestirpe89 @Shambo1 been a pleasure guys. Well done. Really happy with the site


Drop the atomic_supported check for this case.


Judging system for UTPJudge project, this proJect will be connected


What exception are you expecting... rescue all the things makes me uncomfortable. Also, if this is an uncommon thing, let's log it happening


+1 :)


I think when the NVW runs, if you pick a quad, it doesn't end up with 'Manual' in it, at least I'm sure I've seen this before. LGTM though :+1:


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (1984 of 1984 tests passed)

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


No, I don't think so--the default is `true` so it seems unlikely that some mechanism would make it `undefined` (unless that's how ACE returns falsey values for options).


test


Thanks for the comment. You're right, that statement doesn't have an effect atm.


А это уже контролировать невозможно. Переводы строки браузер не понимает, а длину выбирает сам.


Now best is detected incorrectly for http://espn.go.com/video/clip?id=10365079.


Apache? (also in location-options.nix )


This looks good to me.  Is there any issue when this query field is requested for non-exnodes?


May be better "Упоминать пользователей"


Yeah I understand. That would be something for mod / standalone project authors to add in if they want something like that.
I'm not really making any gameplay related changes in this project.
Mostly just engine side stuff and cleaning up the baseline tree.


@ellismg doesn't the condition mean the library wasn't built at all then? how did it pass CI?


I've found it. It was there before and has just been copied to the Android case. Thanks!


Great point - `.rfc3339` will throw `ArgumentError` if the string passed is not an actual RFC3339 date string, but `#xpath` will return an empty String if the element cannot be found... so this was me avoiding some conditionals.  I can make it more obvious for sure!


Good suggestion. I will fix this when we accept Domen's changes. He asked to work as little as possible on resolwe-bio until then.


Thanks for reply. By more specific I mean what `replaceObjectsInRange(_:withObjectsFromArray:)` specifies `otherRange` for you. In other words it provides default value for it. However in Swift default value of parameter could be static only, hence overloaded methods.
Also redirecting it to `replaceObjectsInRange(range: NSRange, withObjectsFromArray otherArray: [AnyObject], range otherRange: NSRange)` allow us to avoid making copy of array in `otherArray.bridge().subarrayWithRange(otherRange)`.
All these thoughts motivated me to make proposal.


That did it


Very well. Perhaps some examples could be included in the docs (maybe in another PR though). I have been having fun cooking images -- here are four examples https://gist.github.com/rvl/c1ca657882a883195f49


This basically means that you should build against SDL2 for binaries targeting RPi. For everything else using SDL, we still prefer SDL1.2.


Yeah, though there are a few that actually test the interpreter specifically, or \*gasp* the integrated REPL.


Downside: merge conflicts for everybody.


Хотяя, судя по всему `
` работает как перевод строки.


How they became unsupported in your branch?
That's what I wonder most.


content-length isn't a mandatory header. Not sure whether this is OK.


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


Did you ever run sb16set.exe?


Feel free to move stuff around, merge conflicts don't faze me :-)


I was suspicious when you said you had no idea so I grepped for `"hi` and found it. I was trying to debug the relocation problem.


Well there's an example for `pullImage` and `buildImage`, I think that's enough and people will eventually write and share their own nix.

Isn't `runAsRoot` there working in your gist? I guess you need a shadow for that `nobody` to work.


Looks like the response will be empty for anything but exnodes.  I would suggest we only make `recfind` valid when querying exnodes and ignore it for everything else.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.4 [Build 31](https://teamcity.labkey.org/viewLog.html?buildId=291063&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 17, passed: 31 Build time: 01:11:49
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGroups: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilterPane: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

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

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

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

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

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


Довольно полезная функция! Вынеси её пожалуйста в отдельный класс и напиши для неё тесты. Супер!


Если только одна из этих переменных может быть `true`, то для однозначности лучше ввести одну переменную, например type типа enum  с значениями том, раздел, глава, параграф. Иначе потенциально возможна ситуация с неколькими true, а то что потенциально возможно иногда происходит реально :)


This is downloading the wrong package paths.
It is looking for
  https://dl.bintray.com/spinnaker/debians/spinnaker-clouddriver_1.36.0-4_all.deb
rather than
  https://dl.bintray.com/spinnaker/debians/pool/main/s/spinnaker-clouddriver/spinnaker-clouddriver_1.34.0-4_all.deb


ok


They didn't become unsupported.  There was no case to handle them before.  Now there is a case which prints a warning.
By the way, I moved the SB changes that were unnecessary for mpu401 to my 'blaster' branch.


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


Ya it does. I did that in my own fork.


Я бы назвал `ItemsRange`, это название автоматически приобразуется в название таблици БД `items_range`. За это отвечает бин `<bean id="namingStrategy" class="org.hibernate.cfg.ImprovedNamingStrategy"/>` в `src\main\resources\hibernate.xml`


why not:
lastIdentity.getAsJsonObject("Identities")


Need #undef REDUCE_LOCAL_DTYPE_CASE_CPLX


Buttons won't reset themself. (They should, but that takes until a random tick.)
But tapping them again unpowers them. So they currently work like levers. Also, pressure plates also work

> Am 11.12.2015 um 19:36 schrieb PocketKiller <notifications@github.com>:
>
> so.. levers and buttons are now supported and turning on redstone? :0
>
> —
> Reply to this email directly or view it on GitHub.
>


thanks for clarification. The RaspBian team is understaffed (well it's a team of two),
so I'll propose some `dpkg-vendor --is Raspbian && ` magic for `debian/rules` &
will see how to do for debian/control ...


Fix-allocation-for-feneSolarDelaySeq (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1448 of 1448 tests passed)

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


гит там итак стоит


Fine with me, but we could/should maybe also add a editorconfig file (http://editorconfig.org/).


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


So does ahk run in the background or does it hang waiting?


Have all the plotting up and running for a single R run now with RFP. We currently save a parameter (and by extension, trace, although it is hidden within parameter) and a mcmc object. With this graphing is possible with all the graphs currently produced by runRFPModel.R script.

Next goal:

Be able to produce all plots with these objects for the ROC model.


@VisualKei сделано.


Этот класс должен наследовать `org.ayfaar.app.model.UID` и там уже прописан `uri` как ID


commit


So you didn't remove the feature, but instead
print the false warning claiming it is unsupported?
Didn't became any clearer...


TeamCity LabKey :: Trunk :: Premium :: Git sqlserver 2014 [Build 103](https://teamcity.labkey.org/viewLog.html?buildId=291084&buildTypeId=LabkeyTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:34


bb


uri будет гененерироваться автоматически в зависимости от `code`


Naw, because ABC won't help you specify the return value / type of a method, just that it exists.


Let's move the initialization of this into `initialize` o.w. `@manifest_metadata.key?` calls in the accessor methods will fail trying to call `key?` on nil.


nn


Nopes I just derped. The _unchanged_ constraints (the posyconstraints, which should now be renamed gpconstraints) are at the front of the list


this does break compilation


I am not sure I understand why you'd want to drop unrelated callbacks:
it looks like it's a "first scheduled wins" strategy, which seems error prone as the implementations of Work are the ones that call scheduleFOO. If it's a bug to call scheduleX and scheduleY, we should abort when scheduleY is called


:crocodile:


This change is wrong. Please see: http://stackoverflow.com/questions/15479335/is-nul-and-null-interchangeable-in-c


Fonction nBitQuantificator (encore sale certes mais...) fonctionne


Should we run this only if `name` has changed? Same applies to sites.


В целом довольно структурный и понятный код, благодарю!


Why are we converting this to `DateTime` at all? We can use the value as is and update the file incase it changes.


Да нет же, продолжительность отдельно в первой строке, как главную инфу, а во второй всё остальное как вспомогательную.


if you define log demain before including sol-log.h you will not need to undef it.
take a look at sol-log.h


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1445 of 1445 tests passed)

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


Зачем? По-моему и так нормально.


precisely! :p this means that anything implementing OneEqConstraint (which is a constraint that has only a single inequality...yeah it needs a better name) needs to implement its own `sensitivities` method or it will be found wanting.


Why not `p.sub(x, 3)` while you're at it? :p


the removing of the line 13 does break compilation (tested on raspbian Jessie)


Can we eliminate "Submission#0" from the display? There is some value in displaying submission # but it diminishes quickly as it's not convenient to count the number of submissions. We should figure a better way to point the user to the actual spot where the problem is.


Для облегчения восприятия же. Или вообще попарно разбить: продолжительность, опубликовано/автор, просмотры.


Looks good. I am merging pulling request.


Fix it ;)


Yes, `sourceComments: true,` under the Sass task in the Gruntfile is causing this, and to be fair, it was a bloody nightmare trying to resolve merge conflicts last night because of it. So, I commented it out so it's there if someone else is doing solo development in the future and wants to use it. It will be merged soon.


@sersut I'm not sure I follow. You need to compare it to the mtime of the file on disk, which will be a `Time` object, and `Time` doesn't have a parse method for an RFC3339-formatted timestamp (which is what S3 provides). So, I'm converting the S3-provided LastModified time to a `Time` object so I can compare it with the local file's mtime.  Do you have another suggested approach?


@uglymunky, we miss you, friend! Great to see you peeking in. :)


Change (log2_proc <= (SHMEM_REDUCE_SYNC_SIZE - 2))


Кстати и висящую запятую убрать в конце строки.


Если только одна из этих переменных может быть true, то для однозначности лучше ввести одну переменную, например type типа enum с значениями том, раздел, глава, параграф. Иначе потенциально возможна ситуация с неколькими true, а то что потенциально возможно иногда происходит реально :)


The control of the coordinates starts in line 35.


@VisualKei сделал.


if this is false you should set mdata->timer = NULL, don't you?


you need to find a way to pass light from the pit into the test chamber and pass shadows from the test chamber into the pit (for bts with pits made up of nothing but cement walls)


`memchr()` is called twice here, why not just something like:

    end = memchr(start, '.', scanner->end - start);
    if (!end) {
        end = memchr(start, '[', scanner->end - start);
        if (!end)
            end = scanner->end;
    }


it should be negative


it should be negative.


0->1 - is there a chance this may change again? Anything programatic about it?


:+1: on avoiding the range allocation.


I've done that - see this [commit](https://github.com/BenVlodgi/BEE2.4/commit/5bc5d19ff06d90829d650a905e23e9180c6c6795)


it's a problem in _sort_formats and not in Ooyala extractor.
i already opened a pull request #7769 to solve this problem(it happen in other extractors as well).
this is how it will be sorted with the change in the pull request:
```
python __main__.py -F 'http://espn.go.com/video/clip?id=10365079'
WARNING: The program functionality for this site has been marked as broken, and will probably not work.
[ESPN] clip?id=10365079: Downloading webpage
[ESPN] 10365079: Downloading webpage
[OoyalaExternal] 10365079: Downloading JSON metadata
[OoyalaExternal] 10365079: Downloading mp4 JSON
[OoyalaExternal] 10365079: Downloading m3u8 JSON
[OoyalaExternal] FkYWtmazr6Ed8xmvILvKLWjd4QvYZpzG: Downloading m3u8 information
[OoyalaExternal] 10365079: Downloading hds JSON
[OoyalaExternal] FkYWtmazr6Ed8xmvILvKLWjd4QvYZpzG: Downloading SMIL file
[OoyalaExternal] 10365079: Downloading rtmp JSON
[info] Available formats for FkYWtmazr6Ed8xmvILvKLWjd4QvYZpzG:
format code  extension  resolution note
hls-meta     mp4        multiple   Quality selection URL
mp4          mp4        unknown
rtmp-280     flv        unknown     280k
hls-300      mp4        unknown     300k , avc1, mp4a
rtmp-580     flv        unknown     580k
hls-602      mp4        unknown     602k , avc1, mp4a
rtmp-1100    flv        unknown    1100k
hls-1152     mp4        unknown    1152k , avc1, mp4a
rtmp-1500    flv        unknown    1500k
hls-1610     mp4        unknown    1610k , avc1, mp4a
rtmp-2150    flv        unknown    2150k
hls-2305     mp4        unknown    2305k , avc1, mp4a
rtmp-3280    flv        unknown    3280k
hls-3515     mp4        unknown    3515k , avc1, mp4a
rtmp-4740    flv        unknown    4740k
hls-5090     mp4        unknown    5090k , avc1, mp4a  (best)
```


@asottile

In reference to #568 (and ex-pr #605) -- This is the only instance of `TaskUtils.getJobArgumentsForTaskId`.

I don't see a reason that it couldn't just use `job.arguments` from `BaseJob` instead to accomplish the same-ish task.

Also FYI: I'm working on a fix for #568 based off of your #605 work in [mcoffin:task-id#568](https://github.com/mcoffin/chronos/tree/taskid-args%23568)


ENOMEM? I think this should be EINVAL or something like that.


Technically these should be ORed. i.e.
```
!local_manifest_exists?(manifest) || (have_both_md5s_for?(manifest) && !manifest_md5_matches?(manifest)) || remote_manifest_newer?(manifest)
```

When we do this with if / else, we are not going to fetch if md5s match and if remote manifest is newer (most likely will impact things but...)


Во, теперь отлично, спасибо.


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1448 of 1448 tests passed)

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


Looks good.


The ownership graph seems weird:
there is no guarantee that being able to acquire a shared_ptr (lock) will keep the parent alive. Right now mParent is an actual reference that may become invalid?


@reedloden Thanks for reporting this. I couldn't find a new version where this fix was included. Is that correct?


using `t` here would be a bit more consistent as it's what is displayed


I don't super like the idea of depending on local time. Ideally I would want to persist the LastModifiedTime we get from S3 on disk and use it here so that we are resilient to system time changes / time zones / file copy operations during deploys, backups, restores, etc...


This was actually a pre-existing bug in our code that I happened to catch in my manual testing. [0] is the entire match `NAME: "Rack 0"...\nPID: R-IOSXRV9000-CH` but all we really want is the substring `R-IOSXRV9000-CH`. Since all of our platform comparisons regexp against this string rather than doing an equality check, this bug was silent most of the time (since it still contained the desired substring, just a bunch of crud besides) but it was a bug nontheless.


таким функциям обычно не место в классе данных. Ведь она же универсальная не зависяцая от значений полей этого класса, в ненешнем виде она будет присутсвовать в памяти для каждого объекта этого класса. Её лучше сделать статической для экономии памяти и вынести из этого класса для соблюдения принципа чёткой специализации классов. Данный класс предназначен для хранения данных, а обработка данных это уже ответсвенность других классов. Соблюдение условия чёткой специализации классов повышает структурность и понятность кода, что весьма важно при работе в комманде.


My comment was meant to be semi-funny :smirk:

> It may be the good time to switch JSON to v7 and change things, but please change all to be consistent there's too much half done things to add more over it.

That's exactly why I started this discussion. Because we have enough time now.

Actually I only wanted to add video stream selection. Maybe I should split that pr and merge the core stuff already.


Refer to ryelle/Foxhound#1, which also applies to this project


what's with the `this->` notation? Aren't they regular methods?


:trollface:


Thank you for help @SysVoid. I'm afraid I can't merge this PR because fix relies on internal php function. We don't use them for technical reasons. As you know web interface is handled by php-fpm server that is running under admin account.  Internal function file_exists() will fail against files owned by any other users if directory or file in not readable by everyone. Can you please relace it with v-list-fs-directory API call instead? Thanks


:+1: :pray:


I see the point of this, and can merge it for now, but in the long run I guess I need a different approach, because with this I cannot easily do a "second build" run to test incremental compilation and caching...

I'll need some extra command to prepare tools that I can switch off for second runs...

If you know of a way the tools could be shaded without actually compiling / testing that would be great. But I guess if you knew, you'd have done that.


After the (testharness.js) test function has completed, prior to the next test being executed.


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


we going to need


I'll check, thanks.


We'd better do that, true.


Ouch.


@donaldjarmstrong glad to help you save the time I took


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

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


Fair.


Yep, without https here's what I get:

`Font from origin 'http://fonts.gstatic.com' has been blocked from loading by Cross-Origin Resource Sharing policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:3000' is therefore not allowed access.`

My solution was to set https explicitly.


how about this test


PP Kolbrún, Tryggvi, Tinna


got it!


please we can stop about complaining about piracy and such? and going on topic


Cargamos la primera pregunta del cuestionario


When u see my website
On 11-Dec-2015 11:40 pm, "Parag Jain" <notifications@github.com> wrote:

> Good
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/vinitparak/School-WebSite/commit/4460f0b1ec8286309ddfcb50fa34d3df25044e28#commitcomment-14931542>
> .
>


ok


I don't think we should merge all exception formatting into a single method. We can provide that method as convenience, but there should be way to format each frame separately as well as API to filter uninteresting frames.

The csi repl used to display the error message in red and the stack trace in dark red. The intent was that stack trace is less important than the error message. It should be possible to keep that implementation.


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*The issue occurred in the lines 2 to 2.*

*This issue relates also to the following locations:*

 * File `/tmp/tmp32gtch_b/test.py`, line 4 to 4
```
        pass
```

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
-        pass
+    pass
```


Will add more here later...


I don't think I am getting these kinds of emails...I get emails when commits are made.  How does one revert a change that has already been merged into develop branch?


RIP
should've used my brain instead of mindlessly copying hydra's code


we can skip this method.


for the PositionState::pinMoveOpensCheck i did following comment under this commit:

-------------------------
"Not directly related to this patch, but anyway writing it here.

in the PositionState::pinMoveOpensCheck function we have:

if (_statePinInfo.rankPin.smallSlidingPiecePos != INVALID_SQUARE && _statePinInfo.rankPin.smallSlidingPiecePos != move.to) {
    if ((squareToBitboard[move.from] & _statePinInfo.rankPin.smallPinPos) && !(squareToBitboard[move.to] & _statePinInfo.rankPin.smallPinPos)) {
        return true;
    }
}
Most of the time condition "squareToBitboard[move.from] & _statePinInfo.rankPin.smallPinPos" will fail, so lets begin with this condition. Also if we check menioned condition first i think we don't need to check if (_statePinInfo.rankPin.smallSlidingPiecePos != INVALID_SQUARE) anymore, because if (squareToBitboard[move.from] & _statePinInfo.rankPin.smallPinPos) condition is true smallSlidingPiecePos is valid. Please take a look"

-------------


I don't think that this is a problem. Before the change it is also 0. Furthermore  if we show this dialog with a running player it is irrelevant (I guess). BTW. the is (or was) a bug when closing a video while this dialog is opened the dialog is not auto closed.


Al contestar la primera pregunta y hacer clic en "Enviar" nos lleva la página de show, ve de esta forma:
![image](https://cloud.githubusercontent.com/assets/10439040/11754067/379aa698-a049-11e5-909d-02d51a8205f1.png)


TeamCity LabKey :: Trunk :: Premium :: Git postgres 9.4 [Build 841](https://teamcity.labkey.org/viewLog.html?buildId=291082&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 24, passed: 223 Build time: 00:30:50
### Failed tests
```
org.labkey.test.tests.RedCapTest.testMergeDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testFailDuplicateNames: <no details avaliable>

org.labkey.test.tests.RedCapTest.testConfiguration: <no details avaliable>

org.labkey.test.tests.RedCapTest.testLongitudinal: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDataTypes: <no details avaliable>

org.labkey.test.tests.RedCapTest.testDateBased: <no details avaliable>

org.labkey.test.tests.RedCapTest.testSeparateDemographicProject: <no details avaliable>

org.labkey.test.tests.RedCapTest.testVisitBased: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testDatasetExport: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelection: <no details avaliable>

org.labkey.test.tests.DataFinderTest.testSelectingEmptyMeasure: <no details avaliable>

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


Oh hey—this is slick @kevinSuttle! Thanks!


develop (evanweaver) - i386-Windows-7-VisualStudio-12: OK (2339 of 2373 tests passed)

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


I don't think we should make ObjectFormatter public on Globals. We should only expose API that allows the user to tweak the printing options.


nit: "Invalid sql." instead of "invalid sql;" to be consistent with a previous log entry "Invalid sql. Unable to substitute".


LGTM!


getContentLength does not reading Content-Length header, it's okay.


Referências para o conteúdo original:
* `BDD na prática – parte 3 – Gherkin.pdf`
 - http://elemarjr.net/2012/04/12/bdd-na-prtica-parte-3-gherkin
* `Desenvolvimento Baseado em Testes (BDD + Cucumber).pdf`
 - http://pt.slideshare.net/edumendes/bdd-com-cucumber-14910137
* `Desenvolvimento orientado a comportamento (BDD) com Cucumber.pdf`
 - http://www.devmedia.com.br/desenvolvimento-orientado-a-comportamento-bdd-com-cucumber/33547


For `_after_attempts()` it might also be interesting to pass the `delay_since_first_attempt_ms`. It is a minimal effort and might be useful, though this will mean the callbacks have different method signature.


Before you merge this commit, another way to do this would be to have `hexo.config.server.port` default to `process.env.port`. Whatever way you want to cut it. I prefer this method, but to each their own.


Did you mean to put `do expressions` here?


Same here


not sure what you mean, this hasn't changed


@kaapstorm Does this all look right to you?  It basically mirrors the session datum in suite.xml.  Is it ever possible to have more than one usercase with the same user id?


TeamCity LabKey :: Sprint :: Community :: CDS postgres 9.4 [Build 6](https://teamcity.labkey.org/viewLog.html?buildId=291071&buildTypeId=LabKey_Sprint_2_LabkeyTrunkCommunity_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 12 (1 new), passed: 35 Build time: 01:08:51
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

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

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

org.labkey.test.tests.CDSTest.verifyCounts: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyAssaySummary: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudyProductsSearch: <no details avaliable>

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

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

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

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


Зачем ты импортишь в каждом файле variables.less, он у нас должен один раз загрузиться в `less/style.less` и всё. В остальных файлах переменные уже будут работать


Спрячь ключи в ENV


Is there a reason you wanted it to be `0`? Would doing `12` work so you don't have to change the logic above either?


ok, then how about shadows if say i place a piller right in front of a observation room and the pit is behind that pillar


This looks wrong


@nmelo this change will possibly break for people who are using Lifx White 800 globes.


@RPangrle @jasoncoolmax I don't have a strong opinion, but retrying at least once is a good idea.  The server folks told us that 500 errors are pretty much the only ones that we should explicitly retry.  The real question is how long to wait before retrying.  @RPangrle 1sec?  Maybe 1 retry?  Thoughts?


Cool project!


@hlinnaka @gcaragea @foyzur @karthijrk @entong

Some facts regarding the correctness of this feature. First, as I mentioned in my previous comment:
> On 10/12/15 22:10, Nikos Armenatzoglou wrote:
> A clarification first. In metadata versioning, we examine the counters (in order to purge metadata cache) after parser has acquired locks and we have processed all invalidation messages. Consequently, at the beginning of optimization  phase, we are sure that if any concurrent transaction has performed any changes, then mdcache has been purged and all objects in syscache are up-to-date.

Although planner may acquire some additional locks, it does the assumption that the root of the dependency tree has been locked already. Please read the comments in plancat.c: 178. Specifically, the comment states: " For each index, we get the same type of lock that the executor will need, and do not release it.  This saves a couple of trips to the  shared lock manager while not creating any real loss of concurrency, because no schema changes could be happening on the index while we hold lock on the parent rel, and neither lock type blocks any other kind of index operation. "

Finally, based on the same assumption, ORCA may not acquire any locks.

To sum up, since we examine the counters (in order to purge metadata cache) after parser has acquired locks and we have processed all invalidation messages, ORCA will always retrieve the latest version of an object. Consequently, there is no correctness issue.

Nikos


Ok, thank you. :) I was suspecting that already.


shouldn't this be `or`?


Ves, to me moti, da vsakic, ko pogledam kaj si spremenu sem mi najprej naspama 500 vrstic autogenerated xmlja in sele pol tvoja java


looks like rest api url's are created based on git. It is not working for bitbucket


Ce bi se dal popravt ime commita, bi ga popravu na neuspesno zehanje


The relevant test case is `(inexact? 1.1+3i)`


we should lock this down :)

because this will pull in the latest blindly :)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.928](http://teamcity.monogame.net/viewLog.html?buildId=12147&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **FAILURE**
Summary: Snapshot dependencies failed: 2 Build time: 00:00:00


Your password, dumbass.


we can just do `mocha tests/manifest-parser-tests.js`

npm will automatically resolve the .bin lookup for us :)

also this will fail on windows :dancer:


Is there a plan to release or branch sg for es 1.7.3?


names


you don't need this file


999999 を MAX の定数にして overlay はそれ -1 したほうがいいかなって思ったけどこれ css だった


If you move the closematch stuff out, this could just be *threshold*


… and we'd probably want developmentDependency="true". And version 2.1, since it's what xBehave is pulling in.


creating awesome_page.md @thehumantaurch


you were probably using rails scaffolding to generate these files. The best practice is to only commit the code that's absolutely needed. So we should remove all these view spec files.


Weren't you going to handle the exact match case here?


Delivered fix for bug #126


I'm getting super nitpicky now, but a failure here will emit a message containing "because Serialization constructor should be protected.".
Probably best to lowercase the initial letter in these "because" messages.


Copyright header


i think it's good!


Doesn't lowercasing cause some problems in some locales? /cc: @DavKean


#4994-ControllerWaterCoil-TemperatureAndHumidityRatio-control-active-when-it-shouldn't-be (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1967 of 1983 tests passed)

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


we should move params.require(:question).permit(:title, :body, :resolved) to its own method so that create/update could share this code

```
def create
   Question.new(question_params)
...
end

def question_params
  params.require(:question).permit(:title, :body, :resolved)
end
```


@m-kuhn thanks. Here's a gift in return, one more dynamic buffer symbology fill example (this time for markers):
![marker-dynamic-buffer](https://cloud.githubusercontent.com/assets/1728657/11734163/13a8b69c-9fe7-11e5-8bcc-6cadeee3c415.png)


nice password


Not sure. Later I also got the same error as Matt on the screencast tonight… Your fix works all the time, thanks!


XE가 웹환경이 아닌 콘솔 환경이나 테스트 환경에서는 이 파일이 여러번 실행될 수 있습니다.
여러번 실행될 때 오류가 발생하지 않도록 하기 위해 if구문이 필요합니다.


Shouldn't this be up with the rest of the actual code files?


Again, the following chunk code code could be extracted out to its own method and shared by create/update methods.
+    @sponsored_post.title = params[:sponsored_post][:title]
+    @sponsored_post.body = params[:sponsored_post][:body]
+    @sponsored_post.price = params[:sponsored_post][:price]

And you could use sponsored_post.update_attributes method in the update action


I'm sorry it broke your LD experience.

Your solution would be better, but also necessitates copious fine-grained fallback code to handle the cases where a driver 1) claims to support an extension but doesn't (and will either produce garbage or just crash if you try to use it, upsetting players) and 2) claims not to support an extension but does. Repeated for every GPU, from every vendor, for every unique extension queried by Spring. If you want to implement this, go right ahead.


comments should only have post_id as foreign key.

we would need to use polymorphism model if we want to make sponsored_posts commentable


you can find time to do this but not to come check out georgia? smh


Polished bf28b216a18041efc52cd429e3ee1ff4b4e452b6


LGTM


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


The while(true) part could be separated from the eventual node adding to make it clearer that the node addition *always* happens. You could call the while(true) part FindParentForNode() or something.


in python 3.5
```
python __main__.py 'https://tv.nrk.no/serie/skam/MSUB19120116/sesong-1/episode-1'
[NRKTV] MSUB19120116: Downloading webpage
[NRKTV] MSUB19120116: Downloading f4m manifest
[NRKTV] MSUB19120116: Downloading m3u8 information
[f4m] Downloading f4m manifest
Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
    youtube_dl.main()
  File "/home/amine/youtube-dl/youtube_dl/__init__.py", line 410, in main
    _real_main(argv)
  File "/home/amine/youtube-dl/youtube_dl/__init__.py", line 400, in _real_main
    retcode = ydl.download(all_urls)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 1675, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 674, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 720, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 1345, in process_video_result
    self.process_info(new_info)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 1607, in process_info
    success = dl(filename, info_dict)
  File "/home/amine/youtube-dl/youtube_dl/YoutubeDL.py", line 1549, in dl
    return fd.download(name, info)
  File "/home/amine/youtube-dl/youtube_dl/downloader/common.py", line 342, in download
    return self.real_download(filename, info_dict)
  File "/home/amine/youtube-dl/youtube_dl/downloader/f4m.py", line 295, in real_download
    manifest = fix_xml_ampersands(urlh.read()).strip()
  File "/home/amine/youtube-dl/youtube_dl/utils.py", line 1379, in fix_xml_ampersands
    xml_str)
  File "/usr/lib/python3.5/re.py", line 182, in sub
    return _compile(pattern, flags).sub(repl, string, count)
TypeError: cannot use a string pattern on a bytes-like object
```


super-nitpick: FWIW, I think the usings look prettier without the empty lines 6 and 8.


Yeah, you're right.


Should we more aggressively prevent the addition of 0-distance edges?


fixed b4b5cfbc93f1432279eea1d75185b7e1ba3120c0

I didn't apply formatter to this file because the result was ugly.
Unfortunately, `@formatter:off`  was not available via maven-plugin


fixed 979e9a4a55d616a126c172bbda1a71aa8a80992c


fixed 70be1281d0705412ed43e70ddd19cde247661a2d


It's not immediately obvious to me why this isn't necessary.


why?


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


I don't want to add logging dependency on this module.


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


fixed b95b38ba8aa9b2e7541f76b00cd1a221075c3d9c


Yes - the reviewer was confused - ignore the bar graph bit.

On Dec 10, 2015, at 4:29 PM, andkovuvic <notifications@github.com<mailto:notifications@github.com>> wrote:

Removing equation is easy. One of the review were asked to make it into a bar graph. I don't agree with this. It serves no purpose and misrepresents the data. Or I do not understand that comment.

I will remove the equations and re-create the figure one in figure 2 s. Is there anything else that I need to do to that graph or to any other graphs?

Sent from my iPhone, please excuse typing and T9 errors.

On Dec 10, 2015, at 4:08 PM, Andrea Piccinin <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>> wrote:

@andkov and @adkelly , those look to me like what we were looking for - thank you!!

I'm a bit distressed by the fact that:

a) there are quite a few scores of "zero", even at baseline when supposedly we have no dementia;

b) the plotted averages, particularly for Diabetes-no HTN seem almost outside the range of the plotted observed trajectories...

This is one of those things we can hope the reviewers do not react to, but it raises more questions in my mind about what is going on in these models.

Amanda, I don't want to hold up your re-submission, so just go ahead, but Andrey - we need to ensure that in the future these types of graphs (without the predicted of course; I am working on this with Raquel, Rebecca, and Jamie) are produced PRIOR to running any models.

Amanda - do we also need new versions of Figures 1 & 2, as well as a comparable Figure 3 for ELSA (or actually 3, rather than 1, graphs for MAP?). If so, Andrey - can you produce them without that equation that does not clearly relate to anything?

Oh! - argh! - and does Andrey need any new data/output in order to produce graphs that actually map onto your new analyses?!

Best,
Andrea

-----Original Message-----
From: Andriy V. Koval [mailto:notifications@github.com]
Sent: Thursday, December 10, 2015 3:30 PM
To: IALSA/Diabetes-Tri-Study-public <Diabetes-Tri-Study-public@noreply.github.com<mailto:Diabetes-Tri-Study-public@noreply.github.com><mailto:Diabetes-Tri-Study-public@noreply.github.com>>
Cc: Andrea Piccinin <piccinin@uvic.ca<mailto:piccinin@uvic.ca><mailto:piccinin@uvic.ca>>
Subject: [Diabetes-Tri-Study-public] updated graphs (3e5f6e2)

updated graphs

@ampiccinin and @adkelly, please see the newest graphs in the [Basic
Graph](https://github.com/IALSA/Diabetes-Tri-Study-public/blob/master/BasicGraphs.md)
report

---
Reply to this email directly or view it on GitHub:
https://github.com/IALSA/Diabetes-Tri-Study-public/commit/3e5f6e25f81e5717625d683d87b8c473451a3359

-
Reply to this email directly or view it on GitHub<https://github.com/IALSA/Diabetes-Tri-Study-public/commit/3e5f6e25f81e5717625d683d87b8c473451a3359#commitcomment-14914892>.

-
Reply to this email directly or view it on GitHub<https://github.com/IALSA/Diabetes-Tri-Study-public/commit/3e5f6e25f81e5717625d683d87b8c473451a3359#commitcomment-14915167>.


Version 1.0 by 2015/12/11


That's pretty tight. Interested in trying it out to see how it feels.


I'm actually not 100% this will work after Alan delivers ng-messages, but we'll deal with that later. What are our error conditions to trigger "$error", I don't see any so I don't think the message will ever show up.


fixed 7ac0810716d1de31678296c7a12be8d8faaab80b


@rwjblue any possibility of a new release? so we can get rid of all the lodash deprecation warnings in ember-cli?


I'll change the collection name to just 'users' as there are already references to that in the code.


I fixed the symmetries of the FKFilterGathers.jl code and added FKFilter.jl and a script for testing FKFilter "test_SeisFKFilter.jl".


thanks @pp- for the comments. updated those in the 66988bb5f4d8c976d4ddf94a71ed052b8ea72d27.


Consider; However, consider it if it contains the string the user typed


![qq 20151211102024](https://cloud.githubusercontent.com/assets/7963299/11734431/2b57d448-9ff2-11e5-956e-13949d892b0d.png)


We should add this to g3.mk if we want this to work; otherwise it's not included. Also, init.galbi.sh already has something similar to set the baseband, so that should probably be removed if you want to use this.


Curious, why is this happening before the check at lines 476-478 for consistency between clusters?


帮忙看下，我这个问题，build都没报错，MainActivity的路径也修改了，但是react-native run-android 的时候就报错。我用的是模拟器。感谢。


It's a bit of a weird API to accept "useThreshold" without allowing them to *change* the threshold. Assuming a threshold of 1/2 based on the use-case of this thing doesn't make much sense for something in Utilities.


Copyright


Please in the future, rebase rather than keep in these merge commits.


100 commits # yeah!


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

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


There's another help item in the menu that will need changing: ` <execute>zenity --text-info --title="About Bunsen Alternatives" --filename="/usr/share/bunsen/bunsen-docs/helpfile-bl-alternatives.txt" --width=900 --height=700</execute>`


Because the check for cluster equality is relaxed when in "build primary replicas only" mode, hence the state of the job must be established (by calling ```negotiateServerSettings()```) prior to calling ```allClustersEqual(clusterURLs)```.


Woops!


sam se 561526 nam jih manjka do Linux kernela


修复无法关闭SSH终端,造成后台进程一直在运行中,CPU会变高.


Yeah....


For safety concern, gpdb use AF_UNIX to create connection between QD and entry db. Meanwhile pg_getnameinfo_all()-->getnameinfo_unix() will set remote_host to a fixed value "[local]", so when entry db trying to setup interconnect with QD, it can't parse the remote hostname "[local]". that's why we adjust the hostname here.


In my test setter has to be there for parsing, but not getter.  So below I have added setter for @Key("Key")


Editor suggested changes


test


your patch was going to fail LTC
http://tests.stockfishchess.org/tests/view/5667294a0ebc592d552a3ecc

you can't change the test procedure in view of test results. that's cheating statistics.

and "proper LTC" has not yet been approved by Gary and Marco:
https://github.com/glinscott/fishtest/pull/75


:-1: Don't merge.  I am testing/improving the patch.


Desconsiderar esse commit. Estava apenas testando e não quero arriscar um commit para desfazer esse e desfazer todo o projeto. Rs


Not a comment on this change, but I'm unsure that it's a good idea that the set of restricted configs can be *reduced* in the config.


Got further clarification. Ivar's statement is correct for ML2 plugin pre-/post-commit APIs: the pre-commit is within a transaction, the post-commit is outside of it. The situation is different for the L3 plugin. These aren't formalized APIs for neutron, and exist in vendor-provided code as analogs to the ML2 APIs. In other words, you need to see how these are implemented to know whether or not they are being called within a transaction or not. In our case, they aren't -- for both the l3_apic.py and the cisco router service plugin APIs.


Nice update!


@mcgloflen heyo Haley, you can use Figaro to hide your API keys: https://github.com/laserlemon/figaro
Otherwise, looks great so far!


Ship It!


I was thinking on providing two callbacks, first for success, second for error.


Aha - there's a subtlety that deserves a comment.

On 11 December 2015 at 13:36, Felix GV <notifications@github.com> wrote:

> Because the check for cluster equality is relaxed when in "build primary
> replicas only" mode, hence the state of the job must be established (by
> calling negotiateServerSettings()) prior to calling
> allClustersEqual(clusterURLs).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/FelixGV/voldemort/commit/66ccea2359d33f4a05f8f18a466045e16fcdf2ce#commitcomment-14916621>
> .
>

--
Greg.


Yes just to be sure I did a "make clean" again and built successfully this time. :)
Now on to why the gcc5.2.0 is causing boot problems on my platform. I'm aware of the u-boot patch.


would be better, by the way, I feel we have some overlap between Auth and LiveData


Yeah... I'm not sure... I just wanted to make this functionality a bit more flexible, but IDK if that's a good idea.


Ok, I can add a comment.


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1445 of 1445 tests passed)

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


I am not "cheating" statistics... just trying to free up resources.
What makes you so sure it will fail?


Does this function need to return the error, so that we can handle the error as early as possible?


develop (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1448 of 1448 tests passed)

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


Is this NPC present in latest version? If so please get a screen of this HTML.


This is a test pull request.


only primitive types can be here


不要把 conflict 放上來啊！
記得推上來前檢查一下


這個也是


not the whole plugin, just two useless imports I forgot before committing from ide.


Remove check here. The caller already checks for cmd length greater than zero.


what happens if i jump or fall into the pit looking up at the ceiling before i hit the teleport?


Привет ворон недавно поставил твою тему awesome скрипт дает множество ошибок если можно помоги мне с настройкой моя страница в вк https://vk.com/id253775991


much cleaner. thanks


Yeah I was hunting this bug when I saw that my uc_close changes caused a silent segfault.


Maybe you made it work since I last looked at it a while ago.  I'll check again.  In any case, it doesn't have anything to do with mpu401.


Better.


Why do you wish to removed the additionalAddress param?


Hi, i agree to your chances. i believe it was a good idea! ztp


Lowest precedence and highest precedence determine how the operator interacts with other operators. But that is a good question to ask: should the pipeline operator be of high, or low precedence?

I'd like to keep it at 0 for now, but I will create a GitHub issue to discuss the matter.


Thanks, will fix this and the other one.


Lets be consistent and call it "container" or keep it as-is as "c".


This is good code... very modular and reusable. Good job


@TeamSpen210  - how will you go about placing instances in the sky box such as test chamber gibs and vac tubes etc without collisions etc?


Why do you want to get rid of comments?


Would it make sense to just define XCTMain() in XCTestCaseProvider on OS X, to avoid the bootstrap script having to manually filter out main.swift?


Is there an advantage to this being a macro? Or just a style thing?


TeamCity LabKey :: Sprint :: Premium :: Git sqlserver 2014 [Build 10](https://teamcity.labkey.org/viewLog.html?buildId=291090&buildTypeId=LabKey_Sprint_2_LabkeyPremiumTrunk_GitSqlserver2014) outcome was **SUCCESS**
Summary: Running Build time: 00:06:17


Will do

On Fri, Dec 11, 2015 at 2:10 PM adamvy <notifications@github.com> wrote:

> BTW:
>
> foam.build.BuildApp can generate an appcache manifest for you.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/foam-framework/foam/commit/7319ec433a4ff2921ef8abae50bd5d61449c9198#commitcomment-14932712>
> .
>


I wonder if it would work to use `int_to_color` here instead of `create_color`?


Why do you want to get rid of checking for error in packet parsing?


looks good


check out https://github.com/xbmc/xbmc/blob/master/xbmc/video/dialogs/GUIDialogAudioSubtitleSettings.cpp#L153
at least theoretically this gets into an endless loop because g_application.m_pPlayer->GetAudioStream() is never m_audioStream = static_cast<const CSettingInt*>(setting)->GetValue(). the setting never returns -1.
feels not very robust


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


TeamCity Chorus :: chorus-precise64-chorus-2.5 Continuous [Build 2.5.1001.bab9ad24dc6ac370cd6b01ef876ee8b815ba2fe8](http://build.palaso.org/viewLog.html?buildId=67221&buildTypeId=ChorusPrecise64v25Cont) outcome was **SUCCESS**
Summary: Tests passed: 862, ignored: 54 Build time: 00:16:44


Issue #1 still exists with this modification. The problem is that the array cannot be exploded and indexed in the same line. If you move the bracketed "[1]" down to line 246, it should do the trick.
i.e.
245:      $exploded = explode('_', $key,2);
246:      $replaced = str_replace(">","",$exploded[1]);


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


Oh, right, forgot about this. I'll fix it.


@ashnwew then unblock me???


r=jedmccaleb


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


It's a macro in other similar files. The idea is that when we have more of these functions, the code looks more uniform. Not super important though.


With which NDK version(s) and Android standalone toolchain(s) do you test this?
Which configure command do you use?

I've tried NDK "android-ndk-r10e", toolchain "x86_64-linux-android" and "./configure --host=x86_64-linux-android" on Linux. It detects getline() and libreadline.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 9155](https://teamcity.mesosphere.io/viewLog.html?buildId=91104&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:17


test


asd


You are awesome :+1: I wasn't sure why all the created factories has the same thing, now I know thanks a lot :)


As always, Suppress Warning are not recommended


replace empty file with `.gitignore`:
```
*
!.gitignore
```


That looks like it will be more effective. Do you have a machine to test this on?


We need this to be defined as it is called on line 42


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 67](https://teamcity.labkey.org/viewLog.html?buildId=291069&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 36 (3 new), passed: 255, muted: 2 Build time: 01:25:18
### Failed tests
```
org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudiesSearch: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

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

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

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

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

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

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

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

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

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


When you merge you get all changes from that branch.  We'll just have to fix it by hand, which I've done for the develop branch.


Why do you want to remove pseudo interface support?


saw approval from jedmccaleb
at https://github.com/MonsieurNicolas/stellar-core/commit/c45de85533e28eeb75c975a6ba0c86a8339d22b1


merging MonsieurNicolas/stellar-core/maintenanceStartup = c45de855 into auto


MonsieurNicolas/stellar-core/maintenanceStartup = c45de855 merged ok, testing candidate = ee2ebbb3


So I can access method Array#empty? like this (below)?

line = Queue.new
if line.queue.empty?
  blah blah
end


looks good, will merge. Next time please add the compare link to the previous reviewed commit.
https://github.com/xavier-fernandez/SmartGadget-Android/compare/6dbad3637bdef3a0052f6a7001f19608e2337ad9...e5f2e7e126fd12cf80ca8a3ed03e79a433e747ff


Do we really want to remove error checking here?


@serghey-rodin sure, I'll try to get around to it shortly. :)


Do we really want to remove error handling here?


LGTM :+1:


github y u not know this me?


@CliffordAnderson @EdWarga I need an opinion on FDM-032-033-000-034, line 135.  In this line the only change is from an enclosing em-dash in the lemma to parentheses in the witness. I used the ``<app>`` element for the entire line rather than having an ``<app>`` element for each piece of punctuation.  Does it make sense to do it the way that I've done?
:smiley:  Ramona


@mv003348 Check your github email settings, https://github.com/settings/emails, to see if they are going to a different address.


Ah yes what happened is I was basing nginx-container off debian then ripped debian out without testing before publishing the gist.

Actually now, without a base image, I can't get `shadowSetup` to work. The commands seem to succeed while building, but there is no `/etc/passwd` in the image. (see https://gist.github.com/rvl/c1ca657882a883195f49#file-test-docker-nix-L66)


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


let me know if you need any help with this HW... The psuedo code looks good and looks like it would have been a fun Choose your own adventure!


Changed the reset code to start at random locs.


CC @mistydemeo and @DomT4 who requested this.


Note the command line option had been changed (fixed with 15ee1bc837b83e3d76b89f056ff472782ecc2cc0).


You know what we need.  A pluralizer.....


This line can actually be removed as passport is being included on line 1.


avc: denied { execmem } for pid=206 comm="drmserver" scontext=u:r:drmserver:s0 tcontext=u:r:drmserver:s0 tclass=process permissive=0

avc: denied { read write } for pid=2174 comm="netmgrd" name="wake_lock" dev="sysfs" ino=57 scontext=u:r:netmgrd:s0 tcontext=u:object_r:sysfs_wake_lock:s0 tclass=file permissive=0

avc: denied { read write } for pid=2207 comm="thermald" name="wake_lock" dev="sysfs" ino=57 scontext=u:r:thermal-engine:s0 tcontext=u:object_r:sysfs_wake_lock:s0 tclass=file permissive=0


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

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


новый шаблон под сертификат


You are getting a null pointer exception because at some indexes, your chess board has null values yet you are still trying to execute a `.getType()` on a null object.


Note this has been changed.


Also, why is everything so static!? Everything is passed around statically which is really, really bad.


@brunowd a sua loja responde como se o módulo não estivesse instalado.

Uso no meu checkout customizado como abaixo.
A minha url que você pode checar é:

    http://casadois-server4you.wjcommerce.com.br/checkoutsimplificado/index/cep/numero/05586001

Meu **JS** ajax padrão é mais ou menos assim:

```javascript
   ajax = new Ajax.Request(url, {
            method : 'get',
            asynchronous: false,
            onSuccess: function(transport){
                obj = transport.responseJSON;
                resultado = obj.result;
                tipo = obj.value;
                txt = obj.txt;

                console.log(obj);

            },
            onComplete: function () {

            }
        });
```

Meu **Controller** fica assim:

```php
public function cepAction()
    {
        $numero = $this->getRequest()->getParam('numero');
        $numero = str_replace('-', '', $numero);

        if (preg_match('/^\d{8}$/', $numero)) {
            $CEP_URL = @file_get_contents("http://cep.republicavirtual.com.br/web_cep.php?cep=".urlencode($numero)."&formato=json");
            echo $CEP_URL;
        }
    }
```


Please keep links unbroken, if necessary on a separate line it it's too long.


Well, thinking it again. I think those are self explainable, don't you think?


Rimraf should be wrapped with:

return When.promise(function(resolve, reject) {


Naaahhh !!! :)


PS: maybe be a bit more careful on what your debug messages say. I don't really mind them of course but some, in the future, maybe would :grin:


And reject("Removed") instead of throw


Please keep original markup, I diff against `en` and such changes make it more work.


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


You need to reconsider your class design. You cannot leave it like this. A `Pawn` should represent a pawn, nothing more. Your `Pawn` contains a reference (static!) to the board!!


This needs to return Fs.unlink to chain the throw -> catch


some tests failed:
failure: https://travis-ci.org/stellar/stellar-core/builds/96347215


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*An issue with the file `test.py` was introduced.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,6 @@
 def main():
     pass
+
+
 def other():
         pass
```


The code does not comply to PEP8.

*Detected by PEP8Bear with severity 1 while running section `python`.*

*An issue with the file `test.py` was introduced.*

The issue can be fixed by applying the following patch:

```diff
--- a/test.py
+++ b/test.py
@@ -1,4 +1,4 @@
 def main():
     pass
 def other():
-        pass
+    pass
```


are you sure it's no problem if mConnectionsAdapter is null or should we do something about it, like calling initPreferencesList()? In other instances within the file you Log.e when it's null.


Wrong date (14:00:00)


Should we hook this function in verifyBaseDeviceUUIDFS()?

That function first verifies UUID of base device. And then probes fs of base device contidionally (if device.filesystem is not nil).

We could change it a bit and do Probefs() if devices.BaseDEviceFilesystem=="".

And then check if device.BaseDeviceFilesystem matches devies.filesystem or not and give warning.

And that way we can get rid of getBaseDeviceFilesystem() function you introduced and code will become even smaller.


This broke the oozie recipe. If you meant to delete it, it should also be deleted from the recipe.


用log把所有的print都替代掉


We can easily drop the type name if it's a submission.  I'm not sure about an alternate location though.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1977 of 1978 tests passed)

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


unack


The server should save the world when it is shut down gracefully.

If you're paranoid, I can add `--saveworld` to the stop and restart commands.


On 11 December 2015 at 15:13, Joseph Wright <notifications@github.com>
wrote:

> Does this need \numexpr or would \number work? I'm wondering about
> accidentally picking up + 10 or whatever following.
>

needs to be \numexpr so \the works (should have \relax I guess) .

Conversely though some of the \protected I added are wrong \pdfcreationdate
needs to expand in an edef

this gives same log for me in pdftex and luatex.
 "same" apart from fact that pdftexversion comes out as 40 or 140,
depending...

\ifx\directlua\undefined\else
\protected\def\pdftexversion     {\numexpr\pdffeedback version\relax}
\def\pdfcreationdate   {\pdffeedback creationdate}
\fi

\message{[\the\pdftexversion]}
\message{[\pdfcreationdate]}
\edef\x{(\pdftexversion)(\pdfcreationdate)}
\message{\meaning\x}

\bye


ack


@edersondisouza I will leave as @note The returned value is the number of seconds since 1970-01-01 00:00:00 +0000 (UTC)  ok?


I've asked @dsgouda to just use TargetsWindows=true for now in this condition in his PR https://github.com/dotnet/corefx/pull/4939.

@akoeplinger I think it passed the CI because the test project still built it unconditionally because of the ProjectReference, but this an interesting pivot to take into account as we are changing our builds for more combinations now.


This breaks our OAuth 2 gem as well

'omniauth-edmodo'


My bad, wrong commit.
I tried it like that in a previous version, but in the end, thought about doing it like the above.
I'll test the town name modification in the editor.


It's a userspace header, so no.


develop (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1448 of 1448 tests passed)

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


Hi there, I will look for the code over the weekend as I am currently away from home.


On 11-12-2015 18:27, Guilherme Íscaro wrote:
> @edersondisouza <https://github.com/edersondisouza> I will leave as
> @note <https://github.com/note> The returned value is the number of
> seconds since 1970-01-01 00:00:00 +0000 (UTC) ok?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cabelitos/soletta/commit/c9ffa3f09f1c5d6cddf10761a5ef6294b3641fe6#commitcomment-14934319>.
>
fair


yeah, gitmate works again :) I really missed it in the two days it was down...


any reason R.array.array_temp_unit isn't one of those bound constants? Not saying it should, unless it's indeed a mistake, otherwise mainly asking to understand.


I'm pretty sure we agree, but just to confirm: it should stay a single method on `ObjectFormatter`, but have more hooks on `CommonObjectFormatter`.

It seems weird that the "textify" API would have a notion of color.  I can imagine returning two strings - header and details - but that seems like over-specification of the behavior.


Inquiry Number: 21691 83749
Date : Fri , 11.12.2015
email : janackert@gmail.com
Please help


LGTM, Thanks!


One line to the rescue!  Thanks!


I think we probably need to discuss this as a group (@kevinh-ms) .


..same here and the R.array.array_season 4 lines up


Since the variable is local before the refactoring, any assignments to it aren't meant to propagate to the caller. Remember that parameters are passed `ByRef` by default in VBA, so not specifying an explicit `ByVal` in the signature could actually change the behavior. I think it should return..

    return "ByVal " + _targetDeclaration.IdentifierName + " As " + _targetDeclaration.AsTypeName;

(or use the `Tokens` static class' members instead of hard-coding the VBA code, but that doesn't matter much really)


veri tabanı bağlantısı için dosya dahil ediliyor


Really nice. This looks like a really excellent add-on feature.

Two ideas:
* From what I can tell, you're only supporting pulling from S3, which implies that the files exist in S3. If that's the case, I'd add to the README what the initial requirements are, namely that these 3 files need to be exist in the bucket. It's obvious, but a naive user may assume that if they don't exist, they'd be created.

* Perhaps add a note that boto needs to be present on the remote side & credentials or IAM profiles need to be set as well.


Yeah, with length < 2 is the same as no entries since there will always be a newline at the end.


This was not intentional.
LightInject failed and I did not notice (see https://github.com/seesharper/LightInject/issues/233).


@jodersky, I don't think it is needed. Probably leftover from before we had `build.properties`.


Hi Phil,

You fixed the gain scaling issue, but broke the integration time scaling :-)

Since you are now multiplying by chscale instead of dividing, you need
to flip the scale fractions in _integrationTime._DATA:

    _DATA = {
        #     time   bits     scale       max
        13:  [ 13.7, 0x00,  322.0/11.0,  5047],
        101: [101,   0x01,  322.0/81.0, 37177],
        402: [402,   0x02, 322.0/322.0, 65535],
    }

A good sanity check for this code is to set the sensor up so it's
getting around 100 lux (approximately what it sees at my desk, facing
toward the ceiling). That's in range for all combos of gain and
integration time. The code should return the same value for all combos -
lux is lux.

  - Chris


We don't use these atm as the approval process for Chocolatey is so long


`_wrapperWrapperFactory`? Really? :wink:

I've given quite a bit of thought about this wrapper thing - I think we don't need it anymore, since we're now able to completely mock the VBE API.


@dtenenba We definitely want to make this change, I wasn't really familiar with the `os.path.join()` in Python until recently, but it's the cross-platform way of doing path concatenation.  It should be done at all times, rather than adding `/`.


I THINK MY CLOCK IS OFF!


Added distro config for the installer too, i will handle this from now on.


I certainly do! I'll give it a shot and report back.


Ah yeah actually I forgot about that too - good to think about.


Done in 6bdb4a4eab0a2289177f0b7316eeedb6f2b358a1


I noticed that this option has a comment of

> // It should be used only with storage drivers providing strong consistency.

Just calling it out since we've had some consistency issues with NFS


see comments, good otherwise. Please post compare URL as comment to new commit when amending: e.g.
https://github.com/xavier-fernandez/SmartGadget-Android/commit/cb0ee5d964331d19e322fd35de9a83cc2f7bb385...newcommithash


This could throw if the add-on was uninstalled or otherwise removed:

Potentially unhandled rejection [3] Error: ENOENT, unlink 'loop-dev/extensions/loop@mozilla.org'

So probably just explicitly ignore: `.catch(function() {})`


inlining it if possible might make it clearer...


:+1 for the refactoring


javadoc?


`IPacker` тоже нужно `Mock`-нуть. Вообще, в идеале реальным у тебя должен остаться только тестируемый класс


1st module to build ?


Don't get me wrong, I don't insist on that change. But I can't see how that method should loop.


Check the test log again. The problem isn't the test cases, but the functionality of the class itself.
Expected: 0.1
but was: 1.0
I don't see any comparisons within the SetpointProfile class that would be an issue.


Ты уже проверяешь это в тестах на `FontCreator`. В этих тестах нужно проверить только то, что в `tag.Font` лежит то, что вернул mock `IFontCreator`-a


@starteleport понял


Wtf? Host key checking is in SSH for a reason. Use your own `ssh_config` if you need that. Don't set insecure defaults for people.


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


Oh loh. Don't listen to me. Saw 00000000001 and automatically assumed that, because it happened to me so much. Carry on then.


It didn't, but I have certainly seen this happen.  I have done:
```
        cmd = ['umount', lazy_option, path]
        if lazy:
            cmd.insert(-1, '--lazy')
```


as far as i can see - all files are in kernel/blah/blah/atmel
but no in out/blah/blah/KERNEL_OBJ/firmware/atmel - when starts building

Moved those files manually (via script) and i have "kernel" file in /out directory (compilation still in progress)


This override seems unnecessary.


Prior when Will made calls to it, the stdout/stderr was discarded. The errors messages are not really propogated up in this case.  Ill noodle on it


Если получится напрямую создавать `Statistic`, не прибегая к помощи `StatisticCalculator`, то станет читаемее, и уберется `_blackListLoader`


At least IRQ select and DMA select worked from day 1.


This is handling the case where openscap, docker, or some other process has the mount tied up.  We have observed with some regularity that openscap seems to have something tying that mount point up but where lsof shows nothing.  So we just implement a stepping timeout to monitor it.  We need to handle it this way because the deferred removal and deleted are *not* enabled by default in our docker configurations.


Nice one 👍🏻


Can you document what xmlSet means?


Тебе нужно проверить именно `Packer`, а не то, как работает с ним `CloudData`.


It would require a rework of code in other places but it's a good point.  I'll circle back with Dan on what he wants.


And how is it different from `setName`?


@marek-safar looks like this breaks a few compiler tests:

```
Loading ./../class/lib/build/mcs.exe ...
Loading verification data from `ver-il-net_4_x.xml' ...
gtest-230.cs...	KNOWN ISSUE (Compilation error)
gtest-fixedbuffer-01.cs...	IL REGRESSION: C: Int32 Test() (code size 368 -> 373)
gtest-fixedbuffer-03.cs...	IL REGRESSION: Bug.RedSkyTimeCode: Void CopyTo(Bug.xxx*) (code size 102 -> 110)
gtest-fixedbuffer-03.cs...	IL REGRESSION: Bug.RedSkyTimeCode: Void Convert(Bug.xxx*, UInt64*) (code size 94 -> 102)
gtest-fixedbuffer-05.cs...	IL REGRESSION: S2: Void Test() (code size 42 -> 43)
test-339.cs...	IL REGRESSION: C: Int32 Main() (code size 98 -> 101)
test-400.cs...	IL REGRESSION: X: Int32 TestPtrAdd() (code size 106 -> 107)
test-400.cs...	IL REGRESSION: X: Int32 TestPtrArithmetic() (code size 277 -> 281)
test-401.cs...	IL REGRESSION: X: Int32 Main() (code size 206 -> 210)
test-408.cs...	IL REGRESSION: T: Int32 Main() (code size 59 -> 60)
test-409.cs...	IL REGRESSION: X: Int32 Main() (code size 40 -> 41)
test-603.cs...	IL REGRESSION: ConsoleApplication1.Program: Int32 Main() (code size 65 -> 67)
test-643.cs...	REGRESSION (SUCCESS -> EXECUTION ERROR)
Wrong return code: 2
test-652.cs...	IL REGRESSION: C: Int32 Test() (code size 35 -> 37)
test-653.cs...	IL REGRESSION: C: Void Test2() (code size 33 -> 37)
test-653.cs...	IL REGRESSION: C: Void Test2(SByte, Int16, Int32, Int64, UInt64) (code size 40 -> 43)
test-654.cs...	IL REGRESSION: Program: Int32 Test(SByte*) (code size 48 -> 49)
test-670.cs...	IL REGRESSION: C: Void Main() (code size 15 -> 17)
test-688.cs...	IL REGRESSION: Test: Byte* GetFoo() (code size 15 -> 16)
test-688.cs...	IL REGRESSION: Test: Byte* GetFoo2() (code size 15 -> 16)
test-689.cs...	IL REGRESSION: Sample: Void Foo(Byte*, Byte*) (code size 43 -> 47)
test-709.cs...	IL REGRESSION: C: Void CopyTo(Int32, Char[], Int32, Int32) (code size 71 -> 73)
test-709.cs...	IL REGRESSION: C: Void CharCopy(Char*, Char*, Int32) (code size 36 -> 38)
test-740.cs...	IL REGRESSION: FixedTest: Int32 Main() (code size 361 -> 365)
test-793.cs...	IL REGRESSION: MonoPointerBugTest.Program: Int32 Main() (code size 60 -> 61)
test-808.cs...	IL REGRESSION: Program: Int32 Main() (code size 70 -> 72)
test-919.cs...	IL REGRESSION: Test: Void Inner(Char* ByRef, Char*) (code size 10 -> 11)
```


Does it make sense to write this instead?

  regInfo.insert(std::make_pair(xml ? "altname" : "alt-name", alternateName));


not sure what you mean here, are you wanting me to return both values and do the conditional check elsewhere?


hahahahahaha😂


`grind test` does more than just run the tests, right? Analyzer n` such?


Currently, I do not have it. I hope to build one and post here when done ..


This swap requires more changes, i.e. in `__repr__` and in the test.


Arrggg, Windows...

This package isn't particularly great at parsing globs either.


Does this work instead?

    regInfo.emplace("name", registerName);


Wrong map? (530)


いいね。クール。


Why is it invalid?


I like how you built a `std::map`. +1


It may be better to use `std::vector<std::pair<std::string, std::string>>`, since you never look up in the map.


TeamCity LabKey :: LabkeyRelease15.3 :: Git postgres 9.4 [Build 106](https://teamcity.labkey.org/viewLog.html?buildId=291088&buildTypeId=LabkeyRelease153_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 45 (21 new), passed: 242 Build time: 00:47:16
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGrid: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutAssays: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyHomePage: <no details avaliable>

org.labkey.test.tests.CDSTest.clickOnLearnAboutStudyItem: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyGridCheckerboarding: <no details avaliable>

org.labkey.test.tests.CDSTest.testLearnAboutStudies: <no details avaliable>

org.labkey.test.tests.CDSTest.verifyAssaySummary: <no details avaliable>

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

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

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


これってちゃんとHP減ります？


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1448 of 1448 tests passed)

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


Do we? I'd assume we'd want it the other way around, otherwise we can't ever rotate


Nit: don't we have `ds2::ToString` or something for this?


Why bother calling `quote` for a literal?


Did you try `Win32_PerfFormattedData_W3SVC_WebService ` instead ? It offers better performance but may require to add additional [counter types](https://github.com/DataDog/dd-agent/blob/master/checks/libs/wmi/counter_type.py)


This has affected me in Infernalis


Nit: join char and string literals?


Dead code?


Now that it is 2.8 compatible, I just might give it a try. I wonder how good it is compared to Configurable.


The problem was here. The sensor header directory was `${CMAKE_SOURCE_DIR}/include` (that is translated in `IDYNTREE_ROOT/include`) instead of `${CMAKE_CURRENT_SOURCE_DIR}/include` (that is translated in `IDYNTREE_ROOT/src/sensors/include`).


This comment is no longer necessary? It seems to apply to the `notyet`.


Not sure what you mean...
The AHK script starts running, and hangs while waiting for the VeraCrypt install to start. Once it starts, ahk catches it and automates the install. Once the last item in the ahk script has run, the script (and process) ends.


> My bad, wrong commit.

I'm confused. What?


nice!

\o/


Are you sure we should count `regno` from `1`? Intent may be clearer with a `for` loop instead:

  for (uint32_t regno = 1; onQueryRegisterInfo(session, regno - 1, info) == kSuccess; ++regno) {


I have no clue what this code is doing. dogscience


I have r10e-rc4 (64-bit), and an arm toolchain level 21.  I configured with
`LIBS="-lncurses -lm" ./configure --host="arm-linux-androideabi" --prefix="${SYSROOT}/usr" --enable-static --disable-shared`
, which is the same as in build_for_android.sh.   I believe if you do not provide the `--prefix` option, it will use the libraries available on your computer, including the gnu libraries that have getline().  I assume you also have a libreadline available to link to.

Runtime note: If `--enable-static --disable-shared` is not given, then configure will try to link dynamically.  I had some issues at runtime with the android linker when linked dynamically.  Static linking with ncurses and libreadline worked better for me.  When I tried dynamic linking before, I was using gcc which also tried to link with glibc, and was not available on my device.  If you want, I could try again with dynamic linking using the NDK compilers, which might fix that problem.


やはり減らなかったか。

なぜ減らなかったかわかります？

そして、せっかくメソッドを局所化したのにarrやturnというメソッドの外側の変数を使ってしまうのはメソッド設計としては良くない、という感覚があると良いです。


This is essentially the same method as in the parent class with a pre-process to sanitize the metric names. Do you think we can refactor here ?


+1 Awesome thanks for this. SHould have been in there all along. Shoud be merged.


Were you going to incorporate this into the https://github.com/csscomb/sublime-csscomb plugin. This is pure class btw.


As you can see, it's already #ifdef'ed for ``__GNUC__``, which means gcc or fully compatible compiler. Feel free to figure out way to filter out clang and submit patch.


Could you refactor this back to using `(read-grading-table schema-file)`? You could show an error on `#f`. Or we could refactor things to expose the exception if wanted.


Racket doesn't have one either -- it just walks the stack and puts source locations there.


I can't construct a real case either and don't want to block this. Go ahead.


It would be a typo, I originally went to add a string but removed it after the fact and did not notice the whitespace removal...


Wouldn't hurt to include a screenshot to make it enticing. It's just unbelievable how powerful an image can be. Definitely on my list of eligible menus.


The erroneous modification of the town name in my previous push wasn't supposed to be done that way (mistake in the list of files I commited).
The current version (after the commit Unicode vs MultiByte), with the GetWindowTextA, works alright. I was able to modify the name of a town in the editor and then play that same map with the modified name.


not sure I follow why this is needed.  It looks like the for loop below will return any relevant errors (or ignore them if they aren't relevant).  Why the extra `ErrBlobUnknown`?


I would start being ruthless (especially since we have this record in Github) and remove the commented out lines. It'll clear up a lot of visual cruft and put a real stake in the ground, saying: We're going this way.


Add a `boolean`. And explain why five is enough for now.


It just runs the tests. There is another task analyze.


Looks like it was already like that before, are we using this property to create a metric too ? (and not only a service check)


Tora does not load mod_tora at all, it instead provides a "mod_neko compatible" but emulated API since it does not run into Apache process.


9feed59 would be a good name for a startup...


This should be called `_submit_service_checks` instead ;)


Okay, I believe you, and it's in another branch now so you don't have to look at it.


Like a harmony of us singing LudumDare???


Or a techno-anarchist artist collective

On Friday, December 11, 2015, Marcus Povey <notifications@github.com> wrote:

> 9feed59
> <https://github.com/idno/Known/commit/9feed59f422407a6f3e16ea6d6d3fc26cf19ae72>
> would be a good name for a startup...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/idno/Known/commit/9feed59f422407a6f3e16ea6d6d3fc26cf19ae72#commitcomment-14935022>
> .
>

--
Ben Werdmuller
<http://goog_1933028737>
benwerd.com | werd.io

+1 (312) 488-9373


You may need a * after the / here. Not positive, would have to try.

Make sure you put in a delete for the old directory :)


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


Makes sense.


I did not mean to commit that, but that is good to know.


-------------------------------------------------Realistic Zombies--------------------------------------------------
* The zombies natural state is to wander around in a random direction chosen every 100 frames
* if humans come within range of the zombie, the zombie will chase the human unless...
* If the player comes within range of a zombie, the zombie will prefer the player over the humans.


<p> убрал. думал он визуально меняет, <label> добавил: 5872243


@jasoncomes I think this will be in ST plugin not so soon. CSSComb 4.0 should be released first and if CSSComb maintainer will have time then ST plugin will be updated.


Agreed


And there's basically one test function per load here?  If so, sounds great.


Tested and this properly excludes these files. I deleted both files and the directory should be gone now as well.


Thanks for the patch! :+1:


I often wonder if people truly do not see the larger picture or if it is the result of the setting


Does the logging in ibp_server auto-rotate?  We'll have to re-work some
of the logging functionality then to make sure it sets the correct
user/group permissions as root, then opens and continues logging as the
specified user.  This update is more than I bargained for...

On 12/11/2015 3:48 PM, Andrew Melo wrote:
> Do we? I'd assume we'd want it the other way around, otherwise we can't
> ever rotate
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/datalogistics/ibp_server/commit/45cca92071e8de29726a53ffd8dd7058a593e578#commitcomment-14934809>.
>


Members: Mollee, Andy, and Gavin.
Purpose: To provide people with way to make media.
Accomplishments: We made many HTML pages, each with separate purpose and various CSS pages to make the website presentable.
Special notes: Please note the vast amount of work(data) put into each page.


@tambry maybe it should've been `"userBlacklist": ["AlexAltea"]`


looks good


fixes 1395


will fix that in PT-2236 PR


Here you don't need a partial function per se, could you change it to pass parameters normally?


> for massive API change

oh boy. :D


I've seen something similar in [Scaloid](http://blog.scaloid.org/2015/08/scaloid-40-is-released.html). It seems to take into account the scope.


Same here.


I see. Thanks for maintaining this BTW.


commit


I could've sworn there is the normal SIGHUP functionality to support external logrotation, but I'm a bit hazy on it.

You're right, though, it's complicated. Conceptually, they seem like two different logical components. Perhaps breaking it up into two features will simplify it somewhat. I'm mostly concerned with keeping the existing functionality :)


Let's make it easier
```
services = instance.get('services', [])
```


This needs to be updated: https://github.com/dain/presto/commit/7340546413b25de7f4aa23871747b03be236a425
```java
Optional.of(ImmutableSet.copyOf(getBucketColumnHandles(handle.getTableId())))
```


Shouldn't the URL here be "whamcitylights.appspot.com"?
@keithkml


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


This one can go to `replumb.load` similarly with the other `-to-try`.
We can actually rename all of them in `file-paths-to-try`...or just `file-path-for`...


Отлично! Первый pull request готов!


I've fixed this.  'useThreshold' is no longer part of the exposed API.


Because in the loop below we'll simply have no children to iterate over.  i.e. firstChild and lastChild will be the same.  so the loop won't do anything.


Actually, this should go in the commit that uses a single split per bucket


That's essentially what TankDriveArmTeleopMode was in my code. Minus the telemetry...


We never generate `output_prod`: not on circle nor recommended locally. Run as is, it will always pass since it's not actually checking anything. If you want to do this, you should at least recommend that users generate this locally and ensure that circle is testing output_prod, not output_dev.


You mean with an exception?  If so, then i'm happy to do that.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1983 of 1984 tests passed)

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


Nice threading ;)


This will likely cause issues with `_get_wmi_sampler` method.


! !   ==> warum die doppelte Negation?


So you are rewriting the query to only extract the needed columns. It's a reasonable optimisation and will certainly shave off some millisecs, but is it OK wrt columns that appear in selection ranges?

Put differently, given a DDL with PK fields `a`, `b`, `c` and a non-PK field `d`, and a SELECT query featuring a 'where' clause which further limits the selection to some range of `d`, will that condition on `d` be observed if you only select `a` and `b`?


Change the error message a few lines down


what about other `armhf` devices ?


I think I prefer your previous version. We really want to allow rippled to run without any log at all?


nice! sieht aber noch nicht ganz optimal bei mir aus.. sah das bei dir besser aus?

![screenshot from 2015-12-11 22-16-27](https://cloud.githubusercontent.com/assets/7880552/11755760/da3ff15e-a054-11e5-8a65-3a324594cb11.png)


It's interesting to pass the host name only when it does not target the localhost (i.e != `localhost` or `.`). Otherwise you override the default `host` tag and can likely introduce errors.


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


typo: `beeing`


LGTM


Let's change `TODO:` to `FIXME:`.  (TODO is an enhancement.  FIXME is a potential future bug.)


#5252_TinDiscrepancyNewVRFModel (rongpengzhang) - x86_64-MacOS-10.9-clang: OK (1977 of 1978 tests passed)

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


The console log is not useful in production. In the case of debug_log not
existing (or not openable), the console log will still cause performance
problems.

On Fri, Dec 11, 2015 at 1:16 PM, seelabs <notifications@github.com> wrote:

> I think I prefer your previous version. We really want to allow rippled to
> run without any log at all?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mtrippled/rippled/commit/dfd34987a2da9edc9fb28ed5ea017690a52481b2#commitcomment-14935398>
> .
>


Is es nicht eqivalent zu:

return (StringUtils.equals(sections, refCompared.sections)) )
&& systemUnderTest != null && systemUnderTest.equalsTo(refCompared.systemUnderTest)
&& ..


`host` tag is missing here (also please see my comment about it above).