@krontogiannis I don't think having one table with a million partitions, each with one row, is not going to perform much differently to having a single partition with a million rows - the benefits you see with sharding / scaling on table storage occur when you start to share data across partitions. As far as I'm aware, having a single row per partition will not help.


El label deberia estar dentro del <form> y relacionarse de alguna forma con el input de texto al que hace referencia.


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (646 of 646 tests passed)

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


OK, going to the next PR.


Interesting, never seen that before.


from [w3c School](http://www.w3schools.com/bootstrap/bootstrap_jumbotron_header.asp) :

> A page header is like a section divider.

we should be use it to divide the page in "Prices" and "Registration".
do you agree ?


This is extremely complex. We have gone from 2 objects, to something like 10-20 objects. I have a hard time even navigating this to where the features are located.

Also, to get the content that we have in our current genome object, we're now going to have to pull many many other objects out of the workspace, and we cannot do them all at once, because we won't know what refs are in the top level object until we load it up. We'll have to do in parallel. So any routine involving a genome just took an extra 1-2 minutes to run.

Designing web pages that use genome information will get massively more complex with this. We will now need a detailed API for handling this object.

Let's also talk about transition plan... how will you transition the current 100K plus genomes in KBase to this new format? This is a massive transformation...Every genome will have to change.

I feel that the justification for this design must be provided. Why do we need all this complexity? What is driving this?


Por que un -h2-, sino se trata de un titular?


Yay! :+1:


Sounds like a useful change. Smart thinking!


command used:
dd if=/dev/zero of=/cache/testfile bs=190M count=1

results vary from 36MB/s to 55/MB/s
the higher value with 512 54.7/MB/s
the higher value with 2048 53.7/MB/s

but:
dd is not made for benchmark and it return very different value each run
to mitigate difference i tryed to run it in recovery, but gap on each run is too much different to make a real winner


vamos a devolver esto al original para prueba


This one is useless as it is the default. However for test purposes you must change the configuration in src/main to be `email-confirmation`


@grossmj  It's strange the dynamips_id is a the root of the node, instead to be in properties.


Cuidado con alinear verticalmente con números absolutos, es una solución muy poco flexible. Además dependes del margin-top que agrega el navegador, puedes llevarte sorpresas.


This a comment on this commit


1. You are right about messing with CXXFLAGS/CFLAGS.
2. I think that having enable-build32 configurable using ./configure is more friendly for some users.  Also it implies a more formal level of support in 32 bit and not something that is on the user responsibility.


okay. thanks


Si estas flotando los elementos es innecesario declarar el inline-block.
Cuando un elemento se flota automaticamente adopta las características de un box-model block.


This seems inconsistent compared to the previous branch.


Woot!


Un margin-bottom seria mas apropiado.


Hubiera sido mejor declarar estos estilos en el body.


Think about what your narrative will look like with even *1* of these genomes in it. You now have 20 objects in your data panel. Now let's say we have 100 of these genomes in it. Now I have *1000* objects in my narrative. And having 100 genomes in our narratives should not seem outlandish. It should seem completely normal.

Now let's say I want to share my genome with other users? What do I share? How to I copy the entire genome to another user? There was a reason the original typed objects were relatively simple and involved few interlinked objects. The workspace is not the right kind of store for objects with this level of complexity. You might as well just store everything relationally if you're going this route.


hmm... thanks


不知道为什么，查询nvarchar类型的字符，前面要添加 N'  .不然查不出


This is only an internal interface (not exported), so I don't personally care whether it's short or long. Jeff, if you like my suggestion of `promote_linear`, I'm fine with that. Or should this just be `promote_type`?


I was thinking about it, but if `TrackSample` there is already something terrible going on. So I decided to does that instead of moving it back to free samples just after processing. On normal usage TrackSample would fail only if allocator is already set, and in never will be at this point. How about just adding assertion for debugging?


Tienes muchas propiedades compartirdas en los inputs (border, font-family, font-size, height). Seria mejor abstraerlas en un solo selector.


asNumber refers to Autonomonus Number that is used in BGP session. I looked at RFC, https://tools.ietf.org/html/rfc5396. There are couple of ways to represent it. So, I picked string so that it gives flexibility in defining it. I can reduce the size to VARCHAR(15).


My suggestion: delete it all and go back to the original object and think more about minimal changes from the original object.


> FWIW On Windows 8 it does call evr!CMFVideoSample::Release.

"It" being?


Same comment as given for EngineDataCenterVO.java above.


Solucion poco flexible, se descentra cuando el texto cambia de tamaño o salta de linea. Vimos una mejor ( transform: translateY(-50%); ).


Ojo, esta solución no es accesible. Si utilizas display:none es lo mismo que no haber puesto el label. Busca en las notas de clase maneras do ocultar elementos y que sigan siendo accesibles, lo hemos visto.


Why is this hard coded to A4? This should simply use the printer's default page size


Which office do I work in?


Yay! Thanks for a quick patch.


Thanks for your feedback. I want to keep `has_readme` because at some point I'll need it, For example:

```elixir
<%= if has_readme do %>
  <a href="README.html">README</a>
<% end %>
<a href="overview.html">Overview</a>
```

I put the `_` prefix as a reminder.


https://github.com/nodejs/io.js/commit/4488a69fac9a00ec8ce941464cfc16b9078249ee @indutny, you fine with this change?


Este componente es bastante chapucero con los h3 con position absolute. Si lo que buscabas era que el titulo quedara fuera de la caja blanca deberias haber creado un contenedor extra solo para el img y los tags.


I've already talked with Tracey a little bit about the idea.  Lemme know what happens/if you want help or contacts


Okay.

Please reduce and push your commit. I will merge it tonight. :)

I will also test it on xen and kvm, running another bunch of tests. If all goes fine, i will get it on ACS.

Thanks!

Cheers,
Wilder

Sent from my iPhone

On 04 Jun 2015, at 18:50, Suresh Ramamurthy <notifications@github.com<mailto:notifications@github.com>> wrote:

asNumber refers to Autonomonus Number that is used in BGP session. I looked at RFC, https://tools.ietf.org/html/rfc5396. There are couple of ways to represent it. So, I picked string so that it gives flexibility in defining it. I can reduce the size to VARCHAR(15).

—
Reply to this email directly or view it on GitHub<https://github.com/sureshr-hasvik/cloudstack/commit/e672a8841967cf7c302aab738a7d19a2501fb703#commitcomment-11525502>.


i was looking also on stock script and sony use caf value

512 on /sys/block/mmcblk0/bdi/read_ahead_kb


Comment


same here - this needs to take the data from preferences


K, done. Thought it was messing with markdown. But maybe wrong..


Mnau is a donut.


ditto


I looked at other commits done when they added ip6cidr. Interestingly some places they had 255 and some had 50.

Actually, we are cidr format. IPv6 is 32 character + 7 separator but we also have need to store the network prefix. So, we can have VARCHAR(50).


À la ligne 75, tu peux rajouter un "| tail -n 3" pour n'afficher que les 3 dernière lignes ;)


same here


as i remember, this partially wrong without damage-related structure, maked for save real damage dealer and first spell...


Ojo con el padding, esta haciendo que el input y el boton tengan diferentes alturas.


Hey! Anything else?


@Tedwikman  Thanks a lot! I really appreciate your contribution to this project.


Looks good!


I think if should indeed be rewritten as
```C
if (gpu_amd(device_info[sequential_id])) {
```
even if `gpu_amd` is a macro.

I grepped for similar code and found an even more confusing code snippet:
```
mask.c-1050-                            if (mask[j] == '\\') {
mask.c-1051-                                    j++;
mask.c-1052-                                    if (j >= load_cl(cl_ctr)) break;
mask.c:1053:                                    if check_n_insert
mask.c-1054-                            }
```
and
```
mask.c:1099:                            else if check_n_insert
mask.c-1100-
mask.c-1101-                            j++;
```


Tou can also have another task, in the next sprint, to fix the places where it's represented as 255.

Thanks again :)

Cheers,
Wilder

Sent from my iPhone

On 04 Jun 2015, at 18:54, Suresh Ramamurthy <notifications@github.com<mailto:notifications@github.com>> wrote:

I looked at other commits done when they added ip6cidr. Interestingly some places they had 255 and some had 50.

Actually, we are cidr format. IPv6 is 32 character + 7 separator but we also have need to store the network prefix. So, we can have VARCHAR(50).

—
Reply to this email directly or view it on GitHub<https://github.com/sureshr-hasvik/cloudstack/commit/e672a8841967cf7c302aab738a7d19a2501fb703#commitcomment-11525584>.


Why? :)


I don't think this is the place to be generalizing `std::swap`. I'm content to constrain the existing overloads.


We are close, but this isn't working.
When the appium driver starts up it installs the App (in my case on a real device) and then starts up the App, but then promptly gets this error:
global name 'set_value' is not defined

Did webdriver.py line 49 need to be removed?
WebElement.set_value = set_value


As clarification: the flusher should have its thread.

And yes, I think that this is a sufficient and obvious performance regression that a rollback is in order.


I realize that using NO_PRINTING is consistent with what we do elsewhere. Still...
if(NOT NO_PRINTING) sounds really silly
let's turn this around into one if/else/endif that starts with
if(NO_PRINTING)
-- handle that case
else()
-- handle the printing case
endif()


En una resolución usas inline-block y en otra float para conseguir el resultados muy similares. No puedes resolverlo con el mismo codigo en ambos sitios?


a line comment


Message formatting is not part of "the API," it's a contingent and unspecified detail of the user interface. People should not be creating programmatic dependencies on them. Are people scraping those messages from logs? Can the messages of `assertEquals` et al be changed to be made more coherent as part of version 4.13? Version 5?

In the meantime, since this behavior does not seem to have been very thoroughly thought through, I'd prefer to not provide the option of passing a custom message to `expectThrows`. It can always be added later, right?


1) It was bad and ugly, but it was way to start. Now anyone who want to implement such feature must start from scratch.

2) Still some forgotten lines in header https://github.com/xbmc/xbmc/blob/master/xbmc/cores/dvdplayer/DVDPlayerVideo.h#L127


RadianceRefactor (rpg777) - 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)


Thanks. I believe you master that area better than me :-)


I like the artistic-2.0 license because it mandates that forks must change their name to indicate that they are _not_ the same as the original project. npm turned me on to it.


So `writeText` would have `allowSubstitutes = false` to minimize cache request noise?


It would probably be better to add an internal ctor to ProcessModule which take these as parameters.


Yes, but the idea is to widen the type so that `yield 0; yield ""` will result in y=`number|string`.


Yeah.


okay. thanks


FYI the `x=None` is in there because when using direct mode, `self.node.size_in == 0`.


This should be cached by making filever_class static and doing if (!filevar_class) ...


these will probably want to be flipped around to match the other scripts, something like
```groovy
params.experiment = "${baseDir}/tutorial"
finalDir = file("${params.experiment}/final")
observed = Channel.fromPath("${finalDir}/**observed.gz")
```
That can be worked out after this pull request is merged.


needs a license header, see e.g. https://github.com/nmdp-bioinformatics/flow/blob/master/main.nf#L3


Hm, I didn't know that. In that case we should nuke the linux.dev reference specifically, or prevent it from being compiled in.


This should be cached.


There is another use of `date` further down this file.


the license header on the expected.xml file does not match this repo; and should the data in there be obfuscated/deidentified at all?


Still don't think `byt` or `surr` need to be declared ahead of time, since `byt = dat[pos += 1]` is indexing into a `Vector{UInt8}` and `get_continuation` always returns a `UInt32`. Even with `ch` you might not need `local`, replacing the `ch = dat[pos += 1]` line with `ch = UInt32(dat[pos += 1])` might work just as well.


I like that for documenting just what those are... they aren't really generic...


LGTM!


thanks.
Indeed, the "STR(VERSION)" was ugly


So basically it allowed to use nested types only for first type I think


I am not really sure about this


double $$ typically denotes a private api in angular, is there no better way to get at this?


Ох ну и.... "Убран"


Donde esta el -form- y -label-?


I thiught world matrix was necessary only for shadow. If possible its  better to avoid.


why do we have to mutate the resolver?


:x: newline please


nullptr is *only* in C++11, though.

A default construction, e.g. fine_context(), should do the right thing, right?


> How about just adding assertion for debugging?

Ok.


would a switch be a better control structure for this?


Actually you can just leave it in the main class not the weird subclass. But that does not matter.


Use `Int` instead of `Integer` :)


¿Dónde esta el -form- y el -label-?


It is the case, it's only added when shadow are enabled.


¿Por que un -h2-, sino se trata de un titular?


Probably, you have already seen this `console.log` so I do not really know why I'm writing this.


It's not strictly speaking regression as it is new code added in commit 4dda4e637e4c17a14db6cd265f36f5e8a5050367. Before this commit root-fsck did not exist in initrd at all and paths were either passed as unit names (and implicitly escaped) or root-fsck service in real system used device name directly.


Wilder, thanks for you comments.

I have changes the sizes appropriately...


Just look into Scope.py and TestScope.py.
Other line changes are made by some mistake when trying to commit.


You accidentally added `+` symbols to tex3pt.sthlp.


@Notsosof1 For your next commit please remove the static images that are no longer used. ie. 1.jpg, 2.jpg...


:+1: Just sat down to try to write a PR to add something like this, but found that you've already done it! Awesome!


-span- no lleva margin ni padding por defecto, no hace falta resetearlos.


Thomas,

I don't really know how else to contact you, but my company really likes you project and would love to use it for a "vitrual window" of sorts between our two offices (through a tunnel) and we were wondering if you thought there would be a way to schedule calls to trigger and end at certain points. Being that you're open source I imagine it wouldn't be too extreme to script it in one way or another, and at worst set up a macro to manually click all the right buttons to trigger a call if need be.

If you'd like to educate me a bit about Homer please shoot me an email at: greg@gocapitalusa.com

Thanks a lot, really love your software!


Just by happen stance, I came into this commit and saw the use of `let`.  By doing this you're limiting this code to be usable on only the most recent browsers and you're excluding safari all together.  Also, I think let is currently only supported by firefox nested within a <script> tag with a version defined.

This could lead to problems down the road, my 2 cents.


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


Remove comments.


Move `script` to the end of `body` (before the closing tag).


Hi Reuven,

As you agreed in class this presentation is also our part 4 of the project.

Thank you!


Remove, we don't support IE6 or IE5.


¿Para que el width 50%?


Follow this style guide for JavaScript: https://github.com/airbnb/javascript/tree/master/es5


I have committed all the changes mentioned in the review comments. Thanks for you comments...


Good choice!


hhhhhh


Cuidado con mezclar porcentajes y pixeles absolutos en el responsive. Te puede dar problemas. Si usas porcentajes en el width, utilizalos tambien en los margenes laterales.


It's not JSON, so just use `var cid = '{{ cid }}';`.


Estas consiguiendo el centrado vertical con numero absolutos, en cuanto el texto tine más de una linea o cambia el font-size comienza a descuadrarse. Hemos visto en clases mejores maneras de centrar verticalmente.


```bash
LastComic="$(curl -s 'http://asofterworld.com/rssfeed.php' | xmllint --xpath "//item[last()]/link" - | awk -F'[=<]' '{print $3}')"
seq "${1:-1}" "${2:-$LastComic}"
```

Would be cool to have alt-text slurped down too; I might find some time to hack on this over the weekend.


Definitely you master autotools better than me.  Still, I need to understand the difference between two places in this commit: lines 106-107 vs. lines 117-118 (regarding properly using yes/no).


Por quê uma tag p vazia e mal identada?


question_id :)


@Ironholds I think so. At this point, it strikes me that we're going to be able to provide some very intriguing, mostly descriptive stats and visualizations of a very interesting dataset. So it's progress to even be in a position to say what proportion of packages are tested w/ our objectively applied def'n. Then to give some info on what's going on in the remainder, based on a sample.


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


okay


How does this reduce $digest issues ?


this should have has_many :answers


okay. thanks


Similarly, it would be interesting to run some packages that we call "tested" thought `covr`.


Hi @Gaurav0, thank you for the upgrading. I realize that you may still working on some minor upgrading for the moment, but once you are done, could you please bump the version on npm to 0.0.17? ```ember install``` gives me 0.0.16 for the moment so I will manually substitute to the latest version here:) Thanks!


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-14: OK (351 of 351 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)


+1 I'd been meaning to do that.


mmm I see your point, cause the `return` estatement keeps the same semantic, and just using `endWith` is quite express-like, I would avoid it, and will make sure that you have to do `return endwith(yourResponse)` so we keep consistency and avoid that people forget to return something that they really want.


@jimlyndon Is `deferred` a keyword in ES2015? I thought @honestbleeps was just fixing a typo.


.. but it's missing the changeset for the database. Please add it to the branch and create another pull request.


*thumbs up*


[using nocrypto](https://github.com/sevenEng/ocaml-ci/commit/7d0a1d23a21373cd8a7125670620d3f43a191ce6#diff-a92cc4708fe8a81779c611793b60afd5R22) right now, I'm not familiar with these libraries, what's the reason that we shoud prefer nocrypto here?


closes #4


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.5-LibC++: OK (351 of 351 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)


well... thanks


again Indentation


Indentation


This fix got us a lot further!

https://travis-ci.org/frenetic-lang/frenetic/builds/65438928


In a very roundabout way. :p

Still looking into it, but I'll explain offline.
On Thu, Jun 4, 2015 at 10:18 AM Thomas Burleson <notifications@github.com>
wrote:

> How does this reduce $digest issues ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/angular/material/commit/5372710a7a4764a908f08deeb06481ae71e539d3#commitcomment-11526071>
> .
>


Ah great. Now I need to recode all of the Drakkensteed ACF and the application of the Devoted Tracker feat... AND all my dinosaurs from MM3


hola


well.. thansk


We should try the decorator `ifNoResponse` in both cases when we do have a response and when we don't have it, isnt?


bodyParser gets added twice.


Cool things about you


Same places as before (see stacks I posted earlier).

The difference is that on Windows 7 it fails on line
`m_pHead = pNode->m_pNext;`
when AddRef is called by assignment operator. And on Windows 8 it fails later on line
`FreeNode( pNode );` when Release is called on destruction.

But crashes are not only restricted to `CInterfaceList`, because it does happen also deeper inside evr code.

```
evr!CMFSample::AddRef+0x0
evr!CMFVideoSample::QueryInterface+0x59
evr!CMFVideoMixer::MFTProcessOutput+0xfd
evr!CMFVideoMixer9::MFTProcessOutput+0x2a
mpc_hc!DSObjects::CEVRAllocatorPresenter::GetImageFromMixer+0x121
```

One thing we could also do is to don't use `CInterfaceList` which works on CComQIPtr and just use list with CComPtr. Not that it would change anything.


We are so dense :person_frowning:


Deleted the extra bodyParser

On Thu, Jun 4, 2015 at 10:23 AM, Vlad Berteanu <notifications@github.com>
wrote:

> bodyParser gets added twice.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fuzeus/telegram-server/commit/b548ec2bfb37d8a0593cdd74e81be272dda1ab3e#commitcomment-11526181>
> .
>

--
Tristan Lovell
Kenneth Warner (Partner)
Co-Founder and Chief Product Officer
909-730-1804; 303-818-1459


ok, we are sure that we will have at least 1 mw, that is the default one, so this is safe


Not sure how much this adds over `id` which is more canonical, in my opinion, and more likely to be used in CSS.  Consider setting only either `id` or `data-action` so we don't have to maintain symmetry in it.


Nice solution for a refutation.


Sắp xếp lại code, thêm vài dòng chú thích, đánh dấu version, và viết lại Readme.
Cậu xem thử ổn chưa.


Interesting! Why is this?


OK great

On Thu, Jun 4, 2015, 19:09 Tuan Kuranes <notifications@github.com> wrote:

> It is the case, it's only added when shadow are enabled.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cedricpinson/osgjs/commit/475a242daeec85799486db0ed864472d5794c635#commitcomment-11525891>
> .
>


Please rename to HtmlTagHelper. We have \yii\helpers\Html and it's name similar to your class


For comparison, this is how I would write it. A few things to point out:

- The closure prevents leaking variables in the global namespace (e.g. `window`)
- `setInterval` is defined here rather than on the `<body onload=..>` attribute. This works as long as the script is moved to the end of the body as noted above.
- Renamed the element ID to be more clear. HTML tags should never be included in the name of an ID or CSS class since it couples the structure and presentation.

```javascript
// Wrap in closure to encapsulate names.
(function() {

    // URL to the specific container.
    var url = '{{ url }}',

    var checkContainer = function(url) {
        var xmlhttp = new XMLHttpRequest();

        // Set the state change handler.
        xmlhttp.onreadystatechange = function() {
            if (xmlhttp.readyState === 4 && xmlhttp.status === 200) {
                // Create a short message containing the container status for the subheader
                var data = JSON.parse(xmlhttp.responseText);

                document.getElementById('status').innerHTML = data.status;
                document.getElementById('stdout').innerHTML = data.stdout;
                document.getElementById('stderr').innerHTML = data.stderr;
            }
        };

        xmlhttp.open('POST', url, true);
        xmlhttp.send();
    }

    // Start the interval
    setInterval(function() {
        checkContainer(url);
    }, 5000);

})();
```


okay. thanks for that


BUAH! Amazing, super clear and easy to follow


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


Using `CComPtr` cannot change anything as far as I can tell. What I cannot understand is how we never got that crash reported before if it's so common...


Also the `url` should be directly included in the template as suppose to constructing it here.


It'll be better if we had interface like `HtmlTagHelper::add($attributeName, $attributeValue)`


Just an x86 dll will not be enough, since you cant load a x86 dll from a x64 python version... So you would need 2....


Fixed now!  Thanks for the catch.

On Thu, Jun 4, 2015 at 1:11 PM, Nils Enevoldsen <notifications@github.com>
wrote:

> You accidentally added + symbols to tex3pt.sthlp.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekwolfson/tex3pt/commit/05523d0ef79799f0da439d5dac0e6c1abc493a36#commitcomment-11525915>
> .
>


Now i can see clearly why you wanted the nullify case.

So we can have a middleware that can potentially decide to reset any previous response, if something is detected at that stage.


WOOOOOO CHECKBOXES


Part of #65


One thing im not against through would be to provide dlls as a separate download for people to use. I just don't feel they have a place in the github repo. Because it can be difficult to find recent versions of freetype for windows without compiling yourself...


Why did you  do this?


How bout them commits? #155


Could you add documentation for this function?


D:


boxed_number_simplifications (lefticus) - i386-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


Uh.. Did you forget to add it?


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


type "createMan_a_geOfferOp"


Thanks Martin, all good changes, thank you for the review!


LOL, now that makes sense to me, previously i was thinking, i need to check the documentation, don't know what xit ;P


Ok i read over that code without being tired, and i do see that is is doing what you said it does.


`data-action` removed.


should be named "mIsPassive"


where are the code path that actually use this?


should this cases be constants?


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-clang-3.6-ThreadSanitizer: OK (351 of 351 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)


Same here, we should make nextAction = NEXT_ACTION or something similar


did you close this scope?
I think a { is missing somewhere


Hahaha Im a programmer not spelling bee champ


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (351 of 351 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)


Please check if that code works with regions which support both sigv2 and sigv4 auth.


Thanks, man! I hope it will be accepted :-)


We're evaluating different expressions in each branch, can we use a switch?


Change header
bz-1097344 change error codes for docker run

to something like.

Change error exit codes to help identify docker run errors from container errors.

```
+func RunError(err error) StatusError {
+	if strings.Contains(err.Error(), "no such file or directory") {
+		statusError := StatusError{StatusCode: 127}
+		return statusError
+	} else {
+		statusError := StatusError{StatusCode: 126}
+		return statusError
+	}
```
Only one

return statusError

Could you get an err constant rather then doing the string compare to  "no such file or directory", it should be ENOENT

Remove this
 +	// no docker error, contained process exit code non-zero
and any additional whitespace

+

```
 		// Make this asynchronous to allow the client to write to stdin before having to read the ID
 		waitDisplayID = make(chan struct{})
 		go func() {
-			defer close(waitDisplayID)
-			fmt.Fprintf(cli.out, "%s\n", createResponse.ID)
 		}()
 	}
``` Why are you removing this?


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/e4be97eb79c435f788b112765f0cb144fac41636/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/e4be97eb79c435f788b112765f0cb144fac41636/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/e4be97eb79c435f788b112765f0cb144fac41636/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/e4be97eb79c435f788b112765f0cb144fac41636/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/e4be97eb79c435f788b112765f0cb144fac41636/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


No, it would make the code uglier than it already is


time at the completion of the task


alright, I could call a compromise at that... :)
no need for the failsafe if the download's provided

however, I could accept this for the people who did what I did and downloaded the repo as a zip.
for automated installation, the dll proveded for download should be included with the installation ;)

so we can get rid of these 2 lines for that ;)

would be nice if we could just load a dll from the web
then we wouldn't have to worry about the whole "latest version" thing :P


time of expiration : one week after creation and not at the time we test it.


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-gcc-4.6: OK (351 of 351 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)


s/replica's/replicas'/


added underscore lib (to be installed in ../collab-client/www/lib)


nit: does this belong to the other commit?


This could be used for tracking and analytics within Javascript


@jgoppert is this supposed to be definitive? Isn't it kinda hacky?


boxed_number_simplifications (lefticus) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.67: OK (0 of 0 tests passed)

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


@richardkchapman Please merge and close


I don't see it that way, but it could go either way. In my view the first commit is the functional part of the change; the second commit removes an error return which will always be nil.


yea, but noobs are noobs and will put the '.dll' :P
why not make use of it... lol


boxed_number_simplifications (lefticus) - Win64-Windows-8.1-VisualStudio-12: OK (350 of 351 tests passed)

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


need to add serial comm. and speed dials. unfortunately ubuntu mei speed dials ka problem hai. Most probably i will shift that in the future.


First deploy failed because I didn't push to github before deploying to heroku.


Not easily, node_gtk_id would actually need to take 4 or 5 arguments and that would be even uglier


In Eclipse, you can press Format (Ctrl-Shift-f) and Organize Imports (Ctrl-Shift-o) for Formatting.
And we use 4-spaces instead of spaces. I always review my changing using 'git diff' before doing a commit.

Their might be some old code not properly indented and eclipse formatter will change its indentation.
Remove that formatting, so that commit will only show your code changes.
okay?


Is there a reason you are not using `mvn release:prepare release:perform`, which among other things would have created the Git tag?


cabecitaaaa


Why hasn't this changed also on Crowdin?


ok, danke trotzdem für deine Mühe.
wenn du noch dazu kommst, Google zu entfernen und eine
neue Version erstellt hast, magst du mir diesen schicken?

email: privat@rbuck.info

Gruß


To be clear, I do think there's a lot of un-systematic common/repetitious structure across a lot of our `configure`/`make`/`make install` derivations that could benefit from some consolidation, but I think it's something whose design (principles) we should agree upon


Why did you exclude the old versions from the build? They worked fine, no?


I reasonably certain the proper fix here was to remove the repeated "are", not to remove "the."


Sorry for the delay. We can ignore except for the artifact xml files, or if
you could figure out how to deploy a flat jar with SBT, even better.

I'm back, can you come in tomorrow¿ Any graphs?

Kentaroh Takagaki, MD PhD
(高垣 堅太郎)
On Jun 2, 2015 1:56 PM, "Slentzen Demian" <notifications@github.com> wrote:

> @ktakagaki <https://github.com/ktakagaki>
> Would it be acceptable if I on the dev branch modify .gitignore to include
> most of the .idea/ folder? It has been my experience that including the
> .idea folder when collaborating quickly can lead to merge conflicts and
> runtime errors that are IDE specific and not code/project specific.
>
> @slentzen <https://github.com/slentzen>
> Reminder to myself to look at this section.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ktakagaki/nounou/commit/4120e6b60f0a3ceb5d57dfaf78e3c62e57c50a4a#commitcomment-11474070>
> .
>


why not?


RadianceRefactor (rpg777) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2124 of 2202 tests passed)

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


95652938-jsoncpp (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1162 of 1162 tests passed)

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


Done, along with a check that the directory actually exists.


That's why I made it optional


epic fail


Don't worry about older browsers support because we transpile es6 code into es5 using traceur


Just an idiosyncrasy of how imgix expects URLs to be signed. It makes sense, as well, since the signature can't be part of the parameters it's signing.


nm :) i am dumb


@rec: The ninja support generates a build file for a particular target, and  we only do one target right now. vcxproj is different because you don't have to specify another target. I'm unsure what syntax I'd use if we made it a target (`scons ninja clang.debug` doesn't seem right because clang.debug is not a target anymore `scons ninja_targets=clang.debug ninja` seems redundant).

I'd prefer `--ninja`, but scons handles argument parsing, not SConstruct. So `ninja=some_value` was the next best choice.

I'm also happy that `ninja=1` required minimal changes to SConstruct.


This isn't really "auto-pluralization", as it relies on the user to add an "s" to the type, which means we can assume the pluralized form will be correct, of course it won't catch stuff such as "Puts the  #mice in the trap" but it's good for 90 % of the use cases


This field is only use to calculate the gravatar and here the different tag declared are identifier to checkup in the process of validate data (where regex and other verification are applied)


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.241](http://teamcity.monogame.net/viewLog.html?buildId=8068&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 497, ignored: 6 Build time: 00:21:39


please remove the generated old AmpqService files (groovy, etc...) as this will create an issue on OSX due to case insensitiveness that cannot be fixed locally.


Yep


Fixed line 62, and double-checked that was the last explicit reference to "debug"


I agree. Maybe something along the lines of `composableDerivation`.


It is random. People generally doesn't care if program crashes once in a while. They just want to go back to watching movie asap and if the crash won't happen again why bother? :)

But I see your point. I even rechecked my recent changes, but there is nothing related really. Only related change were here, but I don't see how it would matter... https://github.com/mpc-hc/mpc-hc/commit/2105de827532b5adaa157df5da0d7975ea92127a


@jlucerond Please remove the static images from the next commit. 1.jog, 2.jpg ...


Consider having this return something similar to the success case, something that quacks similarly to a hash.

Consider the use of ```location_score```, and how you're planning on checking if it is an error. I might wrap this string in a hash with an ```error``` key, such that you can do this:

```
#location_score = the output of location score

if location_score[:error].present?
  #do error case
else
  #do success
end
```


:clap:


Might want to make the VCR name more specific


and, separating VCR cassettes for each different call is not a bad idea.


The tool is currently under active development, and not recommended for production load migration as it is. It has never been tested for juno to kilo migration. The README is outdated as well. My only recommendation would be to jump into the sources and try to adapt it to your needs.


And it's gone https://github.com/RS485/LogisticsPipes/commit/469a1c97421cf75b6c782037150c599c59804b25#diff-a0558779144902e9760b51f34b766d24L325


This looks awesome! I had some minor process notes that might be useful to think about, but other than that :+1:


Correct me if I'm misunderstanding here.  This is an protocol which people can add methods to in order to hook into the generic efficient indexing framework, right? I.e. this is more like `promote_rule` – a hook point into a system – than it is an operation that people directly use. In that case it's not exactly "private" – naming does matter – but it's also not appropriate to coopt a normal operator like `*` or `&` or `intersect`, regardless of if the implementation or behavior
*happens* to be similar. How about calling this `index_rule` or something like that to mirror `promote_rule`?


````
if config["fluentd-tag"] == " {
  return ...
}

receiver := &Receiver{
...
```


Hello. This is the deadly_skills branch


Why not just take the params, pop `s` and go from there?


Some wires got crossed.  The pod name needs to be uniquified.  You previously had this as

````
GenerateName:      "pv-scrubber-" + r.name[:44] + "-",
```

Which I think is better, but wrong because bounds.

```
func shorten(str string, n int) string {
  if len(str) <= n {
    return str
  }
  return str[:n]
}
```

plus

```
GenerateName:      "pv-scrubber-" + shorten(r.name, 44) + "-",
```


is this allowed?


Can we consolidate these debug messages?
`logrus.Debugf("logging driver fluentd "+ host + ":" + port + "tag: "+ tag + "container: " + container)`
... Or something like that


@aclark4life I like the idea, but hope we don't open a can of worms re: UI/UX consequences.  Let me know what you think of thoughts below...

* I like the idea of making tasks nestable.

* I think it makes sense to have the OOTB types subclass plone.dexterity.content.Container

* We cannot guarantee that all other types in other add-ons extending behavior are containerish (those that are not should not support subtasks).  It may also be the case that some other types that are like tasks using the behavior won't want sub-task support (e.g. make this opt-in for the behaviors story).

* Most tasks won't contain sub-tasks, so hiding the Contents tab when empty with an action condition (TALES expression) in the FTI may make sense.  Alternately, we could use a different UI idiom for containment and just hide the Contents tab completely, always.

* "Add new" menu confusion: inside a task, if you add a new task, are you adding a sibling or a child; it is surprising how no matter what you do, someone's expectations are going to get violated.  Possible solutions:

  * Distinct nomenclature (look-alike FTI) ('Add New > Subtask').

  * Hide add-new menu and make a button "Add a sub-task" that just adds a contained task.

  * Solve the add-new/containment problem by avoiding stock Plone idioms:
    * Hide "Contents" tab and "Add new" actions menu;
    * Make View tab for Task FTI a custom view;
    * Custom view has, below task metadata, two UI elements for sub-tasks:
      * If sub-tasks exist, enumerate them in an HTML table.
      * Button: "Add a sub-task"
    * My gut says this regime is preferable; hierarchy and containment without the pretense of the folder metaphor?

* Our "Task planner" type (and other folderish types that would use similar behavior, such as UPIQ's "Form Series" type) is a different case.  The outermost container is not a complete task, but may contain some fields/state, views common to tasks (shared configuration for contained tasks for due-rules, notification-rules, common assigned-parties).  Tasks containing other tasks are non-sequitur to this (this is just a schema DRY/re-use problem, IMHO).

* Notification: should subtasks inherit/acquire their notification rules and assigned parties from their parent tasks if omitted (recursively walking up the tree from child to ancestor/parent)?  There may need to be a checkbox for inheriting these configured traits from parents (which would disable/gray-out explicitly setting them on forms for the subtask)?


CSS outline not supported by: Opera Mini (5.0-8.0)


Very good, looks like you already had a test in here for `degree`.


shorten() should go in util as ShortenString() or TruncateString() or something


oops!  The django debug toolbar stuff was for local testing.  I'll remove soon.


very good


To be honest I don't really think it's a new issue. Anyway we will see if we see some changes with your patch...


We can overload `polynomial` to calculate the degree if it is not provided. That way if you know the degree, you provided it and save on the calculation, but if you don't, you can get it calculated. What do you think?


same comment as above


I don't see a reason why section files shouldn't be proper XML, also I was not aiming for backwards compatibility to gtk-doc here.


@AnHardt Yes, all thermistor are connected, usually it jump from 1022 to 1023, but playing a lot with heaters should have warm them slightly.


The `s` parameter won't and shouldn't be present in any user-defined parameters. It's something that must be computed based on the client token, path to the image, and any present image URL API parameters. Thus, no need to pop it off on the way in. (Although we could consider erroring if it's present.

When adding the `s` parameter back in for the final URL, it must be at the end, hence the string concatenation beginning on line 120.


RadianceRefactor (rpg777) - 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)


This is a redundant else, you can reverse the `if` statement and return early above.
Also nice to have the important code with 1 less indent.


TODO: switch to watch ASAP


hmm, I would say you can do two things, clone the repo directly and use npm link. or use the RVM directly via the standalone runtime installer:
https://openfin.co/developers/openfin-runtime-introduction/
https://openfin.co/developers/rvm/


Does this need to be a public symbol?


looks like i missed this when refactoring for [alpha2](https://github.com/jgravois/esri-leaflet-gp/releases/tag/0.0.1-alpha2).  PRs happily accepted.


Yes, I think we need it. It shows the "Logout" button for logged in users (See https://developers.facebook.com/docs/plugins/login-button)


Thanks!


That'd be better, shall I implement that?


Is either of you good at creating these new-fangled figures with tool-tips? Because we're gonna have some where that would be really sweet. Like, hover near this point or bar and get info which package(s) it pertains to.


Thanks for the heads up. Is it worth factoring this line into its own function?


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-MacOS-10.9-clang: Tests Failed

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


This is great feedback. Thanks!


The code stucks with cblm_max_age=1

There is an error in line 39 of DynamicCacheBasedLanguageModel.cpp

for (unsigned int i=0; i < m_maxAge; i++)

should be

for (unsigned int i=0; i <= m_maxAge; i++)


>numLogin
is **NOT** tested yet!
My blame!


Apparently the nether factory manager has to be last

On Thursday, June 4, 2015, Travis Christian <notifications@github.com>
wrote:

> Do I even want to know why
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TealNerd/FactoryMod/commit/6c6c539282a785b9d76c4d5fc9c6b8f978fa8143#commitcomment-11524486>
> .
>


If you throw Exceptions here, I will not be able to construct a new Searcher inside another class.
Seriously, don't do it.
I need that.


Really sucks but got no time to handle this


Sorry about this. But I will revert this commit as this is still incomplete. On going work is in https://github.com/Homebrew/homebrew/pull/40378


Yeah, go ahead.


Obviously `Line 36` should not be there.
@streambinder any ideas?


@alexmarsev: Since you were the one to try to get MFC to do what we want with Alt/F10 and menu, do you think this can work without side effects? People are unhappy with the fact (Ctrl | Alt | Shift) + F10 shows the menu when the hotkey is not used in MPC-HC but as a global hotkey in another program.


修改基本信息


Nope.  HostPath/NFS have a different public method to call.


Thanks for details, Somehow i need to provide authentication details to set proxy like below
Step 1-Ran this command to set proxy with credential
npm config set proxy http://UserName:Password@proxy.ZZZZ.com:8080
Step 2:-Then ran command---- npm install -g openfin-cli
This work for me... huuray!!! Step 1 achieved..still lot to go...

But the problem i m facing now when i tried to execute step that you shared with me[https://openfin.co/developers/openfin-runtime-introduction/ ]

Step 3:-openfin --launch --config http://goo.gl/4oYrrb
Its says Openfin is not reconized as a internal or a external command...
Any suggestion?


response['debug'] = logs

This line was added when the debug feature was added. However, there are no logs to get if scribe.py is not used.


90664130_AlternativeModelPAT (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)


unit tests. awesome :)


That's for linking users with journals by picking a random role from the database.


choco/installfile


Actually, if we're just trying to pull all terms in a taxonomy, then the route would be `'/wp/v2/terms/' + this.taxonomy` right?


@seanupton Thanks; good feedback. I'm going to open issues to discuss further.


@Profpatsch, I simplified the text in d827dc6. I hope it's better now.


Also, refactoring this into a separate file sounds a good idea. :+1:


I really like these short, well-defined methods! The last line for each of these methods isn't necessary though, as the last line of a method is implicitly returned in Ruby and assignments return the new value. Although on another note, do we even need the assignment? If you just want to return something, the track search, for example, could be shortened to:

``` ruby
def track_search(query)
  RSpotify::Track.search(query)
end
```


It looks like a hard tab snuck in here.


Maybe this is finally time to override entire  ``CFrameWnd::PreTranslateMessage()``, our hacks are worth half of its size already. Are you interested? I can do that but not today.


More hard tabs here.


I'm going to refactor my use of VCR in a separate mini-branch when we get a moment to pair, since I'd like to know more about the convention there.


# README #


Can you review this change please Matt? Not sure why I had to do this :-)


This is fine, but for `$*` methods I think using a `$save(success, failure)` signature makes it easier to read later when you need to change things. Also placing those handlers within the parent function helps later edits too.


Commenting on the updated pull


I just realized that our plan of short-circuiting updates can't work, because there is no way to know if the rerun was triggered by Tracker or props or state. You would need a previousState and previousProps argument to be able to compare.


@jackdempsey can you push version `0.1.1` to rubygems please? Thanks!


Care to explain why?


I have some problems with the audio cutting out, any suggestions?


Find updated changes


Changes


YES!


nice to know: `$(document).ready(function(){ ... }` can be written as well as: `$(function(){ .. })`, it's shorthand :)


I disagree that the upload finished signal should be emitted when it fails.
I think that instead of implementing your own handler of the signal and relying on it being called every time to dequeue tasks, you could call into what is now `handle_upload_finished()` manually. Then that function would gain a parameter whether to emit the signal or not.


first of all, if you don't need to do anything after slide is done, you don't have to put callback, `$('.menu').slideToggle('slow');` makes same thing


tsk_seruntime doesn't be available in the stock linux kernel, and was causing our builders to fail (we automatically build the sysdig driver against ~20 linux kernels).

Based on http://ck.kolivas.org/patches/bfs/4.0/4.0/4.0-sched-bfs-462.patch, it looks like tsk_seruntime is a BFS-specific symbol, and its definition is inside a CONFIG_SCHED_BFS ifdef.


second, nice to know: if you want to pass empty function, there is another handy shorthand for that: `$.noop` which means 'no operation' which is equal to `function(){}`


OK.


The commit messages first line should read:
```
ADIOS & HDF5: use OpenMP to speedup for loops
```


sir, please update the library for FPS-GT-511C1 that contains all the functions of the sensor thank you.


I just saw it myself - I could have sworn the macro was generic, but apparently it is introduced by the BFS patchset, albeit for both schedulers (which is why my test worked). Thanks for spotting!


Ok, no more testing this way. I'm installing lubuntu, fire everything up, and getting stuff done.
I'll also newify my website sometime.


congrats man, this is pice of a very nice jQuery code!


Can't you call `shutdown()` on the mock server instead of doing hara-kiri?


I don't know how to do `e.preventDefault` and `handleSelect.bind(this, pagenumber)` in the same element, so I keep binding click event on `li`.
I think we can replace `a` with `SafeAnchor` when it's ready.


I think we're still missing mention of these two:

```
# FEC_WEB_API_KEY
# FEC_WEB_API_KEY_PUBLIC
```


units are redundant when value is 0, good practice is to use just `margin: 0;` in that example


padding: 0;


Regarding your question, Ctrl-F10 still shows the menu.


all divs have width set to auto by default which will be 100% in that case


`/>` (and `</` in the end tag) is really one token, so I guess it should be here too?


Ohh I see. I just thought that it would give an exception so... Yeah using singletons would not make sense either.

Yeah. Let me know if we need to add other operations. Lol yea but you are better than me at this.

Exactly. And would we need to swap while we evolve the list? I mean Shouldn't mutation or crossover do that for us in future?

Ahh, so ShiftL means shift to the left, stupid me. So, I would assume lets just shift one element/some(random) number of elements in the list and not the whole list. Otherwise it would be just like reverse, right?

Oh, I forgot to add those instances on github. I have it on the same file on my computer. Will patch it.

What should be our next step now? Functions on list?
I have few created in the genprotov1. We can just use them?


This is fine for this version, but we may want to a reminder to optimize this on later versions.


hi


Also in general I am wondering if tests shouldn't have a way to tell the server to shut down instead of having it time out.


Still not fixed StringExtentions[.cs] should be StringExtensions[.cs]


Is nonolo a male or female cat?


Shouldn't `FALSE` be returned in this case?


TCP에선 멀티플렉싱을 위해 select 함수를 쓸 수 있을 것이고.. 서버 코드만 수정하면 됩니다.
현재 코드는 한 클라이언트가 접속해 있으면 다른 클라이언트가 들어가지 못하는 형태입니다.


I'm not sure it's a good idea to read the name of the function like this - just because a function is called "std" doesn't mean it's actually calculating standard deviation, and certainly there's no guarantee that it's doing it exactly the same way the optimized version is.  At any rate if you want to make this assumption we should make it very clear in the docs.


The cause being CMainFrame::OnEnterMenuLoop(), which is also a hack. Without it menu will take focus but stay invisible.


uh huh


Ich glaube, du hast da etwas vergessen. Sind nur zwei Bilder und ein Cache-File. :)

Die Bilder hatte ich nicht ins Repository genommen, weil ich dazu keine Lizenzen gefunden hatte. Würde sonst eher nach frei verfügbaren Bildern schauen oder eigene basteln.


Well done :+1: It looks really nice, I will checkout this later and see how it look in the browser, cheers mate


I have already added it.


Is it OK to have no tests for this bug?


Gfb


Cool, that is valuable feedback. Losing a top-tier gear piece doesn't cause the achievement to be revoked, so $exists does sound like the right way to go.


in the code above you use `"` and here there are `'`s, be consistent, good practice is to use apostrophes in js and quotes in html


Aha, excellent. @crookedneighbor puzzled out the basic purpose of `fields`, but what to do next with it didn't click for me. I'll make that change!


I didn't know you had 2 cats.


Ah yes, you are quite right. Will fix!


The signal was actually already being emitted when the upload failed but only for certain types of failures. Are you also saying we should change that existing behavior? It's going to be pretty tough to test the daemon if there's no way to know when it's finished in the error cases.


These links should be fixed to refer to Emacs-D-Mode-Maintainers accounts on travis-ci.org and coveralls.io.  The two sites report the build status and the percentage of code covered in testing.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: Tests Failed

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


Do we need this in here now?


falta fazer os vínculos agora com as FKs :+1:


Correct spelling for 'paypal'


No es necesario subir este archivo


more cleanup needs to be done.


Se tiene que agregar el carrusel para las imagenes


Nope. I'm not even sure how it's still there -- I could have sworn I deleted it.


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - i386-Windows-7-VisualStudio-12: Tests Failed

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


Oh okay, you can disregard my comment then - but you can still optionally decide to take my suggestion and emit the signal from `handle_upload_finished()` instead of implementing a class handler :wink:


Shouldn't this `!$currentTransition` be done inside the `select()` instead of explicitly having to set it within the ng-click?


```sh
~/Git/docker-bench-security$ docker build -t dbs-rhel -f Dockerfile.rhel .
Sending build context to Docker daemon 371.7 kB
Sending build context to Docker daemon
Step 0 : FROM rhel7
Pulling repository rhel7
65de4a13fc7c: Download complete
Status: Downloaded newer image for rhel7:latest
 ---> 65de4a13fc7c
Step 1 : RUN yum install -y yum-utils; yum clean all
 ---> Running in e251961cf526
Loaded plugins: product-id, subscription-manager
This system is not registered to Red Hat Subscription Management. You can use subscription-manager to register.
There are no enabled repos.
 Run "yum repolist all" to see the repos you have.
 You can enable repos with yum-config-manager --enable <repo>
Loaded plugins: product-id, subscription-manager
This system is not registered to Red Hat Subscription Management. You can use subscription-manager to register.
There are no enabled repos.
 Run "yum repolist all" to see the repos you have.
 You can enable repos with yum-config-manager --enable <repo>
INFO[0041] The command [/bin/sh -c yum install -y yum-utils; yum clean all] returned a non-zero code: 1
~/Git/docker-bench-security$ md5sum Dockerfile.rhel
61672146e6ed847888932c08a3c9191b  Dockerfile.rhel
```

docker version:
```sh
~/Git/docker-bench-security$ docker version
Client version: 1.6.2
Client API version: 1.18
Go version (client): go1.4.2
Git commit (client): 7c8fca2
OS/Arch (client): linux/amd64
Server version: 1.6.2
Server API version: 1.18
Go version (server): go1.4.2
Git commit (server): 7c8fca2
OS/Arch (server): linux/amd64
```


Correct spelling for paypal, compatibility and multiple.


If had to try it to figure it out.


hi, this commit brought up this error during compile
art/runtime/utils.cc:1092:24: error: no member named 'IsValid' in 'BacktraceMap

i am assuming this is due to an older kernel version ? [mine]
if i'm on to it right , can you recommend or point me to a fix ?
or is there a way to put some sort of version check on it ?
Again if i'm seeing it right. Thanks. m


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - 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)


Amended the commit again to include the wiring through ```VoldemortConfig```.


Is this the right syntax, then?

```
var query = {
  'items.gear.owned.weapon_wizard_6': {$exists: true},
  'items.gear.owned.armor_wizard_5': {$exists: true},
  'items.gear.owned.head_wizard_5': {$exists: true}
};
```


comment


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: Tests Failed

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


Correct spelling for 'paypal'


Changeset was part of the first commit https://github.com/ua-eas/ksd-kc5.2.1-ua/commit/185d6a38348526ed52b7ab045b5cbbe9f2b2d4f1


Correct spelling for 'paypal' and 'instance'


Nah I write what I think


There's a lot of changes here that I don't totally understand, but it seems like this update has caused nukes in general to be significantly more powerful.  Was fighting King Arthro and his Waterga's were doing well over my maximum HP when I was 15 levels above him.  As another example, the Under Observation BCNM fight is now almost impossible to win as the Sobbing Eyes's spells destroy.  Even non-NM or battlefield monsters appear to be hitting noticeably harder with their nukes.

I'm not sure how to test/confirm these findings.  Anyone able to help?


//deberias cambiar esta linea


Should be able to just call `.strip()`


@astoilkov This is my first PR and it got accepted! :tada:


Or just go ahead an implement an optmized version such as:

```
diff --git a/avocadoserver/views.py b/avocadoserver/views.py
index c33cd50..daf65b8 100644
--- a/avocadoserver/views.py
+++ b/avocadoserver/views.py
@@ -15,6 +15,7 @@
 from avocadoserver import models, serializers, permissions
 from avocadoserver.version import VERSION
 from django.http import Http404
+from django.db.models import Q
 from rest_framework import viewsets, status
 from rest_framework.response import Response
 from rest_framework.decorators import detail_route
@@ -36,14 +37,13 @@ def statistics(request, format=None):
     Returns basic statistics from jobs
     """
     jobs = models.Job.objects.all()
-    passed, failed, other = 0, 0, 0
-    for job in jobs:
-        if job.status.name == 'PASS':
-            passed += 1
-        elif job.status.name == 'FAIL' or job.status.name == 'ERROR':
-            failed += 1
-        else:
-            other += 1
+
+    all = models.Job.objects.count()
+    passed = models.Job.objects.filter(status__name='PASS').count()
+    failed = models.Job.objects.filter(Q(status__name='FAIL') |
+                                       Q(status__name='ERROR')).count()
+    other = all - (passed + failed)
+
     jobs_data = {'passed': passed, 'failed': failed, 'other': other}
     last_job = jobs.last()
     passed, failed, other = 0, 0, 0
```


@JoeShep Can we add these classes via handlebars / ruby helpers rather than via jQuery?


I am happy to hear that. Have you tested that the bower package can be installed corectly?


@chriscool After thinking this through, Ill squash this with this patch itself, As there needs to be corresponding changes in ref-filter.{c,h}


I'd think it'd be better than the routines written in 1995.


As far as I can tell even overriding `CFrameWnd::PreTranslateMessage` completely wouldn't avoid this issue, right? I must say that I don't understand why `OnEnterMenuLoop` gets triggered on `Ctrl+F10`.


:+1: :+1:


@benwilhelm
La número __number__
should be
El número __number__
also,
line 18, for badPhoneNumber, you can use:
Favor de usar un número válido de los Estados Unidos y de 10 digitos
(same as "valid-format" in json)


@mateusortiz can you do a new PR for this one?


@nbougalis Sorry, I should have been clearer in my original comment. You probably want forwarding ref (`String&&`) or we'll get an unneeded copy when someone uses a non-temp `std::string`.


:-1:

JK

:+1:


@slicesofcheese @Mathiman @OsthatoAlfakyn
Get untraced extracted emoji here: https://drive.google.com/file/d/0B0snajNgG3_lVV9IYWJaWi1LYjA/view?usp=sharing
If you use one of those I will trace them to get higher quallity pngs at any size we want.


And if you choose to implement the optimized count like the one suggested above, your could say here:

```
last_job = models.Job.objects.all()[0]
```

or

```
last_job = models.Job.objects.order_by('time')[0]
```

Which I prefer since this won't change even if the default ordering for the model changes.


:boom: :+1:


@benwilhelm
lin line 18, **suscribir** ('b' appears only once at the end) is an alternate spelling and can be included _as an additional option_, not replacing subscribir


Alternatively:
The cause of those crashes is most likely use after free, which means somewhere refcount of IMFSample reaches zero, but it's still used afterwards. ``IMFTransform::ProcessOutput()`` doesn't increase that refcount, I mean it appears to be fully synchronous - you call it will 1 refcount, it returns with 1 refcount. So it's either our pointer logic goes fubar somewhere, or ``IMFTransform::ProcessOutput()`` has a bug with double release. Two ``ENSURE(refcount == 1)`` before and after the call to  ``IMFTransform::ProcessOutput()`` should localize the problem.

But it depends on whether on not we receive these crashes in recent nightly builds, otherwise in will take too much time.


Made update


Seems good to me. `localDate` should be set at the moment as close as possible to the "server" timestamp.


@locutus2 Thanks for your explanations, Stefan. As all tests have finished almost even, it seems the bonus for having opposition sometimes helps, and sometimes hurts. Or doesn't help at all.


How can we mimic the old behavior? Separate our .sass calls with an output file specified?


Ah, I totally forgot about `shutdown()`. Will do that instead.

When the tests fail they don't bother to tell the server to shutdown, because the shutdown handler we register with [`g_test_add`](https://developer.gnome.org/glib/stable/glib-Testing.html#g-test-add) does not get called. I could have the tests register a handler for some subset of `SIGHUP`, `SIGUSR1`, `SIGUSR2`, `SIGTERM`, `SIGABRT`, and `SIGINT` using `[g_unix_signal_add](https://developer.gnome.org/glib/stable/glib-UNIX-specific-utilities-and-integration.html#g-unix-signal-add)` that tells the server it's time has come. Does that seem better?


Made change


Submit a PR?


df


@benwilhelm I'm glad its all coming together with the twilio!


Oh, well nvm then!  Thanks for the response.


You should just be able to use `animation` here since we're using autoprefixer


Why do you think is better? What data /tests have you collected to know that?


This makes me think we should add `passrate()`, `pass_count()`, `fail_count()` and such utility methods to `Job`.


this change should not be in the PR.


Uhm, I think we can manage to have it simplier than you made it.
Have to check it out. Hope to give you few updates in few hours.

Cheers, mate!


@jpfr, you break the semi-public API of the network layer, @LEvertz will not be happy to see it ) Can we consider it it stable now?


Yes. In fact, the question was asked before and I wrote a (polite) answer:
https://github.com/jenkinsci/monitoring-plugin/commit/bd062e797ba3578118d1f0facda00568d73150f9

In summary, if a tool does not work at this moment, I prefer to use another way with the same final result than losing my free time. It may be better another time.


Same as previous suggestion, I'd write this as:

```
last_job = models.Job.objects.order_by('time')[0:10]
```


Indeed.  Good catch. Thanks.

On Thu, Jun 4, 2015, 13:44 Casey Carter <notifications@github.com> wrote:

> I reasonably certain the proper fix here was to remove the repeated "are",
> not to remove "the."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cplusplus/concepts-ts/commit/81e26c84955d6397a5cd80df88bd6cfb771d3b38#commitcomment-11526596>
> .
>


L'erreur de correction de "**quelles** étaient les gains" vers "**quel** étaient les gains" et corrigé par "**quels** étaient les gains" dans le prochain commit.


Of course the same kind of approach could be used here. It's your call do some or none of it though. Implement first, optimize later.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 49](https://teamcity.labkey.org/viewLog.html?buildId=253592&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:10:59
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

```


Hey @jgmize I removed this class, and now there are some Travis tests that are failing. Wondering if I should also be removing some tests from test_views.py? Can you take a look or point me in the right direction? Thanks!


What would this look like for syntax errors (like I misspelled datsets or gave an incorrect granularity)? For this case, based on what is here, succeeded = 0, failed = 0, failures has 1 or more entries w/ data: nil, reason: <string> - which I suppose is OK.


rax retry tests


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - Win64-Windows-7-VisualStudio-12: Tests Failed

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


Only through Shiny apps :(


> I must say that I don't understand why OnEnterMenuLoop gets triggered on Ctrl+F10.

Throw a breakpoint there and enjoy the stacktrace. WM_SYSKEYUP through the default message handler becomes WM_SYSCOMMAND(SC_KEYMENU).

> As far as I can tell even overriding CFrameWnd::PreTranslateMessage completely wouldn't avoid this issue, right?

Not directly, no. Still we can "eat" WM_SYSKEYUP message there by returning non-zero, but it may backfire somewhere. The correct way is probably to handle WM_SYSKEYUP F10 message before it goes to default handler. Argh, mfc is annoying.


You can just use `options={}` above


Aren't unknown values in data supposed to be represented as `nan`s?


Have to admit. I learned alot from your commits. And I extracted some work you've done here. I have to thank you in name of our community the work you shared.

I was a full o3 pursuit and now I also rolled back to o2 thanks to a suggestion by Sultan and also by have seen this in your git. Never understood the porpuse of having a separate code to activate o3,and o must admit, gives much better results for end user.

Now my  mystery is, why keep thumb with Os flag? Should it be with o2 or o3 to be paired?

Thanks.


Ideally shouldn't edit the options object in case someone wants to reuse it for multiple calls to this plugin for example


very useful indeed


Looks like a good occasion for a helper to support this "empty node" pattern :)


Thank you very much, I did already merge your code and it seems to work. There was another issue with latest telegram-cli which prevented IRC -> TG from working, output syntax had changed slightly. It's now fixed in `fruitiex/teleirc`. Thanks again! :)


OK.


> But it depends on whether on not we receive these crashes in recent nightly builds, otherwise in will take too much time.

What do you mean exactly?

Also I'm not sure the problem is here because if `IMFTransform::ProcessOutput()` really was returning with 0 refcount the sample would have been released already and we would crash just after `IMFTransform::ProcessOutput()` which isn't the case here.


This was intentional; here is the relevant excerpt from this method's documentation:

```
  * Returns %TRUE on success, even if the metrics are intentionally dropped due
  * to space limitations.  Returns %FALSE only on I/O error.
```

This lets us [ensure that no I/O error occurred](https://github.com/endlessm/eos-event-recorder-daemon/blob/master/tests/daemon/test-persistent-cache.c#L1127-L1181) without adding a `GError` to the API.


Maybe `file.relative` would do here?


Ah, sorry, I meant to ask why Imgix doesn't do that. I guess it would change if the params were reordered so the only way to do it without great user error would be to dictate that it always comes at the end, yeah?


Even if Oracle recognize them, the data is translated into pandas and pandas will print a blank space anyways which might be less confusing as Null. I agree


@alexcrichton, could you add `0.3.0` tag to this commit in git?


tested and working


Good Choice


Yeah; you might find this snippet useful
https://github.com/endlessm/xapian-bridge/blob/master/test/test-daemon.c#L13


@craig-riecke this is my attempt to fix the travis test...


At some point I might add the optional argument to specify the name of the table being created/appended so it can use a generic file


Thanks


Are these lines only comments (that can be removed)?


需要选上 CJK 字体。


Relates to 4922899


Maybe this can be replaced by text explaining that this version is not currently configurable. Or is it?


其实可以不用删掉 这个是git submodule，会引用其他项目的commit，win下可以使用吗。删掉的话merge进主分支会造成它被删掉


DistScope should probably be DistributionScope

Lets not shorten names to prevent confusion.

How is this different then License?

It would probably be better to break these up into three discussions.  Rather then adding three labels at once.

Componant, is this a URL to report bugs?  Should we have a ComponantURL?


I for once would like to see this commit split into two: one containing the 3rd party content, and a second one with the code you authored.


i think that should actually be OK, since you can call `stat` on `IO` file objects:
```
julia> methods(stat)
stat(s::IOStream) at iostream.jl:30      stat(fd::Integer) at stat.jl:52           stat(f::Base.FS.File) at fs.jl:279
stat(fd::Base.Libc.RawFD) at stat.jl:51  stat(path::AbstractString) at stat.jl:53  stat(path...) at stat.jl:56
```


Sorry, yes. I just added the API key envvars to `manifest_prod.yml`, since stage and dev don't use them.


I still have the stacking problem on my server :(


Thanks, dude! This'll work for now :-)

On Wed, 3 Jun 2015 at 16:28 Rory McKinley <notifications@github.com> wrote:

> @taylordowns2000 <https://github.com/taylordowns2000> I have hacked in a
> solution that works on my machine (TM). It is not pretty, but may allow you
> to play around, until I can refresh my knowledge of the asset pipeline.
>
>    - In short - you cannot load the particles.js file until the div with
>    id 'particles-js' exists. For now, I have just crammed the config into the
>    html, so that you can play around with it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OpenFn/OpenFn-Site/commit/05aa6e2d7f3c002ef475476deed8afa71e860ace#commitcomment-11508499>
> .
>


Is the 1.x series going to see a patch? Users who aren't using bson_ext (such as users on JRuby) are still vulnerable.


Interesting idea... @ptomato did you suggest a class handler to avoid the possibility of a stack overflow/difficulty of reading the recursive stack while debugging?


This is mandatory for Fedora, a lesson I learned with Autotest and ARC.


For Fedora, a tag is not good enough here.


Perfect!


Again, this would have to be kept as is.


sweeet


Yes, it will be released this afternoon.


Thanks for merging so fast. I think, it would also be a good idea to create a new release with this fix.


these `debian_` files look weird


It's in master already.


The significance of this file is that it's deliberately hard coded in Kubernetes so that clients connecting from pods know where to retrieve the token from. Changing the path would affect any pod that requires it, meaning that any image would need to be rebuilt so I think it's fairly safe to say it's not going to change.

How about I add it as an option to the `auth_options` parameter & default it to this file location? The whole purpose of a known location for the client file is to enable any client to check a known location for a token & use it if it exists. I'd really like it as a transparent option, but understand your reasoning for not wanting to hard code it, albeit I see the risk as very small.

As for your point that not every client is inside k8s... of course not & this doesn't stop anything working outside of k8s, just makes it nicer when you are running inside it.


That's pretty nice. FYI, @ebassi, @ptomato: ^^.


so _tcslen is no more working in the rest of the code as well. We have 612 occurencies of it.. what am I missing?


Perfect, thanks. https://github.com/mongodb/mongo-ruby-driver/blob/1.x-stable/lib/bson/types/object_id.rb for anyone else who ends up here looking for it, like me. :)


Planning on it, @troxler ; want to get the QA tools passing first (there's some issues with namespaces I need to correct).


Grazie Matteo, much obliged :)


I think this would be better suited as actually testing the Regexp rather than testing it's output.


This will certainly bite us when we proposed this to Fedora.

Maybe using `%{_defaultdocdir}/%{name}` right from the start will save us from getting bitten later. This would have to me matched on the `%install` section though.


:+1: for the lazy initialization


Don't worry, I see it too. The problem is that JSX doesn't support any kind of for loop (and i want to avoid including another plugin), so i just temporarily duplicated this and made a note to find other way to do this.


"(Note that there is no more than one variable name in that vector.)" , I think that the parentheses should be removed, no ?


Looks good to me :+1:


I'm not really sure what to do here. It's a good thing you marked this file as configuration. The location of course is not, but I have no better suggestion right now because I have no idea how the client JS code could look for it elsewhere.


Remove "script".


Remove "script"


```ruby
Regexp.union(site.keep_files.map(&Regexp.method(:escape)))
```


from strlen doc say :

"The strlen function returns the length of the null-terminated string s in
bytes. (In other words, it returns the offset of the terminating null
character within the array.)"


Cool - will do some poking at the asset pipeline tonight and over the weekend and see what I can do ito housekeeping.


You could short this to `= link_to image_tag(video.small_cover_url), video`.


ce genre de code est inutile:
```C
str = (char*) safeRealloc(str, lenStr);
str[lenStr-1] = '\0';
fprintf(stdout, "%s", str); // Print the SQ line in the SAM file
free(str);
```
simplement et plus rapide:
```
fwrite(str,lenStr,sizeof(char),stdout);
````


editDisabled is used on the inputs for every managed business page. If the status of the org is pending, then all the fields are disabled with `disabled={editDisabled}`. In this particular case, we dont want orgs changing there deals after they have been reviewed, to I am checking if the status is active, and disabling deal fields.


good riddance!


These Seeds are not right. First you are hardcoding category to video, then you should use `create!` to pop error on validation if any and then you create Video in 1 line, almost unreadable.

```ruby
comedies = Category.create!(name: "TV Comedies")
Video.create!(title: 'Family Guy',
              description: 'In a wacky Rhode Island town, a dysfunctional family strive to cope with everyday life as they are thrown from one crazy scenario to another.',
              small_cover_url: '/tmp/family_guy.jpg',
              category: comedies)
```


Que copado. Ahora lo hiciste a mano o con un script? Taria bueno tener un script para el tema... :)


Indentation is off.


Haces un pull request?


А это что ещё за подруга?)


Why all videos? All videos should be associated under categories. this shouldn't be here.


> What do you mean exactly?

I mean waiting till the next stable and then some will be too long.

> Also I'm not sure the problem is here because if IMFTransform::ProcessOutput() really was returning with 0 refcount the sample would have been released already and we would crash just after IMFTransform::ProcessOutput() which isn't the case here.

Yes, for debug builds. For release builds... it can be anytime. Also, I'm not 100% it's `` IMFTransform::ProcessOutput()`` and not us who messes up.


This whole class is wrongly indented.


I forgot.  Why is this `raw_args_signature` map to a dict and not to a hash?


<kbd>shift</kbd> is missing ;)


Which part handles the velocity?


and that is how 612 occurencies of   tcslen are working in fact. tcsncpy is placing a zero at the end of the copied string.
in pure unicode  that means 0x00 0x00 , right?
sorry but I dont get it.


If it is strictly necessary i think i forgot why. However, i think that logically the compositor and the backends are two different things and this separation makes things easier, also because the compositor may want to extend weston_compositor itself, to store its stuff.


tl;dr

I think you should add a `bearer_token_file` option in `auth_options` (which would be mutually exclusive with bearer_token).


looks good.


I don't like that this is implicit when no auth option is specificed. You should use `bearer_token_file` (as suggested above).


raw_args_signature -> raw_args or raw_args_string


So this is kinda a workaround. After the organization is active, if the manager of the organization changes any of the details. I thought we would want to review their edits before the change could be made. As I have thought about it, I am thinking that the only thing we should limit is the deals. Keywords, locations, and business description will be up to them. I think I will take this logic out.


Lo hice a mano :(


@KradekMFC Excellent investigating and nice JSON dump there!

I think the difference is that sessionstate is sent on a buffered timer of some sort (meaning people joining/leaving won't instantly trigger the message), and that roomdata seems more responsive to instant change, judging by the numbers.

I notice that the _only_ models with any _big_ variation up/down in room count are the ones who already had loads of viewers, which in turn means they are probably the top camscore model on the site (or one of the top) and therefore ranked first in the list, which would lead loads of guests/people who just opened the site to click their name and increase their view count.

So the differences in viewcount are easily explained by different sampling-times for the room count.

Here are two pronounced examples, one going up and one going down:

```
{"name":"AlessiaBoobs","session_state_rc":34,"roomdata_rc":25}
{"name":"lovely_cath","session_state_rc":444,"roomdata_rc":602}
```

But if the sessionstate is allowed to run until the next batch-update/timer/whatever they use for sessionstate, then it would be updated to the correct count again. And it probably wouldn't take long either, maybe a few minutes judging by the numbers.

I'd stick with session state unless there's a need for the seemingly faster update-rate that roomdata uses.


Yes, see below.


I don't know about that. While I agree the sentence is still understood, I couldn't find references on this variant on the Real Academia de la Lengua Española.
Do you know of any formal references for these rules?

Thanks!


Once a user creates an organization, I wanted to set the user status to manager. What would you suggest as a better solution?


As a side-effect this would also remove the need for the (controversial) SERVICE_ACCOUNT_TOKEN_FILE, that anyway imo you can still provide as a constant. So you could do:

    Client.new(..., auth_options: { :bearer_token_file => SERVICE_ACCOUNT_TOKEN_FILE })

Or just define `SERVICE_ACCOUNT_TOKEN_FILE` on your side.


不过port包里并没有这个值的改动啊 应该要在destroyship的时候更新


Exclude data item of paper delivery


I did that because I read it in the documentation that you can specify a `port` and also `use_ssl` with environment variables (`ENV['WORDPRESS_PORT']` and `ENV['WORDPRESS_USE_SSL']`). I can improve this code adding a default port, e.g.
```ruby
def self.default
  ...
  port = ENV['WORDPRESS_PORT'].empty? ? 80 : ENV['WORDPRESS_PORT'].to_i
  self.new(:host => ENV['WORDPRESS_HOST'], :port => port, :username => ...
end
```
Or we can keep only the `use_ssl` code and put back the port by default to 80.


OK I'll update this PR. Thanks for the review so far @abonas & @simon3z.


Shouldn't this line of code be retained?


Couldn't we add a bit of logic when we user deletes or transfers an org to check how many orgs that user will have left, and if 0 then update user status? Doesn't seem to big of a hassle to me, but then again, i'm igno'ant. Is there a better solution you are thinking of?


@LeoNatan Why is there a title to an action? is it because you plan to add buttons?


Fang=Flinch, but also Claw=Crit


Only through ggvis (and/or Shiny), but I think that's what you're referring to? Good idea!


Crap, this totally somehting you would be able to do.....major security flaw. In this sort of situation, how would one protect against that? I still haven't tested in postman, but I will try to get postman and check.


Тут можно было воспользоваться `this->`


Ну и написал бы его тоже в теле класса..)


This commit closes #14744.


Аналогичная хурма с `this->saver()`


No, I suggested a class handler because it's (supposed to) have slightly better performance than connecting to a signal on the same object that is emitting that signal.


This breaks `init.lua` because there's no `if`(anymore) to match the `else` around line 26.


"мінимально" -> "мінімально"


:thumbsup:


The variable is not used.


Unfinished statement (missing `;`)


Unused variable stats.


This should be unnecessary: The switch statement means that in this context, `t` is exactly equal to `val.(core.AcmeStatus)`. Were you seeing broken behavior?


I don't know how I managed to save these changes in spanish/prtips.utxt into Grey-Echo's fork, but now I cannot make a pull request for it, nor edit it any more. I have already created my own fork for furher translations, but this one is stuck in here.

Can any more experienced, wiser, person help me to untangle myself from this?


Please feel free to add a contributors list and add yourself to it.


fixes #29


why walnut?!


This is good paranoia.


Check https://github.com/trayer3/DeathCube/tree/TechExplosoinHelperbranch branch the mine now only needs to call 1 method... particles work  etc...
no longer looping thru the blocks in range etc..


Nit: nicer if you'd use 0.0 and 1.0 in the tertiary operator.


TeamCity LabkeyTrunk :: CDS postgres 9.3 [Build 50](https://teamcity.labkey.org/viewLog.html?buildId=253598&buildTypeId=LabkeyTrunk_CdsPostgres93) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 0 Build time: 00:03:09
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

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

```


Que archivo?
El 04/06/2015 15:39, "gllambi" <notifications@github.com> escribió:

> No es necesario subir este archivo
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/gllambi/msp-bus/commit/f6abf5cbf0d3f4ac9e3a56cd0dde15ed01d59b6e#commitcomment-11527652>
> .
>


is the compare happening here? in `{key: email}`?


While you're working on this file, can you remove the HACK on line 67? Part of issue https://github.com/letsencrypt/boulder/issues/252, of which @bifurcation already fixed one part.


"Размір" - "Розмір".
Також краще сказати "Розмір поля [{0}] класу [{1}] зі значенням [{2}] ....."


Какой безумно хитрый `static`. Надо бы с ним что-нибудь придумать.
Надо видимо как-то трекер юзать для этих целей, чтобы он отдавал `ProgressSaverCounter`. Ведь глупо выводить первую строчку, если остальные выводиться не будут..


wops my bad, but if i leave it there the travis witch is building my repo will notify your chat..


> unless you wouldn't mind putting together a PR for it.

Will try as soon as possible.


good catch, merged


:thumbsup:


Please use curly brackets, even in one line if statements. This will save you a lot of hassles, as Apple demonstrated with the famous goto fail bug ;-)


Frankly I don't really think this is an improvement, just adds more overhead...


We should probably add a linter to this project. ^_^


Quem usa () num return? hahahaha


Depends on how much eloquent code you have, lol.


ё - правки


needs _extra-close_ review.


Why? Can't you just rename "Location" to "Place"?


> I mean waiting till the next stable and then some will be too long.

We got those crashes everyday, even with nightly.

> and not us who messes up.

We will see after this commit. If it changes anything. It basically will ensure that free samples queue have always valid and not used samples. If it doesn't change anything I will dig deeper.


#Performance issue

After committing this, it was probably better to call the bearer function once, since the bearer apparently remains the same...


HELP
Lo tengo instalado, en mi vps, no se como añadir os plugins o como descargar premium.


It seems there have been merged the merge differences instead of resolved conflict results? Or is it intentional, e.g. to mark the problematic places?


Zone-VS-WSHP-system-sizing-draft (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1168 tests passed)

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


"Редагуваня" - "Редагування"


And... here you have: https://github.com/HiddenHost/backstage_logreader/commit/0859adfb0a02131be3c182f22a2a51f627576bd6 . ;-)


"Ві впевнені що хочете відалити?" - "Ви впевнені що хочете видалити?"


> Yes, for debug builds.

Correction - for debug builds under debugger. And even then it's not 100%...

> We will see after this commit.

I simply proposed an alternative :p
If we get the crashes for recent nightlies, either way is fine really.


As I said previously, after the change to use `s-suffix-p` we should be able to get rid of the compat stuff completely as we only target emacs 24.3 or later.


Again created with NetBeans 7.0.1 native Swing editor.

Image:

![visicut2](https://cloud.githubusercontent.com/assets/8248300/7993205/e50a4890-0b04-11e5-9f1b-3e651f56551b.png)

Red: Most buttons are now represented in top main navigation (camera will be changed soon, not now)
Orange: Added menu "Plugins" for plugin buttons
Green / Purple: Changed some distances between elements in toolbar
Blue: Renamed and sorted "Edit" to "Options"


Great! Glad I could help :)


Feel like it may fix some people and then break others like me.


remove need


@agmcleod Why? I get a warning (and rightly so) when installing:

> npm WARN prefer global grunt-cli@0.1.13 should be installed with -g


Fair enough, let's leave it. Although, you could only have the one empty header since it would be keyed by nil or "". I think we're into bike shedding territory at this point so let's just press on as is. I will land it in a bit


The other way would be instead of throwing it in finally, might as well throw the exception here. I prefer this approach actually.


This completes #5.


I guess this is not true anymore. i.e. A user can write before negotiation is done and `SslHandler` will buffer them. What do you think, @normanmaurer ?


Arreglado Issue https://github.com/CamilaAlvarez/IndicadoresDCC/issues/26


Ah indeed - O_LARGEFILE can be removed, AFAIK it has no effect on Mac anyway. I may have some additional includes from installed ports/homebrew that allowed it to build for me.


@dgreensp Should `trackMeteorData` get props and state as arguments, or should people access `this.props` and `this.state` instead?


With the #ifndef __APPLE__ it is ignoring O_LARGEFILE on my end (and I'm on 10.10.3 as well)


1. Delete extra bracket in post.
2. Revert back to default highlight css.


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.233](http://teamcity.monogame.net/viewLog.html?buildId=7986&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 496, ignored: 6 Build time: 00:20:34


Actually, I think the ConfigurableSocketFactory is not used at all for WebHDFS, so these changes are irrelevant ): ...


问提供你 APN 的人。


Nice!


Automated testing FTW :-)


What about Sundays?


radio 未選擇，返回“”
radio 選擇“存在”，迴歸直線未移動，返回“迴歸直線存在時，別忘記移動獲得迴歸直線哦！”
radio 選擇任意，返回true or false


Added new PR #130 to fix this once again...


Ok, that makes sense. Thanks


why not just check length in `validate_chan()`


per -harmony, `NICKLEN=` designates no limit, otherwise default of 9 (same for channellen/topiclen/etc.)


`validate_nick()` should do length validation


why don't you setting span style to `white-space: pre` to render multiple space instead of replacing text?


This is kind of weird. Could we use a different function for this?


Oh, I was just thinking of Google's (somewhat minimal :)) R style guide: http://google-styleguide.googlecode.com/svn/trunk/Rguide.xml

- Doesn't look like it suggests alphabetical imports, though.
- I think the two things I did notice that you don't follow are: `if (...)` (most style guides suggest a space after `if`), and your funny `#'` function comments :D


this is ok


Oh god your name is going to be tied to this.
I'll merge and deploy it when I get home


what is default value for executors?


### VC11(Visual Studio 2010) から VC12(Visual Studio 2013) への移行について

[このコミット](https://github.com/himika/libSKSE/commit/f039cd61d37d8daa493926edb2aeb6e4b7d2b852)から、Visual Studio 2013に以降しました。

Visual Studio 2010 と Visual Studio 2013 では、デフォルトのインクルードディレクトリが変更されているため

> error C1083: include ファイルを開けません。'winsock2.':No such file or directory

このようなエラーが出る場合があります。その場合はプロジェクトのインクルードディレクトリを適宜書き換えてください。

![VC12 Project Property](wiki/images/vc12property.png)

プロジェクトのプロパティから
* **[構成プロパティ]** → **[VC++ ディレクトリ]** を選択
* **インクルード ディレクトリ**を編集 `$(VC_IncludePath);$(WindowsSDK_IncludePath);$(Home)\include\libSKSE;`
* **ライブラリ ディレクトリ**を編集 `$(VC_IncludePath);$(WindowsSDK_IncludePath);$(Home)\include\libSKSE;`


well you've made some contributions to my work in the past. Thought i'd reciprocate.


My thinking was that since it is a dependency, it should be marked as such. Though i wasn't aware of the warning.


AlienLeaderInterrogation 似乎还没有译完, 我先 merge , 请继续补完 :)


ok


Seems correct to me!


Leave a line note


What do you mean QQfoolsorellina? Where do you want me to fix the link?


will do


Where is var `$strong` from?


I always think of you Haru xD


Thanks very much for the improvement in documentation.  One problem, though. Shouldn't **scrops** be spelled **crops** both here and in the following bullet?


try/finally


Does this need to be `0.15`?


update...


nice.
how about using that same show/hide logic that is now on the calendar icon, applied to the heart icon?
that way we can drop the heart tooltip show/hide switch, and the actionbar UI is more consistent?


@pixunil can you have a look at the behavior when cinnamon-settings->windows->behavior->attach modal dialogs is checked?

We get a couple different behaviors now (none of them right) -

- Nemo or Nautilus, right-click, *delete* a file (not trash), a confirmation dialog pops up, and when you close it, the parent window dims, and never undims.

- Firefox, alt-menu, file->open file, the dialog pops up, and you close, and no dimming occurs ever
- Cinnamon-settings, startup apps->add->choose application, same thing, no dimming.

What should happen on all three of these programs:

1) Modal dialog opens, attached to the titlebar, parent window dims at the same time.
2) once the modal is closed, the parent window should undim.


rax retry tests


nice! it passed.


Added more exercises


Very minimalistic of you.


파일 I/O 비용이 더 클거 같아서,
콤보박스 출력만 보여주는걸로 변경


Different value types are already tested in the serialization tests so this won't break. What is not covered, however, is things like ``test_dict[1] = []; test_dict[1].append(2)``


시작 그리고 디버깅


It sounds good to me.


good!!!!!!!!


u r god


We should make it a separate PR.


Yeah, Will fix it.


Ahh, doh - Thanks :grinning:


merging


You created uberdemand? @Cl3Kener


Minor/curious: do you like having points in addition? (For time series, I usually prefer just lines. For line plots, I like having both)


It's only a dependency to run the grunt command, but if it's installed locally, the command doesn't go into the path, and is only runnable with something like

```bash
$ ./node_modules/grunt-cli/bin/grunt
```

Which is awkward and not what we want. :)


thank you, fixed
https://github.com/AkihiroSuda/zktraffic/commit/86dbc26babfada94ccb11107ba2f4226b191f452


Mm... this is the terminology used in plone.scale;
https://github.com/plone/plone.scale/blob/master/plone/scale/scale.py#L71

...but... it does look like a typo to me. Fixed in the PR. :)

On Thu, Jun 4, 2015 at 10:26 AM, Cris Ewing <notifications@github.com>
wrote:

> Thanks very much for the improvement in documentation. One problem,
> though. Shouldn't *scrops* be spelled *crops* both here and in the
> following bullet?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shadowmint/plone.app.imaging/commit/281baf1fdada8cf92c9b6268573ad7ef77e72527#commitcomment-11513154>
> .
>


On Jun 03 2015, Ismael Mejia wrote:
> nice (connected to the json nice one).

Thanks for the comment. As I started writing these tests, I realized how
much boilerplate we have to write for tests with the standard unittest
module. It's almost like writing Java, but in Python.

With pytest, we don't have to create classes and stuff to create tests. Just
simple functions are enough to test other functions. The code in the final
if (to check if we are running as a program or not) sucks so much. The best
thing is to let the tools discover the tests automatically.

Oh, and using those specialized forms of assertions (like self.assertEquals)
is, I think, a thing inherited from Java (please, correct me if I am
wrong). "Vanilla" assert should be enough for this kind of job.

And notice that the code in the four cases is almost the same, with little
variation, namely, the "if the input is foo, we expect the output to be
bar". With the standard unittest module, if we put everything inside just
one function, then if it dies, we don't know which of the cases was the
failing one.

With pytest, we can still use only one function and it will iterate over
that list and regard those as separate tests. If you want to see an example
in action in coursera-dl, take a look at:

https://github.com/coursera-dl/coursera/blob/9d0ee94d/coursera/test/test_parsing.py#L79

Notice that the test function is moderately complex and that repeating that
as special-case functions would just be code duplication.

I think that we should move to pytest here too. :)

--
Rogério Brito : rbrito@{ime.usp.br,gmail.com} : GPG key 4096R/BCFCAAAA
http://cynic.cc/blog/ : github.com/rbrito : profiles.google.com/rbrito
DebianQA: http://qa.debian.org/developer.php?login=rbrito%40ime.usp.br


I don't think you use `by.date` anywhere?


thank you, I will fix it in future


rax retry tests