I guess Thierry did this to make use of dired-async's features in the helm-find-files' actions.  OTOH I agree that loading helm should not change the behavior of a different thing, dired.  That's an error, be it easy to fix or not.

But the error is in dired-async: it breaks with the rule that loading a package should not make user visible changes.


The original version was just "DIN," which is great in the handful of countries that still refer to it that way, but not the rest of the world.

Every ISO- and DIN-standard size has the same sqrt(2):1 aspect ratio.  There are three series (A, B and C) and each series has 11 scalings (0-10), so including all 33 of them isn't an option.  "ISO 216, DIN 476, A4" seems like it might be a good compromise.  I've pushed that and corrected the case, which was an oversight on my part.


> people will only want to send a message with a name

I don't really agree. I think if you only have one of `(name, data)` then it's most likely in fact that you have `data`. I think it adds more confusion than convenience for it take that to be `name`.


Use "address_form" instead of "f"


Let me just say "ewww"


Use "contact_detail" instead of "f"


looks like this import is not used?


Ise the setText for the effect to set the correct tooltip text.


Can you remove this `echo`?


Because our build server doesn't support new C# syntax yet. We'll return to new C# features when TeamCity 9.1 is out.


Maybe better use  "Absorb 1 <i>IIf a source... </i>" as text.


hi


use here the possible parameters so you can remove most of the rest of this class.


If we start doing this, we should think about a proper versioning schema, right? Currently we only use it via git tracking anyway, so there is no point for versions.


Ah! Thanks for catching that


Updated. Thanks!


I didn't mean to add all formats, my "..." was to be taken literally. Your proposal looks perfect to me, thanks.


how about changing task.model.name to always return "" instead of nil


I'll have the same problem with CompatInfo 4.0 branch.
Branches maintenance will be applied tomorrow (I've no time to it before)


Can you leave this as it is. It's being used later on.


Asked Him(Georg) on gerrit and have added you let's see if he replies and
if it helps solve anything.
On May 11, 2015 4:02 PM, "Mrcl1450" <notifications@github.com> wrote:

> There are still pa_therm0 and pa_therm1 in
> /sys/bus/platform/devices/pm8xxx-adc
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Mrcl1450/android_device_sony_huashan/commit/34e10cc34c1d4023b15ef3d51ad8c9a5efb466bd#commitcomment-11130738>
> .
>


FINALLY!


Not exactly. And in this case, the line-height used is +14px. Is it ok?


I become no Quest Complete Credit.


USe the ChooseColorEffect class here.


Hello Marcello,

Please check the documentation about the R-R packet. http://www.zephyranywhere.com/media/pdf/BH_DS_P-BioHarness3-Data-Sheet_20120919_V02.pdf page 5.

There is description of the data which says "Alternating ± sign at new
detection". With Integer there is no alternation of sign at new detection there are just big values.


So it seems related to the replayer. It seems to work with latest replayer. I will give it a shot later.


There are two fake implementations of Gossip: `mockGossipComms` and `TestGossipRouter`.  `testAllocator` uses the former and `makeNetworkOfAllocators` uses the latter.  And then there's `TestCancel` which almost does what `TestGossipRouter` does, for reasons that are unclear right now.

So, `testAllocator()` is something like `makeAllocatorWithMockGossip()`


You cannot simply import referencesource. Build is now broken for monodroid profile.
https://gist.github.com/atsushieno/c8dbcc479386631ff3fe

Unlike previous sources it now depends on Microsoft.CSharp which is likely missing in bare build.


>> people will only want to send a message with a name

> I don't really agree. I think if you only have one of (name, data) then it's most likely in fact that you have data.

Well I think there are valid use cases for both.

However, it is certainly idiomatic that when an argument is omitted, it's value is assumed null.  It's quite unusual to have a method definition for `publish(name, data)` for 2 arguments and `publish(data)` for 1 argument, typically the arguments remain in the same order whether you provide them or not.  I think changing the order based on the number of arguments is a really bad idea.

To avoid ambiguity though, I am happy to require a `name` & `data` argument if that message format is being used i.e. the name & data pair.  If only one argument is provided, then a message or array of messages is required.  That will be far easier to document as well.


Here you need a second static ability, because the color change is done as long as Shifting Sky is on the battlefield (also for permanents that enter the battlefield later).


I live in the U.S., so for me it was really a question of whether or not "A4" was sufficient enough to be recognizable as representing the whole series to people who use it.  If it's a problem, somebody will tweak it.  :-)


I'm not sure if this is a common way of implementing the observer pattern in golang, but certainly in Java it would be surprising to have unsubscription as a 'side-effect' of notify. `notifyAndUnsubscribe` is a bit unwieldy though...


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


* This change was made during 2.1 development, and the migration guide basically states to recreate the archetype.
* We can easily move the companion's `apply` method logic to `GatlingRecorder`


Thank you.

Date: Mon, 11 May 2015 04:18:27 -0700
From: notifications@github.com
To: android_device_sony_huashan@noreply.github.com
CC: marcelo-fil@outlook.com
Subject: Re: [android_device_sony_huashan] Open Source Thermal Management (34e10cc)

Asked Him(Georg) on gerrit and have added you let's see if he replies and

if it helps solve anything.

On May 11, 2015 4:02 PM, "Mrcl1450" <notifications@github.com> wrote:

> There are still pa_therm0 and pa_therm1 in

> /sys/bus/platform/devices/pm8xxx-adc

>

> —

> Reply to this email directly or view it on GitHub

> <https://github.com/Mrcl1450/android_device_sony_huashan/commit/34e10cc34c1d4023b15ef3d51ad8c9a5efb466bd#commitcomment-11130738>

> .

>

—
Reply to this email directly or view it on GitHub.


It is a shortcut. You will always do both things, _to notify_ and _to remove_ all the observers, in a sequence... Separating  both operations would duplicate the number of lookups in the maps...


> This change was made during 2.1 development, and the migration guide basically states to recreate the archetype.

Maybe I failed to notice as my old archetype classes kept on working.

> We can easily move the companion's apply method logic to GatlingRecorder

Yep, it seems to me that this logic should be in `GatlingRecorder `.


This commits breaks the intended semantic. There are memory regions (let us call it "other", which are part of "available"  memory. These regions may be in the list before the actual "available" regions. If the "other" region are processed before the "available" region, the operation "ram_alloc()->remove_range" has no effect. Later the "available" region is parsed (which can overlap with the "other" region) and get added to ram_alloc, which shouldn't include the "other" region. It is essential that first all available memory is processed, and later on all "non-available-memory" is processed, so that it get properly removed from ram_alloc.


:+1: In related news, I am disappointed that there isn't a duck emoji.


very interesting way


Not sure why that happened, but in version with `@Autowired` I got exception with message: Error creating bean with name 'jsonFormatter': Requested bean is currently in creation: Is there an unresolvable circular reference? (this happens in saos-batch integration tests)


@Senorsen Thanks for patching it! I got the same error using MySQL and this patch fixes the problem :)


uset getFromTop()


Don't use the Pick zone, it's an old construct and should be removed everywhere it's still used.


My bet is this will break when we update ruby one day. Would it be smart to round this up before asserting?


use the player. moveToGraveyard method.


use the player. moveToHand... method.


How did you install mraa? Did you compile and flip the right flag to get the Java bindings out?


> publish(data) for 1 argument

I don't propose that, and it can't work anyway because there is no way to tell the difference between `data={<object>}` and a message.

> To avoid ambiguity though ....

Yes, I agree, maximum obviousness and also not irreversible.


no need for this or?


Multiply by `1`? Is that a form of casting again?


@da-anda Is this a general Multi remote or is this a Harmony remote?

Can this become
```-msgid "Harmony remote"``` Im confused why there's Harmony in brackets, it seems to me we already have a uNiversal remote :) entry anyway, so unless this is yet another, then makes no sense to distinguish.


>Maybe I failed to notice as my old archetype classes kept on working.

A bit strange...

>Yep, it seems to me that this logic should be in GatlingRecorder.

Sure, i'll move it.


@ava14: thanks for your great work too.
By the way, you forgot to update the commit message again.


翻译加油


Listing user tags for VM is returning same tag multiple times. It should return a tag only once. Automating this issue to check same user tag is not listed multiple times


__str__ is not a function. Remember what we are writing are NOT functions. They are member functions(theirs a difference) or methods. __str__ is used for stringification. Essentially it returns what would be shown if you were to print your object. How do you print a Dog object? I__str__ tells python the format we want the object to be printed, if the programmer wants to pint the object. Otherwise you'll get a generic print like <object at {insert some position in RAM here}> . But remember __str__ does NOT print, it simply RETURNS the string representation


Notice: Undefined variable: trackers in /var/www/torrent/include/credentials.php on line 27


done.


Good, @SimonWoolf over to you ;)


interesting


c


@KaramJez format je code ln 116 is niet geformateerd.


You have to check if the creature is still in the graveyard.
If, after the last ability triggers, the creature card is removed from your graveyard in response, it won't be returned to your hand.


I'm sorry, I have no idea. Don't even know the Harmony remote ;) Maybe @Memphiz knows why Darwin has these special settings?


remove whitespace


Dont use such notation. Its ok to write Position.X for example


Friend grateful for modification.

So it is not feasible. You use the data attributes for isserir an element in the sun, this is unnecessary since you could use the jquery method directly.

example:

$ ('#element").append('<input type ="file"name ="myfile">');
$ ('input[name="myfile"]').filestyle({buttonText: 'my button'});


@arnaudgelas @zarquon42b
I am in the process of testing it on debian testing and ubuntu 14.04.
With debian testing, there are some strange errors during compilation - but I need some more time to figure out what exactly goes wrong. Could also be my system.


我能看见


Saw this while making the changes. Is this foreach really needed?


Its all a new method


Can you open an issue an issue and provide logs?
Thanks


Why can't you use simply PutIntoGraveFromBattlefieldAllTriggeredAbility class?


This constructor has a lot of creation operation. You need extract methods and make some refactoring


Okay so. I removed the current implementation of AttribPersistence.Outer to make way for the thing I have in mind for the rest. Basically what I most likely need is a way for every RantAction to know their parent RantAction if any and the RantActions before and after them. Messing around with the BlockManager gets really complicated and I cannot figure out a sane way to make the persistences work consistently.


Sure - I was advocating a more descriptive name rather than a change of semantics. Not super important though, it is clearly documented...


Static?


Problem is that futility_margin() is perfectly tuned with Gains. You need to retune it without Gains to make a fair comparison. Just removing Gains like that is unlikely to work. Well, let's wait and see...


Instead of listing the different variants could you just add a generic comment why and under which circumstances the binaries get a suffix. Something like "gem adds the ruby version as suffix to the binary on openSUSE 13.1 and later"?


Dear
I think that
foreach ($sqlServer in $xmlfile.Configuration.MSSQLMetics)
Should be
foreach ($sqlServer in $xmlfile.Configuration.MSSQLMetics.sqlServer)


Dont send to CVS files, which are generated.


Ok so I'm going to be annoying, but I usually think that tests needs to be very dumb (readable).

Here, the timestamp (in millisecond) is converted to seconds, and you add what I think is 2 hours (120). How about writing it like:

```php
$oldTimestampInSeconds = self::$timestampOlder * 1000
$twoHours = 120 * 60 * 1000
// and you can then write $oldTimestampInSeconds + $twoHours
```

It avoids using my valuable brain cells :smile:.


Unfortunately, this still copies a buffer.  You want to use `ByteList#unsafeBytes` instead.


Very Good.


I have a large refactor incoming for Tile and Item initialization, including an Attributes system, etc. Nice rendering system though, I'll merge these into my changes! :D


Need to check the tile entities as well


IIRC got an error before which is why I added the cast


това изглежда добре


whats the error you get?


$botttom... ?


Rather than finding the column by number, you can just access it by name:

```chambers = data.map { |row| row['chamber'] }.uniq```


It doesn't seem right, no.  I was checking line-heights for the other sizes and we're going with +12px for line height.

For example, the veryLarge font-size is 24px and line-height is 36px.


@alex-ab Thanks for the explanation of the intended behavior and how the change breaks the semantics. I did not consider that issue when making the changes. Maybe your explanation could be added as a comment for future reference so nobody else trips over it?
It is probably best to simply drop this commit from staging.


comment ^^


The convention for these is to describe what you're testing (in a human-readable manner that forms a sentence). So here, for example:

```
  it "should have two chambers for Australia" do
```


Fixed in 8a910ec


no sorry, deployed plugins/artifacts can not be removed - as far as I know


So this should be 30?

/cc @joaojusto


Yep. I think it's pretty common in PHP (and also JS), because it's fastest.


Not sure if this is working correctly but when viewing the homepage I see _4 Articles_ listed here which doesn't tally with the amount of articles actually displayed on the page.

![screen shot 2015-05-11 at 12 49 35](https://cloud.githubusercontent.com/assets/941843/7564220/6686373a-f7dc-11e4-9bc8-2445091503b5.png)


Is this supposed to be 30px / 18px then?


Before my time - @davilla does Mr. Glodfish remember?


Hi,
Is it possible to add additional aggressive strategies to DE like "current-to-best/1/bin"?
http://www.dii.unipd.it/~alotto/didattica/corsi/Elettrotecnica%20computazionale/DE.pdf
BR, Pavel


I think it would be okay to return a sequence with 1 token. There are at least three reasons that come to mind: 1) consistency (0 tokens would probably produce Nil, 2 tokens would probably produce a 2-element sequence and 1 token is sort of an odd one), 2) the usecase in Mathieu's work, 3) the name `toks`, which hints at the fact that the result is a collection.


@friedelwolff Looked in Virtaal current code and I couldn't find any place where these changes can be problematic. Maybe you can point me to some place where there is a problem.

And I haven't checked in previous versions of Virtaal. Despite there might problems with those we can just tell people to downgrade to TTK 1.12.

Maybe it is a good idea to think on releasing using pinned versions to avoid having to constantly wonder if some change breaks backward compatibility with any software that might or might not be using TTK. I am talking about Virtaal and Pootle.


No it is finally fixed in https://github.com/mixxxdj/mixxx/commit/524edeea1e1959c7a071839e5d83f0d9b5121d82


That's a valid point. It's definitely not instantly obvious that it's supposed to be 2 hours.


why do you need the cast ?


Wir verwenden den Stil bei allen Modules, und zwar aus dem Grund weil das Angular Team den Vorschlag gegeben hat, um zukünftige Upgrades auf 2.0 zu erleichtern. Die Anweisung lautet: "Klassen und Directives" verwenden.

Getestet hatte ich nur das speichern, was vorher nicht funktionierte. Ansonsten muss man da eh noch einmal ran.


Rem: add the listener


For this kind of ability you can overwrite the adjustCosts methos.
Look at source of Necropolis Fiend as an example.


knewstuff does depends on attica-qt5; KF5Attica (required version >= 5.10.0)

kdeclarative; KF5Package (required version >= 5.10.0)


OK, I'll implement that.


It seems to me that it'd be useful to define liftables that would automatically call `tks` on stuff that you unquote.


"of building a project", "First one" not "in first one", "regardless whether they are stored",


excess string


Could you please change this so that it tries both paths? So if `AppDomain.CurrentDomain.RelativeSearchPath` is set try that path first and if the file is not found there, try the original algorithm. For ASP.NET the old algorithm will always fail because it shadow copies the DLL's, but it is possible that `RelativeSearchPath` is set in other circumstances so I would like to leave the old behavior in as a fallback.


Since C++ version 11 it's allowed to use double >> at template declarations.

But you're right, I've made the change for compatibility with older compiler versions.


extra string


good spot, sync'd the two in the readme https://github.com/novoda/download-manager/commit/88825823fc6ea5b43d1d49ba6aabd9ae4c7be6cc#diff-04c6e90faac2675aa89e2176d2eec7d8


AFAIK, there will be no difference but I think doing that in checkModeChange would have been better to allow 2 consecutive calls asking for a mode change (even if it is not done AFAIK).
But that can be done later (or not at all).


Seems better to me ^^


Good cleanup.


SF


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


这两行为什么放前面呢？


`var packages` is not used on this file.


Cool. I was also thinking of doing that ^^


Thanks mate

On Monday, May 11, 2015, Sebastian <notifications@github.com> wrote:

> Since C++ version 11 it's allowed to use double >> at template
> declarations.
>
> But you're right, I've made the change for compatibility with older
> compiler versions.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SebGDev/zxing-cpp/commit/103489879a2277021cb32dcf8e2602541be2fe40#commitcomment-11131990>
> .
>


@jlucerond done with my review. did you want to try and implement the pinch to resize? we can cover that on a call if you like.


Lol copying my ideas all over:D


I also prefer this name


This is the purpose of `m.nextWord`.


I suppose it should be `payload = { packages: packages };` ?


Hum. I understand the idea, but performance is not really the goal here.

I really enjoy the explicit casting `(int)` more than ` + 0` or `* 1`. During the old days where `intval` was the only alternative, I didn't mind much, but now casting is in the language and the old way feels hack-ish.

Also, it all comes down to "Did he write this code to cast it or was it a typo where he wanted to use `* 10` or something?". When you use `(int)` I have no doubt that you just wanted to cast it :).


OK, https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent defines it as CLOSE_NORMAL, but sockjs just says 1000 in their source.  But also, they reserve 4000-4999 for application defined.  So which way do you want to go? Use 1000 because that's the MDN defined CLOSE_NORMAL? We could define it here since sockjs doesn't yet. Or make up our own? var OCTOPRINT_CLOSE_NORMAL = 4000


Det här behöver någon skriva om, jag har försökt men jag blir bara sjukt förvirrade om vad som är metod och resultat!


Забыл добавить в список обновляемых файлов. :(
Добавил в обновление 1.2.5.2


/ruby-2.2.1/gems/minitest-utils-0.1.2/lib/minitest/utils/reporter.rb:93:in `find_test_file': undefined method `gsub' for nil:NilClass (NoMethodError)


Hi Darius - this change is not working as
  Problem 1
    - Installation request for darkaonline/l5-swagger ~2.0 -> satisfiable by darkaonline/l5-swagger[v2.0].
    - darkaonline/l5-swagger v2.0 requires zircote/swagger-php 2.*@dev -> no matching package found.

Regards


Why would you make an abstract virtual function in a superclass, but not override the function in the sub classes? There's no way that would even compile!


yes :+1:


To be consistent "remove" should be capitalized "Remove ..." at the beginning of the sentence.


extra string


I'm not near my dev-box to check if it's done somewhere outside this class, but shouldn't SendDarksendDenominate call UnlockCoins(); before it returns at https://github.com/UdjinM6/dash/blob/7a6d8605f06bef123059e4ad8fee76969310d280/src/darksend.cpp#L1189 ?


x


There's no issues that we're sending an address with 2 lines in a single field?


Hi, thank you for your PostFinance adaptation.
But can you tell me why you deleted all the alias related files ?


@zamith I had to make this commit to fix sandbox mode. I skipped a Pull Request because I needed this to keep working, but fell free to review and suggest any changes


Hi @jenkoian , can you explain this to me? I wrote the code on a windows machine and am sort of familiar with *nix systems but I don't know enough to understand what the purpose of this change is?

Thank you so much for your contribution :)


I think `R.home("etc/curl-ca-bundle.crt")` is a slightly nicer way of expressing the same thing


You are right, either way works on Windows, but I always forget to switch it because with `/` on windows I lose my autocomplete. This change is fine by me. :)


Oh, right, thanks!


This commit, as well as the commit message was generated automatically by ReproPhylo


Need check if self._widget is None?


@Diego-F-Aguirre I can explain, I think, why this code was causing the app to crash. it's an issue with the checkpoint details.


I meant it's the fastest to type :)

No problem, I'll add explicit casts everywhere.


Good point. I will try that out soon. :+1:


The binary (`bin/featurebrowser`) didn't have executable permissions, so was impossible to run the featurebrowser binary. If you notice, it was set to 644, or rw-r-r, so I just updated that to be 755 or rwx-rx-rx.

The composer.json change is just following advice from https://getcomposer.org/doc/articles/vendor-binaries.md


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Done


As a nitpick to start the code review, I think that consistent indentation (2 spaces per level) would help :)


Looks good to me.


`Lit.Unit` should go to `()`, not to `Unit`


I think I like the idea with `(o, o)` :)


You're right, and the problem here is... Fog doesn't have a way for you to look at the extra_specs of a flavor, so that'll have to be added.


The new comment says "all devices", but fixup_devices is still limited to devices belonging to destroy actions on devices.


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


Why not get API [search.customelements.io](http://search.customelements.io/)


It is more robust to configure this as a directory of JARs, so we don't need to edit this when the version number changes.

e.g.

```
<jarDirectory url="file://$PROJECT_DIR$/../../build/deps/junit" recursive="false" />
```


what's the dash.exe?


right, thanks!


Well, think this won't work. All calls to Oauth2 functions must go to www.plenigo.com.


 使用css3制作的一个平滑过渡的网页

主要使用了以下HTML和css3的知识点：
1.使用radio按钮控制底部button的切换；
2.transform属性控制变换，transition控制过渡效果以及animation帧动画；
3.属性选择器，伪类以及web字体的使用。


x and y stands for the cartesian coordinates, taking the bottom right corner as 0,0


I don't see it, did I miss a commit of yours?


The rest is more of less fine so far, except for the comment about liftables that I wrote above.


Yep likely, but this is a base comparison to see how far we get with this aggressive pruning. I will next add some additional margin based on the average gain used and possible tuning later.


Also, btw `indent` is spelled with two "n"s.


1) Indentation => I will check that soon, no problem. Intelij will do that for me (once I stop using Sublime)!
2) Oups. Will be corrected in my next commit.
3) Instead of finding names, I thought that a picture would be more explicit :p

4) True for liftables, I will do that.

Thanks!


:thumbsup:


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


@cheerfulstoic is there a reason you made this change?

match[1] will match the first grouping in the regex.  since the '/' is outside of the group, it will not get matched.

This change is causing URLs like the following to fail (and thus Google indexing isn't happening):
http://graphgist.neo4j.com/?_escaped_fragment_=/gists/all

Thanks!


+import javax.swing.BoxLayout;
+import javax.swing.JLabel;
+import javax.swing.JPanel;

These are the worst libraries ever written in Java, use others instead. Please fix, I've waited so long for this amazing product.


@teorikraften


Thanks for the explanation. Another case where we also don't need to unsquish is when the formatter returns an empty string, but that case is hard to detect since we don't actually look at the strings until in the renderUnsquish. So we'll pull your contribution.


コメントします。


"In the second one the"


Faltou alterar o "private long idAluno;" para "private Long idAluno;"
Endentar corretamente o código, parece uma escada. ;)
Deve usar nomes de atributos com letras minusculas no inicio. Ex: ra.


Looks good to me.


No issue at all, providing characters passing from two address lines should not exceed Number of character limit for single line address present for salesforce.


"a product of the compilation"


:+1: for consistency


"of building a project"


pluginWindow can still be 0 here, you should first check for it again before calling componentMovedOrResized()


"solve a problem"


This module would probably makes more sense as part of decor than, delite. Also is that strictly identical to liaison code or did you have to make any change?


Thanks!


liaison->decor (if we move it to decor)


- Métodos implementados de interfaces não podem ser privados.
- Nomes de métodos devem possuir ações, lembra do padrão: salvaR, cancelaR, procuraR, fazeR, etc.
- Você vai procurar a pessoal apenas pelo ID? Acho melhor procurar também pelo nome e pelo CPF, as vezes você tem uma ou outra informação no momento.


"for example when macros ..."


"of depending" seems unclear - there should be a bit more description maybe ?


- Faltou ajustar o enum, crie um Enum com o nome "Sexo" junto com as classes de Domain, e dentro dele coloque os valores M e F. Depois só declarar no lugar de "private enum sexo{M,F};" o valor "private Sexo sexo".


@al-martinez these two methods were not compiling because the closing curly bracket } was misplaced for the first method `- (IBAction)textFieldDidChange:(UITextField *)sender {`


Acho que podemos melhorar isso ainda, adicione um comportamento semelhante ao adicionarComentario para adicionar o ProfessorCurso.


How about this instead?

> Your current PostCSS version is *runtimeVersion*, but *pluginName* uses *pluginVersion*. Perhaps this is the source of the error below.


Seems to me somewhere here something got broken.

```
julia> using Docile

julia> Docile.Cache.getraw(Docile)
Docile: updating package list…
Docile: caching 'Docile'…
 - Docile            ✓
 - Docile.Legacy     ✓
 - Docile.Cache      ✓
 - Docile.Formats    ✓
 - Docile.Collector  ✓
 - Docile.Interface  ✓
 - Docile.Extensions ✓
 - Docile.Runner     ✓
 - Docile.Utilities  ✓
ObjectIdDict with 0 entries

julia>

```


I finally rechange it to char path[PATH_MAX];


I find the uncommented code a bit confusing. Please document how the actually used condition works instead.


"There is a defined .." - that is a bit unclear too - reorder of the sentence would be good


@mikeloz2015 you can actually combine these two into one line (fewer lines is almost always best)

`return number + 1;`


+1 for Python 2.6.x

I vote to remove this once SLE11 is dropped : )


@mikeloz2015 these can be combined too.

`return number1 + number2;`


Not well configured


@mikeloz2015 less code = less potential bugs = big win

`return dividend % divisor;`


@mikeloz2015 done with my review. press on.


Yeap, it's shorter. Feel free to make a commit.


Enabling instead of setting


You missed a reference in the pom.


... with respect to the scopes is turned on ...


you want to edit the [source](https://github.com/processing/p5.js/blob/master/src/color/creating_reading.js#L293)
not the build library


Changing the setting for existing project


Refix


woops, yes, good catch, thanks @spasmsaps


isn't apple_id not yet defined here ?


this function can be found here: https://github.com/processing/p5.js/blob/master/src/color/creating_reading.js


Pinging @garthwebb - this test started failing again


As for the indentation problem, let's discuss it tomorrow. I see no immediate solution, but maybe we'll come up with something after an offline chat.


If the project already exists and you would like to get advantage of the ..... you need to update the configuration in the following way


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


Thank you for noticing this.
I am going to make your changes directly to the website.  The repository is in shabby condition so I think it might be awhile before I can deal with pulls, not the swiftest with git.  What is on the website and in the repository are a little out of sync.  Gotten lazy.

That said, wow I messed the little line thingy.   Also noticed that the legal links on the right side have underlining too.  So I took a look at legal-links class and well see below.

    .legal-links {
      color: blue;
      font-family: sans-serif;
      font-size:12px;
                                       <-  looks like something is missing
    }
Your .no-underline class can be used to remove the inline style of the following too.

        © 2015 ebobtron, et al.
 brought to you by me.

Thanks Toby


Thank you so much.


@b-alidra Good question! It was a long time ago but I think it was only because I didn't need them and/or PostFinance didn't need them, can't really remember sorry...


@mcostalba: I'm not trying to create artificial conditions where the patch will fail. I want to know if this patch is a regression, neutral, or an elo gain.
This depth >= 12 pre-condition is making it impossible to measure. Aside from the trivial case of IID, this patch only kicks in at **depth >= 14**: first non-PV node is the second child of the root node, which will get reduced by `r=1` ply at depth 14, so `14-1-r=12`.
I'm not saying we never reach depth >= 14 at LTC; we do, but mostly in the endgame, while this patch is supposed to help with mate detection in the middle-game...

For me this patch should demonstrate an elo gain. It's not a simplification, and it can really choke the search.


이건 왜 바꿨음? string 안에 변수 없으면 ' ' 쓰는게 권장되는거 아니었나?


I think we should do our best to factorize as much code as possible and so to limit the `if (array) {} else {}` parts to the minimum. Here for example it could certainly be reduce to the tracking section as processQueryresult.call & processCollection calls are pretty similar.


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


Hello,

Thanks for feedback and sorry for my late answer.
Actually the solution that we've made is here: https://github.com/PrestaShop/PrestaShop/commit/b2a0d3db

Best regards.


@b-alidra Feel free to make a PR to re-add them if needed from your side...


It's possible to get this mostly working with Transitions...I wouldn't recommend it though. I spent some time creating transitions for the circular progress indicator but abandoned that effort in favor of animations.


I just moved ```find_package( VTK REQUIRED COMPONENTS ${required_vtk_modules} )``` into the vtk6 specific part: 852591d3aa1542baf247d464638ed68734c81f23


sup.call() being called in the two branches it should be placed outside. And the if should probably have a single branch.


this should probably be extended to getNextWods() returning a list of pairs of words with probabilities (and it is getting complicated again...)


еще для ссылки добавь main-nav__link


@marcelluethi I will build the deb-package using pbuilder and debian testing (with tests and examples enabled) and check if there is an issue.


Follower에 authenticate_user와 authorize!가 없는 건 의도된 건가?


здесь тоже для ссылок класс


Yeah you're right :+1:. Actually it's working because I'm assigning the **id** in the object `p.id = id++;`, but definitely should be like your suggestion, good catch!


We should just delete this function if we don't need it.  If we do need it in the future we can look at a previous commit where it existed and pull it from there.


@sinhrks Just to let you know, but the reason that this docstring has to be a raw string (the `r` you removed) is because there are newline characters in the example that have to be displayed (and not interpreted as a newline, because this gives errors)


а модификаторы для каждой --twitter --facebook --vk


Hum, but Class should have its `name` field set, I'll have a look.


Previously I got"

```julia
julia> require("Sandbox.jl")

julia> import Docile: Formats, Cache, Utilities

julia> Docile.Cache.getraw(Sandbox)
Docile: updating package list…
Docile: caching 'Sandbox'…
 - Sandbox ✓
ObjectIdDict with 2 entries:
  example2() => "example2: One section meta: !!section(Lexicon.md/Exported/Meth…
  example1() => "example1: No meta-syntax."

julia>
```


I updated the VC project file in next commit ; DX12_SUPPORT should be defined (it wasn't before)


maybe it is better to test against the presence of the idProperty here? if (store.idProperty) ?


Тут вот независимый блок figure.gallery-item в нем картинка gallery-item__image
описание figurecaption с классом и кнопка удаления


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


да нет, это просто information


Looks good to me


это information-item


isn't kernel32.dll supposed to be a 32 bits only DLL ? doesn't this break 64bits builds ?


`return unless self.class.try(:logger)`


How is iterateFrames involved in the body of the method here?


by doing that is seems to me we totally ignore the work done just above on tmp. Maybe this is ok, but then I would need at the very least a comment explaining why.


add `[AB_testing]` to the msg, will be easier to search for


looks :ok:


Indentation's missing here.


There's no ActiveSupport here, `try` is not available as far as I know.


@chelmuth, @nfeske: please drop commit 2d045142c1cc67ac5eab95fff92616d376f7df20 and add this one


Ben says those methods are going to go soon. So it's ok


to staging


Changed in https://github.com/Duhemm/parsermacros/commit/fadb4c5dca0abb3f5e34efd2d5b9dff5af6058b8


Note: if your distribution uses wine-staging as the DEFAULT wine installation, change the binary variables to match that of your distribution.


looks like it might be time to add a getIdentify method to delite/Store and move that code there to isolate the dstore vs ObservableArray as much as possible in delite/Store itself.


This is broken, there is no last() for arrays in standard javascript.


Apparently, I need an access to https://www.gitbook.com/@feelpp in order to write a part of the book, isn't it ?


We obviously need to check this is not conflicting with other classes extending delite/Store and redefining (differently?) getIdentity (ComboBox, TreeMap etc...).


Not bad! Although it looks like the 0th element in the array will get used twice if you don't pass an argument.


This should be done unconditionally on any possible exit, at_exit hooks will not run with `Kernel#exit!` for instance.


And it would still crash, since in case of NullTest self.class is not nil, it just doesn't have logger method.


But this is a shim, so it's fine, might just be worth a comment.


Good point :+1:


@tvsonar looks like this is a leftover from another commit


You actually seem to enable the fixed height here, not commenting it out?


Impressive turnaround speed :)


Also, I'm not sure if VK_MENU is the key used for Left + Right Alt keys, with the old system I had to use a combination of VK_MENU, VK_LMENU and VK_RMENU keys to get alt to work


can you pleae comment on why this.attached needs to be tested now and why it was not needed before and conversely for store. You might have already told me, but depending on the explanation which I don't remember we might want to add it as a comment in the code.


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


I recommend https://github.com/ai/autoprefixer-rails for handling CSS properties with prefixes.


If you get youre repo sorted out at some stage let me know on slack and I can rebase this commit for a merge :+1:


can you please help me with sample code?..


Thanks for the fix! I am working way to little in Javascript.


It does not break the build. But it might fail to set the Unicode font on 64 bit systems.
I will test it once I have access to a 64 bit windows.  Or can you check it?
Just watch out if "Consolas" is selected in console preferences after Mixxx is started


Here is the build: http://downloads.mixxx.org/builds/1.12/release/mixxx-1.12.0-beta1-1.12-git5435-release-x64.exe


It's just a visual indication to make it easier to see where each file import starts. If you happen to have lots of invalid data in multiple CSV files, I think adding a visual cue for the beginning of each import makes it easier to see which file the errors correspond to. I think adding a new line in between each import would also help.


Would expect an assert for floatie as well


Не совсем хорошее решение.
Вот пример как надо было сделать:
https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


Review by @nicolasstucki for pushing on master.


The maven plugin for MyBatis generator adds the generated source directory to the compile path programmatically.  We don't want to add it explicitly because part of the test is to make sure that it works automatically per Maven expectations.

For eclipse, simply add generated-sources/mybatis-generator as a source folder to the test three projects after you import the projects into eclipse.  Works like a charm.  I have less than 10 warnings in my eclipse workspace related to the generator and no errors.


*.pyc should be in global gitignore file on your machine. Please, undo changes in .gitignore file of this repository.


What are the original arguments here? "no argument" ?


Папки bin, .metadata, test-output не заливаем в рпеозиторий


Not really, those strings would have to be added manually, or maybe we add a general gettext stub file that simply calls all strings that are dynamically used (so the translations can pick them up). Individual strings will have to be added manually like so:

gettext('string1'); gettext('string2');

I believe it would even work like this so it is never evaluated and then it can be embedded into files or functions:

if (0) { gettext('somestring'); }

Thoughts? :)


done.


At the company for which I originally wrote the tool, we actually use it on our production server. After every release, we regenerate the docs, which are mostly for our internal users, but it's all on our production server. I think it's up to the user to decide if they are going to use in production or dev, because if they only have on their dev environment, they have to then get the final docs to somewhere the business/end user can see them right? The goal of the tool is to create the .html documentation for either end users or product owners, to work as a user manual really. Not so much for the developers, we can always open and read the .feature files ;)

Let me know what you think :smile:


All `pubDates` were updated? Oo.


Please extract the common URI ```lilypondUri.resolve("../share/lilypond/")``` in line 60 and use it here. Also, extract the ```ly``` directory name.


Для генерации дат (чисел, месяцев и года) можно было б использовать классы java - Date, Calendar. И генерировать все 3 параметра одним методом.


params_not_allowed.push(key) unless formatted_allowed_fields.include?(key.to_sym) might make this a little more backwards compatible. if not using attributes then post requests fail due to automatically adding to the params_not_allowed.


AutorizationService.isAutorized()


все верно


I don't particularly agree with changing the correctly spelled 'grey' to 'gray'. I prefer real English to an Americanism, but hey....


First test with stb_truetype are ... weird ... I guess I'll have to contribute once my engine is done.


Fixes: "W/DirectVolume(  234): Deprecated implied prefix pattern detected" @airlessproject.


It doesn't compile without it.


En GenericName dice "Viosr" esto influye en algo?


Реализация не самая верная. Вот пример: https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


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


I thought about it as well - but after push. I'll change it to be after the mode is created.


Все это сравнение нужно сделать в классе ContactData в методе equals.


but no *comma* here ;)


Everything else was named this way too (i.e. "color"), so it only makes sense to keep it uniform across the API.


With USE_WIN32 you choose utf8 encoding, while for regular path names you use UCS2.
Is there a reason?


`with` repeated twice.


I found some syntax errors in this associated with Private Key, Publish Key, SellerID missing " " and missing ' ' .  I set everything up, seemed to operate fine, but end up with cURL call fail as error return. Maybe something in the /lib/ is broken cause everything in index.html and payment.php is correct.


I wonder why the table would be in a different order than id? Maybe I sorted it by accident?


`with` missing.


Same.


lgtm


Ок. Верный метод.


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


Много пустого места оставляете.


Все ок.


OK. Agreed


Yes - perhaps I should include some actual arguments.


Bitch add me permission motherfucker


Well I suppose :S  Since the languages are all american created. Eg. System.Drawing.Color


Папочки bin, test-output не заливаем в репозиторий.


Eclipse annotation processors config.
This is the only Eclipse config file checked in currently because it's annoying to do it ourselves and it does not find it from Maven config in my experience.


Maybe we should add a `Truffle::Primitive.at_exit_always`.


+This group has two members
+lpethi02 : Lalin Pethiyagoda
+&
+jdocke01 : James Dockeray


Guess you can drop the `map`. :-)


Strange since MenuModeEditor is an AbstractApplicationMode. Well, I guess it is a strange unexpected c++ behaviour ^^


@feiyuw I think there are others files that still is importing from psutil.error. I could not run here.


@dh10 It's great to hear you're considering to use VarDict as your defacto variant caller in amplicon sequencing data. May I ask what sort of projects you're running and if there might be any collaboration opportunities in oncology?


Ah should have clarified - I mean if you have gotten a binding using something like `Kernel#binding` - that's where the problem is. A binding from a `Proc` might be ok.


I am a nice comment


We have /privacy/ which reads and converts markdown from the current privacy.md on the repo. I think this is obsolete


удалить старый код то, происходит 2 раза печать


TeamCity Ybackend :: synteny/ybackend [Build 25](http://192.168.30.225:8111/viewLog.html?buildId=1215&buildTypeId=Ybackend_SyntenyYbackend&guest=1) is now running


is this correct:
from => to
to => from
?


реализация неверная
вот пример:
https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


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


Anyway: I send a PR for a cleaner Test which works on a previous version.

I will use the previous commit to continues on Lexicon as I really hope to make some good procress this week.


Agreed. I think prismatic schema's default exception should be enough


@tvsonar should be a `Set` in order to provide a fast `Contains()` operation


Why can't it be left as `final val binaryEmitted = current` when `final val current = "0.6.3"`?


>  The goal of the tool is to create the .html documentation for either end users or product owners

Fair point. However, in that case, what I would do, is still use it in `require-dev`, and as part of my CI after successfully passing the tests, I would generate the html files & package / deploy them for the end user / product owner.

What I would suggest is, aim for the best scenario, and some may do it the easier or less ideal way, but I think this is better than the other way around.

Its your project, you are the boss :wink:


Yes, group territory should return `survey_type` of `reserved`


where did u get `gralloc.montblanc.so` & `hwcomposer.montblanc.so`? why do u change it?


:D


Dlaczego rejestrujemy event z wykonaniem funkcji w której go rejestrujemy. Nie ma to kompletnie sensu.


ok, just tested, "consolas" is selected as console font with this build under windows seven ultimate edition SP1 64bits.
Either with console window openning when launching Mixxx with shortcut and the console used to launch Mixxx manually with .\mixxx.exe

I worried for nothing, sorry.


Because very soon (in the next commit), `current` will become `"0.6.4-SNAPSHOT"`. We've come to the decision that we would change this field (and the next) on the version commit, rather than on the next one.


working on it :)


ok


Czy tej części nie wykonujemy już w `placeMarker`?


LGTM


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


@rvansa thanks for the explanation. This is exactly the type of context that I think is helpful to put into the description of the PR to show the justification for the change.


But i cant see and i cant sit chair and toilet :D


Point taken. You win this round of the documentation war!


The rationale for that decision is: as of the version commit, the version number of the IR format is fixed and stable. It's the version commit that makes that true, not the "Towards 0.6.4." commit that comes after. Going towards the next release is not what gives a stable IR version; the version commit does.


Well, if gettext delivers the tools to grep the gettext() calls, I could certainly create a script that just creates one file (or a bunch of files) containing those calls for all translatable text in our xml and json files.
I will come up with a proposal as soon as possible, one suggestion, if possible I would like to create those dummy files into a separate directory because they will have no real function. (and we could choose to either leave them out of git)


Should the model be set with th appropriate namespace if a developer runs `artisan app:name`?


After a gaffer load

Traceback (most recent call last):
  File "/Library/Python/2.7/site-packages/gaffer/process.py", line 710, in _init_cpustats
    except psutil.error.NoSuchProcess:
AttributeError: 'ModuleWrapper' object has no attribute 'error'


Nightly: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-nightly/396/
Weekly: https://scala-webapps.epfl.ch/jenkins/view/Scala.js/job/scalajs-weekly/41/


Thanks. :+1:


TeamCity Ybackend :: synteny/ybackend [Build 25](http://192.168.30.225:8111/viewLog.html?buildId=1215&buildTypeId=Ybackend_SyntenyYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:10:25


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


It is indeed.


Sounds good. `make translate' will pick up any gettext() call in src/ by default so we don't miss anything. Let's see if you can find a place to hide them or if the workflow works as intented. ;)


@jaydson that's because Harmonic uses the host's timezone, see https://github.com/JSRocksHQ/harmonic/issues/84 and https://github.com/JSRocksHQ/harmonic/issues/148


@rvansa thanks for the example, but I'd still like to see a new or updated benchmark included with this PR. In lieu of a features page or documentation, working examples are very helpful!


It was a regression. Do we have a test for that, e.g. a screenshot?


I would go further and say that the config should be passed into the send-event and flush-buffer functions.
The config strikes me as a state about the connection being used (i.e url, client-id), it could even contain connection stats (e.g successfully sent messages).
Essentially there would be a default config defined in the namespace, that would be merged via a function that returns a map/record representing the samsara connection.
This map/record is then passed into send-event and flush-buffer functions, and used. These functions could even add/update certain stats within this map/record and return the updated map/record


Hi, Darek,
 zircote/swagger-php 2.* is not stable so you have to add
```php
"minimum-stability": "dev"
```
to your composer.json file.

Or you can can add aditional line:
```php
"zircote/swagger-php" :"2.*@dev"
```
in you require section in composer.json file.


Hmm.. the other cases should probably also use Instants or some variation there of. Most of the others simply write the ISO-formatted string directly, while storing seconds and nanos will probably take less space.
Will have to look into that...


If this really bugs you, you could add the build helper in a profile that is only active inside eclipse - just make sure it isn't active by default.


alguem acabou com o arquivos


> function example1() end

doesn't get the right line/file info because it has an empty function body. See [above](https://github.com/MichaelHatherly/Docile.jl/commit/a1cfb5a50099aa2d1c547a7ddcb959d8424dd838#diff-daa2a78cfafe9a5434fa4a01fc390ec9R13) where I've added a `()` to the definitions. I think it's a know issue, but I've not had the time to scour the Julia issues list yet.


@mikemcquaid This commit introduces a change in behaviour on my machine where Ruby is in fact available, only via Rbenv. This used to work without having to install Ruby via a Brew formula.

Is this intended or did I just catch a bug? Thanks in advance <3


Hello,

pourrais-tu mettre ces deux logos sur le site avec les liens sur les sites ?
![logomaddyness](https://cloud.githubusercontent.com/assets/7852945/7565650/ee089558-f7f1-11e4-8989-7420f49bb3d4.jpg)
http://www.maddyness.com/

![startup-handbuch-sponsoring-logo](https://cloud.githubusercontent.com/assets/7852945/7565672/1ed67164-f7f2-11e4-8002-1a98ca20d9a1.jpg)


Should this have been changed?


Got it.


想问下这是干嘛用的。。？ 我也这么改了，好像没什么变化


Nice one @chriserin - Imma totally use this! :heart: :heart: :heart:


With this commit, we properly reboot on systemd.

In the current shutdown, we disable swap filesystems as well as other NBD connected devices, note that Systemd already takes care of disabling swap filesystems, so maybe we can also get rid of this here, I'm checking for extra NBD filesystems. Anyway, it is harmless to do extra work here, maybe we want to keep this for other init systemd, this commit only removes what was causing 'systemctl reboot' to fail.


We don't really have a choice, without this constant we cannot compute the
panel height. Why is it problematic here, when it seems that it doesn't
cause any issue with the other constants for portrait/landscape on tablet?

On Mon, May 11, 2015 at 2:52 PM, Herrie <notifications@github.com> wrote:

> You actually seem to enable the fixed height here, not commenting it out?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/webOS-ports/webos-keyboard/commit/826a755728054856b1fd22730033a704c14b8acc#commitcomment-11133100>
> .
>


only *when* relevant. ;)


TeamCity Ybackend :: AnnaPonomareva/ybackend [Build 304](http://192.168.30.225:8111/viewLog.html?buildId=1217&buildTypeId=Ybackend_AnnaPonomarevaYbackend&guest=1) outcome was **FAILURE**
Summary: Exit code 1 Build time: 00:09:14


屌神好有文采…


почти верно, вот пример https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


That historical, right? Maybe @Danimal696 knows more?


Это то не заливаем


@airlessproject This is the way fstabs _should_ be formatted since 5.0


@notspiff are you sure we don't need to free m_cfg somewhere?


I would vote for keeping it consistent.


sounds likely but there is no function to do so in kiss_fftr.h


I assume that this does not work with --root and also with template unit files.


Out of curiosity, why was this needed vs. putting this in the server block only?


This surely belongs in laravel/laravel and should be a "suggest" in here?


Misspelled "Canola"


The current 'clean-all' behavior of systemd is actually weird, because they advocate for the initrd to cleanup everything (as explained in #31), I don't get why we have this double-work done.


So what you need to do is either add an expression to the body of each function or change them to single line ones:

```jl
function example()
   ()
end
```
or
```jl
example() = ()
```


Because `MixinSubject` has multiple targets, the majority of which are perfectly valid for both sides, however the rcon one is not. This is basically one of the intended purposes of config plugins, they allow you to conditionally filter targets based on arbitrary criteria.


final


A minor tweak, but we could drop the ``sponsor.petition.try`` with `if: :petition` as an option (e.g. only run the validation if the sponsor has a petition).

```ruby
validates :email, exclusion: {
  in: -> (sponsor) { Array(sponsor.petition.creator_signature.email) },
  message: 'Petition creator cannot sponsor the petition',
  if: :petition
}
```

I think this would be marginally clearer (I'm not a huge fan of try).  It's up to you though, the solution is fine otherwise and we can merge away.


What does this do? It seems like it might be fragile if lines containing "`# alias`" are added to the default `.bashrc` that are not intended to be uncommented. (Also, the regex is not anchored to the start of the line.)


What is the best course of action in reverting the change to just LibLog.cs in my commit?


If you see in the later commit I tried to limit the dependency with the global state
and make re-bindable in the sense that now you are able to do something like

```Clojure
(binding [*config* my-second-config
               !buffer! my-second-buffer]
    (record-event {:eventName "any.event"}))
```
I used this also to separate the tests are most of the test were depending from the previous execution,
and I created  `with-empty-buffer` which simulate the execution of the block with a empty buffer.


just tried it out on 12.04, not sure it's just me or not....

1. pip install --upgrade pip # to address[0]
1. pip install --upgrade setuptools # to address[1],
1. pip --log install-deps/log.txt wheel -r ./requirements.txt -r ./test-requirements.txt 'distribute >= 0.7' 'pip >= 6.1' # and this command will download the wheels.

 i am not an expert of this. but seems our precise gitbuilder are working fine...

---
[0]
```
$ pip wheel -r ./requirements.txt -r ./test-requirements.txt 'setuptools' 'pip >= 6.1'
Usage: pip COMMAND [OPTIONS]

pip: error: No command by the name pip wheel
  (maybe you meant "pip help -r ./requirements.txt -r ./test-requirements.txt setuptools pip >= 6.1")
$ pip --version
pip 1.1 from /home/kchai/dev/ceph-ubuntu-12.04-kchai/install-deps/lib/python2.7/site-packages/pip-1.1-py2.7.egg (python 2.7)
```
[1]
```
$ pip --version
pip 6.1.1 from /home/kchai/dev/ceph-ubuntu-12.04-kchai/install-deps/local/lib/python2.7/site-packages (python 2.7)

$ pip --log install-deps/log.txt wheel -r ./requirements.txt -r ./test-requirements.txt 'distribute >= 0.7' 'pip >= 6.1'
The directory '/home/kchai/.cache/pip/log' or its parent directory is not owned by the current user and the debug log has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
[The directory '/home/kchai/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
ERROR: 'pip wheel' requires setuptools >= 0.8 for dist-info support. To fix this, run: pip install --upgrade setuptools```


Marketing!
I thought it would encourage people to report, and that in theory the backend should be so much more solid now that you killed gnonlin (shouldn't it?). I can take that part out if @thiblahute thinks the same way.


较之前感觉好了许多。


fixup_devices is all current devices plus all devices scheduled for removal.


codigo del semaforo a falta del rele


I'd rather not bring the mediainfo term "General" into this project. Can we use a term that is more representative of a file format or container as a whole, then alias it to 'General' within the policy checker code.


There's probably a more generic way of defining this, either by defining a new `tone-` class for the colour scheme, or using something less specific than `--legal`, but not a major thing.


Yeah, couldn't think of a better name... any ideas?


Вот пример как улучшить и вытащить больше информации из таблицы:
https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


From PV build. It doesn't matter. The ones from venor are the same.


Why not just use FactoryMuffin here? Seems to basically redo the same thing


Nice!


Because I changed the response names (which are just enums, AFAIU) above... you were using copy-pasted names (APPLY, OK) that had no relation with the actual buttons or behavior :)  also if you look at the if/elif section at the end, you can see that it could not even possibly ever work because it was checking twice for the same condition! ;)


This is for kernel's ramdisk, why would you put this on device tree?


@MathieuDuponchelle yeah well I kept this as a separate commit for clarity for the review. I'm fine with squashing this stuff if you think that's needed. This one here is more about the code cleanliness and logic, while the previous/initial commit was about the wording presented to the user.


Also note this funny line here that pointed to nothing :)


@chrisroos I've updated the commit message with more info about how I did that


A few points:

- Null move is not taboo.

- About eval > beta condition. The point of null move is to say "I'm
winning so clearly that I can do whatever I want, like skip a move". If
eval < beta, you are not winning, so there is no point in doing the null
move. First play the moves that achieve the winning position, then you can
start null moving like hell.

- The impact of your patch is not measurable with current time controls.
Let's assume that in some positions, your patch reduces the search depth by
40% after it kicks in (depth >= 12). Now at LTC where we reach depths like
16, we would in some positions have a reduced search depth by 1.5 ply. This
is not measurable because the impacted positions are so few. But when we
talk about TCEC like time controls where we reach depths like 30, we would
already be losing 7 plies in some positions. So in some positions SF would
suddenly starting to play really badly (compared to its normal strength).

- The only way forward is that the patch needs to prove itself in areas
where we can measure the impact, let's say depth > 5 * ONE_PLY.

- Either the patch needs to be an improvement, or it really needs to
improve some analysis capabilities (similarly to verification search) and
not cause a regression of more than 1 ELO at max. If it doesn't cause
regression with depth > 5 * ONE_PLY at LTC, then I'm confident that it
won't cause regression with tournament level time controls either after we
change the condition to > 12 * ONE_PLY.

On Mon, May 11, 2015 at 7:16 AM, Marco Costalba <notifications@github.com>
wrote:

> @zamar <https://github.com/zamar> Indeed it is not taboo to skip null
> search, for instance we do this 50% of cases due to eval >= beta, as usual
> here is a tradeoff between advantages of null search in some cases and
> higher probability to find something interesting. Maybe this patch can pass
> even SPRT[0, 4], but pelase do choose your testing condition and I will run
> it, IMO makes sense fixed games at VLTC (very long TC), but please you
> decide.
>
> @jhellis3 <https://github.com/jhellis3> Thanks for your support. Yes, it
> would be great if you can compare this patch against master and against
> your best version against a set of tactical positions. Indeed I'd like to
> ask to interested people to try this patch against a positions set and
> compare with master, so to verufy that this patch actually increase
> tactical strength.
>
> @cuddlestmonkey <https://github.com/cuddlestmonkey> Lucas idea is simply
> to push the patch to fail: I don't seen *any* reason you want to do this,
> normally you want to invest resources in make the patch to succeed not to
> fail.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mcostalba/Stockfish/commit/ab5ee7beda44512ef091f1660f1da9eab4a9b49c#commitcomment-11127428>
> .
>


The model shouldn't reference the DOM directly.  See StockView#updateShares in your w08/d03 version.


Nice work, but please don't use this repository as your private notebook, cloud or documentation. use own repositories, gists or your hard drive instead.
Just add elements to the repository, that are supposed to become a REAL part of the application ITSELF. Thank you.


Which part of the code are you referring to?


why? ..now it returns the cloned query $q! and now it doesn't work again


Thank you for the update!


Thanks for merging ;-)
Like you said, I thinks it would a good idea to set it as the default plugin in order to have the maximum number of testers.


Wow, thanks for the extensive testing :-)


I think it shouldn't require strict there


Will you PR this?


Oh is that what happened? I was just going to do a manual conversion.  I'll change it back.  I thought I had accidentally changed it to Text or something.


Yeah, it felt a bit weird but I was in "just get it basic-fixed, not refactor the whole thing" mode :)


How does the CCPM interactive button have anything todo with the secure icon in hub window, Even DC++ shows the hub cipher as tooltip from that icon.


There's a typo here (lowercase "t").

Also, a wording suggestion:

"Writing a test is very similar to writing a standard Python unittest, but has more builtin features and allows for more diverse tests."


Got it! The formatting somehow made me miss the ```devices``` part of the expression.


Good.


Not sure if 'for example' is here ok because a description encloses all cases of 'early compilation pull up' mechanism


Does this lxd ever get killed on test failure?


Yeah, I tried to create a structure similar to Forge. It's working well


Which term? I don't like "Container" because some formats have no "container" purpose (e.g. raw MP3, raw DV...) and I have no better idea.


Oh I didn't realize that. Handy :).


@notspiff mhhh maybe this define? https://github.com/Memphiz/xbmc/commit/981eae8417396338009b2e87391c3ba7e833e362#diff-72e5fdca2bd89f8f980579be536d7892R41


Are you using a mixture of single and double backquotes here on purpose?

Also, I believe we can add `:mod:` prefixes so that users are taken to the API docs an those modules if they click on it (cross reference).


I'm unsure how to test this code. Comments on style and correctness are greatly appreciated.
If you know how to spawn local workers and jobs to test locally, please point me in the right direction!


我发现想要把这个程序移植到linux上相当困难，kbhit函数在linux中没有，只能自己写。。。。


Sure.


Yes, the cleanup() method in tests/main.sh (line 83 ish) kills all the lxd's spawned from the test process.

I guess catastrophic failures would not cleanup, but that's the same for all other lxd daemons started by the tests.


Shouldn't policySets have 0, 1, or many policies? Not sure why policySet is within a policy.


ack


Lemme flip that and reverse it, set inside policy

On Mon, May 11, 2015 at 9:54 AM, Dave Rice <notifications@github.com> wrote:

> Shouldn't policySets have 0, 1, or many policies? Not sure why policySet
> is within a policy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MediaArea/MediaConch/commit/9274037143b407c1fd3878145de0d0360f00f0fc#commitcomment-11134255>
> .
>


but that won't free any of the members of the anon struct.


also could add the others: Image, Text, Other


another solution would be to overload the parse method...something like the following
public List<Integer> parse(String latestCommitMessage) and
public List<Integer> parse(String latestCommitMessage String reqPattern)

In the second method you can override the REQUIREMENT_PATTERN and use the user supplied one


Does this api work atm?


Yes. Just used it today.

On Monday, May 11, 2015, Ryan James <notifications@github.com> wrote:

> Does this api work atm?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/mgp25/Snap-API/commit/c2bef7ecc408f75d82f5927a14b27a4ed23f927a#commitcomment-11134304>
> .
>


@robertmaynard  This causes build failures with the ParaView used by CMB superbuild. Thus smtk/master should not be merged into smtk/release-v1 until the CMB superbuild ParaView is updated.


Yeah, I should of mentioned that I changed it earlier, my bad. I don't want it to break things on the data import, so whatever is easiest. I just noticed that detail while building the views and switched it quickly.


Oh, sorry, I didn't realized this code has changed. Looks like you alredy have LXD_MIGRATE_DIR in the rms, right? I'm fine with merging this as is if you confirm it's already there :)


I didn't pick up on this before when this interface was introduced - it is confusingly named. It's not the zone observer - it's the observed zone. `ZoneObserver` is an interface I would expect to have the notification callback methods defined on; this should be something like `ObservableZone` instead...


Oh right, I was looking at the wrong branch :) It's added already indeed.


There it is :)


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


Краще зробити окремий файл, в якому будуть кольори для всіх алгоритмів та можливо допоміжні кольори для дебагу проміжних етапів
@MariaPetlovana @AnatoliyS @derubioo


@nfeske: Please merge to staging


Why is doing this (creating an object with a single member, iterating over the members, and making the assingment that way) better than making the assignment directly?


Образцовое выполнение!


Write meticulously well maintained documentation. More commits.


HTML tests pass.


Untagged source map tests pass.


unchanged, no conflicts. Reack.


我把一个list当成一个命令了，因为第一种和第三种不会区分。。。


Can you please elaborate why you reverted this and what kind of issue you got, now if I build with debug as target the collector and the query engine don't start since the path is not good, Thx


but the flag is only "sr".


this looks really good!


Looks good.


Hi,

this doesn't seem to work on 2.17 yet?

Thanks for explaining


Не совсем верно.
Вот пример: https://gist.github.com/ilyubin/f7995f18a9adc1b0590b


Hi, this helps with non required fields, like choice filter fields in LexikFormFilterBundle. This is the good part.

But now, also really required choice fields (like those for not nullable database columns) are not validated anymore. If I submit a form leaving such a required field empty, I get a databse error about a missing non nullable column.

When I remove this fix, I get the expected form errors again.

So this fix fixes the problem, but introduces a new worse regression at the same time.

Greets,
spackmat


OK, using pbuilder it builds on testing.


@gonetz I updated my note3 to lollipop in order to get the latest OpenGL ES 3.0 driver (V@84.0) and most texture issues are now gone. Also using framebuffer emulation doesn't rotate the screen anymore but it's still buggy:

![screenshot_2015-05-11-16-03-48](https://cloud.githubusercontent.com/assets/2852068/7566505/6f9b8a94-f7f7-11e4-8271-29c826e131aa.png)


Якась чималенька функція виходить) Може її можна розбити на декілька логічних компонент?
@MariaPetlovana


@cflipse I also need to change it so that we use adapter `identifiers` rather than registered adapter names in the setup. I'm not sure why we decided to do it like that :)


Move inline CSS into custome.css


@cflipse ugh wait, it does work like that...sorry, just ignore me > - <


- -发现BUG，第二种就gg了。。。
再次求日哥提示TAT


@spackmat create an issue and reference to this commit, otherwise your issue won't possible be recognized.


poooorra! esses caras ai errando besterira... AIeiaejiaehwih valeu benja!


Just curious: why do we need a maven bit ?


Could have used `Config.getInstance().getCssBaseURL()` ;)


Move this code into helper method


@yumminhuang Do you want to log this skip in any way?


I think you might have forgotten to update the unitypackage after this commit. I removed the old VFW from my project, downloaded the zip, and unpacked the package into my project but my code isn't the same as yours.


"File Format" or simply "Format"? This would work for raw streams.


Це практично копіпаст алгоритму побудову розділяючого ланцюга з діаграми Вороного, я просто прибрала зайві елементи. (Взагалі, я була б не проти просто модифікувати діаграму Вороного так, щоб там зберігався мап найближчих сусідів; але з іншого боку, це ж у нас типу різні алгоритми мають бути).
А зменшити цей метод, мабуть, не можна, бо його основна маса - величезний цикл по многокутникам Вороного
@knucyb  @AnatoliyS  @derubioo


@ruda first, thanks for providing a dependency graph, it's really useful!

Now, the information that the graphic shows really made me panic. The reason is that it's impossible to clearly identify the dependency strategy we intend to give to Avocado, mainly the lack of circular dependencies. There's just too much information on the graph.

I believe one big improvement would be to remove the plugins from the graph, since they're are not intended to be consumed as APIs in any shape or form. Maybe just by doing this cleanup, we can rest assured that the dependency is sound.

What do you think?


Because in hamachi manifest i found these files are required
![screenshot from 2015-05-11 19 37 02](https://cloud.githubusercontent.com/assets/11899221/7566655/b8de962e-f815-11e4-971f-65d1965cf701.png)


Sounds great!
>I updated my note3 to lollipop
Is it official firmware?


I think Format is good, to imply things at the highest possible level but
remaining general (without using general).

On Mon, May 11, 2015 at 10:11 AM, Dave Rice <notifications@github.com>
wrote:

> "File Format" or simply "Format"? This would work for raw streams.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MediaArea/MediaConch/commit/9274037143b407c1fd3878145de0d0360f00f0fc#commitcomment-11134701>
> .
>


Yes.  In UCS2 each character occupies 16 bits.  This is what the WIN32 CreateFile function expects (when _UNICODE is in effect).  But as far as I can tell, OpenSSL is using fopen to open files, and fopen expects a char*.  So by using UTF-8 encoding (which is compatible with char*) it makes it possible for fopen to access Unicode path names.  Of course this assumes that the fopen being used knows about UTF-8, but if it doesn't at least ASCII path names will work.  The alternative, of using "LATIN-1" or "native" path encoding would also work for ASCII path names and would make another set of assumptions but would certainly not be able to access Unicode path names.  So it is better to assume the best case.


this should be a `throw` most likely


Original commits: https://github.com/OptiPop/build/commit/0fd0a8ada751d54eadcf91a2d536200d5562a05b
I'm see you :stuck_out_tongue:


Capitalise Friday


I am concerned, because you did not sign this commit.

Who are you and why are you tampering with @floriank's commits?


Oops. Fixed. Thanks.


He ppl have been complaining about that for ages

Isarray?obj:[obj] I guess
On May 11, 2015 1:01 AM, "Austin Brown" <notifications@github.com> wrote:

> Sick.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jsdelivr/api/commit/80692d850d4ea162dcd3be3df8f30538b2febee2#commitcomment-11127017>
> .
>


Este repositório será utilizado para fazermos os testes de utilização da ferramenta.


Zusätzlich wurde CHAMPS Partner eingeführt


I have faith in you Heliomance! <Insert cheesy thumbs up gif here.>


Yes, my suggestion was only for the 2nd sentence. The complete paragraph would be just like you commented.


I don't think this should be required. If no trackNumber then we should default, to either "first" or "all". Also the elements should define an 'all' concept. So if you have an MKV file that should have 16 tracks of 24 bit audio, you don't need 16 tests to check.


Trying to get on Commit Logs from Last Night?


I don't have time to look through the code this morning, but as long as no RantAction is a child to more than one other RantAction, adding a Parent property would probably not be that hard.

Not sure about adjacencies, though... For the RASequence, that would probably not be an issue. As for the others, I am not sure.


It worked :+1:


This should be in the same line


Update: I can read it, but not write it ??


With MediaInfo the list of possible reported values isn't known by MediaInfo. I think we can define a vocabulary for it, but shouldn't use a restriction list in the schema.


Alex:
Just checking to see if I can send you email from yahoo via github?Cozy1
      From: L0FKA <notifications@github.com>
 To: rscustom/rocksmith-custom-song-toolkit <rocksmith-custom-song-toolkit@noreply.github.com>
 Sent: Sunday, May 10, 2015 3:18 PM
 Subject: Re: [rocksmith-custom-song-toolkit] *Fixed PS3>PC converter (PS3>Xbox is still broken) (47ed436)

@fabianosan, just some odd pathname issues, like this one—
Reply to this email directly or view it on GitHub.


I agree that send -> updateReader is not straightforward. One way to think through it better is 1. send -> updateCommunicationTerm 2. updateCommunicationTerm -> updateReader


@Anahkiasen :+1: for Factory Muffin, lol. ;)


why don't you use relative links?


It's been a while since I've last used it, but I'll try. Iassume you are talking about the export.sh script? What doesn't work?


if operator="Exists" then value isn't needed, but value is minOccurrs=1, so ¯\_(ツ)_/¯


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.108](http://teamcity.monogame.net/viewLog.html?buildId=7155&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:18:47


It's also the term used by ffmpeg. ;)


Well let's try to stick to our standards and provide the correct patch for this please :)


Thanks for the fix.


No debería ser `main_quit()`?


I meant deterministic, my bad


コンストラクタからプロトタイプ継承されたユーザー定義関数を共有するカウントダウンを刻むインスタンスを作った


Je trouvais la mise en page verticale plus adaptée et plus lisible.
Par contre en fixant la carte dans une DIV de 100% de hauteur pour ne scroller que la partie de droite (liste et fiches attributaires).


hes the best


Seems an issue of Kernel#singleton_methods, thanks for the notice!


Please add an additional comment why skipping frames is OK


Fizeram coisa errada aqui!!! kkkk


Sorry to ask here, but why nobody updates SB v1.4 on github? I see Cocos v3.4 was updated several times in the meantime but SB isn't. And last commit of SB can't be built at all because of some signing issues. So if you're still the same team, please ask somebody to update Cocos2d v3.4 submodule in SB v1.4 and to commit SB that could be built with script like before...


Yep, see the bottom of https://github.com/spatialaudio/jackclient-python/pull/3

Direct link to my commit: https://github.com/amstan/jackclient-python/commit/489a54225c4584d2c0d157e7d6336fb1a6497762


I would like the configure script to be as "automated" as possible, searching the likely places for the library (just like it searches for a C compiler and other libraries).  That way, a naive (or lazy) user can compile Gambit without too much effort.  The configure script should support a --with-openssl-dir option, in particular because the openssl found during a search of the standard places may yield an old version.  Note that when --enable-openssl is specified (which will eventually be the default), and an openssl library can't be found, a ./configure;make should complete successfully without support for TLS.  Otherwise naive users with no need for TLS will lose interest quickly.  Of course in this case the configure script should give an obvious message indicating that TLS has been disabled, and perhaps a few hints on how to resolve the issue (i.e. that the --with-openssl-dir=X option should be used after installing/compiling openssl).


You've used the `"[attr='value']"` syntax in some places and `"[attr=?]", "value"` in others. Is there a reason for the different approaches?


@premasagar that's included in the flexbox documentation because it affects layout


Considering Computronics adds a Driver for the spatial I/O Port...


3つの箱のうち１つタップするとajaxでjsonを取得してuserIDから当たり外れ判定をする抽選機能


lobi


Some tests need the explicit quoting, some don't. But there's probably some consistency issues anyway. Will revisit.


sassはBEM記述


Improve this part, avoid sentences nested


Geotrek like. :-)


What if i need to just create and send the HTML to a client as a standalone?


There isn't base path description


Improve comment


OK, accepted.


I understand, the the Assignment is mediocre quality, yet from the Assignment text:
> awaits the nextRequest from the listener, responds to it asynchronously using respond

You're supposed to use combinators like onComplete, map, etc. here.

Second, `nextRequest()` might be called before `listener.start`.


Support windows paths??


This changed the formula because quantization effects are no longer present.
Ex.: with hoursToWait = 1, being interrupted was impossible, now it is almost guaranteed.
https://github.com/OpenMW/openmw/commit/14696093b2893bfe1487eb6f7e2a51a863b1cc38


Short feedback for Windows with VS2012: The superbuild compiles (for branch vtk_required_modules). I don't think that https://github.com/statismo/statismo/commit/852591d3aa1542baf247d464638ed68734c81f23 would break it, but I can re-compile if needed.


Nice catch!  :)


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


ファーストビュー最適化。ユーザー流入時にURLに付与されたパラメーターからユーザーに合ったコンテンツを最上部に描画させる


Use Paths to support windows paths


画像遅延処理


You could have done all this with RBAC permissions system. ;)


@jeromenadaud ha :D what is this? :D Is this some kind of penguin?


I'll move to search.customelements.io soon, I was just trying the API directly first


How so? I believe it was you who always told me to reject with an error.


Gah, you're right, thanks!  I think when I was testing this I was entering `?_escaped_fragment_=/all` which is why it wasn't working.  I've put it back now


Tweaks for #4


Is this ever used reading?


Why shouldn't `fetch` be used here?


wrong rspec


@jpounts I checked the jar in maven central and it seems 1.3.0 was even compiled against java6. Do I miss anything?


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


What is 64?


Hi @kpodemski , a Puffin, named Preston


I see. Thanks for correction.


Yes it is ;-)


if autoReverse = true,The animation go back from end point to start point in reverse order and then fire **iterate** event
if autoReverse=false,The animation fire the **iterate** event after it ends in forward updating

**autoReverse**
0 ---> 1(fire **reverse**) ---> 0 (fire **iterate**)
**not autoReverse**
0 ---> 1(fire **iterate**)


Please make sure that braces are at the line end in this pull request.


Detta funkar av någon anledning inte!


:smiley:


`index` 액션은 전체 공개라고 생각한 거지. 물론 서비스 방향에 따라서 로그인을 요구할 수는 있고 (`authenticate_user`) 본인만 볼 수 있게 한다던가 하는 식으로 권한 설정을 할 수도 있지만 (`authorize!`) 첫 버전에서는 이런 거 다 배제.


Ah, never mind. Just notices that it is already done in a later commit. Thank you.


I have found no errors


지금 swindler 프로젝트에서 쓰고 있는 rubocop 설정 파일을 참고


This breaks non-Vagrant builds.


Why did I think we needed to pass many params? mah .. :D


I feel like we could just use an int or uint for the size. Starting position = x,y coordinates, size = 20, therefore we know the tiles.Count() == 20? or whatever is passed in? we could even default rooms to have the exact same number of tiles if we wanted to make generation a bit easier? We could also limit them to like 20-40-60-80 in size in order to make generating rooms easier and avoid rooms that are not symmetrical to start.


Thx, that's much easier to understand ^_^.


@jpountz^^


Ups, slipped up on the javadoc. Fixed now.


This has killed build of SP. Says there is an error in line 230
spiderapi.rb:230: syntax error, unexpected tSYMBEG, expecting '}' (SyntaxError)
                     :inclusive => "If set to true,...
                      ^


This commit changes the strings for English not for Italian.
Look at your commit and change accordingly based on your language because this is totally wrong.


hi


Looks good :+1:

For the future it would be great to have
```
decoded_token = JWT.decode token, hmac_secret, true, { :iss => iss }
```
Do the job without taking care of the string vs symbol and additional boolean flags


Hey @jkeiser,
Any plans on releasing 2.1.1? It'd be really nice to have this fix released.


AFAIK, function pointers don't like/support inheritance and ambiguity in functions signatures.


Thanks :)


What are we looking at?


@SellaRafaeli duplicate require


@bradleyboy @ tdominey Hi there!
Can you guys see what is wrong with my dependency above?


为什么不更新了？


At this level, the cost is moot (in light of the other stuff we're doing).  I can change it back, if you prefer.


:+1: #17 #9


途中段階までしか書けませんでした.....。


We'll need to make sure to adjust this to `mozilla` master after merging...


Ok, this seems fine. I think it may need a small change in the future, as I would like to remove the WebSocket4Net dependency all together, but that's not happening now, so should be fine. It also shouldn't break the Queue Middleware that's already using IEventProducer.


This is correct, yet why returning a combined Subscription?


Do this affect in any way previous versions of sugar running under older versions of gtk?


Actually, I added this one to be consistent with the other classes in the module but it's not used by mongo.

I will correct that.


Обробка для правої частини точно такий самий, як для лівою. Цей код можна не дублювати.
@MariaPetlovana


TeamCity MonoGame :: Develop (Win) [Build 3.5.0.109](http://teamcity.monogame.net/viewLog.html?buildId=7159&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 486, ignored: 6 Build time: 00:19:16


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


JSON parsing using awk and sed - good times.

![wat](http://laslow.net/reactiongifs/Batch03/Fish.gif)


y r u so dum?


No, the old state "active" continue available for compatibility with old versions. We just add the new state.


re: commit remark, the app_version is used after it has been committed, to affect retrieval but the v3 app.
The only time it should ever be checked or change is on create, and so I find this to be correct.

I am not particularly concerned about the tests needing to do something a bit more contrived, as this is only an issue in the tests.


No, don't worry. I was just wondering if I'd missed something and `extend` had extra side effects.


I agree, using the if clause makes it clearer!


Good catch!


Your understanding of cancellation requirement is correct, but implementation is not. `takeWhile` will complete instantly forming a second list. Cancelation token must be checked before each `write`.


LGTM


LGTM, I suppose


For fun I tried this. In python2.7 crc32() accepted both ascii and unicode, but in python3.4 calling crc32("this") raised an error. Passing ascii in python3.4 worked fine: crc32(b"this")


@wkennington These lines do not work on Darwin.


You probably have introduced a memory leak here because you never call ThreadLocal.set(null) or ThreadLocal.remove().

If you don't believe then you might want to take a look at https://github.com/dropwizard/metrics/issues/742 which is basically the same situation.

So I think you should keep the WeakReference. For Tomcat is actually doesn't matter because they clean up values via reflection after logging that severe warning. Other servers don't do that and will leak memory then.


@thinkofdeath ?


I just checked and you're right, so 1.3.0 should work on Java 6 out of the box.


you shouldn't need the sed command.
This should work fine.
cat /etc/redis.conf | grep "^\s*masterauth" | awk '{print $2}


That's a good idea on how to deploy to production. I'm going to think this over.


Sorry if I missed in the previous PR comments, but I feel that "that performs" actually change the meaning for the worse.

What I mean is, a test service or grid would indeed be one "that performs" automated testing, while a set of tools and libraries (allows one) "to perform" automated testing. So, I'm favor of keeping it "to perform". Maybe @ldoktor can say if my English logic makes sense.


I think the assignment was about using `continue/With` and `any` (below).


Ok


Just a side note, would be good to investigate what ironic actions we need from fog, I haven't implemented them all, like the power management etc.

List of unimplemented is here https://github.com/fog/fog/blob/master/lib/fog/openstack/baremetal.rb#L59, not sure if there are some new API calls


@Saurutobi Why did we get rid of this file from what i can tell this is how we should be authenticating the login


This will be fetched from the `Appfile` here.


This should apply for the other changes as well:

I think we should keep 'recovery' for a while and just add 'recovering' instead, handling both of them in the monitor.  We should note that the 'recovery' state for these commands is going away and give time for scripts/interfaces to adapt to the change before dropping the state.


Dwie rzeczy:
@mateuszjablonski : co jak podziedziczysz? tak z ciekawosci pytam
@anybody: w metodzie self i self class zwraca to samo, ale osobiscie wole pisac self class bo dla mnie jest czytelniejsze :)


following what I mentioned before, we should also check for `state == "recovery"` for a while.


Grade process doesn't care, but there should be `println("Bye!")` as per assignment rules.


yeah


`fetch` returns a FrozenHash, and the two `prepare_` methods both modify the hash.


I think it would be nicer to extract `"#{@i18n_prefix}.phrases.#{phrase_key}"` into a local variable, but this shouldn't prevent this from being deployed.

Do you know that log warnings are definitely logged in production? It looks as if the `log_level` is set to `:info` in `production.rb`, but I don't know whether this gets over-written on deployment.


:bicyclist:


Was trying to use Python 3 method of referencing parent class name while running python 2.7.9


a


I split it into 2 commits so that I can cherry pick this over to master.

The dynamic checks are finding bugs.


Salesforce address street is a field that has newline characters between lines while freshbooks uses 2 fields... so do we need to split it?  does freshbooks split it automatically or put it all on one line?


random extra newline


else on the next line


This looks correct to me, but it's not a tail recursion.


I don't have access to deploy scripts so I don't know if it's over-written there.


Q4가 아니고 Q5임


Indeed. In Python 3 cc32() asks for a bytearray() or something like that, instead of str(). I submitted a PR that removed the key.encode() and that may cause key hash fail on python 3.


Technically correct, yet the assignment was about implementing `any` manually:

    def any[T](fs: List[Future[T]]): Future[T] // hint - use a Promise


靠 HTTP Header 判断太不可靠了，轻松伪造


And another empty directory.
The .gitignore file prohibits a lot of major files acces.
Try to buld a server without a chat in game is just stupid.

* with these restrictions the http://www.projectswg.com is no longer considered as open source *


yeah you are right @AchimTuran any input on this?


I had to test the attached because of a modification in delite which broke PageableList https://github.com/ibm-js/deliteful/issues/538. It was a temporary solution to solve the problem wkeese and edchat have corrected the bug since. It is removed in the second commit.
For the store, I encountered the same problem, so I removed the store from the list and it works fine.


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


Just another nitpick suggestion: "extend" and "add more" sound like same, so maybe "extend and add new" would be better here.

So, a final suggestion:

```
* :mod:`Plugins <avocado.core.plugins>` that can extend and add new functionality to the Avocado Framework.
```


Nit: can you update the commit message summary to replace "struck" with "struct"?


It should work with --root. I am not sure about template unit files as I did not test that.


Curious about the removal of MBCamera and using the 'cmake_dependency' for things that aren't on cmake.


It is entertaining that a single slide roughly adds 30~50 lines of xml.


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


Can you make a "if" check for non-systemd systems ?


I'm not sure (and I haven't tested it right now), but doesn't the "lwm2m_bootstrap_delete" send the DELETE before the ACK for the initiate?
May be we extend the transaction (set the retrans_time) so that it is not needed to be sent directly and could also be send first triggered by the lwm2m_step()?
This doesn't solve reordering or message loss but may lowers the number of "resends" in that case.
(Or add a COAP_IGNORE also there.)


What about counting the date diff using momentjs? Google & read its API for that.


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


1. It's good you found `blocking`. But, I suggest, you research what it really do.
2. This implementation consumes a thread per `delay()` - there are better options in Java library.


@Diego-F-Aguirre done with my review. press on!


CLI tests passes on Unix.


:+1: :sparkles:


yes, of course. removed the commented lines and recommitted.


look in your mail box obi :)


Yep, I guess so, and maybe in few other places too. But that is another issue - locked inputs (until wallet restart) while this commit is about keypool managing issue that could lead to too fast key usage (or wasting keys actually).


To be clear - if a value begins with a number, it needs to be interpolated (`[id='#{id}']`) rather than parameterised (`"[id=?]", id`)


You can have a look at os.path.relpath and os.path.abspath in https://docs.python.org/2/library/os.path.html


Jason,
    Can you please add the following content/sections..
1) Autoscale using Schedule and a template snippet
2) Autoscale using Webhook that returns the web hooks url in the output and a template snippet
3) Autoscale using Cloud Monitoring and a template snippet.


Yes, but there's no API to it that I could find, so I decided to use my own. Also, the cache holds a limited number (that's set in the properties) and so may not hold really old names/UUID's. One purpose of admin commands is to delete or deal with players who may not have logged in for a long time, they may not be in the cache.
If I'm wrong here, I'm happy to use another way.


So much better now!


Fixes #26


There seem to be several things going on here. I don't understand the IPv6 issue as it is the TCP BTL that is failing, and it should have nothing to do with the OOB. So it looks like some crossover is occurring that I don't understand.

The dynamic issue stems from a race condition - the OOB change exposed it, but isn't the root cause of the problem. I'll have to think a bit about how best to address that one.


our project is same


We need to have the escalation discussion.  I'm not sure a service account *has_one* securitycontextconstraints anymore, and we need to know that before we ref it here.

In brief - we need to decide who can use a service account to launch a pod.  That sounds like a policy decision.  The person who can create or modify a service account has to have the privileges they create on the service account (you can't give yourself the ability to create priv pods on a service account).  Not every cluster admin will give their project admins the ability to run privileged (some will if they constrain the nodes the pods run on).  If SCC is a part of SA then we can't give project admins to edit SA.  Therefore SCC should be distinct from SA and require a separate resource.  That resource might be real or virtual.


Mind adding javadoc-style comments to Init and SetMode?


At least this part looks fishy:
r = readlink(path, target, st.st_size + 1);
if (path_startswith(target, "/usr/lib/systemd/"))


I am a nice comment


I just wanted to get the color names to match the way they are spelled in game:
![gray](https://cloud.githubusercontent.com/assets/12382962/7568219/1ff1d154-f7c7-11e4-8f56-3b45a7fa143a.jpg)


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


I will, in a `master` commit that documents the whole `r_public.h`.


No like, when you call OfflinePlayer#getName, it'll automatically grab it from the cache. I handle this in Factions by checking if the name is null and if it is, I just print the uuid. Otherwise, it grabs the name from the cache.

https://github.com/drtshock/Factions/blob/1.6.x/src/main/java/com/massivecraft/factions/zcore/persist/MemoryFPlayer.java#L353


@Connorhd Sorry missed the original PR. Is this right? Seems odd to force encoding to UTF8 then ASCII-8BIT. Would think we could use something like Content-Type response to figure it out.


I wanted to ensure that we will consider symlink only if symlink itself and its target are both in /usr.


<< lunchbox


package br.com.alvoradamaringa.domain;

public class NivelUsuario {
	private String Descrição;
	private Long idNivelUsuario;

}


We took it out because we reverted all logging in changes when we were trying to debug your stuff thursday.


@tbielawa Oh yeah, I've ran into the forkedness. Thanks sib.


I'm a complete PHP newb, but attempting to follow steps above on latest Yosemite (with zero PHP/Pear custom config) and getting the following errors when installing ncurses:

```
configure: error: Wrong ncurses lib version or lib not found
ERROR: `/private/tmp/pear/install/ncurses/configure --enable-ncursesw=autodetect --with-ncurses' failed
```

May be something on my end, or maybe a step missing above? Not sure, but wanted to toss in my experience here.


Vafan gjorde du här ?


Update the translation for the module purchase_order_report


I love to geek out


Note that you do not use `reversed_method` (nor does `PyArray_GenericBinaryFunction`).


cd /usr/lib/systemd/system
ln -s bar.service foo.service


And this is helpful why?


@zarquon42b @arnaudgelas  It works (compiles, and tests are ok) for Ubuntu 14.04.
I am still having issues with Debian testing, but they are not related to the changes. I will investigate them further and if necessary open a separate issue.


I think this will have to wait until 2.0 release. If we break this now I think there will be a lot of complains about breaking the API.


Yes, I think the "right" thing to do is use the charset in content type if its available, then make a guess based on whether the mimetype is binary or text and the actual contents of the response.

All this change does is makes the different ruby versions consistent (but still wrong in many cases). Without this change older ruby versions will use ascii for the empty string and jruby will use utf-8 for the chunks. This means that for jruby you get binary marked as utf-8 encoded.

With this as it is now ascii text will come through as utf-8, and anything else (including valid utf-8 text) will come through as ascii_8bit.


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


@nfeske: Please add to staging

https://github.com/ssumpf/foc/issues/10


If `NSTimeInterval` conforms to `Equatable`, you don't need to implement this, only `==`.


This duplication of structure between name and inaddr related functions appears systemic; perhaps it's an indication that we shouldn't have the hardcoded division between looking by name and lookup by IP?


I used this a lot, what should I be using instead now that this is gone?