We could be a little more specific here. Not this exact text but..

Object literals extensions support shorthand methods and shorthands for `foo: foo` assignments. Together, these also bring object literals and class declarations closer together, and let object-based design benefit from some of the same conveniences.


Hi Tapas.

Let's say "ad" is a ClassAd that is empty - no keys.  Then,

ad.get("foo")

returns None.  Then,

int(ad.get("foo"))

throws an exception.  However,

int(ad.get("foo", 0))

returns 0 as "foo" is not a key in the dictionary.

If the object is a dictionary defined as:

ad = {"foo": None}

Then, you can still have ad.get("foo", 0) return None.  However, there is no None in ClassAd language, so you don't have to worry about this case.


Hello, Can you add Method setGravity for VisualFont?


`es5 = ` to `es5: `
`es6 = ` to `es6: `

Also please add spacing before/after the + operator here.


This is potentially a breaking change, if people have not set the correct permissions on the vendor directory. What is the reasoning behind moving this out of storage?


please extract userInfo keys to constants


Just a nit - typo in the parameter name here


@Montellese you mean rename to `IInputCodingTableFactory` to match `IInputCodingTable`?


Let's add some comments here to explain what we're demonstrating (how do the ES5 and ES6 examples differ?). Clear to someone that understands,but useful to add a line for those who don't.


These two lines are very similar to prev_x =..., prev_y=... It's better to extract a method in such cases.


This is a first Intro Project


No me acaba de gustar el manifest con todo en la misma línea.

Por el maven, enhorabuena ;)


Btw, Nicholas Zakas has a great chapter on obj literal extensions with some more examples over in https://leanpub.com/understandinges6/read#leanpub-auto-object-literal-extensions. Might prove useful for inspiration on how we can expand the samples here.


Je pense qu’il existe également l’oubli du e à bienvenue dans la fenêtre verte qui s’ouvre à la première connexion. J’arrive plus à le reproduire car je n’en suis pas à ma première.

Nathan


Let's add `jscs` instead, that adds (more) style checks and doesn't duplicate jshint checks.


Is ``configured`` there to check when its_get_devid is set? If so it may happen out of order. This could be an issue depending on when this function was expected to be called, e.g. if we are scheduling on multiple cores.


No, this is an experiment in progress that never should have been committed.

What I was trying to come up with was a simple way of improving the accuracy of the does-this-line-seem-to-end-a-statement test without bringing in the entire grammatical context established by previous lines -- though a more contextual/grammatical approach seems likely the way to go longer-term.

In particular, I thought that there were common false positive and false negative pitfall cases that could easily be eliminated.
One is the case of the semicolon NOT being honored when it is followed by (whitespace and?) a comment that was not terminated by a semicolon.
Another (more serious) is the case of honoring a semicolon at the end of a comment that did not follow a semicolon (+whitespace).

This code glitch probably came from starting to think about the first scenario and then realizing that the second scenario was the worse problem and that I didn't have a trivial pattern fix for it and then walking away. The trailing "--" coverage should probably be taken out, but it has almost no effect in the real world. Or we could follow through on case 1 by making it "(?:--.*)?$" for a slight win? WHat do you think?


i dont understand why we are starting kafka and zookeeper here


Right, I fixed in 2a60194cdde560bc3142bb0711bd70d68a4f7c3c


Nice catch, i'll update this.


Looks good


What I tried, is adding the 2 functions right after the commands and before annyang.init(commands), as below, but that didn't work for me, although in the examples provided (https://www.talater.com/annyang/), they did the same thing and it works.

```Javascript
var commands = {
                'draw': function () {
                    SendScript();
                },
                'clear': function () {
                    ClearCanvas();
                },
                'brush black': function () {
                    brushColour = 'black';  //Changes the brush colour to black

                    //Changes the colour dropdwon to black (just to show the user the current colour of the brush)
                    var colourSelector = document.getElementById("selectColour");
                    colourSelector.value = 'black';
                },
                'brush blue': function () {
                    brushColour = 'blue';

                    var colourSelector = document.getElementById("selectColour");
                    colourSelector.value = 'blue';
                },
                'brush green': function () {
                    brushColour = 'green';

                    var colourSelector = document.getElementById("selectColour");
                    colourSelector.value = 'green';
                },
                'clear script': function () {
                    var userScript = document.getElementById("codeEditor");
                    userScript.value = '';
                },
                'move forward *step': addMove,
                'turn *angle': addTurn
            };

               var addMove = function (step) {
                var userScript = document.getElementById("codeEditor");
                userScript.value = userScript.value + '\n$input.MoveForward(' + step + ')'
            }
            var addTurn = function (angle) {
                var userScript = document.getElementById("codeEditor");
                userScript.value = userScript.value + '\n$input.Turn(' + angle + ')'
            }

            annyang.debug();

            // Initialize annyang with our commands
            annyang.init(commands);
```


The thing is, JSCS is **only** for style. ESLint on the other hand is much better for common errors. The best combo at this point would be JSCS and ESLint in my opinion.


We will get the same problem on Linux! I would suggest to rebuild the structures (which are transferred between user/kernel) by adding paddings, so that they are equal on 32Bit and 64Bit systems.


Note that graph is an object that may exist without being possible to compute shortest paths. Hence findHamiltonianCycle_2opt should not be a method of Graph, but a function or even better a class that implements some abstract interface like IHamiltonianCycleFinder. You have several algorithms that implement interface so it makes sense for the problem.


You should add that anyway, currently this only affected live system, the environment flag was not set in installed ones...
That way we can also remove setting it in the live-cd script https://github.com/manjaro/manjaro-tools/blob/master/lib/util-livecd.sh#L426


this line needs to be removed


Se puede hacer asi, ejemplo:
Iterator<Envio> iterator = repositorioEnvios.findAll().iterator();
List<Envio> envios = IteratorUtils.toList(iterator);
return envios;


we could just make this like:

Add a Comment


Bien vu ! ça sera corrigé dans une prochaine PR.


maybe lets add a header here to designate comments:

Comments


Related issue: https://github.com/kanaka/noVNC/issues/471


I have sent to you @moromete


Verry nice, that resolve the missing account number on html pages.
Anyway to put that information on the PDF version too you need to change as follow:

/bb-modules/Invoice/Service.php

$company_info = __("Name: ") . $invoice['seller']['company'] . "\n" .
                        __("Address: ") . $invoice['seller']['address'] . "\n" .
			__("Company VAT: ") . $invoice['seller']['company_vat'] . "\n" .
			__("Company number: ") . $invoice['seller']['company_number'] . "\n" .
			__("Account: ") . $invoice['seller']['account_number'];

$buyer_info = __("Name: ") . $invoice['buyer']['first_name'] . $invoice['buyer']['last_name'] . "\n" .
			__("Company: ") . $invoice['buyer']['company'] . "\n" .
			__("Address: ") . $invoice['buyer']['address'] . "\n" .
			__("Company VAT: ") . $invoice['buyer']['company_vat'] . "\n" .
			__("Company number: ") . $invoice['buyer']['company_number'] . "\n" .
                        __("Phone: ") . $invoice['buyer']['phone'];


I use jshint+jscs on various jQuery projects. I prefer to stick to that combination.


kk I was in a hurry and I thought they were in English :stuck_out_tongue: From next time I will make sure that they are mentioned explicitly also I will change the present files to those format


Could you point to the places, where tTimerHdl is shared between user and kernel?


not sure how flexible this is going to be with the properties in the container, lets discuss please


Well, if so, it is fine. On my installed system it works. Maybe only live system is affected due this and not installed versions. I have to check it once more ...


Again, heuristics like _2optSwap should not belong to Graph. In particular here you use only route to update it.
What is very bad here: you pass vector as a parameter, so the object will be copied every time. And after that you copy it back. Is it possible to make it in place?


trolrorlrorlro


The "Pretty URLs" section is repeated at http://laravel.com/docs/master/configuration#pretty-urls and http://laravel.com/docs/master/installation#pretty-urls

Note: In the "Installation" page, it does not appear as a section in the navigation links at top of article.


i think we can remove the class from here to make it look more like the sample


Oops. ODBC 3.5 was released in October 1996. And there are still drivers that are not 3.5 ready???


Ah that's a good point. I think they *are* recoverable though, for the most part, and generally expose an edge-case bug or something. @biyeun does that sound right?


change to: < div class="col-md-12 ">


这不应该是`@story.body` 么


Thank you very much!


Please get rid of labels - it's a bad style. In many software companies they are forbidden by style guide.
You can just use an infinite loop plus a bool variable instead. Not a big deal however :)


@Montellese this function is not a charset convert, it's translate `\uxxxx` type string to real unicode.


Ой. Кроме того, что нужно указывать кодировку, нужно и тело письма кодировать и вообще проверить отправку писем, тут у тебя получится нечитаемые кракозяблы :(


@jforissier @pascal-brand-st-dev I'm wrong also. line 260 is to clear other flags in `type` except `TEEC_MEM_INPUT` and `TEEC_MEM_OUTPUT`. Line 262 need use the new `type` value hence line 260 is necessary.


makeRandom shouldn't be a method of Graph. It's better to create a function like GenerateRandomGraph(...)


I kind of agree about the cores/ram. But, we end up running multiple node processes at once for the server and the watcher, and some of the smaller gulp tasks that run have some native dependencies (imagemin, sass compilation) that AFAICT can utilize more than one core/process. 1 core is probably fine, but my thought was I should try to make it as fast as my local machine somehow.

I upped the ram because I got an out of memory error during npm inatall when compiling the imagemin native dependency and saw some other "performance in vagrant" posts talking about it. 2gb might be overkill. The default was 512-ish.

I'll look into the private IP address a bit more. The actual IP seems pretty arbitrary. Maybe there's a safer IP we can use? Not sure.


Es como lo formatea eclipse, además  son atributos de la misma tag y eso no se va a tocar nunca. A mi me gusta mas del otro modo


Do we have a test to prove this?


Instead of writing the code similar to the above one you can just store additional information together with data. Something like from(1 << n, ...). When you select minimum over all j values, you can also store j = argmin.


You should not commit .pyc file, and put it under .gitignore.


1) Do NOT use digits in variable names.
2) Pure coordinates is not a logical part of circle. Coordinates have to incasulated in another class like Vec2 or Point2d. And following code will look better:
class Circle
{
  Point2d center;
  double radius;
}
3) It will be also better use one line for one field.


Sure, feel free to add it yourself then. I prefer ESLint because it truly
is better than the rest.
On Mar 11, 2015 3:41 PM, "Jörn Zaefferer" <notifications@github.com> wrote:

> I use jshint+jscs on various jQuery projects. I prefer to stick to that
> combination.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/XhmikosR/grunt-html/commit/8738e81bf178041e76787a1243c96a158d186a1e#commitcomment-10142587>
> .
>


Looks good


we just ignore ```f``` after this, and in case of ```AMD``` module ignore the value returned by the factory function.


why russian?


Oh sorry, then the way github colours


This file is the correct one


Oh sorry, then the way github colours things confused me.


This is intended to be understandable, then I apply your suggestion in some way.


This is not correct


Remove this line for now. Commit, merge. Then fork from the features branch and keep it synced.


This doesn't seem right to me.  If you want to pass a null pointer, shouldn't you pass `C_NULL`?


shouldn't this be ```sound``` instead of ```info.DestroyedSounds.Random()```?


same here


Will make space change and column limit


IMO the implementation of `IsAddonInstalled()` after this change is wrong because a disabled addon is also installed so that either has to be fixed or renamed to `IsAddonInstalledAndEnabled()`. `CGUIWindowAddonBrowser` uses this `IsAddonInstalled()` and also relies on the fact that it is true for both enabled and disabled addons.

We can remove the `IsAddonInstalled()` that also returns an addon. I used it in one place in `CGUIWindowAddonBrowser` but then removed it again during a re-write.


lol


Ok, fixed it


It already looks like production code!
Good job!


Well, but why you’ve commented it instead of just removing?


What units are data(:,1) and data(:,2)? Is this something like rpm vs. thrust?
Then I suggest to swap X and Y axis to identify a concave polynomial which has thrust as input and rpm as output.
As soon as I understand the task completely it would be a pleasure for me to do make a pull request. :)


FYI Those are the non-semantic changes to existing code I was talking about


First of all, apologies for not including DLNA guideline reference in the commit message -- its 7.5.4.3.2.4.4. I don't see exact way how HTTP version could


Don't try/catch that, just let the method throw the (Parse)Exception


This method no longer throws CloneNotSupportedException


+1 for creating the convenience methods in PacketParserUtils. :)


here return ```m.exports``` instead of ```f````


There is nothing to be so hyper about. I don't even know personally and I don't see any reason as to why I would not retain your authorship over something and say it as mine. Why can't I find it as well? You have the sources, I have them too. :) Understand brother, I am against non retainment of authorship as well. This is not intentional. So please, nothing to be offended about. There comes no point of titles here. You are selected for the developer program that itself shows your worth. I don't know who has provoked you on this. But my intensions were clear.


yay!!


Disclaimer: I'm a moron.
Coming from Haskell, anything with a for-loop looks imperative :) (is this really a for-loop like the traditional sense?) Anyway, something that may be idiomatic Haskell may not translate to idiomatic Scala, and I don't know enough about scala's design/tools to suggest a better solution off the bat. I have a question though (not necessarily related to functional-y-ness). You define "sorted" using the sortWith function...in this case, what does this do that the original definition of sorted doesn't? As I've been writing this, I've been googling and looking at how scala expresses things vs other languages (answered my own question about the for loop-comprehension thing [here](http://stackoverflow.com/questions/4466362/for-expressions-versus-foreach-in-scala). It seems like you're doing things right, and maybe it just looks imperative because that's part of the goal of scala? The only other thing that sticks out as non-functional is that you have to be aware of the state of your streak and topStreak variables/counters? I may be just talking out of my arse and @gfritz can offer more valuable information.


Awesome!


why there are 2 `Features` headers?


We use `goog.bind` elsewhere. It's better to use `goog.bind` for better compression by the Compiler. Note that we use `angular.bind` in the ngeo examples because the ngeo examples are not compiled when hosted on github.io.


C'est mieux de garder les données dans le contrôleur plutôt que dans le scope, de "cacher" les infos dans le service et de transmettre à la vue les infos via le contrôleur.


Shoud we also change ```issueAccessToken``` method?
I except 40x/50x error including ```Left``` exception, which allow to have content body in my understanding.
(Actually 400/401 error statuses are only allowed on OAuth 2.0 specification)

So we can write below?

```scala
  def issueAccessToken[A, U](handler: AuthorizationHandler[U])(implicit request: Request[A], ctx: ExecutionContext): Future[Result] = {
    tokenEndpoint.handleRequest(request, handler).map {
      case Left(e)  => new Status(e.statusCode)(responseOAuthErrorJson(e)).withHeaders(responseOAuthErrorHeader(e))
      case Right(r) => Ok(Json.toJson(responseAccessToken(r))).withHeaders("Cache-Control" -> "no-store", "Pragma" -> "no-cache")
    }
  }
```


Maybe the `\n` should be in front of `Type:` to avoid `Type` being tacked on to the description when there are no organisers.


Its amps vs. trust in newton. Reconsidering where I took the data from I actually think I have to go back and find PWM vs. thrust data. I recommend to contact leonardthall@gmail.com for a chart of a EPP1045 prop, I believe he has done some of that lately and my data is pretty old (2011).


Neat. My concern would be the amount of overlap between them. Does the merge remove duplicates properly?


Good changes to scope. :+1:


Hey!

I guess nowadays, this is considered *bad practice* to use `target="_blank" to force opening in a new tab. People would rather let the user decide and use the mouse's middle click (or Ctrl+click) to open in a new tab. I can think of a few use-cases where I would still use `target="_blank"`, but not that many.

But because this is your project, it's up to you :)


I don't understand the problem that this change is intended to fix, so I can't say whether this is the correct fix.


Hmm, this will cause more overhead in the verbs provider due to branches.


Sorry for the title line, not sure why it shows this!


No the `I` at the beginning means "interface" and the `C` means "class" so since this is a class implementing an interface it should start with `C`.


As a reminder to think about a slf4j version property for testing..


See the other comment concerning the usage of `I` and `C`. In this case we are talking about an actual implementation class so it should start with a `C` but the common pattern is still `InputCodingTable`.


fixed it


@smcv @fabiangreffrath @coldtobi

Hi, could anyone with access to a dualboot PC test this :-) ?


iukiuk


@alcapat if the footer is `display:none`, that's why  we don't see the footer.I guess .


Excellent!


Hey man,

The code is good, but I wouldn't added to the core library. We might want to add this piece of code as an add-on separated from the core.

What do you think?


ICSSolarCollector_#4723 (Nigusse) - 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)


Wofür ist das ini_set?


This is being done under @UnitOfWork annotation, so it's all or nothing as far as database is concerned.  So queue failures can happen and in that case we abort and we didn't set the state so we should be fine there.


@bchavez , for Sandbox Testing , the following are the modifications which we made :


最新的提交


I will add a logger event also for debugging purpose


Can these printlns be removed?  The latest Grails3 builds are picking this up and we are preparing for RC1.


One is quoted, and one isn't! Sheesh!


huch sorry


try this #10


No it works with GML2  and GML3


You should add a test for this, so that we avoid this problem in the future.


Is anyone using this with type logical? It's not idempotent for me.


Yep, you are right!

I'll fix `issueAccessToken`


I made a typo in my commit message, sweet irony.


So here is an example of PEP8 indenting I don't like:

https://github.com/wbond/package_control/blob/61e463a65e71deccd44a09bc9d6261109731616e/package_control/package_manager.py#L746-L748

I particularly dislike arbitrary indentation to a `(` such as at https://github.com/FichteFoll/package_control/blob/fix_update_message_scroll/package_control/package_manager.py#L1284-L1285. This is annoying because you can't just hit enter and tab, but instead need to make sure the lines align and then need to adjust things if a variable or method name changes on the first line.

I much prefer this style (which I found seemed to become popular in the javascript community):

https://github.com/wbond/package_control/blob/b3bdd40478c00320e3bd56b44b4322589a24be55/package_control/package_manager.py#L277-L283

When you have enough params that you need to wrap a line it gets hard to parse through and see what is being passed. This is even more the case if one of the params involved a method call or index access. The one-param-per-line style makes it super simple to parse through. Previously I had written in the PEP8 style I first outlined, but I've decided that the difficulty in reading the code is not worth compliance with PEP8. Same thing goes for line lengths. Arbitrarily wrapping at 80 chars leads to continued lines which are harder to read. Almost everyone has at least 120 these days.


Problem with this commit: #2342


It was a doddle but you did it right!


没必要改吧。反正string:tokens会把最后那个0忽略的


Привет!
Перевод готов?
Я случайно увидел коммиты, но не был создан пул реквест.


OpenLayers.Protocol.WFS doesn't support version 1.1.1 so as It's in  hitCountProtocol ( line 692) , I prefer to force wfs vrsion


Sorry, but I don't have dual-boot. But it sounds like a good idea.


Hi Hraban,

I had a much closer look at GCC and the "objdump -S" output. I'm sorry, I was wrong and you were right. GCC replaces memcpy() with dereferencing where possible. When patching scanmem to always use memcpy(), then there is exactly the same assembly code on x86_64. I'll update my test and create a patch for Wang Lu. I'd like to add a "Reported-by: Hraban Luyat <hraban@0brg.net>" to it to credit your work.

Are you fine with that?

I'll send you the link to the patch for review before creating the pull request.


Дебрифинг.

Эта задача потребовала особых навыков:
- изучение новых библиотек,
- поиск информации вне документации,
- подбор библиотеки под задачу.

Впрочем, последний пункт не был раскрыт до конца, т.к. ты выбирала только из двух уже известных библиотек.

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


This check uses the tag pointer which isn't initialised. Suggest removing 'tag->' from the if at line 69.
Paul


etata aetatae SDF-2


hi, @pesa, @kensington, you right! we should set target as generic, it's obvious from source code. besides I tested with original patch, it will fail to build some packages (eg, dev-qt/qtscript-4.8.6-r1). I pushed this to portage tree, please sync if you're good with this.

btw, I'm doing native compilation on arm64 (not cross compile).


done


Passing the joined tool is as configuration param could be a solution?


为什么把微缩图删了?


Why was /debug flag added?


@Ellega could you verify this does not break something elsewhere?
We had an issue when switching to russian when on IIS.
Thx


Just wonder, any update?

On Tue, Mar 10, 2015 at 2:05 PM, Satish Patel <satish.txt@gmail.com> wrote:

> Superb! waiting for your reply! Thanks!
>
> On Tue, Mar 10, 2015 at 1:15 PM, Bogdan Andrei IANCU <
> notifications@github.com> wrote:
>
>> This is a different story - I have to check if the HEP protocol (used for
>> siptrace to sipcapture communication) does support WS proto.....be with you
>> back after checking with Homer guys
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/OpenSIPS/opensips/commit/178b0cc26b05a81947de150fe1c2df36d600ccaa#commitcomment-10125159>
>> .
>>
>> _______________________________________________
>> Devel mailing list
>> Devel@lists.opensips.org
>> http://lists.opensips.org/cgi-bin/mailman/listinfo/devel
>>
>>
>


If segmentEncryptionIV does get set here, I think you need to make sure that you don't overwrite it at L218 below. That logic should only be executed if a segmentEncryptionIV hasn't been specified using a KEY_TAG, I think? ....


Nice "fix" - just incorporated it into my package that is based on the released 3.2.1 ;-)


This is correct :-)


Not sure how these look like when rendered in the pdf. Please test.


I think they can be deleted


最后不一定只有一个 0，而可能有一长串 0。我希望能把这种情况显示出来。


This is a little guy. I'm going to merge it.


fghjk


....Which you can do by not writing to segmentEncryptionIV. Instead, do something like:

```
String thisSegmentEncryptionIV = segmentEncryptionIV != null ? segmentEncryptionIV
    : Integer.toHexString(segmentMediaSequence);
```


These services are just internal for the raw PID subscriptions.. But note the changes in the http_stream_mux() function - the mux URL now handles the 'pids=' parameter, so you can stream only selected PIDs without the whole mux contents as you proposed later through a HTTP session.


Hi JD,

I will challenge that best practice.

As a marketer and business consultant, I spent years working with clients
and relaying their wishes to the product team.  Users hate being abruptly
taken away from a website, they would rather have it open in a new tab.
Additionally, from a business perspective, I think it is better to have an
external site open in a new tab rather than navigating users away from your
website, where you or your company make money.

So I will always try and think about it from a user and business
perspective.

Plus you should see some of these older generations (my parents included)
try and right click on something with the new laptop track pads; it is
pretty painful to watch! lol

-Ryan

On Wed, Mar 11, 2015 at 10:02 AM, Jean-Denis Vauguet <
notifications@github.com> wrote:

> Hey!
>
> I guess nowadays, this is considered *bad practice* to use target="_blank"
> to force opening in a new tab. People would rather let the user decide and
> use the mouse's middle click (or Ctrl+click) to open in a new tab. I can
> think of a few use-cases where I would still usetarget="_blank"`, but not
> that many.
>
> But because this is your project, it's up to you :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OggimusPrime/Bloccit/commit/a479974bf5ce5434460811599f1bd36919e2611e#commitcomment-10143051>
> .
>


1: M111 unrecognized component name: setup


多少个都没问题啊。你自己试试就知道了么


What is a doddle?


Noted :D


You're right. Somewhere these changes and the upgrade rule were eaten by git.


yea i think i did in master branch sorry about that


Expand on this switch statement.


Silly magical variables


If `datum` is no or the empty string, then it should get set to "WGS84". See XEP-80 Appendix G 2.


traded CacheHeaderStrategy for this - more self-contained and reusable in multiple places.
Note that:
* this deals with only response directives, while other frameworks chose to model all request+response directives in the same class
* only deals with `Cache-Control` - but we could extend it to also write `Vary` headers, for example


Actually I do agree with you: this best practice, is not always for the best. Under the assumption it'll empower users, it actually "break" things for some people, and that should be accounted for.

It's becoming mainstream nonetheless, and because I wasn't sure whether you used that as a test for HTML, or as a deliberate choice, I thought I'd be better describing that "mainstream" policy :)


thanks for the feedback.

im not sure what the magnes paper is trying to say, but i would expect the exponential phase representation `np.exp(-1j * np.angle(fid_pn))` to be more stable than the naive `np.conj(fid_pn) / np.absolute(fid_pn)`. im not a numerical analyst though. either way, the computational cost should be negligible.

good suggestion to put the multiplication with `dyn.tau[k]` in the other line to be closer to the formula in the paper. how shall i proceed? add another commit for the change, or amend and force push?


`maxAge`, `noCache` and `noStore` are opinionated directives that, in my opinion, exclude each other and describe the main intent. That's why here those directives are the only one to be described in a static method.


Ah ok. I thought this was about HTTP 1.0 vs. 1.1 connections. I will re-read the guideline (is that from 2014 DLNA guidelines?)


This is a good write up that explains all points of view:

On Wed, Mar 11, 2015 at 10:24 AM, Jean-Denis Vauguet <
notifications@github.com> wrote:

> Actually I do agree with you: this best practice, is not always for the
> best. Under the assumption it'll empower users, it actually "break" things
> for some people, and that should be accounted for.
>
> It's becoming mainstream nonetheless, and because I wasn't sure whether
> you used that as a test for HTML, or as a deliberate choice, I thought I'd
> be better describing that "mainstream" policy :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OggimusPrime/Bloccit/commit/a479974bf5ce5434460811599f1bd36919e2611e#commitcomment-10143511>
> .
>


TeamCity xBIM :: XbimExchange :: Build [Build 117](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1149&buildTypeId=XBIM_XbimExchange_Build) is now running


http://stackoverflow.com/questions/3906898/to-target-blank-or-not-to-target-blank-that-is-the-question

On Wed, Mar 11, 2015 at 10:26 AM, Ryan Haase <ryandhaase@gmail.com> wrote:

> This is a good write up that explains all points of view:
>
> On Wed, Mar 11, 2015 at 10:24 AM, Jean-Denis Vauguet <
> notifications@github.com> wrote:
>
>> Actually I do agree with you: this best practice, is not always for the
>> best. Under the assumption it'll empower users, it actually "break" things
>> for some people, and that should be accounted for.
>>
>> It's becoming mainstream nonetheless, and because I wasn't sure whether
>> you used that as a test for HTML, or as a deliberate choice, I thought I'd
>> be better describing that "mainstream" policy :)
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/OggimusPrime/Bloccit/commit/a479974bf5ce5434460811599f1bd36919e2611e#commitcomment-10143511>
>> .
>>
>
>


ああ，本当ですね．消しておきます．


Bitte mgl. camelCase-Schreibweise "einloggen"
Üblicherweise nennt man eine solche Klasse "user", weil die Klasse alle User-Aktivitäten zusammenfasst.


U.K. slang for *something easy to do*.


I'd be happy to know what's the U.S. version of this!


Probably safe. Icons can look a bit broken until all cached data has been cleared from browser.


SB


Ist das auch zum Debuggen?


this is confusing, should we have --from-rev and --to-rev ?


I've commented 3 times asking for an explanation of these changes and I still haven't got one. All this does is make those properties of core readonly and non-enumerable/configurable. Why are they that way and not the others?


Mes ```vector<IntervalVector>``` sont des data containers.
Il y aura toujours un probleme si je passe en ```list<...>```!

Et puis la solution est peut-etre de mettre un message d'erreur plus explicatif...


Hi, I'm Pavel, a founder of a small startup called Blynk. It's a platform to connect Arduino, Raspberry Pi and similar microcontroller boards with smartphones. You can take a look at our project here: www.blynk.cc

We've noticed that your account on github is not very active. So we decided to ask if you could be so kind to let us use your account name for storing our open-source code. We had a kickstarter campaign where 2300 people supported us. It would be very helpful from your side to support such an amazing community. Please reply in any case to pavel@blynk.cc.

Looking forward to speaking to you.

--
Pavel and Blynk Team


我是试过的，看到差异才找到这里把它改了的。


I think these extra ctors are redundant, aren't they?


Oh nice! I'm going to put this in my word bank! :)

I watch a lot of English shows and hope to one day go there.

I've been to Germany, Italy, Austria, and Switzerland but need to make it
to England, France, and Spain.

-Ryan

On Wed, Mar 11, 2015 at 10:27 AM, Jean-Denis Vauguet <
notifications@github.com> wrote:

> U.K. slang for *something easy to do*.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OggimusPrime/Bloccit/commit/e82883b43ebfed335e528a1be9c1b00ab49379ae#commitcomment-10143566>
> .
>


No problem, happy to help. Glad we ended up with the same result, I thought I was seeing ghosts.

Thanks for including me in a reported-by :)

Cheers

Hraban


There is no CONFIG_FRANCO_THERMAL in your makefile. Why have you added that ? Maybe a mistake?


TeamCity xBIM :: XbimExchange :: Build [Build 117](http://ncl-xbim.hq.4projects.com:8080/viewLog.html?buildId=1149&buildTypeId=XBIM_XbimExchange_Build) outcome was **SUCCESS**
Summary: Running Build time: 00:03:29


Are we sure that getRootDirectories() can never ever return an empty collection? I have experienced a NoSuchElementException from this piece of code, unfortunately I don't have a reproducer. Anyway, testing a value of hasNext() and throwing an exception with reasonable message could help, what do you think?


Tweets van 1 dag 1 uur


@d10 This is pretty important, even though it isn't required for many slides. If we don't keep it consistent, there will eventually be errors. Examples: L244-255, L308-322.


This will be added.


i think that's the only type i use it with, and it's idempotent for me.
why is it failing for you? can you show us a --debug run?
which version of libvirt/virsh are you on?


We would usually say: "cake", "cake walk" or "gimmie".

I like the way "doddle" sounds over the above.

-Ryan

On Wed, Mar 11, 2015 at 10:29 AM, Ryan Haase <ryandhaase@gmail.com> wrote:

> Oh nice! I'm going to put this in my word bank! :)
>
> I watch a lot of English shows and hope to one day go there.
>
> I've been to Germany, Italy, Austria, and Switzerland but need to make it
> to England, France, and Spain.
>
> -Ryan
>
> On Wed, Mar 11, 2015 at 10:27 AM, Jean-Denis Vauguet <
> notifications@github.com> wrote:
>
>> U.K. slang for *something easy to do*.
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/OggimusPrime/Bloccit/commit/e82883b43ebfed335e528a1be9c1b00ab49379ae#commitcomment-10143566>
>> .
>>
>
>


Same as https://github.com/wei2912/gamedesign-js/commit/3818f2f4f4ecd3e5b184923c177f4eec57fe3de0


Ok, after some thinking it seems that this is unavoidable. Even with event emitter stop_job will need to be split. But can you call 2nd half  i.e. _stop_job? stop_job2 seems like new version.


I knew about "piece of cake". Would people say "it was a piece of cake", or "it was a cake", or something else actually?


Which colorscheme are you using on the new screenshot?


I've considered it. It is only for this case scenario which we use the "--from-rev".
For delta, backfill, backrevisions [1] we don't use --from-rev.
I'm thinking of the users that would never trigger a range. They might find using --to-rev weird.

We could also make both --to-rev and --rev acceptable.

https://github.com/armenzg/mozilla_ci_tools/blob/master/scripts/trigger_range.py#L65


OK, I will move this class to PromotionWebhookMapper class/file


changed by last commit... now you can assign value with or without "0b"


Thanks. Other than the extra ctors in the parser, this looks fine.
I think the commit message is lacking test information, though. Swiften has unit tests for the serializer (although seemingly not the parser for some reason), which should presumably be ported as well.


for this part you can simply call c.getJsonObj() and it will return you this.


That commit did not actually add a comment, just extra spaces :).


same with here, by simply calling p.getJsonObj() will return you a pre-json map


yeah, it is confusing what --from-rev and --rev mean, details to sort out
later

On Wed, Mar 11, 2015 at 10:33 AM, Armen Zambrano <notifications@github.com>
wrote:

> I've considered it. It is only for this case scenario which we use the
> "--from-rev".
> For delta, backfill, backrevisions [1] we don't use --from-rev.
> I'm thinking of the users that would never trigger a range. They might
> find using --to-rev weird.
>
> We could also make both --to-rev and --rev acceptable.
>
>
> https://github.com/armenzg/mozilla_ci_tools/blob/master/scripts/trigger_range.py#L65
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/armenzg/mozilla_ci_tools/commit/1caf0f06d91c5aecc9168a770d326fb6fcb25d01#commitcomment-10143671>
> .
>


"It was a piece of cake", or "it was a cake walk", is more proper (even
though it is slang).  While "it was cake" is more slang and "urban" if you
will.

On Wed, Mar 11, 2015 at 10:32 AM, Jean-Denis Vauguet <
notifications@github.com> wrote:

> I knew about "piece of cake". Would people say "it was a piece of cake",
> or "it was a cake", or something else actually?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/OggimusPrime/Bloccit/commit/e82883b43ebfed335e528a1be9c1b00ab49379ae#commitcomment-10143658>
> .
>


Looks OK as best I can tell. Not sure how to install snapshots outside of the extension registry in order to test it though.
Regards,John

     On Wednesday, 11 March 2015, 12:20, Jan Pilzer <notifications@github.com> wrote:

 Check the latest commit (733434b) please.—
Reply to this email directly or view it on GitHub.


Fantastic! Thanks.


Got it. Subtle yet meaningful. Thank you!


A nice title, do not change it please.


all case switches should have a default clause. if the case is exhaustive the default should raise an exception.


I think this code is pretty ugly.

Do you have a specific (measured) performance issue, or just think this should be faster?

Unless you have an actual performance issue I'm against this change, and if there is a real issue, then I think you can structure the code better.

I'm especially wary of making the encode modal - with encoding and cound/encoding modes.

The especially ugly thing IMO is the unintuitive combined mode.

[I do agree that the code could return the actual size on overflow, but that if we're going to do it it should be an improvement for the base encoder API, which isn't public anyway]


Introduction to course project


I don't understand the comment. Where does `0xc147cf0c` come from? Is it specific to the kernel used by wheezy?


@macumber just yesterday, @kbenne noted my fishiness.


好仔细 :+1:


used `public_info.get('duration')`


I'm a bit lost here. If you mean I should set up the iv only when a KEY_TAG has been set, then it should be something like

  String thisSegmentEncryptionIV = null;
  if (segmentEncryptionKeyUri != null) {
     thisSegmentEncryptionIV = segmentEncryptionIV != null ? segmentEncryptionIV
    : Integer.toHexString(segmentMediaSequence);
  }

Otherwise my code is not overwriting the segmentEncryptionIV because I verify that its null at line 217.

Or did I misunderstood something ?


I don't understand the comment. Where does `0xc147cf0c` come from? Is it specific to the kernel used by wheezy? Same for `0x148` below.


I wonder about the use of a definition list here.  It seems like overkill if there is only one 'term'.  Perhaps a heading element would be better, `<h1>` for the course title?


Hmm... What is this mean? :D This was done in #639


are all of these new? or we didn't need them before?


o.O


Michal, can you please comment this commit? Petr cannot merge the branch https://github.com/UnifiedViews/Plugins/tree/dpu/t-filesToRdf-1.5.1without your explanation


We didn't need them before, not now either, just completing.


ICSSolarCollector_#4723 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (638 of 638 tests passed)

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


How to disable shadow for toolbar?


I've just checked that N9005 Lollipop kernel source does not use this HWDEP_CAL.

Are we sure we can use this feature just because the acdb files are now Lollipop ones?


thanks for your assistance and patience!
I hope I did it right now


I am using the same vg for the kvm host installation. Don't know if that makes a difference.
```
andersbs@common50:~$ cat pool.pp
libvirt_pool { 'vg':
  ensure => present,
  type   => 'logical',
  autostart => true,
  sourcename => 'vg',
  target => '/dev/vg',
}

andersbs@common50:~$ sudo puppet apply pool.pp
Warning: Setting templatedir is deprecated. See http://links.puppetlabs.com/env-settings-deprecations
   (at /usr/lib/ruby/vendor_ruby/puppet/settings.rb:1139:in `issue_deprecation_warning')
Notice: Compiled catalog for common50.iis.se in environment production in 0.08 seconds
Error: Execution of '/usr/bin/virsh pool-define /tmp/pool.vg20150311-5070-bpk8wh' returned 1: error: Failed to define pool from /tmp/pool.vg20150311-5070-bpk8wh
error: operation failed: pool 'vg' already exists with uuid 29b5dd27-7647-4541-9c39-5d344b82002d
Error: /Stage[main]/Main/Libvirt_pool[vg]/ensure: change from absent to present failed: Execution of '/usr/bin/virsh pool-define /tmp/pool.vg20150311-5070-bpk8wh' returned 1: error: Failed to define pool from /tmp/pool.vg20150311-5070-bpk8wh
error: operation failed: pool 'vg' already exists with uuid 29b5dd27-7647-4541-9c39-5d344b82002d
Notice: Finished catalog run in 0.29 seconds

andersbs@common50:~$ virsh --version
1.2.2

andersbs@common50:~$ libvirtd --version
libvirtd (libvirt) 1.2.2

```
Full debug run at https://gist.github.com/andersbs/7b9d591a500fe5bd53bd


@NafaaBout good call.


@acaly I think no. Many times developers need to set different par set and use enum will make it long and not so simple...
You can see method ChestContentRegistration.type(int par1) to know how it work.


There is a race condition here.  In a very simple application, nothing unexpected (such as interrupt activity or a task switch) can insert a delay between lines 223 and 226.  However, if the complexity of the application increases -- e.g. by turning on an interrupt-driven peripheral -- you might get a delay.

This means the data path state machine may not be prepared and ready when the card begins transmitting.

For safety, consider enabling the DPSM (and thus DMA) before issuing CMD17.


@Lysergic-Acid  I have systemUI force close just when boot finishes, will this fix it?


Cool function if you want to show an image in your label.


We initially wanted to use a callback, but then we would need to pass it custom user data, which would involve using CCode attribute and designing custom structure which holds these data; and finally a callback defined/called somewhere. Using closures anywhere in the class would result in capturing 'this' pointer.

In the current code, the structure is either way created by valac in the static methods. In general, these changes were considered to have the same level of "ugliness" as this commit, thus it was left this way. If necessary, we can rework this to callbacks -- if this is the case, just let me know. I also happily accept any ideas, which would result in nicer code


I am just wondering if it wouldn't be better to have a Descriptor object which contains the definition and the list of values. Same for Stats object.

Or am I missing something?


This tests that the Seq(start, start+1, start+2) is called only when the interable.iterator() is called, which was part of the purpose of LazyIterable. But I agree it does not test the streaming of the underlying iterator, which is one of the goals of LazyIterable too.


Hi.
You can find me at magno.mabreu@gmail.com

Thanks.
Em 11/03/2015 07:49, "monicamohammad" <notifications@github.com> escreveu:

> HI My name is.Monica ,
> nice to meet you please
>
> contact me here i have something to Share
> monicamohammadi@hotmail.com)
>
> with you or you can give me your
> mail to contact you please
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/icemagno/malelo/commit/c4cba85986f1f120455ce4778f01ec13a8f3366f#commitcomment-10139326>
> .
>


This will prevent jobs from being stopped if tracking jobs in the database.


thanks, this looks good!


Interesting.  This looks good to me, I tested it and it works.  So, I say check it in.


You’re right, I’ve pushed the local commit. Sorry about the
inconvenience.

On 11 Mar 2015, at 15:59, Julian Raschke wrote:

> This is my first encounter with `git submodule`, but I think this
> commit has not been pushed, and the reference fails when updating the
> submodules.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/textmate/textmate/commit/f54a7cb45b51775a4ebbe739b330a7005ce96f04#commitcomment-10137538


我也又一次推倒重来啦…… 也换了frog，在windows和linux里都可以比较好地使用，满意。

关键点还是对blog既爱又恨。 爱在blog里我可以随时随地发表文章，恨在我这个人喜欢组织，喜欢条理。blog毕竟是一篇一篇的，sphinx文档是跟一本书一样的结构。 各有优势吧……


@ThomasArdal can you explain a little about why you choose to set Replicas = 0, Shards = 1.  I'm working on replacing our production SQL Server Elmah with a central ES instance, one type for all clients with a rolling index and I'm wondering if Replics = 0, Shards = 1 is the proper setup.  I've also put in a ticket with ES dev support to see what they recommend.

Thanks in advance!

CC @Mpdreamz


vec_string_1d_t это же вектор строк (слотов для строк – элементов множества).
Хеш-значение в этом векторе хранить не нужно, оно же просто индексирует элементы массива.
Откуда появляется size_t?


I would reverse the checks to have a positive check.

+ log.Warnf("Docker failed to enable SELinux, please recheck the system SELinux config")

Should be something like

+ log.Warnf("SELinux is not enabled on the host system.  Docker is not able to enable SELinux")


thanks for pointing that out:)


Many stats objects have a similar tree - like structure (see initial example). So can we also include here parent/child on the name only (not on min/max/ref of course).


Если будет переполнение при вычислении в size_t, то просто добавятся хеш-коллизии.
Но они – нормальное явление, так что ничего страшного не случится.


Yeah, damn synthetic tests! I'm also glad that we could agree on the improvement and that it won't hurt us performance-wise. :-) This "objdump -S" output with scanmem is really obvious:

"objdump -S" output on x86_64 before:
            data_value.int64_value    = *((int64_t *)(&data[offset]));
    857e:       49 8b 04 1e             mov    (%r14,%rbx,1),%rax

"objdump -S" output on x86_64 after:
  return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
    857e:       49 8b 04 1e             mov    (%r14,%rbx,1),%rax

With the option "-fno-builtin-memcpy" it's possible to enforce that GCC really uses memcpy() but that results in > 150% overhead. :smile:


If I am not mistaken, 3.10 kernel needs new modem binaries, yes?
Where are you porting trees from, caf?
Could you link me to source please,
Is it LA.BF.1.1.1-02210-8x74.0,  ( https://www.codeaurora.org/xwiki/bin/QAEP/ ) ?


Ran across your code when I was looking for examples to clarify ST's (awful) docs.  Are you confident that this DMA configuration results in complete block transfers?  I assume you'd notice if your application were e.g. failing to read the final 12 bytes of a block....

I ask because, from what I can tell, the SDIO peripheral on the STM32F4 fails to generate the last three DRQs during a block transfer.  I see this with or without PFCTRL set (you're not setting it here), and with or without using interrupts to synchronize.  There's an offhand comment in the reference manual about enabling four-transfer burst mode, which seems to fix it.  You don't appear to be setting it here, and I'm curious if you tried and what your results were.


Yes, we've been working mostly with DLNA guidelines from March 2014 :). GUN of the guideline should be 4DTU5 .


Would it really? It's fun IRL. Is there a way to make mounted entities rotate?


Changed OARequest from GET to POST.
Switched to SLRequest because TWRequest is deprecated.


In this file you can see how to work with multiply windows UI on Python3.4


how long is the testing time now?


Ok.  Still better than deleting stuff we don't want deleted, so two steps forward and one back.


open devices kernel is following AOSP progress

https://github.com/sonyxperiadev/kernel/tree/aosp/AU_LINUX_ANDROID_LA.BF.2.1_RB1.05.00.00.173.012


We store the setup and test scripts in the repo and then just tell codeship to run those scripts, so the setup and test blocks are a single line.


I wouldn't put the "user" suffix here.


Done in https://github.com/UniversalMediaServer/UniversalMediaServer/commit/b27bb0c9c3c80868fa744e3dc4ca48b92f45ccbd


Главное вызвать srand() до вызовов rand() по времени.
Иначе rand() скорее всего будет засеян константным значением, т.е. будет выдавать одну и ту же последовательность чисел при каждом запуске.


Can you offer the option to use `if (k.getKey().equals("_id")) return calculate(o);`?


In case that's useful, I've started pushing the version of these scripts I'm using here: https://github.com/killbill/killbill-integration-tests/tree/master/killbill-integration-tests/load/jmeter.


```java
this.buildHostsArray = new ArrayList(buildHostsArray);
```

is more java friendly :)


For now this is good, but when we have modules, we will have to redo it.


@Furrydaus
I wanna use this for another ROM too
Can I use it
I'll give you credits


find ich supaa


I think you can now remove everything related to the recording config episode-duplicate-detection which was added when my pull request was merged.


das war leonard
nicht ich


V "crs_pod2014.csv" je správne opravený účet "39100", ale zase ľudia volajú, že to nemajú aktualizované. T.j. keby sa dalo, aby nemuseli načítavať znovu generovanie, ale zase niekto už má odovzdané a keď im zmeníme generovanie súvahy, budú mať inú súvahu. Asi najlepšie = nechať na užívateľovi, aby si načítal sám, len mu pomôžeme, že mu napíšeme alert: "Zmenené generovanie pre účet 391. Pôvodne na riadku ... a teraz na riadku ... .


this is priceless


Send me a pull request to merge this back into main.


With this, either the 'modern' or homo sapiens will come to an end. Kill the human, save the species.


@lustrousgorilla - sorry to be a nag here, but if this is going to be part of Slycat, it needs to be an AMD module, without globals.  It looks to me like underscore doesn't have any support for AMD, but you should be able to configure curl to wrap it up as one.  That configuration would go in js/slycat-curl-config.js.  Otherwise, it should be part of the model.


Not a big deal, but a bias of mine is definitely towards "prefer composition over inheritance" and over multiple inheritance and mixins especially, so I'm vaguely concerned by what seems like unnecessary "mixin proliferation".

I don't see anything in here that requires that this be a mixin vs some kind of Service object or even a plain vanilla function that takes a section and a user as arguments.


I tested the original issue without this code and it seems to mostly be fine. The only thing that doesn't work then is Cmd-Z.


actually it was your fault ;) you changed `build/jsxc.js` the build directory is a no-go!! on every build it will be overwritten. you have to change `jsxc.lib.js`


If the intention was to reference the name of your publishing platform, then I think you picked the [wrong character](https://www.google.ie/webhp?ie=UTF-8#q=magic%20roundabout%20ermintrude)! ;-)


Hmm, this won't work with sharding I believe (which might or might not be a problem). Also, using KEYS is not recommended, using SCAN is usually better.


comment


Correction, perhaps all my stuff (https://github.com/tvheadend/tvheadend/pull/301) is already removed/replaced but i don't think the episode-duplicate-detection/dvr_episode_duplicate is used any longer so it should probably be removed.


I don't know if there is a need to pass a generator into a generator. I think being able to pass a custom generator into a robot is good enough.

I


Hello, I'm doing a Degree project on Face Manipulation/Morphin using Java code and am just looking for idea's and inspiration. Struggling to get my head around this to be honest. Regards, Caroline (am new to GIT so not sure exactly what I'm doing either at mo)


Hi,

nice addition, thanks. Maybe, this could also be interesting on this topic: [Mobile_Detect](https://github.com/serbanghita/Mobile-Detect)?

regards


Color `#B0B8BF` should be written as `#b0b8bf`


develop (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1148 of 1148 tests passed)

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


Nice. Thanks!


other directives are optional and can be chained


We could use a closure here for a little more efficiency:
```JavaScript
var self = this;
this.timer = scope.interval(function () {
    self.tick();
}, 1000);


Yeah, I agree.. I thought about that when implementing this. This could easily be rewritten as a function. The namespace pollution of using so many mixins could become problematic.

My self-talk and goal mixins rely a little more on the instance state (setting and accessing a form object on `self`), but I don't know if mixins are the way to go, going forward, for customizing `PageView`.


`くだしあ` はわざと？


May be due to the focusInEvent and focusOutEvent that are in ConstantWidgetBase.


You'll see that `encode_response` [here](https://github.com/sportngin/saml_idp/blob/master/lib/saml_idp/controller.rb#L33) calls to `SamlResponse` which [exposes that option](https://github.com/sportngin/saml_idp/blob/master/lib/saml_idp/saml_response.rb#L28). One thing we could do next is provide simpler access to that variable from the `encode_response` call


ICSSolarCollector_#4723 (Nigusse) - x86_64-MacOS-10.9-clang: OK (1648 of 1648 tests passed)

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


hmm, haven't even noticed this so far, does this work? i mean setting non-standard attributes in html? interesting. wouldn't a non-spaced shorter id be better? not the ports, those should not be exposed externally, as they can change i guess. but the ids (oth,sim,etc.) that could be even used as an id instead of non-standard attributes, and is easier to reference in css.


Getting closer.

![2015-03-11_100234](https://cloud.githubusercontent.com/assets/984608/6599246/bf1539cc-c7d6-11e4-93e3-3083008436f5.jpg)


thank you for reporting this. we fix the typo error


you could `list()` the indexes of `result` to make this more readable


ICSSolarCollector_#4723 (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1654 of 1654 tests passed)

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


This is not an error, 1.1 is the package available through yum.


ここのパッチを頂きました。
http://anago.2ch.net/test/read.cgi/software/1425420543/162


Rename to RESEARCH_DIRECTOR_ROLE


nois. can you handle an active state for clicked settings buttons as well?


We've wrapped the entire document in an "[iffy](http://en.wikipedia.org/wiki/Immediately-invoked_function_expression)", which includes the `this` property as the second argument: `scope`. In web context, `scope === window` which is preferable instead of directly referring to `window`.


Santiago Limón de la Rosa


I think we discuessed this before, but dont know exactly the arguments...  shouldn't any Exception no matter which type fail the promise?


:sparkles:


url can ben /forum/admin/derp if site is under a subfolder should move this below https://github.com/NodeBB/NodeBB/blob/57a64870fd579e6cefcdd3c2a99fd688aeefe16c/public/src/ajaxify.js#L39 after that relative_path is removed from url.


c11? is it confusing c11 with c++11?


ICSSolarCollector_#4723 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1147 of 1147 tests passed)

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


Hi:

Right.

Read http://cs-studio.sourceforge.net/docbook/ch29.html? on product name constraints.

-Kay

________________________________
From: Eric Berryman <notifications@github.com>
Sent: Wednesday, March 11, 2015 8:39 AM
To: ControlSystemStudio/org.csstudio.product
Cc: Kasemir, Kay
Subject: Re: [org.csstudio.product] removed touchpoint that copies .ini file, removed activation of pvmanager on startup, change CS-Studio name to Cs-studio, added m2e project files (cc9c908)

I believe this is an Eclipse thing; must be uppercase, then all lower case. @kasemir<https://github.com/kasemir> right?

-
Reply to this email directly or view it on GitHub<https://github.com/ControlSystemStudio/org.csstudio.product/commit/cc9c90886080f7c3b6fffc120fa179da94e9d1b8#commitcomment-10141283>.


it seems sometimes you also expect 4 elements, so this message might not be accurate


Document


yikes, thanks for testing - i see the issue


This commit should be reverted!
If you launch tests with valgrind you get errors!
This is due to test such as: OnCall(xxx).Return("foo"); where "foo" is no longer copied and a bad reference is kept on it!


Yay, progress. Keep it up!


@hcodes let's move this throwing right after `if(ctx.content) {`


This test fails because since `string` is passed "naked" to `seq`, it gets expanded out to an array of characters. So when there is the contains check for `string`, its checking for the string when its already been broken into a character array
```python
In [5]: string = "abcdef"

In [6]: s = seq(string)

In [7]: s
Out[7]: ['a', 'b', 'c', 'd', 'e', 'f']

In [8]: "c" in s
Out[8]: True

In [9]: l = ["yyy", s, "zzz"]

In [10]: l
Out[10]: ['yyy', ['a', 'b', 'c', 'd', 'e', 'f'], 'zzz']

In [11]: s = seq(l)

In [12]: string in s
Out[12]: False
```


I tried running your tests as is, and I'm getting failures for the two `actions` tests involving `actionOnLink`. It seems the browser is redirecting to `example.com/failure`. Are these in a known non-working state? or do they pass when you run them? I'm happy to fix things, but I didn't want to duplicate work if you're working on this at the moment.


in `Nbsock` naming this would not be a host but a `uri`. `setHost` seems to be not a good name and this error message doesn't seem to fit either.


Fixed on master.


:heart:


@hcodes Exception text is not right, there might be not only menu items, groups are valid as well.


Thanks for writing this diagram. One note, I've seen that you've added 'DroidSans.ttf'. Is this legal? Otherwise, we should probably remove it.


@vicb we should not use the int type


should the `transform` really be part of your public api?


Addint `.po` files won't translate the plugin. Am I missing `.mo` somewhere?


Mistake, is fixed now (and test you added passing)


Can you rebase your PR, @arthurnn?


develop (jasondegraw) - 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)


1: M111 unrecognized component name: testutils


@vicb
* We should not pass reflector around. It is OK for it to be global.
* If you want to make it local, you should change all the usages of the reflector in the system. Not a few selected ones.


.mo should be there


Thanks for the quick release! :smiley: :beers: (... and the credits!)


@schrockwell !!!! This is the issue. :+1: :hankey:


Why can't this just be data_obj[:has_data]. Seems silly to add a variable just to track something that's already available.


Hi Brian,

We were talking about two distinct cases, 1) 'key' not available, 2) 'value' is None. As you said, latter is not possible in ClassAd language, but the reason L1123 is written that way because I did find None as value for some of the keys. I will trust you on this and just worry about (1). We can close this discussion :-)


lol @Waverunner


https://github.com/easydigitaldownloads/Easy-Digital-Downloads/tree/master/languages

mo are there, although no po.


Yes, we checked already:
https://github.com/serbanghita/Mobile-Detect
and
https://github.com/piwik/device-detector

But they are not really lightweight in terms of performance, so we tried it this way for now.
If we see that this isn't practical we'll integrate one of the solutions above.


Primeiro Commit


ICSSolarCollector_#4723 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1654 of 1654 tests passed)

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


Formatter is the wrong name.  In Go, a Formatter would have a Format method.  You means something else here, perhaps Marshaler.  There's precedent for something similar in other packages: http://golang.org/pkg/encoding/json/#Marshaler


Also there is a mess here with po/mo files: https://github.com/easydigitaldownloads/Easy-Digital-Downloads/tree/release/2.3/languages


Add this


Please remember to write this trick in new readme - otherwise  library will not work


It would seem like the exponential phase representation you give is clearer, and as you say the cost should be negligible.
Re the other change. You should be able to simply edit your local branch, commit the changes, then push to github. This will automatically update the pull request.


this will not  work for version with 'extra_version' like 1.2.3b1. Also you are not adding the release there


Awesome! :shipit:


Seems the grunt potomo didn't worked
po should be removed and mo made


Remove commented code. Not a good idea to have commented code.


Waddup @gabehammersmith?


po should be always there, as mo. So user will be able to fix translation as he needs. See WordPress itself


Merged from upstream


Doh, thanks for catching that.


no the translations are centralized on Transifex, the fix will be lost


Since we `dup()` shouldn't `closeFd` always be `true`?


LOL


shouldn't this fail the promise instead? success would mean it successfully subscribed to a channel?


Dito


This seems like overkill to do it then already, maybe you are still bringing them together afterwards.  I am afraid this is more complex than it is at first sight.  Also the split moves in action_done: you should check split moves of split moves also, ...


I'll remove the PO files. @fxbenard no rush but do you think you could correct the pull command to not pull PO files?


This would break current MongooseIM master (and some internal projects).


Is there a reason why we need an attr_ at all? If we need it because we are doing some kind of attr export magic, do we need to set an instance variable for the magic to work?


We can get rid of this for now ?


With 100 languages you can't offer holding 100 po files in your language folder


No. http://en.wikipedia.org/wiki/C11_%28C_standard_revision%29


Also it seems to me, that WordPress locale is not respected - some languages are not in the way WordPress locales are.

But I may be wrong, so would be nice someone will check that.


@pippinsplugins It does already I don't get it


We aren't using any of that.


`bindSocket()` is only used here, so why keep it as a separate function?


@fxbenard They came down when I ran `grunt tx-pull`


Thanks!
Interestingly, I now agree with Brian that yellow and orange are too alike...and yellow might be too light anyways. We might want to consider going back to red as the fifth color.


Do we still need `closeFd`? Seems like it is always `true` now.


[src/librbd/internal.cc:2287]: (performance) Prefer prefix ++/-- operators for non-primitive types.

Change it to:
for (map<string, string>::iterator it = pairs.begin(); it != pairs.end(); ++it) {


double which - maybe use: a data structure made of a list of ...


If segmentEncryptionIV is set by KEY_TAG then you need to retain that value for subsequent iterations of the loop. So you need to remove L183 (note that with this line, segmentEncryptionIV is actually guaranteed to always be null where you do your null check at L217).

Once you do remove that line, the other comments probably end up making more sense.


maps


@pippinsplugins try `grunt potomo`
@slaFFik witch one?


This is more convention but I rarely use `this` when making method calls as `this` is assumed. I'll use it when calling `super`


Hi @jni
I was using profile_line to obtain an 1D array, but I dont get what I want, I mean:
when I run this:
>>> arr_f = profile_line(img[0], (pts[0],pts[1]), (pts[2], pts[3]))
[ 59.  70.  44.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.
   0.   0.   0.   0.   0.   0.   0.   0.   0.   0.]
It is supposed this must be an image intensity profile, but it is not.
Any suggestion, @jaimebayes


This is fine but Exception encompasses IOException


please add utf 8 headers - but wait, you dont need to, i already have at the branch, see PR #7


А что - есть такие предупреждения?


double ``is`` or are you a terrorist?


Should be 1.5.0.


What I would prefer as an intermediate solution is to have a gyp defined in ozone_impl.gyp and make it a build time option to disable this with shared component.


Refers to slipstream/SlipStreamServer#237.


I'm basically trying to change the original code as little as possible -- those files will not be compiled as this is the else case for `ozone_platform_wayland == 1`.


@slaFFik I'm using http://wpcentral.io/internationalization/ for setting up the languages


Did everything work with this ?


Yeah I know. But compiling with Xcode gives you millions of warnings like:
`qcommon.h:861:26: warning: redefinition of typedef 'mempool_t' is a C11 feature`
And starting from version 3.6 is defaulting to C11 anyway. So it's only a matter of time while Xcode switches from 3.5 to 3.6


I still think this is too lenient. If something is occupying the port we want, we don't want to silently ignore that just because we got one of the protocols.


Better make it clear that in the following query, lucene is not used.


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


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1148 of 1148 tests passed)

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


Some more explanation here on what I have so far. Decided to try and get these closer to what we will get when candles revamps to 2.51. These numbers came from a pull of the official svn around June 2014. I felt this was close enough to pre-Spartacus to get us in line with where I think Candles will go with the official revamp. Also removed the commented lines of Spartacus stuff, since once Candles finishes with his, it will match closer to his.


Please add a test to cover this.


and please do not commit directly to master, thanks


@jaredmorgs you must escape quote in the string :
browser\'s

I took care of it, i'll publish soon


In these you test how ::Proxy::BMC::Api#bmc_setup initializes IPMI logger using calls to the controller, which is fragile and makes it harder to understand the test. I would suggest to make ::Proxy::BMC::Api#bmc_setup method public and exercise it directly.


up to 8275e632


update performance


I think you can also use ```terminal in the future :-)


Maybe "scientists, and scientific programmers who..." -> "scientists and programmers". In any case, I'd remove the ","


@moloko   What do you think?


Oh ok that makes sense, thanks Kyle

On Wed, Mar 11, 2015 at 1:29 AM, Kyle Rogers <notifications@github.com>
wrote:

> The layout assumes using the member's unique id. If you change the file
> names then should show up automatically in the correct spots.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/ThetaTauMiami/thetataumiami.github.io/commit/d06763a76ae31cc3ba2ce649665ac52bcf5dd0ad#commitcomment-10135423>
> .
>


According to http://ex60.stopiccot.com:8010/builders/OS%20X/builds/778/steps/shell_2/logs/stdio fresh build emits 188 _is a C11 feature_ warnings.


This might not be lenient enough though. :)
Have you tested how this behaves on a system with IPv6 disabled? That's the only case I can come up with that we should care about.


edd-br.mo - there is not such in WordPress


@pippinsplugins install gettext https://www.gnu.org/software/gettext/


@jaredmorgs *message* is a javascript object and you can't define twice the same attribute.

I took care of it, i'll publish soon
I changed this code like this :
```
content: 'Settings Saved, do not save further configuration changes until HubPress confirms posts were regenerated successfully.'
```


Is this necessary? Doesn't `getaddrinfo()` fill this in via the `service` we gave it?


Maybe "participants" -> "learners" in the sense that it's in the next sentence when you talk about workshops.


Also what is `edd-en@pirate.mo`?


5-8 seconds is heavy. Can we try to speed it up by only using 5 channels or so?


I don't think `SystemException` can sort out the `getaddrinfo()` specific error codes.


utf-8 headers for an empty file? No thanks. I mean, what's the point if the file is only meant to trigger python discovery machinery. Once you put some code, then yes sure, of course.


Breton and pirate


`runtime.get_nodespace` (which is called by the `get_current_state method`) only returns information if the nodenet's current_step is newer than the supplied step. If you don't care about polling too often, 0 is fine.


As I was loading myself out for Nightfalls yesterday, I was really wishing damage type and value were both visible at a glance.


Wrong text here I think. :)


Fair point, will try to send PR for this things as well.


Add the property createdAt in CUser.
Add the property totalPoint and wonBattles in CPlayer.
Initialize the constants of PLAYER_TYPE , of STATUS, of SPELLS.
Do the methods get and insert of CUser and CPlayer in CMysql.


Also there is no `edd-id.mo/po` in WordPress but you have it and `edd-id_ID.mo/po`


I was unsure which way to go:
- the reflector is available globally,
- it is also registered in the DI.

I you feel it is important, I'll change the code to use the global one, let me know.


<ulink url='http://rpm.kamailio.org/docbook/sip-router/branch/master/rpc/kamailio_rpc.html'>
Why "rpm.kamailio.org"

you still have "call ser" that should be "call Kamailio" :-) line 765


checks of code-analysis fail otherwise


Doh! On it.


we need isfinite instead of isnan to also catch `inf`


In the Brackets extension manager you can install from URL to download the latest code from this repository.
See the updated readme: https://github.com/Hirse/brackets-outline-list/blob/master/README.md#installation


The default case (no `--single-process` did) did, but passing `--single-process` did not (which was basically the case before if I remember correctly).
However, if I do not create the `RemoteStateChangeHandler` and pass `--single-process` Crosswalk also does not launch.


> Breton and pirate
I meant - that's not from wpcentral :) there is not such official WordPress thing (at least right now).


but please wait until the branch is merged with those changes, otherwise you create work.


Were these two needed somewhere?


Headless Laptop, I see. :D


rationale ? TS ?


develop (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1145 of 1145 tests passed)

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


you can do the changes on my branch, so all cleanup is together


"investments here": I'd remove here.


I feel like there might be a settings page in our future :)


How come this got moved here?


Elsewhere you use `api_endpoint_v2` instead of the endpoint directly, maybe that should be used here


And perhaps other files, I did just a quick search, the whole list should be manually checked if you want to deliver translations that WordPress officially supports.


that specific code-analysis does not fail if the file is completely empty, I wrote it with that in mind already (and specifically for ``__init__.py`` files): https://github.com/plone/plone.recipe.codeanalysis/blob/master/plone/recipe/codeanalysis/python_utf8_header.py#L20


We probably need to clean these up now as well.


? would it not get me `foo.bar` in case of `[0].foo.bar` so if my prefix is `foo` then it would not match right? I am not sure if that is the desired behavior.


Sure, next time I will try to do that or just send a PR


TeamCity Akka.NET :: Akka.NET PR Build [Build 130](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=223&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


"This realization is at the heart of modern software development processes, but to date, these have only been adopted by a small minority of scientists." Maybe a bit long?

What about?

"This realization, at the heart of modern software development processes, has only been adopted by a minority of scientists."


Thanks for pointing it out though!


This is such a weird fix.


Also I do not understand why are we updating the key. key should be immutable use a different variable.


I haven't checked where this is used, but is it sufficient to just test IPv4? And only localhost?


ok, i wasnt aware of it, can you anyway please revert it? rebasing the branch with this changes is more work than it should be


@it's already checked don't worry id and id_Id are provided - I'm managing more than 100 projects amongst EDD, all the languages are setup as they should be ;)


TeamCity Akka.NET :: Akka.NET PR Build [Build 131](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=224&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


It was using version range.  I wonder if we want to keep using range?


For some reason I thought 1.5.0 had been released last week... I'll update.


love tab bars on ipad


Is this really proper C++? Shouldn't you have a `new` here?


If our Travis build system isn't a ball of duct tape and hacks by the end
of this project, we've done something wrong.
On Mar 11, 2015 9:43 AM, "Braedy" <notifications@github.com> wrote:

> This is such a weird fix.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/CMPUT301W15T07/TravelTracker/commit/af24df7afa2381764bad220635745d085f136534#commitcomment-10145290>
> .
>


Missing comma: "but like the web can accelerate ..." -> "but like the web, can accelerate ..."

I like this paragraph very much.


You're probably right about 'DroidSans.ttf
I uploaded it together with the rest, without paying attention. (we should probably add it to `.gitignore`)


No longer case insensitive. Why not keep the original line?


The single-stack ipv6ers are coming :)


wait, rebase worked


so, keep it in master :)


FYI https://translate.wordpress.org/projects/wp/dev/br/default for breton ;)


"The activities within the Lab, in turn, provide additional learning opportunities around computational thinking while generating new tools and projects."

I think this is in relation to Mozilla Science Lab, should we remove it?


do you know if it's possible to make a test that tests this directly?

On 11 March 2015 at 16:16, Mathias <notifications@github.com> wrote:

> This commit should be reverted!
> If you launch tests with valgrind you get errors!
> This is due to test such as: OnCall(xxx).Return("foo"); where "foo" is no
> longer copied and a bad reference is kept on it!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/dascandy/hippomocks/commit/51f721950b9eb1ce41512e8ae199ab262b432d6e#commitcomment-10144556>
> .
>


I remember having problems with Stream handling, but I can't exactly remember about the Events. It is possibly that I changed it to explicit handling simply because we know very precisely their lifetimes: can never escape from `executeAcc`, so there isn't really any benefit for having them handled by the GC. If you want to extend that lifetime, then sure, it should be fine to put them under GC as well. There is also the question of _when_ the GC is run, which shouldn't be during the `eval`, so there will be some delay in cleaning everything up, and there will be a longer pause when you do.

Anyway, be careful that they still work. I can't remember if it's still possible to use a null Event (in case the finalisers are firing too early), in that waiting on a null Event might just trigger a whole device synchronisation, rather than throwing an error. So run some tests under `nvprof` and check that kernels are still overlapped, etc.


didn't we want to use http://localhost/resource/MotorGarage-1?


regarding `id` - WordPress doesn't have it, so why other plugins should provide it? IMO no need to have `id` at all.
But anyway you are free to do what you think is right :)


Hey, were we talking about wpcentral?


Also remember that the names are mapped to the unique id's in the csv in the _data so you should be able to write a quick script to rename rather than doing it by hand.

> On Mar 11, 2015, at 8:34 AM, loomisdf <notifications@github.com> wrote:
>
> Oh ok that makes sense, thanks Kyle
>
> On Wed, Mar 11, 2015 at 1:29 AM, Kyle Rogers <notifications@github.com>
> wrote:
>
> > The layout assumes using the member's unique id. If you change the file
> > names then should show up automatically in the correct spots.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/ThetaTauMiami/thetataumiami.github.io/commit/d06763a76ae31cc3ba2ce649665ac52bcf5dd0ad#commitcomment-10135423>
> > .
> >
> —
> Reply to this email directly or view it on GitHub.
>


Extension method FTW? :)


```
result = db.session.query(Tag.tag, db.func.count(Tag.tag).label('count'),
                                         db.func.max(Tag.created_at).label('created_at'))\
    .group_by(Tag.tag)\
    .filter_by(username=name)\
```
가 되어야하는거아닌지


This comments doesn't make sense anymore. Could be something like "From the hash, generate paths on the form C1C2/C3C4/rest_of_the_hash.jpeg"


dsadsa


Gotcha, I removed it in 9e58fca322bd09a2816a717f768472ee564e3b8f


Ah I think you are doing it so that the code after this can run properly. OK. ignore my comments.


@slaFFik thanks for the catch I'm removing it right now from Transifex - wpcentral is my I'm based on + WP core glotpress repo also ;)


? 이거 왜고쳤습니까.
`Integer`인 컬럼이 `primary_key=True`면 `unique=True, nullable=Fase`랑 같은효과가납니다.


Holy live-reading sampled analogue values from a SpiNNaker machine Batman!


Why would a `send/2` not return? Specifically in the light of all implementations being `gen_server:cast`s, which return `ok`?


Bump version before this.
up to 51c435d0c


Use the connected method to start the timer?


"Provide peer- and institutionally-recognized rewards to encourage students to make acquiring these skills, and passing them on, a priority."

Maybe simplify to:
"Provide peer- and institutionally-recognized rewards to encourage students to prioritize acquiring and passing on these skills."


develop (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (639 of 639 tests passed)

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


Please rebase this. I cannot apply this with master tot Chromium rev


Although I should revert this i think because the yum dependencies are enforced when the rpms are built not by this. I will make this change and push it up.


there is none, he literally just replaced msm_thermal.c code with his own so there really is nothing to change since msm_theraml.c is already in the Makefile and its enabled by default.


2.x버전대 class 선언스타일 3.x에선 안쓰는걸로


Ok I'm beginning to see the light. What you mean is that if we see a key tag with an IV, we should always use that IV with the key for each media sequence. If there are none, then we should use the media sequence number for each media sequence. Is that what you mean ?


Is it possible to assert something on the returned `UnknownTypeException` so we know it's thrown from the code path we expect?


yes


See note on `send/2` above.


Ooo, your own Twitter account, noice!  There really is a shield for everything.  Think I will have to steal this idea too :-)


Are you sure that prefixing the key by "SQ-" is a good idea ? Could it be possible to use a more explicit key ?


Oh no, if my [pull request](https://github.com/Panopto/Moodle-2.0-Plugin-for-Panopto/pull/39) will need to be applied on top of this commit, then it will be a nightmare to rebase...


we need to use this stubs as we still have some custom va calls


This number is very outdated, right?


In css add https://paste.lugons.org/show/c5MJgum6K2JbAfWIAlmo/


Another issue, the `dup()`-ing might be unsafe with the `shutdown()` method. Calling `::shutdown()` on a connected socket affects all duplicated file descriptors, not just the current one. Not sure what it does on listening sockets.


These checks are redundant with line 86, aren't they?


Yes. I think the right thing to do is revert everything back to how it was, then at L217 do:

```
String thisSegmentEncryptionIV = segmentEncryptionIV != null ? segmentEncryptionIV
    : Integer.toHexString(segmentMediaSequence);
segmentMediaSequence++;
```

Then pass that into the Segment being created.


:-( It happens to me time after time. Intellij shows it as only one file and I forget that there are several. I'll try to remember it next time.


Oh. And.. Thanks ;-)


Yeah, I will see if I can get some logs for you regarding 13.2 and bootstrap failures.


Should not be removed (needed for phrasea engine).


The way you're supposed to do this is:

```
try:
    import something
except ImportError:
    pass
```

And by convention, conditional imports are below normal imports, because it looks cleaner. If you have it open, you could just change that quickly.


I'd prefer to keep the `+gpgme` as the idea with the features is it's something a user/consumer would want to know.


IIRC, argument evaluation order is undefined in lua.
The `monotime()` might happen before OR after the `:step()`


Currently returning a Map of headers.
This method could return another type, but this should handle gracefully `null` headers, so that they're not written to the response.


(But we can delete this)


What do you suggest ?
The probability that another dispatcher use the same key is low.


Now that this package contains only one class, maybe `CacheControl` should be moved in another package?


if (creatureinfo && spawnpoint)


TeamCity Akka.NET :: Akka.NET PR Build [Build 130](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=223&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:12:13
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


@staabm I believe this is correct. When a subscription promise is returned its consumers register their own callback via `Promise::watch()` to receive notifications when new data is available. The promise should only be failed if the connection is unexpectedly severed. Unsubscribing results in a successful completion (resolution) of the promise.


No problem ! We're a team.

> On Mar 11, 2015, at 8:53 AM, Ivan Zaera <notifications@github.com> wrote:
>
> Oh. And.. Thanks ;-)
>
> —
> Reply to this email directly or view it on GitHub.
>


here the cert item is highlighted.


TeamCity Akka.NET :: Akka.NET PR Build [Build 131](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=224&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Persistence.Tests\bin\Release\Akka.Persistence.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHel... Build time: 00:11:41
### Failed tests
```
Akka.Persistence.Tests.dll: Akka.Persistence.Tests.GuaranteedDeliveryCrashSpec.GuaranteedDelivery_should_not_send_when_actor_crashes: <no details avaliable>

```


ah, I knew I forgot something :/ now I remember


Because "no-cache", "no-store" and "max-age" are quite opinionated in this implementation, the XML configuration is trying to mirror this. But this is not conveyed by the XSD right now, as everything is an attribute in the `<mvc:cachecontrol />` element.


Would this enable reducing the cost of other cards? Not only self-adjusting cards like Affinity?


Please do not use switch case, use simple if statement to limit processed numbers between [0-9] by looking up their ascii encoding:  then pass the byte to blinker, blinker doesnt care what number its displaying, it should just display the number. switch case is just an overkill.


Guys, please be careful not to merge changes like this to master. This commit should have been rewritten to use the new release version of `imglib2-algorithm` containing the needed functionality, and _not_ `0.1.2-SNAPSHOT`.


Popping in late on this: Using centroid and then doing nearest neighbor doesn't always work in the case that the polygons are not convex.


Would it be better to override this in organisation? I think we always want to retrieve the type too.


This can't work, since you fetch your filters only when you have datagrid results, but you didn't create your datagrid yet.

I think you should do $this->setFilter right after execute, without the if structure around it.


Shoot, you warned me on top of this.


@maclaw I'm having systemUI force close, do  you know how to fix?


This was 2 in the old exmaple. it takes very very long otherwise.  ok @agramfort ?


> It seems that the SVG file viewed in a browser seems to show that grey overlay. Don't know why...?!? Maybe a transparency layer on top of it?

That is exactly what kept me from using the SVG so far, @garvinhicking. I don't know what it is, either. I assume the same – it is some kind of transparency layer that gets misinterpreted in the version of Illustrator that was used to export it, but worked in the version of Illustrator that it was created with (or in whatever program it was created with). It is definitely necessary, because if you remove the layers for it in Sketch, the logo does not look right, either.


I was thinking the other day, all html output should ideally use [output renderers](https://docs.moodle.org/dev/Output_renderers), in combination with [Output API](https://docs.moodle.org/dev/Output_API) methods - similar to how it is done in the other parts of Moodle. Not a quick job to refactor existing raw html though, may be a task for TODO list.


Maybe, but I found this "SQ-" not very meaningful...
What is the difference between this dispatcher and the other one with key "MyNewIssues" ?


Good point. I blame lack of coffee and a couple of late nights for any other such mistakes...


Basically you're trying to access a Sandbox company over a Prod URL or add a Prod Company over Sandbox URL.

If it's the first scenario, make sure you've got the `$IPP->sandbox(true);` line somewhere in your code.

If it's the second scenario, it might be interesting to use Intuit's API Explorer and see if you're able to add a customer that way.


No need for this @gkimaiyog , you can just say `LED[j]=k&1;` it can only be zero or one.


@rallytime Thanks!


`git config pull.rebase true` :)


I think `options = options ?? new RequestOptions()` reads more easily.


No need for this @gkimaiyog , you can just say ```c LED[j]=k&1; ``` it can only be 1 or 0


Glad to see that @tpietzsch fixed it with e76bc865c6e80d82fee896dc3f53718a72b5aa2c. The downside is that `git bisect` will now never work correctly through the commits from this topic branch, since that `SNAPSHOT` will disappear from the repository and none of these commits will build later.

Hopefully that makes sense. Let me know if not, and we can write up a wiki page describing the tradeoffs in more detail (it is something we discussed at length verbally at the last ImgLib2 hackathon in Madison, but I know that doesn't help you, @tinevez).


Ok sorry it took so long for me to understand. The draft is not clear abount that point. I'll close this pull request and create a new, hopefully clearer one with a new branch.


This fix makes ``virtual-ows`` tests pass again. Some change was introduced in ``geotoolkit`` which adds feature identifier attribute as FeatureType property. Questions was raised on ``geotk`` list: mainly because the feature identifier is not explicitely part of the FeatureType. Distinction is made with other properties (property name is prepended with ``@``)


You removed the testing for the regressor, multi class and predict? Isn't it more important than gaining a couple of time in the testing?


Good point. We're never setting it. And my suggestion of attr_reader was wrong anyway, I should have said attr_writer. But we don't need either.


Instead of a comment, perhaps this can be extracted to a method called `FindServerSideResponseType()`?


loving this.


Oooh sorry, I didn't see that you changed it. I'll change it back.


Let me know if I need to add a commit for this or if someone can just delete that line on master...


toggle LED if not sending [0-9] numbers instead of disabling


You can safely pn_ssl_domain_free(domain) after you've successfully pn_init_ssl()'d.  The pn_ssl_t keeps a referece to the domain (which is reference counted).


Does this need a public setter?


This commit allows to get a batch centroid with an id, list all of them with some criteria and create a new one


Much better signature indeed. :)


Ok po files removed.


TeamCity Akka.NET :: Akka.NET PR Build [Build 132](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=225&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


@kingjr nope we need to save the seconds they ramp up. We cannot test everything. Often we write these grid-like tests during development but only keep the most basic ones. Similar pattern in the ICA code.


GetCounter return LowType (uint64) https://github.com/TrinityCore/TrinityCore/blob/6.x/src/server/game/Entities/Object/ObjectGuid.h#L158


In the DiscountCurve case, the array y would contain discount factors, would it not? In that case, do you mean to apply the shift to the discount factors or the corresponding zero rates?


Azahara María Porras Tejada


In this new implementation, HTTP1.0 methods are now deprecated, even though CacheControl does not provide ways to configure "Pragma" or "Expires" headers.


Dont use switch case in embedded, optimize your code


This one is fine I think.


lol. the "features" shouldn't actually change anything, so I'm not sure that they are actually features :P


RA isn't used (or rather, provided) for containers either.  IIUC that's why Spike added the Proxy NDP stuff into Felix.


once Strings.splitStringToArray separates the indicesBoost argument by comma, shouldn't a different delimiter be used to separate the index name from sBoost?


FYI, SI-8462 is already fixed but it's merged only to 2.12 branch.


What does `returnExportsGlobal` bring against `returnExports`? The code sample at UMD repo is buggy?


Do these processing steps with an [ES Transform](http://www.elastic.co/guide/en/elasticsearch/reference/current/mapping-transform.html)?


fair enough, your call. Could we keep the old version in disabled? I see the multiclass, predict and decision_function as a future source of errors in our next PR.


Do you remember why?  I forget.


lgtm, good catch and thank you


should probably have this as a lockPref - we don't want to be pushing up hundreds of MBs of cache to the server


@kingjr I kept the lists and the for loop. Next time you can just add cases there if you have doubts.


using a private boolean that indicates when to use HTTP1.0 headers - this is enabled when one of the deprecated methods is used. Using only `setCacheSeconds` would result in the new implementation with HTTP1.1 headers only.


Wrong file path here, also perhaps give it a more descriptive file extension, like `.js` or `.cfg` instead of `.txt`?


Looks good, modulo the other comments I've already made - thanks.


Cool.


Either `returnExports` or `returnExportsGlobal ` should be replaced with the real module name -> `Swipe`
See the newest codes,  <https://github.com/lyfeyaj/swipe/blob/master/swipe.js#L10-L25>


I would recommend getting rid of this section entirely. Given that prefs.js will be synchronised with the server, the user will see an update message once (to tell them of new features) and then it will be saved.


Not sure about moving IPv6 down here, because I think it's covering some 'how it works' material as well as 'how to configure it'.  Thoughts?


Brent;
@roryk knows this code better than I do so might have ideas as well. I think you're right on with the difference. It looks to be from annotating with the extra VEP columns. I guess after normalization one of the inputs doesn't match and so it won't pull in that info. That's probably the price of doing business if GEMINI will be normalizing/adjusting the variants. Nice work on all this. Brilliant additions.


`ResponseEntity` is now using `CacheControl` for defining the "Cache-Control" header. This concept fits well here; but if we decide that `CacheControl` should be able to write other headers, then the API would be less intuitive here.


Remember to nuke the skel for Firefox - ideally it shouldn't be there at all (but it might be needed for the generation of user.js)


s/programmed/passed/ ?


Not in 3.3.5 where this was developed. :)

https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/game/Entities/Object/ObjectGuid.h#L102


Is the underlying transport/connection/etc dead now?   Should a 'continue' be inserted after this?  else we fall through to the SASL config.


Good catch thanks!


similarly s/programs/passes/ ?  Makes the para a bit lighter.


Or "programs those rules using ``iptables`` and ``ipsets`` commands" ?


Good job.  You could also rewrite this as

```ruby
@like = @song.likes.new
```


This method isn't actually necessary, since we never show the `new` page.


Seems like we're leading the reader up the wrong tree here.  Can we be clearer upfront about what the agenda of this part is?


TeamCity Akka.NET :: Akka.NET PR Build [Build 133](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=227&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) is now running


@chapmanb and @roryk during all of this, we kind of decided that it's probably best to decompose and normalize before annotation. I just sent a msg to the mailing list and it'd be good to get your feedback.

but yeah, I decided that the missed value was because the normalize changed the position enough (which also led to realizing that a lot of the annotations *should* change after normalizing).


Is this generated GemName.sample_app(args) method still valid?  I see we pulled it out of LM.


TeamCity Akka.NET :: Akka.NET PR Build [Build 132](http://petabridge-ci.cloudapp.net/viewLog.html?buildId=225&buildTypeId=AkkaNet_AkkaNetPrBuild&guest=1) outcome was **FAILURE**
Summary: System.Exception: xUnit failed for the following assemblies: D:\BuildAgent\work\49b164d63843fb4\src\core\Akka.Tests\bin\Release\Akka.Tests.dll   at Microsoft.FSharp.Core.Operators.FailWith[T](String message)   at Fake.XUnitHelper.xUnit(FSharpFunc`2 s... Build time: 00:10:36
### Failed tests
```
Akka.Tests.dll: Akka.Tests.Actor.InboxSpec.Inbox_have_a_default_and_custom_timeouts: <no details avaliable>

Akka.Tests.dll: Akka.Tests.Routing.TailChoppingSpec.Tail_chopping_router_must_throw_exception_if_no_result_will_arrive_within_the_given_time: <no details avaliable>

```


Hey, they fix some bug :stuck_out_tongue:


<3 this


:+1: :tada: :clap:


Really close -- I'm guessing you set this up and never came back to remove the unused routes.


Yes, sorry about that. Usually I rebase inside Eclipse, and this time, for no good reason, I just did pull / push from the command line, and did realize that the setting wasn't set.


Good start @HenriqueCarvalho -- your JS was great.  Just ran out of time?


Ah, yes! You're right, I didn't see that. Moreover, that is akward, I do have such an UMD implementation in my own libraries. Thanks for fixing this!


The reason I'm not doing a manual alignment based on the content of the previous line (like you did above and in many other places), but use a 3 character indentation, is because `hbformat` would change the former, while keep the formatting of the latter as-is, or would in fact format it the latter way if not formatted correctly. IOW, I'm trying to use a machine reproducible format. I know manual alignment may look better in most cases, but as soon as someone else touches the code, it will not be followed and will create extra noise when running `hbformat` on it. Same reason for `value := ;' lines and `RETURN ;` lines.


Interesting. Sorry for the bad PR ;)


Also, great work on committing regularly.


okay, I suppose this works. Better would have been to instantiate the access token only before the test, not once, globally. Anyhow, we can merge this for now.