labofsahil/github-event-dataset-2012 huggingface dataset:
I don't see how that would be possible. I don't think it would be sensible for Eclipse to read config out of an ant build file. 

I think the right solution would be for EclEmma to support per project configuration of these settings. That would allow the settings to be checked in with a project and then all coverage reports run in that project would use those settings. 

However, it should also be easy to override the settings so that a developer can quickly view coverage with/without source directives from within Eclipse.
`rposition = new RegExp("^("+jQuery.cssExpand.join("|")+")$", "i"),` is only one byte bigger!   :trollface:
To make it more obvious when it fails. The right value is "041275" (which is added by `request()`). When I accidentally made it fail, I wasn't sure whether quotes were being added to the "right" value, or whether it was a different value.

Why the quotes are there to begin with is a mystery to me, no idea what the point of this assertion is.
This still won't work. Consider if you set `method` on `ajaxSettings` and `type` on the options given to `ajax`. This will always use the value of `method`in `ajaxSettings`. That's why I still think it would be better to handle this in `ajaxSetup`: the logic is not that simple.
Nope, should be empty string. Fixed
This should not be a macro but a proper function.
Probably the right thing to do is to raise an error if `df <= 0`. Also, we should make `df` an optional argument (which currently means defining an overloaded UDA).
Why doesn't this work?  It's just doing some pretty standard fork/exec stuff...

On Windows we just don't compile this file at all.  Perhaps we can add your platform to that list?  Search for 'browse' (with the quotes) in configure.py to see where we add it, maybe we can just adjust the 'if' statement above it.
I don't know what the right answer is, I'm just trying to reason about this.

It sounds to me, at the moment at least, that if we were to allow a srp user to run actions against their twitter account then we might as well completely auth them and update their user to contain the new twitter service information (we don't have a UI for this at the moment but at some point we should.)

I guess basically all I'm saying is -- why not use the accounts system? Is there any service that would want to allow a user to auth via Facebook and not store that data on the user object?
Fixed. GitHub didn't seem to mark this comment as outdated, since I only added two lines after this diff.
Why are we parsing `rule.cssText` ourselves instead of using cssutils` functionality for parsing a rule into individual properties? I'm concerned that this implementation is probably naive about some edge cases regarding comments or whatnot and will break with some cross-browser hacks people are probably still using in their CSS.
No sudo; indentation is 2sp.
You can get this programmatically:

```rb
ENV['SMLNJ_HOME'] = Formula.factory("smlnj").libexec
```
You can drop the brackets
We prefer sha1 now, but basically I am just testing comments on diffs for multiple commits.
There needs to be a better way to extract this - this is not a good mechanism.  URL parse is better.
The idea is that modulo bugs, this executor is only used in the lexical scope of the Future class, it's never used for the app's callbacks.

By "app" I just mean "code outside the Future implementation"

Another way to look at it is that _if_ the app has any callbacks on the Future, there is always an app executor "after" this internal one (associated with the app callback). You can see later in the code that in methods that take an implicit executor from the app, that executor is manually used rather than this internal one.

So this internal EC should be an undetectable implementation detail. I tried a different way to do it (adding virtual methods to use rather than the existing onComplete) but I think this way is less code and functionally equivalent.

It also adds the elegance that the patch uses its own new feature (lexically-scoped custom executors) to implement itself - kind of fun!
einfach statt der ternaries einfach 

```
$this->getConfig('firephp_maxdepth', 5)
```
At a glance, this line seems > 80 chars?
Changing bashrc should an option, otherwise you work around stuff people may have changed otherwise already.
Quite; they end up being iterated over, decorated and put in an ArrayList eventually.
Is it intentional that we don't set `pi.auth.profile` as we usually do?
I think the intent here would be clearer if we wrote "But we want to..." (because we're specifically _avoiding_ the tinycolor behavior).
``$("#main-toolbar li#" + this.id + " > ul.dropdown-menu"`` overqualified? Does ``$("#" + this.id + " > ul.dropdown-menu"`` work?
For this raw data export, it may be more helpful in the interim to return all results instead of computing the average. Someone else might want to see the raw data points and get min/max values too. Just something to consider.
On my machine, this makes the build number text only slightly lighter than the rest of the text. I prefer it with the lighter color.
Does it make sense to factor these changes out of the in-browser branch?
Super minor nit: some of the colon spacing is inconsistent.
charCoords calls clipPos(), so you don't need to get the line length here. If length is undefined, just pass in Number.MAX_VALUE.
I'd still put the keybindings in ext/language, and then the language specific stuff in ext/jslanguage.
What is @instances used for?
Goodbye for now, friend.
Fix the trailing comma -- this JSON won't parse
Indeed. And I especially don't like the increment there. Pfiou... at least git tells me I am not to blame for this :-)

I'll clean this up.
Please indent this one a bit better, it's hard to read : I thought there were three different calls, but there"s only one.
`
        widget=widgets.SelectDateWidget(
            years=[
                y for y in range(datetime.datetime.today().year,
                (datetime.datetime.today() - datetime.timedelta(models.LOWER_DATE_LIMIT)).year,
                -1),
            ]
        ),
`
This is too much right?
butter.page isn't already a module?
Why throw?  Why not just return _ghost?
No. These just simple toggle what elements are enabled and then toggle event listeners on the text boxes themselves.
Mind making this a properly-cased sentence to fit with the rest of the documentation we're starting to have in Butter?
Why not just leave this as it was and extend the if/else block here https://github.com/mozilla/butter/pull/1363/files#L0L168 ?
Space before the parenthesis `if (!styleSheet) {` - That's why the linting failed :)
It should not but i will have to verify.
Shouldn't this be to the `enabled` directory here?
For some of this stuff it might make sense to depend on zipkin-common and use the Span object in there. That already has code for extracting service name for example. We can leave that for a follow up branch though.
Yek header kolli bede
hich css ham nadashte bashe
dakhelesh ye section.banner ye ul.menu va ye figure.logo dakhelesh besaz
Yeah, makes sense, I guess I'd just have that be the default but make it configurable. What if somebody wanted it to be `{{series_name}} {{series_id}}` for example?
( should be _(
Declare at the top of the function since there is no block scoping.
Why do we do this? I know using Exception will catch some problems that are not coutched  wthh '=> e' (it handles errors, but not exceptions, https://gist.github.com/948354). Is that what we want to do?
Hm, can we catch more specific exception here?

In any case we should at least include the original exception message in the custom exception we re-throw.
Done in 6ba2b43.  This is nice,  first time I wrote an exception class that does more than `pass`.
also fixed above
I think this old position was more useful; after all, we'd like to know when the IRCSeed thread starts to do actual work (connecting to IRC), not so much the pre-flight checks.
ah, ok. separate commit, but same pull, do you mean?
This strikes me as flawed by design, though perhaps that's inherited from boost... does it not have any kind of "is loopback" method?
Even IPv4 has 16,777,216 unique loopback addresses.
These can go, too :) No need for dummy unused static declarations.
Damn I thought I got this one...
In mijn hoofd is `($this->record['is_featured'] === 'Y' ) ? true : false` leesbaarder, en aangezien true en false booleans zijn en een conditie altijd ne boolean is is de laatste stuk nutteloos ;-)
Very nice. Yay LibGit2Sharp!
To aid user support in the future, please add something like:
} else {
  NSLog(@"Clipboard access denied for CopyToClipboard");
}
I changed this to integer ms to be comptible with `HeartMonitor.period` in the Controller config.
```
options = { escape_html: true, auto_links: true }
BlueCloth.new(text, options).to_html.html_safe
```

の方がよいですね。
I don't believe it was intentional to treat the -1 case as a "pass" (but not zero) but I could be wrong.
I'd commented it out anyway, so obviously not required - deleted.
@tanoku @benstraub `git_object__dup` added and deployed.
I have added a button. Currently it just launches KeePass. I need to add a test to to login if KeePass is started already.
I can't remember if the capitalization on the `L10N` part is vital or not, as I usually see `L10n` used.
"guage" should be "gauge" but I'll just fix that in the merge. r+ otherwise.
no idea, that's what was there?
@johnzachary pointed out that `_local` actually assumes that the input is UTC, and renders it using the local time zone.  So the blurb in the diff is right, and my mod is wrong.  Can you update the documentation of the existing `cork_timestamp_format_local` to have the correct text?
AAAAAAAAAAAHH!!
@igorw what do you think about this ?
Why are you talking about sockets here? I don't get it. But anyways, the old method is safe as this is just a shortcut for `setTrustedProxies(array($request->server->get('REMOTE_ADDR'))`.
the way you handle priorities is inconsistent with the other places in Symfony. You force using unique priority, which would require knowing all services being tagged to be sure it is not used twice (as one of them would be lost here), which totally defeats the benefit of tags (not having to know all services using the tag)
you should not use ``we`` in the phpdoc (who is it in the code ?)
Selecting by primary key (token) should be faster than any count...
Wrong condition. Doesn't work when $index is negative (e.g. -1).
Why not simply `isset($this->elements[$index])`?
You wander from the subject. I fully understand your use case and I think it's a good feature. But IMO it should be implemented differently. That's what I'm saying the whole time.
> you cannot say that we don't have complicated plural messages in the core validators, as any message needing plural is complicated in some languages

`DefaultTranslator` does not support translation :) Just interpolation, i.e. parsing of parameters into messages. It does not have to care about what translations look like.

> So, if I get you correctly, we are reinventing a way to "fake" plural messages?

To make a long answer short, yes. People *can* use intervals in their constraint classes, but then these constraints are bound to the Translation component and not compatible for example with Drupal's translation mechanism.
No idea. I should probably ask on the chat, or something...
Looks like both test descriptions need to be updated to reflect new method names.
Style: indentation two spaces left.
so you want to chomp string? I think you can do it more elegant this way:
```
output  = regexpsub( output, "^(.*)\n$","\\1" ) || output
```
well, I don't like much missing chomp command and behavior of regexpsub, but it is as least more obvious what happen. another option is using substring to check if last char is "\n"
这一段太细节了，而且说实话我觉得一般的需求很少会遇到……
太多的文字描述会让大家觉得繁琐
不如就推荐大家看lshort的公式相关章节来的直率
也没必要重复造这个轮子
哈哈哈哈~~~
Did settimeout not work for you here?
Do you need to normalize here? I think you normalize again later anyway.
This breaks the wiki link. To include "here", do this: [[Bitcask here|Bitcask]]
I can see the difference between the later files and earlier files
Why would mgmtClient be null? If it's ever null, sounds like something is wrong with whatever code passed that. And then that test will fail when we run it in an IPv6 environment.

AIUI any hardcoding of "127.0.0.1" or "localhost" is forbidden in the testsuite/ modules.
@basiclines rulez!!! :-p
I believe that crash dialog would regress by this statement. Because it is designed to have a visible class.
Modify crash dialog style or remove this too-general style.

c.c. @leibovic 

I agree that we should put all XXXX-dialog in the same bucket, but you know that.
Ups, missed that debug line. Thanks!
we're going to get rid of Zepto and move to native methods, but we'll leave it to a separate refactor pull request
Strange to have MAY 2012 hardcoded here
We could use and `h2` here :D
Yes, the STK spec defines that this text shall be showed to the user and wait until it accepts it ... so I think an alert is ok for this. It isn't a debug line ;)

I implemented this other method in order to be able to work async. since in some case the alert box should be closed after a defined time, so this new solution didn't block the JS execution while it works fine ;)
As far as my understanding we do have a transition on "call-screen" (which is an <article> at the same level as the "views" <article>. As a consequence, we have to explicitly hide the keypad.
umm I'm only used it because I though that was faster. I don't have problem using this another approach and you are right what this alternative is more extensive. For example, if in the future we will add more classes to these DOM elements, implementing classList.contains everything will work fine and with className  === the algorithm will be broken :)
this.view?
right, and that's what I did throughout in this pr (go to the lowest common level on all common flow paths); however, this is the win flow path, and railsinstaller 2.1.0 (current) installs rails 3.2.1
Put your email in here.
NO MEANS NO CHRIS!
Close f after you're done reading with it, or justify why you need it open.
I'd vote to keep the module, possibly remove the code, and insert what you just described as a comment. That's so people don't wonder why we aren't testing the Presentation class. Let us know if you have any other suggestions like this, you're currently our test expert.
I was trying to stop Freeseer's verbose output during the test suite execution (detailed in my final blog). I thought I removed them all. I'll get rid of this one too as it isn't being used.
Why does the previous channel need to be saved? Surely that's extending by plugin again?
yea just a consistency. I think, for example, it’s less clear to use `@view = new UserView {@model}` instead of `@view = new UserView(model: @model)`, but I use it for cons.
Hey, i tested it on my ARCHLINUX 64bit machine, but no errors occured without using *va_copy*.
What's the distribution of linux you are using?
sprintf is missing
please use scss for this file.
you can just do

resources :key_rings, :only => [:show, :create]

and then the post should just be when someone does a POST to /key_rings
The usage says that bootSize is optional, but ' - $2' will raise an exception if empty. Either correct usage or catch the case where bootSize isn't provided.
Do you have a tip how to do that? I'ts not that easy since cpp know nothing about the size of the structure.
Nope, sorry chinese people :-)
Right. I've updated the code.
Результат в html => надо отдавать text/html :)
Instead of this, I think you can just do the `pthread_join`s here.
If you don't have a release mechanism, one thing you can do is do detect that user is using workers (or single): in this case, the packet treatment is made in the Loop function on you can do zero copy on suricata side by using PacketSetData.
two underscores?
Let's just keep it as `doc` without the return keyword. The other filters also keep this style.
I think we should only depend on FRB when we start using it
This looks like if we fail to enable localization we will just log error. Could explain why in the comment.
Why do you have to specify require(chrome) and require(api-utils/loader) ?
Can you specify where these require calls are made so that we can know to which modules we are giving priviledges to require chrome, loader and runner.
I thought we only had to specify modules required from bootstrap.js.
Shouldn't we test for `if (!uri)`? or `typeof uri !== "string"`.
(it won't throw if resolve returns undefined.)
So I think what are doing here is:

- change `isPseudo` so that it returns `true` for 'chrome'.
- Remove `isBuiltin` and associated condition.
Ah, ok. So the popup on signup does close itself on verifying the email if you leave it open. Both paths are valid things to test, yes. Although it should differentiate the cases since prematurely closing the signin popup is a race of its own.
Something we should improve here is to let users add variables not only at the end of the string but at the current cursor position. I'm happy with a follow-up for this RFE.
max_value : None or float, optional
Shouldn't be a call to the `getValidationFailures()` method on the `$book` object, not `$this`?
@vrthra not sure if we should use an existing parameter or add a new parameter...
It was nominally supported but didn't work, #3910 is about fixing it. You can specify access controls on the master using auth.conf.
And more activated() vs. triggered().
This should be a private variable (__interacts); it's certainly not something we want the user to mess with.
The user doesn’t have access to that variable, because we aren’t just using `from interact_sagecell import *` in the user namespace. The only “special” variable visible to the user is the `update_interact` function, which exists as `sys._update_interact` in the user namespace.
due the changes in #1586 the underscore must be removed
That was behaviour from the helper already. But the $container is passed by reference, see the & in the method signature. This protected method is afaik only used in setContainer(), you can check it there it's called and the return is not used.

Imho it's not the nicest solution but I wanted to fix the bug only and do nothing to the rest if the code.
This should be removed if you agree with my previous commit
Put spaces around the dots
Because you can't `sudo echo >`, because then `echo` gets run as root but `>` doesn't. See: http://stackoverflow.com/questions/82256/how-do-i-use-sudo-to-redirect-output-to-a-location-i-dont-have-permission-to-wr
You can't call a super method of an interface. You actually need to add the value cancelled in the event and also a method for isCancelled to override the parent.
For transactions the getDbAdapter must be used, getDbSlaveAdapter is for read-only operations. The inserts and updates wrapped within the transaction might otherwise point to another server where no transaction has been started and be committed automatically. This will create unexpected behaviour.
This is not OK.
`controller.namespace.entriesController.filterBy` is probably a better way of doing this.

Anyway, if this works it's fine, but I'm not satisfied. It's just not that simplicity what Ember is supposed to offer.
Still waiting for current route state, somehow pragmatically, to be exposed to `action`.

Thanks @tdreyno for the patch! :+1:
Just followed your coding style from other generators (which used unicode even if not needed).
Well that file is now generated by the dumper, this controller is just there in case people do their own setup and don't call the packagist:dump command. Anyway the controller was outdated, so yes it changes it, but it just brings it in line with the dumper command.
get rid of this in the example easyconfig, only use it when actually installing ORCA on our end
D'oh forgot to revert after testing.
Note that accessing as strings and not as `GLUT.lastX` is not necessary here. Strings are only necessary when connecting between code compiled with closure and code not compiled with closure. So `event` comes from the browser, which is not compiled with closure, and needs strings, but GLUT.lastX is entirely in closure compiled code.

I'll fix this after I merge, just mentioning this here for future stuff.
Typo: image
turns out net is not even needed.  So I have removed it completely.
I'd opt for utilities then, since they are technically that, and in some time most of the functions will probably be replaced by the tuque wrapper when it gets written.
This works, but since there are so few, you might as well just hard code [256, 16, 1].
Wrap this in a try/catch (of type Error) since some SWFs may not be running in an HTML page.
This could be `if p = @app.sitemap.page(path)`
I would add a break here, once you find your plugin, there is no need to check the rest
I just re-used this from elsewhere, but it looks like some sort of fix made it into helenus that filters the column key out of the results. I'll go ahead and remove this across the codebase.
Can you verify that making someone a tenant admin invalidates the user?
memberships is probably better here.
Ok, this also explains my [above](https://github.com/scala-ide/scala-ide/pull/179/files#r1358662) comment ;-)
Nope, this won't work, you need to also check to see if the player's name is the same, not just if it's an instance of a player.
Added space around: ready!
It does not work :(

```bash
$ time php UnitTest-e8af5d1.php | grep -E '^F'
F 0/abc
F abc/0
F abc/0/def
```
Репозиторий у @hmmvot, я запутался со своими юзерами.)
I originally performed this operation using the find call within the
controller, however it is used by the student's view as well and will
affect the order their assignments appear in as well. So if we can live
with that, then it can go in their. I guess for worst case scenario I can
also do the sort right before the admin call happens in the controller as
well.

On Mon, Oct 1, 2012 at 2:36 PM, Severin Gehwolf <notifications@github.com>wrote:

> In app/views/main/index.html.erb:
>
> > @@ -16,7 +16,8 @@
> >  <div class="wrapLeft">
> >  <p><%= I18n.t(:welcome_to_markus, {:user_name => @current_user.first_name}) %></p>
> >
> > -  <% @assignments.each do |a| %>
> > +  <%
> > +    @assignments.sort{ |a,b| b[:due_date] <=> a[:due_date]}.each do |a| %>
>
> As with another review, this sorting should be done in the controller and
> not in the view. Thanks! It would be good to have a functional test
> asserting that assignments are sorted that way. Perhaps we could even
> adjust the find() query to return things sorted already.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/MarkUsProject/Markus/pull/862/files#r1732623>.
>
>



-- 
Derek J. Dowling
B Sc. Honors Computing Science
dowling@ualberta.ca
587-588-5810
Is there a tab here, or just extra spaces?
+newBackgroundScheduler
It'd be nice to offer background schedulers of all GCD priorities, maybe through an additional method like:

```objc
+ (instancetype)backgroundSchedulerWithPriority:(dispatch_queue_priority_t)priority;
```

Then we can prioritize reactive operations just like manually-enqueued blocks. For example, maybe git operations are high priority, and cache refreshes are background priority.
Good eye. Also fixed along with others.
This section is slightly wrong, but that's my fault for writing bad docs. I'll clean it up.
>is this something we would like included in the current pull request?

@jamill If that's ok with you, I indeed think that it would be nice to have it. If only to make sure the fetch process is fully covered.

I have no idea of how to implement it though... Maybe an additional enum based parameter with a default value...
do.. didn't remember my own code ;-)

I would not check for NaN (otherwise we would have to check everything for NaNs...) just use if/else with `if len(fmin) == 1 and fmin[0] == -np.inf:` on line 728
we already use the decim keyword in time_frequency code and decim is also what the keyword used by mne_process_raw I would stick to it for consistency.
:-)

On 19.10.2012, at 20:56, Martin Luessi <notifications@github.com> wrote:

> In mne/artifacts/ica.py:
> 
> > +            pre_whitener **= -1
> > +            mixing *= pre_whitener
> > +            # mixing *= pre_whitener.T
> > +        else:
> > +            mixing = np.dot(mixing, linalg.pinv(pre_whitener))
> > +
> > +        if include not in (None, []):
> > +            mute = [i for i in xrange(len(sources)) if i not in include]
> > +            sources[mute, :] = 0.  # include via exclusion
> > +        elif exclude not in (None, []):
> > +            sources[exclude, :] = 0.  # just exclude
> > +
> > +        # restore initial sort, then mix back the souces
> > +        restore_idx = np.argsort(self._sort_idx.copy())
> > +        # print restore_idx
> > +        # print self._sort_idx[restore_idx]
> delete debug things
> 
> —
> Reply to this email directly or view it on GitHub.
>
Figured out a better way to fetch device stats. Use /proc/net/dev file and read the stats on it.
Include all the parts of the sentence please. Not all languages follow the same constructs as English :)
This never fails and is required to redefine an advice.
s/fix/fixed
you can remove ``(or any exception extending this exception)``. An exception extending AuthenticationException is still an AuthenticationException.
I think this line doesn't have to be changed.
Trailing spaces are not good, but a patch must only include the things concerned with the patch itself.
Please remove this change.
There was no consensus, but some people thought it should be. Personally, the only thing I care about right now is getting this stuff into stdlib, so that I can add execution-context support to parallel collections.

Once the alternative is agreed upon (e.g. we just use longs for nanoseconds everywhere), someone can refactor this appropriately. It can even be me :)
Лучше впихнуть в предыдущую секцию qresource.
I only added one line to that file on the MergedDictionaries,

<ResourceDictionary Source="pack://application:,,,/MahApps.Metro;component/Styles/GridComboBox.xaml"/>

I didn't touch anything else.

I think the issue is that I uploaded the first time an old version of the Generic.xaml and it should be that it remains the history of changes.

I as said you is my first time using GitHub (sorry) is there any way that I can delete all that history, so you can see that I only added that line?
I'd write 
```(json/read-json (browser getEval (str "JSON.stringify(" notice-array-js-var ")")))```

as

```
(->> notice-array-js-var 
     (format "JSON.stringify(%s)") 
     (browser getEval) 
     json/read-json)
```
If you do this (which I like), then you'll want to remove out lines 7 and 8
(int) $_POST['copy_from_theme']
Creative writing makes homework more interesting. And keeps me from
accidentally copying verbatim from the book. I'm glad it's making it more
fun for you, too!

On Wed, Dec 5, 2012 at 9:54 PM, nellshamrell <notifications@github.com>wrote:

> In week7/homework/questions.txt:
>
> >  5. What is the difference between a singleton class and a singleton method?
> > +*A singleton method is a method that is specific to a particular object (as opposed to a class).
> > +*(see question 2)The singleton class, or Eigneclass, is an anonymous class that bursts into appearance when a singleton method is is defined. It holds the definition of the singleton method. It becomes the class of the object that the singleton method is attached to, pushing that object's original class up a level to become its superclass.
>
> I like that "bursts into appearance"
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/UWE-Ruby/RubyFall2012/pull/148/files#r2331253>.
>
>
I was hoping you would have one method that works for both arrays and 2 numbers, but this is fine.
Please use descriptive names, always. Even in an interface where that name is not used any further. During a review, it is completely unclear, what "s" might be. Some javadoc comments for the methods might also help in reviewing (or modifying the code later, as much more code is changed during software development than is written).
Always change the user config and never the global and we are good. :D
Should the key file path be configurable via an attribute?
Comment on pull request diff
Dependency for Django
sFINALLYul!!!!!!!!!!!

Sent from my digital shackles. 

On Jun 8, 2012, at 11:28 PM, sacharya<reply@reply.github.com> wrote:

> sFINALLYul
Right! Making adjustments and updating PR!
The idea was to provide a meaningful metric, even if the agent wasn't configured. In this case, useAgent==false causes the metric to fall back to counting instances of an particular type of object rather than the actual size. The raw object count could then be used as a multiplier for the type of per-object size measurements that ehcache uses, either programmatically or by somebody who is observing the metric exposed through JMX.
No, it is not going to break clients. Trust me, all the tests I had are unchanged. Infinispan 5.1 and Infinispan 5.2 nodes will not be compatible to exchange map reduce messages though.
can't these two if be merged ?
Something needs to be added to the hash even if the file doesn't exist.
it should default to ``true`` for BC, no ?
It would be nice if one could change rc params even after the initial import, but I'm not even sure any of the other backends support this.
We probably only need to test this with one format. Perhaps png is best performance wise.
Hmm, I don't think my users would understand what it was for. I'd quite like suteki shop to be as simple to use as possible without having to understand SEO principles. It should do a good job for SEO without the user really having to think about it.
nit: extra space between "supports  "WPA"

also, please add a ":" after "following"
Typo: Forgoten/Forgotten
ok, it's an alternative
Btw. Yes it is thread save. It creates for each connector a complet new instance with a new serviceId
Does it return true if it falls on the tile edge?
@bp323 Can we make sure that the first parameter for every callback is a `boolean` for whether is was successful or not?
Like we're doing with `getGroupData`
Casting to `Integer` or `String` leaves out the chance of someone using `Long`. Instead of the `try..catch`, you can use `PropertiesUtil.toInteger(content.getProperty(COMMENTCOUNT), 0)` or `Integer.parseInt(String.valueOf(content.getProperty(COMMENTCOUNT)))`.
why that change? if you want a different precision make it an optional parameter
get_kernel_matrix will get you a matrix - which you have to free via matrix.free_matrix() later on otherwise it will leak memory.
Can you have a look in the themes/default.scss file if there's a background colour you can reuse? Maybe just a base color through darken/lighten?
It seems like a lot of this code could be shortened if we just do `jToggle.attr("class", "hint-marker-positioning hint-marker-" + type)`. This would also make it easier to have more of these sorts of widgets in the future, since we wouldn't have to constantly update line 74 to say `jToggle.removeClass("hint-marker-error hint-marker-help hint-marker-foo hint-marker-bar etc etc")`. (In other words, before line 74 existed, this module wasn't strictly tied to only having two types of messages.)
That's where they were before :)

Maybe once we have this extracted to a separate repo it would make sense. But otoh the whole idea of this change is to put everything, e.g., Hipchat-related into one directory/namespace. 

After all these are the "instruments that come with the addon hipchat".
Should this say `make pkg`?
Are you sure you want to override the parameter ``$host``? This often makes code less readable. Please also check line 298.
Are you sure ``$this->getMock('Zend\\Db...iver\\ResultInterface')`` has the correct type as ``object<PHPUnit_Framework_MockObject_MockObject>`` is not a sub-type of ``object<Iterator>|object<IteratorAggregate>``?
异常还是通过callback(e)的方式传递吧。
Sandboxing, perhaps?
This method does not exist anymore, we might want to add a getPageName method and use that.
Tror også det er lurt å slenge på en `creates => /root/$elastic_package`, hvis ikke ser den ut til å laste ned `.deb`en hver gang den kjører.
well, if this page is displayed from the web (which is the case), the new code gives you the appropriate output. The only issue is that he forgot to pass the request object to the template
definitely rethinking this, I wasn't taking into account the possibility of the element having other filters applied, and once that is added into the mix the tests aren't passing anymore. Making a helper seems to be a good idea.
Non-unique IP address
Ehm. This does not exist anymore. And this line you can remove, it was not here before.
Change arguments name to `browserid_assertion`, `browserid_audience` and `request`, in this order.
```
class HoroscopeBase(object):
    ...


class SlovenianHoroscope(HoroscopeBase):
    def get_language_code(self):
        ...

    def fetch_data(self):
        ....
```
No need to test all this again, if they are tested already in other tests. Concentrate here on assertions needed for your test case, for notifications. So probably you have to check is published flag, and that there are no comments. You could also be dirty and go and directly access the database and check if subscriptions are as they should be.
Yes, you are right, that sounds like a better approach. I kind of went for it with the update URL but did not go all the way with the filter elements because I only knew about form.serialize(). Thanks for the suggestion.

The change would not be difficult to make, just that I don`t have the resources right now to make it.

Even so, I`m guessing that this is not really a blocker for the merge, right?
OK. Note that I copied it from SourceDialogs.ycp
This is basically the count of the EquipmentSlotId enum right? If so it would be alot better actually read the length of it instead of having a hardcoded length that needs to be updated:

this._equipment = new uint[System.Enum.GetValues(typeof(EquipmentSlotId)).Length];
I think it's opinionated to embed the order_id in the authorization, especially when the order_id is already part of the request. Do you know why it was embedded here in the first place?
There are a few ways, yes. Will discuss on IRC.
Woops. Thought I changed it back to what it was...
Do we need the commented versions? If so, could we we add a comment be added about them?
It's the action datastore_upsert (there's no datastore_insert or it's not documented) with "method" set to "insert" in the send_request function. (So we agree.)

I may have had other problems while testing that lead me to believe create would fail if it already existed. The delete isn't the main thrust of the change here, though --- I'm not opposed to deleting the call to delete, though I think it's clearer to keep it in if that's the behavior anyway.

As I read the code, previously it would call datastore_create on each 100-row block that it wanted to upload. It can call create on the first iteration, but it should call upsert on later iterations. And then it would call create_mapping for the final 100-row block, and that wasn't documented and I didn't know what it did, so I just deleted it.
No, I meant using the windowID of the correct window if there's a way to know that? This way the default headers of that window are sent, I believe. Would be nice in case of private windows. See http://operasoftware.github.com/scope-interface/ResourceManager.1.3.html#CreateRequest.CreateRequestArg.headerPolicy
'runtimes.getScript' and 'runtimes.getRuntime' can both return 'null'.
window_info
Probably none of the elements created above are needed when we end up here, maybe move those creations into the `if` part?
Could we just pass `nil` as the parameter on line 57 and get rid of this variable? It's value never gets changed that I can see.
why does cdf specify the kertype but pdf doesn't ?

I think ckertype will need more options. maybe later
Thanks for catching this. The problem was that LocalizedTime doesn't keep track of a date and therefore was defaulting to today's date, which varies by timezone. We discussed the possibility not allowing "ago" to work at all for a LocalizedTime, but decided that there are certain use cases where it makes sense for a user to ask about time on the current day. To test this I'm now just using a timespan with two times which are x seconds apart by definition (e.g. time2 = time1 + x).
the queue _will_ be deleted once the channel is closed (or actually the connection I think).
This might never occur since $package is replaced by the foreach loop on line 20.
ow how I long back to the days where we allowed to use `ide.removeEventListener("afteropenfile", arguments.callee);`
needs more `<marquee>`
Works for me... CoffeeScript feels more Python like to me. I guess that's why I inadvertently used underscores.

It's probably a good idea to change these.
I think what you want is at this point:
```
try:
    result = _galsim._...
catch HSMError:
    if strict == True:
        raise
```
Jim might correct me if I'm wrong though.
Here's an example of where it seemed to me that the `.view()` method call was not needed (c.f. comment at line 189)
Ah, right you are, I'll do that here!
This looks like quite a lot of stuff to be adding to the public api. Do we really want to add it all on this interface, given that implies supporting it for a long time?

Could we have it as a separate view or something, so that the public api interface is kept simple? Just thinking out loud...
groovy multiline string """
nohup...
for i in ..
do
  grep ...
...
"""

is easier to read (and type)
not _being_ pragmatic
I fixes it now.
side-note: it's more akka-style to use mapTo on futures than using asInstanceOf on results

(remoteFront ? RemoteTask(action, self, logger)).mapTo[ActorRef]
"First, we copy"
This also needs to use the standard connection string then, but I'll ignore it so we can fix it for all tests in one go.
You missed it last time. However, this is how the functions in the Python standard library tend to work, and I chose to do this to match the behavior a Python programmer would expect.
These actions have side-effects and so need to be in a form that submits via POST
What is this class supposed to do when len(new_ids) > 1, should it go over ntrials or not?
Do we need a `depends_on :x11`, too? Just asking, I haven't tested that.
I like the idea, I think that's something the Admin and not project managers should worry about.
What spacing around if statements should be used?

```javascript
if (selLength>0) {
if(selLength>0) {
if (selLength > 0) {
if(selLength > 0) {
```

And following on from the function def, should the opening bracket be on the same line?
Am 22.11.2012 um 13:46 schrieb Andreas Kaiser:

> In js/deform_bootstrap/__init__.py:
> 
> >  from fanstatic import Library
> >  from fanstatic import Resource
> >  from js.bootstrap import bootstrap_js
> >  from js.chosen import chosen_jquery
> > +from js.deform import deform_css
> 
> Why is this needed? I'd rather leave out deform's CSS completely as it's ugly as hell.

It is not needed but I made it only equal to js.deform where the beautify_css is loaded, too.
(In my project I actually removed the beautify_css via monkey patch, but the deform_css does not seem to harm.)


Mit freundlichen Grüßen
-- 
Michael Howitz · mh@gocept.com · software developer
gocept gmbh & co. kg · Forsterstraße 29 · 06112 Halle (Saale) · Germany
http://gocept.com · Tel +49 345 1229889-8
Python, Pyramid, Plone, Zope · consulting, development, hosting, operations
Тест зависит от локали пользователя. У меня, например, не работает.
I think all the necessary options can be isolated in JRUBY_OPTS, e.g.:

JRUBY_OPTS="-Xcext.enabled=true -J-d32 -J-client -X-C"
Can we have an accesskey (from a DTD)?
In fact it should be: `public SpellLoader SpellLoader = new SpellLoader(this);`
or even public SpellLoader spellLoader = new SpellLoader(this);
I know that but I want the assert to be visible and be able to see what are we asserting.
I think this duplication is accepted :D
sorry about that. will fix.
That explains a lot, thanks for the heads up, wasn't aware of the issue.  I'll try and get it working locally so I don't keep pushing dumb mistakes up here. :)
I would say ``parameters to choose`` or even ``hyper-parameters to choose``, as in some sense, the centroids could be seen as parameters and they are optimized to have the smallest distance to the cluster members.
You can use the `floating` type. This will produce the two functions (double and float) for you. See:
https://github.com/scikit-learn/scikit-learn/blob/master/sklearn/metrics/pairwise_fast.pyx
the special case for 'huber' and 'epsilon_insensitive' - I'd rather not have it but I didn't find a reasonable solution which does **not** require to change the signature of the loss functions (not good because old pickled SGD* objects cannot be deserialized with the new release)

lets leave it this way for the time being.
Then maybe change the title to somehow mention that the model predicts the house price otherwise it's not possible to interprete the plot without reading the companion paragraph.
Can you please elaborate on what you mean by "as a property" ?
either 'that guarantees' or 'to guarantee'.
Yes. I wasn't quite sure what the right choice would be. I'll have a look at KernelPCA. What do you think about kwargs?
And gamma should be ``1/n_features``, I think.
I think that thins can simply be written symbols.add(o)
If $_GET['v1'] undefined?
Klammern sind nicht notwendig
i certainly understand the motivation towards _less_ ambiguity and being explicit, but this specific `import` style seems to be standard among most of the officially bundled extensions found here http://selenic.com/hg/file/41fc1e078d68/hgext

i don't have any illusions that this particular extension is going to make its way into the main hg trunk, but if we expect outside hg devs to poke around and patch this, it seems to be fair to at least adopt the general style of the bundled core mercurial extensions (especially since they are the de facto documentation for how to write extensions). 

It's your call, though.
_Too few_ test files can also be problematic, though, as it decreases the granularity of the status reports if the test suite does not pass. For DMD, this might not be as big a concern, as the test suite is developed in synchrony with it anyway. If you are working on an alternative implementation/port, though, it can be quite annoying if a big test case fails, and you can't quickly judge the scope of the breakage. I'm not sure what the best compromise here is.
e.g. here
This should be `<QDateTime>` or `<Qt/QDateTime>`
Please keep this into a Warning markup (as in the corresponding french rst file). It should be something like this:

```
.. warning::
  This method should only be used when there is a high degree of trust between the resource owner and the client.

.. seealso:: Voir aussi `http://tools.ietf.org/html/draft-ietf-oauth-v2-10#section-4.1.2 <http://tools.ietf.org/html/draft-ietf-oauth-v2-10#section-4.1.2/>`_.
```
Is it possibly copyright London HUG?
In that case, the comment is invalid "allow developer to check if the client is already synced with a user.", since developer can get other information as well.
This is ignored by maven 3 as using the enforcer plugin is better, but it has Maven2 builds fail eagerly (rather than have weird errors because of plugins defined below are not consistent)

mvn dependency:... goals are much happier with it being defined.
Strange case mixture.
That is a good point. I have only changed the ones in unittest dir. I'll have a look around.
Doesn't calling linkrows() mean it will leak?
Any reason we don't also set the next_url here? Wouldn't we still know the request.path they asked for?
Why not use something like `--force`? Or make a general switch with `--yes`? It could answer for ppl and get stuff done. 

Kind of like `apt` does it when you do `apt-get -y install foo`.
Is this the problem you mentioned in the pull request?
These annotations aren't necessary any longer. They can be removed.
`0 === strpos($parsedBranch, 'dev-') || '9999999-dev' === $parsedBranch`
Rather than the try/except and call to `keys()`, it might be better to do `if callable(getattr(self.table_pagination, 'keys', None))` or some variation of that.
add try catch whoever interacting with the page.
@christianv Jira: https://jira.media.berkeley.edu/jira/browse/CLC-853
Because I'm dumb. Maybe I tried it and it didnt work? Ill try again.
If the element isn't a timepicker instance, then we do nothing.
do you have an example of a module that incorporates this type of functionality?
vprint_status could be used here, which allows to delete the "if @verbose" check, you can check Msf::Module where vprint_status is defined.
should be an optint, no casting should be needed
I would end end this sentence with ", and yields the downscaled images. Note that the first image of the pyramid will be the original, unscaled image".
derrrp. nice :+1:
Can you use markdown syntax here?
Does this need to be exposed?  `BillboardCollection` use to expose it, but now it determines what billboards are static, dynamic, or stream, and manages them behind the scene.  The hint to WebGL is not that important depending on the driver, but grouping vertices in buffers by their update frequency is very useful, so, for example, if only 10% of the vertices are updated, 90% are not touched.
I just realized this code is in a Sandbox example.  So it's probably better for the code to be cleaner, and not necessary faster, better optimized.  If you agree, just leave it the way it is.
How many new matrices in this change are likely to be used every frame, e.g., because they are used in lighting the central body?  The cascading dirty flags are pretty confusing so my trend has been to minimize using dirty flags for:
* Things that are quick to compute, e.g., `viewRotation` from `view`.
* Things that are likely used every frame.

Some of these fall into that category, and you are welcome to simplify them.
same :p
You shouldn't put debug code in final code
Please lose all of the single-quotes. We pretty much only use them for string literals in text, mostly for event names, like 'click'.

In this case, I don't think there's any loss if we just say, "Previous simple menu example"
yay infinite recursion! can't believe I missed that one.
typeof - оператор, а не функция - от скобок смысла нет
Can you write "(pitchAngle > minPitchAngle || (pitchAngle <= minPitchAngle && mousePitchDelta > 0.0))" as "(pitchAngle > minPitchAngle || mousePitchDelta > 0.0)". The same goes for the if statement below.
`s/Grubers/Gruber's/`
and actually not only the menu bundle needs it, all the cmf uses it by default.
Why `HEAVY` on `SRC_STD_1` and `SRC_STD_2`?
maybe instead of opting out in eclipse, we could opt-in in a mac/linux terminal? For example:

```
var env = Platform.environment;
return env.containsKey("TERM") && env["TERM"].startswith("xterm");
```
Please also add a test here where we expect to replace a local folder with a new folder, whilst adding children to it.

I want to make sure that `replace` still functions under batching.
The thing is that this particular method is used in 2 invocation handlers. I'm trying to make it more OOP-like right now
Can you please explain that to me in more detail?
@jburel: Can we get these commented out lines removed?
What is the purpose of this line? Unless I’m missing something, please remove it.
This line is actually not needed. Inserted it while trying to get the settings loaded right.
Change comment, and change to `86400`
Please fix the phpdoc. There is no boolean here but an array
yea I forgot about this behaviour. fixed! :P
aha, you got back to our popular check in the validation :)
0.3.1 ??
And you still need to address the issue of these calculations not taking class length into account.
Can you just do `#define SEQACK_TABLE_ENTRIES PACKET_DATA_BUFFER_ENTRIES` instead?
Verwende ausschliessende Formulierungen für die Validierung:

if($order->getState() != Order::STATE_OPEN){
  // ERROR
}

$this->remove($order);
Is there a particular reason this variable is declared twice?
typo: 'restTemplate'
I agree. I basically went off the original JIRA but I am more than happy to change this to something less obtrusive.