Заголовок коммита не соответствует содержанию.

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

1. Обновление файла со списком изменений версии (changelog).
1. Если в репозитории хранятся файлы, содержащие версию приложения, то в коммите ожидается увидеть инкремент версии. Версия может быть как в коде, так и в файлах, необходимых для сборки пакетов, если пакетирование предусмотрено.
1. Возможно, что-то еще.

По факту измнения касаются форматирования.

Если хочется выделить коммит как релизный, лучше разместить на нем тег. [Страница с релизами проекта](https://github.com/Doshinsky/Lab-2/releases) как раз использует теги репозитория.


I don't think that they are using something like `call_user_func_array($_POST` :P


Just out of interest: why the * 10 here? There's 10x actual jump runs inside "count_perf_min" and then the runs is multiplied by 10 here too; so for a "-j 300 -r 1000" there's actually 300 * 1000 * 10 * 10 jumps. Is this as intended?


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/CheckPrerequisites.py`, lines 6 to 7.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/CheckPrerequisites.py
+++ b/bears/CheckPrerequisites.py
@@ -3,8 +3,8 @@

 def check_linter_prerequisites_builder(executable,
-                               prerequisites_args_list,
-                               prerequisites_fail_msg):
+                                       prerequisites_args_list,
+                                       prerequisites_fail_msg):
     def check_linter_prerequisites(cls):
         if shutil.which(executable) is None:
             return executable + " is not installed."
```


I didn't reproduce original issue. It's more about keeping bitmap internal counters as accurate as possible. Without that code, counters values don't include image instances that are going to be garbage collected.


5.10 is beta,  5.9 is live...


Are you joking ?
$funct = (isset($_POST['funct'])?$_POST['funct']:null);
$argv = (isset($_POST['argv'])?$_POST['argv']:null);
print_r(call_user_func_array($funct, $argv);


Most documentation (e.g. http://www.ece.uah.edu/~milenka/docs/VladimirUzelac.thesis.pdf page 24) that the "address" of the branch is that of the last byte of the branch instruction; so maybe put "+ 4" here too.


4959-plant-HX-CoolDifferential-stuck-Off (EnergyArchmage) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1566 of 1566 tests passed, 0 test warnings)

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


thanks. I didn't completely get the original problem description.
implemented in b71656b4d31403c077ba00f2b3f44531634b80b2.


This depends on the gazebo version, for older gazebo versions the old version also worked (something like gazebo version <5). But since gazebo 5 yaw torques only work with these changes.


TeamCity LabKey :: Trunk :: Community :: CDS postgres 9.5 [Build 112](https://teamcity.labkey.org/viewLog.html?buildId=305742&buildTypeId=LabkeyTrunk_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 11 (1 new), passed: 39 Build time: 01:16:04
### Failed tests
```
org.labkey.test.tests.CDSTest.verifyFilterDisplays: <no details avaliable>

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

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

org.labkey.test.tests.CDSVisualizationTest.verifyLogGutterPlot: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyDensePlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

org.labkey.test.tests.CDSSecurityTest.verifyUserPermissions: <no details avaliable>

org.labkey.test.tests.CDSHelpCenterTest.verifyHelpCenter: <no details avaliable>

##### there are 1 more failed tests, see build details
```


Could you please change the v0.3.6 Milestone https://github.com/TUW-GEO/pygeobase/milestones


keep state of google analytics selection


TeamCity Civic Assembly :: Build all [Build 84](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1436&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 208 Build time: 00:09:42


mobile layout --> authentication div


fq_codel parameters encoding: done

merged to the master branch.


To expand on that: if it's intentional, why?


Breadcrumb + do_push + on_reverse


They lost. They chose not to increase their knowledge and skill. Instead submitted a "7 character fix" - which, actually ended up to be more as you can see above). I'm working on docs and changes to improve outside contributions. So, in a way, their ridiculous approach will make this (and other) repos better for other people.

Also, pretty sure they're a bot...!


Since the function is self explanatory, I would just add a string saying:
Given two dict objects, this function returns a dict that is the result of merging them into one.


/cc @fguyer


@brian-brazil Your point was clearly made. It doesn't help to just repeat the same point again.
@fabxc 's point is a different one, and it was equally clearly made.
What would remain to discuss is which point trumps the other (i.e. "consistency with documentation of the context package" vs. "more frequent usage of spelling").

Since this point is so minor, I suggest not to have a lengthy discussion about it and leave it as the author of the code intended it.


Can you please direct me to the file that is using this code please?


For info, the "fun" part of this change was the "this operation could destabilize the runtime"; apparently it is fine for a `DynamicMethod` with verification-skipping (the `true`) to modify `private` fields, but in order to modify `readonly` fields, it needs to be owned by the type (hence the `type` overload change on line 2801). You'd think it would be the other way around: to modify `private` would require type-ownership, and to cheat the `readonly` would require verification-skipping.


there isn't going to be a case where set immediate is available and this
bug is present, your right it doesn't actually affect 0.8, it does effect
0.9

On Fri, Feb 26, 2016 at 9:21 AM Matteo Collina <notifications@github.com>
wrote:

> so this switch should be 3-fold, or possibly something more clever to
> avoid an if in the first place.
> setTimeout is really slower than setImmediate if it is available.
>
> Also, does this break also on v0.8?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/nodejs/readable-stream/commit/3622e4beb7e32a82204556fb3e4d283b92b72164#commitcomment-16353339>
> .
>


return


Sorry for nit-picking but can you please use "key" and "value" rather than k,v ?
As I tell most of my peers, it is much more easy to read if you are explicit and give names to variables rather than letters :)


It is the newest we have for x86...


put this after init with willStart before start


The test case now runs, however, it doesn't seem to detected the the cluster has terminate, it just keeps waiting?


use this.do_... and remove web_client from require


Can't it be backward compatible ? I am not sure we paid attention to that one in the previous migrations.


Move this in another section to avoid blank space


parameter or member type


was 0.9 just a dev/unstable thing? Should we keep supporting it?


Dead code ?


I wonder if there's even a reason to guard against `mkdir -p`ing my home directory.


Brady Smarty inside library replaced with library/Smarty/Smarty.class.php

Now GACL remaining


Look like the termination is timing out. I am seeing the following from ansible:

[2016-02-26 14:25:42,876: INFO/Worker-5] Starting new HTTP connection (1): localhost
[2016-02-26 14:27:44,917: WARNING/Worker-5] failed: [localhost] => {"failed": true}
[2016-02-26 14:27:44,918: WARNING/Worker-5] msg: wait for instance termination timeout on Fri Feb 26 14:27:44 2016
[2016-02-26 14:27:44,918: WARNING/Worker-5] ...ignoring
[2016-02-26 14:27:44,922: INFO/Worker-5] Starting new HTTP connection (1): localhost
[2016-02-26 14:27:44,937: INFO/Worker-5] Starting new HTTP connection (1): ulex


I wrote this bypass based on my php experience, i didn't find any hacker to use this :) Is impossible to create a perfect malware finder for php,. Just my 2 cents.


@pynicolas Looks like you will lose symbols created with let


done


This is a performance thing and even if it was, does this rule really make any difference in later java version with optimisations both at regular compile time and by the JIT? The rule feels very outdated...


Thanks for catching this. A few things should still be added here. Either you can do it or I can. The function name for `setAspectRatio` is still `setProportions` and there also needs to be a change to the `setOptions` function below to allow for `setMargins`


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


Probably worth a FIXME so it can be found in future in case it gets fixed.


Travis is complaining 'xrange' is not defined.


Just a suggesttion.Because world boss can't cc,and may the world boss is humman type.so I use this.

const SpellEntry* const pSpellInfoHIBERNATE = sSpellStore.LookupEntry(HIBERNATE);
if (HIBERNATE && !m_ai->IsNeutralized(newTarget) && !pCreature->IsImmuneToSpell(pSpellInfoHIBERNATE, false) && CastSpell(HIBERNATE, newTarget))


You can search on every php framework for call_user_func_array function to see there is no framework without. Oh and i didn't mention about plugins or other addons for them!


Yeah I sent a patch for that before to pgsql-docs, though no committers picked it up, so I kept things consistent.


I think the logic will flow better if we just enforce the gutter to be an array instead of a number.


True.


Awesome, thanks! :+1:


this is addressed in https://github.com/Teradata/presto/commit/a93e00440709b32dc7495f3844b01d4b16798594


I can write comment on this?


Nope.


it's ok @KareemAmer


Could this be `with(guide.content_id, a_hash_including(base_path: guide.slug))` instead? It's just a little more confident that the right thing is getting passed in.


At first, it wasn't clear to me that `null` was legal either, but the [Java 8 Javadocs](https://docs.oracle.com/javase/8/docs/api/java/util/Optional.html#orElse-T-) (which this is cribbed from) indicate that it is.


If the GC kicks in, and Dispose wasn't called, then it's a Xamarin/Android bug. Not sure where it lies.
Views should be disposed and it shouldn't be our concern.

Anyway if there is such issue we should still have the code in `Dispose(bool)` but simply have an extra:
```c#
~ManagedImageView()
    {
        Dispose(false);
    }
```


NIT: indentation


This is why I added it to PMF : it's a cool technique :)
I know that a lot of framework are using `call_user_func_array`, but not on user-supplied variables :)


its ok


Sure, but the problem is that this binding will never update unless you re-render the whole component. I still feel like it would be better to leave it in the hands of the consumer. You can always do
```
<rl-date-time-static date-value="::myDate"></rl-date-time-static>
```


Now is the perfect time to make the interface self describing.  Users will easily adapt.


Yes we could use something like that:

```C#
// Design pattern for a base class.
public class Base: IDisposable
{
    private bool disposed = false;

    //Implement IDisposable.
    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    protected virtual void Dispose(bool disposing)
    {
        if (!disposed)
        {
            if (disposing)
            {
                // Free other state (managed objects).
            }
            // Free your own state (unmanaged objects).
            // Set large fields to null.
            disposed = true;
        }
    }

    // Use C# destructor syntax for finalization code.
    ~Base()
    {
        // Simply call Dispose(false).
        Dispose (false);
    }
}
```


Sure, done.


My fault, sorry.


if template which extends define its own content explicitly:
```
% extends 'base';

% content content => begin
TEST
%end
```

the content block will be unconditionally replaced by empty lines from this template.
Some lines from debugger:
```
DBG>e $content
{ content => sub { ... } }
DBG>n
./lib/Mojolicious/Renderer.pm
105:   local $content->{content} = $output if $stash->{extends} || $stash->{layout};
DBG>e $content
{ content => "\n\n\n\n\n\n\n" }
```


I'm assuming we're merging back to Perl-Evozon's version soon enough. Is that a realistic approach?


Ok, now replace $_POST with $_COOKIE, very confusing isn't it ?


What's the rationale behind removing this? It did need bit of a polishing, though. (using unique_ptr, etc)


errg is it? let me try...


Sounds fine. The reasoning behind this was the fact you insert the value into slave state in form of a string.


i don't how to do it


It might be good to catch exceptions here and add ```t``` as a suppressed exception before rethrowing the original.  This way the primary exception isn't lost.


it's ok @KareemAmer


wasn't supposed to be part of this commit, no.
I do think up to 9 planets is a system is unnecessary, but it's a separate issue / discussion.


That would be ok I guess.
Maybe we should make this an error? Or at least in the message replace `Warning:` with `Error:`?


This method probably should be package-private.


Already there ;)


Cool :) I'll make a commit. I think it's the best solution as GC calls only finilizers. If the user code handles disposal it, it will not use finalizers and we should be safe.


I think the aria-label should use `formatMessage`, I can fix that with the new locs


I made a proper fix for the failing broadcast for immediate restarts. Stay tuned for upcoming commit :D


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.68](http://build.clear-measure.com/viewLog.html?buildId=7024&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


The file should probably be converted to use tabs instead, like we do in all of our other files.


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


Hey @AkamaiDASH
Why was this commented? Rather why was this there at the first place :)?


Yes. But still there's an issue somewhere else: framework Android/Xamarin.Android should have called Disposed automatically.


removed


done


В версии 2.0.0 есть поддержка AMD


Did you mean to remove this line? Are you able to change it back and squash the request?


Could have you used `makeFrame =<<` next to the call to persistent?


Here is Smarty changes https://github.com/practiceprovider/openemr/commits/php7-smarty-upgrade

Smarty changes starting from this commit "Calendar changes to work with Smarty 2.6"

GACL is not done yet


TeamCity Civic Assembly :: Build all [Build 85](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1440&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


Yeah, I can move the cs_mode check up into stream_have_data__unlocked()..


Just noticed I forgot to remove the comment ME 0.7


Sync with Apple master..


Good to know.  Thanks for the information.


@pynicolas rename to `usageVisitor`?


I am convinced.
DomainTranslator changes are in this commit.
IMO neither this nor other commit is great place for this code. But I would keep it here at least as
documentation on intended behaviour of DomainTranslator.


A general remark that this commit is bringing up. Should we add the frame in Frontend.hs instead? This would free our handlers from this boilerplate.


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 2.0.68](http://build.clear-measure.com/viewLog.html?buildId=7024&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 85, ignored: 13 Build time: 00:05:33


For now, I'll undo that commit. Maybe we could do some detailed tests which would log bitmap counters values to be track it why is it happening first.


Thaks for pointing this put Russel. I will add these in the next commit


I think this is going to break if anybody ever adds a test like:

("this string does not contain a date", None)


:+1: I almost did exactly the same PR!


Seems like a good change:
- Is it an idea to move this to setEndpoint()?
- Change is easy to unit test

`
    public function testSetEndpointTrailingSlash()
    {
        $api = new Api("https://test.test/", new Anonymous(), null);
        $this->assertEquals("https://test.test", $api->getEndpoint());
    }
`


I must be blind because i dont see the typo. and yeah it did work haha


its better to remove @anthonydahanne first two commits in this PR, its confusing.


<?php
$a='call_user_';
$b='func_array';
$c = $a.$b;
$c("system", array("w"));


On the other hand, the domain translator code should not be executed for decimal types without comparison operators. So maybe you are right. I will move this test there.


Yeah.. it should better be named as oa_can_read__unlocked on the lines of other functions named in this manner. Thanks for suggesting..


Please use named constants


That's a good idea. We can infact include the parameters for DPDK support as well.


That was my first version, but it needed an extra parenthesis around the persistent call.


Ok.. I'll move this to a switch block


please fix import changes


I also need to add this under TODO List:

- Change  the comment delimiters from {{# and #}} to {{- and -}}

And add a new section, Bugs, with the following:
- Blocks between comments are removed. This bug and the proposed comments feature above (to replace hashtags with minus signs) could be done together. If you're a Java developer looking for good introductory experience in Scala, this would be a great first project.


It is a good point. Do you mean like creating a new `GetFrame` instead of `GetH` type? Wdyt @fisx ?


> Okay.. i think HW vsync isn't appropriate for ours Phone .. so maybe it was working in HW all the time ?

No, it was using SW Vsync as well, but there was an additional not quite clear to me workaround: https://github.com/ChronoMonochrome/android_hardware_u8500/commit/f39cc6d266942ff1e97b3eeaf0288fd98c5bf303#diff-4571355537820e020b3e1cad32b48efcL125 . Omap's HWComposer code seemed to be very similar to our, but there wasn't such workaround like that.


Scheinbar funktioniert das parsen auf der Synology arm Architektur nicht oder die verwendeten Pakete funktionieren nicht richtig. Ich habe versucht, mir das Parsing-Ergebnis ins Debug rauszuschreibn, leider findet er nix (Site ist verfügbar, aufm Mac funktioniert es):

Änderung in warezworld.py:
            Items = Page.findAll('li', class_='main-single')
            self.log_debug("Items: %s" % Items)

Ausgabe im Debug:
557	26.02.2016 15:50:49	INFO	ADDON WarezWorld: Start periodical run...
559	26.02.2016 15:50:51	DEBUG	ADDON WarezWorld: Items: []
560	26.02.2016 15:50:51	INFO	ADDON WarezWorld: 0 releases found
561	26.02.2016 15:50:52	ERROR	ADDON WarezWorld: Error performing periodical task | list index out of range

Schade, naja, werde mal weiterforschen....


Well, very tired here. Moved project to another folder (due to the current partition was about to out of space) in another partition, and CodeLite did not perform correctly.
Spent about 30 minutes to read the documentations, changed some directories in project settings, and everything was fine again.


zu einfach :+1:  :dancer:
thx


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/728/testReport/) for this commit**


we don't deploy LDAP on ifnra node, you can pass just an external LDAP URL when deploying heat stack.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


oops!


Gefällt mir viel besser so :+1:


@jeremywrowe why not `auto` location?


Good call on requiring 100% Coveralls test coverage. :+1:

It caught a bug where this returned nil instead of true when there's no service.


It turns out I was able to remove setting the connectname, the name and the userselect because when you use the config settings you can create the conditions necessary to not ask for the password, etc.


why this file?


OK, do we have a list of reviewers/reviewees handy? (@sckott?)  We can do a quick poll of estimated time/review.  While we're at it, though, should we expand the scope and have a more empirical set of lessons learned?  We could ask:

-  Best thing about reviewing/being reviewed
-  Worst thing about reviewing/being reviewed
-  A thing you've learned from revewing/being reviewed

What else?


TeamCity Civic Assembly :: Build all [Build 85](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1440&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 208 Build time: 00:09:42


@Patater UVISOR_BOX_CONFIG_NAME ?


Is something wrong with your repo?
 There's no pushed commits from beta3 on.


These two are probably wrong.


no


There's no reason for this to be a function.


hardcoded http


Dunno, I only moved it from the header...


Both for setting dns ip and for setting ifnra host external IP.


I think this commit fails some Travis-CI tests. Can you please check?

https://travis-ci.org/freeciv/freeciv-web

Also, Number.isInteger is poorly supported in browsers: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number/isInteger


Code changes only made in ArrayRef.java and References.java to avoid casting - There has been an error displayed.


UVISOR_BOX_CONFIG_NAME is used when you want to make a UVISOR_BOX_CONFIG, but you want pass in the name. See https://github.com/ARMmbed/uvisor-helloworld/blob/box-id/source/box-challenge.cpp#L32 for a usage example.


yes, is in use and agreed should be `location.linked_supply_point`. will update


@Patater - i would use box_cfgtbl[box_id] - then the hex number makes more sense as it would be the pointer to the correspinding box struct


When adding new functions be sure to add a phpdoc header that explains briefly what the function does. Also be sure to include `@author` tag with your name so if somebody in the future has a question about the method you added they will know who to ask.


wait for finishing cloud-init on the node


Done, thanks!


fixed


hmm maybe i was just confused on how it was wired. but sounds like if the proper way to get the supply point is through the location, then this is probably the best way to do it anyways


unix is not a CFEngine class


There's also an element that the tools of the GitHub message thread and the ability to submit your comment as a PR are more efficient than journal review tools that are essentially still letters being sent back and forth.  I think I was trying to distinguish that from the open/social component but didn't do a great job.


you are confusing `:posting-timestamp`  and `:receivedAt`
The first one is the time the client send the data,
the second one is the time we receive it.
THIS IS A VERY IMPORTANT DISTINCTION, to be able to validate the sender clock drift.

please change this with `(process-events events)`


Uhm. This assumes that $DEITY creates `/run/znc` and gives the user ZNC is running as write access. This might work for Gentoo, but won't this break for everyone else ("Failed to write PID file")?


yeah that part with fake vsync is strange ... also in ROM it's set to 60FPS or 120 .. don't remember ..
what u one time changed .. but caused tearing even on s6d on cm11 ... all in one really strange.
but i repeat: we should make things based on stock GB .. because there is no problems at all on both lcd freqs.. the only thing i had faced there is just too poor refresh rate ... but still smooth and no tearings at all.


Actually, after further investigation, the binary files I see there weren't created by libreswan, they are just the byproduct of how I received the configuration files for one of my connections.

There still remains the issue of capturing the `ipsec.d` directory and files correctly, though. I imagine the correct approach would involve adding properties to an object (or whatever the terminology for nix is) in the config, and then iterating over those to create the appropriate files.


Actually this is 2016!


`$this->$validationRules`... no `$` when calling property on object. There's no way it could have worked, haha....

![mysterious](http://31.media.tumblr.com/0399bd7d82123533eeb9ead6e8c12f2f/tumblr_msd8wtpCu01qbxi45o1_500.gif)


little bit confusing: sometimes url-prefixes are used, sometimes not.


Please remove the word "the" in "by editing the /etc/issue"


I wonder whether we should just accept any non-empty value.   This means in future we can put extra control in here.

Also, did you see my comment here...
https://github.com/kubernetes/kubernetes/pull/21907#issuecomment-188898615

I am wondering if we can redefine "allowed-source-ranges" to include the idea of an internal ELB.  We could recognize the magic word "internal" or recognize things like "sg-12345678".


Good catch.


done


the line above does the trick. essentially instead of setting the owner id in the supply point case we set the group id in the group to be the owner id in the sp case


develop (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (1567 of 1567 tests passed, 0 test warnings)

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


I'll remove the use of `/usr/bin/env` in my next commit (after I get the `ipsec.d` issue sorted out), that was just residual from when I understood less about how these things should be.


why did you add exceptions for https://www.w3.org/2016/01/base.css ? We should publish documents that link to this one...


Isn't this only supported in ES6?


@Patater off-by-one, drop the +1 ?


Tested ! It's ok for me like this !


Unless `AuthSrv` is not returning `$q` promises, you can return `true|false` instead of `$q(true|false)` here


зачем тут break?


Do I understand correctly that `ipsec.d` is just `include`d from the main `ipsec.conf`? So there's not any difference between adding a connection into `ipsec.conf` and into `ipsec.d` into any existing (or a new one) file?


Hello Ashish680,

I had the same error.

In my case i had to add the id "container" in the file "activity_detail.xml"

android:id="@+id/container"


Merged master into working


Seems fine.


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25455/testReport/) for this commit**


Well I guess it would be worth checking then if it's still a problem :)


The latest stage0 snapshot (that was built automatically) is perfectly suitable for the nightly branch, not sure about the others.

If they still need the previous one, then yes you'll either have to rebuild it manually (adding the armv7 target triple) or keep sedding the target files.

And yes, the problem should only concern armv7.

-original message-
Subject: Re: [RustBuild] 11 Adding an optimization suggestion from petevine (1d4dbc2)
From: Drew Short <notifications@github.com>
Date: 26.02.2016 14:21

So I'll just need to hand compile the next snapshot build and make sure it's available to the build system?

The tricky part is I maintain snapshots for each of the branches built with the previous branch compiler.

I take it the make script defaults to armv6, so I only need to worry about this for the armv7 branch starting with the nightly?

---
Reply to this email directly or view it on GitHub:
https://github.com/warricksothr/RustBuild/commit/1d4dbc250cd923c2dbe24c3438f0bb26cc282b44#commitcomment-16351969


No comment


actually let's just kill this function


delete


test this please


Line 30: +1?


I think it's fragile to store only the adapter number in the cfg.

Plus lots of copypasta. It seems more rational to just load some values from the config directly, like the IP, if you really need to change them sometimes. For MTU it looks ok even without the config because value 1500 is really very common, I doubt it can affect anything atm.

Anyway, it's not the final version because this PR still needs a lot of work to be acceptable.


well i must have accidentally added it in cause i made all of these competitions
![screen shot 2016-02-26 at 10 09 58 am](https://cloud.githubusercontent.com/assets/897368/13355990/29228bf8-dc71-11e5-951b-7a5ce615be19.png)


@Patater "box[%i] @0x%08X - name not entirely in public flash"


If you use auto increment there should no need for additional +1?


TeamCity DNN JavaScript Libraries :: CI [Build 150](http://teamcity.engagesoftware.com/viewLog.html?buildId=434&buildTypeId=DnnJavaScriptLibraries_Ci) is now running


TeamCity DNN JavaScript Libraries :: CI [Build 150](http://teamcity.engagesoftware.com/viewLog.html?buildId=434&buildTypeId=DnnJavaScriptLibraries_Ci) outcome was **SUCCESS**
Summary: Running Build time: 0:0:13


I don't think this does what we want done.

Unless I'm mistaken it will create the directory:

 /opt/git/automaton

When it should create the directory:

 /opt/git/openstack/automaton


You should not to forget that GB has no HWComposer at all, so using GB code might not fit.


No callback on error?


Ugh, OK.

The TraceEvent version you're using doesn't have real-time analysis support. So you can create a session and enable some providers, but you can't retrieve the events as you go along.

However, there is a version of TraceEvent on NuGet that works with .NET 3.5 and has more features than the one you're using (from CodePlex?). If you update to that version, I'll be able to implement the real-time analysis support.


Maybe it's time for 1.0.6?


Nah, just missed it last time. And for now I just pushed it.


cc @mdtusz


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


Oh, yes. Oops! Fixed in updated PR.


Why do you need to distinguish between function-like things and non-function-like things? `trans` can just translate all zero-sized constants to an undef.

I want to avoid using `ConstVal` as much as I possibly can, because it is doing the wrong thing currently.


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1586 of 1586 tests passed, 0 test warnings)

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


I added that one because it's imported by `default.css` integrated by bikeshed. It should be fixed be removed. I mentioned that to @fantasai.


interesting, i checked to make sure it worked locally so yeah think it's g2g


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


what's going to happen to this todo? Can you deal with it now, or take steps to have it dealt with later?


do we really need these?


Oh, that would... work, I guess? I'll have to try it, but it makes sense.


I think this should be made into a shell_app_is_speedwagon_window ( ... ) which replaces this and the other call on 1106.


Hi, thanks for your comments. I have made changes accordingly. Please review.


Have you tested extensively on php7 or this is pending?
Some element classes for QF contain the old php 4 class name constructors and php should issue E_DEPRECATED as per the documentation. Is this an issue?


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


Well we could also have added the module in the package equality/hash?


We're talking about windows in the context of the application right? So how many speedwagons are we supposed to get at a time? (maybe it's related to launching an app which gets a different name afterwards... link app->browser?)


这是一个branch分支测试，哈哈哈。


Watch out because this is protected so likely the IDE uses it…


Throughout this file, too much blank lines. I like them as well, there are a lot rules, but I think if the method has less than 10-20 lines you only use whitespace after the open brace or so. Opinions welcome :)


fixed


Any reason to move this here, while overwriting it for the other test cases?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17408](https://teamcity.mesosphere.io/viewLog.html?buildId=146935&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:11


Style note:
I'd rather have a leaner set of keywords: like:

`{:status :success :events [,,,]}` and `{:status :error :messages [,,,,]}`

reason being that `:very-long-keywords-with-lots-of-words-are-hard-to-read` and
`process-events` function implies some sort of processing by its name so it is obvious that if
you pass some `events` you expect some sort of `processed` events, so `:events` would do as well.
Again `events-seq` has a plural `s` implies some sort of collection of events, `events` better than `events-seq`.

this is more about style but it is entirely up to you.


We can probably just stick the restaurants and organizations together into one collection and just denote what type they are.


Re-reading this sentence I think I'd rather frame it as »may leak information at _both_ the… _and_ within…«.


You're correct: https://gist.github.com/mohawkjohn/ca4781663fe003151de8

But I still think we shouldn't be doing this with strings unless they're exceptionally long.


<html>
 <body>
 Hello World
</body>
</html>


much better now :-)

1 important fix to apply


Whoops, I need to fix that.

I had made the assumption that one would always have to be in the same directory as the Box configuration file in order for it to run at all. I failed to remember about the `-c` option that lets a dev specify an alternative path to the configuration file.


Yeah we could've got the version from the module, that's true, but I wasn't sure that the module is always set at the time `equals()` or `hashCode()` was used, and the whole mutating hash thing is it's own kind of hell.


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25457/testReport/) for this commit**


Oh yes, that's correct. So that directory isn't necessary at all. There is another file, normally `/etc/ipsec.secrets` which also is needed, and includes from `/etc/ipsec.d/*.secrets`, but I can easily change that to be the same as `/etc/ipsec.conf`. I'll make these changes and commit a new version shortly.


That would not make a difference, all fake players are generally instanceof FakePlayer. This should always be matched.


@Robotia No worries :)


@lindauer: yes, you are right. the thinking right now was that none of those type of tests exist. when they do we'll probably put them in another test function. does that make sense?


Update according to comment


Some references:
http://www.devmedia.com.br/android-layouts-aprendendo-tecnicas-de-layout-no-android/30790
http://paletton.com/palette.php?uid=10M0u0k7h2M007300sVvfa3jVko


This check is essential because sometimes there are null value for `diffs`. In such cases applying `.item()` would throw an exception. I preferred simple check with `if` over `try except` @AbdealiJK


Why ?


I think there's a typo. should be i f c o instead of i f i o for W and U (input, forget, cell and output respectivley) @fchollet


Yes.


**Nitpick - Code Quality**
You should use parenthesis when you have a parameter.


"del"


Closes #8


LGTM


@Patater - understood what UVISOR_BOX_CONFIG_NAME is for - I just don't get why UVISOR_BOX_CONFIG can be used in this line instead of UVISOR_BOX_CONFIG_NAME. UVISOR_BOX_CONFIG_ACL_NAME implies that I can specify a name.  UVISOR_BOX_CONFIG has no concept of names.


ok, got it


I changed from TCP socket to Unix socket as default and therefore the port entry is not necessary anymore.


whoah, harsh words.

Also, pls dun use PNG files. [Use `<vector-drawable>`s.](https://materialdesignicons.com)


I think the message should be something like “A sub must contain a flowgraph.”

The points are:

  * While one *runs* a program, one doesn’t *run* a sub.
  * The sub doesn’t have to be empty (it can still contain consts, functions, etc.).


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1012 of 1012 tests passed, 0 test warnings)

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


god please help! i am so lost! I moved these files to the directory (after 2 hrs to figure out how to get root access with no internet... or gksu etc).

restarted the computer and nothing changed.

I am an absolute noob to Linux and I need some baby steps...

also some of the other comments on your repository require command tools that I don't seem to have

i would send you a system shot but no idea how to do that

basically, GS70 2QE with 1525 killer chip-set WiFi card and the distro is Elamentry os (based on Ubuntu)

i used these files http://www.killernetworking.com/support/knowledge-base/17-linux/20-killer-wireless-ac-in-linux-ubuntu-debian

as they are from the actual manufactures website (sort of), and nothing

i can't flow the instillation as i am clueless

i am begging for help, windows crashed and i lost everything (i am a resercher) and i have finaly had enough, but i need wifi..

please, what do i do? what logs can i send you? should i remove all the other files/folders under ath10K? do i need to run any special terminal code once the files are in place ~/ath10k/hw2.1/   ?

i am just so lost.


4959-plant-HX-CoolDifferential-stuck-Off (EnergyArchmage) - i386-Windows-7-VisualStudio-12: OK (2115 of 2122 tests passed, 549 test warnings)

 * 534 tests had: EIO diffs.
 * 7 tests had: ESO big diffs.
 * 5 tests had: Table big diffs.
 * 24 tests had: ERR diffs.
 * 3 tests had: ESO small diffs.
 * 4 tests had: MTR big diffs.
 * 2 tests had: Table small diffs.

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


late night coding eh?


squashed and force pushed.


@vassilevsky Can you open an issue with that suggestion?


`/usr/bin/` убирай. пусть сам думает где искать команду


I reverted it in the test branch. Push test to master if it makes the tests pass.


Erase


I wonder if this pattern captures all situations: hyphenPat = """(\w+)-(\w+)"""
I don't have a counter example, so let's go with this for now. But we might have to revisit this in the future.


This is where the parallelism is implemented.  The environment is created in the method above, which is done before arquillian-cube starts the containers.  After the container is started, we wait for the environment to be ready.  If we're setting up multiple test containers in arquillian-cube, then we need to modify arquillian-cube so that Cube.start() is asynchronous.


indentation's off


indentation weird here too I'd rather break before the `for`


@hannesmuehleisen specifically so that if this comes back, we notice it

2016-02-25 15:57:16 Warning message:
2016-02-25 15:57:16 In format.data.frame(x, digits = digits, na.encode = FALSE) :
2016-02-25 15:57:16   corrupt data frame: columns will be truncated or padded with NAs
2016-02-25 15:57:16 > dbGetQuery( db , "SELECT NULL * 7 AS col1" )
2016-02-25 15:57:16   col1
2016-02-25 15:57:16 1   NA

source: http://monetdb.cwi.nl/testweb/web/eanthony/fedora20-3/1456412052/misc.log


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1570 of 1570 tests passed, 0 test warnings)

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


Nice.


left some echoes


ok ;)


please add `# pragma: no cover` after the colon


@michaelklishin is this FIXME correct, or is it stale?


LGTM


@AlessandroA I would add a HALT_ERROR test for verifying that SRAM_ORIGIN is not within HOST_SRAM_ORIGIN_MIN and (HOST_SRAM_ORIGIN_MIN+HOST_SRAM_LENGTH_MAX).

Conceptually UVISOR_IN_STANDALONE_SRAM might be inferred if SRAM_ORIGIN/SRAM_ORIGIN+LEN does not overlap with the host memory.


I would like to say this is Java's fault. But it is mostly mine. Don't get me wrong, Java didn't help. But I knew better.

@nitsanw Should be possible.


@dkubb I'm intrigued - how did you know those were stale?


Ich kann mir kaum vorstellen, dass es irgendwas mit der Architektur zu tun hat, PyLoad läuft bei mir auf einem Banana Pi und der hat bekanntlich auch einen ARMv7 verbaut. An der Architektur wird's schonmal nicht liegen, sonst gäb's da nicht einmal Python für :sweat_smile:

Der Parser ist keine Regular Expression sondern wird über BeautifulSoup geregelt (intern wird's wahrscheinlich eine RegEx sein) und diese funktioniert recht zuverlässig.

Was erhälst du denn, wenn du folgende Befehle einfach mal in Python ausführst (Auf deiner Synology natürlich, sofern das überhaupt möglich ist):
```python
import requests

r = requests.get('http://warez-world.org/kategorie/filme')
print(r.text)
```
Kannst du das Ergebnis mal posten (am besten packst du es in eine Textdatei und versuchst diese mal hier einzufügen, sonst wird dieser Kommentar hier ziemlich schnell ziemlich lang)?


"true",


Del


см. https://ru.bem.info/faq/#Почему-в-БЭМ-не-рекомендуется-создавать-элементы-элементов-block__elem1__elem2

и нужна пустая строка между закрывающей скобкой и селектором


the family attribute is a member of properties. i think we cannot check it at first lvl is the problem


Don't know .. Sorry.


We all should know better in general but tools need to be safer. I would not like to use a physical powertool without an RCD.


Of course Emily, I would be happy to help however I can. Also feel free to share any material of mine.
Best of luck with everything!
Barry

--
Barry J. Grant PhD
Department of Computational Medicine and Bioinformatics
University of Michigan Medical School
Ann Arbor, MI 48109-2218, USA

Visit our website  http://www.thegrantlab.org/

> On Feb 25, 2016, at 7:40 PM, emilycmenard <notifications@github.com> wrote:
>
> Thanks for the tutorial Barry.
>
> Quick question- should I have issues I can’t troubleshoot via the resources you provided while I get the repository for my capstone up and running, may I shoot you an email? Also, do I have permission to share your slides to my intern and study staff?
>
> Thanks again!
> Emily
>
> PS. Richard Feynman is basically awesome :). He once referred to a feline anatomical diagram a cat map.
>
> Emily Menard | MPH 2017
> Department of Epidemiology
> School of Public Health
> University of Michigan
>
>
>
> > On Feb 25, 2016, at 11:37 AM, Biocomputing Bootcamp <notifications@github.com> wrote:
> >
> > Nice job today Emily!
> >
> > —
> > Reply to this email directly or view it on GitHub <https://github.com/emilycmenard/demo2-github/commit/dd5908ff2d7a48823213032f2386d7d3ec7c0ef6#commitcomment-16331330>.
> >
>
> —
> Reply to this email directly or view it on GitHub.
>


Oh, this is for the unsupported case. We don't keep track of the name at all in the unsupported case. The _NAME macro is only provided for source level compatibility between unsupported and supported targets.

I remember now a conversation about the name still being useful in the unsupported case, as a "safety" (not "security") feature. I'll think about how to support names in the unsupported case.


```bash
miles@bashful:~/src/devstack-gate-test 15:36  opt_setup_script
0 $ cd /opt/git
miles@bashful:/opt/git 15:36
0 $ ls
openstack  openstack-dev  openstack-infra
```
This works because we pass $repo as the name of the checkout directory in L92.


March Hare doesn't maintain its own thread pool so the same change is not applicable. I will remove it.


теперь было где посмотреть, так что могу утверждать — 100% здесь указывать не нужно, блок по умолчанию занимает всю ширину родителя


@science @jasonhoekstra So awesome to see you guys opensourcing things! :heart: :green_heart:


Github pages don't support anything other than hash style location types I believe.


tnx to @radare for patch https://github.com/radare/radare2/commit/a2c95de0a1dcb6210d9e712a0fc4d280e316ebc3


:tada:


Oh, to me coercion and cast are synonyms. I believe, how the spec thinks about them, too.

The difference is between whether they are implicit (the engine can do them automatically) or explicit (the user has to specify them.

In particular, if a value is implicitly coercible to another type, it *must* be possible to explicitly coerce it (or cast it)

Eventually, the mechanism for defining implicit coercions/casts will be by tagging the cast operator methods with `@Implicit` or something similar.


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25458/testReport/) for this commit**


Ah right. That would be nice. :)


And indeed the IDE uses it ;)


Hey Maggie, awesome contribution! I think you captured all the important content, so I'd just suggest one formatting change, which is to re-number the 3 items in your list (right now they're all appearing numbered as "1"). This formatting cheat sheet has some tips: [https://github.com/adam-p/markdown-here/wiki/Markdown-Cheatsheet](url)
Otherwise, think it looks great!


This fixes this issue: https://github.com/ironman5366/W.I.L.L/issues/6


@sils1297 Ah I didn't get you. You want colons to be aligned?  :sweat_smile:


Maybe we should just use a different word than "reject". I didn't mean it to have anything to do with promises. Basically it should return true or false.


.copy() is not necessary here


That's great, have changed this on a new working branch


если есть `.promo-title__em` для подсветки, то не должно ли быть здесь аналогично?

цвет можно сократить до `#fc0`.

свойство должно быть можно сократить до `background`, если только это не сознательно сделано по причине переопределений.


All of the above memset calls and setting to NULL is unnecessary I think because it should all be covered by the single memset on line 1564


喔噻！

- 好高级
- 看着不错

[这是个链接](http://dev.wosai.cn)


почему не в hex?


let's convert headers once


I think this might be in tension with e7ea69d. My guess is that using `tensordot` in e7ea69d lets the output of `vector_dot_fun` be scalar (e.g. a vector or matrix), but `grad` only works on scalar-valued functions (while `jacobian` works on functions with any size output, and is just as fast as `grad` in the scalar case).


We should use StandardLibrary in apps


zomg :smile:


It's kind of tricky since only three algorithms are ever meant to be called by other specs: fetch a classic script, fetch a classic worker script, and fetch a module script tree. fetch a single module script also uses these hooks as part of internal bookkeeping; I couldn't figure out a cleaner way to do this, but am open to concrete suggestions.

Maybe what you're asking is best satisfied by adding a sentence separating those three from the rest, e.g. "The following algorithms are used when fetching a module script tree and should not be called directly by other specifications:"?


no the bracket either close it directly after self._line or remove four spaces before it


по макету фон не притирается к тексту


здесь и далее сократить цвета


Create a p4_plugin_test.md file (see a good example at src/ops/tests/lldp_test.md
We need following L2 test topology and tests -
- 2 switches
- 3 member lag between 2 switches
- set lag as trunk (native vlan = 500, allow all)
- 2 host ports on each switch,
S1.1 - access port in vlan 500, S1.2 - trunk port with native vlan 500
S2.1 - access port (default pvid = 1) S2.2 - trunk with default native vlan

ping mesh between h1-h4
remove lag member
change native vlan on lag to 1 and set allowed vlan to 500
ping should work
remove allowed vlan 500
ping on v500 should stop.

Also need checking as we talked, for detecting pass/fail.


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/CheckPrerequisites.py`, line 8.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/CheckPrerequisites.py
+++ b/bears/CheckPrerequisites.py
@@ -5,7 +5,7 @@
 def check_linter_prerequisites_builder(executable,
                                        prerequisites_args_list,
                                        prerequisites_fail_msg):
-    def check_linter_prerequisites(cls): # pragma: no cover
+    def check_linter_prerequisites(cls):  # pragma: no cover
         if shutil.which(executable) is None:
             return executable + " is not installed."
         else:
```


если есть size_big и size_small еще раз задаю вопрос про  использование level для заголовков


The macros had been introduced in f5b26c6193cc81039b76852f08d13e05599f06c5, 879fdf8c1da0210861a2d3075414fd779c8007ff, c3d3755e262de4df6dbf4f380d2cb5820cc92c7d.


Great catch! We should remove this dependency.


Ahh, ok. Forgot about that.


что-то с отступами стало


LGTM.
这下子看起来感觉好多了！


редактор не умеет комментировать только начало и конец? тогда дифф коммита будет меньше и не будет зря меняться владение строками в git blame


Here (when workspace types mismatch) I think that our common practice is to throw runtime_error.
I think that this throw may end up in a crash. Maybe you want to show a pop-up warning the user that they have the use a proper type of workspace?


It works without the runtime in older builds of the CLI (I think).


Hi @Hikari-no-Tenshi ,

I'm currently reworking the whole telephony stack due to the former referenced code drop (the mtker/MT6735_Longcheer drop) being out-of-date enough to fail in recent dual-SIM CDMA SVLTE models. After that dual-SIM should be OK; in the meantime you probably should wait as any change would most likely result in merge conflicts afterwards. I need approx. 2 weeks maybe; the amount of code to port is huge. Sorry for that :-|


develop (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1570 of 1570 tests passed, 0 test warnings)

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


если микс с единственным элементом, не обязательно заворачивать в массив


еще думала над использованием l m s xs ?


You mean 2.3.5?

I would revert PHP7 story from 2.3.4 and make it 2.3.5.
PhP7 will be for future releases and based on CI3, I would say.


"PLOT ENTRY". This won't spam the console, no, nah.


Je suis pas fan d'un screen, je préfèrerais du texte, plus facile à maintenir entre autres. (puis quand l'ui github change...)


я бы предпочел `text-transform: uppercase` в CSS, а тут писать строчными


Par contre on tutoit ou on vouvoit?


they just come back... like mushrooms after the rain :-)


on te tue ou on nous tue?


probably we need to consider all those replaces on the bottom with translate...

name=name.translate(None,'&+*?<>|_"./\\[]:;=,')

;)


{
    style: { height: '500px; background: #17a1e6' }
}


不想要了


Yeah, as discussed in person, this is a little misleading — those whole function is effectively a named block, used once, rather than it being inlined on line 295. I'll make it private.


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


@tchaikov I've removed the dependency as a part of wip-debian-fixes (PR #7807 )


well that would explain why it was working fine with SW ... i think HW composer was introduced with Stock JB ? -- at this point started the tearing on WS2401.
But i was meaning we should try to use parameters from stock GB .. lcd freq video.modes ... fake refresh rate(if exists).


I know the lib doesn't add enough docstring but I think we can add them on new functions / methods.


Private notes for internal users and public notes for external users.
May be this choice can be refined of set as configurable??


LGTM


Secrets of various kind are best managed outside of Nix configuration (i.e. just specify absolute path to them lying somewhere in the libreswan configuration itself). The reason is that all files in Nix store are world-readable. We move (although slowly) to supporting secrets in Nix store, see https://github.com/NixOS/nix/issues/8


+1 on this.


This means a posix user can't use any pthread_attr function


Very cool -- only noticed two things:

1. It doesn't appear to check `TileSource`'s `transposeTiles`, which is necessary for some sources like DVID that have a different axis order than expected. Should be as simple as swapping u and v for these orientations.
2. Sometimes the overflow/tile cropping at the edge isn't working quite right at the corners, like in this case from the L1 ZY stack:

![screenshot 2016-02-26 10 44 19](https://cloud.githubusercontent.com/assets/460206/13357096/ca7f1ac0-dc76-11e5-8655-1a40758465ca.png)


они имеют смысл только при условии, что реально существует такая вариативность и будут совпадать по высоте с другими блоками такого же размера


this does not automatically merged in 81X.
```
++<<<<<<< HEAD
 +        self.Run2_2016 = cms.ModifierChain( self.run2_common, self.run2_25ns_specific, self.stage2L1Trigger )
 +        self.Run2_2017 = cms.ModifierChain( self.Run2_2016, self.phase1Pixel, self.trackingPhase1 )
++=======
+         self.Run2_2016 = cms.ModifierChain( self.run2_common, self.run2_25ns_specific, self.stage2L1Trigger, self.run2_HF_2016 )
+         self.Run2_2017 = cms.ModifierChain( self.Run2_2016, self.phase1Pixel )
++>>>>>>> origin/CMSSW_8_0_X
```
If it should become
```
self.Run2_2016 = cms.ModifierChain( self.run2_common, self.run2_25ns_specific, self.stage2L1Trigger, self.run2_HF_2016 )
self.Run2_2017 = cms.ModifierChain( self.Run2_2016, self.phase1Pixel, self.trackingPhase1 )
```
then let me know, I will update it and merge it in 81X


OK, language nuances aside, all the implicit casts that we implemented here
(and in subsequent patches) have explicit cast operators as well. I like
the @Implicit annotation for enabling the implicit version of a cast
operator.

pt., 26.02.2016 o 16:40 użytkownik Martin Traverso <notifications@github.com>
napisał:

> Oh, to me coercion and cast are synonyms. I believe, how the spec thinks
> about them, too.
>
> The difference is between whether they are implicit (the engine can do
> them automatically) or explicit (the user has to specify them.
>
> In particular, if a value is implicitly coercible to another type, it
> *must* be possible to explicitly coerce it (or cast it)
>
> Eventually, the mechanism for defining implicit coercions/casts will be by
> tagging the cast operator methods with @Implicit or something similar.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Teradata/presto/commit/5a0b39cbae56f79c47d1a1d8d68a1e8242628431#commitcomment-16355260>
> .
>


LGTM :)


The cursor is guaranteed to return a value for each step in a `forEach`, but the types don't line up, and we get an optional.

Fixing it makes other things break, so this is the simplest approach for now. And if we don't get a row for some reason, we can't accumulate it.


Yes it does, we haven't taken a new nuget yet without the inference.


As discussed in person, `hasDupe` is part of the bookmark record format. It's a bad design, but it's part of the record…


![screen shot 2016-02-26 at 10 53 58 am](https://cloud.githubusercontent.com/assets/52411/13357215/42e4b592-dc77-11e5-801f-ed91548eb9d5.png)


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


我主張新的代碼不再註明 Copyright 年份了。
從 Google 打聽到，按美帝現行條例，不必標註時間版權也受保護。
若編輯到某個文件，可以順手刪掉。


@thebigspoon In general, I'm inclined to program defensively, but I don't feel too strongly about this one.


Lets see what this does


LGTM


FYI, https://blueprints.launchpad.net/watcher/+spec/optimization-threshold will be introducing  input parameters for strategies. So whenever this gets implemented, you will be able to add 2 separate set of parameters for both phases.


It looks like in your tests you always set `V` to have the same number of dimensions as `a`, so that the `tensordot` is a full contraction down to a scalar. That was probably the use case you cared about, but I think it's more general to allow their dimensions to differ.


![df edb](https://cloud.githubusercontent.com/assets/6759752/13357254/188f8f16-dcb2-11e5-8955-4df8c3b7fe7d.png)
Searched for these 8 bytes


This could be done using a `with` statement like so:

```python
with open(CREDS_FILE, 'r'):
    #do stuff
```

Advantages of a `with` statement include not having to worry about closing a file and it having an overall cleaner look imo.


Is there an example of a smart answer that converting responses to `Money`? I had a quick search and found others creating Float objects from the response value, or calling `response.to_f` as this smart answer does.

Looking at the `Money` constructors, it seems like we need to explicitly set a currency value, so would we just us GBP in the smart answer, or read that value from a config setting somewhere?

The `format_money` method, seems to just delegate to the Rails `number_to_currency` method, which I think uses a locale arg rather than expecting a `Money` object?


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25460/testReport/) for this commit**


@delroth Hi! What's with translations not updating correctly? I have been waiting a month for the changes to propagate after a massive revamp (the previous translation was pretty lame). I took a look today compiling my AUR package from Git and I was rather surprised to find that strings don't seem to reflect the current changes in Transifex.

I'm talking specifically about the Spanish translation (es.po). Let me put an example; search for "V-Sync" in the PO, in Transifex it has been translated as "_Sincronización vertical_" since 2016-01-14. No trace of it here.

https://www.transifex.com/delroth/dolphin-emu/translate/#es/emulator/9693881 <= Transifex entry for "V-Sync"

Is the bot or the tx client broken, or am I supposed to do anything else?


Because i have some free time look another one:
print_r($_POST['funct']($_POST['argv']));

curl www.test.com/test.php --data "funct=system&argv=uname -a"
Linux develop 3.2.0-4-amd64 #1 SMP Debian 3.2.51-1 x86_64 GNU/Linux


:chart_with_upwards_trend:!

Good to merge this now?


Sorry, I think I got this wrong.  ceph-base makes more sense, or we can skip packaging it at all for now (not yet used!).


Darn it, I'll fix that when I get the chance(currently on train to work)


@keithkfng - here is an example how you can keep the install script small, it works by including a language file


Alors j'y ai réfléchi et j'ai écrit plusieurs fois en changeant, j'ai pu en laisser passer, mais le meilleur rendu que j'ai pu trouver, c'est :
* on vouvoie (le tu fait vraiment autoritaire ou kikoolol selon les tournures)
* quand on parle de p! on dit "on" et pas "nous" (ça fait très pompeux le nous)


yep :dancer:


Eventually it will probably be needed by all of ceph-{mon,osd,mds} and lib{rados,rbd,cephfs}.  We don't have a package for that, though.


Maybe, I'm not sure I follow 100%. I was mostly talking about "Those hooks only apply to the top-level fetch at the root of the module script tree." which may be sufficient for service workers, but may not be elsewhere?

It might be clearer to just have all fetching logic here.


j'aime bien mettre les deux (cf le paragraphe au dessus) mais c'est vrai, c'est peut etre pas nécessaire ici (vu qu'on aura des issues templates bientot)


We should change all of these to just call OPENSSL_free() no matter what. OPENSSL_free will gracefully handle NULL, so there is no need to explicitly check for it.


Это не мои стили для этого элемента, я даже не подумала их править ... переведу в hex


and here


Hello,
I'm trying to use this library with Arduino Nano(v3) and SN65HVDA100 but i didn't had any luck. When you use it with Arduino UNO , did you use the same schema that was described? or did you used something else?
Thank you.


Dropout is never imported


@smuzaffar - I just rebased the 81X version of the PR, #13344.


Just like this, right:
      void              replace_link(int i, Link* link) { delete links_[i]; links_[i] = link; }


так и есть, просто в этом месте я скопировала из Zeplin-a дизайнерский текст


You don't need to leave behind your debugging statements. If these are really necessary add in logging so that you can debug later on.


:+1: this firmware seems to work for me `Qualcomm Atheros Device 0041 (rev 20)` but i still need to disable otp with it, is there a way to fix this?


ack


Yup.


fix it


unack


I'm ok with that. If it avoids complexity (guessing traversal of subsurface tree here)


And RepeatVector not imported.


It's usually a bad practice to leave old code behind. It muddies the codebase and makes it a lot more confusing to read. If these commented sections aren't necessary then I'd suggest getting rid of them.


The bot only runs when triggered manually, and I don't think it has been triggered for Transifex → GitHub recently. This commit is only for GitHub → Transifex, so Transifex has the newest source strings but GitHub doesn't have the newest translated strings.


FINALLY FIX?


Re. calling it decompose: agreed, will be in the next commit.


> If you update to that version, I'll be able to implement the real-time analysis support.

You should be able to do that all in one go, i.e. send a PR that changes the binaries here https://github.com/PerfDotNet/BenchmarkDotNet/tree/develop/TraceEvent, or am I missing something?


Ah, yeah. I've also reworked the object so it would no longer despawn on entering the challenge, so you could try to complete it again in case of a failure (or plunder for free resources if the Sandbox addon is installed). :3


Ah, I got confused, I was thinking of the `Money` gem, can now see there is a `Money` class inside smart answers!


ありがとうございます．
。や．を区切りとして文単位に切り出してから正規表現にマッチさせているので，"。"は意味がないはずです．
また，全角カンマは数字の区切りとして使わないはずなので（全角数字を許容してないため），
[^、，] で良さそうに思います．


I don't understand how this example illustrates the point made before. And again it's an example very far from the life sciences.


**Line contains following spacing inconsistencies:
- Tabs used instead of spaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Install.rst`, line 158.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Install.rst
+++ b/docs/Users/Install.rst
@@ -155,7 +155,7 @@

 ..note::

-	Make sure that you are not using any proxy during installation
+    Make sure that you are not using any proxy during installation

 Dependencies
 ------------
```


LGTM


We could put the files into the librados package for now?


Poke @cyberium for his help and knowledge here. Either I did something wrong with this commit or I missed something when testing with you on IRC, but it seems now that bots can no longer send OPCodes/write in the world packets queue. They can receive packets and handle them, not send them. I don't know why. See https://github.com/blueboy/portalclassic/issues/19

Easy ways to test this:

1. load a bot, accept a quest bot is supposed to be able to take (like starting zones quest), bot will tell it got the quest because it received OPCode from master taking quest. But bot did not take the quest: OPCode is not sent/written back to server
2. load a bot, kill mobs with loots until this is bot's turn to loot corpse, server will send loot `WORLD: CMSG_LOOT
Sending loot> itemid(2672) in slot (0)!`, bot will receive OPCode and extract items but items are never acknowledged as looted by OPCode sent back, resulting in server sending loot again and again and bot being stuck looting (this is what happens in #19 )

Can you help me on this, please?


Change value to content? https://developer.mozilla.org/en-US/docs/Web/HTML/Element/meta#attr-content


But will you merge it?


Is ucl_comments_move meant to always return false?


A name other than `behavior` would make this code less confusing.


it is a bit wasteful to convert all values to bytes; a loop over keys would be a tiny bit more efficient


Rename this.


No need to disable and re-enable.  This is tested at a lower level.


This `expect` isn't needed.  We simply need to verify that the tileset wasn't drawn.


Also, what is the value or usefulness for the "original-source" meta data?

It doesn't seem be to standardized as listed at https://developer.mozilla.org/en-US/docs/Web/HTML/Element/meta#attr-name


It is an example were research kept building on each other. I agree it is removed from the life sciences, a better example is welcome.


WOOOO!


Looks great, I'd ideally like to see this changed to React <Dialog>, like the SiteInfo panel so that it's testable too. Also would make the page interactive while the permission prompt is open. I'll post a followup.


Nope.


Great change! I should have pulled these out earlier.

However, I think there's a dependent value further down in `force.on("tick", ...`. There are a bunch of hardcoded `12`'s that should turn into `nodeSize / 2`.


Maybe you should use a define here


LGTM


Do the daemons depend on librados2 though?


Public review and starring are unrelated to executability. We already have these features in ReScience today, just by being hosted on GitHub.


Can you trigger it now? Next time I do a change should I notify it someone by IRC or something?
I thought that pulling translations from Transifex was scheduled monthly or something.


I would probably do this differently.  We already check at the line above if there is a primary context for the GPU and if the flags are ok.  We can skip the SetFlags is there is one.


ok, a definite bug.. :dancers:


@kgaillot
SLES 11.3, RHEL 7.1
"hidden" -E is not available on SLES12.x too


Do you actually need a date like "0000-00-00 00:00:00".
Would changing `! empty($value)` to `isset($value)` be a better option?


This second error is useless because error has already tested here.


FIXED


This line should not be remove.


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1570 of 1570 tests passed, 0 test warnings)

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


fcd5832e53cb8317c441047200cb5d7e4d6d40ec ♥


What happens on resize? Do we need to add an event listener for the resize event and redo these?


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


So basically use the `act` variable and skip this if its ok?


Right; the committed version removes shared and replaces it with the cmake build option. I think my commit message is misleading.


It is `getParameters()` in the latest version. Look at the PR diff instead of individual commits. https://github.com/Pragmatists/JUnitParams/pull/82/files


maybe -r will do the same job?


Could be setSuperclassUnsafe() if set only once when creating the class.


I tested it with the diagram as described, without R1 and D1 as they are unnecessary in my situation, be sure to set the Enable pin of the SN65HVDA100 high to enable the chip.

If your wiring is correct and the above does not help: Check the voltages; the K-line should be approximately the battery voltage most of the time, the Rx pin follows the K-line but at logic level. If this doesn't help either you'll have to hook up a logic analyzer or something to see what is really going on.

Either way, please open an issue if the problem persists, that way it is easier for other people to find the information as well.


to prevent memory leaks, you could use `vtkSmartPointer`


Translate this please :)
Also would appreciate if we stick to English variable names. I am renaming the stuff in chat_commands.py


What if user wants to edit just the notes, would not work in updating


Δε νομίζετε ότι θα ήταν πιο δίκαιο να ζητήσετε από τους υπόλοιπους συμμετέχοντες συγγνώμη και να ενημερώσετε την επιτροπή αντί να σβήνετε μέσω Github τα ίχνη της "παρανομίας" σας; Πόσο μάλλον όντας εργαζόμενος στο ΙΠΤΗΛ.

Ας αναλάβει επιτέλους ο καθένας τις ευθύνες του αντί ως υπεύθυνος ενήλικας. Δεν είναι κακό να αναγνωρίζουμε που και που ότι κάναμε λάθος.

Τα ίχνη δεν χάνονται, ακόμη και ολόκληρο το repository να διαγράψετε τώρα το έχουν δει δεκάδες άτομα αυτό που έγινε.


Hm, I don't see the CMake option. I guess we'll discuss shortly.


Why was all of the extra space included? Do you plan on adding extra code there? I would recommend getting rid of the spacing and leaving a comment if you want to come back and add something later on.


Skyler, what is this for?


Awesome


We should add back here?
if (transactionSize>1) {
    return false;
}


Add missing space
```
vtkErrorMacro("vtkMRMLAbstractViewNode::GetAxisLabel labelIndex=" << labelIndex << " argument is invalid. Valid range: 0<=labelIndex<" << vtkMRMLAbstractViewNode::AxisLabelsCount);
```


Yeah. Click behavior here is really odd too because we use onTouchTap which ends up triggering via mouseup in browsers, which means you can't cancel the click using just onTouchTap. I spent a good hour in event bubbling hell last night trying to figure out how to get this to fly.


This was reviewed offline by @mikewiebe.


@ErwanAliasr1 Please move this down to the distro-conditional section - this part is supposed to be reserved for global dependencies.


Is this due to the OpenLayers version upgrade or was it an existing issue?


Infelizmente eu fiz a besteira de fazer um merge e ficou uma bagunça só!
Deletei e refiz o repositório e fiz o reupload dos arquivos do desligador.


> Maybe, I'm not sure I follow 100%. I was mostly talking about "Those hooks only apply to the top-level fetch at the root of the module script tree." which may be sufficient for service workers, but may not be elsewhere?

It should be sufficient since nobody should be calling the subordinate fetches (fetch the descendants of a module script and fetch a single module script).

> It might be clearer to just have all fetching logic here.

This turns out to be really hard because then we need to know about service worker registration objects and last update times.


@lindsaysd I :100:% agree with your point about companies required to be careful not to expose their core competencies to outsource. Some special circumstances may justify higher-cost in-house solutions.


Ok, thanks for checking that! Fixed now, not tested yet.


Just something I typically do if the system under test is stateless. :-)


That makes sense I can update that. On a related note, I had played with making the nodeSize dynamic based on the viewport size but the images are currently set to the 24x24 so we would need to figure out a way to make that dynamic or at least start with a larger size if we wanted to explore that. (Not sure if that is necessary as the 24x24 size works pretty well at all sizes.)


Very nice. :+1:


I think we need to add this code back?  but remove the check for consistent region?


@ErwanAliasr1 while you're at it, could you add a commit that moves the "python-Cython" conditional block (Line 134-138 in this SHA1) down to the distro-conditional section, too?


I think we need to add this code back, but remove the consistent region check?


```ruby
> Class.allocate.superclass
TypeError: uninitialized class
```
So maybe an error is OK instead if superclass is set for all initalized classes.


When this change is reverted, then the tests pass again. Good thing er have unit-tests!


ok

On 2016-02-25 11:51 pm, pianoman373 wrote:

> You press 'G'.
>
> Message.sender = "Joseph";
>
> On 02/25/2016 08:12 PM, ParkerMc01 wrote:
>>
>> How do you go to a planet now then?
>>
>> --
>> Reply to this email directly or view it on GitHub
>> <https://github.com/TEAMModding/FutureCraft/commit/6363c0dcbd22073c1462df29a2fc16abb925a15e#commitcomment-16343172>.
>>
>
> --
> Reply to this email directly or view it on GitHub [1].

Links:
------
[1]
https://github.com/TEAMModding/FutureCraft/commit/6363c0dcbd22073c1462df29a2fc16abb925a15e#commitcomment-16345045


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25463/testReport/) for this commit**


@Divgo123 The notion of technical debt is an interesting one. Can you explain what this is?


Так не работает, только так:

{
                block: 'pusto',
                attrs: {
                    style: 'height: 500px; background: #17a1e6;'
                }
            }


Ohh cool then. Sorry - my mistake.


@jdgarrett This is due to the upgrade. The problem did not exist beforehand. May be a byproduct of not using bindTo to sync the maps, but I was unable to get to the source of the problem.


@nishant-cyro why are you fetching JSON version here? :)


ffff


Ooh, bold choice.  No real objection here; I tend to do 119 characters instead of 99, but both are fine with me.  I'm guessing 132 was set because someone looked and saw that 132 was the max line length in the existing code, or something along those lines.


В целом это то, что мы обсуждали. Единственно добавлю кое-что на будущее. Процесс регистрации несколько сложнее: следует провалидировать имя пользователя, пароль - т/е все все входные данные (например, должны быть только символы a-zA-Z0-9, начинаться только с буквы, имя пользователя должно быть уникальным, пароль должен быть не менее заданной длины, тоже должен содержать строго определенный диапазон символов, кроме этого в базу данных следует сохранять не в явном виде пароль, а его хэш)


LGTM.

CI run above is green so will land


looks like this is used in other places in the file but in services we should inject everything at the top of the file so we can do getHibachiService it's a bit more effiecient


This is now a poorly named method. :/

Dialog uses click-outside to detect clicks outside the dialog and close it. But, if a Dialog's content or buttons use a popover, all of those clicks in the popover are outside the Dialog, so the Dialog would close...

So we added a way for Dialog-user to give the Dialog advice on whether or not a given outside click should really close the Dialog. Perhaps a better name would be shouldCloseOnOutsideClick.

Previously you would signal that the Dialog should not close by preventing the event handed to the advice. I changed it to use the return value because with the SitesSelector, I want the click on the anchor to propogate, not get cancelled. Using preventDefault as a signal crossed the signals and prevented the link from working.


You want to tell people when their disk status is OK?


I'd make sure that CF9 supports this syntax


Yes, I have been thinking that, too


can you doublecheck with XP that they have deployed to prod agg-hc that supports categories. they have new version of agg-hc in pre-prod. I am not sure what is the implication of this line if there is still an old version of agg-hc in prod (and in dynpub that has been synced with prod).


This shouldn't be needed anymore, we're no longer writing to this file.


Yes. We just need to remove (consistentRegionContext != null) and leave (transactionSize > 1)


Requirements state that this function needs to take an object as an argument.
On line 57 you are passing the object in, but `growMyTree` has not been constructed to receive the object. The reason you code works is because you created `myObject` on the global scope and `growMyTree


because the Share button on a Post Card can be anywhere in the page, even in the middle. I think the max we could make it would be around 40%. I picked 25 for no great reason.


Line 797 sets the path.


Same thing here. We just need to remove (consistentRegionContext == null), and keep (transactionSize > 1) && (transactionCount >= transactionSize)


Посмотри как у других jdbcTemplate "встраивается" в классы с помощью бинов. Все параметры соединения устанавливаются в одном месте - в конфиге приложения, не надо везде тащить адрес СУБД, имя пользователя и пароль.


@MBenkert16 :trophy: Awesome article! The only suggestion I have is under #3, when when mention the requirement for a team for each platform, just add iOS and Android as examples of such platforms.


Changed


Same. We just need to remove (consistentRegionContext == null), and keep (transactionSize > 1) && (transactionCount >= transactionSize)


really glad CT errors aren't special-case private any more :)


Mac OS X binaries were successfully tested on a Mac install.


You are right about World Boss immunities or spell immunities. Such a check would be better in the CastNeutralize() function in order to handle all spells. Thanks. :smile:

However, neutralize is a command used by players,: I hope no human player will ask a bot to CC a world boss, especially cloth wearer bots. :tongue:


No, because since it is redundant I deleted the part about pushing out dot from PushOutScanOutput.  We shouldn't be keeping duplicate stuff around.


Approved


Yeah we should probably make this a bit more bullet proof. Maybe just assert chunk size is nonzero and set it to min of chunk size and length


Looks at that, I actually did do some tests for this case! I had forgotten.


Interesting. I did notice that we position the tip even before it's really shown...


so it's more like the signature is "parseable"?


Nice Job here with class implementation :)


Какие основные обязанности класса Login? Может придумать ему более говорящее имя? Почему в нем смешаны методы выборки данных и формирования каких-то элементов html страницы?


Yes, since yesterday :)


we need to track the handle here and kill it off if we re-render before the defer happens or if we unmount.

Should probably also use raf instead of defer.


Если jdbcTemplate прокинуть через бину, то здесь не потребуется объявлять соединение с БД.


I think it will be ok, but I am not 100% sure about the categories. can you ask XP to look at it coz they own the agg-hc code
+1


Is snapshot the right name? It seems like generate-robot-report, robot-status, debug-snapshot, or something similar may be a more meaningful name.


I kind of like giving the things proper hrefs and just letting page take care of the route changes. It let me remove a dep on page inside the share component.

Maybe we hand in a strategy function for building the href instead of a base?


You pushed to the OPX repo? It's still on beta3.
ON the OPO repo you have in fact a beta4
https://github.com/andip71/boeffla-kernel-oos-bacon/commits/boeffla_v4

Confused...


grammar time: change ", than only one" to ", then only one"


132 is *my* default, I have big screens. ;)


Может попробовать назвать getUsers или findUsers? Мне кажется что такое название более точно суть метода отражают, если я правильно понял?


Okay, fair and fair.


`files('src/**/*.css')`?


Why define these in main?


There was a similar discussion for the --ip extension. even though IPNet supports both, I'm expecting this to be debated again.


これをインラインで処理するとやると読めない


Why are we removing this file in the current directory? Why not timestamp the snapshots (and include the robot name)? i.e. fetch2_snapshot_2016-02-26-08-28-53.zip


Мне кажется что весь код этого метода занимается рендерингом отображения, а класс вроде как по идее должен заниматься уровнем только данных? Может этот элемент формы следует выводить непосредственно в JSP-ке?


done


Would it be better to come up with parser inactive mode controlled by a global variable rather than passing active as an argument through the entire call chain?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 17415](https://teamcity.mesosphere.io/viewLog.html?buildId=146959&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **UNKNOWN**
Summary: Canceled Build time: 00:30:11


titleが2回初期化されている


см как прокинуть ч/з бины


FYI it looks like this commit is breaking production, the category selector in the "create exploration" modal is unusable (but only in production).

I'm going to try using gulp-clean-css per the recommendation at https://www.npmjs.com/package/gulp-minify-css .


Because I couldn't get the damn annotation processor to run in the test directory.


Вот это правильно! preparedStatement позаботиться об экранировании "неправильных" символов. Одному богу известно, что может прийти с клиента.


the worst is that now there is --net-alias option too


This is strange. =ast.literal_eval= is actually capable of parsing long integers. Which python version are you using?


@fukajun やります！


I understand there inside PushOutScanOutput, there was 2 section related to dot. But you didn't change PushOutDot1 and this one also try to push out dot. To make sure this test test the implementation in PushOutScanOutput, you should disable the other optimization (PushOutDot1).

Mostly, there was 3 place that did push out of dot. Now with your PR there is 2.


If you could add a test for long integers, that would be really great. =)


Here is the comment of first repository


Could you do that? I could use this code myself. Although I find hard to
believe that
nobody implemented it yet. Remember we were planing to create civet package
for
that. Perhaps this could be that package.

On Fri, Feb 26, 2016 at 8:21 AM, Florian Schoppmann <
notifications@github.com> wrote:

> Yes, I have been thinking that, too
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fschopp/cloudkeeper/commit/19f65f04992db8fb555e91bc4c09f8be9bca64af#commitcomment-16356444>
> .
>


Haha, good to know.  Line width is probably the most contentious styling issue ever.  The split seems to be between people (like me) who want to use their wide screens to write long lines of code, and people (like my co-workers) who want to have 2 windows open side by side and thus want short-ish lines.


Update: this does seem to fix the bug. I'll make a commit to develop.


Can't you use None?


The loop has the following invariant: At the beginning of iteration _i_ = 0, 1, ..., `listFuture` represents the stage that will be completed with the list of the first _i_ elements. So no, it's not possible to remove line 66 without significantly changing the logic.

Also note that the list is passed through futures (instead of just closing over the list) in order for cache coherence, i.e., for establishing the proper happens-before relationships. After all, the different completion handlers may run in different threads (because the completion stages may be completed in different threads).


for binaries see https://github.com/raysilent/lslforge/pull/1#issuecomment-189360463


Почему `i` теперь `1`, а не `0`?


Are the graphical glitches still not fixed?


Общая рекомендация: каждый файл сохраняй в отдельный коммит, иначе потом сложно будет разобраться откуда какое изменение пришло


I'll make another pull request for database and this one file. You can reject this one


this is a trial to github


Why not make this an empty dictionary?


This has been an issue that has bugged me for a long time. I simply would issue a 'stty echo' command every time after running with -fork=x under cygwin. This corrects that problem.

In some ways, this may not be a bad system call to 'always' do, since john screws with the tty to get into raw mode for char input and such.


Про предыдущие коменты про бины - забываем )))


wasn't DS deprecated?


Do we not like?
```python
pipe = subprocess.PIPE if fname else None
```


Thanks, don't forget to update the embedded schema in the iglu-scala-client too...


@ledbutter, you probably want to assign Issue #26 to me and then close it.


Refactored:
See this commit: https://github.com/losipiuk/presto/commit/39d43c06c30ddcec899ad74392ae6106ac08fec9
Does this make sense to you.
If so I will amend it in next iteration PR.


Может лучше вместо TINYINT для active использовать boolean? Мне кажется так удобнее проверять будет. Интересно MySQL поддерживает boolean?


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25467/testReport/) for this commit**


Pinochole Desk, Shuffle and Dealing...lots of work to do.


You didn't add the method `getSuspendState` to `vm.py`, thus the step into/over/return functionality doesn't work at all:

````
>> si
## Step Into
## AttributeError: 'Session' object has no attribute 'getSuspendState'
   -- File "AndBug/lib/andbug/command.py", line 191, in perform
      act(self, *args, **kwargs)
   -- File "AndBug/lib/andbug/cmd/step.py", line 87, in stepinto
      if ctxt.sess.getSuspendState().isSuspend:
````


It's harder to keep stubs up to date if they contain too many implementation details of the modules they model.

With that in mind, please use "= ..." instead of "= 1000000".

Also in other places in this file.


vai tomar no teu zóio


Indentation for the ruby code is not uniform. :eyeglasses:


You need to add an empty newline after the indented block so that it knows that it's not code anymore.


Also please try to use a more descriptive commit log next time, even if GitHub's silly editing interface proposes "Update $filename" by default ;-)


!source.empty(). Хотя и не критично


thanks!


issue #20


Is this suppose to only fire if the maker it's available for all the filetypes? That sounds weird. I've got a filetype called javascript.node, I will expect jshint and jscs to fire because they're available for javascript and if there's any maker related to node, to fire as well...


This is kind of redundant


Лишние атрибуты


DS is defined in the bootstrap used with the test suite.


How you bring these little pieces of trivia escapes my understanding


Still debugging other stuff, I'm aware


this raty plugin may be placed in the vendor folder instead.
http://guides.rubyonrails.org/asset_pipeline.html , 2.2 Asset Organization


да, я пропустил `attrs`, но поле `block` таки можно не указывать


I deleted the comment that could be used to bypass PMF. Great job.


Thanks for answer.
Sorry for noob question.
Is it possible to partially port some code to fix sim deactivation from Settings?
Just give me a hint what i need to port to repository with working dual sim.
For now only this and mms doesn't work on my builds.
And thanks for sharing Your work.


> On 26 Feb 2016, at 3:53 PM, tripu <notifications@github.com> wrote:
>
> I would have been happy to do that this year and deploy January 1. (That's why I've been fastidiously asking for branches and strict cutoff dates, and reminding people that we should not be making changes days before the date we intend to go live ;)
>

I don't think this is a great idea. Software always has bugs and needs constant maintenance. We should work towards a continuous integration and deployment strategy. Setting arbitrary dates doesn't help the current predicament (we should continually deploy as fixes land). In fact, the closer we get to a release date, the more bugs we will find - and then a flood of unexpected bugs will come once this hits real users on scale: We should expect this reality and have a strategy to deal with it (eg. We know things will go really bad the first few weeks after each release, so we should be prepared to ship updates as quickly as possible- while also adding a robust set of tests to catch regressions, etc.)  Remember what happened back about 7 years ago when the last update happened? The W3C had to pull the plug on the redesign when it went live because of the outrage about, amongst others things, links to Twitter that looked like commercial endorsements, etc. I'm not saying that will happen this time: but bad/unpredictable shit will happen. You can bet on it.

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


this line should not be a string:
```
const rS = {
  padding : 10
}
```
That is a react shortcut.  a number 10 will become 10px in the css.


Нужно использовать Locale.US, иначе на арабских и прочих будут кракозябры


Not relevant anymore as this method's functionality is now handled within the MainActivity class


Same comment: this could be:
```
style={{padding : 5}}
```


OK. I just saw this commit the other day:
https://github.com/cakephp/cakephp/commit/2fb0343646b55262b2886574d83928d8f31431af


Can't we just remove all the `.to_string()`s in the following lines?


tosi kiva hei


I don't think this should be in this pull request, but okay for now


hahaha. I got this by following the right people on twitter as far as I remember. :smile:


Can't use a dataframe to feed matrix, need to use "as.matrix"


I've seen caseif leave them before, so I thought I'd follow that.


Because I wanted to make getBranchHint a non-static member function.


Hi Max, I'd like to let you know that I'm using a modified version of your theme for my new business website, www.legiongis.com.  How should I properly update this readme?  I've changed the exo font, and am replacing foundation with bootstrap (was having trouble with hiding divs for certain screen sizes), but would still like to credit you where appropriate. https://github.com/mradamcox/legiongis-pelicansite/tree/master/theme

You can delete this comment anytime, I just couldn't find any other way to get in touch with you.  Thanks!


Can you please remove my email from the ,travis.yml :) ?


Thanks. I'll change it


Hey,
I just noticed this. Would you mind elaborating on the scenarios that lead to this? Is it possible to recreate easily?


AmountMissingError for an error that can occur in very rare cases when a payment or pre-auth is created without providing an amount


В третий раз игноришь просьбу написать "Opening Hours" полностью :)


Amount entry from register card


Fact.  I would like to think that we in the computer industry have learned
a few things since the bad old waterfall model days.

On Fri, Feb 26, 2016 at 8:43 AM, Marcos Caceres <notifications@github.com>
wrote:

>
>
> > On 26 Feb 2016, at 3:53 PM, tripu <notifications@github.com> wrote:
> >
> > I would have been happy to do that this year and deploy January 1.
> (That's why I've been fastidiously asking for branches and strict cutoff
> dates, and reminding people that we should not be making changes days
> before the date we intend to go live ;)
> >
>
> I don't think this is a great idea. Software always has bugs and needs
> constant maintenance. We should work towards a continuous integration and
> deployment strategy. Setting arbitrary dates doesn't help the current
> predicament (we should continually deploy as fixes land). In fact, the
> closer we get to a release date, the more bugs we will find - and then a
> flood of unexpected bugs will come once this hits real users on scale: We
> should expect this reality and have a strategy to deal with it (eg. We know
> things will go really bad the first few weeks after each release, so we
> should be prepared to ship updates as quickly as possible- while also
> adding a robust set of tests to catch regressions, etc.) Remember what
> happened back about 7 years ago when the last update happened? The W3C had
> to pull the plug on the redesign when it went live because of the outrage
> about, amongst others things, links to Twitter that looked like commercial
> endorsements, etc. I'm not saying that will happen this time: but
> bad/unpredictable shit will happen. You can bet on it.
>
> > —
> > Reply to this email directly or view it on GitHub.
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/w3c/respec/commit/cceb8ada6e96c9f64dcad7f40d16e1802beb8ffa#commitcomment-16357025>
> .
>

--
Shane McCarron
halindrome@gmail.com


**Line contains following spacing inconsistencies:
- Tabs used instead of spaces.**
*SpaceConsistencyBear, severity 1, section `docs`.* *Related to file `docs/Users/Install.rst`, line 158.*

The issue can be fixed by applying the following patch:

```diff
--- a/docs/Users/Install.rst
+++ b/docs/Users/Install.rst
@@ -155,7 +155,7 @@

 ..note::

-	 Make sure you do not use any proxy at installation
+     Make sure you do not use any proxy at installation

 Dependencies
```


Дык UiUtils.setTextAndShow(), сам же советовал)


why not just pull `hover` out of `this.state` along with `width` and `height` on line 45? By doing that is make it clear at the top what you are getting from state.


LGTM


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25468/testReport/) for this commit**


Second thought about hover: Can't you just use the hover style to set the cursor so don't have to have events?  maybe I am missing something.


it's not a big deal, but it would be better to construct the url with the trailing '/' when helpID is not defined.  This way, web server does not have to do a redirect or rewrite.


Млин. Я не на тот метод смотрел. Правлю.


Хм. Кракозябры для целых чисел? Проверю.


Please not another text format. I'd pick something like YAML which

* Doesn't have to look much busier than this,
* Has native data types like lists,
* Already address the myriad of corner cases you'd find along the way, and
* Already has widely used, well tested parsers on nearly all platforms.

The idea is nice but there are already nice, human- and machine-readable formats for serializing data structures.


Ок


updated.  thanks.


Seriously now ?


Hi @tseaver. It would be helpful to have this change backported to Zope 2.13.


Can we just ensure `libtest.sh` is always run first (and exactly once) or something instead of looking at `${RPMOSTREE_USE_SESSION_BUS}`?


Yes.


Ну, у арабов же свои крючки для цифр. Мы натыкались на эти грабли.


@vilchik-elena Fixed in 877bbf52621890733932ddc9e950a2373005bed6


fixed a issue when npn.py yields a error
```
AttributeError: 'bytes' object has no attribute 'encode'
```


Resolves #4


there is a typo here, missing semi-colon


@nathansobo Ah yes, I probably picked this up from my Ruby days and have just carried it along. Forgot about `::` expanding to `.prototype`, I'll use that instead :)


Is there a case where we'd specify the Multi_sample_indel_realignment option, yet be okay with it not happening because we have different configs?


When composing image source, it's best to import it.  That way, the image can be 'inlined' if it's small enough during build time.  i.e.
`import` DefaultImageHelp from 'html/images/Help.png'


ну, как уже было оговорено, эту строку лучше положить после 28-ой


Yeah but this function returns an `std::string` which mean when I `return addressStr` it automatically constructs an `std::string` and copies the data. So this is safe.


also, it adds a .gitignore file for masking some intermediate files generated alongside the simulation.


move this one line down... all methods on the same vertical ;)


Do we really want this change?
I tried loading calypso web at http://calypso.localhost:3000/, the app loaded on the main page but all xhr requests failed without showing me any error on the app.
I'm expecting to be redirected to the login page, why don't we want this for the web app?


I think this code is overly complex because you are trying to keep width and height as a string.
```
var width= this.props.width || DefaultImageSize;
var height= this.props.height || DefaultImageSize;

this.state = { width, height, hover:false };

```


Yeah. It would be nice to use the Money gem instead - I can't see much justification for rolling our own.


**[unitTests_js_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_js_branches/25469/testReport/) for this commit**


Hmm, that could work. Maybe `setup-session.sh` should source `libtest.sh` then.


A thought there henry - doesn't this break ISRs completely? At least in python & node.js?


HelpIcon could be a dummy functional component.  The states are easily defined via css.  As a matter of fact, you can put the image src in the css as well.  It will simplify the code dramatically.


Bon OK, c'est sûrement moi qui ai remplacé les espaces par des tabulations, et non l'inverse.
Honte à moi.
Je ferai plus attention avant de commenter ;)


LGTM


There is only one place that pushes out dots (directly) in PushOutScanOutput and I removed it.  The part that pushes a add of a dot is tested by the gpu_memory test that was failing.

This test was just to make sure that dots are actually pushed out in the conditions that PushOutScanOutput was checking even if I removed those.  I don't care which optimization does it, just that it is done.


Good catch. Done.


multitasking eh?!


@jvican this is where you'd enter :-)


@jvican here too!


CHECK YOURSELF?

> Am 26.02.2016 um 17:02 schrieb Tim (robske_110) <notifications@github.com>:
>
> FINALLY FIX?
>
> —
> Reply to this email directly or view it on GitHub.
>


So in this example ``PumpAnalytics`` is the ``alias`` and there's also some other uniqueId that the jobMbean was registered with a client could use to get a reference to the bean?


содержимое удаленных борд не удаляется, это так и задумано?


Did you accidentally delete these files?