Hmmm, I don't have any samples.

I do like your approach of checking just for the "14" as it should be a
good catch-all.

I have seen 401 responses for sure.

I used to try to treat each one separately, but found it too cumbersome.
Now I simply try to do the refreshAuthToken() for any http exception, even
if it did not include the 14.

// Sean

On Fri, Feb 19, 2016 at 1:02 PM, Yaima <notifications@github.com> wrote:

> @StrykerSKS <https://github.com/StrykerSKS> I was trying to get all
> possible scenarios for the responses, but I was able to trigger only the
> 500 from my local app. Do you have any response samples available?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SmartThingsCommunity/SmartThingsPublic/commit/13d9137c9afeca7c5de0753403de9ae4434bfbf1#commitcomment-16211100>
> .
>


Btw, a non-zero tailCall could be one of PREFIX_TAILCALL_EXPLICIT or PREFIX_TAILCALL_IMPLICIT.  That is the call could be an explicit ".tail" prefixed call or an implicit tail call.  In both the cases tailCall param would be true given the logic at caller of this method.

I think the reason why we have to not model user-call implemented intrinsics as GT_INTRINSIC node is
a) For back compat reasons on desktop .Net 4.6/4.6.1
b) It will be non-trivial task or too late to re-materialize a surviving tail prefixed GT_INTRINSIC as tail call in rationalizer.

Please capture the above as comments.


:smile:


New doc is so much better. Gives context and lets the developer get to the code post haste, where the true information is.


OK, I'll bug them.


[style] These should appear on the same line `], [`.


[](url)Eric


Can we limit each string to max 128 bytes here?


$VAR[LogoVarListInContainer] was removed with https://github.com/Ignoble61/skin.aeon.madnox/commit/edd1cd0e46f1f1a09393380914519cf57ce24b70 ;)


should be


This looks good!


Oh yes! Good catch!


it would be nice to also have methods with Runnable as parameter instead of TimedTask in order to support methods without return type


> it not worth because one-two lines here to drop support for 0.10/0.12.

you make a great point. I'll revert


IOWorkers


[](url)my last name: Frimpong.


Does it always work to transform both of these up, especially if there will be two chubbyy speeches in a row?


> 4


this is already available in the libc crate, please use it from there instead.


OK, tested building on master. And it works for me fully:

===
Unknown compiler version - please run the configure tests and report the results
msvc.archive ..\..\..\bin.v2\libs\program_options\build\msvc-14.0\release\link-static\threading-multi\libboost_program_options-vc140-mt-1_60.lib
msvc.link ..\..\..\bin.v2\tools\auto_index\build\msvc-14.0\release\link-static\threading-multi\auto_index.exe
msvc.manifest ..\..\..\bin.v2\tools\auto_index\build\msvc-14.0\release\link-static\threading-multi\auto_index.exe
common.copy auto_index.exe
..\..\..\bin.v2\tools\auto_index\build\msvc-14.0\release\link-static\threading-multi\auto_index.exe
        1 file(s) copied.
common.mkdir ..\..\..\dist
common.mkdir ..\..\..\dist\bin
common.copy ..\..\..\dist\bin\auto_index.exe
..\..\..\bin.v2\tools\auto_index\build\msvc-14.0\release\link-static\threading-multi\auto_index.exe
        1 file(s) copied.
...updated 78 targets...
===
Oh, I saw you reverted on master.


викинь цей коміт, залиш тільки два інших


``eric


I'm not sure if this is working on a server, I'll test it later


Worked.
Although our constants are not exactly perfect, it appears. But whatever.


This should be fine, modulo using the function from libc instead of writing our own extern decl for it.


upload data set for a new MRS for RTVS sample.


> 10*2


``hello my best friend!


Maybe "Semantic" is a more apropos differentiator? We can do more with `picture` els  than hi-ppi.


No worries! I was just joking. Thanks for working on this


Nit, use consistent google-style docstrings for public methods


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


Hey I have an important Job Interview for some company Ken Zen or something like that and they want me to build this exact same thing... Do you mind if I clone it and turn it in with your name on it?


Since this is just a sample file, it might make sense just to have it in a big comment in the source code.
Lets revisit next week after we know what'll happen with the server side.


Looks good to me.


> 12*7= 84


Duplicate info? It's in a macro file, so maybe "Gallery" would suffice?


It should be `mapSize.real[i]^2`...


1> 2


This looks good.


We decided initially to have a fixed framecount to be 50, which we decided after testing with different frame counts, 50 seems to be quite smooth and giving a good impression of the footage. So we should try to keep this idom. Other then that you don't want to do a check on a file on disk at runtime, if we decide to have arbitrary number of thumbnails, we should put it into the meta data which will be coming from the database/json


It might be known, yeah, but if even if not it was probably fixed in 0.8.


Yeah @StrykerSKS, that approach got us into some trouble before. It is better to refresh only if we need to.


I can read the code and I'd like to propose a different name but I'm still at a loss. Can you describe what `unit` does in natural language?


OK, cool. won't worry about it then! :smile:


Un fallo tontín en el nombre de la Url ;)


Hi @wooorm, I think adding alex.js and alex.min.js to .gitignore was a mistake :)


Can we indent HTML structure, for better readability?

Once there are a bunch of these I would love to convert a couple to Jade, and run them by the team.


está muy `quemado` esto... si le cambias el orden se friega... más bien listeners al finalizar los eventos..


Oh, I see, there are a bunch of macros per file. "Galleries"?


Does it matter whether that line comes before or after the previous one `getStyleClass().add("styled-text-area");`?


I've sent FDs over MQ with this, and FDs do have several nul bytes in them, so this should be good. Also, there's no mention of any special NUL handling in http://linux.die.net/man/3/mq_send -- it should orient just on the buffer length.


We might also consider moving the docs/examples to a wiki page where we would also document the server side.


:wat: x2
`contentMusic = $document[0].getElementById('music-moi');`
`getElementById` <- si lo estás usando


asking because i'm not familiar with DataInputStream: is the DataInputStream necessary? from the documentation, it looks like it's needed for reading java primitives, which i don't think is needed here. might be worth trying to just pass the fstream straight into the InputStreamReader and seeing if that still works.

it'll make the code look more deliberate, which would look good to other people who look at the code


**Shortlog of HEAD commit is too long.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


Some of these have one trailing star, some have two, some have none. Is there a pattern we should follow?


nit: remove commented out line


Yes, I saw the same. So in my own test app I created (to only test the authentication flow), I have been using this:
if ( (e.statusCode == 500 && e.getResponse()?.data.status.code == 14) ||  (e.statusCode == 401 && e.getResponse()?.data.status.code == 14) ) {

But I plan to change it to your approach and drop the HTTP values and only look for the status.code == 14.


I don't think it matters.


can the GsonBuilder declaration be moved out of the loop so it's not created each loop iteration?


nooooooo, lo que tu pusiste es en el controlador del tree, mas no de la directiva


oh i see - it's named like that because of gson. that's fine - you can ignore my comment :)


Accidentally committed to iter1 instead of iter2. Mistake has been fixed.


No I only want my prompt to show local versions of ruby. Checking `$(rbenv global)` would make sure that either `system` or any other global version of ruby won't show.


OK, it also could be an IP address. I do it now.


I think it's not so good to mix the input value for offset with the bbox that should be computed.  bbox should be an optional out parameter I guess


It looks like my shell messed up my commit message. {;*;} was supposed to be {$($body:stmt);*;}.


hey @rmnl the plugin is throwing an error in the plugin definition.
`Parse error: parse error, expecting `','' or `';'' in /Users/rdinh/projects/wordpress/wp-content/plugins/wordpress-plugin/jw-player/include/jwplayer-api.class.php on line 15`

 Let us know when the plugin is ready to be submitted and we can have @enobt submit it to wordpress vip.

Thank you


I think it is better to keep the `cast` here, so that future graphs all have int64 variables an inputs.


After using babel, namespacing with php is so bad...


could you use `-1isize as uid_t` (`gid_t`) to make it clearer we're using `-1` as defined in the spec? Also if there were a system with signed `uid_t`/`gid_t`, we wouldn't end up doing the wrong thing. Not sure if it's allowed by posix, but hey better safe! :-)


Other than the slight-overhead of shelling out to an additional `rbenv` command, I agree that using `$(rbenv global)` is probably the better way to accomplish this.


var matcher = /^(?:\w+:)?\/\/([^\s\.]+\.\S{2}|localhost[\:?\d]*)\S*$/;

can be simplified to

var matcher = /^(\w+:)?\/\/([^\s\.]+\.\S{2}|localhost[\:?\d]*)\S*$/;

(ie: the first "?:" doesn't do anything.) I would submit a PR for this, but I don't want to download the repo. hopefully this comment will spur action in a different way.


not sure what's going on?!


This looks good after the one change.


I'm not quite sure what the relationship between the .md and source files is. I don't see a picture gallery in the macros below.

Can you tell me a little more about this? What would happen if we eliminated the md file and only tracked documentation in code comments?


Correct.


This can just be self.update_metric(...)


Yes, because of `com.proofpoint.reporting.Key`


This should all be merged, but I may have screwed up, so I'll leave it here.


This should be fatal until a portable `cpuid()` implementation is added.  Something like this:

```
        AC_CHECK_HEADER([cpuid.h],
           [AC_DEFINE([HAVE_CPUID_H], 1, [Define if host has <cpuid.h>]),
           [AC_MSG_FAILURE([*** cpuid.h missing])])
```


same with the ArrayList< Listing >() above


ah... entiendo... peor todavía... no sé como funciona si hay dos elementos del DOM con el mismo id... como sea creo q es mejor sacar esta lógica de la directiva... tal vez puedas hacer que la directiva de los gifs emita un `$broadcast` o `$emit` :notsureif: y la directiva de los sonidos escuche eso... así puedes sincronizar sonido + gif... o q pasa si se hace videos sincronizando eso? así se va a tener la certeza q audio-video está sincronizado @macool


This and update_tag() can be staticmethods since you're not using self anywhere. Also, docstrings please.


+1 for making the legacy behavior optional


hmm, not sure what you mean. Do you mean not mixing `offset` that is in/out parameter with aabb that is only out parameter?


sztos! :+1:


To ten z lekcji xd


Why flip the order?


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


nit: remove unnecessary newlines


Edited to try and work out what's going on!


It's really easy to get something added! Well, except that their CI is pretty unforgiving in exactness. :-)


It looks like these macros are already filled with content. I'm guessing that in the future they'll be templates that take a JS object, but that for now we're hard coding?


:wat:


bueno al momento la música que tendría que ser detenida es la del intro, eso sucedería cuando el usuario decida dar click en el nodo y cambiar de estado e ir al contenido de ese nodo


I don't think that's correct. The code one line below reads 64 bits from the memory and interprets that as an int64, this will give wrong results if the array is not actually in NPY_INT64.


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


:cry: Not an easy fix, but I can't run this simple test of a pure function using `mocha test/codegenTest.js` because...

1. We're testing a function in codegen.js
2. codegen.js uses utils.js
3. utils.js uses Hammer.js
4. Hammer.js initializes internally with a reference to `window` and breaks when window is not defined.

Sometimes I hate JavaScript.


Yeah, the old behavior definitely looks like a bug. Thanks for finding it!


The `ReportCollectionProvider` constructor will do a null check, but the corresponding method in `NamedReportBinder` is missing a check.


It's probably not great.  QoI could definitely be further improved here.


Sure, that's definitely something to consider. The tricky part is we now almost have two types of async data sources. dataService and clientServer just make a request to get all data matching the current filter set, and apply everything else on the client. serverSide and smart include the full filtering, sorting, and paging information in the request and only get a part of the data set.
One possibility would be to have another intermediary data source base class, but it seems like that starts to over-complicate things. Thoughts?


Comment is a cliffhanger


`size` here is a string, holding the name of the variable containing the size in the C code. Casting it does not make any sense.


lmfao


##### ChIPseq_BAMtotagAlign


You would have to increase the version number here.


This should read:

`					<configurationFile>${project.build.testOutputDirectory/orientdb-server-config.xml</configurationFile>
`


I'll take a look, but will comment on the individual PR when it's opened. Thanks for the link!


Properties are defined as key value pairs in a Java .properties compliant format where they are readable employing methods defined in java.util.Properties


consider just using a for i = 0 to listings.length loop, since you need the 'i' anyway. arraylist item access is O(1) so i don't think there's a noteworthy performance improvement of using an iterator here.

yeah i think this sorta corroborates what i was thinking
http://stackoverflow.com/questions/1879255/performance-of-traditional-for-loop-vs-iterator-foreach-in-java


for what is this?


The following doc(s) have been deployed to the [527rsyncl](http://527rsyncl-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/logs/downloading-live-error-logs](http://527rsyncl-static-docs.pantheon.io/docs/articles/sites/logs/downloading-live-error-logs)


quest can be simply backported from WoTLk branch... where works perfectly fine


What would you recommend in this case?


Yesssss, I'm glad the `<var>` catcher is helping! Don't forget that you can limit the scope of the vars by putting an `algorithm` attribute on a container or heading, so that a var used in one algo doesn't accidentally validate an incorrect usage of the var in a different algo.


pedantic nit: this return is not hurting anything but also not necessary


Pushed fixup commit.


New one is Wasp 47 now. I first changed it accidentally to something else
(the one that the trap fit code chokes on).
On Feb 19, 2016 11:00 AM, "Fergal Mullally" <notifications@github.com>
wrote:

> The old example was wasp 47. What's the new one, and why the change?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/barentsen/dave/commit/a0ab837fbeaa2bd858658a35c2d541c8e31400be#commitcomment-16211052>
> .
>


OK, interesting... what motivates this exactly? Is it your reading of the DTD for a job.properties file?


2083_radiance_version_fix_again (rpg777) - i386-Windows-7-VisualStudio-12: OK (2359 of 2395 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.5%25-yellow.svg)


Changed here


Variable is undefined :) (did you mean PathFlagsAudioChannels?)


Ah, sorry. I answered while being on the road by just reading email notification. I didn't notice this question was directed at disabling face detection instead of memory heaps you asked about earlier. You can disable this commit too if it causes issues for you. I don't know why face detection even suddenly woke up at all.

By shimming I meant shim library: https://en.wikipedia.org/wiki/Shim_(computing)

But since there was this mixup with question and answer I don't think my earlier answer is that valid nor helpful.


I don't think join is a good name. 'Join' is just one operation of a `CompletableFuture`, why should this method name focus on just that?


This forward declaration isn't need for the following static inline and can be removed.


no


Variable is undefined :)


adding bacl


The offset parameter is what the user of the api may set, while aabb is passed in with values depending on previous calls. I would prefer something like
```
void draw(const LineLayout& line, glm::vec2 offset, float width, LineMetrics& _measured);
```


Maybe overkill, but you could also assert the following:

```javascript
    it("no argument", function () {
      assert.throws(function () {
        codegen.calculateCumulativeLength();
      }, Error);
    });
    it("no text", function () {
      assert.deepEqual([0, 1], codegen.calculateCumulativeLength(''));
    });
```


You little shit! It's pig fat, not salo


This makes me think of another potential portability problem -- what if `isize` is _smaller_ than `uid_t`/`git_t`?


Will update both.


I wondered about that. I don't think checkstyle should exclude src/test/java.


use generic iterator: Iterator< Product > it = products.iterator();
and remove the cast to Product in it.next();

(note that i'm adding spaces between the arrow brackets in Iterator< Product > only cuz github doesn't show text properly if you don't add spaces.)


Thanks @evadyer!


[style]

```
static inline void
cpuid(unsigned int *eax, unsigned int *ebx, unsigned int *ecx)
{
...
}
```


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


>  plus bonus for eliminating any LoC from the `loader.js` metacode.

Totally agree with that. Not so sure about making `page` calls less explicit here, since there already all over this file. What about the alternative

``js
require( ' + JSON.stringify( module ) + ' )( page, controller.renderElements );'
```

with that second argument function invoked inside `my-sites/themes` after elements have been created (much like it's now on `master`), as discussed in Slack? Arguments to that would then be `router` and `renderer`, which I find kind of appealing, since it captures quite nicely client and server specific differences. (We can totally rename `renderElements` to `clientRenderer`, BTW).


That is a valid point. The watches should only fire when the text actually changes though, so it'll only trigger the other watch once.


Ok, will make that change. Can you explain this offset param and how it is used?


If I'm reading this right...
> Taken from CSS Reference guide: Section "Scene, Parent and SubScene Stylesheets"
>
> The implementation allows designers to style an application by using style sheets to override property values set from code. ... This has implications for the cascade; particularly, when does a style from a style sheet override a value set from code? _The JavaFX CSS implementation applies the following order of precedence:_ **a style from a user agent style sheet has lower priority than a value set from code, which has lower priority than a Scene or Parent style sheet. Inline styles have highest precedence. Style sheets from a Parent instance are considered to be more specific than those styles from Scene style sheets.**

Then it follows that precedence is:

Low Priority <----> High Priority
UserAgentStyle --- Code-set Value -- Scene --- Parent -- Inline Style

So, I'm guessing it doesn't....


They don't seem to have the syscall numbers, but they would probably be welcome. The `syscall(2)` function is defined:
- BSD-like: https://github.com/rust-lang/libc/blob/master/src/unix/bsd/mod.rs#L317
- non-BSD-like: https://github.com/rust-lang/libc/blob/master/src/unix/notbsd/mod.rs#L636

The constants are architecture specific, so if you add them, you'd be doing it somewhere like [src/unix/notbsd/linux/other/b64/x86_64.rs](https://github.com/rust-lang/libc/blob/master/src/unix/notbsd/linux/other/b64/x86_64.rs).


Undefined variable :)


I've tested this changes and it passes all tests


nit: you can have makeListingsIndex return the created index instead of creating it here and passing it in. semantically i think it is more natural if the 'makeListingsIndex' is responsible for creating the Directory instance


113360729_HumidifierOASys (Nigusse) - i386-Windows-7-VisualStudio-12: OK (2102 of 2102 tests passed, 7 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: RDD diffs.

![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)


Creo que se podría definir una variable $columns para no tener que repetirla en todos los UNION


Done.


I feel like there was some reason to exclude src/tests/java, but do not remember what it was.


остаток от деления n-го числа Фибоначчи на m


oh dear, that is a good point. maybe `-1i64` as the largest currently expressible signed type in Rust?


I always use one var statement per function thus am sure about it.


Yes, for now I am just copy and pasting code over. This is very much a first step in the implementation. There are open issues for refactoring and cleaning up the highest priority modules. The goal will be template like macros. Everything will need to be a variable for i18n.


@StrykerSKS I still need to cover the cases where we change set points with an expired token without loosing the request body.


Alternatively we could just drop the server-side data source now that we have a better one. In practice I don't think anyone's ever going to use it


along with a comment? :-)


Another similar back compat difference from Desktop 4.6/4.6.1  due to modelling of intrinsics as GT_INTRINSIC nodes which if survived front-end optimizations could materialize as calls:  It is possible that the call could be an in-line candidate.  Since the call was modelled as GT_INTRINSIC node in-lining logic would not kick in.  Essentially an in-line candidate expanded by .Net 4.6/4.6.1 may not be expanded now if GT_INTRINSIC node survives front-end optimizations.

Note that importer (impImportCall()) tries to determine whether an call is an in-line candidate (impMarkInlineCandidate()) after it attempted intrinsic expansion (impIntrinsic()) and expansion hasn't happened.

One possible fix is that in-lining logic also looks at GT_INTRINSIC nodes marked as GTF_CALL and marks them as in-line candidates and expands them.

Please file a bug to track this issue.


you'd still want to `continue` even if it's just a no match error? I guess you expect gvrs to be empty in that case?


@ratoaq2 how fast are you hitting LTV? maybe they are blocking too much hits per second

I have 12 files eps to search for subs, only the first 2-3 I can search. after that I got: "Could not search in legendastv provider. Discarding for now" - when any connection error happens


Wrong GIT Ignore


Stale comments. Let me get rid of them and save what's valuable. Now's the time.


This fixes the locale dependent unit test failures on OS X.

Thanks @shirou!


Fixed by change in DSL.


Bro Aaahhh .i want join team marsh rom  :+1:


good that you have this explanation here. it's good to be transparent about the reasons for your choices.

if you want extra brownie points, how about adding a small bit of code that gives you some indication of how well your algorithm is performing? i.e. track how many products found no listings, and track how many listings didn't get into the results.jsonl. you could perhaps just System.out.println these stats at the end.

this would tell you how well your algorithm is working, which would be good to know before submitting the code


Will the software improvement be continued with [another update suggestion](https://github.com/GPUOpen-Effects/GeometryFX/issues/3 "Newlines at end of source files")?


I was using this file temporarily to stash old documentation. I probably should have checked the docs first and kept only what's relevant.


972154-NewSimpleChilledCeiling (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (997 of 997 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-28.62%25-green.svg)


I don't see how the precedence rules would apply here, since there is no precedence conflict to resolve. Quite opposite, the two lines work together to achieve the result: one add a style class, the other add a stylesheet, that defines some style for elements of that style class.


New test for GroupBy(Int) to test generality, see #19 .


Only one barcode per Sample possible? Then perhaps we should use .uniqueResult()


This command offends me.


> you'd still want to continue even if it's just a no match error? I guess you expect gvrs to be empty in that case?

There are two ways to drop through:
 1. `NoResourceMatchError`, in which case gvrs is empty
 2. No error, in which case I want to add them.

I'm willing to add an explicit continue if you prefer.


I'm trying it out because usually I tend to agree with the changes you make. Tbh it's not that much of a difference to me, I am not sure yet if I find this better or not. I use vim daily so I'll quickly have an opinion :)


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `setup.py`, line 96.*

The issue can be fixed by applying the following patch:

```diff
--- a/setup.py
+++ b/setup.py
@@ -93,7 +93,7 @@
           version=Constants.VERSION,
           description='Code Analysis Application (coala)',
           author="The coala developers",
-          maintainer="Lasse Schuirmann, Fabian Neuschmidt, Mischa Kr\xfcger"\
+          maintainer="Lasse Schuirmann, Fabian Neuschmidt, Mischa Kr\xfcger"
                       if not on_rtd else "L.S., F.N., M.K.",
           maintainer_email=('lasse.schuirmann@gmail.com, '
                             'fabian@neuschmidt.de, '
```


and here


> Oh, I saw you reverted on master.
>

Right on develop I see:

...found 1 target...
...updating 1 target...
msvc.write-setup-script
C:\Users\John\AppData\Local\Temp\b2_msvc_14.0_vcvarsall_
x86.cmd
...updated 1 target...
...found 1 target...
...updating 1 target...
msvc.write-setup-script
C:\Users\John\AppData\Local\Temp\b2_msvc_14.0_vcvarsall_
amd64.cmd
...updated 1 target...
...found 1 target...
...updating 1 target...
msvc.write-setup-script
C:\Users\John\AppData\Local\Temp\b2_msvc_14.0_vcvarsall_
x86_arm.cmd
...updated 1 target...
Performing configuration checks

     - 32-bit                   : yes (cached)
     - arm                      : no  (cached)
     - mips1                    : no  (cached)
     - power                    : no  (cached)
     - sparc                    : no  (cached)
     - x86                      : yes (cached)
     - symlinks supported       : no
     - junctions supported      : yes
     - hardlinks supported      : yes
     - has_icu builds           : no  (cached)
...patience...
...patience...
...found 2602 targets...
...updating 6 targets...
common.mkdir M:\data\boost\boost\tools\auto_index\build\M:\data
The filename, directory name, or volume label syntax is incorrect.

         if not exist
"M:\data\boost\boost\tools\auto_index\build\M:\data\\" mkdir
"M:\data\boost\boost\tools\auto_index\build\M:\data"

...failed common.mkdir M:\data\boost\boost\tools\auto_index\build\M:\data...
...skipped M:\data\boost\boost\tools\auto_index\build\M:\data\boost for
lack of M:\data\boost\boost\tools\auto_index\build\M:\data...
...skipped
M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost for lack
of M:\data\boost\boost\tools\auto_index\build\M:\data\boost...
...skipped
M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist for
lack of M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost...
...skipped
M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist\bin
for lack of
M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist...
...skipped
<pM:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist\bin>auto_index.exe
for lack of
M:\data\boost\boost\tools\auto_index\build\M:\data\boost\boost\dist\bin...
...failed updating 1 target...
...skipped 5 targets...


It's a pig fat, not salo


Actually since this seem to be x86 specific that's not going to work.  How about adding an `#error` to the non-implemented `cpuid()` function below.  We don't want to accidentally get used until it's implemented.


Ok, just remenber to fix the problem.


Cool. If it's just a reference, does it make sense to keep it outside of git history? I saw it as a new pattern.


I'll change the comment though.


OK, I checked, results:
```$ npm pack && less coininfo-0.8.1.tgz```
![screenshot at 2016-02-19 22 36 36](https://cloud.githubusercontent.com/assets/2633065/13186893/44e5f784-d759-11e5-8f3c-c6187888d2f8.png)

In npm source code: https://github.com/npm/npm/blob/580f88a8183375a9c045529f10bf1aaea75f7bdd/lib/utils/tar.js#L64


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


The offset defines the translation of all quads. It's more interesting when drawing range of glyphs to advance to the next line :)


Colored scoreboard items kinda suck because they can not be localized properly. We either need to give up coloring or provide translation strings for all colored variants :(


Sure thing. I feel obligated to note that these templates originally came from elsewhere, specifically,

PR template: https://quickleft.com/blog/pull-request-templates-make-code-review-easier/

Issue template: https://bl.ocks.org/auremoser/72803ba969d0e61ff070

and I modified them slightly to my repo's needs.


:+1: Yay :)

I never liked `hologram` much myself. Nightshade v1 was super fast proof of concept and I was so beaten down by rails in my first round to replace the asset pipeline that I went with a rails friendly generator.


:hankey:


Awesome changes, keeping them all.


The general aim of the XeTeX class loader is to cover existing use cases (as the mechanism is at best limited). However, here it did seem to me that the old behaviour was defective as `CJ` chars were simply not considered at all. It therefore seemed to me that a change was reasonable.

In terms of the set-up mechanism, I'm not 100% sure I understand. Perhaps you mean something like
```
\ifdefined\XeTeXcharclassID
\else
  \chardef\XeTeXcharclassID=1 %
\fi
\ifdefined\XeTeXcharclassCJ
\else
  \let\XeTeXcharclassID\XeTeXcharclassCJ
\fi
```
_i.e_ with one 'dominant' setting?


LMAO!


Use databinding to bind a model property to the element.


I don't understand this result. Clearly none of these match the input resource? Shouldn't this return an empty list?


References to dirty/store slice in the commit message should, of course, be to maps, not slices.


Never, really. Thanks for catching this. I had decided to remove it. Didn't catch this one.


![win-preferences-single](https://cloud.githubusercontent.com/assets/210240/13187002/bfe83f48-d716-11e5-9bc7-063868c01c1d.png)

++ Looks good


How about, `#error "Requires portable cpuid()"`


remove this


Thanks!


Doesn't the message contained already take care of that? Also this is yet another case where a custom ADT may be a better fit. So we can match on the ADT and have custom errors better represented than through the Ex msg field. I'm all for more verbose message but the nested return type before was very hard to handle. Feel free to change it around in this same PR if you think it can be improved as long as we have a reasonable return type to deal with.


The pointer access can be async-signal-unsafe.


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


[style]

```
return ((eax == 7) && (!!(ebx & AVX2_FLAG)));
```


Looks like that open letter finally paid off, huh?


There's a race here between the check and the add, but it might not matter.


The input kinds are expected to have no version filled in? Please add example in & output to the comment.


Fixed #1


Same comment.


a more foolproof way might be to use jquery's .text() method to add the user input text to the dom, since that method does not allow html and hence automatically escapes


So a more idiomatic way of doing this would be to use the pipe. `state |> Map.has_key? |> reply`


Ditto below.


Turns out libpng is not used at all by mozjpeg (was used by libvips instead), so removing it from this depends cut it entirely from this build. thanks for catching and for the tip about abuild.


@baytechnathan @baytechmichael I do not see this class listed anywhere in the CSS file.

Again, please remove unwanted classes.

@dcancilla tagging you here.


maybe there should be a space in the ":" when you re-create the hader to push to response_headers?


@robwierzbowski :+1: glad to hear you like the top level TODO, that was my proposal for how to leave cleanup notes on files. Ideally, todos are also tied to issues.


> I don't understand this result. Clearly none of these match the input resource? Shouldn't this return an empty list?

Sorry, its a test harness that always returns a fixed result.  The `MultiRESTMapper` logic doesn't care how its constituent `RESTMappers` get a result, it must simply faithfully collate the results.


I get it, we might call it translation then? Or position? I got a bit confused by the name.


I feel obligated to note that these templates originally came from elsewhere, specifically,

PR template: https://quickleft.com/blog/pull-request-templates-make-code-review-easier/

Issue template: https://bl.ocks.org/auremoser/72803ba969d0e61ff070

and I modified them slightly to my repo's needs.


Nice. In general, it's better if actors have coarse APIs over granular ones.


Created issue #28


Nice. In general I'm a big fan of minimal documentation, and using small files and well named functions and vars to self document code.


Much complicated, very WOW.


`20:27 DerpyHooves: ( ͡° ͜ʖ ͡°)`


Cool. Although I think I didn't think it completely through. I think we need to compare the dot product with the previous block. Then make sure that we internally have the  same orientation.

I'll see if I can't create a nasty test case.


Gotcha. OK.

On Fri, Feb 19, 2016 at 11:45 AM, David Eads <notifications@github.com>
wrote:

> I don't understand this result. Clearly none of these match the input
> resource? Shouldn't this return an empty list?
>
> Sorry, its a test harness that always returns a fixed result. The
> MultiRESTMapper logic doesn't care how its constituent RESTMappers get a
> result, it must simply faithfully collate the results.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/deads2k/kubernetes/commit/ea12fa4aa17c7326d5a7a9455210fd51ee78f886#commitcomment-16212151>
> .
>


Ah, I didn't understand the difference between style class and style sheets. Thinking about it more, I see your point: there is no conflict.


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


Good.  Go ahead and remove the `@todo` annotations when we make these updates.


Was there a reason to remove the mention of China National Railways?


Is it possible that the fix was actually in your previous commit? I think that's the one that changed when the render middleware was called (specifically _not_ anymore for any non-isomorphic chunks, i.e. chunks that ignored the `router` argument).

(I kind of liked it better without the repeated `navigation, siteSelection` :-) )


The only time my hair doesn't stand on end with comma styled `var` is when they are hoisted manually:

```javascript
function foo() {
  var bar, baz, froboz;
  bar = 'bar';
  baz = 'baz';
  froboz = 'electric co';
}
```

Otherwise my sense of style and indenting gets all out of whack for example I've seen abused comma `vars`:

```javascript
function init() {
  var a = 'bar',
      onError = function() {
        // 60 lines of code
      },
      onSuccess = function() {
        // 50 lines of code
      },
      // This pattern continued for **hundreds** of lines
      returnValue = 'foo';
  return foo;
}
```

Just make my head hurt... 🤕


На самом деле это не баг Яндекс.Карт, а просто неуникальные ключи в JS-объекте =)


@mgmccarthy At a glance, I don't understand why this using statement is needed, can you remove it?


> The input kinds are expected to have no version filled in? Please add example in & output to the comment.

Will drop out in combination with other pulls.  If you're largely ok with the rest, I'll combine the pulls into a single unit and you can have another shot.

This existed to preserve legacy behavior of choosing "the best" version for return to a client.


I think I've found a better solution: `(0 - 1) as uid_t`!


Right.. I reproduced it now on develop+master. Will try and fix ASAP.


What about `::createEmpty` and `::createRandom`?


It would be helpful to include the "why" in your commit message (you can add it in a new paragraph after the summary).


I really hate context.Transaction.InnerTransaction

That bleeds a lot of the internal state all over the place.


optimizations for BOD on IDEv1.6.7


Looks ok.

-David Russell
APRS World, LLC
http://www.aprsworld.com/

On Fri, Feb 19, 2016 at 11:24 AM, Charlie Detviler <notifications@github.com
> wrote:

> changed the way the url property is handled
>
> may need @davarus to review this
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/aprsworld/wsBroadcast-WebClient/commit/55ab1e1c7c3fdb0c8b54a4f9a17bcbcad7ea227c
>


Like it. Sometimes I measure the complexity/size of a TODO with whether or not to make a story for it. But now that I think about it, anything that's too small for a story I should probably just take care of in the current PR.


What happens if we have multiple collections?


I haven't seen a DTD for job.properties file..   Are you referring to using Hadoop xml option for properties.


Can't resist, I just recently read this great article: http://jalf.dk/blog/2010/03/singletons-solving-problems-you-didnt-know-you-never-had-since-1995/

`mapped_memory_cache` is a perfect example of what it's about -- it's just a namespace for global functions acting on global state. I think I'll try to refactor it just to be sure I'm not imagining it's that simple :thought_balloon:


Why do we have `"Collections": "Users"` here?
I would expect this to be an array, instead.


Hey Greg,
Thanks for taking a look at this! I didn't notice I was missing branch. I submitted those changes you suggested. Take a look and let me know if that works.
Thanks!
Allyson


Yes, that sounds good, thanks.

On Fri, Feb 19, 2016 at 11:48 AM, David Eads <notifications@github.com>
wrote:

> The input kinds are expected to have no version filled in? Please add
> example in & output to the comment.
>
> Will drop out in combination with other pulls. If you're largely ok with
> the rest, I'll combine the pulls into a single unit and you can have
> another shot.
>
> This existed to preserve legacy behavior of choosing "the best" version
> for return to a client.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/deads2k/kubernetes/commit/ea12fa4aa17c7326d5a7a9455210fd51ee78f886#commitcomment-16212221>
> .
>


Needs a Haddock comment.


Ditto.


Fixed. It's an error – merci!
Ich würde gerne Cap auf C und Cost auf \mathcal{C} umstellen. Einsprüche?


@wangkx Looks reasonable


ta verzija ne dela


I believe I can include documentation for this on the run_jobs task under oozie.thor.


moram updatat.


mam na usb


@tochman then it will look like a freaking javascript :)
Ok I will fix this!
It will be nice to write some code style standards for contributing to the project.


a bos ftb igral?


Yes, the fix actually comes from the parent, but this one is necessary to make the whole thing work. Without it, you can't break the mw chain after the render, because you still need the page handling to trickle down to the `navigation` and `siteSelection` middleware.

So the more correct labelling for this commit would've been _Fix sidebar no longer showing when logged in._ or something. ;)


Here retrieve our meta data as a new entity and update accordingly.


tl;dr: bad attempt at splitting commits for history


(facepalm)


Cheers,

is this to fix the issue that some custom visuals are currently not loading on app.powerbi.com ? This is being discussed on the forum (http://community.powerbi.com/t5/Service/Power-BI-custom-visuals-won-t-display-properly-in-Power-BI-Web/m-p/19765#M5727) but there's no mention of it in the PowerBI status page.


also a space is needed before the opening parenthesis here


Yes, this was useful during development, but we don't need to keep it around (specially if the final source of the config file is easily available). A good documentation explaining how both sides work will be much better.


-ggdb can be removed. Debug information is not needed in bin_descend.


Here you use {}.


Here you use dict().


x86_64 bit instruction translation is not confirmed. It does not work now


Uh, really? /me scratches head. It's just that it... seems to wfm without this commit :-)


'position' would fine by me. though 'to offset' translate to german meaning of 'move sth by' which seems pretty accurate


Feel like it would be better if I could just put these two together. Thought of doing something like:
```ruby
if FORWARD_MOVES.include?(move)
    return can_move?
elsif CAPTURE_MOVES.include?(move)
...
```
where FORWARD_MOVES = [[1, 0], [2, 0]] and can_move? checks if the space is occupied and if it is moving 2 also checks to make sure it is the pawn's first move. Anyway, it works the way it is but wasn't sure if this other way would be better.


:laughing: i hear you on looking sorta weird. I'm open to a prefix. I opened up https://github.com/CasperSleep/nightshade-core/issues/23 to us to discuss.


The inlininer has some pretty strong assumptions that anything inlineable is a GT_CALL, and that anything that's an inline candidate has been hoisted to a top-level expression. So it's not going to be as simple as just setting a flag. Basically at the point we fail to convert to an intrinsic we need to rewrite as a call and go through the normal call processing.


create a directory named images in your asstes folder outside application i.e booksnstud->assets->images


We can only have 1 collection in autoindex? Assumed this from the constructor.


Wait, I had a typo in my semi-manual revert...


What does git+https means exactly ?


Not sure I understand. Anyway, I was confused by "java compatible format" and continue to remain confused by it, but that's inconsequential.


autoindex can only have one collection?


Remove the ^ to avoid problems ?


So, document in a "desc" statement then, yes?


Not sure how to handle that. What would you suggest?


Yes, that might also just be my own understanding :)


LOL! That was some beautiful grammar right there...
Thanks for the spelling fixes.


Maybe a little xo would be nice :)


Do we really need to center by subtracting the mean? Theoretically these should have mean 0, since they are sums of normal random variables both with mean zero. Similarly, the variance should be two (since variance is additive), so I think we should be able to divide by the theoretical standard deviation (sqrt(2)) to get unit variance.


## This is a markdown file


Yes.


zakaj?


Why did you add this comment here?


Hmm, seems to wfm with a proper revert tho, too. (And the way I read mw invocation order, it shouldn't change with or without tihs commit.)


You're right, that's not what I intended. Since [Default] is a fallback, when there is a product entry, the product entry should take precedence for all of its keys. Although not likely, we could use the configuration you mentioned to put all of the products on hold except one.


wget doesn't live on windows?  Is it supplied by git perhaps?


Yeah, trying to get closer to the Android convention where 1 event = Listener + interface; multiple events = Callback + static abstract class.


This is why I don't use PHP.

...

Well, it's part of the reason.


#22 resolved


support txt file for lightning talk


Mas está em JS, e não é uma biblioteca... eu fiz na mão.
Só que para poder reutilizar o código eu coloquei em um fonte separado.


please add ?> after the last echo


thank you


Nitpick: maybe use single-word or camel-case parameter names, just for consistency with other `medic` subcommands.


Could the Jasmine code just `require` these?


You're welcome, and thank you for spotting the bug in my original patch. The new fix is: https://github.com/kplatfoot/android-play-billing/commit/d6e39b1ba339c5f0225adc0d301237066a38e2cd.

Besides the async race conditions, I also saw crashes from Google's IabHelper calling queryIntentServices and assuming the result is non-null (apparently not always true). There's a separate pull request open (https://github.com/googlesamples/android-play-billing/pull/4/commits) that fixes that one.


Are the ifdefs really needed at all now?  And I don't think Arduboy needs to include the .c file... isn't it going to get compiled by default?


@gforcada I've reverted all my changes, but it looks like there are still some test failures that crept in around when I pushed my reverts (http://jenkins.plone.org/job/plone-5.0-python-2.7-robot/4098/changes).  The reverts shouldn't have impacted the build, since none of the reverted packages were in checkouts, but I didn't want to prevent any work being merged into master those packages.  I've got branches for these fixes ready locally (including a fix for the multilingual issue), but I'm holding off until I understand the current set of test failures.


This code means that the following keys look the same
http:/hello , http#hello
which is a risk. Why not just Hexencode the keys? http://blogs.msdn.com/b/blambert/archive/2009/02/27/blambert-learnings-don-t-base64-encode-partitionkey-and-rowkey-values-in-azure-table-storage.aspx


Comment je fait please??

Le 19 février 2016 à 01:53, Christelle Scharff <notifications@github.com> a
écrit :

> @farhanat <https://github.com/farhanat>
>
> Bonjour il y a un probleme. Je ne peux ajouter votre nom. Essayer de voir
> comment faire pour que je puisse merger votre pull request. Bonne chance!
>
> Regardez ici:
>
> 3390901#commitcomment-16193262
> <https://github.com/paceuniversity/courses/commit/33909013b1563f965ee6ed0e351b7e77c803f11d#commitcomment-16193262>
>
> egalement
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paceuniversity/courses/commit/33909013b1563f965ee6ed0e351b7e77c803f11d#commitcomment-16193332>
> .
>


@mattwala


maybe just call this system.global-api-key


i think it will need to handle teams somehow, but im not 100% sure


++, this looks much better!


This is an extraneous `path.join`. Either use `path.normalize` or `path.join` individual components. Also, please make this a constant or compute it from passed arguments (e.g. what if we run this file from one directory below `cordova-medic`?),


Please separate starting iOS proxy and starting Appium into two separate functions.


I'll take a look when I get back from holiday


Adição do método para botão info dos annotations do mapa


@dvicklund I think this is what we want in order to start up the node server - yes? Defaults to port 3000. Change back if this is wrong, but just "node" doesn't work on my setup and wasn't having luck with nodemon yet either. Is this true or do I need to fix something in my setup?


"optionally one or more"?


Could you factor this into a helper method on resolver?


It will be fine if you restart nova services.


likewise, helpers for each object which can be allocated from an arena would be good.


It seems that your editor doesn't like you…


Ditto.


We typically have been using the coding style where the opening brace is on the same line after the method's parameters i.e.
blockNumbers(array $numbersToBlock) {
..
..
}


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


Would be nice to confirm this and the methods below to that


applying this change by copy/paste resulted in RSOD (red screen of death) at startup.
Error message = "Uncaught ReferenceError: e is not defined"
Fix = change semicolon at end of line 386 to a comma so 'e' is defined as a var.


My appologies.

Sent via the Samsung Galaxy S™ III, an AT&T 4G LTE smartphone

-------- Original message --------

From: Peter van Dijk <notifications@github.com>

Date:02/18/2016  11:38  (GMT-06:00)

To: PowerDNS/pdns <pdns@noreply.github.com>

Cc: irinatodeva <todeva@tocy-net.net>

Subject: Re: [pdns] schema change gmsql and gmysql backend (7166723)

Please see https://github.com/PowerDNS/pdns/blob/master/modules/gmysqlbackend/enable-foreign-keys.mysql.sql

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


@outofmap 커밋메시지에 " delete 'on update current_timestamp' " 는 코드에 드러난 건 아니고, 서버에 테이블 설정을 그렇게 해 줬다는 거죠??


I think this is not quite right. A gep instruction is `gep=addr+off`. If you have `sz=getObjectSize(addr)`, the assertion to insert is just `assert(off + used <= sz)`. No need to deal with `base` and `ptr`.

If size is unknown, then the assertion is something like:
```
  if (base == addr) assert (off + use_sz < tracked_size);
  else if (ptr == addr) assert (tracked_offset + off + use_sz < tracked_size)
  else if (ptr == addr + off) assert (tracked_offset + use_sz < tracked_size)
```
The last condition might be unnecessary. The cases can be unified into a single assertion by naming individual components of the offset and setting them to 0 when necessary. I don't think your asserts are correctly conditioned on `addr == base` and `addr == ptr`.


@farhanat

Regardez comment faire une pull request.

Ceci peut vous aider:
https://help.github.com/articles/checking-out-pull-requests-locally/


Ah, ok -- I thought a pure virtual class could be made that way, although the second one clearly isn't.


Sure you can.  For any code that the user enters, it's checked against the scratch codes (line 1760, check_scratch_codes(...)) but if that fails it's checked against the otp codes (check_counterbased_code(...) or check_timebased_code(...)).  It doesn't choose only one or the other based on the length of the value the user entered.
This file is talking about the format of the per-user ~/.google_authenticator file, so the phrase "Any all-numeric sequence of eight-digit numbers" refers to a line in that file, not a code that the user enters.


add chaungcey's <p class=" impression ">


The 2 new attibute would be shared by all instance of this class if you set them there. I don't think this is good. Define them in the method on_attach. This way, each instance will have a different value.


Click "View", then "View Raw" to download


 je pense que maintenent c'est bon, merci

2016-02-19 2:21 GMT+00:00 Christelle Scharff <notifications@github.com>:

> @farhanat <https://github.com/farhanat>
>
> Regardez comment faire une pull request.
>
> Ceci peut vous aider:
> https://help.github.com/articles/checking-out-pull-requests-locally/
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paceuniversity/courses/commit/33909013b1563f965ee6ed0e351b7e77c803f11d#commitcomment-16193641>
> .
>


nice!


This is not really a problem...


@farhanat Pouvez vous le refaire?


Why this if? I don't see a need for it. It probably come from copy/paste, but it isn't useful here.


Something to keep an eye on. This patch was suggested by someone who understands these timers better than I.


can you add tags as well? :)


Is it possible for the user to select a drive that doesn't have any associated drive letters? Does this handle that case?


Damn :( yeah I didn't really test this when I pushed....


good catch on this!


Please describe clearly


is this PR-able?


2016-02-19 2:25 GMT+00:00 Christelle Scharff <notifications@github.com>:

> @farhanat <https://github.com/farhanat> Pouvez vous le refaire?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paceuniversity/courses/commit/33909013b1563f965ee6ed0e351b7e77c803f11d#commitcomment-16193685>
> .
>


I find this name a bit confusing, although I 'm not sure of a better one.


Hola


isn't sqlite3 part of the default python library?


nit: prefer 3 lines for this kind of `if` expression (I realise name resolution has a lot of one-liners, but it would be good to move away from that in new code).


Or just...
default:


I think we should show this in the sysinfo summary view as well (not just in the section dialog).


@jazzsaxmafia What is the purpose of this line?


Task Manager uses a space (rather than ',') as a separator. I think that's a bit cleaner and easier to read.


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


Not terribly important, but why 1023 and not 1024? At least the docs state "in units of 1/1024 V".


I should have tried swapping xml with json in the url -___- Did not know there was a json variant of the jenkins api


You could use new pagination functions that have rel attributes out of the box. I use for single.php:

```
				<?php
				// Previous/next post navigation.
				the_post_navigation( array(
					'next_text' => '<span class="screen-reader-text">' . esc_html__( 'Next article:', 'tinyframework' ) . '</span>' .
						'<span class="post-title">%title</span>',
					'prev_text' => '<span class="screen-reader-text">' . esc_html__( 'Previous article:', 'tinyframework' ) . '</span>' .
						'<span class="post-title">%title</span>',
				) );
				?>
```

and for archives:

```
if ( ! function_exists( 'tinyframework_archive_page_nav' ) ) :
/**
 * Display navigation to next/previous archive pages when applicable
 *
 * @since Tiny Framework 1.0
 */
function tinyframework_archive_page_nav() {
	the_posts_pagination( array(
		'prev_text'          => esc_html__( 'Previous page', 'tinyframework' ),
		'next_text'          => esc_html__( 'Next page', 'tinyframework' ),
		'before_page_number' => '<span class="meta-nav screen-reader-text">' . esc_html__( 'Page', 'tinyframework' ) . ' </span>',
	) );
}
endif;
```

I still use your code for image navigation rel attr. If I give too much feedback, please let me know, don't want to bother you too much.


Your name is still not here:
https://github.com/paceuniversity/courses
@farhanat


Can you explain the purpose of `mask`?


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


Nice! 2~3s is a really awesome duration in comparison to 30 seconds.


Ok


This is @tastythredz ' commit.  Server sent incorrect user


yes


> do we still need bower.json?

We need it in case someone decides to do `bower install` due of force of habit or automation.

> do you need to update README.md to not ask to isntall bower?

Absolutely.


I think it would be good to update the comments in jshardware.h https://github.com/espruino/Espruino/blob/master/src/jshardware.h#L166-L171 to explain what really gets returned. For example, it says that jsPinAnalog returns a value between 0 and 1 but not what range that's supposed to represent. Or should it return the voltage? On the esp it would be the same, but on a system whose ADC can measure 0-3.3v or 0-5v that would be different. Similarly, it doesn't explain what the 0..65535 returned by jshPinAnalogFast are supposed to represent.


Yep, looks like you're right.  Nodemon definitely figures it out by itself, but I guess vanilla node is just not quite smart enough to parse the package.json.


Can you create an "issue" with your suggestions?


Is this what you want, it prevents generating pulses on open-collector pins, doesn't it?


I tested honami device using l-mr1 branch for device-sony-common and vendor-sony-kernel repos. I suppose that change was not applied to kernel's l-mr1 branch.


everything in between needed deleting, its already fixed


Seems like a perfect use-case for separating value and display.  I don't know why the other filters don't use it, but worth looking into.


Isn't this the default behavior?


This was actually me webteckie  doing this commit (per Jed's permission) :-) Didn't realize I was logged in with this 'epihacker' account I use for work :-)  Sorry about that!


The method below is called `iter_ancestor_location_types`, is that what this is supposed to be?


@DavidGoldwasser I believe with this commit you just reintroduced the build failure.


This is going to be a small lecture, so bear with me.

One of the most important rules of design (applicable to most design-able things, not just software):
Keep things as simple as possible -- and then try to make it simpler. If you're making something more complicated or expensive (in any way), then you better have a very solid reason for doing so.

Now lets talk about exceptions.

**Pros:**
Exception gives you a nice way of handling errors by encapsulating a lot of information in one object:

1. Error type. You can create your own exceptions by [inheriting from the exception class](http://php.net/manual/en/language.exceptions.extending.php) and throw a different exception object type to identify the error. Then you can use [multiple catch blocks](http://stackoverflow.com/questions/8439581/catching-multiple-exception-types-in-one-catch-block) to handle different error types in various ways. The difficult and less readable way of handling this is by using different return code values and associating meaning to each of them as [often done in C](http://www.virtsync.com/c-error-codes-include-errno) since it doesn't support exceptions.

2. Exceptions usually have an error message which you can use to obtain more information. You can even tag on other information if you create your own exception class.

3. Another very useful feature is that exception objects usually include the [stack trace](https://en.wikipedia.org/wiki/Stack_trace). You can [get the stack trace from the exception object](http://php.net/manual/en/exception.gettraceasstring.php) and that will tell you which functions were called, in what order and where the error occured along with relevant line numbers

**Cons:**

1. The process of creating an exception object and keeping track of it using try/catch blocks can be [expensive from a performance perspective](http://stackoverflow.com/a/17684984).

2. For cases where you just want to verify success/failure and don't have any more details to provide, exceptions and try/catch blocks can seem like "over-engineering" and excessive.

Now how do you approach your query() issue given the above:

1. You just want to say if the query execution was a success or failure. There is no more information to add and an error message as in your example does not provide any usefulness in debugging.

2. If you go the exception route, you will have to use a try/catch block *everywhere* you call the query() to make sure no exception escapes. Now you have a lot of usecases (like INSERT, DELETE, UPDATE queries) which don't have any result except true/false. For those cases you can collapse the function call and return value to one line:
        `return parent::query($sql);`
Compare that one line to the five line try/catch block implementation you suggested. Which is simpler? Does the exception solution provide any more benefit than the one line? In both cases you just return true/false in the end.

3. You seem to have a difficult time identifying areas where you are increasing coupling. Your PHP database library should not be integrated into your website output and should be as stand-alone as possible. Think of the database folder as a black box, all you care about is the function, the parameters and the return values. You should not be dependent on the get_items() function to echo an error message onto your website.

  To give you a practical example:   What if I want to write a separate tool which used the PHP library to query the database (insert, update, delete, retrieve data)? You want your database library functions to properly inform the caller if there is an error instead of just doing an echo of an error message. Your website might display the error to the user, but if I write an automated command-line tool which updates the inventory every time a customer buys food, then the tool will have no way of knowing when an error occurred because you were just joyfully echoing messages instead of establishing a solid error reporting/handling system.

**Conclusion:** Throwing an exception in the base class query() method serves no purpose since at that level you have no extra debugging information to provide and all you want to do is return the result of $conn->query($sql).


uart_recvTask is for RX only and uart1 doesn't have RX (pins used to access the flash chip)


74715752-Constant-Flow-Radiant-Heating-Not-Turning-On-Bug (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2131 of 2131 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)


I'm not sure how much this function would be used, it might make more sense to keep it with it's usage (maybe as a private utility function, for example) unless you know you'll need it again.


Nice catch on the Runic properties.  You could also just remove the object from the list as it applies.  I think that's the original idea.


thanks for taking a crack at the esp serial!


put the comment above the line; you shouldn't break up a line of code to put a comment in there


I'm not a fan of punning on `Result` like this. I think a new enum would be clearer. However, the setup with glob and named bindings seems a bit odd, I feel like there ought to be a cleaner design - maybe just one binding + a 'binding kind' enum?


and if it's the `dist/12.0` part, just use a separate variable:

```
public double getAngle(double distanceInInches) {
    double feet = distanceInInches / 12.0;
    return 1.862 * feet + 37.49;
}
```


74715752-Constant-Flow-Radiant-Heating-Not-Turning-On-Bug (Myoldmopar) - 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)


-mfpmath=sse combines with -march=silvermont (or another SSE-capable processor) to use scalar FP instructions instead the old 387 fp. I didint found --fmath option, is it an specific Linaro option?


I suggest you read this wiki:  https://en.wikipedia.org/wiki/Software_design

The most important section is "Design Considerations". Read up on each of the considerations listed. When I talk about software coupling, I'm essentially talking about modularity. Reading that wiki will give you a good perspective of what factors go into making design decisions.


I tried using a single menu item but the code for that is a bit messier, so will stick to two menu items.


Hi, I've pushed a patch with the following code for _inverse_cdf:
```python
    # Sort pdf and bin centers - this should already be the case for
    # marginalized pdf, but not for raw sample data
    rec = np.rec.fromarrays([bin_centers, pdf])
    rec.sort()
    bin_centers = rec.f0
    pdf = rec.f1

    # Normalize pdf so that area is one
    pdf = pdf / sum(pdf)

    # Find the bin width.
    bin_width = bin_centers[1] - bin_centers[0]

    # Shift all bins forward by 1/2 bin width (i.e. bin edges)
    bin_edges = [bin + (0.5 * bin_width) for bin in bin_centers]

    # Insert first edge so we have n + 1 edges
    bin_edges.insert(0, bin_centers[0] - (0.5 * bin_width))

    # Build a list of (parameter index, cumulative posterior weight).
    # Note we insert an initial entry at index zero with cumulative weight
    # zero to match the first bin edge.
    cumulative = list(enumerate([0] + list(np.cumsum(pdf))))

    # Special case where prob > max(cumulative): return the rightmost bin edge.
    if prob > cumulative[-1][1]:
        return bin_edges[-1]
    else:
        # Find the index of the last param value having
        # cumulative posterior weight <= desired probability
        index_lower = filter(lambda x: x[1] <= prob, cumulative)[-1][0]

        # Find the index of the first param value having
        # cumulative posterior weight >= desired probability
        index_upper = filter(lambda x: x[1] >= prob, cumulative)[0][0]

        mean = 0.5 * (bin_edges[index_lower] + bin_edges[index_upper])
        return mean
```

This actually gets us (mostly) back to the old numbers for credible regions, because by taking the mean of two adjacent bin edges, we're back to reporting bin centers. The code this replaces did the opposite - by averaging bin centers, it was reporting edges.

Note the doctests for statslib.one_dim now fail - I'll hold off updating them until you've confirmed this code is doing what we want, or we come up with something better.

Also, slight caveat here - if the requested probability is exactly equal to the cumulative probability at one of the edges, that will be returned as it is both <= and >=. *All* other probabilities get an average (i.e. a bin center). Which seems a bit iffy. I don't know if it's worth doing something about this, but if so we could either take the next / previous bin center.

Or if we wanted to go all in we could do a linear interpolation between the bin edges (i.e. a weighted averaged based on how close the point is to each edge). That seems nicer to me but I don't know if it's appropriate here. Not hard to do though.


Regarding `JavaScript.sublime-syntax`, it looks like you may have a typo in several places: `puntuation.accessor.js`?

Also, if you wouldn't mind explaining, it looks like the way lines 709-715 are structured, that in that situation the `.` will not get a scope? Am I wrong about that, or is that the desired behavior?


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 437](http://ci.infinispan.org/viewLog.html?buildId=36079&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for adebdedf43f0d1fb0038dc1e8f03bcc9e237fa1f
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:06:49

Process exited with code 8


j'ai repris

2016-02-19 2:34 GMT+00:00 Christelle Scharff <notifications@github.com>:

> Your name is still not here:
> https://github.com/paceuniversity/courses
> @farhanat <https://github.com/farhanat>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/paceuniversity/courses/commit/33909013b1563f965ee6ed0e351b7e77c803f11d#commitcomment-16193794>
> .
>


I recommend you use git pull --rebase instead of force pushing, and getting commits like this


Alternatively, this is pretty similar to the `SQLLocation.get_ancestors` method, so you could mimic that as a method on the `LocationType` class.


AddCoolGithubThings (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2099 of 2099 tests passed, 6 test warnings)

 * 6 tests had: AUD diffs.

![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)


It was an unexpected error - I forgot I had merged a pull request through the website when I tried to push some local changes.


**FAILURE** - TeamCity HotRod C++ Client :: Pull requests monitor - RHEL [Build 438](http://ci.infinispan.org/viewLog.html?buildId=36080&buildTypeId=HotRodCpp_PullRequestsMonitorRhel6&guest=1) for dadc79d2b3440345930fe4a117a80911aea08c3e
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:06:36

Process exited with code 8


@kbenne sorry, All I intended to do was sync my local git with the web one. I had not made any commits locally. All I had done was made new temp branch.

Should I have thrown away my local repo first and re-checked it out? Can you re-kill this commit. I'm still a little confused. You have two commits from Dan that are also on develop. That is all that was in Dan's OrphanLifeCycleCost branch. That branch builds for me on mac.

When I accepted his pull request initially I did it from the web interface.


It's true that the line you change is doubly wrong. (I wonder what I had been drinking when I wrote that.)
But you can fix it like this:

```
expr "$TEST_OS" : "CYGWIN_NT" >/dev/null || test_set_prereq POSIX
```


It happens :)


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


Thank you @felipefrizzo


What commands are you using to get the error? Note, I can at minimum get b2 to attempt building with the default invocation. Although the compile fails. Probably because I'm using the rather old msvc-12.


It isn't your fault, although it is true the remerge did re-break the build. It is mostly my fault. The root of the problem was this bb3c8f292cd3072063b5dee04c3f2f7601eddf13 idd change. not actually sure why, but I reverted it to free up the build. That Idd change was an addition of IP units which I did the merge on. I think that gal/min is an invalid units string. @evanweaver you should have a working develop again.


+1,  please wrap the check action into one function. In fact, there are many other cases(change memory/cpu weight) to trigger restart worker.


I think it'd be good if these tests were added in the same commit that change the behaviour of the flow.


ping @ingvagabund guess the license part of the review is handled. What else do we need?


Aside from some consistency nitpicks, this LGTM. However, the commit comment no longer matches the behavior of the code.


We have a python script internally bound to cmake to do this if it helps.


We don't want to do that. This is costly for each request.
See: http://issues.hibernatingrhinos.com/issue/RavenDB-4371

Let us think if we can do better.


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.34.0](http://build01:8888/viewLog.html?buildId=14313&buildTypeId=OpenSourceLibraries_Sourcerer_CI) is now running


Both of these CSS files are not loading.  In the case of the Reset file, the reason is because it doesn't exist in your project folder.  You need to have a folder named "css" within your main project, and within the "css" folder, put the `reset.css` file from the starter code in it to be able to load the Reset files.

You also will need to put your CSS file into this folder as well and rename it to `style.css` (currently your file is named `Relaxr Css.css`).


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.34.0](http://build01:8888/viewLog.html?buildId=14314&buildTypeId=OpenSourceLibraries_Yaclp_CI) is now running


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.41.0](http://build01:8888/viewLog.html?buildId=14315&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) is now running


Тут ещё и комментировать можно.


TeamCity Open-source Libraries :: YACLP :: CI [Build 1.0.34.0](http://build01:8888/viewLog.html?buildId=14314&buildTypeId=OpenSourceLibraries_Yaclp_CI) outcome was **SUCCESS**
Summary: Tests passed: 20 Build time: 00:00:29


TeamCity Open-source Libraries :: ThirdDrawer :: CI [Build 1.1.41.0](http://build01:8888/viewLog.html?buildId=14315&buildTypeId=OpenSourceLibraries_ThirdDrawer_CI) outcome was **SUCCESS**
Summary: Tests passed: 8 Build time: 00:00:29


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.56.0](http://build01:8888/viewLog.html?buildId=14316&buildTypeId=BlogMonster_CI) is now running


Seems nice, thanks ! :)


Sadly, a switch case is probably going to be twice as fast.
See: https://ayende.com/blog/173121/good-fast-pretty-code-how-to-choose


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.39.0](http://build01:8888/viewLog.html?buildId=14317&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) is now running


Just a comment on the name of your HTML file -- typically, you want to name the file for the main page of your website `index.html`.


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.56.0](http://build01:8888/viewLog.html?buildId=14316&buildTypeId=BlogMonster_CI) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 12 Build time: 00:00:19
### Failed tests
```
BlogMonster.Tests.dll: BlogMonster.Tests.WhenImportingPostsFromAnInstagramFeed.ThereShouldBeAtLeastOnePost: <no details avaliable>

```


Why do we need this, instead of just using default(T) where needed?


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.521.0](http://build01:8888/viewLog.html?buildId=14318&buildTypeId=ConfigInjector_CI) is now running


TeamCity Open-source Libraries :: MvcNavigationHelpers :: CI [Build 0.0.39.0](http://build01:8888/viewLog.html?buildId=14317&buildTypeId=OpenSourceLibraries_MvcNavigationHelpers_CI) outcome was **SUCCESS**
Summary: Tests passed: 21 Build time: 00:00:25


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.60.0](http://build01:8888/viewLog.html?buildId=14319&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) is now running


TeamCity Open-source Libraries :: Sourcerer :: CI [Build 0.0.34.0](http://build01:8888/viewLog.html?buildId=14313&buildTypeId=OpenSourceLibraries_Sourcerer_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:01:17


How about `zfs_raidz_math_enable`?  It's consistent with `zfs_raid_math_impl`.  Or perhaps some better name.  Regardless, you'll need to add both these new module option to `man/man5/zfs-module-parameters.5`.


Unnecessary else case?


Looks like @mscoutermarsh  had better solution for this one using created_at and updated_at, so probably will end up changing this to use that instead. @mscoutermarsh I was wondering though, how would you write a spec for this if we used the created_at and updated_at method?


Please don't change the whitespace policy of the file.
I can not review the changes anymore, since github will just state
`1,337 additions, 1,338 deletions not shown because the diff is too large. Please use a local Git client to view these changes. `

I hope this commit does indeed only change whitespaces.


TeamCity Open-source Libraries :: ConfigInjector :: CI [Build 2.0.521.0](http://build01:8888/viewLog.html?buildId=14318&buildTypeId=ConfigInjector_CI) outcome was **SUCCESS**
Summary: Tests passed: 38 Build time: 00:00:34


TeamCity Open-source Libraries :: NotDeadYet :: CI [Build 0.0.60.0](http://build01:8888/viewLog.html?buildId=14319&buildTypeId=OpenSourceLibraries_NotDeadYet_CI) outcome was **SUCCESS**
Summary: Tests passed: 3 Build time: 00:00:58


As it was explained in the article I linked there is a problem with overload operator. This code compiles fine:
```
uint32_t x = 3453535;
bitox::OutputBuffer output_buffer;
output_buffer << x;
```

But this code fails to compile:
```
bitox::InputBuffer input_buffer (output_buffer.begin(), output_buffer.size());
uint32_t y = 0;
input_buffer >> y;
```

You will have to overload each std type but even that would not be a trusted implementation.


We certainly can, are you trying to avoid complete extremely long garbage in the DMI data? I expect this would already be taken care by the kernel, when it reads the data from the firmware.


Nice!  BTW, I already merged the babel branch into master.


Can you be more specific when this is needed? Which OS, which python, ...? Why -lm is needed anyway? This could depend of the version of mkl.

I don't see big problem to always add it, but it wasn't needed in all cases for mkl. Why not try the flags without it and try with it only if it fail without?


This  is a lot of delegate invocations to pay for, see also the comment about the switch from earilier


Ok, very cool!


@k are you demoing this somewhere?


This will be augmented once we have libear (or similar) support in place.


It's already cached by AppFavorites; the complexity is an array lookup linear in the number of pinned apps. Not worth caching here IMO. :)


:dancer:


@garbear I'm happy to help. I also wanted to circle back to Moonlight for Kodi but hadn't yet found the time. Feel free to email me if you have thoughts or questions.


Why? Because `scr` might not always point to a proper location? Anyway, the solution seems to just have a simple variable for this and not in `scr`.


74715752-Constant-Flow-Radiant-Heating-Not-Turning-On-Bug (RKStrand) - x86_64-MacOS-10.9-clang: OK (2093 of 2093 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.

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


I also took the opportunity to simply the backward code, taking advantage of the sideOpposed bitmap, but
this is exactly equivalent to master code.


Bwahaha, dili diay ka mag laravel? or PHP?


I've been trying to use Beautify but it doesn't work as well as Tidy. Also haven't been as diligent on macros that are going to get refactored. I'll see if I can find a better plugin though. Having well formatted html helps refactoring.

:+1: on converting to jade. That would be great! Jade was the other templating language I was considering as an option. It would be great to have some templates to test performance with. Both are great templating options


notes of the first chapter


DMI itself has a limit of 64kb. I think we should have the cap a bit lower.


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


The header pushed to _response_headers_ is the same as before, with space after the colon.


wat?


po co?


kubelet first (so it self-registers with the API)
then plugin (so it inits with the Node record from the API and silently unblocks the kubelet pods via a channel)
then proxy (so it has the plugin filters for multi-tenant)


indent


ok


I thought i made a pull request template -_-


Good catch! Definitely should reuse open-state-changed for this.


972154-NewSimpleChilledCeiling (RKStrand) - Win64-Windows-7-VisualStudio-12: OK (2117 of 2117 tests passed, 550 test warnings)

 * 550 tests had: AUD diffs.

![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)


useless val


This is wrong.
What we should do is compare last mapped document & last tombstone document to the last document, then return false if they don't match for each collection.
No need to do any allocations of the etags array


May you can try the Pollard' rho algorithm for fast integer factorization: https://en.wikipedia.org/wiki/Pollard%27s_rho_algorithm

Described by:
```C
int gcd( int a, int b) {
	int remainder;
	while (b != 0) {
		remainder = a % b;
		a = b;
		b = remainder;
	}
	return a;
}

int main () {

	int number = 10403,x_fixed = 2,cycle_size = 2,x = 2,factor = 1;

	while (factor == 1) {

		for (int count=1;count <= cycle_size && factor <= 1;count++) {
			x = (x*x+1)%number;
			factor = gcd(x - x_fixed, number);
		}

		cycle_size *= 2;
		x_fixed = x;
	}
	printf("The factor is %d ", factor);
}

```


comments


@linhphi9x94 welcome aboard!
Lol, do you have a OnePlus user name?


Default mapping mana ry and F5 :+1:


This would make it inconsistent with current `select` in that it would use `FdSet` from `libc`. How should I proceed?


We need to see how we can avoid allocating a slice here for the collection inside the `GetLastDocumentEtag`, since this is going to be called a lot.


Auto open ni sa nerdtree? :+1:


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


worth adding that as a comment, actually, and in the issue I assume you'll open to make someone fix all those dependencies


I understand that you can still modify the `__connection_params` later on if you really want to but I think it would be a waste of resources to guard against that (checking before every connect).


972154-NewSimpleChilledCeiling (RKStrand) - i386-Windows-7-VisualStudio-12: OK (2117 of 2117 tests passed, 550 test warnings)

 * 550 tests had: AUD diffs.

![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)


Seguinte: no seu settings, vc tem essa configuração abaixo:
````python
STATIC_URL = '/static/'
STATICFILES_DIRS = (
    os.path.join(BASE_DIR, "static"),
)
````
Só que não existe essa pasta `static` não existe na raiz do projeto, é que pra onde o `os.path.join(BASE_DIR, "static")` vai apontar. Essa pasta na realidade está dentro do diretório da sua app `connectedin`. Então a sugestão é: retire essa linha, porque assim o Django vai fazer a busca automática dentro da sua app.

E a sintaxe para fazer isso seria:
````html
<img src="{% static 'img/image.jpg' %}" alt="Image">
````


You don't need this at all. The header.jsp is never directly called by a user.


du coup on pourrait utiliser la convention utilisée par les développerus de Scala js dont je te parlais la denière fois qui est, dès qu'un truc après le case fait plus d'une ligne on saute une ligne avt les autres cases (c'est quand même plus clair à mon avis)


Since you know that email is guaranteed to be unique in your system thanks to validations, you could use `User.find_by(email: params[:sender])` instead, which would be more "Rails Way". `find_by` returns the first matching result from the database, `where` returns all matching results.

Behind the scenes, these two lines of code run the same SQL though, so it's not a big deal. Just a PROTIP.


That is correct at the moment, but why limit ourselves?
If I want to search by name across users, employees & cars...

Note that I'm 99.99% convinced that auto indexes will stay with a single collection. But we'll need to use this code for standard indexes as well, and there you have multi map.


I'll queue a patch for that.


Might as well move this to one line.


would we want to prevent them from running the multi-tenant plugin separately from the kubelet? if so, `actual.Has(ComponentKubelet) != actual.Has(ComponentPlugins)`?


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

```


same here


Move some stuff to the context.Transaction ?
Such as Write("tree", "key", stream) ?

Maybe just put a TODO: ugly law of demeter violation ?


@mscoutermarsh this is the line that @mmeitl and I are having problem with... it always fail the test


add a logger


Thanks, going to make the code consistent and fix the comment


I'm not sure if these are still considered _dependencies_ instead of `devDependencies` since are used for development and we don't have the standards working without a build step. Since the build step is happening before we distribute everything, then I think we can leave most things in `devDependencies`.

http://stackoverflow.com/a/22004559


Hey r u on


[The data type "sig_atomic_t"](http://en.cppreference.com/w/cpp/utility/program/sig_atomic_t "Description for sig_atomic_t") is the portable interface for the software design that you seem to try out at the moment.


Thanks for the pull request. You seem to have missed a few strings, https://github.com/garwedgess/PassKeep_translations/commit/bdea5fff2c62934c93ce56ae83d2ba723272dfae

And then this one line:
https://github.com/garwedgess/PassKeep_translations/commit/ad0cd32dde531162ef56b4e2eb8412bb777076bf


I would approve this PR if we change "Twitter Bootstrap" to "Bootstrap". I feel that is an outdated relationship and for as long as I have know about Bootstrap it has been a freestanding project unrelated to Twitter.

http://getbootstrap.com/about/#brand


True


@szepeviktor - apologies, I can't tell what line you are referring to - the line comment seems to be talking to the entire code block.  Can you update with the actual line number?  Thanks.


but please remove that part - it shouldn't be needed any more


Cool. Looks good to me.


Ха, отсутствие перевода строки в конце файла, GitHub считает залётецом.
Будем знать.


when merging branches, they got removed from master


Please extract a helper method here; anytime I see a `break;` I think it would be cleaner as an early return.  This looks like it would extract very much like what you did with `shouldCustomMarshalObject` above.


Cool! Is this a built-in way to order query results (the +/- thing) that mongo provides?


@szepeviktor - now I spotted it - the last line.  Fixed - many thanks for reporting!!


Does timing always come with navigationStart?


The username should come from the form, not always be "post.username"


Yes, I even marked this array with TODO couple of lines below. Need to figure out how to bind this with QueryResult.LastIndexedEtag


@richardkchapman this PR is ready to be merged.


Yeah! I had to dive deep in the docs to find it. What you were doing getting all posts and the using a for loop worked great, but this one liner might be faster since the DB is doing all the processing. We definitely don't care about performance, but it looks pretty!


@rzarzynski what about other error codes?


Azoknak, akik a kétméteres lombokat nem érik fel. :)


It certainly does! Nice find, Dan


Misnamed parameter documentation.  Should be `intFunc`?


OK, I'll change to setBackgroundDrawable and let super handle it. Yes, I'm not calling "this" to avoid a second call to updateBackgroundDrawableAttrs, but I can also remove that call above and let de setBackgroundDrawable handle it


D'oh -- apparently bugs related to missing `navigation`/`siteSelection` middleware are subtle ('Themes' not being highlighted in the sidebar is one indication). They _are_ fixed by this commit.


@nbansalvox please uncomment this line when you pushed your part. Thanks!


Hrm. I don't know right now! If you have feelings about API design and project directions, you could comment on #190... ;-)


Nice!


This style will not actually inline the code inside gnix_ep_atomic_write into the application. It will just get rid of the double deref.

I think FABRIC_DIRECT can be better than this, and directly include the code for the underlying functionality in the app. This will have multiple benefits at compile time. e.g. if some arguments to fi_xx call are constants, then the compiler can fold all the constants through the stack and eliminate a bunch of branches. Especially important for SHMEM, since it has single element put/get calls.


Nice!! :-)


наведение на чекбокс и радио, задисабленые состояния из стайлгайда


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


please for to be using the `post_float` (and related) functions for accessing input parameters :-)


LGTM


all tests pass:
success: http://buildbot.bos1/builders/python-client-test/builds/476


`performance.timing` can be `null`, but if it's not `null` then yes, there should always be a `navigationStart` https://www.w3.org/TR/navigation-timing/#processing-model


Cleaned it up in 8636a540344891970db48111f058cce850449b17


I removed `(quiet)`, `(no-boot)` and `(no-connman)` because they don't mean much. i.e., they aren't options but listing them kinda suggests they are.


Yeah, looks good.


Thanks for the tips. Will have them in mind next time :))


papas


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-MacOS-10.9-clang: OK (2093 of 2093 tests passed, 3 test warnings)

 * 3 tests had: AUD diffs.

![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)


@hppritcha


Argh, sorry, sloppy of me.


I need Push Access apparently.


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 148](https://teamcity.labkey.org/viewLog.html?buildId=304425&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 10 (4 new), passed: 22 Build time: 01:15:12
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

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

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

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

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

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

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

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

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

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

```


Hooray for dependency injection, but doing this on a namespace (rather than an instance) is one more dependency on only having one global `jsInterpreter`.  Fine for now, but we may want to refactor this into a helper class that must be constructed with a `jsInterpreter` instance at some point.


убери обводку outline по фокусу


Bahahahahaha!


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


Opened https://github.com/openshift/origin/pull/7477, see if that works for you.  OS_OUTPUT_GOPATH=1 is legacy behavior, and you can still set OS_EXTRA_GOPATH if you want.


When I saw you introduced a `GameLabSprite` file I thought "Great, we'll have a place to put any of our own extensions to the Sprite class!"  Do you think this would be an appropriate place to do that?  I'm not so sure after seeing its current application.

Maybe I'm wishing more of the sprite-wrapping code was in here as well, not just the custom implementations.


since we need to have memory fence also for object reading, we need instead a full memory fence or a store_acquire


LGTM


After reading some more - I agree. It seems only volatile sig_atomic_t can be modified inside a signal handler portably: "This compliant solution sets a flag of type volatile sig_atomic_t and returns;" (from your link); also from a C99 draft: "<...> or refers to an object with static storage duration other than by assigning a value to an object declared as volatile sig_atomic_t (7.14.1.1). ". Thanks for your patience.


Note: the `public/eloquent.js` file added in this commit was automatically generated by running `php artisan eloquentjs:generate` command.


sdsd


Missing headers


Totally! Thanks for the reminder.


I'm on ubuntu 15 and this path doesn't exist for me.
On my computer the path to the icon is: **~/.local/share/icons/code.png**


Hmm, that was not my intention.  My first change used spaces instead of tabs which seemed to not follow the existing whitespace pattern in the file so this last change was meant to convert those spaces back to tabs on just the lines I added/modified.  I will look into it.


Also, this should've been two separate commits.


Working perfectly now with https://github.com/EpochModTeam/DayZ-Epoch/commit/da59f314a0e6afe96b40fa83afe69f802a2b4658


Yup - also for build URLs with query params take a look at `http_build_query`


Is edge like latest? and can we just use a version number if it is? makes debugging a remote setup easier


fast-forwarding master to borshop-integration-436-features/lrb/gh-434-riak-client-tools = 23f167b7


<نسخة أكس = "1.0" ترميز = "UTF-8"؟> <؟ XML-أنماط أ href = "http://www.blogger.com/styles/atom.css" نوع = "نص / المغلق"؟> <XMLNS تغذية = "HTTP: //www.w3.org/2005/Atom" XMLNS: أوبن سيرش = 'HTTP: //a9.com/-/spec/opensearchrss/1.0/ "XMLNS: مدون =' HTTP: // schemas.google.com/blogger/2008 "XMLNS: = georss العنوان http: XMLNS //www.georss.org/georss ': GD =" http://schemas.google.com/g/2005 " اكتب 'نص' => pfghrd43 </ عنوان> <نوع العنوان الفرعي = 'أتش تي أم أل "> </ العنوان الفرعي> <رابط يختلط =" HTTP: //schemas.google.com/g/2005#feed "نوع =" التطبيق / ذرة + أكس "أ href =" HTTP: //pfghrd43.blogspot.com/feeds/posts/default'/> <رابط يختلط = "الذات" نوع = "التطبيق / ذرة + أكس" أ href = "HTTP: //www.blogger كوم / إس / 2658780847637174665 / وظائف / الافتراضي "/> <رابط يختلط = 'بديل' نوع =" نص / أتش تي أم أل "أ href =" HTTP: //pfghrd43.blogspot.com/'/> < 'المحور' صلة يختلط = = أ href العنوان http: //pubsubhubbub.appspot.com/'/> <المؤلف> <اسم> alkas googly </ اسم> <البريد الإلكتروني> noreply@blogger.com </ البريد الإلكتروني> <كلمة المدير العام: صورة يختلط = "HTTP: / /schemas.google.com/g/2005#thumbnail "العرض = '32 'ارتفاع = '32' النسخة = "7.00"


ah yes ... I didn't see that. you are right of course.


I'd suggest removing the RAIDZ_USERSPACE define and just using !_KERNEL.   I know the intention here was to make this clearer, but for me at least it ended up as a point of confusion as to what exactly RAIDZ_USERSPACE means.


@seasick No, it wasn’t! You can can use the [releases](https://github.com/wooorm/alex/releases) from now on!


Dużo ważnych zmian. Bardzo mnie trapi, że pętla, którą napisałam w testach nie działa. Zakomentowałam ją.


113360729_HumidifierOASys (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2102 of 2102 tests passed, 7 test warnings)

 * 6 tests had: AUD diffs.
 * 1 test had: RDD diffs.

![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)


eu deixaria isto a cargo dos clientes


I filed https://bugs.swift.org/browse/SR-775 regarding this.
On Darwin, `pthread_t` is a typealias to `UnsafeMutablePointer`, while on Linux it is a typealias to `UInt64`.
The questionable removal of the default initializer on `UnsafeMutablePointer` comes home to roost.


PrepFor85Release (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2099 of 2099 tests passed, 550 test warnings)

 * 534 tests had: EIO diffs.
 * 6 tests had: AUD diffs.
 * 16 tests had: ERR diffs.

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


Well, you guys do what you wish, I just created the quest out of curiosity because I've never done anything with any MMORPG server/engine. It's working nearly perfectly now though if you look at Killerwife's commit which is the final version.


I'm not sure updating `schema.rb` is enough to remove these table.  This file is just a record of the current DB structure.  I think you'll need to write a migration.


should have prefixed with `deps: `, but already signed release so not gonna rebase


Clean lines up with whitespace


Thought I did...


Is this and the previous hunk related to this patch?


Yeah, it's an annoying bug. I think for the mean time it's going to have to be wrapped in

```swift
#if os(Linux)
self.thread = pthread()
#else
self.thread = pthread(nil)
#endif
```

I'll try and make the changes tonight if I get a chance


dumps/* тогда уж


:) I using htc one m8 . I can dev for m8 with marshrom . Love marshrom
Vào 20-02-2016 3:09, "Aaahh Ahh" <notifications@github.com> đã viết:

> @linhphi9x94 <https://github.com/linhphi9x94> welcome aboard!
> Lol, do you have a OnePlus user name?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MarshRom/android_frameworks_base/commit/f7f43121a8b3937a588caf1baba96b8fff40b1bf#commitcomment-16212694>
> .
>


bir sorum var, bunu loop icerisinde yapmak zorunda miyiz?


can we move it to the top of module?


also removed comments from main.c


Would it be possible to do something more elegant like have a way for a compiler with good IPA to deal with this if we installed provider src code somewhere accessible to the compiler?