There is no exit path between `hw_descriptors = g_hash_table_new_full ()` and `g_hash_table_destroy (hw_descriptors)`, so I don't think it is leaking. Anyway (just as a curiosity), did you check if `gs_unref_hashtable` also frees the keys and values with the respective destroy functions passed to `g_hash_table_new_full`?


WoopWoop


Nice feature !


I think CamelCase is used for constants and it might be better to use dv instead of DV


This should not be a configurable. It is always the first IP in the ServiceCIDR. Unless this changed upstream...


thanks for all your hard work on this one...


can't we just keep mother tongue in english? coala is english, translating the messages of languagetool would be inconsistent.


unack


Оке.


will conflict after rebase but looks good in principle


Shouldn't be configurable in the first place. Should just derive it from ServiceCIDR


I looked it up. It is not an open spec. It is here: http://www.jeita.or.jp/cgi-bin/standard_e/list.cgi?cateid=1&subcateid=4 Not sure if we should reference it. Let's let the editors decide once you submit this as a PR


I think PEP8 requires 2 blank lines before the function def.


Yes, this is doing nothing in the current implementation, unless we decide to change it once again to follow the origin file in some case :) I rather keep it for now, as even if we forget to remove it later, it is doing no harm.


Should also confirm that dnsServiceIPAddr is not the first IP in serviceCIDR


@oliparson My bad! Will correct this tomorrow!


Nice!


because `<a></a>` can be shortened to `<a/>`, so this linter will eitheryield syntax errors which isn't really helpful because people rarely produce syntactically wrong files, so if we use the stdout and `gives_correction` we can give some more useful linting beyond syntax checking!


done in 3973892e29e5755d8b940870c6884fdbe94dd138


oui on touche à la flexibilité de la thématisation c'est le propre de l'utilisation d'un FW (et nous devons l'accepter et le limiter au maxi c'est ce qu'on essaie de faire ) mais ça reste modifiable quand meme ... en bricolant ... Bon quand t'aura du temps fait moi passer un pages.php


No problem man I think I speak for both of us when I say we're glad you're using the program.

I'm glad spoinky showed me how to support corsair as well :).
Stay tuned for an update (maybe this weekend) that adds another type of display for rocket league.


inverse of a matrix rather than compute it repeatedly.
Below are a pair of functions that are used to create a special object that
stores a matrix and caches its inverse.


ack


We recently reported to @dadhi a lock contention issue which Maksim then fixed. 2 threads doesn't stress the container enough under load, such as Web services which create one thread per request.

Basically, this is a good start but maybe a separate load test for true concurrent performance benchmark.


`rippleCredit` needs a journal passed to it. I passed it a journal with the category "view" to keep with the rest of the system. If we don't mind passing a journal with the category "flow" to `rippleCredit` we can get rid or `Logs` here and just pass `Journal`


Makefile:1048: recipe for target 'ChangeLog' failed
make[4]: *** [ChangeLog] Error 1

are you sure this is related? there is no ChangeLog target in here. And we don't need git for downloading the tarball(although I should probably add the same git rev resolution as in autotools)


Nobody really cares about tracking that file.


This is definitely necessary. When handling pointer focus, we use surface->view to get the view, even when the surface is a subsurface. If you are strongly against it, this could be changed, but I think this is a better option(after all, this subsurface belongs to the view in some sense).


Yes.


seems like more of the regexp's need a space.. closing this one.


nit: why always the relative paths? Is there a specific reason to no just use ".gitignore"?


can you do a test without the config?


The original proposal here: https://wiki.whatwg.org/wiki/ImageBitmap_Options, states that "colorspaceConversion only applies when creating ImageBitmaps from bitmap media data (i.e. blobs and image elements with non-vector images)". I believe it applied to Blob and HTMLImageElement excluding svg-images.


Yes. (since the generation script can check the default-locale, we could even automate this, but _rename.tab is easier for now)


Renamed.


Not yet mentioned in the readme :)


Shifted. Also added check.


`credentials/*.pem\n`


I'm taking in the html changes.


all tests pass:
success: http://buildbot.bos1/builders/test-bitcask/builds/580


VZ wrote:
> Unfortunately this doesn't work, `GetContentScaleFactor()` always
> returns 1 before the window is realized, so I never see the new menu
> item.

Damn. That's going to be a problem.


Malinda, I always have to dos2unix your files for some reason. There are always a bunch of strange newline characters in them. That's why the entire evalSide.html is diffed...


Sorry for responding so late, I didn't receive your message somehow...

![screenshot from 2016-02-25 01 25 46](https://cloud.githubusercontent.com/assets/6505000/13296255/4c42efaa-db5f-11e5-97bd-156a70799521.png)

1. Version (from CHANGELOG): 2.0.2 (2016-02-02)

2. `find /usr/lib -iname "*libexpat*" -type f`
```
/usr/lib/x86_64-linux-gnu/libexpatw.so.1.6.0
/usr/lib/x86_64-linux-gnu/libexpat.a
/usr/lib/x86_64-linux-gnu/libexpatw.a
/usr/lib/x86_64-linux-gnu/libexpatw.la
/usr/lib/x86_64-linux-gnu/libexpat.la
/usr/lib/i386-linux-gnu/libexpatw.so.1.6.0
```
3. Architecture: 3.19.0-51-generic #57~14.04.1-Ubuntu SMP Fri Feb 19 14:36:55 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux


hm, that doesn't seem to work tests are skipped on circle at least when I look to coverage. Maybe @Redridge can help?


Что-то типа ``tagExists`` или ``tagPresented``


NO MORE DISCUSSION, I AM LAZY


we deserve bonus for the teal background lol


Achivement unlocked: TooManyChanges!
xD


Done.


Yes, but you only showed me one instance of correction.
Also, the correction gets passed through `stdout`, how can I retrieve that since I use `stderr`?


easier to do

```js
handle = e => {

};
```

and not use any binding


@borsch не зовсім, в першому випадку замість нула краще кидати ексепшн взагалі. а так то так, все вірно


how does this work for the win32 case? msys2 downloads the files using the same logic, so we'll probably need it for cmake as well?


why not just use proptypes?


it should be indented 3 more spaces


Where does the ```value => shift```  key/val pair go in your rewrite?


Add also gvg_te


can you put `target` as first argument in this callback function


gvg_te description ere too


first time


What about importing in `patterns/__init__.py`? Could this make the refactoring of the import statements in the single modules obsolete?


Hahahaha xD Dieser Zufall xD @NAAVOO


Modify this line


ME respondes pls?


change and add sleep time here


nit: this is semantically unclear. You're calling "WriteToFile()" and providing a path.

Also, I've never personally tested this (or tried), but all these relative paths (rather than using path built-ins) will likely break support on non-nix OS's. There may be other reasons this tool wouldn't be supported on windows, say, but it shouldn't be because of paths.


@sils1297 No actually config is necessary for scalastyle.


@brandur should it be `strconv.FormatBool(params.SubProrate)` instead?


While we are updating this, lets use the right type t.decimal with a :precision and :scale set.

http://api.rubyonrails.org/classes/ActiveRecord/ConnectionAdapters/TableDefinition.html#method-i-column


You can call this: Code Style change.


Maybe because it is v8 specific?


What I understood from the links you pointed is that both `msg->response_body` and `msg->response_body->data` are filled in the client side after the response message has been fully read. `soup_session_send_message` is synchronous, and only returns after the response has been fully read, so I think we're good on that.
https://developer.gnome.org/libsoup/stable/SoupSession.html#soup-session-send-message

About dereferencing the pointers directly, I thought the "right way" of accessing a gobject property was using `g_object_get`, that's why I chose to do it this way. So leaving the memory concern that was discussed  in another comment with @cosimoc aside, which approach should we take here (I'm fine either way)?


`operator==` is not a member function. Any function may compare `Step` for equality.


Edit to "Chavez, Cesar."


lol


FixGSHPweirdOutputOfAsterisksEtc (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2119 of 2120 tests passed, 0 test warnings)

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


var userChoice = prompt("Do you choose rock, paper or scissors?");
var computerChoice = Math.random();
if (computerChoice < 0.34) {
	computerChoice = "rock";
} else if(computerChoice <= 0.67) {
	computerChoice = "paper";
} else {
	computerChoice = "scissors";
} console.log("Computer: " + computerChoice);

var compare = function (choice1, choice2) {
    if(choice1 === choice2) {
        return "The result is a tie!";
    }
        if(choice1 === "rock") {
            if(choice2 === "scissors") {
                return "rock wins";
            }
            else {
                return "paper wins";
            }
        }
        else if(choice1 === "paper") {
            if(choice2 ==="rock") {
                return "paper wins";
            }
            else {
                return "scissors wins";
            }
        }
}
compare(userChoice, computerChoice);


I think it's described here :

https://www.securecoding.cert.org/confluence/pages/viewpage.action?pageId=637

I can't find where is the correct rule but it's an issue with initialization order. With singleton you control the way your variable is initialized. Moreover it could be used to change your random engine (even initialize it with a more robust seed, ...), and gives you the ability to use multiple random engine (with the upcoming support of local thread storage this could avoid race conditions). There is no overhead of using the singleton since compilers can do a perfect job. Finally global variable names could not be overloaded instead of function names.

Remember that it's not because you don't encounter the issue that there is no issue ;-) (moreover with implementation specific classe like this).


@remistr So it's a little tricky to look at, but because the conditional is based purely off of `params.SubProrate` being true, it doesn't make a difference if we just use a raw boolean value of `true` here. This seems to be how it's written elsewhere in the library, so I just copied the convention for consistency.


Edit suggestion: "Type-written letter on green paper"


ah ok, I'd find it better if we could generate a config automatically. But that could be a new issue.


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


Sorry for the delay, but I was curating some models that are already vetted in Phoenix by our group. You can use them as a starting point and then take it from there.. I will create an issue for each drug and upload the files required for that particular model. Cool?


Yes, @kunalspathak had an explanation in the commit description. Adding comments here would make it easier for readers to understand.


yeah let's keep it that way and then we should file issues to autogenerate configs for all those bears


Olá, não temos acesso ao cookie. Estamos, em /siga e o cookie está em /sigaidp. Não existe jeito elegante de pegar essa informação. Só fazendo um iframe, aí fica feio demais. No mais, não é uma questão de ser "inseguro", se estiver errado, simplesmente não conseguirá autenticar. Não entendo que exista risco de segurança... Tks!


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

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


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


He can summarize fine what he was doing.
In short:
Adding 0.14 Entitity classes

- Witch;
- MinecartChest;
- MinecartHopper;
- MinecartTNT;
- TripoidCamera;
- ThrownEnderPearl;

That difficult is it not right :)


@brandur: the other one works the other way though and is called `NoProrate`. Your code seems to mean we can never disable proration on the upcoming invoice no? Since you never allow it to be set to false


[`assert.doesNotThrow`](https://nodejs.org/api/assert.html#assert_assert_doesnotthrow_block_error_message) can be used here.


What if we'd just check if `uname` is Darwin and if it is then use glibtoolize ?


You are welcome to merge this into your main repository if you would like; however, if I remember correctly I ran into some troubles using this. I had a lot of trouble convincing bower to handle the commonjs require statements correctly. Eventually, I switched to webpack, which seems to be more popular in the React community.  (Also, I am no longer using react-faux-dom for reasons that are more political than technical.)


http://manage.dimagi.com/default.asp?218191
![screen shot 2016-02-24 at 1 42 15 pm](https://cloud.githubusercontent.com/assets/918514/13296753/11fd7c6e-dafd-11e5-8bd5-908a5a492317.png)
![screen shot 2016-02-24 at 1 42 12 pm](https://cloud.githubusercontent.com/assets/918514/13296754/11fe9e00-dafd-11e5-8cf2-ef13b9e95cfb.png)


What it this


Same for my 1+ with Vodafon Germany Carrier.


In all-php files, the closing php tag is redundant, and is better left off to avoid issues with "header already sent" errors.


nit: ReadFromDir()?


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2115 of 2115 tests passed, 0 test warnings)

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


I think using just the one partition "Flow" would make more sense


@jianchun Oh yeah. That would be better.


fast-forwarding develop-2.2 to borshop-integration-814-dr/bump_eleveldb_for_2.2 = 7b947cd6


I don't think we need to restrict in pre-re since TE is only for renewal


Please accept this.


`URL mismatch in POST data (expected \'%s\' but found \'%s\')`


fast-forwarding develop-2.2 to borshop-integration-94-dr/bump_eleveldb_for_2.2 = 871a2f80


`Tampered field \'%s\' in POST data (expected value \'%s\' but found \'%s\')`


`Unexpected field \'%s\' in POST data`


`Missing field \'%s\' in POST data`


@odolha, I'm working on a plan to consolidate all the escomplex repos into a single project (i.e. under the [escomplex](https://github.com/jared-stilwell/escomplex) project). There should be more details available by early next week. I'll keep this PR open until that happens and it becomes a little clearer whether a fork of this project makes sense or not.


準備自己的庫，因此所做的改變


I've reworded some of the exception messages for brevity. Let me know what you think.


Doesn't this save the "SimpleOutput" / "FilePath" twice now? It is once saved from the simple output input field above and again here. Since the 2 fields are linked together anyway and should always contain the same text, it doesn't make sense to save it again.


should be "politics and society" (no capitals)


TeamCity LabKey :: LabkeyModules15.3 :: Git postgres 9.4 [Build 119](https://teamcity.labkey.org/viewLog.html?buildId=305368&buildTypeId=LabkeyModules153_GitPostgres) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:55


That wasnt all :p

> Am 24.02.2016 um 19:46 schrieb Danny Terweij <notifications@github.com>:
>
> He can summarize fine what he was doing.
> In short:
> Adding 0.14 Entitity classes
>
> Witch;
> MinecartChest;
> MinecartHopper;
> MinecartTNT;
> TripoidCamera;
> ThrownEnderPearl;
> That difficult is it not right :)
>
> —
> Reply to this email directly or view it on GitHub.
>


Why?


Congratulations, well done!


Použil jsem \fullref z http://tex.stackexchange.com/a/121871, avšak generuje "section 2 Implementace", protože je složen z \autoref*{#1} "section 2" a \nameref*{#1} "Implementace". Nezná @tlapicka způsob, jak vygenerovat pouze "2 Implementace"?


Btw, I got some feedback where it was wondered why imageOrientation was needed. I guess it applies to isFlipY too.


ok, the problem is dvdcss: https://github.com/xbmc/libdvdcss/blob/master/Makefile.am#L10-L16


Expand this description - should tie your location in with your object/article.


guess so ... just trying it ;-)
but I have copied your yaml
hopefully no it will turn to green ;-)
thx

Von meinem Handy gesendet

> Am 24.02.2016 um 19:19 schrieb Iceman <notifications@github.com>:
>
> Cool, having the automated build server is great. You can check out the travis file in my fork to get it running.
>
> —
> Reply to this email directly or view it on GitHub.
>


How about using git subtrees?


Add your citations - especially the citation for your item.


Did we confirm the set statistics output is for  the select * from stat_check


Done. Andreas mentioned this also.


FixCrashWithZoneAndSystemERV (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2121 of 2121 tests passed, 0 test warnings)

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


Fixed (was able to remove the line all together)


I would also mention the Cassandra port (9042), the Jolokia port (8778) and the Prometheus port (9100) here.


@jwilsson wouldn't u think, we should publish 3.1.2 version as we have at least two bugs fixed? Do we wait for some reason?


Had to change all this to address @SzymonPobiega 's comment


thanks :D


this shouldn't be here, just in `Game`


Pushed an update, please re-review https://github.com/Particular/NServiceBus.NHibernate/pull/159


Take out the catch


use this.repository in stead. It's a "bind", so it could be added later and then the repository won't exist


I suggest `===` instead of `==` and single quotes instead of double.


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

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


I don't think this is correct. This means we will always check out the latest commit in a tag, instead of the lock. The fix is to *always* use the lock, as it was before, except if the lock is out of date.


but i think its not published to npm? could you please do publish it?


CCurlFile::ParseAndCorrectUrl extracts pairs after a Pipe (|) and translates known values in CURLOPT values. Its common usage and a lot of things can be sttered already nowdays.


meh, I guess that would work, proptypes are a weird ugly way to add typing to a language IMO


This can be a non-optional ``let`` instead of an optional ``var``


Yup did consider that scenario but decided that frequency of that happening was low enough that it wasn't worth the change.


? still need to remove the handler


111624768_HWRadiantConvBB (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


If we make responsesDataController then we can get rid of this ``guard``


```js
if (!resource) {
  return;
}
```

If there's no resource, it's up to the user to supply the count.


Python ile browser yapımı için yapılan bir repo


I would suggest putting the `const` assignment near the top of this function. It's nicer when you can see all variables at once then go through functionality.


Alternative way - search entity.histfig_ids vector for unit's hist_figure_id. Dwarf Therapist uses this method.

auto entity = df::historical_entity::find(df::global::ui->group_id);
if(!entity || unit->hist_figure_id == -1)return false;
return std::find(entity->histfig_ids.begin(), entity->histfig_ids.end(), unit->hist_figure_id) != entity->histfig_ids.end();

But game seems to use entity links. If you delete link - unit imediatly(without unpausing) become uneditable in vanilla way.


Again, if you're using ``!`` to force unwrap, you're probably doing something wrong.


Should this be in `Client` do you think? And instead provide `getDefaultLocale` (or maybe `getServerLocale`) on `Server`?

Client has a defined locale (chosen by the user), whereas on serverside there is the "fallback" locale of the server and the individual locales for the players themselves.


Well at the time of writing the doc, v3 support wasnt there for rgw yet, I guess I'll have to update with v3 options as well, havent looked into it yet.. I'll take a look at the pr which was merged and add the v3 docs too


For win, the CMake file still just searches for the prebuilt libs from the downloaded win dependencies. If with msys2 we're able to build them directly, the bottom of this file has to be adapted. I could probably take care of this at the weekend.


AccessDeniedAccess Denied96F6EAA41413F50279Uz8SlRKLpZV6pYcxzbVHNcXVfOiNo07na9YSpZVJrzjdFNyprKTbyKuc0IkubtlC9kwzu7Rx4


In that case, DNODE_FLAG_USEROBJUSERD_ACCOUNTED shouldn't be set anyway. Maybe I misunderstood your question?


I don't like console.error in plugins. Perhaps use a throw, emit an event, use an attribute to handle errors... Check how other plugins handle this :)


I think it is safe to remove this commented out old code.


This will read better with a space in each side of ``..<``. In general, we should always put a space between an operator and its surrounding context. Follow Apple's code examples for proper spacing.


Fix this


typo. should be "increment"


- Faltou você colocar o nome da imagem (404.png)
- No alt pode colocar "Erro ao encontrar a página"
- Pode remover o atributo title


Done.


@kashike rationale?


See https://help.github.com/articles/dealing-with-line-endings/


What is incompatible about it exactly?


Honestly I think all these changes on stats functions can be optimized a bit before merge, like this example:
```
case CHC_MAXHITS:
{
	if (s.HasArgs())
	{
		long iStatRaw = s.GetArgVal();
		iStat = (iStatRaw > SHRT_MAX) ? SHRT_MAX : iStatRaw;
		}
	else
		iStat = 0;
	Stat_SetVal(STAT_STR, static_cast<short>(iStat));
} break;
```
these stats codes can be changed to
```
case CHC_MAXHITS:
	Stat_SetMax(STAT_STR, static_cast<short>(minimum(s.GetArgVal(), SHRT_MAX)));
	break;
```


Mezcla realizada


Weird. Any idea why?


----------------
	User:
	Device Name Dev2787
	Model samsung, SM-T235
	App: TrackView, v. 1.4.2-pro
	SDK Version: 19
	Language: vi_VN
	Carrier: VN MobiFone

Mon Feb 15 00:41:38 GMT+07:00 2016:
[12:41:38 AM]: VApplication.init
[12:41:38 AM]: Loading settings {"m4":0,"ry":0}
[12:41:40 AM]: valid date: 2013.9.30-2016.6.30
[12:41:40 AM]: parsed date: Mon Sep 30 00:00:00 GMT+07:00 2013 Thu Jun 30 00:00:00 GMT+07:00 2016
[12:41:40 AM]: Command: enablepaidversion

[12:41:40 AM]: Command: setrelay

[12:41:40 AM]: onStartSession com.trackview.activity.TrackView@41fe65e8
[12:41:41 AM]: onStartSession com.trackview.login.LoginActivity@4202a8c0
[12:41:42 AM]: onEndSession com.trackview.activity.TrackView@41fe65e8


If you switch the single quotes in the `editButtonHtml` to backticks you can use string interpolation like
```js
`<a href="${editHref}">`+
```


@Mumfrey Last I heard, we aren't adding everything and their uncle to `Sponge`, this counts as one of those in my opinion.


Please, remove that "wipeingui" flag. If the users wipe this partition, they will lose their IMEI numbers.

Thanks.


@dbnicholson I think this change covers what you mentioned here: https://github.com/endlessm/eos-updater/pull/26#issuecomment-187869086 Please let me know if you meant something else.


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


Single quotes instead?


You can also make this a documentation comment (that shows when you option click on the name) by using ``///`` slashes. If so, you can also make it a little more detailed like: "Given a new comment, looks through responses and increments the ``childCount`` of the parent of that comment."


[{"_id":"1","lat":"1.3243889","lon":"103.9251209","accuracy":"63.7960014343262","provider":"fused","date":"1444964239072","date_s":"16-10-2015 10:57:19","address":"31 Chai Chee Ave,Singapore","imei":"866376029640841


@aleffert we are populating DiscussionResponsesViewController from storyBoard so we don't have init method for this. So making this non-option results in error.


I elected to remove the boolean since the function already implies setting the status to public.


This can be changed back to ExecutionMode (and the static cast below removed) now that you have added `.load()` everywhere.


Done.


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


I believe it is important to stick to one key word for polynomial. The docs and online help suggest 'polynomial' but the actual keyword to use in the code is 'poly'. In addition, 'p', 'po' and 'pol' are also accepted. If this is the intended behavior, it should be noted in the docs.

The docs suggest that \<model_table\>_random is used internally and not meaningful on its own. Is there a reason why it is not dropped by the end of function and left to user for cleaning up?


runStatus doesn't seem to have a purpose. It's always 0 (STOP is 0). I don't know if it used to do something in the past.


Now these are completely optional and default to some versions (see below)


``private let responsesDataController = DiscussionResponsesDataController()``


Defaults applied (0.4.6 and 1.16)


:heart:


Done; I s/skill list/SkillList everywhere else in this file, too.


The same way we install the grunt-cli for commodity, we can install other stuff, versioned or no. If not versioned will go here. If versioned will be similar to the shifter/recess case.


Only if there is not package.json


Deleting CamVerify I guess @OxLemonxO


Please change the choices here, they do not make any sense.
What is "pinned"? what is "rolling"?
We should strive to be intuitive to others not to outselves.


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


Are you interested in continuing the work?


Maybe other servers do not want it.


It doesn't seem correct.
If a user set a duty_cycle_percent she expects it will be updated with period changes, specially because you can't guarantee order of packets deliveries.

So let's say it receives a packet setting period to 2000ns and duty_cycle_percent to 50%.

Depending on the order it may end up with duty cycle of 0ns or 1000ns...


"Boggler"


@wluyima what settings do you recommend - do I go for auto?


Note I planned to have all the requirements (shifter, recess into some array) and then run the same code to install them, but shifter's --version behavior forced me to handle it apart. In general the recess block below will be reusable for any other future package that we want to install with a version.


This is a somewhat fragile process. Ordering of template/encode is important and can only be run once.

Maybe as an alternative we build a config of our raw assets, and something like templateAndEncode just returns the templated/encoded data for use (rather than overwriting itself with now templated / encoded data).


Shouldn't we check for m_function->IsInDebugMode here?


Good catch. I'm trying to use 'collectionBackendObject' as the canonical example for the raw data used to construct the collection frontend domain object, so that other frontend domain objects also use a similar pattern. I don't fancy the idea of using 'dict' anywhere in the frontend, since JS doesn't really have a concept of what that is. At this point, it's really just a JS object, anyway.


old school redirects FTW!


Done.


IMO, the data kept here should be moved to `DbConstantsStore` as a boolean value


I had to bisect to found why my application breaks when migrated to 3.3.x.
I use old behavior: ```collection/id/.format```. Was it really necessary to break the behavior?
It would be great to mention this in ReleaseNotes or issue compatibility Warning, like Django does.


This is perfect - if get_destination returns something. If it returns nil, which it does when there is nothing in that direction (or if we don't have the world file for what should be in that direction), we can't set the player's location to nil.


Здесь, наверное, нужно изменить данную реализацию. Как нам сегодня говорили: "Если ошибка не критична, то можно просто написать cathc(Exception ex)".


[The patch has been applied](https://sourceforge.net/p/netatalk/code/ci/eac98606745f4263897244d027ebb8d3de36887c/).


Why not gtkglsink if available, and gtksink as a fallback? Could be easily implemented like https://github.com/sdroege/gst-player/blob/master/gtk/gtk-video-renderer.c#L107


The other PlanNodes don't seem to have a toString.  I'm not sure if there is a reason for that or if it was just and oversight.


I see what you are doing here, but it assumes tight coupling with a certain WebView. If you had a third-party WebView engine, this will not work.

Try using, `[self.commandDelegate evalJs:jsString]`


If the idea is to use it to feed pwm I don't get the reason to convert it to percent.
I've commented about that on the previous PR already.
Anyway, it would be compatible with DUTY_CYCLE_PERCENT, not DUTY_CYCLE


Done


Good suggestion; changed. Thanks!


Hello,

On 02/24/2016 03:39 PM, Ville Skyttä wrote:
> Makes sense to me, but this causes three test suite failures: one for
> scp, and two for sftp. See the Travis test details link for more info.

Ah, I missed there are two sftp failures, but scp also fails for me
without my patch, and one of the two also fails for sftp here.

Can you confirm that scp and sftp are already broken? The new sftp
failure should be fixed in my master branch.

Best regards
Uwe


jksadlfjksjkdlf


yayy! \o/ :+1: :sparkles: :fireworks:


it's pretty empty


why remove the comment ?


do we need to add "1.2.0" here as well, or was that where we broke binary compat and we can't do that?


Doesn't really matter because `os.path.expanduser` gives the correct directory. I guess this *could* fail on some Windows machines. Ughh..


The idea is that a user does not have to type the whole word. This can be both convenient and help eliminate error cases caused by typos. This is also consistent with the behavior in gaussian's case.

<model_table>_random should only be removed if <model_table> is removed, because prediction relies on <model_table>_random


commented on https://issues.apache.org/jira/browse/BROOKLYN-227 and addressed


TaskCluster-GitHub: https://tools.taskcluster.net/task-graph-inspector/#Lnu0NaerTsumt9d07li0Qw


Why this file exists? I did not find any references in shell scripts and the generated html is only the body without head tag (for link or include style).


IEntity owner не нужен, так как у нас в каждой сущности, которая имеет какую-то зависимость определено свойство <ИмяЗависимости>ID. Пример:

public class TodoListModel : IEntity
    {
        public TodoListModel()
        {
            TodoItems = new List<TodoItemModel>();
        }

        public int ID { get; set; }
        public int UserID { get; set; } =====> указывает на ID пользователя!!!!!!!!!!!!!!!!!!
        public string Name { get; set; }

        public virtual ICollection<TodoItemModel> TodoItems { get; set; }
    }


When dealing with this kind of situation our proposal (followed by most of the other node types) is to warn the user and invert max and min for her instead of returning an error.
We consider it less frustrating yet educational. So she'll be able to test her flow and fix it later.


Hi @cedricmessiant @sgeulette

the collectionwidget id is not always "c1", this is the purpose of the utils.getCollectionLinkCriterion, could you please correct?

I think it could be easier to systematically branch or fork and do a Pull request, this is easier to review.

Thank you,

Gauthier


is this notation for dual stack IPv4,IPv6?


"cancelled", not "canceled"


@hmmr As I mentioned in the intro, I'd like to use the relative error instead of absolute error, so we do need the division.


one line import, from datetime import datetime, timedelta


wow


no empty line here


oh, I guess not, what's this?

`0.0.0.0:3232,[::]:3232`


an alias for "any address"?


try catch here in case of your data is fucked up and the unpacking fails


Awesome ! :+1:


yes, it's better to explicitly tell libtorrent to open in IPv6 too


plone.schema has no stable release.

https://pypi.python.org/simple/plone.schema/


The commit message mentions non-existing option `--output`.


Please document it should be a positive value on JSON.
It may seems obvious but...


Done


Let's leave this out.


Joli Willy


Instead of aborting the flow on this case you may warn the user and follow with default value if it can't be harmful (not sure if it could damage user's equipment).


Is dat echt zo?


UVISOR_SVC_SLOW_INDEX has already 0xFF filtering - same with UVISOR_SVC_MODE: thus 8 bit is guaranteed


Did you mean `php artisan`? lol. Not sure everyone has the `art` alias.


Ziet er niet best uit.


Здесь предлагаю назвать LogModel, так как у нас сущности БД заканчиваются на Model.


@vfonov Does this change mean that InstallManPages need to be included in the superbuild?


I forgot to add the "optional from" member.

The loader for the data subRecord use DynamicCast to detect which form the class inherits and loads the appropriate data. The change added the TESWeightForm class as a new parent to the TESAmmo class. The loader first fill the Data in memory with zero then tries to load 4 bytes for value, then 4 bytes for Weight. If there isn't enough bytes, it will ignore it.


Node requires 'use strict'; right now for it, but might be worth thinking about 'let'
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/let


self.occurs = defaultdict(set) so you can get rid of the setdefault


I think this should be:
```csharp
public abstract Task CancellationTestQuery(CancellationToken token);
```


Likely should be in both `Server` and `Client` now that you mention it. `getDefaultLocale` is current equivalent with the already existing `Locales.DEFAULT`, `getLocale()` is meant to return the `Locale` that has been configured in the config file. I learned recently we already listen for client settings packet and you can, in fact, already do `player.getLocale()` on the server and it will give you the client's `Locale`.


I am not sure this is what you guys meant, but it doesn't work. There are other places where the login state doesn't match this and I'm not certain where they all are, but issuing the second "login" doesn't result in a pinentry with this change, and the ui just continues to show the user is logged out (though toasts pop up saying they are). @maxtaco, you said we weren't changing the whole API's notion of being logged in, but I don't see another way to keep the UI consistent. What do you think we should do?


@SanketDG can you file a newcomer bug to change to_line to end_line and so on? This way people can see how the params are named.


What do you mean "requires strict"?  That's optional, and I don't think any of actionhero's files have that at the top of the file.


fixed with commit f7c798e


ditto fix :)


Oh man, great catch! I'm glad that you took review duty :)

Alright, in 5e6c138 I've "inverted" the prorate parameter which I think will allow proration to be properly disabled. This is modeled off of [how it's done in the subscriptions package](https://github.com/stripe/stripe-go/blob/5e6c138361cd33c681140c354abd47329701d986/sub/client.go#L115,L117). Does it look better now?


Good points.

On Wed, Feb 24, 2016, 19:54 Henning Jacobs <notifications@github.com> wrote:

> I would also mention the Cassandra port (9042), the Jolokia port (8778)
> and the Prometheus port (9100) here.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zalando/planb-cassandra/commit/72e5c92a3a292bef2285db8dd61db6d347ed145f#commitcomment-16306156>
> .
>


Unfortunately, there is one problem with using "==" instead of equals. I mean the case of the local game. Technically it means that the player plays against himself, but crossPlayer != zeroPlayer. The problem is that they share the same screen to draw game info and to get input from player.
Using "==" is still possible, but it requires some hacks to be implemented.


let's GH on this...


You're missing an `x`.


Edit description


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1007 of 1007 tests passed, 0 test warnings)

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


Always use triple equals (or, alternatively: `!results` if you are just checking for truthiness).


A3][A4][A6][vi_VN] Support for leeoh369


@emb03 is this target needed and if so is it supposed to be `_blank`?


:+1:

I think the stddev testing error illustrates that the relative error is useful (and incidentally makes for a nice test case for the changes, thanks).


A good practice when it comes to form submissions is to, instead of attach the action to the button, create a `<form onsubmit={{action "submit"}}>` and leave the button without any action, because forms can also be submitted by pressing enter by example.


@slavapestov LGTM. Thanks for working on this, Slava!


I think you can use `Long::sum` here


Fixed (and improved by removing the need for allocating two buffers on stack and copying the string twice -- once is enough) - I'll push after I checked the other issues as well


Horizontal direction should work the same way as the example in README.md. The only additional property which is needed is "rowHeight". So instead of

    ...
    // ANDROID ONLY
    columnWidth: 150,
    verticalSpacing: 10,
    horizontalSpacing: 10
    ...

you have

    ...
    // ANDROID ONLY
    rowHeight: 150,
    columnWidth: 150,
    verticalSpacing: 10,
    horizontalSpacing: 10
    ...

Note that in my modification the horizontal direction is hardcoded, as I only need horizontal and not vertical. It should easily be extendable to make this a creation-time property though.

Hope this helps.


The repo does not seem to use git submodules so gitsm is unnecessary.

My earlier feedback was about addinfg '+git${SRCPV}'


Can we remove all the verbose traces for now?  It should not be in this code review


changed min SDK version from 18 to 16


However, I agree that the keyword used in the code should be the full keyword 'polynomial' rather than 'poly', such that it allows a user to input any prefix of 'polynomial' including the word itself.


Sorry, my sentence was poorly phrased. I was speaking to using 'let' instead of 'var', though it's a more general statement rather than specific to here.
It's just you can't use 'let' without a 'use strict'; line right now. :(


Looks like you also added this


Reverted it back


Yeah, I totally get why static analysis can't figure it out. Was just pointing it out.

But regarding your second paragraph, I don't think that's true. Calling `raise e` where `e` is the exception passed to our `rescue_from` handler will re-raise in `e`'s original backtrace.


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


ahh, sure.
There's a large refactor pending where I guess we should change (almost) all `var` to `let`


@sopel39 is correct.  An outer join has the normal `LookupJoinOperator` as an output and a separate `LookupOuterOperator` which outputs the unmatched rows.


ohohoohoho! lol!


Sure, no worries. Not sure why you guys are so happy to see this, it is required as part of the protocol resilience work, I don't want things to just crash when it's all done and you build the library with -O. :-)


Maybe a refactor in the future could be to pass a deck object instead of a longer list of parameters? Seems fine now, but I can see this getting pretty long in the future.


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2153 of 2153 tests passed, 0 test warnings)

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


"allowwed"


Isn't that available in ES6? Is it ok to use it?


minecart is not moving

WIP?


This should remain as-is (without the slash).


Haha, wieder fehler...hab den text von dem letzten push nicht verändert


FixCrashWithZoneAndSystemERV (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Oh Okay! I think I got it wrong I will reupdate it


If we're bothering to inline these templates in the binary, may as well template these too. Or at least use our default values (rather than having them defined twice).


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


it looks like we're putting `ceph.logrotate` into `/etc` here, rather than `/etc/logrotate.d`? can you confirm?


True, thanks for the reminder!


Ah, good point ... I must have started working on this and forgetting to add code to include it. Will fix!


fixes:
* `size_t` -> `long` : size_t is unsigned
* float comparison:  ` 'sth1' == 'sth2' ` ->  `fabs( 'sth1'-'sth2' ) <  FTL_EPSILON`
* casting: ` (sth) ->  static_cast<sth>( ) `


This XML file does not appear to have any style information associated with it. The document tree is shown below.
<map>
<string name="EI">352560074356491</string>
<long name="timestamp" value="1455990989697"/>
<string name="DID">352560074356491</string>
<string name="UTDID">VsiozTBPs6YDAEQHLkb2v1QE</string>
<long name="S" value="2490635226"/>
<long name="t2" value="1455990989624"/>
<string name="SI">452019138756591</string>
</map>


I don't understand here. freq is init as 1. in the whole loop you only do freq += 1, aka increment, but no decrement. And, the yield condition is depending on freq > 0, which will always be the case no matter what your loop does. Is this really what you want?


@vincentchabannes I would prefer to *duplicate* some data to let us write:
```cpp
for(auto const &it: model.materials())
K.on(_range=markedelements(mesh,it.first),
_expr=(
   vf::chi(idv(T)>it.second.Tliq()) * it.second.Kl()
+ vf::chi(idv(T)<it.second.Tsol()) * it.second.Ks()
+ vf::chi(idv(T)<=it.second.Tliq()) * vf::chi(idv(T)>=it.second.Tsol())* 0.5 * (it.second.Ks()+it.second.Kl()) ));
```
If there is no duplicate, I'll integrate two times over the same range - that can be time consuming isn't it ?

Ideas or remarks are welcome :-)


The llvm-3.8 branch fixes the issue using `"%0.*" PRIx64` instead. (No action required on your end, just telling.)


should be if (!Units::isOwnCiv(u) || Units::isOwnGroup(u) || Units::isOwnRace(u))
if creature's historical figure has entity link "MEMBER" with your group (alternative: if creature's histfig_id is present in entity.histfig_ids) - it is not a pet for sure.
Only historical figure can be isOwnGroup. Ordinary pets have histfig_id "-1" so they are not isOwnGroup regardless of method.
Even historical figure pets are not members of group. They are members of civ only.

And that pull request is not pushed yet.


I do not think we should use {} here.


It's a test code. We should not maintain backward compatibility for that. :) No need to make alias.


I have pushed a modified version as it was not 100% correct

in MSYS, i have no problem with relative paths. Can you give me an example of command you use ?


@G07cha, that image is magical. Mind if I highjack it?


This should all be gone in master -- no more uri hackery at all.


is there a reason to prefix with `this.`?


Please can you do this invert based on a DMI quirk. Also, note the desired _DSD property in the commit message.


small penis


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


This shouldn't have a constructor if it doesn't do anything.


for a rule that applies to multiple selections, the preferred style is to give each it's own line. we can do a more cohesive linting later.

https://github.com/airbnb/css#formatting


Nice commit message @danomatika :laughing:


That's a good point. Form `onsubmit` provides a better UX. I will update it.


These methods with deck copying seem like they could be moved to a service level at some point. I know we don't really have that concept yet though so it seems fine here.


I have a question here:
When should I use the helper _application_helper_ and when should I use the _application_controller_ with the **helper_method**


These constructor calls probably aren't necessary.


Same as clif_parse_Broadcast


with the (minor) exception of the css rule, lgtm. there are enough other violations of that rule for now that I don't think it's worth fixing here.


@crutchcorn, sure, because I've found this one in google by "notice me senpai" query


worried about the perf impact here.  this is basically the innermost loop in our simulation engine.

can we just check the cancel policy once a day?


Look at the column for Chrome latest: https://kangax.github.io/compat-table/es6/


JS broke binary compat from 1.2.0 to 1.2.1


Part of #2135 and #2957


LGTM in terms of functionality. +1


A few of the options was never hooked up to anything and I'm not sure if all the available ones makes sense even. This turned out to be a bigger effort than I anticipated. I think the best place to track these issues would be on the [split out plugin](https://github.com/keis/panel-plugin-gauge). Feel free to open issues about your issues.


Heh. I was seeing if you had any particular styling for case citations, and just happened to notice this.


It is showing fully implemented. So I guess I'll add it.


You probably don't want to be using a mutable object here as a contructor default. http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments


The points created by this loop are just *outside* the polygon, right?


Is this a memory leak?


The diagonal index in the model is differemt than in the server and in the ui

Sent from my iPhone

> On Feb 24, 2016, at 11:11 AM, Trenton Adam Hyer <notifications@github.com> wrote:
>
> Out of curiosity, why the lines returning an edge location given an edge location in your various states? I chased the code around to the parent class, but it appeared not to do anything that changed the input drastically, except maybe creating a copy object. Was that the intent?
>
> —
> Reply to this email directly or view it on GitHub.
>


I still think this is a strange field to have in a `Sha256SignedMessage` but we can change it later


Modified index.html


I'd like us to keep it `private readonly`. My design goal here was to hide the underlying storage instance from the tests. And externalize only the minimum needed. see what I mean by looking at other comments.


Right now this `cancel_policy.execute` gets passed a dt, an event, and a cancellation function, and is responsible for calling `cancel` and returning a bool indicating whether or not to log a warning?

It seems like those behaviors should be flipped: the policy should just be responsible for returning YES/NO indicating that the order should be cancelled, and the blotter should be responsible for actually performing cancellation.  If the cancel policy wants to log a warning, it can do itself so before returning `True`.


fixes #32


`itervalues` doesn't exist in Python3.  You want `six.itervalues`.


This is causing issue in some places , most browsers at end of the line trim the spaces in block, so for an example if I have text "A Certificate will be issued in view of:" and the block can only have content of length "A Certificate will be issued" (notice missing space in end) then the chrome and firefox are keeping the word 'issued' on same line and breaking after that,

![image 5](https://cloud.githubusercontent.com/assets/17438011/13298441/bb305a7a-db04-11e5-8290-7bb5d578f8fe.png)
![image 6](https://cloud.githubusercontent.com/assets/17438011/13298440/bb2f251a-db04-11e5-894e-46c9746cf009.png)

Chrome and firefox are behaving close to second scenario and are allowing the text even if it has trailng spaces going out of bounds of block. The old code without UrlAware breaker was behaving close to the chrome and firefox


Experimental USB drivers


probably


I'll limit it to MAX_INVENTORY (the packet refers to inventory positions, so it should make no sense for it to have more than MAX_INVENTORY items). I'll also backport the underflow check from 2ec5d0d5dad2a54c4af5f3ea1352a4ecb544a160


LGTM; feel free to merge to master.


I think this should be: (see other comments)
```csharp
var task = sut.RunCancellationTestQuery(tokenSource.Token);
```


I think this should accept the CancelPolicy object itself rather than strings.  The current design prevents a Zipline user from implementing their own custom cancellation policy (e.g. "Cancel at EOD on Fridays").


@ju graphql will throw errors for unaccepted arguments. That is not ideal to permit all arguments for all attributes of that record.


tyutyuty


@stevenjswanson I just added GadgetMaker2 to PYTHONPATH.


Are you referencing #778?


ouo


Why not just have this at the bottom of the func? We do all of the `wg.Add()`s before getting there, right?


But I really liked that.


I think this should be: (see other comments)
```csharp
if(!sut.SupportsCommandCancellation)
```


Yaser


This one generates points inside the polygon.


does this function need to apply to the obs in `update_matched_obs`, not just in `initialize_filter`. I have something which does something similar to this in the dual band stuff


Tested using http://pastebin.com/Qa6ZpyyD


tst


why


Close #6

Does this work?


Oops! Thanks! 😃


I personally found them totally useless with colors as it just feel like noise (and Maven output logs are already sooooo noisy) but I get your point :smile:

This configuration was the default one for users installing `maven-color` with `brew` for OS X. No one asked me to restore log level before.

I will open an issue and see if other users manifest themselves ! If not I may restore it in next release, I can continue to remove them with `brew`.

For now, you can just change back your logging configuration.

Thanks for the feedback :+1:


На самом деле, есть способ сделать это с меньшим числом операций. Я собирался показать его, когда готовился к занятию, но это совсем вылетело у меня из головы.


@akselsm
- please move this region into the background regions for now
- why does the ACL require UVISOR_TACL_EXECUTE ?
The background here is to make region 3 available for dynamic use


@ademar  if `System.Web` and `System.Web.Extensions` are useless, you can remove it from `project.json` too


@chrisfsampaio esse projeto eu dupliquei do outro e alterei de imagem para butoes, dai eu simulo os sensores alterando status. Porque ele só altera 1 vez, mesmo clicando varias vezes ? Ele só funciona de novo se eu voltar pra mainScreen e ir de novo. Sao 20 botoes, com Tags.


That's a good commit!


This one generates points that are inside the outer shell. These points will be inside for the polygons that does not have hole. For polygon, that has hole, it' can be random - either inside or outside the polygon depending on if generated point if inside or outside the hole.


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

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


imageOrientation is useful for WebGL use cases. Apparently some WebGL games use shared assets with DirectX implementations (possibly ported from DirectX?) where (0,0) is the bottomLeft rather than topLeft of the image.  That said, flipping the image when texImage2D is called is already supported, it is relatively low overhead and it never requires a redecode of the image. So it would be safe to say that imageOrientation is the least important of the three options. In the initial proposal of these options, there were more values for this parameter to do things like ignore/respect image orientation meta-data (from EXIF headers). This seems more useful because there is no workaround, but there is little dev demand for that functionality AFAIK.


@ademar  add the `Utils/CultureInfoCache.fs` file in `project.json` too, under `compileFiles`
The file list is inside `project.json`


I was also unsure about this, but I like your suggestion for a CollectionPlaythrough object, so I implemented one of those to have better separation of responsibilities, here.


From here onwards, the points are outside of the polygon - outside of outer shell envelop.


See Pull Request https://github.com/domoticz/domoticz/pull/515


looks good


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


Does it have mathlab's situation ?


This does not mean it is in the BSN HCL. There is no BSN HCL in this module.

The sff.supported flag means the eeprom data was parsed correctly and the module has been properly identified.
Perhaps this should be sff.identified.

All the same, this check must still be performed.


good to know - I didn't know you could do this.


Amazing code. SkypeBot offers a vast variety of commands. Microsoft should totally buy this.


@rzarzynski do we also need to check that on get_http_status() we got a 2xx response?


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


111624768_HWRadiantConvBB (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2120 of 2120 tests passed, 0 test warnings)

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


AddCoolGithubThings (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2119 of 2119 tests passed, 0 test warnings)

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


Here and below, instead of "throw new Error" it would be nicer to use "ErrorReporter.errorAbort". That message shows a nice message to end users and gives the option to show the stack trace.


Don't encode `"requested_validity"`, and just use the default.


I was not really referring to that specific line (i.e. the "disposed" flag), but to the onEvent handling for the `WikiDeletedEvent` and `DocumentDeletedEvent` events which would need to do basically what is done in the `dispose()` method.

Maybe I did not pick the best commit to add this comment, but the idea still remains.


I think we should add the property:
```csharp
public bool SupportsCommandCancellation
{
   get
     {
        return DbConstantsStore.GetDbConstants(storage.InvariantName).SupportsCommandCancellation;
     }
}
```


111624768_HWRadiantConvBB (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (2120 of 2120 tests passed, 0 test warnings)

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


Yes, it's in minc-toolkit-v2 already (my latest branch)


@SimonCropp - Rather than either `MessageToSend` or `MessageTypeToSend`, shouldn't this be something like:
`Namespace.MessageToSend, Assembly, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null`
cc / @mauroservienti


T'as oublié de mettre la partie javascript et le fichier tex dans ton commit pour que ça marche


Why do you allocate memory on heap instead of stack here?


According to the API reference, Architecture and Version should not be returned for "PackageType=repo"

Also, there should be double-quotes around the $CFENGINE_TEST_PKGSRC_MOC test in line 12.  As written, it always returns true.

    $ unset VAR
    $ if [ -n $VAR ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    TRUE
    $ VAR=''
    $ if [ -n $VAR ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    TRUE
    $ VAR='value'
    $ if [ -n $VAR ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    TRUE
    $ unset VAR
    $ if [ -n "$VAR" ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    FALSE
    $ VAR=''
    $ if [ -n "$VAR" ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    FALSE
    $ VAR='value'
    $ if [ -n "$VAR" ] ; then echo "TRUE" ; else echo "FALSE" ; fi
    TRUE
    $

I know neither of these deal with the patch you submitted, but this is the first time I've looked at this module.


ok i will add it to the old minc-toolkit (which I still use).

On Wed, Feb 24, 2016 at 2:53 PM, Vladimir S. FONOV <notifications@github.com
> wrote:

> Yes, it's in minc-toolkit-v2 already (my latest branch)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/BIC-MNI/minc-tools/commit/e3e63921f5b74ba49091cedabfba64f2da71e1cb#commitcomment-16307644>
> .
>


More on the above comment , basically it is treating the regular space at end of line " " as   , non breaking space


Additional info: https://github.com/odoo/odoo/issues/7696


@jbredeche I think there's an eventual world where it would be nice to be able to say model something like "Cancel this order if it's not filled in the next hour."  If we supported that, then I think this would be unavoidable.  In the short term, I could see this being only an EOD operation.


Still have some problem. Need fix.


hey, the change looks good, the commit message is neat but instead of `Fix typo at line ...` I'd rather have `Fix typo` (the line is really useless for the reader) or something like `coa ais -> coala is`

That and a rebase and we should be able to merge it :)


It depends on which OS you're using, github is hosted on a unix machine so on OSX you should set it to **input**, [this](http://git-scm.com/book/en/v2/Customizing-Git-Git-Configuration#Formatting-and-Whitespace) explains it pretty well depending on your platform


Is it really necessary? I don't think so.


Very good point, maybe se should expand a little bit and add a comment on full type name and the implications it might have.


Not exactly. This is absolutely necessary for collection node; it's reflecting the exact backend structure. If you trace it, exploration ID is used a lot in and through this object. The icky pattern was having exploration ID and the actual exploration (with its ID in the exploration sub-object) within the same node. Since this node no longer stores the summary of the exploration, I don't see this as a problem anymore. There's no duplication of information.


The following doc(s) have been deployed to the [ttrowellht](http://ttrowellht-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/code/htaccess](http://ttrowellht-static-docs.pantheon.io/docs/articles/sites/code/htaccess)


Another valid point. This is fixed with the introduction of a ``CollectionPlaythrough`` object, which is not used by the editor (though could very well be in the future for handling fake playthroughs of the collection editor for players; this object even makes it easy to pre-setup configurations such as faking certain explorations being completed and seeing how it will look for the user).


no thanks


Code to run as an android application on device.


new PR that refactors HomeController to use MediatR


is it typo ? "log_error" ?


@abakst can you submit this against develop please? (we're just pulling from that into master.)

thanks!


nit: Personally I would like to be able to point this at a destination directory rather than always having to cd into a workspace.


This should be fixed here: https://github.com/Team-Singularity/KernelAdiutor/commit/1b5a5bc8275046282d12cf7dbbfe0584c3587899


My bad, shall be `labelSelector` instead of `podSelector`


it's real ,I am ok


Well I thought you made a mistake, sorry I should ask you first.
I made a little update but I didn't test it yet


Could use more explanation of what this does (beyond "now you have a default cluster")


I'll make a note to investigate the use of `Inflector`.

Good point about this logic eventually belonging in a helper or presenter object. I'm going to leave it for now and address that once everything's been moved to the `MarriageAbroadCalculator`.


s/filterdpods/filteredPods/


Would belpful to add brief explanation of what is contained in each of these locations


ds.Result{Code: cmd.ErrorDatapoolNotExits，
code没必要去掉


In the tests that you have here, the retain never has to go through RCIdentity for pairing purposes. This needs tests for the case where the retain is the one that has to go through RCIdentity as well.


Here, I think there's no need to call `delay_response!` because the controller will respond with `:ok` if you _don't_ call `respond!`


Actually, I believe that we may already have these tests but the FileCheck lines may have been insufficient.

Look at arcsequenceopts_rcidentityanalysis.sil. Either way these tests should be in that file.


s/if node/If node/


Here, I'm going to create a new error, `Houston::Slack::NotInChannel` or something like that.


nit: s/you'll/you will


I guess this is still a work in progress. How do you envision this cred_srp classes being used? Do you see them used internally in bonafide? Or somehow being used from a bonafide user? Or is just a proof of concept on how a cred module will look like?


same with 3 events


```
s/checks/Checks/
```


So does this work at all or is it functionally equivalent to "launch a new cluster"? If that's the case, lets not ship with this option enabled until we have a supported upgrade path.


indent


inline


ok fine you win ok. i still have my spare copy to override though.


In general I can't see this being safe unless you're wiping all state (see same comment as above).


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


:clap: that's the spirit! Now, woudn't this affect captive portal users?


:cool:


Syntax error, colon expected after if statement


It looks like the problem may have already been fixed on develop, depending on what `pprintTidy` does


コミットの単位についてですが、理想的には
１つの修正の単位、１つの機能追加の単位としたほうが望ましいです。
後からコミットログを見た時に見やすいですし。

従って、本コミットのコメント文追加修正は、
それぞれの↓に含めてしまったほうが良いかと思いますが、いかがでしょうか。

   ofctl_rest: Support port number in get_port_stats()     ※コミットコメント修正
   ofctl_rest: Support group id in get_group_stats()       ※コミットコメント修正
   ofctl_rest: Support meter id in get_meter_stats()       ※コミットコメント修正
   ofctl_rest: Support meter id in get_meter_config()      ※コミットコメント修正


I haven't hosted the binary yet. It's kind of a TODO:


this is teh leetness right here


Alias is simpler but we'll never be able to use https.  I tried setting up cloudfront with an amazon no-cost cert and it works with browsers but not java.  Hopefully java will get its https shit together before too long, though.


.............BUT YOU JUST COMMITTED THIS KEY


This is really bad. It also breaks compatibility with non-gentoo distributions like exherbo.


Those parameters are required so we have to declare them in our manifest even though we don't care.  If the client wants to use a different value they need to add the override.


This change accidentally will return all users, not just external users with a rep:externalID property. The result of `getIdentityRef(auth)` is not checked for null anymore.


Thanks for adding the sound filtering and the other enhancements. I can definitely hear the difference in both dos and win95 (such as with the sbpro/16/ess drivers). The sound is smoother and without abrupt transition, particularly apparent in music generated by the sb/ess emulation. I didn't know it would be possible to improve the audio further or that the filtering can be reasonably adapted to code.


Nit: import ordering: stdlib, remote, local


NOTE source code level 8 is implemented, java 8 mandatory


many thanks.


I add a note of using `-Dfreenect2_DIR=` in https://github.com/OpenKinect/libfreenect2/commit/680d934374060c3fbe197c6d99fa4212801a04cd.

This could be personal opinion but I don't want to let user sudo everything and pollute their /usr/local. If you feel it's justified to really install to /usr/local, I could revert the instructions back to /usr/local.


Essa alternativa apenas foi usada para que a função voltasse a funcionar.

O problema principal acontece com a incapacidade do comando ``` file ``` dentro do ``` zzencoding ``` identificar a codificação correta, e impede que o comando ``` iconv ``` dento de ``` zzutf8 ``` faça a conversão correta, gerando o erro.

A melhor solução será corrigir esse problema no ``` zzencoding ```, ``` zzutf8 ``` ou principalmente no ``` zzfeed ```.


Btw, here's a fixed build: http://bit.ly/1XLwTwQ


Nah, we actually strip it out from the dependencies when preparing the
plain cjs package. (async-es depends on just lodash-es as well).
On Feb 23, 2016 5:57 PM, "John-David Dalton" <notifications@github.com>
wrote:

> Should lodash-es be a devDep?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/caolan/async/commit/3d1781cbb9a215b30fc03c2135309fe3579e5689#commitcomment-16286210>
> .
>


なるほどねぇ。

ちなみに、なんでこれが以前のdef rootの中にある状態だとうまくいかないの？
rootの中に同じような宣言してあげてるだけなのに。


QuickFixIDD-Memos (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2119 of 2119 tests passed, 0 test warnings)

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


No, you should use `godep save ./...` and v54+ . With v54+ and the newer resolver (which was already in) you should get only the stuff you need in `vendor/`. Similarly, once you re-save your deps with v54+, to remove an "extras" that could have creeped in you should also be able to use `go install ./...` because everything needed would be included in vendor. And since we don't save the tests for dependencies `go test ./...` should also work.


Personally, I would not use `assert_difference` here. It introduces coupling between this UI test and the underlying model implementation. It's definitely not a huge deal, but I think it's unnecessary because we already go back to the image index and assert that the image is not present.


Will the published package have a dep on either?


Error Exe?


done here instead: https://github.com/epantry/epantry/commit/687bb18ff89145860d43932e0655d7c81d3fecc7

i merged your change into the master branch


ok! didn't know that /#/ was a react thing too!


Thanks! I will be adding a lowpass filter next to supplement the slew rate filtering when I have time tonight.


This breaks windows.


For cordova:?? --> should  be For Android ?


It's not name or SHA1 or tag. It's the neutron's pbr version got via "pbr.version.VersionInfo('neutron')"


https://github.com/c5t/Current/pull/390 :+1: Thanks!


Changed in https://github.com/AppFolioOnboarding/Li_Image_Share/commit/f09dfa81cff77b661a9c848fb32db3f2acb9b4ca


@bobzoller I believe this line breaks source maps for coffeescript and babel, 'cause `Error.prepareStackTrace` is last in wins :(


修改好，请接收。


not use this test app on windows.


d


why keep this rtcore set separate from android part ?


@pragmaticivan thanks :+1:


I was going to test WoE:TE stuff this weekend. I've been busy once I get home from work so I haven't had much free time to test it yet. I saw @Atemo removed the branch and figured he had some plans/reasoning for it.


You're right. That was my mistake.


@tmd45 から、次の「お気づきかと思いますが、全てメールで通知されます。」がお気づきではないという指摘を受けて。そもそも h4 の css が見づらいという問題が大きいのですが。


looks like you forgot npm again…


Yep, it will depend on the corresponding version of lodash.


return?


没看见修复代码？


113837367-Fanger-PMV-Equation-Bug (RKStrand) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1563 of 1563 tests passed, 0 test warnings)

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


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


teste


I'm going to update the messages for the commits that use `quacks_like` so that I can get this merged. I'll happily remove `quacks_like` from my tests if we don't think it's a good idea.


#!/usr/bin/env python
#_*_ coding:utf-8 _*_
__auther__='QianMo'

from sys import argv

a=[1,2,3,4,5,64,75,7,42,4,63,6,2,35,2,45,24,52,52,5,]    #  有一个列表

def check_service():
  if len(argv) != 3:
    print "请在脚本后面输入两个数字供脚本比较!" ;exit()

  if argv[1].isdigit() and argv[2].isdigit():
    pass
  else:
    print "请输入数字" ;exit()

def number_comparison():
  check_service()
  num1,num2=argv[1],argv[2]
  if num1 > num2 :
    print num1
  elif num1 < num2 :
    print num2
  else:
    print "两个数字相同"

number_comparison()


Symlink for using on terminal:

```sh
$ subl -v
> Sublime Text Build 3103
```


Thanks. I'll keep that in mind. :)


ttttt


test ok


I think I will need to significantly upgrade my speakers to distinguish the difference between the emulation and the real thing.


Seen.


Fixed in 84bce9392c2dd5786e710a2f879334504365144e


please file a new issue with some reproducible code and rule config. Thanks


Seen


I didn't find a way to make the second placeholder inside the quote. If I wrote, instead `import $1 from '${2:${1/([A-Z])/-\l$1/g}}';`, the cursor fall at the begining of the module name and the module name is not selected.


seems to me that dropping the `//` here would break the canonical file URI scheme in the normalized name if this were ever to be used for something other than `urlparse.urlparse`, no?

according to https://en.wikipedia.org/wiki/File_URI_scheme this form should still be valid even for windows paths (but apparently with a 3rd `/`).


since both branches do the exact same thing, maybe combine them into one boolean expression to eliminate the duplication?

```
if not purl.scheme or (WINDOWS and len(purl.scheme) == 1):
  # In the case of Windows, this is likely a drive letter.
  purl = urlparse(self._normalize(url))
```


how about an assertion message to make the context clearer if this ever failed the assertion?


upOpts isn't defined in this package. I guess it works, but feels a little weird to me.


We will need to support more keywords before class/task/function (like static, local, protected, automatic):

https://github.com/kaushalmodi/.emacs.d/blob/master/setup-files/setup-verilog.el#L395-L400


S.T.D demo for protocol


added with https://github.com/RayZopf/lslforge-raysilent/commit/32e362d02bde659172c22f9abec0c5ac052f0101
thanks for pointing out!


Wow python magic, nice.


QuickFixIDD-Memos (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


I hadn’t seen this before merge … I feel this `singular_with_article` thing is cluttering `entities.json`, and a slightly less elegant error message such as “This is not a collection for entity type $url.” would have done the job, too.


as a heads up I would leave it as a TSTR instead of a WSTR
http://www.codeproject.com/Articles/76252/What-are-TCHAR-WCHAR-LPSTR-LPWSTR-LPCTSTR-etc
it should already support unicode characters


Restricted xD


extraneous else


I think you forgot
// 4096 - cannot be used in zone 8 maps


@neighborhood999 原本這兩行讓所有東西都指到 how-to-back-up-music.html 囧~


There is gvg_te mapflag to add in the docs


nit: why add relative "./" prefix to all paths?


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


Why did you add these castle ? even if they are disabled)


`uri` and `method` are not used. But seeing the implementation of the classes that use this metaclass, Why do we pass `action_name` to `_get_uri` and `_get_method` instead of `uri` and `method`? It looks like `_get_uri` and `_get_method` don't care about the action_name and only use the uri and the method.


I will remove this function


コミットログについて、

This patch adds attributes of the following messages.
以下のメッセージのために属性を追加しました。

とありますが、
下に並んでいるのは

Add the port number for get ports stats.
get_ports_statsのためのポート番号追加。

等の、メッセージではなく
対応そのものの説明に見えます。


MF_GVG_TE_DUNGEON not defined
maybe you want MF_GVG_TE ?


★ `-` 으로 표시한 줄들은 추후 정리 할 필요성이 있음 ★


Serves me right - I only tested that on windows. Windows `pathname2url` includes the `//`, while the linux one does not. I'll look into fixing this with a `urljoin`.


I'm a noob in src so don't blame me if I'm wrong
I think you forgot case MF_GVG_TE


Note this is just a buxfix and does not require a new version.


Add the attribute for message
コミット文ですが、一番タイトルが大事です。
タイトルを見て対応内容が想像できると良いです。
「メッセージのための属性を追加」だと少し分かりづらい気がします。。
過去コミットでいい感じの無いですかね。
https://github.com/shinpeimuraoka/ryu/commits/9e1353d1c2ce1c286fa34c2a4bfc1b861aa45d58/doc/source/app/ofctl_rest.rst


pc_canParticipateSiegeTE(sd->class_) already return 1 or 0
script_pushint(st, pc_canParticipateSiegeTE(sd->class_));


あとは、以前岩瀬さんからコメントあったかもしれませんが、
Add the port number for get ports stats.
Add the group id for get group stats.
Add the meter id for get meter stats.
Add the meter id for get meter config.
は
 - Add the port number for get ports stats.
 - Add the group id for get group stats.
 - Add the meter id for get meter stats.
 - Add the meter id for get meter config.
とかしほうが列挙であることが明確なので見やすいと思います。


Okay I don't seem to be able to open an issue on your pull, but I just wanted to point out that on line 91 of init, you are calling `PrecacheParticleSystem("bday_confetti")` which is supposed to be called on both client side and server side to cache the particle. So I think you need to move that out of there and just stick it into shared.lua xD I haven't tested anything else out but otherwise it seems good. As you have probably noticed I have been short on time unfortunately.


`_get_uri` and `_get_method` are exactly the same in the subclases of `BaseProvider` we could move them here.


I'm not sure if I see the point of creating `_get_method`, as at the end the only thing that it does is to return `method`. Maybe this `get_%s_method` methods could return them directly.


that works?  Robot needs two spaces or a tab to distinguish, seems like this would force "${jsondata["data"]["value"]} > 0" to be a single value, not two things to compare


You can just use

```yml
language: node_js
node_js:
  - "stable"
```


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


Jose,

 I'll take a look through and fix the exceptions and see if Files.copy fits.

 Thanks,
   Matt

On Tue, Feb 23, 2016 at 6:44 PM, Jose Manuel Duarte <
notifications@github.com> wrote:

> Again check for nulls explicitely
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/biojava/biojava/commit/148519fc9e8a02289a3d8a78090fe5d65d3e0cc6#commitcomment-16285107>
> .
>

--
Matt Larson, PhD
Madison, WI  53705 U.S.A.


Aiiii Sorry.
Will fix the earlier commits later.


why not do `argumenrs = '--noout {filename}` outside `run()` ?


sqrtの計算をするよりもshop.rangeを二乗したほうが速いはず


I have been so foolish,hahaha.  Thanks

2016-02-22 18:15 GMT+08:00 Yen Chi Hsuan <notifications@github.com>:

> Simply:
>
> youtube-dl http://www.iqiyi.com/a_19rrhb8ce1.html
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rg3/youtube-dl/commit/73f9c2867df5048e78c6f60d2fc1d9691edb4b2d#commitcomment-16236118>
> .
>


メモ：経度・緯度はkmを度に直す定数


You need a comma inside the rounded brackets - otherwise `valid_files=["<node>hey</node>"]` and when it iterates over it, each character will be taken as a file


It's okay--that line isn't visible in-game anyway! Also, I thought it sounded snarky enough to be informal as narrative slang.


more more love


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


criação do repositório


调整说明


io.CopyBuffer()?


``temp1
"ImageData"
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=4&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR failed loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=4&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/active_ping?channel=p_100011123532687&part…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/AllPhotosAppCollectionPag…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=1v&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/WebEgoPane?__pc=EXP1%3ADE…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=1t&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/PlacesAppSectionPagelet?_…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=1x&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/SportsAppSectionPagelet?_…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=1z&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/ManualCurationAppSectionP…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=21&__rev=2196638".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=4&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/InHouseOGAppSectionPagele…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=23&__rev=2196638".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=0&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://www.facebook.com/notifications/sync/?lastSync=1456274465&__pc=EXP1%…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=24&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/InHouseOGAppSectionPagele…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=27&__rev=2196638".
XHR finished loading: POST "https://www.facebook.com/ajax/mercury/thread_sync.php?__pc=EXP1%3ADEFAULT".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/InHouseOGAppSectionPagele…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=2a&__rev=2196638".
XHR finished loading: POST "https://www.facebook.com/ajax/photos/photo/tags/tags_init.php?__pc=EXP1%3ADEFAULT&__a=1".
XHR finished loading: POST "https://www.facebook.com/ajax/proxy.php?__pc=EXP1%3ADEFAULT".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/WebEgoPane?__pc=EXP1%3ADE…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=2b&__rev=2196638".
XHR finished loading: GET "https://www.facebook.com/ajax/pagelet/generic.php/NotesAppSectionPagelet?__…y3fh8WUJkiVWxeUlxi9Hx2bwTADDBBwDKqcBGqewIUsz8gCxm5k&__req=2h&__rev=2196638".
GET https://www.facebook.com/sw?s=push net::ERR_FILE_EXISTS
GET https://www.facebook.com/sw?s=push net::ERR_FILE_EXISTS
u2028
Uncaught ReferenceError: u2028 is not defined(…)
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=2&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: POST "https://www.facebook.com/ajax/bz".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=2&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/active_ping?channel=p_100011123532687&part…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=2&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=2&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=3&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: POST "https://www.facebook.com/ajax/bz".
XHR finished loading: GET "https://0-edge-chat.facebook.com/active_ping?channel=p_100011123532687&part…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=3&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=3&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=3&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=4&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: POST "https://www.facebook.com/ajax/bz".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: POST "https://www.facebook.com/ajax/webstorage/process_keys/?__pc=EXP1%3ADEFAULT".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=5&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=6&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/active_ping?channel=p_100011123532687&part…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=6&parti…&viewer_uid=100011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy".
XHR finished loading: GET "https://0-edge-chat.facebook.com/pull?channel=p_100011123532687&seq=6&parti…00011123532687&sticky_token=99&sticky_pool=atn1c08_chat-proxy&state=active".


Possibly fixed remove chat error (untested)


I would lean toward this returning encoded data from the buffer rather than updating the buffer in-place.


Are usernames allowed to start with `-` or `_`?


doubtful, that was just a shuffling around of stuff, no serious
modifications were made. `pprintTidy` is the only method in the type class
now.

On Tue, Feb 23, 2016 at 6:10 PM, Alexander Bakst <notifications@github.com>
wrote:

> It looks like the problem may have already been fixed on develop,
> depending on what pprintTidy does
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/abakst/liquid-fixpoint/commit/0fc98459e912998f85ef422971c907081f927d9b#commitcomment-16286370>
> .
>


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.34](http://build.clear-measure.com/viewLog.html?buildId=6088&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


Скинул файл на форуме в ЛС


I was just trying to adhere to http://coala.readthedocs.org/en/latest/Users/Tutorials/Linter_Bears.html#finished-bear

I will make this change.


OK then I will test against develop and resubmit if necessary.


Sugestão: Trocaria aqui 'padrões' por 'padrão' ;)


how do u slove this problem?


Hm, do you add `\0` to the start of the buffer?


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


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


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


This is unnecessary.


hello wheres me


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


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


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


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


The printer flags and generator flags are both adding the `output` option at this point which will cause this to panic at runtime.


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


@lpereira I talked to @anselmolsm about it.

It seems it would be better to mention this in the file reader.


TeamCity Open-source Libraries :: BlogMonster :: CI [Build 2.0.61.0](http://build01:8888/viewLog.html?buildId=14386&buildTypeId=BlogMonster_CI) outcome was **SUCCESS**
Summary: Tests passed: 13 Build time: 00:00:28


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


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


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


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


you can write :
```
std::map<std::string,std::pair<ModelMaterial,ModelMaterial> > mybimat;
mybimat["marker"]=std::make_pair( mymat1,mymat2);
for(auto const &it: mybimat)
 {
     auto const& mat1 = mymat.second.first;
     auto const& mat2 = mymat.second.second,
     K.on(_range=markedelements(mesh,it.first),
          _expr=( vf::chi(idv(T)>mat1.Tliq()) * mat1.Kl()
                  + vf::chi(idv(T)<mat2.Tsol()) * mat2.Ks() ) );
 }
```
If your are anisotropic, you need to introduce 2*6 parameters for the conductivity, idem for elasticity... Also we need to add parameters defined as a function so we duplicate also by two.
Also I will change  Tliq and Tsol by the parameters Tmin and Tmax for the temperature range where the model is defined.
It's just a proposition, maybe I didn't see something in your complex model.


Thank you!


Great, thank you, @thierryvolpiatto


Can combine two lines above


That looks great, feel free to commit the changes or I could do that in the next commit too... Thanks!


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


Just being curious, what is this property for?


@athanatos Incorporated your comments..


@ramilexe, why? Would you please explain what benefit I would get from using `array_merge` over using the union operator here?

I chose `+` (and the operand order) quite deliberately, as I don't care about numeric keys and it maintains the same defaults, but allows them to be overwritten.


Ah, right. Missed that. It just calls `g_hash_table_unref`, but since you registered `g_free` as the destroy func for keys and values, this will destroy everything if the ref count goes to 0.

https://developer.gnome.org/glib/stable/glib-Hash-Tables.html#g-hash-table-unref


May want to convert to uppercase, and perhaps check that all characters are hex.


I have updated the release number to 1.0.0 in preparation of a release.
https://github.com/plone/plone.schema/commit/4c94d8b466a24589ff8b87c1c9059129458274ea

I have not released it yet, because our Jenkins server currently has a failure. (I do not mean the failure linked in a previous comment. The current failure has got nothing to do with this package).


Perhaps pop up a message on error saying what it should look like


@meriac Will do! This kind of feature was exactly what was missing that lead to considering the tradeoff between "wasting" a static region and having to add boilerplate to every single application. I'll get it moved over to the helloworld example for now.

The execute permission is a copy-paste error from the main flash block. I'll remove it as well.


The following doc(s) have been deployed to the [rachelwhitt](http://rachelwhitt-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/backups/backup-creation](http://rachelwhitt-static-docs.pantheon.io/docs/articles/sites/backups/backup-creation)


Was the reversion of the nanopb commit intentional?


Probably better to say "wifi adapter" instead of "wipi", since it could be other hardware.


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1561 of 1561 tests passed, 0 test warnings)

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


@yehudasa In theory we could do that. However, I wasn't 100% sure we don't introduce a regression here, so I added only this single check. Responses other than ```401``` will be handled in exactly the same way as currently.


Sebastian Wiesner <notifications@github.com> writes:

> Just being curious, what is this property for?

Which one ?

--
Thierry


I like these new instructions on setting up the DIspatch module on your swift-toolchain directory! Small nit, though- I think we should tell them to run 'make install' instead of 'sudo make install' because the installation happens on the user's directory not system-level. In this case, it creates some libraries in the user's directory owned by root.


No sabia que dejaba rastros el sublime, para que lo pusieras en gitignore lol
Que tipo de cache deja?


never give up ;)


Thanks for the name suggestion; updated. I added a comment why it exists. I agree that usually one source of truth is preferred, but I think these domain objects allow us to do slightly more efficient patterns like this, since maintaining two sources of truth is easy here (due to it being well-isolated and hidden behind the domain object interface).


Got it!

On Wed, Feb 24, 2016 at 2:48 PM, Enrico Sada <notifications@github.com>
wrote:

> @ademar <https://github.com/ademar> add the Utils/CultureInfoCache.fs
> file in project.json too, under compileFiles
> The file list is inside project.json
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SuaveIO/suave/commit/e82a42a172a00024d3c2764ab5b1bfd82e086278#commitcomment-16307528>
> .
>

--
Ademar Gonzalez
+1-647-891-3606
http://ademar.name
https://github.com/ademar


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


so complicated for such a simple thing! Good improvement at least even if it needs a reload.


Pour les microdonnées, Open Graph ou les Twitter Cards, le plugin jekyll-seo gère ça très bien.


I think the command is not needed inside `execute_get_input_device_info`, because it's already checked.


Sebastian Wiesner <notifications@github.com> writes:

> Just being curious, what is this property for?

Ah, helm-only, sorry, yes it allow hiding symbols with this property in helm-M-x.

--
Thierry


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


**This file contains unused source code.**
*PyUnusedCodeBear, severity 1, section `python`.* *Related to file `coalib/bearlib/abstractions/Lint.py`, line 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/coalib/bearlib/abstractions/Lint.py
+++ b/coalib/bearlib/abstractions/Lint.py
@@ -4,7 +4,8 @@
 import tempfile

 from coalib.bears.Bear import Bear
-from coalib.misc.Shell import escape_path_argument, run_shell_command
+from coalib.misc.Shell import escape_path_argument
+from coalib.misc.Shell import run_shell_command
 from coalib.results.Diff import Diff
 from coalib.results.Result import Result
 from coalib.results.RESULT_SEVERITY import RESULT_SEVERITY
```


D'après ce que j'ai vu sur le site temporaire en ligne, il fait des doublons :/


Is this the only file affected with wonky line endings?


From what i can work out this is the line i want to play with. Ideally doing something like:
`if (firstPropSeparatedGroup !== secondPropSeparatedGroup && thisGroupsEmptyLineBefore === "always") {`

But from looking at the `expectation` that is passed on L18 it is an array of the groups. Is there anything on `firstPropData` or `secondPropData` that say which index in the array they belong to?


Yeah, it makes sense that sudo isn't needed anymore. We can get rid of it.


Are you sure that `pcmd.cmd` can be compiled?