We need to remove the section on Overload Protection. Talked to Fred and he agreed that we should not advertise this at the moment. The way the things are now, sometimes users will get an error and sometimes the node will crash. There is a ticket to fix this.


Initalizing functions.


Alternative:
https://github.com/Zegeri/Player/commit/49c34d39e2ceecf6df12711bb6a652298d1a252b


Updated per suggestion, please take another look. Thanks.


@seemaj, please move this discussion to the salient [PR](https://github.com/basho/basho_docs/pull/1629).
This is a deprecated commit.


Here's a little improvement for that section:

``` ruby
unless Post.exists?(title: "This is a test string")
  Post.create!(…)
end
```

In Ruby, `unless` is the logical reverse operation of `if`.


right. will submit PR soonish


This will only loop once, the loop can be removed.


We're still in 2015 :)


should be `if n <= uint64(j) {`


@schism: i am not going to talk philosophically here, but for me it's just so absurd to be pissed off for just a copy of files in a public platform that mostly hosts open source projects. Personally if someone would fork or copy my projects i would be honored and happy, since i feel that i have done something good that can help somebody else and not only me. credits or not.

This repository is just for testing, nothing is going on production and noone is judging your work, i think @necrose99 was just curious, and its normal and perfectly human for me. I bet everyone here has at least one time copied some file on github

I just hope that this is just a big misunderstanding, i would be maybe pissed off by the commit message at most.

Good luck if you want to keep private your repository on github without paying, i think the only way would be encrypting them.

bye


A inveja é uma merda :+1:


It wasn't for a new day, it was the same day.

Lori Hill
Sent from my iPhone

> On Apr 10, 2015, at 11:16 AM, Chris Lim <notifications@github.com> wrote:
>
> I wonder why it got stuck in the first place.
> If it ran for a new day, it would send the kModelRefresh notification.
> That would hit the refreshPageView selector, and the last step of refreshPageView selector would call hideLoading...
>
> —
> Reply to this email directly or view it on GitHub.
>


+1 to Kate

On Fri, Apr 10, 2015, 11:38 AM Katherine Whitlock <notifications@github.com>
wrote:

> I'm always super skeptical of commits that are named like this...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/rit-sse/crazy-train/commit/ebb897d0c0f3225a2adc28ab9c71913fc939e657#commitcomment-10671764>
> .
>


Hello those optimizations are deprecated  https://android.googlesource.com/platform/art/+/2e879694544744436a595eb30cd0b8ad4580c9db%5E!/

So you can update those ;)


@cptvitamin, thanks for pointing me in the right direction regarding the accessibility issue, I completely misunderstood your first comment regarding the issue. Not sure, though, if we want to provide course number to the screen reader or not (please see line 27).


chammer commits igendwie like? ;)


Thank you Kartik :D


I would keep a class of utility for logging. Are you sure that you can not separate the logger?


No eclipse, clique em File->New->UIMA Ruta project, e dê um nome ao projeto. Será criado um projeto vazio. Copie um arquivo de saída do cogroo-uima para a pasta de entrada de seu projeto (*.xmi). Copie os arquivos de cogroo-uima/descriptors para a pasta descriptors. Com a pasta script selecionada, clique File->new->Ruta script e crie o script ruta. Rode o script, selecione como typesystem o arquivo TypeSystem.xml e vá em frente. A saída será um arquivo .xmi na pasta output.


look this is her husband we have been together for 9 years with 2 kids and I would like you to get hold of me at 502 203 8602


think this is also done in kiwix/tasks/main.yml or was it removed

I checked and the structure of the 64 and 32 bit files is the same


I think it should be easy enough -- just need to figure out the most appropriate place the logic would fit.


Would this functionality make more sense to be part of `DestinationType()` below, which does something similar for Java?


@pinaf I think that checking `Thread.currentThread().isInterrupted()` is an anti-pattern in Java. you should not do this. instead, you should call methods like `TimeUnit.sleep`, which is exactly what you're suggesting.


Probably because I didn't know about it.  Will update the code.


Your code causes all these unnecessary casts to be generated in Java.


That's right!

I will try it out

thank for all the insights

Dan Williams, SIQ Agile Coach
CSM, CSP, PMP
Cell: 425-877-6333
________________________________
From: Jean-Denis Vauguet <notifications@github.com>
Sent: Friday, April 10, 2015 12:08 PM
To: danielwilliams12/new_options
Cc: Dan Williams
Subject: Re: [new_options] Updated Seed data file (d92e147)

Here's a little improvement for that section:

unless Post.exists?(title: "This is a test string")
  Post.create!(...)
end

In Ruby, unless is the logical reverse operation of if.

-
Reply to this email directly or view it on GitHub<https://github.com/danielwilliams12/new_options/commit/d92e147ecfad2ba06938e17afe112b713ba4ac08#commitcomment-10672207>.


Could you explain how this could be better than my simple fix in #154?


just added pending numbers


I tried this out with the technique recommended here: http://dgrid.io/tutorials/0.4/single_query/

If subsequent calls to `refresh` return different results, the dgrid will display _all_ results at the same time, including those from previous calls to `refresh`.


Shh it's so close to working.


We need to rid of Russian words. Use Ukrainian ;)


With mine you don't have to turn your screen off/on after boot up, any reason for doing this?


let's avoid outright swearing in a public OSS repo :)


looks fine to me; however I'd like Ken Muchison to give the final ok since he's the recur guru.


Mostly it's misunderstanding on the part of every single one of the additional commenters.

I'm simply asking for future politeness and cooperation in the spirit of open-source collaboration.  Yes, I was mildly put off by the unintentional "crap" connotation and was frustrated that I wasn't granted the opportunity to have input on how my work was used, but I'm not "pissed".  Pissed would be taking the repo _back_ private, or throwing around a lot of angry words.

Politeness and being considerate.


This is correct, should return the old bucket because it's full if addEvent() returns false


I'll fix this in a subsequent PR by first adding tests for it.  Right now we still return a result for invalid inputs, but I'm hoping to clean this up a bit.


I prefer fixing the issues using code over hard-coded entries in a file whenever possible. These two particular cases are easy to fix by code, and I don't need to worry if the author of gpairs removes the `gpairs: ` prefix in the future, or the KernSmooth author adds the backslash before `&`.


You shouldn't you should keep the logger in the class that's gonna use it.


What do you mean? Not sure I understand the question, but g_i is the group assigned to item i, and phi_{g} is the parameter assigned to group g. If two items i and j are assigned the same group, then g_i = g_j and phi_{g_i} = phi_{g_j}.

The confusion might be in my indexing notation (the way I indexed all the way up to n in both g_1, ..., g_n and phi_1, ..., phi_n) :(


It's clearer to rename this to `$_requiredArguments` or `$_requiredArgs`.


Thanks, I'll try it


I was having issues getting yours in on the latest Verizon source and it
actually compile and work as when I did finally get it in, the whetstone
toggle to turn it on and off didn't actually do anything. Was strapped for
time and this was easier to implement. Haven't had a chance to go back and
mess with it yet.
On Apr 10, 2015 2:23 PM, "savoca" <notifications@github.com> wrote:

> With mine you don't have to turn your screen off/on after boot up, any
> reason for doing this?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/holyangel/LGG3/commit/bbcac184b7b1e7a020136a549e7b4580756330c4#commitcomment-10672387>
> .
>


This should be "data" not "variables"


prac


Yes, it needed to be changed to a log instead of an error. However, I also changed it so that it will get a minimum of one row to render. See the commit for 1.2.1.


:heart: Thx !


That's my confusion. Indexing up to `n` seems to indicate a 1-1 correspondences between `g`'s and `\phi`s that isn't quite right.


why this 'harsh 2' is added here ?


I would say "arguments" rather than "parameters". It's not really the same.

From [MSDN](https://msdn.microsoft.com/en-us/library/9kewt1b3.aspx):
> To communicate this information to the procedure, the procedure defines a parameter, and the calling code passes an argument to that parameter. You can think of the **p**arameter as a **p**arking space and the **a**rgument as an **a**utomobile.


'harsh 3' should come in separate file with 3,6,9,etc.


Isn't just adding (out) enough for this one?


just check harsh2 and harsh3 comments


:bomb:


Ej molek, bo ja dorzucilem jakies syfki od db, moglbys zrobic cos zeby McPlayer byl zapisywany do bazy (sqlite najlepiej, albo jakis plik, byle nie mysql).


Ah I remember big merge conflicts with that too. Try this one https://github.com/savoca/furnace-g3/commit/025fc7958e5935990b1713368567225e785af2de as it's based on LG's 5.x source.


Yeah, definitely need to lock before checking. I'm pretty sure the first approach is best.


@julianrichen Could you make this more readable?


The code looks good, but I don't really see the impact in the wall claiming rate so far. As far as I can tell my workers claim walls at least as fast as they claim ground tiles (I even wonder if it's not faster, it's like they touch the wall and it's claimed, and then they linger around a bit before moving someplace else).


Fix confirmed :)


This line is breaking the purpose of the AsyncTask, the main thread shall never be blocked in order to avoid freezing the UI. In addition a long wait in this thread will generate an "Application is not responding" message.


Open CV link is here!!!!! http://stackoverflow.com/questions/27406303/opencv-in-android-studio


Yeah this is just on minimising, the current code for margin bottom for the
composer when its open seems to work as intended so no need to remove it

On Fri, 10 Apr 2015 13:23 Barış Soner Uşaklı <notifications@github.com>
wrote:

> There should be code that adds a margin-bottom when you open the composer
> so you can see the last post when you are typing.
>
> If we just remove this the margin-bottom will stay on the topic.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/NodeBB/NodeBB/commit/59909894fcba034e31a7d799c19ff8e683feac93#commitcomment-10670436>
> .
>


11*3 = 33 :-)


Thanks =D


I approve of case insensitivity, I can't think of a reason or use case to keep them case sensitive in our scope and sensitivity adds unnecessary complexity.


to keep in order basically.


ok


Personally, I don't care which variant is used, although I think in German grammar there is no discussion about it and the comma before or/and is verboten. So we're gonna roll with that, it also involves the least effort since this commit won't need to be changed again.


Debug prints, pls.


buradan yukarısı test için var. denersen diye. ben test ettim gerçi ama. class lı halini de pull request ettim zaten


+!!!!!!!!!!!!!!!!!!!! ->->->-> [o.O] <-<-<-<- !!!!!!!!!!!!!!!!! (http://m.memegen.com/aysu72.jpg)


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

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


Good catch, missed that one!


Set the value to 0.1 and you will see ;-)


Three reasons:

1. Matching column names.
2. Matching Sync record fields, where that applies. (GUID is "id").
3. Matching the majority of code in the tree, which uses "id" and "guid" unless they follow as camelGUID:.


How does one select the player able to do the config? Is this only the backend, and the frontend has yet to be implemented?


Even though we don't use this for filtering anymore, I think it's still valuable for the sheriffs when they may need to update the regular expressions for new jobs (or to refine things).  Some day we can remove *all* references to buildernames and life will be a cleaner place.


Matching record fields.


I think if you look at the source code of [array_rand()](https://github.com/php/php-src/blob/94bea670de40c937d5926839d52dc262750921f4/ext/standard/array.c#L4590) you'll see that it does a foreach loop through the array keys each time. You might want to use the fact that you know this is a numeric array to just pick a random number between 0 and `count($this->ids)`.


>Hmm, I kind of want `dist/` in git as that's the downloadable browser bundles.

Isn't that `build`?


Yeah, I asked you a question on IRC before I wrote this, and didn't get a reply, so I rolled on. We can make a `DatabaseError` struct.


Nope, it's all kinds of crazy.


I was thinking of keeping the sign-in form even once they've set it up, in case they change accounts or something or want to re-do if their token get's changed with a different server install maybe?

Or I was thinking maybe hide it but give them a button that's part of the "plexAuthEnabled" div that brings the form back up if they want to re-do it. That way keeping it clean for those that are setup and don't need it but letting those that are setup but need it easy access to it?


Adding a comma for the last item as well is better for version control.


You mean in the mock? It should be.


I'm not sure if JURI::base() is the right server (it will not get the CDN domain name, but the main server one) ?


Sure.


Indeed the difference is really clear with 0.1 :-P
I'd propose to lower the default value to 0.5 as a first balancing test.


nevermind, this appears to work even with the user isnt' root.


Yes! I self-corrected on this a little after this was written.


Which problem does removing these lines solve ?


Awesome! Thanks for the pull.


Nope... FreeCAD ATM doesn't read/write IfcMaterials (only IfcSurfaceStyles). They are all lost. But this is a very good test case to implement it. I'll work on it this WE. We're doing pretty big steps forward!


Hey - what's the reasoning behind this? Since we're using LDAP.search to confirm that LDAP usernames are real in apps, this breaks that, but we can implement that a different way if needed.


I think I tried this and failed (you can do it with class functions, but I think instances are too stupid to pretend to use `.apply`), but I'll give it another shot.


I chose the cheap route for now 13e8c84ac5993f985c6aff043c36634420108ab6 :)


We null-check above. I'll add a `let`.


Right. #getting-old. Nevermind me! ;)


So why not add the other dependencies (jquery, bootstrap, etc...)?

Also for the versions there is there a way we can get the names without versions and match so when we bump versions it doesn't require updating this file?  That may be an enhancement.


Sorry! It was definitely not intentional. I forgot to pull before making changes and unfortunately relied on my hazy memory of what to do to fix it. Sarah's sorted me out on what to do in future, so it shouldn't happen again.


I'm not sure. I just got it from the old code:
https://github.com/joomla-extensions/install-from-web-server/commit/e66e9c60c986318ce317b5fb4cf897b8fc59814b


@julianrichen There is a typo in `_addStructureContentsParamters()`. It must be param[e]ters.


You'd have to ask @emavro . It looks like it's not needed, based on the deduction of https://github.com/joomla-extensions/install-from-web-server/commit/e66e9c60c986318ce317b5fb4cf897b8fc59814b


Hey @c4milo – we suspect this is an upstream issue that's been resolved, can you help us confirm?

Try running:

    $ make updatedeps
    $ make dev

and let us know if you can still reproduce this? Thanks!


`n >> 1 == n // 2`, that was the problem


It looks like the CSS wasn't needed, so the file was removed from being called (but the CSS remained in there).

Now we need the file, but only the CSS at the bottom (that I recently added for a fix).

So the old CSS that isn't needed can be removed.

Hope that makes sense. Let me know if you have any questions and I'll be glad to answer.


Where is this problem? It's available for sure in VS2008, VS docs says it's available since VS2005:
https://msdn.microsoft.com/en-us/library/41w3sh1c%28v=vs.80%29.aspx


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 297](http://192.168.30.225:8111/viewLog.html?buildId=1110&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


space after semicolon?


attempt_more_exception_reduction (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (335 of 335 tests passed)

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


Update: the same thing happens when using an OnDemandGrid instead.


@teodoryantcheff ,

this looks fine, but can you add an example how would you using this code?


And maybe add "(there can be gaps in the numbering)". This is certainly something people can be confused by.


Do you know if cert_marginalen2.pem is still required? Can it be removed?


Ah, yes that is likely wise. I like the second option.


The docs are obviously not ready yet and will change, but in the future this should get more specific.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 298](http://192.168.30.225:8111/viewLog.html?buildId=1111&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) is now running


Yeah I removed all references to ostruct and just use hashes everywhere since they serialize to JSON a lot better


SQLite can be added soon as well.


definitely better


Didn't see much worth in coupling this with the other test shell wrapper we have (mostly because of etcd), can if you feel it's worth it


Good. Thanks. Now I got it.
Then `eventsBySource.remove(source)` is redundant anyway: `put` overrides and old value.
Anyway we should think more here to make it robust for multi-threaded publishers, e.g. we don't need the `!bucket.addEvent(event)` check, if the `bucket` is really new from the previous code block


I'm sure there's a cleaner way to do this, plan to dig around a little. I just want to avoid the noise from the master services alltogether.


@julianrichen Isn't EditorConfig working? Not a big deal, but there should be an empty line at the end.


Thanks. I'll give that a shot in a little bit.


Exatamente. Se as funcionalidades descritas nesse documento podem ser usado por pessoas com perfil distinto, restringindo as ações permitidas e os resultados obtidos destas, significa que se trata de 2 features, devendo existir um arquivo .feature para cada uma delas.
Para nunca esquecer... a Feature descreve a necessidade de um usuário em um dado papel...
As a <papel>... I want to <feature>... So that <valor de negócio/benefício>...
Inclusive, nesse arquivo a descrição da feature remete ao papel do professor, não fazendo sentido misturar aqui cenários envolvendo qualquer outro papel.


This seems to be the only unresolved commit and it's a good discussion. I'd like to resolve this.
@ThirteenFish
(9:41:38 PM) jack_haydn: thirteen_fish: is there no concept of private/public vars in python?
(9:41:57 PM) jack_haydn: i mean, having a var defined and initialized on a class level doesn't necessarily mean you want to use it outside that class, if you tag it private


Ok, ogarnę jutro ;)


Thanks! Are you reviewing that stuff?


lol your the comment for windows users!
Why don't you setup mariadb?


plot1, plot2,plot3,plot4


Nice! Will do, thanks!
On Apr 10, 2015 2:33 PM, "savoca" <notifications@github.com> wrote:

> Ah I remember big merge conflicts with that too. Try this one
> savoca/furnace-g3@025fc79
> <https://github.com/savoca/furnace-g3/commit/025fc7958e5935990b1713368567225e785af2de>
> as it's based on LG's 5.x source.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/holyangel/LGG3/commit/bbcac184b7b1e7a020136a549e7b4580756330c4#commitcomment-10672515>
> .
>


Aiee! Some time ago, my enterprise takes part of Hyper-V private beta test. I hope that we can test this too. Go! Go! @coreysa @taylorb-microsoft :-)


86046200_CondenserToZone_New (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1638 of 1640 tests passed)

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


emita -> o sistema emite


`g->Data4[0]`

or

`g.Data4[0]` ?


Tobias: As per my understanding on new ckeditor the following things also needed.
   In Old version I had worked only `.cke_wysiwyg_div` but new version I
    noticed that when ckeditor opened its loading `.cke_wysiwyg_frame, .cke_wysiwyg_div`.
    So we need that fix here too.
    So far my understanding main cause is in webkit based browser like
    chrome and safari when ckeditor is loading then a inline css is going like
    element.style {
    width: 0px;
    height: 100%;
    }
    As the width is getting 0px, so the ckeditor is being grayed out.
    Relatively I am guessing that should resolve as the same fix I did on test server
    and no one has complained after that.
presets.css
--------------
.cke_wysiwyg_frame, .cke_wysiwyg_div
{
	background-color: #fff;
	width: 100% !important;
}

I had fixed that but I did notice that I have no permission to push. If possible please change this and push again.
Thanks


@Tyestor it's referring to the change in https://github.com/UniversalMediaServer/UniversalMediaServer/commit/c561b6c0fe844c1c123c325b47e8d90d604efd81 which enables UPnP detection for Android


Oops I forgot to make these changes in the merge. I've now done it in a separate commit: 7a884009af6ab7b60f3a27f3c9fe998226db1a87


@Simn What error did you get? I'm having the opposite result that having these lines will result in error:
```
echo let version_extra = Some " (git build development @ v3.1.0-2995-gc4f1b4a) " > version.ml
ocamlopt.opt -g -I libs/extlib -I libs/extc -I libs/neko -I libs/javalib -I libs/ziplib -I libs/swflib -I libs/xml-light -I libs/ttflib -I libs/ilib -I libs/objsize -c version.ml
File "version.ml", line 1, characters 54-57:
Error: Syntax error: operator expected.
Makefile:179: recipe for target 'version.cmx' failed
make: *** [version.cmx] Error 2
```

My version of make:
```
$ make -version
GNU Make 4.0
Built for i686-pc-cygwin
Copyright (C) 1988-2013 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.
```


OSPF checksum is not computed so other OSPF routers drop the hello. RFC 2328 A.3.1


DID YOU SERIOUSLY UPLOAD A PRIVATE KEY HERE


為啥改這樣不會 Error?


yeah, that corrects a grammatical mistake.


thanks for the thorough walkthrough!


Rename this `Map` too?


@SubJunk Ah, I see. Thanks again!


Looks like you're using a tab character here.
Spaces-only please :)


If only Matlab had keyword arguments....


just the temp program I made to generate the 8x8 px boards


I have done the task but I am unhappy that authenticated yet not authorized user can delete someone else's product. :cry:


seemingly not, or at least, the trailing-space-highlight i'm using still sees a space.
(it's not the biggest deal)


where do i insert the "now it works" code ? does it have to be added to the "obabp.py" file ? i cant seem to connect to the usb pen in the original code tobabp.py


Prefer single-quoted strings when you don't need string interpolation or special symbols.


Final newline missing.


hii


The original http://www.springframework.org/schema/security/spring-security-2.0.6.xsd url is returning 404 and I can't find another official Springframework or spring.io link to this file. I don't like grepcode reference too, but in the absence of a better one, it works and makes the xml validation error on Eclipse to go away. Any suggestion?

About the pom.xml, I'm new to Git and don't know why it identified the change like that.


第一次提交


Just curious, why there are no builds for 5.4 against single sites and none for 5.2, 5.3, and 5.5 against multisite?


Jake, you are too cool.  This is YOUR repository, and you are treating it like a polite foreign resident.  What can I do to help?


Is this Drop frames with unknown SA.


I remember we agreed on SAI_PORT_LEARN_MODE_PENDING


老师我是4月七号第二节没去逃课的同学之一，这是我的作业


This resolves #10.


I feel like the "strong" or "alive" (perhaps to a lesser extent) names don't tell me anything, because that is my expectation of a normal Haskell object already. I think they are both better choices than `Lifetime` though, because they give my brain some hints on context.

Having the name suggest in some way how these are different from regular weak pointers and ---perhaps more importantly--- foreign pointers, will be difficult, but I think that that is the thing my brain really wants to know.


因為分開跑好像就不會在 venv 裡了，不然每一個 run 都 配 /venv/pttnotifier/bin/activate 我猜應該也會對


这里写错了吧，是 0>=x


I'm also new to git, and it could have been an error with git/eclipse/etc that tagged the line numbers wrong.  As I stated, though, it is not incorrect as the intended change is valid, however from a git history perspective it looks...wonky.

Good as any reason to upgrade versions of Spring. Checking the spring website, just looking at the security directory, http://www.springframework.org/schema/security/, there seem to be a few xsds that might be useful enough.  What happens if you use 2.0 or 2.0.4?  The reference to 2.0.6 is a little meaningless considering the package version of spring-security is 2.0.8.RELEASE.


Don't we want to compare to the sqrt of the variance?


`dow-votes`


You're good Robert, I'm the author but you're the owner, I'll have a pull
request soon:)

On Thursday, April 9, 2015, Robert Plummer <notifications@github.com> wrote:

> Jake, you are too cool. This is YOUR repository, and you are treating it
> like a polite foreign resident. What can I do to help?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jakiestfu/Medium.js/commit/2a40a652285e9e297acabb546e46177f89ef2947#commitcomment-10658506>
> .
>


不行，Cell 也用到 tempGrid


'stylish' is actually the default, so you could leave this line out, and not pass any options.


Good.


Trying to copy Lab10_TrafficLight into GitHub


biu clock is already enabled, it is original code and modified in patch 1d5b0bbf3900ffeaa646ece41c533f065b4480fa


Robot Car


Add a note about how in v1beta3 you must convert your api call from command to args


YW!


Looks fine but a little crude and arbitrary.  What about @marcusps suggestion of considering the angle subtended by the two tangent lines to the noise blob that pass through the origin.   When that angle is greater than the purported phase precision at that step ``k`` then the step is giving no meaningful information.


Another way would be to use `--no-default-keyring --keyring file` with a file containing only the Hashbang Team key so that checking exit status would be sufficient. Well, at least assuming that rest of the configuration is sane.


@jasonLaster Just a heads up, we are merging changes to the test setup here with updates to SpecRunner.html.


I think it's because you didn't throw it, you just created it.  You need to put a "throw" in the front.


See my latest commit.

Can we run the sparql as part of the travis job?


For that matter, it's not our job to compensate for Source Engine rendering issues, it's the job of the team who manages the Garry's Mod codebase.


all private/protected member variables should start with understore'_'
and in this case, these variables should be set as private, not protected.


qweqweqwqwe


This should be dependent on the ACL stage. Could we enable INGRESS_SAMPLE on EGRESS ACL.


screw the points we dont has to be perfect anyhow c;


constructor should initialize all member variables which don't have default constructors
need initialize
+ float _sigma_r;
+ uint32_t _imw;
+ uint32_t _imh;


yes u ur


Could we have a generic action definition:
SAI_ACTION_INGRESS_SAMPLEPACKET_ENABLE and use it for port, acl etc.


Looks good!


I could check what happens if I use 2.0 or 2.0.4, but I think it's pointless because Spring don't check whether the xml is valid or not. This will only affect Eclipse xml validation.


succeeded in pushing


Oh man!!! Thank you about a million times!!

=-Steve


@mroliveira7 me tira uma duvida, as alterações que há no `lex.yy.c` ocorreram pcausa das linhas q vc add no `scannes.l` ou vc meteu elas na mão mesmo?


Also, any assertion library probably has 'truthy' assert, so instead of comparing something to true, you could probably say `test.assert(lilbrary('array', o));`


This resolves #2 for hover, but it you open the link in a new tab, then mouse off of it, the line still appears.  At this point, it might be enough of an edge case to ignore, but I'm going to try a few things first.


how?


plots 1,2,3,4


French is coming... @ikeydoherty :smiley:


develop (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (336 of 336 tests passed)

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


expressar de forma mais concisa em conjunto com a linha anterior, já mudando para when...
When eu faço o upload de uma foto de tamanho menor que "5" para uma questão da prova "Biologia Parte I"


A lot of my users have uppercase in their usernames so I converted the usernames array to lowercase and also the value entered in the form.

There is also some refactoring of the XML phrasing which should make it a bit cleaner.

Also I thought instead of saving the Admin username in the collection, you can access the plex.tv account username?


This isn't required for correctness but it does speed things up.   All the "managers" handle missing data that later show up and do the right thing.

We're really establishing a partial ordering: the snapshots go onto the rules/tag manager queues before we tell the endpoint manager about the snapshot.  This means that, from the rule manager's point of view, the endpoint code can't request rules (via an incref on the rule manager's queue) until after the rules manager has handled the snapshot (because that must already be on the queue).

It's more efficient if the rules manager already has the data rather than creating the profile and then filling it in later.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 297](http://192.168.30.225:8111/viewLog.html?buildId=1110&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 73 Build time: 00:11:28
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


TeamCity Ybackend :: Feodorov/ybackend [Build 133](http://192.168.30.225:8111/viewLog.html?buildId=1112&buildTypeId=Ybackend_FeodorovYbackend&guest=1) is now running


And clico em "salvar"


Unfortunately not, I still get the same issue. In 0.4.0 it works perfect, though.


emita -> o sistema exibe


To clarify the intent here, this is because our code is currently working against the webhooks that UserVoice sends out (which includes `data` and `event` keys), so this means we can consume from raw endpoints or via this service hook.


@stinos: FYI


@c4milo Unfortunately me, @catsby, and @phinze can't reproduce this with upstream (though we did reproduce it with 0.4.1). Can you gist your configuration to repro it?


oh, this is the piece I don't like... Maybe we should add more packages!!!!!


xpath?


Closes #1


Esse step pode sair


We could probably cache the server version in the rest-client. There are also other things like session details cached. That way not every user of the rest-client would need to cache it by itself. What do you think?


You're implementing an agent interface. You can't return structs of your own, so yeah I guess it's either a middle package or it's agent's Deployments.


Given eu estou na tela de "criação de provas"


Pode tirar


BigDecimal would be better than Double IMHO.


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 298](http://192.168.30.225:8111/viewLog.html?buildId=1111&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 73 Build time: 00:11:45
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


TeamCity Ybackend :: biocad/ybackend [Build 186](http://192.168.30.225:8111/viewLog.html?buildId=1113&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


how do i extract these javadocs to form an API?


Não reflete o significado do cenário.


Since we are only ever invalidating thumbnails throught the api, we no longer need this margin


Hah! :smile:


Esto es una prueba donde te digo que algo esta mal y lo cambies


? what's wrong here?


This logic could probably also be part of the rest-client. I'm quite sure the IntelliJ plugin will not be the only user of this version stuff forever ;)


@tserong
looks good, I will merge upstream


Didn't know you could use uppercase so smart choice to lower case it all, and I didn't look into the XML deeply just noticed it wasn't in the list but that's definitely cleaner way of doing it than having a field in the collection.


Suggest removing TODOs. Not sure of the intent for this pull request but if this is going to be fixed/resolved in a future Backlog item then we can just remove the comment or change it by removing TODO and saying it doesn't handle multiple cubes. Otherwise we should fix the issue.


And eu clico em "salvar"


asasasasasasas


Po tej "poprawce" w trakcie update z wersji DIRA mysq ma problem z OFFSETem

Wystąpiły błędy w obsłudze bazy danych!
Zapytanie: ALTER TABLE documents ADD fullnumber varchar(50) DEFAULT NULL
Błąd: Duplicate column name 'fullnumber'
Zapytanie: SELECT d.id, cdate, number, template FROM documents d JOIN numberplans n ON n.id = d.numberplanid WHERE numberplanid <> 0 ORDER BY id OFFSET 0 LIMIT 1000
Błąd: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'OFFSET 0 LIMIT 1000' at line 3
Zapytanie: UPDATE dbinfo SET keyvalue = '2014072500' WHERE keytype = 'dbversion'
Błąd: Table 'dbinfo' was not locked with LOCK TABLES


Asserts do not cost anything in non debug mode, so you should put some whenever necessary. But in this case we can forget them.

For the loop, you can just do:

         while (currentX < xLength) {
             if (accidSpace->at(topPos + 1)[currentX]) currentX += 1;
             if ((currentX < xLength - 1) && accidSpace->at(topPos)[currentX + 1]) currentX += 1;
             else if (accidSpace->at(botPos)[currentX]) currentX += 1;
             else if ((currentX < xLength - ACCID_WIDTH) && accidSpace->at(topPos)[currentX + ACCID_WIDTH]) currentX += 1;
             else if ((currentX < xLength - ACCID_WIDTH) && accidSpace->at(botPos)[currentX + ACCID_WIDTH]) currentX += 1;
             else break;
         };

I think this should work without the extra accidSpace.


Is better $order_info['language_directory'] then 'english'.


emita => o sistema exibe


:warning: Working on line 101 bug in CheckoutFormFragment to signal the checked boxes


What is the point of `$('<div></div>').text(v).text()` ?


same as https://github.com/pminutillo/modeler/commit/f66f7da12f326da77964200fb496aca99fa1aafb#commitcomment-10673154


:+1: Nice one


I prefer to have constants right below the class declaration.


OK, you can do it if you want and update this merge or I'll  merge your current changes and just make that small change to the `plex` page tonight/tomorrow.


This was actually my first approach after, we have removed the extra loop for this.
But it turns out to mess up the code, if we want to make it performance natural.
The scaling issue allow is already complicated enough.
Do you see a chance to integrate it in a maintainable way?


Why can't I find a build manual in English anywhere on any site linked to this project. I printed all parts bought rest and it is sitting doing nothing


Nuevo commiet


I think it can be removed.


develop (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (336 of 336 tests passed)

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


Oh, should it return null instead?  I probably got confused and did this to make sure it works with debugMode, but I can't recall at the moment.


PS One assert you should have is actually

assert( topPos < accidSpace.size() - 1)

because you do accidSpace->at(accidTop + 1)


Finally settled on 99a40cee47fe8be75dbe19bef813bc0afba4ec3c


woot :D


Yes, but as @edolstra mentioned, this file is copied to be used as /etc/nixos/configuration.nix in amazon images.


Thanks for your attention to detail @chriscct7


I'm not sure if it is easily possible (I have not checked out this code in my IDE yet), but in some other cases I have added backward compatibility in catch-blocks (so the overhead of getting the version is not required). See for example GerritUtil#getChangeDetails.

What do you think?


TeamCity Ybackend :: Feodorov/ybackend [Build 133](http://192.168.30.225:8111/viewLog.html?buildId=1112&buildTypeId=Ybackend_FeodorovYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 72 Build time: 00:11:45
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


TeamCity Ybackend :: biocad/ybackend [Build 187](http://192.168.30.225:8111/viewLog.html?buildId=1114&buildTypeId=Ybackend_BiocadYbackend&guest=1) is now running


That wasnt enough, the problem persist just in less frequency.


Okay - I'll test it out a bit and see what I can find with debugMode (which I still haven't used yet...) it's not breaking anything in production.


Excellent.


You do not need `.to_s` here since the return of the `.split` call will be an array of strings.


That's exactly how it works for anything that we create in this process. This cleanup call simply deletes anything that's left-over from a previous run of felix.  We use ref counting for ipsets and profile rules.

It would have been possible to load the whole start-of-day state from iptables/ipsets and create pre-increffed actors for those (I like that idea) but when I tried to make that work it ended up being hard to get right.


vmdb_obj can be any object so it does not seem safe to assume it will always expose a name attribute.  Should this method be more specific to what object you are processing (for example tags) or be modified to handle objects that do not expose `name`?

Maybe something like:
```ruby
options_value_array << vmdb_obj.respond_to?(:name) ? vmdb_obj.name : "#{vmdb_obj.class.name}::#{vmdb_obj.id}"
```


Manter um padrão nos termos usados, evitando sinônimos no mesmo cenário... o título e os demais steps desse cenário usam o termo "questão", somente aqui fala-se em "enunciado".


I moved to init_qt, made a slight change to is_available() because it requires a wallet, but load_wallet() is yet to be called. It worked.
The bigger problem is when creating a Trezor wallet by the installwizzard - it ultimately tries to access the device (and get_client()) long before init_qt is invoked.


Perfect.


A feature já define que o stakeholder está no papel de professor.... isso nao precisa ser mencionado mais em canto algum.... daí é seguir a convenção de falar em primeira pessoa...
When eu crio...


Question: isn't this outdated? I believe that options now is a Seq[(String, String)]. Can this be changed?


86046200_CondenserToZone_New (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1644 of 1646 tests passed)

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


trova l'errore!


Looks nice :)


Then a nova questão argumentativa é criada // o que significa que é salva no sistema


@Gioyik please create markdown links to both events and last community updates, otherwise it looks good to me. I'll be able to merge it and publish it


When eu removo...


TeamCity Ybackend :: biocad/ybackend [Build 186](http://192.168.30.225:8111/viewLog.html?buildId=1113&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 73 Build time: 00:11:44
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


Is it possible to just initialize here?


Folder structure
![](https://cldup.com/y4e0vikDyr.png)

Each file's content
https://gist.github.com/c4milo/97c86a1b16eae76d9449


This makes sense. I can't imagine anybody really wanting to get a pyjobject for these


86046200_CondenserToZone_New (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (620 of 622 tests passed)

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


Then a questão com descrição "Quem foi Rômulo e remulo?" não existe mais no sistema


Nothing. I just though: constraints are aligned with 11 and 32, their common kgV is quite high :-)


As mentioned above, this is only trying to delete left-over state form a previous run of felix.  Using async=False blocks this greenlet until that iptables updater actually finishes its cleanup and returns.

We allow blocking calls to iptables updater because it's at the lowest level of the tree and it makes the rest of the code simpler on balance.

Each actor has its own greenlet so, even if it spins the scheduler, it still can't process another message off the queue in parallel.  The cleanup blocks anything else until it's done.


usar verbo sempre no presente... ver correções em outros steps similares...


o professor -> eu


Attention Christophe, j'espère que tu as bien reset tous ces passwords


parce que les bots vont voir l'historique aussi


The stuff below this if statement is being hit every time in 2TA and 3TA, resulting in the allocation of a new timer every iterating as well as a function call to open the arms.


trocar "enunciado" por "questão" para uniformizar o texto do cenário


Good point. I'll start one so we can figure out what those rules are.


Then  a lista de questões argumentativas contem a questão "Quem foi romulo e remulo?"


que esteja => eu estou


SELECT o FROM OpwKandydat o WHERE o.firstname = :firstname AND o.lastname = :lastname


Is there anything else that we need to validate against other than the name?


Se o objetivo do cenário é descrever a criação de uma nova questão, esse deveria ser o resultado do cenário.


ユーザデータを読み込んでからnowを代入するまでに、データが更新される可能性があるので、55行目より以前に代入してください。


Czepianie się ale: http://stackoverflow.com/questions/925423/is-it-better-practice-to-use-string-format-over-string-concatenation-in-java


Looks good, keep up the good work :+1:
Oh wait this is my commit :smiley:


TeamCity Ybackend :: biocad/ybackend [Build 187](http://192.168.30.225:8111/viewLog.html?buildId=1114&buildTypeId=Ybackend_BiocadYbackend&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1, passed: 73 Build time: 00:11:02
### Failed tests
```
ru.biocad.ybackend.backend.MetaServiceTest.MetaActor should list config if user has RO access: class java.nio.charset.MalformedInputException: Input length = 1
java.nio.charset.MalformedInputException: Input length = 1
      at java.nio.charset.CoderResult.throwException(CoderResult.java:277)
      at sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:338)

```


Deveria fazer parte do when o preenchimento do cadastro da questão com os dados necessários, usando valores específicos, passados entre aspas.
Lembrar.... cenários são exemplos de uso do sistema, logo se usa exemplo de entradas e saídas para escrevê-los.


ляляля


We're moving away from the admin entry under the Settings collection so you can't use it going forward.


Co jeżeli kandydat ma więcej niż jedno imie?


комент


Thanks for the clarification, that does sound better. Gonna need to update other instances of that.


We could do:
```php
            return rtrim(
                // Creates dsn string: mysql:host={~host};dbname={~database};{~port};{~charset};
                str_replace(
                    // Replace any instance of "{~value}" with the config value.
                    array_map(
                        function($value) {
                            return '{~' . $value . '}';
                        },
                        array_keys($config)
                    ),
                    array_values($config),
                    $parse
                )
            );
```

however splitting the code into another method would seem like over kill.


Thanks ^^


Thank you!


Both of these comments make sense to me... I made these changes.


@brandtheisey probably should remove this, right?


Awesome stuff.


I'm not sure how you're getting the requesters name? Are you adding their name into the Settings collection under plexuser when they authenticate and then calling that variable on the push bullet notification?

We'll be adding the requesters name to the movie entry itself, and then pulling it from there.

Unless I've missed something entirely wrong (reading at work while running out the door).


Was [this](https://github.com/vivo-isf/vivo-isf-ontology/commit/49e9700e823a6e68fe5ce03494d9756ba6215718#diff-c60afe2b89a6b0301214c34218533691L1879) intentional? It seems like the other ERO properties still have rdfs:labels and I would argue maintaining support for it is helpful with most editing tools.


It was fairly straightforward to execute some JS to define a JS function.  But I was doing *something* wrong and the function wasn't doing what I wanted to.  Probably just something sillly I was missing, but it was very hard to debug what was actually going on within the JS code. After spending some time on it, I figured it wasn't worth it at this point.  I've got the code stashed somewhere if you want to take a look.


This looks like it could go on forever...


Was this intentional? It seems like the other ERO properties still have rdfs:labels and I would argue maintaining support for it is helpful with most editing tools.


To powinno wylądować w encji :P
http://stackoverflow.com/questions/21477034/what-does-transient-annotation-mean-for-methods


That isn't a variable this call has any access to, I'm assuming this is old?


Haha I have t done anything to it yet. Still got some learning to do before I proceed

Sent from my iPhone

> On Apr 10, 2015, at 1:24 PM, Steven <notifications@github.com> wrote:
>
> Looks good, keep up the good work
> Oh wait this is my commit
>
> —
> Reply to this email directly or view it on GitHub.
>


This needs a description somewhere.  A comment on how we store the various pieces (secure/in-secure) would help.


With the changes, this is the only script that is now needed. The other script for listing the branches can be deprecated after someone else tried out that it works for them.


    W dniu 2015-04-10 o 22:25, pwl89 pisze:

      Co jeżeli kandydat ma więcej niż jedno imie?
      —
        Reply to this email directly or view
          it on GitHub.

    Na karcie wyborczej i w protokole są wszystkie imiona wpisane
    Tutaj nie wydaje sie to konieczne.

    Przemek


Here is a minor problem. If variable 'm_col_width' does not defind here:
void mmListCtrl::SetColumnWidthSetting(int column_number, int column_width)
{
    Model_Setting::instance().Set(wxString::Format(m_col_width, column_number), column_width);
}
in that case the function
void Model_Infotable::Set(const wxString& key, const wxString& value) will save value for empty key.
How to reproduce: Open GRM - select any report - Execute SQL by Test button then close GRM.
As result the record will be in mmexini.db3 file.


Replace "key" with "opaque token", add verbosity around Sync/collection keys, etc.  Also, perhaps we should just derive the opaque token from the syncKeyBundle?


Can we enforce this with some kind of pair `(KeyBundle, Keys)` or similar?


Does this do good things when the optional is nil?  I think so, but worth checking.


pode apagar essa linha.


This Scratchpad has a bit of the feeling of `SyncConfig` from android-sync: it's a big ball of mud.  We'll see how it grows and how much we can keep the state immutable.


Thanks! We're digging into this now


This breaks the assumption that g_strcmp0 will not choke on NULLs.
You should really add some other boilerplate to make it behave like g_strcmp0

```Handles NULL gracefully by sorting it before non-NULL strings. Comparing two NULL pointers returns 0```


The variable `@task` does not need to be an instance variable, it can just be a local variable `task`.  The only places it is references are in `parent_task_dialog_information` and `save_parsed_dialog_information` which are both called directly below and should just get the task object passed to them as a parameter.


Gah! Having all those web elements show up in the docs will make them very noisy. Please only use jsdoc-style comments above functions that are high-level, public facing functions. Leave "private"-y stuff like web elements, helper functions, and such undocumented so that they don't show up in the resulting documentation.


Thanks! BitHub has sent payment of  $11.33USD for this commit.


Limitar a quantidade de registros para trazer


@chrisscooke We move more towards NPM as it lets us handle dependencies easily and it takes away the maintenance load. The version of ShellJS in the repository was very outdated. NPM makes sure that we can update all dependencies easily without having to put all that external code in the main repository.

If there is no way to install NPM locally (and there are ways for that, see for example http://tnovelli.net/blog/blog.2011-08-27.node-npm-user-install.html), you can always work around this by downloading ShellJS yourself and changing the include lines, but for our repository it is best not to have all that code in the main repository.


esto lo puedes hacer asi...


You could also update this in your PR to be `<?php` instead of `<?`: PSR-2 states opening tags *must* use `<?php` or `<?=`, never `<?`. Minor tweak but since you're changing this block anyway, it makes sense


Awesome, thanks guys!


I like this a lot better.  We're not leaking database backed objects into the celery task now.  We created the backend for exactly this purpose.


It could indeed, but only if the user continually presses "Retry" forever without resolving the problem. If there's another way then it's an oversight.


This is pointless.


El XML tiene que rehacerse por completo ya que los monumentos cambiaron en el 1.3.6


Doesn't this mean the first visitor will be visitor 2?


Great. Thanks for answering Scott


Absolutely none ... I guess it is kind of late here. I thought I was escaping some symbols like that, but now when I look at it - I am not. Thanks for the catch.


I mean: do you really need "array" in this name? :-)


Doesnt work lol


Why would this ever not be the case? If `$previous` isn't valid input for `unserialize`, then `unserialize` shouldn't be called, as this emits an `E_NOTICE`


dejaria un enter mas no?


this is great.  It would be nice to document `plot.lmList4.confint` better, e.g. the `order` argument.  The implementation seems a bit weird (i.e. order, xlab, ylab, seem to passed through ... rather than being named arguments -- some odd generic-compatibility issue?)

I'm happy to do this, but don't want to put it in the wrong place.


hay que corregir el copyright?


Your capitalization scheme is inconsistent here.


CIAA Firmware?


estos 2 errores estan en todos los archivos creo.


Much better. :+1:


86046200_CondenserToZone_New (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1644 of 1646 tests passed)

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


@shinnn, mind adding this to the other plugins too? :)


Also the function
void mmListCtrl::OnItemResize(wxListEvent& event)
{
    int i = event.GetColumn();
    int width = GetColumnWidth(i);
    if (!m_col_width.IsEmpty())
        Model_Setting::instance().Set(wxString::Format(m_col_width, i), width);
}
is useless now.

And the function
void Model_Setting::Set(const wxString& key, const wxString& value)
{
    Data* setting = this->get_one(SETTINGNAME(key));
    if (!setting) // not cached
    {
        Data_Set items = this->find(SETTINGNAME(key));
        if (!items.empty()) setting = this->get(items[0].SETTINGID, this->db_);
    }
    if (setting)
    {
        setting->SETTINGVALUE = value;
        setting->save(this->db_);
    }
    else
    {
        setting = this->create();
        setting->SETTINGNAME = key;
        setting->SETTINGVALUE = value;
        setting->save(this->db_);
    }
}
looks strange. May be better:
if (....)
{}
else
{}


oops


A+ thanks for your help @arthurvr


You are inconsistent with using `echo "foo"` or `echo ("foo")`. The former is typically preferred.


Nice idea.


He compiled the plugin with Java 8, so you also need Java 8 to run the plugin.


I am not sure if it is this commit that is causing the problem, but sound & notification setting shortcut widget FCs when pressing it. Log: http://pastebin.com/Vi1U7EXF


You are inconsistent with using spaces around your assignment operator (i.e. `$a=1` vs. `$a = 1`)


I'm not sure the right way to handle this. i.e. I want to say "Fall through to all the default stuff", but you can't have an empty statement in the switch. I moved all of the default handling inside here instead.


Does this insert the CSS with a link tag still or does it inline it?
Inlining can be a problem for CSP, I think.


Debug code left in?


Is this intended?  I expect the scratchpad to be deeply immutable.


1591_AttributesInArguments (macumber) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@kirtfitzpatrick, This commit isn't in master or any other branch. Is it OK?


What is your intent with using `return` here? Where will this message be returned to?


sonucu ters slash lı halde verirse db ye girerken sıkıntı yaşıyoruz.. serverda sorun çıkartmaz bu url ile vereceği için adresi fakat localde C:/ ye göre baz aldığı için adresi sıkıntı yapıyor. db ye prepare ile girerken onlar temizleniyor..


Reminder to self: I left a debugging statement in at line 132.


Está bien, manten el Y a -94.5 y 94.5 en -94 y 94 apareces mal


I would probably look at the arrayFilter function in lodash: https://github.com/lodash/lodash/blob/master/lodash.js#L1400

(Really we want the equivalent to the _.find function, but the code for that is not intuitive.)


saw approval from bsparrow435
at https://github.com/basho/riak_kv/commit/a82d67e705aee592d025a7b111da8185f55a14a8


merging basho/riak_kv/jhc-portwarn = a82d67e7 into borshop-integration-1102-jhc-portwarn


basho/riak_kv/jhc-portwarn = a82d67e7 merged ok, testing candidate = 530e7a76


Merged as 5d8d2ddc75ba97fcd4ff2a920b83c59731167eb0 with modifications---all methods should be non-enumerable as part of ES6 classes.


1591_AttributesInArguments (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

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


r+


duh.  missed that.


Merged as 87b4ff93334c5f1cbebda638101c6a145a601a4b


It kinda grew organically. Origin is like 2003, perhaps a single format at that time. Now nearly all formats has legacy code from it :smile:


Please use the PFMT* format strings for such things for better portability


86046200_CondenserToZone_New (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1644 of 1646 tests passed)

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


There should be a division somewhere... It is realistic for just test to have > 32 players.


Is updated, this way works better and prevent from new ID creation. @opdenkamp can you check.


I like the Builder pattern but I'm worried about the potential for races with `checkpoint`.  I guess this is always a concern, but it will be really quite easy to make a mistake if and when we add more threads.


I've added you as a contributor to the main halide repo (probably should have done this a while ago). In future if you make fixes as branches inside the main repo they'll be easier for us to check out and test.


Oui oui, c'est bon ;)

Le 10 avril 2015 22:20, Thibaut Assus <notifications@github.com> a écrit :

> parce que les bots vont voir l'historique aussi
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/krichtof/edgar/commit/4a4c8482be60b3cfd757fa841e3f8c3e7029877d#commitcomment-10673379>
> .
>


Ilakoze,
When you finalize these changes,
It's always wize to let the team, know that it's complete.


saw approval from graydon
at https://github.com/graydon/stellar-core/commit/474b97829c23be9c5d21cf72e6c5f0c92915e8ef


merging graydon/stellar-core/fix-travis = 474b9782 into auto


graydon/stellar-core/fix-travis = 474b9782 merged ok, testing candidate = db4c0c1a


delete


Merged as 048ef2bf2302513cf3d6e33df064388fb51ccb34!


I think this is fine.  I can't remember if we handle this one way or the other in android-sync.


Nice progress,
I recommend you put descriptive messages in your commits, not just "save current doc" or "updated doc",
Saying what you have actually updated and in what part of the doc for which good reason, is really important for future reference.

And it's good programming practice.


Man, Sync clients are just a giant stream of punches to the face.


I swear I though JavaScript had a `RuntimeError`. Opps. :confused:

`data.op` is a mistake.


I'd prefer forward('', '400');


Sql update:

ALTER TABLE `sayygo` ADD COLUMN `home_location` VARCHAR( 800 ) CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL;

ALTER TABLE `sayygo` ADD COLUMN `languages` VARCHAR( 200 ) CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL;

ALTER TABLE `sayygo` ADD COLUMN `my_experience` ENUM( 'Been around the world', 'Experienced international', 'Experienced domestic/regional', 'Moderate experience', 'Little experience', 'Never traveled' ) CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL;

ALTER TABLE `sayygo` ADD COLUMN `phone_number` VARCHAR( 20 ) CHARACTER SET latin1 COLLATE latin1_swedish_ci NULL;


Let's develop a habit understand what a function should do, writing a test, then writing the function.  Does anyone really work that way?


A może by tak burze rozpętywać tutaj: https://github.com/adamkowalewski/OtwartaPlatformaWyborcza/issues ;)


Make sure you indent line 16 properly and to end line 17 with a semi-colon. =)


Missing a semi-colon on line 26 after the curly brace. Looks like proper indentation is also needed for lines 19 through 25.


just in time!


Missing a semi-colon after your render function call.


Fixes #160


Maybe someday lol.


Seems so, updated.


cool :)


Merged as 96e2c5f6047bd03050636dd9d550246c1eb09d82, although the strategy might start throwing if #317 goes through.


@rbran100 Trying to apply this update to custom OS installation and I'm getting the error

Fatal error: Unsupported operand types in /mnt/www/html/ekiczek1test/docroot/includes/database/mysql/schema.inc on line 85


we should make sure that if we're depending on cookbooks they are also up on the supermarket and properly versioned and documented. to really get adoption of this cookbook we'll need to be super-transparent about what it (and its dependencies that we own) are doing to your machines


test


Yeah, I altered that naming elsewhere, failed to do so here.

I didn't derive this for a few reasons. One is that in theory I'd like to be able to clean up without having your keybundle. Another is that I don't want to accidentally load keys just because the sync key was the same. Randomly generating a token offers all of the benefits with none of the problems, with only the cost of persisting that token alongside the rest of the state.


I would consider moving this line into the `find_wiki` method. eg.

```ruby
def find_wiki
  @wiki = Wiki.find(params[:id])
  authorize @wiki
  @wiki
end
```


IE tech preview won't throw for the sparse array case. I've filed a bug on them. For now the `{1:1}` case will throw.


Thanks @tricknotes this was trolling me for a while


Dzieki :)


Also not sure if you need to be concerned with tech preview / or non-stable release issues.


Yeah, the `?.` operator means this'll just be the same as `\(nil)`.


This is not necessary, these variables will be initialized with false and "", just as they are if you used:

var hasTag bool
var tagId string


Here you would do:

hasTag = true
tagId = tag.Id

return


Just to verify, is the last part correct? `{~port};{~charset};`


I'll merge after the line notes are resolved.


Another way of doing this is to set the default value in the database itself. You would do this with a migration.

Inside a new migration you would write something like this:

```ruby
def change
  change_column :users, :role, :string, default: 'standard'
end
```


@IbnSaeed I fixed in this commit: https://github.com/jhonnymoreira/jeet/commit/17d4e95656345a09274fd94aa9bf0406facbf85f


That gets hairy: when we switch out keys (which is really a `Fetched`!) we'd need to rebuild the pair. This is more of a caution to me than to anyone else :D


Ignore this, I'm gonna fix this eventually.


Be careful with using real email addresses in your seeds or tests. It's too easy to send emails to them by accident. I always use `@example.com` for the domain for all my seed and test data.


Maybe we should call this `<beast/win32_workaround.h>`


I would if I had a way to test it easily on my Mac (browserstack doesn't appear to have it). Any suggestions?


all tests pass:
success: http://buildbot.bos1/builders/test-riak_kv/builds/1996


```php
// Binds DSN string like: mysql:host={~host};dbname={~database};
```

I can live with that, but it's not just about understanding what it does, but also how it's being done.


This should say `Also see |:terminal|.`


Good idea.  Will update on next commit.


Ok, I can take out the TODO on the next commit.


I see your point, but:

1. It's much more convenient to do this, and in practice using a mutable collection here is equivalent.
2. From a certain perspective this is more correct, because these timestamps — other than meta/global and crypto/keys, for which the state machine is the "synchronizer" — are really tied to persistent local storage. Once you've applied a record locally, an earlier Scratchpad with an earlier timestamp is no longer correct, and it should reflect the new collection timestamp. So this (again, arguably) is a valid breaking of immutability.


There should be a data for this as well {{content}}.

Not sure if we'll use body, but add the data and we can merge it.


Now take out the background.


remote.modern.ie.


all patterns need to have a class section in their html and data


add a class to the button


not yet working, need to look into configuring email through env vars. this should work, right? at least we're using this stuff in prod... nvm, will try later


add a class


Minor nitpick, but could ```text``` here be replaced with ```expected``` as with the ```Element Should Contain``` keyword?


Yeah, no the problem was a nasty segfault because the QAmqpChannel itself had already been destroyed and what I was left with was the QObject.

This triggered a nasty segfault:
https://travis-ci.org/mbroadst/qamqp/jobs/57880744#L493


:+1: I'll add that to the next batch.

I rebased a few times, so had few different set of reviews, but if you think I'm merging things too hastily then I can let things "simmer" for a little longer.


tak naprawdę to trzeba pisać po polsku komentarze skoro już taką przyjąłeś konwencję


Yeah. NSUserDefaults is thread-safe for individual operations, but not for the whole sequence we do. I expect we'll use the Deferred queue mechanism to ensure that these all happen together.


Let's reset this change for now.
We can discuss failure case later.
However, blocking all events isn't a good idea..


I felt that way as I wrote this code, yes.


Merged as 4702137b5a0f1b7f99a206df7147a40a8f19c69a, with some tweaks to the property descriptors


This removal along with #161 seems to breaks the widget.


Except now it doesn't exit early, so behavior below is undefined. :stuck_out_tongue:


:fire:


This may not work as you expect.

We want to ensure that a `Vote` is only valid when it's value is either 1 or -1.

To achieve this you need to setup the `@vote` variable and modify it's value and check it's `valid?` method.

eg.

```ruby
describe Vote do
  describe 'validations' do
    describe 'value' do
      describe 'when greater than 1' do
        subject { Vote.new(value: 2) }

        it { should_not be_valid }
      end

      describe 'when equal to 1' do
        subject { Vote.new(value: 1) }

        it { should be_valid }
      end
    end
  end
end
```

There are another 3 cases you need to check, when value is 0, -1 and less than -1.

Try writing these other 3 test cases.


Looks like I deleted a `datetime=` here :(


Also, shouldn't this be before the calls to file_get_contents to begin with?


If we put `int i = 0;` here, we can also eliminate the `else` branch below.


port & charset are valid arguments, as well I was surprised to find out that PDO accepts `mysq:host=example;dbname=test;;;`. So leaving the extra `;` is ok. You can also reverse the order `mysql:host={~host};{~port};dbname={~database};{~charset};`

Is that whay you are verifying? Not 100% sure.

I could like into that adds the `;` dynamicly so it would look like:
```

Now that I think of it, we might want to add `unix_socket` support:
http://php.net/manual/en/ref.pdo-mysql.connection.php


In general, your commit message should be a brief description of the actual changes made. Versioning should be handled outside this "channel".


Well, on this hosting, we don't have Java 8 :/


Did you create a separate Google application for production?  That might
explain why we're not seeing the Google login issue.

On Sun, Mar 22, 2015 at 5:18 PM, Michael Lai <notifications@github.com>
wrote:

> remote as in s3?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/howardc93/dukeexchange/commit/bca25e2f5e779ed7c6a4fcf4686e15d6c7aef114#commitcomment-10335173>
> .
>


Please  let me know if there is still something unclear or if you find some issues   .
I have added a lot of comments to  make it more understandable .


Fixed now...didn't make it into the most recent pull request, though.


Suggestion: s/fixed/fix/ in the commit log.  Makes it fit in 50 characters too. :-)


I'm lost for a while already between all the id's, _id's, etc. so I won't even try ...


@Pyrohh No I think everything is going well :)


typo


you think?

I was kind of considering by the time this is called, the act of choosing is in the past, hence chose.


Please go ahead.   `plot.lmlist4.confint()` is one of the few things I did not touch at all (apart from s/lmList/lmList4/ renaming).


oh, I thought I was missing some assumption about the environment in andmore.


But, do we need IsBlankText when it's the exact same as string.IsNullOrEmpty? :)


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


You're getting *too* awesome.


aha.. nice, good to know


@singpolyma We're looking into this! Thanks for bringing this to our attention. We'll get back to you soon.


One cannot export a path variable here because the existing path would be overwritten by any following calls to `fw_depends`. Example: `fw_depends java7 resin maven` - if all three of these set PATH, then each $PATH would be resolved when being appended to the `.installed` file and the last script would "win".


cc @retep998, can you check this over?


Seems like this breaks stuff along with PR #161... It feels like my #154 fix while working for simple widgets broke the Heavy ones. Maybe 5c18d5425156973de098e8eaaf76b7e9826ecce4 would have worked better.

Needs testing but so far an AutoHeavy widget fails to initialize with "selector is not defined" because it is lacking the
```window.django_select2.%s = function (selector, fieldID) {
    var hashedSelector = "#" + selector;```
 part


Not sure if there's a better (read: automatic) way to ensure that the file can be executed. This works, but feels like an extra step.


create


(y) Looks good to me!


Always source the newly created `.installed` file to get variables into the environment.


So, would `isSearchable`, `isDisplayed` and `clear()` be enough? (more? less?)


develop (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


https://github.com/paulmillr/es6-shim/commit/1d80b06264d51166a4d8c8c1bc51b43b5b6be098 works in IE 11 TP


I'm surprised that there aren't compiler warnings about unreachable code with the `return` after the unconditional `throw`.  But I suppose there could be compiler warnings the other way around about a missing `return` statement...  I'm good with whatever keeps our compilers happy.  Nice use of `{}` on the `return`.


Wow, that is really strange. Need to optimize import...


fast-forwarding master to auto = db4c0c1a


I am going to omit these because (a) they are really tests of the implementation's JavaScript implementation; (b) I think in ES6, for strict functions, `arguments` and `caller` go away.


Do we need this try/catch then?


These should in fact be named


This will be changed back to `2-3-stable-mrh` when the fix in the referenced branch is merged.


Meilstein03, Einzelaufgabe


Confirm then we don't need to clean up any children?


I don't think it applies cleanly any more.  This section of s_client.c changed quite a bit.

@shigeki Can you capitalize and punctuate the body of the commit log?


typo? `capacit()`


This works without a comma? WTF python still surprising me..


Small change, but fixes an irritating issue :+1:


I would use a `finally` statement here, just in case `close()` fails:
```java
try {
    fem.close();
} finally {
    AsyncJPA.bindForCurrentThread(null);
}
```


Hey @c4milo – https://github.com/hashicorp/terraform/pull/1487 was just merged into `master` to address it, can you take it for a spin and let us know? Thanks again!


Did you actually rename this somewhere in the python codebase? I'd vote to keep the variable names expressive in python and terse in bash


No! Just an oversight.


Правильно ли я понимаю, что этот ассерт проверят, что тип памяти должен быть Programm и половина памяти была кратна размеру инта (4)?
Если так, то создать сторадж можно только для Programm?


I would use `finally` here too.


Nice!


Zobacz w konsoli czy dasz radę to zapytanie wykonać.
W pgsql OFFSET jest oczywiście bez problemu wspierany.
Chyba powinno być: LIMIT 1000 OFFSET 0


OK, this seems like the simplest way to go. Hopefully it'll only be a cycle or two before we can bring in at least a draft replacement.


They both existed and I was simply accustomed to using `iroot`. I'll revert it.


I know we discussed this in IRC, but for the record I hate dis :-P Turns our beautiful complex bash into a hack. I'll see if I can make the more conventional approach work and reply if I get something


Que grande @ldlsegovia !!


I need some feedback if data model is correct. Any comments and propositions are welcome


Broken in reference implementation due to https://github.com/google/traceur-compiler/issues/1879


@shigeki This commit should also apply this patch:

```diff
diff --git a/configure b/configure
index e4148a7..675581b 100755
--- a/configure
+++ b/configure
@@ -525,13 +525,6 @@ def configure_node(o):

   if target_arch == 'arm':
     configure_arm(o)
-  elif (target_arch == 'arm64' and
-        not options.shared_openssl and
-        not options.without_ssl):
-    # FIXME(bnoordhuis) It's not possible to build the bundled openssl due to
-    # deps/openssl/asm/arm-elf-gas/modes/ghash-armv4.S, which is 32 bits only.
-    warn('not building openssl, arm64 not yet supported')
-    options.without_ssl = True
   elif target_arch in ('mips', 'mipsel'):
     configure_mips(o)

```


typo - forgot the function name


Done


1591_AttributesInArguments (macumber) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2085 of 2197 tests passed)

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


Why?


For reference, the issue is that `sudo -u %s -E _H stdbuf -o0 -e9 bash -ex` by itself drops you into the new user and does NOT copy over exported functions - only variables. If you can find a way to pass the functions over, then by all means.


I don't think these are required anymore and I simply forgot to remove them.


Note that if you want to allow people to create databases using Trestle, this be removed.


I don't understand this phrase and I'm not sure if there's a grammatical error on it or I just did not get enough sleep last night.


Here's the most minor of nits: You can keep the methods in the same order they are declared in the header by moving the implementation of `store` below the implementation of `fetchBatch`.


Fascinating...


Small typo in commit log: s/form/from/


It seems to allow them to enclose the value, when decoding we need to take this into account, in case of reading such a cookie written on the server.

  > cookie-value      = *cookie-octet / ( DQUOTE *cookie-octet DQUOTE )


Merged as d6f0c9590e207eb681374ab331ab7e5d8fe50d19


I'd rather start with writing tests for the expected behavior...


Corrige esto y usa  addCSS


Can't we reuse existing roles?


Dats a bunch of bananas :P


r+


Can you add outputs, clean and check-{cc,asm} to .PHONY?


Esto  proboca errores de coNpilazión


I must admit I have absolutely no clue what's going on here. Cargo culting at its best!


Por favor corrige el código


Oh oops, I ended up fixing this in a few later commits.


FYI this is the old implementation of `read_link` (e.g. this was copied from `sys::windows::fs`)


s/wdelete/delete/


Congrats!


estou trabalhando em um aplicativo de rede social em 3D
eu tenho apenas os Briefings/Wireframes, gostaria de compartilhar com vocês


Ok then!


develop (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (336 of 336 tests passed)

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


Yes, it's on the list here :)


![Uploading WIREFRAME IOS SMART SOCIAL 3D.jpg . . .]()
![Uploading WIREFRAME IOS BARRA DE INDICADORES DE COMANDOS.jpg . . .]()
![Uploading WIREFRAME IOS SMART SOCIAL.jpg . . .]()


I'm inclined to say that the presence of SSE2 can safely be assumed.  I don't think ia32 V8 even runs without SSE2.


WTF why my commits aren't my.


to practice OOP design skills :)


1591_AttributesInArguments (macumber) - i386-Windows-7-VisualStudio-12: OK (2151 of 2197 tests passed)

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


line 18, $('.plexAuthComplete').remove(); should actually be the below:
$('a.show-plex-form').show();
 $('.plexAuthComplete').hide();


following the convention


Thanks for finding this bug @ryancbarry ! This happens when `setQueryParams()` is called at some point, but was never called from `componentWillMount()` before, then `_this.currentParams` doesn't have the `this.props.queryParams`.


Talked with @arv about this. There's no easy fix in Traceur, which means I'm going to have problems making the corrected versions pass in the reference implementation on io.js until Chrome 42 is released with classes and thus io.js gets classes and thus I can turn off Traceur's classes compilation in favor of using the native V8 ones.

In the meantime, maybe I should add updated tests, but comment them out with a note on what's blocking them? Or, somehow try to layer something on top of our current testing setup that allows failures?

Sorry about this :-/


Note to self: we should change the warnings to "io.js configure error"...


Line note


parameter/DefaultElementLibrary.h missing licence update


Uhm the day-month-year one I guess. My watch has the American format so that's why I've made that mistake I guess..?


¿Qué hace esto?


parameter/Android.mk missing licence update


g.Status != 'unknown' && g.Status != 'error' should be (g.Status != 'unknown' || g.Status != 'error')


r+


You usually do not test private APIs. Not sure that `filteredContent` is part of the public APIs

To stay consistant, the second test should probably not rely on `filteredContent` but on what is visible to the user.


wait... maybe not... but in any case... it still wont let me bulk close errors


Shame ...


Thanks Mat. I had to move things around a bit as I lost my popoover annotations but looks like it’s sorted now.


Is this used? I don't see it here.


r+wc


nit: doesn't -> don't


It's slightly more pythonic to write that as e.g. `return int(match.group(2) + match.group(3)) if match else 0`.

Also, I don't think the calculation is really future-proof.  3.1 == 4.0 with the current approach.


Style: can you indent the comment?


No, I don't see any thing needs cleaned up.


Some licence update are missing


Looks great.


maybe IllegalArgumentException would be more descriptive.  Should eventually translate to a 400 bad request


Won't this be subject to the static-initialization-order problem? (ie: list_head might not be initialized before the first call to halide_runtime_internal_register_metadata()). That's why I had it wrapped inside a function as a static, to ensure it was initialized before first use. (Can we rely on it being zeroed before first use?)


Style: can you add one more blank line below?


Don't know. But with my attempt, I achieved a size reduction to *82,2%*, compared to the "old" version – if we take into account only the PNG files. However, for the *total size* of the release binary, the size reduction of the PNG files is kind of negligible anyway! The size of the Setup.exe was reduced to *98,0%*. Therefore, shrinking the PNG files by another *1* or *2%* probably isn't worth the effort...


86046200_CondenserToZone_New (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1133 of 1135 tests passed)

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


@mstembera
Congrats to you too.
However, before I pull...I want to be certain it doesn't regress...with all the new changes that been recently merged, such as your cmh weight.


The reason I hided them before is I don't want a indent action to interfere VS editing.
Remove this as re-throw doesn't make much sense.


This could be a bad idea. We have the Genomics people in Uppsala who want to use Karamel for OpenStack. You're now tightly coupling the code to AWS. Why not check if a VPC field is set in the provider, and if it is set - then cast it to AWSSecurityGroupApi.


liveness probe is actually another point of failure, unless you really need one for the test


using ports is generally discouraged, if you need to, can you use a service?


Are you actually polling this server anywhere? If all you're doing is looking for hostip suggest using a simpler pod (look around other e2es for something, will just the pause pod suffice?)


suggest keeping this pod spec to a minimum unless you really need labels etc


Why are you hard-coding the vpc here?!?


suggest using json instead of using the internal type (feels like best practices is not to use the internal types for anything except internal apiserver communication, though the previous test might have violated this already)


I suspect GH indents the tabs quirky when they're part of a diff.


Can you add clean to .PHONY?


Suggestion: maybe use perl here?


Beautiful :heart_eyes:


Again - why do you need to specialize to AWS? There are other platforms - GoogleCompute, OpenStack, Rackspace, etc. Please find a better soln.


Removed.


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


stupid confirmed


@oz123
shouldn't we support nvd3 1.7?


die.


As discussed, please remove this extra log statement and also add a comment above the runAt line explaining why it needs to be running in the server (and how the converter is running in the client)


The original proposal provided a readymade receipt subscription during message subscription which could then be forwarded to the AS. Part of the rationale for transitioning to the "receipt subscribe" was that the AS could then create multiple subscriptions for receipt ACKs.

Why not return to that simpler model? Then the PS could decide whether to return the same receipt subscription URI which is then a signal for the AS about reuse?

Also, I might expect that a PS SHOULD provide the same receipt subscribe URI to UA's that are pre-configured with the same message subscription endpoint. But -what's the rationale?


Why not keep the else statement and declare `int i;` at the top?


Please space the equal sign.

var lines = message


I believe that as a global with no constructor we can rely on it being
zero. halide_mutex is set up such that zero is a valid value representing
an unlocked mutex.

On Fri, Apr 10, 2015 at 3:01 PM, Steven Johnson <notifications@github.com>
wrote:

> Won't this be subject to the static-initialization-order problem? (ie:
> list_head might not be initialized before the first call to
> halide_runtime_internal_register_metadata()). That's why I had it wrapped
> inside a function as a static, to ensure it was initialized before first
> use. (Can we rely on it being zeroed before first use?)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/halide/Halide/commit/ab5d1f5c8ae26238e28b75b9ed2821c6387c639f#commitcomment-10674792>
> .
>


Aktualnie jest to tylko mock.


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


space the equal sign, and `<`. Also add a space between the `;` and the `i++`


Hm unfortunately I may have to remove this entirely, the paths returned all start with `\\?\` and apparently `gcc` chokes if you pass it those paths (for some reason or another).


That's a nifty feature - I ran into it a bunch of times. Good to see its resolved!


@areski that's probably a good idea. Do you have all your examples working ? Including IPython demo?


Yeah I did, it's on my account. Is it being deprecated or something? Did you find out about the change just because you were trying to run it and it didn't work or did you see like an email about it?


\o/


:smile:


Slicing MSVS_VERSION doesn't hurt but it's not strictly necessary either.  String comparisons are lexicographical, so e.g. `'2015-rc1' >= '2012'`.  That said, it's not wrong.

This line is overly long though.  You can write it as:

```

['(OS=="win" and MSVS_VERSION[0:4]>="2012")'
 ' or llvm_version>=33 or gas_version>=223', {
```


Missing brackets


TeamCity Azmq :: Build (Windows) [Build 112](https://78.46.71.115/viewLog.html?buildId=455&buildTypeId=Azmq_Build&guest=1) is now running


TeamCity Azmq :: Build (Linux) [Build 86](https://78.46.71.115/viewLog.html?buildId=453&buildTypeId=Azmq_BuildLinux&guest=1) is now running


TeamCity Azmq :: Build (OSX) [Build 70](https://78.46.71.115/viewLog.html?buildId=454&buildTypeId=Azmq_BuildOsx&guest=1) is now running


r+


Typos in commit log:

* 'add support older asm compiler' => 'add support for older assembler' or 'support older assembler' ('asm compiler' is something of a misnomer.)

* s/asm_obsolute/asm_obsolete/


saw approval from graydon
at https://github.com/graydon/stellar-core/commit/57622e2640761acd1f9b9580883a55ce3de7b830


merging graydon/stellar-core/faster-event-cancel = 57622e26 into auto


graydon/stellar-core/faster-event-cancel = 57622e26 merged ok, testing candidate = 9093ad00


A receipt **subscribe** resource is used by the application **server** to create


let's instantiate the source, sink, and transforms in one place instead of spreading it out throughout the validation. Also,  should catch exceptions that might pop up, since failure to instantiate the class is most likely due to invalid input from a wrong class name or something like that, and we want those types of errors to result in a 400.


kubernetes/pause image is easiest if you don't want to do anything to the pod.


No, hostPorts are discouraged. containerports are fine and necessary for a service to work.


TeamCity Azmq :: Build (OSX) [Build 70](https://78.46.71.115/viewLog.html?buildId=454&buildTypeId=Azmq_BuildOsx&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:55


TeamCity Azmq :: Build (Windows) [Build 112](https://78.46.71.115/viewLog.html?buildId=455&buildTypeId=Azmq_Build&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:01:59


oh i totally missed that was a containerport :\


Yeah I agree with this one, though hopefully this test won't even last long enough for this to matter.


TeamCity Azmq :: Build (Linux) [Build 86](https://78.46.71.115/viewLog.html?buildId=453&buildTypeId=Azmq_BuildLinux&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 19 Build time: 00:02:39


Hmmmmmm


What's this :-)


We haven't been good about this... we're going to have to fix so many places that one more isn't going to be an issue. I would suggest NOT using json though, because that's not compile-time checkable; rather, use a versioned api object if you want to go down that route. (e.g. pkg/api/v1beta3)


@bors: retry


Is it necessary to check in the PDF file?


similar comment for other places where the config is invalid


@splinterofchaos just suggested this change :) (damn github discarding old commit comments  )


s/opnessl/openssl/


s/wrote/written/


The former.  I had to change code to get it to work locally though, so I
don't think the new code will work in production.  What I did was go to the
developer console and enable the Google+ API.

Basically this issue:
https://github.com/jaredhanson/passport-google-oauth/issues/46

On Fri, Apr 10, 2015 at 6:07 PM, Michael Lai <notifications@github.com>
wrote:

> Yeah I did, it's on my account. Is it being deprecated or something? Did
> you find out about the change just because you were trying to run it and it
> didn't work or did you see like an email about it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/howardc93/dukeexchange/commit/bca25e2f5e779ed7c6a4fcf4686e15d6c7aef114#commitcomment-10674894>
> .
>


There can be no integer implementation of getrf, so we really shouldn't have a pending here — it should just skip for integers.


Drat. That may in part have to do with the maximum length limits.


heh oops, but was there something wrong with the suggestion since it wasn't applied?


Wow. Good catch.


You are right, a lot are broken :/
linePlusBarChart
cumulativeLineChart
multiBarHorizontalChart
multiBarChart
multiBarChartDate
scatterChart


If you want to remove the **MUST** then we should be consistent. There's another one in Section 3 for the similar Location case. Is there a reason that it should be eliminated? Style? Just curious.


Shouldn't we comment (or remove) unused parameter name instead ?


@ajlanghorn thanks for this pull request! We'll merge it shortly.

We're also looking into creating a VMWare fusion supported box.


Nono, I meant he suggested the reverse change (remove else)


Do we want to be consistent about "learned", "provided", "distributed", etc for the two resources that are shared with the AS by the UA?


You know, if maintaining the assembly version of RC4 is a pain to support, I think a reasonably case can be made for dropping it.  The cipher itself is being phased out and OS X is not a hugely performance-sensitive platform.


![1 me](https://me.jpg)


develop (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2099 of 2197 tests passed)

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


But it's nice to have documentation!


I'll be fixing that one!


Are we sure that `500` is the correct error code for this? It seems that `404` which is generic "not found" error code on the Web might be most appropriate...  Let's double check with Shannon via tracker, unless, you of course already have. Thanks.


@cmungall sparql now sits here https://github.com/jhpoelen/eol-globi-data/blob/master/eol-globi-data-sources/src/main/resources/org/eol/globi/data/find-jsonld-interactions.rq and is included in [a test](https://github.com/jhpoelen/eol-globi-data/blob/master/eol-globi-data-tool/src/test/java/org/eol/globi/data/StudyImporterForJSONLDTest.java) that is executed against [a static dataset](https://github.com/jhpoelen/eol-globi-data/blob/master/eol-globi-data-tool/src/test/resources/org/eol/globi/data/globi-jsonld/globi-dataset.jsonld). I am open to moving (test) data around if you have other suggestion for location.


The package is already functional, will tag it later today.


and while I _slightly_ prefer declare variable at first assignment (as it is right now), I could do it whichever way :)


Yeah, a bit of googling confirmed that the standard requires zero-initialization prior to anything else. Does that mean we're good to go on this, or do want more testing/contemplation?


Great - thanks! That's one thing less that I need to poke @matjohn2 about ;)


Matteo happened to specify a session ID like `./2753247532452938568`, which is in principle invalid.  So the session was not found in the DB, obviously, but the utils then fall back to read from a cache file, like:

```
open ("%s/%s.json" % (cachedir, sid))
```

which happens to work with that specific ID format, so it could read the data from cache all right after all :P  But the session ID in the frame was taken from the input parameter, and thus remained wrong -- now its taken from the data itself, and is thus cleaned.

Independently, Matteo fixed his ID parsing, of course ;)


Nothing to do here


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.188](http://teamcity.monogame.net/viewLog.html?buildId=6116&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 481, ignored: 6 Build time: 00:14:40


新規作成


Yes on line 421


Nit.

*as was included* - which was returned (or created) with the push message subscription ...


haha :)


Oi, abigos


Ah okay, I just brought it up because it looks like a redundant initialization when any if statements match. I suppose it's up to you, doesn't really matter.

On April 10, 2015 6:20:56 PM EDT, "Björn Linse" <notifications@github.com> wrote:
>and while I _slightly_ prefer declare variable at first assignment (as
>it is right now), I could do it whichever way :)
>
>---
>Reply to this email directly or view it on GitHub:
>https://github.com/bfredl/neovim/commit/6e5621c435238dbb4452cddf4f4f3c4d005fe859#commitcomment-10675035


You should not add this.


Can you revert this parasite change ?


I would use `YYYY-MM-DD HH:mm`


Just added a partial to make things less cumbersome


I like that, to be able to specify multiple sessions -- but shouldn't we put the loop over session IDs into main, so that this works for all the xyz_session methods?  Like, for plot, stats, etc?


-L/usr/lib still remaining


Cool! :)
By the way, I'm also the author of https://github.com/BitStore/BitStore-iOS and https://github.com/GeoTappy/GeoTappy-iOS


Hmm. What would a UA do with the information about the push and receipt subscribe resources? It is useful? Seems more important to remind the AS with this information in response to requests?

Unrelated question - what happens if a UA or AS attempts to DELETE a receipt subscribe resource?


In terms of names, I was not able to come up with something that's more accurate than `SGD`. It is widely used in both optimization and machine learning (at least at the same level as `NMF`, etc)


@DarkDimius, @odersky: I just realized I needed this special case which seems a bit ugly, two alternatives I can think of are:
- Change `SymDenotation#companionClass` to inspect the type of `companion$class` before Erasure.
- Add a flag `DoNotSemiErase` or something like that, and set it for `COMPANION_CLASS_METHOD`

Of course my solution is not that bad and could be left as is too.


I imagine its here so it can be next to `struct_tail`, which makes sense. Moving to trans also makes sense, but I imagine this *could* be useful outside trans.


@agustinf I'm just trying to say: "you need to run devise, devise invitable and active admin installers." I removed this, is confusing and unnecessary.


:smile:


Yeah I was trying to think of another one, and kinda failed. I mean, I instantly guessed the meaning but I'm biased I guess. https://www.google.com/search?q=sgd+-singapore seems to suggest it doesn't stand for anything else you'd confuse it with.


This seems a bit wordy. If we eliminate the section on returning the link references, then:

*In response to this request, the push server MUST generate a server push for all stored messages that have not yet been delivered.*

-or-

*In response to this request, the push server must generate a server push for all messages that have not yet been delivered.*


Eeep. Will remove.


How about StochasticGradientDescent?
On 10 Apr 2015 17:30, "Dahua Lin" <notifications@github.com> wrote:

> In terms of names, I was not able to come up with something that's more
> accurate than SGD. It is widely used in both optimization and machine
> learning (at least at the same level as NMF, etc)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/JuliaLang/METADATA.jl/commit/5e936a2e89aff61e2c98adadacad166a4c37d52d#commitcomment-10675132>
> .
>


For future commits similar to this, please include the name of the signed version tag the JARs are produced from.


I find the step numbering a bit odd - in the &thin to &fat case, step 3 happens before step 2.


This commit message splits the commit - consider what happens when the subtree's changes are pushed to beast, we will have a commit that says "define NOMINMAX when including boost/coroutine" but the only changes are to add win32_workaround.h which has no mention of coroutine. The commit message could instead read:
"Add workaround include for Windows.h NOMINMAX"


I had this ordering of the struct fields to indicate the logical order of operation, rather than strictly reflecting the struct.


Please attach a GPG-signed "v33" tag to this commit.


Your word-wrap is being a bit too aggressive here.


Thanks a lot! Finally found the needed lines for supporting a swiss-keyboard...


Fix for #39


What about `SGDOptim`?

This makes it clear that the package is about optimization.

I googled a bit, and found `SGD` is also widely interpreted as Singapore Dollars. I guess `SGDOptim` may reduce confusion.


I'd suggest to rename this cask to "emacsformacosx". This is not the original Emacs you'd get by building it from the Gnu Emacs website or installing it via homebrew. EmacsForMacOSX is highly customized and the user should probably be aware that s/he is installing this version.


The reason for allowing the AS to control the receipt subscriptions it that otherwise it can't control how acknowledgments are delivered to it.  If the one push message subscription is distributed to multiple backends, then each backend needs to filter out acknowledgments for messages that it sends and that's a real burden both on the backend and on the push service, which now sends multiple sets of acknowledgments out.


I don't get what you mean by "in order" here. This is I guess the literal order in which trans does the coercion? That is a little confusing because the type in the unsize field is pre-autoref, so the logical order is deref, unsize, ref, not deref, ref, unsize.


Nice work!


Ack.


Yeah, I'd rather not use the M-word unless it is needed.  Here, we are simply defining what success looks like, so MUST is too much.


@bithavoc, you can `npm publish` whenever.  Or add me as an npm contributor or whatever, and I'll do it.


merge


Nice :)


*results in* **a** *response* -or- *returns a response*

or perhaps we could simply echo a variation on the equivalent case:

*A 204 (No Content) status code with no associated server pushes indicates that no messages are presently available, potentially because the messages expired.*


Fixed the other one.


I like it, for what its worth


That would be relatively easy, but I didn't know whether there is a use-case for the other modes too, that's why I did it inside the prof mode for now.


This seems like it will duplicate the work done in `walk_autoderefs` to some extent - is that a problem?