AddNetAreaColumnToEnvelopeSummary (mbadams5) - i386-Windows-7-VisualStudio-12: OK (2052 of 2052 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 know and that works for link tags, the trouble is that the meta tags look like this:

```HTML
<meta property="og:image" content="someimage.jpg" />
```

So we need html-loader to check the `property` attribute of the `meta` tag, and if that is `og:image` it should `require` the `content` attribute.

As far as I can see that's impossible with just the `attrs` query, but I'd love to be proven wrong!

<sub>*(Also see https://github.com/webpack/html-loader/issues/17#issuecomment-170833361 )*</sub>


This does not seem to be used anywhere, so it should be safe to remove.


I thought about it, but they are not exactly the same for 2 reasons:

1. we don't have `--log` available during `early_start`, so when you use `--log early`, the messages are printed when the `reconfigure` happens. (whilest DEBUG_EARLY logs immediately)
2. the user might not want to have `DEBUG_EARLY` env set when running avocado (eg. if he executes avocado inside avocado)


couple things here:
1. name your file with a .js extension, file names should not have spaces. randomNumber.js would be a good file name
2. your code is not properly indented. everything inside the for loop should have 2 spaces to the right so we can easily see that those code are part of the for loop.


@rom1504 aha, it's in node-minecraft-protocol index.js but not browser.js which currently only has Client and supportedVersions: https://github.com/PrismarineJS/node-minecraft-protocol/blob/master/src/browser.js

I guess browser.js should be updated to add states, but I'm not sure what else if anything should be added to browser.js from index.js


I thought I did that bit, sorry :P


(you know me, I'm paranoiac)


Ahaa ... this is now making shove not work at all :(.


You don't need to specify the `type` when an `enum` is used


Oh sorry I was just following one of the other examples I saw.

From:  Mads Kristensen <notifications@github.com>
Reply-To:  MCKRUZ/schemastore <reply@reply.github.com>
Date:  Tuesday, January 12, 2016 at 2:05 PM
To:  MCKRUZ/schemastore <schemastore@noreply.github.com>
Cc:  Matt Kruczek <kruz79@gmail.com>
Subject:  Re: [schemastore] Added HTMLHint schema (8e3b698)

You don't need to specify the type when an enum is used

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


This hasn't fixed the no-launch issue on Windows 10 Pro 64x.


I don't have a very clear suggestion here, but I feel the user may get confused with the order of precedence here. What I mean is:

```
$ avocado --log none,app run /bin/true
```

Gives completely silent output, while

```
$ avocado --log none,all run /bin/true
```

Gives complete verbose output. It may just be a matter of documenting that behavior though.


You're welcome!


Ej, ta metoda przecież nie ma sensu. Co ona ma robić ogólnie?


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


Конфиг YARD как раз должен быть в репе. Если бы это был реальный гем — а вдруг кто-то захочет помочь тебе с разработкой (и скажем пофиксить доки)? Он не знает, с какими настройками ты генерировал доки (и скажем в Gemfile зависимости от YARD тоже не указано, ето зря!)


Indrømmer du at du tager fejl, @svip?  I en diskussion på Internettet?


Looks really good. Thanks!!

There are a few issues with declaring the property `type` when using an `enum`. They conflict in this case where the Boolean value `true` is valid from looking at the `type` but invalid when looking at the `enum` array. Removing the `type` property fixes this.


Looks good to me :) Kako to razumeš v glavi?

Lp,Primož Verdnik

On 12. januar 2016 18:24:31 GMT+1, Klemen Verdnik <notifications@github.com> wrote:not sure if there's a word missing here:  will tie the events that would otherwise be sent together in a batch.?   —Reply to this email directly or view it on GitHub.


isnt? Nie lepiej po staremu, `!=`? W co to `isnt` jest kompilowane do JS?


I use Windows 10 64x so you might want to get more descriptive.


Hm... I think this should be done inside `Tools/ardupilotwaf/mavgen.py`, no? You can just declare a function called `options` there, just like it's done here, and it'll work fine.


Is there a need to check for labels collision?


Is that something you need me to do and repush?

From:  Mads Kristensen <notifications@github.com>
Reply-To:  MCKRUZ/schemastore <reply@reply.github.com>
Date:  Tuesday, January 12, 2016 at 2:07 PM
To:  MCKRUZ/schemastore <schemastore@noreply.github.com>
Cc:  Matt Kruczek <kruz79@gmail.com>
Subject:  Re: [schemastore] Added HTMLHint schema (8e3b698)

Looks really good. Thanks!!

There are a few issues with declaring the property type when using an enum. They conflict in this case where the Boolean value true is valid from looking at the type but invalid when looking at the enum array. Removing the type property fixes this.

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


Nope; labels are for the user's benefit.  If they want to reuse a label, we'll let them.


Yeah, I think I thought that I would care about something related to this.


Should this go upstream, perhaps? Seems it's a bug in their code.


Zrób to jakimś selectem. Poza tym, przekazuj idki rozdzielone przecinkiem, to na backendzie będziesz mógł zrobić po prostu. `users = User.find(params[:id])`.


Alright. To me, it seems like an expected functionality out of one action. So, we seem to be testing the same function twice. I guess this is why we have the S in the SOLID principal.  Either way, looks good!


Hello


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1217.5f24da3855f145cdd801bef94d0d092da7f0e9bb](http://build.palaso.org/viewLog.html?buildId=67876&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:21:09


FOR THE LOVE OF CHRIST, stop ignoring the fucking chisel3 compatibility warnings.

[warn] fpu.scala:439: Chisel3 compatibility: nodes cp_ctrl_fma, cp_ctrl_ren1, cp_ctrl_swap12, cp_ctrl_round, cp_ctrl_sqrt, cp_ctrl_single, cp_ctrl_fastpipe, cp_ctrl_wen, cp_ctrl_ren3, cp_ctrl_ren2, cp_ctrl_div, cp_ctrl_swap23, cp_ctrl_wflags, cp_ctrl_ldst, cp_ctrl_toint, cp_ctrl_fromint, cp_ctrl_cmd should be wrapped in a Wire() in class rocket.FPU


@stevvooe This commit tries to avoid calling ReadStream on uploads that are in progress. The resumeDigestAt was called at three places. Once in Commit after all data has been written to verify the digest. This usage should be safe. The other two where in ReadFrom and Write. I made sure that these invocations of resumeDigestAt  no longer try to recover the digest in case the offsets of the writer and the digester get out-of-sync.

I also updated the blobWriter.Write function to update the digest with the bytes that were successfully written by the storage driver. This should prevent the digester from getting out-of-sync with the filewriter.


it would be great if you could just remove the 3 instances of the `type` properties and commit the changes. The PR will automatically be updated with that change


I think this string should be named "submit_button" instead of "send_button".


I recognize the value of the environment variables, and I mean to keep them! My suggestion is that, as much as possible, the "same" behavior is achieved **either** with `--log` and the corresponding env var.


Can you define the redirect policy with `http.CheckRedirect`.  Also if you can store a reference to the `http.Client` it would enable the underlying TCP connections to be reused.


Sure,
1. Not sure yet, it may be that Qt including works for all possible platforms but wanted to keep manual includes as an option for now. Qt is anyway the default now.
2. I haven’t needed AA_UseOpenGLES anywhere else so wanted to highlight that this is specific to windows for now. If windows starts to work with full OpenGL without ES+Angle this will of course be changed, probably these lines can then be removed altogether from each app.


The GUI never opens and the process kills itself after a couple of seconds of being opened.

![](http://i.imgur.com/1e1X1i9.png)

1.4 works fine, 1.5 and 1.6 do not.


I wonder how much time it will take to go in Debian stable? If I can help someone with French translation then send me a message.


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


that wasn't an "if nobody contributes, neither will I"

I'm going to take a break and learn elsewhere to stop writing garbage. My flaming shitpile api implementation vs yours is another example of why.


I am a nice comment


Bardzo bardzo bardzo kiepsko, tyle ile będzie parametrów, tyle zrobisz zapytań do bazy danych (razy jeszcze 2, bo o `users_groups`, czy jak to się u Ciebie nazywa, też pytasz w każdej iteracji). Zamiast tego proponuję: `users = User.find(params[:user_ids]).includes(:invitations, :groups)` - i tak musisz to wszystko pozapisywać, ale przynajmniej wyciągniesz za jednym zamachem, zamiast się bawić w osobnego selecta dla każdego idka.


Any reason to use `seen_add` instead of `seen.add`? (if 80 chars then you can break the line before the if)


This line only ever sets `@need_encode` to false. So the only case where `@need_encode` is actually true, is when the source is converted to UTF-32LE.


I'll go and do some playing around, honestly all updates presented above are from 2to3, nothing of my own doing.


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


Redownload 1.6


Changed* Spelling.. *facepalm*


完全同意！


looks great, thanks!


manually fixed it in the new _base.py file, please let me know if you have issues.


AddNetAreaColumnToEnvelopeSummary (mbadams5) - 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)


完全同意。我想这个PR先把流程尽快跑通，然后来优化这些要点。我创建了一个issue，连接到你的这个ocmment。接下来几个PR实现它。


Well actually it's impossible to do that, because the `--log` is initialized later, then the `DEBUG_EARLY` is used. So they can't be merged...


# #Redesign
The redesign is fixed, converted to Unix, amd merged tonight, my friends and :octocat:s!


Same comment as above (reuse `http.Client`)


Thanks!


Years in changelog are wrong from v17 to v22. Also there is a time paradox between v22 and v23.


Ok, I agree. Finally it is possible to pass ``render=bool(template_name)``. This will work exactly the same way, but is much more verbose variant.
I'll fix this with next release. Thanks!


(btw for certain really nasty failures the `DEBUG_EARLY` is actually the only way to get output)


pagarme_class não é um bom nome.


An out-of-range filemode should be an error


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


Just for the record, this branch was a one-off branch, which I was not intending to merge in as is.


porque essa declaração não está no topo?


Is it better now?


An update on this: Caching 0-𝚽 mice causes the cache size to balloon, slowing parallel computations to a crawl. This is a wont-fix until we have global caching working.


Arf, tu m'as grillé, je l'ai corrigé dans ma branche de travail :-/


loooooooooooooool :laughing:


Yep, one of them needs to win.... (try to explain when --log none,all stands for... :smile:)


dfsdf


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2351 of 2384 tests passed, 0 test warnings)

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


most of the header patches (case sensitivity) look like they could be passed to the upstream OSG repository at: https://github.com/openscenegraph/osg (i.e. they are not yet merged upstream apparently: https://github.com/openscenegraph/osg/blob/master/examples/osghangglide/hat.cpp )

I assume that this would help reduce the maintenance overhead for mxe once upcoming versions of openscenegraph have these patches included ?


True...


Is communication always over http?


the `npm` build does include it: https://github.com/aframevr/aframe-core/blob/dev/lib/three.js#L15


The app can easily accept DELETE,
but how do I tell my browser to send a DELETE request?
 a <input type='hidden' name='_method' value='delete'> does not
do the trick ?


I did

On Tuesday, January 12, 2016, vadim <notifications@github.com> wrote:

> Sorry to bug you -- can you push bumped version to Hex, please? Thank you!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeremyjh/dialyxir/commit/8945488a537fd43e86ef6d1bc59f579a082fe0b1#commitcomment-15400892>
> .
>


OK - Will take
Thanks


@ragesoss Just pushed! Now we just run `./jscoverage` from the app root, and it'll run the tests, generate the coverage, and open the generated HTML in the browser! :tada:

As a side benefit, it also exposed components that weren't used anywhere, so we were able to delete some dead code. :tada: :tada:


有道理。改！


I think we need an option to turn the Chisel3 compat warnings in to errors.
It's kind of hard to tease out which warnings you actually need to pay
attention to.

On Tue, Jan 12, 2016 at 11:20 AM, Yunsup Lee <notifications@github.com>
wrote:

> Just for the record, this branch was a one-off branch, which I was not
> intending to merge in as is.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405807>
> .
>


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1218.935a05d260c1a3eb8551199b12a2ecb1931e60d5](http://build.palaso.org/viewLog.html?buildId=67880&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:21:19


Added issue #12


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


арм boost 1.54 консольный клиент крашится


Done


So i should update code to use SetNick? I hope this will not set alot of players name to custom one...


I think returning the `message` portion within error messages would provide value.


You can't do that, that's cheating.


Can't we push this check down into `store_operand`?


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


Agreed if the Chisel3 syntax is stable enough.
Otherwise, IMHO errors could enforce duplicated efforts such as move the
arguments of Vec(n, type) back and forth.

-Wei

On 12/01/2016, 19:23, Howard Mao wrote:
> I think we need an option to turn the Chisel3 compat warnings in to
> errors.
> It's kind of hard to tease out which warnings you actually need to pay
> attention to.
>
> On Tue, Jan 12, 2016 at 11:20 AM, Yunsup Lee <notifications@github.com>
> wrote:
>
> > Just for the record, this branch was a one-off branch, which I was not
> > intending to merge in as is.
> >
> > —
> > Reply to this email directly or view it on GitHub
> >
> <https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405807>
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ucb-bar/rocket/commit/55b4f9c174b4dcc0879c322fb2e258dc302e30d8#commitcomment-15405884>.
>

--
Dr Wei Song ( 宋威 )
Research Associate
The Computer Laboratory, University of Cambridge
http://www.cl.cam.ac.uk/~ws327/


redundant line. I'll remove it.


I guess that would mean either repeating check here or building a dead GEPi. But either, frankly, seems fine. After all, we are repeating the check here anyhow.


Mind if I move this to a parameter to -removeEdit: and _updateConflictsForRemovingEdit: ?

IMO the pattern
```
_foo = false;
//do stuff
_foo = true;
```
should be avoided it possible.


added a fail, but the test is still green without the database migration.


@8191 yes it should, do you have time to create a pull request for it? I'm not around a terminal at the moment.


What is the reason for writing to the local disk?  You have an `io.Reader` as a parameter which is what `FileSystem.Create` requires


`build/` should be omitted here...


> Definitely more work but probably
> the cleanest (I think this is what you wanted since the beginning).

Yes, basically, every-time the AugmentationLoop calls the `toAugmentors`, now, it should invoke a special `AugmentorInterface` member function. So that the `HttpAugmentorInterface` will send the message through http where as the `ZmqAugmentorInterface` will send it over zeromq through the `ZmqNamedClientBus`. It's exactly the same scheme as the `BidderInterface`.

So, basically, the `AugmentorInterface` should have a `sendAugmentMessage` pure virtual member function that both `ZmqAugmentorInterface` and `HttpAugmentorInterface` will override. Then, the `AugmentationLoop` will call `augmentorInterface->sendAugmentMessage(auctionId.toString(), ...)`.

Does it make sense ?


Przed zapisaniem ma zmieniać pierwszą literę imienia i nazwiska na wielką


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


associate with #10


why store the error in  `exp` ?


Is it just copying computationStrategyInfo?  Can it not use the same copy?


added `redux-either` to handle when the `Future` isn't rejected, but the `stat` from flicker is `==fail`. (f.e. when you click on search without entering a query)

@drboolean what would you do to get rid of the ugly `map(map(toPhoto))`? Would be great to get your help on that.


Thank you!


There's also `unit.getCallableReturnType(functionModel.typedReference.fullType)`, which I guess does about same.


CoffeeScript compiles == into ===, and != into !==. In addition,is compiles into ===, and isnt into !==.

W dniu 2016-01-12 20:07:45 użytkownik Marek Lipka <notifications@github.com> napisał:
isnt? Nie lepiej po staremu, !=? W co to isnt jest kompilowane do JS?
—
Reply to this email directly or view it on GitHub.


That explains it, I was creating the result incorrectly =(.. However, don't we need a check for `!item` in the `CdbPathLocus_IsHashedOJ()` case as the original code had? My patch was, apart from being incorrect, slightly different.
```
diff --git a/src/backend/cdb/cdbpathlocus.c b/src/backend/cdb/cdbpathlocus.c
index 0e2186a..c0d2165 100644
--- a/src/backend/cdb/cdbpathlocus.c
+++ b/src/backend/cdb/cdbpathlocus.c
@@ -422,6 +422,7 @@ cdbpathlocus_get_partkey_exprs(CdbPathLocus     locus,
     List       *result = NIL;
     ListCell   *partkeycell;
        ListCell   *i;
+       PathKey    *item;

     Assert(cdbpathlocus_is_valid(locus));

@@ -430,7 +431,6 @@ cdbpathlocus_get_partkey_exprs(CdbPathLocus     locus,
                foreach(partkeycell, locus.partkey_h)
                {
                        PathKey    *pathkey = (PathKey *) lfirst(partkeycell);
-                       PathKey    *item;

             item = cdbpullup_findPathKeyInTargetList(pathkey,
                                                                                targetlist,
@@ -446,7 +446,6 @@ cdbpathlocus_get_partkey_exprs(CdbPathLocus     locus,
                                result = lappend(result, copyObject(em->em_expr));
                        }
                }
-               return result;
     }
        else if (CdbPathLocus_IsHashedOJ(locus))
        {
@@ -458,19 +457,26 @@ cdbpathlocus_get_partkey_exprs(CdbPathLocus     locus,
                        foreach(pathkeylistcell, pathkeylist)
                        {
                                PathKey    *pathkey = (PathKey *) lfirst(pathkeylistcell);
-                               PathKey    *item;

                                item = cdbpullup_findPathKeyInTargetList(pathkey,
                                                                                targetlist,
                                                                                NULL);
                                if (item)
+                               {
+                                       foreach(i, item->pk_eclass->ec_members)
+                                       {
+                                               EquivalenceMember *em = (EquivalenceMember *) lfirst(i);
+                                               result = lappend(result, copyObject(em->em_expr));
+                                       }
                                        break;
+                               }
                        }
+                       if (!item)
+                               return NIL;
                }
-               return result;
        }
-       else
-               return NIL;
+
+       return result;
 }                               /* cdbpathlocus_get_partkey_exprs */
```


Also added issue #13


Not a problem @phuslu!


I get error "Internal Server Error" when I send contact form on the web site if base_phone module installed:

2016-01-12 19:28:44,167 6612 ERROR openerp openerp.addons.website.models.ir_http: 500 Internal Server Error:

Traceback (most recent call last):
  File "/opt/openerp/v8/addons/website/models/ir_http.py", line 199, in _handle_exception
    response = super(ir_http, self)._handle_exception(exception)
  File "/opt/openerp/v8/openerp/addons/base/ir/ir_http.py", line 147, in _handle_exception
    return request._handle_exception(exception)
  File "/opt/openerp/v8/openerp/http.py", line 666, in _handle_exception
    return super(HttpRequest, self)._handle_exception(exception)
  File "/opt/openerp/v8/openerp/addons/base/ir/ir_http.py", line 173, in _dispatch
    result = request.dispatch()
  File "/opt/openerp/v8/openerp/http.py", line 684, in dispatch
    r = self._call_function(**self.params)
  File "/opt/openerp/v8/openerp/http.py", line 310, in _call_function
    return checked_call(self.db, *args, **kwargs)
  File "/opt/openerp/v8/openerp/service/model.py", line 113, in wrapper
    return f(dbname, *args, **kwargs)
  File "/opt/openerp/v8/openerp/http.py", line 307, in checked_call
    return self.endpoint(*a, **kw)
  File "/opt/openerp/v8/openerp/http.py", line 803, in __call__
    return self.method(*args, **kw)
  File "/opt/openerp/v8/openerp/http.py", line 403, in response_wrap
    response = f(*args, **kw)
  File "/opt/openerp/v8/addons/website_crm/controllers/main.py", line 96, in contactus
    lead_id = self.create_lead(request, dict(values, user_id=False), kwargs)
  File "/opt/openerp/v8/addons/website_crm/controllers/main.py", line 32, in create_lead
    return request.registry['crm.lead'].create(cr, SUPERUSER_ID, values, context=dict(context, mail_create_nosubscribe=True))
  File "/opt/openerp/v8/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/opt/openerp/v8/addons/base_action_rule/base_action_rule.py", line 178, in create
    new_id = create.origin(self, cr, uid, vals, context=context, **kwargs)
  File "/opt/openerp/v8/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/opt/openerp/v8/custom/addons/crm_phone/crm_phone.py", line 32, in create
    cr, uid, vals, context=context)
  File "/opt/openerp/v8/openerp/api.py", line 241, in wrapper
    return old_api(self, *args, **kwargs)
  File "/opt/openerp/v8/custom/addons/base_phone/base_phone.py", line 97, in _generic_reformat_phonenumbers
    % (vals.get(field), e))
except_orm: (u'Error:', u"Cannot reformat the phone number '+380441111111' to international format. Error message: 'bool' object has no attribute 'upper'")


Either works for me. The initial version I had also checked for it inside store_operand.


is this an atomic operation?


If you want to cover internals of how this function works then we can test that the last 16 chars of generated hash still the same for user class objects and for internals class objects (e.g stdClass vs SplObjectHash),

Here is what I mean: before this changes:

```php
$o = new stdClass();
$s = new SplObjectStorage();
var_dump(spl_object_hash($o)); // 0000000046d2e51 a000000003e3e4a43
var_dump(spl_object_hash($s)); // 0000000046d2e51 a00007fc62b682d1b
```
after:

```php
$o = new stdClass();
$s = new SplObjectStorage();
var_dump(spl_object_hash($o)); // 00000000054acbe8 0000000050eaeb6f
var_dump(spl_object_hash($s)); // 00000000054acbe9 0000000050eaeb6f
```

But testing this goes out of contract that `spl_object_hash()` declares.

From googling through github to look who and how uses this function, it looks like that vast majority object are user classes instances, which mean that second hash part is same for all of them.


Very much appreciate you looking into and commenting on this!
Good idea! Done.
https://github.com/Whonix/whonix-gw-firewall/commit/7ef15d52e4d61a80f82df0e02b0b3f3ba21c942d


Não esqueçam de apagar as linhas das views que foram comentadas


It goes out of the contract of it, but as I read the message you build upon this assumption in your weakref extension...?


@dcsjapan i followed password change modal ([line 142](https://github.com/sijad/flarum-ext-english/blob/480fc3427bf1fe2e697aa633b4ce4e3e940bbedd/locale/core.yml#L142)), any way this issued has bin fixed so it's not important any more :smile:


Agreed, but this function is only in this intermediate commit. I remove it later on after I merged the block/boundary object storage using the new warehouse.


your code sucks


:beers:


to be nitpicking these placeholders should be %1$s, %s2$s etc...


@wolf31o2 @dereklwood Can you please review this commit? Added dependency of cdap-hbase-compat-1.0-cdh-5.5.0 on the cdap-master.


use expf instead, since i don't know if the compiler will recognize that it is being truncated to float32


same here


is this text ok to be hardcoded? (no gettext) https://github.com/skyverge/woocommerce-customizer/blob/8e047b0d3a6c98b6fe8ccc1c9263765199b9cd0f/includes/class-wc-customizer-settings.php#L291


@LMCMLJ I added my projecy directory to pythonpath. I don't think it's the prettiest way, but it worked


Why is the enable_diego_ssh_in_cf.yml file obsolete?  I was just looking for the client config needed and I can't find it documented anywhere else.  Is there some other place we should be looking now?


I got the values from thin air... but these values passed a local no-regression sprt test at shorter time control.


Heh, I'm *shocked* I got this right on the first try


It's just a speed optimization. Straight out of stack overflow and the like.
On Jan 12, 2016 5:15 PM, "Lukáš Doktor" <notifications@github.com> wrote:

> Any reason to use seen_add instead of seen.add? (if 80 chars then you can
> break the line before the if)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/clebergnu/avocado/commit/f86e7dad65806a8be1ce36fa1e5eb4e8a6d3cbe0#commitcomment-15405702>
> .
>


Irei modificar


El cambio va en la segunda entrega...jaja...el paquete es PRACTICAS2 y la clase SumPares2b


these are gone in the latest master I think.  If not, they will be gone soon.  You should use the appropriate saved search to retrieve data from logs.


OMG! a patch I did 9 years ago. Thanks @larsbrinkhoff!


In my extension I replaced object handlers tables, which obviously changed handlers pointer, which lead to hash change:

```php
$obj = new stdClass();
var_dump(spl_object_hash($obj)); // 0000000046d2e51 a000000003e3e4a43
$ref = new Weak\Reference($obj);
var_dump(spl_object_hash($obj)); // 0000000046d2e51 a00007fc62b682d1b
```

which may lead to unexpected behavior when using ext with php libraries that relies on `spl_object_hash()` output.

Even before writing my ext I had a conversation with @nikic about why hash uses two values: object handle and pointer to object handlers while object handle fits the contract and got a reply that there are no reason still doing that, and after some discussion in SO chat yesterday we come to a solution to remove handlers pointer from hash generation but keep hash length and form the same.

Are there anything you know what can be affected by this change?


You probably could delete this #define.

Much of the only reasons why I'm seeing this macro ever get used is repeatedly this sort of thing:
```c
#ifdef USE_WINDOWS_API
    m_hFile(INVALID_HANDLE_VALUE),
#else
    m_hFile(NULL),
#endif
```

I would just do instead:
```c
/** #ifdef _WIN32
    #include <windows.h> **/

#ifndef INVALID_HANDLE_VALUE
#define INVALID_HANDLE_VALUE
#endif
```
... then you would only ever have to write `INVALID_HANDLE_VALUE` instead of checking #ifdef all the time.


Shishir, can we move even this check inside checkGrowBaseFs()? Trying to keep nesting depth of if statements to the minimum in setupBaseImage()


I thought for free plugins we had to manually bump the version and not use the `-n` suffix handled by grunt?


Probably not necessary to specify 'Obj'.  By nature of defining a class, obj is implied.


While '>, <' work with Date() lets use 'upload.initiated.getTime() > latestUpload.initiated.getTime()' to make it future proof, so that anywhere else in the code when using '===' or '!==' people will use '.getTime()'


Ah, got it, thanks.


Como é function declaration achei melhor colocar perto do seu prototype ao invés dos métodos genéricos usados.

O que prefere?
Jogar pra cima é tranquilo tbm rsrs


Just do.

 if err := devices.checkGrowBaseDeviceFS(); err != nil {
   return err
}


Are you sure these are supposed to be hex values? (0x16 = 0b10110)


I dont have concret cases in my mind


$ cat /tmp/example.js

    var Example = goog.defineClass(null, {
      constructor: function() {},
      method: function() {
        for (const target of []) {
          var f = () => console.log(this);
        }
      },
    });

$ java -jar target/closure-compiler-1.0-SNAPSHOT.jar --js=/tmp/example.js
java.lang.NullPointerException
	at com.google.javascript.jscomp.Es6RewriteBlockScopedDeclaration.varify(Es6RewriteBlockScopedDeclaration.java:149)
	at com.google.javascript.jscomp.Es6RewriteBlockScopedDeclaration.process(Es6RewriteBlockScopedDeclaration.java:113)
	at com.google.javascript.jscomp.PhaseOptimizer$NamedPass.process(PhaseOptimizer.java:285)
	at com.google.javascript.jscomp.PhaseOptimizer.process(PhaseOptimizer.java:217)
	at com.google.javascript.jscomp.Compiler.check(Compiler.java:777)
	at com.google.javascript.jscomp.Compiler.compileInternal(Compiler.java:693)
	at com.google.javascript.jscomp.Compiler.access$000(Compiler.java:89)
	at com.google.javascript.jscomp.Compiler$2.call(Compiler.java:651)
	at com.google.javascript.jscomp.Compiler$2.call(Compiler.java:648)
	at com.google.javascript.jscomp.CompilerExecutor$2.call(CompilerExecutor.java:93)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)


Can't take info lock after devices.lock.


Somewhere we're dealing with a const node that was removed from the AST and not added back in, or something.


who is the owner of an alert?


I think growFS() can still be in a separate function and we will simply call it from checkGrowBaseFS(). That way growFS() will be callable even when we do per container storage options.


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


D'oh. Thanks for catching that.


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


deveria especificar que existe no sistema


why did you remove check-ui-defs?


@AdSchellevis i can do that ;)


Ah sorry. I see you moved it now.


First attempt at implementing #6 #8 #9


do sender/receiver belong in the alert or in the producer?


Shouldn't this be a git submodule ?


Let's leave this out for now.  Needs some thinking through.


Do we remove the ability to convertColumn or is it something that we never finish?


comment about "put what you want to optimize here" etc?


alerts probably don't need a protected flag.


can these be passed in?


should "on on" be "on" ?


Why is this needed ?


Sweet! Can you make it run on travis?


would rather call subject and message -- short_message and detailed_message or something similar.  Since we will have other producers than email, let's avoid email semantics.


Let's just use the example obj function, and maybe a sleep(1) so that they don't get a bad user experience with our suggestions.


AddNetAreaColumnToEnvelopeSummary (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2084 of 2084 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)


Why the requirement for > 2.2?


OK - I see that now.


limit at 100? Otherwise this ends when we start throwing errors about limits. It would be cool if the result was "Check out your experiment at <url>" instead of errors.


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


1. with Qt4 you need to use utf16() instead of toUtf16()
2. compiler doesn't like. there are errors for example:
C:\projects\qtkeychain\keychain_win.cpp(61) : error C2440: '=' : cannot convert
from 'LPCWSTR *' to 'LPWSTR'
        Types pointed to are unrelated; conversion requires reinterpret_cast, C-
style cast or function-style cast


Please don't change the indentation here


AddNetAreaColumnToEnvelopeSummary (mbadams5) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (952 of 952 tests passed, 0 test warnings)

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


80 column limit.


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


Ditto.


I tried to separate independent tasks into their own methods to make them self-descriptive and avoid long methods.


Sure, no problem. Let me know if there's anything you're wondering about before I've had a chance to rewrite it.


On Tue, Jan 12, 2016 at 11:33:28AM -0800, Simonas Kazlauskas wrote:
> Either works for me. The initial version I had also checked for it inside store_operand.

I think I prefer to push these checks as deep as it makes sense for
them to go, even if we sometimes repeat them further out to avoid even
more work -- and I can't ever see a good reason to actually store a
ZST operand.


I only found out today that Debian was sitting on a lot of patches for httptunnel.


Thanks Chaveiro.

**/var/log path in not accessible on shared hosting. Defaulting there may cause issues with people trying to install there.**
Would it be adequate to cover this in the shared hosting & Windows installation guides, which already describes modifying the datadir path settings to match the installation in step 5, also perhaps [adding this](https://github.com/emoncms/emoncms/blob/master/default.settings.php#L52) to settings. It's an additional step for shared host & Windows users to undertake, but this is outweighed by the other advantages gained by moving the logfile location for the majority of users.

**Currently emoncms has a log file size limit of about 250Mb (log is reseted reaches this value).**
Yes, I saw that in EmonLogger.php but maybe using Logrotate (which is running by default in most Linux derived distros, including Raspbian) is a more complete solution as it rotates the log, and makes managing it's size easier, and in the same manner as other system logs, which is especially important when in Low-Write mode as tmpfs has currently only 30Mb to accomodate them. OS's which don't use Logrotate will fallback on the existing 250Mb reset.

**$config_file_version is to signal structural changes on the config file variable names, not the default values.**
I think it's important that changes we commit to the 'master' branch are fully tested before they get merged to 'stable', and if I had not forced a settings update, the logging would not be moved to /var/log/ and therefore most of the changes that I had made would not be initiated, or user tested. We could then have been in the position where those changes would not have been apparent for many weeks/months, and by that time those 'un-tested' changes would have been merged to the stable branch.

Paul


just pinging @colder maybe he remembers something which would explain the importance of the two values used for the hash(I've seen him and @jpauli discuss this PR on irc, and it seemed that they did not have any immediate problem with it though).


Also, does this line actually make any difference? The compiler does not really store or load variables there, it is using SSA anyway.


Evidently this prevents shove gestures from ever being recognized on some Samsung devices, including the S3, S5, and S6. The root cause is not clear yet, but if I can't determine a solution quickly then I'll simply revert this commit.


Ok, let's do it!


Same thing, I think you could just split it into two lines without compromising any performance.


@dustContributor You didn't read all of the changes. Look at the top of the page. The sampler is bound to unit 0 and sampler 0 in BatchRenderBackendCoreProfileInternal, as intended.

It is NOT the job of CoreProfileSaveGLState to make judgements about which sampler or texture binding Nifty is using. That's the render backend's job. All of the changes from #396 are already covered.


done - it is #656


Few things:

The simplest solution:
Future's Error branch probably be called from within `Http.get` on this one. That way, when we `fork` we can bubble the error up to where it should be displayed (in the component).

But if we didn't want to use Future's async error - like we had a `Future a (Either b c)`, we could choose to use Functor Composition (like here: https://gist.github.com/DrBoolean/c4949a0f4b8f9ba8261f). I think Functor Composition could be made nicer in JS if we passed in the `of`'s.

or if we need `chain` define a monad transformer, which is the normal solution.

Typically, we define something like an `App` that's our monad transformer stack that holds all the capabilities. (see here: https://gist.github.com/DrBoolean/352b354a47b97a8cf342)

But the debugging transformer stacks sucks (which can be made dramatically better with show instances: https://medium.com/@drboolean/debugging-functional-7deb4688a08c#.14xqojctv) and it's kinda hard to see what the hell's going on.

I'd like to work on a standard `App` that is `StateT(EitherT(Future(x)))` that is fully showable and could be used for this purpose, but I haven't made the time.


>my earlier attempt to get in the BitVector32 Debug one was rejected.

@Clockwork-muse is this what you were talking about? Was this already discussed elsewhere?


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


How would we access the report? It just runs `mocha`. Travis already does that.


could be just `$this->checkAndRemovePackageSymlinks($collection->getStorage())` because $storage isn't used later on, right?


Same here; I think having `getRetryTimeout` in a separate function makes the code more readable.


Nice one, looks great!


Hey there @steam, happy to know that Ello is using FLAnimatedImage!

We've made some changes to FLAnimatedImage that completely decouple it from any particular logging solution, you can get logging by setting a log block on FLAnimatedImage in version [1.0.10](https://github.com/Flipboard/FLAnimatedImage/releases/tag/1.0.10) and higher. We've made some other performance improvements as well, you should be able to safely update to the main fork without any CocoaLumberjack-related issues!


oh, we actually _can_ add constraints to existing colums. See commit 9410d7184


I'm not sure: will $streamMetaData['uri‘] always be set, no matter which kind of stream? If not, we should add some extra check here.


provide this as a default setting for from_address?


Não sei se seria algo avançado e/ou desnecessário, mas me parece que o erro é um bom momento pra comentar sobre o que é move by default e o que é copy, onde o erro não aconteceria. E comentar que derive Copy, etc replica este comportamento.


Lovely!


Gah! Those are fixed now too. it was probably an automated update/change that introduced that. :-(


@carriepl @lamblin is this what you meant regarding broadcasting the first dimension?


get subject and message (short_message, detailed_message) from the Alert.  No need to make them fields in the model for the producer.


Should I move this to a closure specific package?


Just realized it's probably status 200, even though it's a failure (thanks flickr). Which is why you made the Either the way it is. So...Compose or Transformer...


以后还是提pr吧，大家都能看到哪里做了调整


添加一个新的文件


111411352_ATDeprecateFields (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2085 of 2086 tests passed, 0 test warnings)

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


It's not actually `json.dumps()`, it's Flask's default JSON encoder... not sure what to call the encoder in the docs though... ??


I don't. But thanks!


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


Not sure on this one, but I think it's better to fail noisily than pick one and silently drop the other.


personally i just say once the migration file is committed, you never change it again. Name them with timestamps to dictate the order and then "migrate up" by running them in order, starting with the base schema.


Would it make sense to throw this into a proc or function? It was initially hard to read since i thought the `end` closed off the unless.


Thank you for your contribution. However, please have in mind that the (eventual) goal of pyfuzzylite is to follow the same standards as fuzzylite C++. Having said so, the Right and Left Shoulders no longer exist in fuzzylite C++. If you are keen to contribute to pyfuzzylite, a nice first start would be to rewrite all the terms accordingly. I will be happy to merge pull requests adapting the terms from fuzzylite C++. See documentation here: http://fuzzylite.github.io/fuzzylite/index.html , and see latest source code here: https://github.com/fuzzylite/fuzzylite/tree/master/fuzzylite/src/term . Cheers.


My goal was to have minimal changes or special dependencies. I think it's fewer tweaks than listed in the HOWTO which seems to be updated from older versions.  The clang compiler is fine for the C and gfortran is used for Fortran. Arpack is the only auto-dependency. Let me know if you see any problems.

I'm thinking cgx should be possible as well but I haven't tried yet.


Looks good.


The only thing I might change is the nomenclature in use here, although I readily admit that my opinion here may be due to the context (I thought "Send" as in the oibit trait used to mark things that can be sent to other threads), and that it may be unreasonable to change the names of any of these items at this point...

...But basically I was thinking that it might be better (given that "Send" is a Rust trait name) to use another name for this that makes it clear that IrcSend refers to *sending* things via Irc rather than sending some item having to do with irc? And now I'm entirely blanking on what other names could be good...

Ok, barring changing the name of the trait to `Irc` and then changing the name of the `IrcStruct` to `IrcState` or `IrcConnectionManager` or `Phillip`, I have no idea, but I just thought I'd mention the `Send` confusion anyway in case you have a stroke of inspiration regarding names.


This looks like a copy/paste error. This is still using 1.8


Consider a boolean value?


See below


Shouldn't we still simulate some favorites?


★rip


I like this solution, and in fact I have already modified my bot to use it. What I was playing with was much too specialized (built for just my use case) and I think this is more generally applicable. I do have one question for you (that I might be able to answer by reading the code, but which you are probably much more qualified to answer--I'm sure you already thought of this):

What happens if someone creates an `IrcHandle` and then just keeps it around forever. Will its wrapped `Writer` run the risk of falling out of sync with the one it was cloned from? If so, what consequences might that have? Not that this is a risk that I run myself--I intend to time out after just a few seconds rather than waiting forever for an API call--but people do goofy things sometimes.


I notice 'xrandr' has returned to the "Requires" list in the top comments. This is a command, while all the other items are packages. I would like to replace it with 'x11-xserver-utils', the package which provides xrandr. Any objections?


111411352_ATDeprecateFields (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (2053 of 2054 tests passed, 0 test warnings)

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


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


Connect is also written directly but is only one message. sendCommand adds to the end of the `pending` queue, but the client should be authenticated and subscribed before anything else is sent. It felt like a hack to write to the front of `pongs`, and writing connect and sub to the front of the `pending` queue seemed less desirable to me than writing them directly (feel free to disagree). It may also be worth setting `client.infoReceived = true` inside the connectCB or some other callback to ensure that pending messages are not lost if the server rejects the authentication (assuming some other server in our list accepts it).


If you issue a Get command at "https://htmlpreview.github.io/?"+\<anything\>, you'll get a 200 OK response. So at line 109, I remove the prefix. The suffix should be a valid URL, but it isn't. I remember we investigated this together, the conclusion is that htmlpreview's js did some magic.


I'll change it to https://github.com/kubernetes/kubernetes/blob/HEAD/docs/api-reference/v1/definitions.html#_v1_nodestatus, then we don't need this special case here.


Typo.


ok I think unless there are alot of subscriptions this should be fine. Node itself will buffer to some degree under high load as well. Might be worth a quick test for 1k subscriptions what the difference in performance and cpu usage (due to context switches), just so we have an idea.


param: PW[1], best: 208.56, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[2], best: 199.26, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[3], best: 173.51, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[4], best: 172.53, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[5], best: 157.98, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[6], best: 139.03, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[7], best: 148.34, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865
param: PW[8], best: 178.27, start: 208.00, min: 0.00, max: 416.00, c 66.027557, a 485.433865


no


just stick to web


@rainabba - nope! :) No Bluetooth nor backlight.


The keys are sorted, but it is still inefficient to do it like that and a bit counter intuitive to play the last note instead of the first.


nah worked 2 many hours


小郑同学，见笑了


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


Reminder to move the Rcpp dependency before the readr dependency here.


Are there always only 5 barcodes? If not, could generalize to N barcodes.


Sorry about that @walf443! Tag now pushed


Is there any reason to open and close files after each write? What about opening all files before loop and closing after loop?


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - i386-Windows-7-VisualStudio-12: OK (2050 of 2051 tests passed, 0 test warnings)

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


Trying using argparse to work with command line arguments: https://docs.python.org/2/howto/argparse.html https://docs.python.org/3/library/argparse.html


Reminder to add all of these to the menu unqualified and then do the command management over in Application.initializeWorkbench (which will have an impact on the visibility of both these menu items and their counterparts on the main menu).


Finally I've made great update with support of template inheritance feature. Also slightly changed the `render` argument behaviour. Now it is just ignored when no template_name provided. It is still orthogonal, but a bit more flexible now )


添加设置collection的接口


Oops, thanks!


This file cause a freezing crash when using kirby against mewtwo and kirby can't use roy's power and will freeze the game

any solution I like to play sami with a fix kirby powers.


Fixes #1


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


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


I think this was just auto-formatted. I prefer @Override on top though.


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (383 of 383 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)


develop (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: Build Failed

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


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


Clojure should be capitalized here.


No; of course there should only be valid package names in the Requires: field.


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


Hi Malathi,

    This need to be corrected. Propertyservice used to get assessment
installment, this can be done by installmentdao.... please do changes... In
case of issues late have a call.

Regards,
Ranakrishna I.
On Jan 11, 2016 22:50, "egovernments" <notifications@github.com> wrote:

> By adding @Service <https://github.com/Service> without Scope making it
> non-threadsafe and will surely break in concurrent environment due to the
> following class variable
>
>         private SimpleDateFormat dateFormatter = new SimpleDateFormat("dd/MM/yyyy");
>     private BasicProperty basicProperty;
>     private Map<Installment, BigDecimal> installmentWiseDemand;
>     private Map<Installment, BigDecimal> installmentWiseCollection;
>     private Map<String, Date> installmentAndLatestCollDate;
>     private Map<Installment, EgDemandDetails> installmentWisePenaltyDemandDetail;
>
> If these class variables to be maintained then add @Scope
> <https://github.com/Scope> to prototype
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/egovernments/eGov/commit/5bba8502d65990d4ff7cae3434167bec77c6b73c#commitcomment-15375537>
> .
>


Hi, I wanna ask how to enable Matlab plugin compilation? thank you


The second sentence is a bit awkward. Maybe something like this would flow better:

> Designed as a general purpose language, Clojure combines the approachability of a scripting language with a robust infrastructure for multithreaded programming.


yeah, I think that's a good plan - I wish there were some way to enforce that. maybe we can just make them read-only.


@JobV where?

See the case, there is no Note author name in the form:

![2016-01-12 10 42 01](https://cloud.githubusercontent.com/assets/5518/12253213/2545e668-b919-11e5-85d3-7b7b6fa2f255.png)


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2081 of 2083 tests passed, 0 test warnings)

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


I've tried that as well. That works great in the browser, but we also test (the functionality of) our (universal) bundle in a Node test runner, and aliasing doesn't work there in Webpack. No easy solution. Our best bet  might be to just fork or wrap this package to add back a `browser.js` file.


First Comments


shouldn't it be mouseState? I don't see any 'state' field/var


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


775-CurveAsResource (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2328 of 2373 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.1%25-yellow.svg)


同上


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


This assumes that all return statements without a return location (ie, all void returns) have an argument, and doesn't consider `->`. Right now, the compiler accepts ones without an argument.

I'd probably disallow returns without an argument, replacing them with `-> void`.


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - 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)


[It is ?](http://www.getsymphony.com/download/releases/version/2.6.1/)

![image](https://cloud.githubusercontent.com/assets/771169/12253264/9e1732bc-b8ac-11e5-9007-14e01f6205fa.png)

How should we phrase it ?

Also, you are using a regular expression to parse it ? Why not used simple_xml (which would could handle such a change)

> This is a breaking change,

I do not agree. It's a implementation detail... ;) As always, good tester to break things.


It's parsing values @msmirnoff , see ``amount=`` method and ``currency=`` method)


isn't that what the n->expr.nargs is guarding?


Me too.


It's shooting me down...


i like it much better!


And cool!


I truly hope so.


```
I see you decided to let <x:after ==>-10 x:before> in. (=\> relation) as it's demanded.

For the rule language it means, whenever you see =\>, it is only allowed to match with negative intervals like ==>-10, while =/> rules are only allowed to match with positive intervals like ==>+10
Probably a meta-rule can make sure of the order, but I didn't have the key idea for this, the best generalization which came into my mind for the rule language was Order:ForAllSame, and I didn't want to get too complicated with this meta-specifers so that it can at least be easily guessed what they do if the related rules not considering any meta-info and preconditions are considered.

 .mustBelieve(cycles, "<x:before &&+10 x:after>", 1.00f, intersectionConf, 10);  <- It seems that I convinced you that Sequences are an important thing? ^^
```


I blame Wiki for putting the wrong one and confusing me


Oops, yes -- this should be fine.


If I ever do an OpenGL framebuffer for non-Windows targets, it could be just as relevant there.


Let's pull these into dimens.xml and give them a descriptive name


**bold**About this Program:
**bold**Input Data format
The given postings lists can be downloaded from Piazza. The index is contained in file term.idx. The data format is
summarized in the following:
• The file is a self-contained index including the dictionary. Note there is no document dictionary.
• In the index file, Each posting is on a separate line in the file
• Every posting has three values: the term, the size of the posting list and the posting list itself. Each posting is
of the form X\cY\mZ where X is the term, Y is the size of posting list and Z is the posting list
• The posting list itself is expressed as [a/b, c/d, e/f,...]. The square brackets denote the start and end of the list.
Each entry of the form x/y means the term occurs 'y' times in document id 'x'.
Detailed Requirements
The following functions should be implemented for this programming assignment. The results should be written into
a log file (more descriptions on this latter), and the corresponding output formats are also defined in the following.
• getTopK K: This returns the key dictionary terms that have the K largest postings lists. The result is expected
to be an ordered string in the descending order of result postings, i.e., largest in the first position, and so on.
The output should be formatted as follows (K=10 for an example)

FUNCTION: getTopK 10
Result: term1, term2, term3…, term10 (list the terms)
• getPostings query_term: Retrieve the postings list for the given query. Since we have N input query terms,
this function should be executed N times, and output the postings for each term from both two different
ordered postings list. The corresponding posting list should be displayed in the following format:

FUNCTION: getPostings query_term
Ordered by doc IDs: 100, 200, 300… (list the document IDs ordered by increasing document IDs)
Ordered by TF: 300, 100, 200… (list the document IDs ordered by decreasing term frequencies)
Should display “term not found” if it is not in the index.


Missing the class modifier here, right now without it, it's considered package-protected. We'll want this to be public since we can reuse this for the database work


That said, I agree that the naming is confusing, the parsed values should be saved in a way that makes the parsing obvious to the caller, e.g.
```ruby
@amount = parse_amount options.fetch(:amount)
```


fixed LED trigger patch mistake and reverted to default clock rate


For logging, generally you want the tag (the first arguement) to be the same on all logs in a class. The convention is to usually create a private static final string called tag and give it the class name, like this:

private static final String TAG  = MainActivity.class.getSimpleName();


Oh hey, thank you :)


@igormukhingmailcom regular Ansible copy only copies from local -> remote.  This is a remote -> remote copy which takes place only on the server.  Using rsync preserves idempotence in that it can use the --checksum feature to check the content ahead of the copy, making it preferable even to the cp command.


这块已经改了，去掉两个队列了，直接用了一个优先队列


Done in the new commit.


I was planning on using this to build a bunch of product/component pairs that I'd show if nothing could be found from the product search or from the API. I can deal with that later on. Removed it for now in the followup.


Start using some CPP code in material project now? Would you consider adding more cpp code in this project?


我都没有时间来维护github了.我的都是一堆垃圾代码.啊,,,哈哈哈...有时间一起维护一个项目啊...


Removed them all. They were useful while I was playing around with the code at the beginning. Less useful now. Removed them all in the followup.


I've added a shim file in #661, but it was unfortunately closed.


Done in the followup.


could you please add a title in markdown?


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


Sorry, but that was really not obvious for me.


Yes, in his weakref ext @colder uses casting const to non-const, which has side effect (see discussion about that - https://github.com/colder/php-weakref/issues/21), where I was at replacing handlers table (https://github.com/pinepain/php-weak/blob/master/php_weak_reference.c#L181) so we both was at the points whether we had to choose which evil is less. I couldn't make a choice so this is how this commit appeared.


email producer should be a subclass of an abstract producer with a send method.  send_email violates that abstraction.  The send method should probalby take the Alert as a param, and use it to construct the message.


Yes. :)


Combine two queries into one outer join query.


I think we need to restore passing parameters in this call, onDataLoadedCallback


Fixed.


@bitfed
First off I got no issue with people forking this project and making improvements on it or making there own version of it or really anything HOWEVER when they fork it and create a repo with the same name as the project then go to the original projects members and forums telling people to switch then yes I got a god damn problem!

@ulchm
I purposely have ignored bug reports and pr's for good reason being that the release I'm creating those pr's and bug reports are useless for and have been already implemented not to mention I'm focusing on the code and doing anything else takes away time from that which is my time that I'm taking away from family and my life so unless you want to step in and fill the gap how about show some damn respect!


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


What is the motivation behind this change? It doesn't seem to improve or fix anything?


I thought: what if we created the task generator inside a `build` function in `Tools/ardupilotwaf/mavgen.py` as well. Additionally, we could tell waf that `mavgen` task should run before any `cprogram` and `cxxprogram`, or seting `ext_out = ['.h']`. If the latter works fine, I would prefer that. See https://waf.io/book/#_build_order_constraints .


Shouldn't have deleted either of those.

a) I added them because without them, tens of undeclared function errors ensue.
b) Your commit says removed usage of windows.h.  Here you're removing stdio.h and string.h instead.

Now I need to PR both of those #include's back in because without them you can't use FILE, memset, memcpy, sprintf or any of the other features the `md5.h` code is dependent on.  Maybe in your case it works but that's because the standard C library headers on Windows are done in Microsoft's tangled way.


*nods*


unused imports I think.


Fixed with d08047d0


Awesome @timonus, we'll take a look and delete this fork if all works well. Thanks so much for your hard work. We :heart: `FLAnimatedImage`


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


Why are you removing coalescing?


why are you filtering this?  Wouldn't you want to process all AlertSearches?


Yay!


Fixed with 9b08a1e and d7c7293


:smile:


2nd push add course path


Should have asked earlier.  Is there any reason we are not adding the new field name and computation strategy to the end of column info so that existing fields do not have to be moved?


As a general rule, we prefer the arrow package for datetime management.  It does a much better job of timezones than the datetime and pytz packages (IMO).

import arrow
arrow.utcnow().datetime


I think it was left hardcoded since it copies the text from WC core and is meant to match that.


No worries, I think you made a valid point.


Please refrain from leaving debug artifacts in core files. It may be a good idea now and then, but we've been doing it far too much.


Suggested edits from 1/12/2016 standup:
-For ASM/BSM diagram, add legend explaining grey triangles
-For nBCR diagram, convert to Visio, add grouping rectangles, generally make less messy


I would have done the opposite (const uint8_t* p =...). In theory it would allow the compiler to generate more efficient code because it can assume the pointed data doesn't change.


There are still some changes that i would like to do in the minimax java class as there are some errors in the code.


I doubt that you need this.  May just need to specify the filter in your view, but probably you already inherit an appropriate one if you are extending SavedSearchViewSet.


That's great, thanks! :heart:


Valeu pelo feedback.

Acho que o topico é um pouco mais avançado pra introduzir nesse ponto, e que poderia inclusive ser uma outra apresentação. Quando falar sobre isso vou avisar que tem mais sobre o assunto.

Nos próximos tópicos eu tento explicar um pouco sobre `move semantics`, mas bem de leve. Você acha que deu pra entender, considerando ser introdutório?


Why not extend SavedSearchViewSet?


I already took care of that. It is derived from targetLooptime. I do need to initialise it once
https://github.com/borisbstyle/betaflight/blob/betaflight/src/main/mw.c#L662


This could be done with just one call to `prepend_value` :-)


Implementation of Functions


5 minutes


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


Updated


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


Implementation of PageObjectModel (POM) Framework


Well, just hashing the handle is enough yes, however some user code could use the value for some black magic tricks. It even happens that the same hashes show up through several requests because of memory management layouts filtering OS ASLR (reusing pointers), I guess some user code could rely on such a scenario (even if it should obviously not).
So I'm wondering if we couldn't break something merging that in actual stable branches.
Of course, we can merge that against 7.1, but what about stable releases ?

Perhaps bringing more people to the discussion could be good ? A topic on internal too ?


it should be LearnCode_Academy_YT


Format this sql into multiple lines.


I certainly don't mind having formatting changed but removing @author is quite harsh ;)


Also, for this 2nd run,
I used the following code
return make_score((mg_value(score) * PW[pos.count(PAWN)(Us)]) / 256,
                                (eg_value(score) * PW[pos.count(PAWN)(Us)]) / 256);
instead of
return score * PW[pos.count(PAWN)(Us)] / 256;

Cannot check this now, but apparently the second way is faulty.
maybe some overflow ?!


I did similar on my branch - great minds think alike :)

https://github.com/sawilde/opencover/commit/4b19ccdfec4a7fe68f6f19b302eb7dc32a546f96


are you sure it is coming from the local config, or from the remote one? What is if maxChunkCount = 0? Do we need the same for maxMessageLength, something like bytesLeft?


Yes, @mariospr reported on the upstream issue at https://codereview.chromium.org/1324513002/.


IMO, tudo está inteligível sim.


thank you, you are a scholar and a hero. this is what it's all about man. Feels good


Thanks for submitting this. I just started using atom for Processing and couldn't figure out how to get that console window to go away. It works great.

john


@ledvinap
My plan is to have a pointer function for dynamic coefficient initialisation eventually. I was also considering to seperate it, but since someone told me memory is not an issue I kept it like this to be more clear what belongs to what. But it is a waste to use same variables for same array of state elements indeed.


None of the other files have `@author`.


When travis runs `npm test`, it generates some command line summary output, and also creates an lcov file. The lcov file gets sent to codeclimate when the run finishes, and then you can view all the coverage details there: https://codeclimate.com/github/WikiEducationFoundation/WikiEduDashboard/coverage


Ya I like that better! :)


@Memphiz
right. I was led by false premise that default dir would not be user home (if not set) - because sftp:// doesn't work as expected (from a layer using common libssh framework).

by looking into the code it just looks it can't as it is forcing own settings over user's ~/.ssh content/settings.


Super to wygląda :)


Looks like this can be automatic json codec.


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


le head


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


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


@solekiller I'm so happy that you love it. It gives me joy in life!


No, it won't, I added a delay to the bitmap cacher so you guys could actually see it, otherwise it would take a split-second. But yes, once we have about 200 files, it will last about 5 seconds, even without the delay.


The bad result might have nothing to do by SPSA, (BTW I used the default ck)
but possibly a coding error.

See my latest comment on https://github.com/Rocky640/Stockfish/commit/22252f61298c172088c5e09d716f5317a96e3fa2


Poder ser útil para uma certa audiência ter que comentar a utilidade de não se ter um garbage collector. Muitas pessoas consideram qualquer coisa que não seja garbage collected coisa baixo nível a nunca ser usada e preconceitos do tipo.


ACK. Functionality looks good. Conceptual growth (can-kick) looks good too.


But should we care about black magic?


I noticed, but that's not exactly unexpected because they are written all by you :)
I just stumbled upon this since i refreshed my codebase and it vanished - which confused me.


Can't do that, since we have this Star Wars font, the capital O displays as a capital N, not sure why though. We're going to have to make our own font anyways.


This causes the error msg to not show errno any more.


So, I applied a different change that mentions the fact that lastpass-cli is available in Fedora as a binary package, as such I don't think this change is as relevant.  Also, none of the other distro building steps mention asciidoc, though there is a blurb at the bottom that states that AsciiDoc is a prerequisite.

If you think it's still useful, we could just tack asciidoc onto the end of the ```yum install...``` line for prerequisites for older redhat releases.


Use automatic json codec.


There are problems with (! $lastlineread =~ /^.*\z/).

! binds tighter than =~, so the test is equivalent to ((! $lastlineread) =~ /^.*\z/) which always succeeds.

It should be written as either (not $lastlineread =~ /^.*\z/) or ($lastlineread !~ /^.*\z/), but more importantly, the test is wrong. .* will match everything up to but not including \n so /^.*\z/ matches if there is no new line and doesn't match if there is a newline so it should really be

  if ($lastlineread =~ /^.*\z/) {
   push @{$body}, "\n";

However, I too am concerned that appending '\n' to the body will break all signatures, not only DKIM but also PGP or whatever may have cryptographically signed the body.


Abbreviation does not thrill me.


I think it's better to be clear to people who are reading the code for the first time.


.withEvent("startup", "from restart") would be better here so that the fact that it's a restart is captured in the event details field.


up to 2ffd11bf824dab


looks good, and verified on jenkins


On 12/01/16 21:32, Daniel Gustafsson wrote:
> That explains it, I was creating the result incorrectly =(..

FWIW I think it was my bad actually, I rewrote that function already in
my previous patches.

> However, don't we need a check for `!item` in the
> `CdbPathLocus_IsHashedOJ()` case as the original code had? My patch
> was, apart from being incorrect, slightly different.

Hmm, I guess you're right.

- Heikki


good one bro


Using the commit message for commentary. Hopefully proves useful *crosses fingers*


This should go in a separate file i.e. `updater4pyi/upd_iface_pyqt5.py`


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


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


May want to `assert(drStream)` if this is now production code.


I'm just a user @echel0n, and from my perspective, it's a good change and part of the evolution of SickBeard... er wait I mean SickRage.  I respect the community as a whole, and it appears the community as a whole has moved on.  I think it's a great sign they will decide together on a new branding to move on to, rather than a single person lording over it with their ego.  I was impartial until you started calling people names and saying how superior your coding is, because that has not been the experience as an end-user.


Correcting image link


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


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


Or maybe not, given how few people run the ee in debug. Might want to throw a fatal if it's null.


develop (lefticus) - i386-Windows-7-VisualStudio-14: Build Failed

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


@stulees I'm wondering if this should be `rise-storage-invalid-format` instead of the pluralized version? There may only be one file in the folder, and I think singular may be more intuitive.


develop (lefticus) - Win64-Windows-7-VisualStudio-14: Build Failed

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


Oh! Is there documentation for that fact? If not, I can add it, and then yes! I *can* make it run on travis!


I remember posting a comment about this. It got lost with the squash...


I think that you can get away without having `__builtin_types_compatible_p()`, see this: http://www.cplusplus.com/forum/general/21246/#msg112085

For example (not tested, just a little more sure than banging randomly in the keyboard):
```
#define _SOL_TYPE_FORMAT(type, format) constexpr static const char* _sol_int_format(type x) { return format; }
```

Then I would have something like:
```
_SOL_TYPE_FORMAT(int, " (%d) ")
_SOL_TYPE_FORMAT(int8_t, "(%" PRId8 ")")
```

It should be possible to have the same thing as in C, i.e. having this all being resolved during compilation time.


Documentation? What kind of operation do you think we're running here?


Hey @yvettecook ,
 Thanks for the PR! Any chance this could be consolidated a bit?  I prefer to use inline blocks rather than breaking them up in variables.

Also, this is using `nil` for the services array and not the HeartRate UUID.

I need to make a nullability pass on this to make the optionals a bit less painful. Hopefully I'll have something soon.


moving > symbol


@winterz Ooops, wrong commit. Will fix that asap


Ugh... so now we're going to churn at least 4 2MB buffers every tick interval? Guess it's not that much worse, but blech...


smarty?


Why not just try this optimistically and fail on error? We could pass something back from fr.Seek indicating it cannot be seeked.


Oh, I see. You're only churning that active one. That's fine, then.


Hey @Robinlovelace , can you resist the temptation to force push to pct-data.  Any other repo is fine but force pushing to this will cause merge conflicts all over the place as it is pulled automatically.
Nice work getting all this in btw!


@jpauli If anybody was extracting pointer values based on this, then this commit fixes an address space information leak.


I think this should have been a PR, but did not make it to my repository. Maybe you could do this manually? Thanks!


Fine, but I don't even see where mesh_interface is used, would be good to drop if not necessary.


oh, this looks like Spring before it had Annotations and programmatic service wiring.


@donnapep If we go singular I think we need "folder" in the event name. What about a compromise, "rise-storage-folder-invalid"?


The real check that needs to happen here is that if `offset` doesn't match the current upload offset, this should fail. Perhaps, this check needs to be further up, but that is the basic gist of what needs to happen here. Basically, it requires the current upload progress and the PATCH offset to always match.


Thanks Nimisha!  I do love itertools.


> I also updated the blobWriter.Write function to update the digest with the bytes that were successfully written by the storage driver. This should prevent the digester from getting out-of-sync with the filewriter.

Isn't this what `io.MultiWriter` does effectively?


Using the 1.5 android gradle plugin http://developer.android.com/tools/revisions/gradle-plugin.html
the build threw an error regarding:
'Cannot set the value of read-only property 'jniFolders' on task'.
I found this and attempted to fix via the google group link here: http://stackoverflow.com/questions/33758421/google-appinvites-break-build
Everything built fine and looked good so I committed.  I later did full clean's on ndk and gradlew and rebuilt and was left with an apk of half the size.  It doesn't seem to be getting the different binaries.
I tried a few things to fix but could not.
Until I have more time will revert the change and use the 1.3.1 plugin which I don't think will error.


@stuartc , #48 , this works well and is super easy... look at the "data" that I pass to this table!

My question for you is whether you can figure out how to add back the "inspect" and "retry" buttons. I'm at a loss. Eventually, they need to be values for their own keys in that "data" object.


This is very nicely done! I tested a bunch of scenarios before even looking at the code and got results that made sense for everything. Only thing left I guess is to remove the ambiguous situation from Gen 3/4. :+1:


I'll do the same for receipts if this seems workable.


At any time, there should be only two DR streams being used, either regular or compatible.


delete all project


TeamCity LabKey :: LabkeyRelease15.3 :: CDS postgres 9.4 [Build 127](https://teamcity.labkey.org/viewLog.html?buildId=296405&buildTypeId=LabkeyRelease153_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 42 Build time: 01:21:02
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <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.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

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

```


then let's make the pwm easier to use? Maybe by changing its ports, maybe by adding helper ports, maybe another pwm node?


SYSERROR() will cause list_defined_containers() to yell on stdout or stderr. What could we do instead?


teşekkürler!! kapatma meselesini kod satırının yanında çıkan ünlem işareti nedeniyle öğrendim. iki ayrı scanner sınıfını da acaba olabilir mi diye denemek için yapmıştım. integer ve string için farklı scanner sınıfları oluşturmak gerektiğini de öğrendim bu sayede. tekrar teşekkürler.


@stulees Yes, I think you're right. It needs "folder" in there. I'm fine with your suggestion. Thx.


SASSA


Final decision about properties, I don't add properties due to the lack of a way to define them "defined but unknown value".

This is how it would look like if there would be a value, without the value:
```js
    var symbol = new AST_VarDef({
        name: 'arguments',
        value: {
            properties: [
                new AST_ObjectKeyVal({
                    key: 'length',
                    value: null, // TODO insert AST_Node instance for unknown but defined value
                })
            ]
        },
    });
```


doesn't this happen auto-magically in the data source?


Hi all, hope all is well.


should we use size-specifier for old/new?


I think this should be 'UTF-8', in caps.

Also it's not safe to simply assume this encoding - there should be a twiddlable option, or this is going to blow up on latin1 documents.


Tkank you ;-)


size specifier? I guess `int8_t` will do.


removing the bracket around (5/9) fixed issue of method always returning zero. This is because 5/9 will return 0 unless it one of those numbers is a float (docks values after the decimal), and the whole expression is multiplied by 0.


Perfect. ;)


I've got some boilerplate stuff you can probably leverage over in my eventsauce project. It's got Mocha, Istanbul, Isparta coverage all integrated together. It's a pain to get it all working together at first, but it's definitely worth the effort since you can reduce the toolchain dependencies for contributors down to 'npm install' :)


Adicionei comentários no https://github.com/bltavares/presentations/commit/6c01ecd68fea19d4204558ccd63dc355cb68a33d

O que mais você adicionaria como beneficio?


Any chance we can make this non-Envision specific?


This commit doesn't compile and also removes the spawning of smoke particles?


just a formatting update


I was wondering why the math but then I realized this is actually a bit operation. So what if we translate this to bit in order to make it clear?
```c
priv->new = (priv->input_a << 1) | priv->input_b;
```


oh . I've forgot to include html file too!! next commit...


TODO: improve these regexs, using the following stolen from PHPCI (sorry Dan!)


@ryanfb and/or @hcayless do you see anything alarming in this change? What I'm trying to do is recover a bit more gracefully from failures on the call to @publication.origin.submit_to_next_board .  With the new pass through community, this step can result in a call to post to an external API and I'd like to be able to abort finalization with an error to the user if this step fails).  Thanks!


Your fix caused another issue. When model is not registered asynchronously, then event listeners are registered twice #45.


same here, it's actually a bit operation, not an arithmetic, it would be cleaner to write:
```c
quadrature_encoder_matrix[(priv->old << 2) | priv->new]
```


googleDns (System.Net.Sockets.UdpClient) has no IDisposable interface.
Only method I found is .Close().
Is this false Coverity alarm?


Copy/paste error. Should put back the original Netflix copyright statement (keeping 2015 as well).


WILeave a line note


I don't think adding a VarDef for arguments is the way to go.


Using find is better practice incase it isn't found. The end-user gets redirected to a 404 page instead of seeing an error.


develop (lefticus) - i386-Windows-7-VisualStudio-12: OK (382 of 383 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-99.74%25-yellow.svg)


Add an else statement in case it doesn't work.


this check and error can go inside checkGrowBaseDeviceFS() as well. This would be the first check and if this is the case error out. Second check will be if sizes are equal. If they are then, simply return. Otherwise continue to grow fs without doing any checks.


The two streams appear to generate their sequence numbers independently. This appears to be a determinism problem. Scenario:
1. Normal DR stream is active
2. Generate 3 txns; normal stream sequence=2, compatible stream sequnce=-1
3. Take trunction snapshot
4. Connect consumer; transactionally switch to compatible stream
5. Generate 5 more txns; normal stream sequence=2, compatible stream sequnce=4
6. Fail without truncating

Now, on recovery, two things are going to happen. 1. We're going to truncate to the snapshot sequence number. This, itself is already a problem--we want to truncate everything, since it's all in the CL, but we are going to truncate away only the last 2 txns instead.

Then, something even worse happens--we play all 5 txns from the command log. Now, the first 3 txns are duplicated, with two different sets of sequence numbers, and the last two have different sequence numbers than before.


Let's call this something a bit more precise. E.g. `projectsFromIncludes`


Should we just do "info.hash"?


Yay


looking at this array + switch + the operations above to get the array index, isn't it easier to just write in pure-C the checks? Like keep `input_a` and `input_b`, add `previous_a` and `previous_b`, then you and/or these 4 booleans.


develop (lefticus) - Win64-Windows-7-VisualStudio-12: OK (382 of 383 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-99.74%25-yellow.svg)


This can be fixed by either a) always dumping both sets of sequence numbers, or b) always keeping the sequence numbers in sync between the two streams by transferring them over when you switch from one stream to the other.


IIRC msg.headers is a `defaultdict(list)` so this should be
```python
msg.headers["X-Relay-Country"].append(" ".join(result))
```


Why is there [0] here?

Also use .first instead for the "Ruby way"


You are perhaps overcomplicating it. I also saw that we have repetitive code in the TypeCollection. Until we have lambdas, I believe the old approach is better.


Technically writing to a well-known file is a security hole: if another process races your process writing to and reading from the file, then whoever owns that process get you to read/display whatever they want...


Unnecessary


here if both ports changed at the same time we'd be emitting 2 packets. Is this okay? Otherwise we need to add a 0-timer to emit a single packet.


Deliberately didn't do it that way; two queries more accurately represents the shape of the model.


Interesting that no ID was automatically created.


This class is abstract. Shouldn't it just be a "normal" class. TypeList isn't abstract either, and I suppose we just want to use TypeMap in many cases.


It can't be.  It can be a `WrapperJsonCodec` though.


can this be more specific?


It most have as I can use `using`

https://msdn.microsoft.com/en-us/library/dn823305(v=vs.110).aspx


Do mongoose.model('Review') instead. It's a better practice, because relative file paths can be messy.)


a release tag (or lacking one a commit hash) would be preferable


This shouldn't ignore minor commits


Also can't be, but could be a wrappercodec.


and the 1.3.1 android plugin doesn't correctly recognize 2.10 as being > 2.2 so have to use 2.9 for now.


I'm a little confused, does this happen once per translation unit, or once per project?


Shouldn't it ignore minor commits if cutting a patch release?


use .find() without the {} argument. Functionally equivalent, but looks cleaner.


Do we need to say it is a map? Normal players don't know what a map is anyway.
Also there might be a better syntax than what I suggested, but idk.


Maybe use %s?


res.json not res.send


@thenickcox We can undo these now, correct?


:reviewId instead of :review


findById instead of find


res.json


Errors should be a flash[:warning] instead.


Let me know how it goes..


Review.create(req.body) is enough


This should be nullable. Don't continue on null, and just use getVisual rather than getVisualInner.


dp[i][j] 表示前i个数, 第i个数调整为j 满足条件, 所需要的最小代价


for these ports, are we looking into actual booleans and always emit the packet or should we convert them to empty packets and only emit when that action should be triggered?

I guess by using some real example we could see that. If we end using "pulse if true", then we should be converting to empty packets.

Also would reduce the number of packets in the flow (would result in only one packet instead of 3).


That's a good point, I didn't even think about it. Should we just remove them from everything?


Again, just use getVisual rather than getVisualInner.


reviewId


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1219.f05222b8be447ff21cc51195d12027dc7803e091](http://build.palaso.org/viewLog.html?buildId=67881&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 824, ignored: 7 Build time: 00:07:07


#4702-AirloopHVACUnitarySystem-autosizing-incorrect-for-heating-only-systems-when-using-FlowPerHeatingCapacity (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2082 of 2083 tests passed, 0 test warnings)

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


Is there any way of pretty printing the output from the traverse function? Or at least making it a bit easier on the user when creating the output to display?


findById


I currently don't have alternatives (yet). This is handled as token and is threaded differently that way.


o primeiro comentário sobre poder ser usada de outras linguagens, [Go é um contra-exemplo mais recente disto](http://blog.ralch.com/tutorial/golang-sharing-libraries/), então acho que não é tão vantajoso o comentário, apesar de que um lib Go carrega todo um runtime, o que é algo não muito convêncional mesmo.

Sobre outros exemplos, o que me vem à mente é a possibilidade de entrar no campo de AAA games, que hoje em dia é dominado por C++, e em tese, algum dia possa ter ótimos game engines Rust. Nenhuma linguagem GC balançou esse campo até o momento.


Example Ability:

Holy Light heals a target upon your cross-hair as long their health is below 60. If a enemy or there is nobody on your cross-hair, the ability will heal you if available.

```python
@Paladin.ability
class Holy_Light(Skill):
    'Heal yourself or ally upon ability.'

    ability = 1
    max_level = 8

    _msg_a = '>> \x04Devotion Aura: \x05{name} \x02cannot \x05be healed.'
    _msg_b = '>> \x04Devotion Aura: \x05Healed {name} for {heal} health.'
    _msg_c = '>> \x04Devotion Aura: \x05You were healed by {name}.'

    @property
    def _heal(self):
        return 20 + self.level * 5

    @cooldown(5)
    def player_use(self, player, **eargs):
        target = player
        if player.view_player:
            target = player.view_player

        if target.health > 60:
            self.player_use.remaining_cooldown = 0
            SayText2(self._msg_a.format(name=target.name)).send(player.index)
        else:
            target.health += self._heal
            SayText2(self._msg_b.format(name=target.name, heal=self._heal)).send(player.index)
            if player.userid != target.userid:
                SayText2(self._msg_c.format(name=player.name)).send(target.index)
```


Missing )


I'd say so.


If I'm not wrong, now that we have the versions in the metadata there is no need to put them here unless we want to override them. Let's leave only the ones that need the git repo in the Berksfile.


son dediğine dair ufak bir düzeltme: farklı scanner sınıfı metodları kullanmak gerekiyor (farklı scanner sınıfları oluşturmak değil)


With a TypeEntry (if such a Type is implemented) you could just call that with getVisual and this loop would be shorter (but sill the same logic, just moved elsewhere).


I actually almost copied mine gulp setup from yours, skipping isparta and using simple config.js instead of config module. :)

You say it was _almost_ ready to be cross-platform, I don't have a Win machine at the moment, what was the problem with that gulp setup?


On my VM, this did not propagate ``max_retries`` down to the ``chord_unlock`` task, which still has ``max_retries`` of ``None`` (no limit).


whoops, looks like it! feel free to patch


Try to access the second element of an array object.


Not totally sure, named thusly because `.get` grabs everything on on the Org, whereas `getInfo` just grabs

```
{
    "created": "xxx",
    "deleted": "xxx",
    "description": "",
    "name": "xxx",
    "resource": {
        "human_name": "xxx"
    },
    "updated": "xxx"
}
```


would this have caused a crash? cause one of my janus instances just had a seg fault and i had the latest version before this commit:

[12043593.360931] worker[60421]: segfault at 18 ip 00007ff8e9d2b35b sp 00007ff8caffc850 error 4 in libjanus_sip.so.0.0.0[7ff8e9d18000+19000]


Version 0.6.0 should be already OK. Just after releasing (and tagging) I always bump the version to avoid conflicts. Last tag and version in supermarket is 0.5.0, so we can safely work with 0.6.0 until we release again.


TypeMap is very much comparable to TypeCollection. There are different types of collections, as there are different types of Maps. Map has no constructor, its and interface liek Collection, thus we always need an exact type such as HashMap to define what implementation we want to construct when getting a new one (and we do this very very often)


Oh, and the color thing was just some extra bitmaps I added of colored rectangles to test what it would look like when we had several bitmaps.


ACK


could/should we consider the range's min/max? This would also be the question when we send the packet.


IS there a colon (:) between key and value? I believe there should be.


On Tue, Jan 12, 2016 at 12:34:37PM -0800, Christian Brauner wrote:
> SYSERROR() will cause list_defined_containers() to yell on stdout or stderr. What could we do instead?

something like

./src/lxc/lxc_usernsexec.c:85:          printf("WARN: could not reopen tty: %s\n", strerror(errno));

would be fine.  No big deal - just wanna see the cause of the inability when
error reports show up (i.e. enoent vs eperm)


to insert code


sorry if this is wrong place for it but 'common' seems to be excess here, might throw "undefined name 'common'" or something :wink: just sayn'