Also I think these changes can live safely within the existing skip, so I
will put them there.

Looks like the big change is to pass an Id to skip, to log it correctly in
teamcity, in console, and to increment a skip counter and print it at the
end

On Tue, Oct 13, 2015 at 3:33 PM, Chris Holt <lefthandedgoat@gmail.com>
wrote:

> Can you send it as a pull request?
>
> On Tue, Oct 13, 2015 at 3:21 PM, George Alatrash <notifications@github.com
> > wrote:
>
>> Hi Chris,
>>
>> I've added some changes to allow reporting ignored tests in TeamCity and
>> other reporters. Would you please review and apply these ASAP if you
>> approve them.
>>
>> diff --git a/src/canopy/reporters.fs b/src/canopy/reporters.fs
>> index 75da3dc..b013e39 100644
>> --- a/src/canopy/reporters.fs
>> +++ b/src/canopy/reporters.fs
>> @@ -6,6 +6,7 @@ open types
>>
>>  type IReporter =
>>     abstract member testStart : string -> unit
>> +   abstract member testIgnore : string -> unit
>>     abstract member pass : unit -> unit
>>     abstract member fail : Exception -> string -> byte [] -> string -> unit
>>     abstract member todo : unit -> unit
>> @@ -14,7 +15,7 @@ type IReporter =
>>     abstract member describe : string -> unit
>>     abstract member contextStart : string -> unit
>>     abstract member contextEnd : string -> unit
>> -   abstract member summary : int -> int -> int -> int -> unit
>> +   abstract member summary : int -> int -> int -> int -> int -> unit
>>     abstract member write : string -> unit
>>     abstract member suggestSelectors : string -> string list -> unit
>>     abstract member quit : unit -> unit
>> @@ -61,7 +62,7 @@ type ConsoleReporter() =
>>
>>          member this.contextEnd c = ()
>>
>> -        member this.summary minutes seconds passed failed =
>> +        member this.summary minutes seconds passed failed ignored =
>>              Console.WriteLine()
>>              Console.WriteLine("{0} minutes {1} seconds to execute", minutes, seconds)
>>              if failed = 0 then
>> @@ -72,6 +73,10 @@ type ConsoleReporter() =
>>                  Console.ForegroundColor <- ConsoleColor.Red
>>              Console.WriteLine("{0} failed", failed)
>>              Console.ResetColor()
>> +            if failed > 0 then
>> +                Console.ForegroundColor <- ConsoleColor.Yellow
>> +            Console.WriteLine("{0} ignored", ignored)
>> +            Console.ResetColor()
>>
>>          member this.write w = Console.WriteLine w
>>
>> @@ -80,6 +85,12 @@ type ConsoleReporter() =
>>              Console.WriteLine("Couldn't find any elements with selector '{0}', did you mean:", selector)
>>              suggestions |> List.iter (fun suggestion -> Console.WriteLine("\t{0}", suggestion))
>>              Console.ResetColor()
>> +
>> +        member this.testIgnore id =
>> +            Console.ForegroundColor <- ConsoleColor.Yellow
>> +            Console.WriteLine("Ignored Test: {0}", id)
>> +            Console.ResetColor()
>> +
>>          member this.testStart id =
>>              Console.ForegroundColor <- ConsoleColor.DarkCyan
>>              Console.WriteLine("Test: {0}", id)
>> @@ -123,7 +134,7 @@ type TeamCityReporter() =
>>
>>      interface IReporter with
>>          member this.pass () = consoleReporter.pass ()
>> -
>> +
>>          member this.fail ex id ss url =
>>              let mutable image = ""
>>              if not (Array.isEmpty ss) then
>> @@ -147,12 +158,14 @@ type TeamCityReporter() =
>>              teamcityReport (sprintf "testSuiteFinished name='%s'" (tcFriendlyMessage c))
>>              consoleReporter.contextEnd c
>>
>> -        member this.summary minutes seconds passed failed = consoleReporter.summary minutes seconds passed failed
>> +        member this.summary minutes seconds passed failed ignored = consoleReporter.summary minutes seconds passed failed ignored
>>
>>          member this.write w = consoleReporter.write w
>>
>>          member this.suggestSelectors selector suggestions = consoleReporter.suggestSelectors selector suggestions
>>
>> +        member this.testIgnore id = teamcityReport (sprintf "testIgnored name='%s'" (tcFriendlyMessage id))
>> +
>>          member this.testStart id = teamcityReport (sprintf "testStarted name='%s'" (tcFriendlyMessage id))
>>
>>          member this.testEnd id = teamcityReport (sprintf "testFinished name='%s'" (tcFriendlyMessage id))
>> @@ -246,8 +259,7 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>              consoleReporter.describe d
>>
>>          member this.contextStart c =
>> -            contextStopWatch.Reset()
>> -            contextStopWatch.Start()
>> +            testStopWatch.Restart()
>>              contexts <- c :: contexts
>>              context <- System.Web.HttpUtility.JavaScriptStringEncode(c)
>>              this.swallowedJS (sprintf "addContext('%s');" context)
>> @@ -260,9 +272,9 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>              this.swallowedJS (sprintf "addTimeToContext ('%s', '%im %is');" context ellapsed.Minutes ellapsed.Seconds)
>>              consoleReporter.contextEnd c
>>
>> -        member this.summary minutes seconds passed failed =
>> +        member this.summary minutes seconds passed failed ignored =
>>              this.swallowedJS (sprintf "setTotalTime ('%im %is');" minutes seconds)
>> -            consoleReporter.summary minutes seconds passed failed
>> +            consoleReporter.summary minutes seconds passed failed ignored
>>
>>          member this.write w =
>>              this.swallowedJS (sprintf "addMessageToTest ('%s', '%s');" context w)
>> @@ -271,9 +283,14 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>          member this.suggestSelectors selector suggestions =
>>              consoleReporter.suggestSelectors selector suggestions
>>
>> +        member this.testIgnore id =
>> +            testStopWatch.Restart()
>> +            test <- System.Web.HttpUtility.JavaScriptStringEncode(id)
>> +            this.swallowedJS (sprintf "addToContext ('%s', '%s');" context test)
>> +            consoleReporter.testIgnore id
>> +
>>          member this.testStart id =
>> -            testStopWatch.Reset()
>> -            testStopWatch.Start()
>> +            testStopWatch.Restart()
>>              test <- System.Web.HttpUtility.JavaScriptStringEncode(id)
>>              this.swallowedJS (sprintf "addToContext ('%s', '%s');" context test)
>>              consoleReporter.testStart id
>> diff --git a/src/canopy/runner.fs b/src/canopy/runner.fs
>> index de34732..b1d3d50 100644
>> --- a/src/canopy/runner.fs
>> +++ b/src/canopy/runner.fs
>> @@ -51,6 +51,7 @@ let ( &&&&& ) description f =
>>      let lastSuite = incrementLastTestSuite()
>>      lastSuite.Always <- lastSuite.Always @ [Test(description, f, lastSuite.TotalTestsCount)]
>>  let mutable passedCount = 0
>> +let mutable ignoredCount = 0
>>  let mutable failedCount = 0
>>  let mutable contextFailed = false
>>  let mutable failedContexts : string list = []
>> @@ -60,6 +61,10 @@ let pass () =
>>      passedCount <- passedCount + 1
>>      reporter.pass ()
>>
>> +let testIgnore id =
>> +    ignoredCount <- ignoredCount + 1
>> +    reporter.testIgnore id
>> +
>>  let fail (ex : Exception) id =
>>      if skipAllTestsOnFailure = true || skipRemainingTestsInContextOnFailure = true then skipNextTest <- true
>>      try
>> @@ -99,7 +104,7 @@ let run () =
>>              else if System.Object.ReferenceEquals(test.Func, skipped) then
>>                  reporter.skip ()
>>              else if skipNextTest = true then
>> -                reporter.skip ()
>> +                testIgnore test.Id
>>              else
>>                  try
>>                      try
>> @@ -157,7 +162,7 @@ let run () =
>>      history.save failedContexts
>>
>>      stopWatch.Stop()
>> -    reporter.summary stopWatch.Elapsed.Minutes stopWatch.Elapsed.Seconds passedCount failedCount
>> +    reporter.summary stopWatch.Elapsed.Minutes stopWatch.Elapsed.Seconds passedCount failedCount ignoredCount
>>      reporter.suiteEnd()
>>
>>  let runFor browsers =
>>
>> P.S: I tried to attach this diff as a file but GITHUB kept telling me Something
>> went really wrong, and we can’t process that file., so I pasted above
>> instead.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/lefthandedgoat/canopy/commit/a5705ca997e2704d2969ed1191f857bae3050cfc#commitcomment-13751224>
>> .
>>
>
>


YuiCompressor has had its time.


That was a desperate move to disable warnings reported by @houndci, because rubocop wasn't making such warnings at all.


Deployment config


Special thanks go to @coolwanglu, @anthraxx, @calculuswhiz, @fizzfaldt, @elfring and @cheukyin699! Thank you very much for your contributions and making this release possible! :clap:


deployment config


@ukernel take a look?


and all the other places in strings


`(*y)*256*256+ (*cb)*256 + (*cr)`


I updated the code.


`(*y)*256 + (*cb)`


Project Group Members:
Rudy Gonzalez
Brad Parker
Darnel Clayton


<img width="1532" alt="screen shot 2015-10-13 at 4 29 13 pm" src="https://cloud.githubusercontent.com/assets/9892704/10467610/b608c4a6-71c8-11e5-8f81-ddb89360cd93.png">


Also changed motion update interval in PlaySoundsVC


TeamCity Bender :: Deploy [Build 3.0.28.0](http://build.mikeobrien.net/viewLog.html?buildId=2477&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3268, ignored: 5 Build time: 00:01:34


Disable ssl verification


please post the link here if you have posted.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 712](https://teamcity.mesosphere.io/viewLog.html?buildId=58182&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:08:37


Please Mr. Kevin this code when i run it , it does not run yet it is not showing me any more errors
What can i do please


Fixed thanks. I squashed the fix into the pull request. github confuses me, so I'm not sure what exactly happens now that I force-pushed the branch but at least it's fixed :)


I'll split these out to separate files once they're somewhat finalized.


@dcramer Babel does support this http://babeljs.io/docs/usage/experimental/#stage-2 You need to enable `es7.objectRestSpread`


**valid-jsdoc**: Missing JSDoc @returns for function.


**Eslint**: 'return' outside of function


**new-cap**: A function with a name starting with an uppercase letter should only be used as a constructor.


**semi**: Missing semicolon.


Nathan, please add this to the logger while you are working on updating the print statements.  If you find other messages that are not logged, please add them to the logger as well.  Thanks!


This breaks 'include' expansion


Push to your oduacm.github.io gh-pages so I can see if the events look better this way


remove these spans from a previous commit


moglo se i ime sobe staviti u kriterij pretrage pa samo provjeriti je li lista prazna


At last you increased the character limit. :)


Okey sorry I'm tired, I was looking in wrong file. It turns out that `searchEngine`is missing in the template context since it is not added in language/index.js

Ill do a PR ritght away


why gut all this?


Das ist sowieso alt. Es gibt kein ioBroker.nodejs


Nah. Originally it was the split between clr and application. I don't plan to change that in this refactoring.


Fix these things and you're good to go! Also, I see you cropped the images but was wondering why the are scaled down a little?


Will do. This can also be a concrete class since were still injecting discrete interfaces


Yes! in addition, the canvas should increase in order to preserve all pixels. These features will be added soon by @Fabian0007


@rcrowder - Why is this necessary?


TeamCity pull requests :: netty [Build 406](https://garage.netty.io/teamcity/viewLog.html?buildId=2266&buildTypeId=pulls_netty&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 5625, ignored: 21 Build time: 00:42:16


:grimacing:


yeah we eventually figured that out


@kubark42 Excellent, should have checked first. :D


This can be deferred:

defer func() {
code...
}()


haha great find!

Yes, it does look incomplete ;)


Sure, here it is: https://forums.aws.amazon.com/thread.jspa?threadID=217748&tstart=0


It should be "_eventHandlers", not "_handlers"


http://www.wowhead.com/npc=12665 there is no npc entry 12665.


Victor says @IgorMinar said these tasks are okay.


The *typical* case for development is to run `tox -e py27` (or whatever version you want to test with), which calls setup.py on a possibly-dirty tree. Thus, there needs to be a way to say "yes, I know this is dirty".


Не нада писати кожний раз метод коли ми хочемо щось відправити юзеру. Користуйся тими які вже є там.
sendMessage(String email, String html)


Fixed


Oh that's to skip out on deleting the tmdata container?  Maybe we should add that back then.


@tdhopper you can do the same and now be part of the official commiters...think about it.


Discussed with @amsagoff and we decided to disable the popover for case properties until we have support for `#case_type/property` special syntax.


You merged without testing? wtf art :p


NEWLINE


you can further destructure this
```javascript
var { runnables, className, ...props }  = this.props
```
Then render the chart below using `<div {...props} className ...`. This is a just a nice-to-know comment, I don't think it makes much difference for <Chart />

className has to be treated specially with destructuring since we merge it. There's probably a nicer way to do it than the boilerplate below, but I never looked into it.


Bonkers, I had my nice pin out diagram trashed...


If people run `git add` on the submodule pointer, we have the "long-standing PR" problem (also, remember that when merge commits are made in typeshed, the hash will change anyway).

It's better to let the submodule pointer follow typeshed's master, i.e. always be ahead of the commit mypy points to.


@lkpttn  I've built it in iOS so we can dynamically change the text based on whether or not we have a connection, or no actions. Is there a specific reason to combine?


+1 No errors today


Could we have a different default value for `execute_on` for restarting? If somebody wanted to rewrite the values they would have to explicitly say `execute_on = initial ...`


Yeah, this seems reasonable to me.


@mlaprise
![image](https://cloud.githubusercontent.com/assets/611122/10467937/58c39594-71ca-11e5-87e7-12de32c33aa1.png)


nice ! :smile:


Correct, the beta7 instructions for CentOs do not work with beta8 bits. I have made a note of this in the beta8 version of our docs: https://readthedocs.com/docs/aspnet-aspnet/en/1.0.0-beta8/getting-started/installing-on-linux.html. This will get fixed in RC1 later this year.


Thanks for the suggestion, that worked well.


Spyros, this HTML markup generated within the function, is this a best practice in the WP world? I would expect that it would be in a template and rendered accordingly?


IMO a parameter list is more readable over options, but that's just a personal preference. I think this options idiom became popular because javascript didn't have default parameter args, but ES6 supports them


`export default`


Yes, I will make this an upgrade note.  We already have directions on how to reset them.  Also, I made sure that the supplemental group pre-allocation code in the admission controller will fall back to the uid annotation if for any reason we come across a namespace that does not have the supplemental-groups annotation.


@osvalr

1. Did you do functional testing? According to me. it should exploit to the create a claim and claim lines because invoice_date is DATETIME format.

Please, If you make strong changes, make strong tests. you change line_id.date_due by self.invoice_date. Do you see the cycle for in the code?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.993.61fec5f3873733c6b72f4ae7b751f5a4a7f1d05e](http://build.palaso.org/viewLog.html?buildId=64221&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 543, ignored: 6 Build time: 00:05:18


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 713](https://teamcity.mesosphere.io/viewLog.html?buildId=58188&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:03:59


yep, just realized that :)


it can be a constant.  I made the group strategies reusable for fsgroup and supplemental groups so this just helps it make well formed error messages based on what it is validating.


I agree I don't think it should be common. In this case, however, the reasoning was wanting to extract this into a helper method that was used only in one place, without adding a 400 line gap between definition and where it is used.


@osvalr Test with DEFAULT_SERVER_DATETIME_FORMAT or '%Y-%m-%d %H:%M:%S'


You can use `git rebase` instead.


Scott, you may want:
url="jdbc:phoenix:${server}:/**${hbase-zk-znode}**"

with ${hbase-zk-znode} pointing to a variable defaulting to 'hbase-unsecure'.

That part of the JDBC address is from a configuration in HBase's hbase-site.xml:
```
$ cat hbase-site.xml | grep znode -B 1 -A 2
  <property>
    <name>zookeeper.znode.parent</name>
    <value>/hbase</value>
  </property>
```


About this, is that "rbd" should not be before "dir" ?


@GordonSmith this merge conflict might of wiped out the input changes that i just redid PR for?


Changes:
* important python cache bugfix (PR #121)
* Python 2 and 3 Unicode conversion bugfix (PR #121)
* important PIE bugfix (PR #124)
* gui: Export DISPLAY and XAUTHORITY (Ubuntu 14.04 with MATE bugfix)
* scan performance optimization: **-38%** time if all matches, **-13%** time if nothing matches, **-21%** time if all and then nothing matches
* some clean-ups


Not super proud of how this page is looking at the moment. Am looking to rejiggle it a lot, still.


Increadibly detailed commit name, as always :D
[http://xkcd.com/1296/](http://xkcd.com/1296/)


I have a check earlier when they are set.


VD=true means I already have a data container. your command will return an error if it already exists. by setting VC to some directory, you can copy in a directory. eg if you have a local ~/.tendermint with priv_val, genesis, and config, setting VC=~/.tendermint will copy it into the data container, so when you run the mint container, it will have what it needs


Same issue here. Is this the proper way to generate XML markup in PHP? I would think there would be a builtin library to utilize?


Make a few commits and then move on to the next project :)


Yeah, a lot of different people touched that function last week (and it was a bit messy to begin with.) At the very least, I think splitting out the code that sorts jobsteps by condition splices rerun jobsteps together would be a good and simple first step (I don't even know how those two interact with each other.)


Do we want to add the ```socket.dir()``` call to repopulate list? That would cut down on a lot of repeated code. We could make repopulate list just take a path.


I did wonder if that might be an issue; I looked at Blinding Powder to see what I should do here, so that might need an update too.


I wouldn't select on hidden irrelevant because other things on the page could be hidden with the same class and this would reveal those too. also good to prefix jquery variables with `$` so `$irrelevants = $('.hidden-irrelevant');`


I think what caused the build to fail is the missing `>` here


@osvalr this will mark error. invoice record is a account.invoice, please change invoice.invoice_date by invoice.create_date. Make tests.


very nice, I was wondering how to do that.


how about `form__welcome-text` and form__clear-action`?


Have you checked if our existing SWIG typemaps work correctly with this and similar changes?


Fixed


Is it supposed to be unused?


I think this same thing needs to happen above on line 63.
Als, is it ok to ignore `TreePhoto` and `MapFeaturePhoto` here?


@nacho87 can we add normalize as a bower dependency ?


Have you tried? That gives me:
```
drivers/misc/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c:47:29: fatal error: arch/arm/mm/dma.h: No such file or directory
 #include "arch/arm/mm/dma.h"
                             ^
compilation terminated.
```


Не варто повертати пароль до клієнта. Фактично любий юзер може послати запрос GET user/1/changepass, скинути пароль юзера, отримати пароль, зайти в його профайл і там змінити пошту і ще раз скинути пароль) Аля такій собі хак)


Hey WebSpider, thanks for the reply! I'm not seeing any settings in SickRage for this- where is this found? I've checked search provider settings, all of the tabs, and all of the general settings tabs. I also browsed through but didn't find it elsewhere...

Thanks!


Caribles?  Did you mean Variables / Members?


And also I think this may not do the right thing for collection UDFs


also this logic would fit nicely as a method in the `ActionBar` backbone view.


return cell ?: tweetCell;


That is really cool.

It should generalize to categorical as well, no?  Just sum over k things instead of 2.  The weight will be multinomial, I guess.


did you one better there and get the actual error to be raised instead :) pushed the updated version


Yes, that's already been noted on #32. A fix will be out in the next ~10 minutes, assuming all tests pass.


@albertshau let me know if this is fine, should we add a JIRA to track and remove this in later release?


@osvalr ``self`` is not. see the cycle for.


süß


Can we add this in a separate file @nacho87 ?


Ah, OK; I looked at Dragon Throne Of Tarkir for this one but somehow missed quite what it was doing.


when it comes to subviews on uitableviewcell, use self.contentView instead of just self.


Have the user of the script bootstrap from the `zc.buildout` that was installed in the virtualenv and then use `bin/buildout` as they normally would.


Tbh I don't care, I'd keep the tests sign agnostic as we might change this later


Thank you for the fast response!


LGTM


Go back to old arg parsing for config_file here.


This doesn't make any sense...

tweetWithDictionary should return TWTweet object instance.


Please use descriptive variable names.


mkae tests please. ``DEFAULT_SERVER_DATE_FORMAT`` by ``DEFAULT_SERVER_DATETIME_FORMAT``


Так даже слишком хорошо! :D


Haha skulle precis fråga om det här i smsgruppen, tänkte fixa klart tillägg av noteringar för items imorgon på förmiddagen i linux. Du får gärna dyka upp då om du har tid!


There is surely an easy way around this in the ORM.  Can't we just add a `.filter(model__in=...)` in the view function?


@jreback
@shoyer
I think this might be a good point to wrap this one up, as it's getting quite long, but I'd be happy to add more examples if you think that's needed.


saw approval from cuyler
at https://github.com/basho/riak_ensemble/commit/c788aa6d7a4917b6b1c773b4e73f34142ddcc35c


merging basho/riak_ensemble/feature/bch/upgrade-lager-to-2.2.0 = c788aa6d into borshop-integration-90-feature/bch/upgrade-lager-to-2.2.0


basho/riak_ensemble/feature/bch/upgrade-lager-to-2.2.0 = c788aa6d merged ok, testing candidate = 1a79eed4


I looked it up, it is not specified, it just returns an 128 bit int (which speaks for unsigned-ness, but that's not guaranteed^^)

--> ack


If user only selects RQ, plot:
1. IC vs Q of RQ
2. TradeOff bars of RQ
if user only selects SQ, plot:
3. IC vs Q of SQ
4. TradeOff bars of SQ
if user selects both, plot 1, 2, 3, 4 and:
5. IC vs Q of RQ vs SQ
6. Bar plot of Ct vs IC
Google map are always plotted if selected.


I think I have it mostly implimented

On Tue, Oct 13, 2015 at 3:35 PM, Chris Holt <lefthandedgoat@gmail.com>
wrote:

> Also I think these changes can live safely within the existing skip, so I
> will put them there.
>
> Looks like the big change is to pass an Id to skip, to log it correctly in
> teamcity, in console, and to increment a skip counter and print it at the
> end
>
>
> On Tue, Oct 13, 2015 at 3:33 PM, Chris Holt <lefthandedgoat@gmail.com>
> wrote:
>
>> Can you send it as a pull request?
>>
>> On Tue, Oct 13, 2015 at 3:21 PM, George Alatrash <
>> notifications@github.com> wrote:
>>
>>> Hi Chris,
>>>
>>> I've added some changes to allow reporting ignored tests in TeamCity and
>>> other reporters. Would you please review and apply these ASAP if you
>>> approve them.
>>>
>>> diff --git a/src/canopy/reporters.fs b/src/canopy/reporters.fs
>>> index 75da3dc..b013e39 100644
>>> --- a/src/canopy/reporters.fs
>>> +++ b/src/canopy/reporters.fs
>>> @@ -6,6 +6,7 @@ open types
>>>
>>>  type IReporter =
>>>     abstract member testStart : string -> unit
>>> +   abstract member testIgnore : string -> unit
>>>     abstract member pass : unit -> unit
>>>     abstract member fail : Exception -> string -> byte [] -> string -> unit
>>>     abstract member todo : unit -> unit
>>> @@ -14,7 +15,7 @@ type IReporter =
>>>     abstract member describe : string -> unit
>>>     abstract member contextStart : string -> unit
>>>     abstract member contextEnd : string -> unit
>>> -   abstract member summary : int -> int -> int -> int -> unit
>>> +   abstract member summary : int -> int -> int -> int -> int -> unit
>>>     abstract member write : string -> unit
>>>     abstract member suggestSelectors : string -> string list -> unit
>>>     abstract member quit : unit -> unit
>>> @@ -61,7 +62,7 @@ type ConsoleReporter() =
>>>
>>>          member this.contextEnd c = ()
>>>
>>> -        member this.summary minutes seconds passed failed =
>>> +        member this.summary minutes seconds passed failed ignored =
>>>              Console.WriteLine()
>>>              Console.WriteLine("{0} minutes {1} seconds to execute", minutes, seconds)
>>>              if failed = 0 then
>>> @@ -72,6 +73,10 @@ type ConsoleReporter() =
>>>                  Console.ForegroundColor <- ConsoleColor.Red
>>>              Console.WriteLine("{0} failed", failed)
>>>              Console.ResetColor()
>>> +            if failed > 0 then
>>> +                Console.ForegroundColor <- ConsoleColor.Yellow
>>> +            Console.WriteLine("{0} ignored", ignored)
>>> +            Console.ResetColor()
>>>
>>>          member this.write w = Console.WriteLine w
>>>
>>> @@ -80,6 +85,12 @@ type ConsoleReporter() =
>>>              Console.WriteLine("Couldn't find any elements with selector '{0}', did you mean:", selector)
>>>              suggestions |> List.iter (fun suggestion -> Console.WriteLine("\t{0}", suggestion))
>>>              Console.ResetColor()
>>> +
>>> +        member this.testIgnore id =
>>> +            Console.ForegroundColor <- ConsoleColor.Yellow
>>> +            Console.WriteLine("Ignored Test: {0}", id)
>>> +            Console.ResetColor()
>>> +
>>>          member this.testStart id =
>>>              Console.ForegroundColor <- ConsoleColor.DarkCyan
>>>              Console.WriteLine("Test: {0}", id)
>>> @@ -123,7 +134,7 @@ type TeamCityReporter() =
>>>
>>>      interface IReporter with
>>>          member this.pass () = consoleReporter.pass ()
>>> -
>>> +
>>>          member this.fail ex id ss url =
>>>              let mutable image = ""
>>>              if not (Array.isEmpty ss) then
>>> @@ -147,12 +158,14 @@ type TeamCityReporter() =
>>>              teamcityReport (sprintf "testSuiteFinished name='%s'" (tcFriendlyMessage c))
>>>              consoleReporter.contextEnd c
>>>
>>> -        member this.summary minutes seconds passed failed = consoleReporter.summary minutes seconds passed failed
>>> +        member this.summary minutes seconds passed failed ignored = consoleReporter.summary minutes seconds passed failed ignored
>>>
>>>          member this.write w = consoleReporter.write w
>>>
>>>          member this.suggestSelectors selector suggestions = consoleReporter.suggestSelectors selector suggestions
>>>
>>> +        member this.testIgnore id = teamcityReport (sprintf "testIgnored name='%s'" (tcFriendlyMessage id))
>>> +
>>>          member this.testStart id = teamcityReport (sprintf "testStarted name='%s'" (tcFriendlyMessage id))
>>>
>>>          member this.testEnd id = teamcityReport (sprintf "testFinished name='%s'" (tcFriendlyMessage id))
>>> @@ -246,8 +259,7 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>>              consoleReporter.describe d
>>>
>>>          member this.contextStart c =
>>> -            contextStopWatch.Reset()
>>> -            contextStopWatch.Start()
>>> +            testStopWatch.Restart()
>>>              contexts <- c :: contexts
>>>              context <- System.Web.HttpUtility.JavaScriptStringEncode(c)
>>>              this.swallowedJS (sprintf "addContext('%s');" context)
>>> @@ -260,9 +272,9 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>>              this.swallowedJS (sprintf "addTimeToContext ('%s', '%im %is');" context ellapsed.Minutes ellapsed.Seconds)
>>>              consoleReporter.contextEnd c
>>>
>>> -        member this.summary minutes seconds passed failed =
>>> +        member this.summary minutes seconds passed failed ignored =
>>>              this.swallowedJS (sprintf "setTotalTime ('%im %is');" minutes seconds)
>>> -            consoleReporter.summary minutes seconds passed failed
>>> +            consoleReporter.summary minutes seconds passed failed ignored
>>>
>>>          member this.write w =
>>>              this.swallowedJS (sprintf "addMessageToTest ('%s', '%s');" context w)
>>> @@ -271,9 +283,14 @@ type LiveHtmlReporter(browser : BrowserStartMode, driverPath : string) =
>>>          member this.suggestSelectors selector suggestions =
>>>              consoleReporter.suggestSelectors selector suggestions
>>>
>>> +        member this.testIgnore id =
>>> +            testStopWatch.Restart()
>>> +            test <- System.Web.HttpUtility.JavaScriptStringEncode(id)
>>> +            this.swallowedJS (sprintf "addToContext ('%s', '%s');" context test)
>>> +            consoleReporter.testIgnore id
>>> +
>>>          member this.testStart id =
>>> -            testStopWatch.Reset()
>>> -            testStopWatch.Start()
>>> +            testStopWatch.Restart()
>>>              test <- System.Web.HttpUtility.JavaScriptStringEncode(id)
>>>              this.swallowedJS (sprintf "addToContext ('%s', '%s');" context test)
>>>              consoleReporter.testStart id
>>> diff --git a/src/canopy/runner.fs b/src/canopy/runner.fs
>>> index de34732..b1d3d50 100644
>>> --- a/src/canopy/runner.fs
>>> +++ b/src/canopy/runner.fs
>>> @@ -51,6 +51,7 @@ let ( &&&&& ) description f =
>>>      let lastSuite = incrementLastTestSuite()
>>>      lastSuite.Always <- lastSuite.Always @ [Test(description, f, lastSuite.TotalTestsCount)]
>>>  let mutable passedCount = 0
>>> +let mutable ignoredCount = 0
>>>  let mutable failedCount = 0
>>>  let mutable contextFailed = false
>>>  let mutable failedContexts : string list = []
>>> @@ -60,6 +61,10 @@ let pass () =
>>>      passedCount <- passedCount + 1
>>>      reporter.pass ()
>>>
>>> +let testIgnore id =
>>> +    ignoredCount <- ignoredCount + 1
>>> +    reporter.testIgnore id
>>> +
>>>  let fail (ex : Exception) id =
>>>      if skipAllTestsOnFailure = true || skipRemainingTestsInContextOnFailure = true then skipNextTest <- true
>>>      try
>>> @@ -99,7 +104,7 @@ let run () =
>>>              else if System.Object.ReferenceEquals(test.Func, skipped) then
>>>                  reporter.skip ()
>>>              else if skipNextTest = true then
>>> -                reporter.skip ()
>>> +                testIgnore test.Id
>>>              else
>>>                  try
>>>                      try
>>> @@ -157,7 +162,7 @@ let run () =
>>>      history.save failedContexts
>>>
>>>      stopWatch.Stop()
>>> -    reporter.summary stopWatch.Elapsed.Minutes stopWatch.Elapsed.Seconds passedCount failedCount
>>> +    reporter.summary stopWatch.Elapsed.Minutes stopWatch.Elapsed.Seconds passedCount failedCount ignoredCount
>>>      reporter.suiteEnd()
>>>
>>>  let runFor browsers =
>>>
>>> P.S: I tried to attach this diff as a file but GITHUB kept telling me Something
>>> went really wrong, and we can’t process that file., so I pasted above
>>> instead.
>>>
>>> —
>>> Reply to this email directly or view it on GitHub
>>> <https://github.com/lefthandedgoat/canopy/commit/a5705ca997e2704d2969ed1191f857bae3050cfc#commitcomment-13751224>
>>> .
>>>
>>
>>
>


при редактуванні юзера тепер кожний раз буде пароль шифруватися - юзер втратив пароль свій


@ahmedaljazzar The djangojs-missing.po is too big, and it contains nonjs translations. Looks like we've got a bug in the Edraak i18n new logic. I'm putting a revert script for this (if I can).


Suppression du match sur Number of Drives et Span Depth, tu peux m'expliquer ?


Only one entry for RQ and SQ


A priori si j'ai matché Disk et Drive ici c'est parce que j'ai vu les deux dans les output megacli


Pourquoi ne pas matcher (Disk|Drive), j'imagine que le terme a changé entre deux versions de megacli


Now Decay_lvl3_v1.xml can be loaded in COPASI
Denis

On 13.10.2015 19:50, Denis Kazakiewicz wrote:
> That's what we have now
> >>> D = {'eggs': 3, 'eggs': 1, 'spam': 2}
> >>> D
> {'eggs': 1, 'spam': 2}
>
> Working on patch
>
> Denis
>
> On 13.10.2015 19:34, Denis Kazakiewicz wrote:
>> All right
>> I figured about the bug in our code,
>> We store the data as dictionary so non-unique species appear only once
>>
>> Example:
>> There are two species MG_015_MONOMER in the Decay.xls table ID
>> column, 1 for each compartment
>> But when we loop through the dictionary inside our code, each key
>> appears only one.
>>
>> Need some more time to get through it
>> Denis
>>
>>
>> On 13.10.2015 18:49, Denis Kazakiewicz wrote:
>>> yes, please hold on
>>> rate law should be fixed soon
>>> working on it
>>> Denis
>>> On 13.10.2015 18:47, Martin Scharm wrote:
>>>>
>>>> @den2042 <https://github.com/den2042> did you comment @manikitos
>>>> <https://github.com/manikitos> patch on purpose?
>>>>
>>>> —
>>>> Reply to this email directly or view it on GitHub
>>>> <https://github.com/whole-cell-tutors/whole-cell-reduced/commit/ac12ba924aef4feefccd7bede19fbb7e52ea61a5#commitcomment-13745944>.
>>>>
>>>
>>
>


Finns nu! (L33–L35)


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 714](https://teamcity.mesosphere.io/viewLog.html?buildId=58189&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:05


Exactly!


DOCKER/README.md is a complicated, I figured it would be simpler to just show the basic commands laid out in docker.sh, without turning it into a final user-friendly tool.

We can add more comments in DOCKER/docker.sh

Let me add the VC back in as a comment...


The source code is available here: https://github.com/Exzap/Cemu/releases


dont worry about this, have removed it in the next commit.


Thanks. The change is not handling goto out of a block. I'll investigate.


nothing... weird.  I'll see if I can revert this line change.


It should avoid having nonjs translation being placed in the djangojs files
files, apparently it is not working!

On Tue, Oct 13, 2015 at 10:44 PM, Ahmed Aljazzar <notifications@github.com>
wrote:

> @OmarIthawi <https://github.com/OmarIthawi> Could you please tell me why
> is this line written?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Edraak/edx-platform/commit/2db3e5c37130ebbba740afed68ffa6eb094f44c2#commitcomment-13750375>
> .
>

--
Omar Al-Ithawi

Senior Engineer, Edraak.org <http://www.edraak.org/>

Queen Rania Foundation

for Education & Development

T +962 6 4016464  Ext. 700

M +962 7 90574405

E oithawi@qrf.org

<https://twitter.com/edraak> <https://www.facebook.com/edraakorg>
<http://www.youtube.com/user/edraakorg> <http://instagram.com/edraakorg>


Initial commit


Why is this removed?
This messes up the entity blob tests with the following error:

```
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:testCompile (default-testCompile) on project forskolenatet: Compilation failure: Compilation failure:
[ERROR] /Users/anders/Workspace/forskolenatet/src/test/java/com/forskolenatet/web/rest/PhotoResourceTest.java:[46,56] cannot find symbol
[ERROR] symbol:   method createByteArray(int,java.lang.String)
```

I'm putting this back


from translation files on disk


Thank you, @faho!


i think it makes more sense if this is its own class, since its used in multiple transforms


I think you need to commit the new dummy rpm, you said you changed it?


@mitogh Yes no problem


Vilken tid?


this could extend the other script context


Removing extra space.


@mitogh yes


I see the logic behind "share_owned" as the name but it still reads weird to me. I think I might be OK with "share_owned_by" - i.e. you call this when you want to share this object with the user, owned by the parent you pass in.


What?  provenance? The project name will be shown on the TM result in the translation editor.


This closes issue #24


The issue here is that `data_dir` is a core API, and we shouldn't use from utils.


Instead of converting things to strings, could we just pass the exception object back through the queue, and on the other end do something like `raise packaging.PayloadError from msg`? I don't know if that would actually work, and I like this change regardless.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.994.95dfc32e9c889ec4dd710ddd5378b421ebbdc98f](http://build.palaso.org/viewLog.html?buildId=64222&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 546, ignored: 6 Build time: 00:03:53


True.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 715](https://teamcity.mesosphere.io/viewLog.html?buildId=58191&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:06:18


TODO: with proper productions, this should be unnecessary.


You get less data, see https://docs.python.org/2/library/stdtypes.html#file.read

In general, 1024 should be enough to contain all magic number parts. You are right that, in case of an empty file, the file is NOT considered as a text file. Instead, `'application/x-empty'` is returned. Considering the application of this procedure (i.e. preventing people from uploading wrong file types), this is OK, because an empty file seems to be a pointless input.


What square brackets are you referring to?


ok the simple "just sync me with jae's test net" makes sense. in that case I'd also advocate just moving docker.sh and Dockerfile to the root so we can get rid of DOCKER, and just doing `tendermint node` in the Dockerfile. and then of course point to eris-cli for a more user friendly tool


I didn't see anywhere in the docs that deal with what language the TM input is in? I might have missed it though.  And I suspect if we don't have it in the file we can't actually override it?  Or override it if the code isn't a match for the codes we use in Pootle?


as-tu changé le backend aussi (c'est ta méthode et je ne la vois pas dans le diff)


@aobruno pode ser submetido o PR?


Let's make this an info message please


And there it is :) Answered my own question


Or if the code is incorrect.


check the ```ideal_list.cpp``` @ravirnjn88


I'm pretty sure that you still have a race condition here: you are appending to the list from separate goroutintes (i.e. without locking).  If two append statements happen at the same time, then you could lose one of the files.

Copy this onto your machine into race.go: http://play.golang.org/p/rmyA2UrOlb

    $ go run -race race.go
    ==================
    WARNING: DATA RACE
    Read by goroutine 7:
      main.main.func1()
          /Users/dhowden/race.go:14 +0x32

    Previous write by goroutine 6:
      main.main.func1()
          /Users/dhowden/race.go:14 +0xea

    Goroutine 7 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178

    Goroutine 6 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178
    ==================
    ==================
    WARNING: DATA RACE
    Read by goroutine 8:
      runtime.growslice()
          /private/var/folders/q8/bf_4b1ts2zj0l7b0p1dv36lr0000gp/T/workdir/go/src/runtime/slice.go:51 +0x0
      main.main.func1()
          /Users/dhowden/race.go:14 +0x87

    Previous write by goroutine 6:
      main.main.func1()
          /Users/dhowden/race.go:14 +0xc6

    Goroutine 8 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178

    Goroutine 6 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178
    ==================
    ==================
    WARNING: DATA RACE
    Write by goroutine 12:
      main.main.func1()
          /Users/dhowden/race.go:14 +0xc6

    Previous write by goroutine 11:
      main.main.func1()
          /Users/dhowden/race.go:14 +0xc6

    Goroutine 12 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178

    Goroutine 11 (running) created at:
      main.main()
          /Users/dhowden/race.go:16 +0x178
    ==================
    [0 1 4 6 9 8 7]
    Found 3 data race(s)
    exit status 66


Hi, I am getting compiling errors here such as:
net.c:255:10 error: 'struct udphdr' has no member named 'uh_sport'

I realize you may be working on this and this is not ready for release, but I was curious if you had any releases available that were considered complete in some phase?  I love mtr and would really enjoy using your version with the graphcairo visuals and historgrams.  I don't see anything from this fork in the "releases" page I do not think (version 0.85 appears to be a link to mtr proper).

I found this fork by looking at a feature request in MTR:https://bugs.launchpad.net/mtr/+bug/1405904

Apologies for making a comment on this diff, I don't know how else to message the owner of this project.


k


prompts still suck though


ik what you can do


create a modal with callbacks


Can we try flag these properly if we're going to use them.  I think Django does that for ``DATABASE-ENGINE``


@yaniaular wrong change for my side, that's why not always a sed-based replace is done. thanks


It's in Config > General > Advanced Settings, 4th option down.


every TN server must have its own unique index.


as per our discussion, this is not easy in the ORM.


fixed


Good point.

Fixed.


This should default to ``False`` if not provided.


As per our discussion, correct about cudfs and photos, not an issue for like 63.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 716](https://teamcity.mesosphere.io/viewLog.html?buildId=58192&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:07:24


change text


Change this print statement to use logger.info please.


...provides a weighting factor to alter the final score for TM results from this TM server.


Go ahead and remove the LogIt function please.


--name


reworded.


TMHEAD here but TMCOMMIT in docker.sh?


Just curios why are we initing FileParser here whrn DBParser isn't inheriting from it?


@osvalr so it's always good to test. after this change, try to create a claim with claim lines...


ah yeah, should be the same.  thanks


wtf


Vid 9

Skickat från min Sony Xperia™-smartphone

---- Joel Edström skrev ----

>Vilken tid?
>
>—
>Reply to this email directly or view it on GitHub.￼
>


Fixed in d9df2947044ea472b03a2386694a28e943ab0b0e, just pushed.


this methods seems to be a quick hack,. I don't think this is a good idea.


logger.info message please


A default clause wouldn't work because of the weirdness around rebalancing.


Pick up my rewording from earlier


Same here for rewording.


lgtm.


You need to send this as a pull request for me to merge though.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 717](https://teamcity.mesosphere.io/viewLog.html?buildId=58193&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:04:50


Do this inside the quotes. There is no guarantee scheme is a keyword list at compile time. :)


Current cellResc implementation is incorrect, that line does not affect for accuracy


You want to remove the leading slash, I believe.


Is the '\' at the start of this string needed?  The other logger messages don't use this for broken lines.


This is useful methods for CLI display, however it is tightly coupled with the core library. consider putting it in CLI project instead of DoLah project.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 718](https://teamcity.mesosphere.io/viewLog.html?buildId=58194&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 190 Build time: 00:05:42


I defined uint8, sint8 etc. in rct2.h, so they can be typedefed to what ever is required for the platform and compiler. Please do not use uint8_t.


Thanks for the PR, however I would rather have it done the other way:
status is unrelevant here as Http status codes are a duplicate (my bad).
Better throw away the other status attributes from all other requests.


I'd use Markdown for the README file.


We shouldn't do this. `__load__` is always called with the correct prefix already, which may be nil. This would make it impossible to nillify  a given prefix. :)


Will do.


Line 45


I believe we can get rid of this signature.


This is a good change.  Thanks!


Hmm. I'm intrigued by this concept of "their preferences had to be combined". So, was Pei Wei making a template that all other styles had to follow? As of now, almost all news sites have ended up looking the same anyways. I'm curious if there existed, 'webpage' tastemakers. People who could set up and propagate a template all over webpages. Did online blogs like Xanga or GeoCities use these templates?


@Kiniamaro est-ce que c'est vérifié ?


А это не то же, что pattern-matching?


see #935


Ouch, apologies!


Correct, it makes a mess (sometimes a big mess) when master gets merged into a branch & then that branch gets merged back into master; rebasing keeps it much cleaner.  I've been reviewing this the last couple days with TheSilentOne.  He's changed git clients now and I think we're getting his new process straightened out OK.


```python
>>> iter([1, 2, 3]) == [3, 2, 1]
False
```


Hi Frank  @berghaus :)
I noticed a minor difference in Ganglia configuration among queues, and found this commit. Is the intention to have
``` globals:
  override_hostname: "%f"```
for all the yaml files?
I noticed that the CERN yaml file has neither of these lines. Should I add them in?

Thanks!


@legoboyvdlp

Hi Lego
I don't understand the commit title: What is gen?


In theory, should check that length of `m.captures` is 5 also, otherwise cryptic `BoundError`s will be thrown on the subsequent lines


`django-celery` is not a required package, it only adds admin views to manage tasks. It shouldn't even be in `requiremetns_analytics.txt`, in fact, `requiremetns_analytics.txt` shouldn't exist either.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 719](https://teamcity.mesosphere.io/viewLog.html?buildId=58195&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:06:09


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


Since cgroups.procs cannot be a directory, we can probably remove the IsDir() call, no?

Also, since iErr is checked after file is compared to 'cgroups.procs', errors on other files/dirs will be masked. Is that intentional?


this line still got  hidden danger,  and should to be change  to `guard` or `if let` statement


Renaming to `bt_send_nosigpipe()`


Shouldn't this be `make ... all` as well for consistency reasons (as a few lines above)? It just so happens that `all` is the default target but I don't think we should rely on that.


The number of matching groups in the regexp does not change, as `(\/[^\/]+)?` remains a matching group even with the trailing `?`. The difference is that `match[4]` is `null`.

For the purpose of elegance, I would keep setting the initial variables this way:

```js
var tag = match[3];
var asset_name = match[4];  // eg. total, atom-amd64.deb, atom.x86_64.rpm
var format = match[5];
```

and do

```js
if (asset_name !== null) {
  total = false;
  asset_name = asset_name.toLowerCase();
}
```


Fixed in ef5659c7a445c0ccc39b1a80500a915e1ab2ff7a.


Just came across this discussion about the CSS display property, which enables you to apply particular kinds of functionality to boxes: [http://stackoverflow.com/questions/11805352/floatleft-vs-displayinline-vs-displayinline-block-vs-displaytable-cell](http://stackoverflow.com/questions/11805352/floatleft-vs-displayinline-vs-displayinline-block-vs-displaytable-cell).

One of the answers pivots toward flexbox, all of which might help you consider some next steps.


Is this kind of thing potentially a breaking change?
Do we care if so?


Will fix.


@lvotypko  Caused [JENKINS-30934](https://issues.jenkins-ci.org/browse/JENKINS-30934).


You should be able to replace this with something like:

```python
if key in userCommands.keys():
    if message.content.startswith(key):
        userCommands[key](client, message)

elif key in adminStrings.keys():
    if message.author.name in gAdmins and message.content.startswith(key):
        adminStrings[key](client, message)
```

and reduce a lot of this iteration you're doing.


This is enough of a head-scratcher that I'd almost rather put a named span around the actual text (or use `<i class="icon icon-remove"></i>` instead of `<button class="close">x</button>` for the x, so you could just use `$this.text()`)...but it's really six of one, half dozen of the other.


"an exception will be thrown" _now_, by this method, or later, when using the pattern?


Good points!


No, the `queue_` has to be locked for the `prepend_entries` operation


@maxiloc do you have any tentative plan to merge these changes in master?


@thaJeztah That's a good point. as `libcompose` is inteded to be a library, I would naively think it has options to set up the API version, but the *handling* of `COMPOSE_API_VERSION` should be handled by the software using libcompose.


@jyuan can you include the `&` symbol?  Or should it be escaped?


@jyuan remove this commented out code.


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/235


Isn't that generally true, though? If I try to parse a LocalDate using the pattern "MM'-'dd", I'm going to have a bad day on 02-29, aren't I? Or did you mean something more specific than that?


@jyuan same here, no commented out code (remove anywhere else it's found in this file too)


I'm having a hard time finding pricing for pure carbon monoxide without packaging, can you provide a value?


This will rebuild the container on every single run of the Makefile which is quite inefficient.


saw approval from macintux
at https://github.com/basho/yokozuna/commit/ffe11588b1deca4516110a917e404ad02e4c398f


merging basho/yokozuna/merge/bch/2.1-onto-timeseries = ffe11588 into borshop-integration-574-merge/bch/2.1-onto-timeseries


basho/yokozuna/merge/bch/2.1-onto-timeseries = ffe11588 merged ok, testing candidate = 6216ad7e


That's originally how @biyeun had written it, but it was breaking a bunch of tests and this was a fairly easy way to fix that. However, I see your point about it not being very readable.


bugger. which then means that LGS must have some magic database to know what 0x8* stand for on each device. Or there's some other request that tells us that, but I doubt it.


> implying implications


Actually I can create cgroups.procs directory there, so it's just additional safe-check :)
Yeah, skipping errors is intentional, you might not have access to all things in subcgroups.


i guess this change (replacing register format with amd) prevents using Aurelia with Firefox when firebug is enabled - see https://github.com/systemjs/systemjs/issues/814
Are there still some benefits of using amd instead of register module format?


wrong test data committed


I totally understand where you're coming from, but surely you can see that what you're asking me to do is to agree to maintain code that I didn't write and don't use, for free, forever? You'll (probably) stop working at GitLab within 5 years, but we'll still have to maintain every built-in alias. Maybe GitLab should provide a snippet that your users can copy and paste if they want :gitlab support in their Gemfiles, so that GitLab can support those users.

In an ideal world, Bundler would have paid maintainers and it would be no big deal to maintain aliases for lesser-used platforms. Today, Bundler is maintained almost entirely by volunteers, and none of them are paid by GitLab. The limited paid time spent on Bundler is funded by Ruby Together, where I notice GitLab is also not a member. Supporting the Ruby community before making demands of it seems much more likely to achieve your goals.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 722](https://teamcity.mesosphere.io/viewLog.html?buildId=58200&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:49


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


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1373 of 1373 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.36%25-green.svg)


@manish172 Yes we will release that soon


Cool. nice clean up.


Triple quotes are only used for docstrings.  Code inside of these is not seen by the program and doesn't run.  For a variable that's a string, like you're trying to use, you can use single ' or double " quotes.  Put your first line up on the same line as the variable name, and use a backslash \ to have text on the next line when you get past 80 characters.  To have a line break in your output, use /n.  Like this:
```ONE_CUT_LETTER = 'Dear {donor},/n Thank you so much for your kind donation of \
{amount}. We here at the Foundation for Homeless Whales greatly appreciate it.'```


@mvl22 on my local env this was stopping reque running, which would stop any background jobs sending out emails when new messages are added to threads.  Have you noticed this?  I've tested in dev.


You're right about `IsDir()`. It isn't possible, I'll remove this check


thanks


Generalizing from the Beta/(array-of-)Bernoulli to the Dirichlet/(array-of-)Categorical conjugacy requires representing the Dirichlet distribution, which we have only done in the case of order k=2 (i.e., the Beta distribution). Higher orders require multidimensional density recognition.


One of falcon objetives is the performance, is not faster to use '\\'.join([...]) instead of '\\'.join(...) ?


RIght, I add sync.Mutex to Lock and Unlock write operation


I fixed that, thanks for the warning.


So me will be forced to transfer it to Sigsev orga !


@kfranqueiro, is this in line with what you were thinking?


Get rid of these triple quotes too.


From my perspective, since 5.2 plugin performance is very good. And
according to the release note there was a fix about just that.

Quick tests show same fps with both methods.

Cheers!

Le mar. 13 oct. 2015 22:25, smdol <notifications@github.com> a écrit :

> Our native code clobbers a lot of GL state (and for whatever reason,
> GL.InvalidateState() did not work for us), so for quite some time we've had
> code in the native plugin to save and restore bits of GL state during the
> distortion correction pass. It cropped up as a performance hit, more than
> we liked, mostly stalling the CPU. So we moved that functionality up into
> Unity (see CardboardPostRender.cs) in order to not have to do all that
> state preservation. Maybe the way to say it is that by using
> GL.IssuePluginEvent we were in a situation where we didn't know the exact
> state of the pipeline, so it introduced an "ignorance overhead".
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/googlesamples/cardboard-unity/commit/e94bb104a2ef5924103a4d887a0a451ed56bca41#commitcomment-13751300>
> .
>


I agree, we should ignore the latter two (hence the _unused_ tag). If we can get the frame rate to work, that's a good start.

also agree on the sensor frame rate (that's the wording in the spec) and given that 0x64 "time between reports" uses a different terminology that's the most likely explanation.


I wouldn't make this change, explanation will be in the general comment.


@maxiloc Great. Thankyou so much. I have opened another issue (https://github.com/algolia/algoliasearch-magento/issues/225) and hoping that it should be easy fix.


fast-forwarding 2.1 to borshop-integration-90-feature/bch/upgrade-lager-to-2.2.0 = 1a79eed4


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 723](https://teamcity.mesosphere.io/viewLog.html?buildId=58205&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:03:44


This is a .ckan not a .netkan. If you need something changed it needs to be applied to the [.netkan](https://github.com/KSP-CKAN/NetKAN/blob/master/NetKAN/Deltaglider.netkan).


all tests pass:
success: http://buildbot.bos1/builders/test-yokozuna/builds/1400


@scottpurdy It may not be necessary. Potentially needed during the ```make install``` step. Without it see line 816 here https://travis-ci.org/numenta/nupic.core/jobs/85205848


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 721](https://teamcity.mesosphere.io/viewLog.html?buildId=58199&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:07:59


didn't remove the sass errors


It looks much cleaner and more accurate, thanks.

I wouldn't add the range restriction as you have to deleteOrder. The reason for this is that the intent wasn't to make sure it works like that but rather show people how they can document a min-only range, with no max. Keep in mind that users can create orders, so those would reach beyond the id of 10. Now, for getOrder the range is applied for documentation purpose, and that's again just to exhibit the documentation capabilities of Swagger. The goal is not to have an accurate API but to show the capabilities of Swagger.


At the moment, I'm going for spreading out if/else. Rustfmt will fix this up later if we decide to go for one line if/else.


Why did you remove this line?


Great.  You can actually just use the map approach if you put protect it with a `sync.Mutex`, this is a nice way to do it cleanly when you don't need it anywhere else (embedding in an anonymous struct): https://talks.golang.org/2015/tricks.slide#15.


nice work!


Example by Dan Appleman


I do not quite understand how it is supposed to work. Could you please add more comments explaining your approach?


thanks!


huh didn't know that! thanks


Je kan misschien beter een groot try catch block maken:

```java
try {
    //
} catch (Exception e) {
    e.printStackTrace();
}
```

of anders schijt hebben aan die try-catch blocks en het in de method definition zetten:

```java
protected JSONObject getMagisterResponse(String apipage, Map<String, String> params) throws MalformedURLException, IOException {
    //
}
```


I am weary to add this behavior because it breaks the DOM Event convention - returning false usually prevents the default action of a given handler.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 724](https://teamcity.mesosphere.io/viewLog.html?buildId=58206&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:00:27


Looks like the first two changes are obsoleted by the next commit?


Hey @inacionery!

Here I have to confess, I gave you bad instructions. Reviewing this pull request with both the taglib changed and the calendar changes proved harder than necessary even for myself. I bet it would be better split into two tickets, in spite of what I've said.

Fortunately, the part *you*'ve done is nice :)  So I hope to have it merged soon.


Should be merged into the previous commit because otherwise the previous commit on its own doesn't really do what it should.


I agree this should be changed - maybe we just create a simple counter and bump it for each new handler?


Hey @mustig !
It seems like an interesting fork you have there, but unfortunately the rename of the zsh-theme itself didn't go well. It is quite hard to follow your changes.. I bet there are some nuggets we want to have in the mainline. Could you be so kind and have a look on that?

One other thing: I am very curious about how your terminal looks like. Can you provide a screenshot in your readme? That would be awesome! And, if the changes in your fork are only technical nature and not necessary for the look, put that screenshot and your configuration in our [wiki](https://github.com/bhilburn/powerlevel9k/wiki/Show-Off-Your-Config)?


Addressed by dadb3af :-)


Dans les versions recents de MegaCli, cela n'existe plus:
#  /opt/MegaRAID/MegaCli/MegaCli64 -LDInfo -lall -a0 -NoLog|grep -i virtu
Adapter 0 -- Virtual Drive Information:
Virtual Drive: 0 (Target Id: 0)
Virtual Drive: 1 (Target Id: 1)

#  /opt/MegaRAID/MegaCli/MegaCli64 -LDInfo -lall -a0 -NoLog|grep -i number
Number Of Drives    : 2
Number Of Drives    : 4


Please don't push code directly into master. Changes like this should be made in a branch so that we can generate a pull request and go over changes together.


Uncomment


@dpgeorge : FYI. And actually, it wouldn't take that much effort to run them in Travis.


The method parseLine() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.


Android is going to instantiate our app for us. We don't have the opportunity of defining a main method.


Am I blind or is neither `strTrack` nor `strAlbum` actually used anywhere in this constructor? The same seems to be true for the deleted/replaced variables.


I created a new method "get_author_names" in team.rb.


@bmuschko, Actually, we got existing code in one of the applications like

```java
for (URL url : ((URLClassLoader) (Thread.currentThread().getContextClassLoader())).getURLs()) {
            if (url.getPath().contains(artifactName)) {

```
Code trying to check if a specific artifact jar existing. When we are building with "tomcatRun",we can see  only "/build/classes/main/" returned from _(URLClassLoader) (Thread.currentThread().getContextClassLoader())).getURLs()_
but I can see the same code works good with gradle-tomcat-plugin 0.9.9 & tomcat7.x, also with standalone tomcat 7.x and 8.x but not with gradle-tomcat-plugin 2.x with embedded tomcat

 As the latest versions of the gradle-tomcat-plugin  looks like setting up only the tomcat classpath but not webapp classpath  in [AbstractTomcatRun](https://github.com/bmuschko/gradle-tomcat-plugin/blob/master/plugin/src/main/groovy/com/bmuschko/gradle/tomcat/tasks/AbstractTomcatRun.groovy#L219-L221), I understand the intention of this, but I believe, [Tomcat.start() ](https://github.com/bmuschko/gradle-tomcat-plugin/blob/master/plugin/src/main/groovy/com/bmuschko/gradle/tomcat/tasks/AbstractTomcatRun.groovy#L359-L359) code path may be using the it's parent  / system classloader which resulting missing of the webapp classpath? And I observed the [TomcatRun](https://github.com/bmuschko/gradle-tomcat-plugin/blob/master/plugin/src/main/groovy/com/bmuschko/gradle/tomcat/tasks/TomcatRun.groovy#L145-L153) too, but looks like this is not refreshing contextClassLoader with this webapp classpath.  Did I miss any other code which really doing it and the root cause is different?

And also, I came across another issue
```
org.apache.jasper.JasperException: The absolute uri: http://tiles.apache.org/tags-tiles cannot be resolved in either web.xml or the jar files deployed with this application
```
root cause seems to be same because of the jar scanning process failing to load the TLD and cache then as no webapp classpath and not really scanning the webapp classpath jars? I just tried by appending the webappclass as shown in this  [commit](https://github.com/bmuschko/gradle-tomcat-plugin/commit/55ed7986bd0ddc014b9ed6085b0c15b4070f27e4) and can see the  issue _"org.apache.jasper.JasperException: The absolute uri: http://tiles.apache.org/tags-tiles cannot be resolved in either web.xml or the jar files deployed with this application"_ resolved with this. Is this looks reasonable or do you have any other recommendations or thoughts about it?

But the the initial issue still existing, as I can see the [tomcat.start](https://github.com/bmuschko/gradle-tomcat-plugin/blob/master/embedded/src/main/groovy/com/bmuschko/gradle/tomcat/embedded/BaseTomcatServerImpl.groovy#L56) starting up few daemon threads and one of the thread - "localhost-startStop-1" seems to be not having webapp classpath still but main thread having it. I am investigating on what going on around this, please let me know if you got any some thoughts on this too.


Log.i


Log.e


final


all of these should be final


Nice work, thanks! I'll merge it as soon as possible. I'm struggling a bit on the axolotl branch still...


I get an error from Match when trying to run the Harvest Person Data from CSV option in VIVO.
2015-10-13 15:53:47.666 DEBUG [o.v.h.u.a.ArgList] running Match
2015-10-13 15:53:47.669 DEBUG [o.v.h.u.a.ArgList] command line args: -i vivo/config/h2-sdb.xml -ImodelName=http://localhost/vivo/harvested-data -IdbUrl=jdbc:h2:/usr/local/vivo/harvester/vivo/harvested-data/csv/30FE126DFA5BFC23B376FC4B3D490754/score-data/match -r -t
[snip.....jenaconnect log..... snip]
2015-10-13 15:53:48.414 ERROR [o.v.h.s.Match] r has no parameters
2015-10-13 15:53:48.539 DEBUG [o.v.h.s.Match] Stacktrace:
java.lang.IllegalArgumentException: r has no parameters
        at org.vivoweb.harvester.util.args.ArgList.get(ArgList.java:177) ~[harvester.jar:na]
        at org.vivoweb.harvester.score.Match.<init>(Match.java:140) ~[harvester.jar:na]
        at org.vivoweb.harvester.score.Match.<init>(Match.java:119) ~[harvester.jar:na]
        at org.vivoweb.harvester.score.Match.main(Match.java:443) ~[harvester.jar:na]
2015-10-13 15:53:48.543 INFO  [o.v.h.s.Match] Match: End

If I try the prior commit, the script just hangs. Any ideas?


Oh, thanks.  I will use it.


For these, make a global
```
var errTypeRuleNotDefined = errors.New("a TypeRule is defined without a type")
```

Then return errTypeRuleNotDefined and in your unit tests, you can check that the returned value is errTypeRuleNotDefined.


Stop writing garbage in our read me file, Varlese.


The class TapParser has 12 methods. Consider refactoring TapParser to keep number of methods under 10.


OK, good.  This is one of those changes where you may not see much of a difference, if this path is not your bottleneck.  It was for at least one of our apps.

Can you see if your CPU utilization has changed?


I can envision this for any configuration that is passed to start_test. In that case the call would actually be more generic. BTW, we may want to eliminate the "settings" function that pull all that in start_test.


lgtm


Any reason for changing the order here? IMO in the music playlist view the "first" sort method should be `SortByPlaylistOrder`.


BTW, if you remove the spy_groups, then only spy_points that do not specify a group are enabled.


This breaks my local debugging setup with a "socket closed" red box error.


Looks like `SortByTrackNumber` is added twice but with different label formats?


Je n'ai pas vu ce type d'output depuis plusieurs années avec MegaCli V8, cela date peut-être d'avant.
J'ai testé cela:
$ sudo /opt/MegaRAID/MegaCli/MegaCli64 -V

      MegaCLI SAS RAID Management Tool  Ver 8.07.14 Dec 16, 2013

    (c)Copyright 2013, LSI Corporation, All Rights Reserved.

Exit Code: 0x00
$ sudo /opt/MegaRAID/MegaCli/MegaCli64 -LdPdInfo -a0 -NoLog|grep -i virtual
Number of Virtual Disks: 2
Virtual Drive: 0 (Target Id: 0)
Virtual Drive: 1 (Target Id: 1)


You lost this assert. I like to have such asserts to enforce the typing of the interface.


Good.


You also need to remove these settings from `settings.xml`.


I am not sure if you are getting the above inline comments automatically, hence this comment.


test


It should be checked that the key is actually in the map before accessing and returning it.


@Naios : I presume this update is equally useful in the 6.x source, because the old error text is still showing up in the Travis 6.x PR builds. Are there any priorities on the 3.3.5 => 6.x cherry picks, or are the developers more comfortable with just knowing knowing how to read the 6.x build errors as it is? :-)


Well I guess you are right. Using iteration can reduce memory usage but I agree with performance is more important.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 725](https://teamcity.mesosphere.io/viewLog.html?buildId=58208&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 194 Build time: 00:04:14


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


Ok. Anyway this was the only one that seemed more readable in one line, in prefer I like the multiline versions as well


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 726](https://teamcity.mesosphere.io/viewLog.html?buildId=58209&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:06:29


We can add more buckets here as needed in the future


Yep! It works now.

Well done.


Links to master with line numbers can be crooked when the file changes. Maybe this should link to a specific commit?


This x looks like a typo.


NAVER - http://www.naver.com/
--------------------------------------------

josephrkim@naver.com 님께 보내신 메일 <Re: [matplotlib] Merge pull request #5228 from mdboom/no-more-sourceforce (f04b6c9)> 이 다음과 같은 이유로 전송 실패했습니다.

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

받는 사람이 회원님의 메일을 수신차단 하였습니다.

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


why not arbitrarily assign it to j?


Correct me if i'm wrong but the client secret is not required for implicit grants used by public clients / browsers. So why are we raising an error here?


resolves https://github.com/openresearchdata/ckanext-oaipmh/issues/17


These appear to be new in this version, which is a heck of a lot better
than the previous one in this respect after pull request (#786).

As we are so close to release of sdo-phobos, currently in an effective code
freeze, I suggest we look at this and a few others that have crept in /
have been missed early post 2.2.

On Tue, Oct 13, 2015 at 8:59 PM, Jonadabe <notifications@github.com> wrote:

> And here again! 2 spaces. ;-)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/schemaorg/schemaorg/commit/36a27bea9b930296c23a5987a63f4cd653eff76d#commitcomment-13750684>
> .
>


This rounds to the nearest degree? That's super coarse, right?


Better use `Array.isArray()`.


What it is?


cool shit


Ok.  Well, if you want to try and get it running under Travis then I'd support that.  You can use the travis-testing branch.


Good point, indeed Blinding Powder is not correctly implemented and I committed a fix for that.

So my first suggestion was not correct because this case (ability is applied to another object) the check is more complicated. Look at the corrected Blinding Powder for a correct solution.


Doesn't quite work yet, changes after a goal online


You don't really need to specify 'no border'. Elements don't typically naturally have borders.


'sets' can be deleted as this is const.


応急処置で作ってた
getSceneObjectInfo()、setSceneObjectInfo()を消しても大丈夫かも。
Macでは、この二つのメゾットをsetFrameからコメントアウトしても選択範囲が変化しなかった。


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 728](https://teamcity.mesosphere.io/viewLog.html?buildId=58214&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:03:57


Whoops, haha, thanks for catching that!


Maybe it should be `static struct db_main fakedb;` - we might keep some pointers into this after init?


Why the odd time control: sprt @ 40+0.4 th 1 ?


Please be specific. what response header do you write? WriteResponseHeaderForServer?


or ForCluster?


Check what precondition?


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 729](https://teamcity.mesosphere.io/viewLog.html?buildId=58215&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 180 Build time: 00:04:12


Done.


Do we also revert the VALID_AGG_OPS?


this is so strange.... why not isRemoveError?


minor nitpick... the current git workflow suggests to do:
```
profiles: fix typo in p.mask
```
to have a short subsystem name ('profiles' here)


Changed this so `repopulateList` makes the call to `dir`. It gets the path from the element property `this.path`.


Now, by a method that this method calls. It'll end up as an ArgumentOutOfRange exception, with a parameter name that we don't supply - which is why I didn't want to document it as an <exception> here. It doesn't feel worth wrapping the call in a try/catch just to change it to a different exception... thoughts welcome :)


Fixed in 743bf69e9705e7b6a62cb80a195d935b05e233d5.


my bad


np :)


@mattmuroya yes you're correct--they're very different. I used the wrong word here--was typing kind of quickly. (eek)


@corbt If you can submit a separate PR for docs that would save us work :)


It's the new standard LTC time control : https://github.com/glinscott/fishtest/pull/67


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1911 of 1911 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)


Well, the default template value is 2000-01-01, so it would still end up being valid. Changing to use a different template value (i.e. not in a leap year) would cause a problem, but the difference is you'd be explicitly choosing a template value then - whereas choosing a template value via this method is a bit less explicit.


@cston You are right, we can use _waitIndicator.StartWait directly. Fixed.


Thanks for fixing my oversight.


@jamischarles why this was reverted? I was thinking about extending this regex to actually match lowercase, since that is what our team uses


This should come from ```SmartSync```. Chained dependency. The compat library is required only if we support older ```Android``` versions too.


Yup, pretty easy, see https://github.com/ably/ably-ruby/blob/master/spec/unit/models/token_details_spec.rb#L55-L82.  Simply set the expiry to within the buffer and check that it is considered expired


That's fine, and on re-reading that's probably clear from what's written ("if... and then this method is called with..., an exception will be thrown.")

I tried rewording a bit, but it ended up sounding awkward. Probably not worth fiddling with.


See 2fbf89b20155760f0efdc691f9ef47701a1e0438


As long as you have unmatched parenthesis/bracket/brace you don’t need to use `\` for continuation. PEP8 AFAIR even requires you *not* to use it.


I am apparently still unable to master the rules for leap years in the Gregorian calendar.


would break asymmetry. which is probably fine except in some weird cases... you never know what could happen. there are some cases where i have to deal with ties and then i flip a coin (which is "symmetric") but i don't want to do this in this tight loop


Ok, great, thanks.  I am just thinking about future proofing it because if you later want to support specifying the mode or algorithm in the constructor, it may be difficult without breaking the API


Published later, will try to always publish to CocoaPods at the same time as when publishing to npm.


Ok, perhaps my knowledge of PHP is limited, but as far as I can tell [it does have properties](http://php.net/manual/en/language.oop5.properties.php), and in the Java version it's just a normal property too (i.e. not a special accessor or method under the hood), see https://github.com/ably/ably-java/blob/master/src/io/ably/rest/Auth.java#L165.  I am not clear why this needs to be a method at all?


saw approval from cuyler
at https://github.com/basho/riak_core/commit/b53eeec6b7dab08717188040792b92be37baff0c


merging basho/riak_core/feature/bch/upgrade-lager-to-2.2.0 = b53eeec6 into borshop-integration-789-feature/bch/upgrade-lager-to-2.2.0


basho/riak_core/feature/bch/upgrade-lager-to-2.2.0 = b53eeec6 merged ok, testing candidate = cade2d16


Ah, OK, makes perfect sense.


You have a really good starting point here! Keep it up.


@mrdoob Removing the "shadow" getters from `Light.js` is silently breaking code, as the values are simply undefined.

Would you consider restoring them?


Archive method is public, maybe someone using it.

I suggest that keeping it and calling DynamicArchive internally. And decorating it with obsolete attribute "Use DynamicArchive method." note,

```csharp
[Obsolete("Please use DynamicArchive method")]
public bool Archive(string archiveFileName, string fileName, bool createDirectory)
{
     return this.DynamicArchive(archiveFileName, fileName, createDirectory);
}
```


Hi @airy how are you? I remember from the beginning of the summer that you were in the middle of a larger pull request for this gem that is half merged into main so far. What is the status on that? If I remember correctly you had asked me not to bump the version of the gem until the your full changes were completed.


Yes, I will squash everything. I think I figured, that people might follow the progress of the PR and like to see the diffs between the versions without the need to search for them.


I'm using tabstop=4 are you using other?


No, because it does not cause the error -- but eventually we should refer to vega schema's json rather than referring to files directly.


List of things to do.

**Chris’ comment. Audience is what really matters. **

Digging into the spec more, searching the Charles Babbage Institute for any historical materials related to code-commentating.

**Lhyen.  It helped the grader understand your intent. Importance for developers to enable their pages to function properly. Commenting needs to be done correctly if it’s done at all. **

**Sarah. Auto-documentation.  Commenting from the code and putting it into different formats so that it all makes sense.**

“You should write code that explains itself”.  explanation IS additional content.

To do list.

There is an audience that code comments, (CC) are written for. *Who are they? Why do they exist? Why is it necessary? (Look up historical information about it.)*

The use of comments is important. It helps the reader understand the intent of the author. A parallel that could be drawn is the use of a footnote in a novel. How has commenting developed because of a collegiate approach to Computer Science education. *Have Computer Science programmers changed because they’re following models and not their authorial intent?*

Auto Documentation is relatively new and I don’t really understand it. Look it up and see what it means. *Does it make it easier on the author and the reader? *


Thanks.  I will remove it.


Isn't `./node_modules/.bin` already in the path when `npm run` executed ?
https://docs.npmjs.com/misc/scripts#path


What's "magma)y" ?  :stuck_out_tongue_winking_eye:


For some reason it was not included with the previous commit.


dynamic programming
Here don't save the whole substring for s[i-1], just the length, since the substring is fixed and just part of s!


Actual tabs :smile:


Thank you so much for the suggestions and links! I'm planning on talking about both the element and the attribute center and how they work in similar or different ways. I think you can use the element center for images because I did that in my index practice web page that we did a few weeks ago.


is this test necessary?  We are always going to have a groupid unless something has gone wrong, right?  And if we don't have a groupid, it doesn't matter how we serialize it, does it?


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


OK, I see that the deserialization depends on it


Thanks, all- yeah, I'm going to go look up that book ASAP, Mao.  I would be curious, too, to see how the various default fonts for each generic family are decided on.  If it's Fantasy, for example, which one ends up being the least objectionable to the most people? Is that even the right question to ask regarding that one.
Re: Comic Sans, @alexandradepalma, I just have to share this essay I don't hate (though it looks to me like it's seen as more of a "cursive" font): http://www.mcsweeneys.net/articles/im-comic-sans-asshole


MaxParallelThreads	Overrides the maximum number of threads used per assembly. The default value is determined by the test framework (for xUnit.net v2, it is the number of virtual CPUs in the PC).
Applies to: xUnit.net v2 (from https://xunit.github.io/docs/running-tests-in-parallel.html)

Would it better to use the default rather than hard coding it to a specific number?


Hello,
would you please update tarball on PyPi? Thanks.


One thing I keep on noticing as I read and reference Duckett is the lack of use of comments, especially in example blocks of code. I agree with @murtosl that more commenting should be implemented in coding since there is usually more than one way to achieve the same goal. When I took an introductory course to C++ back in high school, the comments in the examples and on the code we analyzed really helped me grasp the goals of the programmer who wrote the code.

With this being said, @lingeringcode could it be beneficial--especially for novice HTML coders--to have some sort of "master" file that includes all of the commenting a coder would want to add, then when developing the final product, simply omit all of the comments to conform to the expected norms of the HTML community? @linuxbo it would be interesting to write a HTML document with a multitude of comments then compare it to code from other languages--you could then analyze what your document looks like with and without coding next to a document written in another coding language.


I suggest you use the existing definition of LqdCO as a reference for the cost


Added this file to `include/Makefile.am`


Why do we return a list of groups and now just a filtered down set of PwPairs?


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1911 of 1911 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)


This line could be changed to "MAGICK_THREAD_LIMIT=2 ulimit -t 14400 && ulimit -v 5000000 && ulimit -f 10000000 && nice -n 19 " to limit processing to two cores.


lol still on it huh?


> low-level code-representation to serve as a compiler target

We should explain what this means :)


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1775


Told you I tried to break it on purpose. I'll fix that with the other 'on purpose' regressions you might still find (and I got in 'on purpose' by accident :wink:)


Renamed file to send.h


> Maybe best would be to create it's own user (garb) with some default home directory (/var/lib/garb) and run daemon as this user with chdir to it's home dir.

indeed. But then it also requires fixing packaging scripts :(


No, just don't like things keep things uncommented

On Tuesday, October 13, 2015, James Liu <notifications@github.com> wrote:

> lol still on it huh?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jamesliu96/gen-st-162-project/commit/4ebeab223eda4cdc930d93881a203768a5daef38#commitcomment-13754164>
> .
>

--
Follow your heart, then life will lead you to the place you should be.


Hm, thinking about this a bit more and I see how it could be useful, but I'm not sure what the best way to expose this is.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 731](https://teamcity.mesosphere.io/viewLog.html?buildId=58222&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:03:52


fix for bbc/microbit-extras#2023


This is a crucial fix for f35e06b60fe21fac2dac34ab16a0a7e4888d7b84... otherwise the preview doesn't always update properly...


@pinkdoremi Here is the behaviour that I expect:

- If there is one dropdown list shown, then
  - clicking **outside** the dropdown will close it, and
  - clicking **inside** the dropdown (selecting an item) will also close it
- If there are multiple dropdowns shown, then
  - clicking **outside** all dropdowns (somewhere on the page) will close all dropdowns
  - clicking **inside** dropdown X will close **only** dropdown X.

Is this the behaviour you were expecting as well?

I think you noticed that when there are multiple dropdowns shown, clicking inside one of them closes all of them -- I've fixed this [here](https://github.com/ubyssey/dispatch/commit/900266f55ff3954344b739a3b39d7fbafd246ffe) so they are now independent.

Thanks!


Don't forget to issue a pull request for this commit so I can put it into the main repo.


93653630-MultipleYearRunPeriodForTabularReports (JasonGlazer) - 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)


Did so above? Didn't I?


Since you have 2 different elements here, put the `<a>` and the `<img>` into a new line to help make it more readable.


For the opinion side of the article, we could try to answer the question of whether we think WebAssembly will be adopted easily or be resisted by the community, and also what your thoughts about it are.


That is my understanding.
Le 13 oct. 2015 11:27, "Pascal Lamblin" <notifications@github.com> a écrit :

> So, is it expected that this code can be imported from two different
> places, and we don't know which one will be called first?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nouiz/Theano/commit/32f8a112160ebd3e5fa3b59d5fcd2dde939f2d4d#commitcomment-13744002>
> .
>


I would add a short concluding paragraph after this, and bring "the future is bright..." down to the conclusion.


@Kaysilin тут надо написать так:

```js
function initScroll() {
...
clearTimeout(timeoutDisappear);
timeoutDisappear = setTimeout(isCloudDisappear, TIMEOUT);
}

function isCloudDisappear(){
var elementCloudsPosition = elementClouds.getBoundingClientRect();
if (lementCloudsPosition.bottom <= 0) {
  window.dispatchEvent(new CustomEvent('cloudsdisappear'))
}
}

window.addEventListener('cloudsdisappear', function() {
     parallaxActive = 0;
    });
```

Пускай пока, что parallax эффект будет отключаться если мы проскролили вниз


Nope, I think that ```SortByTrackNumber``` just slipped in


Good catch, will remove one.


оставил сортировку по вычисляемому полю!!!
по другому никак не получается...
3 запроса выполняется при просмотре новости


dfsfsd


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 732](https://teamcity.mesosphere.io/viewLog.html?buildId=58223&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:03:55


setTimeout первым аргументов принимает функцию. Сейчас ты всегда посылаешь событие без задержки в 100 милисекунд.

Надо было написать
setTimeout(function(){ window.dispatchEvent(new CustomEvent('cloudsdisappear'));}, TIMEOUT);
тогда бы мы откладывали событие на 100 милисекунда


Is this (the whole file) really relevant?


Yes, your right. Was like this from the start. Blindly refactored it. Will change it.


I think we want to uncomment ``moment`` & ``momentTZ`` because later we use the moment-timezone.js function `.tz( ... )` down on Line 37.


A couple of issues with this:

- It seems to short-circuit the previous checks for certificate validity.  Now if hostname verification fails, the user will get a dialog asking whether they want to continue.  If they click "Yes", then no further validation is done on the certificate, and the user is never given the option to add it to the trusted certificate store.  I would expect that the verification encompasses a chain of operations that must all succeed in order for the certificate to be valid.

- There is no way for a user to "trust" a particular hostname.  I would expect that, upon answering the trust question once, that question should not be asked again for that particular host.

- I don't fully understand what this is verifying.  What does a certificate need in order to pass this test?


Please avoid committing space changes, it would be easier for everybody ;-)


I'm curious if this is working with the `require` for moment-timezone?


`font-family: 'Open Sans', Helvetica, Arial, sans-serif;` I don't think your font actually applied because you linked it after you linked your stylesheet. Remember that the browser reads you code line-by-line. So when you provide your stylesheet with your desired font, it doesn't know what it is yet if you provide the link for the font after.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 733](https://teamcity.mesosphere.io/viewLog.html?buildId=58224&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 180 Build time: 00:03:59


@DougFischer porque removeu essas outras 3 empresas? Pode fazer o merge com o master por favor?


This commit is from December, if you think this is an issue, find and mark the line in the current code base


I just got some of the ARM stuff working on Make, will see if it is problematic.


first commit


That field name is confusing..


Comment added!


Most of the times when doing recursion or in this case iterative-recursion, you want to put your base case in the top. or as the first instruction of the while loop
This way you validate your input before doing something.
For instance:
* what if x and y are already out of boundaries of the matrix?
* what if the matrix is an empty matrix?
* What if the original color is the same as the new_color, do we have to do something?

You can either change it or we can discuss this tomorrow


This is good and i'm glad to see minimal does not have this so we keep it minimal.


Ай-я-яЙ!!!


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 734](https://teamcity.mesosphere.io/viewLog.html?buildId=58229&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:06:13


This won't fix the issue because all it does is remove the recipe to craft planks.  However, slabs are a different item.  I am working on a fix and I will send a pull request when I am done.


This won't work here since it is opening a file on the machine running kola on and not the just spawned master VM. Your limited to interacting with your new VMs through the TestCluster interface. So, you can just `master.SSH("cat /etc/machine-id")`  and collect the stdout, then trim the space or w/e.

I think the confusion stems from the fact that the coretest suite of kola tests actually uses kola's native code functionality.  See:
https://github.com/coreos/mantle/blob/master/kola/coretest.go and https://github.com/coreos/mantle/blob/master/kola/tests/coretest/testgroupglue.go - -The second link is the actual kola tests that then execute the registered native functions in the spawned VMs. Its a little cumbersome I admit, but it made the porting coretest to kola straightforward.


Cool change, man!


@fabn when you're planning to release this version? :)


I did not even now you can do this, cool!


shouldnt this be a list with zero elements rather than null?


was meant to erroneous "comment" not commit!


Initial commit


> - decide on the final place for those (see #123)

should read

> - decide on the final place for those (see #124)


:+1: i see now


Very nice and clean code overall!


Whoops, I didn't notice the typo


A couple of reasons:
- QLatin1String docs suggest that it's the safest + fastest approach
- For Qt5 builds QLatin1String is faster than the char* overloads as the char* overloads use slower utf8 decoding but QLatin1String doesn't (see http://woboq.com/blog/qstringliteral.html)
- it will make it much easier to identify strings which we can flip to QStringLiteral when we kick Qt4 to the kerb.


To run numeric sequence calculator project. Please download the source code and double click on the solution project 'NumericSequenceCalculator'.

This should open your visual studio solution. Then click on run to run the project.


serasera


This seems like an anti-pattern anyway. Can be removed in favor of `New().WithArgs()`


fast-forwarding 2.1 to borshop-integration-789-feature/bch/upgrade-lager-to-2.2.0 = cade2d16


Probably we need to return a copy of incoming options here with defaults provided for cases when options are undefined


My tests were not initially passing using ```let```.  It turns out in the last test ```first_parent.children``` did not call the Family.create method before ```family.parents.create``` so I was always getting ```0``` on the last test instead of the expected ```2```.  I was thinking I did something wrong with my associations, but after rereading your emails and the Rails Guides, I knew I had set up the associations correctly.  Thus the error had to be in the test.  I knew with the ```before do end``` syntax, the variables were redefined for each spec so I had a feeling it would work and it did.  I could not have debugged like that when I first started with you.


fast-forwarding end-to-end/timeseries to borshop-integration-574-merge/bch/2.1-onto-timeseries = 6216ad7e


nil is the NoEmailer value - It's returned here: https://github.com/coreos/dex/pull/146/files#diff-cbaaf9e34ffda3d54135814606573888R129 - the reason I used nil is that nil is assumed elsewhere in the code - password links are only returned to callers in the presence of a nil emailer.


Thanks a lot!


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


This is a RFC for now.


Make sure they don't refer to this name anywhere in the cookbook (e.g. via the `cookbook_name` attribute on some LWRPs).


Here is the new error I got:
```
..................................................46050
...........................Traceback (most recent call last):
  File "/home/wulab2015linux/warren/Software/spladder_master/spladder/python/spladder.py", line 255, in <module>
    spladder()
  File "/home/wulab2015linux/warren/Software/spladder_master/spladder/python/spladder.py", line 239, in spladder
    count_graph_coverage_wrapper(fn_in_count, fn_out_count, CFG)
  File "/home/wulab2015linux/warren/Software/spladder_master/spladder/python/modules/count.py", line 146, in count_graph_coverage_wrapper
    tmp = sp.hstack([sp.atleast_2d(x.edges) for x in counts_tmp[:, c] if x.edges.shape[0] > 0])
  File "/usr/lib/python2.7/dist-packages/numpy/core/shape_base.py", line 274, in hstack
    if arrs[0].ndim == 1:
IndexError: list index out of range
```
Let me know what you need. I can open up a new issue if that would be better.


TeamCity OSS :: Kubernetes Mesos :: 5 - Smoke Tests (docker/mesos) [Build 736](https://teamcity.mesosphere.io/viewLog.html?buildId=58235&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 189 Build time: 00:04:26


PHPCS: There must be one blank line after the last USE statement; 2 found;


Method BuildService::addBuildToQueue does not have a Docblock comment.


Class BuildWorker does not have a Docblock comment.


Ah, very good. It will make Travis tell us more precisely what is going on. :+1:


This will actually be initialized to "Hello, how are you?"


Probably we need to return a copy of incoming options with default value for cases when options is undefined.


It might be simpler to do `curveOption.union(.AllowUserInteraction)`.


I'm aware of that, and it's sort of an inside joke. Leaving the proper SQL
password in the code files was purposeful.
I appreciate you taking the time to alert me to it, as it shows you're a
decent person and clever enough to have located it.
I'm fairly certain as you said that the localhost restrictions in my SQL
config will keep most people out.
Anyway, I no longer run the server.

Cheers,
Alexander (Sshado)

On Mon, Oct 12, 2015 at 11:20 AM, skull132 <notifications@github.com> wrote:

> Just as a general note. The values you, @Isratosh
> <https://github.com/Isratosh>, and @sshado <https://github.com/sshado>
> keep editing here are placeholders. They're initialized with generic as
> hell strings, and later updated to the proper ones through config
> initialization. The config file you want to look at is dbconfig.txt. Having
> passwords and (luckily you were using localhost, but still) databases
> addresses like this in the open seems like a stupid move.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sshado/Aphelion-Code/commit/696cd50aa6f7952290851e9e4ff609fcbceac3c7#commitcomment-13719574>
> .
>