`div#output` selects the div with the output class (i.e. `<div id="output">`


Yep, I know. But having `|| val === undefined` there is wrong way of handling this problem :)


***Code Quality***
You're not testing your `create` function. Your test should start there right.


There's a lot of duplicate code here.  This block is repeated 3 times so, I'd suggest functioning this section out.

        if '\n' in data:
            rows = data.split('\n')
            if lines_read < max_lines:
                if first_row:
                    guesses = analyze_header(rows[0], guesses, dialect, has_header)
                for i in xrange(1, len(rows)-1):
                    first_row = analyze_row(rows[i], guesses, dialect)
                    lines_read += 1
                data = ''
            else:
                return guesses, has_header, COMPRESSION_TYPE


119640277-User-file-fails-with-NAN-in-thermostat-error-message---may-be-due-to-room-air-model-#5533 (lgu1234) - i386-Windows-7-VisualStudio-12: OK (2201 of 2201 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)


Sounds good! Thanks a lot :)


@rsfbarreira I submitted another PR https://github.com/torch/cutorch/pull/412 , on my machine all tests passes.


NodeJS is THE WORST!! 👅


@joeltaylor I added these link variables in an included view, but it may make more sense to make things into something smarter on the back end.


\o/


Is there a reason to keep track of the index of the array? It'd be a bit cleaner to do `for row in rows[1:-2]:`


Hello
This not solve the problem i think that missing a comma isn't?
 signatureVersion: 'v4',
 httpOptions: {timeout: 600000}


you can write "not is_bonus" instead of "is_bonus == False"


you can write "is_bonus" instead of "is_bonus == True" as it is a boolean value in itself


Yes you can remove my name too


be consistent with spaces around comparison operators


missing space before and after =


missing space after -=


I think this should be taken out of the `if '\n' in data:` block.  If the last chunk of the file you read is the last line of the file and doesn't end in a newline character, the `get_<compression>_guesses` functions will return `None` even if there are guesses.


I'm currently not well-versed in the amd/ati gpu situation... does our proprietary driver support xserver-1.18 already or not? I'd like to update it.


name = raw_input('please enter your name')


I think this can, and probably should, also live under the `dirty == false` check


I can't test AWS as they don't sponsor us, but yes indeed it looks like a coma is missing


You got the reference right, but not quite how it is used. It works like this. The compiler sees that you're supporting e.g. API levels 9 through to 23, but you've got an API call to a function only found in versions 14 and later. So normally, it would give you a warning. What @TargetApi does is give the compiler a hint that you're aware of the issue and you're doing something like a runtime check to ensure you don't hit that function on older OS releases. The parameter given to @TargetApi is generally the API level in which the function was introduced. So in this code, Camera.open(int) was only introduced in Gingerbread, so that is the hint.

But of course as you noted, this is a moot point for this code, because we don't support anything pre-API level 15 now anyway. So in fact the runtime check can be removed too.


issue #63


I Cant join a server with this Nukkit version :/


But headphones are okay on 4.9 too, my kernel is running over 4.9 without any problem.


edit tenant name and variables


Check this: https://github.com/zf5/Kernel/commit/72c04c8eb54dcb3e8b0561bed68d589e024d16df#diff-59f6363fe6ba61aab68db8ac18d4271e
It was renamed to `init.platform.usb.rc`


CameraConfig should be initialized with index 0. With this solution,  you need to iterate over the whole collection twice, the first time in cconfig.All (c => c != null)).  Instead I would do this which is simpler to read.

`
if (cconfig == null) {
  return;
}
for (int i = 0; i < cconfig.Count; i++) {
  if (cconfig [i] == null) {
    cconfig [i] = new CameraConfig (0);
  }
}
`


might this break in some browsers? would it be better to cache the value and explicitly restore it after the test, just in case?


The server computes a package hash when it receives an update (i.e.:  via the CLI's `release`, `release-cordova` or `release-react` commands).  When an update is available for a client the server returns the package hash to the client during an update check.

In the code above `update` is a [RemotePackage](https://github.com/Microsoft/react-native-code-push#remotepackage) instance and represents an available update.  `update.packageHash` is the package hash returned by the server, the same hash that was computed server-side.

There is a [special case for Android](https://github.com/Microsoft/react-native-code-push/blob/master/CodePush.js#L66) where a hash is retrieved from the APK.  This hash is represented above via `config.packageHash`.  This hash is ultimately generated by generateBundledResourceHash.js.


сам  ты # fuck!


# !!! CORRECTION AHEAD !!!!
# !!! ONLY READ ON IF YOU CAN HANDLE CS !!!
# !!! TRIGGER WARNING !!!
Okay so that's one method being overloaded once, so its ONE overloaded method, and it has been overloaded ONCE. The THIRD method is NOT overloaded because it has a different nae. Idiot.


testing


test comment


Remove inline css


Confirmed! confirmed!! This is awesome! w0000t!
![](https://lh3.googleusercontent.com/-6fLkN9z2Z7A/Vz3jLGeIDsI/AAAAAAAAVM8/vGA0aj2ZBfkcMbp2QxzTDwU1HbONkJu3wCLcB/s620/001.PNG)
![](https://lh3.googleusercontent.com/-xk0IaqtdwVw/Vz3jLHK8cFI/AAAAAAAAVNA/VHPJSMcy2uwDqgqcKioips1Y-y_nk_DvACLcB/s620/002.PNG)
![](https://lh3.googleusercontent.com/-TfY6jjN_yQs/Vz3jLNuumrI/AAAAAAAAVNE/mGEIslJcS1EdeqjhImEUR1dbpwYln48WwCLcB/s620/003.PNG)
![](https://lh3.googleusercontent.com/-eReDb7S8fMA/Vz3jLk8l0aI/AAAAAAAAVNI/6Lt6exdoY_0WV0TUYzYBpa-HitZVifw7QCLcB/s620/004.PNG)
![](https://lh3.googleusercontent.com/-GSsa2EDdHQI/Vz3jLtPWaGI/AAAAAAAAVNM/A3g37X46iXsyA5VlcDRP3n6RFJHAL07LwCLcB/s620/005.PNG)
This is a first big leap for the native IOT debugging scheme/platform!! And radare2 started it!
I am a SUPER happy r2 fan & user, and person mockis r2 or OpenSource is a dead meat. ;))


if you are going to submit this for upstream acceptance, its worth making this patch minimal.  not adding semi-colons.  that could muddle the change and make it harder for the upstream author to accept.


Name came out strage from a bad flag. should read "replaced all instances of /page/ with/


If you need the bzImage built by me, based on latest kernel code, **without dates so people won't complain about kernel version**, on GCC 4.9, use this https://github.com/zf5/android_device_asus_ctp-common/blob/marshmallow/kernel/blobs/kernel


Much better ty fam


Yep, all tests pass now here as well! Well done :)


@marekcirkos Will be able to provide you with more context here, he's responsible for the bulk of the work on `FBDeviceControl`. I just open sourced it as he was on PTO at the time.

I have some ideas that we may be able to bypass the `DVT` frameworks for Application manipulation since `MobileDevice` and the `AMDevice` family of functions provide Application manipulation functionality to 'Apple Configurator'. The `MobileDevice` functions are also very fast and don't require linking half of Xcode.


malformed like ur penis xddddddddddddddddddddddd


Cool, thanks !


missing whitespace before and after =


In this commit has been added the library to read file.json for configuration aims.
The possibility to rotate is obtained adding the  heading towards "lateral" positions(lattice elements) respect to the current one.


I had to format my data partition a few times in TWRP to get rid of
encryption each time I tested it. If the area reserved for the footer was
lost because of that then all the more reason to not use a footer at all.
On May 19, 2016 8:50 AM, "engstk" <notifications@github.com> wrote:

It's working fine for me on the 19/05 CM nightly with ext4.
Although I had to format the partition, because going from a build with hw
encryption set true to a build with it set false will derp the initial
layout setup, vold goes nuts.
You should try it mate, this will introduce unnecessary fragmentation.

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
<https://github.com/sultanxda/android_device_oneplus_onyx/commit/4f7312a9d701c3511b4c4589b9d65de4921f3be7#commitcomment-17542930>


we will only get to this branch if we passed the two before,
so if n is not a multiple of 3 or 5
but is a multiple of both
= never


Why not add custom_vcl_include_before and custom_vcl_include_after :)


Nope, doesn't really work: https://travis-ci.org/citation-style-language/styles/builds/131455594

Was hoping we could collapse everything before the actual test output, but we probably can't.

Got the idea based on:

https://github.com/Homebrew/legacy-homebrew/pull/43976#issuecomment-140483599
https://github.com/travis-ci/travis-ci/issues/2285#issuecomment-42724719
https://github.com/macbre/travis-fold
https://blog.travis-ci.com/2013-05-22-improving-build-visibility-log-folds/


I have no such error.


What problems? I got FDE working perfectly, I just couldn't get TWRP to
decrypt it.

Form my testing, I don't think we'll get HW encryption working. It feels
derped and that's why OP must have disabled it on OOS by default.
Many other snapdragon 80* devices disable it also because it's know to have
problems.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
<https://github.com/sultanxda/android_device_oneplus_onyx/commit/630bc33f313db84c77e28063d6ac756e064babbe#commitcomment-17542853>


my thoughts 2


Thanks for helping to think through this. The 401 is sent by us in views.py (look down for `request.is_xhr`). I don't know if it's the right way to do it, hence feeling uneasy.

That said, I don't think I'm going to submit this patch for PR/review. I got a good start on #1049 last night which will make this problem go away, because we'll go back to doing "regular" form submission and handling auth purely on the server side.


@HerbyW @FGMEMBERS/fg-aircraftdevelopers

Hi Herbert

For some reason, the guys in the devel list never came around to update the follow me in FGADDon. I was waiting to sync downstream by default.
But since they are not moving ahead, I decided to be proactive and updated the follow me in FGMEMBERS to ship with FG2016.2

Looks awesome (as always, coming from you!)

Thanks
@IAHM-COL


Hm okay but can you fix the chat for Windows 10 ? :)


Alright,Thanks.
I will fix that now


satoshi --> atoms


Now I have problems with the Internet. I look a little later.


Hi @DaniTheLion

I hope you're well

I've implemented this feature in a different way using the docker compose API, see https://github.com/francescou/docker-compose-ui/commit/ef0472e9b32e3b87cbd33e5404da6f659e361f92


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 143](https://teamcity.labkey.org/viewLog.html?buildId=325584&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 9 (2 new), passed: 46 Build time: 01:37:04
### Failed tests
```
org.labkey.test.tests.cds.CDSPlotTimeTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

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

org.labkey.test.tests.cds.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

```


Okay Thanks :)


Atoms/1000 bytes


also fixes #1150 which is an earlier copy of #1175


Shouldn't this me 1000000 since we are doing .ToCoin() ?


@adambaratz oh, my misread, thanks for the correction. If there's no behavior change, probably it's fine. As not sure how much breakage would this bring (certainly it's a bit more than converting numeric strings).

Thanks.


issue #61


I think you just need to call `builder.complete()` now. Example [here](https://github.com/meteor/meteor/blob/1000af63d3f023ebf75f03b88c89ae49fd4d52fe/tools/isobuild/bundler.js#L2164).


@dash-rai check this out


jika dosen pembimbingnya berubah, jadi harus ubah secara hardcode ya?


@dash-rai read the above comments :)


The point of this file was to have a single one per-repo that's easy to localize, not duplicating them per-package.


I agree that it is good to be consistent with the rest of the factory module. However, I don't like the fact that "normalization" refers both to the 2-norm (in mpnum.factory) and to the MPS normal form / normalization (in mpnum.mparray). Any suggestions?


Pull request: https://github.com/PrestaShop/PrestaShop/pull/5525

And as you said, it allow us to alter `$template_vars`


Good catch!


Sayak Dutta added to the names.txt list


@ctrueden, I discontinued the artifact mpicbg-trakem2 and replaced by trakem2-transform.  trakem2-transform contains the required transformation from trakem2_tps and I removed it there.  Please note that trakem2_tps includes a standalone plugin for Fiji, i.e. it has to stay in pom-fiji and fiji/fiji.  This fixes the "circular org dependency problem".


We're not running on an AVR chip. This part is Arduino-specific and will likely raise an error if used on the BeagleBone.


Do you think it is possible to load results from a first inversion that for instance did not sufficiently converge ?
That would enable to save time and start directly at the end of the previous inversion.
I imagine that it would be possible to do that instead of doing the first initial walks, isn't it?


mungkin tutorial ini bisa membantu
http://www.tutorialrepublic.com/php-tutorial/php-file-upload.php


Thx, doesn't see this PR :)

But, why do you need this one much more than the two others already existing ? What's the point ? It's for understand, I ask, how you think to use it :)


Nevermind, did not look at the `ifdef`


GenerateEDDwhenOutputEMS (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2195 of 2195 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)


The semicolon is invalid syntax.

This should be: `<meta charset="utf-8" />`


I prefer to include the optional slash at the end. It is called a SOLIDUS.

```
<link rel="stylesheet" type="text/css" href="style.css" />
```


Browser-breaking commit.
I love it!


Introd


by updating the react-native-progress to 3.0.0
fix the issue https://github.com/KevinHu2014/RN-test/issues/5


I have Updated it.


Comment syntax should be:

```
<!-- COMMENT HERE -->
```

Note the hyphens


Transoptor to pierwszy pomysł jaki mi przyszedł do głowy jak wpiąć uC w
obwód hamulca. Ale rzeczywiście wystarczy dzielnik napięciowy, tak by na
wejściu mikroprocesora wartość napięcia wynosiła maksymalnie 3V.

2016-05-18 10:13 GMT+02:00 ptaq666 <notifications@github.com>:

> Spoko, w każdym procku jakoś inaczej jest rozwiązane definiowanie
> Interrupt Handlers, zawsze jest z tym mały problem. Ogólnie problem fajnie
> rozwiązany, widzę że projekt już prawie skończony. Po co jest ten
> transoptor?
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/ptaq666/Mechatronika_2015-16_Proj_TN/commit/d8aa713d507e3f4d48e30451f6dccbaa1a62a894#commitcomment-17518763>
>

--
Sławomir Wawrzyniak


I don't understand your point.
What you mean the area from the footer was lost?
You need a partition or a zone of a partition to store the key right?
If you want to unencrypt what's the difference between formatting just a partition (if key is sotred in footer) or formatting a encrypted partition and a second partition with the key?
I really don't see the need for this, storing key on footer works (and always worked) fine.


css is looking good. nice work. remember to indent.


be consistent with spaces around +


factory girls installed, yet not sure why we skipped the following configurations:

(1)
Generators for factories will automatically substitute fixture (and maybe any other fixture_replacement you set). If you want to disable this feature, add the following to your application.rb file:

```
config.generators do |g|
  g.factory_girl false
end

```

(2) Default factories directory is test/factories, or spec/factories if test_framework generator is set to :rspec; change this behavior with:

```
config.generators do |g|
  g.factory_girl dir: 'custom/dir/for/factories'
end
```

gem url https://github.com/thoughtbot/factory_girl_rails


missing space before +=


Interesting... Is a loop rate of a 100Hz enough though? Let's run it with this and test it out. I'd obviously like to run the rate PID as often as possible.


Please make a Pull request for this. Thx!


will village box be updated to 1.9.4 as well?


Maybe we should get rid of this 'prefer' here too?


We have FDE working in sw mode, cpu+neon extensions.
hw extensios, TTE and so on are know to be troublesome, even some nexus have it disabled (and some devices on CM also disabled it)


i got boot it on p1 now thanks a lot. but on my device ram shows up 561 mb:) its incredible p1 has only 512 mb ram:P


need to add the old authors here as well ..


LGTM


Looks good, please Pull request this.
Add/update unit tests as well please!


Also, I don't think the noise he talks about affects the absolute angle too much. Probably only to do with the rate. Let's set it on level ground and get some readings the next time.


develop (kbenne) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2357 of 2419 tests passed, 0 test warnings)

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


lgtm


Is that in the official eCSIRT II taxonomy?


unnecessary empty line


@LordUsagi - doxygen has not markdown support as we're used to. bold, emphasis etc that starts with `*` or `_` needs to have the first and last byte to be a char in alphabet. See [this page](http://www.stack.nl/~dimitri/doxygen/manual/markdown.html). So you just need to replace `*/etc/mysql/my.cnf.*` with `<i>/etc/mysql/my.cnf</i>.`

Also prefer markdown links and syntax before html's one. You have also there double quotes used `"` you should replace them with `"`.


This reverts our change of switching the roll and pitch axes assumed by i2cdevlib.


Is that needed? Maybe (and I don't understand the need yet)


check the modulo operator instead https://docs.python.org/2/reference/expressions.html#binary-arithmetic-operations


Should be 0.01 since the config parses a float and converts coins to Amount, and 0.01 DCR/kB is the default mempool relay fee (this is still using the default BTC/kB value for bitcoin).


Oh.. speaking of which, shouldn't we also have [that stuff](https://code.google.com/archive/p/pcsx2-playground/source) archived before it's too late?


@pjbreaux if we have link to both then we need to have separate entries in this file.  Since the content is the same I think we can safely point to the most recent and use that.  It will save time if we need to make changes later.


From a quick overview, I didn't see any change that might fix our problem of angles > 90° not being measured properly. @KarthikNayak Could you see if there's anybody else with the same problem?


issue #51


develop (kbenne) - 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-warnings-yellow.svg)


I just learned that there is a difference between checking `__len__` and `collections.Sequence`:

```
>>> import collections
>>> hasattr({}, '__len__')
True
>>> isinstance({}, collections.Sequence)
False
```

Insteresting! (via [Stack Overflow](http://stackoverflow.com/a/2937122))


Sorry, a space where?   Any where does it look better?  In the Google Authenticator UI?   I've not seen %20 in URI in the wild, have you?


+ fill it in the m variable instead of just printing out the result


There are 0 spells with this flag?


Why not process the terraform state directly in json here?


Oops, I see… I'll investigate about this.

Thank you.


lgtm


Logs can be obtained from journald. If consul fails, journalctl -u consul > out.txt. We may not even need to pull files.

If we can run a handler on the system if a component fails, we can just ssh and pull the output.


@leont this is not the only spot sadly. The full list currently is:
```
t/par.t:12:  my ($have_c_compiler, $tmp_exec) = check_compiler();
t/par.t-13-  if ( ! $have_c_compiler ) {
t/par.t-14-    plan skip_all => 'No compiler found';
t/par.t-15-  } elsif ( ! eval {require PAR::Dist; PAR::Dist->VERSION(0.17)} ) {
t/par.t-16-    plan skip_all => "PAR::Dist 0.17 or up not installed to check .par's.";
t/par.t-17-  } elsif ( ! eval {require Archive::Zip} ) {
t/par.t-18-    plan skip_all => "Archive::Zip required.";
t/par.t-19-  } else {
t/par.t-20-    plan tests => 3;
t/par.t-21-  }
t/par.t-22-  require Cwd;
--
t/ppm.t:17:  my ($have_c_compiler, $tmp_exec) = check_compiler();
t/ppm.t-18-  if ( ! $have_c_compiler ) {
t/ppm.t-19-    plan skip_all => 'No compiler found';
t/ppm.t-20-  } elsif ( ! $PPM_support ) {
t/ppm.t-21-    plan skip_all => 'PPM support modules not installed';
t/ppm.t-22-  } elsif ( !$Config{usedl} ) {
t/ppm.t-23-    plan skip_all => 'Perl not compiled for dynamic loading';
t/ppm.t-24-  } elsif ( ! $HTML_support )  {
t/ppm.t-25-    plan skip_all => "HTML support not installed";
t/ppm.t-26-  } elsif ( ! eval {require Archive::Tar} ) {
t/ppm.t-27-    plan skip_all => "Archive::Tar not installed to read archives.";
--
t/xs.t:13:  my ($have_c_compiler, $tmp_exec) = check_compiler();
t/xs.t-14-
t/xs.t-15-  if ( !$have_c_compiler ) {
t/xs.t-16-    plan skip_all => 'No compiler found';
t/xs.t-17-  } elsif ( !$tmp_exec ) {
t/xs.t-18-    plan skip_all => 'Compiler is dysfunctional';
t/xs.t-19-  } elsif ( $^O eq 'VMS' ) {
t/xs.t-20-    plan skip_all => 'Child test output confuses harness';
t/xs.t-21-  } elsif ( !$Config{usedl} ) {
t/xs.t-22-    plan skip_all => 'Perl not compiled for dynamic loading'
t/xs.t-23-  } else {
```

I am not sure why `check_compiler()` doesn't do the `$tmp_exec` check itself tbh, thus not supplying the trivial patch to `t/par.t` and `t/ppm.t`


use 'single quotation marks'


Closes #115


Zeitzone muss noch berücksichtigt werden


missing space before +=


Has anyone added an active state for the state labels that they can share?


Ugh, that is silly. I'm gonna try to get 12.1.0 out asap today. Sorry for the breakage.


This looks great, thanks Jordan!


to check parity


try printing "len(x) % 2 == 0"


you can write "not is_bonus" instead of "is_bonus == False" as it is already a boolean value


@xfurry i can trying help you.
``` CPP
SPELL_LEOTHERAS_GREEN_BEAM          = 37626,
/* *************
** npc_spell_dummy_leotheras_green_beam
************* */

// Desctiption: You can Send AI Event Custom A for add counter. If aura remove from caster = decrease count. If count = 0, remove aura Banish and set in combat with zone. Delete script check from instance.cpp.
// Description Note: Add separately as a you add npc_dummy crusader strike in Culling of Stratholme.
bool EffectDummyCreature_npc_spell_dummy_leotheras_green_beam(Unit* pCaster, uint32 uiSpellId, SpellEffectIndex uiEffIndex, Creature* pCreatureTarget, ObjectGuid /*originalCasterGuid*/)
{
    // always check spellid and effectindex
    if (uiSpellId == SPELL_LEOTHERAS_GREEN_BEAM && uiEffIndex == EFFECT_INDEX_0 && pCaster->GetTypeId() == TYPEID_UNIT)
    {
        // only apply this for Leotheras.
        if (pCreatureTarget->GetEntry() == NPC_LEOTHERAS)
            pCreatureTarget->AI()->SendAIEvent(AI_EVENT_CUSTOM_A, pCaster, pCreatureTarget);
        // always return true when we are handling this spell and effect
        return true;
    }

    return false;
}

/* *************
** npc_spell_dummy_leotheras_green_banish
************* */
// Desctiption: Remove server spam spell error. Not handled in custom case.
// Description Note: Add in leotheras_cpp script.
bool EffectDummyCreature_npc_spell_dummy_leotheras_green_banish(Unit* pCaster, uint32 uiSpellId, SpellEffectIndex uiEffIndex, Creature* pCreatureTarget, ObjectGuid /*originalCasterGuid*/)
{
    // always check spellid and effectindex
    if (uiSpellId == SPELL_LEOTHERAS_BANISH && uiEffIndex == EFFECT_INDEX_1 && pCaster->GetTypeId() == TYPEID_UNIT)
    {
        // only apply this for Leotheras.
        if (pCreatureTarget->GetEntry() == NPC_LEOTHERAS)
            pCreatureTarget->AI()->SendAIEvent(AI_EVENT_CUSTOM_B, pCaster, pCreatureTarget);
            pCreatureTarget->SetStandState(UNIT_STAND_STATE_KNEEL);
        // always return true when we are handling this spell and effect
        return true;
    }

    return false;
}
```


There is no functional difference in the location that the key is stored.
However, the footer isn't a magical 16 kB region that'll always
conveniently be available for our use; if it's lost from formatting or
something else, then you'll have to manually resize /data to get it back.
This is what made it an absolute pain to deal with back when OEMs used it a
lot more often.

I was very close to shipping h/w encryption, so I'm not too worried about
the fragmentation this causes. If/when I do get TWRP to decrypt
h/w-encrypted data, there'll be more fragmentation anyway.
On May 19, 2016 9:27 AM, "engstk" <notifications@github.com> wrote:

> I don't understand your point.
> What you mean the area from the footer was lost?
> You need a partition or a zone of a partition to store the key right?
> If you want to unencrypt what's the difference between formatting just a
> partition (if key is sotred in footer) or formatting a encrypted partition
> and a second partition with the key?
> I really don't see the need for this, storing key on footer works (and
> always worked) fine.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/sultanxda/android_device_oneplus_onyx/commit/4f7312a9d701c3511b4c4589b9d65de4921f3be7#commitcomment-17543517>
>


It is too late :(


This shouldn't have been a problem.


All right, so i have trouble understanding https://github.com/igorpecovnik/lib/blob/dff98cae2e3c1c691e8f46f40a036a7866a15806/patch/kernel/pine64-default/sun50i-simple-corekeeper.patch#L25 as the indentation is misleading.


Unused in 3.3.5a


print numbers from _1_ to 100 -> start range from 1


@nyalldawson @volaya there is already a ticket about this issue?


issue #52


P.S. Я сделал нормальное наследование LinkedList от ListComparable, но теперь не работают тесты, не знаю почему. В main() есть код, повторяющий тест, при этом код работает, а тест - нет.


you ~~don't need to~~ shouldn't increment the loop variable in a for loop


develop (kbenne) - x86_64-MacOS-10.10-clang: OK (2381 of 2419 tests passed, 0 test warnings)

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


Hmm, I kind of do that in Main to a degree, although Main handles all the content as well. The only time I use the main static void method in the whole thing is to create the gui and create an instance of Main.


Another kernel patch (or a few?) is required, but other than that it works
perfectly on onyx.
On May 19, 2016 9:30 AM, "engstk" <notifications@github.com> wrote:

> We have FDE working in sw mode, cpu+neon extensions.
> hw extensios, TTE and so on are know to be troublesome, even some nexus
> have it disabled (and some devices on CM also disabled it)
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
> <https://github.com/sultanxda/android_device_oneplus_onyx/commit/630bc33f313db84c77e28063d6ac756e064babbe#commitcomment-17543563>
>


Found a post on TCRF that says that only the Pokemon Centers in Johto used the '() PC' graphic, in the Kanto region, all versions of the game use the old style of sign, so that may want to be taken into account. I'd do it myself, but I don't know which images correspond to each region.


i have added the log code i am waiting on a new error
Regards


recordings for lanefollowing


Please contrast it with this one. https://github.com/uaarg/uaarg.github.io/blob/d6a19df15c51ff53426ac3a7d68d45e224a5532c/Rakefile

Also see the readme. https://github.com/uaarg/uaarg.github.io/blob/source/README.md


fixed a tiny typo


@usr110 suggest we create a new data repo for non census data - sound good @nikolai-b - this one is way more than 100 Mb, putting lots of strain on my internet connection!!


use 'single quotes'


Doesn't matter. Minimizing branch differences is a goal too.


weird... why did the line get added instead of replaced?


The following doc(s) have been deployed to the [twootenpat](https://twootenpat-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/wordpress-solr](https://twootenpat-static-docs.pantheonsite.io/docs/wordpress-solr)


Can you update the plunkr to include a component?


Typo


NO me has dado tiempo ni de mirarlo. Se puede usar `melt` para hacer eso de una forma un poco más simple, pero en todo caso tiene buena pinta.


needs `<pre class="codesnippet">`


Source: https://tcrf.net/Pok%C3%A9mon_Gold_and_Silver#Pok.C3.A9_Mart_And_Pok.C3.A9mon_Center_Signs


typo, missing 'e'


My PC did not recorgnize a phone without init.a500cg.usb.rc


Yes I noticed, I'll change this :dancers:


In my case it is dead on GCC v.4.9.


needs <pre class="codesnippet">


做的没问题~


hi, when using 74LVC245 Octal 2-Way Tranceiver the ports A8 and GND are inverted when you look at schematics and protoboard views


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 144](https://teamcity.labkey.org/viewLog.html?buildId=325591&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7 (3 new), passed: 49 Build time: 01:43:09
### Failed tests
```
org.labkey.test.tests.cds.CDSTestLearnAbout.testLearnAboutStudyProducts: <no details avaliable>

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

org.labkey.test.tests.cds.CDSTestLearnAbout.testLearnAboutStudyProductsSearch: <no details avaliable>

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

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

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

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

```


DepricatedFields (mitchute) - i386-Windows-7-VisualStudio-12: OK (2202 of 2202 tests passed, 556 test warnings)

 * 556 tests had: AUD diffs.
 * 2 tests had: BND diffs.

![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'd just revert that commit ;)


Me too. I did, didn't really find anything. :(


DepricatedFields (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2234 of 2234 tests passed, 556 test warnings)

 * 556 tests had: AUD diffs.
 * 2 tests had: BND diffs.

![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)


Whoops. I could've simplified this (one less line).


I feel like the worst thing to do would be to serve arbitrary content through our own server. This might cut some corner somewhere, but it should stop us from serving up scripts as secure content, right?


no need for space between method name and opening bracket


DepricatedFields (mitchute) - 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)


Hi my name is JEDSUPHA AUNTASEE
I studying computer engineering at SURANAREE University


I still wonder about this code. Why don'y we just remove the parameter and overwrite by default (b onto a). If you don't want to overwrite, you need to reorder the parameters to the function (`update(b,a)` instead of `update(a,b)`).


LGTM


again :)
https://google.github.io/styleguide/pyguide.html?showone=Whitespace#Whitespace


this would make the last item in self.elements to contain the whole list minus itself


OK er dette en endring?


Yeah on the other hand the Gelf folks don't seem like they are very willing to address the issue, so maybe we should rather keep truncating?


use spaces around `>`
https://google.github.io/styleguide/pyguide.html?showone=Whitespace#Whitespace


I think it actually does as it's an unsynced RNG so a pointer can seed it just as well as anything else.
But yes, it's ugly as hell.


Just removing something that was not needed...


lgtm


testing..


@pskumar448 Than you make build-flow-plugin 0.19 a mandatory dependency?
FlowRun from build-flow-plugin is needed.  See: https://issues.jenkins-ci.org/browse/JENKINS-34803


Let me know if you have any question!


You should switch to boringssl.

https://android.googlesource.com/platform/external/boringssl/

On Fri, May 13, 2016 at 3:39 PM, hgbread <notifications@github.com> wrote:

> Hello. I apologize if this is a spam, and hope you can help us.
> In our Android App, we are using CSipSimple that uses OpenSSL. So we use
> the latform_external_openssl project to compile OpenSSL for Android. Now
> with the new requirement from Google (
> https://support.google.com/faqs/answer/6376725) to upgrade OpenSSL to
> 1.02f/1.01r, we are looking for the proper way to accomplish this upgrade.
> I have noticed you have upgraded the OpenSSL for this project a few times
> in the past, so I was wondering if you intend to do this soon also for the
> latest version and if not, could you please give us any input as how to do
> it? We tried using the README.android instructions, but found that
> import_openssl.sh is not up-to-date which makes it a real challenge to
> upgrade the OpenSSL code. Or maybe there is another alternative to get the
> latest OpenSSL code compiled for Android..
> Hope it not too much of a bother. Thank in advance!
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/android/platform_external_openssl/commit/21b86b154c3cbb88bca801b07e93db4fc8face8e#commitcomment-17474608>
>


nooo single quote is better :)
(when no inner single quote is present in the string)
https://google.github.io/styleguide/pyguide.html?showone=Strings#Strings


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


okay... that is AWESOME.


the previous solution was closer to the lame exercise text :)


b


implementações #7 e #4


Where did SunshineSyncAdapter.java come from? I don't even have that file.


unnecessary empty line


use space either both sides or neither of `+`


I would suggest just to add isAndroid() check into the if (..), like this:

if (!isAndroid() && (doPrivileged == null || doPrivileged.trim().equals("0"))) {


:+1: was planning to suggest this :)


Why did you make this change? These are not story requirements and they severely limit your application.


try printing "len(x) % 2 == 0"


Honestly I haven't understood everything in the code, but it did seem like incomplete and as you're saying so, I guess one should just wait for a real confirmation that it works either on Win98 or anything else; then we might consider this mouse emulation. (note: I do know nothing about DOS calls/interrupts, I'll be hardly of any help)


'is_bonus == False' can be written as 'not is_bonus' as it's a boolean in itself


you defined a global attribute in `Circle` for containing the value of `pi` but its never used


That is exactly how docker works.  It only follows the pid 1 of the container; even if you have systemd start up a bash shell, docker wouldn't be able to attach to that child process in a `docker run`. If you want a bash process running in your container with systemd, then you need to start and attach to one; for example, run ssh in your container and connect in.  You could also run a bash session with a `docker exec` but that would only be in the namespace of the systemd process and not managed by systemd.


Can Android choose the correct library for the platform itself?

The code you skip here is actually designed to detect platform (32/64, ARM/Intel)
and to choose the native library, that matches the platform.


classes should be inherited explicitly from `object`
https://google.github.io/styleguide/pyguide.html?showone=Classes#Classes


lgtm


Thanks, @NoahKunin.  With respect to my _public collaborators_ document question, I was curious to know if someone from outside of GSA on Slack, government or public—"public" was mentioned in the draft blog post—shared **their** Google docs and what data Slack indexed once a user permitted the integration.


class names should start with capital character
https://google.github.io/styleguide/pyguide.html?showone=Naming#Naming


@Ettatabe This is not the right way to make Javadoc. Please fix this..


again from object and `ClassName`


again :)


Hah, good catch.


great <> greet


May be better: `"Can't find " SEVEN_ZIP_PACKAGE "/SevenZip class"`


Zopfli performance summary: This change adds ~2% speed overhead. Tested with LTO + PGO.


Alright. I'll check that.


use single quote if there is no single quote in the string
https://google.github.io/styleguide/pyguide.html?showone=Strings#Strings


single quote


Check the bzImage I linked you to.


LGTM


Você está dizendo eles estão no HTML e não em um CSS externo porque ele não estava achando o arquivo externo? Como você estava fazendo ?


use spaces around `<`
https://google.github.io/styleguide/pyguide.html?showone=Whitespace#Whitespace


I understand removing Cal- and CardDAV as SOGo is better, what am confused on why removing the scheduled mail backup function as it was quite a useful feature.


chắc ko :v nó đang tính theo customerkey mà, còn cái kia ô lấy ID của nó kìa, ID với customerKey giống nhau à


It should start as soon as the client confirms the cinematic started client side (since the loading time is an unknown). I'm not sure how it can be pre-empted otherwise, so there's always the chance of this odd delay. The odd "flashing" at the start is because I used a creature as a method of moving through the waypoints. Blizzard for sure don't do this (not from all the sniffs I've seen anyway). They send nothing to the client about a creature/dynobject, or indeed anything so far as I can see. So they simulate movement entirely server side and just send the objects around that "virtual object".

To me, it was a trade-off between doing something we can do with existing core API, without writing something new just to handle this.


use spaces around `=` when its a whole line statement (eg. not in default param value)


lgtm


? Isn't the repository there available to download/mirror/export?


Lovely


number of candies, not number of students 🍬


yes, definitely simpler :)


ererw


We may consider to use this "Android" style of finding classes for all other platforms as well. It may have a little initialization performance drawback (we should cache all class references anyway), but will reduce the amount of `#ifdef`s and though possible undetected bugs.

 What do you think?


no need for spaces between function name and opening braces


add-curve-unit-tests (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2213 of 2214 tests passed, 0 test warnings)

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


single quote


use spaces on both sides


I think, this is a good thing to do anyway. Different JVMs could have same restrictions. So fill free to remove `#ifdef`.  I will test it on all other platforms later on.


Can we remove the mscorlib.ni.dll now?  Or do we still need NI's of full-facades.


Original code (`budget_get_any_online_cpu`), here indentation is wrong already and code can be simplified, but it works.
```
static int budget_get_any_online_cpu(const cpumask_t *mask)
{
	int cpu,lastcpu=0xffff;

	for_each_cpu(cpu, mask) {
		if ((cpu != boot_cpu) && cpu_online(cpu)){
			if(lastcpu == 0xffff)
				lastcpu = cpu;
		else if(cpu >lastcpu)
			lastcpu = cpu;
		}
	}
	return lastcpu;
}
```

Based on this `budget_get_any_offline_cpu` would look like this (I changed `cpu_is_offline` to `!cpu_online`, but didn't push new version of this patch yet):
```
static int budget_get_any_offline_cpu(const cpumask_t *mask)
{
	int cpu,lastcpu=0xffff;

	for_each_cpu(cpu, mask) {
		if (!cpu_online(cpu)){
			if(lastcpu == 0xffff)
				lastcpu = cpu;
		 	else if(cpu >lastcpu)
				lastcpu = cpu;
		}
	}
	return lastcpu;
}
```


invalid syntax, this code is not running!!!
you shouldn't commit anything without trying


use spaces on both sides or neither of `+`


__API Description file syntax error on line 1935__: base type 'Expense' is not defined in the document (Autogenerated message from apiary.io)


invalid syntax!!! never commit things like this! comment it out at least


bin\\app and bin\\test paths failed in osx.  changed to forward slash


How much did it show in previous kernels?  It should be roughly 60MB high than it showed there (and I believe that P1 actually does have 592MB of RAM, but I'm not sure why (look at https://github.com/feroxx/android_kernel_samsung_aries-1/blob/f55d0edf9fc16a3716c46e95355f404a740f0a29/arch/arm/mach-s5pv210/mach-p1.c#L7493 - this only reads 127 in aries devices - and if you add the three banks together you get 591 (with 1MB reserved for RAM console)))


And using flexboxes


dont use spaces between variable name and index


GenerateEDDwhenOutputEMS (JasonGlazer) - i386-Windows-7-VisualStudio-12: OK (2201 of 2201 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 think this will fix https://github.com/Fluorohydride/ygopro-pre-script/commit/c70072201dd8f2f25d453be1863df492e6e62728#commitcomment-17518303, but it need more test.


This commit broke telemetry monitor for FrSky protocols on 7e (crash when entering telemetry monitor page, can be reproduced on emu_7e).


Bunch of defines with the same name:

In file included from ../arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi:47:0,
                 from ../arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts:47:
../arch/arm64/boot/dts/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h:77:0: warning: "RESET_MIPI" redefined [enabled by default]
 #define RESET_MIPI  29
 ^
../arch/arm64/boot/dts/include/dt-bindings/reset/amlogic,meson-gxbb-reset.h:76:0: note: this is the location of the previous definition
 #define RESET_MIPI  28
 ^


@hashar what version are you on currently? The tagging system claims this is part of irc bot plugin version 2.27.


try not checking the part that is already sorted - use i in the inner loop's limits


The following doc(s) have been deployed to the [1581cookie](https://1581cookie-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/cache-control](https://1581cookie-static-docs.pantheonsite.io/docs/cache-control)
- [docs/caching-advanced-topics](https://1581cookie-static-docs.pantheonsite.io/docs/caching-advanced-topics)
- [docs/clear-caches](https://1581cookie-static-docs.pantheonsite.io/docs/clear-caches)
- [docs/cookies](https://1581cookie-static-docs.pantheonsite.io/docs/cookies)
- [docs/debug-cache](https://1581cookie-static-docs.pantheonsite.io/docs/debug-cache)
- [docs/drupal-8-cache](https://1581cookie-static-docs.pantheonsite.io/docs/drupal-8-cache)
- [docs/drupal-cache](https://1581cookie-static-docs.pantheonsite.io/docs/drupal-cache)
- [docs/mobile-redirection](https://1581cookie-static-docs.pantheonsite.io/docs/mobile-redirection)
- [docs/pantheon-workflow](https://1581cookie-static-docs.pantheonsite.io/docs/pantheon-workflow)
- [docs/pantheon_stripped](https://1581cookie-static-docs.pantheonsite.io/docs/pantheon_stripped)
- [docs/guides/terminus](https://1581cookie-static-docs.pantheonsite.io/docs/guides/terminus)
- [docs/unsupported-modules-plugins](https://1581cookie-static-docs.pantheonsite.io/docs/unsupported-modules-plugins)
- [docs/varnish](https://1581cookie-static-docs.pantheonsite.io/docs/varnish)


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 145](https://teamcity.labkey.org/viewLog.html?buildId=325602&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7, passed: 48 Build time: 01:40:36
### Failed tests
```
org.labkey.test.tests.cds.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

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

```


I gotta admit 'be tired' is pretty good advice.


DepricatedFields (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1088 of 1088 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-33.36%25-green.svg)


maybe it would be easier to use a range here instead of manually keeping track of the index


You may need to edit UseItemPacket.php add the following,

 public $unknown;
 public $slot;

also

   $this->unknown = $this->getSignedShort();
   $this->slot = $this->getSignedShort();

Just FYI


remove duplicate items from union


Support for gemm with int (it's originally from tensorflow's quantized dot products :))


please make a PR for the Team Cyrmu bots (including source code and unit tests). Thanks!


This is more correct anyway; you ought to be able to get a mutable copy of an immutable path. 👍


PR please with unit tests. Thanks


hey @runoshun
tried your hack, worked fine to quit moonlight. But it didn't stop the game on PC side whereas i would expect it to stop too. I guess that's the normal behaviour ?


try list.index(value) to remove duplicates instead of sorting


seems like we can merge this into a generic filter bot?


LGTM


I think I wrote that, yeah, with the idf contents ripped right out of a test file.


Ohh, opengraph. I had to google it haha. Oks!
El may 19, 2016 11:27 AM, "Franco Leonardo Bulgarelli" <
notifications@github.com> escribió:

> Actually og may be ignored in mumuki-offline.
>
> —
> You are receiving this because you commented.
> Reply to this email directly or view it on GitHub
> <https://github.com/mumuki/mumuki-atheneum/commit/77dfbe126aa10453dc14808ef4319e647a2c09d5#commitcomment-17541362>
>


use self.radius instead of a method param


ugly :)


__API Description file syntax error on line 1935__: base type 'Expense' is not defined in the document (Autogenerated message from apiary.io)


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 146](https://teamcity.labkey.org/viewLog.html?buildId=325605&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 7 (1 new), passed: 48 Build time: 01:37:52
### Failed tests
```
org.labkey.test.tests.cds.CDSPlotTimeTest.verifyTimeAxisFilter: <no details avaliable>

org.labkey.test.tests.cds.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

```


This seems like a simple whois bot. Can you compare with the ripe_abuse_c bot?
Maybe stats.ripe.net abuse finder already solves exactly this problem. Also for BT (?)


PR please


We are facing a similar issue which is caused when the GELF message is too big: https://github.com/bzikarsky/gelf-php/blob/master/src/Gelf/Transport/UdpTransport.php#L112

So there are actually two problems:
- each key-value pair can be too big for ES (32766 bytes)
- the whole message can be too big to be transported by the GELF UDP protocol (1420 chunks * 128 bytes = 181760 bytes)

The ES limit is not really a problem as it just generates errors on the ES server which do not affect the app. But the message limit throws an exception in the GELF php implementation which can ruin the app.
I don't know how exceptions in log handlers are handled in general by monolog. PSR-3 doesn't specify AFAIK if the log methods can throw exceptions. But IMO logs should never throw exceptions. Often you explicitly catch an exception and log it to keep going. But if that logging throws an exception as well, then it doesn't make sense at all.

So shouldn't monolog catch exceptions by handlers by default? Or should handlers not throw exceptions in the first place (which I'm sure is impossible to enforce). As explained above I think this is more important to tacle.


And there was much rejoicing!


@jeffbyrnes FYI shipped this one commit without PR because it was about to create a ton of alert spam ;-)


Please make a PR and add documentation (HOWTO connect intelmq with logstash).
This could go into the contrib/logstash folder.
Great stuff!


Ah no, si es que hoy iba a ponerme con otra cosa, pero he tenido una idea feliz de pronto y me ha funcionado perfectamente, por eso lo he cerrado, xD.


can this be part of a generic filter bot?


use Capitalized classnames


use lowercase instance names


can be probably improved. Look at stats.ripe.net API and ripe_abuse_c bot


Capitalize class names


I think it would be best to disable this during blocksplitting.


Hi @fasterthanlime, maybe I missed something here, but it looks like it doesn't say what should be in the `directory`, or if it can be an `.exe` (e.g. innosetup installer), or an `.msi`, `.deb`, etc. instead of a `.zip` file. Or what should contain the `.zip` file?


This is not part of the Data Harmonisation format. It will be harder for others to use this since they don't know this field


"removed /vender folder from git ignore" would have been an acceptable commit message as well


We have a WhatFailureGroupHandler if you want to suppress/ignore exceptions from handlers. I personally want to be aware if my logging is failing as it's kinda critical, so I am rather against doing it by default. Let's focus on the GELF issue please :)

If you think an improvement can be made here, I'd appreciate a PR. I never used GELF myself so I am not extremely familiar with it.


I am not sure if I understand this bot. Maybe a README.md with some examples?


Create a PR please to the 2.1.x branch


dont use semicolons, they are unnecessary


please don`t use the built in methods


TaskCluster: @hotsphink does not have permission to trigger tasks.


This indeed looks ugly.


use single quotes unless the string contains single quotes
https://google.github.io/styleguide/pyguide.html?showone=Strings#Strings


I did not check now if the functionality is exactly the same, but it looks cleaner


PR please. Let's officially review . Add a unit test thx :)


Tilraun


single quotes


Tilraun líka


Amaaaaaazing! Congratulations Loco!


I'd be happy to help. Can you be more detailed what kind of issues you had that need patching and caused removal of meta-beagleboard?


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


holy sh_t balls!
# :+1: THANKS @tiagonoronha!!!!!! :+1:

_that's sOOOOOOO much more intricate than the solution I had whipped up_

Dear God: Thank you for rocket ninja geniuses who code **way the heck better** than I can. ~Amen

:wink:


Please also allow for the "official" JSON with comments format: //
However, there might be quite some drawbacks with that! Not every JSON library supports comments.
So... This might be a no-go...

Might make more sense to add a _comment: "blablba" value everywhere?


unused variable _grades_


unnecessary param for average


This one needs to refer to 'the other interface' (or just 'the other') instead of 'the other two'.


if one token is too short that should be okay, should only return empty result if all tokens are too short. see https://github.com/humhub/humhub/issues/1737#issuecomment-220399014
also mb_strlen should be used to make it work with multibyte chars.


type mismatch, adding a list and an int


self.size()


Why did you move this back to data2.s? Is it because of that comment I wrote?

I meant that this should be in data2.s

```
gUnknown_0840B878:
 +	.incbin "baserom.gba", 0x0040B878, 0xF @ unknown
```

The rest can stay in credits.s


\yii\db\ActiveRecord[] <- array!


use a range instead of a list instead of manually keeping track of the index in a for loop


convert item to array inside the push method instead


indent more


Error signing in with i4a integration when paidthru is null and the user is a valid member


enum gemaakt voor status


Sorry I mistook the card name.
btw,  I think you missed https://github.com/Fluorohydride/ygopro-pre-script/commit/d7d5a939b2a864fad3e96b95df8dd707f0669259


is nie protected, niks gaat erven van product


DepricatedFields (mitchute) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1646 of 1646 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.97%25-green.svg)


start by building it on master.


lijstje van tafels he, niet van ints


@flekschas: we should use `settings. ANONYMOUS_USER_ID` instead of -1 here.


Hi,

Please take a look at the updated tree: https://github.com/eugenesan/keepassx/commits/2.0-http-totp
I've fixed default binding to 0.0.0.0 on machines with IPv6 support enabled.


Now works, but the db don`t work: Sorry deployment for this database is not possible
Where is the configuration of this?


This introduce a BC break, 2.4 dev is the same as 3.x stable version


GenerateEDDwhenOutputEMS (JasonGlazer) - 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)


TaskCluster: @asurkov does not have permission to trigger tasks.


Why fixing on 1.2.x versions ? the old notation allow >1.2, <2.0


develop (kbenne) - x86_64-MacOS-10.10-clang: OK (2375 of 2419 tests passed, 0 test warnings)

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


same here


Ship it!


Thanks; removed now! (I added this pre-emptively expecting we might want to hook up to a e.g. interrupt handler / signal but I think we'll have a more explicit mechanism for interrupting single non-R chunks)


No, google has taken it all down, they have kept the revision notes (not the changes) and the latest revision of the repo, that's about it :(


Something like this: https://github.com/intellij-rust/intellij-rust/commit/9e2ee67c648faecccdf1bbcefdb18d08c798f344


I think I'd also add the starting hex into the logging, as well as the dest and path.


just putting a space between house names, judge-me :see_no_evil:


This introduce a BC break, 2.4 dev is the same as 3.x stable version.
Interface has been BC in 2.0 of this bundle.
Use 1.* branch instead and make compatible merge.


Yep, `PsiComment.value` would be a handy method, but is a bit tricky to implement. There are a lot of flavours of Rust comments, and all of them should be handled properly.


The following doc(s) have been deployed to the [1581cookie](https://1581cookie-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/cache-control](https://1581cookie-static-docs.pantheonsite.io/docs/cache-control)
- [docs/caching-advanced-topics](https://1581cookie-static-docs.pantheonsite.io/docs/caching-advanced-topics)
- [docs/clear-caches](https://1581cookie-static-docs.pantheonsite.io/docs/clear-caches)
- [docs/cookies](https://1581cookie-static-docs.pantheonsite.io/docs/cookies)
- [docs/debug-cache](https://1581cookie-static-docs.pantheonsite.io/docs/debug-cache)
- [docs/drupal-8-cache](https://1581cookie-static-docs.pantheonsite.io/docs/drupal-8-cache)
- [docs/drupal-cache](https://1581cookie-static-docs.pantheonsite.io/docs/drupal-cache)
- [docs/mobile-redirection](https://1581cookie-static-docs.pantheonsite.io/docs/mobile-redirection)
- [docs/pantheon-workflow](https://1581cookie-static-docs.pantheonsite.io/docs/pantheon-workflow)
- [docs/pantheon_stripped](https://1581cookie-static-docs.pantheonsite.io/docs/pantheon_stripped)
- [docs/guides/terminus](https://1581cookie-static-docs.pantheonsite.io/docs/guides/terminus)
- [docs/unsupported-modules-plugins](https://1581cookie-static-docs.pantheonsite.io/docs/unsupported-modules-plugins)
- [docs/varnish](https://1581cookie-static-docs.pantheonsite.io/docs/varnish)


Hi Mateusz, you do not need here sparkling-water-genmodel
I would prefer to tree TFIDF merge separately to simplify testing for now.


I hate this change but @Hidendra doesn't seem to want to update his own service.


TaskCluster: @terrence2 does not have permission to trigger tasks.


a working sample: @Mohammed-Gh

       <?php
                function func(int $x, int $z = 1) : void {

                }

                function func2(int $x) : string {

                }

                func(10); // with default value
                func(1,50);// with default value overriden
                func2(10);
        ?>

first `FunctionDefineNode` has nodeType of **TypeFunction** that was constructed by the defintion, the **FunctionType** has `uniqueName: func0`

seoncd `FunctionDefinenode` has `nodeType` same as above with `uniqueName: func21`

**now note here:**
first FunctionCallNode `func(10)` has nodeType of **TypeVoid** which is the return type of the function definition, and It has a `functionType` which is the **TypeFunction**, which we can grab the `uniqueName` from it.


@sayanchowdhury basic task issue is fixed. Now getting NULL constraint violation in blog_url.  probably document not able to parse it  properly. will check it later.
check if it is working in your system. Also why redis in requirement.txt ? I have tested it with redis though.


use the loop variable instead of stars?


@rodrigc
Today me too faced issue after upgrading to newer version. I am updating and immediately will release patch for this


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


GenerateEDDwhenOutputEMS (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2233 of 2233 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)


Note to self: remove this.


unnecessary space before (


use 'single quotes'


I know this comment is outside the context of a PR - but the knowledge of the exact (relative) location of the "root path"  doesn't feel like it belongs here.

It's really an environment detail that this class shouldn't have to depend on.

@mikeknep ?


indentation should be 4 or 2 spaces, but not 8


try the exponentiation operator **


This special configuration (only enabled in test mode) is quite useful to prepare for a full-SVG platform in new versions. Any particular reason why you commented it? (I understand it made it difficult to test the normal size of icons as per your other change above, but you could have done that out of the test mode, anyway...)


first


connectionstring snoepen


hello


bence iyi


Tradução lançada em 19/05/2016


Hi,

It would be great to add an icon for do-files in Stata.

Thanks!


119640277-User-file-fails-with-NAN-in-thermostat-error-message---may-be-due-to-room-air-model-#5533 (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (2201 of 2201 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 like this. But the text needs to be padded in a bit from the sides of the window. Right now it is just too flush for to seem comfortable. It feels squeezed or like its going off the page.


same


(2) Extra asset / column features MAX 30H


Fixe it! Don't push A here! It's mBloc that must be pushed, then mBloc takes the value of A...


/w\


unittest2 should actually only be needed on Python 2.6 and lower. On Python 2.7 you can just use unittest. A try/except would then be needed so we use unittest2 on Python 2.6 and unittest on 2.7.
But with the current tests we must indeed add this dependency. Thanks.


@matsen If I understand the semantics of `Eigen::Ref` correctly, this is more or less equivalent to returning a reference to `marginal_` and should allow you to use it as an lvalue. If it works that way, it's perfectly proper C++. The only other thing you might consider is adding back the const version of the method for use when the `Smooshable` object is const. (You can overload methods in this way if one is const and the other is not.)


I agree. I think this class/file probably has a little too much knowledge about where everything is. Seems like more the responsibility of the `config.ru` to wire everything up. I also am starting to question whether or not this file belongs in `lib/`. Kinda feels like it should live somewhere else—possibly root, possibly under an `app/` directory.


I recommend using http://eslint.org or better yet http://standardjs.com which would report such cases.


damnit, I searched for `opam info threads` and `opam info thread`. Didn't realize was base-threads. How can I include that now in the least painful way, back to the opam publish made dir and change opam file there? then what, `opam publish <?>`


@mcking65: I've squashed and merged this branch into master (commit 93f7cee7c4). If everything is as you expect, feel free to delete this branch.

Related aside: We should be (and used to be) able to use `rdef`, `sref`, and `pref` repeatedly in a single section and have the repeats magically converted by our scripts to use `code` so that we don't wind up with a bunch of links. However, that functionality seems to be broken currently. So if you're wondering why textbox (for example) is repeatedly showing up as a link, it's due to my having changed the tags deliberately from `code` to `rref` because this functionality should be in place. I need to figure out when/what broke the convert-to-code-tag magic and talk to Shane. Undoing the current link spam should be something done by script; not in spec.


Why the fuck is this back in code?


when will it be updated again ?


test


Congrats for this!
@loco-loco: When will you make a pull request?


What is going on with this comment?


this widget is wrote principally by beherith, i have only optimized some part.


yeah, I'm working on moving this stuff out of here now. Originally this was the entry point for the application, I just didn't move these things out yet.


GenerateEDDwhenOutputEMS (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1087 of 1087 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-33.36%25-green.svg)


Yeah, I realized that wouldn't work after committing - see #60, which is still open.


GenerateEDDwhenOutputEMS (JasonGlazer) - 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)


Await new updates :+1:


This shouldn't impact reading the normal ADC pins though


Since things are being done in-place, you'd end up modifying the wrong dictionary.

Also, the fact that you have nested dictionaries makes this a little bit trickier.


Oh...goddamit.
There's still some stuff on the [web.archive](http://web.archive.org/web/20080904210820/http://code.google.com/p/pcsx2-playground/) for the records.


Wasn't h/w encryotiin something brought up with the Nexus 9, and wasn't the general consensus that this was a bad idea because it forced the user to have encrypted data? I think the commit was pushed to enable it after a while, but it's definitely something to think about if you ever decided that you did want to. I'm not saying it's a bad thing. I'd for one love to see it. I just don't want to see users complaining that your version of CM forced encryption from the start.

I may have this all wrong though, and not completely understand it.


Because it didn't work, the pattern matcher was matching everything


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


:+1: with preference for the latter there are s few places in the docs that point to `master` functionality but not in the released vagrant.


@nthanvi I think a couple of comments got lost in other conversation.

- Use Misc.getFullTableNameFromRegionPath instead of custom conversion to keep the logic in one place
- rename "regionName" to something like "qualifiedTableName"


Am confundat problema 3 cu problema 2.


Probably you made this change by error, but it would be better to remove it.


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


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


DepricatedFields (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2202 of 2202 tests passed, 556 test warnings)

 * 556 tests had: AUD diffs.
 * 2 tests had: BND diffs.

![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 think I'd have to modify MovePath as I don't think it has a "getStartCoords" method or the equivalent.  Not a big deal though.


I'd make a copy of the default dictionary before updating it. I guess I'll just try and send a pr.


The patch looks good to me, but I think that I would explain on the commit message why this change is needed, and why it worked before. I know that I tend to be too verbose sometimes, but taking into account that this solves a regression I think that it would be good to explain it a little at least.


i have make it
the widget is on branch testing
i experiment some painful problem by working with github
This branch is 1 commit ahead of Tarendai:master.
i have try with pull rebase fetch merge and a large amount of combination of command
can you give me triks?


> mercury233
The commit at https://github.com/SalvationDevelopment/Salvation-Scripts-TCG/commit/f217c3c5fd1ff7ac414423752e814a3b612c82ce has the Chokoikoi fix


@remotevase can you do the grammer fix again after i finish the plugin code and fully upload it


>  the 0x4e was our old catch all "this seems to work" setting, but I can't remember exactly what it means without looking.

Is this what you were looking for? The [ancient SourceForge repo](https://sourceforge.net/p/pcsx2/code/HEAD/tree/) will get you pretty close. (r396)

```
void cdvdReset()
{
#ifdef _WIN32
	SYSTEMTIME st;
    //Get and set the internal clock to time
	GetSystemTime(&st);
#else
    time_t traw;
    struct tm* ptlocal;
    time(&traw);
    ptlocal = localtime(&traw);
#endif

	memset(&cdvd, 0, sizeof(cdvd));
	cdvd.sDataIn = 0x40;
	cdvd.Ready   = 0x4e;
	cdCaseopen = 0;
	cdvd.Speed = 4;
	cdvd.BlockSize = 2064;
	cdvdReadTimeRcnt(0);

    // any random valid date will do
    cdvd.RTC.hour = 1;
    cdvd.RTC.day = 25;
    cdvd.RTC.month = 5;
    cdvd.RTC.year = 2007;

```


This line should be modified to reflect the RC 1 and to be visible in the NPM search


adicionar o dado ba (bateria) no banco


No, I didn't: this recipe isn't enabled for any architectures.


This is probably wrong because it pass when there is actual error, condition should be:

```swift
guard error == nil else { return self.showMessage("Please download the API seed so that you can call it.") }
```


@ramnathv, been playing here tying `formattable` with `DT` and other table libraries just in case you might be interested


I forgot to do the Add Pages page. Will finish later today.
Not done with 8.


przypadkowy znal


You're thinking of the forceencrypt flag: https://android.googlesource.com/device/htc/flounder/+/android-6.0.1_r26/fstab.flounder#9

I don't want my device encrypted, so there's no way I'd add that to my builds. All I did was relocate the area where the encrypted key is stored.


Whoops


można ewentualnie przenieść spację za >


@dadarek , @mikeknep I tried moving this stuff to the config.ru file and it is giving me some trouble with getting the views to load. Can we make fixing this problem the next pull request?


Does it mean that now we don't have in response's errors that reason (i.e. wrong vehicle) but a thrown exception instead?


I'm not familiar with `gtest` but is there a way to warn/notify the user that this functionality isn't being tested?


@MrCK1 I fear you replied in the wrong place.
And anyway I had already saw that, and went even further back.

We were discussing of comments now instead.


Just saw this. Yes, definitely, leave it for a separate PR. As mentioned in Slack, I opened a PR that might help you with this.


Follow-up: It's not that something broke; it's that it doesn't work in list items. So I just opened issue 377 for that.


Thanks, I'll backport to 16.03


These should be #if defined(FEATURE_PAL) right?


@Tolo0  I can only say that, It will be updated again when it update again.
Need time to test or other something.
Maybe fast.
Maybe slow.


I tested this report and I had  error
ERROR: query failed: SELECT t.id, t.refer_date, t.reply_date, t.body, ut.organization, uf.facility_id, p.pubpid, CONCAT(uf.fname,' ', uf.lname) AS referer_name, CONCAT(ut.fname,' ', ut.lname) AS referer_to, CONCAT(p.fname,' ', p.lname) AS patient_name FROM transactions AS t LEFT OUTER JOIN patient_data AS p ON p.pid = t.pid LEFT OUTER JOIN users AS ut ON ut.id = t.refer_to LEFT OUTER JOIN users AS uf ON uf.id = t.refer_from WHERE t.title = 'Referral' AND t.refer_date >= '2015-05-04' AND t.refer_date <= '2016-05-19' ORDER BY ut.organization, t.refer_date, t.id

Error: Unknown column 't.refer_date' in 'field list'


By the way, I'm sorry for stopping your build 😢
("Thanks", Jenkins web interface)


This completely disables garbage collection, and leads to an unmanaged memory leak, as far as I can tell.

Is this intended?


same for
def addRecordingTime


Maybe a reasonable default here would be ``DataType.VARCHAR``?


`std::cout` :P


GenerateEDDwhenOutputEMS (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (2201 of 2201 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)


@carlos-lopez-garces Still very WIP, but this is to have stateful RNGs so we can actually test things like the selectors. It is a bit forward looking thinking that we might want different RNGs inside a GA (one for selection, one for mutation, I dunno) so we can name them.
I want to add a 'counter' of generated numbers so we can easily restore state on the RNG.

Thoughts?


No change to file name.


@Eluinhost I just uploaded that from my school pc so I can download it when I get home and cleanup the code/make it compilable


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


TaskCluster: @wanderview does not have permission to trigger tasks.


Hey @djangojack ! What exactly do you mean? It does currently have components?


Sorry, I should have separated this in another commit: see the new try & catch in `GraphHopper.calcPaths` which makes handling errors a bit simpler. But one can still add multiple errors like done for location lookups and return afterwards.


#30 Fixed


Use this instead: https://github.com/Muqsit/Genisys
I wanted to create a PR but it's just a modified version of the current pull request with fixed typos!


I have no idea at all on the topic to be honest. As soon as a stable router appears, I will make the needed research and updates.


At first glance it's now hard to tell if the AEGIS+GEMS+GOODS and simulated-agn samples are a double-peaked distribution in the same sample or not.. maybe make one of them the solid-line histogram?

Also I think the color would be fine to add back in for the online version, with a desaturated version for print...


refactor 👍 nowy feature pozwalający na wyszukanie childValueObjects wydzieliłbym do osobnego commita


Cuál debo utilizar? Estoy viendo muchas pero estoy un poco perdido


Oh hm I should probably have left this note on #123...


Yeah !!! Important if you don't add this placing items won't work !!!


This Is Second Working System


I dont agree the first one is easier to debug


Unfortunately it doesn't.


I dont agree it depends sometime naming temp things make it easier to grasp and to debug.


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


@Nunnery Did you change any java code in your fork?


TaskCluster: @mrbkap does not have permission to trigger tasks.


I forgot to mention: react isn't actually being referenced or bundled anywhere.


Awesome to hear that. Thanks for the clarification!


if 'Value3' in queuedData: data['value3'] = queuedData.get('Value3')


add-curve-unit-tests (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2251 of 2252 tests passed, 0 test warnings)

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


The first my_dict['foo'] = 3 is more explicit, you should say update with many line. Otherwise people will use update just for one value.

 my_dict = dict(my_dict, **my_dict2) is ugly just my_dict.update(my_dict2)


Should be using `<scope>test</scope>` for all these.


@jgw96 If you could add a single custom component to the default plunkr, it would make demonstrating issues related to components faster.


A space after the colon. Basically, if the issuer is Foo, the label would be "Foo: bar@example.com"

This looks much better than "Foo:bar@example.com", especially since it looks like the "Foo:" is part of the e-mail address (or username or whatever the label is).

As far as %20 in URI, I have in a few places. Slack would be one example.

Here's Google Authenticator's UI (random image search found this... just see how it would look). Just visualize what "Foo:bar@example.com" looks like compared to "Foo: bar@example.com".

![Example](https://www.sterndata.com/wp-content/uploads/2015-01-22-11.46.35.png)


Might be good to specify that this is an id here?


Thanks @davidbrandfonbrener! We should move your function to `pubmed_parser` folder too so people can use the function when they build the library.


OMG
https://archive.org/details/archiveteam_googlecode
Repo might be potentially there, somewhere.
We should find it first though.


Thanks Martino for contributing.
Lets for next time do not modify the master branch and lets create our own branch of the develop branch. Master is only after our final code is working perfectly fine then we will merge it with master.

Thanks


@dog194 ok no worrys.  keep up the good work 😉


I'm not sure; I've never even used the `using Foo: bar` syntax.


add-curve-unit-tests (jasondegraw) - 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)


I think those are good suggestions, and I'd like to keep the color.

@rjsmethurst, would you mind trying that? (and make it on a new branch with a pull request, instead of a direct commit? that's usually easier if anyone has followup questions or suggestions).


Remove space between ** and \`


Since t.refer_date was removed from the code by this commit, I'm thinking that what you tested must be the old version of the report.


@kamcio96 bukkit-plugin-parent (my version at least) isn't a fork. It's just a collection of common settings for my Bukkit plugins.


@IAHM-COL @hyde-tech

OK thanks!

Best
@vezza


Need to update the rst whenever the signature changes, and preferably run `make docs` and commit the rest of the rst change at the same time.


But, did you change something in code, since you forked project?


looks good, now fuck it up!


it could have been put on the end yes, but it's an important variable so its better practice to keep them in a good order


Yes I understand that, but it would require a lot of edits to the revive system, because kill data is held on the client and only broadcast once the player is truly dead. Crediting a kill on disconnect would require 10 times more work than just crediting the kill on injury.


Пожалуйста назначьте атрибуту id - Primary Key, Первичный ключ должен быть, и желательно чтобы он был "повешен" на уникальный идентификатор записи


@develop7 Thank you for the heads up! Fixed in https://github.com/hishamhm/htop/commit/645057d81aa619f4f9704cd221e63b357ac7f749


Nice. We should double check that we don't report events twice in native and webview


2235-OtherEquipment-schedule (kbenne) - x86_64-MacOS-10.10-clang: OK (2380 of 2419 tests passed, 0 test warnings)

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


@davidbenjamin @samuelklee @LeeTL1220 oops, I pushed this commit directly to master on IntelliJ rather than making a pull request.


ddfsfsdf


:clap: Thank you!


orly?

I guess that makes sense in this commit; in the subsequent ones when I have a real build step defined, some of these become hard deps, though, right?  at least workflow-steps-api?


Woooww)Awesome


I'm uploading a ton of notes


TaskCluster: @terrence2 does not have permission to trigger tasks.


In AmpMenus, no. There have been no code changes. Just distribution changes so that I could get it into my own repo for use.


Build fails at this line


T


@kieregh  `클릭`이라는 용어대신 `터치` 라는 용어를 사용하는 게 좋을 것 같습니다.


```js
if (!self.context.players.length) { // ????
```


#!/bin/bash or #!/usr/bin/env bash


TRUE
That is why I need to git pull upstream all the time.
Thanks Rod


I think you forgot to replace the last line with 2016_05_19_00_auth.sql update.


oops


latest commit to Heroku as of 5/19


Yes, but always reformat only your changes (never the _Whole file_ or unrelated code). Commits and PR should be clean. It eases the code review and later on looking back to these commits.

Ctrl + Alt + L is the shortcut for reformatting code. Before saving changes, use it and enable _Only vcs changed text_.

![Format code dialog](https://cloud.githubusercontent.com/assets/1794599/15406302/456ca7a4-1e06-11e6-9355-1795d64a5e84.png)

Maybe useful as well https://gist.github.com/robertpainsi/cb81b13b1008f6cf31f8 and https://gist.github.com/robertpainsi/cb81b13b1008f6cf31f8#gistcomment-1748867

There is a lot more to say about commits in detail (should compile, tests green, good commit message, be formatted, be atomic, no checkstyle, lint or pmd violoations...), but you may already know all of these :wink:


#5612-User-file-trips-on-new-fan-coil-control-warnings-Cold-Water-control-failed-in-fan-coil-unit (jasondegraw) - x86_64-MacOS-10.9-clang: OK (2194 of 2194 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)


Expected; this has an unsubmitted dependency --- https://github.com/pagespeed/mod_pagespeed/pull/1312 has a slightly outdated version thereof


os_2_0_develop_big_merge (kbenne) - x86_64-MacOS-10.10-clang: Tests Failed (0 of 0 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-NA-red.svg)


"wikipdia"? I think you mean "wikipedia"


Thanks for the addition !


till 20 may update


@jhofstee better to make that qt5 one a https:// checkout?


You have a tab or something here. Should just be an empty line.


ah, nvm, this is a new method going in the apache side, disregard the previous comment :+1:


Same here and probably elsewhere. Gotta turn on something in your editor to strip the whitespace from the end of a line.


@ejholmes what if we just passed in `&resp` here?

Since these are cloudformation provisioners, it seems like accepting `p.Provision(req, resp)` seems like a sane thing to do.

I'm not a fan of returning more than 2 variables from a function


s per discussion on today's tech call, we'll be pulling this whole assemblage out of the core ontology into a package with other similar material and getting it into `fcrepo4-exts`. Identifiers for the `NamedIndividual` will be changing to use hash URIs in the same fashion as the rest of the ontological equipment.


Enter should be lowercase.


Ok, I'll fix that ASAP


See #42


unless this enforces that provisioners always return a PhysicalResourceId and Data at compile time?


Is there any hope of fixing that? I find this setup completely unacceptable.


maybe it is better?
`public bool IsSparseTable => false;`


Hey, why not use there the real pcap file?


OK, so I'll postpone it a little bit and then probably make it conditional in nixos based on your selected driver.


Good job, quite clean to compile c-exts with Sulong and the tool :)


I'll rename it to "WriterFactory", I'm open to any suggestion.


Fixed.


Pressurestat (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (2203 of 2203 tests passed, 556 test warnings)

 * 556 tests had: AUD diffs.

![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)


add-curve-unit-tests (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1105 of 1106 tests passed, 0 test warnings)

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


Delivery Status:
----------------
|Verify|
|:--|
||
||
||


Totally xD


Non-uniform function arguments


OK, removing it. Known side effect is that it will not revert to fallback language of English if language file is not present. Seems to  need registered available languages for fallback to work.


I know a shortcut for this :)
`File.expand_path(CONFIG['out'], CEXT_DIR)`


Yes.... hmmm. The code seems to be done well enough to run.


If it seems safer with import, let's use that...


I already took that into account on server-side.


Ajax does not work with local file system. One may need to host a local server. The simplest would be : python -m SimpleHTTPServer 8080. Still too much work. Might shun this approach. Hopefully setting up Firefox Extension is easy. They start using npm now and I heard Node is not good so we will see.


I think I have a couple broken tests with this, will update this or the tests as needed in a followup.


> [](url)


I had a look on archive.org (where they are pointing) the repo is very sporadic, some of it is there, some isn't, it's not really a lot of help as we can get the latest repo direct from M$ still.


보다 정확한 용어는 탭핑 이네요. 아래 예문을 참고해 주세요.
---
backdrop 을 탭핑하가나, 테스트탑 테스트에서 키보드 back키를 클릭하여 취소할 수 있습니다.
There are easy ways to cancel out of the action sheet, such as tapping the backdrop or even hitting back on the keyboard for desktop testing.


Yeh I didn't want that setting as it isn't right, especially for here.


see several issues on transitioning the rest of the docs enyirely to markdown. we're not making the same mistake of redoing the docs right before a release again, but hopefully during 0.6.


:-1: to that function.


Tutores, IGNORAR ESTE ARCHIVO


Thanks :stuck_out_tongue_winking_eye:


👍 I forgot. Will do so in the future.


I was wondering if this was possible because numberOfItems is a primitive int. Therefore it cannot be null - it will always be 0.

Happy to add it to future proof if you think it's still necessary.


I don't believe that "The monks let you" should have been changed, due to the pluralization of "monks".


I don't like very much the idea to have all the fields even when you don't need them. But if this this the way we want to proceed it's ok. We need to format the mood, too. @michelangelo what do you think?


@lucaswoj why was this changed to add the listener in any case, with empty in case of `false`? Just wondering.


The `<body>` tag should always go immediately after the `</head>` tag.

A basic HTML5 template usually looks like this:

```
<!DOCTYPE html>
<html lang="en">
  <head>
    <!-- HEAD STUFF ALWAYS GOES IN HERE -->
  </head>
  <body>
    <!-- ANYTHING THAT IS MEANT TO BE DISPLAYED IN THE WEBPAGE ALWAYS GOES HERE IN THE BODY -->
  </body>
</html>
``


ip-table-conversions (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2232 of 2232 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)


minor: i think if `this.state.url` doesn't exist, the `certErrorAdvanced` button shouldn't be shown either


Any noticeable effect?


oops, that's actually what I  meant to do. Trying to do too many things at once


A `<div>` cannot go inside header tags such as `<h1></h2>`. If you need a block-level element here, consider using `<span style="display: block;">` instead of `<div>`. Although I do not think you need anything inside the `<h1>` at all except the text. I will cover block level elements in next class.


`<br>` is obsolete in html5. Consider either `<p></p>` or `<wbr />` or css `margin-bottom`.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 141](https://teamcity.labkey.org/viewLog.html?buildId=325417&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 8 (2 new), passed: 47 Build time: 01:41:20
### Failed tests
```
org.labkey.test.tests.cds.CDSTest.verifyHomePage: <no details avaliable>

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

org.labkey.test.tests.cds.CDSMeasureStoreTest.testDemoMeasureStore: <no details avaliable>

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

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

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

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

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

```


Good job on using classes.


Brent, Have some questions, how can I contact you?


The end body tag `</body>` should always be the last thing before the end `</html>` tag.


@michaelsproul  @auchri  Could anyone help me generate the application to my messenger ???
I am unable to have any tutorial
------
Alguém poderia me ajuda a gera o aplicativo para meu messenger???
não estou conseguindo tem algum tutorial


It is good practice to write standalone tags as `<hr />` with the trailing slash.


lgtm


Stephane, thank you for posting the detailed instructions for Ionic/Cordova for Wakanda.  My question is, can you provide details regarding what is necessary to be able to do the same thing, but with an existing Solution and Project that was created over the past several years, before the App Factory?  I am expecting that there is more that needs to be done, beyond copying the mobile folder structure and modifying it for the existing model/classes.

Thank you again for the time you invested to set up the NSCurious article for new projects.

Kirk


合并


{"print $0'}


Oh, I see. I don't use permissions, which is why I don't understand them very much XP
No problem for the grammar. I do that for lots of people


Thats expected. By default PREFIDA will ignore any extra variables found in
the inputs structure. This behaviour allows for variables that are device
specific.

On Wed, May 18, 2016 at 5:47 PM, gzhao2014 <notifications@github.com> wrote:

> A more question. Do you already update check_struct_schema?:
>
> [image: 9999999]
> <https://cloud.githubusercontent.com/assets/15125494/15379270/8dddc338-1d39-11e6-9034-f71ba375f63b.png>
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly or view it on GitHub
> <https://github.com/D3DEnergetic/FIDASIM/commit/dba2d559431822df85b44403c38d2a131f810e4c#commitcomment-17533310>
>


Oops :stuck_out_tongue:
Fixed


second paragraph


samplemovie


Thanks @tmeasday, didn't know that existed.


sounds good when should I come over


@bradjc @brghena check it out


TaskCluster: @hashedhyphen does not have permission to trigger tasks.


DWSim Tutorial


TaskCluster: @aethanyc does not have permission to trigger tasks.


Fixed in 6ba0aedaf8bdbdbe69b9bc1876ebd3668c8e1cca


Told less indentation, more comments, slight perf boosts, etc. Still not sure why github doesn't show jengaroot.ml as binary even enough I've marked it in .gitattributes. Works locally in my view.


ip-table-conversions (nealkruis) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1086 of 1086 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-33.35%25-green.svg)


ip-table-conversions (nealkruis) - 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)


add some to readme file.


In case you're interested, this one requires a logic shift (due to avoiding the method where you switch the pin between input and output [but by default set it to low]):
[https://github.com/NicoHood/Nintendo](https://github.com/NicoHood/Nintendo)

Different style, I believe NicoHood followed that path due to potential issues (or just for clarity without such a trick):
[https://github.com/brownan/Gamecube-N64-Controller/issues/2](https://github.com/brownan/Gamecube-N64-Controller/issues/2)

Side note: The current method assumes that, after the setup() code, setting pin 2 to an output pin will make it drive low. That may be hardware dependent (for sure works on Uno but may need to double check on your boards). Also, the pin register locations are also hardcoded in AVR, so that may need to be adjusted if your boards are different.

Oh, and current state in this repo is that I can read transactions well, can read from a controller, but having trouble acting as a controller and responding to the initialization probe command.


response added


- [ ] vars set/get & passing
- [ ] build tools


Haha, phone tried to change Readme.md to that


'npm run dev'


Ojo esto
`$http.get('http://localhost:8081/movies?Func=filter&Filter=popular')`
debería ser
`$http.get('/movies?Func=filter&Filter=popular')`


Thank you!


Si hace eso yo estoy haciendo pruebas para usarla en tryton :)


$ git init


ojo que en el $http en angular estabas diciendo que la url usaba el puerto 8081 aca dice el 3000


I think the line should be removed too. But looks like this will break mouse hovering(`onMouseLeave` and `onMouseLeave`). Is there still on going work on this?


Find longest sequence of zeros in binary representation of an integer.


Ya no se permite Anonimos


acordate lo que mostraron los demás, ellos usaron
`app.get('/movies', function()....`
para escuchar todos los get a /movies
`app.get('/generos', function()....`
para escuchar todos los get a /generos
Eso te ahorra todo este gran IFs que tenes por lo menos para saber a q GET estas escuchando


Right, "conservative update" sounds like it.

> It adds a small cost to improve accuracy (read counters, find smallest, update)

It may also reduce the cost as not all counters get updated. But you ran the simulator and I'm just guessing.

> I think you mean "admission filter" vs "admission filter + conservative update ".

Actually, to me all four possibilities seem to make sense. But I really meant what I wrote as both features are designed to make the counters growing slower (or to allow to use less memory), right? I expected CU to be really helpful, while you proved me wrong. This shows how guessing meats measurement. Thank you for your very informative answer.


good point, I'll find a way to test it before I merge this PR


2233-baseboard-enhancements (kbenne) - Win64-Windows-7-VisualStudio-12: Build Failed

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


#412 appears to be completely fixed after this.


为了新的测试，删除


bfraser@geoanalytic.com


so is there any  ESB4.8.0 patch for this issue?


ip-table-conversions (nealkruis) - i386-Windows-7-VisualStudio-12: OK (2200 of 2200 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)


Please make the same changes in Font class. Pull request https://github.com/PHPOffice/PHPExcel/pull/900


Ahora se desactiva el boton al darle clic no? Bien hecho! Cada que hagas un cambio especifica que cambiaste.


This just feels wrong, I don't know why. 😭


Confirmed working. Many thanks!


why this is a wrong assertion?


@asaf could you plz release a new version for this patch?


Yes, I agree, I'll fix this.


OK. We will try it. Thanks!!!
BTW, it says in its README.md file that it is not intended for third party use, "because there are no guarantees of API or ABI stability". But I guess it is our best bet for now.


GenerateEDDwhenOutputEMS (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1645 of 1645 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.96%25-green.svg)


We decided not to use region in our model.


javadoc should be here, not before package declaration, that is the place for copyright header of the class.


there is a (working) build of 1.9.4 forge now


The following doc(s) have been deployed to the [1616format](https://1616format-static-docs.pantheonsite.io/docs) Multidev environment:
- [docs/code](https://1616format-static-docs.pantheonsite.io/docs/code)
- [docs/create-wordpress-site-networks](https://1616format-static-docs.pantheonsite.io/docs/create-wordpress-site-networks)
- [docs/external-libraries](https://1616format-static-docs.pantheonsite.io/docs/external-libraries)
- [docs/migrate-wordpress](https://1616format-static-docs.pantheonsite.io/docs/migrate-wordpress)
- [docs/settings-php](https://1616format-static-docs.pantheonsite.io/docs/settings-php)
- [docs/wp-config-php](https://1616format-static-docs.pantheonsite.io/docs/wp-config-php)


@valib cool. Are you happy for me to do a release this weekend? There isn't much but I think it's enough to add a lot of value for some users


This breaks the build --> https://travis-ci.org/dashpay/dash/jobs/131496337#L1485


you are mixing the concepts of triangular and rectangular tiles, they should be separated in specific abstractizations.


where is the price used ?


@0x0FFF can you submit a PR on gpdb4 ?


Thanks. Done https://github.com/jembi/bsis/commit/3dcd505de73baed6141c76ff70c34f967e394a28


@lucaswoj It looks great but I think the first test should be removed.
Thanks a lot for the help!


good! this makes a lot of sense!


Changed in the same commit referenced above: https://github.com/jembi/bsis/commit/3dcd505de73baed6141c76ff70c34f967e394a28


this should be in triangular abstractization


Waiting for the outcome of my comment above about adding a required check for numberOfUnits.


0.8.0 is released https://github.com/TryGhost/Ghost/releases


Awesome!


the price here is for a piece, why do you use the calculation for surface ?


What are these about? We'll always target latest Ruby so do we need version checks?


this should be the triangle  abstractization specific, has nothing to do with a generic tile


Thanks for catching this.


Hello from the other side!
Have Fun & Take Care!


cc @harrisonkrat


I'm usually wary of moving code like this because it breaks git annotate.
and makes it harder to track the change history...