It looks like I've missed a use case. What was the problem you found?


FYI: @tribal-tec, @eile, @rdumusc, @bilgili


You need to prefix this commit message with the ticket simliar to how you did the pull request.

https://github.com/phpbb/phpbb/pull/4082

Though not sure how the PM count can go negative to begin with.


ref ant-design/ant-design#668


@basepi Woh, how this could have been merged... :-1:


Names and names and names.  It might resolve itself by breaking out the description of the two commands, but as it stands I'd suggest being specific about <service name> and <selection name>.  Don't give the reader an excuse to get confused!


@pass-by-value why did you did this, this removes LVM support


а все, я ступил


**Code Quality - Nitpick**

Your error code here could be one in the lines of `406` which means the request is **Not Acceptable**. A status code of  `401` in general, indicated that authentication is required, which is currently not the case.


-> "... which means use the container's IP address reported by docker (i.e. as from `docker inspect ...`)"


I'd roll this paragraph into the option descriptions above.  This explains what motivates the two options, which will aid comprehension.


Shouldn't this be a `ConcurrentDictionary` for thread safety?


anotace k $xmlFile? Šmik a levačka je v čudu...


this is a wrong fix , in fact lxc-create => fssize, lxc-clone -> -L...


No problem


> tg-cli command line commands

expanded turns into

> Telegram command line command line commands

beautiful


Isn't this about as gender biased as it was before your change? You might want to get some inspiration here: https://github.com/whatwg/html/commit/08c99122c6fda980b95c90558a2727c69efc1d8f about how this is handled correctly.


Paragraph break before "For example".  It reads a bit like you are going to give an example of the image/tag distinction, rather than the general use of selectors.


Sökfunktionen i toppen strular just nu, går inte att skicka 2 forms till samma sida så den prioriterar den i botten


can commit


**Code Quality - Critical**

Your spacing and indentation should be consistent across your code base. This line should be written as
```
……
  } else {`
……
```


:-1: non-descriptive commit message


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


$region anotace


查看课程方法


necheckouješ, že tam ten index je


According to https://faustedition.uni-wuerzburg.de/wiki/index.php/11._Dezember_2015#Titel


Diese Woche in der Testauswertung für E2:
eressea: /home/eressea/eressea/git/src/kernel/region.c:635: rsetmoney: Assertion `value == 0' failed.


            var client = new RestClient(url);
            var request = new RestRequest();
            var response = client.Execute(request);

            var content = response.Content;

            return content;


@stieglma Is it a problem? Can't we just add lots of symlinks all pointing to the same place?


This commit is poor workaround. Of course I should rebuild a new device-tree for bliss, based on cm-13.0.


Does this list of amberctl features add much? Might be better to just say "... for full details of the use of the amberctl command".


isn't this return pretty useless?


vzhledem k tomu, že jde o api a nemůžeš zaručit, že ti ho nezavolají současně, může ti tady při persistenci nastat duplicate key exception mám takový dojem


TeamCity YaccConstructor :: YC.Core General Build [Build 513](http://teamcity.codebetter.com/viewLog.html?buildId=204312&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) is now running


Uhm im wondering if this is not a lxc-utils retrocompat issue..


It fails somewhere here [1]. Please check serialization stuff in newly
added classes.

[1] https://github.com/yade/trunk/blob/master/py/tests/core.py#L85

Best regards

Anton


how to disable the yellowBox ?? why i release my iOS APP,(schema is release) ,and still have the yellowBox


make sense to the lkl_ prefixed types.

what about ifdefs of machine dependency ? is there any other way to do the job ?


TeamCity YaccConstructor :: YC.Core General Build [Build 513](http://teamcity.codebetter.com/viewLog.html?buildId=204312&buildTypeId=YaccConstructor_YcCoreGeneralBuild&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:01:06


It was the `m.rsc.category.id` that got in the way. Which might have been a confusion with the `m.rsc[category].id` just before it.


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


@liskin please read https://github.com/whatwg/html/pull/405#issuecomment-164907469. Overrepresentation in the other direction is not a problem.


Yes, it was fixed later in https://github.com/structuremap/structuremap/issues/365


舰娘？我没看错吧


loool wtf, im seriously retarded.


@ArthurClemens The problem was that the category category is returned from `m.rsc.category.id`. Obviously, we want to refer to the *variable* `category` instead, which requires `[category]`.


@annevk All right, makes sense (a bit). I was just trolling, anyway.


Nice!


Porušení DI - pravé závislosti jsou jen streetRepository a partCityRepository, prosím, nebuď líny a vypiš ty konkrétní :-)


Wouldn't it be nicer to use `value()` For places where a check previous to `.at()` is done with a subsequent `if( entry ) return;`?
It's easier to read and we're sure not to forget the `.size() < 0` part.
Mainly asking for your opinion, not asking you to change it


Directly pushed to master ... - revert this if you don't like it


$includePartCities = FALSE


No, `.fast` is not needed after the `unmarshaller(_)`.
But you are right in that it should be `.fast.flatMap(...` and `FastFuture.sequence`!


:+1:  :+1:  :+1:

Vi sjunger en sång som sprids över världen
från fri republik och förtryckare stat
den ropar ett namn som håller oss samman
den sjunger om Richard Stallman, vår vän och kamrat


Thanks a lot!


Wouldn't 80 chars be a little bit short? I mean, this is not the kernel, nobody will read this file from a 80x25 terminal (or at least they will have proper auto-wrapping).

I'm fine with wrapping it though, but I'm not sure how XML would deal with the spaces that will be introduced to do proper indentation. Which is partially why I left it all in one line.


thanks


Just nice to show the default as well.


Original had the @ sign missing for the annotation.


Looks worse imho.


No. It was generated by my IDE :(. Thanks!


Some typos here


@anosach-luxoft, good that you've found this peace. :+1:


nevermind, it's right


I added an inotify watch and now I will monitor the fd with sol_fd_add()


File is correct. Spelling is "Ensembl" (no 'e'). Sorry for my commit message, which was wrongly corrected!


awesome @caseyjhol! congrats


While this isn't necessarily wrong it's nice to show field injection in an example too. I think this is the perfect candidate.


tohle ti vrátí vždy v hlavičce kód 200


;) I'm sure it was here for a reason ...


:+1: looks like this could possibly be a `with self.get_sql_engine() as engine` call one day


r+


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


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


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


Ok, I will do that and release a new version asap.


Because the connection code was EVEN shittier and never closed connections to MPD, so the battery drained and MPD's connection pool was exhausted :+1:


awesome.
Design wise, I think it would be beneficial to make more of a point of what icons we have and the icon name for usage.
Again, something similar to http://rizzo.lonelyplanet.com/styleguide/design-elements/interface-icons


Shouldn't this be `@tparam number padding` and `@tparam[opt=tag.selected()] tag t`? The point of `@tparam` is that one can also document the expected type of variables


A better way to do this (using actually correct english) would be to tweak the language a bit further, so you have "a trusted site" instead of "a site they trust", "pick his preferred colour" to "pick a preferred colour", etc.

English (like most european languages) *is* gendered, and *numbered* (well, it at least has concepts of singluar and plural - weaker than e.g. slavic or arabic). Breaking the latter in an attempt to dispense with the former strikes me as a bad path to follow if you're trying to reform the way people think.


why does this work at all with empty query string?


I the version up on github is non functioning. For example that line you
mention. It was a push to test the package would update. I'm not very
experienced with node or js. I plan to finish testing my local version and
push it up today. I'll submit a merge when I am more confident it will
work.

On Wed, Dec 16, 2015, 4:23 AM Mario Kozjak <notifications@github.com> wrote:

> Has this proven valuable to you? Thinking of merging this commit...
>
> Also, why 'var.rxdShellPrompt'? Are you modifying the 'var' object?
> Why not 'var rxdShellPrompt'?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/papaburgs/node-telnet-client/commit/52a4b3e822dd94f9e0d9b0be1bf9781aa178634a#commitcomment-15009852>
> .
>


Was this what you wanted?

`git tag -a v0.7.4 -m "lorem version 0.7.4"`
`git push origin v0.7.4`


sweet. never _really_ knew what fluff did. seems pretty similar to UCR though


Errr, OK desktop GitHub. Why did you merge this?!
I literally have no idea why?


Excellent ! yea I will send more money too.
I'm working with my brother as amateur on a project where this plugin is needed (you can check out some graphisms I did for it there : beloceanpictures.tumblr.com) and that is litteraly unsticking us.
We would have a couple more of little requests though so that it's really fully usable with C2, we will post them in issues. Cheers


Decided to go fancy and use rems :open_mouth:


Вопрос смысловой нагрузки данного действия ?


Das assert scheint mir okay, wer will denn da was anderes als 0 setzen?


Maybe you should wrap the negative `$grid-gap` in parentheses. So `margin: 0 0 0 (-$grid-gap)`. There's no  any difference on the compiled CSS, but might be the right way to do it, i think...


That's a good shout actually, the example/HAML thing isn't super clear for it and also lead to some iffy code which will be good to avoid


`sol_platform_get_system_clock()` returns int64 and `irange.val` is int32, it can easily overflow to a negative value


Yes, that is the one.


Ok i see, there is a possible race of double kwarg , i will fix, ..., that wrong fix to the propper fix.


how come this needs to be in a try except?


Don't see in my head how it is going to be used yet. So probably yes.


> But you are right in that it should be `.fast.flatMap(...`

But doesn't `Unmarshaller.strict` already make it return `FastFuture`?


Oops.  I left all the good info in the commit message and forgot the
comment.   Will fix.
On Dec 16, 2015 6:28 AM, "Tom Aldcroft" <notifications@github.com> wrote:

> E.g. "The obs tstart and tstop times are derived from the OFLS maneuver
> summary file, which for implementation reasons gives the maneuver times
> offset by 10 seconds from the actual command timing. Fix this here."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sot/starcheck/commit/df921578c9dd5120b2a2e7b4c969626088e8dcef#commitcomment-15011195>
> .
>


just realizing all this stuff is just moved. not reviewing closely


If we assume it it always allocated, that simplify things


bø


**Code Quality - Critical**

The indentation used in some code blocks in the script is 1 space.
You should adjust your spacing and indentation so it's consistent across your code base.


maybe better do not recreate buffer all the time?

you can reuse buffer and just extend it when necessary


I see, thanks for the explanation. Merged in 1d4765de5d04155efa387fef2cbb87ba90f200e1


Curiosity mostly but also because cljs-devtools does a bunch of stuff I do not understand (yet). I intentionally made it non-extensible (eg. no protocol) since I don't want dev stuff leaking into production code.

I'm not sure I'll keep this around, integrating cljs-devtools is easy enough and already does more. For now I only wanted a simple implementation I can play around with.


Awesome, thank you :smile:


dadadasd


ggdfd


+ systémové message z výjimek napatří na output


Hopefully solves test-failures for #213.


hht


hfhghggggggggfrtrt


dfggt


I wanted to be platform independent and I got inspired by Linux, where time_t is a signed int


dgge


hththf


Yes, but I have to externalize the allocation of the table outside, and I need it internally to deal with shrinks and growth operations anyway. I will continue today with the tree, I trust I will find out what I can assume and what I cant by then


```CreateIoCompletionPort``` share ownership of ```ck``` pointer (i think in some windows kernel object), so ```ck``` should't be deleted "anyway", only in case of failed ```CreateIoCompletionPort```.


grgrhr


dfgdddf


gdgdfggr


I will talk to you about this function in person.


I am not able to use virtualbox >= 5 because I use windows 10. sorry


rteter


ertetet


ok


gffghhfhf


hhrr


`git commit --amend`? :wink:


There are some lines I don't know how to divide to respect the 80 columns.


@ayende How can I achieve this???


Yes, true. So the `FastFuture.sequence` is the only thing needed, I think.


I'm posting my Source Code to help the public what potentials AutoIT have when it comes to pixel based Automation.


yep, sorry


Hello how can we help you for language traduction (french) ?


again :)
vma_area_list->shared_longest = 0;


My first comment should of course have been "if PvNode is false, isn't RootNode also necessarily false," so the right hand side doesn't change anything.

From your comment I think you want to make some threads disable pruning at PV nodes, so your logic should be one of the following equivalent constructs:

If the node is not a root node AND (the node is not a PV Node AND we are in modified search)

If the node is not (a root node OR (a PV node if we are in modified search))

Truth table:
                     Normal search     Modified Search
RootNode         Don't Prune         Don't Prune
PvNode              Prune                 Don't Prune
Neither               Prune                     Prune


why?


And as usual defining the one-off `$manvr_sum_offset` actually reduces readability for me.  The value of 10 is hard-coded into the OFLS and will never change.


**Test Coverage - Nitpick**

You should use the `.toBe` jasmine comparator so that your test result has more context, when the test fail. Like so:

```
expect(res.status).toBe(403);
```


missing ;


Ok, cool. Thx for confirming!


clearSearchQueries: function () {
    var i, j, len, ref;
    ref = SearchQueries.find().fetch();
    for (j = 0, len = ref.length; j < len; j++) {
      i = ref[j];
      if (i.who === this.userId) {
        SearchQueries.remove(i._id);
      }
    }
  }


I can do a release after Dürrenäsch is deployed.


if you change the sensiolabs project configuration that this is not a web application you do not need these hacks...


but I find your approach is applicable if we just add ```ck_holder.release()``` after last ```throw_system_error_if``` in this scope. as result we can remove template version of ```throw_system_error_if```. great!


Ok, cheers!


Yes, I saw after pushing this commit.


I'll fix this


Do you want me to revert it?


No appreciable change in code behaviour upon workshop projects that do size-bias adjustment or stratified estimates:

### Cluster solutions (preferred method of size bias correction)
```
[[2]][[1]]$`E(s) by ln(s)_g(x)`
         Statistic Distance_value mrds_value  Rel_diff     Pass
1                n      8.800e+01  8.800e+01 0.0000000 <U+2713>
2       parameters      2.000e+00  2.000e+00 0.0000000 <U+2713>
3              AIC      3.187e+02  3.187e+02 0.0000000 <U+2713>
4          Chi^2 p      9.105e-01  9.104e-01 0.0000000 <U+2713>
5              P_a      6.035e-01  6.033e-01 0.0002258
6          CV(P_a)      1.154e-01  1.161e-01 0.0062016
7   log-likelihood     -1.574e+02 -1.574e+02 0.0000000 <U+2713>
8          density      4.991e-02  1.688e-02 0.6617384
9      CV(density)      2.732e-01  2.822e-01 0.0328839
10     density lcl      2.904e-02  9.252e-03 0.6814336
11     density ucl      8.578e-02  3.081e-02 0.6408255
12      density df      4.195e+01  1.230e+01 0.7067263
13     individuals      3.570e+04  1.208e+04 0.6617354
14 CV(individuals)      2.732e-01  2.822e-01 0.0328839
15 individuals lcl      2.077e+04  6.618e+03 0.6814356
16 individuals ucl      6.136e+04  2.204e+04 0.6408280
17  individuals df      4.195e+01  1.230e+01 0.7067263
```

### Stratify solutions
```
[[2]][[1]]$`E(s) by ln(s)_g(x)`
         Statistic Distance_value mrds_value  Rel_diff     Pass
1                n      8.800e+01  8.800e+01 0.0000000 <U+2713>
2       parameters      2.000e+00  2.000e+00 0.0000000 <U+2713>
3              AIC      3.187e+02  3.187e+02 0.0000000 <U+2713>
4          Chi^2 p      9.105e-01  9.104e-01 0.0000000 <U+2713>
5              P_a      6.035e-01  6.033e-01 0.0002258
6          CV(P_a)      1.154e-01  1.161e-01 0.0062016
7   log-likelihood     -1.574e+02 -1.574e+02 0.0000000 <U+2713>
8          density      4.991e-02  1.688e-02 0.6617384
9      CV(density)      2.732e-01  2.822e-01 0.0328839
10     density lcl      2.904e-02  9.252e-03 0.6814336
11     density ucl      8.578e-02  3.081e-02 0.6408255
12      density df      4.195e+01  1.230e+01 0.7067263
13     individuals      3.570e+04  1.208e+04 0.6617354
14 CV(individuals)      2.732e-01  2.822e-01 0.0328839
15 individuals lcl      2.077e+04  6.618e+03 0.6814356
16 individuals ucl      6.136e+04  2.204e+04 0.6408280
17  individuals df      4.195e+01  1.230e+01 0.7067263
```


I can use something else than underscores, but I still prefer members and constructor arguments to have distinguishable names, so that there is zero risk of mistaking one for the other in the constructor body.

Granted, in theory there would be no ambiguity: in the constructor body, "srcfile" would be the argument and this->srcfile would be the member. But an inattentive developer might too easily forget the "this->" and the result may well compile without even a warning, and cause some waste of time debugging why the member is not affected by the constructor as it should have been.


`set` should be changed here as well


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


Postgres compatability


Frech translation.


The logic in these files are likely not 100% correct - I suspect this is the core issue causing https://github.com/jruby/jruby/issues/3505.

@enebo - do you anticipate to have time to look into this in the near future or would you like assistance? I don't hack Java every day so if I will do it, I think I need a pointer on how to build & run the unit tests etc, for starters...


nice. I can push this but it needs to be documented [here](https://raw.githubusercontent.com/pazz/alot/master/docs/source/configuration/hooks.rst).
How about calling it `sanitize_attachment_file_name` ?


а чоч, стримом никак? :) импиративщик


Hi Stefan,

Thanks for the patch.
I think it looks good, but I'ld like to change the param `http_get=False` to `force_http_get=False`.
That might better indicate that it is a boolean value.
I'll try to commit tomorrow.


Nice


@EugeneButusov please pay attention to what you commit. Do a diff before committing to make sure that you don't commit unwanted stuff


good job


my friend !


J'avais tout mis en ordre anti-alphabétique tu as tout cassé :'(


+ e ? :)


In Android 5.x `JNI_OnLoad` is called for all dependencies when `System.loadLibrary` is called. This change guarantees that NativeScript runtime is initialized once. ping @tailsu


and at last
```auto ck_holder = std::make_unique<completion_key>(handle, dirname, impl);```
but we must determine, which version of c++ standard shoud match dir_monitor? which toolsets?


что именно?


ноуп, это не @PathParam, это @QueryParam :)


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


No there is no IRC channel. Currently the rest of the team sits in front of
me :-)

2015-12-16 11:55 GMT+01:00 Saad Zaher <notifications@github.com>:

> Do you have an IRC channel for this package or how do you communicate with
> the rest of the team ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/kontron/python-ipmi/commit/4413c7313067ca84d586a0528b65d5302f587aef#commitcomment-15010629>
> .
>


Hi Eric,

I found your change because I was facing the same problem in eventmap. I first found your "swap" i nthis commit : https://github.com/Eric-Cadwell/nHapi/commit/80ae7b7c7fd433272fd9a00cec01c6a8b6194e4c
Now I find this one where you revert the fix.

What was wrong ? Events are meant to be on the left side while the structure type are on the right, aren't they ?

I will be glad if you can explain this.


I don't see how symlinks should help, because it would have to be located in ```/localhome/stieglma/cvc4-2015-12-15/builds/x86_64-unknown-linux-gnu/production/src/.libs/libcvc4.so.3```. And this directory is just not available for other people than me.


How should I proceed? should I make a new PR with the suggested edits? Or is
it enough to edit+commit+squeeze?

Quoting Patrick Totzke (2015-12-16 13:52:37)
>    nice. I can push this but it needs to be documented [1]here.
>    How about calling it sanitize_attachment_file_name ?
>
>    —
>    Reply to this email directly or [2]view it on GitHub.
>
>    Reverse link: [3]unknown
>
> References
>
>    Visible links
>    1. https://raw.githubusercontent.com/pazz/alot/master/docs/source/configuration/hooks.rst
>    2. https://github.com/cinghiopinghio/alot/commit/1300a85a45fc6d1429b5ccd2834768588b59a9a2#commitcomment-15012945
>    3. https://github.com/cinghiopinghio/alot/commit/1300a85a45fc6d1429b5ccd2834768588b59a9a2#commitcomment-15012945


If you dont have any issues or stuff you want to bring in I can do a new
package for pypi this week! Is this ok for you?

2015-12-16 13:57 GMT+01:00 Heiko Thiery <heiko.thiery@gmail.com>:

> No there is no IRC channel. Currently the rest of the team sits in front
> of me :-)
>
> 2015-12-16 11:55 GMT+01:00 Saad Zaher <notifications@github.com>:
>
>> Do you have an IRC channel for this package or how do you communicate
>> with the rest of the team ?
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/kontron/python-ipmi/commit/4413c7313067ca84d586a0528b65d5302f587aef#commitcomment-15010629>
>> .
>>
>
>


This line is unnecessary, as the first line of this function already sets `mdata`.


Jeje, yo pensé lo mismo, pero luego al ver que existe un archivo de irregulares supuse que los afijos no se aplican y que los irregulares por ser excepciones se ponían en un archivo aparte. Estoy viendo los archivos de afijos -sobre los cuales no sé absolutamente nada- y voy a investigar cómo agregar estas irregularidades.


this event not using any 3.party bundle but we can need soon. So ı won't roll back


It was working for me before what was the issue exactly?


Wait, what part is un-necessary?


Se debe mandar solo el id


return vector_of_shapes.at(constr.obj1).options & GrShape::FixedPoint;


Thanks for fixing it, i just saw it when rereading it.


sure, it's documentation. :)


Yeah, you're right. But then it should be something like:

```ruby
cannot :update_password, User do |target_user|
  enabled = Site.current.local_auth_enabled?
  local = !target_user.no_local_auth?
  if target_user.superuser
    !local
  else
    !enabled || !local
  end
end
```

More complex but it makes more sense for the user.

Maybe we should also not just hide the password inputs, but show them disabled with an alert that "you can't edit your password because you are signed in via Shibboleth/LDAP". But that's another ticket...


ah, ignore me, I didn't see that the previous `lengthCol <- "Length"` had been removed!


tx.Flags == TransactionFlags.ReadWRite


disable**s**


poner en containers y columnas (container, row, col)


Is this still useful? Not all developers have the database, of course, but I suspect that this is a thing you yourslef will no longer use. correct?


The numbering is wrong here


No worries.  R being a strange and new land to me, please to give feed back on what, if any, silly things I have done here ;P.


Maybe rename this event with more generic one


Fixed.


Numbers are incorrect


you can write `return finder.findFullText(entry).flatMap(url -> MimeTypeDetector.isPdfContentType(url));` or something like it.


I suggest removing "This will not be based on any ratios", as it clutters the document with things that don't really need to be there. It also limits admins if they come up with a ratio to use.


To be honest, I would just say:

Subclass of gzip.GzipFile with CRC checks disabled

Monkey-patching is something slightly different.


Yes, right now the app has only one language, and no `messages` is set.


Test


Fixed (2)


unrelated whitespace changes.


faltan espacios


I'm also posting this source code in order get donations for my son Thirdy.


Github is GitHub (sorry).


We must remove empty div and use offsetting : http://getbootstrap.com/css/#grid-offsetting

And if you need to "clear" a row : add an new row : http://getbootstrap.com/css/#grid-example-basic no ?


`head()` gives the first 6 entries by default - is this what you want?


A definitive time period and consensus method would be helpful (like five days after PR is posted, all admins can vote to merge).


No issue here:
```
$ brew fetch -s ssh-copy-id
==> Downloading http://ftp.openbsd.org/pub/OpenBSD/OpenSSH/portable/openssh-7.1p1.tar.gz
######################################################################## 100,0%
Downloaded to: /Library/Caches/Homebrew/ssh-copy-id-7.1p1.tar.gz
SHA1: ed22af19f962262c493fcc6ed8c8826b2761d9b6
SHA256: fc0a6d2d1d063d5c66dffd952493d0cda256cad204f681de0f84ef85b2ad8428
```


Stylistically, this doesn't really need to be on multiple lines and I think reads better when it's not.  You don't need to change it to pass code review, but I'll probably go in and polish it later.


This part should be a title, right?


We also need add the body of the readme.md file


No --- I just want the first one; the front of the list if it's a list.


This is fundamentally broken, you assume here that `viewDate` is always Monday (start of the week), but that’s definitely not true.


możesz <linear-chart chart-data="salesData"></linear-chart> zamiast, imo czytelniej


mjk


Do I just want `head(files, n=1)` or is there a better way?


There are some subtle differences between the meaning of `get()` and `poll()` here.  I've gone back and changed every instance of `poll()` to `get()` in the current code base, so let's do that here.  I need to huddle with Stephané about which we will use, but at the very least it needs to be consistent until we make the call.


I notice all the `adda` lines have an identical structure.  Maybe `adda` could be made into a mixin?  Or maybe that is generally considered bad practice?


> I can see that the duplication isn't ideal.

I checked, and it's only about 200B with clang -O3, so I can't get too worked about about it. :)


kontroler lepiej per dyrektywa


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


It's an error if it's null and shouldn't go silently. I guess it would be useful to have a proper error message here


J'ai pas vu ça dans l'énoncé, pour moi il dit juste de mettre dans un fichier d'output la séquence de déplacements de toutes les voitures, et dans un autre fichier la situation initiale. C'est vrai que la première liste dit que ca doit produire la liste des déplacements de Goal, mais on ne voit aucun exemple avec uniquement ça. Puis de toute façons ça afficherait Est, Est, Est, Est, Sortie...


Let's add `OrientedBoundingBox.fromCorners` that takes `minimum` and `maximum` parameters.

@e-andersson I totally understand if this is too far outside the scope of your work.  If so, @lilleyse or I can do it.


The original has `start time` here, not `events`...


I was thinking that those changes are useless for me..... because I use set_cpu_min_lock fn only for touchboost....


dużo czytelniej jest imo tak jak jest w tym papastyle'u:

app.controller('SalesController', SalesController);

function SalesController($scope) {
 //..
}

SalesController.$inject = ['$scope'];


Et de toutes manières des gens lui ont déjà demandé des précisions sur les output, et en gros il dit qu'on fait un peu comme on veut ^^ donc si on tombe sur un truc pas cohérent, faut trancher, meme si on respecte pas certaines lignes de l'énoncé


forum mesajında username olmaz zaten giriş yaptıkdan sonra cevap yazarsın


why don't you make the name default=host ?


`css.js` is really ugly name, at least `css-utils.js`


//this is an update to the method parse msg..
//the bug was that unset repeated fields were printed as well
// added "if" condition and chaged "if=else" to "continue" condition.

static rapidjson::Value* parse_msg(const Message *msg, rapidjson::Value::AllocatorType& allocator)
    {
        const Descriptor *d = msg->GetDescriptor();
        if (!d)
            return NULL;
        size_t count = d->field_count();

        rapidjson::Value* root = new rapidjson::Value(rapidjson::kObjectType);
        if (!root)
            return NULL;

        for (size_t i = 0; i != count; ++i)
        {

            const FieldDescriptor *field = d->field(i);
            if (!field)
                return NULL;

            const Reflection *ref = msg->GetReflection();
            std::string str;

            if (!ref)
            {
                delete root;
                return NULL;
            }
            if (field->is_optional() && !ref->HasField(*msg, field))
            {
                continue;
            }
            if (field->is_repeated() && ref->FieldSize(*msg, field) == 0)
            {
                continue;
            }

            rapidjson::Value* field_json = field2json(msg, field, allocator);
            rapidjson::Value field_name(field->name().c_str(), field->name().size());
            root->AddMember(field_name, *field_json, allocator);
            delete field_json;

        }
        return root;
    }


Just noticed with the new solver that the print to screen is still default. Can you make this an option rather than default as we discussed?


shouldn't we add them as we have them?


bu blog cevap kısmı olmuş ben sıfırdan blog yazısı için bir form istemiştim


EA jest domyślne, więc to jest redundantne.
Ja zazwyczaj zostawiam samo 'E', chyba, że dyrektywa ma z jakiegoś powodu być atrybutem


bump of the version has been done there

https://github.com/OCA/project/commit/c56b243a53609721211ea4dedc8e57e97390cc38


Related: https://github.com/okulbilisim/ojs/commit/23f646ca49baa3e35d856e477d208e24cc341ac3
I used `OJS_ISSUE_SHOW_VIEW ` is it right use for generic? @hmert


jeden var i kolejne po przecinkach, tzn:

var padding = 20,
  xScale,
  yScale,
itd


aldığım notlardan farklı olarak şifremi unuttum ekranından sonra şifre sıfırlama ekranı da olması gerekiyor


otherwise ack.


template na zewnątrz


nie można całego tego svg do markupu przenieść ?


Basic 不是跟 User 有關連嗎？所以應該加上一個 user_id？以下也是。


True, that's a pretty drastic response. In theory, this never happens, so it's really only there as a better response than doing something worse. I kind of think that this is actually a good thing, but a try catch block might be better?


這邊應該是 basic_id 才可以關聯起來。


@jsvd this has already been merged to master, but I reverted the Gemfile changes in a different PR.


That goes beyond the purpose of this patch :smile:
It's a fine suggestion but it's a refactoring that can be done in a latter time.


Ира, спать нужно в это время! )


[code]shortest > distance[/code]


link imo w ogóle nie potrzebny i atrybuty dyrektywy powinny być na skołpie


Let's remove the title attribute and make the link translatable:
```
<a href="<?php echo esc_url( __( 'http://glotpress.org/', 'glotpress' ) ); ?>"><?php printf( __( 'Proudly powered by %s', 'glotpress' ), 'GlotPress' ); ?></a>
```


It's not weird, transpiler probably converts the integer keys into string keys.

*Dump of var expected is:*
```
{ '3':
   [ { startsAt: '2015-12-03 12:00:00' },
     { startsAt: '2015-12-03 14:00:00' } ],
  '4': [ { startsAt: '2015-12-04 12:00:00' } ],
  '5': [ { startsAt: '2015-12-05 12:00:00' } ] }
```


please use the right indent.
And I dont think following lines are necessary
<os>
    <family>Linux</family>
     <arch>x64</arch>
 </os>


@mtias The only way to get the site out of the sites-dropdown component is via the `onSiteSelect` callback. That means if the user doesn't make a change in the drop down, we wont know what site they have selected.


Done!

On Thu, Dec 17, 2015 at 12:17 AM, Gareth Kennedy <notifications@github.com>
wrote:

> Just noticed with the new solver that the print to screen is still
> default. Can you make this an option rather than default as we discussed?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/underworldcode/underworld2/commit/61e54ea60491ffb30bd4f4a2d02815d0338974cd#commitcomment-15013468>
> .
>


Should the output 3D stereo be perceived at this point as another check or is this much later in the process?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1141.6f2927948177501875ccd43e9943a1ab27625c3b](http://build.palaso.org/viewLog.html?buildId=67364&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 775, ignored: 6 Build time: 00:06:41


I probably would have removed the duplicated code for clarity with something like:

```
namespace {
bool isSp2SulphurRingAtom(Atom *atom) {
  return (atom->getAtomicNum() > 13 && atom->getHybridization() == Atom::SP2 &&
          atom->getOwningMol()->getRingInfo()->numAtomRings(idx));
}
}

...

  if (isSp2SulphurRingAtom(mol->getAtomWidhIdx(aid1)) distTol *=2;
  if (isSp2SulphurRingAtom(mol->getAtomWithIdx(aid))) distTol *=2;
  if (isSp2SulphurRingAtom(mol->getAtomWithIdx(aid3)) distTol *=2;
```


why these changes?


just a test
[x] yes
[o] no


This is something I'd definitely want to fix. If we can just adhere to plain PSR-2 in the future, that would be good.


this place needs a like button ;)


Out of curiosity what's special about the first row here?


I think we could actually simplify things, and  merge `test_worker.rb` and `runner.rb` -- the latter could be a normal Ruby class, which would just conform to the Sidekiq interface by having the `perform` method? (Of course its existing `perform` method would have to be renamed to eg. `run`.)


..oops


Is there a reason the show instances were recently commented out? While I can see that they would be slow on large ArrayBuffers in real code, they would be super handy for debugging.


cool test.


here's just a refactor, it does not leak if the original implementation does not.

I did thought of replacing it with a smart pointer, but refrained at the comment line right above. I am not familiar with the completion i/o APIs enough to tell how they are going to do with the pointer being passed in.

If that comment line was put by you and you feel it's OK to use smart pointers now. I'll make the changes accordingly.

BTW, if any change is going to be made, is another PR required or this PR will just get updated automatically?


Better to use 'linux-x86_64'


I'd rather name this a bit differently, to be "safer"? Eg. `BenchmarkWorker`, `PerftoolWorker`?


Do you have a tracker item? Because right now Spyder is broken on 2.7


@piotrasd I haven't tried compiling Openelec 7.0 yet.  I suspect something has changed or is missing config options from the linux 4.3 kernel.  Are you able to SSH in to your Openelec instance see if there are any folders under here?:
`ls -la /sys/bus/virtio/drivers/9pnet_virtio`

If that folder doesn't exist or there are no subfolders then it's something with the kernel (missing options maybe).  If you see a subfolder, named something like `virtio0`, then I will need to review "_netdev" and see what is stopping that from working (maybe systemd update or some other userspace issue).

Thanks for looking in to this!


Yes, I'd better leave that for you two. :-)


That is not the point.
The point is that the 14 lines below are unnecessary and doing double work.


I'll be editing the labels for the public to understand my code. So expect an day to day update from it.


Write those fixtures into a `conftest.py` file and pytest will automatically find them.


Not if the removed line does not. They are equivalent.


The atoms are in general bigger and "softer". The major impact is that they distort the geometry of rings away from ideality.


masz angulara w node_modules a nie masz w dependencies w package.json

JAK DO TEGO DOSZŁO ?! :)


Why not default to a file in the current working directory instead?


I'm posting my Source Code to help the public what potentials AutoIT have when it comes to pixel based Automation.

I'm also posting this source code in order get donations for my son Thirdy a 6 month old preterm new born. I'm looking for medical assistance.

I'll be editing the labels for the public to understand my code. So expect an day to day update from it.


This commit is both unnecessary (++ works just fine in bash) and incorrect (it changes those two lines into no-ops).


Would occur later in the process. At this point no embedding has been done at all. We've just generated the bounds matrix and then applied triangle smoothing.


agree, I'll change Linux to 'linux-x86_64'


The original code never set ```self.direction``` to ```nil```. If there was no result it would favour north. From the description of direction in the constructor I gather that direction should be set to (any) one of the cardinal directions?


The script needs `import sys`, otherwise `sys.exit()` won't work.


Done :)


floor(pointer / 10 / 126 / 10) is different from floor(floor(floor(pointer / 10) / 126) / 10). I don't know the context but wanted to make sure the former was intended?


Noooooo!


You probably don't want this: I tried and gave up given how difficult it is to get travis to push to github.


Can you explain the reasoning for looking in a common system directory like this? Xcode uses this to build all projects by default - it will likely contain build products from many different projects.

Given that Xcode provides code coverage reports itself, the only reason to use CodeCov is in a CI environment, and therefore I would suggest that it is a mistake to build products into `~/Library/Developer/Xcode/DerivedData`, and instead the CI tool should use a bespoke path.

From an integration point of view it would be simpler if CodeCov required the directory to search for and didn't pollute the results with arbitrary build products.


ouuuups erased everything (not on purpose) sorry i'm fixing this


ok.
so I just rm the <os></os> part, or change this to something like
           <property>
          <name>platform</name>
          <value>linux-x86_64</value>
        </property>


good point. I'll fix that


Да, имеет смысл.
https://github.com/v-nikulichev/mastermind/commit/7e58c9afe6f4dac92a7ad903697e898b05f92b1c
https://github.com/v-nikulichev/mastermind/commit/25f6b0d67ce74303217e46d3aad82a618bfba044
https://github.com/v-nikulichev/mastermind/commit/83db838a3c8c59ec9d055e39a23e573f34c71d6c


Is this check unnecessary?


`files[1]` is the simplest :)


Just out of curiosity, why replace "deallocate" with "deallowcate"?


Quoting Cinghio Pinghio (2015-12-16 12:58:39)
>    How should I proceed? should I make a new PR with the suggested edits? Or
>    is
>    it enough to edit+commit+squeeze?

The latter should be fine. iirc, it will automatically change this PR because
the PR tracks your feature branch, which is what I will be merging anyway.
Thanks


can you describe how it detects usage of -h for help vs -h for hostname?


saw approval from hmmr
at https://github.com/basho/riak_kv/commit/45fdf7b8b058f63b9156546492d6c8820d620238


merging basho/riak_kv/feature/gg/refactor_query_worker = 45fdf7b8 into borshop-integration-1305-feature/gg/refactor_query_worker


basho/riak_kv/feature/gg/refactor_query_worker = 45fdf7b8 merged ok, testing candidate = 00d4d71b


but note that indexing out of range is not an error, it just gives `NA`.

e.g.

```r
> a <- 1:10
> a[11]
[1] NA
```

So don't forget to check the vector contains the element you're looking for.


This 1-based indexing ... Not sure how I feel about it ;P.


Yes I don't thinks it's needed right now. We'll move the platform section in commons if we start using batik at commons or rendering level.


As far I understand when `cd` fails it means that dir doesn't exist or we don't have permissions. For me it's unusual case and looks like an error. If we are just exiting with code 0 other external scripts (like `ansible`/`puppet`/etc) may recognize it as successful return value but it was not.

I would rather return 1 or another non-zero value to signalize about error.

See also LSB specification: https://refspecs.linuxfoundation.org/LSB_3.0.0/LSB-PDA/LSB-PDA/iniscrptact.html


I was just being consistent with ActiveMQ


With all the space between each chart they do not fit on the screen *all at once*.
This means you have to scroll up and down to see the changes in distribution resulting from your selection.
Thus you cannot see what size the bars changed from or to (in particular with the elastic y-axis) -- this is where the animation assists the eyes.


Attempt #2: building a CreateFunction expression and converting that.


Great! It does work!


Are those jobs in live job executed? If yes, can it happend that the test is too slow and the job finished before it is being paused by the test?


Thanks!


@woodall

1. Relative units are great, no matter which flavor you use. What I like about `rems` is they are relative to the root font-size, which I set to 10px, so there's no need to calculate any inherited values as you would with `ems`. Having the root font-size set to 10px makes the math incredibly easy, whereas 1.2rem = 12px. Using relative font sizes is great because I can essentially _zoom_ the entire layout by just changing a single property… `body { font-size: 105%; }`.

2. `vh / vw` are great. I haven't had an opportunity to use them. They are great for items such as headers where you may want to control the line length or size the type relative to the viewport.


This was essentially unused, right?
If so then a separate commit would have been nice.


@josevalim
yeah, nice.


Attempt number 1(below): doing manually the conversion that would be performed on a CreateFunction (I actually left some checks out; this approach was causing the evaluation to suddenly exit silently)


LGTM


用这个检测是为什么呢


Yep. this got messed up in the last refacto.


That actually broke stuff for me. Sometimes I want to pass the id to a data attribute (for example in lists). The id of 0 is not passed in this case.


I was trying the same thing too with 0.13.4, unfortunately this doesn't appear work on IE.  I was testing in 11, which throws:
```javascript
document.dispatchEvent(new Event('foo'));
```

Error: Object doesn't support this action


I don't know if it is worth writing after you reverted this commit, but anyway: I tried a combination of `interpreters.set(this)` along with `workCopies.clear(); hiddenEnvMaps.clear(); oprImpPluginsCache.clear();` and had performance issues with my specification and later on the engine hung in a endless loop and didn't compute steps anymore


Cho hoi anh hoc khóa nào HUBT vậy, Lớp nào ajk, Em cũng học HUBT K17


Thank you for this, have monkey patched it for now. Now to figure out how to NOT iterate through all pages before returning a query - seriously why would we want to return 500 results with 10 queries to YT on every search?


Nice. Wish the project was still active to merge your PR.


Wouldn't it be easier to read all conditions when avoiding `not` negation?


Since this commit Clementine will crash on OS X when starting to play a song (or it might be commit 978d849164d88bf30119620bcac2d4b69fac1ab8 but I don't see how this can have an impact).

Building myself is fine (no crash).
But (strangely) I can't create a valid DMG for my OS X version (10.10.5). That's probably a problem on my side, but that prevents me to be able to reproduce the issue currently unfortunately.


I've been multilining any use of a builder, even if it's only a single argument. I find it easier to read but will fit in with the house style.


This change breaks Solaris support.  It's also incomplete as the AC_NETATALK_LDAP function in macros/netatalk.m4 is still checking for ldap_init.


@woodall Normalize is great. I've usually modified it into a bit more of a reset but now I'm putting my reset into a separate file. Right now I'm okay with this. Normalizing is great because it's going to give a nice amount of consistency across all browsers… and then my reset is a little more specific to the needs of the particular site.


Thanks for "adding excitement"


@sbelsky, if we have to explicitly call release(), we actually lose the autoness of smart pointers. It may not be a good   case of smart pointers here, although I am all for smart pointers in general.


Improve the name of the test. (Why do you test for running task = empy?)


Reeeeeeeee


@untitaker Thank you!


Good point, thanks. At this stage we don't know if we're running as an init script so exiting with with correct value per the LSB specification will be tricky. Exiting with 1 seems like a reasonable compromise. It's fine for the non init script case, and also the right value for all but the `status` action when being used as an init script.


hm, why not skip `out` and return `NULL` directly?


Actually, I need to take those out. I think they just got copied into the file from an old project and forgot to remove. I know longer use that system to calculate my grid since it involves doing arithmetic by hand rather than by function.


that means we will have to rewrite that once again with Ember CLI, any way to avoid that?


add 3 constructors and method comparePrice


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


Redundancy.


This one was fine.


Remove these T.T


This was the one I wanted inlined :smile:


Ah, ok, I was only referring to passing it down as the `selected` prop. It's not a big deal anyways.


This is wrong. If a long long were to have something other than 64 bits, things would break here.

Why not alter the declaration/definition of argconfig_parse_comma_sep_array_long to take a __u64 instead?

Even worse is the case of argconfig_parse_comma_sep_array where an int is certainly sometimes 64 bits and sometimes 32 bits. When filling the __u32 with values, you'd interleave the values with 0's as you'd skip one 32bit value for every inserted value. And after that, you start overwriting the stack.


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


This file is missing btw.


They don't ship this part of clementine anyway.


OK.
Over and out.


JSLint


Looks OK.


In OMP master it is still called 'extends PKPRegistrationForm'...


@lilleyse I added this to #3177 for after we merge this PR.


typo: player instead of payer


Poza zmianami w obecnej ustawie projekt PiSu zawiera również następujące artykuły:

**Art. 2**
Jeżeli postępowanie wszczęte przed wejściem w życie ustawy nie jest prowadzone przez
skład sędziów określony w tej ustawie, postępowanie wszczyna się na nowo.
**Art. 3**
Jeżeli przed wejściem w życie ustawy orzeczono o rozstrzygnięciu sprawy na posiedzeniu
niejawnym sprawę rozpoznaje się na rozprawie, jeżeli uczestnik postępowania złożył o to
wniosek w terminie 14 dni od dnia wejścia w życie ustawy.
**Art. 5**
Ustawa wchodzi w życie po upływie 30 dni od dnia ogłoszenia


Why do we start with double `<`?


C# is striking back: lowercase variable names please. :-)


Thanks a lot, going to test this :)


No builder involved here.  (which might explain the commit that followed this).


remove new line.


You shouldn't do that because those addresses conflict. `*` means all IP interfaces and "all IP interfaces" includes `127.0.0.1` ...


htobe64 is unfortunately not very portable, see here for some horror (which I rather avoid in kafkacat if possible):
https://github.com/edenhill/librdkafka/blob/master/src/rdendian.h

Do you really need to print it in big-endian?

Also; Kafka message lengths are 32-bit, not 64-bit.


Nice :)


Thank you for pointing this out, there isn't, there is a bug here.


ksjdf


Could we add here 'parent::execute($user)' ?


good


К слову, чуть раньше ты используешь `gtest_SOURCE_DIR`, у меня есть подозрение, что когда создается новый проект `project(mastermind)`, то переменная `mastermind_SOURCE_DIR` может создаваться автоматически.


yay fix


Yes you are right, would have been cleaner :-)
On Wed, 16 Dec 2015 at 14:27, koefoed <notifications@github.com> wrote:

> That is not the point.
> The point is that the 14 lines below are unnecessary and doing double
> work.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cnrd/IDE15-16/commit/65ce63ad3793ba969f74aad70b65b39ad776c61e#commitcomment-15013699>
> .
>


Please keep the JavaDocs params and descriptions on a single line. Just add a line between the message, the parameters and the return and throws clauses.


@woodall

1. Probably to give you a better a picture of what those units mean, take a look at [gridulator](http://gridulator.com/). I use that to create a design grid that will not leave any fractions of a pixel or leftovers.

2. They don't PX units because these are variables that I'm using to run calculations to get a percentage value. Being this is responsive, we always want to use percentages for our grid. They are not used as CSS properties.


This change is wrong as it might cause a deadlock.

The suspend initiator cannot suspend as it hold the suspend lock, meaning no other threads will be able to resume it or trigger a GC.


thanks

why did you change it to FLASK_PORT? I was using PORT because a lot of PaaS are using this variable.
good point adding the host. thanks!


Caleb Preston


Can you do it on the cli, meaning if --name is not filled it would assume --host?

It would be an user expectation IMO.


[Uploading tmp_13942-B2GDroid-mozilla-central-nightly


ahffaaaaaaaaaaaa , commits, yay


I've been looking through packets for ages then I refresh my twitter timeline and see that it's been fixed!


FWIW *"It's needed for the test"* indicates you've written untestable code :)


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


Sorry, should have resized them instead, and put everything including the
map on 3 lines the height of one graph instead.
On Wed, 16 Dec 2015 at 14:37, koefoed <notifications@github.com> wrote:

> With all the space between each chart they do not fit on the screen *all
> at once*.
> This means you have to scroll up and down to see the changes in
> distribution resulting from your selection.
> Thus you cannot see what size the bars changed from or to (in particular
> with the elastic y-axis) -- this is where the animation assists the eyes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/cnrd/IDE15-16/commit/e6b16cd6f3b7ed9d41cf457260a52f1c1f94ba40#commitcomment-15013937>
> .
>


It cannot happen because liveJobs is just keeping track of job/tasks states but it does not actually execute the tasks.


This will crash everything since it'll pass a None back to the controller.


@chmanie are u sure your problem is there? We check the 0 value https://github.com/riot/riot/blob/master/lib/browser/tag/update.js#L154


Shouldn't it be `ArgumentError` like in update and others?


@woodall

I don't think it's really in use on this prototype as of yet. So how am I using them? I don't like using grid class names or CSS frameworks. They are too bulky and make too many assumptions. I like being able to use a width property from my design grid and apply it however I may need, whether it be floated, displayed inline, using flexbox, absolutely positioned, or whatever.


don't need var


They should be really like this.

```java
/**
 * An MD5 weight
 *
 * @param joint    The index of the weight's joint
 * @param bias     The weight's bias (ratio between joint orientation and position to vertex transform)
 * @param position The position of the weight
 */
```

Also align the params, and don't worry about going to a new line, the line width should be 120 characters, which is when you should be wrapping.


Corrigido.


Hey @andrewgould! Fixed to at least version `>=5.0.0` :-)


I think it's my fail `value != 0 ` => `value == 0 `! Sorry I will update it in the next release


Aerys should check for conflicts then. ;-)


Any example URL?


Oh wow!


Changing the test by verifying that the jobSubmitted method in the listener is called one time


Great point.


oh I missed that, :+1:


I see at least 2/3 reusable methods in this part of code.
Like getPointX, getPointY, getCenterOf and that kind of thing.


Thanks for the fix. Will test today if I get a chance.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1142.5b69c3219851b679cf9dcb01ccc96c4a32b6293b](http://build.palaso.org/viewLog.html?buildId=67365&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 775, ignored: 6 Build time: 00:06:21


For copy constructors, you have to copy the Vector3 objects too. Otherwise a reference is copied, and when you change the `weight.position` then the clone's position is also changed. Use the Vector3.copy() method.

```java
this.position = weight.position.copy();
```

That should do it.


fast-forwarding ts1.1/timeseries to borshop-integration-1305-feature/gg/refactor_query_worker = 00d4d71b


I like the idea of visual separation of parts of the test, however I would use `rlPhaseStartTest ... rlPhaseEnd` or somehow unique lines instead of several `=`.


I wonder if we can support a `PropertyDescriptor` here as well? So we can do

```scala
 config.extract(INSTRUMENT_KEY / FILTER_PROP).as[Filter]
```
instead of

```
 config.extract(INSTRUMENT_KEY / FILTER_PROP.getName).as[Filter]
```
as it seems this will be a common use case?

However, adding a dependency here to JavaBeans isn't great. Thoughts?


Same in the setter methods for vector3 objects, you have to set them, not change the references. Use the Vector3.set method to set the positon object.

```java
this.position.set(position);
```

That should set the values of this position to the same as that position object.


Thanks


Since we are improving this code, could we rename this class to Flamingos2 instead of FLAMINGOS2 so it's consistent with everything else?


Relax :smile:
This comment is just a duplicate from:
https://github.com/sysadminmatmoz/project-service/commit/61581a67299d5487580d7460c097a4d5f22f8946

Sorry about that :wink:


@rdlowrey point is, it doesn't matter with 7.0.1 anymore, it's just a static check. This function is not meant to be particularly tested anyway, it's directly tied to the underlying system.


there, fixed it! Again, sorry about that :)


@ilbelkyr right.. fix pushed


![screen shot 2015-12-16 at 12 58 00 am](https://cloud.githubusercontent.com/assets/2508142/11842809/f2198a7e-a3ba-11e5-9071-04a71b9d9457.png)


Same with this one, this should be Tcs, not TCS


![screen shot 2015-12-15 at 5 43 05 pm](https://cloud.githubusercontent.com/assets/2508142/11842823/120a2f32-a3bb-11e5-83cf-b56fce4cba69.png)


This `error_msg_ignore_crash` call replaces `perror_msg_and_die` call without providing the description of errno. However, the `perror_msg_and_die` call is wrong too, because it must be right behind `off_t ulimit_c = strtoull(argv[2], NULL, 10);`.


Use constant for 4*1024, you use the value again in line 448.


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


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


I don't think we can do this. It will break any existing registry DeploymentConfig, won't it?

@liggitt @miminar @sdodson @brenton @smarterclayton


1. JSON
2. Add publicmenuitems.html
3. Flash
4. Code style fix
5. Add readme.txt


@woodall

1. Not quite sure I know how to answer that. They are required in RWD. In my opinion, RWD is the only way to do web design.
2. Start with the smallest screen. This mitigates loading unnecessary assets and the number of overrides you'll have to make which in turn reduces maintenance.
3. The breakpoints I've picked for this prototype are random. I don't have a real design for this and just plugged in any old number. In production, the design and its stress points are going to dictate what your media queries should be. Each site is different, so each site will have different breakpoint values and a different number of breakpoints. Never tailor breakpoints to specific device widths… or only add breakpoints for mobile, tablet, and desktop.


Could you also look at the old function parse_bbc ... a few lines into it it has
```return $parser->parseMessage($message, $smileys);``` so all code after that point is not reachable.  I'm assuming that function is there for compatibility with 1.0 addons..


You forgot to `git rm src/gui/search/engineselectdlg.cpp`


上二つを再定義しちゃってる


Shouldn't this check if `len(xlim) == 2`? Same for ylim.


Yep, good idea.


LGTM :+1:


wäre hier bei aktivem ssl ein redirect auf ssl sinnvoll?


I'm not sure, but this seems to be non-idiomatic go: A function should either return an error, but now meaningful return value, or err = nil and a return value. In this case, returning string, bool, error is probably better (folder name, multiple found, error)?


Yep, I'll do that in a separate PR. I have a few more stacked up.


Could have been more concise; although would have changed data structure (hash to array).


Yikes! Yes, just pushed that.


I do not think this constant name really applies. Found for what? How about MSG_CORES_MULTIPLE_VERSIONS_FOUND?


This is interesting. :smile: It should probably be in its own commit, though, if it's coming in.


Do you mind removing formatting changes?


neptune pipes? sounds interesting


I am skeptic about having comments in the directives. For instance I doubt the xml is encoded properly if there are various unicode, utf, etc characters. And I think it can cause trouble when using it also. Your addon does not work if there are comments as far as I see.

This:

    re_define_time_bits_64 = re.compile('^\s*#\s*define\s+_TIME_BITS\s+64\s*$')

Does not match:

    #define /*1*/  _TIME_BITS /*2*/ 64 // 3


raus


It think the commit message could be improved slightly too, I was a bit confused by the mention of "semver", not knowing that it is a library to do version comparisons (I had to read the diff to find out).


this is exaclty doing nothing ;)


kommentare raus


This needs `#ifdef QBT_USES_QT5`


Maybe we could abstract this out to an IAdminThemeService?


I'm not sure about the name `cast` here—it seems more like a parsing or decoding operation.


Why copy? These are still exported in docker/distribution.


Null or Completed Task?


`#ifdef QBT_USES_QT5`


the corresponding C++ datamember is named src as far as I see.


Can we make these more generic?—e.g. signatures like this:

```scala
implicit def decodeOpt[A: DecodeRequest]: DecodeRequest[Option[A]] = ???
```


This and `shouldF` look like reasonable additions, but can they be in a separate commit / PR?


forgot to remove self.next and self.previous


sorry i forgot them (...again)


Also I think I'd vote for simply naming this `filterF`.


`qDebug("Destroying PluginSelectDlg");`


@ncdc `DOCKER_REGISTRY_URL` is set as a part of `CMD` when `/dockerregistry` binary is run inside a container. Deployment config should not matter in that case.


Working great so far, this also fixed the VMAD issue with "array of structures" property type refusing to copy. Now need to decode the rest of fragments and can left VMAD decoded by default.


@MaxMorais I have commented this for now. The error in jinja is displayed in the UI via the web's error page.


how can i make intellj to show scalastyle errors i tried to import them in ./idea and project and edit the style menu to show me errors but nothing shows


ok


import java.util.*;
import java.lang.*;
import java.io.*;

/* Name of the class has to be "Main" only if the class is public. */
class Ideone
{
	public static void main (String[] args) throws java.lang.Exception
	{
		String input1 = "1#2,2#4,3#1,4#5,5#3,6#6";
		int i,j,k,l,c,max;
        c=0;
        max=c;
        for(i=0,j=2;j<input1.length()-4;i=i+4,j=j+4)
        {
            c=0;
            for(k=i+4,l=j+4;l<input1.length();k=k+4,l=l+4)
            {
                if((Character.getNumericValue(input1.charAt(i)) < Character.getNumericValue(input1.charAt(k))) && (Character.getNumericValue(input1.charAt(j)) < Character.getNumericValue(input1.charAt(l))))
                {
                    c++;
                    if(c>max)
                    max=c;
                }
            }
        }
        System.out.println("Count : " +max);
	}
}


somewhere I think we need to document src and def better.


How about for multiline textareas? it's only the first line that gets indented
![screen shot 2015-12-16 at 14 17 33](https://cloud.githubusercontent.com/assets/13032789/11843155/488f265a-a408-11e5-9e87-f304a2084048.png)


TeamCity Dirigo Edge :: Pull Requests :: Pull Request Smoke Tests [Build 156](http://teamcity.corp.dirigodev.com/viewLog.html?buildId=1839&buildTypeId=DirigoEdge_PullRequestFullChain) outcome was **SUCCESS**
Summary: Tests passed: 6 Build time: 00:01:17


Pourquoi pas const/let?


panic???


Maybe you want to use [`raise-syntax-error`](http://docs.racket-lang.org/reference/exns.html#%28def._%28%28quote._~23~25kernel%29._raise-syntax-error%29%29) instead of `error` here?


oops. thanks for fixing this, but I think data.directives should be data.configurations


Ммм, а что там осталось-то?


I just checked the [MSDN](https://msdn.microsoft.com/en-us/library/aa363862(VS.85).aspx), `ck` is attributed as `_In_`, not `_In_Out_`, which suggests we should not manage its lifetime, unless we want to claim it back at errors. The original implementation seems better. And it's the win32 Api to blame that modern c++ is not used here.


Line is too long. [106/100]


Hello. I'm interested on your project. Spoke with Lord gex and get said to ask you. Now that I have a bit of more time, I would like to include your in my source in order to compile from source. Do you think you can help out on doing that? Thanks :)


keep the field name as registryClient


#1 게시판 + 로그인 기능 구현


no err handling?


Great, thank you!


Remove the Apps :)


Try to avoid abbreviations like this. manifestService, manifests, ms, or m seem better than manServ to me.


replacing function with boolean field? too weird


C'est tellement plus agréable à lire!


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


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


Can you give me two numbers where this matters? E.g., going from 201599 / 10 / 126 / 10 to 201600 / 10 / 126 / 10 seems fine. I suppose I should technically use more floor() though since it's supposed to all be integer division.


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


Are you looing at `v2.2.1`? I can't find them.


you only increment upon success.


Could replace it with CRM_Core_Error::fatal, or else throw an exception? Sorry I'm not exactly sure what this code is doing, but calling `dsm` is going to crash 99% of production systems, since the vast majority of them will not have the devel module installed (or will not even be drupal).


delete final spaces


Walter - I have some examples now up on my Ubuntu PPA. If you have time can you test and report back? Still some work to do cleaning up names and versions but we're getting there:

https://launchpad.net/~sbates/+archive/ubuntu/ppa [ I think you can access this page]

Stephen


@fefifofum this commit will fic fb,snapchat and other apps fc ...you can pick this


its ok


yes in this case error handling doesn't need


No, 2.1.1 as that's what I thought I saw. I'll check 2.2.1.

On Wed, Dec 16, 2015 at 9:24 AM, Michal Minar <notifications@github.com>
wrote:

> Are you looing at v2.2.1? I can't find them.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/miminar/origin/commit/ca9bbb204f5efdeac2f72b9acc7a55c04bb4ea26#commitcomment-15015342>
> .
>


No, just wanted to double check.

This change was the first "test" of the new API backwards compatibility checks that we have, so I just wanted to make sure that changing the API was valid here.

Thanks!


这是新增的一个文件？


Good point !
I didn't test this usecase.
Can you add a test to cover it before close it ?
Thanks


Looks good.


This is very large code duplication, we certainly don't want to duplicate that much code for every solver.
Just duplicating methods from `UnsafeFormulaManager` is already bad.


- docs need update
- when i run with 6 cores within a vm testing ubuntu 12.04 i had problem with resources not being enough... What will happen if i run on 4 core machine? We need to think about the default resources maybe 1 slave?


C'est allé plus loin, j'ai force-pushed le commit. https://github.com/Polyconseil/sentry-sourcemaps/commit/a6c0b7e5cf559e877d3b5af65740379bca8696b4


i checked eveything few time if i missed some your changes but no ... looks like be init from busybox not provide changes even folder called /unraid is no created its strange because in all location like initramfs, buxybox and inside packed initramfs.cpio file is correct with all changes
here to view http://pastebin.com/rip2h7kX

command  mount_virtfs is not recognized also kernel modules all need are enabled
here to view
http://pastebin.com/G1kvtw6z


is a arkmanager restart --warn working ?


oubli, oubli... c'est très très très WIP !


See reference/regexp.go. Will those work?


right. let's have a `__main__.py` then :)


I haven't tried to recreate a pathological test to demonstrate the bug, but it looks like it could be a nasty bug (with a really simple fix). I guess this could be applied to GROOVY_2_4_X as well as master.


マジ？？ちゃんと動くの？？


This change requires a migration (seems that this triggers an `AlterField` when `./manage.py makemigrations` is invoked, both in Django 1.8 and Django 1.9)


without this, the inputs is empty.
wm423-2d0:~ gtiao$ java -jar cromwell-0.14.jar inputs hello.wdl
{

}


You are completely right, wrapping a "warning" information in an error is not idiomatic nor a good idea, but I am still moving the first steps in the builder and wanted to do less noise possible :smile:
Changing based on your comments and pushing soon, thanks a lot! :wink:


Line is too long. [106/100]


*がカレントディレクトリの全てのファイルになってしまいそうだけど


I think it will be cleaner to use a consumer because you are not intervening with the future.
Following your comment that they need to be calculated together you cannot cache them as in getAsync so it is pointless to extract them to a variable. and then you can inject the count from the scope


3lach zadti hadi a sahbi


ánnansnan


badl hadi


@andkov, I'm about to look at this now.  Sorry I overlooked it yesterday.  Is it still a problem?


Yeah, seems pretty atomic to me, I can't see it creating issues. However, I don't understand the problem that this intends to fix.


Why the order change here? (Also, shouldn't the (lack-of) spacing around the "*" make flake8 unhappy?)


This incremented inside `startAsyncOp();` too and the check inside that function will fail...


Well, floor((7/4) / (1/4)) != floor(7/4) / floor(1/4).  But I think if all the divisors are integers it shound be okay?


Comment these methods of the Page


This class should be reverted to private access, since it should only be accessible from the outer Fragment class. Also, why the rename from `CommentsLoader` to `DiscussionResponsesLoader`? The "Discussion" is redundant, since we're in a Discussions related Fragment, and since the model and task both use "Comment" in their names, I think the original name was fine.


This should be reverted, it just adds 270 lines of duplicated code.


@fengWei029  是的


@wibeasley , great. please sync up, I've just pushed.

Yes, it's still a problem. I can't debug it yet and I have to move on. Any leads would be very helpful.


+1 but same remarks as for the Section:
- I don't like detail::Morphology ;-)
- should be a nested class too?


Backup incase all else fails


My bad, I was thinking this was set in the pod template in the DC.
Nevermind!

On Wed, Dec 16, 2015 at 9:31 AM, Scott Dodson <notifications@github.com>
wrote:

> Yeah, seems pretty atomic to me, I can't see it creating issues. However,
> I don't understand the problem that this intends to fix.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/miminar/origin/commit/716dc2b59e5f683c94db6f3d177c4ee803b45739#commitcomment-15015586>
> .
>


My bad, used outdated branch, sorry.


This is currently not called anywhere. The implementation will come with image pruning rework. Until then, `panic` seems appropriate to me.

I could have implemented it now, but it would need a rewrite for the image pruning.


Order change here because ~~it better reflects ~~ I'm stupid.  I'll switch that back.

The lack of spacing is fine as far as PEP8 goes, in this case I prefer it because (while I don't need reminding of BODMAS) I think it improves readability...


ok.


# 这是一个标题符号


@bergquist this should be fine, but you can check the comment for original ID name at https://github.com/grafana/grafana/pull/3440#discussion_r47780241


you can return fetchLatestDate() directly here, without wrapping in a Promise function


Haha! Totally forgot you could subtract vectors directly lol. Silly me.


I've updated the PR.
thanks

Mauro

Quoting Patrick Totzke (2015-12-16 14:36:37)
>    Quoting Cinghio Pinghio (2015-12-16 12:58:39)
>    > How should I proceed? should I make a new PR with the suggested edits?
>    Or
>    > is
>    > it enough to edit+commit+squeeze?
>
>    The latter should be fine. iirc, it will automatically change this PR
>    because
>    the PR tracks your feature branch, which is what I will be merging anyway.
>    Thanks
>
>    —
>    Reply to this email directly or [1]view it on GitHub.
>
>    Reverse link: [2]unknown
>
> References
>
>    Visible links
>    1. https://github.com/cinghiopinghio/alot/commit/1300a85a45fc6d1429b5ccd2834768588b59a9a2#commitcomment-15013909
>    2. https://github.com/cinghiopinghio/alot/commit/1300a85a45fc6d1429b5ccd2834768588b59a9a2#commitcomment-15013909


I think this error should be authentication_request_error


hehe

On Thu, Dec 17, 2015 at 12:25 AM, Gareth Kennedy <notifications@github.com>
wrote:

> this place needs a like button ;)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/underworldcode/underworld2/commit/367c3455305321a470bfb26ba8b30a7d0b2a8fee#commitcomment-15013646>
> .
>


I *much* prefer that spacing for the same reasons you do but I always presumed PEP8 outlawed it... Nice!


Is it possible for this to be invoked via an http route?

On Wed, Dec 16, 2015 at 9:35 AM, Michal Minar <notifications@github.com>
wrote:

> This is currently not called anywhere. The implementation will come with
> image pruning rework. Until then, panic seems appropriate to me.
>
> I could have implemented it now, but it would need a rewrite for the image
> pruning.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/miminar/origin/commit/2bfc48231ea7696098c990917f056594e89de259#commitcomment-15015670>
> .
>


This isn't needed since you call `startAsyncOp();`


Maybe you should split up the combo.
Test a combo of just search related tweaks.  Another test of just eval tweaks.


We have a method to do the connection obtain the value and close the connection


when you say that, you definitely want to supply explanation


All those things are essential in the sense of "it won't work if you don't do them"


This isn't needed too.


Yeah, someone should tell those guys Chaucer and Shakespeare not to use singular they.
https://en.wikipedia.org/wiki/Singular_they

Although I'd certainly welcome additional patches to either rewrite to avoid gender or else use plural subjects (where everyone agrees that English is not gendered).  We can collaborate.


Sorry about this one, I should have created a new branch and then created a pull request for you to approve.


System out Print, Delete please


TIkras branch https://github.com/Heartless27/Praktinis2/commit/0d3293e899ce1d500bfbb512f528bd4eccfe84fa


$ target/pack/bin/main 2 4 '*'
と入力させるのではだめ?


I haven't tried `neat`. I like bourbon because it nicely handles vendor prefixing, which is very necessary. I prefer doing my own thing with the grid because: a) i think what's really important is the width because percentages are impossible to memorize; 2) `neat`, in my opinion, is just another set of documentation to read and could potentially bring in selectors I don't need. It's a lot faster and leaner to use the appropriate CSS properties.

I guess in other words, I think that `neat` attempts to be a grid framework that accounts for all the ways a grid could be implemented. For each site though, a grid is typically implemented in one way. So for me, it's still a little much. Their approach is dead-on though. The grid should rely entirely on Sass instead of presentational markup.


`firstID`


You have copied (instead of moved) this from aux.


あんまきれいじゃないか


But whether it is a constant tuple, the behaviour  seems are same: just create a tuple(but use different methods).I didn't see where we let the compiler "build the tuple only at the first time to compile the method"

And

```
# statcheck: noninit_count('gc_registered_bytes') <= 1000000

for i in xrange(1000000):
    (1, 2, 3, 4, 5)
```

Failed with `Exception: ("noninit_count('gc_registered_bytes') <= 1000000", 'gc_registered_bytes', 20000024)`, in this branch.

BTW, why choose `100 000`?


This patch breaks the `ghc` build on NixOS: https://github.com/NixOS/nixpkgs/issues/10752. I'll investigate the technical details a bit and the report my findings at https://ghc.haskell.org/trac/ghc/ticket/11147.


Can you fix the indentation here and below.

I'd prefer ((gpr_atm)1) << ... for it's overly explicit obviousness about what's being cast.


We have a database method to open obtain the value and close the DB


Hercules*


そういうことか。まあ引数の幅を広げるということでアクセプト！！


Thanks.


But then, when I think more about this, are these examples really representing women as we want them to be represented?

 - So then she pointed at the tiger and screamed
 - I put a bike bell on her bike

Wouldn't it be more fun if someone had to put a bell on a man's bike? I for one would be quite ashamed if I couldn't put a bell on my bike myself. :-(


No, this is intentional. The `is*` methods should get inlined inside `visit` and then be removed. Implementing these methods has proven to be too error prone.

It is much easier to implement a single method per solver that handles all cases correctly than that wild bunch of methods, because in a single method it is more obvious what cases exist and need to be handled.


`Moodlight` you say. Meesa sense a conspiracy to change name from `LEDOS`. Meesa no like it.


No need to comment.


Yes, I have stumbled on this a bit after.  IE11 doesn't support the Event constructor on custom events. I used this polyfill :  https://github.com/krambuhl/custom-event-polyfill

Note you will have to use 'new CustomEvent' rather than 'new Event'. I will update the docs as well.


That's cool, thanks for the help man :D


Why?


Why the culturefeed_calendar_get_url function? l(t('Cancel'), 'culturefeed/calendar') is more then enough.


the description is not readable, please re-write it.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1143.103fd53f8a6198cfd5d59d42ccaec5920ebd9633](http://build.palaso.org/viewLog.html?buildId=67372&buildTypeId=bt431) outcome was **UNKNOWN**
Summary: Canceled (Tests failed: 1 (1 new), passed: 774, ignored: 6) Build time: 00:14:03
### Failed tests
```
GlyssenTests.dll: GlyssenTests.ProjectTests.UpdateProjectFromBundleData_BundleDoesNotContainLdmlFile_MaintainsOriginalQuoteSystem: <no details avaliable>

```


Comment


The new toXML() method implementation result in malformed set request xml.

<iq to="pubsub.chenchdi2" id="dii5X-41" type="set">
  <pubsub xmlns="http://jabber.org/protocol/pubsub#owner">
    <affiliations> node='Application'><affiliation xmlns="http://jabber.org/protocol/pubsub#owner" jid="jack@dkncx" affiliation="publisher"/>
    </affiliations>
  </pubsub>
</iq>


This only changes the version that is downloaded, but will report no errors on a system with 1.5.1 / 1.5.2 pre-installed.


I'm concerned about wrapper--staff--assessment not being unique (it's the same class that is used for staff override.


Can we simply change the definition of `norm` when we specify the faces?  So that it is no longer a unit normal.


thanks!


`engineName`


or more correctly `pluginName`


Quoting Markus Unterwaditzer (2015-12-16 15:40:23)
> You have copied (instead of moved) this from aux.

I moved some stuff from aux to conftest.
Do you think I should just have renamed aux.py -> conftest.py?


missing var. Please use eslint.


Both aux and conftest contain the `locale` dict.


菜单


Да, создаётся. https://github.com/v-nikulichev/mastermind/commit/7d238806e784865509b96e76eb6ac02b8f615988


We have a method to obtain a value of a Key review DBMethods class


This file is added under src/main/resources directory.. Is it fine ?


Patches welcome!


We could use `reference.nameComponentRegexp` if we exported it. We can't use `reference.NameRegexp` because now it accepts a hostname part.


I've removed this because it's no longer necessary.. I do that check in the `_processMessage`. Any other reason to add this back that I'm not aware of?


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1144.103fd53f8a6198cfd5d59d42ccaec5920ebd9633](http://build.palaso.org/viewLog.html?buildId=67374&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 775, ignored: 6 Build time: 00:05:32


Ah, sorry, didn't see that from the email.


is ril working on cm-13.0


Pull requests for more and better examples are very welcome, as per https://github.com/whatwg/html/pull/405#issuecomment-164907469


Warning: Use of "-f" without parentheses is ambiguous at ./uhu-installer-disable line 953.
Use of /c modifier is meaningless without /g at ./uhu-installer-disable line 953.
Warning: Use of "-f" without parentheses is ambiguous at ./uhu-installer-disable line 956.
Use of /c modifier is meaningless without /g at ./uhu-installer-disable line 956.
Warning: Use of "-f" without parentheses is ambiguous at ./uhu-installer-disable line 960.
Use of /c modifier is meaningless without /g at ./uhu-installer-disable line 960.
Unknown regexp modifier "/e" at ./uhu-installer-disable line 953, at end of line
Unknown regexp modifier "/t" at ./uhu-installer-disable line 953, at end of line
Unknown regexp modifier "/e" at ./uhu-installer-disable line 956, at end of line
Unknown regexp modifier "/t" at ./uhu-installer-disable line 956, at end of line
Unknown regexp modifier "/e" at ./uhu-installer-disable line 960, at end of line
Unknown regexp modifier "/t" at ./uhu-installer-disable line 960, at end of line
Bareword "create_disks_model" not allowed while "strict subs" in use at ./uhu-installer-disable line 1199.
Bareword "create_grub_model" not allowed while "strict subs" in use at ./uhu-installer-disable line 1200.
Bareword "fill_grub_model" not allowed while "strict subs" in use at ./uhu-installer-disable line 1201.


At least BooleanFormulaManager should depend on UnsafeFormulaManager to avoid all the copying from the UnsafeFormulaManager...

I understand your logic, yet on the other hand the visitor syntax is very unwieldy.
And the logic duplication across all solvers is rather large, and it's quite easy to make them unsychronized with each other.


that really looks wrong to me... a test proofing that would be very nice


I wanted to avoid using spaces here so I didn't use titles, suggestions are welcome


Is this line to remove ?


实现数据库类（构造与析构）


Spacing here between parens kills me a little bit... but since there is nothing in our [style guide](https://github.com/medic/medic-docs/blob/master/md/dev/style-guide.md) about it I can't fault you. Fits into the keep modules consistent category. Also a meaningful commit body or some docs in the module would be nice, about why the module exists and how to use it properly.  I know the issue is linked which gives some context but GH isn't the same thing as git and if it's not in git then I assume it doesn't exist.  Minor stuff anyway -- nice work on pulling this one off it's really slick!


EconomizerIntegrationNew (lgu1234) - 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)


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


PcGamesList.txt - ჩამოწერილია PC თამაშების სია, რომლის დასორტვაც შესაძლებელია ამ პროგრამით


buggy code, doesn't work


数据库类


it seems silly saving bools as strings, perhaps i could do "1"/"0"


and this is still a mess, keeping some of it in conf and some in aux is probably idiotic. I guess I'll move everything back to aux, only but the fixtures in conftest and import everything needed for aux into conftest.


更改菜单内容


Stress test. there is a very rare null exception that only seems to crop up when you do thousands of calls. ill tackle that next.


Because your local repository was out of date from the repository in the cloud, so before being able to push your commits up, it had to merge the other commits from the cloud into your local branch.


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


更改菜单内容对话框的头文件


showErrorDialog was not the right function... that was one i caught during commit. that will be changed to a proper info log message.


Blabla Test


Seems the rst headings open a new `<div class="section">` but the closing `</div>` doesn't occur until after the raw html. Making the headings raw html avoids this problem.


Hmm thought it was too generic and might cause conflicts with other things running on the same server.

Since it's only for dev.


OK, I'll backtrack through the callers to find how it could actually fail. If it can't, I guess the code is actually useless, because right now it doesn't work :)


Is this change needed upstream?


@Miguelcldn the new name was necesary to help new people find where to start when the enter directly to the repository version of the book. Although it is not fancy , it works, so let's agree to maintain it like that.


Ideally we'd use something from upstream, but I don't necessarily want to hold this up waiting for nameComponentRegexp to be exported. Maybe create an upstream issue or PR?


Normale einfache While-Schlaufe


hier ginge auch i = i + 1;


There's no such route yet. The method is being added to the `ManifestService` with [1050](https://github.com/miminar/origin/commit/aff38976bec1c5cd60a67179d100984a08d52d94) which aims to provide *local* interface for pruning, it doesn't provide neither a route nor binary to do that. Our admin prune routes don't enumerate either.


@andkov, my guess is that `tanh()` is misbehaving because there are `NaN` and +/-`Inf` values in its input (ie, `pc_CORR_11`).

Working backwards, it appears the infinity values in `pc_CORR_11`, were produced because `pp_TAU_11_est` has zeros.  (Recall `pp_TAU_11_est` is in the denominator `pc_CORR_11`.)

Do these tau zeros exist in the Mplus output and are being parsed correctly?

If this doesn't solve the problem, consider moving this to an issue so we can track it better.

![image](https://cloud.githubusercontent.com/assets/1372890/11844000/1597fc9e-a3d2-11e5-8159-1389d7dc59a0.png)

=================

![image](https://cloud.githubusercontent.com/assets/1372890/11843704/a02babe6-a3d0-11e5-8da9-2678700278dd.png)


Added the showMoreSuggestion option.
Usage : <auto-complete ... show-more-suggestion="true"></auto-complete>
Default is "false"
![2suggestions](https://cloud.githubusercontent.com/assets/8235116/11844120/5fbaa67e-a40d-11e5-9dda-16d6a247e3d7.png)
![4suggestions](https://cloud.githubusercontent.com/assets/8235116/11844128/6541ad2c-a40d-11e5-847d-d59c6fdef230.png)
![nomore](https://cloud.githubusercontent.com/assets/8235116/11844129/67963b56-a40d-11e5-8a84-f218422938ff.png)


Will do.


Ok, thanks. panic is fine.


Please report this in the forums.


cloud.gov should always be lower case, even at the start of a sentence.


Good point. We should alert the user in those cases (v > 1.5.0).


This is sparsely commented atm. basically it is a static class that stores a dictionary if instances stored by file name. When it writes a value(or values) to an instance, it starts a task to store them. The class is long but it is very efficient and can be accesed without worry from any time and anywhere with only 1 line.


"What we offer"


This seems to be a glitch


Some tips about TREE.


orchestration


lower case monitoring


One way to have only the `visit` method for introspection and to avoid the code duplication would be to give the is* methods protected visibility and to move the visitor code to AbstractBooleanFormulaManager.


预定餐位对话框


ah, true. and we are not running on any PaaS like heroku anyway, right? (fyi. here is the info from heroku about `$PORT` https://devcenter.heroku.com/articles/dynos#web-dynos )


I started off using clock_gettime, but CLOCK_MONOTONIC wasn't exported by Glibc. I'll make a note to revisit this.


the equals and hash overrides are so we can search for it in vectors, and it will consider something a match if its propName matches, but it will ignore the value. I felt it would be easier to only worry about 1 vector of pairs, rather than 2 vectors, one with keys and one with values.


把客户预定信息添加到数据库


The linux config and init code you link to seem correct.

You may need to delete these two folders and try building again to really update the initramfs that's packaged in the linux kernel:
`build.OpenELEC-unRAID.x86_64-6.95.0/initramfs`
`build.OpenELEC-unRAID.x86_64-6.95.0/linux-4.3.2`

I'm guessing at the version numbers in the folders above but essentially I would like you to delete the `initramfs` and `linux-4.3.2` subfolders that live under the `build.OpenELEC-unRAID.x86_64.X.XX.X` generated folder.  I sometimes have to do that to get the latest changes when I update code in `packages/sysutils/busybox/scripts/init`


"Cost estimates will vary depending on the labor required for provisioning a compliant service and the cost of the  acquired service." I think that's the same meaning as the original sentence.


"The power to your computers must be turned on."

Maybe I'm missing something, but is it essential for the user to know about it, or do something to achieve it?  If not, why bring it to their attention? The second point has the same issue.


Line 50 contains a typo "uncluding" where it should be "including".


Are you sure the Cache is really updated in the `else` statement?
I would suggest to add these lines
```
    else
    {
        this.Cache.LastWrite = DateTime.Now;
        this.Cache.cacheBits = this.Serialize();
        db.UserTokenCacheList.Attach(this.Cache);
        db.Entry(this.Cache).State = EntityState.Modified;
    }
```


this is where the aforementioned null exception is coming from, more on that later.


Boas,

Sugestões:

806 - O número é demasiado extenso

106 - Sem mais tarefas para hoje

312 - Download detalhado

As palavras em chines são para traduzir?


Should we use a manual list instead then?


预订餐位对话框头文件


Should maybe consider updating Boost to the latest version, otherwise, IIRC, there will lots of annoying "unknown compiler" warning spam when building with newer Visual Studios.


I'm not sure if it is ok to make this blocking, right now if the file wasn't yet open, it waits until it is. If we make this a thread, it may forget the order of the properties to add.


菜单对话框头文件


This is not necessary. `git describe` adds the hash automatically. The format is *latest_tag-number_of_commits_since_then-g+current_sha*, which should be sufficient.


Hello,
You should use the collection field type. Symfony documentation : http://symfony.com/doc/2.3/reference/forms/types/collection.html
Example : https://github.com/thelia/thelia/blob/master/core/lib/Thelia/Form/ProductSaleElementUpdateForm.php


because in this case we want to get info about roles and if such file isn't exist or error happens we don't need to send this error.


Yes !


yes i know, i even saw after delate manual and recompile initramfs.cpio and .install_init is not recreate in build folder, so i made make clean and recompile one again all.


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


@rockonedege,
1) anyway there was ```ck``` lifetime management in origin code. and it was tricky.
2) ```release()``` just releases the ownership without deallocating memory. it's okay if you strongly understand what do you do ;) I think it will be quite normal comment.
as for me, berkus's offer look's more elegant and reduces the amount of code. and don't forget about ```ck``` at line 256.


定义应用程序类的行为


Wonderfull :)


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


The code is catching an error that should never happen in production. So I want the error to be clear and unequivocal and interrupt whatever is happening. Sounds like CRM_Core_Error::fatal is a good choice.


this is where the aforementioned null exception is coming from, the pending changes object is null somehow. I will fix this but haven't put any time into solving it yet.


Additional patches are indeed welcome, and I also like to think that we can collaborate :)


Hey Juan, why did you add the -pg flag?  I've not researched it, but it seems likely that it would make APBS run much slower if it's collecting all that profiling data...


Well, I'm a bit hesitant to have Travis fail on styles that pass validation and follow our style requirements. Contributing styles is difficult enough for new users as it is.

Some of these things (like the "class" and "demote-non-dropping-particle" attributes on dependents styles) are marked in the 1.0.1 CSL schema as being obsolete, and I would just disallow them in the next big CSL update (for CSL 1.0.1, I wanted any style that validated against the 1.0 schema to also validate against the 1.0.1 schema). They never made much sense in this context, but the 1.0 schema wasn't set up to really tell dependent and independent styles apart very well.


Здравствуйте, я по поводу изменений в файле: ActivitiRestObjectCustomsController.java

+/*@ComponentScan(basePackages = { "org.activiti.rest.controller" }, basePackageClasses = {
-   org.springfox.ApplicationSwaggerConfig.class })     +   org.springfox.ApplicationSwaggerConfig.class })*/

Эта аннотация что-то выбивает?


Simon, hilfeeeeeeeeeeeeeeeeeeeeeeee


The problem I found when I used just git describe was that with current tag being on a separate branch it didn't work well. I wasn't sure if the release branch is always going to be where things get tagged by tito etc. It's kinda hard to test with a single real release there


@AnriShitko


The order is wrong, it should be
```
u16 pow_weak = (u16)(65535 * pow_l);
u16 pow_strong = (u16)(65535 * pow_r);
```


Unassigned should have been gone over here


first for function is delay
second for function is move text
last for function is print  Hellow

sorry;;; i don't speak en very well....


Done.


This and resolve pending changes should be flipped, i bet that is what is causing my null exception


主对话框


Done


owner_user_id is not important here.  Is not there a responsible?(technical, ...)


how i can split them in separete commit?


+*.php diff=astextplain
+*.json diff=astextplain


Have you reviewed this PR yet? :)


Boas...
Então força nisso :)
As cenas chinesas deixa tar que não devem aparecer...
Nornalmente so aparecem quando eles traduzem p inglês...


Gute Idee, werde ich anpassen.

---
Daniel Krämer
Master of Science (Informatik)
Software Engineer

anderScore GmbH
Frankenwerft 35
50667 Köln
Deutschland
 <BLOCKED::http://www.anderscore.com/> www.anderScore.com

Tel.:         +49 (221) 3558 3530
Fax.:        +49 (221) 3558 3531
Email:        <mailto:daniel.kraemer@anderScore.com> daniel.kraemer@anderScore.com

Geschäftsführung:    Sibylle Ohm
Handelsregister:        Amtsgericht Köln, HRB 56533

Von: muescha [mailto:notifications@github.com]
Gesendet: Mittwoch, 16. Dezember 2015 03:54
An: goldschmiede/2015-12-11-Wicket-Workshop <2015-12-11-Wicket-Workshop@noreply.github.com>
Cc: dkraemer-anderScore <daniel.kraemer@anderscore.com>
Betreff: Re: [2015-12-11-Wicket-Workshop] Added related stock items (f4752c5)

wenn null übergeben wird würde ich eine leere Liste schonmal erzeugen, so dass die liste nie NULL ist

—
Reply to this email directly or view it on GitHub <https://github.com/goldschmiede/2015-12-11-Wicket-Workshop/commit/f4752c5dc570b46357881d438728ba35c756378d#commitcomment-15003662> .  <https://github.com/notifications/beacon/APcCncI8bQrwFoFJ2fz9rNj391FOJBNqks5pQMm1gaJpZM4G2Mia.gif>


done


Split Administrator menu into Administrator + Editor


It's alright, as Nullable can only take non-nullable types as a parameter, which means Nullable<Nullable> is illegal, as Nullable represents a nullable type (duh!)


调用计算机本地时间，已显示时间。


It should be prioritized over other formats to be always `best` I guess.
Can you also add a test?


A rewrite of this paragraph which you can feel free to use, discard, etc:

With MirageOS, we've taken a clean-slate approach to building systems with a focus on safety.  Our approach is just one way to build specialised, single-address-space machine images with a library operating system (or unikernels for short), and we're happy to see more implementations being discussed and released at an increasing rate!


更改菜单对话框


And that indexing out of bounds behavior is just lovely, huh?


`supportedCat`


There also is the other side of the coin and that's why we should really think twice if we actually want this fix.  The thing is, it would make it impossible to only clean up the cache specific to e.g. a certain $releasever, which may otherwise be quite useful for some users.

Let's say you let system-upgrade cache the packages before you decide to upgrade them:

```dnf system-upgrade download --releasever=23```

Later, while still having those packages prepared in the cache for future use, you may need to clean up the F22 cache of the same repository that pulled the F23 upgrades.  What you would normally do, is run:

```dnf --releasever=22 clean all```

However, with the fix, this would purge both the F22 and F23 cache of the repo.

So, while on one hand it may not be intuitive to have to specify --releasever when running "dnf clean" (hence this PR), it may come in handy in other cases.

Alternatively, we could rewrite and redefine the clean command so that it simply purges the whole contents of the cache directory (/var/cache/dnf).  The sole purpose of the command would then be "to free up some space on the disk".  In such a case, I would also suggest getting rid of the subcommands (metadata, packages, ...) which would then make little sense.  Historically, the clean command was also used to force yum to refresh its cache, however, since introducing --refresh, that's no longer the recommended practice so that's why I think that redefining the clean command is also an option.

What do you think (or am I overthinking this)? :)


@teodimoff Sorry, I know literally nothing about IntelliJ. :frowning:


结账对话框


Здравствуйте, я по поводу изменений в файле: ActivitiRestObjectCustomsController.java

+/*@ComponentScan(basePackages = { "org.activiti.rest.controller" }, basePackageClasses = {

org.springfox.ApplicationSwaggerConfig.class }) + org.springfox.ApplicationSwaggerConfig.class })*/
Эта аннотация что-то выбивает?

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


Ahh, that makes sense. :-)


IMHO, yes it is, and that's why I reported [GNOME bug 759457](https://bugzilla.gnome.org/show_bug.cgi?id=759457) already (which turned out to be a duplicate of [bug 756648](https://bugzilla.gnome.org/show_bug.cgi?id=756648)), but for some reasons I quite not understand the maintainer does not seem to be keen on accepting it, which is why I removed the reference to the bugzilla from the commit message.

But you can be certain we (and other debian-based distros too) need this :). I'll talk to @amigadave on IRC, see if I can convince him to get it landed in the meantime.


There are fancy ways to do that kind of thing but to be honest I'd just remove them and then add them in a new commit manually.


sorry panda but done removed ! is a replacement not a removing