labofsahil/github-event-dataset-2012 huggingface dataset:
Je suggère qu'on utilise la convention de nommage suivante:
methodePublique()
_methodeProtected()
__methodePrivate()

Cette convention s'applique aussi pour les variables membres:
monObjet.variablePublique
monObjet._variableProtected
monObjet.__variablePrivee
Cool. I didn't know that. I'll leave the quotes out then. Thanks.
@igorzi - Thanks, merging.
is Children.Find() case sensitive? that would suck :/
@filipeamoreira, hands related :)
Maybe I'm missing something, but can you explain why `innerHTML` has to be used? I thought the process went like this:

If the button node is an `input`, then set its `value` and return. Otherwise, look for a nested `"yui3-button-label"` element and set its `text`, if a label element does not exist, then simply set the main button node's `text` and return.

This is the logic I was trying to convey in the [`_setLabel()` implementation above](https://github.com/derek/yui3-1/commit/b90d3fc725a0a4b62fcc7dd8b3516571fd8582ff#commitcomment-1024226).
HTTP/1.1 defines the sequence CR LF as the end-of-line marker for all protocol elements except the entity-body
FYI, it looks like we could have just done something like
```
if( !strcmp( type, @encode(CGRect) ) ){
  //then this is a rect
}
```
rather than all the string searching stuff.  

I'd be the first to admit that the whole of DumpCommand.m is a mess :)
в последней версии в заголовке написал
Like this? "payload" : ""Message""
I like that commit-message :P
end missing ;p
Oops! Good catch in later commit though.
Why does it break your code? Can you reproduce it and write a test?
Correct again: there is no point even trying any value that is greater (or equal since sizeOfSet != 1) then the total we are trying to achieve.
Again, we avoid iterating over numbers that we have no hope of being able to use.
Can you be more specific about *which* optimization phase?
force_encoding is Ruby 1.9, and won't work in Ruby 1.8.7...
of course! They wouldn't be magic numbers if they were wrong.
Manaserv is indeed checking for the range, then checking all the floor objects and checking the position requested before picking up.

Sending the item public id could be a good solution as it would increase the code efficiency for both the client and the server (Yet, I don't know whether floor items have got a public id known by the client atm.)

There are also two ways of thinking about this feature:
1. Use the item id as you suggested, and make the server check the rest. The main drawback I can see is that you can pickup only one item at a time, and cannot, depending on how the client is handling it, pick up items which are below a drop steal protected one, even if the case is not that common.
If we adapt the client to try every items id found on the location when there are more than one, I guess that drawback part is gone.

2. Use the position where to pickup and make the server do the job for everything that's pick-able for you, as I think it was implemented atm.
This is a little heavier for the server maybe, but a little bit less messages are sent when picking up several items at a time.

Even if my will is to quick-fix the pickup feature for manaserv, I'm in favour of the first option, with the client modification proposed. What do you think about it?
i think it would be easier to just retype the line and comment out the other lines.
Ca pourrait être bien de mettre des quotes autour de la valeur non ? `<%= k %> = '<%= v %>'`
Sinon on est obligé d'écrire quelque chose comme `'http_path' : '"/mypath/"'`dans la config
I'am sorry, my mistake
getting better!   I think I may be able to take this one level further.  Soon, there will be no code left :-)
为什么是exit？不是return？
这个函数，改成Group类的一个方法吧，用来判断某个用户是否在这个group里
oh, and please feel free to fix it!
I know Webkit is currently the only "supported" browser, but it would be handy if $.os.opera and $.os.fennec should be added too. 
$.os.opera=UserAgent.match(/Opera/)?true:false;
$.os.fennec=UserAgent.match(/fennec/)?true:false;
Really? u thing that crashes are in 1608? For me stable, i thing problems are most on 1611 and maybe 1622, 1623
mmmm... good :) if frequency of freezes in VMAP (BIH) code and pathfinder code be less - it's be very good things.
when Npc : 18326 cast spell 32663 i have this error :
Check existing of map file '/server/data/maps/5563131.map': not exist!
Spell::DoSummonGuardian: impossible place for create creature entry 18701, spell 32663.
Why ?
It's pretty stable for my 1650, if you mean it.
Your crash related to VMAPS not, to issue with [11898]. I have it too randomly.
now paladins cant heal after use divine shield could be problem of this commit?
bladestorm not affected by last changes. but slow mechanic not rejected by bladestorm, as i remember.
I'm trying to wrap my head around this line. Why are you attempting to append v to a value that may already exist in params?
Just noticed that this is dependent on the exception channel change... But easily removed if you think that the exception channel change is not favorable.
Ehm, har ni funderat på _vad_ som kanske är bra att testa? Dessa (mfl) blir underförstådda om test "närmare specen/den som konsumerar API-et" går igenom.
Use case which is currently setup in the Application module bootstrap for now. 

$events->attach('ZfcUser\Service\User', 'register-pre-save', function($e) use ($config, $app) {
            $params = $e->getParams();
            $form = $app->getLocator()->get('custom_register_form');
            return $form->update($params["user"]);
        });

        $events->attach('ZfcUser\Service\User', 'register-post-save', function($e) use ($config, $app) {
                $params = $e->getParams();
        });

        $events->attach('ZfcUser\Controller\UserController', 'register-validate-error', function($e) use ($config,
            $app) {
                $params = $e->getParams();
                var_dump($params["form"]->getMessages());
        });
this works without `self.refresh`ing?
maybe TextUtils.isEmpty(str) is better. and the primaryFolder alway impossible = null. 
thanks your advice, i will taken it.
Оперативно! Спасибо гигантское!
Please have a look at http://www.mantisbt.org/bugs/view.php?id=13935
This seems to be introduced by our commit.
The old detection was better than the current "fix". I already pointed
to a mobile device that has a Chrome 8 like browser (or at least V8
and isn't auto updated like the desktop flavor). So the newer weaker
inference is a bust. The solution is to revert and add
*fake.style.background=''*

On Feb 15, 2012, at 2:56 AM, Alexander Farkas
<reply@reply.github.com
 > wrote:

> @jdalton
> I knew, that someone would come and point this out. The code before
> tested, wether element.hidden results in a display: none style and
> adds a bunch of "HTML5 styles" if not. So the old code was already
> lame.
>
> The new code only tests, wether the property exists, which should be
> reflected by the style. If it is not reflected, it is a browser bug.
> If you have a browser with such a bug, let me know. But Chrome 8
> doesn't really count.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/aFarkas/html5shiv/commit/93f37df622661aff1cf83821cab47fd46c8d43eb#commitcomment-968224
I've rewritten this answer like 5 times before being completely confused...
That being said, I don't like the idea where there's no way to distinguish between critical OKAYs and warning OKAYs. IMHO an OKAY message acknowledging a FAILURE message is *as important* as the failure message itself.
As I said (don't remember where) there should really be OKAY_FAILURE and OKAY_WARNING. Then you should be able to choose from NONE, WARNING and FAILURE for NotifyLevel, which would result respectively in logging none, all, and failure+okay_failure. I hope you see where I'm getting. Or then again, completely discard OKAYs. I wonder how NAGIOS is handlng this.
Good thinking about the warning.
This function is still used on line 296 of reveal.js and now throws an exception when called.
Using pull requests (even between branches in the same project) made it
one-click simple. It was awesome. Sorry, I'll let you do it next time. :)

On Thu, Jan 19, 2012 at 9:03 PM, Nicholas Hagen <
reply@reply.github.com
> wrote:

> I figured there was an easy way, just didn't around to looking/asking. The
> other thing I want to better understand is all the merging between branches
> and forks.
>
> Thanks for the tip though.
>
> Nicholas Hagen
> www.znetdevelopment.com/blogs
> * sent from my iPhone *
>
> On Jan 19, 2012, at 8:46 PM, "Brian R. Jackson" <reply@reply.github.com>
> wrote:
>
> > If you had written "fixes #4" it would automatically close the issue
> when the changes were pulled into teatrove/teatrove.  see:
> https://github.com/blog/831-issues-2-0-the-next-generation
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/teatrove/teatrove/commit/863f2a54e44d9f639828ba82a2d7c9af41bf6f71#commitcomment-884538
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/teatrove/teatrove/commit/863f2a54e44d9f639828ba82a2d7c9af41bf6f71#commitcomment-884572
>
Always a good idea to use in-development utilities for production…
soga，但是有保存功能的话就要用动态网页，不能用静态了，安全性是否能过关？建议吧ACE作为一个未来演进功能
Yeah I put in the $primary_key_field as a variable so it could be used in line 280 - so all good
Looks interesting Max. I've had the transform code in Recline working again since January and it would be great to be able to integrate the client side transform stuff there with backend support when that's available -- as it appears to be in datacouch (it will be a lot faster I imagine ...)
I think moving this event breaks the events.
why not if (hiddenPlayers.add(player)) return;
Cool! When it's all ready I'll look at moving it to the 2.7 branch as well.
New keyboard controls:
 '1' = set all eyes to wander
 '2' = set all eyes to idle
 '3' = set all eyes to follow mouse       
 'space'  = delete most recent eye
Great - I'll try that out now.
So, I hacked `eliloalt` to make it work (setting and removing the EFI vars) on 2.6 kernels (using `/sys/firmware/efi/vars`).

Besides setting the variable it's necessary to either run `elilo.efi` with the `-a` argument, or set `checkalt` in the `elilo.conf` file. It's also necessary to remove the `prompt` directive from the config file, because that means "force interactive (and select first by default) mode". 

The `elilo.efi` boot recognizes that the variable is set:

```
Loading: Red Hat Enterprise Linux AS                                
Starting: Red Hat Enterprise Linux AS
found alternate default choice : vmlinuz-2.6.9-100.EL root=/dev/VolGroup00/LogVol00 console=ttySG0
```

Still, there's one missing piece to the puzzle: loading the initrd. It should be possible to do it in one of two ways:

* Passing the '-i argument' to `elilo` via `efibootmgr`
* Setting `initrd=` as a top-level (global) line in `elilo.conf`

I'll check how both options work out.

So, I'll give this another short try, and if we fail, I guess we can postpone this feature to a later boottool version.

PS: I somehow mistakenly assumed that an implementation for RHEL 4 would be based on `/proc/efi/vars`, but it also runs a 2.6 kernel.
<script type='text/javascript'>
												var amzn_wdgt={widget:'Carousel'};
												amzn_wdgt.tag='babyvote-20';
												amzn_wdgt.widgetType='Bestsellers';
												amzn_wdgt.searchIndex='Baby';
												amzn_wdgt.browseNode='165796011';
												amzn_wdgt.title='';
												amzn_wdgt.width='600';
												amzn_wdgt.height='200';
												amzn_wdgt.marketPlace='US';
												</script>
												<script type='text/javascript' src='http://wms.assoc-amazon.com/20070822/US/js/swfobject_1_5.js'>
												</script>
$ лишний?
Typo, missing ), 
-if (roll_chance_i(66)
+if (roll_chance_i(66))
I don't think so,    

self.add_contact(contact, self.offline_group)  will be  self.add_contact(contact, None). 

Inside function add_contact, this will be added to the root, or  if  self.group_offline is enabled, the group will be created
LOL, je demande un copyright Bibizz
use text() function
much nicer browsing.
I don't think I understand the question.

`$world` is only set in order to have something to `instance_exec` on. It's the `self` in a step definition.
Nice. Didn't know! You might as well get rid of it completely and make Syncorro require Firefox 10. This is a totally reasonable assumption to make since Firefox 10 will be out soon, definitely before Syncorro will hit any sort of Beta audience.
Is there a reason we issue a warning rather than raising a NotImplementedError?
should we change this to an xslt?
I will convert all of these script to new security model; since the changes will be extensive would like for your group to then test it before committing it to sourceforge.
Can you make your commit message more readable for others?
I thank you a lot for your help, I had a cache problem that prevented the bundle from finding to appropriate file.

I am still beginning with Symfony and bundles like yours are really helpful.

Have a good day
Hi. Are you planning to refactor this code to make it database agnostic?
Thanks.
Okay, I going to use the metadata. Like so:

    if obj_info['hash'] == cloudfiles.Object.compute_md5sum(file):
        continue

I also noticed a thinko which is that we needed cloud_obj to be defined at that line. I am wondering why I did not meet that before. I am not going to duplicate the code for the md5 as it is used by the library internally and I'd rather have the same bugs (or implementation) in both places.

    u'hash': u'56d3661264cce360ac3df5fca33c9d3a'

That's the same as the etag, thus an MD5. The cloudfiles.object.Object.etag returned from 'hash'. A SHA1 would be 40 characters. It is also moderately harder to compute and colliding files are so unlikely that MD5 is actually a sound choice.
agreed! dont teach the new generation wrong stuff hehe
My bad, I thought this loop was nested, but I see you are right.
этот файл можно не коммитить
(This looks like the same information again...)
nein, rebase is noch völlig untauglich für normal use :D  hab ne normale skyfire genommen
ha. ha. ha. ;)
Neat. Putting the common case first is always best, where possible.
Merging Trial
for python3 mapply function should start like this (because of renamed function attributes):

 def mapply(self, ob, positional, keyword):

        f = ob
        im = False
        if hasattr(f, '__func__'):
            im = True

        elif not hasattr(f, '__defaults__'):
            if hasattr(f, '__call__'):
                f = f.__call__
                if hasattr(f, '__func__'):
                    im = True

        if im:
            f = f.__func__
            c = f.__code__
            defaults = f.__defaults__
            names = c.co_varnames[1:c.co_argcount]
        else:
            defaults = f.__defaults__
            c= f.__code__
            names = c.co_varnames[:c.co_argcount]
emmm.. ice-water.
Shouldn't that be wrapper => 1?
Great.

This local `previewClick` declaration was overwriting the global one, anyway.
Merge looks good overall. I only reviewed the previously unreviewed code.
Could you also translate the File to English and provide more information for installation?
Then the Pull Request would make more sense, thanks
Dear developers. Please fix critical crash
https://github.com/TrinityCore/TrinityCore/issues/5256
See if you can avoid another view model, we want to simplify the code as much as we can.
DELETE FROM `gossip_menu_option` WHERE `menu_id`=28138;
Agreed.  I'm not sure which of those I prefer.
Yes, that's it exactly.

My eventual goal is to promote the general permissions checks to the w2p_Core_BaseObject abstract class and eliminate it from the modules. That way *every* object would automatically support permissions without explicitly needing wiring.

Then if the object *doesn't* need permissions check (the exceptional case), the module implements its own.

I primarily care about the "90% of the time" scenarios..
Yes, but Buzz has not been tagged since November.
Minor - The log message should have been for "Entity bean" instead of "Stateless EJB". Same applies for next log message in this code block.
can there even be a filter for @Child?
At the very least you could have thrown an UnsupportedOperationException to give the user some feedback that the operation is not implemented yet.
i personally hate libraries without a changelog and most people using my stuff would kill me if i wouldn't provide a summary of what's changed.
it only takes a little effort to present an overview of changes and then people are free to look at commits if they think they need to.
tg airflow is a lot simpler, it would only present slight conflicts in terms of realism as is.
Yes I was, there is a new commit after this one to fix it though.  I made the mistake of changing how the string was formatted without testing it afterwards, although I thought that I had.
this isn't actually lazy getting the value, as lazyGet.get() is being called while guice is wiring things up.  I think it is better to make StartVBoxIfNotAlreadyRunning implement Supplier<VirtualBoxManager>, then in this module's configure() method, bind(new TypeLiteral<Supplier<VirtualBoxManager>>(){}).to(StartVBoxIfNotAlreadyRunning.class)

and make sure you never inject StartVBoxIfNotAlreadyRunning anywhere, instead.. Supplier<VirtualBoxManager>
No love for the female bill collectors and counselors beating out the males?
Does version.bat work properly for you?  I think I removed it because it was either failing to run (due to  something missing in my path), or because it was always generating "0.0.0.0" (can't remember which.)
yes this core is different in some ways then the old core. & I copy pasted into browser so tabs apparently deform it :)
Thanks to kaelima for some code.
Are you sure?

Tbh I only had tried it on the AnimationBenchmark which is using a SpriteGroup. Let me try again tomorrow... =)
In theory it "can". I've been running this for a good while now and never had a problem.
You actually add nothing to content.opf, so I assume that your function is not complete What is already there looks good, though. Oh, please don't forget indentation.
Hi An,

thanks for the confirmation. I took the fake item. Now, I'll try to implement the solution again and set up a real item and check every step carefully. I'll let you know the outcome.

thanks,
Ulli
https://github.com/robotlegs/robotlegs-framework/issues/66
Thanks for contributing.  I've applied a similar patch except I removed the anonymous functions in setTimeout and named it hide() instead of clickCall.
Part | Points
------------- | -------------
base | 1
*Total* | 1

Passes tests. :sparkles: 

I had to update the .gitignore thanks to IDLE. :rage:
ow wacht het valt best wel mee hoe kut het eruit ziet.. het leek er ff op dat het kopiieren van backupcanvas naar basecanvas niet goed ging omdat ie moest schalen oid maar het gaat goed. er zitten nog wel wat bugs in trouwens! map wordt nog niet getoond en bij het doodgaan wordt ie te ver getekend.. ik ga nog ff kijken of ik ze snel kan fixen maar ik moet eig ook zo naar bed :P
TBH I think the right thing to do would be to move it into the docs really.
@burntblark Glad you liked it. :-)

I will happily look at a pull-request for this change. For me the $template param is usually very simple (just a p tag plus a class) and it only appears in one place in the layout. I can see that others might want something different though.
That would be better written as this:

    $ctx-> _hdlr_pass_tokens_else(@_)

I should have shifted off the $ctx object in the first line.
You may also want to rename the generated file from t/00-assure-deps.t to t/00-check-deps.t.
Changed header generation (didn't work before)
I guess the way to look at it is:  
The expression inside the !() brackets is TRUE if
 - tmp is is not any of { ‘NA’, ‘NaN’, ‘Inf’ or ‘-Inf’ } (c.f. ?is.finite) OR
 - tmp is NA
In such cases we don't want to replace the expr value with 0
In all other cases we do. 
Does that make sense?
Are these references to hazard logic tree paths? If so, this seems like a bit of an obscure way to reference the hazard input used to produce these results. What requirements have Risk science people given us?
Yours works, just found odd that the same doesn't work with my url.
The bulk of the implementations for both getOffScreenNodes and getOnScreenNodes seem like they could be replaced with Y.DOM.inRegion().  I'd like to at least capture any issues preventing it from being used in this context.
Worth noting, it is not necessary to check for a flick event (unlike removed code) because this  method will not execute in between a finger lift and the flick animation.
This line it too long. Besides, what's the purpose of this line?
OK, that does make sense. But then, what about the case where the "download" feature is not supported? The code has changed for that case, so I'm expecting the user is going to get "exportFile.zip" instead of "a.zip".
Enabled by default is fine.
See http://villavu.com/forum/showthread.php?p=1077419&postcount=6, let me know what you think.
Can you also put this in `config.rst` under a subsection at the end? Otherwise people who don't usually look at the man pages (like me) might miss it.
Ok, fine, I think you're right, but what about the config information below it?
Why pass the extension around?
demonstrates
Latency would be useful though, but it does not fit in this framework  
because it has to be measured from the outside.
I think we should point them to http://doc.akka.io/docs/akka/2.0.3/project/migration-guide-1.3.x-2.0.x.html
I think this should be accompanied by a factory which accepts `ActorContext` and has the appropriate WARNING signs (and uses context.system, obviously). Then I think this would make sense :-)
None of these associations were used.  If we want references, use cards.
This should consume any whitespace after `!optional`.
add , model='ar1'
We cant to use the db.py for the migration script that migrates from the old datastore to the new one. Because on an issue with translations in ckan core, error messages raise an error themselves when we run the migration. In addition to that, the migration script should run without a whole ckan installation. With the change above, it is possible to copy the db.py to the migration script and then run the migration independently from ckan.
testsysadmin wasn't already being added to the admin role?
Same as above. If you do change it, don't forget to change it here too.
NOMINMAX here removes 'min' and 'max' defines from windows headers
memleak of valueLower/Upper?
We usually have the else (and the { ) on a new line.
Yes. and it should always be valid to pass the pts through. (if we always should use it could be debatable)
'regarding'
No spaces around method params.
Just the match if the between data is empty, else the pair.  newck is an instance of one of the chunk classes.
I.e. the companion object to this class seem completely unrelated
Should be: `null === $limit ? 1  : $limit`
Please don't mix single/double quotes. Rosy uses double quotes.
Because they might contain gunk prior to entry, I found some during debugging, so I put them up there.
I thought you had to push/pop these to restore the previous state?
And nobody will understand except superpythonistas
Dimension or component?
закомментированный код убрать, если не нужен
I think you could add a line to test `sum` to here as well, right?
I thought about it, but chosen this indent because here we do methods from self. If first line will be object - for me it's ok to do indent after, but here all lines are methods.

```ruby
Object.
  method1.
  method2...
```

```ruby
#self.
  method1.
  method2...
```
I think you've added the functionality of passing an array of records to `IM`, not? It could make use of it here, but in any case, I still don't know if we'd want that feature to exists there.
Yup, probably, I added those because I was having problems with my editor. E.g.

https://github.com/lifo/docrails/commit/f069ec162fe92523eb5f0e8c0139005fed671a0c
FYI

    mexists(classParamss)(isDisallowed)  

exists for this purpose - higher in both efficiency and aesthetics.  There is also mforeach, mmap, etc.
Removing 5 and 6 sounds good. I would put some more of what you want to get across, like your note 3) above, into words. That's much clearer then let the user draw the conclusion from a lot of numbers.
This code is outside the scope of a test and therefore will be run outside the context of a test. Please put it inside a `before` block if it's setup.
Ouch.  Synchronizing on the handler's monitor every time we dispatch an event to it.  Apart from the inherent performance hit, there's a real risk here of surprising the client with deadlocks when his own code doesn't even perform any locking.  Suppose you have two handlers which end up transitively firing events to each other and expecting a response.  Especially given the decoupling nature of an event bus, that could be quite hard to track down.
i miss gmock: http://code.google.com/p/googlemock/
On 13.04.2012 05:33, Christopher Smith wrote:
>> +        # TODO better data structures!!!
>> +        #print len(P), len(S)
>> +        # Use the "normal selection strategy"
>> +        lcms = [(i, monomial_lcm(sdm_LM(f)[1:], sdm_LM(g)[1:]), sdm_LM(f)[0]) for \
>> +                i, (f, g) in enumerate(P)]
>> +        i = min(lcms, key=lambda x:O((x[0],) + x[1]))[0]
>> +        f, g = P.pop(i)
>> +        h = NF(sdm_spoly(f, g, O, K), S, O, K)
>> +        if h:
>> +            S.append(h)
>> +            P.extend((h, f) for f in S if sdm_LM(h)[0] == sdm_LM(f)[0])
>> +            P = prune(P, S, h)
>> +
>> +    # Now interreduce it. (TODO again, better data structures)
>> +    S = set(tuple(f) for f in S)
>> +    S2 = set(S)
>
> You don't need S2 to protect S during iteration over the permutations; don't forget to play in the sandbox :-)
>

I'm never going to trust this on my own ^^. But done.
I also did not immediately notice it.
How about `"Turn a Compound into a SymPy object"` (no "Tuple")?  If you would like to keep the word, I'd ask you to lowercase it.
The printing system provides two possible implementations:
  - methods in each class like here (`_pretty`, `_latex`, etc)
  - what is in sympy/printing

The second option completely disregards namespaces. Some of the classes here already use the second option, however I will refactor them in order to use the first option everywhere as the clashes arising from the merge of all namespaces is non-trivial (for instance `Point` and `Differential`)
It looks like you're only using this function directly in `setdicteq`.  Don't things work with the conversion to string?

I don't really like the idea of converting to string to compare for equality, because it slows things down and may sometimes fail miserably.
public $disabled;
> String value would be treated as the class name (more details available in {@link Yii::createComponent}).

I'd rather say:

> The value of this property will be passed to {@link Yii::createComponent} to create the generator object.

This would be more consitent with other property documentation that are passed to createComponent.
@creocoder I dont want to continue this arguing. You are not being constructive. Just giving insults.
@cebe is assigned here. So Im waiting his thoughts.
Looks like I made a forward declaration for windows based off of what was in windows.h, but didn't bother to update the non-win32 case. I'll address this in another commit.
But the result of the assignment persists after the if statement...  Therefore, regardless of whether the result is nil, so takes the value of fTarget before getting passed to RemoveTarget
Don't know. If yes, we could probably do the same with another one at line 128.
You're right, it should have been inheriting LetTest or LetWithLambdaTest.
Sounds like we need a different implementation of deepEquals(). Probably some assumption is no longer valid.
typically these operations are also fast because they don't require: 

1. sending data between the clients
2. moving the record in the data store.
Did you add the blank line or should I?



On Tue, Sep 4, 2012 at 4:06 PM, Sam Kleinman <notifications@github.com>wrote:

> In source/administration/replica-sets.txt:
>
> > -   reused by the new member.
> > -
> > -   .. code-block:: javascript
> > -
> > -      cfg = rs.conf()
> > -      cfg.members[0].host = "mongo2.example.net:27019"
> > -      rs.reconfig(cfg)
> > +Replacing a member is useful if you have an existing configuration
> > +and only want to change the hostname of a member rather than completely
> > +remove all configurations related to the existing/previous set member.
> > +
> > +Use :method:`rs.reconfig()` to change the value of the
> > +:data:`members[n].host` field to reflect the new hostname or port
> > +number. The :data:`members[n]._id` field does not change as a result of this
> > +operation.
> > +.. code-block:: javascript
>
> blank line before this line
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mongodb/docs/pull/176/files#r1527042>.
>
>
nice use of 205. :)
do we want to add a test for `wiki.content.extract_code_sample('The_following_is_a_new_section', doc_src)`?
can you set that var as a default to one .. e.g. 'reports_view_report' and then override it 'if edit or not report', then just set the redirect_url below? will be a bit cleaner and not as many lines :)
Authorizations sounds good, will change.
acp_ prefix? I cannot guess at all as to what this event is for.
Casts should gain a comment explaining their purpose here.
It only hides it if the line the comment is on was changed. Since you didn't change anything about the `);` it doesn't hide it. ;) No worries.
`count` -> `sizeof` per the coding guidelines.
Please add parameter and return value documentation to the docblock since you changed this function.
I seem to recall this having been changed to the current version, in which case this change needs to be taken out.
Sorry, I meant `logging.error`,
Ești sigur că merge? filter nu scoate listă, ci QuerySet. S-ar putea, totuși, ca ei să fi măsluit __equals__. Altfel, aș fi testat: `not formula.count()`
This seems very strange to me.  My understanding was that the `path` library was supposed to do filesystem path construction in a way appropriate for each platform.
The "express" confuses me? If this is the timeline for the entire request, I would prefer to see "request" or something like that.  Also, since this *is* a timeline, why the ":timeline"?
A small matter of style:  we sorta have an convention of naming the variable containing the libraries as `libfoo`.  It might be worthwhile to maintain that for consistency.
the curly brace should be on its own line.

and the typehint should use the interface instead of the implementation, and use a use statement to make it shorter
I'm scared :)
404 - Page not found
=== here as well, and below.
Actually, maybe its not "obvious"...
Needs a newline before the start of the text. i.e.
```
        """
        Creates a ...
```
Are you aware of the result of this expression, depending on the value of lang ?
This line and 187 should be `{stop, normal, State}`, not `{stop, Reason, State}`. We don't want the sink supervisor to print errors about this process dying, and all the other processes that care about when this one dies are monitoring it, not linking to it, so exiting `normal` is fine.
Please use curly braces in these cases too.
I read your note!!! in the class doc comment.  I also don't like the implicit assumption of Prototype==Model.  However, every use-case for `clone()` I can think of wants a Model* as a result.  So why not move the cast into the clone() method and have it return Model* instead?
Please don't abbreviate the name. It will be more readable
`post :create, :format => :xml` because `request.env['Content-type'] = 'text/xml'` from `before` section has no effect.
title2 not title1 :)
While record notation binds tighter than function application, it's nothing that's immediately clear, so I prefer to have the extra `$` here.
Okay I see, -text is to unset.
On second thoughts, I'm just going to drop the homomorphism bit...it's just pretentious.
Prepend a "FIXME:"?
I used gpdb 4.1 in my laptop. I will try gpdb 4.2 and check the error message again.
Yeah… I was kinda just taking a tip on what preexisted in HEAD already. I'll remove it for now.
This paragraph doesn't make sense to me.  Do you mean the total number of jobs?
`.*` is a greedy match and gobbles everything up. It's probably good to add a ? after it to make it less greedy. e.g.

       match = re.match(r'Mozilla.*?\((.*?)\) (.*?)', ua)

Further it's probably better to match more specifically with "everything up to the next (" or "everything up to the next )". You can do that like this:

       match = re.match(r'Mozilla[^\(]+\(([^\)]+)\) ([^\(]+)', ua)

(I haven't tested that, but it should work.)
Is this change acceptable?
the variables are hardcoded in etc/yaws/yaws.conf  (see below)

should i change it to read

HOMEBREW_PREFIX/yaws/1.94/lib/yaws/examples/ebin

or do you have a different structure that is ideal?


########### BEGIN CONF FILE

# conf for yaws

# First we have a set of globals that apply to all virtual servers

# This is the directory where all logfiles for
# all virtual servers will be written.
logdir = /usr/local/Cellar/yaws/1.94/var/log/yaws

# These are the paths to directories where additional
# beam code can be placed. The daemon will add these
# directories to its search path.
ebin_dir = /usr/local/Cellar/yaws/1.94/lib/yaws/examples/ebin
ebin_dir = /usr/local/Cellar/yaws/1.94/var/yaws/ebin

# This is a directory where application specific .hrl
# files can be placed. Application-specific .yaws code can
# then include these .hrl files.
include_dir = /usr/local/Cellar/yaws/1.94/lib/yaws/examples/include
this line uses the helper from nifty-generators.  In this example, Hogjaw is the page title and is displayed at the top of the browser.  "false" means the title doesn't appear in the body of the page (you can set it to "true" and the title is wrapped in an H1 (I think) and displayed at the top of the page.)
Good point. We don't need "the reddit server".
for future Python endeavors, I wouldn't recommend putting constant variables like these _inside_ the for loop (very inefficient)
mean should be means
I think that `localhost` and the default port are quite hardcodable, and since we don't have a different session store that doesn't require these you can either throw a *no store configured* error or just hardcode the mongo defaults (which seems fine to me). Defaulting to base session store (which is an abstract class if I remember well) seems just wrong.
@fred-o what do you think about moving this into the java bundle and then to check if the completion bundle is active?

`(cabbage-bundle-active-p 'completion)`

@jone what do you think?
What's with the extra comma? Your jshint should pick this up.
You're right! What a silly API.
Newline at the end but inside the brackets?
shouldn't that be #else here?
在编译的bin目录下~但是直接使用会有点小问题，主要还是library和lib的问题。

On 2012-3-31, at 上午12:48, Yihui Xie wrote:

>> @@ -60,22 +60,22 @@ sudo apt-get install r-base-dev
>> 对传统或高级Linux用户来说，装任何软件都还有一套古老的办法，那就是用伟大的`make`命令从源代码自行编译，很有黑客范儿。大致步骤很简单：从网上下载软件的源代码，通常是一个`*.tar.gz`压缩文件，解压缩，然后在终端中切换目录到解压缩文件夹下，运行：
>> 
>> ```bash
>> -./configure
>> -make
>> +./configure --enable-R-shlib --with-x
>> +make -j8
> 
> 不需要吧，make之后就已经有可执行文件了，具体在哪个目录下我不太确定，但我基本上肯定是存在的。make install的过程基本上就是把编好的文件们拷贝到相应的系统目录下。
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/yihui/r-ninja/pull/2/files#r629493
I removed this because I believe you are supposed to get a unique 0-length file, or get an `IOException` - never a file that has content already in it. But I may be wrong.
Remove CoreMatchers. in this line.
I'm fine with either.
This should be public, right?
I don't like that this leaks outside of the DOM namespace. We should move all references back into Batman.DOM.
Might want to assert that you got the JSON you expect, something like:

    if (!(profile && profile.identity && profile.identity.userId)) return cb('unexpected data!');

If the pid break, EVERYTHING breaks.
i don't think we need to check for `this.link` first, because `dojo.isObject(this.link)` wil return false if `this.link` is undefined. Same below with `this.title`.
Why ``update-notification`` here instead of ``module.id``?
Missing translation?
You can nest these conditions so you are only checking the value of lineSelection once, and the code is a bit more clear:

```
if (!lineSelection) {
    if (multipleLine) {
        // For a multiple line selection transform it to a multiple whole line selection

    } else {
        // For one line selections, just start at column 0 and end at the end of the line

    }
}
```
I think we should actually be committing newColor here (and having that be what we overwrite above in the `!newColorOk` case).  Otherwise non-normalized input from the user (e.g. "rgb(10,20,30)") will get changed to normalized text (e.g. "rgb(10, 20, 30)") as soon as they type the last char.
I think the world is divided in this one... Chrome uses "Editar", but iWork and Office suites use "Edición". Lion as well. I guess we should go with "Edición" then. And make it consistent with "Visualización" for "View" and "Navegación" for "Navigate".

This doesn't bother us, though. I haven't been able to locate a discussion about this anywhere :)
Changing removeMenuItem to delete the parent li of the menu item appears to make this test work as expected now.
change made in 1bce0f3
I'm sure I'm guilty of this as well, but we need to be consistent. Either uniformly adhere to assertIn and assertNotIn, or assertTrue and assertFalse.

Actually, I'd like to convert to using nose test runner so that I can use standard assertions:

```python
assert 'items' in self.db.tables
````

I won't bother with this until after the first player release though. Should definitely open a ticket though at some point.
I didn't create a separate mu4e package because there isn't a separate package.  mu and mu4e are distributed together by their author.

From http://www.djcbsoftware.nl/code/mu/mu4e.html, "mu4e is part of the normal mu source package and also available on Github.".  The elisp for mu4e is here: https://github.com/djcb/mu/tree/master/mu4e.
I modified hostname to #{@hostname_command}
The JSON functionality was removed. What is unused and therefore useless, or it has just been moved somewhere else?
Can you also add a few words to the main doc mentioning that it does both continuous and discrete variables?
Register CSS assets as well. It's long been missing in hasjob.
Could you add a xsd:deprecated element?
is a fully qualified type really needed?
So no, this is ugly.
Add one empty line before and after the *load* block.
A 400 Bad Request would be more appropriate here, the problem isn't with the server.
In my opinion, this is the responsibility of the test. We should know what page we want to load.
Do you actually need two callbacks here? What's the downside of just having one that takes care of both scenarios? `createPropertiesFromManifest` will already take care of attaching the elements to the correct divs, right?
Capitalize the "m" in Maker.
A warning should happen in this catch. It's a bizarre enough situation if a backup fails like this that we'd want to know.
same counter thing as above.
nit: Doesn't look like data size ever changes put it in a var.
英語が「Current view」なので「現在のビュー」としていましたが、機能は現在表示している位置をブックマークしているので、
「現在のビューをブックマーク」に修正しました。
This is a legacy pickle test and i can't change it to passive-voice. Also, this is a operation so i think it goes better as a `when`:
`
    Given a database exists with security_group_name: "<rds_sg>", ec2_sg_to_authorize: "<ec2_sg_to_authorize>"
     When I go to the database's page
     And I follow "Start" using a cassette named "<cassette_name>"`
I'd rather see a stackoverflow upgrade to the current forum than a simple forum upgrade !

But it's your forum, so your choice...

Le 1 nov. 2012 à 13:22, Florent Blaison <notifications@github.com> a écrit :

> In src/ZfrForum/Service/ThreadService.php:
> 
> > +     */
> > +    public function __construct(ThreadMapperInterface $threadMapper, AuthenticationService $authenticationService)
> > +    {
> > +        $this->threadMapper          = $threadMapper;
> > +        $this->authenticationService = $authenticationService;
> > +    }
> > +
> > +    /**
> > +     * @param  Thread $thread
> > +     * @param  Post   $post
> > +     * @throws Exception\LogicException
> > +     */
> > +    public function addPost(Thread $thread, Post $post)
> > +    {
> > +        if (!$this->authenticationService->hasIdentity()) {
> > +            throw new Exception\LogicException('A user has to be logged to add a new post');
> Yes the prupos of this is to show good coding practices but we want to respond of one need too. Z-f.fr's forum is very old and we want to upgrade it with this forum.
> 
> So we must learn from what is currently on the different forum ...
> 
> Later why not doing a Stackoverflow clone ;)
> 
> —
> Reply to this email directly or view it on GitHub.
>
Looking into this further, it does look like it's causing some problem. SASS now seems to be placing everything else under #tandem_page_links, so styles are mostly completely broken.
这种能指定到具体的产品页么？
Seems like ':notify => true' should be the default behavior.
Do we still need the checks for packages array length? I think they are redundant since we've removed the offset.
> gettext does allow for a comment to be added to the po file to provide context for the translators

It does. E.g:
#: app/views/layouts/_footer.haml:6
msgid " Red Hat, Inc."

Good translator should either check source or see it live in product and see the context.
Good suggestion.  Updated w/ commit d68a667e40f7ebe5fd666f490101794718603498
I think the fill_value attribute has significant value for a Table, where
each column may have a distinct fill value.  Then filled() actually
iterates over every column and fills independently.  Doing this without
fill_value would be inconvenient.


On Mon, Nov 12, 2012 at 6:58 PM, Erik Tollerud <notifications@github.com>wrote:

> In astropy/table/table.py:
>
> > +        self.format = format
> > +        self.description = description
> > +        self.parent_table = None
> > +
> > +        self.meta = OrderedDict()
> > +        if meta is not None:
> > +            self.meta.update(meta)
> > +
> > +        return self
> > +
> > +    def __array_finalize__(self, obj):
> > +        BaseColumn.__array_finalize__(self, obj)
> > +        ma.MaskedArray.__array_finalize__(self, obj)
> > +
> > +    @property
> > +    def fill_value(self):
>
> I know this is taken from numpy, but I'm not sure the answer to this: is
> the fill_value attribute used by anything other than the filled and
> similar functions/methods? That is, is it used in arithmatic or sum or
> something? If not, what do you think about eliminating it? I've found this
> to generally be a confusing aspect of masked arrays the few times I've used
> them (that there's both fill_value and the arguments to the functions).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/astropy/astropy/pull/451/files#r2104372>.
>
>
This should be "./utils" - haraka fixes up the paths for plugins so it should load right.
Methods ending in a question mark should return `true` or `false`, not a Relationship.
Me too, that would make it an owning reference to the underlying data. The memory referenced by .c_str() is deallocated when the underlying std::string goes out of scope, so after the Init() function. Keeping a global reference to it results in use-after-free problems...
ACK, but it was not used before :D.
Quite sure it won't; maybe on other platforms?
I think we should just use ``qApp->applicationName()`` instead of ``tr("bitcoin-qt")``.
Also, you might need a `ctx.save()` at the start to match the `ctx.restore()`.
why are you not using // //
Am 30.07.12 21:12, schrieb David Buchmann:
>> @@ -0,0 +1,111 @@
>> +<?php
>> +
>> +namespace Midgard\CreatePHP\Type;
>> +
>> +/**
>> + * Common base interface for all type definitions
>> + *
>> + * @package Midgard.CreatePHP
>> + */
>> +
>> +interface NodeDefinitionInterface
>
> i agree we don't want to build more classes. but if we are serious about
> the interfaces, there is something missing here. the interfaces need a
> way to get from node to type/property/collection and probably the other
> way round too. otherwise how could we render a property of an entity in
> html?

Well, we can't. That is somehwat the point I want to make. We have an 
RDF structure and we represent it in HTML by modeling it in classes that 
implement the node interface. But the rdf structure itself is 
independent of this representation and could be rendered into some other 
format, too, if someone wants to do that (e.g. at some point we might 
want to render json-ld from an entity)

> if the only link between node and rdfa elements is that coincidencially
> they are implemented by the same class and represented by the same
> object, we gained nothing. but i think its really just the getNode on
> all rdf elements and the getRdfElement on the node (allowed to return
> null in case there is no such element bound)
> the render method is on the node, the rdf elements are really pure
> container elements for metadata, potentially bound to a domain model object.

having two separate and parallel structures might be easier 
conceptually, but I think in practice this would lead to a lot more code 
and strange error conditions, since you would constantly have to switch 
between the two and keep them in sync. So I would definitely stick with 
one object tree, where every object implements the node interface, and 
one of the rdf interfaces. The only difference to the situation now 
would be that the rdf connections (like rel and rev) are also accessible 
with separate functions.

>
>> But this could be easily fixed by having a getProperties method in the
>
> hm, but then what about the collection children of an entity?

Actually, calling entities in a collection children was a bit of a 
mis-nomer on my part. I've been thinking about a better term for them, 
but in the kind of ill-defined hybrid dom/rdf model, children seemed 
kind of acceptable.

But they are not really children in the sense that there is a 
hierarchical 1:m relation between an entity in a collection and the 
entity that has the collection as its property. It can be any kind of 
relation, e.g. if a person entity has a collection called "friends", the 
person entities in this collection wouldn't be considered children (they 
have a "friends" property of their own, they might be in multiple 
collections, and so on). So technically, the collection's function 
should probably be called getLinkedEntities or something like that. 
Maybe there's even a fancy RDF term for this, but we'd have to look that 
up :-)



>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/flack/createphp/pull/12/files#r1266527
>
OK. What I understood from your email is that I shouldn't bother using `handle` because httr will take take of re-using the correct handle for me (is that right?). If it is so, then I'll probably pass `url` to `spider_svn` rather than a `path` and a `handle`.
You prevent all the logic after the return statement to apply if `new` is a relative path so i don't think this implementation is the right one.
@tanoku Curious what you think of this. I wanted to be sure that a zeroed-out `git_filebuf` was not mistaken for an initialized one, so I added this `fd_is_open` bool. Without this, calling `git_filebuf_cleanup()` on a zeroed-out struct would have ended up calling `p_close()` on fd 0. There was at least one error path in the codebase which could have resulted in that (had the error ever actually occurred).
On second thought, I wonder if `GIT_ENOTFOUND` is the correct error code in this particular case.

Shouldn't a reference pointing to a non existing object be considered as corrupted?

If this is the case, my vote would rather go to returning `-1` and setting a explicit error message.
agree with eonei002 above, otherwise ship it ...