old supg (broken) <--
new supg (fixed) -->
![newsupgresults](https://cloud.githubusercontent.com/assets/7593038/10080636/bc306d2a-62be-11e5-8636-2dca37ee98c1.png)


@tomwhite just wanted to confirm regarding this point.  Should this method be reimpl'd using the more complicated version?


meh, IMO, i'd recommend `ex` or `exception` or something instead of `e` since a lot of people use `e` to represent `event` from an event handler. so yeah, `ex` or `err` is preferred over `e`. (that's just nitpicking, but just always want to side-step any confusion where possible)


Didn't realize this was the case for so long. Rather than pull the recipe, might be best to just add a note to the Descriptions. As far as I understand, the version is basically random, but only for the .pkg recipe is there the possibility that there will be an error, due to the different pkg structure.


....................... PRs?


We did update the docs for this transform to mention that filtering is performed. This is just a temporary guard to prevent errors -- eventually we will add support in `AddContextDataToReadSpark` for unmapped reads and remove the filter, but until we do I think we should have this built-in protection rather than relying on the caller to filter.


I don't think the issue should be fixed here tbh


A comment for this section would be helpful


Your final elif statements for both of these could be changed to else, since If n is not equal to 0 or 1, then we want always to run the code inside of your final elif statement. You would use elif if there was another condition you wanted to check. Think about what would happen here if your series started with a negative number.


Regular expressions... :anguished: Otherwise, this looks nice. It's nice to have the implementation for pulling out the tests in pulled out into its own file


"noise" might be desirable if the user was writing a plugin and simply forgot the shebang, in which case it'd be customary for the proxy to say why it rejected a seemingly correct plugin.  I guess it actually depends on the case, and should be left at the discretion of the proxy author.


rax retry tests


Ah, for shared source I would definitely avoid resources at the moment. No good way to share the resx in a meaningful way.


не должно быть запятой в конце


เยอะเกิน


shouldn't need to to have this duplication; you can just pass the value, and if it's a thing it'll display in the input, if not, it won't.


This is not a commit to master, its a manual rebase && fast-forward merge of #2243. And what else was I to do about this cast without squashing it into warmist's changes?


Does this make a difference? Don't see much changes.


This was a necessary evil. Without it, karma was failing because angular got loaded for each test individually. This bootstrapper just pulls all the tests into one webpack bundle for the tests to run. The reg ex gets simplified later. I only put `filters.*` in there to limit which test files ran since most of them weren't compatible this early in the migration. Becomes: `/\.tests\.ts$/`


@droazen so should this filter be removed and checking for unmapped reads should be performed in `AddContextDataToRead`?  Let me know how you would prefer it.  If you think the filtering should occur in `AddContextDataToRead`, let's just make sure that this is documented.


Sweet!


you guys are funnny


Add a distingish show version foreach repo. So we're able to check the version of seat seatapi and further using console.


two*


@DAB0mB Lets say we added a file called import.html into this package like this `Api.addFiles(['import.html'])`, content of this file looks already familiar to you:
````html
<head>
   <script>
    document.load(function() {
      System.import('client/app')
   })
   </script>
</head>
````
Will it get eventually to  user app's html, to the head section?


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


I know isn't that nice? No `/// <reference` anymore... all deps clearly spelled out... all types magically imported... ! Makes me so happy!


это что за функция?


Ugh, yeah... that's going to get gross...


Seems cleaner if this function just takes a count parameter (the caller could set this to math.ceil(min_perc*n))
Also, I'd add a comment that it returns `None` if the `min_perc` cannot be satisfied


s/fast/early/, but that's no biggie


Suggest to rename the option as `resolve`, `reverse` or `lookup`.


SMART using snakecasing instead of camelcasing for your column and table names to avoid the whole need for quoting camelcased columns/tables :)


Still trying some things, but I’m trying on the cluster now, because of the very low precision there of the clock. Seeing how different clocking tools work. As soon as I know more (also about platform compatible timings) I will see if it needs to be updated.

> On 24 Sep 2015, at 19:21, David Blom <notifications@github.com> wrote:
>
> Does this make a difference? Don't see much changes.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/davidsblom/FOAM-FSI/commit/3df3c6dab1a7e7050a6f10fa1d92d89a692874b1#commitcomment-13420612>.
>


Requesting a stomt by ID or URL available


We could always do what NuGet.exe did :trollface: https://github.com/NuGet/NuGet.CommandLine/blob/master/src/NuGet.CommandLine/NuGetResources.resx#L602


похоже, это какой-то левый файл


@augustf I'm not sure how to resolve this, but since this version contains a change to the javascript, users will have to run `bundle exec rake assets:precompile' after the plugin is updated.  Otherwise the js changes would get included in the application.js.  At least that is what was required in my production environment.  This will be true for any plugin with javascript changes.


Good call. Thanks.


Just keep this as `get` - there's no reason to change the name.


`kie-maven-plugin` should have been moved to `droolsjbpm-integration`. It does not exist on master anymore, but there are some leftover files here on 6.3.x branch. @mariofusco I guess these 6.3.x files should be removed?


@azturner The fix that we drafted forces the block to update the cookie when the query string changes.

New call hierarchy:

- OnInit, CampusContext checks for query string ( campusId=2 )
- Update/replace cookie if the ID value changed
- Trigger a new RockPage.SetContextCookie, with refresh
- Page continues load with the CampusContext matching the query string campusId


I'm pretty sure that would error but I could be wrong.  Its used in extension only but the mvn version plugin won't report it so that was my driver behind adding it under site as a dependency.

Sent by Outlook for Android

On Thu, Sep 24, 2015 at 10:01 AM -0700, "dbwiddis" <notifications@github.com> wrote:
Question (for my education).  Is it necessary to include the version both times?  If we left off the version under "extension" would it just use the version from the dependency?

---
Reply to this email directly or view it on GitHub:
https://github.com/dblock/oshi/commit/cf577e52cbed8a83d99bd276f7b9c145712d6886#commitcomment-13420137


udpate


Hmmm, I can see it in landscape. Is that a version specific issue again?


This indent is wrong, the dcd should be closed *after* all steps are written. In any case, rewrite with the context manager protocol if possible.


Sounds good!


Makes sense. `reverse` is good in terms of DNS. I'll go for it


indent is wrong, yield should be inside the loop.


Contribucion "https://github.com/martynbm/cm-12.1-20150430-UNOFFICIAL-millet3g"


indent does not make sense


Oh, so that's why I was confused. This isn't actually too bad. (The regular expression)


1730 should be inside the loop


There is no longer a difference between building remote or local.
You should be able to just `USE_CONTAINER=1 make build` and it will build against your docker engine.


Why was this whole code block deleted??


This should be the exact opposite. If there is a space in the text field, we want to do a Google search, otherwise open the website.


Nice! VSCode is awesome


this line is not right


There's weird stuff in this commit. Should be redone and then amended/rebased.


you want to break out of this loop once len(output_hotels) == n, otherwise you are iterating for no reason.


test


So, another win of this whole change is that we don't need the `bower-*` repo anymore.


We'll be publishing this repo and consuming it directly.


Ah, right. As I commented later. That does make it a lot clearer. I was confused about what this was trying to do here.


The git commit messages do not make sense. Please use `git commit --amend` to rework. Then, use `git push --force` to refresh.


Yes, it should be done at the `AddContextDataToReadSpark` level as in `dr_lb_spark_bqsr` (there is already an existing comment to `add()` mentioning the filtering). UNLESS, that is, you need something to do while waiting for `dr_lb_spark_bqsr` to finish code review, in which case you could add support in `AddContextDataToReadSpark` for unmapped reads (unmapped reads should be returned paired with empty or null `ReadContextData` objects, I think).


Ideally declaration of `p` would be in the loop now (but don't bother too much, I know I'm annoying)


I think I can actually work on Hop today, let me see what I can do

On Thu, Sep 24, 2015 at 1:30 PM, Oliver Beckstein <notifications@github.com>
wrote:

> There's weird stuff in this commit. Should be redone and then
> amended/rebased.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iwelland/hop/commit/ae4d5ca665904983f9ecb80447f9d827615215cd#commitcomment-13420813>
> .
>


Sweet!


That's a big win


Comments can relate to specific lines of code.


isn't this call redundant with the one on line 2322?


this is great!


same, declaration in the most-inner scope now it's not needed in the loop condition


Ok, I'll move it to  be consistent with what you already have for now, in an effort to avoid the add'l PR to have too much creep.


Seconded. This commit breaks programs using -Wextra -Werror.


So if no scope is specified, a new scope is created off of the root scope? Will this cause issues because the local context is lost, or will the consumer generally specify the scope?


don't we already have this value in variable `client_envs`?


Should we allow options to be null and default them to empty internally?


great!


so is it 20 or 21 parameters in ptv.par? according to:
https://github.com/OpenPTV/openptv/blob/master/liboptv/src/parameters.c#L212

it's 21 for num_cams = 4


FYI, circular references are strictly legal, but can cause some awkward resolution timing issues like you ran into, so they are often best avoided.


neat.  Though, I think `_node` and `next` should be renamed: `_node` is technically reserved, and `next` seem a bit too common.  Maybe using `q##_node` and `q##_next` would be better?


Can you elaborate? I don't think I fully follow the flow we're using to publish this package. I thought we published it to atmosphere (which requires a version bump)?


bin en fait si (je me suis cr�� un fichier tools.py dans lequel j�ai mis une copie de tes m�thodes sans le _) car si je me rappelle bien, je n�arrive pas � �crire dans un fichier directement avec la m�thode myKey.get_pub()� je fait donc un .write(str(myKey.get_pub().decode()))

Ce qui est peut �tre la raison qui explique mes probl�mes.

Quand tu as sauvegard� tes clefs, comment as-tu fait ?

On 24 Sep 2015, at 18:55, calve <notifications@github.com> wrote:

from monrsa.crypto import _serialize, _unserialize ?

Mais a priori tu ne les utilise pas par la suite :s

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


I added this because I need an internal DB client to modify the system span and set the commit trigger.


Check in the designs!


I see.  I renamed them since I changed how arguments are passed so either way I broke backwards compatibility.  I do think they should stay as `protected` however, so they can be used by inheritance.

What are you thoughts on that?


Although useful, we don't do this to avoid putting potentially sensitive data into logcat. Like:

Response code: XXX for really.embarrassing.or.secret.url.here

Note that the exception itself does contain the DataSpec, so on the application side it's possible to pull the uri back out and print it there, if desired.


No, this actually doesn't change the behavior. If no scope is specified, angular-ui defaults to the rootscope. (The dialog scope is a child of that scope, in that case)


Sure, that would be nice


r+


Page 1 is persistent. Only page 0 is not.


@thibaultCha well this is still an issue when the request doesn't have a body, the library should defend itself from this edge case.


Perfect.


Will this throw if the delete didn't succeed https://github.com/Particular/NServiceBus/blob/3b5158d30980783a10e3fb70f6fe970e034ac792/src/NServiceBus.Core/Persistence/Raven/TimeoutPersister/RavenTimeoutPersistence.cs#L155 ?


and because we have to put the header (containing the data length) before the data, but we can only know the data length by splicing it.


clearly, this code needed more comments :-)


Looks good, can you add tests for `Observable.throw`, `never` and `empty`?


Your function here needs typing (even for just a forEach function). You can use ```function (key: string): void {```.

This line is throwing a TSLint warning on the dev branch:
```(typedef) lib/Utils.ts[132, 52]: expected call-signature to have a typedef```


Makes sense


one tab is missing :P


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


merging jedmccaleb/stellar-core/master = b0900702 into auto


jedmccaleb/stellar-core/master = b0900702 merged ok, testing candidate = 027a32f4


> Hmmm, I can see it in landscape. Is that a version specific issue again?

I'm going to check with this new code, as "configChanges:orientation" might fix the issue with nested linearLayouts.


Can you explain what the code path is that can lead to codec ever being null here? Thanks.


factorcode.org is now running the latest code!

also restarting all of the masons soon.

:dolphin: :dolphin: :dolphin:


good idea allowing the caller to decide whether to count these calls


Good job!


On pacemaker_remote nodes, the tools like crm_mon and crm_verify still require the xml schema files. But pacemaker binary package that currently contains them doesn't have to be installed on remote nodes. So should we probably move the schema files into pacemkaer-libs,  otherwise let pacemaker-cli require pacemaker package?


Why are you switching to the non-default 8800 port?


Stripping smart quotes isn't quite right; need to put normal ascii quotes in instead.


Hmm...which files?  Is this livemedia-creator doing relabeling?


This is the first line


And actually pacemaker requires pacemaker-cli currently.


that’s not a change I was making (intentionally)

From: Venu Uppalapati <notifications@github.com<mailto:notifications@github.com>>
Reply-To: thomas-couchbase/showfast <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Thursday, September 24, 2015 at 10:47
To: thomas-couchbase/showfast <showfast@noreply.github.com<mailto:showfast@noreply.github.com>>
Cc: Thomas Anderson <thomas@couchbase.com<mailto:thomas@couchbase.com>>
Subject: Re: [showfast] Update config.json (d199574)

Why are you switching to the non-default 8800 port?

—
Reply to this email directly or view it on GitHub<https://github.com/thomas-couchbase/showfast/commit/d199574570d500d28611cc44adae00b4c27c669e#commitcomment-13421150>.


That was fixed in the 2nd commit on this PR


same here.


There is not an code to see right now, maybe fixd in the futher!


em-dash elimination here makes for 'usee.g.' which isn't right.


smart-quote elimination bad here.


new-feature-doc-fix (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (816 of 816 tests passed)

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


It took me a while to figure out what is going on :)

The disadvantage of my approach is that it requires the extra buffering pipe pair for all cases. I figured that since it is zero-copy that should not matter, but I must admit that I have not benchmarked it.


Not according to the doc I have under my eyes :)
But it might not be completely accurate for all devices: I agree that having a page 1 persistent makes more sense given @phomes discover regarding the profiles list.


Nous avons au MTQ un nouveau serveur https, le getInfo pass via le proxy alors avec une chaine user et mot de passe encrypté et la methode BASIC authentification on est en mesure d'avoir un resultats positif. Normalement le protocole https ne pass pas par le prox avec ce changement il sera permis.


you are hosting it on IIS do you still need to give or write the ServiceAddress?


I think you're mentioning the wrong @fantunes because I am not involved not I haven't forked/stared this.


new-feature-doc-fix (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Ich han d search methode implementiert gad under dere methode wo du demfall entfernt hesch. Ha denn recht viel merge conflicts gha und vermuetlich zviel wieder ineta. Gheisch das mit de tokens wieder use ;)


Forge should do this automatically, and afaik (= last I checked) Sponge keeps the link stuff in there. In most cases, custom chat plugins will destroy these information by simply converting the text with `Texts.toPlain()`, e.g. to replace something in the message.


Isn't that what it's doing? I ran the app and tested a bunch of times and it worked. What am I missing?

> On Sep 24, 2015, at 10:29 AM, Falko Buttler <notifications@github.com> wrote:
>
> This should be the exact opposite. If there is a space in the text field, we want to do a Google search, otherwise open the website.
>
> —
> Reply to this email directly or view it on GitHub.
>


Looks like you forgot this.


this might just be me, but much prefer a single if statement with an && to nested ifs, wherever possible


a little more commenting would be helpful


Wuut, did i merged my own pull request on the main branch oô ? Heuu sorry guy ! I'll discuss in the pull request thread !


Good catch, unintended white space change.


Petite faute d'orthographe


Thx :+1:


I like :)


r+


Not strictly, as the default is the equivalent of -m64, but no harm in putting in an explicit section for s390x. We didn't have an s390x section for 0.12.x or 0.10.x.


Should this say pluralize?


Can l use this before using minetest.get_node in unloaded chunks?


Sorry, I am stupid. You are right :-)


nonancel? nocancel.


Good catch, will remove.


good catch


Perhaps none. However, passing it in will allow us to make the [changes I suggested](https://github.com/mwatson2/encrypted-media/commit/0d33734864311d6c61ad0f1af3f2213fd0611e39#diff-f72607e47a6f74e53dc90eab8ee094e2R760). For now, I suggest adding a TODO to consider removing it, especially depending on possible changes related to that suggestion. We could even file an issue and assign it to me.

As for the note at line 625, the concern was about interpreting it as the consent is for exactly the accumulated configuration. I'm not sure what you were trying to say, but it may be been an effort to address [this comment](https://github.com/mwatson2/encrypted-media/commit/0d33734864311d6c61ad0f1af3f2213fd0611e39#diff-f72607e47a6f74e53dc90eab8ee094e2R621).
Perhaps:
>User consent to use Distinctive Identifiers is specific to the <var>origin</var> and may be limited to configurations sharing certain properties with <var>accumulated configuration</var>.


Should this read `Oj` instead of `JSON`?


think this comment is stale now?


Oops, nevermind:  I see `JSON` is valid as well.


> `getservent()` and `endservent()` are used in tcpdump. I've used these libpcap's functions in windump.

Were they actually reading the *Windows* services file, at `"%SystemRoot%\drivers\etc\services"`?

> So now tcpdump should add a `Win32/src/getservent.c` instead.

Yes, or a `missing/getservent.c`. I don't view it as WinPcap's job to provide replacements for UN\*X routines not in Windows if it doesn't use them itself.


Ploop tool has a built-in default for filesystem, and I want to use that default rather than add another one.


should rename to `GetDataForIOV` since this is now autoreleased instead of caller-free.


some tests failed:
exception: https://travis-ci.org/stellar/stellar-core/builds/82010829


Why bother with a fixed time here? This should evaluated to true fairly quickly, and if it doesn't the normal test timeout would catch the problem. Putting a 500ms timeout here seems like it will ensure a failure on CircleCI at some point.


there was some `data.concat` is not a function error when data is not an array (then it is an object `{}`). To clarify how this code works, I now test using `g_info.data_is_object` which indicates whether `some_data` should be an  object (for geom_line, polygon, ribbon, path) or an array (for anything else).


Sorry my bad it was a typo, port is always 4022! :p


OK, will fix.


For performance, you can avoid an extra copy of data by combining the copy and the null skipping.


Will fix once I get rid of system()


changed to 10s


This is incorrect behavior. It's correct that the entire sample should be written at the current position. This clips the end of the sample in the case that the current position is non-zero.


rax retry tests


I'm very unclear as to what crash this is actually fixing. Could you please elaborate? Do you have a stack trace? Thanks.


ditto for split_test.go


Oh cool!  Watching the first talk now :+1:


Hey thanks for the review!
Do you mind creating a pull request when you are done so I can start merging?


わかった
_,*w=は多重代入の構文の1つだった
a,*b=[1,2,3,4] みたいにすると a=1, b=[2,3,4] とかになるらしい
多重代入の左辺の*を使ったテクニックは使い方が広がるは


commit should read "changed context from $scope to this". I didn't realize $ variables are not read in the console commits


saw approval from graydon
at https://github.com/nullstyle/stellar-core/commit/cb4878c4ada0fb07fb0bff9617e955cf6f6cffa4


merging nullstyle/stellar-core/devportal-docs = cb4878c4 into auto


nullstyle/stellar-core/devportal-docs = cb4878c4 merged ok, testing candidate = a4c39951


Is there a reason you're using `sqlDB.Begin()` here while above you're performing a single `sqlDB.Exec` containing `BEGIN/COMMIT`?


Since this is now autoreleased, you should surround this entire function with an autorelease block, as

1. you don't know the call is done in an autorelease block (if not you get a error message coming on stdout, and you'll have fun looping).
2. if there is an autorelease block, you don't know when it will end and release the objects here.


shouldn't we also handle long, long long, float types directly?


Yes, it's better to fix twise than no fix at all :)


I don't understand why this is a 'bad behavior'. Throw an exception is the expected behavior when adding an action without exec function. Could you explain a bit?


Currently fails:

    $ npm test

      …

      370 passing (38s)
      6 failing

      1) oauth_proxy message integrity: headers should support POST requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

      2) oauth_proxy message integrity: headers should support PUT requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

      3) oauth_proxy message integrity: headers should support DELETE requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

      4) oauth_reverse_proxy message integrity: headers should support POST requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

      5) oauth_reverse_proxy message integrity: headers should support PUT requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

      6) oauth_reverse_proxy message integrity: headers should support DELETE requests with non-'utf-8' charsets declared in the header:

          Uncaught AssertionError: expected 404 to be 200
          + expected - actual

          -404
          +200

          at Assertion.fail (node_modules/should/lib/assertion.js:180:17)
          at Assertion.prop.value (node_modules/should/lib/assertion.js:65:17)
          at Request._callback (test/utils/validation_tools.js:55:58)
          at Request.self.callback (node_modules/request/request.js:198:22)
          at Request.<anonymous> (node_modules/request/request.js:1063:14)
          at IncomingMessage.<anonymous> (node_modules/request/request.js:1009:12)
          at _stream_readable.js:908:16

    npm ERR! Test failed.  See above for more details.


There is missing ui file for this formula


Very true, also the current code doesn't warn you when instancing non-implemented versions of the templated functions, leading to opaque linking errors. I'll fix that for the next version.


While this patch is merged, we still need to address the issue of discovering server through sub-nets. i.e if there are no arguments provided here. Controller should scan through the local subnet as necessary.

While it is not the final cut, it is still important that we implement it and see it working and see any caveats.


I suspected that the browser might be doing that work. The extra code doesn't hurt, I just wanted to make sure I had the right idea of what was going on since there wasn't really an explanation of that part. It's good to have an example in case you're doing something that the browser won't take care of on it's own.


I think I'm missing something here: what is the difference between `ResectionSquaredResidualError` defined here and the one in defined in sfm_robust_model_estimation?
Also, but maybe it's related to the previous question, can't we just change the implementation of `sfm::robustSection` so that, if `error_max` is passed as input it computes `dPrecision` as it is done here?


they are just redundant in the arithmetic mode..
got it from here: https://github.com/koalaman/shellcheck/wiki/SC2004


This should be put in a pull request immediately as projects with the RMStoreAppReceiptVerifier CocoaPod require an updated version of OpenSSL to compile successfully in Xcode 7. It looks like your change here fixes that exactly as I had hoped myself.


Caller should already guarantee that the input isn't freed, for the duration of this function call.

I'm not sure how passing in an `NSData` makes a difference.


Hi guys,
Thanks for this library, it's really useful.
Looks like this commit is breaking our services because `Content-Type` header shouldn't be set on GET requests.
Issue is here: https://github.com/request/request/issues/1786

Cheers
Chris.


This you are missing a "," here.


Hi Adam,
I still haven't approached Redmine 3.1, but I verified that your change is backward compatible. I will pull it into my branch.
Thank you


Sure, will do


Yeah, I agree. Is that how glmer does it when you group by something that
is not originally of class factor?

On Thu, Sep 24, 2015 at 12:50 PM, Jonah Gabry <notifications@github.com>
wrote:

> @bgoodri <https://github.com/bgoodri> Before this commit, a model like
>
> stan_lmer(mpg ~ wt + (1|cyl) + (1|gear))
>
> will include parameters
>
> b[(Intercept) 4] b[(Intercept) 6] b[(Intercept) 8] b[(Intercept) 3]
> b[(Intercept) 4] b[(Intercept) 5]
>
> without it being clear that the first three are for cyl and the second
> three for gear. In fact the b[(Intercept) 4] for gear won't even be
> included by the print method because the print method uses setdiff to
> remove 'log-posterior' and that will only retain one of the b[(Intercept)
> 4] parameters (in this case the one for cyl because it's first).
>
> With this commit these names are now
>
> b[(Intercept) cyl4] b[(Intercept) cyl6] b[(Intercept) cyl8] b[(Intercept)
> gear3] b[(Intercept) gear4] b[(Intercept) gear5]
>
> I'm not sure that this is the best way to format the names, but we should
> at least have the names of the variables somewhere.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/110112b5e663291170a5b3d52977af4c9e4ac1ae#commitcomment-13419900>
> .
>


This is exactly what was needed


I'll change it ;-)


I would say you use startwith() just in case we have things like "win64-debug" or "win64-pgo".


Thanks for that!


reshape_arrow_minimize should be reshape_arrow_stop


@mkonicek - thanks. I published 0.11.1 and rolled it back because Android was missing. I can publish 0.11.2 today.


@jurysch my change rolls back the latest change, not moving us to the most up to date version. Thoughts on that?


Actually, somewhere else in the doc they mention that page 1 is the directory and the page 0 is RAM (for the devices we are looking at). So I guess the 0/1==RAM must have been introduced first and then they decided to change it.


If this EKF only uses the primary IMU isn't this for loop unnecessary?


you forgot `final`s


sorry, here you go...


`getId()` as well.


@jurysch If you're interested in using my repo as a stopgap feel free, you can use this in your podfile.


Can you wrap this at 80 columns?


@drewfisher314 typically the information in the metadata.json for supported modules is only the supported platforms. Since there's no testing on Solaris12 we didn't want it in the metadata.


This file--`package.json`--is an **npm** package definition file. This file only exists to enable continuous integration via TravisCI. We'll never actually publish a `meteor-mturk` npm package.


likewise here.


r+


I think a FIWARE Lab token is needed to make this work, I will take a look into how to solve it, but uncomment this code


Near the top of the file, can you also add a link to https://www.iovisor.org/


Will this show up in the PR?


ต่อไปการเรียก resettoken() นอกจากจะเป็นการลบ token ออกจาก microgear แล้ว ยังจะส่งคำสั่งไปขอลบ token กับ netpie ด้วย


This looks like a mistake, filter here should probably be 'url'.


Here an example:

![image](https://cloud.githubusercontent.com/assets/7888201/10082182/c8bb3540-62f8-11e5-87b9-f410647197b1.png)

# Mandelbulber settings file
# version 2.05
# only modified parameters
[main_parameters]
camera 3,764376803015136 -7,135074826943302 2,28288807774876;
camera_distance_to_target 5,075139665603642;
camera_rotation 31,45468911618299 -14,78826288042063 0;
camera_top -0,1331944033283134 0,2177398401444148 0,9668756967345385;
DE_factor 0,3;
flight_last_to_render 0;
formula_1 5;
formula_2 56;
hybrid_fractal_enable true;
image_height 1000;
image_width 1000;
keyframe_last_to_render 0;
linear_DE_mode true;
target 1,203770643145705 -2,94911834536 0,9874703637611406;
[fractal_1]
power 17;
[fractal_2]
platonic_solid_amplitude 1,8;
platonic_solid_frequency 0,3927;
platonic_solid_rhoMul 0,8;
[fractal_3]
[fractal_4]


It's not strictly necessary.
This just ensures all problematic files are reported, instead of stopping on the first one.


Yeah sorry many of these testing things I left in purposefully, including the old code and comments, for the team to see and look over before creating a pull request. Tomorrow after we meet in person I'll fix it all up, comment everything, and commit again and then send a pull request.


In your .out file, I think you need a new line character at the end


seems like it should be -UDISABLE_NETWORK because doesn't -DDISABLE_NETWORK=OFF still define it?


What is the wrappee function for exactly? I don't see what this is useful for.


Good catch, didn't notice that. Yeah it was meant to be 'price'


Why are you setting an position to top top, updating on every scrolled pixel? I guess you only need to toggle a class at intended scroll position instead of blowing browser performance. Rest will do the css.
For example:
.affix-top {
  /* before element reached position */
  position: relative;
}
/* when element reached position */
.affix {
  position: fixed;
  top: @navbar-height;
  z-index: @zindex-navbar-fixed;
  width: 100%;
}


This is related to https://github.com/discourse/discourse/pull/3767#discussion_r39598368.

If we add the date into the key, the rate limit will reset every day instead of waiting for the count down to expire. :thought_balloon:


It should be `region` and `stack` instead of `region` and `instanceId`. Fixed in https://github.com/euforia/vindaloo/commit/0348f20758de36bdfa40269b9aed3130bf55a125


This is my first project on github
I wrote this in Qt


I'm not sure about this, but the condition vor Visual Studio 2015 (version number should be 14.0 I believe) is missing, right?


I expected this one.

Note this header is private, i.e. it is internal to ploop and doesn't have to do anything with the rest of the code, i.e. not meant to be included from some other code etc.

Let me give you an example -- kernel-wide includes are under top-level ./include/, but private include files are not, for example:
./init/do_mount.h
./mm/slab.h
(find . -path include -prune -o -name \*.h finds 18000+ such files in the kernel sources)

To avoid confusion, I will rename it to ploop-internal.h


There are too much spaces ;)


Nice


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


+1 on converter-driven binding of a type-safe Map - also for this as well as the multi-binder work do we want to introduce something like `spring.cloud.stream.defaults` ?


The answers for your last two test cases are flipped.


This works well but could be time-intensive if the number of users is huge.  Can you think of a good way of saving time (since this is supposed to be a prediction with an error margin anyway)?


OHHH, haha, sorry, I think you might've misunderstood. I meant that my suggestion was probably not the best idea. Hmm, I guess I might've worded my comment a bid bad. Read it like this

//This might not be a great idea, but you could try adding atest to see if you pipeline is empty.


TaskCluster, Authorization Failed ```{
    "info": "None of the scope-sets was satisfied",
    "scopesets": [
        [
            "queue:*",
            "docker-worker:*",
            "scheduler:*"
        ]
    ],
    "scopes": [
        "assume:worker-type:test/test",
        "assume:worker-id:test/test",
        "assume:scheduler-id:*",
        "scheduler:*",
        "queue:define-task:test/test",
        "queue:create-task:test/test",
        "queue:define-task:test/test",
        "queue:claim-task",
        "assume:worker-type:test/test",
        "assume:worker-id:test/test",
        "queue:report-task-completed",
        "queue:resolve-task",
        "queue:route:tc-treeherder-test.*",
        "queue:route:tc-treeherder-test*",
        "queue:rerun-task",
        "queue:route:taskcluster-github*",
        "docker-worker:cache:*",
        "docker-worker:feature:*"
    ]
}```


Have you tried the link with webpack yet?


No need I think, I will update as you suggested.


I'll need to check on that

On Thursday, September 24, 2015, bgoodri <notifications@github.com> wrote:

> Yeah, I agree. Is that how glmer does it when you group by something that
> is not originally of class factor?
>
> On Thu, Sep 24, 2015 at 12:50 PM, Jonah Gabry <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>
> wrote:
>
> > @bgoodri <https://github.com/bgoodri> Before this commit, a model like
> >
> > stan_lmer(mpg ~ wt + (1|cyl) + (1|gear))
> >
> > will include parameters
> >
> > b[(Intercept) 4] b[(Intercept) 6] b[(Intercept) 8] b[(Intercept) 3]
> > b[(Intercept) 4] b[(Intercept) 5]
> >
> > without it being clear that the first three are for cyl and the second
> > three for gear. In fact the b[(Intercept) 4] for gear won't even be
> > included by the print method because the print method uses setdiff to
> > remove 'log-posterior' and that will only retain one of the b[(Intercept)
> > 4] parameters (in this case the one for cyl because it's first).
> >
> > With this commit these names are now
> >
> > b[(Intercept) cyl4] b[(Intercept) cyl6] b[(Intercept) cyl8] b[(Intercept)
> > gear3] b[(Intercept) gear4] b[(Intercept) gear5]
> >
> > I'm not sure that this is the best way to format the names, but we should
> > at least have the names of the variables somewhere.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/stan-dev/rstanarm/commit/110112b5e663291170a5b3d52977af4c9e4ac1ae#commitcomment-13419900
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/stan-dev/rstanarm/commit/110112b5e663291170a5b3d52977af4c9e4ac1ae#commitcomment-13421654>
> .
>


No. But I'm pretty sure it will work.


Preemptive strike here. :grinning:


I think your .in file should have 1000-2000 lines


Good catch, @blakecallens.  My bad.


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


saw approval from jedmccaleb
at https://github.com/jedmccaleb/stellar-core/commit/45c5c5af6335d6655516962891ab3d385417809f


merging jedmccaleb/stellar-core/master = 45c5c5af into auto


jedmccaleb/stellar-core/master = 45c5c5af merged ok, testing candidate = 68ad2927


But I do know that the Z matrix that lme4 creates for this model with cyl
and gear only has the level numbers as column names without indicating the
variables (which is annoying), and that's probably why we weren't getting
the names without this fix. But if cyl and gear are made into factors
before calling stan_lmer then maybe my fix here will lead to duplicate
names. I'll double check and remedy it if need be.

On Thursday, September 24, 2015, Jonah Sol Gabry <jgabry@gmail.com> wrote:

> I'll need to check on that
>
> On Thursday, September 24, 2015, bgoodri <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
>
>> Yeah, I agree. Is that how glmer does it when you group by something that
>> is not originally of class factor?
>>
>> On Thu, Sep 24, 2015 at 12:50 PM, Jonah Gabry <notifications@github.com>
>> wrote:
>>
>> > @bgoodri <https://github.com/bgoodri> Before this commit, a model like
>> >
>> > stan_lmer(mpg ~ wt + (1|cyl) + (1|gear))
>> >
>> > will include parameters
>> >
>> > b[(Intercept) 4] b[(Intercept) 6] b[(Intercept) 8] b[(Intercept) 3]
>> > b[(Intercept) 4] b[(Intercept) 5]
>> >
>> > without it being clear that the first three are for cyl and the second
>> > three for gear. In fact the b[(Intercept) 4] for gear won't even be
>> > included by the print method because the print method uses setdiff to
>> > remove 'log-posterior' and that will only retain one of the
>> b[(Intercept)
>> > 4] parameters (in this case the one for cyl because it's first).
>> >
>> > With this commit these names are now
>> >
>> > b[(Intercept) cyl4] b[(Intercept) cyl6] b[(Intercept) cyl8]
>> b[(Intercept)
>> > gear3] b[(Intercept) gear4] b[(Intercept) gear5]
>> >
>> > I'm not sure that this is the best way to format the names, but we
>> should
>> > at least have the names of the variables somewhere.
>> >
>> > —
>> > Reply to this email directly or view it on GitHub
>> > <
>> https://github.com/stan-dev/rstanarm/commit/110112b5e663291170a5b3d52977af4c9e4ac1ae#commitcomment-13419900
>> >
>> > .
>> >
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/stan-dev/rstanarm/commit/110112b5e663291170a5b3d52977af4c9e4ac1ae#commitcomment-13421654>
>> .
>>
>


I'm sure I'm just misunderstanding some logic here, but why did showNextView() get changed to showPreviousView()


hmmm....So a log.Fatal that aborts the program **is** useful :-)


Looks like I broke the Travis build validity again in a previous commit: https://github.com/theopak/oauth_reverse_proxy/commit/538607c70720461b98848307c5498e96cd2e8e4a

    370 passing (39s)
      6 failing


Consistency with the cache disabled test. But yeah, collapsed into a single statement and added comment clarifying why I'm keeping the next one exploded per statement.


Can you add a legend for circle sizes?

On Thu, Sep 24, 2015 at 12:27 AM, realmarcin <notifications@github.com>
wrote:

> [image: screen shot 2015-09-23 at 11 47 44 pm]
> <https://cloud.githubusercontent.com/assets/4625870/10067634/0072e056-6253-11e5-9836-e3ab9d01ef56.png>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kbase/data_api/commit/4dbb9c2aa879e303451f6f75bdb825a3cebbbc11#commitcomment-13408302>
> .
>


Yeah, I think we're heading that way re: defaults - but that's a different refactoring story IMO.


agreed it's a different story - but possibly a similar approach to the other different story you mentioned (e.g. using a DefaultProperties type-safe Map)


You are welcome :)
Does  cli lack more elements, or  are we feature complete?


It's a cmake option, not compiler's
http://cmake.org/cmake/help/v3.0/command/option.html


Ignore me, completely misread the diff


On the G500, I get same error message (ERR_INVALID_PARAM_VALUE) reading too far (page 19 for the G500) before or after page 31. The limit varies with devices (G700s as 27 pages). Why not let hidpp10_request_command fail? The device would set its own limit.


Potresti aggiungere un `;` a termine riga.


Thanks dude! We just caught this bug a few minutes ago when I was going through our makefile!
That is really awesome of you to catch this too and reach out to us!

-Manny


Two blank lines here. Only need one


I see. Yeah.

Got any ideas? I'm not sure how to make it how to handle situations like that in general. If there were some way to specify the IP address of a given host in the inventory, but still be able to use the name on the command line... but it's not clear to me from [these docs](https://docs.ansible.com/ansible/developing_inventory.html) how one would do that.


Is this doing the same thing as before?  Is loadClass the same as forName with initialize=true?

It looks like before, loadClass would fail and super.resolveClass would call forName with initialize=false.  From the javadoc, it seems like this doesn't matter for Array types (the class is not initialized either way).


Are we on Java 6?


nome do commit 10/10


Another double blank line here. Needs to be one


What's this line supposed to do: attributeDivisors[ attribute ] !== meshPerAttribute ?
It gives warning on build. Supposed to be '='?


What's this line supposed to do?
It gives warning on build. Supposed to be '='?


oh ok


I'm not sure about that atom.setPosition test with negative values.

It seems there is another mechanism in Electron which is aligning new windows to display boundaries. So the effect of setPosition calls can be altered by this repositioning. On Mac, a window belongs to exactly one screen (display). It can be visually spanning multiple screens but it belongs to exactly one. It seems that setPosition call has some logic which takes that into account. Or it is a default behaviour of Cocoa. Anyways, this is an issue to discuss with Electron guys.

I would recommend pointing them to this section "Constraining a Window’s Size and Location"
https://developer.apple.com/library/mac/documentation/Cocoa/Conceptual/WinPanel/Tasks/SizingPlacingWindows.html

Particularly `constrainFrameRect:toScreen:` method should be overridden to prevent default screen edge snapping:
https://developer.apple.com/library/mac/documentation/Cocoa/Reference/ApplicationKit/Classes/NSWindow_Class/index.html#//apple_ref/occ/instm/NSWindow/constrainFrameRect:toScreen:


Gotcha


Зрозуміло.


I think filing an issue for that first point would be a good idea, since I confess I didn't fully understand that comment ;-)


Please rewrite this commit following a 72-character limit in the commit message. It is best if you use the pull request workflow as well, this code is used by several people, so it's important that any changes are vetted.


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


@jayaajay ^


I like the notion of having an explicit 'components' subdirectory within an activities package -- I always find myself struggling to decide where to put stuff that I want to refactor out of an activity file.


I know, because I tested Windows/Visual Studio before I pushed :wink:


@flekschas see here


If I change it to '=' though, then it gives the following error, since extension is null sometimes:
Uncaught TypeError: Cannot read property 'vertexAttribDivisorANGLE' of undefined


Maximum number of candidates is 20


"for exactly" Same as the other ones: easier to handle httpc context state.


@jemusser Please use the pull request functionality when changing the ibp server code in the future. It's used in production environments, so giving people the chance to vet changes is important.


Any reason why busList isn't private?


This can just be an `else` not `elif`


c'est totalement useless, on a un sommaire au dessus. La phrase "Allons à la découverte de ces nouveautés" était plus légère. On n'est pas dans une rédac du bac.


You need 50 choices per ballot


in general with `htmlwidgets` we recommend separating dependencies instead of one big bundle since very often different `htmlwidgets` require the same dependencies.  `htmlwidgets` has dependency conflict resolution to help avoid namespace problems and duplication.  While probably not that big an issue with `datacomb` since we probably expect full screen viewing, I would still advise isolating each of the dependencies as I did in the pull request.  I could foresee once we build in more `Shiny` integration `datacomb` paired with things like `parcoords`, etc. where this could quickly become problematic.


>Do you need any textures for it? If you want I can try to come up with some shiny metal texture such as the one you posted in the photo above.

That would be really cool!

Thanks thevirtualfer .


Ack.


Thanks for digging into some of the sordid details here.  There are various skeletons that lurk in the shadows of the verse.


There are better ways to do this.


find-links має пріоритет над PyPI, так?


But the pod dependency statement using the 1.0 would include the 1.0.204.1 version of OpenSSL, which appears to be what's needed here. Isn't that going forward? Not sure I follow.

Also admittedly I'm not sure of the syntax of using an optional CocoaPod (in this case, RMStore/AppReceiptVerificator) for a specific fork while also referencing the original repo RMStore 0.7 CocoaPod.  Any insight there would be appreciated.


Should be fixed in latest


We need to be careful to namespace all the `datacomb` stuff and avoid setting global `html`, `body` type css.


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


možná bych mohl s něčím pomoct :D


Try this:
```
# Mandelbulber settings file
# version 2.05
# only modified parameters
[main_parameters]
ambient_occlusion_enabled true;
bailout 91,2011;
camera 1,856437362928866 -2,422686912436439 -0,8627060682821982;
camera_distance_to_target 2,10150643595171;
camera_rotation 39,0095844309535 16,46744416755642 0,7529812462532607;
camera_top 0,1886267771663262 -0,2119770814769542 0,9588981467623962;
flight_last_to_render 0;
formula_2 56;
formula_iterations_1 2;
hybrid_fractal_enable true;
image_height 1000;
image_width 1000;
iteration_threshold_mode true;
keyframe_last_to_render 0;
linear_DE_mode true;
N 5;
surface_color_palette b09021 12130b aab05a 8d9339 c0c0c 534715 af4a36 7f7d54 5a6b59 2a320d ;
target 0,5879030737081283 -0,8567132145563205 -0,2669910063898834;
view_distance_max 15,89437755231822;
[fractal_1]
power 3;
[fractal_2]
platonic_solid_amplitude 1,8;
platonic_solid_frequency 0,6;
platonic_solid_rhoMul 0,73;
[fractal_3]
[fractal_4]
```

I found that this formula works better with MaxIter mode (DE calculation is very inaccurate)


Ok, I will do that! Do you already have it UV-mapped? And do you already have a texture file for me to work on or should I create a new one from scratch?


Is `. sidebar` a typo? Maybe it works but it looks weird.


I suggest putting parens around the import list, and then lining up the second line with the open-paren.


Наскільки я пам'ятаю, ми "плювалися" на такий find-links, тому і пропоную
його не додавати

On Thursday, 24 September 2015, selurvedu <notifications@github.com> wrote:

> find-links має пріоритет над PyPI, так?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/selurvedu/openprocurement.chronograph/commit/b80b2c2d9ca44a8b8a6d8cdedaca3289cb270ae2#commitcomment-13422283>
> .
>

--
....................................................................................................................................
Myroslav Opyr   ▪   CTO   ▪    Quintagroup   ▪   +1.917.475.4725   ▪
http://quintagroup.com
˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙


Given that the zfs project makes very regular releases, is there any reason to keep zfs-git around?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.607](http://teamcity.monogame.net/viewLog.html?buildId=10265&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:18:06


omg. what is this, and how can I support it?


Maybe this isn't super gross anymore? :)


`self.log.log(pexLog.Log.INFO,` --> `self.log.info(`


Thank you, JeDa


/cc @evliu


Uh, these use whitespace and not tabs.


> Were they actually reading the Windows services file, at "%SystemRoot%\drivers\etc\services"?

Sure. The command `windump -i1 udp` gives here:
```
18:22:58.634255 IP router.43730 > my-host.syslog: SYSLOG kernel.warning, length: 216
```
>  if it doesn't use them itself.

*it doesn't* meaning libpcap? If so, it would be similar to the stuff in `pcap/namedb.h`. I'd rather have the missing `getXY()`in libpcap (prototyped in `pcap-stdinc.h` or `pcap/namedb.h`. Implemented in ` pcap-namedb.c`) than in tcpdump.


A better test would be to have checks in the code that the outputs match your expectations rather than having a human inspect the output.


@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<module external.linked.project.id=":shadercam" external.linked.project.path="$MODULE_DIR$" external.root.project.path="$MODULE_DIR$/.." external.system.id="GRADLE" external.system.module.group="com.androidexperiments" external.system.module.version="1.0.4" type="JAVA_MODULE" version="4">
+<module external.linked.project.id=":shadercam" external.linked.project.path="$MODULE_DIR$" external.root.project.path="$MODULE_DIR$/.." external.system.id="GRADLE" external.system.module.group="com.androidexperiments" external.system.module.version="1.1.0" type="JAVA_MODULE" version="4">
   <component name="FacetManager">
     <facet type="android-gradle" name="Android-Gradle">
       <configuration>


Here as well.


FYI: See the angled tubing that goes from the square tube in towards the wheel rim (about 10-20 deg). I think I will edit that and have it connect to the vertical tubing right above the square tubing instead of going into the square tube. It looks like the real image is designed like that. I wish I had noticed it before I joined all for braces per side per ski.


## Notes
> 3) b.	Do not show the bus route code or the route name in the Edit & Create input areas or in the Index, Details & Delete property listings.  It’s still required to create and edit a route-stop record, so figure it out.

Since the project specifies that we can't have input fields for the bus route code, but we have it stored in our cookies (Ensure in your program that there is an existing cookie), I initialized the Create view with a
``` c#
return View(new routeStop() { busRouteCode = b.busRouteCode });
```
and put a hidden field on the view. That way the hidden field value auto-selects the current bus route code (with what we initialized and sent back to the view model).

## TempData

I wrapped the TempData visual display in a if statement so that it doesn't break the page weird when there is no TempData.

``` html
@if (TempData["message"] != null)
{
    <div class="well" style="margin-top: 15px">
        <h4 class="text-danger"><strong>@TempData["message"]</strong></h4>
    </div>
}
```
![Conditional Well Div](http://i.imgur.com/vAOnAu9.png)


And here too.


I think we can nix this parenthetical comment (doesn't really matter what it was).  But please refer to "PosixRegistry" rather than "filesystem registry", for consistency.


Would be good to return your best guess at this point and maybe an extra value indicating error bounds due to timing out.


Why console.error() instead of THREE.error() ?


Don't leave in commented-out code.


Ditto.


0.11.2 is up on npm and `react-native init` creates an Android project too. @mkonicek thanks for your help!


Two questions:
* Why does the minimum ROOT version need to be increased to 5.34?
* Should the parentheses around LD_LIBRARY_PATH not be curly braces?


OMGGGGGGGGGGG


For that, we need to define and describe a way of error reporting. Like what a create_tessera function is supposed to return. As I see from the current code, it's either 0 or -1.

Looking at it, I think we should add an API function like ``bool tessera_exists()``.


`if isinstance(self.registry, PosixRegistry):`


This code is from the provided example ConvServer, I didn't choose to omit an exit here.


thanks for spotting @big-guy . I'll change it to initialize=false.


Very cool!


Hmm, maybe I can do something like preparing a few strings like dir and ddxml value. The only problem is I don't see any private data for tessera what I could use for that. Where would you save this thin volume ID to?


This is challenging, we want to use the generic prefix (http://fedora.info/definitions/v4/repository#), but I'd prefer to lock it to a specific URL (e.g. http://fedora.info/definitions/v4/2015/07/24/repository). Any ideas?


changed in d6e3a9a2


@onox, the changeover logic and gui are pretty much complete and could be used as is if you want a pull request. I could use some help with a translate or rotate on the skies. I think they are designed to raise the front and lower the back , slightly (bungee or spring load on the front), when off the ground, then on landing the ground contact with the back edge rotates the front down to close to level or landing configuration, wheels drag or roll (surface dictates).


I know this is super nitpicky, but this should be a `TypeError` because it's a type mismatch.


womp :(


Inline these for readability. They will be very public as they are part of the SQL user interface and thus won't be changed.


@dmpayton this is an FUSD initiative to get middle school girls into programming. @vanderbeam is designing and building the site as a lady in tech!


Thanks, I was debating which was the better type.


Instead of hard-coding a list of supported registry types, can we put this functionality into the registry?


Yes, I have no idea how that slipped into the code. Thanks for catching that! :)


Most of options are done
I need to do keyframe animation rendering (needs similar changes flight animation, so should be easy)
There is needed some nice console output for progress of animation rendering (you can do this for flight animation which already works)
We need also to check if there are needed any additional error messages when syntax is wrong.

When CLI will be done, we will start doing rendering queue


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


This is fixed in a later commit. please ignore.


No problem, correct error reporting is important issue (which is still unsolved even in criu ;) )
The tessera_exists() call is OK too, but not in the first place.


Fine, then we don't need to do anything in ->open handler.


@cklosowski should be 'reversible'


Ack.


Suggest prepending an underscore to the method name, to indicate this isn't part of the public interface (which is defined by the parent class).


Yes it is uv-mapped to the floats.png (nothing that could be refactored any easier than remapping). I think a new texture would be needed and I'll just remap it.


r+


Right now I don't have any private data on tessera, indeed. Just because we didn't need one.
As far as thin IDs saving is concerned. I thought of it. It looks like we'll have to maintain some metadata file on, probably, the metadata tessera. And put user's one in a subdir.


Вообще левый =)
Видимо. первый вариант кода диммера. Прибью следующим коммитом.


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


Good catch! Do you want to submit a pull request?
Am 24.09.2015 09:34 schrieb "mikee805" <notifications@github.com>:

> The documentation
>
> https://github.com/EsotericSoftware/kryo
>
> needs to be updated to reflect this change:
>
> In many situations, you may want to have a strategy, where Kryo first
> tries to find and use a no-arg constructor and if it fails to do so, it
> should try to use StdInstantiatorStrategy as a fallback, because this one
> does not invoke any constructor at all. This is actually the default
> configuration and could be expressed like this:
>
> kryo.setInstantiatorStrategy(new DefaultInstantiatorStrategy(new
> StdInstantiatorStrategy()));
>
> Thanks,
> Mike
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EsotericSoftware/kryo/commit/bfc02befd7f479165cf86fc7c8b22b75c2ff35ca#commitcomment-13408401>
> .
>


Right, now that you mention it I think someone explained that to me already...

In that case I'm pretty sure "copy file; commit; delete unnecessary bits" will work better than "cut and paste into a new file".


I'd put the `(` on this line


А ты сможешь это пул-реквестом оформить?


create_ploop() did it wrong, thanks for catching. I'll add a macro to straighten things out.


In fact, is there a reason this shouldn't be an inline function in `lookup`?


Well, in this particular case, we can always update the vocabulary definition in [lib/rdf/vocab.rb](https://github.com/ruby-rdf/rdf-vocab/blob/develop/lib/rdf/vocab.rb#L49) to use that as the source. In general, there's an issue with different people needing different versions of vocabularies, which is a problem outside our control.

The entry might look like the following:

```
fcrepo4: {
    uri: "http://fedora.info/definitions/v4/repository#",
    class_name: "Fcrepo4",
    source: "http://fedora.info/definitions/v4/2015/07/24/repository"
},
```


saw approval from jedmccaleb
at https://github.com/jedmccaleb/stellar-core/commit/57b1b7164488745cee0d6a2febae267094b4c1f4


merging jedmccaleb/stellar-core/master = 57b1b716 into auto


jedmccaleb/stellar-core/master = 57b1b716 merged ok, testing candidate = 2a57f129


pooor Cameron...


Commit is for:
TUNEBI-45 dbimport keeps exiting on s3 errors during configurable shard testings.


Much better, thanks


soooo good.


Yeah, I always consult [the docs](https://docs.python.org/3/library/exceptions.html#TypeError), because they're pretty good.

> exception TypeError
> Raised when an operation or function is applied to an object of inappropriate type. The associated value is a string giving details about the type mismatch.
>
> exception ValueError
>      Raised when a built-in operation or function receives an argument that has the right type but an inappropriate value, and the situation is not described by a more precise exception such as IndexError.


Agreed, this needs to be merged in. Works great!


Uppercase `AS` (or remove)


Didn't see until after I merged that this file doesn't end in .php.


I haven't made any changes recently because I have been waiting for Alex to integrate my changes into the main codebase. I'm not sure what his plans are though. I think he has made a lot of changes recently so integrating my changes will be more difficult.

Your ideas sound great but I don't personally have a need for them. I'd suggest submitting a feature request to Alex on the main repository.


Anyway better if we cache brushes, dispose them every frame is quite expensive


@mkristian This tested out okay for me, but please review when you have a moment.


For ploop, default fs is "built in", and it's used unless we explicitly specify one (or "none" for not creating a file system).

I can't really check if given FS is valid without running "ploop init" which is done in create.


This indentation seems off


For both the insert tests, insert several times and very rows after each insert


I suggest checking for inequality rather than using `is not` (http://stackoverflow.com/questions/1504717/why-does-comparing-strings-in-python-using-either-or-is-sometimes-produce).


So I guess we need to make ``.mount = bind_mosaic_subdir_loc`` a default so if a driver doesn't care it should be implicit.


The 2 large methods in this file could probably still benefit from being split into a few more, smaller, well-named functions if you felt like it. Whether or not it's worth the effort I guess just depends on how much we anticipate adjusting them in the future, but would be awesome from a readability standpoint. I know this was already a lot of work though, so up to you if that's best left for pass #2.


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


rax retry tests


Feel free to create a pull request. There doesn't seem to be anything wrong with it and it does appear more organized.


OK,

i will figure something out for progress in cli.


Fine, we can say that "no fs means ploop def fs"


I'll use `assertEqualsToFile`.


What does this comment mean?


should we re.escape on the plugin slug?


done


Does this do a new scan every time I want to `butler.get`?  If so, can we cache the results for each template?


new-feature-doc-fix (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


new-feature-doc-fix (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1354 of 1354 tests passed)

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


Why isn't this using the version without a message? This error will have an empty message


I'll add it.


Thanks, fixed


So the aficionado is increasing the humility factor?? Honesty factor seems optimum but the entertainment score of the site has now dropped by a huge margin.
Good for the guy, I guess ;)


The ploop .mount is already such :)


Когда добавляю к объекту float=right, то его высота уменьшается. К сожалению


Unwrap all these


"Cannot"


Так там и стилей почти не осталось
.nooui-widget {
	padding-bottom: 5px;
}
.nooui-btngroup {
	border-spacing: 3px;
}
.nooui-button {
	cursor: pointer;
}

Первый раздвигает виджеты по вертикали, пока это не решено снаружи, второй (поля вокруг кнопок) вроде в bootstrap решения не имеет, третий можно заменить классом th-ponter, он остался в наследство от старого варианта.

Что здесь прибить (кроме третьего)?


No need to wrap


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


Why this isn't 'else if'?


Ну, высота не проблема, можно её указать явно.


Changing the minimum ROOT was a (failed) early experiment to debug the build.  I think it is not necessary.  I forgot until after I committed that we had not put it back to 5.28.  As I understand it, the curly braces will use the local value of the variable, whereas on constance we need the value to inherit LD_LIBRARY_PATH from the environment (round parentheses).  We found that without that we had strange version shear where not all parts of the build were using gcc 4.8.2 consistently (this is related to the libmpc issue that is peculiar only to PIC apparently).  I assume there is a more elegant solution to this, but this was the best I could come up with.


This was bugging out some things... will fix later


delete `>>>`


`allPaths.items` --> `allPaths.iteritems`


This seems to be unused


define `sent`


![screen shot 2015-09-24 at 2 03 23 pm](https://cloud.githubusercontent.com/assets/7316321/10083730/6303c916-62c5-11e5-9667-203455165eef.png)

mmmhmm...


What's the purpose of the indeterminant progress bar?


I'm not sure this is the right error. Probably `NOT_SUPPORTED` is better


Why you deleted _RustLexer?


Slash am going to be the asshole who corrects your spelling :) (indeterminate)


Added Honeeepi to the Honeypot sensor section


I think it makes sense to move the schema files to pacemaker-libs, or possibly a new (arch-independent) pacemaker-data package.

Somewhat related, it might be a good idea to put the ocf:pacemaker agents in a new pacemaker-agents package, as currently any pacemaker_remote node that needs to run one of them needs to pull in all of pacemaker.


Deleted data files from tests


On 09/24/2015 06:14 PM, Hartmut Kaiser wrote:
> Hmmm, shouldn't we leave |id_type| in place for at least one or two
> releases before removing it?

Yes we should.
https://github.com/STEllAR-GROUP/hpx/blob/remove_id_type/hpx/runtime/naming/locality.hpp#L72
should take care of that.

>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/STEllAR-GROUP/hpx/commit/aaf5423395f855b6d9f08dc7c8a50e525594ab66#commitcomment-13419092>.
>


`retItems.append((pathDataId[prop] for prop in lookupProps if prop in pathDataId))`

Actually, I would be tempted to write both `for` loops as a single list comprehension.... but that's prolly going a bit far.  (:


@kevinct That part is correct and works like retail then, at least for WoTLK and later. The moment a creature dies, its loot policy is set in stone, and changing the group's loot method does no longer change anything for that particular corpse, only for future corpses.


On 09/24/2015 06:19 PM, Hartmut Kaiser wrote:
> This should be:
>
> |hpx::async(action_type(), locality, std::forward<Ts>(vs)...); |
>
> You can think about this as invoking a member of the locality.

You are right, that would be the correct syntax. The reason why it's the
other way around is due to locality being an executor ... So i guess
this design decision is not solid enough.

>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/STEllAR-GROUP/hpx/commit/aaf5423395f855b6d9f08dc7c8a50e525594ab66#commitcomment-13419211>.
>


LGTM.


... К сожалению, не могу сейчас показать, как это выглядит, и не смогу в ближайшее время. Поэтому посмотри, пожалуйста, сам, как это выглядит. У меня, если что, браузер Google Chrome.

Если такое изменение высоты не является проблемой, то я могу открыть пулл-реквест. Собственно, поэтому и открыл Issue - я не силён в HTML и, возможно, там можно что-то добавить, чтобы сохранить высоту.

Есть два варианта:

1. Добавь float=right и явную высоту от своего имени.
2. Скажи мне, как добавить высоту, я внесу её в свой форк и открою пулл-реквест.

Извините за разорванное сообщение, пишу со смартфона.


Please restore the newline at the end of the file.


There should be a simple legend for what the circle means. I think you mean a scale with absolute sizes? That will be tricky ...

> On Sep 24, 2015, at 11:26 AM, Dan Gunter <notifications@github.com> wrote:
>
> Can you add a legend for circle sizes?
>
> On Thu, Sep 24, 2015 at 12:27 AM, realmarcin <notifications@github.com>
> wrote:
>
> > [image: screen shot 2015-09-23 at 11 47 44 pm]
> > <https://cloud.githubusercontent.com/assets/4625870/10067634/0072e056-6253-11e5-9836-e3ab9d01ef56.png>
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/kbase/data_api/commit/4dbb9c2aa879e303451f6f75bdb825a3cebbbc11#commitcomment-13408302>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Ah ok never mind on the question, read the ProgressBar documentation


Magnifficent!


new-feature-doc-fix (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


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


Ладно, я на выходных попробую выделить время и всё это проверить и закоммитить.


Change this `import` to an absolute (`import lsst.daf.persistence.fsScanner`) or relative (`from .fsScanner import FsScanner`).


> I thought a currency should be displayed the way it is formatted in its origin locale.

Well what would be the origin locale for euro? :P

> Is that "allowed" in the ICU formatting rules?

I don't know but I don't think any user would expect a mix of formatting rules on the same page. For example on a live exchange rates website:
* an American would expect to read `$1,234.56 = €1,100.41`
* a German would expect to read `1.234,56 $ = 1.100,41 €`
* and both would be confused with `$1,234.56 = 1.100,41 €`

The English Wikipedia article about [euro](https://en.wikipedia.org/wiki/Euro) states "more than €995,000,000,000 in circulation". So: the euro symbol displayed in American format.


@udalov
Please take a look at the last three commits: https://github.com/JetBrains/kotlin/compare/rr/fix_kt9178


nice but just for the record, im already annoyed. vcsh shouldn't allow itself to be called from within a subshell spawned by vcsh. because that's just annoying, even if it could be useful pratically IF IT'S 1970 AND YOUR TERMINAL EMULATOR SUCKS. use tmux. it's like use bcrypt, but for 2015.


1. Лучше было бы сделать веб-приложение или что-нибудь под мобильные платформы.
2. Хорошо было бы определить границы проекта, так как в функциональных требованиях все крайне размыто. Например, подсчет калорий кажется излишним, потому что товары могут быть непродовольственными, продукты могут испортится, покупаться для кого-то другого или что-нибудь в этом духе.
3. Так как приложение работает с деньгами и делает статистику, хорошо было бы добавить к нефункциональным требованиям точность вычислений.
4. Из текстового описания не совсем понятно, каким образом будет составляться список покупок.


Que miedo


Спасибо.


@AlfioEmanueleFresta Fatto, hai ragione. Grazie.


A couple of thoughts come to mind here.

At first I was going to suggest wrapping this in a function like we do with all the javascript for the [college parent theme](https://github.ncsu.edu/ncstate-ucomm/college-parent/blob/master/functions.php#L22-L42). Then I realized nothing in this theme seems to be using the wp_enqueue_scripts [hook](https://codex.wordpress.org/Function_Reference/wp_enqueue_script#Using_a_Hook). So what we should really do is make sure all javascript and styles are registered and enqueued inside of a hooked function (again, like in college parent).

We should also concatenate the mobile nav js onto the main js for this theme to reduce http requests. In order to do that, we should incorporate gulp. Jonathan or I can help with that if needed, but we have a [good template](https://github.ncsu.edu/ncstate-ucomm/college-parent/blob/master/gulpfile.js) to work from in college parent.


You mean by creating a custom css class? I'm creating css classes in a second moment, that was just a quick trial because I'm not sure we'll keep that formatting. Were you thinking of something else?

On 24/09/2015 19:39, Andrew Hankinson wrote:
> There are better ways to do this.
>
> —
> Reply to this email directly or view it on GitHub <https://github.com/bassculture/BassCulture/commit/79f1442e00317180c89ed2ba157cfc4a69fd4b92#commitcomment-13422282>.
>


Very nice.


Thanks!


You could use https://github.com/sindresorhus/is-video instead of the whole `path.extname(file).slice(1).toLowerCase() in extns` and `videoExtensions.forEach(function (el) {`.


Sure I can do a PR on the docs


This probably ought to go somewhere, I thought right under the package declaration was a good spot. Is there a reason you removed it?

Feel free to add your name to it, of course.


Collateral damage. I wonder how I was able to run the plugin after this...


> *it doesn't* meaning libpcap?

Yes.

> I'd rather have the missing `getXY()` in libpcap ... than in tcpdump.

Why?  I see \*pcap as a cross-platform library for packet capture, saving captures to files, and reading those files; it's not a general "fill in the stuff Windows doesn't have that UN\*X does" library. Not every program that uses \*pcap needs `getservent()`.

And that raises the issue of whether *tcpdump* needs it.  That depends on whether `getservbyport()` is linear (or worse) in the number of known service <-> port mappings or faster, and whether enumerating all the entries in the map is expensive.

At the time tcpdump was written, `getservbyport()` linearly scanned the `/etc/services` file, so `getservbyport()` was linear, and enumerating all the entries in the map was not much more expensive than looking up the last entry in the file, so it made sense.

However, if, for example, the map is accessed by NIS a lookup is probably sub-linear (NIS servers tend to store the entries in dbm or other fast maps), and enumeration could be expensive (an NIS "dump this map" operation or iteration through the list over the wire).

The same applies if the map has a local index file, as I think it might on some BSDs; in that case, lookups are probably sub-linear.

At least at one point, on OS X, tcpdump took a significant time to start up because `getservent()` went through the Directory Services daemon, so reading each entry in the file took at least two Mach messages (request and reply) and context switch (to the daemon and back to tcpdump), so that loop was slow.  I seem to remember seeing that `getservent()` changed to read the services file directly and only talk to the Directory Services daemon when it got to the end of the file, or something such as that.


Спасибо, учту пожелания


+1: "These tips are good practice to help you meet Web Content Accessibility Guidelines (WCAG) requirements."


what's the reason for the change here?


good idea, yes, noise is quite an issue with this formula.


CA1305 - updated that test code starts in new line
CA1309 - removed disabling CS0219


Hopefully you can just reference it directly from Bower since it should already be registered: `"vui-icons": "~0.7.0"`


same here,  why the date change?


Does it make sense to use `to_owned` here as well as `to_string`, just to introduce an extremely common idiom and performance improvement?


So this "clean up" nuked the instructions that just had it use the environment.yml file, and used out of date requirements for the ipython notebook.

We need to be more careful when "cleaning up".


@sungeunchoi  looks pretty good.  had one question about the  time stamp above but otherwise good.


Hi Esk0r,

Since you don't have forum threads for your assembly, I was wondering if you'd be willing to add a Lane Clear and Jungle Clear function to this assembly with the typical logic (toggles for each skill, min. % mana slider, etc.).

Thank you.


Curious , right? :)

The "https:" is a label (for a possible go to) and the rest is just a comment. Was a valid expression after all.


How about AUTHORS file at the root like https://github.com/golang/go/blob/master/AUTHORS ?


Should it be on runtime?


commmment


excellent!


Good catch!


I think I missed it while writing templete, thanks for noticing !


I still think that retrying is like a sweet sugar and it shouldn't be part of this service class, it gives it more responsability that just providing function for each phase of a chunk upload.
Having that sweet sugar provided by the `Facebook` master class makes sense, it's an easy way to do for any standard user.
I think it's up to the advanced user that directly use this service to handle retrying an upload.


I'd maybe break out the function that maps a URL -> icon type into a separate file. That way we can write unit tests around it in isolation.


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 709](https://teamcity.labkey.org/viewLog.html?buildId=274069&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 34, passed: 9 Build time: 00:25:24
### Failed tests
```
org.labkey.test.tests.CDSVisualizationTest.verifyAntigenScatterPlot: <no details avaliable>

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

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

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

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

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

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

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

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

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

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

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


The backport have the advantage to simply drop it if we deprecate Python 2.6.


Why is this always being written to the outState as true? From my understanding this is supposed to be the value of hasFormLoadBeenTriggered?


No,what's happened?Where my mistake?


Some useful info here: https://github.com/rollbar/rollbar.js/commit/1cc899b1c184743b6c4b1d1917271c63733dbf1f


whoww.. thanks.. now I will read this thoroughly and try to do so in core :-)


dvbhdhomerun also need similar patch.


Until we figure out how to package the CSS, could you generate this file into `dist` as part of the `build-sample` task?


I would make this part of the `GenericViewer`


Shouldn't we test on nightly (currently 0.5-dev) in addition to 0.4? Or is this just for testing?


Looks good locally. did you want one for northeastern:drs:staff?
<img width="637" alt="screen shot 2015-09-24 at 3 29 15 pm" src="https://cloud.githubusercontent.com/assets/5439169/10084373/2238ae04-62d1-11e5-9abe-9f55fdd5d6b5.png">


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


Fixed for real in https://github.com/theopak/oauth_reverse_proxy/commit/a17271db4d00aac95e01d8575fad568fea361850

The issue is that the Travis worker uses a shell that doesn't support env var array syntax.


The bug is already fixed, new `3.5.5` packages are out (only Debian and RPM packages were affected).


Yep, but the doctest was failing in a way which I couldn't see an easy solution for. The travis test just said failure and showed me that the expected output was `'Python 2.6.2'` and the actual output was `''`.


[> Make unparenthesized exponentiation expression as operand of unary operators an **early** error.](https://esdiscuss.org/topic/exponentiation-operator-precedence#content-60)


Is this not supported in the backend or in the SDK?


Cool!


@michaelklishin yes, we are already reverting it: https://review.openstack.org/#/c/227495/

It blocked our CI, we had to find a workaround.


lol


I would have the if here. Just as a persona preference. As long as the variable is not used elsewhere.


nice :+1:


i know... but unnecessary is not "wrong" ;) hrhr

actually the example given in SC2004 lacks readability in my opinion...
a="1+1"
$(( a * 5)) is (1+1)*5...?
this might not be obvious in any way no matter if $a or just a is used..
if it expands to a string first it should still be $((1+1*5))
this is somehow weird shell behavior anyway..
the fix to write clear code should be a="(1+1)" and not $((a*5)) in my opinion ... ;)


Isn't the deleted line correctly handling that flag string?


Are the fonts the same now everywhere in the game?


Sounds nicer than before.


Ahh, ok - makes sense. However I'm not sure if a `locality` should be an executor type (I admit however the idea is compelling).

We already have the `targeting_distribution_policy` to turn an `id_type` into a distribution policy and the `distribution_policy_executor` to turn any distributon policy into an executor. So I think the code should do something like:

    locality l = ...;
    targeting_distribution_policy tdp(l);
    async(make_distribution_policy_executor(tdp), action_type(), ...);

That would also simplify the implementation of the locality type itself.


TeamCity LabkeyModules15.2 :: CDS postgres 9.4 [Build 21](https://teamcity.labkey.org/viewLog.html?buildId=274067&buildTypeId=LabkeyModules152_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 4 (1 new), passed: 21 Build time: 00:33:54
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

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

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

```


That'd end up being more duplication though (e.g. if you changed another of the parameters, or decided you want to use another keyword arg, you'd have to change it in two places instead of one)


This whole thing can be simplified to:

```js
return files.filter(function (file) {
  return fs.statSync(file).isFile() && path.extname(file).slice(1).toLowerCase() in extns;
});
```


no  <!-- Slim Shortcut Actions -->  for CreateShortcut.java


Great!


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


Completely agree on the enqueuing changes - I'll start working on modeling the function after college-parent's to get everything loaded the same way, and I'll also get gulp working with this :)


But it's git, right? nothing is ever gone..... ;-)


or just write `b=10` ;)


:) :+1:


adamsonich?


I was unable to translate encounter_method_id 17 since even the english translation does not make any sense to me.
Could this be something like "Walking in long grass"?


Ok.

Do we have to make both conversions implicit, though (`id_type` --> `locality` and `locality` --> `id_type`)? I think extracting an `id_type` from a `locality` should be made explicit.


When compiled and run, no?


output images are completely grey, any clue?

I tried with:
./mandelbulber2 -n --res 328x328 -s 4 -e 25 -F /usr/share/mandelbulber2/examples/flight_anim_menger\ sponge_3.fract -o /tmp/


Right, the managed groups. Yeah, there should be values for staff, faculty, and all, too.


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


One of my push button issues is are things breaking (or going backwards) when doing cosmetic adjustments. Along the lines of PEP8-ing a file.


My comment was more directed towards, why have another layer of indirection (deriving from an external class in another library) when the base language offers the same feature with similar expressivity.


This is super cool.

@vanderbeam I'm a fan of the Margaret Hamilton pic on the homepage. :+1:


@xrwang the gitignore files is there to ignore other files.  It can't ignore itself


so if i use  https://github.com/sindresorhus/is-video then it will get simplified to
```js
return files.filter(function (file) {
  return fs.statSync(file).isFile() && isVideo(file);
});
```
right?


Unfinished job. Chill and wait a bit ;-)


my page now goes to this: https://ktvocalstudio.com/paywall


please use x.y.z formatting for the version. This is how Kodi stores versions in core for fast comparison, so anything else might have unexpected behavior (e.g. maybe kodi looks for a zip named `plugin.program.iarl-021115.0.0.zip`)

Also, I recommend using year-month-day order for dates (see ISO 8601). Most of the people on the Kodi project are european, so when the year is last they expect to see day-month-year.

Also, putting the year last totally breaks sorting. Kodi will think December 2015 is after January 2016.


Render a little more frames, then you will see :)


This sounds like a pretty complicated interface, mixing a few modes of operation depending on whether some parameters are null. Is "dumpExitCode" referring to the exit code of the OCaml, and if so that sounds like a very confusing way of controlling the output from running the ocaml program.


r+


Yes without question


The `Icon` should likely go into the `GenericViewer` component, so that this always just returns `<GenericViewer />`


saw approval from jedmccaleb
at https://github.com/jedmccaleb/stellar-core/commit/1bca795df7f2eadbd3f1c70abd61f54836b4501e


merging jedmccaleb/stellar-core/master = 1bca795d into auto


Here's what I've done with the competence scores, per @twiecki 's advice. Is this along the lines of what you had in mind, Thomas?


jedmccaleb/stellar-core/master = 1bca795d merged ok, testing candidate = 32730292


Don't forget to tag the 2.5 release as well. Great work on this plugin by the way. It keeps getting better.


Maybe we can integrate this in our legacy parsing. to improve the usability of the Text API a little bit.
Especially plugins that provide schedulled and configurable text messages will benefit from this.


Ty @zlotap!


`<link>` tags need to go in the `<head>` element.


Should have a period


`id_type` to `locality` should be explicit, as not every id represents a locality.
`locality` to `id_type` can be implicit, every locality should be represented by an id.


Yup


@bdbergeron We build BDBOAuth1Manager & AFNetworking into an OS X framework, and this change breaks our build. Is this change for iOS only?


Seems to me `Win32/Include/net/if.h` isn't used too.


The final `:K` isn't necessary any more after #1710, it should be changed back.


Yes, that looks great!


Well, you could achieve the same by symlinking `~avocado/data/avocado-vt/isos` to your NFS dir. I don't see why I should accept adding another attribute to accomodate this very specific way of using the NFS dir.


oh no :D


I see why this is necessary, but it's unfortunate we have to move the syntax array from the rules.


You are illiterate man, word like uneeded doesn't exist ...


yeah that's right.. it still does the calculation the right way :+1:


Nope. They're defined to llu/lld respectively.


I think I did it correctly this time.
https://github.com/MacGyverr/Metropolis/tree/new-refactor


Thank you for the bugfix! Can you please correct the ArrayList size? I'll merge the fix afterwards.


This is now skipping 2-6, not just 6.


@Salaboy I commented out this include introduced by your last commit. This file doesn't seem to exist and the include breaks the build. Can you either remove or fix the include?


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.608](http://teamcity.monogame.net/viewLog.html?buildId=10269&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 539, ignored: 6 Build time: 00:19:07


Feel free.


Correction...I will not fix this since checking for number being 1 is a base case for my recursive sequence.  This is because of the code higher in this function that says:

    number /= 2;

If `number` is 2, then after executing the above code, `number` becomes 1.

Also, my `IsPrime` method already handles the 1 case by default since the loop counter starts at 2.


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 23](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=196&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) is now running


Yes! That's a very good illustration, linking "degenerate edges" words to it would be nice.


Yeah, it's branched off of upstream, so that'll work for now.  In the future I recommend you don't pollute your master though.  But it's up to you.. I can work around these sorts of things anyway :)


TeamCity Xbim :: Xbim Windows UI :: Xbim Windows UI [Build 23](http://xbimbuilds.cloudapp.net/viewLog.html?buildId=196&buildTypeId=Xbim_XbimWindowsUi_XbimWindowsUi) outcome was **SUCCESS**
Summary: Running Build time: 00:00:39


:thumbsup:


You can use AppContext.BaseDirectory for this.


replaced the oil with olive oil


This is shit.


74564912Issue4347BBRadConvHW (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1889 of 1892 tests passed)

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


Last commit is not merged to develop. How is it going to break publishing?


Add PSR2 Coding Standard.


The only real change seems to be this. The rest are just style changes, correct? Please provide a separate style change commit if you think you need one.t


You can use the "yield" keyword right here to produce an IEnumerable. Try that variation, and then remove the 100 upper-bound.


Same comment as before: internals should be renamed similarly to `SocketAddress.Buffer` in common instead of using `._`


See https://github.com/twitter/scalding/blob/develop/version.sbt


I think, fixing enable_log_facility to return SUCCESS if already enabled (makes sense based on set_default_log_facility  return values) is the right patch.


Mostly cleanup: I found it inconvenient to need to provide all three properties when changing one property, and I couldn't think of a strong reason to keep a composite update method. If you need it, I can re-add it though; let me know.


74564912Issue4347BBRadConvHW (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


102966118-HeatRecoveryBypass-4568 (Myoldmopar) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Right, but are we not tracking in a release branch thats supposed to be setup for the next rls -- https://github.com/twitter/scalding/tree/releases/0.15 see the other changes for the other rc's are there


muchas gracias, @alexose!


![ski0](https://cloud.githubusercontent.com/assets/11069680/10063778/ff681738-6236-11e5-84ea-a16435473e9b.jpg)
I think this is what I want to convert the skies to. I really like the simplicity and you can take off from pavement or dirt. So time to refactor already!


what a whore....a picky whiney whore


That's your boy right there.


why is the test lost in this otherwise mechanical change?


Any way we could cache this instead?


it doesn't function as is - it can't start a machine with empty AWSOptions{}. we 'test' NewAWSCluster & co quite thoroughly through the kola tests anyway, which can provide the ami id, etc


requirement gemini version 0.17.1


correct, 15.04 is not supported


Archivo revisado


Ok, I will revert the last commit. I thought we were doing release from develop but that's not the case it seems


There's an open issue (#17) for supporting asymmetric delimiters (e.g. opening with `{` and closing with `}`). In preparation for that, I think it would make sense if we changed this interface to this instead:

```
String getOpeningDelimiter();
String getClosingDelimiter();
```

What do you think? In our cases, both would be e.g. `"*"` or `"**"`, but would allow for more flexibility.


in theory yes because the only thing that calls this is create (which is a long-running Python process), but in practice it won't work very well as-is. There are something like 8 different worker processes which won't share an in-memory cache. We can cache it to disk (yuck) or reduce the number of workers maybe.

You'd still need to query LDAP though since the cache may not reflect the true state of LDAP for any number of (unlikely) reasons.


最好不要这样。
#include .c文件的，要么是新手；要么是怪才。
会破坏以后的设计模式


This time I've fixed it as intended.

https://github.com/dankogai/p5-encode/commit/923e57be845754a8471670503e3920b901f183dd


legal


teste


I've been getting an error on the redis. Is that my any chance why you removed it?


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


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1355 of 1355 tests passed)

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


這一步跳太多了，會顯得不太知道為什麼需要在 Dictionary 與 List 兩種資料結構中間切換。是不是 API 可以直接開成 List<Order> 或是 prod code 可以直接用 Dictionary 來做呢？


i got problem on build error

device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp: In function 'int mapBatteryStatusString(const char*)':
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:27:12: error: 'HEALTHD_MAP_CONTINUE_SEARCH' was not declared in this scope
     return HEALTHD_MAP_CONTINUE_SEARCH;
            ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp: In function 'int mapChargeTypeString(const char*)':
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:34:16: error: 'BATTERY_CHARGE_TYPE_FAST_CHARGING' is not a member of 'android'
         return android::BATTERY_CHARGE_TYPE_FAST_CHARGING;
                ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:36:16: error: 'BATTERY_CHARGE_TYPE_UNKNOWN' is not a member of 'android'
         return android::BATTERY_CHARGE_TYPE_UNKNOWN;
                ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp: In function 'void healthd_board_init(healthd_config*)':
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:41:13: error: 'struct healthd_config' has no member named 'batteryChargeTypePath'
     config->batteryChargeTypePath  = "/sys/class/power_supply/battery/status";
             ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:42:13: error: 'struct healthd_config' has no member named 'mapBatteryStatusString'
     config->mapBatteryStatusString = mapBatteryStatusString;
             ^
device/asus/mofd-common/libhealthd/healthd_board_mofd_v1.cpp:43:13: error: 'struct healthd_config' has no member named 'mapChargeTypeString'
     config->mapChargeTypeString    = mapChargeTypeString;


Will change this later when i have understood build-problem


I don't understand the name of this function, GetEpoch should return the epoch, not the age in days since the CoreOS epoch.  GetCoreOSAge() ?


docstring?


Slider


#4402-VRF-outputs-and-multiplied-floor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1356 of 1356 tests passed)

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


is this right?


因為在第二個測試案例就把全部需求的 prod code 寫完了，所以這邊測試案例加上去，反而沒有紅燈。

這會阻礙TDD的開發節奏，而且會使得通過第二個測試案例的 prod code 花很多時間跟心力。如果，沒有第六跟第七個需求，那多寫的 prod code 就是 over deisgn。


docstring


docstrings throughout...


写log这个功能，考虑向上层提供可开关的选项。因为路由器的ROM，一般是没有可写权限的。想写也只能写到插进去的U盘等设备里（或者我们今后自己的硬件compute stick有自身存储，但无论Flash Nand, SSD还是eMMC，寿命都有限，这样高频率写日志，估计三四个月就坏掉了）


If roll is noisy, then we could use demanded roll instead.
Some airframes fly at a trim angle greater than this. i don't think integrating during climbs or descents would be a problem.


This comment doesn't make sense, especially when ew have a cancelled state below and it's not mentioned here.


can't these all just be in a single case?