labofsahil/github-event-dataset-2012 huggingface dataset:
This makes the mobile experience pretty unbrowsable... might want to remove the fixed position on mobile
This string from Issue #2 (it should be excluded before export this localization)
Could replace with something analogous to `RepoUtils.logBookmark`.
Why don't you write it rather 'get_serializer' ? To match the rest of the code base :)
I know what you're trying to accomplish; though we're still in a manual here and the style should be adapted to it if possible.
If I got it right invocation of callbackOnSuccess only occurs in the positive result (when user has right for this digital good SKU), but if he doesn't have right would callbackOnFailure be invoked or neither of them? In that case user can't be sure that the response is negative.
Similar comment is apply also for isAppWorldInstalledAndAtCorrectVersion method below.
for kernel, use my source, i will add BLN and torch function too, my kernel source is compatible with cm7 and cm9, so you only need build one kernel and use zImage for cm7 or cm9 ;)
i will update if i have new patch :)
Yep, comrade. Our script not only 4chan, and, largely, not for 4chan - but for russian imageboards.
It is powerful crossboard tool, seen. Look at full list, comrade: http://www.freedollchan.org/scripts/stats
Many wakaba and kusaba imageboards doesn't have original file names, and #imgn was fully useless in this case.
I decided, that #imgn is good alternative for #img, as yet another operator for filter image parameters, not only image names.
Also, early I has write some article for this stuff, and added #imgn there. You can translate it from russian in google:
http://www.freedollchan.org/scripts/spells
That not to write three hundred expressions for each imgboard, I use one common crossboard xpath to most of them. Yep, It is almost the same as getImgInfo, but not contains "em" - specific wakaba and hanabira "(size widthxheight)" container. Maybe, I will merge that.
Also, you still can filter image names, isn't it?
I Like it! It increases the connections when there are more keys. And It allows a lot of phones to connect when there are no keys so that a 4S can get on. And all the objects from other phones are dropped anyways..
Get your lil birdie butt in irc lmfao
How would you suggest i inject an excon instance created with :mock => true into the code under test? Play with Excon.defaults ?
Why does allow_none=True work here? If None is passed, and a numpy.asarray([], ...) is returned, won't that cause an invalid access if you try to use the dat's data?
The previous line assigns the value to object's property, so it's safer to refer the latter in case you do a write operation on this value. 
Is it enough to justify this line of code?
then I might as well `c.get(name)`
You mean that the handling of errors should move to the EndpointActor (do  
the same stuff as above when the Future fails)? I was actually thinking  
about that and talked with Roland about it. He told me that as we already  
have an ActorRef, a Future might be overkill. I am personally a bit  
undecided about this. I conceptionally like the Future version better, but  
on the other hand I feel it is a bit heavyweight. Does it have any  
performance implications? What is your opinion?
How about putting the ExecutionContext into the creation of the breaker?
for java it should be `Duration.Undefined()`
there is a CrossVersion thingy in this file also, is that correct?
Great, let me know when you've checked it
I also think site is the way to go. The stable and milestones already have nice versioned directories and `-SNAPSHOT` can just be point to `snapshot`.

Can't we use the doc string replacement tool for this?
ok will change.
I think this should be 'DomainNames.member.%d'
I think we should either add keyring as a dependency in setup.py or put a try/except block here to give the user a readable warning message.  Thoughts?
The code in this system_check is mostly identical between SVN, git and CVS. Except that the code in CVS was modified a bit since your branch was created, but those differences can easily be unified again.

So, we should try to factor the common stuff here into a method of SelfUpdate::Base. This way, we have to maintain and fix the code only once.
Only nitpicking here, internet is a network, and nothing technically forbid using smb accross the internet (i did, once), even if it's not really sane from a security POV ^^. So comments are not really informative :)
why all the empty lines? please don't put empty lines at end of file :)
I cannot do that if I want to have proper conversion from given instance to string - the example is DateTime and TimeStamp which have different representation in XML
Isn't this file duplicated in ckan/public/base/vendor/resize.js?
Uhh, missed that one. Sorry.
Typo

url = url =
I wonder why you removed the vim-trailer, appearently u're using another editor though. but that's okay.
I'll merge it after having slept and will also work through all your other comments then. Have a nice night.

Christian.
A few whitespace issues (there's some extra spaces in front of sorting.sortAttributes above as well)
Yea will readd the replace logic. I dropped it because DVDOverlayCodecFFmpeg always sets replace true. But you are right - i should have checked the other subtitle implementations which need this.

Will also change += back to = and add the offset outside of that method for pgs and assign pts and dts to the avpacket as you suggested. Will ping you again when that happened.
item[0].m_strPath contains "videodb://1/2/", not sure why.
The movies are store  from 1 on
Geen index op logged_time?
Hasn't been added to `AsyncClient`.
Good to know. `env python` is still nicer for testing a checkout of the code.
> Correct me if I'm wrong, but AFAIK SSL evolved into TLS, so it's
> basically the same thing.
It was a brain fart on my side, sorry. I meant SSL and *compression*.
Sorry, I just realized that parseFlags() wasn't static.  Given that mPermamantFlagsIndex is a member variable, can we make this a one-argument method (taking only flags) and have parseFlags() directly manipulate mPermanantFlagsIndex?
Il faudrait virer `'mon_fichier'` et prendre le nom en paramètre de la fonction.
You just need to be pulling the right header. This function ought to be coming in already from rtems/score/rbtree.h I think.  It's OK for posix code to call score code, so you do not need to use rtems/rbtree.h
In ruby you have methods ending with a "?" (it is a convention to say the method returns a booleaen). It has more (ruby) flavour ;) You should use it.

c.f. : [Object.nil?](http://www.ruby-doc.org/core-1.9.3/Object.html#method-i-nil-3F) [String#empty?](http://www.ruby-doc.org/core-1.9.3/String.html#method-i-empty-3F)
Can't we search for videos using a keyword instead ? This could come in handy if we want to adapt the behaviour to another silly query. And having hard coded video links is ... erk :(
That's definitely stuff to document. The value is entirely arbitrary and needs to be experimented with. The thing is, simhashing calculates distance between hashes as the hamming distance between the values. As such, the intermediary hashing algorithm and how many bits it has (here we use MD5) will impact what distance is significant, and what distance isn't.

I'm thinking this is purely a documentation issue, and showing some examples may be nice here, rather than shoving it in the config file.
I considered that, but IMHO a variable name like 'histogram'  isn't a good name for "configuration data pertaining to histograms".

I also realized, as we add configurable behavior, it doesn't make much sense to split out bits of the config object into new objects, it also make things more complicated.  

So I suggest that each backend just has a ref. to the full config object. that way new configurable logic can use the config data without needing to add more of these `subsection = config.subsection` things
I've just done that.
I agree with you but I've not done that because it is much more complex to implement.
Graahhh! I always do this =___= I'm sorry, it's been re-commited.
This `typedef` is unnecessary, and just obfuscates the type of the properties below.
Headings must be nested hierarchically.  You may not put an H4 section directly underneath an H1.
Not that important - but is it not possible to do this somewhere in the initialization path?
不要加这些乱七八糟的内容，这是git改干的事情(维护作者信息)
think this should be "on as separate" rather than "in a separate"
En plus d'ajouter du support pour d'autre browsers, comme IE et Opera.
This block is only triggered when the number of components exceeds the configured maximum.   (i.e. I moved the `FrameDecoder.maxCumulationBufferComponents` here.)
OK, I checked the [http://docs.python.org/c-api/long.html](Python 2.7) documentation and -1 indeed indicates a possible error, however the docs also show a cast for the unsigned types, i.e., `(unsigned long long) -1`.
This is also mentioned above in the parameters, but it would help if we could clarify *when* this would be the case.
I think this is generic enough for a change in OpenLayers.Format.GeoRSS, i.e. in upstream OpenLayers.
What is this code for: if (sizeof(s)<=0) {
Spaces around `{}`: `{ |ex| ex.message }`
Plese -> Please :)
Hmm, Agreed! (how to fix that, PR again?)

but I think we will add https://developer.mozilla.org/en/Introducing_Content_Security_Policy pretty soon to the same initializer - and dispatch initializer is huge enough.
What happens if I define ``def myInnocentMethod[T,U](block: => T@cps[U]) = reset(block)`` and use that instead of reset?
Will do this when documenting reflection in bulk.
Minor, but would prefer a newline after complete function definition.
You could use Ransack here: 

```ruby
Spree::Promotion.code_cont(@order_coupon_code).
where(:event_name => event_name).first
```
On Tue, Mar 13, 2012 at 7:44 AM, Aaron Meurer
<reply@reply.github.com>
wrote:
>> +                if _is_indirect(member, obj.__doc__):
>> +                    c_indirect_doctest.append(member)
>> +
>> +            # Iterate through it's members
>> +            for class_m in dir(obj):
>> +
>> +                # Check if the method is a part of this module
>> +                class_m_mod = None
>> +                class_m_obj = None
>> +
>> +                # Gutsy hack; need to expand reasons
>> +                try:
>> +                    class_m_obj = getattr(obj, class_m)
>> +                    class_m_mod = inspect.getmodule(class_m_obj)
>> +                except:
>> +                    continue
>
> Does this happen regularly?  What exceptions are thrown? I don't like this bare except, because it will mask any problems with the code in the try block, potentially skipping modules or classes with some other problems without any warning.

This error is caused by a function, __abstractmethod__, which when
combined in getattr throws "AttributeError". It has shown a notorious
behavior in past. (googled it). I will currently fix it by not using
the above bare except, and rather put a check up front for functions
to be skipped, similar to the earlier approach.

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sympy/sympy/pull/1114/files#r549553
Yes. `CoordSystem` can reach into `Patch`. The only place where this **may** be necessary is in `_fill_in_gaps` where when connecting two `CoordSystem` the other connections are automatically deduced. I will remove it **for the moment** and see later if it was indispensable.
I don't mind the "iff" abbreviation like you used before.
velcoity -> velocity

Make sure to run your spel checker. In vim it is:

```vim
:set spell "on

:set nospell "off
```
I saw, that too many augeas related code is used many times through out gloves' agents. This code doesn't belong into agents in my opinion, so I created the augeas_wrapper as a quick "fix". The problem is that I use ruby as a C/C++ because I'm not so experienced in it.

Note to API:
I expect serialize/deserialize would be only one of many read/write like functions/approaches which the wrapper will expose. Just because those are too strict - not all augeas trees can be meaningfully flattened by those functions. The best way, as I see it now, would be to have only one couple for read/write actions and allow injection of some code into those wrapper's methods. This injected code would have modify the conversion according "high level" needs. So, the current approach (conversion from augeas tree to hash) would be one of many. This is where I want to get the wrapper ... but I have to read a bit before I'll get it into this state ;-)
add

```
#' @rdname hook_animation
#' @export
```

above line 77; similarly for `hook_r2swf`
in the past i have used password, password_hash or blowfish_hash. i'm not too bothered. but strictly speaking, it is not a password, it is a combination of a salt and a hash
Consider renaming to `registerUniquMetaTag`.
I don't like the new methods mkdir() and getModeFromOptions(). They make simple code harder to read.
Should be 1 tab instead of 4 spaces.
Indentation is wrong here. Follow PSR-2
Should maybe say "is not valid with the nodal solution method." here and above.
Is the difference between build and create whether the newly built object is saved?  Is the current object never saved even though the foreign key is updated?
This was testing fixture inheritance.
Der Kommentar ist schon ganz gut
typo: untill --> until
yes, gantt is still beta... there are probably more debug shortcuts should be removed
Why is the check mark here too -- since it is behind do we really need it?
/for you/d
Why is this change even required?
Per #798 error handling in template engine is a free for all, anything from passing through invalid tags to shooting the php runtime in the head.

For template events I'm thinking of dropping language variable consultation and just outputting hardcoded messages. Or maybe I will output a hardcoded message if there is no global $user and keep the language stuff if there is a $user.
See the previous pr.
A comment to what effect?
This should be camel case as per the coding guidelines (JS is slightly different to PHP, it was changed a few months ago)
I guess it's okay. `post_edit_count` should probably have an index.
Can we just cast to int here?
Don't do it. Pick better variable names. ;-)
I think its fine as it is.
Ok, this is basically what https://github.com/phpbb/phpbb3/pull/1132/files#r2426792 is asking.
Yep, I forget.
Under "header" I meant src/OpenGLWindow.h, not global.h.
I wanted to avoid the situation where you're sitting down to run the test, but have forgotten you have stuff in your Redis db. So rather than flush right away, I want to just make sure that it's empty before we do anything. It's actually saved my butt a couple of times
1) I know, but that doesn't mean that we should go on with that approach?
2) By "pseudo-hide" I mean code, that is not relevant for the normal use-case. (Couting FPS) Sth that could be moved in an isolated module in most cases. 
3) Afaik, no. In case of "hidden classes" a new class is created (that inherits from an existing hidden class) when ever an unknown property is added. In this case that would mean the "old" class w/o the "_isLoggingFps" is the new parent of the a newly created hidden class *with* the new inst-var "_isLoggingFps". From now on the new class is used.
You could move `this._frameTimes` in the constructor `this._frameTimes = [];` and you could get rid of the existence-check and assignment here.
Oh, okay. Is there a particular way I should install npm? Thanks.

On Mon, Oct 01, 2012 at 08:40:46AM -0700, yourcelf wrote:
> > @@ -28,9 +31,17 @@ System-specific instructions:
> >  
> >      sudo apt-get install poppler-utils pdftk imagemagick rubber rabbitmq-server python-dev postgresql-server-dev-all rubygems
> >  
> > +    git clone https://github.com/joyent/node.git
> > +    cd node
> > +    ./configure
> > +    make
> > +    sudo make install
> > +    curl http://npmjs.org/install.sh | sudo sh
> 
> Hey,
> 
> I'd like to avoid having the pattern "curl (non SSL URL) | sudo sh" in installation -- that's a huge security vulnerability, and basically giving full root access to anyone who can hijack the DNS or man-in-the-middle.
> 
> But even better: npm ships with node these days, so it doesn't need to be a separate install!  Please remove the curl npm bits here and below and I'll pull this.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/yourcelf/btb/pull/18/files#r1729342

-- 

M. C. McGrath (Shidash)
A person shielded by a true, benevolent passion is invincible!
Something is funny about your indentation here.  Tabs instead of spaces?
I think it would be cool if there would be a argument for specifying the repository URL from which
the package should be built. Could also be mandatory.

This would allow to build from any Github Fork.
I'm confused. Does this assume (or create) sorted columns at any point? I don't see that that's the case...
In my applications, speed of eigh is not critical, but I would love to see how G and g can me represented as theano graphs.  I have similar expressions elsewhere that can benefit from a speed-up.   Yes, I did try to avoid introducing an opaque Op, but my theano skills are not strong enough to figure that out.
``var_input`` is not used?
True, will fix. Thanks.
It is not intuitively obvious why source argument should (continue to) precede target argument when calling updateTuple and updateTupleWithSpecificIndexes, but target argument should (as of this change) precede source argument when calling updateTupleNonVirtual.

Does this seem LESS confusing than when arguments were passed along in the same order?
So the answer is no, because we use -isystem on the command line to tell the compiler to look for our boost first. 
The risk is that I could include a boost file that I have on my system, but not in /third_party/cpp, and it would build when it shouldn't.
The upside is that 1) boost is a library so it should be in brackets ("the right thing to do") and 2) the makefile won't list boost stuff as a dep.

So I'm going to leave it?
Why isn't this more specific Exception subclass? We have no way of taking logical action on the generic Exception, e.g. if this is used as part of a user interface.
That comma was good.
I'm not sure how much this matters, but for what it is worth, some platforms have this as ```char yes = "1"```
http://www.beej.us/guide/bgnet/output/html/multipage/intro.html#solaris
As well as Win 7.

Obviously not the main stream target, but worth noting. Usually I use some ifdefs for platforms, which sucks, but I haven't seen a better way.
It's addressed in the next pull request.
On Oct 3, 2012 9:12 PM, "Paul Wilson" <notifications@github.com> wrote:

> In src/Core/Utility/BuildingManager.cpp:
>
> > @@ -34,43 +34,43 @@
> >  void BuildingManager::doMakeBuildDecision(const Commodity& commodity,
> >                                            double unmet_demand) {
> >
>
> This whole method is commented out, but seems necessary for the tests.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/cyclus/cyclus/pull/388/files#r1757683>.
>
>
Yes, it's for backwards compatibility.  I'm not sure it's really necessary though since the load function seems to only be called by the Client constructor.  Is load meant to be an internal method and not a regular API method?

In practice, the registry is just passed in through the Client constructor.  Registry modification can happen through raven.base.Client class methods.
Right, we are not joining paths in this part of the code, so we can let native versions and tests still pass. I will remove the upath call here.
Probably should escape `v` at the very least, maybe `k` as well.
Cpt obvious strikes again :D
Question: why not just set it to nothing?  Is the empty string significant here?
are you testing for just the ampersand character itself or an empty escape character?
And what about property "skip" and projects with packaging "pom" ?
or maybe
<pre>
.animate({
    opacity: to 
}, speed, easing, callback );
</pre>
What is the right way?
Why did you remove the whitespace?
Yes, I've been looking for a good way to register C++ objects in fn_extra, but so far the only way to do it is to remember objects somewhere in the heap and hook transaction callback; ugly.
No need for else. In the "else" case some required variables will not be set, and cmake will give an appropriate error message. That's handled in find_package_handle_standard_args() below.
We should remove this line, since the CART doesn't depends on svec and array_ops any more.
Before this, I would check for pString.get() being NULL, or pString.getType() == jstNULL (from bson/bsontypes.h).  If either of those is true, return Value::getZero().  That will handle returning zero for a non-existent or null valued field.
https://github.com/Lawouach/WebSocket-for-Python/blob/master/ws4py/server/cherrypyserver.py
This is confusing and you're right--my text sucked. Maybe change it to "look elsewhere"?
FWIW `HOMEBREW_MAKE_JOBS` is a really useful debugging tool for stuff that fails due to parallelization issues. (So thanks for restoring and I'll look at important stuff. ;) )
what's with this lastProjectDir thingy?
We are very, very close to not needing rhc-check anymore.  Don't spend too much effort on it.
Display methods should not perform rest calls.  Make the calls in the command, pass the required values into the display_XX methods.
Would it be cleaner to just start having a parse token for non-empty strings? I suspect we'll want one anyway.
Does this create a single item or multiple menu items? If the former, it should be called `createMenuItemFromJSON()`. If the latter, it should take an array, not an object (and probably be called `createMenuItemsFromJSON()` since it doesn't actually create the Menu itself, unless I'm misunderstanding).
Copy-paste error: the key should be "CMD_SELECT_LINE"
ContextMenu is the only one of these that doesn't call its super constructor.  But it easily could, since all Menu does is initialize `this.id`.  Should we do that?
On first read, this function name sounds to me like it'd retrieve the version of the _currently running_ copy of Brackets.  Maybe something like getUpdateInformation() or getAllVersions() would be more clear?
It's not needed to make Brackets run, but it is needed to avoid an error message on the console. Would you rather have it removed?
It would be nice to have a shim for requestAnimationFrame() so it works across browsers. For example:
http://paulirish.com/2011/requestanimationframe-for-smart-animating/
Do we still want to keep those modes that have mime type like text/x-xxx to be X-xxx? eg. text/x-c++src or  x-java or x-csharp
If the registration server crashes, we need the dispatch table to live on.
thought about refactoring some of this quasi-copy/pasted bit from the report routing over to here but that function also seems to want access to "model_path" and it looked like it would either create a weird tuple-returning API or force deconstructing the name back out from the report so I decided to let it lie.
Fixed for good with commit bcec4f876361cd7c5c1468bd249a409bfac8e2e4 in iMedia 3.0.

Not sure whether commit is applicable to 2.5 without adaption (e.g. parserIdentifier not present in 2.5)

On 04.10.2012, at 10:36, Mike Abdullah <notifications@github.com> wrote:

> In IMBiPhotoParser.m:
> 
> > @@ -1031,7 +1031,6 @@ - (void) populateEventsNode:(IMBNode*)inNode withEvents:(NSArray*)inEvents image
> >                  }
> >                  else
> >                  {
> > -                    #warning @Jörg This is a preliminary "fix" for the Events bug with iPhoto 9.4
> 
> @jjac, what's going on with this at present?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
You'll want to make this change in src/raphael/raphsel-set.  This file and r2d3.v2.min.js are generated files :)
Very good observation. Later on we will combine this into one URL. But for now change the template so that Facebook logout URL is displayed if the user is logged in through Facebook, no?
Noooo no no no no http://www.youtube.com/watch?v=0O_IHOwgW0w
We don't seem to support this in the UI, should we remove?
Can we get rid of this padding? Not necessary, and makes gmaps get cut off...
use `/^data:image/.test( popcornOptions[ item ] )` to check for the string
確かに理由があったほうがいいですね。
理由を追加して修正しました。
> I'm new to redis, but as far as I know that's the only way to ensure that the "set" call only succeeds if the key has not been modified since the "get" call.

For the example you provided I think this used to be the only way to achieve that, but with the introduction of scripting you could do that with the EVAL command. So you might want to look into that if this is something you do allot. So not a solution the issue, but it might help your programming problem.

I'm however still hesitant about the semantics of this code. So I understand your problem, but I don't like the solution. I don't think there is anything wrong with the code. I just think it's messy to put a SyncCall on a AsyncClient. And if I pull this I know you depend on it, and it would be harder to remove, than to add at a later stage. I really appreciate the effort you put into the issue. I suggest you add a helper function until I figure out a good way to do this, or just maintain the small patch.
Whoops, would have been best in its own branch. That's the js error dialog popup you can get with `index.html?debug`, enabling it did not work.
在freebsd下，你不需要使用这个选项，因为libc里面已经是jemalloc了，所以没有影响的，不需要加判断。
No.  :(
There seems to be a small typo here.
I've removed the TTL, prio is still in the where. It now looks like:

$ ack-grep update-query
pdns/backends/gsql/gsqlbackend.cc
283:  d_UpdateContentQuery=getArg("update-query");

modules/gsqlite3backend/gsqlite3backend.cc
106:    declare( suffix, "update-query", "Update query for DynDNS", "update records set content='%s', ttl=%d where name='%s' and type='%s' and domain_id=%d and prio=%d");

modules/gpgsqlbackend/gpgsqlbackend.cc
97:    declare(suffix,"update-query", "Update records query for dyndns", "update records set content=E'%s', ttl=%d where name=E'%s' and type=E'%s' and domain_id=%d and prio=%d");

modules/gmysqlbackend/gmysqlbackend.cc
99:    declare(suffix,"update-query", "Query that updates record", "update records set content='%s', ttl=%d where name='%s' and type='%s' and domain_id=%d and prio=%d");
Updating this to:

add_to_groups
remove_from_groups
It is probably not the best; however, we've had to do it in certain places for helptips to have them render better.
Yes, this is not being pulled into master, just the pulpv2 branch
But might cause false expectations
is printing to the error log here for debug or worth keeping in?
Also, what about `int` (`long`?) overflows?  It looks like `Number` maps to a `double`; so, maybe I should be using `qint64` (`qlonglong`), instead?
`set_temp` only works in a loop, which is not very convenient for my tests. But I added codes to reset to default values in `teardown_class`. Thanks for pointing that out.
It might be clearer to use the symbolic names for the levels, rather than numbers. i.e., rather than 10, use `logging.DEBUG`... etc.
从老的系统写
What is the format of strDate?  And shouldn't there be a check here to make sure strDate is reasonably close to the current time, to prevent replay attacks?
This was causing a problem I think, the end block was not looking random with this bit removed. https://github.com/cakephp/cakephp/pull/829
I see. And I now realize this is related to your changes to the WFSGetFeature and WMSGetFeatureInfo plugins.

We should make sure these changes do have negative impacts on the FeaturesWindow plugin, which also listens for querystarts and queryresults.

I think the querystarts queryresults sequence logic should be unit-tested. I'm volunteering to take a look at it.
Format this css properly.
TODO: /* WORD token may consist of any characters other than special characters */
Just deleted it instead.
We need to make this use LibGit2Sharp if possible.
I don't think we should introduce or expose a type like this.
Nothing to `stdout` please. Either fail, or ignore the warning if it's not critical.
We *need* to set an error message here. Cannot return a negative value without error message.
`Number.NaN` to `0/0`
Hm, wouldn't it be easier to have a do-while loop here?
thanks!  great tip. fixed.
are we using 'get' here just to suppress the KeyError if the key is missing?  If those keys are missing, is it really not a fatal error?  This question may be more for adrian than for peterbe.
Sure. Again, it's equally arbitrary, but there's a bit more of a hint. I am curious (outside of this bug), will it be possible for a user to have the same log in from BrowserID, BlueVia+BrowserID, Foo+BrowserID, etc? How will the user know which account to use?
Is there ever a case where email wouldn't be set?
Re @tillmo: OK, thanks! @tillmo, @okutz: I think there is no way in OWL, i.e. SROIQ, to express this, is there? If there were, I'd like to make it explicit. But I think it is rather only possible to forbid (not in the sense of constraint violation, but rather inconsistency) certain combinations, e.g. via disjointness axioms.
такі речі винось в конфіг
так. стоп. я зрозумів, що в коммітах робить файл з логотипом конфи :)

тобі не треба накладати це все на кратинку. достатньо вивести на чистій html сторінці зображення коду і ФІО участника + номер квитка
shouldn't you remove the header after that ?
We do this in other places as well (the firewall for instance), and using a separate listener would complicate things here
The interface makes other implementations possible, so that's really part of the `EventDispatcher`.
You should add an empty line before return.
This isn't bc break, or?
`setStrictParameters` is not part of the UrlGeneratorInterface that is returned by getGenerator according to the phpdoc.
So it cannot be called safely.
the `has()` check is not needed and can be safely removed.
you should set the default values on the property declarations instead
Actually, I take that back. Is there ever an instance of just `Post`, or does everything inherit from it? In that case, yes, abstract class is the way to go.
Do we need all those repositories here?
I created another ticket about adding Archive size (#2976) and moved it there
Why a title="" here ? 
If it's necessary for the tooltip, why isn't it on every other tooltip  ?If it's not mandatory, you can just remove it
brainfart.
removing `--show-reachable` should clean up the output
not sure what "and like" is supposed to me. Comment in braces is unnecessary, could simply be "results in some performance gains"
this is wrong I guess, it should be `build-model`, right?
I'd say it looks better without this border
You're right. Hadn't thought of that.
Could later be enhanced with ``$strict = false`` in constructor (<, <=)
Il te manque deux clef que tu as en Français
I think we should just fix SimpleOperationDefiition#getDescriptionProvider

```java
if (entryType == EntryType.PRIVATE){
return new DescriptionProvider() {
       @Override
       public ModelNode getModelDescription(Locale locale) {
            return new ModelNode();
        }
}else{
     return new DefaultOperationDescriptionProvider(....)
}
```
Agreen, switch to `null`.
While this passes, the vash.config.htmlEscape functionality appears to be completely ignored.
Sorry for adding something not ready for the localization.
I will be more careful on this in the following pull requests.

Besides, thanks for the following clean up on l10n. :)
Same comments as the previous file for those scripts.
mmhh.. wait we don't need to handle them for the background services just the window manager.
if (cmd) should be enough
I am talking about `- 10`
Can you define as:

````js

var options = {
  myvar: x,
  ...
};

```
Localization for different languages than en-US, ar, fr-FR, zh-TW does not live anymore in the main repo. You can remove this.
The fast_gaia path compares local stuff in $gaia/profile with /data/local on device. Now we have some data that is in /system/b2g and some in /data/local
Please don’t include this file in this PR, as it belongs to another component.
See the policy about name I mentionned earlier.
kill it it should not be there. At one point I had one locally I must have committed it by mistake.
properties separate keys and values with \s*[:=]\s*
Please add white space before and after "+", then I think we could merge this change.
ok but Etienne said me: "we can live without comments in another pr"
try not to reuse the variable to confuse yourself

    var textContent = conversation.body.split('\n')[0];
    var htmlContent;

   if (...) {
     htmlContent = ...
   } else {
     htmlContent = ...
   }
We really need to show a UI here to customize the name of the bookmark. Can you open a new issue and reference it here for that?
return了，clear无法被执行啊。
Yes that's the convention I'm following
I think this should be added back so that you can see somehow what proc types an app has
This is exactly the thing that regexes are designed for. They're extremely common, so just learn how to do it - it's not that complicated.
Si, creo que sí
Agreed, i kinda just had to stick it in somewhere
This could be a little neater using each_with_object - http://ruby-doc.org/core-1.9.3/Enumerable.html#method-i-each_with_object
There are some rogue tabs in here (and elsewhere)
Can you move all the open braces on new lines to the line above them? (that's our style for the project)
It's harder than it sounds, particularly with all the madness we allow between `(` and `)`, to write a regular expression for this.  And Python doesn't make it very easy, either:
<pre>
>>> "ab%(c)! d" % {}
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'c'
>>> "ab%(c)! d" % dict(c='x')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: unsupported format character '!' (0x21) at index 6
</pre>

That said, I bet a fake interpolation against either a sequence of the right length, or a defaultdict, could do the trick.
The first of those is in `master/buildbot/util/maildir.py`, so won't be imported in this particular case.  The second is in `__init__.py` could be an issue - at the least a spurious warning appears.  I don't think we really need to be using a warning for this, so I'd be happy to just remove it.
Do we care if this is set to something stupid (ie, not a function)?

gst_exports.hasOwnProperty potentially differs from Object.prototype.hasOwnProperty- also, we maybe want to go up the prototype chain here?
I take it this is the typo? :)
Did you measure the increase of PROGMEM for the bifield handling of convert_delay?
Just being paranoid for possibility of DDOS. Probably not needed. ;-)
В чем смысл json: {html: html} ? Лучше уж просто render text: html
Couldn't this be achieved with helpers?
How GIT_WORK_TREE helps for the Language detection? I do not see the relation. Can you explain?
I asked Jeeves, he says that might not be completely safe

http://stackoverflow.com/questions/4456231/retrieving-html-attribute-values-the-dom-0-way#answer-4456317
You're right. Forgot to IP_GET_RAW_VER was not checking the protocol inside the IPv6 header.
As far as I can tell, this is just swapping pointers. These are two separate, running, instances of a Phantom Webpage. All Casper does is maintain an internal "then" stack, wrapping access to the phantom api (`this.page`) in some higher-level primitives.

Generally, this is a hack. What I'm saying in this code is that all that casper is good for is maintaining an internal step stack, and as I need to model complex behavior, use casper like a GIL and swap "page" contexts at each step.  I'm less pleased with this code the more I think about it...
You are already freezing Function.prototype. So that this additional lambda feature looks useless, isn't?
Mhm, I strongly suggest to don't call a function with the same name of a property, and just with the "private prefix" on top, it's totally misleading. Also, we should avoid to add property / method to the object (`PageMod`'s instance in this case) if it's not necessary.

If you want to avoid to repeat all `this.attachTo.IndexOf(...) === -1` stuff, you could use `has` function from [util/array](https://github.com/mozilla/addon-sdk/blob/master/lib/sdk/util/array.js#L21):

```js
    if (!has(this.attachTo, "top") && !has(this.attachTo, "frame"))
```
Array.may -> Array.map
This is nice but not quite what I meant, sorry I was not very clear about it. Can you instead of using `sp.prefs` a non-unloaded one (`require('simple-prefs').prefs`) to make sure that an actual pref change does not call's listeners on the unloaded prefs module.
Above, you talk about still using "some_name_i_chose", these should remain consistent.
I would prefer if we could do that with Cu.import().
I using localhost since I want to test on my server, forgot to change it to
HTTP instead.
And the problem with cannot using the ../../, i think on faq part. so have
to avoid using the ../../

On Mon, Jul 9, 2012 at 11:13 PM, Hilary Ho <
reply@reply.github.com
> wrote:

> > @@ -22,7 +53,7 @@
> >                       </div>  <!--end div search-->
> >
> >                       <div id="logo">
> > -                             <a href="../../CoinCod"></a>
> > +                             <a href="http://localhost/CoinCod"></a>
>
> why do you put in localhost?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nvcHilary/CoinCod/pull/2/files#r1122923
>



-- 
Regards,
vincent
So all of these `test` changes should be reverted please.
This is for compensate an extra whitespace at the left of the sentence
space between foreach and (
Before you comment on this, this it the correct author, the function was missing the doc block so i added it
lowercase letters are better :)
I think @stof mean something like:

```
$nbRows = $em->getConnection()->executeUpdate($query, array($minDiff, $minThreshold));
Could we put this in a separate file?
Use a constant for the Enter key:

```js
var ENTER_KEY = 13;

...

if ( e.which === ENTER_KEY && $(this).val() ) {
```
Also, could be nice to have a kind of output for the user; so logging this would be cool.
why would it return false? IIRC, the syntax is `.get(key, fallback_value)` on a dict; isn't it?
If you change "riak_etop" to say "np_etop" or something generic I'm +1
what's up with the empty lines here?
this should be done in ``setUp`` IMO, not by overwriting the constructor
Missing spaces around `!` sign.
you should only change the ``$className`` variable when the rewritten class exists, or you should throw a special exception here mentioning both possible classes (which would be even better). Currently, only the rewritten class will appear in the error message 4 lines later
To make our CS happy, you should always comment *before* the code, like in:
```
//@todo Inject relations or change spec to only generate link, old: $content->getRelations(),
array(),
```
I've done this because it was already the case with ```ezpublish.api.storage_engine.legacy.transformation_parser.class: eZ\Publish\Core\Persistence\Legacy\Content\Search\TransformationProcessor\DefinitionBased\Parser``` which inject the ```kernel.root_dir``` as well.
See: https://github.com/patrickallaert/ezp-next/blob/4434846675b861ec550748f6dc17ecfdcad3e2e3/eZ/Bundle/EzPublishCoreBundle/Resources/config/storage_engines.yml#L111

Not that I am against, but if this has to be done, then it should probably be the same for https://github.com/patrickallaert/ezp-next/blob/4434846675b861ec550748f6dc17ecfdcad3e2e3/eZ/Bundle/EzPublishCoreBundle/Resources/config/storage_engines.yml#L22

But I guess @andrerom explained the reasons behind that.
we can actually ignore all the .egg I think, not only six.
Sorry, I'm not sure to understand correctly: should we use ``<link rel="stylesheet" href="{{ ASSET_URL }}">`` or ``<link rel="stylesheet" href="{{ SITEURL }}/{{ ASSET_URL }}">`` in our jinja templates?
If the latter, then we should mention this also in the plugin documentation.
"GCC" can be replaced by toolkit.GCC when pull request #38 is merged.
Why the \ at the end?
you're right, it works. I'm updated the links
I created a new account on a Fedora box and I'm running a test build without the JBoss repos now.
Now that it is it's own class it should not be static anymore
I have no idea.  It works.  I don't know when you need a semicolon and when you don't.
-----Original Message-----
From: knipster <notifications@github.com>
Date: Fri, 17 Aug 2012 15:12:51 
To: knipster/update_ip<update_ip@noreply.github.com>
Reply-To: knipster/update_ip <reply@reply.github.com>
Cc: jlehrer<github@lehrerfamily.com>
Subject: Re: [update_ip] adding a logfile (#1)

>      echo  "IPs Match" > $EXTRA_OUTPUT ; 
>  else 
>      echo "IPs do not match" >$EXTRA_OUTPUT ; 
> +    echo "`date` $DNS_IP_VAR -> $ROUTER_IP_VAR" >> `dirname $0`/update_ip.log

Does this line need a semicolon? 

---
Reply to this email directly or view it on GitHub:
https://github.com/knipster/update_ip/pull/1/files#r1407521
This should be attached via the API `on` so it can be managed by Popcorn
Wouldn't we want title="additional tags" here too, for consistency? (Maybe we don't, I'm not 100%, but it seems like the only place we don't, now.)

Otherwise, looks good :)
`td, th {...}`としてpaddingとborderあたりを纏めるとか、mixinを作るとかしてDRYにして欲しいです。
あるいはCSSフレームワークを入れて楽するか。http://imperavi.com/kube/ とか軽いし1ファイルだしでお薦めできます。

でも別に今回頑張るべきところでは無いので、このままでいいと思います。
Will this logic make "StandardFields" become "StandardFieldss"?
You are fast. Fixed.
Must be something wrong with the wait above this if this line is required. This line is waiting for one image whereas the throbber should be there until at least one image is present but preferably all of them loaded.
no, Author text is very complex!
doesn't work for me... have to change this to addClass(i + ''). Or we should change bonzo to handle passing 0 in. Or i'm a n00b and missing somethign obvious.
I would split this function into two functions. One functions that calculates that path. This could be done in the constructor and then the lockPath could be final. The other part of the function is to create the path. This can be called on lock. As it is, creating this path can happen a zillion places which makes the code hard to reason about.
Should this be "username" to match the field label?
https://github.com/SpoutDev/Spoutcraft/blob/master/src/main/java/org/spoutcraft/client/packet/PacketGenericItem.java

Also see the rest of the PacketGeneric* in that package.
do you really need to use reflection here ? It is clearly a hack in this place.
Added observable to control whether or not a column is able to be sorted on.
Should be constants, at least to add some documentation
remove the spaces
Same here. We could just use `IO.readfile`
This part doesn't make sense.
Is this tested? `store_pane`.

Also, not every image is meant to be a block-level element.
Yes. remove_index should be before rename_column.

When you change something in particular order,
you roll back your changes in reversed order.

If you encapsulate instructions in "change" method -- you don't need provide up & down methods.
Rails usually can handle it implicitly, but sometimes it may be impossible and up & down methods has to be written explicitly.
Standard double-== is sufficient for string comparison
You are right. Done
Change to existing function.
No need for clang::, I guess.
That line was removed because couple lines above, there is a line identical for shape. In other words, this line was a repeat.
The bookmarklet plugin needs this return value
The `<condition> OR <expression>` syntax isn't currently in the style guide.

Neither of the syntaxes is wrong, because the first one is recommented by the style guide and the second one isn't mentioned at all. I've tried to start a discussion on this with the Reactor team and EllisLab and we were kind of split on this one. There are however cases like this, where IMO what I'm proposing is way more appropriate.
"indeterminate", not "undetermined"
Вы имеете в виду что доступен NSMutableArray со всеми своими методами?
в Set невозможно 2 элемента одинаковых положить:
The NSMutableIndexSet class represents a mutable collection of unique unsigned integers (https://developer.apple.com/library/mac/documentation/Cocoa/Reference/Foundation/Classes/NSMutableIndexSet_Class/)
Это не ошибка, просто поясню, что произойдёт в этом случае - после выполнения всех действий получится число с плавающей точкой, но так как это число будет присваиваться переменной типа int, дробная часть будет игнорироваться, будет использоваться целая часть. Так, например, при значении x=39.84 значение x будет 39. Если хотите использовать округление, то воспользуйтесь соответствующей функцией.
Should this be s/provide/prevent
Yeah, that was before `RACSubscribable` was a concrete protocol. So `id<RACSubscribable>` only had `-subscribe:`.
If this if statement is not traversed it likely means the user hit back and does not want to login with openid. In that case nothing should happen.
ディレクトリ名はbodyかと思います!
      text_path = File.join('body', File.basename(path, '.yml') + '.txt')
for correctness, this should be inside 'if (!closed)' check.
@dahlbyk encountered some issues with `CustomMarshaler`s and callbacks with Posh-Git. #172 *seem* to have made those disappear.

Although, we're not 100% sure this was the root cause of the crash, I'd prefer all the callbacks to be non decorated (at least for now). Could you please take care of fixing this?
Oh, it was made by pull rq #91. Perhaps I should merge the master branch of node-irc to my fork...
I think we can still add `return_generator` but it can't be True if `n_jobs > 1`. But I wondered as well if parallel processing here is necessary. For me it seems very fast to apply inverse to epochs.
rename ret -> fig, so it is consistent
oops should not


On Sun, Dec 16, 2012 at 10:19 PM, Alexandre Gramfort <
notifications@github.com> wrote:

> In mne/viz.py:
>
> > @@ -1109,6 +1115,10 @@ def plot_image_epochs(epochs, picks, sigma=0.3, vmin=None,
> >          (data.shape[1] == len(times))
> >      show : bool
> >          Show or not the figure at the end
> > +    scalings : dict
> > +        The scalings of the channel types to be applied for plotting.
> > +    titles : dict
> > +        The titles associated with the channels.
>
> titles is still there ...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/221/files#r2430657>.
>
>
On Thu, Apr 12, 2012 at 4:57 PM, Pascal Voitot Dev <
pascal.voitot.dev@gmail.com> wrote:

>
>
> On Thu, Apr 12, 2012 at 4:51 PM, Julien Richard-Foy <
> reply@reply.github.com
> > wrote:
>
>> > +   *
>> > +   * JsArray(JsString("value"), JsNumber(123), JsBoolean(true))
>> > +   * ====> Json.arr( "value", 123, true )
>> > +   *
>> > +   * There is an implicit conversion from any Type to JsValueWrapper
>> which is an empty trait that
>> > +   * shouldn't end into unexpected implicit conversions
>> > +   *
>> > +   */
>> > +  trait JsValueWrapper
>> > +
>> > +  private case class JsValueWrapperImpl(field: JsValue) extends
>> JsValueWrapper
>> > +
>> > +  implicit def toJsFieldJsValueWrapper[T](field: T)(implicit
>> w:Writes[T]): JsValueWrapper = JsValueWrapperImpl(w.writes(field))
>> > +
>> > +  def obj(fields: (String, JsValueWrapper) *): JsObject = JsObject(
>> fields.map( f => (f._1, f._2.asInstanceOf[JsValueWrapperImpl].field)))
>> > +  def arr(fields: JsValueWrapper *): JsArray =
>> JsArray(fields.map(_.asInstanceOf[JsValueWrapperImpl].field))
>>
>> I suggest you to remove `trait JsValueWrapper` and then to rename
>> `JsValueWrapperImpl` to `JsValueWrapper`, so youll be able to get rid of
>> those `asInstanceOf[JsValueWrapper]`
>>
>>
> No this is the main idea of this implementation actually :D (discussed
> several times with Sadek)
> We want to have an implicit conversion to trait JsValueWrapper which can't
> do anything, has no method. In this way, we don't fear the compiler will
> resolve an unwanted conversion.
>
>

Don't worry, this is a Sadek's idea: my Scala is not good enough to imagine
such trick ;)


>  ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/playframework/Play20/pull/231/files#r676609
>>
>
>