I think you are right.

This pull-request is more than needed, but you can believe me, that it is to hard make changes one by one and after every change in code wait 15 minutes to check the reboot problem. So I just take full method from original SDK and its working properly.

Main question for me: Isn't it just delay problem beginning moment? May be it can repeat after 15*X (where X is divider) minutes or some other interval?I think we should understand how it happening to be sure.


How about something like https://gist.github.com/torarnv/2ad85d424faeb469d201 ?


True but let's wait the initial feedback of advance tester


That's so weird. `cls.view` will usually just wrap it with `cls.wrap` by default...


Вот по контрибьюторам я бы советовал что не стоит пока - они и так светятся на github. Здесь действует другая особенность http://oss-watch.ac.uk/resources/cla

Ты тогда должен будешь указать и финансовую составляющую контрибьюции - лично я не рекомендовал бы это.


Uhh I don't see how this changed anything? Or why anything needed to be changed? the level doesn't reset the progress bar... the progress bar and level are only set by the score. And the level increased AT 100, 200... rather than over 100, 200, points... so all you changed was that it levels up at 101 and 201 now?...


I, like Anakin, will succumb eventually. :P


that's just for development. we can always add postgresql later...


actually the font size is relative to the browser/device default size, which makes sense for me.


Can this be squashed into `1c533d2`?


@kordianbruck would you please enable issues and pull request for this repo! We can track things which need to be done via issues. This allows me to assign my self to features I'm starting to working on etc..

Just an idea: What about sending pull requests before committing something? This way can do a peer review before committing code instead of reverting things. May it work, may it will not work. Never coded something in such a collaborative way.


Yes! intended behavior. I'll continue to improve upon this but let me know if I can close this up.


The problems is here. You need to check if your start_time and stop_time are valid.
I also don't like the idea of hiding short runs silently and unconditionally.


How are you going to keep this in sync with the canonical source?


Fixed in d36a9ee


la universitatea de vest


Squashed gist into updated PR


MarkdownDocs (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1677 of 1677 tests passed)

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


Ah, great catch. I think this in particular is the line that will cause problems?


@dio85 *clap*


because their coordinate system was defined differently from ours. I went through and changed them to have the same coordinate system.


By redefining the coordinate system, we no longer needed a yaw argument.


Looks like the right arm has an updated forearm and palm while the left only has the palm. Was that on purpose? Were the inertias wrong before? And can we squash this into some previous commits?


see above note


This is a good idea. I see you are sourcing contrib/intel_env.sh from run.sh, which makes sense..  Why do we need the contrib/intel_env_vars_list.txt in addition to intel_env.sh ?


kadamski I think you author of the library so you have more complex vision of it, and will be better if you'll make requried changes as you want, but I think we should be sure what problem wouldn't apear again in different time. Did you have ideas about that?


@j6mes - just want to double check before I get rid of this - it's only there for debugging? Don't want to delete and re-create the view everytime.


@mkresin you mean the spaces in the javascripts? Huh, don't really know how to change that in this commit. You got some manual or howto?

Issues are now enabled. PRs should be on by default, but you will need to use branches to create a PR to this fork of miniflux. Make sure to create select kordianbruck/master as the compare repo.


Interesting. Looks like jython matches any portion at the beginning of the string, but python does a full match on strptime

```
$ jython
*sys-package-mgr*: processing modified jar, '/System/Library/Java/Extensions/QTJava.zip'
Jython 2.5.2 (Release_2_5_2:7206, Mar 2 2011, 23:12:06)
[Java HotSpot(TM) 64-Bit Server VM (Apple Inc.)] on java1.6.0_65
Type "help", "copyright", "credits" or "license" for more information.
>>> from datetime import datetime
>>> datetime.strptime('2015-04-01T00:00:00Z', '%Y-%m-%dT%H:%M:%S')
datetime.datetime(2015, 4, 1, 0, 0)
>>> datetime.strptime('2015-04-01T00:00:00Z', '%Y-%m-%dT%H:%M:%SZ')
datetime.datetime(2015, 4, 1, 0, 0)

$ python
Python 2.7.5 (default, Mar  9 2014, 22:15:05)
[GCC 4.2.1 Compatible Apple LLVM 5.0 (clang-500.0.68)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from datetime import datetime
>>> datetime.strptime('2015-04-01T00:00:00Z', '%Y-%m-%dT%H:%M:%S')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/_strptime.py", line 328, in _strptime
    data_string[found.end():])
ValueError: unconverted data remains: Z
>>> datetime.strptime('2015-04-01T00:00:00Z', '%Y-%m-%dT%H:%M:%SZ')
datetime.datetime(2015, 4, 1, 0, 0)
```


What is this


I think these should be `checkNotNull` to match the rest of the code base


done.


@AliSoftware: As discussed, this could be reverted. Better this is agnostic about the existence of asset catalogs. Reverting will cause that some additional tests fail, because the fixture podspec is misconfigured in that way.


Thanks @hyclak :+1:  I hereby merged your changes into master. This module has been on the forge for quiet a while, but a couple of months ago I decided to remove it due to the my use of puppet and a lack of time. Since the usage of puppet changes from job to job - I'm willing change the module to the latest structure/guidelines and push it to the forge.  Are you comfortable helping and/or giving feedback? If so, let's do it :-)


Ok you are right, it looks like it works, thanks!


some values got updated in thor cad and roped into that


renaming commit


Mike, I think that it would be good to exit here with some kind of error code/message. See a call to exit several lines down....


MarkdownDocs (axelstudios) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1172 of 1173 tests passed)

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


Yeah, that was a bit of "early morning" (I just got up a bit ago) flailing.
I had previously changed it so that the level switched at 100, 200 (instead
of 101, 201 like you'd had it). Then I noticed a bug I thought I might have
introduced with the change, so I fixed the bug (order of operations), but
in the meantime I undid my change.

In the end, I just simplified it slightly and moved the progress bar
handling to the score class, which has the added benefit of animating the
progress bar.

On Tue, Apr 7, 2015 at 1:19 PM, Falon <notifications@github.com> wrote:

> Uhh I don't see how this changed anything? Or why anything needed to be
> changed? the level doesn't reset the progress bar... the progress bar and
> level are only set by the score. And the level increased AT 100, 200...
> rather than over 100, 200, points... so all you changed was that it levels
> up at 101 and 201 now?...
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/falon3/Wordless/commit/5a672b98dc17bcca7298c3cdb9c72058f5ae9442#commitcomment-10609709>
> .
>


Interesting. I'll give it a try.


Same thing as the comment I just made for smackverify.py. I think that just calling sys.exit() means that you are exiting normally. So in erroneous situations I think that you want to invoke exit with an error message.


The nice thing about this script is it only relies on the std python libs. I think adding the requests library will be better for a fully functional project, but this is meant as an example script and is extremely portable.


I can't wait also! 8-)


haha :+1:


It looks like this relies on the knowledge that the "this" variable will be the first one in the iterator. Add a `checkArgument` to verify that


If you get this file from upstream at http://www.dnspython.org/kits/1.12.0/dnspython-1.12.0.tar.gz you can eliminate the unzip dependency.


HeadlessFixes (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Unfortunately git wouldn't easily allow me to add only the relevant lines changed. This is to add the warning message string that we want to filter out so that the user and the tests don't get alarmed.


I've filed rvagg/nan#306 asking whether TLS would be in scope for Nan.


relevant change


from [W3C](http://www.w3.org/TR/css3-values/#rem-unit):

> **rem unit**
> Equal to the computed value of ‘font-size’ on the root element.

where *root element* is the *html tag*


Speaking of which, checkout [the latest news](http://spark.createsend.com/t/ViewEmail/t/5A49A30D85B7B8ED) on the Photon timeline.


Welcome aboard!
http://sidekiq.org/about/


done.


There should be type checks that return an error if input is not an int, e.g. float.


Should be lowercase.


That was on purpose. They weren't necessarily wrong, just a little off. I'd
rather not squash them to retain the history, as this was over the course
of several days.

On Tue, Apr 7, 2015 at 3:25 PM, Nikolaus Wittenstein <
notifications@github.com> wrote:

> Looks like the right arm has an updated forearm and palm while the left
> only has the palm. Was that on purpose? Were the inertias wrong before? And
> can we squash this into some previous commits?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/TRECVT/hdt_7dof_description/commit/c95012b6227644b0b6a609ac144bf76c4076157f#commitcomment-10609829>
> .
>


Pre-existing, but this `record_superlifetime` call seems weird -- is that just a holdover from when closure bodies had a parent region?


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


This one getting wierd.
On the newly installed babun (dist-1.1.0 installed in a different `/targer` folder) the `open shell here` doesn't work with the zprofile being sourced.
Commenting this line out gives me no error messages and opening shell works fine. I'll double check with the original #205 issue and try to trace it back


Il manque un espace sur cette ligne ;)


cuillère, sacre bleu ! :anguished:


This one should be BEAM on %s (add "on") :)


`checkNotNull`


Yeah, when it's ready for primetime I'll fix the UI. Just let me know.


Haha, je l'attendais : http://fr.wiktionary.org/wiki/cuiller


"Cette orthographe était autrefois nettement majoritaire, mais elle perd du terrain, tout en restant courante."

Place à la relève, ma proposition est plus *hype* désormais :stuck_out_tongue:


Why no 'includes', 'does not include', 'is included in', or 'is not included in' for email?


lgtm


Fixed. Odd that JSCS doesnt catch it. Maybe it would be an idea to include all files in the check?


10000 -  краще уникати таких значень у коді, старайся прописувати іменовані константи


This appears to be an error—`string_to_datetime` takes a string and returns a datetime, whereas here it's passing in a datetime and expecting a string. Probably meant to use `json_format_datetime` or something.


can anybody post me were this is changed? can't find it


The build team has reviewed. Please let us know if this needs back-porting.


Чудово, оновлюйся з dev та мерж


I think this will fail for `void.class`. Add a check that the type is a reference type


Shouldn't line 168 be like 39?


I pushed the test with symmetric values, 50,000 games, only mg values (32), and -100,100 limits. The uncertain thing for me is how the double equality works.


@kalashnikovisme мб стоит в названии обозначать, что этот метод формирующий url. Кирилл у себя делал вида `profile_curl` (типо custom url). У меня curl вызывает не те ассоциации, мб можно сделать более длинное profile_custom_url, или просто profile_url, хз.
в любом случае стоит обозначить, что делает эта функция, например - social_profile_url


Can this be lowered to ```aeson >= 0.7.0.3 && < 0.9```? 0.7.0.3 is the version that will be in Debian Jessie. It would be nice, if gitit2 could be packaged and backported for Jessie.


HeadlessFixes (axelstudios) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


@kalashnikovisme  а зачем?


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

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


I follow a recommendation that variables should be declared in the "smallest" scope possible. In this example
first is only used in the consequent of the if and so might be better declared in that scope.

Then, purely as a style thing, one might choose to do

if (int count = et->constants.length) {
  bool first = true;
} else {

}

I wouldn't have done that before starting with Chapel but I see a lot of our code leans on this
newer C++ declaration style.


Why do we need this string to be a translateable resource? I thought we purposely were not installing all the localizations of Hg, so won't we always get an English warning? Can't we just put the string in C# as a literal?


For documentation purposes,

This PR fixes missing MIDDLEDOT (U+00B7) support in hashtags. It was requested in issue 4, see https://github.com/twitter/twitter-text/issues/4


check


:fireworks:


/team size?


should we even include print() anymore?


maybe, but I did try some different things here, and I vaguely recall thinking that this was in fact necessary.  I'll try removing it, and if it turns out to be necessary, I'll try to add a comment elaborating why.


@prashanti does this look right?


this can be done with streams


TODO?


Der er mange måder at laver compilers på... måske skal vi specifere at dette blot er en af dem?


"statement constructs" ? Måske noget om scope eller type i stedet?


It would be ideal if we can remove it for all validators, but I'm not sure how to do that. Since the ScreenNameValidator only provides an interface, we can't have implementation in there to remove the Slash. Should we create a base validator for this?

If not, maybe we can just document it in the portal.properties? That way, developers who want to create their custom validator are aware of this.


nu am mai testat aseara, o sa incerc sa vad acum de ce nu mai merge


constructed* og GABMLE som reference til acronymer


HeadlessFixes (axelstudios) - x86_64-MacOS-10.9-clang: Tests Failed

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


Is this accessible securely? You may need to pull this down and serve locally.


Actually, I think the solution is going to be a bit more tricky because the instance number is relative to the series, right? So I think we're going to have to track the series number here and reset the instance number counter when we get a new one.

Can we assume that all dicoms from one series are going to be written before any dicoms from the following series?


Not sure this is available securely.


The problem is at this point we don't have the internal console, and so any type of error while importing Pandas makes Spyder to crash :-)


MarkdownDocs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


@kalashnikovisme
def has_auth_provider?(provider)
   authentications.find_by(provider: provider)
end


great commenting going on here...


@Ultra2D

if you're referring to `!isset($config['auth_ldap_group'])`, it is there because that is what existed previously.  The attempt is to not break backward compatibility.  If that portion can be removed, then great.  I don't have the environment to test, though.  I `unset()` that variable in my config, and there aren't any issues with the current code.  If I remove the `!isset($config['auth_ldap_group'])` portion, it still works.  I can remove it if it's superfluous, but it was legacy code and I'd prefer to not break someone else's installation.


Ooo nawet nie wiedzialem, jutro dodam.


Does the Flickr slideshow work with the https:// switch?


27 75 00 00 f5 00 06 <.......login packet......>

note: i replaced any identifying information with repeating digits.

```
10101 0000f5 0006
    0000001e00841691
    00000028:66666666666666666666666666666666666666666666666666666666666666666666666666666666
    00000007 00000000 00000001
    00000028:39366661313030646161366135376566326232313932396438346561626161303932366631356437
    00000000
    00000010:66666666666666666666666666666666
    ffffffff
    00000008:47542d4939313935
    001e8483
    00000002:6573
    00000024:66666666666666662d555555552d333333332d444444442d666666666666666666666666
    00000005:342e342e34
    01
    00000000
    00000010:66666666666666666666666666666666
    00000000 01
    00000000
    6734ac91
```

the rest i don't understand, that does not look like clash of clans.

```
37 b6 00 00 1e 00 00 cb 86 a3 4d a2 d8 39 f4 0c 6f 36 31 ab 8a 3a 2a 59 99 3b 73 37 89 65 65 a9 81 9d 33 2f 77
37 16 00 00 0c 00 00 af 1c 99 fd 86 eb 89 6c de b5 c4 7a
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 7a 7a 44 d1 ad ad 1a 7b b5 5d 3f 28
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 93 e9 6a 2b 66 ac f1 14 99 ad 92 86
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 2e 0f 19 f6 71 be 3f a1 a1 b3 5b 4e
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 5e 72 88 18 b7 44 ef a1 4e d6 0f 52
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 06 81 f2 b1 04 83 9b de f6 69 2f 4c
27 7c 00 00 00 00 00
37 16 00 00 0c 00 00 ac cd 96 fb 99 ae ed 06 80 6c 46 44
```


I think the `s/chapter/section/g` is a bit too violent :)


Is this available securely? (Is this available at all? If not, maybe we should pull this test.html file)


i'm running the binaries not the source - so not sure i can 'patch; that readily - i see what your suggesting in the code snippit  add the Null pointer check - however why cant it find the spring loaded jar ?  is that to do with the gradle.build file ?  and spring boot bits ?  I'd have expected it to be there no?


This should be text.is_a?(String) || text.is_a?(doc_class)


Show Tiago, ficou top!


Make a pull request, this looks good


sdfasdf


I don't see jquery-1.6.min.js in the /media/js/libraries directory.


fds


May be it also can be related to overflow problem? If it is 312.5 times larger.


este Then ficou muito longo,deveria encurtar ele ou então fazer uso do And


In this particular case I agree that it's a problem, though if I move this block there might be repercussion with other combinations of sentences. The current filters are arranged to be on the conservative side.


Yes, as Ramon mentions, Python is now under the "Air" category of DroneKit, which implies a companion computer.


This should be the position parameter, `position` not the spawn point


Didn't see the bottom of the file!


Wherever possible never use a Singleton (UmbracoContext.Current). We try to expose an instance of UmbracoContext or ApplicationContext on anything that might require it.

We are trying to move away from singleton usage as much as possible in the core so that things are unit testable.

In this case, this needs to be:

     docRequest.RoutingContext.UmbracoContext


poderia apenas ter dito que os dois apareciam na lista, e que "Restaurante Universitario" aparecerá antes de "Cantina Area 2"


As I said in https://github.com/SiCKRAGETV/sickrage-issues/issues/1122 the hardlink method is working great in Windows 7, and it's supposed to work on Vista too (or any modern Windows version to that matter). Symlinks are also supported (NTFS6), so even when I haven't tested them as much as hardlinks, they should work too. Why are you removing them?


Ehrlich gesagt bin ich mir da auch unschlüssig. Eigentlich hab ich keine deutsche Übersetzung gefunden, die dem englischen Original so richtig gerecht wird.

Ich gehe davon aus, dass man im Bordell steht, da Durmsey IMHO (aus dem Textverständnis heraus) eine männliche Prostituierte ist, die im Bordell arbeitet. Das macht dann auch den Satz noch problematischer bzw. schwieriger zu übersetzen...

Auch das Textverständnis ist hier gar nicht so leicht. Warum gestikuliert er und warum zuckt er mit den Achseln? Was will er dem Spieler damit in der Situation sagen? Meine Interpretation ist, dass der Mann die leise Hoffnung hegt, dass der Spieler ihn aus dem Bordell herausholt, gleichzeitig aber resigniert ist und etwas Derartiges nicht wirklich erwartet. Man könnte den Satz natürlich im Deutschen mit drei oder mehr Sätzen umschreiben, aber das wäre dann schon eine starke Abänderung des Originals und imo nicht meine Prio.

Wenn er aber tatsächlich vor dem Bordell stehen sollte, dann passt meine Übersetzung natürlich nicht, das ist klar. Ich hab das Spiel leider nicht installiert hier auf dem Laptop und bin auch nicht daheim, daher kann ich das gerade nicht selbst im Spiel checken...


Sounds good -- should be nice for quick testing of more complex maps! It is very frustrating to have to think through all the back-connections when manually making maps.


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


Why not just use olive oil?


For the record: from now on we have a recommended OpenCL driver version (only a hint).
>= 346 -> NVIDIA
== 1573.4 (14.9) -> AMD


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1683 of 1683 tests passed)

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


deveria especificar que agora a lista que ja estava sendo mostrada na pagina "Geradores de Residuos", esta mostrando a lista organizada por volume de residuo


The reboot part is straight out of jumpy installer code :).


@HowardHinnant That is broken. Good catch.


# Github user repo created for CIT 281 15S


seriously I'd hold off. I'm _so close_ to 100% backcompat and providing these patches and more. I would love your help digging through the 14 remaining failing test cases (which are probably nitpickey edgecases, I think i've fixed all the glaring-refactor mishaps). Once I get my branch passing existing unit tests 100% I intend to add tests to duplicate a lot of these fundamental option-toggles and deprecate the old tests in the process. (the tests can go away when the support for the syntax goes away)


<?php

echo "hello word!";

?>


<script>

alert("hello word!");

</script>


what is the value of using (above) identical(.id, NA), whereas here you use !is.na. Given that @hadley recommends identical(..)? would it not also be more consistent to also use !identical(.id, NA)?


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


91244204-HeatExchangerAirToAirSensibleAndLatent-Supply-Air-Outlet-Temperature-Control-problems-#4794 (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1174 of 1174 tests passed)

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


Sure. May be change email to homepage . http://harikt.com . So it don't get
spammed :) .


LGTM


time模块好像没有用到


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


@biddisco: John, may I ask you to try to conform in your formatting style to the surrounding code? For the code above this mainly boils down to adding a space after the comma in argument lists. Thanks!


这里要两个空行，请参考pep8标准，或者运行 pylint 加文件名检查一些code-style


backup_new（）属于 back模块的功能，utils 模块放的应该是backup和restore共用的代码，这个函数显然应该放到 backup.py 里面


nihao


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 10](http://teamcity.otenv.com/viewLog.html?buildId=1240811&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) is now running


TeamCity Architecture Team :: Java Platform :: otj-logging :: build-config [Build 10](http://teamcity.otenv.com/viewLog.html?buildId=1240811&buildTypeId=ArchitectureTeam_JavaPlatform_OtjLogging_BuildConfig) outcome was **FAILURE**
Summary: Exit code 1; exit code 2 Build time: 0:0:0


this is a test


LGTM. (Sorry, was that my bug?)


"[subtype] you control" handled in MagicTargetFilterParser, there is no need to add a custom rule for it


Corrigindo, exercicio 3.2


I corrected the capitalization


So I took a look at the branch in your fork, and I don't feel comfortable with changes as massive as that in a single branch.  There _has_ to be a way to do this in small iterations and not such a massive change, and there are way too many things to discuss in that set of changes for us to be able to take small parts of it and discuss other parts of it separately.

This set of changes addresses some of the specific issues that @Sergiy-Babich had opened pull requests for, and I would like to get his stuff in and attempt to iterate on top of it, I don't see the benefit of blocking these changes for something so huge that could take a very long time for us to agree on.

The type of changes in your pull request will take some time to talk out and think through and we should be in a position where we can iterate and reflect, not be "stuck" debating various ways of changing what is essentially a re-write of the whole code base, thus preventing anything else from going on in the meantime.


@SubJunk this is premature imo, we should have a few people try it first and see if it actually makes any difference before editing the defaults.  It's easy enough for users to test on their own.


Thank you lianxu!

The idea is good. But you use sizeof(1024*1024*1024), which is actually 4. Do you mean to set all the 1024*1024*1024 bytes to zero?

PS. A space after every comma is welcome,:)


Hi  iulukaya,
                      this code not working for me its return me this

An client error occurred: (list) unknown parameter: 'location'

can you help me on this whats issue when i'm searching youtube video via location and radius
see the parametere :-1:
  $searchResponse = $youtube->search->listSearch('id,snippet', array(
        'type' => 'video',
        'q' => $_GET['q'],
        'location' =>  $_GET['location'],
        'locationRadius' =>  $_GET['locationRadius'],
        'maxResults' => $_GET['maxResults'],
    ));


Hey, great job so far! I'll accept a pull request when it's done, thanks!


This is in relation to this pull request: https://github.com/jnicklas/capybara/pull/1219

Unless I'm understanding this incorrectly (it's certainly possible!) these warnings are actually not supposed to be happening but the bug fix hasn't been included in a new release of the gem yet. For the time being, I'll just use the latest github version because the warnings are bugging me.


I believe that is correct as is. `pages` is the number of pages currently in the wal file (see: https://www.sqlite.org/c3ref/wal_hook.html). After a checkpoint the wal file is small again.


nice flooring technique


Although I have found out that the preview pane is put on the right via a CSS rule in the theme.  I still wonder why moving this preview div-block after the edit div-block causes "Recursive transclusion error in the transclusive widget"?


gotta love escaping escape characters


Oh well.


I included it so .except() would work


First of all, I'm sure confidId here can not be negative, I would like to judge, the second. I think is negative is of no significance


Adding GIANT_YOU_CONTROL constant is fine, I mean 'single.put("giant you control", GIANT_YOU_CONTROL)'


@rodricios try out this build, the tests run in less time, and the extraction is much faster with the ``assets/google_wiki.html`` file.


I keep forgetting about it too.


These comments are completely unneeded.


develop (axelstudios) - i386-Windows-7-VisualStudio-12: OK (2174 of 2197 tests passed)

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


MarkdownDocs (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (661 of 661 tests passed)

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


merge 2014 data


e..... @arrowrowe  ....


Okay, I'm just trying to say that I apperiate all the hard work @DaneEveritt has put in.


Oh I see! It's just an old habit - I always added there before the other methods were implemented. Fine to remove that part :grinning:


I don't see jquery-ui-1.8.12.min.js in the media/js/libraries directory.


commit message: This actually fixes the bug :)


I looked into this further and it seems it is using more than one thread, but is limited externally by the RxNorm API. From what I see the lookup is using 1 worker (which is what threw me), but multiple threads.


maybe put that into an own function? `show_nonexistant_tests` or so.


make comment


LGTM


lol @weierstrass


Here's where I found out about it in the change log for vhs from September of 2014. Search for "allowMoveToFooter" as you have to scroll a little even with this link. What is says here is:
> '*allowMoveToFooter*' property renamed to '*movable*' in asset definitions through TypoScript

https://github.com/FluidTYPO3/vhs/blob/development/CHANGELOG.md#200---2014-09-05

Here is the code that is allowing deprecated allowMoveToFooter to continue working for the time being:
https://github.com/FluidTYPO3/vhs/commit/0e2c6ea90a2efc9abd09622d4299ba4184b7da47

##################
Also as a side note, vhs should have this documented on this page but it's not and I don't know how or where to do a pull request to update it... maybe @cedricziel could help me with that???
https://fluidtypo3.org/viewhelpers/vhs/development/Asset/ScriptViewHelper.html


(shrug)


Thanks -- I'd actually noticed that earlier today, and was preparing a fix, but `git rev-parse` should be cleaner/more robust. Thanks!


Great job!


Mike, my question is why it didn't work if I wrote this `if ($job_state_ref->finished != 0) {` instead of `->{"finished"}` ?


Method does not take a lock on `buffer` though all add operations do take it. Is this intentional?


add key codes for +/- on main keyboard;
add to comment, if its num/main key code
see for key codes: http://www.cambiaresearch.com/articles/15/javascript-char-codes-key-codes


Fingers crossed!


@balhoff freq should be the number of taxa the class is annotated to. I guess that's the same as the number of instances of the class. The normalization looks right.


Forgot to translate these, thanks :)


to have this interact with main_keyboard and num bad, i guess this will be a tricky one :-P


You should always use the terminal version regardless. It offers a lot more control.

You can work in eclipse for editing but do your commits from the terminal.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 1:29 PM, oltanklo <notifications@github.com<mailto:notifications@github.com>> wrote:

I think its because I made too many branches. I would make a branch, comit the test, comit the code for the test, and merge with master; I would then repeat for the remaining 2 tests. I will certainly say that I prefer the terminal side of github rather than the visual IDE. I just may need more practice.

--Aaron Miller

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com>>
Sent: April 6, 2015 3:26 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

Again here, this commit changed the whole file, not just the new method you added.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10590314

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10608694>.


hey, @jrperritt do you plan to add something to specify security group directions (like Ingress/Egress)?

Is it already implemented?


For this assignment you were to use Eclipse and check the files out to your local machine. That also means you need to install Git. Review the assignment instructions.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 11:09 AM, mkaempf1 <notifications@github.com<mailto:notifications@github.com>> wrote:

I'll attempt at redoing the implementation and tests, in regards to the indentation, I do not know how that can be fixed, it is appropriately indented in Nitrous so I don't know what caused that to happen.

________________________________
From: Brian Hogan @ CVTC <notifications@github.com<mailto:notifications@github.com>>
Sent: Tuesday, April 7, 2015 3:48 PM
To: mkaempf1/TDDWithJUnit
Cc: Kaempf, Michael
Subject: Re: [TDDWithJUnit] added test2 implementation (cfea52d)

Deceleration means slow down. So why are you adding to the speed?

-
Reply to this email directly or view it on GitHub<https://github.com/mkaempf1/TDDWithJUnit/commit/cfea52d3028bfe710ee26c377c4ad46785d5fece#commitcomment-10605408>.

—
Reply to this email directly or view it on GitHub<https://github.com/mkaempf1/TDDWithJUnit/commit/cfea52d3028bfe710ee26c377c4ad46785d5fece#commitcomment-10605810>.


lål lål Begos portfolio och hemsida...


I'm trying to compile gitit2 with the pandoc version from Debian Jessie, 1.12.4.2, and this line causes a build failure:
```
Network/Gitit2.hs:35:51:
    Module
    `Text.Pandoc.Writers.HTML'
    does not export
    `defaultWriterState'

Network/Gitit2.hs:35:71:
    Module `Text.Pandoc.Writers.HTML' does not export `WriterState'

Network/Gitit2.hs:35:84:
    Module
    `Text.Pandoc.Writers.HTML'
    does not export
    `inlineListToHtml'

Network/Gitit2.hs:35:102:
    Module `Text.Pandoc.Writers.HTML' does not export `unordList'
```
You mention the problem in the commit message. But isn't their a solution that works on Debian Jessie?


I am not sure about that two times exactly the same line ?


Same locking question, though here it is not that important


Is the side-effectful updating of the performance required here?

Also, looks like we can do the `get_account` just once in this function and share it between the two?


will also need to somehow get "latest" in this array but only on master...


We went with a single three field index. This seems to be working for our users.


This was a duplicated category, I haven't seen it before...


Is it a safe assumption that the content is centered in the viewport?


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/5114ca9d357f3387d1d8376f53c76825cfe246ff/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/5114ca9d357f3387d1d8376f53c76825cfe246ff/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/5114ca9d357f3387d1d8376f53c76825cfe246ff/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/5114ca9d357f3387d1d8376f53c76825cfe246ff/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/5114ca9d357f3387d1d8376f53c76825cfe246ff/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


MarkdownDocs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


Shouldn't this line be element.hammertime = newHamer.Manager(element[0], defaultOpts); ??

Otherwise you are never setting element.hammertime that you are checking for.


16, of course you're right!


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/1f74bfd9b355f1747beb8f4478a0d483b9745609/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/1f74bfd9b355f1747beb8f4478a0d483b9745609/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/1f74bfd9b355f1747beb8f4478a0d483b9745609/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/1f74bfd9b355f1747beb8f4478a0d483b9745609/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/1f74bfd9b355f1747beb8f4478a0d483b9745609/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


Adding a constant so there is a good place to put a comment about why this exists.


Are we going on an SQL server or storing it in a server folder.


@mcostalba
Swordfish is getting burnt to a toast...I don't want to waste any more resources on this.  Please stop the test if you feel it is necessary.  The test results shows the benefit of syncing the move order with lmr...at LTC I believe this benefit will increase.


I'm not the author, the code was created by lwip project, then Espressiff did their port and all I did was to clean this port up.

I will try to reproduce this problem in the weekend and then provide the fix.


Ok, I certainly used the IDE's visual comits , merges, etc. Maybe that had something to do with it. I am assuming the terminal part of github is the "Git Shell"? -- in relation to the desktop applicatiom from github.

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com>
Sent: April 7, 2015 3:05 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com>
Cc: "oltanklo" <tankz0r@msn.com>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

You should always use the terminal version regardless. It offers a lot more control.

You can work in eclipse for editing but do your commits from the terminal.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 1:29 PM, oltanklo <notifications@github.com<mailto:notifications@github.com>> wrote:

I think its because I made too many branches. I would make a branch, comit the test, comit the code for the test, and merge with master; I would then repeat for the remaining 2 tests. I will certainly say that I prefer the terminal side of github rather than the visual IDE. I just may need more practice.

--Aaron Miller

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com>>
Sent: April 6, 2015 3:26 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

Again here, this commit changed the whole file, not just the new method you added.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10590314

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10608694>.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610613


I started real basic but if you want you can do that.


This is not a banal mistake; see that, since the name is wrong inside the ```try``` statement, the *"Undefined function or variable"* error wouldn't be thrown and would be mishandled instead.

I considered pulling the two ```setOutputFrame``` lines out of the try statement but I'm not sure what would be the effect of doing that in the other case where ```'glider_trajs.mat'``` is not available.


AddingMirror
line 5 —> { in a new line

reflection_level_4.cs:-
line 7 —> { in a new line
delete line 8 and 9
line 32 —> { in a new line
line 136 —> where is the block of the else part ?
method extendLighX —> number of tabs

rotatebttonslevel4:-
remove the start and update methods
line 4 —> { in a new line
@mohamadbadr


Yea it is. You can also reinstall and choose the option to add  Git to your regular path.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 3:12 PM, oltanklo <notifications@github.com<mailto:notifications@github.com>> wrote:

Ok, I certainly used the IDE's visual comits , merges, etc. Maybe that had something to do with it. I am assuming the terminal part of github is the "Git Shell"? -- in relation to the desktop applicatiom from github.

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com>>
Sent: April 7, 2015 3:05 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

You should always use the terminal version regardless. It offers a lot more control.

You can work in eclipse for editing but do your commits from the terminal.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 1:29 PM, oltanklo <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>> wrote:

I think its because I made too many branches. I would make a branch, comit the test, comit the code for the test, and merge with master; I would then repeat for the remaining 2 tests. I will certainly say that I prefer the terminal side of github rather than the visual IDE. I just may need more practice.

--Aaron Miller

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>
Sent: April 6, 2015 3:26 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com><mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com><mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

Again here, this commit changed the whole file, not just the new method you added.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10590314

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10608694>.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610613

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610749>.


An aside/reminder here, that this section should have been a DataFrame.

(Saying this as the one who committed that injustice.)


Nice! Thats makes me happy.

--Aaron Miller

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com>
Sent: April 7, 2015 3:14 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com>
Cc: "oltanklo" <tankz0r@msn.com>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

Yea it is. You can also reinstall and choose the option to add  Git to your regular path.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 3:12 PM, oltanklo <notifications@github.com<mailto:notifications@github.com>> wrote:

Ok, I certainly used the IDE's visual comits , merges, etc. Maybe that had something to do with it. I am assuming the terminal part of github is the "Git Shell"? -- in relation to the desktop applicatiom from github.

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com>>
Sent: April 7, 2015 3:05 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

You should always use the terminal version regardless. It offers a lot more control.

You can work in eclipse for editing but do your commits from the terminal.

Sent from my phone.

Brian P. Hogan, Faculty
Information Technology-Software Developer Department
Chippewa Valley Technical College
BEC 159A

On Apr 7, 2015, at 1:29 PM, oltanklo <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>> wrote:

I think its because I made too many branches. I would make a branch, comit the test, comit the code for the test, and merge with master; I would then repeat for the remaining 2 tests. I will certainly say that I prefer the terminal side of github rather than the visual IDE. I just may need more practice.

--Aaron Miller

--- Original Message ---

From: "Brian Hogan @ CVTC" <notifications@github.com<mailto:notifications@github.com><mailto:notifications@github.com>>
Sent: April 6, 2015 3:26 PM
To: "oltanklo/TDDWithJUnit" <TDDWithJUnit@noreply.github.com<mailto:TDDWithJUnit@noreply.github.com><mailto:TDDWithJUnit@noreply.github.com>>
Cc: "oltanklo" <tankz0r@msn.com<mailto:tankz0r@msn.com><mailto:tankz0r@msn.com>>
Subject: Re: [TDDWithJUnit] Made test for deceleration (65349f0)

Again here, this commit changed the whole file, not just the new method you added.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10590314

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10608694>.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610613

—
Reply to this email directly or view it on GitHub<https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610749>.

---
Reply to this email directly or view it on GitHub:
https://github.com/oltanklo/TDDWithJUnit/commit/65349f0242df7afbc4d67dfa89c4da2c9d592b09#commitcomment-10610771


Oh, you are little new line eater :smirk:


Thanks for the mention! :)


OK. I have just compiled .apk file and there was 3 the same lines..


Ok, found an issue, it'll be fixed after some rigirous testing.


Done.  https://github.com/schwehr/libais/commit/875f893bde53fbadf2b9139e10a25302d37bcf49


Is this correct? `event.actor_name` is the `email`?


You must have perl downloaded to either "C:/Strawberry/perl/bin/perl.exe" on windows or perl <- "/usr/bin/perl" on linux


I think we should for super simple debugging purposes while we're still developing. I think we definitely strip it out when we're close to being finished.


@JonLim Since you're already getting a list of all people from the database. There's no point in doing another query to find a random person. We can just use the `Array#sample` to get a random person from the list of people we already have.

Try replacing the next two lines with: `@current_person = @people.sample`

http://ruby-doc.org/core-2.2.0/Array.html#method-i-sample


Hi, thanks for the comment. This is not the main development branch nor the main repo of gitit2.
The main branch is the master branch of jgm/gitit2.

I developped this branch to be able to have table of contents including subpages.
For this I needed to hack pandoc as well, in the PR jgm/pandoc/pull/1771, but the PR has not (yet) been accepted there.

So it is definitely not available in any distribution, but if you want to install anyway, you will need to patch your version of pandoc with the above PR.


Is this way acceptable?


Thanks for your input


why not have a variable for the expected `metadata` like in test below?


I've to double check this once more ...


No CamelCase


> i see what your suggesting in the code snippit add the Null pointer check

@woodmawa I am not suggesting that you do anything with this code snippet.  You won't see this change in play until we ship another release, unless you are building your own distribution from source.  I don't know why the spring loaded jar can't be found.  The issue you reported at https://github.com/grails/grails-core/issues/620 will track that.


No worries, thx for merging J

De : Jon Schipp [mailto:notifications@github.com]
Envoyé : mardi 7 avril 2015 20:25
À : hegyre/nagios-plugins
Cc : hegyre
Objet : Re: [nagios-plugins] Add sudo for linux + switch user variable (f9becae)

Sorry for the delay, applied, thanks

—
Reply to this email directly or view it on GitHub <https://github.com/hegyre/nagios-plugins/commit/f9becaee435900388d381a71b3cabcc2d9921381#commitcomment-10608592> .  <https://github.com/notifications/beacon/AH7rF9f5ltBKgPxlrnmx6oNUWlDUsRODks5n9BhhgaJpZM4D71s9.gif>


There is a bug, according to : https://www.teeworlds.com/forum/viewtopic.php?id=11295


It's assumed that the replay wouldn't occur concurrently with other operations. But it won't hurt to lock anyway, so I'll add that.


My only reservation is the name of the method and the parameter name.

should it be more like

```
def isMountpoint(self, path):
    # etc.
```

?


It's already done almost at the end of "def configure_system(self):"


love it, thanks for your work


Do you use final on local variables for the sake of the compiler or the developer, @boq ?


@shinnn Chokidar works fine on windows. Tests are running (and passing) on appveyor. Seems like you're taking a few old comments out of context to draw your conclusion here.


So there seems to be no test for the `dependencies` methods and they are public... is this correct?


With this change, when you `setFullScreen(false)`, you end up not being back to the right screen size... Is this intentional?


All good, feature and implementation wise. Well done!


Looks good, tyler, go on skype


Ну твою же налево: error C2065: 'i' : undeclared identifier


migration added. see 62847af5c322e74075ff454fffcd824a2ebb2b57


best commit name


I plan to update the translation periodically (every month i suppose)


I like this hack, but we should work on refactoring that crazy cat logic.


there is an alternative: we could make the return statement mandatory and fix the ported comps. Happy to help with it - one time pain.

I fear this will be brittle. What do you think?


No more $0.10 cats!


С другой стороны, measure first. Дайа пока оставим так.


looks good to me - i like the tests


Oh my god, that works? That's amazing!


yes, please see 62847af5c322e74075ff454fffcd824a2ebb2b57


This totally works. It does create more routes than needed however. I wouldn't create a route for :edit, :update, or :destroy if wasn't going to use them. Specifically because it opens you up to more ways an attacker can access your app. Don't trust people on the internet. They are all liars, cheaters and swindlers.


New package created with the purpose of switching name from "example" to "game".


Thanks!


This class implicitly depends on all `writeLog` calls to be guarded by lock because `SimpleDateFormat` is not thread safe. Not sure if this is a problem, maybe smth worth documenting?


That specific function, which is core to the feature, is not easy to understand. It deals with some Warden specifics, handling multiple cases at the same time. I would need to investigate Warden's code to understand *exactly* what's going on there, but one can get the global picture just by reading it (handling login apart from the other actions, etc.) and that's enough.

The only real problem is that it's brittle code, because if Warden and/or Devise were to introduce some breaking changes, it could fail and one would need to start asking questions about how to fix that… or start diving in the underlying code. But that's the job!

One thing I usually do in my projects, is adding a comment like `# Monkey patching` or `# Brittle code`, so that I can quickly look for those "hot spots" I need to take care of (especially when upgrading third-party libs).


I don't understand why do you need pipe here?


Thanks, @kirbybubbles! :sparkles: :+1:


:rocket:


We don't need to use the `respond_to do |format|` syntax if we are only responding to html.
If we wanted to respond to both html and json we'd need to use it.
You can simplify a bit and do this

```ruby
if @url.save
  redirect_to @url, notice: 'Link was successfully created.'
else
  render :new
end
```


> Supercalifragilisticdittoumbracobaseconverter

LOL! :laughing: Touché!


I am okay for it to not work on my repo

On 08/04/2015 12:08 am, "Daniel Wolf" <notifications@github.com> wrote:
>
> @halfnelson: I'm not sure what to do about this one. The baseurl must be
identical to the GitHub repository name. Otherwise, it will work fine
locally, but the GitHub pages will show a 404 error for everything but the
root index.html.
>
> Seems like the official repo and most forks (including mine) are called
moai-dev, while your fork is called moai-sdk. So it either works with your
repo or with anybody else's. :-(
>
> Is it feasible for you to rename your repo?
>
> —
> Reply to this email directly or view it on GitHub.


très bien mindi


Creo que podemos aprovechar este PR para mejorar las descripciones/purposes de los overlaps de una vez.

El overlap del Martes lo hemos cambiado a solo Workflow. Me gustaría que el título y la descripción de abajo reflejaran más eso. Propongo:
- Tuesday: Workflow
  + Talk about our work processes at a metacognitive level. Propose new Pull Requests.
  + Address open Pull Requests (if any).


Oops! I pushed it to the Sonatype staging repository but forgot to go on there and release it *derp*. I will fix this - once I can get their UI to load >_<


Ook, will revert then.


Why do you write 0 to pipe?


No, I don't think we need it (For GOST I can not test).

For ECC and RSA, there is _sc_card_add_rsa_alg() and _sc_card_add_ec_alg() to be called by the card drivers. The functions can be called with or without SC_ALGORITHM_ONBOARD_KEY_GEN independently. I did a quick test, it registers CKM_RSA_PKCS_KEY_PAIR_GEN and CKM_ECDSA_KEY_PAIR_GEN like a caller of those functions would excpect it, even if for example it is **only** supported for ECC or the other way around.


There is no other way to return from blocked select() on linux. On OS X select() unblocks upon closing the file descriptor, but on Linux it would remain blocked.


The upload_to path is wrong. It should be cortex.core.models.Mod.generate_imagefilename


name='Mod',


selection delete merge acum, era de la mode.finish(), insa delete all nu merge cu sau fara alert dialog


name='ModVersion',


I'm not sure I like having the link elsewhere. It forces you to render the Markdown (or search a bit) to access the reference. I guess I'm so used to reading code that skipping the inline link never felt like a big deal.


upload_to=cortex.core.models.ModVersion.generate_filename


turu


@pose shouldn't we `401` if `onLine === true`?


name='Person',


@maqifrnswa yes i don't envy you trying to package ide 1.6, especially with all the new jssc and java8 cruft. luckily it doesn't depend on avr-gcc 4.9 or we'd really be in trouble.

p.s. note that we don't support the arm sam stuff yet, mainly because the toolchain hasn't stabilised and we don't have any sam boards to test with.


to='core.Mod'


DesignDaysGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: OK (2179 of 2196 tests passed)

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


We need to write something to the pipe, so select() unblocks and returns in another thread.
It does not matter whether it is 0 or something else, the only thing that matters is that there is something to read from the pipe.


to='core.Person'


to='core.ModVersion'


Hey, the images are not square anymore!


9+10 = 21


to='core.modVersion'


Hello Ralph, this XOR formula permits an easy differentiation of the black squares from the white squares. How would you do it?


to='core.String'


could we name it `connection_refused_timeout`?


Really its probably better to just recreate this migration.


Verified that the tests all pass on WP8.1. However, they used to run on Windows, and are now all pending. Why is that?


Hi Karolis,

I agree, but these changes came as a complete ossec_agent.tcl from Brian Kellogg:
https://groups.google.com/d/topic/security-onion/QZ1aVC58LYo/discussion

You could compare the old and new ossec_agent.tcl files side by side and look for code changes or you could follow up with Brian Kellogg directly if you have further questions.


That's a good point.  I had placed the variable first before I decided to wrap that entire section in the if/else statement and had forgotten to move it when the scope had expanded.  Thanks!


Merge


The commit message should say AlbumAdapter and Album classes instead of ContactAdapter and Contact classes


@DanielSperry is this more in line with what you had?

Creating unit tests for the deny side of things is proving difficult as the selectNode() loops indefinitely until another stage becomes available.


I'm proud of that one :smile:


Oh, that's why my code was different! lol


Style note, we've been favoring a kwarg per line.

```
benchmark_returns=None,
algorithm_leverages=None)
```


Hi,

Thanks for the suggestions.  Here are the changes as requested.

If you have any questions, please let me know.

Thanks,

Michael


Yes, seems to be forgotten old code. Still have to test why I got a failure now ...


Gemfile


So, on my end it looks like this commit is breaking this site in some way. I did some poking around and it's in this commit that the site is breaking. I'll be honest, I haven't found the exact location and nature of the bug, but I'm pretty sure there's one in here somewhere.


Yes. I didn’t fix every single formatting issue with what I pushed, as I don’t want to fix stuff and then have to reapply the fixes. I don’t understand why a beautify script is needed for non-generated code. Humans always do a better job at this (at least those that care about formatting, but almost all experienced developers do). Is the “beautifying” script applied to Python & JS code as well, or just CSS/HTML? If not, why?


Yes,  this little fella is a bit odd:
https://github.com/benoitc/couchdbkit/blob/master/couchdbkit/client.py#L944

In this case the view returns a row where 'doc' is None and 'value' is None
and so the whole row dict is returned when we actually want 'None'.


HeadlessFixes (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2138 of 2197 tests passed)

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


Very  unexpected. Where did you get compilation errors, on what platform I mean?


This breaks existing tests that were using that public static variable. See http://ci.xwiki.org/job/xwiki-enterprise-test-ui/878/console

I am not sure why we removed it, but either we fix the existing failing tests to not use it any more (if possible), or we put it back.


^^^^ +1


Final on local variables in not even compiled, so it's just me.


With the current modification, if you have unset  `$config['auth_ldap_group']`, line 168 is always true. I think that check should be placed before looping through the ldap groups, like on line 27.

Also, `ldap_compare($ldap_comparison === true)` looks like it should be `$ldap_comparison === true`.


It does provide minor optimisations, I wondered if that was the reason.
Using final is a good practice that I don't do often enough.


Unfortunately, this doesn't solve the problem at all. When not null, the type is incompatible when executed in JRE7 if compiled with JDK8.


 - __style__: [src/chaiscript_stdlib.cpp:18](../blob/59103b5a228d43f7ce572704e033b70096ecdc36/src/chaiscript_stdlib.cpp#L18) The function 'create_chaiscript_module_chaiscript_stdlib' is never used. (unusedFunction)
 - __style__: [src/main.cpp:169](../blob/59103b5a228d43f7ce572704e033b70096ecdc36/src/main.cpp#L169) The function 'get_eval_error' is never used. (unusedFunction)
 - __style__: [src/main.cpp:140](../blob/59103b5a228d43f7ce572704e033b70096ecdc36/src/main.cpp#L140) The function 'help' is never used. (unusedFunction)
 - __style__: [src/main.cpp:216](../blob/59103b5a228d43f7ce572704e033b70096ecdc36/src/main.cpp#L216) The function 'myexit' is never used. (unusedFunction)
 - __style__: [src/main.cpp:158](../blob/59103b5a228d43f7ce572704e033b70096ecdc36/src/main.cpp#L158) The function 'throws_exception' is never used. (unusedFunction)


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


@pose what do you think of this?
```js
if (0 === er.status && !onLine) {
  // offline
} else if (0 === er.status && isTroubledIE) {
  // override with 401
} else if (0 === er.status) {
  // connection_refused_timeout
} else {
  // parse response status
}
```


Probably better to add units to `rotationThreshold` & `rotationDelay` fields


@pbauer @thet I'm testing the unified listings now from 1.1.x branch after the merge in a project, but in collections the batching is broken: 2nd and further batching parts list no contents at all but only show the batching navigation top/bottom. I checked the master checkins for maybe any bugfixes.

Since the unifiedlistings were merged to master for Plone 5 end february, did you spot anything like this there?

![pac_batching_collection](https://cloud.githubusercontent.com/assets/394784/7033072/ca0a0c7a-dd77-11e4-9ef3-9207d7862241.gif)


Not sure need to check.


I added a jira ticket that we should eventually migrate this to a git repo that doesn't track history.  Obviously, we have plenty of other data sets that need the same thing.


Your right, its not :) I downloaded it and added it to media/js/libraries


Since this merge also includes some new features, I'm going to bump the MINOR version


puede


Probably not. This is not one (of two) compile-time optimizations (only `static final` ones are inlined) and 'final' on local variables is not encoded in class files (IIRC, there is no such possibility in current format), so no runtime gain.


With the naming convention we have been using, would `signal.types.ts` make more sense?  Also, `signal` reads just a tiny bit off, maybe `signals` as the name?  No strong feelings on this (I'm actually not even sure which way I prefer)

General note: I would rename all the places you have used `s` as a variable name to `message`, I think that is a lot more descriptive (I might also do the same for some of the places the variable is still called `signal` actually)


Can you remove the `JSON.stringify` here?  It gets automatically done by the logger and that handles the case where it throws (not that we would expect that to happen here)


This in sync with max_drawdown, so this isn't a blocking comment, but looking at that function again it did give me pause that the `calculate_max` doesn't set the new max itself, as itself max functions that rely on state set the new value itself. (Again saying this as the one who probably did that in the first place.) Also `.max()` may be fast enough in favor of having less moving parts.


Progress spinners have only been added to XML. Their visibility is not yet handled programatically in AnsweringFragment.java.


Got confused a little here. Thanks for this.


"keuy" - typo :-)


Does it matter that there's no SConstruct file?  I'm ok with it as long as the auto builder doesn't care.


Could you just add another foreach loop around the


awesome fix! i was already wondering, how the mobile menu was meant to work...


Should we (or are we even required to) indicate that the logo is used with permission in the "About" section?


Why not just add another for (IImportScope scope = namespaceScopes; scope != null; scope = scope.Parent) around this loop? And avoid the allocation of iterator and array.


I think it is better to use 'Neo4jJobScheduler' here so all kernel's thread management is in one place


why switch to `requireNonNull`?


Fix : update angular.d.ts same time as you updated angular upload.d.ts


Perfecto que hayas hecho un Main para poder probar. A partir de la próxima entrega vamos a ver de reemplazarlo por algo mejor: tests


J'ai fait le choix du mixin dans l'éventualité que l'appel à l'api sera fait par d'autres class routes ou controllers; il me suffit d'étendre la classe. Cependant j'ai "abusé" du mixin dans mon précédent projet et je me suis retrouvé avec des problèmes de code dupliqué.

Le service semble plus adapté pour la gestion d'une api, je ne l'avais pas encore utilisé.
donc:
Un singleton injecté dans les instances des objets choisi pour le service VS étendre le prototype du constructeur pour le mixin

Je passe sur un service!


The 1.9.0 build has now been released, and will be available on Maven Central in a few hours.


Блин, так и чувствовал, что что-нибудь оставлю.


for (int i = 0; ....?


One of the unit tests in sims_catUtils already depends on this repository.  It has passed buildbot without difficulty, so I think we should be okay.


Можно проще:
```python
field.coords = [[None] * field.weight] * field.height
```


I'm not sure that this is true. Our bower.json has:

    ```"open-sans-fontface": "1.2.1",```

How else would bower know to update it?


I don't think we are required to but I'll add a notice to the license dialog.


I'm going to be a little nitpicky here, so excuse me :smile:. I'd prefer if we stuck to Qt style conventions throughout the project (as the intent is to eventually merge this in as a Qt component). Having said that, there is a preference for using descriptive enums e.g.

```
enum ChannelState {
   OpeningState,
   OpenedState,
   ClosingState,
   ClosedState
};
```


I had done very little reading until now, but apparently it optimizes arrays slightly and multi-threaded access. In this example however the generated bytecode would not change, as you said the final is not stored. It's only the resulting bytecode that might change. Though it does not in this example.

Thanks for the discussion. (:


again here, Qt style would dictate the removal of needless tabs: ```qAmqpDebug() << Q_FUNC_INFO << "Channel is not open";```


`deps: ['jquery']` is like writing `require('jquery')` in the source file itself. It is a separate thing to a package management dependency.


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


@lookfirst ^


Al igual que en Smalltalk, vamos a evitar utilizar estructuras tipo do/for/forEach, y en su lugar utilizar mensajes de orden superior.

En particular, las colecciones de Java tienen un mensaje `stream()`, que te da un objeto sobre el que podés realizar operaciones interesantes, por ejemplo:

```Java
ventasDelDia.stream().mapToDouble(venta -> venta.getPrecioVenta()).sum()
```

Más información:

http://uqbar-wiki.org/index.php?title=Lambdas_en_Java_8


Please better commits. Fix what?


No problem, that's my job anyway :v:


И почему это не в конструкторе Field делается?


MarkdownDocs (jasondegraw) - x86_64-MacOS-10.9-clang: OK (1677 of 1677 tests passed)

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


also cleaned up unnecessary extra raw random generators


MarkdownDocs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


Ah, I see to what you were referring now.  Typo/edit mistake on my part.  I've added a new commit that corrects `ldap_compare($ldap_comparison === true)`.


Note: grappa is on the heavy side dependency-wise; maybe it would be worth the trouble here to create a custom package with a custom classifier.

Also: I'm really not sure about the reference to the first element of `moduleArtifacts`; I happen to have "forced it" out of the dependency and I'm really not sure whether this is the correct way to handle this situation...


Thanks for this.


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1683 of 1683 tests passed)

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


Also, a good point on unsetting `$config['auth_ldap_group']`.  That doesn't have much of anything to do with this PR, though, and is more related to my local configuration than anything.  I may, at some point, look into this to make it so that `$config['auth_ldap_group']` is no longer needed--though still supported.


Да, я с этим Text.set попадался несколько раз.

Ты закоммитил файлы проекта. Они точно нужны?


Perfecto que hayas hecho un método para construir los objetos fácilmente!

Tip: en lugar de declarar el método acá, declararlo como un constructor **en la clase Venta**, así:

```
public Venta(Prenda p, int cantidad, Date fecha) {
  this.prenda = p;
  this.cantidadPrendas(cantidad);
  this.fecha = fecha;
}
```

Y luego usalo así:

```Java
private static void venderPrenda(Prenda p, int cantidad, Date fecha) {
		Venta v = new Venta(p, cantidad, fecha);
		ventasDelDia.add(v);
}
```


m_pButtonOptions = vgui::SETUP_PANEL(new vgui::Button(this, "btnOptions", "options"));
	m_pButtonOptions->SetSize(256, 128);
	m_pButtonOptions->SetPaintBorderEnabled(false);
	m_pButtonOptions->SetPaintEnabled(false);
	m_pImgOptions->SetImage("menu_options");
 	m_pImgOptions->SetSize(256, 128);

	m_pButtonOptions->SetCommand("gamemenucommand OpenOptionsDialog");


precioFinal, dado que devuelve algo calculable, debería ser un método y no un atributo (así si el precio o el valor fijo cambian, el precioFinal también cambia)
```Java
public dobule precioFinal() {
   return precio * valorFijo;
}
```


Why do we need to store archived streams? Is it possible to simply close them right away?


Sorry for the typos:
`Libuv_INCLUDE_DIRS` must be `LIBUV_INCLUDE_DIR` and `LuaJIT_INCLUDE_DIRS` must be `LUAJIT_INCLUDE_DIR` of course.

Since both pathes are `/usr/include` for me I haven't noticed earlier.


Ok, on verra ça une fois le merge, il ne me semble pas avoir de fichier concervant ces valeurs dans le "/world"


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3611/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3609/


I think the naming here needs to be consistent (we use "Show Diagnostics" for one command and "Analyze Project" for the other). I'd suggest:

Show Diagnostics -> Current Source File
                              -> Current Project

This assumes a menu cascade, which I'm not sure is the right approach (since when there is no project the cascade will have only one item). I'd suggest moving them both to the main menu (in their own section delimited by separators) and using one of the following (I have no clear preference yet)

Show Diagnostics
Show Diagnostics (Project)

or

Lint Source File
Lint Project Files

or

Analyze Source
Analyze Project Sources


static import


Ah, right, because there might be in-flight log calls.


Tip: no pongas los setters salvo que el problema los requiera (es decir, que mutar al objeto sea un requerimiento).

Si no, son código inutil (y cuanto menos código escribamos, mejor!)


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

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


This cannot work because psqt table is never updated. I will write something for you in these days. You have started tuning with a difficult one :-)


@tgalal :smiley:


Devolvé directamente el resultado de la cuenta, sin usar variables locales (nuevamente, porque cuanto menos código escribamos, mejor)

```Java
return precioFinal * tasaImportacion
```


Your right it wasn't in that commit, but it was in the last one :)  https://github.com/coati-00/blackrock/tree/https/media/js/libraries


Não ouvi nada, apenas achei estranho escrever algo como "APÊNDICE A CÓDIGO FONTE" em vez de "APÊNDICE A <CÓDIGO FONTE>". Faz sentido ter alguma separação. Mas se puderes ver com a Bea, melhor.


I dont get why you changed this.
objects[args.modelRef][args.instanceRef] is something like a model instance. what you want is to replace something like 'actor': 'User:id' with objects['User']['id']


Mismos comentarios de antes: evitá setters y variables locales de más :)


aggregatedActivities is an array, why the keys?


So you don't want to make this branch into a pull request anytime soon? I'm planning to make a PR to split up the gitit2.hs source code and wanted to give you the chance to merge your stuff before that. Otherwise it'll be hard for you to rebase.
There's interesting stuff in this branch. Maybe you'd like to do smaller PRs on the different topics?


Make the names of the classes a little more verbose. Is alignV vertical or something different?


@chums923 Let's see if we can isolate the issue. It may be due to the changes in depends.py. Do you want to see if you can isolate the issue?

Please feel free to add screenshots too if that helps you demonstrate an issue :+1: Thanks.


Hi


Please add padding here for all children nodes


You can actually do `abstract def render`, you can have abstract methods in Crystal.

However, you can't make an existing method abstract, at least not right now. So `abstract def to_s(io)` won't work.


Default should be false since it is a new function and I am sure not everyone needs it.


Nay, sir! We don't want to calculate stats twice, the level cap should be applied before they're done the first time even


Did you ever create a switch on the client object for doing auto updates?


@voidcase hittade en bugg, om man backar tillbaks i target (med E-knappen) så att man går backar till att ha playern som target, sedan inget som target, och sedan en gång till, så kraschar programmet.

&& så kan man targeta sig själv. Heh.


@bartavelle heeelp ;-)

Any chance you remember this code? I have yet to digest it. I have fixed almost 2,000 instances of hard-coded vector widths throughout JtR including, I believe, every single one in sse-intrinsics.c except the md5crypt functions...

Compared to the code you wrote, MMX_COEF is now called SIMD_COEF_32, and we also have a shortcut for it, VS32.


Still have to remove the creature_addons. Working on that!


Please explain what the idna converter is


looks good. Where you have xbox_live..can either leave it as that or if you want to short-hand it. "xbl"


уже удалил


Use `FormattedLog. OUTPUT_STREAM_CONVERTER`?


Although now I have to convert it back to Java as I'm including the same code in `omnij-core`...


@norine-badr @maikany


Add the TODOs to move the fields in Container


Here you introduce warnings about manual object closure.


Structure this like VolumeSource, embedded *


Na verdade, acho que nem um nem outro está correto. A norma original da ABNT (14724, item 4.2.3.4) diz que deve ser usado um travessão.

No meu modelo baseado no abnTeX2, eu usei um travessão, e o pessoal da bibilioteca olhou e não falou nada a respeito disso.


It would be good to change the line in functions.php to update the version date:

`wp_enqueue_script( '_s-navigation', get_template_directory_uri() . '/js/navigation.js', array(), '20120206', true );`


Dependent on https://github.com/plexinc-plugins/Services.bundle/pull/807


I would like to do a PR of this branch, but without the PR on pandoc, this PR will not work. So I am waiting the PR on pandoc to be accepted first.
I will do a PR with the preview in edit, cause I think this is the most needed piece indepedent of pandoc.


Done - sorry :)


21 means android 5.0.0 - it is to high for minSdkVersion


We actually need a little bit more
https://github.com/jflex-de/jflex/pull/185

Concerning the prerequiste, I wrote some time ago
https://github.com/jflex-de/jflex/wiki/Development-environment

Le lun. 6 avr. 2015 à 06:19, Gerwin Klein <notifications@github.com> a
écrit :

> See also 48914c9
> <https://github.com/jflex-de/jflex/commit/48914c94b953ff4ba7e1a7c7f32b45b1f5a1d795>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jflex-de/jflex/commit/d945e5b8551a859675711d9903506f57eb318167#commitcomment-10580640>
> .
>


No need to set `conf_writer = None` anymore?


@eg04lt3r this monitor should be created in the constructor, not every time `accept` is called


this actually just made an exception so the inflector corretly generates "WP_User" from wp_user, instead of "Wp_User" which would probably be confusing


I'm not sure how we should maintain the documentation. Github source browser natively supports README and CONTRIBUTING, we have a few wiki pages, and also the LaTeX Guide.

I was a big fan of LaTeX when I was a student, but it's hard to maintain, and we don't really need it. I'd love to use Markdown everywhere.

This test looks quite nice. Have you done it manually or used a tool such as `pandoc`?


Haven't you forgotten `exitcode = 0` in the `else` branch?


Oh god, it's coming back to haunt me !

What are you trying to achieve here, before I start looking ?


@rohitdholakia do u remember why we have this change?


So it turns

```
{
    actor: '3',
    foreign_id: 'Pin:7',
    id: '1bea1ba4-dd44-11e4-8080-800056798e20',
    object: 'Pin:7',
    origin: 'user:3',
    target: null,
    time: '2015-04-07T16:35:32.379',
    to: [],
    verb: 'Pin'
 }
```

Into

```
{
    actor:
     { _id: 3,
       username: 'Thomas',
       avatar_url: 'https://avatars0.githubusercontent.com/u/125464?v=3&s=96',
       __v: 0 },
    foreign_id: 'Pin:7',
    id: '1bea1ba4-dd44-11e4-8080-800056798e20',
    object: 'Pin:7',
    origin: 'user:3',
    target: null,
    time: '2015-04-07T16:35:32.379',
    to: [],
    verb: 'Pin' ,
   item:
     { _id: 9,
       user: 0,
       image_url: '281485446934954481_c04cfc76c018.png',
       __v: 0,
       pin_count: 0 }
}
```


send


And according to docs, `abstract` will not work with structs, is it true?


@xiaom actually, i have no idea what that change is. i don't remember making it, hmmm.


Did we have a CR for that?


It works with structs. Where does it say that?


Oh, that makes more sense. I should have made that assumption.


seems okay, ack


why adding the `item` key if the original activity does not hold any ref to it?

what is wrong with the previous version?
```
args.activity[args.field] = objects[args.modelRef][args.instanceRef];
```


You don't need to use `uk-text-center` here as the parent element already contains it.


Same goes for the code here


Two things bother me:
1) With existing logging framework it was possible to configure logging levers on package basis. It was quite useful for production setups for HA. Am I right that now we'll be able to only on/off debug logging?
2) _Possible_ performance hit as `AbstractPrintWriterLogger#log`s are synchronized and `RotatingFileOutputStreamSupplier#get` does some heavy things.

@cleishm what do you think about this?


Kaip suprantu, čia nori pasiimti masyvo elementą su raktu 'ID'. Tokiais atvejais būtina naudoti kabutes (t.y.  $row['ID']) - nors tavo kodas dabar veikia, kur nors kitur apibrėžus konstantą ID jis netikėtai nebeveiktų. Beje, jei programuotum įsijungęs warningus, turėtum matyti apie tai pranešimą. Ta tema stackoverflow atsakymas: http://stackoverflow.com/questions/2405482/is-it-okay-to-use-arraykey-in-php


If we want to print the number or name errors then will need to go through all steps


fixed


Yup


added


Build is still broken. Please see http://ci.cgeo.org/job/c-geo/3612/console
Build was first broken at http://ci.cgeo.org/job/c-geo/3609/


need to rollback tx here


Here: http://crystal-lang.org/docs/syntax_and_semantics/virtual_and_abstract_types.html
It doesn't say that you can put `abstract` on methods though..


i don't think it counts as spaces with the section breaks. my pylint didn't complain.


[Here & Now](http://www.hereandnow.org), NPR's midday show produced by WBUR

[Jeremy Hobson](http://www.twitter.com/jeremyhobson), host

[Robin Young](http://www.twitter.com/hereandnowrobin), host

[Rachel Rohr](http://www.twitter.com/liotalk), digital & social media producer

[Full Staff List](http://twitter.com/hereandnow/lists/here-now-crew)


Weird that 'doc' would be none with `include_docs=True`


Does it matter that there's no SConstruct file?  I'm ok with it as long as the auto builder doesn't care.


@eg04lt3r the calling thread should be kept somewhere, like in a thread-safe structure, along with the calling timestamp. the monitor thread should iterate through that structure to check which threads should be interrupted.


Doc for sure is outdated
Isn't there any drawback to force user ID in container this way ? I mean : without this set to 1000, is uid some random value depending on build / run environment ? Can this be predicted ? Is there some risk this collides with other containers / host constraints ?


Testade och ser ut att funka rätt bra (förutom den åvannämda buggen samt att man får exceptions om man rullar target innan man scannar). En del grejer i koden ser dock sjukt konsiga ut, du får berätta sen varför  alla skepp skuggar sina equals().


I got the feedback that it was confusing. Agreed you want to say in the example it can be absolute or relative. How about we simply add `The partial Upload URLs MAY be absolute or relative`?


Out of curiosity why did you leave the conditional in here? (when data.next() became next(data))


Without the or check it was passing onto the else. With this check back in place, after a user signs in it goes straight into its check same users. Is this a desirable solution?


Where is the .3 in this line?


ah that was typo. agreed


I just sticked to how the previous version of the sample app worked. We can use your version, and just change how the sample app uses the data.


Executor is a much simpler, more general (less specific) interface. And, alas, Neo4jJobScheduler is only available in kernel.


what if its a shared library?, this is file-specific and we should use a different more generic way to handle when to show rbin info or not.


Ah, maybe the docs aren't very clean. It says "Note that virtual types only apply to classes, never to structs." A virtual type is when you have a union of `Foo | Bar` where, for example, `Foo < Bar`. In this case the compiler will type it as `Foo+` and will include other subclasses of Foo. But this is only if Foo is a class, not a struct.

You can apply abstract to classes, struct and methods. You can even have [abstract methods in modules](https://github.com/manastech/crystal/blob/master/src/io.cr#L135).

I can't fine where it says that you can't have abstract methods. In any case, it doesn't say that you **can** have abstract methods. I'll update the docs soon...


Thank you, Marco. I removed it.


By the way, just tried to do `abstract def to_s(io)`. And it haven't make any compile error. So is it just ignoring abstract in this case and use default `to_s` implementation?


@sirusdv This is looking great!

The essential thing is the fact that it's pluggable. :+1:

You've written: *"This would be much easier if we were able to get the interface class name here."*. Good point. Few free to add an [utility] method to do this if you think it's important.

About testing, `selectNode` is using `System.currentTimeMillis()` and should timeout in 30s, if you change that to `execution.clock().millis()` you will be able to inject the time and force an early timeout. (from the test cases: `clock.incrementTimeMillis(xxx)`)


I don't see any problem with setting the user's ID to a specific value like this. The UID that gets used if you don't specify one is not entirely random, but it's not easy to know in advance. In the case of the `jetty` image, I've just chosen to document the fact that it uses 999:999.

Your choice of "1000" is reasonable because it eases the use of host-mounted volumes under `boot2docker` or `docker-machine` + VirtualBox where "1000" is the uid of the `docker` user.


er no, it shouldn't thanks for catching that. please feel free to revert
that commit.

On Tue, Apr 7, 2015 at 10:18 AM, Peter Hughes <notifications@github.com>
wrote:

> Should this be hard-coded to your machine?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/zass30/SharpChess/commit/cb2d393eeb1e70eb005c6794ef9bd8dc673c10fd#commitcomment-10607167>
> .
>


If you don't mind, yes please for the sake of keeping things clean.


MarkdownDocs (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


might want to break apart the partitioning and grouping+partitioning requirement since we cannot enforce grouping yet


    if not entrydesc.text.strip():
AttributeError: 'NoneType' object has no attribute 'strip'

fixing


@waterlink Yes, but only because I didn't think of that use case, so it's a missing feature


replace new by view


With the couple changes you've made, the behaviour has remained unchanged, right?


So I went through the whole thing, refactoring code slightly and adding in all the new inSquare code, right?

Then I was like, "Man, I don't see what makes this different from inSquare". So I ran "diff src/inColor.c ../inSquare/src/main.c".

Result:
2c2
<   inColor
---
>   inSquare

Well, I tried. Custom color coming soon.


I was trying to do this:

```crystal
abstract def render
end
```

And was wondering why I get compile error. And concluded, that there is no `abstract` keyword for methods. But turns out `end` shouldn't be ther


Ah Got it. Though it sounds obvious, IMO we should clarify the Server should return it's most preferred version along with 412. what do you think?


It might be worth to throw `IllegalStateException` if this supplier is closed


Since this change affects other beamlines, have you done a full build of Acquaman suite?  Better to check now rather than getting an email from Mr. Jenkins.


Why is entry in the ChangeLog for this PR?


that seems like a big bug


Commit fixes bug; however, current validation does not allow some pages to proceed to next page.
Ex: Victim page to defendant information page.


N/m. didn't notice that this was a merge commit


No, there was no explicit request for that. This refactoring was done during implementing of new feature. Related code was moved from ncview.cpp which is still about 5000 lines. Some duplication of code were removed also, but it is in the next commit.

Why you against it?


Sounds good


MarkdownDocs (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1683 of 1683 tests passed)

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


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

VSO State: Done


I agree with @parkr @alfredxing it feels weird to have the links separate from the content.  I did wonder why the second [] was empty so thanks for pointing and further explaining that!  As far as the argument of using html instead of inline links, that's a semantic argument meant for the spec not for us.

I don't know that I disagree with the reference links style, but I do agree it would take a lot for those of us who have it muscled in and for those who don't click immediately that they are all the way at the bottom, but is it really worth it to break brains for something so trivial?


Reverted in https://github.com/peachepe/stream-nodejs/commit/e541d9a7bd17a20d2261b5cf15b47f4d9bbfc277


The release-checklist one? I did a full pull for re-adding the release checklist, including adding a changelog entry (I technically renamed it from `release.txt` to `release-checklist.txt` and figured that was changelog-worthy. I assume the entry is here because of a sync with Master.


@tfoote, The ABB package release was corrupted or something.  For whatever reason, bloom failed to make the right tags in the [release repo](https://github.com/ros-industrial-release/abb-release).  @Levi-Armstrong tried to re-release it to fix the problem.  Have you seen this before?

I'd like to try a new release, but I'm not sure what to do different.


Oh yeah, didn't see that, was fixing it in a hurry for a server I work for and didn't see the error, but thanks for telling me! ^-^


Well @ImALuckyGuy We were gonna just delete the banning system, due to McBans exists on your server...


This one closes #107.

And I don't know if this is the correct way of fixing it. Maybe we should add .attach() to all the cards that uses the .summon() function?


Закоммитился только ксайлинксовский мусор.


Isn't https://github.com/docker-library/docs/tree/master/jenkins generated from contributed project README ? I never proposed a PR to this "docs" folder


I agree -- I'll change these to `Show Diagnostics` and `Show Diagnostics (Project)` for now, and we can then refine later if we feel the need.


I knew there is something ugly and terrible!


Non mais... pourquoi c'est ce type qui fait le commit alors que je l'ai coder et push avec mon compte... Vraiment ouf ce bug !!!


Sorry folks! This is actually v0.2.4 :)


@ExSport we have a PR for that too :) https://github.com/UniversalMediaServer/UniversalMediaServer/pull/517


it wanted me to commit for some reasons, I don't think anything actually changed.
________________________________
From: John Lewis [notifications@github.com]
Sent: Tuesday, April 07, 2015 4:57 PM
To: jflewis/UMLatron-editor
Cc: Gregory D Polhemus
Subject: Re: [UMLatron-editor] Fix (9246bb6)

Please better commits. Fix what?

—
Reply to this email directly or view it on GitHub<https://github.com/jflewis/UMLatron-editor/commit/9246bb6a7c503ad0f431be8f7560760041f85267#commitcomment-10611569>.


Actually, I think it should concatenate when items are numerics, strings... But when items are objects, items should be merged according to their order.

For example :

merge.recursive(
  {
    arrayProperty : [
       {
           firstProperty : "A value",
           anIntValue : 45
       },
       {
           firstProperty : "Another value",
           anIntValue : 50
       }
    ],
    standardProperty : 5
  },
  {
     arrayProperty : [
       {
           firstProperty : "The new value"
       }
    ],
    standardProperty : 10
  }
);

should result :

  {
    arrayProperty : [
       {
           firstProperty : "The new value",
           anIntValue : 45
       },
       {
           firstProperty : "Another value",
           anIntValue : 50
       }
    ],
    standardProperty : 10
  }

In order to remove an element, we could add a null value in the second arguments array at the index we want to delete.


WHY IS THIS STILL EXISTING

WHY DID THIS EVER EXIST


Sorry folks! This is v1.6.14 :)


:+1: #5


Did a little history rewrite. Hopefully it should work now.


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

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


This should go inside the curly brackets.


@norine-badr


Can you add a test that reproduces the issue caused by this?


This is a bash script. It should end with ".sh", not ".py"


:heart:


MarkdownDocs (jasondegraw) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1173 of 1173 tests passed)

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


`m_pythonEnvironment.reset(new pdal::plang::PythonEnvironment());`


@guybedford thanks! makes sense...


`m_gdal_debug.reset(new pdal::gdal::GlobalDebug());`


Hey @ethanlindsey do you want me to add these or would you like to fork and add them?


Re: grappa don't mind that test is reasonably straight forward enough.

to query the classifier we need the [ResolvedArtifact](https://gradle.org/docs/current/javadoc/org/gradle/api/artifacts/ResolvedArtifact.html) and i'm assuming at this point the dependency will have a single artifact (i don't like this, seems a little... hacky)


In short, the code **looks** like it's vector-width-agnostic, using SIMD_COEF_32 in many places. But it isn't. Like the comment says, there are many instances of hard-coded width of 4, disguised as eg. "+4", "&3" or ">>2". And some instances of eg. "*16" may likely be "4*SIMD_COEF_32".

My problem is I can't tell which of them is widths and which of them should actually still be "+4", "&3" or ">>2"... We need the code to run with AVX2 (SIMD_COEF_32 == 8) and AVX-512 (SIMD_COEF_32 == 16).

Like I said I have sorted out nearly everything else, bar the dynamic format. Jim gets to fix that.


Looks reasonable. If there is a regression, we should just fix that.


Did my part.


This commit made the formatting on Annottrack.js a little wonky. I would change it back but it would make merging in changes from other branches difficult.


@Ahmed-bastawisy the last arrow should be inside the else part


Btw, who can tackle any of these for the next release?https://github.com/corporateshark/WCMCommander/issues?q=is%3Aopen+is%3Aissue+milestone%3A0.19.1


Hey,

I did offer to take it down when I submitted the test. I thought it was a bit strange that there were publicly available answers. I thought of this a bit too late to be of help to me though :p.

I will back the fork to my local machine and delete the fork from github hopefully sometimes tomorrow.


Really?


This function must recieve a Vector character "VC" and a list "L" of two lists "L[1] and L[2]", where each line connects respectively a sample with his experiment.
The output, is a list sorted in the same order as VC and each row connects the sample with his experiment, and assign a color to each experiment.
IMPORTAT: The names in VC and L[,2] must match EXACTLY.


Easter eggs !


### [Codacy](https://www.codacy.com/app/jatinx-lol/Countify/commit?bid=2233767&cid=8993915) detected an issue:
#### Message: `Avoid unused local variables such as '$html'.`

#### Occurred on:
+ **Commit**:  46eed18ace7fa973a0f21ba8ab4b373751c6bdd5
+ **File**:    [class.countify.php](https://github.com/iStorry/Countify/blob/46eed18ace7fa973a0f21ba8ab4b373751c6bdd5/class.countify.php)
+ **LineNum**: [87](https://github.com/iStorry/Countify/blob/46eed18ace7fa973a0f21ba8ab4b373751c6bdd5/class.countify.php#L87)
+ **Code**:    `               return $html = curl_exec($ch);`

#### Currently on:
+ **Commit**:  46eed18ace7fa973a0f21ba8ab4b373751c6bdd5
+ **File**:    [class.countify.php](https://github.com/iStorry/Countify/blob/46eed18ace7fa973a0f21ba8ab4b373751c6bdd5/class.countify.php)
+ **LineNum**: [87](https://github.com/iStorry/Countify/blob/46eed18ace7fa973a0f21ba8ab4b373751c6bdd5/class.countify.php#L87)


@twizmwazin sorry I just wanted to see how it looked


same here as the aggregation, we should split up the partitioning and grouping+partitioning requirements and enforcements.


probably just an oversight. It is still in the code, so if you want to a PR to clean this up, very welcome!


You can test a smart pointer without the dereference, I think.  Otherwise it isn't very smart.


yes!


@skeptical fair call. I had been planning on getting users to test before releasing the change but it was lazy to put it in master :)


segue


where does this comment go?


Adds test coverage for duplicated debug docs.


Hi @yorikvanhavre
As we discussed, here's the isolated IFC file for the locker rooms.

Will be fun to hash this out.


Thanks to Connor Sawaske for the tikz!


HeadlessFixes (axelstudios) - x86_64-Linux-Ubuntu-14.04-clang-3.5: OK (2144 of 2197 tests passed)

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


Oops, my bad, I left something out, will submit a new pull request.

line 9 should be

values, indices = zip(*sorted((a,b) for (b,a) in enumerate(l)))


So to sum it up, the commit above reverts my mistakes and takes the code back to original state.


Nope. The docs only come from https://github.com/docker-library/docs at the moment.


This is the file that would need to be updated: https://github.com/docker-library/docs/blob/master/jenkins/content.md


nice!


See also #1177


This name still bugs me, but I can roll with it. :smirk_cat: