labofsahil/github-event-dataset-2012 huggingface dataset:
According to the HTML5 spec it's not required: http://stackoverflow.com/questions/7048833/does-html5-require-an-action-attribute-for-forms

I don't know if we have a webdev guideline, but I prefer actionless and would like to keep it that way on Affiliates (IIRC this is not the only place Affiliates has an actionless form, so consistency rules here anyway).
Nit: ClickStats is the actual object (and you can computer word it with \`\`ClickStats\`\`).
By now I think you mean local. I can't find another commit yet.
Couldn't we just move the removeTrackEvent call to before this if function check? Or is that going to cause sync issues.
Simplified rendering of partials - render 'hidden_fields', :form => f

(They explain it better than me: http://guides.rubyonrails.org/layouts_and_rendering.html#using-partials
and
http://archives.ryandaigle.com/articles/2008/11/20/what-s-new-in-edge-rails-render-stops-being-high-maintenance )
だらだらと長い. 原因はいくつかあるが，以下の点について再考するとよい．

- tweet_id は nil を取る可能性があるのか?
- remove_noise(nil) を評価する必要があるのか?

```ruby
def find_source_tweet(tweet_id)
  return unless tweet_id
  keyword = twitter_client.status(tweet_id).text
  remove_noise(keyword)
rescue Twitter::Error::NotFound, Twitter::Error::Forbidden => e
  nil
end
Fixed in a6dcdec.
This sorting is working too. This sorts for the Show pages. The controller only has a sort for the Index. This is for all rows with values, index uses all rows that don't have values.
https://github.com/freerange/mocha/blob/master/RELEASE.md indicates 0.13.x doesn't need monkey-patching any more, but doesn't indicate which minimum version of minitest is required.  We could probably change mocha to >= 0.13.0 then, if that works for ruby 1.9 with a "recent" version of minitest.
@irrationalfab It’s a bit tough to understand, but I see why you do it. It should be good, at least for now.
Two Questions:  

We currently have three button colors? I only see references to two of them here I think - the white and white/red version.

Should we !default the button colors, this seems like one of those elements that someone might want to customize and easily could.
Seems to look okay to me. They're close, but doesn't seem too bad (unless you're seeing something different).
This is the new logic in LazyCachedNode that uses the new `supportsGetChildReferenceByKey()` method, and if yes then does the old logic. If no, then it goes directly to the DocumentStore (and to the Connector).
Yes, so that in future we don't risk to have too many arguments.

Ciao!
   Michele
I've also emailed 1and1 requesting their help on giving a clearer explanation on the issue from Share. This might help writing a better description in the INI file.
изучаем do {} while
allllright we'll put it back in. JUST FOR YOU
Ah, good point. It's currently using `local.json` in the root folder of the repository. I've added another, test-only configuration file and updated the path accordingly.
working on it. player word main-character
О, именно так, как я жду :) 100% будет
Формула из обсуждений проекта, насколько я помню предложена студентом Sachs.
Вообще, так лучше не делать. Есть правила, по которым создают и используют объекты разных классов. Будьте уверены, что когда напишете свой класс и отдадите его другому программисту для использования, он будет инициализировать объект методом init, так как это конструктор по умолчанию. Его вам и нужно было переписать "под себя".
я как раз об этом и сказал, sry если не понятно выразился...
вместо этих двух строчек
self.toggle(:is_right)
All of the other messages validate (and document) the parameters. Any reason not to do the same for this message?
It might be a good idea to check `application` against null here for completeness, but other uses of applicationByConsumerKey() in this file don't, so I didn't either. We can probably safely assume that well-written providers will only set `application` to null if `err` is non-null.
None is regular lattice connectivity assumed by ndimage, we should mention this.
same here. Put default arguments in the function signature
Worth mentioning to translators the text is about searching the site.
Please don't advocate removing the list entirely, as it mean AsseticBundle would scan all bundles for assetic tags, which is slow
Please don't change that neither. I could be please to add you as a contributor but not as an author (an author in general don't change).
How about “encode delayed operations”
`@param` and `@return` are not documented in a global function
Пробел в начале (и в ошибке 311 тоже).
А ну и хрен с ней тогда.
А, гм. Тогда лучше явно укажи `Qt::QueuedConnection`, чтобы сразу было ясно, зачем.
The file you switched this to doesn't exist.
Just be all "Yurts!" She'll totally get it. Yurts are universally hilarious
:)

On Wed, Dec 5, 2012 at 9:53 PM, nellshamrell <notifications@github.com>wrote:

> In week7/homework/questions.txt:
>
> > @@ -3,7 +3,27 @@ Please Read Chapters 23 and 24 DuckTyping and MetaProgramming
> >
> >  Questions:
> >  1. What is method_missing and how can it be used?
> > -2. What is and Eigenclass and what is it used for? Where Do Singleton methods live?
> > +*It's the method that Ruby looks for after it can't find the method that's been called anywhere in the class heirarchy that it was called in.
> > +
> > +
> > +2. What is and Eigenclass and what is it used for?
> > +*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.
> > +
> > +2.5. Where Do Singleton methods live?
> > +*In yurts.
>
> Kelsey just came running in "What???"
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/UWE-Ruby/RubyFall2012/pull/148/files#r2331250>.
>
>
Check file dimension (if applicable)
Mainly just a convention thing for me and takes care of any global before/after blocks if their needed :).
yeh, good catch once again, I've changed description of the wrong test.... Corrected on my side already but didn't re-submit since started to look into this equality thing (see my previous comments).
Yep, I figured out why a double login would fix it. There was a problem in my setSync() function that would spontaneously cause an incorrect sync time to be parsed. I think the problem comes down to Java's substring() function that when given certain strings will take off 5 bytes rather than the strictly required 4 bytes. Of course it could also be a char[] to String conversion issue. Either way I've figured out that the sync string isn't anything fancy, and the system time can be used instead (as long as our phone's internal clocks don't get off in time).
`This feels wrong`

DOES IT?!?
can the throwable ever be null there? AI guess the only situation might be when the passed in argument is null, in that case I'd rather explicitly throw an NPE and don't support null args.
Also this is a general purpose utility, so might make sense to have it as a static method in org.infinispan.util.Util. And an simple unit test would be nice :)
Same goes for this class. It is identical to the ```FFMpegWriter``` save the registered name and the class name. A simpler inheritance scheme could reduce the redundancy here too.
You are right. My mistake. Should be something like:

    raise AttributeError(attr)
I guess it is just an example, and if its not used in the example, then its not actually needed...
Reset everything because this is ASP.Net, where ALL behavior is undefined.
AFAIK the only consumer of mozmill is mozmill-automation/mozmill-ci, but reordering of the arguments is an API change.  We shouldn't make API changes just for the sake of readability.
'no longer present'
yes it's a good check :smile:
Auto translation again...
We compare ``cmorph.dilate`` to ``xxx`` to show how computational costs increase with ....
It's probably out-of-scope, but I think some kind of visual feedback that the data was saved or not would be nice
I changed this to a switch statement, it does make it easier to read.
also you don't want BC breaks
since you never use svm locally you can avoid the SG_REF / SG_UNREF below
Removed these comments.
Do you want us to put this back?
We're never clearing process_locked through this path, so we're going to unlock the spinlock twice.   I think you can just delete line 105.
Oh sh1t, I just recognized, that timer, we set in `sleep` (in `wait`) could wakeup that Fiber between call to `signal` and procession of `next_tick` :( so that we ought to rescue from FiberError here, or have other mehod to catch this case.
Converted to JIRA: DVPL-1318.
It's an issue with the test. Some rooms don't have bedTypes given, and the one the test used didn't. A real fix would be to find a hotel with rooms that list bedTypes.
BTW, does not Google Maps API something to bounce just few times? I think I remember this.
Define here default attributes:

```
language = None
source_name = None
source_url = None
```
Add to TODO at the end: "and do not use textarea content but use data from the server (it might be processed)"
Because I did so without harming backwards compatibility. Previously, `buildFragment` sought a document from the `0` property of this argument, or used the global `document` if that didn't work. After this change, it still does that but is *also* able to just accept a document directly.
Should we instead, add a boolean parameter for whether it's in an unwind? is that possible?
`$object` is a bit too generic, maybe `identityClassName` ?
> Yep, backticks are not allowed in the compiler, and ideally shouldn't be used in any of your code either.   There's no need for them. If there is, it should be an open issue.

What about problems that are a closed issue but are still needed?  I have to use `var x;` in many places..
Thinking about catching the exception on redeploy inside the loop (and rethrow it later?) to update as many deploy targets as possible!? Or leave it this way for fail fast behavior?
Mark as deprecated?
same thing here, importing should be explicit.
Based on PHP [PSR-0](https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-0.md) it's recommended not to have that closing tag
Yes, it only makes sense if "td" or "th" is the first item of the list. The reason why it's not two function is that I can do e.g. 
```javascript
   if (!req.request_headers)
    ret.push(ui_strings.S_NETWORK_REQUEST_NO_HEADERS_LABEL);
  else
    ret.extend(this.headers_list(req.request_headers));
```

and later use `map` with that function. headers_list can have either returned a `pre` with content (raw) or a `[[th], [td]]` row (parsed).
I would let 'return_value' take 'return_values.rt_id' and 'search_term' as first arguments and then use bind instead of a new inline function.
```
>>> np.__version__
'1.5.1'
>>> np.arange(5).ravel(order='F')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: ravel() takes no keyword arguments
>>> np.arange(5).ravel('F')
array([0, 1, 2, 3, 4])
```
I agree, didn't see it's only the denominator before
不创建 -> 不编译
Name it maybe "home"?
`<b>` tags are deprecated, please consider using `<strong>`tag instead.
A point that came up in Reiko's review of random.py, which is relevant here.  The `dev.applyTo(image)` syntax isn't really intended for use by the end user.  The `image.addNoise(dev)` is the preferred syntax, since the latter works for both Image and ImageView, while the former only works for ImageView.  So I'd remove the reference to `applyTo` in this section, and instead just forward reference the `image.addNoise` method here.
Good point; fixed and pushed.  (For reference, since you're newer to python just like I am, I will mention that in that first line you need a \ after the %, otherwise python is not happy.)
Have you considered also mentioning the [spring-amqp](http://stackoverflow.com/questions/tagged/spring-amqp) tag on StackOverflow?
Yes, my testing indicates that it works fine. The chef step should just become a default package install of django_storages and python-gdata I think.
Perfect - I've switched over
personally don't like these convenience methods here -- they belong in Entities or somewhere
Ditto, can do:
`log.debug "{} deployed {} to {}: {}: result {}", entity, f, hostname, dest, result
Just the variable names - but wrongly. I've fixed it now.
did not tried it but I think a logical step to reload that class after we click_create_profile_button
Whitespace introduced. Please ensure that there are no trailing spaces on any line :)
Please can you indent like we had? This is very hard to read.
to match with testing style already established in the gem, please use `DataMiner.unit_converter.must_be_nil`
Is this really XML fragments, or does it return a well defined XML document? I thought it was the latter.
Oh yes. Some Common Lisp.
this curly brace should be on its own line
Probably better to just do:

if (secs < 0)
  secs = 0;
``veritcal``? :)
4 spaces not 8
Can you add a space either side of this `!` and instead of putting the whole controller in an else just put a `return` under the template line, so the else can go away.
And that :P
This file was mostly me burning the heretical "src."'s everywhere.
powder.h declaration of protons
`if (Object.isFunction(logCondition)) {`
Are we sure this is safe when using 10 channels and the ca
A bit outside this scope.  I'd rather have this discussion in the signal routing thread at PP forums.  I'd love to get to the point manual control doesn't have information like "best climb rate for autonomous mode".
wasn't sure if this should be
``` ruby
@data = @resp
Ooh, that's already done? Great :-)
Не нашел у него метода проверки на то что существует то, на что он указывает - только существование следующего. Может я что-то упустил.
Не знаю, зачем notDelete нужен, но есть ощущение, что это как-то не очень прямой способ не удалять что-то откуда-то.
```
InputStream reader = null;
try {
  reader = new ...
} catch () {
}

readFrom(reader); // исключение может произойти внутри readFrom, причём не только IOException, которое ты перехватываешь, но и RuntimeException, но и другой Throwable
reader.close();
@ctweney makes sense, thanks for clarifying
Any idea why we are doing this sizing in code instead of in a stylesheet? I'm feeling uncomfortable with this whole 'resize' function and I think it might be the time to reconsider if we want it at all.
s/isn't/aren't/
blank line contains whitespace.
You should really only ever have one QDeclarativeEngine; they're not cheap and it does not make any sense to mix them in the same scene. Generally you want to create everything - other than the main QML component - from inside of QML, so QML can control all of the UI.
Better use `Py_ssize_t` here. (At the moment nearly all cython code in skimage uses `int` here, but `Py_ssize_t` is definitely better)
Do we keep the `monitor` mechanisms? It is a very good idea in my opinion, but maybe out of the scope of this PR. I think we should discuss first of a common monitor API to use within all scikit-learn.
You can use the built-in function `repr` for that.
I tend to prefer the more recent Cython idion: 'from libc.math cimport exp, lgamma'
не завадило б винести рендер списку шпалер в окремий шаблон. цей код повторюється на головній, в категорії, в юзера, тут
Possibly stupid question - but out of interest why are we using dataFilter - can't we just request format json (in which case the success callback gets JSON and we don't need to worry about JSON.parse )?
It would be good to add a test case to ensure this is not broken again, does anybody know a simple example that triggers this check? I'll try to figure out after fixing it but if somebody more familiar with the code saves me the work it will be appreciated :)
New code should not be using the `data` member of arrays.  Use `(*args)[0]` instead, it's typed correctly.
@jmikola I am happy with this being removed, I don't think we need to enforce specifying the defaults. It looks like just removing the should be enough since they are passed to the Elastica_Client as an array anyway
Did you mean to rename namespace? (or function)
Creating new helper file and keeping this kinda utility functions was suggested by David C.
We can move the get(TIObject* ...) functions from NativeUIControl to here, or add to do for later re-factoring.
Maybe, but i made that to be able to reference it from processor (https://github.com/excilys/androidannotations/blob/129_GenerateApiHelperClasses/AndroidAnnotations/androidannotations/src/main/java/org/androidannotations/processing/SharedPrefProcessor.java#L291)
Same problem as with `TextChangeProcessor` here (fixed a few days ago) :  these event processor must happen before `@AfterViews`
Ah, here it is... It's better for comments to be on the declaration of the class and not the implementation, so people can easily see them when following the symbol (and some IDEs can show on mouse-over).
Wuid was moved from old line 44 to new line 39.
it'd simplify the URI change if we keep those things to a minimum.
зачем will_paginate и kaminari вместе? Они же дублируют функционал друг друга?
same as above, please set default as `null`
@shacker Can we link to http://newscenter.berkeley.edu/category/news/ here?
@shacker The second link should explain this.
How is a notification summary different from (issue comments)[http://developer.github.com/v3/issues/comments/#list-comments-on-an-issue) and commit comments? Could we just link to the appropriate comments list from the notification rather than introducing a separate but conceptually very similar resource?
The main problem that I ran into when writing this is that some awards had sponsor names, the sponsor might have changed, the award name might have changed slightly, but the award itself is the same.
Though I can see why it would be beneficial to allow "unknown" award types.
might be a crazy idea, but why not use a QList<int> and pass QContactDetail::ContextHome and friends directly? QML can be told about enum values afaik
Stack-based buffer overflow in VanDyke Software AbsoluteFTP ....
indeed, I'll update the PR, thank you
I removed "computes" ;)
I think this would read better if you moved this into a variable called `swirled`, and then call `swirl` to unswirl it. ( That sounds a bit cryptic, but I think it makes sense.)
Yes - sorry, didn't really know it can contain much more - never used it for that.

Lets then use your proposal with naming it "local" and using the ENV. :)

Shall I update the pull request accordingly?
Formatting needs to be fixed here
[Ordering here should be](http://mozweb.readthedocs.org/en/latest/coding.html#import-statements):

    from django.conf.urls.defaults import include, patterns, url
`this` here is the context object where `EventHandler` is called. It can be `window`. I don't think returning such an unknown, and possibly unrelated, object is a good idea.
On Android, as it turns out, EMBEDDED == FULLSCREEN == DEFAULT. There is only "show controls" or "don't show controls".
This function is only called from within Backend, right? Then it shouldn't be public, but protected.
Wrong! This is a horrible feature.
это значение и так будет по умолчанию
Why not just do `foreach(i; 0 .. n)`? I don't know if there's a performance difference between the two, but the `foreach` seems clearer and more idomatic to me.
```
[[1 2 3]
 [4 5 6]
 [7 8 9]]
```
*ping*

Unfortunately it's not possible to keep the lock due to how advisory locking works:

> If you have a lock obtained with GET_LOCK(), it is released when you execute RELEASE_LOCK(), execute a new GET_LOCK(), or your connection terminates (either normally or abnormally). 

See here: http://dev.mysql.com/doc/refman/5.1/en/miscellaneous-functions.html#function_get-lock

So no matter how our ruby process ends the lock will be removed because its connection will be killed.
are you sure ?
why is there no option to make_installdir to allow what you are trying to achieve?
Need to adapt the email
I don't think this is right.  See `GlobalSession.enabledEngineNames()` and document what null means here.
Please differentiate between `enabledEngineNames` (existing code) and `selectedEngineNames` or whatever you want to call them.  This is super confusing.
Is this file needed? Why is it empty?
Be consistent the way you write the terms "Java bean", "@Model", "runtime"
no commas here
Not sure if we should depend on this package!
Could you change this to 5.3.2?
Looks like we can just do `uploadProfilePicture(user, SERVER_URL, callback);`
oh hi, sorry, was looking at this earlier and got distracted. it feels a little weird to bind all of the job data. probably better to bind this in the job definitions. write it out a few times in the job definitions themselves and then we can write a macro to hide the binding. really simple macro, a good test piece to advance understanding of them.
Worked fine on my MySQL instance
Script will crash if 'request_content_param_converter' key will be undefined won't it? ;-)
The logic for this is also wrong, because this concept doesn't mean what you think it does. I'm going to rewrite it to use the most recent encounter instead.
What about something like this:
```
                    var typeClass = Type.GetType(typeName);
                    if (!typeClass.IsInterface && !typeClass.IsAbstract)
                        instance = ReflectionExtensions.CreateInstance(typeClass);
```
It wasn't used anyway. I am planning to improve error handling soon, however.
This is moar better.
Drop the `$` prefix, perhaps `var elem`.
oh, sorry for that...
This concerns me as we're scanning all of the resource keys in the catalog for each component in the registry key path. Shouldn't we stop at the first match?

With that said, I think the whole issue could be resolved by simply downcasing the paths at munge time. We already save off the subkey (in its original case) and use that when create/open/destroying keys/values (so it will be case-preserving).
@guilhermeblanco you were using the BC (deprecated) RegistryInterface of the bridge. I wanted you to use the ManagerRegistry interface available in Doctrine Common, not the ``Symfony\Bridge\Doctrine\ManagerRegistry`` implementation of ``Doctrine\Common\Persistence\ManagerRegistry``
Update the changes here to conform to the style guide
Why are these fields const instead of immutable?
this is wrong. ``:method:`` is used to link to the Symfony2 API doc, not to the Doctrine one
This should be all on one-line (`"""A base class to authenticate and work with Marketplace OAuth."""`).
Since I am not showing the view controller myslef, I did not want to dismiss it either. But I agree this one is debatable. Josh? Randall?
@apex13 Shouldn't this id change to something else? I see it twice in this document.
Sorry, that should have been a Yes. s/Nope/Yes/.
shouldn't this be `[self downPoint:tapPoint]` (rather than `upPoint`)?
collect looks interesting, I'll have a play...
I'm ok with copying copyrights. :)
What is the use of the @test annotation? Is it a PHPUnit annotation?
an prototyped array node is always set as it has a default value (an empty array)
This code looks quite like the one above, maybe you can make a function out of it? 
In python, you can define them nicely in a local scope (can't do that in java/c++ this easily). This could also help structure the code more, it's a lot of code where it's not immediately clear, what it does. So maybe you could also add a few lines at the beginning at this whole block explaining what all this is about, such that it's not necessary to actually read the code to be able to maintain it.
You do not need to define options here if not using.
Must it shall be
!!!!
naghshe in br chiye ?
Yep, it does. We are good.
We can't edit the original FSS files: they come from the Infusion package. What we'd need to do instead is override this setting inside the idi theme file somewhere, so set 'overflow' to the default somehow.
Nit - I know this class isn't something users can access, but I usually still like to add parameter validation via Debug.Assert so it doesn't slow down production code but you make your constraints clear (and get more specific debugging help when things go sideways).
I've been working on the implementation of this, and I think using
GerritTriggeredEvent with a fully unique hashCode as the key is more
like the behaviour I expect from the plugin.

If we use GerritTriggeredEvent as the key but create custom hashCode
methods, builds from separate events do indeed get attached to
different MemoryImprints, and Gerrit will receive separate reports as
expected.  But if a second event of the same type arrives (and thus
has the same hashCode) we'll still have the same confusion you're
worried about.  The build may be attached to the previous
MemoryImprint, overwriting a previous build of the same project.  Or
the second event may trigger several jobs, some of which get attached
to MemoryImprint 1, and after MemoryImprint 1 is complete, the
remainder get attached to 2.  Then a full report from MemoryImprint
1 is given to Gerrit, but MemoryImprint 2 only reports a subset of
the jobs that actually ran.

This isn't possible currently because you can't have multiple
patchset-cretaed events, however you can have multiple comment-added
events that effectively look the same (apart from timestamps).  In the
OpenStack project, we watch for comment-added events (with specific
code review votes) in order to trigger testing of changes, and if they
all pass we submit the change and merge it into trunk.  If the last
set of jobs Jenkins runs contains only a subset of the intended jobs,
we may end up merging a change that failed tests.

Here's an example from my testing:

  https://review-dev.openstack.org/#change,370

It shows (starting at 12 past the hour) three approval votes
(comment-added events) in a row (a, b, c).  You can see that the job
comment-added-2 build 37 replaced 36 in one MemoryImprint, and also
that the job test-all build 89 ended up on its own MemoryImprint.

So here's an alternate proposal: use GerritTriggeredEvent but have
compareTo use a unique hashCode (eg, java.lang.hashCode) rather than a
type-specific hashcode.  That models what I would expect out of the
plugin: that each event from gerrit will trigger a set of jobs, all of
which run to completion, and the full complement is reported to
Gerrit.

I'd like to include one other change to ensure this behavior: make
sure that the build parameters are unique even for very similar
events.  Otherwise, if multiple (at least three in this case)
comment-added events are receieved and the gerrit trigger plugin
schedules all of them, the first one will start building, the second
will be queued, and the third will not be queued by Jenkins because it
has the same parameters as the second.  By including a parameter that
is different every time (perhaps the hash code of the event), all
three will be queued and run.  This shouldn't normally be an issue,
but solving this problem will mean that all of the jobs the user
expects to run will run for each event.

Here's another example from my testing illustrating this:

  https://review-dev.openstack.org/#change,372

It shows all three jobs running in response to all three events, with
the full report for each event.
add an if here for each page to build the reinformcement image file names
Например? Я сейчас добавляю простенькие алерты, которые будут сообщать о загрузке/ошибке. Алсо, как назвать саму настройку? `'add4chanSnd'`.
> I'm using it for integration tests in APE [my fork] and I don't have jta
> in the classpath - looks fine :)
>

Excellent! :)
Missed this, thanks.
Unit test has been added to the test/unit/lib/overlayWebView.js
where are these values defined from?
would this file make more sense under the pps folder and also be called pps-network or something?

I don't like polluting the root of lib with interfaces that are single purpose.
Need an integration test for this method.
this whole class should probably be deleted
Is it possible that Request or Request.Query can be null ?
Can you change this to:

```LOCAL_MODULE := lights.$(TARGET_BOARD_PLATFORM)```