PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed (1759 of 2088 tests passed, 0 test warnings)

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


Suggestion : It would have been a lot easier to add a withFileName().withTitle() .. etc additional to getters.


This one doesn't work–I don't see an item.


I think that should be OK.  You probably would need to make a similar change to the "Fortran-shaped" checking regex a few lines below.


This doesn't need to be here. Can just be `@_producer_name`, no `||= nil` needed.


no need to add relative file path for no_thumbnail.jpg?


@atifsaddique211f what is the purpose of this docker file?


Uncontreversial :)


parse(...) will modify host and port (which is the whole point of passing them), so I can't use const refs.

Do you want me to implement the name resolution? It's pretty straightforward, but do we allow communication on ports that are reserved by a service? I feel like we wouldn't.


Ok, thanks!


jajajajajajaja


String was an error, any is the way the build filter is setup from the beginning.


This is incomplete


@atifsaddique211f plz remove the commented parts


as in astroids? might as well remove the generation and make chips available from vendors for lots of cash. Intel inside.


It would be helpful to add a comment explaining why do we do this -- i.e. that this addresses zero-length subnamespaces in obfuscated code.


It'd be better defining an `enum` with cases for the (small) range of notes that you support instead of letting clients pass in arbitrary strings. You could parameterize cases with the octave if necessary.

    enum Note {
        case A(Int)
        case A#(Int)
        etc...
    }


make a std string and then use std::string::data() and std::string::size()


Dear Rachel,

Thanks for the Comment!!

Best,
Emma

On Tue, Jan 19, 2016 at 8:18 PM, Rachel Smyth <notifications@github.com>
wrote:

> In line 30, you would want move to be evaluated first, the way you had it
> before.
> Also, consider refactoring your getWinner() function using either switch
> case or ternary operator.
>
> Overall GREAT job! Very clear! [image: :100:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AuvEmma/wdi-fundamentals-rps/commit/4c06a2fb3f1e32882b63b7762d9ac9c7d334a71e#commitcomment-15546042>
> .
>

--
Yue (Emma) Hou
Columbia University
The Fu Foundation School of Engineering and Applied Science
Electrical Engineering, M.S. Class of 2014
Graduation Date: 02/2015


@atifsaddique211f how will handle the case when docker-registry is running on another server? maybe parameterize this as well?


but as far as AE2 making lag - like only 1 person had a system up, if any for the past week.


Introduce a helper method in IsSingleNodeVisitor for `new IsSingleNodeVisitor().visit(tree))` that better conveys the intent.


this file is for go agent,


almost all scenarios realized in this test case rest on the same verification: user has not enough resources to sell. for now this test case works the same as the next one (should_fail_when_player_has_not_enough_resources).

you should give resources to player first


In that case, you should probably switch to a [better search engine](https://duckduckgo.com/tour). This is what @demhydraz meant AFAIK:
![](https://i.gyazo.com/057bcb183f0b182eaeceb7f04ef516bd.png)
[Source](https://en.wikipedia.org/wiki/Word_%28computer_architecture%29#Size_families)


I'd rename `allowExpressionInCalculation` to `allowComplexExpression`


Interesting, it may have never been there. Ill pull in.


put `columnWiseUnion` on a new line to force the indent... it makes the "equals" easier to read


same here


It's a good habit to wrap debugging statements like this with `#if DEBUG / #endif`. There have been instances where people checked in code that printed user's passwords to the Console...


Great effort! This change will definitely make the scan process faster. Lets discuss on the comments I have made on some of the changes .


I'd rename this to `IsSimpleExpressionVisitor`


What is a purpose of this username and pwd?


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.163.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68180&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:35:55


Instead of storing each note as a separate audio file, you could consider using `AVAudioUnitSampler`/`AVAudioEngine` with a Marimba sound bank.


Wootric Kit


All of these will become constexpr when all our compilers support it, right?


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


There must be a smarter way to do this. Registering constants in the main plugin file? DRY?


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


Is this tab indented for my pleasure? ;) Idiomatic for Rails is to 2-space-indent all the things. :/


it a username and password for login of go cd web console


There are new options for the `bins` argument to `numpy.histogram`: http://docs.scipy.org/doc/numpy-dev/reference/generated/numpy.histogram.html. Something like:

    Automated histogram bin size estimation
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    The ``bins`` argument of ``np.histogram`` now accepts the strings ``'auto'``,
    ``'fd'``, ``'scott'``, ``'rice'`` and ``'sturges'`` to indicate the type of automatic
    bin size estimation to use.


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


There must be a smarter way to do this. Registering constants in the main plugin file? DRY, right @ryanhellyer?


"**Complex** expressions not allowed ..."


These are not classy enums


Renamed MPConstants to MPIConstants


As we saw last night, this does not work. You have to specify the submit method as `'get'`


These could be enums like above


this is a line comment


requireNonNull


Wootric Kit


@magnudae kjeder seg tydligvis på jobben... :trollface:


test


Working on the stretch requirements. Awesome


What you have there is not a data type. It is a processor word. Not a number.
See here, the definition: https://en.wikipedia.org/wiki/Word_(computer_architecture)


If I read this correctly, it looks like this is looking for the version number out of the id of the entry. There is nothing that guarantees the id will always contain the version. Instead, to get the version, use the `<as:version>` element in the entry. That is required to be there and only contains the version number you want.


As a best practice, you should use a `guard let...` here and terminate early in case `avPlayer` is ever `nil`. Since you're probably never expecting it to be `nil`, you could place an `assertionFailure` in the `else` clause so in debug builds you'll get trapped in the debugger as soon as things go wrong and in release builds this is just a no-op.


This should be in `X-Rester` format.


These could probably be classy


Interesting way of doing this. Great work


ditto


Idem


In fact, I added that to the version of this which was merged: https://github.com/lsst/meas_base/blob/master/include/lsst/meas/base/ScaledApertureFlux.h#L81


Thank you @iangreenleaf for this noble revision. I support this change


ok! we should add a bit of explanation and purpose of each file


this change looks good to me. Just one very minor thing:
Could we add some comment in the beforeEach to explain why we addEventListener for 'urth-viz-render' and then when it's fired we remove the listener, maybe something like: "to ensure that the table has been rendered before calling done( ) and testing"


another comment


another line comment


Perhaps you could use a lazy variable or `dispatch_once` instead of having an `initialized` variable.


test again


File Formats/Images/UCG Image Format version 3.md

It looks like ![](http://i.imgur.com/F5abGCK.png), unless that's intentional.

I'd rather have it as a tree.


it's gone. see `master`.


I still need to review this outside of Github's interface


Needs a license header.


Oh hey, sorry for that then, I was apparently wrong (and @demhydraz as well). You can just call them words then I guess


Why should a new method be added that is already deprecated?


Please keep identation 4 spaces


Hello, @futtta

Nope, Cloudflare works as a proxy, where you can signup at [http://cloudflare.com](CloudFlare), add your site, and replace the DNS config of your domain with CloudFlare's DNS servers, and after that replace, it try to cache and optimize your site, and Rocket Loader is just an option (which is in Beta state yet, but..) in CloudFlare's panel that try to asynchronously load Javascript files found in the page that can block the rendering of the page.

So, CloudFlare does not depend directly on a plugin, but...you can use it the CloudFlare's official Wordpress plugin with that proxy, but just to fix the IP of the real user so moderation systems can detect the correct IP correctly.

Here's a [overview of CloudFlare](https://www.cloudflare.com/overview/), may help to understand better the system. =)


Please remove this change from the commit


Um.
> In computing, word is a term for the natural unit of data used by a particular processor design.


for what exactly is this field? been wondering


sangar is reviewing my PR -- can we do one of two things
1. wait to PR the fix, or
2. push your fix to your own branch here and i'll merge it by hand into my PR
Sangar is reviewing my enormous PR, and I'm working on reducing memory cost to openos boot
I should have 2 PRs for openos 1.6, the first is the bulk of the work, the 2nd will mainly be memory optimizations (with a few minor bug fixes I find along the way)
PR1: https://github.com/MightyPirates/OpenComputers/pull/1610
You can test that with your terminal code


Look at you working with events! Great job


`@leadcontributor/s`?


Great job in making this work with two functions and manipulating the DOM


I'd also like to point out that
> The first widely adopted 8-bit microprocessor was the Intel 8080, being used in many hobbyist computers of the late 1970s and early 1980s, often running the CP/M operating system; it had 8-bit data words and 16-bit addresses.


Please remove this change from the commit, and all the other unrelated files as well


a


I was trying to talk about this paragraph:
https://github.com/junit-team/junit5/blob/0d96e17fb66ba6e7dc174c5c87a1c4cfc3fade33/junit-launcher/src/main/java/org/junit/gen5/launcher/Launcher.java#L17-L21


ای بابا ریدییییییییییی توش. الان میخام ربات نصب کنم ارور تی جی میده

fatal: reference is not a tree: c6ddcad25c7683bac8be130967f26fa383bfdc00
Unable to checkout 'c6ddcad25c7683bac8be130967f26fa383bfdc00' in submodule path 'tg'
patch: **** Can't open patch file patches/disable-python-and-libjansson.patch : No such file or directory
autoconf: error: no input file
./launch.sh: line 87: ./configure: No such file or directory
Error. Exiting.


squashed.


Exactly. That means a processor word, and not a word as a data type as used in C, C++, Java. (they're called "short" there)


Make sure checkstyle passes. I saw errors in the build:

> ProtoUtilsTest.java:1: Line does not match expected header line of '/\*\*'.
> ProtoUtilsTest.java:6: 'method def modifier' have incorrect indentation level 8, expected level should be 4.
> ProtoUtilsTest.java:6:1: File contains tab characters (this is the first instance).
> ProtoUtilsTest.java:8: 'method def' child have incorrect indentation level 16, expected level should be 8.
> ProtoUtilsTest.java:9: 'method def rcurly' have incorrect indentation level 8, expected level should be 4.


Shell script utilizing dialog to navigate directories. The current working directory can be printed.


No, that wouldn't cause it. As I said, almost has be from partially committing changes that are outstanding, unless you wrote code that won't compile and committed it intentionally before moving to the next file :grinning:


There is a comment where we set possiblyHavePairsWithDuplicateKey to true. I am not sure if we have to go that specific though, I am not making a claim that this is the only possible cause for the broken sort order.


Did they just make trade confirmations (e-mail or mobile) mandatory?


SELECT TabId should be specific to a table as [dnn_vw_Modules] and #TabsLatestPublished both have TabId column


This will crash if you forget to call `chooseProgression` since `scales` will be uninitialized. One possible solution is to use a `lazy` variable for the necessary state. You could also terminate early if `scales` is ever nil by placing a `guard let...` early on. Inside the `else` clause you could put an assertion so things fail gracefully in Release builds but in Debug builds you get trapped immediately. You could also use `precondition`.


@robneu just a heads up you can do:

```scss
@include span(last 9 of 12);
```

this will produce fewer css properties since its aware it doesnt need margin etc.


Ich bin ja paranoid - ich wuerde die Zahl noch Größer und komischer machen


A short is a processor word.
Operations on them `*w (word)` family of instructions.

A C short is not guaranteed to be 16 bits. It happens to be 16 bits in x86, because x86 uses 16 bit words.

If you _need_ 16 bits in an integer data type, you should use `int_least16_t`.


This public method was deleted in a previous commit,.  You asked it to be put back fir now as deprecated.  So I did :)


Geht das so - einfach die Zeilen löschen? Und warum?


@RyckRichards, you forgot the closing quote and the `true`.


Removed option to print working directory. Added working directory in place of title instead.


Wootric Kit


Looks good, don't forget to change the Exception in `avocado-vt`...


It might be worth writing an extension on `Int` that generates random numbers instead of casting to/from `UInt32` everywhere.


Test comment


Added option for traversing a current directory. This utilizes another shell script trav.sh.


Another comment

...


Yes. It does.


If you represent notes with an `enum`, you could define in an extension a convenience method that returns the next note.


thanks for that!


OK, prepping a patch, should be easy enough!


Can we use `time.Ticker` here instead of checking `time.Since` on every metric?

Like having yet another `<-reconnectTicker.C` ?


Ah, I missed this. Thanks!


Great work. I like the use of the push here. Another possible solution for this was to add the elipses on the splice method: `longTextArray.splice(numWords, removeWords, "...");`


looks good


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


Great looking code. I like how you printed everything to the console step by step.


Well I wouldn't limit the statuses to what user should see. All `status.mapping` items are valid statuses and the test can finish with them. But others might have different opinion on this one...


Minor changes. Deleted something by mistake in last commit, fixed it here. Some security issues with previous commits (chmod 777), changed to chmod +x. Changed some variable names.


I'd rephrase this as "Could not evaluate type expression %s with parameter bindings %s"


This is the spinner we *currently* use! http://www.virtuosoft.eu/code/bootstrap-touchspin/


If `avPlayer` was never initialized in the first place I'd expect this to just fail gracefully.

This would be more safely expressed as:

    if avPlayer?.playing ?? false {
        avPlayer?.stop()
    }

You could also use a `guard let...` and return early at the top of the method.


Here are some minor fixes you could do:

* Use wpdb->prepare rather than string concatenation for building up queries (Ref: http://codex.wordpress.org/Class_Reference/wpdb#SELECT_a_Variable). It's a good practice and avoids SQL injection by default.
* Probably faster to do `SELECT 1` rather than `SELECT count(TABLE_NAME)`
* Do not need the ending `;`
* It's best to create a PR for this to explain the reasoning behind this change. You could also use the commit message to do so. The essential problem was that the `SHOW TABLES` query and `in_array` lookup is very slow for WordPress multisite installs with thousands of sites.

Good work making this operation faster!

Thanks.


@cmaher push to npm svp


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


Renamed trav.sh and output text file trav.txt to BranchMonkeyTraverse


r+


Pressurestat (lgu1234) - x86_64-MacOS-10.9-clang: Tests Failed (1663 of 1980 tests passed, 0 test warnings)

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


saw approval from jedmccaleb
at https://github.com/graydon/stellar-core/commit/c3a215a5a5746539c0412cbc7bf0e063b5c1ca6a


True. But this can also make it difficult to find a specific question when scanning the text.

______________
Søren Palmund

> Den 20. jan. 2016 kl. 19.22 skrev cholewa1992 <notifications@github.com>:
>
> That was the whole point of it. I at least like that I don't have to keep track of the numbering of questions and subquestions.
>
> —
> Reply to this email directly or view it on GitHub.
>


merging graydon/stellar-core/bug-282-newhist-safety = c3a215a5 into auto


loa should read (l/a) as it does in line 71


graydon/stellar-core/bug-282-newhist-safety = c3a215a5 merged ok, testing candidate = 9d9d4cdc


I'd set a challenge for yourself and try writing Swift code with the minimum number of force-unwrapped optionals possible.


Ahhhh, well then.... try harder. :stuck_out_tongue_winking_eye:


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


:lipstick: by our styleguide this constant (and it's contents, maybe?) should be frozen.


> If you want, I could just switch to the term "short", that would remove all the ambiguity.

That would _STILL_ be ambiguous because a short int is only defined as a processor word.


Why? It just takes the data stored in the variable and does not need to call getHolder multipile times


That issue about peerDependency may have been relevant in the past, but with npm3 it [will no longer automatically install peerDependencies](http://blog.npmjs.org/post/110924823920/npm-weekly-5). Which means, when users specify `express-handlebars` in their package.json, only `express-handlebars` will be installed, without `handlebars` dependency.

If you were always required to install `handlebars` dependency yourself, and then providing an instance of that object to `express-handlebars` lib, yeah sure, it would make sense to keep it as a peer dependency. But as a user of express-handlebars, my expectation is that it will already include it for me so I don't have to install it myself.

In our previous version when upgrading to handlebars v3, you did quite a bit of other changes, plus handlebars v3 I believe had some breaking changes. Looking at the release notes of v4, I couldn't find any breaking changes, just bug fixes, and besides that these compatibility notes:

- Depthed paths are now conditionally pushed on to the stack. If the helper uses the same context, then a new stack is not created. This leads to behavior that better matches expectations for helpers like if that do not seem to alter the context. Any instances of ../ in templates will need to be checked for the correct behavior under 4.0.0. In general templates will either reduce the number of ../ instances or leave them as is. See #1028.
- The = character is now HTML escaped. This closes a potential exploit case when using unquoted attributes, i.e. <div foo={{bar}}>. In general it's recommended that attributes always be quoted when their values are generated from a mustache to avoid any potential exploit surfaces.
- AST constructors have been dropped in favor of plain old javascript objects
- The runtime version has been increased. Precompiled templates will need to use runtime of at least 4.0.0.

And from 4.0.3:
- ] can now be included in [] wrapped identifiers by escaping with \. Any [] identifiers that include \ will now have to properly escape these values.

If you think these are big enough changes I can remove tag 2.0.2 and create a new tag 3.0.0.


Great job.
Some other options, and just to mention them, because what you did work as well.
Add it on the spice method: `longTextArray.splice(numWords, removeWords, "...");`
Or instead of `concat` use `push`


All kidding aside, yes... the implementation details should be moved to the implementation.

I'll tackle that unless you prefer to do it yourself.


Edited version of the Dialog script. This one makes no use of dialog, For POSIX systems without dialog installed.


add the word "template" to this so it doesn't actually run it.  Unless you intend for it to run.


OK.


Why are even discussing on this? I define the term "word" as a 16bit binary number. I could also define the term "xajihaodwa" as a 3bit number if I wanted. There's no need to discuss on this.


Once you implement an `enum` to represent notes, perhaps you could define a `struct` that represents a scale and holds an array of your note values.

Since there's a consistent mathematical relationship between all the notes in a scale, you could have a convenience initializer that initializes a scale given a root note instead of duplicating this logic for all 12 keys.


> Why are even discussing on this? I define the term "word" as a 16bit binary number. I could also define the term "xajihaodwa" as a 3bit number if I wanted. There's no need to discuss on this.

Then add that definition to the top of the standard


fixed via https://github.com/PerseusDL/lexica/pull/13


I'd rename to "computeParameterBindings"


If string was an error, I assume someone is passing in something that is not a string?


tvOS SDK


Updated names from previous commit. Still checks to see if dialog is installed, if not, it continues using this script, otherwise, it runs BranchMonkey.sh


Hey guy, do you like HikariCP with Jooq ?
From what point jooq generated Author etc.. ?


This would be a bit clearer if you had a dedicated `Scale` struct.

`var scales: [Scale]?`


https://github.com/omnirom/android_device_oppo_msm8974-common/blob/android-5.1/Find7Parts/res/values-fr/strings.xml
Türkçeye çevirdiğiniz dosyayı bu şekilde values-tr klasörüne atarsanız daha doğru olur. Sizin yaptığınız şekilde yabancı kullanıcılar anlayamaz.
Teşekkürler


Just saw this.  I think we should remove callback and check from arguments.  I'm putting up a PR today for CameraRoll that also moves to promises tomorrow.  I'll cc you on this so you can see it.


using 1,1,1 the mob basically jumps z levels every tick if that is unimportant then disregard this.


For the current commit the following, added to `C4Layer`, never prints for rotation or transform:

```
public override func actionForKey(key: String) -> CAAction? {
    if key == "rotation" {
        print("r")
    }
    if key == "transform.rotation.z" {
        print("t.r.z")
    }
    print(key)
    return super.actionForKey(key)
}
```

It only prints the following:

```
delegate
opaque
delegate
delegate
opaque
position
bounds
backgroundColor
opaque
onOrderIn
```


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (956 of 956 tests passed, 0 test warnings)

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


The traverse file which is used to traverse a current working directory in BranchMonkey.sh / BranchMonkeyBasic.sh


Please do, thanks!


Seems like a good candidate for a dictionary that's indexed by the root note of the scale.


Broadcast of the session start notification may incur a delay if the SDK is being started


Please add getGLAccount() again as deprecated delegate to getGLAccounts.


These could also be stored in a dictionary indexed by the root note of the key.


As you can see, it says ``Word: 16bits / 2bytes binary number``


@cheeseng sure thing!
Please see my [sentinel repo](http://github.com/massenz/sentinel).


So hmm, it occurs to me that we'll have two sets of signatures after this is run: a set of originals with their hashes calculated with the new algorithm, and a set of old ones with the old hashes (but signatures recalculated using the algorithm suggested).

I need to think a bit about what the best way forward is here, we might want to recalculate the old hashes as well. Give me a day or so. :)


Now a field for config isn't necessary anymore.


@masters3d Oh, I was wondering why I wasn't showing up here: https://github.com/exercism/xswift/graphs/contributors

I fixed it... I think.


PS: And please put the link here so we can merge them at the "same" time.


Sorry, all good. I'm was going commit by commit and started at the end, not the beginning ;)


page 5 from Murukesan14: "Generally, frequency domain analysis is done in four frequency bands: ultra-low frequency (ULF) (0–0.0033 Hz), very low frequency (VLF) (0.003–0.04 Hz), low frequency (LF) (0.04–0.15Hz), and high frequency (HF) (0.15–0.4 Hz)." Why upper bound is 0.5?


It would have made sense to have changed *id* to a *Node* too to completely remove the *.ToString()* calls from the caller.


e.g. the chakracore advising, and news about our acitvities, e.g. we're gonna run an ed program or something.


iya ahahaahah


@Morwenn  heh, and I *just* sat down to work on the titles and depths. Thanks.


Good catch, thanks! Explains #17 and https://github.com/ParsePlatform/ParseUI-Android/issues/122


?Extra space in the message between ")" and "requires"?


És recomanable utilitzar wp_die (també pots passar un missatge)


LGTM


PlantCompBaseClassAgain (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed (1727 of 2056 tests passed, 0 test warnings)

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


Clearly, since we are having this discussion I _can't_ see.
So make it bold.
And make an index.


## This is a markdown file


Done in 7d50114292d95b0f9bfc037aa11118e4c280eaa8.


@timmyliu This was not required on NVIDIA and AMD GPUs because all elements accessing `workspace` were within a warp /  wavefront ensuring that the operations were synchronous. On CPUs (atleast on Intel), this is not the case which caused value errors. The same is true for all the barrier statements in this commit.


/* ******to be conform to polylang 1.8 and avoid warning in debug mode ********* */


The SSIS Framework Community Edition.v12.suo is a binary file that you can remove from git and add to the .gitignore file. I think some of the github tools in Visual Studio can setup a default .gitignore file for VS related files that you typically do not need to check into version control


The index is the first thing of this markdown file.


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.650.06d35c127cd61c6257bd5273b1d978f988323a51](http://build.palaso.org/viewLog.html?buildId=68201&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3324, ignored: 125 Build time: 00:05:05


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (957 of 957 tests passed, 0 test warnings)

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


It's still not bold.


Plus, your index is borked.


wkwk, sheharyar.me/logo_small.png nih bisa. di repo nya gak ada config. mungkin disetting di member area domainnya?


@Plotzy How are things looking on round 1?


Ajoute ``related_name='+'`` pour éviter de surcharger le namespace d'un ``User``.
On pourra le retirer si nécessaire par la suite.


Hi, Chris. The set of visible initializers affects overload resolution. Please revert this and only do it as a fallback.


@nicklockwood @foghina @klvs
Yea, I figured I was missing something like that. Sorry for the bother & thanks for the tip!


mind that the validation of toleration value should be the same requirement with taint value.


ok


AND! word is not in the index.


unack, about the commit message: You forgot a space between `Result:` and the rest of the message :)


How to install GNU Plot on MacOS

First, get brew installed

```
$ ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)"
==> This script will install:
/usr/local/bin/brew
/usr/local/Library/...
/usr/local/share/man/man1/brew.1
==> The following directories will be made group writable:
/usr/local/.
==> The following directories will have their owner set to Pepe:
/usr/local/.
==> The following directories will have their group set to admin:
/usr/local/.

Press RETURN to continue or any other key to abort
==> /usr/bin/sudo /bin/chmod g+rwx /usr/local/.
==> /usr/bin/sudo /usr/sbin/chown Pepe /usr/local/.
==> /usr/bin/sudo /usr/bin/chgrp admin /usr/local/.
==> /usr/bin/sudo /bin/mkdir /Library/Caches/Homebrew
==> /usr/bin/sudo /bin/chmod g+rwx /Library/Caches/Homebrew
==> /usr/bin/sudo /usr/sbin/chown Pepe /Library/Caches/Homebrew
==> Downloading and installing Homebrew...
remote: Counting objects: 3950, done.
remote: Compressing objects: 100% (3796/3796), done.
remote: Total 3950 (delta 32), reused 2093 (delta 21), pack-reused 0
Receiving objects: 100% (3950/3950), 3.21 MiB | 0 bytes/s, done.
Resolving deltas: 100% (32/32), done.
From https://github.com/Homebrew/homebrew
 * [new branch]      master     -> origin/master
HEAD is now at 42bcd0f dynamodb-local: fix typo in desc
==> Installation successful!
==> Next steps
Run `brew help` to get started
```

Then, install GNU Plot

```
Martinez-Home-iMac:Downloads Pepe$ brew install gnuplot
==> Installing dependencies for gnuplot: libpng, freetype, fontconfig, jpeg, libtiff, gd, lua, readline
==> Installing gnuplot dependency: libpng
==> Downloading https://homebrew.bintray.com/bottles/libpng-1.6.20.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring libpng-1.6.20.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/libpng/1.6.20: 17 files, 1.2M
==> Installing gnuplot dependency: freetype
==> Downloading https://homebrew.bintray.com/bottles/freetype-2.6_1.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring freetype-2.6_1.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/freetype/2.6_1: 60 files, 2.5M
==> Installing gnuplot dependency: fontconfig
==> Downloading https://homebrew.bintray.com/bottles/fontconfig-2.11.1.el_capitan.bottle.4.tar.gz
######################################################################## 100.0%
==> Pouring fontconfig-2.11.1.el_capitan.bottle.4.tar.gz
==> /usr/local/Cellar/fontconfig/2.11.1/bin/fc-cache -frv
🍺  /usr/local/Cellar/fontconfig/2.11.1: 449 files, 2.9M
==> Installing gnuplot dependency: jpeg
==> Downloading https://homebrew.bintray.com/bottles/jpeg-8d.el_capitan.bottle.2.tar.gz
######################################################################## 100.0%
==> Pouring jpeg-8d.el_capitan.bottle.2.tar.gz
🍺  /usr/local/Cellar/jpeg/8d: 18 files, 712.4K
==> Installing gnuplot dependency: libtiff
==> Downloading https://homebrew.bintray.com/bottles/libtiff-4.0.6.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring libtiff-4.0.6.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/libtiff/4.0.6: 259 files, 3.4M
==> Installing gnuplot dependency: gd
==> Downloading https://homebrew.bintray.com/bottles/gd-2.1.1.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring gd-2.1.1.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/gd/2.1.1: 34 files, 1M
==> Installing gnuplot dependency: lua
==> Downloading https://homebrew.bintray.com/bottles/lua-5.2.4_3.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring lua-5.2.4_3.el_capitan.bottle.tar.gz
==> Caveats
Please be aware due to the way Luarocks is designed any binaries installed
via Luarocks-5.2 AND 5.1 will overwrite each other in /usr/local/bin.

This is, for now, unavoidable. If this is troublesome for you, you can build
rocks with the `--tree=` command to a special, non-conflicting location and
then add that to your `$PATH`.
==> Summary
🍺  /usr/local/Cellar/lua/5.2.4_3: 82 files, 687.2K
==> Installing gnuplot dependency: readline
==> Downloading https://homebrew.bintray.com/bottles/readline-6.3.8.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring readline-6.3.8.el_capitan.bottle.tar.gz
==> Caveats
This formula is keg-only, which means it was not symlinked into /usr/local.

OS X provides the BSD libedit library, which shadows libreadline.
In order to prevent conflicts when programs look for libreadline we are
defaulting this GNU Readline installation to keg-only.

Generally there are no consequences of this for you. If you build your
own software and it requires this formula, you'll need to add to your
build variables:

    LDFLAGS:  -L/usr/local/opt/readline/lib
    CPPFLAGS: -I/usr/local/opt/readline/include

==> Summary
🍺  /usr/local/Cellar/readline/6.3.8: 40 files, 2.0M
==> Installing gnuplot
==> Downloading https://homebrew.bintray.com/bottles/gnuplot-5.0.2.el_capitan.bottle.tar.gz
######################################################################## 100.0%
==> Pouring gnuplot-5.0.2.el_capitan.bottle.tar.gz
🍺  /usr/local/Cellar/gnuplot/5.0.2: 44 files, 2.2M
```

Easy enough!

The script still generates some errors but the PNG files are created

```
usage: ./iostat-readwrite-requests.sh <iostat.log> <disk name> <out.png>
procesing iostat-server1.read.01202016_1738.out for device xvdap1, plotting xvdap1-requests.png
usage: ./iostat-readwrite-throughput.sh <iostat.log> <disk name> <out.png>
procesing iostat-server1.read.01202016_1738.out for device xvdap1, plotting xvdap1-throughput.png
usage: ./iostat-queue-length.sh <iostat.log> <disk name> <out.png>
procesing iostat-server1.read.01202016_1738.out for device xvdap1, plotting xvdap1-queue-length.png
usage: ./iostat-queue-size.sh <iostat.log> <disk name> <out.png>
procesing iostat-server1.read.01202016_1738.out for device xvdap1, plotting xvdap1-queue-size.png
usage: ./iostat-service.sh <iostat.log> <disk name> <out.png>
procesing iostat-server1.read.01202016_1738.out for device xvdap1, plotting xvdap1-service.png
```


I must have been sleeping when I wrote that. You are right.


Likewise, the following allows `actionForKey` to be called, but has no effect:

```
public class C4Layer: CALayer {
    internal var _rotation: Double = 0.0
    @NSManaged public var rotation: Double

    public override func actionForKey(key: String) -> CAAction? {
        if key == "rotation" {
            let transformKey = "transform.rotation.z"
            let animation = CABasicAnimation(keyPath: transformKey)
            animation.configureOptions()
            animation.fromValue = rotation
            animation.delegate = self
            return animation
        }
        return super.actionForKey(key)
    }

    public override func animationDidStop(anim: CAAnimation, finished flag: Bool) {
        if let animation = anim as? CABasicAnimation {
            if let key = animation.keyPath where key == "transform.rotation.z" {
                print(_rotation)
                _rotation = animation.toValue as! Double
                setValue(animation.toValue, forKeyPath: key)
                print(_rotation)
            }
            print(animation.keyPath)
        }
    }
}
```


The 1st level entries are bold

Why is it borked? Do you look at the file in markdown or in raw mode?

You can find the definition of the term "word" in the "Definitions" section ;) (which is in the index)


Adding the ./ here forces the user executing the script to be in the directory as NodeInfoExtract-1.0. Having it on the path no longer matters.

If we are including NodeInfoExtract-1.0 in the same directory as KamanjaClusterInstall.sh then KamanjaClusterInstall.sh should determine its own directory and look for NodeInfoExtract-1.0 there.


Je ne pense pas que ça soit nécessaire d'avoir un champ ``number``, surtout qu'il pourrait être incohérent avec le ``date_started``. Je propose qu'on ajoute une propriété ``number`` qui calcule le numéro de la manche automatiquement en fonction du nombre de manches dont ``date_started`` est plus petit que le ``date_started`` courant.

Je pense que tu peux aussi ajouter un ``title`` ici. Tu peux sans doute déjà définir ``__str__`` comme étant ``'Manche {} - {}'.format(self.number, self.title)``


I can start switching it out to that one now if you'd like?


you also need to add a `rotation` property to `C4Layer` and make it `dynamic`


Shouldn't the order be vice versa? Now you switch the format strings:
```python
# should be
result.loaction_repr(),
repr(result.origin)
# instead of
repr(result.origin)
result.loaction_repr(),
```
Would be also nice if you put the argument in a separate line, so doing
```python
func(paramA,
     paramB,
     paramC)
```
instead of
```python
func(paramA,
     paramB, paramC)
```


Do we want to add 3.5 to our list of supported versions? Pypiserver works well for me in 3.5.


Keine mutable map verwenden


I had some trouble, having accidentally initialised a git-repo in a folder 2 levels up. To solve this problem I delete the .git and started the repository 'from scratch'. This is the work from Day-1 (with Elaine) and Day-2 (with Godfrey) as one commit. For the original repo please see https://github.com/tcpickard94/old-boris-bikes


![](https://camo.githubusercontent.com/448f517cdbb915a731f4ab9d879bb77b23f08016/687474703a2f2f692e696d6775722e636f6d2f4635616247434b2e706e67)

Note the lack of "Word"


Not perfect for me. Changed to 50 and the program stopped responding (my
screen did the "turn to grey" thing).  I currently have 30 thumbnails.

On Thu, Jan 21, 2016 at 5:39 AM, Krzysztof Marczak <notifications@github.com
> wrote:

> Works perfect!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/buddhi1980/mandelbulber2/commit/5f571d1da265b740b61a04dd619a43128312d65f#commitcomment-15562515>
> .
>


I looked up the history of the file, and I think this is appropriate:
https://sourceforge.net/p/netatalk/patches/146/


Nice.

If you have a branch with all you changes, We can prepare a version and push it in curse.

Can you use [curse](http://www.curse.com/ws-addons/wildstar) tool to update you addon in China ?


Pourquoi ``approved`` ? Elle sera implicitement approuvée si on l'attache à une manche, non ? A moins que tu ne souhaites que les lexpagiens puissent aussi proposer des questions (mais je pense que ça devrait plutôt se faire via le forum, c'est un peu overkill que d'envisager un système de propositions de questions pour l'instant).


This function would need an own test^^


Ich denke man solle dem user die Möglichkeit geben selbst zu pagen


Can you explain why `declaredParameters.size() != actualParameters.size()` isn't always an error? Why is the case of `actualParameters.isEmpty()` special? (It may easier for me to understand if you can provide a concrete example)


```
dynamic public var rotation: Double {
    didSet {
        setValue(rotation, forKeyPath: "transform.rotation.z")
    }
}
```


aha... No ja myślałem że ma sprawdzać czy są takie same...

W dniu 20 stycznia 2016 19:21 użytkownik Bartek <notifications@github.com>
napisał:

> Nie może być, ponieważ to nie sprawdza czy obiekty mają takie same pola,
> tylko to, czy obiekt sprawdzany jest DOKLADNIE tym samym, który był
> utworzony wczesniej. Paweł w którymś z commitów napisał co trzeba zrobic
> zeby ten sql connector dzialal. Poszukaj w jego commitach.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BartoszBaczek/WorkshopRequestsManager/commit/45bc3584657447b0fc29459a5df1b388a3bbf22d#commitcomment-15565558>
> .
>


Can we tag versions please? This has caused many issues. Cannot access protected properties.
```
Spreadsheet_Excel_Writer_Worksheet::$selected in \vendor\pear\spreadsheet_excel_writer\Spreadsheet\Excel\Writer\Workbook.php on line 534
```
```
Spreadsheet_Excel_Writer_Worksheet::$title_rowmin in \vendor\pear\spreadsheet_excel_writer\Spreadsheet\Excel\Writer\Workbook.php on line 876
```


Nit: In the interest of self-descriptive methods, I saw this function and thought, "what does it do with the dags once it finds them?"  Checked the docstring: it finds them.  Checked the code: It logs a warning.  Can you move that information up from the code?  Either rename the function "_log_warning_on_dags" or update the docstring to something like "Find any DAGs within the given block structure and log a warning.  This will help us remove support for dags in the future."


Hi Saleh,

Can you please remove ./ in front of the script?

Thanks

Pokuri

From: William Tarver [mailto:notifications@github.com]
Sent: Wednesday, January 20, 2016 11:13 AM
To: ligaDATA/Kamanja
Subject: Re: [Kamanja] Fix KamanjaCluster Script (81db375)

Adding the ./ here forces the user executing the script to be in the directory as NodeInfoExtract-1.0. Having it on the path no longer matters.

If we are including NodeInfoExtract-1.0 in the same directory as KamanjaClusterInstall.sh then KamanjaClusterInstall.sh should determine its own directory and look for NodeInfoExtract-1.0 there.

—
Reply to this email directly or view it on GitHub <https://github.com/ligaDATA/Kamanja/commit/81db375d35b6325c6d8767a2b10e059b3675c374#commitcomment-15566843> .  <https://github.com/notifications/beacon/AITH1O8-av4E8lQLAoj1O5yUfk-wpRT6ks5pb9PRgaJpZM4HI47v.gif>


'clean' more like add more whitespace


TeamCity Chorus :: chorus-precise64-master Continuous [Build 2.6.571.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=68196&buildTypeId=bt323) outcome was **SUCCESS**
Summary: Tests passed: 931, ignored: 56 Build time: 00:16:54


@jennybc Let me know if this works well for your use case.

@hadley @yihui Any feedback/enhancements you can suggest are most welcome!


I'd rephase this as "Number of parameters for declared type %s don't match actual type %s"


Besser 2 Methoden. Eine die einen String nimmt und eine die keine Argumente hat


It has been pushed to the 3.4.x branch


foreach verwenden


will do


Would be nice if you provide documentation here, I think you've already seen this style in our code :D
```python
def add4(a):
    """
    Adds 4 to the given number.

    :param a: The number to add 4 to.
    :return:  a + 4
    """
    return a + 4
```


Can remove `expiration`, `path`, `domain` & `secure` from the signature & usages in the function body, as we won't use it.


You don't need to document parameter `self` btw :P


Better yet, mention it in the option parsing help.


"declared type %s doesn't match actual type %s"


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


Yes sure np

On Wednesday, 20 January 2016, ligadata-pokuri <notifications@github.com>
wrote:

> Hi Saleh,
>
>
>
> Can you please remove ./ in front of the script?
>
>
>
> Thanks
>
> Pokuri
>
>
>
>
>
> From: William Tarver [mailto:notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>]
> Sent: Wednesday, January 20, 2016 11:13 AM
> To: ligaDATA/Kamanja
> Subject: Re: [Kamanja] Fix KamanjaCluster Script (81db375)
>
>
>
> Adding the ./ here forces the user executing the script to be in the
> directory as NodeInfoExtract-1.0. Having it on the path no longer matters.
>
> If we are including NodeInfoExtract-1.0 in the same directory as
> KamanjaClusterInstall.sh then KamanjaClusterInstall.sh should determine its
> own directory and look for NodeInfoExtract-1.0 there.
>
> —
> Reply to this email directly or view it on GitHub <
> https://github.com/ligaDATA/Kamanja/commit/81db375d35b6325c6d8767a2b10e059b3675c374#commitcomment-15566843>
> . <
> https://github.com/notifications/beacon/AITH1O8-av4E8lQLAoj1O5yUfk-wpRT6ks5pb9PRgaJpZM4HI47v.gif>
>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ligaDATA/Kamanja/commit/81db375d35b6325c6d8767a2b10e059b3675c374#commitcomment-15566925>
> .
>


Let the #treta begin!

![](http://geradormemes.com/media/created/0x0u77.jpg)


Also we don't use a dot at the end of the commit message, just uses space :D


same here, and for the unused `return`


There is also some content in the 4.1 Section.

ohhh, now I can see what you mean with borked. I only looked at the file in Atom using the inbuilt Markdown Preview package in Github Flavored Markdown mode. It looked like this:

![contents](https://cloud.githubusercontent.com/assets/2488440/12460248/222d2e78-bfb3-11e5-948d-f20a1ecbf2b4.PNG)


Consider resetting the top/left CSS properties on the legend-container instead, so we don't need to do any calculations.


C'est un many-to-one ici, pas un one-to-one (pour la saison).
Donc utilise plutôt ``ForeignKey``.

Idem pour le ``user``.


no sure but i may be related

On Wed, Jan 20, 2016 at 11:05 AM, Rob Hudson <notifications@github.com>
wrote:

> @masters3d <https://github.com/masters3d> Oh, I was wondering why I
> wasn't showing up here:
> https://github.com/exercism/xswift/graphs/contributors
>
> I fixed it... I think.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/robtimp/xswift/commit/03ec8e212c7b396f7a303a6a40734b429dc4fbda#commitcomment-15566647>
> .
>


Well, let’s say we have one vinyl file, `file`, and an array of vfiles, `cli.files`, which has one item in it. Then, to set the transformed content of the vfile back onto a vinyl file, you’ll execute something like:

```js
var file = someVinylFile;
var files = [toVFile(file)];
// process...
file.contents = new Buffer(files[0].contents, 'utf8');
```

The messages relating to virtual files can be found at `vfile.messages`. Read the docs [here](https://github.com/wooorm/vfile#vfilemessages). You may want to use [vfile-reporter](https://github.com/wooorm/vfile-reporter).


You're using a light theme.

And yes, the GFM parser between Atom and Github differ.


[optparse](https://docs.python.org/2.7/library/optparse.html) is deprecated. The recommendation is to use [argparse](https://docs.python.org/2.7/library/argparse.html) instead.


I wonder why this didn't bite you?  Maybe I customized my Coq faces?


'.report' will not work with new url_for, must use 'report'.


@zertosh @babel any use?


Opened issue #871.


@zertosh @babel any news?


This option was removed for 3.14.


@tobscure I assume these large changes to the dist file must be coming from more recent Bower dependencies on my side?


Yeah, I thought about doing it that way, and I felt like accepting garbage might be bad. But the `ds2 gdb` thing makes sense


line after reading `e_dry` should read `coeff_fric` to satisfy the new part.config


Cela va être lourd de computer cela à chaque fois, non?


I still don't get where this is coming from. saw it a couple of times now


There you go, should be fixed now.


Add badge


@spoonikle Working on lacun, descension spawn today. the rest of pack I'll enable tonight


This should be `if (name)`.


If I'm understanding correctly, what would only work until the user has moved the legend. The ResizeHandle dojo thing sets inline style rules that use top, bottom, height, width, etc. I assumed we wanted to preserve the user define shape and position when minimizing.


Better.

BUT WORD IS STILL NOT THERE


It could be useful, but I don't think I can do it in near future.


Looks like arparse support was added in django 1.8.  See https://docs.djangoproject.com/en/1.9/howto/custom-management-commands/


ok. I also removed this one back in Nov: 69dfb5d818f6a8b73695593fe11243ddc3c9a5c8

I'll change that to a warning too


`real coeff_fric;` should be declared here


"hair-repair products"?


would be more clear to me if you used vehicle-relative terms, like port/starboard or left wing and right wing
I read the existing comment as motor 1 on right wing and motor 2 on left wing. I assume that's wrong.


This language is imprecise. Are those files just concatenated together?


Ok, it doesn't make much sense anyway. There is `operator const char*()` which can be used to constrict std::string or any other for that matter.

And while we are already discussing this. I think `GetStdString()` is inconsistent with `operator const char*()` because one will convert int, the other won't. I personally think that it shouldn't convert anything like that behind developers backs. Though this is not my decision to make. Similar case is with other operators.

``` cpp
operator char() { assertTypeOrInvalid(TypeString); return *u.asString; }
```


Aren't some of them internal only? Like `START` and `RUNNING`? `status.user_facing_status` was filled considering the `RESULTS` UI line.


Huh? what is being protected?


got rekt


Since you need to do a small fixup, can you change this to read: "Find the keycode of the key with a given name" - we don't explain "keyname" anywhere, so I'd prefer not to use that term.


Please add java doc comment for parameter customViewKey.


Here too, the parameter can be just "name".


Hier wären getrennte Testfälle (`it`) mit getrennten Beschreibungen besser


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1026.06d35c127cd61c6257bd5273b1d978f988323a51](http://build.palaso.org/viewLog.html?buildId=68200&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:35


Yes. The COLUMNS option was removed in a subsequent check in after it was removed in Mambu


Not sure you can use the photo, btw. Whoever clicked the picture doesn't seem to have the rights to upload derivatives of the original work. I would suggest using a different image.

Also, it's better to get something in landscape mode for the website. :)


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


:+1: thanks


Looks good.


Done

On Wednesday, 20 January 2016, Saleh Abddallah <saleh@ligadata.com> wrote:

> Yes sure np
>
> On Wednesday, 20 January 2016, ligadata-pokuri <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> Hi Saleh,
>>
>>
>>
>> Can you please remove ./ in front of the script?
>>
>>
>>
>> Thanks
>>
>> Pokuri
>>
>>
>>
>>
>>
>> From: William Tarver [mailto:notifications@github.com]
>> Sent: Wednesday, January 20, 2016 11:13 AM
>> To: ligaDATA/Kamanja
>> Subject: Re: [Kamanja] Fix KamanjaCluster Script (81db375)
>>
>>
>>
>> Adding the ./ here forces the user executing the script to be in the
>> directory as NodeInfoExtract-1.0. Having it on the path no longer matters.
>>
>> If we are including NodeInfoExtract-1.0 in the same directory as
>> KamanjaClusterInstall.sh then KamanjaClusterInstall.sh should determine its
>> own directory and look for NodeInfoExtract-1.0 there.
>>
>> —
>> Reply to this email directly or view it on GitHub <
>> https://github.com/ligaDATA/Kamanja/commit/81db375d35b6325c6d8767a2b10e059b3675c374#commitcomment-15566843>
>> . <
>> https://github.com/notifications/beacon/AITH1O8-av4E8lQLAoj1O5yUfk-wpRT6ks5pb9PRgaJpZM4HI47v.gif>
>>
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/ligaDATA/Kamanja/commit/81db375d35b6325c6d8767a2b10e059b3675c374#commitcomment-15566925>
>> .
>>
>


`printf("coeff_fric = %lf\n", coeff_fric);`


Here you can use the internal function `XkbKey`.


I didn't actually write the JSON one yet, but that is what this should be.


This can be an `#ifndef Q_WS_MAC` rather than an empty if with an else.


'.report' here is incompatible with new url_for implementation, must use 'report' instead.


I feel every 30 minutes is too often.  Maybe every hour.


@danielmoore doh... i'll be the first to admit this was a frantic "oh s***!" resolution and sucks... we can do better.  i've filed mongodb-js/extended-json#32 to resolve.


The name (get_*) makes this look like it should be returning a course object, but we're using this for its side effects...


Thanks! :+1:


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1238.95ca714213a72034be6661676951c99ef81fea32](http://build.palaso.org/viewLog.html?buildId=68207&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 842, ignored: 7 Build time: 00:05:55


Does our pylint fail if we have an empty __init__ module?  This seems like a wasted comment.


This TODO should be resolved.


why not put the operator and value validation directly in validateToleration? value validating are closely related to the operator validating, and since both validating of these two fields are very simple, is much readable to have the validating in the same function instead of increasing the calling depth while the actual functions are very simple.


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


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


"numeric literal"


Doesn't `xkb_atom_lookup` return 0 on failure? This is supposed to check for an alias if the initial lookup fails.


111768812_VRFNoLoadOAFlow (Nigusse) - x86_64-MacOS-10.9-clang: OK (2051 of 2051 tests passed, 0 test warnings)

 * 3 tests had: AUD diffs.

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


Please remove this method as the COLUMNS feature wasn't implemented in the end.


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


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.661.06d35c127cd61c6257bd5273b1d978f988323a51](http://build.palaso.org/viewLog.html?buildId=68202&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3303, ignored: 106 Build time: 00:05:33


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


Please remove this case.


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


ah, got it


Hah no worries, I was just foolin around :)

> On Jan 20, 2016, at 11:32 AM, Luke Rodgers <notifications@github.com> wrote:
>
> Update readme with compiler warning update
>
> @jcolicchio previously updated the compiler warning comment to be more in line with correct code he created which is now merged into the develop branch.
>
> As the develop branches readme.md is the thing people see when they hit this repo, I want for it to make sense and tell the original story.
>
> I've updated the contributing section to raise the fact that these compiler warnings have been fixed, and for it to maintain my original story.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/convenient/mazeman/commit/43177034857694557029e8245fbc08615ae4aea5


Il y a 10 manches maximum par saison à priori, donc le nombre d'items présents dans la db ne sera pas bien grand. On va de toute façon devoir lister les manches d'une saison par date, donc je m'attends à ce qu'on place un index sur ce champ. Sans doute peut-on faire cela sur la super-classe car on aura besoin du même type de tri pour une saison.


@wibeasley, do we need to describe these **png** images as well?


`threadedScan` can't be tested very cleanly because it runs forever and there aren't any "hooks" for controlling it (except the sleeper, but that's a bit janky). I'll brainstorm ways to improve it when I add clean shutdown to the hostdb. The basic pattern seems pretty easy:
```go
for {
    select {
        case <-hdb.closeChan:
            return
        case <-time.After(scanSleepTime):
            // scan hosts
    }
}
```
the problem is that `scanSleepTime` isn't a constant, it's set randomly on each iteration. Also, `time.After` leaks memory, although this isn't a huge concern. So a bit more work is needed.


Great to hear, Josh! :smile:
Let me know if you run into any problems or want anything changed.


Thanks for the changes :+1: Truly amazed someone took the time haha.


Kalau di index.html nya bisa aja sih di paksa ke `S` pake javascript tapi klo di file seperti `file.js` dan `file.css` tidak bisa tapi saat di load ke main situs libscode bisa pakai both protokol. kalau emang sudah seperti ini tidak apa-apa lah toh saat di load ke halaman libscode sudah ke https :smile:


Hi, i like your project MPMPlayer. I think do something like this.
Your project is an excellent example for bass and QT libraries.
I am planning to do a small local radio player.
Your help would have meant a lot in the beginning. I can not compile the program the reason probably missing icons.


ok je vire


empty line :smile:


This seems like something generic enough. Probably we have similar helpers in other classes as well.


You got it.


Thanks!


@nelsonic
I think there is a problem here.
I think we need more like
```
html2Amp = require('../lib/templates.js').html2Amp;
```
similar prob in other test..
on the tube for the next hour..


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


Replace the Sleep with a c.Flush()


Both canonical names and aliases are interned, so if the atom lookup fails you should fail immediately.

If the lookup succeeds, it can be either canonical name or alias (or neither). If it's an alias, then `XkbResolveKeyAlias` will resolve the alias to a canonical name, otherwise you should stick with the original.


Please remove this case.


@istos if you wouldn't mind giving this commit a sanity check i'd appreciate it :)


Switch to the canonical representation for parametric types: `array(varchar(%s))`, `map(boolean, double)`


The COLUMNS and all related to it code was already changed and is not in the final version (in subsequent check ins in this pull request it was removed).


@polivilas *remove unnecessary whitespace, did you even use your eyes


Все хорошо эта функция будет работать, но есть пару вопросов и комментариев.


@ctsims


Зачем здесь перед a два восклицательных знака?


@agrebenkina, формат должен быть такой же как у тебя, т.е
 Наставник: [Имя](ссылка на профиль).


all tests pass:
success: https://travis-ci.org/stellar/stellar-core/builds/103674883


Sorry, just noticed this -- it should be _semantics_, not semantic. We can fix that in a follow up commit though.


Что если мы захотим проверить на число второй параметр? Может лучше сделать вспомогательную функцию с проверкой на число?


This is good actually, because the port situation here was complicated and probably not correct.


@kkhenriquez esta es la pagina que necesito ayuda con el CCS. avisame si me puedes ayudar para mandarte los Mockups de lo que tenia pensado y vemos que se te ocurre y se me puedes ayudar.


@ctsims Hm, just noticed the CommCareApplication_().getCurrentVersionString() method. Do you want all of the information that's in that, or just the version?


Sorry, I saw you have done that already. And I think we need to add comments about the formatting requirements in API definition.


I wonder what means "lit"


Ah, good idea. So this script is bazel on your path, and then tools/bazel is a script in your repo that redirects to the appropriate bazel-real. Cool.


neue Version, ich werde es mal damit bis morgen Abend laufen lassen.

    if($buf) {
      Log3 $hash->{NAME}, 5, "SIGNALduino/RAW (ReadAnswer): $buf";
      $mSIGNALduinodata .= $buf;
    }
    $mSIGNALduinodata = SIGNALduino_RFR_DelPrefix($mSIGNALduinodata) if($type eq "SIGNALduino_RFR");

      $idx = index($mSIGNALduinodata,"\003\n");
      if($idx != -1) {
        $cut = 1;
        if($mSIGNALduinodata =~ m/\002.*\003\n/) {    # vollstaendige Signal Nachricht
          Log3 $name, 3, "$name/RAW (ReadAnswerCut002003): $mSIGNALduinodata";
          if(defined($regexp)) {                      # kein parse wenn von doInit aufgerufen
            SIGNALduino_Parse($hash, $hash, $name, $mSIGNALduinodata);
          }
          $mSIGNALduinodata =~ s/\002.*\003\n//;
        } else {                                      # Signal Nachricht ohne Anfang
          Log3 $name, 3, "$name/RAW (ReadAnswerCut003 $idx): $mSIGNALduinodata";
          $mSIGNALduinodata = substr($mSIGNALduinodata, $idx+2);
        }
        Log3 $name, 3, "$name/RAW (ReadAnswerCutDone " . length($mSIGNALduinodata) . "): $mSIGNALduinodata";
      }

    # \n\n is socat special
    if($mSIGNALduinodata =~ m/\r\n$/ || $anydata || $mSIGNALduinodata =~ m/\n\n$/ ) {
      if(!defined($regexp) || $mSIGNALduinodata =~ m/$regexp/) {
        if ($cut == 1) {
          Log3 $name, 3, "$name/RAW (ReadAnswerCut): $mSIGNALduinodata";
        }
        return (undef, $mSIGNALduinodata)
      }
    }


Что будет если мы захотим добавить еще одно действие в цикле, но программист забудет поставить скобочки? Лучше переписать с использованием фигурных скобочек.


Why not inheriting from `threading.Thread` ?


I disagree with these two commits, can you please revert them? See my comments at https://talk.openmrs.org/t/uiframework-error-missing-property-only-in-development-mode/4342/7?u=darius
Also, can we have a ticket for this? :-)


Absolutely, I did exactly how you've suggested. But here is a problem. Linter produces a messages and it looks like every new message is pushed into `VFile.message` property makes a new file and I get something definitely weird.

Here is what is happening if we do `console.log(file)`:

```
[ VFile {
    contents: '…',
    messages:
     [ [Object],
       [Object],
       … … …        // here are really too many messages
       [Object],
       [Object] ],
    filePath: { [Function: filePath] toString: [Circular] },
    history: [ '/windows/Projects/_Repos/gulp-remark/README.md' ],
    directory: '/windows/Projects/_Repos/gulp-remark',
    filename: 'README',
    extension: 'md',
    sourcePath: '/windows/Projects/_Repos/gulp-remark/README.md',
    data: { 'remark:cli': {}, mdast: [Object], 'remark-lint': [Object] },
    move: [Function: locked],
    offsetToPosition: [Function: offsetToPosition],
    positionToOffset: [Function: positionToOffset],
    quiet: true } ]
```

And w this continues until it throws an vinyl file error:

```js
throw new Error('File.contents can only be a Buffer, a Stream, or null.');
```

I dunno how to correctly explain what is going on. Just try to run it.


Hm okay, I've made some changes which should work based on my understanding of what you've said.


Issues mentioned in this commit (1998c28ff92ced211d8e758f0fee45cbf2a4bd11):

* [Issue #4](https://bugs.mtasa.com/view.php?id=4)
* [Issue #900](https://bugs.mtasa.com/view.php?id=900)


Issues mentioned in this commit (1ad5eefd5f82ce505a397b0d39ccfba858dc8f7c):

* [Issue #900](https://bugs.mtasa.com/view.php?id=900)


Latest 2.1 is 2.1.12


System.ot


I'll give it a shot?  Any battery changes keeping the 2nd core on all the time?


То же самое, что и для предыдущего цикла. Если не нравится, что это будет занимать много места и if else раздуется, то можно вынести все во вспомогательные функции, которые мотом можно будет переиспользовать в случае необходимости, а не дописывать, очередной if else.


Please remove parameter.


PlantCompBaseClassAgain (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1507 of 1507 tests passed, 0 test warnings)

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


fast-forwarding master to auto = 9d9d4cdc


    OK, so do you want PKCS11_ecdh_derive to be renamed as  an internal
    routine only?

    On 1/20/2016 12:40 PM, Michał Trojnara
      wrote:

      I think we should do it the OpenSSL's way, i.e., only use the
        engine for the lowest level for key derivation. It should be
        sufficient for most practical applications.
      —
        Reply to this email directly or view
          it on GitHub.

    --

 Douglas E. Engert  <DEEngert@gmail.com>


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


We have multiple IDs in archive so I tried to word it a bit generically.


Why are you reformatting the license to not be like the other files in tensorflow?


Access Snippets


Aaaaaawwwwesome.  Thanks @lefticus.

Already in action, @mjwitte. @rraustad, @wfbuhl, @EnergyArchmage.  [This](https://github.com/NREL/EnergyPlus/commit/8ca1d67c4b8a5b65f53be82dcc555999d9b5391c#commitcomment-15567283) shows the audit file _does_ show diffs, even though we have decided previously that audit diffs alone shouldn't cause a test to be marked as a failure.


lit is a past tense and a past participle of the verb "to light". So lit_state is "the state in which the LED appears... lit!"


Are you saying that, with the code at dde66cfac7a0c5d59f08e5cd8f31f21fd6ad41e5, you’re getting those results, or are there other changes?


on Centos 7/aarch64 this appears to be printed as "Time zone:".  I have no idea why there's not a parseable output format.


Some authors do not recommend inheriting from threading.Thread. In this particular case, can you explain what would be the advantage of doing so?


this is specifying the order in which they appear in the tar file and the hash of those files (if order matters, which I assume it does) must be the same.


Kombinationsfeld


@americool DWMR (done with my review). This is good - both the code and regarding your question about what files to include. Press on!


Testing them one by one before adding them to list, default would be 2.2.4


Reverted at https://github.com/openmrs/openmrs-module-uiframework/commit/fb725216928d71c0200a9c68a65f502cb0507ea5 :-)


How about 'washu washes your car while you shop!' ?


I'm going to bet on bad testing on my side :)


I'd push the ratom conditional into the block like this:

```
atom = xkb_atom_lookup(keymap->ctx, name);
if (atom) {
    xkb_atom_t ratom = XkbResolveKeyAlias(keymap, atom);
    if (ratom)
        atom = ratom;
}
```

but the logic is correct, yes.


I get this if I do this:

```js
var file = someVinylFile;
var files = [toVFile(file)];
// process...
file.contents = new Buffer(files[0].contents, 'utf8');
```

Just run `gulp` at last commit in this branch and look what will happens.


Wouldn't it make sense, to use the min/max value of the distance_sensor-topic here?


Since zlib is required by both unix and mac, we can consider pulling zlib unconditionally. Can you move the code to line 59?


Is the self-invoking function really necessary?


Wow, that's... disappointing. I can tweak that regexp. Is there a machine I can test it on?


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


@samehta These feature specs are well named and well divided. :+1: ace work


NOTE:  Else is not needed. Code at line 79/80 where it was used was simplified.


# Assignment 1 Grading Rubric (15 points total)

## Screenshots

- [ ] Assignment 1 contains directory of files for user (0.5 pt)
- [ ] Repository contains screenshot of [github][] account (0.5 pt)
- [ ] Repository contains screenshot of [Slack][] session (0.5 pt)
- [ ] Repository contains screenshot of [Atom][] installed on users computer (0.5 pt)
- [ ] Repository contains screenshot of screenshot showing that user installed [linter-flake8][] (0.5 pt)
- [ ] Repository contains screenshot of the Anaconda [REPL][] on user's computer (0.5 pt)
- [ ] Repository contains screenshot, in the repository, displayed by Github Desktop (0.5 pt)

## jupyter notebook

- [ ] Repository contains ipython/jupyter mini-tutorial as ipynb file (0.5 pt)
- [ ] ipynb Task 1 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 2 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 3 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 4 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 5 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 6 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 7 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 8 is presented with a description explaining what is occurring (0.5 pt)
- [ ] ipynb Task 1 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 2 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 3 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 4 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 5 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 6 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 7 appears to be answered correctly (0.5 pt)
- [ ] ipynb Task 8 appears to be answered correctly (0.5 pt)
- [ ] You can successfully re-run every step of the ipynb notebook (3.0 pt)

[screenshot]: https://en.wikipedia.org/wiki/Screenshot
[git]: https://git-scm.com/
[github]: https://github.com
[Slack]: https://biolprogramming.slack.com
[Atom]: https://atom.io/
[linter-flake8]: https://atom.io/packages/linter-flake8
[syllabus]: https://github.com/biolprogramming/syllabus
[ipython]: http://ipython.org/notebook.html
[jupyter]: http://jupyter.org/
[REPL]: https://en.wikipedia.org/wiki/Read%E2%80%93eval%E2%80%93print_loop


Est-ce que tu sais si on peut brancher un signal sur un modèle abstract ?

Si oui, ne serait-il pas intéressant de déjà définir un handler pour un *pre_save* sur ``DatedWithMessagesAndAuthor``, afin de calculer ``date_started`` en fonction de ``starting_message`` (si non null), idem pour ``date_ended`` ?


That works!!!!

On Wednesday, 20 January 2016, Chris Kelly <notifications@github.com> wrote:

> How about 'washu washes your car while you shop!' ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Dewale/321superteam.github.io/commit/1d9c372ec7046cae36f60a3dd56ebc374c63f0a0#commitcomment-15567676>
> .
>


Hmm... this PR will break older clients. Let me think about whether there's anything we can do to mitigate compatibility issues.

We could present `VARCHAR(MAX)` as `VARCHAR` so that older clients continue to work, but it's not going to solve all issues (e.g., SELECT 'foo') would have a type of VARCHAR(3), so older clients wouldn't be able to deal with it anyway.


Issue5276 (wfbuhl) - x86_64-MacOS-10.9-clang: OK (2038 of 2047 tests passed, 127 test warnings)

 * 129 tests had: AUD diffs.
 * 136 tests had: EIO diffs.
 * 8 tests had: ESO big diffs.
 * 4 tests had: Table small diffs.
 * 3 tests had: ESO small diffs.
 * 3 tests had: MTR big diffs.
 * 5 tests had: Table big diffs.
 * 1 test had: MTR small diffs.
 * 1 test had: ERR diffs.

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


This should send and log an event to update existing browser windows similar to the `twenty_four_hour_time` feature; you can test that you did this right by adding a test in test_events.py similar to the existing one for twenty_four_hour_time.


I was thinking that the size & position should always reset but it's more of a design decision.


This case should rather throw an exception.


how are we supposed to dominate the world without a manual? :P


we don't have one right? it's just a container div?
It looks like it adds some clarity to the layout having a <main>. What do we need to do to make that work?


Hatten wir uns nicht auf 4 **Spaces** geeinigt?! :crying_cat_face:


Sinon on mets 2 décorateurs


No, it was decided to add a valid implementation method on all devices because system may be not work properly with default values. Gazebo implementation is already done.


@demhydraz see the new commit, I think it should be fine now ;)


*Dated* -> *Datestamped*.

Il faudrait réfléchir à un nom plus évocateur pour ce modèle abstrait ^^
(Ce n'est pas bien important pour l'instant, c'est un refactoring interne qui n'aura que peu de conséquences le jour où on trouve un chouette nom).


@pekohl DWMR. This looks solid. Good. Ship code.


You should use your name here (and 2016 above).


thanks @ramalho I need to improve my english ;-)


Rename to run_tests_with_atexit_manager instead


@pekohl DWMR. More solid code shipping.


Thanks Damien! Awesome. This was just the first commit I received from a PR :)


You can remove these.


@edolstra Do we, perhaps, need a new release (e.g. 1.11.2) with this fix?


How about using `^` instead of `*`? Should be more strict.
http://www.w3schools.com/cssref/sel_attr_begin.asp


Would be nice to test an alias as well.


Pretty sure that was automatically generated code.  What issue does this resolve?


Je pense qu'on devrait tout de même avoir un champ ``visible`` (et/ou ``opened``), qu'on peut mettre à jour automatiquement en fonction de la valeur de ``date_started`` (et/ou ``date_ended``) vu qu'on va faire énormément de requêtes impliquant un tel statut.


Why times is divided by 1000? For different divisors there are different periodogram.
Same question for intervals: fror different divisors of intervals we have different features.


a simple `HDDDEV="$((HDDDEV + 1))"` is sufficent because empty strings are treated as 0 in arithemtic expressions


Je propose aussi de changer ``date_started`` et ``date_ended`` respectivement par ``start_date`` et ``end_date``


111768812_VRFNoLoadOAFlow (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2057 of 2057 tests passed, 0 test warnings)

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


Does Groovy do non-local returns from closures at all?  Does this just return from the `plugin.executions.each {}` block or from the whole code?

We should also update the README documenting this the execution ID rewriting, and now this optional skipping.


@ExSport darn, thanks for the reply :)

@UniversalMediaServer/developers feel free to use the video testing suite to test your renderers anyway. I bet most haven't been tested with the more rare combinations like opus or vorbis. Also if you have any suggestions for other combinations I'm open to that. Because of the way 7Zip compression works, we can add many more variations without affecting the filesize much, it's great at matching similarities.


Yes, we do need an internal function for raw key derivation.  It probably does not need to be as complex as `PKCS11_ecdh_derive()`.


@kd0aij Well I wasn't sure if people would know where to look from if I said e.g. "left wing".


I found in some others projects this way of doing (inheriting `threading.Thread`) so I wonder why not doing this way here (because it was quite unconventional for me). It will be great if you could find why "Some authors do not recommend inheriting from `threading.Thread`"

on the other side I found using Google and "python not inheriting threading.Thread" this article
http://www.ibm.com/developerworks/aix/library/au-threadingpython/
"The threading module was designed with inheritance in mind"

I'm just asking for my personal knowledge


Yes, functionally equivalent.  Modifying the not_found method resulted in the following stack trace for cases where the request is Ajax and the ontology object is nil:
```ruby
NoMethodError (undefined method `flat?' for nil:NilClass):
  app/controllers/application_controller.rb:360:in `get_class'
  app/controllers/concepts_controller.rb:83:in `show_tree'
```
I added the if...else block to fix that.


It is because ruby was crushed because it cannot load libimf.so which is linked with stringio.so and the path is specified by LD_LIBRARY_PATH environment vairable. --disable-gems prevents to load stringio.so.


I think it's `g[dbserver]`, not `[g]dbserver`.


Actually, I think it should this -

1. If the request index is null, use the operation index (which may be null, that's fine)
2. If the request index is not null and the request index does not equal the operation index, use the operation index
3. Else use null (which will use the request index, which may be null, that's fine)


サンプルなんであんまり指摘なしでいきますね～
後々気になるであろうところをメモ程度に残しておきます（今回のPRでの対応は不要です）。


I think this message is enough for product that only have one license type.

For XenServer usage, do we need to have more output here? So user can easily understand which license they need to by?


If an optimization that allocates vregs applies we will need to resize. We increase the size exponentially, relative to the initial number of vregs, so just a few times maximum. Also, mempool allocs are cheap and short lived.


Just realized this code (not yours) is wrong. If the log level is `kLogLevelDebug`, we should push `--debug`; `--debug-remote` should be used if we have `kLogLevelPackets`.


should check toleration.Value not nil/empty here.


Lel

Sent from my Cyanogen phone

On Jan 19, 2016 5:44 PM, "connie m." <notifications@github.com> wrote:

wow

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


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


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


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


@math-u, the generator has now been updated to work with both Foundation 6 and Foundation 5 projects, along with several other enhancements (ECMAScript 6 support with Babel)! Run `yo`, select `Update your generators`, and select `generator-foundation-gulp-browserify` and then give it a whirl!


Nothing,Still nothing!


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


This adds a new entry for the ISO being downloaded from a torrent magnet file.  Users should still check the validity of the file as usual, of course.  I apologize for having fucked up the git commit message -- I did this through the Web in Github directly.


This is pretty cool. I wonder if you could clean up your code a little bit. It's usually frowned upon to chain conditionals more than once ( .. || .. || .. && ...etc. ) but there are some tricks you can use (like switch case statements and the ternary operator). Good job! :+1:


the error message here should be invalid operator, not invalid value.

I guest you have some misunderstanding with switch statement in golfing? There is a implied break at the end of each case. Ref https://golang.org/ref/spec#Switch_statements


这样修改后元素不跳动了，但是快速拖拽元素又不流畅了，鼠标脱离元素就不能拖了


camel표기법


I think it might be better to do a `switch (argv[1][0]) { case 'g': [...]; case 'p': [...]; case 's': [...]; }`. That might end up accepting things like `ds2 goat_cheese [...]` but makes the code a bit more concise, and allows users to do `ds2 gdb` which should be valid.

I really don't feel strongly about this one, so feel free to do as you please.


It's nice to make JA friendly to blind people. It is the 21st century after all.


ASIHPdev (shenb) - i386-Windows-7-VisualStudio-12: Build Failed

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


It looks good! I agree that there could be a problem when not running with elastic ip's but I think your approach solves the problem. In addition, given the nature of this project, I think majority of the cases will be for using it inside an ec2 instance with an elastic ip assigned, not the other way around.


I'd love to see you try to clean up your code and reduce the amount of if/else statements you use. Try looking back through the pre-work for some other options you can use. Overall great job! :100:


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1202](http://teamcity.monogame.net/viewLog.html?buildId=13793&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2056 of 2057 tests passed, 0 test warnings)

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


I like that we can get rid of this big block of diff :smile:


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


Can this colour be made into a variable? It is an official SilverStripe brand colour (#005a93)


I think you didn't update the README to account for this flag changing.


Fixes #35


Really beautiful job! I wonder if there's a way you could clean up all those if/else statements and "switch" them for a cleaner option.... Stellar work! :star2:


恩，我是因为iOS相关的许多工具比如fastlane,shenzhen,homebrew,cocoapods 都是ruby写的，所以想好好学一下ruby，之前看了一下ruby编程第二版，发现ruby写命令行工具真的非常方便，多谢 @NaixSpirit


Yeah I've got these changes in my development branch, thanks.


In the AbstractFactory pattern each factory should create a different type of object


This is something we can figure out later, but I'm not 100% sure that platform mode combined with reverse connect is valid. And if it is, I'm not sure that the slave workers would be reverse-connecting to the debugger as well.


Fixes #36


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


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


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


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


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


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


I love the quit functionality: nice attention to detail!

In `getWinner()` you could use some else statements to break the chain of ifs and prevent errors... OR... you could "switch" the if/else logic for something even cleaner. ;)

Great job! :100:


恩，我也是通过Fir-CLI 工具认识到作者的，所以才看到xxbao的，非常佩服作者呀， @NaixSpirit


When you say that `list` is needed consistently, what do you mean?  What is the error produced by using a tuple here instead of a list?  I'm not sure what you mean by the control flow link, since if you use `()` the control flow lines go away.


This test will hang as all the threads in the pool (`Executors.newFixedThreadPool(2)`) have been exhausted and are waiting on previous resolves to complete.


hi.
can we make our ui look like this style
http://map.norsecorp.com/


This is odd to include `err` in the `if` but let the modification escape. Don't do this.


Can someone please review this one? The CVE reference I've used is the Red Hat one. There were a couple of others used by Oracle, and IBM.


This condition is already checked above. Try to reuse that.


> Unfortunately, I think the Write method is dead code. As far as I can tell it is not called directly (except for tests) and io.Copy prefers calling ReadFrom over Write.

This is true, but that is the interface.


Validator was added by Joe, but as you can see from the discussion in JIRA there were issues with validating URI. That is why the JIRA was reopened.


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


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


the function here, I think should be validating toleration operator as well as value.

according the description in design doc
>     // operator represents a key's relationship to the value.
    // Valid operators are Exists and Equal. Defaults to Equal.
    // Exists is equivalent to wildcard for value, so that a pod can
    // tolerate all taints of a particular category.

I think the `Defaults to Equal` means if the value of operator is not given, we regard it as Equal, does not mean it accepts all the other string values of operator.


然而中文是不用空格的，这样也不会影响格式化输出。


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


109490574b-UnbalancedFlowWarning-4293-PlanB (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Is that link ok? I've got "Could not create download: Could not find any trackers" when trying to load it into `rtorrent`.


RangeError: Invalid string length


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


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


I guess as long as it's not just the ckeditor config but also the other editor configuration (I.E. enable_filemanager / enable_sitemap) this would be ok.


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


Please see discussion in JIRA about the validator and why there are issues using it. Basically, we can't control URI structure of a system that does not belong to us and AMQ URIs can get very complex and only AMQ knows which URI is valid and which is not. So instead of using the validator we are simply delegating to ConnectionFactory and ensuring that user sees the appropriate exception message instead of NPE (as was the cause for JIRA in the first place).


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


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


I am making sure my verification worked.


Opened issue #20


[萤石](https://zh.wikipedia.org/wiki/%E8%90%A4%E7%9F%B3)。
的确，[中文Minecraft Wiki也是这么写](http://minecraft-zh.gamepedia.com/%E8%90%A4%E7%9F%B3%E5%9D%97)，但是我觉得一定是Wiki弄错了


fast-forwarding master to borshop-integration-307-gh296-fix = 0076f317


fast-forwarding 2.x to borshop-integration-314-goldrush-018-2.x = d9bed3e8


sadsd


@olegz  - yes, based on the discussion in the jira, I was expecting this to be tweaked, but not removed. I can see how you interpreted the discussion as complete removal, though.


111639676-UniqueThermostats (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2348 of 2384 tests passed, 0 test warnings)

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


Should say `registers reloader config`


FYI, the app secret has been reset.


这是什么意思?


This change should be functionally equivalent, right?


Small clarifying question above but other than that this looks good to me


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


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


It is against the dev branch. We would need to merge it up.


TeamCity Gensokyo Lobby :: Lobster Server [Build 0.0.1.98](https://pixelated.design/build/viewLog.html?buildId=336&buildTypeId=GensokyoLobby_LobsterServer&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:19


TeamCity Gensokyo Lobby :: Lobster Core [Build 0.0.1.95](https://pixelated.design/build/viewLog.html?buildId=337&buildTypeId=GensokyoLobby_LobsterCore&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 267 Build time: 00:00:16


TeamCity Gensokyo Lobby :: Lobster Client [Build 0.0.1.40](https://pixelated.design/build/viewLog.html?buildId=338&buildTypeId=GensokyoLobby_LobsterClient&guest=1) outcome was **SUCCESS**
Summary: Running Build time: 00:00:18


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.1202](http://teamcity.monogame.net/viewLog.html?buildId=13793&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 560, ignored: 6 Build time: 00:25:58


maybe just inline all this as `.map(columnHandle -> checkType(columnHandle, TpchColumnHandle.class, "columnHandle"). getColumnName())`


Problem with mochawesome: RangeError: Invalid string length
    at join (native)
    at Object.stringify (native)
    at stringify (D:\Document\Company\licaibao\Test\node_modules\mochawesome\nod
e_modules\json-stringify-safe\stringify.js:5:15)
    at Runner.<anonymous> (D:\Document\Company\licaibao\Test\node_modules\mochaw
esome\lib\mochawesome.js:114:20)
    at Runner.emit (events.js:129:20)
    at D:\Nodejs\node_modules\mocha\lib\runner.js:735:12
    at D:\Nodejs\node_modules\mocha\lib\runner.js:626:9
    at next (D:\Nodejs\node_modules\mocha\lib\runner.js:283:14)
    at Immediate._onImmediate (D:\Nodejs\node_modules\mocha\lib\runner.js:319:5)

    at processImmediate [as _immediateCallback] (timers.js:367:17)

why！


OK; let me update the guides back to the control-h/control-q commands.


SpriteTextures were created by using .png files that were hand drawn.


if the path is a directory


Arpit Agarwal
Indian University Bloomington


@olegz  - did you intend on removing the ```StandardValidators.URI_VALIDATOR```, because unless I'm reading the pr wrong, it is still doing some cursory validation, which we should test for.


might be able to drop this. we don't really support windows


Hey! Thank for forking the repo.

Instead of introducing an artificial `RAW_GET` HTTP method, I suggest analyzing the response's ` Content-Type`. If it is `application/json` do the JSON processing (like the original code does), otherwise return the raw response as a string.

Does this make sense?


Sorry guys, I think I messed up. Was I suppose to $ git put origin master at the end or $ git push origin (branch_name_i_created)? I am not seeing an option to make a pull request, so I think I messed up.


Ok, I will remove it.


It's scary because I don't know how much you're joking haha


Oh yeah, I actually did that in the other place


why was it removed? I'm kinda confused because you still have the actions and store folder, but removed flux from dependencies, isn't it included in flux?


How about providing a little more detail about your use case? Which version
of mochawesome? How are you calling your tests? How many tests are you
running? Has it worked for you in the past? Is this a new issue? What
version of node?

I'm happy to help troubleshoot but you gotta give me more than just an
error message and a "why!"

On Tue, Jan 19, 2016 at 9:36 PM XUYUYANG <notifications@github.com> wrote:

> Problem with mochawesome: RangeError: Invalid string length
> at join (native)
> at Object.stringify (native)
> at stringify
> (D:\Document\Company\licaibao\Test\node_modules\mochawesome\nod
> e_modules\json-stringify-safe\stringify.js:5:15)
> at Runner. (D:\Document\Company\licaibao\Test\node_modules\mochaw
> esome\lib\mochawesome.js:114:20)
> at Runner.emit (events.js:129:20)
> at D:\Nodejs\node_modules\mocha\lib\runner.js:735:12
> at D:\Nodejs\node_modules\mocha\lib\runner.js:626:9
> at next (D:\Nodejs\node_modules\mocha\lib\runner.js:283:14)
> at Immediate._onImmediate
> (D:\Nodejs\node_modules\mocha\lib\runner.js:319:5)
>
> at processImmediate [as _immediateCallback] (timers.js:367:17)
>
> why！
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/adamgruber/mochawesome/commit/4707e256df6a273dc56e34906ec9bf2b36f86433#commitcomment-15547035>
> .
>


I'm still deciding whether or not to make this an option.  Or to just automatically delete empty buffers as soon as they are empty.

The main argument for providing this option would be if someone is using DeleteEventAfterFirstRetrieval and they expect a high, steady stream of events on categories that are being actively longpoll-ed for.  In that case, we'd keep deleting EventBuffers and then re-creating them the next event that comes in.  It might be more efficient to wait a little while before deleting the empty buffer.  BUT, on the other hand, this is probably a pre-mature optimization and the amount of work the code would have to do to enforce the delay before deleting an empty buffer might be more expensive than actually re-creating a new buffer.

So I'm on the fence about this, but starting to lean towards getting rid of this option and just automatically deleting empty buffers.