labofsahil/github-event-dataset-2012 huggingface dataset:
Signing them off is a good practice, is basically means that you've reviewed
then and approve of them. They're somewhat less useful if you're only signing
them yourself but the end idea is that multiple people will sign them to show
who looked into the commit and thought it was a good idea.

More information on this can be found here: http://stackoverflow.com/questions/1962094/what-is-the-sign-off-feature-in-git-for
Well if this option was named `create`, then I think `save()` would only be called if the model was new.

I'm just thinking about how add/modify/remove map onto create/update/delete.
You're right. My habit.
Whoa, wasn't aware that --author="Curtis Wensley" --author="Atsushi Eno" didn't work to add two authors. The code change is from http://bugzilla.xamarin.com/show_bug.cgi?id=2927 thanks to Curtis.
Who knows why limitA was 0... I guess I wanted to confuse people.
Ahoj, jaká byla motivace k téhle změně? Doteď jsem si pohodlně layoutoval i komponenty (layout jsem automaticky nastavoaval z PHP, nikoliv pomocí makra extends) a teď nevím, jestli to vůbec bude nějak pohodlně možné...
`$('a.delete-button').click();` do that in your javascript console and just click okay for all the alerts, all your messages should delete, and theoretically the notifications should stop coming in. Maybe?
I don't know how to do that, sorry.
Haha you're right, it's just a preference thing and at the end of the day a version > 1 doesn't really effect anything... 

Keep up the good work :)
Over-engineered, yup, but mostly because XBMC already does most of that stuff for you ;)

A couple of points.

1.  Things that generally need acceleration in XBMC already automatically do that acceleration for you.  All you need do is:
    a. Send repeat CKey objects to OnKey() reasonably frequently (once a frame is enough).
    b. Make sure the holdTime is set to the length of time the key has been held down.
    c. You might need an initial delay before sending the repeat events - not 100% sure.  Most places that support held/repeated events handle it already I think, but you might want to check how SDLKeyboard is set up.

2.  The code would be simpler without pointers in the queue/deque.

3.  You may wish to consider having it at a higher level again (CKey level rather than at the Event level?) - the code would essentially be the same both ways.
I think that statement is no longer correct.  Now that the server is yielding more, there could be multiple yields during a query resulting in a non snapshot like result.
Same reason as under [check test above](#commitcomment-861598).
Feck! Ik dacht dat ik tot laat in de nacht had zitten nerden vrijdag. Gaat Urs er ff 2-3 uur overheen.
Can you open a new bug report so we remember to revert this?
make a note/verify that this is actually recreating the files on disk
Sure thing... But do you mean there should be no NULLs at all in the whole web.client project? Your s2js doesn't seem to care about that...
Make a comment in the Gemfile.lock which would be weird to do
We should probably add comments to various formulae "x.even are stable versions, x.odd are development versions; please do not update the stable release to a development version."
They are passed by `no_cover_passthrough_test` and `cover_passthrough_test`.  In one case `pass_cover` is not enabled and in another it is.

Ahhhh, wait, you are wondering if eunit sees this as a simple test?  My guess is if the function is arity 0 it ignores it?  I could add a 0 arity that calls this.
Formatting comment!
yes it actually copies all log files from all nodes
Alright, fine with me.
ok, apart from that, is it pushable?
From [Handlebars.rb](https://github.com/cowboyd/handlebars.rb):

```rb
Handlebars.register_helper(:twice) do |block|
  "#{block.call}#{block.call}"
end
template = Handlebars.compile('{{#twice}}Hurray!{{/twice}}')
template.call #=> Hurray!Hurray!
```
Do you have anything specific to say here?
Una buena práctica que siempre mejora la calidad del código, otorgando modificabilidad, facilidad de lectura y de utilización es elegir buenos nombres de variables y funciones.
Así el usuario de la clase, función, o parte del código no necesita prácticamente ninguna explicación para comprender de que se trata cada parámetro y como tiene que utilizarlo.
En este caso, si utilizara la función constructor, no se entendería a simple vista a lo que se refieren los parámetros "ti", "ra" e "id".
On ubuntu with gcc and g++ an extension of .c can be either C or C++ but an extension of .C can only be C++.

cppcheck$ cat both.c
#ifdef __cplusplus
#include <iostream>
#else
#include <stdio.h>
#endif

int main(int argc, char * argv[])
{
#ifdef __cplusplus
    std::cout << "C++" << std::endl;    
#else
    printf("C\n");
#endif

    return 0;
}
cppcheck$ gcc both.c
cppcheck$ ./a.out
C
cppcheck$ g++ both.c
cppcheck$ ./a.out
C++
cppcheck$ mv both.c both.C
cppcheck$ gcc both.C
/tmp/ccZe7vuW.o: In function `main':
both.C:(.text+0x14): undefined reference to `std::cout'
both.C:(.text+0x19): undefined reference to `std::basic_ostream<char, std::char_traits<char> >& std::operator<< <std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&, char const*)'
both.C:(.text+0x21): undefined reference to `std::basic_ostream<char, std::char_traits<char> >& std::endl<char, std::char_traits<char> >(std::basic_ostream<char, std::char_traits<char> >&)'
both.C:(.text+0x29): undefined reference to `std::basic_ostream<char, std::char_traits<char> >::operator<<(std::basic_ostream<char, std::char_traits<char> >& (*)(std::basic_ostream<char, std::char_traits<char> >&))'
/tmp/ccZe7vuW.o: In function `__static_initialization_and_destruction_0(int, int)':
both.C:(.text+0x51): undefined reference to `std::ios_base::Init::Init()'
both.C:(.text+0x56): undefined reference to `std::ios_base::Init::~Init()'
/tmp/ccZe7vuW.o:(.eh_frame+0x12): undefined reference to `__gxx_personality_v0'
collect2: ld returned 1 exit status
cppcheck$ g++ both.C
cppcheck$ ./a.out
C++
:CAKE:
а мы не на тринити :) у нас модель выбирается из списка либо по эвент-дате либо рандомом, а список - все ненулевые. тут никакой эвентдаты нет - имеем рандом из всех 4х моделей.
as i think, this - typo from mns :)
based on customvoltage code.
Why get $controlle rhere again ? :) It should already be there.
I don't think we need the parens here. That's tough to translate into other languages and it's ok to talk about "selected items" even if there only happens to be 1 item selected, since the number itself doesn't appear in the sentence.
Doesn't matter, as we apparently pick them up correctly in Socorro UI, and the current correlation scripts just don't know about how we're doing betas nowadays.
When correlations will be re-implemented in Socorro proper, we'll improve on that, but for now, let's go for the easiest path that gives us something useful (I know Marcia is longing for having this).
sorry, I introduced some empty space!
Sorry for the messup, when I realized this was going to become a pull request I had already pressed enter... to much coffee. I was just trying to illustrate my answer to this tweet (http://twitter.com/antirez/status/164011454406729728) by @antirez. Basically this fixes the Markdown issue that does not highlight the indented blocks starting with (gdb) as code. (at least when displayed on GitHub)

Feel free to close and ignore the pull request. (Or let me know if I shall convert the remaining code blocks)
you could use `.rstrip('=')` instead of `[:-1]`
Yes, I also looked into this, though the layer-button is controlled via Javascript in Leaflet's code (https://github.com/CloudMade/Leaflet/blob/master/src/control/Control.Layers.js), so the change will be a bit bigger. I was also thinking of changing the whole sidebar-system so that each section in the sidebar will be its own collapsible button, somewhat like the layer-button. That would make the worldlist, playerlist and layerlist all behave the same and would also solve the spacing problems that happen on phones (since all of these are collapsible).
This is unnecessary, there is already a target called `make dist` that does this, as well as `make distcheck` that checks a lot of things before creating the tarball.  These are standard targets for Autotoolized packages, very common in Linux packages.
Really ? Signfinder ?
Didn't you mean Vlad´s scripts modified by me(orphus==faelon) in the "independent" repo ?
but i suppose its not your fault,.... entirely...
finding a real is sometimes difficult.
We don't generate a cache key, we just use the cachedWithin attribute of cfquery.

Some background info on this weird CFML behavior here:
http://blog.rubicon.je/coldfusion-cachedwithin-max-rows-bug-or-featu
short: return (file != null);
Way to diagnose while actively engaged in a meeting! I wish I had your multi-tasking skillz.

r+, un-tested.
Same thing, you changed the msgid.
if `_.reduce` follows spec then you shouldn't need to pass the `,0)` it will instead will make its first pass `a` the first item in your list.
You are right here.

I did not include the wiki front page in the module as [it seemed very Kolding specific](http://www.koldingbib.dk/wiki). What do you think is the appropriate way to handle this?
Did you write down that initial implementation I sketched during the 1st review?
@Ablu:
Ok, thanks for the info, I'll try that, also.
It is not useless, as my scenario above points out. User first sets his window size, then tweaks font size until it fits. If you restrict minimum window size he has to keep reopening config dialog and changing window size all over again.
Er, you wanted to do this in the Rakefile. Note the header in hadouken.gemspec


    1 # Generated by jeweler
    2 # DO NOT EDIT THIS FILE DIRECTLY
    3 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
    4 # -*- encoding: utf-8 -*-
Looks like you miss this line of code - <?init class="org.zkoss.zktest.test2.B2199361"?>
Typo in the commit message. Intended "a step towards #19 implementation".
Check the comments below the commit 26fb6bb8f4b9727eb6a20d7b81a1e9ca01eb12a8
I *think* @mrharper did the test that @shamelmerchant is now doing...
:)  Yah this repo was me literally going through http://llvm.org/docs/tutorial/ and typing it in as I read through the docs so the code only touches on stuff that was in the tutorial.  When reading it is easy to gloss over something (at least for me), but when you type it all in by hand it really makes you ponder each line/item one by one and if it doesn't make sense you go and poke around in the API docs or source until you figure it out.  While it might take a lot longer it was a lot more educational. (and submitted a documentation patch back to llvm in the process :)
Seems like it'd be really good to include a test with this one.
you should change pref (whatever that is) to accept PNames.DEBUG_MODE (whatever that is)
I still don't get why it is that hard to have a "working" libnfs here. Is there a reason?
IDK, some random shit, haha.
something went wrong... will remove it
May be that we include this for 1.1 if it's baked enough then make it optional when the builder launches. I'm not a huge fan of the old fixed bars but we just need to discuss versioning tomorrow on our call. Assume this needs to be in good shape Scott and well just decide whether it's in 1.1 or offered separately. Keeping this as an extension is ideal.

.................................. .  .    .        .
Todd Parker
Partner, Filament Group Inc.
102 South Street #3 Boston, MA 02111
todd@filamentgroup.com // 617.953.1617


On Mar 7, 2012, at 8:57 AM, "Scott Jehl" <reply@reply.github.com> wrote:

> This is a proof of concept at this point, but yes that's what it does. We're discussing how best to offer it, and whether it'll be included in a release or offered as an external option you can include.
> 
> 
> On Mar 7, 2012, at 11:30 AM, Robby Dermody wrote:
> 
>> So this means that browsers like iOS4 and Android 1.6 will get the "floating" toolbars that jQM 1.0 had then? Awesome! (The static positioning can be confusing for users if the mobile page is long.) Is this going into jQM 1.1 or 1.2?
>> 
>> --- 
>> Reply to this email directly or view it on GitHub:
>> https://github.com/jquery/jquery-mobile/commit/801e4aa6eed135aeacef7938cba274a3e46f2089#commitcomment-1051858
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jquery/jquery-mobile/commit/801e4aa6eed135aeacef7938cba274a3e46f2089#commitcomment-1053547
@shyam-habarakada thanks for the reminder, pulled in mulistage support and released a new gem version. Let me know if it works ok.
Thanks!  This new Eclipse integration is so much nicer =)  The one problem I still get is that it always complains about a required folder named 'test' missing:
Project 'jobs' is missing required source folder: 'target/scala-2.9.1/src_managed/test'
Won't this be nil if zero categories are selected?
I didn't find that yet, still playing around with it. For more details 
see: http://www.eclipse.org/egit/

Best regards,
Marcel

Am 27.02.2012 15:41, schrieb Thomas White:
> Mazi-
>
> That set of commands works fine in Git Bash, but I don't see a way do to git diff origin in either NetBeans or in SmartGit.
>
> Can you do that in Eclipse?
>
> /Tom
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/LimeSurvey/LimeSurvey/commit/b93d63b9fd0db243a7b76ede9719b68f660c13d2#commitcomment-1015185
>
>
Okay, if `$_attributes_array` evaluates to `FALSE` the whole `if` block's result changes. The `$_attributes_array` seems superflouous nevertheless. I suggest to revert this line.
I'm not sure what that colon is doing there, but it's almost certainly not what you expect. If you want a symbol, it should be :latest.
@blafoo And I'm used to colleagues who have the topmost priority of always having green tests. Whenever a unit test fails, this is more important than any other feature or bug being worked on. Because from the moment of failing unit tests nobody in the entire team knows whether or not his own code changes are correct or not. You cannot rely on the assumption that the red result of today is the same red result of yesterday. So maybe you should think about _your_ priorities instead of having critics with this commit.
小问亲~ 你是想将一个bufferHelper对象重复使用么。其实我的意见是，没必要将一个对象重复使用，最好通过闭包释放让V8回收掉不使用的对象。
Kestas, it appears this has been pushed to live and people can no longer create anonymous games.  See this thread:
http://www.webdiplomacy.net/forum.php?viewthread=810333#810333
That didn't work quite right to me, so I went for this:

`$content = $xpath->query('//table[last()]')->item(0)->textContent;`

However, as I try to parse other pages, such as the members list, I'm struggling to get anything to return:

`$emails = $xpath->query('/html/body/form/center[1]/table/tbody/tr[3]/td[2]/a');`

I should be able to return the first email address in the list from this, but I can't get my head around it... Any ideas?
The Where is superfluous, you can pass the predicate directly to LastOrDefault.
/cc @jashkenas (or someone else who has commit access).
On Thu, Jan 5, 2012 at 11:21 PM, David Y. Ross <
reply@reply.github.com
> wrote:

> Perhaps this is true, but the task doesn't exist with the deploy.rb and
> play_recipes.rb that this module gives. This patch SHOULD make "cap
> play:logs" work out of the box. Otherwise, more changes are needed.
>
>
Isn't "cap setup" a default command from capistrano ? We don't need to
redefine it in our recipes.
But yet, I agree that if there is a problem with this, we must correct it!!!


> Can you check on your apps that use capistrano? Can you see if "cap
> play:logs" works?
>
>
Apparently it works on my hypertrout deployment. Here are the logs.

> cap play:logs
  * executing `play:logs'
  * executing "tail -f /opt/play/shared/log/hypertrout.log"
    servers: ["87.98.167.34"]
Password:
    [87.98.167.34] executing command

87.98.167.34: 01:14:10,897 INFO  ~ loading
http://api.klout.com/1/users/show.json?key=bfs2g93vm5dn7qr72ju7baxq&users=tonyubot
01:14:11,206 INFO  ~ loading
http://api.klout.com/1/soi/influenced_by.json?key=bfs2g93vm5dn7qr72ju7baxq&users=ffmercatino
01:14:11,457 INFO  ~ loading
http://api.klout.com/1/soi/influencer_of.json?key=bfs2g93vm5dn7qr72ju7baxq&users=ffmercatino

Pascal



> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/dyross/play-capistrano/commit/97154f1c9ea0d7d276875b2f0c92fd6ac40ecdc8#commitcomment-841878
>
I can't 100% remember but I think I was testing stability and determined it shouldn't be an issue so I switched back to TINY_PREEMPT_RCU.  Check 4 commits later.
Hmm. Cool. I knew about Hash.merge, but hadn't thought about using it here.
eye of acherus fall...
Why is "the stupid" fixing your db errors?
@malcrom 

using SAI wih event type 64 will summon instantly
im sure there is quests out there that require to end the casting bar while active the object...

so how this is going to replace?

and btw all Script_events are working in my server inculding summons only id 13685 doesn't.
**Part** | **Points**
------------- | -------------
base | 1
**Total** | 1

```python
nd = str(n)
print "1. Your number, "+nd+", is even."
```
don't do this. instead:

```python
print "1. Your number, %s, is even." % n
```
it formats it appropriately for you. See http://docs.python.org/library/stdtypes.html#string-formatting-operations

Your logic is reversed in section 4 #3.

```python
print ("Enter a grade [0-100], and I will convert")
```
Putting a space before a `:` or the `()'s` is highly discouraged. Don't do it, I'm surprised it worked.

```python
if (grade>96):        #a series of ifs checking the letter grade
    print "Congrats on the A+!"
elif (grade>93 and grade<97):
    print "Congrats on the A!."
```
There is probably a better way to do all this. Drop the ()s around your conditionals please. `93< grade <97` works by the way.
Let's add a date filter here like the one on the outstanding invoices page.
@sobrinho ah I thought that level would only allow for future betas, thanks.
I though you were going to convert it to LaTex. :)
-Ross


On 1/5/12 11:38 PM, "Stuart Campbell"
<reply+c-842703-aba105cf0d66f96dfda64c49f72f6f2f5ca795ca-1301479@reply.githu
b.com> wrote:

> It would have been ok just to leave it as odt if you had wanted! :-)
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mantidproject/documents/commit/11a04aa5668545af530392d8a513
> 98b440b54d78#commitcomment-842703
>
Tree.getDepth and TreeNode.getDepth seem to be unused. They could be removed if this is the case.
Yes. Haven't I merged that?
Possibly. Though, to my mind, “time as string” sounds clearer. “Time string representation” is three conseсutive nouns and this is “heavy”.
Hm, what about the ones using imperial feet in Sweden ? I think only a few people will use that feature but you never know ,-)
oh print friend
Если поменять местами:
maxDim * originalImage.size.height / originalImage.size.width

То можно обойтись целочисленными вычислениями.
build failed
1>game.lib(Player.obj) : error LNK2019: unresolved external symbol "public: void __thiscall Player::SetHonorPoints(unsigned int)" (?SetHonorPoints@Player@@QAEXI@Z) referenced in function "public: bool __thiscall Player::LoadFromDB(unsigned int,class SQLQueryHolder *)" (?LoadFromDB@Player@@QAE_NIPAVSQLQueryHolder@@@Z)
1>game.lib(Level3.obj) : error LNK2001: unresolved external symbol "public: void __thiscall Player::SetHonorPoints(unsigned int)" (?SetHonorPoints@Player@@QAEXI@Z)
1>game.lib(Player.obj) : error LNK2019: unresolved external symbol "public: void __thiscall Player::SetArenaPoints(unsigned int)" (?SetArenaPoints@Player@@QAEXI@Z) referenced in function "public: bool __thiscall Player::LoadFromDB(unsigned int,class SQLQueryHolder *)" (?LoadFromDB@Player@@QAE_NIPAVSQLQueryHolder@@@Z)
1>game.lib(Player.obj) : error LNK2019: unresolved external symbol "public: void __thiscall Player::ModifyArenaPoints(int,class ACE_Refcounted_Auto_Ptr<class Transaction,class ACE_Null_Mutex> *)" (?ModifyArenaPoints@Player@@QAEXHPAV?$ACE_Refcounted_Auto_Ptr@VTransaction@@VACE_Null_Mutex@@@@@Z) referenced in function "public: void __thiscall Player::RewardQuest(class Quest const *,unsigned int,classObject *,bool)" (?RewardQuest@Player@@QAEXPBVQuest@@IPAVObject@@_N@Z)
1>game.lib(ArenaTeamMgr.obj) : error LNK2001: unresolved external symbol "public: void __thiscall Player::ModifyArenaPoints(int,class ACE_Refcounted_Auto_Ptr<class Transaction,class ACE_Null_Mutex> *)" (?ModifyArenaPoints@Player@@QAEXHPAV?$ACE_Refcounted_Auto_Ptr@VTransaction@@VACE_Null_Mutex@@@@@Z)
1>scripts.lib(cs_modify.obj) : error LNK2001: unresolved external symbol "public: void __thiscall Player::ModifyArenaPoints(int,class ACE_Refcounted_Auto_Ptr<class Transaction,class ACE_Null_Mutex> *)" (?ModifyArenaPoints@Player@@QAEXHPAV?$ACE_Refcounted_Auto_Ptr@VTransaction@@VACE_Null_Mutex@@@@@Z)
1>game.lib(Player.obj) : error LNK2019: unresolved external symbol "public: void __thiscall Player::ModifyHonorPoints(int,class ACE_Refcounted_Auto_Ptr<class Transaction,class ACE_Null_Mutex> *)" (?ModifyHonorPoints@Player@@QAEXHPAV?$ACE_Refcounted_Auto_Ptr@VTransaction@@VACE_Null_Mutex@@@@@Z) referenced in function "public: bool __thiscall Player::RewardHonor(class Unit *,unsigned int,int,bool)" (?RewardHonor@Player@@QAE_NPAVUnit@@IH_N@Z)
1>scripts.lib(cs_modify.obj) : error LNK2001: unresolved external symbol "public: void __thiscall Player::ModifyHonorPoints(int,class ACE_Refcounted_Auto_Ptr<class Transaction,class ACE_Null_Mutex> *)" (?ModifyHonorPoints@Player@@QAEXHPAV?$ACE_Refcounted_Auto_Ptr@VTransaction@@VACE_Null_Mutex@@@@@Z)
1>D:\build\bin\Release\worldserver.exe : fatal error LNK1120: 4 unresolved externals
i rewrote the module in coffee script https://github.com/johansenj/Caffeinated-Img-Load
i believe that it is more up to date with jquery standards
Fixed the broken link.  Should we point to the Git Community Book by default?  I know Scott has a different book too - Pro Git (which rocks).
These basho_bench runs are 20 minute 50/50 mix of put and gets, run on my laptop.  

We're still struggling a bit with providing a stable rate of operations compared to InnoDB; and while our average and mean numbers match InnoDB, our 95+ percentiles are worse for GETs.  But since we are not doing any B-Tree inner node caching, there should be room for improvement still. Plus, we're just 3 days into this project :-)
Wait, the event-argument is now used to determine the event-type, right? We could remove the ``MultiverseCoreListener`` then.
datetime est un objet
->http://www.php.net/manual/fr/datetime.format.php
je vais tester ca !

La taille des colonnes pour les articles ne se fait pas correctement
J'ai mis une fonction qui permet de couper un texte dans fonctions.php (ca peut etre pratique !)

ya toujours une erreur sur le fichier test.sql
It is `[ci skip]`.
This makes the FM radio to be broken.
@Luzifr You can simply write „Fixes #7“ in your Commit-Message. This will automagically close the issue (and link it as well)
Nono, mi manca solo *Extending and Implementing Classes* e poi abbiamo finito :)
Hey Brian,

The extension is not working anymore for me now, i think you forgot to update the partial path in Deface::Override, it's:

:partial => 'shared/recently_viewed_products'

but should be:

:partial => 'spree/shared/recently_viewed_products'

Bests,

Richard
Colin Miller-esque tabulation
Do me a favor and check node_modules into git b/c npm sez so (see my commit)
Это как-бы не стандартизированная возможность (потому-то и используется cssFix). Есть только черновик: http://www.w3.org/TR/css3-animations/
А опера, как всегда, особо не торопится реализовывать черновые стандарты.
Perhaps use

    Dir.glob("#{song_path}/*.yml")

instead of

    Dir.entries(song_path)[2..-1]
위 설정 빼먹어서 다음 커밋에 추가.
There is one downside to it: if someone was to base their theme off of Radiance -- they probably wouldn't think to keep radiance.js -- so they'd lose the remove functionality. A part of me thinks that remove buttons should also be added with javascript, so there's no components that break without radiance.js. As a bonus it'd also provide a more sane experience for users without js enabled.
If the composite is implemented correctly, we shouldn't need the anon class. shouldn't it be enough to return enumarations as it is?

I created a jira for this:
https://issues.jboss.org/browse/JBRULES-3368
It's all good. The trick is committing often. Eg every time you write a method, git commit -am "Implemented Blah.method()", or git commit -am "Fixed x bug", etc. You can commit a whole bunch of times before pushing.

Making ShapeObject abstract is what I was thinking too yeah.
test it?
I guess something like this might make sense. Is this the same as [BVAL-208](https://hibernate.onjira.com/browse/BVAL-208)? If so we might address this in a generic way, independently from method validation.
Can't the custom repository be added to the FOSFacebook composer.json?
Those aren't 4-space tabs
That string ain't anywhere in the sysfs when I'm booted on the U8160 kernel... lemme try poking around in the kernel for some of that debug code which prints the serial.
This shouldn't restrict to a single gridfield per view, the selector should be ".ss-gridfield .relation-search"
Ok, kein Problem

Am 31.01.2012 17:07, schrieb Oliver Schrenk:
> Ich schieb das erst mal bis auf weiteres vor mir hin. pygtk ist noch nicht im packet-manager bei mir drin. Da sind wohl noch ein Bugs die das ganze auf OSX blockieren.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/platzhaltr/easy.rapid.gtd-on-your-fs/commit/c053616e4728b7ce819fa753993a8b73941b6461#commitcomment-918811
>
Yes, that looks better. I'd put a test for the Django version in admin.py and pass the admin site in if necessary, that way it works in all versions.
It seems like what you wanted to check for us lat/long but your seem to be checking on the presence of a key for the county name in the dictionary.  This seems like it wont produced what you wanted it to.
There should not be any delay. You commit them locally on your machine and then push them to github. The changes should be immediate.
Update your code
Yes, anything will do in fact, see:

https://github.com/openstack/nova/commit/6e35b5785d12513dc0076145f4de5e1f98034250

Since it is not a well documented feature, I have call it in the same way Horizon does.
Shouldn't this be `o.on.failure()`?
This should include also include a note that `mousewheel` is not _supported_ on horizontal ScrollViews.
Is there any risk of cycles here?
@rkuhn I was hoping for a suggestion from you here. How do I derive the path of the supervisor from an ActorRef of a remote deployed actor? Any existing utility?
this looks suspicious, but you only use it in the internal diff functions, so that's fine
Is it OK then?
Don't mix default param value with aux ctor
behavior == state, hence a restart must clear it, so why would we want to move it into the context? because it is a var in a trait? (and yes, it won’t be private because of the constructor business)
instead of the match, just do a "map"
you are pretty generous with that `flow` thing … (in this case it’s superfluous)
I will do that change to LeaderChanged. It will be harder to keep track of, but more useful. https://www.assembla.com/spaces/akka/tickets/2518
Whoa, nice catch, bad IDEA
With something like this, we would not have to list out any.  Only the exceptions to the pattern.
Using Wagn::Set as a load helper
Counting all users. The result is cached
Please do not do this, instead add xbmc/cores/AudioEngine to the include paths.
The rest of these are on you. Trim EOL whitespace on save, dude! =P

If you want me to back these out for the sake of having a non-nasty diff on this commit, let me know...
better to import thing in the top of a file than later on
import ckan.lib.base as base
What's the state of version 2? Is it basically just a no-op now?
Is there a place to see what the format of "scheduling data" should be? Is this a first class object or a dict, and if the latter, what are the valid values?
The parent span id is optional and as such I prefer keeping the option.
grrgh, another local test, great spot!

That should be 720i.
zip here too
Thanks for having added also docs for it, that's rare :) But I think we should try for a better example? I usually hate "foobar", "someotherid" in documentation...
As I said, let's delete tests like this - the whole 'page.execute_script' thing is too brittle
if you implement `__eq__` you should also implement `__ne__` as per [Python DataModel documentation](http://docs.python.org/reference/datamodel.html)
thanks for the tip, trying it now!
yes, that's the ugly part I want to fix in a better way.
Only this line is in spaces... weird
let's keep this local.
This can be simplified to:

    if (ch.writableTasks.isEmpty()) {
        unsafe.flushNow();
    } else {
        for (;;) {
            task = ch.writableTasks.poll();
            if (task == null) { break; }
            processSelectedKey(ch.selectionKey(), task);
        }
    
        k.interestOps(k.interestOps() | SelectionKey.OP_WRITE);
    }

We also should move this loop into a separate method, along with the catch block below.
Huh, what? If you replace `window.onload` with `window` you'd overwrite the global scope with said function.
In this case, you don't even need to surround parts of the string with parentheses. Every string following `help=` will be concatenated.
This will cause problems because ```format_p_value...``` will return a string if there aren't enough permutations (and the float conv will fail).
This should be ``option_parser.error(""%s doesn't appear ...``
Hey, I think this should read `redirect_with_null_bytes` instead, not?
Is this needed, since this is a Concern?
yes, that sounds suspicious, since the symbols bound by a polytype are not free (by definition :))
Ah, I see - I thought you've copied that line from the doc comment. Will add.
I don't think this works as intended? The fill value is assigned only according to the first axis?
I think `symbols('i j', cls=Idx)` would be even better.
The only differences should be the ones specifically related to mutability. Anything else I would consider to be a bug.
You're going to need to import reduce from compatibility for this to work in Python 3.
we don't need nulls. you should remove them.
@klimov-paul, personally in common i don't think it's so necessary to clean up `/tmp`, runtime, temporary, cache and so on folders. In our case these two files would have been overwritten anyway. But i've adjusted unit test for purity. :-)
Yelp coding style says if you're going to break this up, put one argument per line. We don't always do that in github'd code but I find it more readable and recommend it.
Agree.. I think the case in which a browser supports `performance.now()` but have yet to update their `rAF` API with the new timestamp is the only one getting the "bad" end of this deal.

Also agree, we'd need benchmarks in any case to even identify it as a perf problem. In which case we could have a perf battle with your feature-detect magic :)

On the plus side, this check only happens per-frame and not "per-tween-per-frame." Small victories.
Is ignoring the return value a good idea? At the very least you should handle EINTR here, shouldn't you (updating the timeout accordingly)?
If you set it to "" (empty string), you can do away with an #ifdef below.
I think we would need to completely separate repairs and reports in the backend, and then in the frontend we would re-add this drop down box, have it contain `scan-completed` and `scan-failed` and call it something different. This sounds like a separate patch.
revert. 

the c++ driver is somewhat special cased in this case.
I'd drop the hyphen in *dot-notation* it'll work fine.
Unless you have an idea as to how, I don't think so. Decorators mainly wrap the ins & outs of a function (ie. parameters & return value). Here, `allow_add_attachment` is neither: It's a part of the data structure passed into `jingo.render`
pep8: 156, 157, and 160 are too long?
And is it worth using [the celery version](http://arecibo.readthedocs.org/en/latest/client/django.html#using-celery)?
Very soon this will be derivable and this code can go away… but that time is not now.
why remove those assert.
Since the JSON that gets decrypted is always wrapped, we do not have to worry about automagic inflation of the parsed JSON into classes. Is that right?
the grammer there is wrong. Do you mean "Get PMs the user sent, but has yet to receive" or "Get PMs the user sent, but have yet to be received"? The former means messages sent by the user to himself, the latter means messages sent to other users.
Use proper query formatting. Same below. Use `$sql` variable if you find it ugly without using a variable.
This will need to be changed because the kernel no longer exists as a service because we are now using the HttpKernel Symfony component instead of our own kernel class.
Again, use the `.value()` function here
Perhaps

    If specified, a link to this URL will be prepended to your board’s breadcrumbs. The board logo will also link to this URL.
Is there an assertion that will fail should someone lowercase these tags?
`#` ?
I would prefer the table name also be passed as a constructor argument instead of being a global constant.
Strictly speaking, not on the clients side.  The client does need to download the correct arch which it does not currently do, mostly because that information is not burned into the binary.
this should be in a test utils somewhere? binary_paths.luvit or something?
I am using the hidden arg variable as described in PIL: http://www.lua.org/pil/5.2.html
String wrappers and constructors like this are generally a "bad part" :). Probably best to just take "String" off and use:

(al || '').trim();
The stack doesn't.  I think the e.message contains the error, so yeah that should be in the output as well.
I think this might be where we want to break stuff out into functions. Maybe roll this into some sort of a getRedListCounts(rows) method. Also, You can condense the code here using an associative array to count vs a 2D array.
This isn't necessary as the logging API handles this itself.  Part of the benefit of which is that it doesn't have to generate the string if it isn't going to actually log the message i.e. log leve = warn.

See: http://www.slf4j.org/apidocs/org/slf4j/Logger.html#debug(java.lang.String, java.lang.Object, java.lang.Object)
Please remove it, don't comment if it's not needed.
I think we should ask JSS people which field they want, because Designers had all four in views.
We probably won't, but just in case there is no username, Guest is okay for now.
Here, you can have a counter k = k+1 and check if k===2. If so, we want to call final patientlist
It would be more informative to calculate based on MAX_ENTRIES, rather than hard-coding to 200.
Don't use "i" if it's not an integer!
Actually this shouldn't matter that much since we have a clean ring every time, as long as the test uses the common setup/cleanup functions. Of course, that assumes there's no blip DNS/DHCP between when we detect the hostname and when the net_kernel starts.  Still something to watch for on our testing platforms, as you say.
this could be simlified to `any(link != from_field for link in model._meta.concrete_model._meta.parents.values())` I believe.
`qlc` for the win!
It seems to? It's variadic so it's nil-terminated.
Would this account for all possibilities?

s.type = options.type || s.method || s.type;
Weird, I always test with the globals check on and this wasn't caught...
Dang, got lost in the totally modified logic, you're right.
Yes, we might want to factor this out into a separate commit.
This line throws a compiler error on Android.  Cannot add int to IntPtr.  Should be
	dataPtr = (IntPtr)(dataPtr.ToInt32() + dataSize);
These four lines should be condensed to just `sitescan = models.SiteScan.objects.create(batch=spider.batch, site_url=item_domain)`
Would be more readable using the ternary operator:

`repouser = $1 == "homebrew" ? 'Homebrew' : $1`
Stands to reason, figured it would be tricky. Thanks!
Pushed in 146e253, thanks!
so the original behavior, using curl to download the file and pipe it to another curl to upload, did have a benefit: streaming through a pipe means that we never need to have the entire virtual disk in memory (could be many gigabytes large), or on disk.

if i'm reading this code correctly, this may use quite a bit more memory and disk space than the previous implementation (it downloads the file to disk, then reads it all into memory). excon supports streaming responses according to the readme. can you use those instead?
Add a line note
We start to see new warnings after 2008 due to the database having more recent data.
I have no idea what the JUnit style is for this, but putting a space after the > looks... weird.
Use attributes rather than service names?
In that case, 'tis cool. Though I still say get it out of the repo and add it as an npm dependency, that way we get updates for free! :smile_cat:
I remove the old model before add the newest, so the equals problem shouldn't happens.

And yeah, i agree, i mean the synchronization can be totaly useless on the renderChunkQueued.
No, this is not necessary. Removed call to jQueryIdEscape().
The code assumes that it's in a tag "if '>' or white spaces after testPos". Instead of checking for "white spaces after testPos", I think it would be much more reliable to check if there's an unmatched '<' before testPos, then the code would not get to this point, and this change is not necessary.

Note: I fixed a performance bug in DW where code would scan backwards through entire (100K) file looking for '<', so limit scan to a reasonable length (I used 4K).
What's the impact of doing this all at startup time? Obviously it's most convenient and least error-prone to do this substitution here, but would it be noticeably more efficient to only do it at the time when a caller wants to use a given string that references the app name?
Typo: targePlatform -> targetPlatform
Could we document up here (or down in ```createSpecFilter()```) what each of these two filters are for?
done. agreed.
since you dont use them anywhere might be good to remove them
Turned the above string into comment lines and added a space between `#` and `I` here. Tried to reduce the amount of code cleanup stuff in this commit.
it looks like overwrite is (intentionally?) being ignored here?
This should probably be done using a progress dialog instead of showing the entire activity as loading.

You can do that by extending `ProgressDialogTask` in the  `FollowUserTask` class
I know your changes didn't involve the hex code, but it might be nice to have that not hard coded in this specific method in case you want to tweak it later.
You can do this without a begin? I learned something :)
Another option for `update_*` is using Redis' [HSETNX](http://redis.io/commands/hsetnx) which will not create an object if it does not exist in the database.
with
```
osmosis --rb switzerland.osm.pbf --wx - | python import_osm.py
```
standards are great, there are so many to choose from :)
Do you need to do this inside of this `if` block? I don't think a view should be required to make this work. That if statement looks fishy anyway.
whitespace on the right
Hahaha I'm glad you're letting me know. Can you put that comment in the css though?
You've been just patiently waiting to get me on a spacing mistake haven't you?
Again, no message?
These create conflicts with transitions. Better to remove them and preserve our transitions instead
can you put the comments above rather than on the right side of the code? :D
lapsus calami
The `try/except` block is required for the above line.

If salt is not installed, or not in develop mode, or not in the system path, which is the case for a system which is getting salt installed for the first time, the above import will most likely fail.
init_pool这个变量必定为1？
Please remove all comments from the spec.
LuaD summary lines tend to be punctuated, let's strive for consistency.
just curios: what does use the apidoc path?
Yes, my argument was that system shouldn't really know the internal details of the system group permissions :)  But yeah, i do like the SystemGroup.systems_readable  method.  I'll add that.
Is there a possibility skip would return something else then done? For me @run == 'once' seems useless, unless there are more possible values than 'once' or 'always'
katello-generate-passphrase has moved to class scope of Puppet::Parser::Functions. That doesn't seem 100% correct to me, but it needed to get called before the template expansion happens ( exec "generate-passphrase" was not getting called until after the katello.yml.erb and candlepin.conf.erb were expanded, so they always fail back to clear text). 

I wasn't sure what the best way to get that command exec'ed before the template expansion, but that approach was suggested by one of our IT puppet admins.

Spliting it into a common sub package sounds like a good idea.
Should we be using `em` instead of `i`?
do we need space after ;?
This should be awaited so maybe it needs to happen elsewhere?
This line is super long can we make it clearer? Maybe use an if instead of ternary operation.
I heard the exact opposite. :-(
Thanks for the comments/review!  Thanks for the reference to @ariya's article, too!  Good read.  I agree with is point(s) for external APIs.

I think the `QVariant` handling is necessary because the IO/A spec for the method is: `read([n Number])`, where `n` can be a `Number`, `null`, or `undefined` (i.e., when not specified).  The QtWebKit bridge automatically calls the method that "matches" the calling signature.  If no argument is specified (i.e., `read()` vs. `read(null)`), then Qt throws the following exception:

```
SyntaxError: too few arguments in call to read(); candidates are
    read(QVariant)
```

So, both `QString read(void)` and `QString read(const QVariant &)` are required to satisfy the spec.

As for the internal `QString read(const int, const bool)`, I was hemming-and-hawing about whether to use `-1` as sentinel, just as you have suggested.  But, in the end, I decided that this "do-it-all" method (`QString read(const int, const bool)`) would be `private`, and thus not really an "API", per say (i.e., nobody outside of the class may call it).  I figured that was Good 'Nuff™ ;)  Rewriting it would be trivial; but, do you really think it needs to be rewritten, even though it's an internal API?
@astrofrog - I've adjusted this example to just be ``a.format(u.hour)`` (which results in what you expect - it fills in the `sep` appropriately).  I don't think we should actually check for ``sep=='hms'`` or something, because then we also have to check for ``sep == ('hr','m','s')`` and so on.  I think by changing the example to ``a.format(u.hour)`` (and also adding ``a.format(u.radian)``), it'll be clear that's what you want, and `sep` should only be used when you have a specific reason (and then we hope they know what they're doing).
The idea was that this function will be used in one context: to add additional elements to the `include_dirs`, `library_dirs` and `libraries` kwargs of a call to `distutils.Extension`.  Requiring all of the callers to do that work, which admittedly amounts to three lines of code appending lists together, just seemed like unnecessary duplication.

But I do think, given your question, that it could be made more obvious what it is doing.  For example, there will always be a local variable named `library_dirs`, because it is an argument to the function -- and yes, it must be a list, but that's mentioned in the docstring.  Maybe having an `if` statement in the loop rather than using a mapping dictionary to sort the arguments into categories would be more obvious?
`riak_core` does not contain an `apps` directory, so `build_plt` and `check_plt` both fail unless I modify the Makefile.
```
dialyzer: No such file, directory or application: "apps/*/ebin"
```
Your text here doesn't match the example below.
On Fri, 17 Aug 2012 10:05:43 -0700, Sergei Lebedev wrote:
> 
> In _oasis:
> 
> > @@ -23,7 +23,8 @@ Flag tests
> >  Library biocaml
> >    Path: src/lib
> >    FindlibName: biocaml
> > -  BuildDepends: core, threads, camlzip_findlib, sqlite3, unix, batteries, xmlm
> > +  # zip & camlzip: setup.ml remove the one that does not exist
> > +  BuildDepends: core, threads, zip,camlzip, sqlite3, unix, batteries, xmlm
> 
> How is zip and camlzip different?

They are not.  Unfortunately godi use the name "ocamlzip" while
upstream and Debian use "zip".  We have to account for both.  A pain
really.  If godi were releasing a "zip" package we would be out of
this mess...
Why did this change? It should not have. Make sure you've run all migrations correctly. Or just `rake db:schema:load` the original version of this.
real_strprintf is called a lot of times in many places (for example, HexStr). 

I don't think clearing it every time adds anything. The buffer is only written by _vsnprintf, which has well-defined semantics and doesn't do anything with the contents of the buffer beforehand. It returns the number of characters that will be in the output string. Nothing in the strprintf function relies on zero bytes anywhere.
Oh, that was a bad left-over. Removed it and re-pushed.
For a pointer you need %p
Added here @jrbasso
What about the other constants? Or are we assuming that it will be loaded via parameters.yml?
Do we really want to add the Hooks stuff to a file already 250+ lines long?
shouldn't affect any guice problems, just a bit clearer
Another option was to do a `object.uid.indexOf(this.uid) !== -1` and get rid of the strict option but this was a bit faster and you will likely want strict on if parsing a large ICAL file rather then small caldav responses.
Ayup, we're gonna need a `WIDEN` macro to turn `LIBGIT2_VERSION` into a wide string literal.
there is room for more tests,  $len for eg.
this isn't section used, go ahead a delete it in the theme.
`.strip()` should be at the end. Unlikely to matter.
where are all the good old `<script>` tags? and `<span/>` should be `<span></span>`
this'll need to be #411 now :wink:
Will this be ok with multiple arguments of the same class?
I did some digging, and to fix this bug you will need to do the following:

1. Remove the entire conditional in the `forceShowLogin` function in topnavigation.js
2. Wrap the call to `forceShowLogin` in topnavigation.js#init with the conditional that you removed in 1
3. Modify the `.live()` binding to read

```javascript
$(document).on('click', '.topnavigation_trigger_login', forceShowLogin);
```

The issue is that the conditional lives in the wrong place, and should only be in init, as when the user has a 'url' argument in the querystring, that means that they were redirected here because they were logged out and tried to view a page that requires permissions to access. This is why it only should exist in init, and not inside of the forceShowLogin function itself.
Good call.  2814e00
I did notice that but I misread it and convinced myself the two config options overlapped in their tool usage. Ignore me.
In the other pull request I did.
I havn't done that, because we would also have wanted to save the counter only for the classes passed to the load method, so it would complexify the code.

In all cases, I don't think performance is the main concern of people dumping several cache files in the same process :)
You can now simplify this with just `sprintf('http://symfony.com/doc/$branch/index.html', Kernel::VERSION)`
you should probably use the constant for readability:

```
\MongoBinData::BYTE_ARRAY
```
コメントありがとうございます。
確かにsymbolの方がいいですね！（推奨されてるならなおさら）
以後、気をつけたいと思います。
**Update:** I take that back!
These are jpeg bytes, we can't pass that to imshow().
necessary rather than needed, perhaps? Then sentence could do with rewording.
Note that users should be able to override the default for these attributes, e.g.

   @celery.task(backend=RedisBackend())
    def x():
        pass

and this would reset that to the default again
i'd name this class more explicitly, something like Translitteration
Right. This fixed a different problem.

The real solution is to use markdown, as you've said before. I'll get to that eventually...
This isn't being set anywhere.
Sorry about that, ignorance from my side.
Do we need the example-notification image?