labofsahil/github-event-dataset-2012 huggingface dataset:
мне тут приходят уведомления на новые комменты, 
мне  интересно тебе эти оценки действительно важны ? для чего? или это привычка из школы:? Самое главное это чтобы ты сам разобрался, а эти баллы я думаю чтобы отсеять совершенно ленивых и невменяемых. Сколько я ставил в оценку я даже не помню
Do we want the sprint number as the 2nd qualifier? Seems like we might have more flexibility using those 4 qualifiers as: version,release,sprint,dot-release, so sprint would be the 3rd qualifier.
Just trigger the event, no need for this method
Du kan sannsynligvis reprodusere problemet ved å f.eks.

1. revertere denne endringen tilbake til f.eks. 1000 (ett sekund)
2. naviger til løftesiden
3. legg inn `sleep 3` i den controller-metoden som blir kalt med Ajax
4. klikk på noe som trigger Ajax-kallet og vent. Denne bør da time ut og trigge alert()-en.
Hi Mark, there's definitely some duplication in the locale file -- we've been moving from the "_{record_name}" I18n keys to now using the  template name where the label was defined.

I think the duplication is in the _names and _agent_contacts entries - the agent_name_person, agent_contact_details. etc, entries are the ones the UI use.

When some labels are shared between templates (e.g. 'rules' on the agent_*_name definitions), I've begun erring on the side of duplication to give greater control of the separate contexts -- this may become more useful when also defining tooltips and placeholders for specific fields.  However, I can see this duplication may also cause confusion.

If preferred, I can follow the pattern whereby we define the labels in a generic entry e.g.

agent_name: &name_attributes
  rules: Rules
  rules_tooltip: Rules by which the name was formulated
  ...

and then reuse that pointer for each of the name templates:
agent_person_name:
  <<: *name_attributes
agent_software_name
  <<: *name_attributes

Thanks,
Payten
I left this inside the try because I would otherwise need to add an extra "try/catch/rethrow as GitException" block around the getRepository. Do you want me to do that?

No release doesn't throw.
@Haacked Those parameters are supposed to be references to volatile data so that if the Clone is being run in one thread, another thread can check on the progress and potentially report on it. Since Clone can take a long time, this was our solution for handling progress without embedding threading or awkward progress callbacks into the libgit2 internals.

I have no comment re how to name things and/or manage volatile data idiomatically in C#.
`int` is either 4 bytes on x86 or 8 bytes on amd64.
`IntPtr` has its size aligned with the architecture.

I know, it's far from being explicit, but I haven't found a better alternative yet :)
why is it still there?? I removed it


On Sun, Dec 16, 2012 at 4:38 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.
>
> keep_only_data_channels is not documented and titles in not an argument.
>
> I don't really like keep_only_data_channels. When this problem happened in
> the past we agreed that picks=None was equivalent to
> keep_only_data_channels=True
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/221/files#r2430076>.
>
>
checks also that keys are strings otherwise it will conflict with the getattr that are integers or a slice used to select some epochs
should st_mask_from_s_inds be public? name is not explicit and it seems like an auxiliary function.
I think concept name search should always be case insensitive otherwise this will case a mess if an implementation sets it to false
Make it ConceptDAO.
language: "SSL offloading" should probably be "SSL termination and offloading", since you explained what SSL termination is, but haven't used the term offloading.
This should be `[AcmeFooBundle]`
We're not consistent everywhere, but I try to write without using "we" or "lets" whenever possible. Instead, how about being direct:

    First, create a ``schema.xml`` file inside the ``Resources/config`` directory of your ``AcmeStoreBundle``:
`<strong>not meant</strong>`
`_filldedent` can help you with the wrapping process:

```
>>> from sympy.solvers.solvers import _filldedent
>>> print _filldedent(str(((x-1)**20).expand()))

x**20 - 20*x**19 + 190*x**18 - 1140*x**17 + 4845*x**16 - 15504*x**15 +
38760*x**14 - 77520*x**13 + 125970*x**12 - 167960*x**11 + 184756*x**10
- 167960*x**9 + 125970*x**8 - 77520*x**7 + 38760*x**6 - 15504*x**5 +
4845*x**4 - 1140*x**3 + 190*x**2 - 20*x + 1

```

Then you just put that expression inside of parentheses, like:

```
e = (
x**20 - 20*x**19 + 190*x**18 - 1140*x**17 + 4845*x**16 - 15504*x**15 +
38760*x**14 - 77520*x**13 + 125970*x**12 - 167960*x**11 + 184756*x**10
- 167960*x**9 + 125970*x**8 - 77520*x**7 + 38760*x**6 - 15504*x**5 +
4845*x**4 - 1140*x**3 + 190*x**2 - 20*x + 1
)
Thanks for pointing that out. I simply used the way abrombo wrote in http://groups.google.com/group/sympy/browse_thread/thread/8a00e9175352398/7e78ac467299f2fc#7e78ac467299f2fc.
Fixed now.
Can you translate that one :-)
¿La fecha de fin de Sprint no fue el miércoles 11/7?
En este caso yo no devolvería error porque tenemos todos los datos de la postulación, y, suponinedo que esto luego funcione de otra manera, por ejemplo, asyncronicamente, no podremos devolver error. Igual no me parece muy importante
MIssing language parameter or how would you give back multilingual questions?
I'm not sure this is what you meant... 

```
{|vehicle| vehicle.make == "Honda"; vehicle.color == "blue"; vehicle.model == "Accord"}
```

This will resolve to (assuming it is a blue Honda Accord)

```
[true; true; true]
```

But if it was a blue Toyota Accord, it would be:

```
[false; true; true]
```

I'll add some code to the main description on better way to do this
task_id should be a parameter
I made a boo-boo in not putting $timeout into the function's parameters. This is why I couldn't get spec to work the way I wanted to.
We should be consistent and use single quotes for all strings unless the string contains more single quotes than double quotes.
```this.options["indexFile"]  ||  "index.html"``` ???
You could split on `path.sep` instead:
    http://nodejs.org/api/path.html#path_path_sep
The tree is the left side as per the phrasing, I will be more explicit. 

FWIW the docs in libgit2 aren't great here either.
I will do no such thing. Open source means that **I** get to dictate what **you** will be using, not some democratic model from your hippy lala-world.
It's because the import takes two lines.
I think the issue is that this function is included twice -- only the second copy below gets defined, so I think I'm cool with this change.
Just wrote a test for modifying rc params. This is pretty cool, I can even switch from xelatex to pdflatex within a single script :)
https://github.com/pwuertz/matplotlib-backend-pgf/blob/master/examples/test_rcparams.py
Typo: Verifies
Typo: available
I removed it because with the MMI tests possibly coming through I thought we may need to clikc * and # keys.
it is declared in https://github.com/bebef1987/caseconductor-tests/blob/7aac885477dde7c21b54dd43a009d15349f7009f/pages/create_profile_page.py#L42
А мне текущая реализация нравится :)
$trans = InnerTransaction::begin(); //вполне себе самодокументированно и хорошо.
упс ))) я просто прогонял тесты и эти ошибки задрали уже, случайно запушил сюда!

Надо бы их в отдельном реквесте пофиксить!
Da sind 3 Tabs drin, github stellt Tabs als Spaces dar.
Und die Zeile wird später beim Update eh gelöscht.
@adrexia continuing our discussion about event handlers (this.on vs beforeActivate vs ontabsbeforeactivate).

Agreed that we should drop beforeActivate, good move.

Still, I strongly feel that applying events conditionally with `if` statements has exactly the result you are afraid of: making the code more difficult to follow because of excessive nesting. See how we are 3 levels deep in this line? (`if` in `actionTabs` >> the `if` above >> `this.on` over here).

Wouldn't it be easier on the future readers of this code to pull out all events into separate entwines in the same way the rest of the CMS does it? It should be possible to:

* move the `actionsMenu` to separate `.cms-content-actions .ss-tabset.ss-ui-action-tabset` entwine
* break the `siteTreeActions` into:
  * ontabsbeforeactivate handler in `.cms-actions-row.ss-tabset.ss-ui-action-tabset` entwine
  * onhover and ontabsbeforeactivate handlers in `.cms-tree-view-sidebar .cms-actions-row.ss-tabset.ss-ui-action-tabset` entwine
  * ontabsbeforeactivate handler in one more entwine that applies the "expanded tree view" panel.

This has the advantage of getting everything down to 1 level of nesting, and removing the large `siteTreeActions` which really only exists to no other purpose than to install these 4 handlers.

What do you think?
Can we add a quick example for each?
Ok good point, I will adjust this.
do we load Helvetica Neue through typekit? Because it's not the free version of Helvetica that comes with every OS, it's a much better helvetica font that you typically have to buy.
This line is really confusing to me. It'd be nice if there was an easier way to write it, but it's mostly a nit. I wonder if it could be made easier to read if we defined e.g. a `lastToggle.relatedElement` expando property instead of putting the related element in a function closure...
k. I preferred having less redundant statements, but I realized that this function is screwy because it's really two entirely separate actions with variable arguments.
This is duplicated from usage.rst, should probably be removed from there.
Ideally you would try to create the file, and if permissions are wrong delete the file, set umask, create the file, reset umask.

If you don't want to do that at least reset umask back to whatever it was (umask returns the old value) as soon as you create the file.
Are you sure ``array('parameters' => array('a' => 'b'))`` has the correct type as "array<string,array<string,string>>" is not a sub-type of "array"?
- Are you sure ``'Symfony\\Component\\Finder\\Iterator\\FilterIterator'`` has the correct type as "string" is not a sub-type of "string"?
- Are you sure ``array($i)`` has the correct type as "array<integer,object<FilesystemIterator>>" is not a sub-type of "array"?
this change actually was not needed - it is not breaking for BB. reverted back
I dig this approach. Nice.
This sentence is obsolete. I will remove it. Is there a way to modify the pull request?
I'd remove the empty line after the function definition.
I'm not sure if we should pass RAILS_ENV, makes it behave differently from the original command e.g. `db:create` would create test databases instead of development, and you can still do `RAILS_ENV=test rake parallel:run['rake xxx']`
That's an intersting point.  I think a better solution would be to allow escaping of the `:` if you intend it to be used in a non-variable capacity.  I'll have a play with that.  I'd rather not use double colons if I can avoid it.
Change type to SimpleXMLElement
I'd must leave this at return false so we don't change the API. (A message which is a non-empty string would evaluate to true)
the fuq?
The comma at the end of this line and the following line give me a synax error when trying to run `make`.
Mnja, we always use *e-mail* also in Slovenian translations. Don't now start with *e-pošta*. ;-) Or do it everywhere. ;-)
And probably you want to do rather:

```
self.panels = {panel.get_name(): Panel() for panel in self.get_all_panels()}
```
OK, just this channel ID is a bit ugly currently. So we will probably have some general channel for "home" view, so you could name it home, no? (Also elsewhere in the code.)
Bljak!
You comment is not complete. So I am not sure what you are asking. But you don't have to call save after [create](http://mongoengine-odm.readthedocs.org/en/latest/apireference.html#mongoengine.queryset.QuerySet.create), object is already saved.

How do you query for State objects?

Read more [about inheritance](http://mongoengine-odm.readthedocs.org/en/latest/guide/defining-documents.html#document-inheritance) here.
You should keep track of last unsubscribe, not of last subscribe. (OK, last subscribe could be interesting, but last unsubscribe is what is necessary.)
If you start overwriting `dispatch`, you are probably doing something wrong. Then you are not using the class-based views, just a function view, wrapped in a class.
I believe xtr.get("json") will return the JSON parsed data object.
Indentation. The next few lines are messed up too.
I feel like an option passed in with the request would make more sense. If you want to enable it globally it could be an option to initialize as well. This should not be the default as it breaks backwards compatibility.
Sorry about that. @ntalbott 
I think we have 2 simple solutions in this case.
We can either take locality out of the equation by stub the date conversion or 
keep locality in the equation by calling .to_time.utc.iso8601 on the dates or re-use the conversion function from the gateway for the assertion;

Case 1 stubbing

    start_date = Date.strptime('02/21/2012', '%m/%d/%Y')
    start_date_iso = '2012-02-21T05:00:00Z'
    @gateway.stubs(:date_to_iso).with(start_date).returns(start_date_iso)
    
    end_date = Date.strptime('03/21/2012', '%m/%d/%Y')
    end_date_iso = '2012-03-21T04:00:00Z'
    @gateway.stubs(:date_to_iso).with(end_date).returns(end_date_iso)

    options = { :start_date => start_date,
                :end_date => end_date,
                :receiver => 'foo@example.com',
                :first_name => 'Robert'}
    request = REXML::Document.new(@gateway.send(:build_transaction_search, options))
    assert_equal start_date_iso, REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/StartDate').text
    assert_equal end_date_iso, REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/EndDate').text
    
Case 2a converting the dates

    start_date = Date.strptime('02/21/2012', '%m/%d/%Y')
    start_date_iso = start_date.to_time.utc.iso8601
    
    end_date = Date.strptime('03/21/2012', '%m/%d/%Y')
    end_date_iso = end_date.to_time.utc.iso8601

    options = { :start_date => start_date,
                :end_date => end_date,
                :receiver => 'foo@example.com',
                :first_name => 'Robert'}
    request = REXML::Document.new(@gateway.send(:build_transaction_search, options))
    assert_equal start_date_iso, REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/StartDate').text
    assert_equal end_date_iso, REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/EndDate').text


Case 2b re-using conversion function

    start_date = Date.strptime('02/21/2012', '%m/%d/%Y')
    end_date = Date.strptime('03/21/2012', '%m/%d/%Y')
    options = { :start_date => start_date,
                   :end_date => end_date,
                  :receiver => 'foo@example.com',
                  :first_name => 'Robert'}
    request = REXML::Document.new(@gateway.send(:build_transaction_search, options))
    assert_equal @gateway.send(:date_to_iso, start_date), REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/StartDate').text
    assert_equal @gateway.send(:date_to_iso, end_date), REXML::XPath.first(request, '//TransactionSearchReq/TransactionSearchRequest/EndDate').text

Let me know which solution you like best
Sorry ch3cooli for changing my mind, but I think we should drop the whole temp-file deletion code from TortoiseMerge. In TSVN TortoiseMerge has it's "own" tempfiles and TortoiseProc is executed often enough to clean temp files...
why do you cast it?
Not all epic items are listed (from what I noticed, only cataclysm epics are)
Are you sure this is the correct way? there's probably a spell or something that does it.
https://github.com/TrinityCore/TrinityCore/pull/7873#issuecomment-9476005

Last line ;)

```
I kept the _MASK constants in enums for backup. 
```

I can remove them if you want me to.
This is definitely not the way you want to handle getting payment status change. You should not do it synchronous to "online view" for numerous reason.

You should implement a celery task (like in PayU backend) to handle it. The online view and ```process_notification()``` method should only check if request is correct, schedule celery task to get a confirmation, and response "OK" as soon as possible to the payment broker.
Both static and class methods can be called from uninstantiated class objects. The only difference that I'm aware of is that the latter automatically passes the class type as the first argument. Both have this same "gotcha" that a user expecting `compute` to manipulate an instance instead of returning one might throw away the result of `compute`.
Should be filterBy both parts.
This is not the most efficient way of computing 'p'. Vec4.kl should have support for operations of type (Vec4*Mat44).  This is a workaround, unless we want to define transpose(hC) = transpose(C)*transpose(h) in line 36 above, so we don't do transpose at every step. Leaving it to the FE folks to decide.
Ok, i'll do, also i'll remove any English writing crimes I did. If I can.
Ah, I mean the `http_status_codes`. If you say that can't be missing then it's good too.
Don't cast away constness from read-only strings, they end up in the .data section that way instead of .rodata.
Not much point adding 0.0.
This isn't referenced from anywhere. Did you mean to include it in this PR?
please add it only for the 'usb'  module to minimize side effects, probably like:

'usb':        ['support/rthooks/pyi_rth_usb.py', 'support/rthooks/pyi_rth_ctypes.py'],
Would you mind also correcting the comment block - it still mentions the old -g switch, then I think we're good to go!
How'd this escape my search&replace...?
Sorry if I'm just missing something, but shouldn't this be alias_threshold, not 1-aliasthreshold?  (i.e., at most 0.005 of the flux is aliased, not at most 0.995?)
Well, it's my own fault.  I wrote the code in the first place.  :-S
Here and for countChildren, let's use underscores (as you do in the other new test class)
Were you able to test this @jrbl ?  If so, and it works, then remove the commented out lines.  If you weren't able to test, then we shouldn't pull.  I can help w/ the testing if you need it.
What if destroy fails in some way? Could we be leaving detectedMachines in some inconsistent state?
can be re-enabled?
There is a VisibleForTesting guava annotation you can use
On Fri, Nov 09, 2012 at 12:13:29AM -0800, Jeanneret Cedric wrote:
> > @@ -0,0 +1,52 @@
> > +# file managed by puppet
> > +<VirtualHost<% ports.each do |port| -%> <%= port %><% end -%>>
> > +  ServerName <%= name %>
> > +
> > +<% aliases.each do |serveralias| -%>
> > +  ServerAlias <%= serveralias %>
> > +<% end -%>
> > +
> > +  RewriteEngine On
> > +  RewriteRule /(.*) https://%{HTTP_HOST}/$1 [R=302]
> 
> well, if the user has something weird in his resolv.conf or hosts, that's *not* server's problem.
> We won't do something crappy as:
> RewriteCond %{HTTP_HOST} alias1
> RewriteRule ...
> RewriteCond %{HTTP_HOST} alias2
> ....
> 
> This is non-sense. Really.
> HTTP_HOST isn't random as long as we do not have some """power-user""" bitching with /etc/hosts or dns resolver.

We indeed *never* have to deal with users messing around with their systems
configuration ;-)

I agree it's not the server problem. But we don't only manage the servers.
We also have to deal with tickets from dumb users complaining about error
messages in their browser because they have "search domain.tld" in their
resolv.conf.

Well it's up to you, it's only a detail...
Un peu degueu comme syntaxe. Utilise _whereas_
Ya, you are very right !

Hm.. I totally understand what I was missing. Alright, I will have to redo the migrations.

I will give you another  pull request soon..

I appreciate you took notice of this !
Any particular reason why this one was changed?
isn't fsync super strongly discouraged?
I'm a little confused why its being introduced at this point?
updated message.py comment, this documentation is correct.
I basically took the code from the loadUserFromUsername method
Same as above, return ..()
AHAHAHA.
Note to self. Make this proc (and New()) automatically detect the presence of a door, permitting it to self-manage if the connection is direct/indirect (and thus if the zones can merge)
WHY.  ALL THIS GEAS CRAP.  WE ALREADY SOLVED THIS.  (We have a check in mob/move() )
Also didn't realize that my change for https was included there. That needs to be a bit smarter as well; the code wasn't picking up https requests with just the http match.
Better idea -- just treat this as a `{release_worker, Worker}` message, because that's effectively what it is.
Почему не boolean?
Название пакета по CC со строчной буквы.
Не вижу этого теста.
Assert missing?
This should be a comma separated list.
s/prepaproval/preapproval
The base.py is used for functions that are available globally across the site, for example the header or footer which is common.

These two `click_next` and `click_prev` are only relevant to the Statistics page so it should be stored in `statistics.py` so they can only be run when the test is on that page.
is good :)
I may be remembering incorrectly (@rbillings, @stephendonnerk, or @retornam correct me if I'm mistaken) but I don't think we want to advertise the <code>--destructive</code> flag in the readme. Community members new to the test suite may be unaware of the danger of the flag and run these tests against our production environment.
I think we'd need about a dozen or so helper methods to cover all of the cases.  I can do that, or can just log the caught exceptions at WARN (since there are 35 places where we catch UnsupportedEncodingException, only 5 of which are not in loci.common.utests.EncodingTest).
Do clients need to be scoped per `user_id` or can anyone with access to Central view *any* client?
Not sure but shouldn't this be `"""`? Or is that just convention?
just a minor thing: you can remove the opening and closing triple quotes above.
> The AsScorer(fbeta_score, args=dict(beta=3)) is a bit to verbose for me still.
> Maybe **kwargs?

Possibly. I have no love for **kwargs, but it might be a usecase.
I think I would like all classifiers to have a ``classes_`` attribute.
Not sure if I would implement it like this, though ;)
No, I meant: what's `$now= time();` used for?
I have no idea what this comment means.
It wouldn't fit the callback signature to pass TiUIBase* to _getChildren
Because TiLogger is a Ti object, we've been using the standard of TiXXXObject, i.e. TiLoggerObject.
Test surely need to verify some basic features :)
The best way to do so is to follow something like the `setUp` in https://github.com/zendframework/zf2/blob/master/tests/Zend/Paginator/Adapter/DbSelectTest.php
Why change the realloc() call? This will make things slower for long lines. See http://en.wikipedia.org/wiki/Dynamic_array#Geometric_expansion_and_amortized_cost
Yes, I see why it's logicalname now, as may be post logical name rename.
But (see other comment) what if exception occurs before it succeeds in renaming logical name..
Would rollback() still be called on that kind of early failure?
If so, shouldn't it check that 'logicalname' != file->queryLogicalName() ?
yes, probably should be stopping, at least for clarity.
As you spotted though, only relevant if stopping==true anyway.
I don't think this check is valid.
Although it's the default, the log dir _could_ be different and might not contain the thor process name..
Does moving up (inside the non-Win32) part, mean we will never use gcc or clang on Windows?

I think I chose that place because it's the nearest, platform-independent, place to where the compiler is set. I didn't want to assume much.
This constant seems useless to me. Either we should inline this or make it configurable via settings.
За ингэж өөрчлөе.
Irakli is currently working on refactoring all APIs in order to use `Base` and `Namespace` instead of api-utils, traits, light-traits, ...
So that we should mention api-utils anymore.
I'd say Base, NameSpace and Events are the top three API to look into, if you want to build a module "like in the SDK".
Or if you want to learn how SDK works. If you start looking at future code (after irakli's rewrite), you will have to understand these two modules.

Note that `events` module is going to be replace by two new ones calles events/core and events/target. (Still living in a pull request).
Nevermind, I see it below in Type.module_name. I'm not super keen on the idea of having #module_name= that a user can call and set the name to any object.
Again, I just want to be sure that devs don't have to call `loadProfile` to actually load the profile of a user.
@shacker Do we have a follow up jira for this somewhere?
Why are we using <font> tags instead of <span> tags? Can you namespace those classes a bit more - change "days" to "kickoff-countdown-days"? Want to make sure we don't accidentally alias them later.
Is there any way to delete an award?
`not` would do the same as this `if` construct
why did you need to set a reference position for the lbr? this has no influence. The arm has to be referenced using the Kuka KRC.
Added a comment, but: `quote` seems to only quote and return the first list. So that.
lowercase "turn off"
qWarning?
{ here goes on newline
This function looks awful similar to cmd_exec() in lib/msf/core/post/common.rb
This is never really long enough for me. The exploit drops the vbs and mof file, and then I just keep on waiting until the module expires -- no shell. I tried to drop a random file in the mof directory, and that seemed to speed it up for some reason.  It doesn't make much sense to me why that helped, but right now I'm having a hard time investigating further, because after a reboot my box is no longer serving Beehive even though the port is listening.  What a b*tch to test.
The purpose of extra spacing is for distinguishing itself from next logical line, see

testcases.py:55:13: E125 continuation line does not distinguish itself from next logical line

But I'll change it back in my next commit.
I wish XHTML had caught on ;-)
typo: link to them.
Like I said in IRC: maybe just add something like `TODO: when we have a better layout import from mozillians` or whatever.
One thing to note is what the conflicts method looks like currently:

https://github.com/norman/friendly_id/blob/master/lib/friendly_id/slug_generator.rb#L43-L51

```ruby
def conflicts
  sluggable_class = friendly_id_config.model_class

  pkey  = sluggable_class.primary_key
  value = sluggable.send pkey
  scope = sluggable_class.unscoped.where("#{column} = ? OR #{column} LIKE ?", normalized, wildcard)
  scope = scope.where("#{pkey} <> ?", value) unless sluggable.new_record?
  scope = scope.order("LENGTH(#{column}) DESC, #{column} DESC")
end
```

This was changed in a recent pull request #239
The original code had that, I changed it because it didn't make sense to me. But it's a breaking change so I put it back to how it was originally.
Yes. I tested it in the Starter/Hello World and Two Canvases Sandcastle apps.
Instead of using anonymous objects, it would be a lot faster to create a local private Node class at the top of the file with child1, child2, x0, x1, y0, y1 member variables.  The rest of the code won't change but it will speed up a lot.  Replace the root object 50 lines up as well.
This logic is still wrong. `result` becomes `true` if ANY point is inside the path. Given this, #L2R51 doesn't have any effect
I assume `sketchabot` is another main.js, which is the whole entry point of the project (correct me if I am wrong). Our project is clearly not an extension to Raphael. There is no point to add the "main function" into Raphael class, and call the "main function" as a method of Raphael.
They are just used for testing exceptions routes, like in [testOnException](https://github.com/danbev/aerogear-controller/blob/AEROGEAR-426/src/test/java/org/jboss/aerogear/controller/router/DefaultRouterTest.java#L136)
Move this after the <title>, and just set a base font for now. We can extend that later.

Look at any demo file for a few other things, e.g. setting encoding to utf-8.
Ah, Ruby 1.8 uses an array of strings (http://www.ruby-doc.org/core-1.8.7/Module.html). I'll update this to do the right thing in either case.
that router is now called DynamicRouter (i realize you just reformatted the text and it is currently wrong, but while we are at it we might as well fix this)
need comma after "folder" instead of period
the litmus for this is 22918, as it was before.
So how should I workaround this:
```D
T f(T)() { assert(0); } // line 1

void g() {
	f!(inout int)();
}
```
Output:
```
main.d(1): Error: inout on return means inout must be on a parameter as well for T()
main.d(4): Error: template instance main.f!(inout(int)) error instantiating
```
?
cut and paste :-)
I'm assuming that if GCC creates a 64-bit library, that's what it should be using as well. 

If it wouldn't be a 64-bit build, the `lib64/libstdc++.a` shouldn't/wouldn't be there.
either add an issue or remove the commented code
I think this dependency should be removed
This line broke linux. video still plays but frame stepping for example is broken because of the pixel format
I am wildly conservative ;-) I've been bitten way too many times in the past.
Yeah... I initially had some more abstraction, but with only one kind of concrete filter, it was really hard to tell what abstraction was appropriate, so I just dumbed it down ...
Never mind, I sorted it out.
Does this table exist on Eden too?
Can you link to documentation and explain what this is doing exactly?
Also confusing for the same reason: 2 raises are expected
Есть предложение - не выкидывать исключение, если хотят подвинуть вверх верхний элемент списка, а просто тихо ничего не делать.
Нет, я предлагаю дать свойствам нормальные названия, типа StatusHeader, TitleHeader, и все такое.
Все равно, сделай ее локальной для цикла.
secondary links are now rendered through a list_item function. needs some extra css
added $min_result, to check if a field has the minimum amount of documents. Oddly enough solr doesn't limit this on the solr side like normal facet fields. So in the case of date facets, fields with 0 references in the index get included.
Which is actually kind of nice, because I need those fields (empty or not) to create the [start TO end] filters.
2 minor things:

1) I would prefer if (x instanceof Y) { ... } else { log } - only because it's easier to read. I personally like to see the cast immediately after the instanceof check with no lines between them. It's safer in case someone ends up changing the logic here some day (imagine the current if block gets longer, the instanceof then changes, and somebody fails to notice that the cast might no longer be safe because it's so far away from the check

2) the variable name probably should not be gConversionService anymore (that was originally a shortcut for "GenericCS" which is no longer necessary the case). Even though it's long I think I'd prefer "configurableConversionService" just to be explicit yet another way what we are casting to and working with.
Ask glenn about azure / origin.
Just as above, the injections don't work for me with an alias, but do work with a class name.
Indent. Always indent for each HTML or Django template block.
Does stylish add a specific, desired trait that is not currently available (I guess I'm trying to figure what exactly a stylish browser based test would look like? Does she wear Prada?)? Could stylish mean user-friendly? Cleanly designed? Readable by a human?

For the second sentence, I'd re-phrase it to state that the first sentence is the goal you want to achieve at the Hackergarten. Currently it sounds like this is something that is almost/already done. 

Ex: One way to achieve this goal would be to abstract a yyyy something (what's the next technical description of layer?)  of Thucydides and run it with Arquillian Drone...
@apex13 Wouldn't it be better if the method name was `save_language`?
@apex13 We should use a mixin for this.
http://compass-style.org/reference/compass/css3/gradient/
Because it's needed to work with firefoxos :\ so in dev. version by default could be adjusted as non-ssl

Also can be changed in each machine ...
Oh wow, totally missed that. I guess it is fine then.
> I stumbled upon this post which discusses our very example :)

:-)
getCompileArtifacts() is deprecated... do you know which method to use instead? We have several warnings due to this method...
I find this list with items for parameter *and* return value validation a bit confusing. Actually there is no kind of validation routine which validates parameter and return value constraints, it's either parameter *or* return value validation.
Maybe two separate lists are better (one with parameter validation, cross-parameter validation and cascaded parameters, one with return value validation and cascaded return values)?
You only added this snapshot for the boot/kernel image. Did you mean to added it for the FS / swap space hard drive as well?
This may not actually be needed, the issue though is that if no month is selected then the input field gets a:

`0null/2012`  for a value

I could just use selectedMonth and default it the same way I am defaulting the defaultMonth.  Does that seem better?
もとのままでよろ
а почему с конца? мы же обсуждали всегда хождение с начала. чтение задом наперед работает медленнее, а главное --- это как-то через зад :)

почему тоже самое не сделать с начала на перед?
What's the difference between getLandscape as a method and getLandscape as a property accessor?
The last 3 statements will not work on IOS. Maybe put them under an if(android) condition.
In this case we may also want to log a `WARN` for the developer that they have set an inappropriate value for the property.
Good idea. Changed name. DONE
Context is important but it's dangerous to tie this to device type. Our data shows us people access the site on mobile & tablets throughout the day and therefore in a variety of contexts. I'm not saying the kind of need mentioned is redundant, just that device detection makes a lot of assumptions.
Why do you specify these callbacks (onShow and onError) if they are not being passed in the options parameter in any test?
Should be in application API in the webplatform
is it skipped or abstained? Because there seems to be a fourth group of people that are not present or are present but could not be bothered to press the "abstain" button.
This is annotation is why you are not seeing any results returned.  You need it to be HttpStatus.CREATED.
split into two tests
This import is not needed anymore
No luck, I keep getting fatal errors stating I'm missing the Traversable interface:

Class Sonata\DoctrinePHPCRAdminBundle\Tests\Datagrid\NodeMock must implement interface Traversable as part of either Iterator or IteratorAggregate

tried: 
abstract class NodeMock implements \PHPCR\NodeInterface, \Iterator {}
and 
abstract class NodeMock implements \PHPCR\NodeInterface, \IteratorAggregate {}
and even
abstract class NodeMock implements \PHPCR\NodeInterface, \IteratorAggregate, \Traversable {}
You need to use `fudge.verify()` at the end of the test method to actually verify that `get_preset_thumbnail`  was called.  Otherwise `expects_call` should be `provides`.
Fixed it to be {global,$env}.config.php now