labofsahil/github-event-dataset-2012 huggingface dataset:
apologies about the Supplier Server vs Provider Server dance.  I think the latter is the cleanest impl
The prefix attribute should be sent to the template as well, but it's up to you to use it.  Hmm.. I guess I didn't document that one.
The below was adapted from this code:
http://stackoverflow.com/questions/4438244/how-to-calculate-shortest-2d-distance-between-a-point-and-a-line-segment-in-all
da hab ich euch ganz gut getroffen oder? ;-)
This commit breaks the minimizing of the toolbar in firefox 7.0.1 (linux).  So the 0.9.2 release is broken for me.
fixed: https://github.com/innoq/iqvoc/commit/cfa49ce1d2cef4c06838ad6f44bd55348d8b2224 (no pun intended)
these changes causes the following exception:

Fatal error: Uncaught exception 'Zend_Cache_Exception' with message 'cache_dir must be a directory' in .../library/Zend/Cache.php:209
Stack trace:
#0 .../library/Zend/Cache/Backend/File.php(154): Zend_Cache::throwException('cache_dir must ...')
#1 .../library/Zend/Cache/Backend/File.php(121): Zend_Cache_Backend_File->setCacheDir('...')
#2 .../library/Zend/Cache.php(153): Zend_Cache_Backend_File->__construct(Array)
#3 .../library/Zend/Cache.php(94): Zend_Cache::_makeBackend('File', Array, false, false)
#4 .../library/Centurion/Config/Directory.php(70): Zend_Cache::factory('File', 'File', Array, Array)
#5 .../public/index.php(42): Centurion_Config_Directory::loadConfig('...', 'development', true)
#6 {main} thrown in .../library/Zend/Cache.php on line 209

I use an up-to-date version of Centurion with default cache.ini file
Thought I'd mention that including armeabi-v7a (by the 'all' method) should boost performance quite a bit when dealing w/ floating-pt numbers (namely Lua & ESPECIALLY the physics libraries) since armeabi does all its floating point math via software emulation :(   armeabi-v7a adds other things, but most importantly hardware floating-pt support.

Hope this gets pulled into the next release.
replace LLSDSerialize::fromXML(mLLSD, import_file); with S32 status = LLSDSerialize::fromXML(mLLSD, import_file);

and remove the duplicate, it only needs to be parsed once lol
I was also seeing occasional timeouts for the tests. I noticed that when I got timeouts, the tests ran for > 15s which is the whiskey default timeout. After I have increased the timeout for whiskey I have not seen timouts any more.

The test `testPooledConnectionLoad` is inserting 3k records, so that can take some seconds....
Changed $ to jQuery to make code cope if prototype is running as well, added functionality to replace the correct missspelt word in the text with the one you select from the list (if there are multiple instances of the same missspelt word), added the option for diferent types of spell checker to use.
watch out for this typo, it's corrected in the next commit.
you need to update your gemspec since you removed schema.rb. I get the following warning rom bundler:

    devise_rpx_connectable at /Users/emkman/.rvm/gems/ruby-1.9.2-p290@stickyleaf/bundler/gems/devise_rpx_connectable-e475bd5d7129 did not have a valid gemspec.
    This prevents bundler from installing bins or native extensions, but that may not affect its functionality.
    The validation message from Rubygems was:
      ["lib/devise_rpx_connectable/schema.rb"] are not files
Fix it, less QQ and more PewPew
I am not so sure whether I like this approach. I understand that we have problem here if we want to have a clean separation, but there a couple of things I dislike here:

* I find having _META-INF/services/delegate/javax.validation.spi.ValidationProvider_ confusing from a developer and user perspective
* We are reading even more files via the class loader. I am afraid this could fail on some systems. AFAIK, not all OSGi platforms make for example the META-INF directory accessible. 

WDYT?
son of a... crap.. thank you I will fix that :-(

On Wed, Mar 7, 2012 at 12:47 PM, Scott McClaugherty
<reply@reply.github.com>
wrote:
> You just deleted the code for my comment macros.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/gamefreak/Reddit-Enhancement-Suite/commit/90cda982613181cb01bd719c421331423cc3600f#commitcomment-1054976
Wow, that was swift work. I just forked it so I could send you a pull request to do just that!
`<code>` blocks are easier to read
Great you already included this one. :) It has been my next recommendation.
Yeah, you should be able to set Excon.defaults[:mock] = true prior to the tests in question and then set it back to whatever value it might have been before (likely false, but not necessarily) afterwards.  You can also use Excon.stubs.shift to remove any newly added stubs and return stubs to the state you got them in.
Should be ThumbnailListItem
Classes should be autoloaded, no?
Yep, it definitely doesn't belong here. But couldn't come up with a better solution.
311 really isn't the appropriate status code for this. It should be something in the 4xx range.
Will apply.
Should also be enforced by type system
You might not want to do plain ".get" as you'll want to escalate it as a configuration problem, see how the other extensions to it (they use recover with rethrow of wrapped exception)
is this java.lang.Iterable? If so, import it at JIterable so it's obvious that it's not scalas Iterable.
Big fat deadlock warning?
this calls `provider.tempPath` while spinning: wasn’t `Registering` meant to be used to prevent wasting names like this?
Because the HeapMemory.unapply returns an tuple in this to make it look like an ordinary case class extractor when used from Scala. This was what I talked to @rkuhn about yesterday.
as you see the constructor is private, and it can't be negative, except if you have negative durations, which I could add check for, if we like the defensive style
I think we can do away with this config option by by just specialcasing "0.0.0.0", no need to introduce yet another invariant.
Agreed; though it would have been a smaller fix without volatile vars to just put the barrier into the original code. The sendN/expectN is an improvement regardless.
Coding style convention that we follow suggests adding spaces after commas (as you would do in normal sentence in English) and surrounding arithmetic operators with spaces, see http://python.org/dev/peps/pep-0008/#whitespace-in-expressions-and-statements
this looks like mhrivnak influence
Ok, let's just say this whole file needs a gettext audit, and I will withhold further gettext comments. :)
Sure, but we could integrate everything before these last updates first, then the cleanups.

My point is that are build sequentially on the branch, so it isn't a problem if my want to split them.  So the line could would only apply chunk by chunk.
has to be configurable
No need for the empty string initialization.
new name: get_job_flows(options)

fixed.
Make it private and non-static
Maybe we could add a parameter to TaskAction to decide either if this call block or not. I would not like to block a thread that could continue safely.
Yes, header type should be decided by  **first group** #s
What do you mean? I agree that resetTimer should not be called on editor scroll - this is waist of cpu.
In other cases resetTimer is really called when we need stop blinking(so we need remove ace_smooth class, and show cursor ) and start again.
Shouldn't be `$test = new Twig_Test_Callable($name, $options);`?
You brought this on yourself! ☺
Do you want to re-rename?
This should be two lines, within braces. (Or, even better, it should be moved into the behavior of a `setHideKnobTimer:` setter.)
Can you point me to an example?
Move this to node_internals.h.
Changing it this way may break existing users' code, which is unacceptable if we're going to pull this PR into 0.8
Maybe say "Asked to remove block <blockId>, which does not exist", so users don't just see an unexplained "block does not exist" message.
Please change this to...

              <tasks>
                <property name="spark.classpath" refid="maven.test.classpath"/>
                <property environment="env"/>
                <fail unless="env.SCALA_HOME" message="Please set the SCALA_HOME environment variable and retry."/>
              </tasks>

I was using the CentOS 6.3 scala package which doesn't set the SCALA_HOME env variable.  It took me a while to realize what was happening so I hope this simple  check saves others the trouble.
This should be "true" all the times r I'm wrong ?
Hi @ahocevar , this is what I tried to do originally - xtype of gxp_editorgrid, config.push as an item, etc.  But I couldn't get it to work this way.  FeatureEditorGrid has only a ptype, no xtype.  I tried using "ptype: gxp_editorgrid" instead, but got exceptions (object.render undefined).  Maybe I missed something obvious?  Adding it as a plugin, like FeatureEditPopup does, was the only way I could get it to work.
jhrtrhjkthkjtet

```python
print 'fgfgdfg'
```

```ruby
puts 'fdsfsdfds'
```

```javascript
alert('ok')
```
`cacheDuration` ---> `statement.cache.expires`.
ну вот и вынести это в константу
`to to` => `to be`?
Yes, one space after a period, please.
Can you change this `nil` to `Time.now - 5`?
The new coding standard does say to indent after private, though.
Sounds ok
Ah, alright then. I didn't know about one :)
Subtaxon should be probably smaller than main taxon. i guess that is why it was H5. Not only be size but also by meaning IMO.
We could add the `_as_real_imag` splitting method. (You can compare to the implementation in `sin`.)
assumptions fail to detect odd

```

>>> var('n', integer=True,odd=True)
n
>>> (-1)**(n**2 + n)
(-1)**(n**2 + n)
>>> _.exp
n**2 + n
>>> _.is_odd
>>> var('n', integer=True,odd=True)
n
>>> (-1)**n
-1
```
I wasn't changing style (much); that's what was there, I just added to it.
Noted. Will be fixed.
Looks a bit off, doesn't it? But it's in fact needed: Models won't save properly otherwise.
I'm not sure I understand why you're doing this, rather than using the single-wire support that the UART offers.  Is there more missing here?
Does this consider associations? If all fields are the same but an association is added/changed the generated code still changes so we probably want the hash to change.
This is a matter of style, but I strongly prefer full words to abbreviations. This makes the code much more readable :-)
replaceEntry is the common code for three specific use cases: history
replace, pushState replace and reload.

updateLocation is used to tweak the actual document location of the current
entry, and is supposed to deal with redirection. If updateLocation is
broken, it needs to be fixed.

On Monday, December 17, 2012, Dave Dopson wrote:

> In lib/zombie/window.coffee:
>
> > @@ -351,6 +349,15 @@ loadDocument = ({ document, history, url, method, encoding, params })->
> >            return
> >
> >          window._response = response
> > +
> > +        # DDOPSON-2012-11-12 - we do history.replaceEntry here so if we get redirected, the browser's url reflects the last url in the redirection chain
> > +        # This is important because if we load A which redirects to B, which is a form, and the user submits to C, the 'referer' for the request to C must be B, not A
> > +        # Lack of fidelity in this behavior would block Facebook OAuth login scenarios
> > +        # DDOPSON-2012-11-13 - Note that we must also call history.replaceEntry BEFORE processing the document
> > +        # or else relative path linked assets will be loaded incorrectly for redirected responses
> > +        # eg www.facebook.com/login.php -> mysite.com, and mysite.com loads /assets/mysite.js.  This would incorrectly resolve to www.facebook.com/assets/mysite.js
> > +        history.replaceEntry(window, response.url)
>
> The problem is that history.updateLocation does not alter the history
> entry. history.replaceEntry calls history.updateLocation, but it also
> updates the URL in the top history entry.
>
> Changing this to history.updateLocation, the "form redirect" test case
> fails because the "referer" header points to the requested URL instead of
> the final redirected url.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/assaf/zombie/pull/454/files#r2440954>.
>
>
I would change `path` to `url` to be forward compatible with hostname support in the routing
Google CDN?
зачем гигантская копипаста?
когда можно было просто logging манкипатчингом навставлять где надо.
Le 15/03/12 16:15, jp-ganis a écrit :
>> @@ -133,9 +132,6 @@ function parseSpellTable( hydraTable )
>>   		conditions = spellTable[2]
>>   		target = nil
>>
>> -		-- conditions default to oncd
>> -		-- if conditions == nil then conditions = true end
> why are these lines removed?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jp-ganis/JPS/pull/95/files#r563302
>
because you have out nil conditions in fct conditionsmatched
and when these line

if conditions == nil then conditions = true end

is read before a nested condition in a table, I don't know why but the conditions remains true inside nested table
reason why the priestdisc was casting dispelling instead of healing
' dispelling is the first nested able

if I remove these line , rotation disc priest is working fine
@jpslav I believe we have to change this one? Never heard of list manager, and list member is not exactly assigned to a question, but rather to a list.
`Option(view)` is nicer than `if (view != null) Some(view) else None`
add note about this being dependent on driver implementation.
possibly rewrite as if (monthly_reports or mentees_reports or all_reports) ?
I don't know how this whole resource will work on Windows, but the readlink system call certainly won't...
This entire code needs to be taken out.
Move filename cleaning into a function.
Point #3 in the comment is the reason why there is no else, what are your thoughts on it?
Includejs?
Hum... You're right... I wonder why that got there...
```</script>``` in new line, i.e.:

`//]]>
</script>`
If it works, then I suppose it's fine. I was just going by what I've seen
done elsewhere in the code. I am not completely familiar with all the
bit-related stuff and will need to look into it eventually. For now, as
long as it works it should be fine.

On Sun, Jun 3, 2012 at 12:29 PM, brunoais <
reply@reply.github.com
> wrote:

> > @@ -227,7 +227,7 @@ function main($id, $mode)
> >                                       'REPORTER_NAME'
>       => get_username_string('username', $report['user_id'],
> $report['username'], $report['user_colour']),
> >                                       'U_VIEW_REPORTER_PROFILE'       =>
> get_username_string('profile', $report['user_id'], $report['username'],
> $report['user_colour']),
> >
> > -                                     'POST_PREVIEW'                  =>
> bbcode_nl2br($report['reported_post_text']),
> > +                                     'POST_PREVIEW'                  =>
> generate_text_for_display($report['reported_post_text'],
> $report['reported_post_uid'], $report['reported_post_bitfield'],
> OPTION_FLAG_BBCODE | OPTION_FLAG_SMILIES, false),
>
> That: OPTION_FLAG_BBCODE + OPTION_FLAG_SMILIES
> Is only equivalent to:
> OPTION_FLAG_BBCODE | OPTION_FLAG_SMILIES
> because there don't have not shared bits.
> I.e.
> OPTION_FLAG_BBCODE = 001
> OPTION_FLAG_SMILIES = 010
> OPTION_FLAG_LINKS = 100
>
> The bitwise OR is the standard(?) (at least the most logical to me and
> what's teached). I don't know about the specifics and which one is faster
> but the bitwise OR is what makes more sense to me. If there's something in
> the coding guidelines that I missed, I'll fix it.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/phpbb/phpbb3/pull/786/files#r919889
>
I think title should be required here as well.
Kinda seems like this belongs into `util/terminal` or `util/prompt`
use VIRGO_DEFAULT_BUNDLE_NAME_SUFFIX and/or VIRGO_DEFAULT_EXE_NAME_PREFIX/VIRGO_DEFAULT_ZIP_FILENAME
Wouldn't using it under the non-commercial license mean we can't use the qless server for any project that makes us money?

At the very least, it complicates the license you put on this project.  I don't think an MIT-licensed project can include code from a project like highcharts that has a non-commercial but non-MIT license.
Часовите зони са рошава работа като цяло. По принцип правилото е да ги съхраняваш в UTC и да ги показваш в локално време. Ето [една статия](http://archives.ryandaigle.com/articles/2008/1/25/what-s-new-in-edge-rails-easier-timezones) по въпроса, която обяснява нещата накратко. Малко е старичка, така че не съм убеден дали не е outdated, но не мисля, че някой е променял нещо по зоните оттогава.

Иначе, видях един ред в конфигурацията, в който настройваш зоната на "Sofia":

``` ruby
# config.time_zone = 'Sofia' # UTC +0200
```

Само дето този ред е коментиран, така че едва ли има влияние :). Има един метод `set_time_zone` в `ApplicationController` обаче, който би трябвало да слага правилната зона, така че не знам от какво може да е причинен бъга с различното време.
fixed again :)
Y23M!!
Let's call it significant whitespace then ;)
which `System` prefix can be removed?
Hm, one other problem, I believe. Right now you have one global var called 
```var keyMap``` which handles all key mappings. When you destroy the keyMapping, now, I think it only destroys whatever mapping was last assigned.

You'll need to allow multiple, separate keyMappings. Could change keyMap to a dictionary, so that you can have > 1 keyMap
```
keyMap = {}
```
and then ```addKeyMapButton``` (current called keyMapButton) adds an item to the dictionary and ```destroyKeyMapButton``` finds it in the dictionary and then destroys the item.
Eventually will want to abstract these out into CONSTANTS
yeah i think so, but it still working..
Blame @bjlittle ;-)
Docstring is not needed IMHO, a single comment would suffice.
"below"
Since it's so limited right now, I would suggest to move this whole remarks section into a single question in the FAQ, which would be called something like ``What are Theano's limitations?``. And it should say that this lists only *some* of the known limitations (since it's obviously incomplete).
set of variables or nodes.
Why this lazy import stuff? I looked at the import dependency and I think we can make a normal import at the top of the file. Is this a left over from previous version?
Reminder from when I was adding (void) to asprintf to avoid warnings. In this case it was easy to integrate checking its return value with nearby code. You should take it.
We search for all Assets the have DeviceInfo = we are searching for Devices. 
DeviceInfo lookup meaning wilil be - search for DeviceInfo only objects.
Apparently, this is not true.
this method could be split into two methods searchByLocation and searchByCategory.
And apparently those 2 years still aren't enough to fully grok the way attributes work. You're right, ``getattr`` should be mentioned.
missing one indentation level here
When a test requires cleanup of generated files, I like to surround the test code with a `try/finally`. That way, if you encounter an exception or your test simply fails on an assertion, you still do the clean up no matter what.

Here's an example: https://github.com/gem/oq-engine/blob/61c0fb390527fc2a5620d740562a90e6a4a8f02e/tests/export/hazard_test.py#L41
You're right—currently it only parses lambdas at the top level of an expression. It should be possible to lift this restriction, though it will make parsing slower because more subexpressions will need to be tried as lambdas.
I'll amend now, thanks for the feedback
Move the initialized, but not assigned to their own line - thanks.

```js
var posX, posY, glue, len, 
      ret = curCSS( elem, "backgroundPosition" ),
      i = 0;
```
Could you change this to `Minion_Task::compile_task_list()`? Seems that is our convention.
The uber-correct way would be using cmake commands like execute_process. I am not sure if this works if the user is using a shell we are not expecting.
Showing them that there's something like `meteor mongo` can be quite helpful knowledge. Good idea. +1
Do we really need netbeans related files to get in?
Why not reuse ``file:seach``? Is probably just as good/bad as having ``local``.
I originally used filter, but then I came across a bit of ES documentation that said

> There’s one important distinction to keep in mind. While search queries restrict both the returned documents and facet counts, search filters restrict only returned documents — but not facet counts.
>
> If you need to restrict both the documents and facets, and you’re not willing or able to use a query, you may use a facet filter.

Using a facet filter is normally taken care of automatically by EU, if you
do `S().facet(..., filtered=True)`, but it isn't automatic when doing raw facets, so it
would have required duplicating code.
Get a real editor. :P
`raw_search()` is the replacement. I'll remove the comment.
I understand this module is external, but still, where's the 1-var-rule :) Ah, I kinda feel strange saying this all the time :) I hope you don't curse me.
I dont find in what cases the cached response can be a Request object
the error is: 

```
Warning (initialization): An error occurred while loading `/Users/senny/.emacs.d/init.el':

File error: Cannot open load file, /Users/senny/Projects/cabbage/electric_end
```
Ah.  Ok.  SGTM.  Nevermind then. :)
Heh.  Ok.
Why would it be useful to assert on position and isn't size already asserted?
这个我真不清楚，要是我合并进来，你得负全责啊！：）
I don't think this mechanism should be included in the default list. This won't be usable by code not aware of "X-FACEBOOK-PLATFORM" mechanism, as the api_key and access_token still have to be provided.
A little more documentation about what this rule is?
I say do away with the second check here
Yes, i'd rather have no support than halfassed broken unreliable support.
style nitpick: a space after the `//` makes the comments more readable.
Yes, that is the API. I added a comment to avoid confusion
I intended this comment to be for all tests. Ultimately, we want to pass all of the old test, and maybe add some new ones.

For any old tests that were broken, then keeping them in place and reversing the logic is perfect.
Good catch. I guess this is similar to file quick open where you can have the same file names in different directories. Yes, this is a bit of work since the results will have to be changed to differentiate the two.
Is this wait necessary?
Missing doc?
I don't understand the new condition you changed
What does it simply mean ? (what the value of hintsContent.text is changing to )
There was an issue with data.filters where if it contained a javascript error in the javascript for the filters then the "loading...." box just stuck around...better to show something more useful in the console about what error actually occurred
to clarify, the couch view uses {} as the app_id for forms that have doc.app_id = null. Providing something other than None here is the best way (using make_form_couch_key) to determine whether i should specifically look for blank application ids and use the key prefix "<foo> app" or whether I don't care about what app id this form has and leave app out of the key prefix.
What's wrong with ```el-get-package-is-installed```?
esto es una chusta
maybe put this in a textfield so it's easy to copy/paste?
Also Done.
Import only modules, not directly their contents.
But you have changed the comment for the other view. I am saying just that if you change one, change also the other. Be consistent.
Order order order. First system, Django, other libraries, then ours ...

Never import everything. Do only `import mongoeinge`.
Flip the order of these calls, since we want to short-circuit if _mediaReady is false.
EOL white space
This line is indented 1 space too far
Why is this function here?
File a new bug for this - as I've noticed there are a couple of other links that point to the US version
Already thought that when thinking about it yesterday. But thought it wouldn't be possible (guessed that there couldn't be gaps between events), and as it wasn't part of the condition already I didn't add it.

Will add it this evening.
good catch.  I didn't want an emphasis on white-space changes. Hence this patch is not consistent on white-space.
Yes, it's a requirement to use an existing rds snapshot. This one belongs to aws-blue however we can use any to record the VCR.

If we want to have the functionality of running non-isolated tests, we would have to run a rds and create a snpashot as setup step. Let me know what you think.
Yes, sir. They were all pretty small and tested individually. This all got started for zmq3 so as
to make everything build nice in the dailies & I've just been merging debian-experimental diffs 
to get as in sync with them as possible. Be glad to make some notes on commits for you
if you like.
+0, how you guys want it. Something like [<id>] <message> ?
Could this variable be allocated by statck?
"Více o ..." se nedoporučuje používat, použít moje, jen zaměnit množné číslo za plurál.
I'm using [autoenv](https://github.com/kennethreitz/autoenv), mostly for defining convenient aliases:
```
alias t='./manage.py test'
alias s='./manage.py runserver'
alias syncdb='./manage.py syncdb'
```
I guess it would only make sense when we are actually editing data, however it is a nice feature in a lot of web-gis applications (measure lines and areas).
We should probably be adding a migration that moves the old tables, right?  Modifying existing migrations should be pretty rare, as I understand things.
Let's separate the type hierarchy determination from the display of the type hierarchy--in other words, this function shouldn't take `io` as an argument. I'm also not sure about the underscore prefixes--truly internal functions I think get a `_jl_` prefix, but I'm not sure this needs to be hidden.
This is the only change on the Katello side. Please note I am only adding a new method to the Pulp Glue code, it is unused in Katello. No other changes, this is important feature of my pull request.
I would like to see comment there:
# this is dependecy for ruby_parser
so next time we will be due diligence of this file, we will not be confused why it is there when we do not use that (directly)
s/update/upgrade/ even here
代码中不要出现不可见字符
In a follow-up patch, if you want we should patch the b2g.js file when we flash the phone and change a couple default prefs (e.g. default locale, default keyboard etc).
out of curiosity what does `dumpn` do?
Maybe it's just me, but I tend to think of the term "glob" mostly applying to files?  I think more people would know what you mean here if it just says "wildcard pattern".

Also you can put this docstring in the getter function itself, and it will be used as the docstring for the property--something to keep in mind when changing the property() calls to decorators.
`adjaceny` -> `adjacency`
Passing multiple booleans to a function results in fairly hard-to-read code (which argument is what, again?). Not meant as a showstopper, but normally I'd suggest using enums. Or maybe add a comment before each, like,

    fBound |= Bind(CService(in6addr_any, GetListenPort()), /*fExplicitBind*/ false, /*fError*/ false);
Nit:  how about:
if (nHashType < SIGHASH_ALL || nHashType > SIGHASH_SINGLE)
This seems like a fix unrelated to this pull, which is also part of another currently open pull.
If @laanwj will ever accept an official name of Bitcoin-Qt for the GUI client (which I like), this should perhaps be the description string from the meta-data pull I have open or we should harmonize this.
Ok - so leave it as is? Since 'checkBalanceChanged' is already called for each 'updateTransaction', restarting the timer if it's running seems to be sensible.
"Thrown when an object is modified"
TODO: /* Ignore any comments and any space characters */
Sure that's fine. I just used template because it required changing less code :)

You're probably right though. (I'd never heard of string.Template until I looked it up just for this)
I think this docstring needs to be more user focused - remember it will be exposed through `%R?`. Something like this:

```
Execute code in R, and pull some of the results back into the Python namespace.

In line mode, this will evaluate an expression and convert the returned value to a Python object::

    %R 1+3    # Come up with a better example!

As a cell, this will run a block of R code, without bringing anything back by default::

    %%R
    foobar
    plot(y~x, data=something)
```
I would just remove the readline install.  We can just point anyone who has issues with tox your way :)
`cast_bytes_py2` and `cast_unicode` only expect bytes/unicode input, so the right thing to use here is `str` and `unicode` respectively.
probably need to log the exception and/or be more explicit: ex. NoSuchElementException
> Generally, if an allocation fails, we don't require code to free other memory that was allocated within the function.

Ah, okay. I can't remember that I've read that somewhere explicitly. However, given your reasoning and the fact that on almost all platforms the code won't be executed ever anyway, I now think, that it's safe this way...
know this is existing code, but do we need to do this in the context of an `eval()`?

couldn't we just do:
```js
var IS_IE = /*@cc_on!@*/false;
```

if it has to be in an `eval()`, we should leave a comment as to why
Bootcamp styleguide doesn't like this style: http://mozweb.readthedocs.org/en/latest/js-style.html#conditionals-and-loops
L10n: `{{ _(product.description, 'DB: products.Product.description') }}`
Same thing here that I said regarding `_db_products_for`.
should this be {"elements": ALL} - or is ALL the default and you can just make a set of field names instead of a dict (didn't see that in the docs)
the `\` is not needed when it's already in a bracket
no Idea about this either
如果图片存在另一个domain下呢？比如我打算把它们放到 http://download.acm-project.org/xiaoyou-album/ 下面，这样可以避免很多开销，至少是session-free的
Are we using the ForbiddenOverwriteException? not seeing it in the code
Missing part of the comment?
also the author is using jshint and his settings are in build.json
Ok.
This point is my real issue.

I can add another factory, but it will a copy / paste of FormLoginFactory.
Moreover, a start to work with FormLoginFactory, but I will have to duplicate HttpBasicFactory. And maybe others ...

So I don't think it's a good idea.

We need feedback here.
Why did you change the above lines?
this means that the RequestContext is not initialized anymore whereas the generator uses it too, thus breaking things.
When the condition is as long as:

    if ($this->getCurrentLineIndentation() == $currentIndentation && $this->isStringUnIndentedCollectionItem($this->currentLine)) {

multiline notation is much easier to read. Should I change it into one long line?
CS: this should probably be unindented.
Nice :). Next commit includes that change :)
Space before boolValue
What if instead of creating a new job and using a module to share behavior, we rename the old job and pass in the name of the email (the thing that changes?) as a symbol?

```ruby
class VoteEmailJob < Struct.new(:vote_id, :email_name)
  PRIORITY = 1
  DELAY = 3.minutes

  def self.enqueue(vote, email_name)
    Delayed::Job.enqueue(
      new(vote.id, email_name),
      run_at: DELAY.from_now,
      priority: PRIORITY
    )
  end

  def perform
    if vote && no_other_recent_votes
      send_vote_email
    end
  end

  private

  def send_vote_email
    UserMailer.send(email_name, vote).deliver
  end
  #...
end

class VoteCreatedJob
  #...
  VoteEmailJob.enqueue(vote, :vote_notification)
  VoteEmailJob.enqueue(vote, :vote_confirmation)
end
```
Convention in this app has been to use `step %(` to call other steps as in:
https://github.com/yammer/sched.do/blob/master/spec/acceptance/step_definitions/authentication_steps.rb#L31
在cls文件中不需要使用"@"标记的
fișierul acesta nu trebuie modificat că eu l-am redenumit în master.
You don't need the extra spaces imbetween the key: value
Should we use a shebang here? Like `#!/usr/bin/env python` ?
可以，这里就是这么用的，相当于 Private 成员变量。
fixed :-) got distracted by area cat

2012/10/4 David Cramer <notifications@github.com>

> In src/sentry/services/http.py:
>
> > @@ -35,7 +35,7 @@ def __init__(self, host=None, port=None, debug=False, workers=None):
> >          options = (settings.WEB_OPTIONS or {}).copy()
> >          options['bind'] = '%s:%s' % (self.host, self.port)
> >          options['debug'] = debug
> > -        options['daemon'] = False
> > +        options.setdefault('daemon', 30)
>
> I think this should be False :)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/getsentry/sentry/pull/619/files#r1764157>.
>
>



-- 
http://zemanel.eu/pages/about-me
this is an attempt to get the new fields ot show up in advanced serach when the user does an import. i moved it to the app router, whcih is more reliable and ensures that anytime the user clicks on Advanced Search they always have the freshest fields.
Ah good point. When testing the underlying Connector did not set a cause but that does not mean that all implementations will act this way so let me add this. 

Thanks,

/Daniel
I think this was to allow multiple iterations for the same version installed., so it's a needed feature, though perhaps not coded correctly.

The idea is taht if you have package version "1.2.3" you want to permit "1.2.3" <= version < "1.2.4" - this allows 'iteration' values in versions like "1.2.3-10"
Switched to using isEmptyObject in 92f6749
remove the `i` after data-l10n-id="ok".
Should I bug you about the anonymous function ? ;p
'p' doesn't make sense for me, please change for true if you want to check this hash in another part of your code
_this is self in GAIA
Is this necessary to add editMode property for handling edit mode checking? We can check the status via hash. If we have  this property, we need to update it  every time while entering/leaving  edit mode.
But it works like this - even if I reset and reboot everything I'm connected on startup. Any thoughts why?
> Is it possible for a system message to launch an app that is not an activity, but does not need to have its size set?

I don't see the possibility.

> I've heard that the homescreen launch is going to stop being an activity. If that happens, then this test will need to come after the homescreen-related code that is below it, I suspect.

@vingtetun would have to fix that in the bug he is fixing, not here.
non-blocking comment:

For localization, you could either get the names from `(new Date(..)).toFormatString()` or hook them into l10n APIs. (the later is better)
indentation is wrong and i-edit -> toolbar-edit?
Why would you want to cache **all messages** into ConversationListView ?

1. If you want to cache it, put it in MessageManager
2. If you want to cache it because of RegExp search, search functionality should be abstracted into MessageManager.
**ii**nstallState? :)
You should be able to figure out the filename from the URL on either side - the Ruby side, or the shell script side - and inject that directly, I think.  Precaclucating this in Ruby seems a reasonable strategy.
- Saved: *Saved ジョブ* は実行コマンドセットとディスパッチオプションを定義しますが、名前を与えることができ、グループに格納できます。

→

    - Saved: *Saved ジョブ* も実行コマンドセットとディスパッチオプションを定義しますが、名前を付けることができ、ジョブグループに格納できます。
There is no point to new MacRuby 0.10 installations, and users should upgrade, but I had not considered if it would break existing installations.
Ah yes, that's much more elegant. Let me make the changes.
Because who searches for sage really?
Hatena's OAuth always require 'oauth_callback' parameter. If missing it, OAuth failed.
Also if set 'oob' for 'oauth_callback' parameter, it changed PIN based authentication.
The PIN code that Hatena OAuth provided is complicated. It is difficult to input by mobile keyboard.
Hello you're right. This one could be shortened.

The comment was added because of PEP8

Lubos

On Mar 19, 2012, at 9:43 AM, alon wrote:

>> 
>> -        def add_qxl_vga(help, qxl, vga, qxl_dev_nr=None):
>> -            str = ""
>> -            if has_option(help, "qxl"):
>> -                if qxl and qxl_dev_nr is not None:
>> -                    str += " -qxl %s" % qxl_dev_nr
>> -                if has_option(help, "vga") and vga and vga != "qxl":
>> -                    str += " -vga %s" % vga
>> -            elif has_option(help, "vga"):
>> -                if qxl:
>> -                    str += " -vga qxl"
>> -                elif vga:
>> -                    str += " -vga %s" % vga
>> -            return str
>> +            # Less common options
>> +            tmp = optget("spice_image_compression")
> 
> def addopt(k):
>    tmp = optget(k)
>    if tmp:
>        spice_opts.append("%s=%s" % (k, tmp))
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/autotest/autotest/pull/233/files#r573120
It's not good to fill the default value to <....>

comment them or fill by some default values
        username = root
        password = 123456
        .....
        .....
What does that mean? AFAIK 0.6 is the current stable, so 0.7 would be the next release which might break something, right?
I blame my emacs. Will fix it.
I might pull out `.DS_Store`, I think those files should go in a global `.gitignore` file...
public properties should come before protected ones
most likely need to base64 these bad boys.
I stand corrected :+1:
`guardiannews.co.uk`? Check out what's really at [guardiannews.co.uk](http://www.guardiannews.co.uk) and have a bit of a cry.
ACK (nice!)
I'd rather see the `do not have authorithy to load module` exception here, in a `else` statement.
@salmonz - I think these last two steps should be optional, some users may not want to cache their credentials.
I mean: &lt;span id="logger" class="disabled"/&gt;
May I ask why you remove that line? In case of the browser.js it's present. Btw. where does storedTitle come from? I can't find a reference.
It depends on when the code gets called. If it's during the startup phase then we can do it on shutdown. Otherwise it has to be done before possible references get invalid.
extra line break? also, should there be a comma after content on line 9?
`unique*` may not be the best name here, what do you think?
Instead of doing `Facter::Util::Config.stubs(:is_windows?).returns(false)` and `Facter::Util::Resolution.stubs(:exec).with('uname -s').returns('Linux')` it would be better, and a little safer for testing, just to do `Facter.fact(:kernel).stubs(:value).returns("Linux")`.
Same as the comment on line 588
Hmm. That has to be a debug leftover. That's a completely sane check. Can i edit just this line somehow, or should i put in another request?
`|| process.env.DEBUG_COLORS` would be fine
No need for apologies. You're welcome ;-)
Oh, thanks. Not only that but more importantly the priority.
`al manager dei moduli di questi` -> `a Zend\ModuleManager di farlo`
Let's extract the above into a separate method.
zend-service are a bunch of independent subcomponents
Use spaces rather than tabs in the readme to keep everything standardized
There is a typo on your README, search for "applyibng"
please remove this column used only for the design and add a margin-left on this table instead.

and you should use ``<th>`` instead of ``<td>`` for the column headers
I think
```java
if (world.getGenerator() instanceof TheEndGenerator) {
    // do stuff
```

would be more logical here.
you've got a double forward slash here :)
You don't need to define this, if not present it will not be called.
Shouldn't this be uncommented so we build the stylus assets each compile?
get_or_create is to be removed for db.open_or_create

for StaticDocument, one should pass a instance, that will get a id, or be replaced with the one from the db
User don't need autotools to build from a released tarball.
is it allowed to override a field with a get/set later on? in that case could you simply have

    var data;

?
yes - already taken care of, realized this was dumb pretty quickly
Actually, we shouldn't define buildstats here like this, we should simply call a function to get the buildstats. 

Let's fix that in #257.
check whether the binary which delivers the main program is there (or is that ecore.sh?)
Info on all arguments added
done (index created)
I think the return is superfluous, isn't it?
Should probably go ahead and use the promise interface: .done(), .fail(), .always() on the jqXHR object.
I wrote this, and feel it's accurate, but localizers may want to chime in and offer advice.
"That's OK; it allows us to use Task.delete()."
pad the `str` with spaces
Yea... looking at it now. Let me know what you find.

On Jul 6, 2012, at 10:26 PM, Kilian Ciuffolo<reply@reply.github.com> wrote:

>> +NsSocket.prototype._fetchBody = function _fetchBody(chunk) {
>> +  var raw, event, data;
>> +  var chunkLength = chunk.length;
>> +  var bytesLeft = (this._eventLength + this._messageLength) - this._bufferJoiner.length;
>> +
>> +  if (chunkLength >= bytesLeft) {
>> +    raw = this._bufferJoiner.add(chunk.slice(0, bytesLeft)).join();
>> +    event = JSON.parse(raw.slice(0, this._eventLength));
>> +    data = this._messagetype ? raw.slice(this._eventLength) : JSON.parse(raw.slice(this._eventLength).toString());
>> +
>> +    this._eventLength = -1;
>> +    this._messageLength = -1;
>> +    this.emit(['data'].concat(event), data);
>> +
>> +    if (chunkLength - bytesLeft) {
>> +      process.nextTick(this._fetchHeader.bind(this, chunk.slice(bytesLeft)));
> 
> I think this is not working.
> 
> Could it happen that scheduling this on the next tick, a new `data` event fires before our `_fetchHeader` call?
> This will break the header. ;/
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nodejitsu/nssocket/pull/11/files#r1117822
also, does this generate api docs correctly?
This line is around 130 chars long. Can you break the line at 80?
Not quite.

If I want to release a new version of govuk_content_models I should increment that repository's version.rb file in master.

If I need to update the version of the gem used in an app (as I'm doing here) that should be done in the branch that requires the new version.
Hm, good question. I didn't realize there could be something other than `eZContentObjectTreeNode` there. I think that all operations expect `$object` to be  `eZContentObjectTreeNode`, don't they?
Uses directly  $RBAC->loadPermissionAdmin() in this line
yep, you're right. what if we call the "empty()" method?  it's already implemented and provides the same functionality.

it would be something like that:

			QueueManager queue = (QueueManager) getCapability(resource.getCapabilities(), QueueManager.QUEUE);
			queue.empty();
Yeah he's been inactive for a while -- there are a couple of other pull requests pending as well.
I think a bunch of projects still use Django 1.3 and jingo-minify and the requirements file for jingo-minify specifies Django 1.3. Because of that, I was trying to maintain backwards compatibility with Django 1.3.

If you don't care about 1.3, we can update the requirements file and I can change this to use the Django 1.4 override_settings.
ups, can't be merged automatically...

can you please rebase master, and bring back this filter again?
do you need source_type or class_name on this? (and the next line)
Seems like we should know for sure what's going on, here...we're proceeding assuming it's a new database, so I think we should just log a message to that effect.
Having the path that was attempted to be written to would be helpful in this message.
There seems to be a few places that still have `'global'` hardcoded:

'''
/tmp/Hilary$ git grep "'global'"
node_modules/oae-principals/tests/test-users.js:        var globalTenant = new Tenant('global', 'Global tenant', 'localhost:2000');
node_modules/oae-profiles/tests/test-profiles.js:        var globalTenant = new Tenant('global', 'Global tenant', 'localhost:2000');
node_modules/oae-tests/runner/beforeTests.js:    var globalAdmin = new User('global', null, 'administrator', 'private', 'en_GB', 'Europe/London', 'Admin', 'User', 'The global admin User', true, true);
```
Yeah, it's much clearer now! :+1:
Looks like this should deserve a similar FIXME note as https://github.com/scala-ide/scala-ide/pull/221/files#L11R174 (I need to check)
Put spaces after type casts.
согласен. спасибо.
При 100 элементах в массиве и 100% замене вы можете получить более 1000 итераций в цикле формирования случайных чисел, т.е. эффективность 10N.
Если установить к-во элементов DNA в 100 000 и менять 100%, одна мутация (на моем ноуте) занимает приблизительно 2 минуты 30 секунд.
Попробуйте рассмотреть например такой алгоритм формирования массива
NSInteger indexArray[[self.DNA count]];
for (int i = 0; i < [self.DNA count]; i++) indexArray[i] = i;
for (int i = 0; i < [self.DNA count]; i++) {
     int rnd = arc4random() % [self.DNA count];
     NSInteger swap = indexArray[rnd];
     indexArray[rnd] = indexArray[i];
     indexArray[i] = swap;
}

при 100 000 и 100% мутации работает не более 1 сек., но в этой реализации используется больше памяти. Возможно вы найдете что-то среднее по времени выполнения и использовании памяти.
Все, ушел гулять с ребенком :)
Создайте свой конструктор, который назовется, например, initWithCapacity:(int)capacity. В нем так же нужно будет вызвать конструктор базового класса, а вот массив будет инициализироваться с помощью переменной capacity. 

Обратите внимание на название конструктора - оно "англицировано", я даже не знаю, как правильнее выразиться. Об этом говорил Рахим на лекциях - нужно стараться задавать названия методам так, чтобы они были понятны. Как пример, у типа NSNumber есть конструктор init:

NSNumber *num = [[NSNumber alloc] init];

Переменная num создана с каким-то значением по умолчанию, в данном случае 0. Но так же есть конструктор initWithInt:(int):

NSNumber *num = [[NSNumber alloc] initWithInt:100];

Создастся переменная с значением 100. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
красивее было бы: `mutatePercentage = MAX(0, MIN(muatatePercentage, 100));`
found should be equip_found, it should be of type bool, and initialized to false.
Yes, please. That will help others like me.
This constant is still used on the Mac.
The block on the background scheduler may not have completed by this point, though.
Not an API change. The test was using the API incorrectly. From what I
could tell, the correct way to set manipulate! options is from the method
arguments, not the block. I corrected the test to reflect that.
On Jun 6, 2012 10:44 PM, "James Miller" <
reply@reply.github.com>
wrote:

> > @@ -80,13 +80,20 @@
> >
>  ::Magick::Image::Info.any_instance.should_receive(:quality=).with(50)
> >        ::Magick::Image::Info.any_instance.should_receive(:depth=).with(8)
> >
> > -      @instance.manipulate! do |img, index, options|
>
> Are you changing the public API here? Please don't remove existing tests,
> we need to know that everything that existed before your patch will still
> work after it is applied.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jnicklas/carrierwave/pull/754/files#r941410
>
这个名字感觉长了点，是否改成bloom-contains?更短一点？下面也是一样。
@xumingming 貌似不用...这里b只是作为参数，不会调用b的方法....我打开reflection-warn看了没有警告的
Convenience function for quickly comparing tokens for equality.
I'd prefet an row_cals + col_cals init to None and then

```
if row_cals is not None:
   del row_cals
if col_cals is not None:
   del col_cals
```
if we avoid the branching then we won't need these tests. Also in windows n_jobs > 1 don't work unless you start the calls from a main function so tests with n_jobs > 1 will break on windows :(
Adobo Chrome Extension Skeleton
Does that actually work? You'd think the inbox would need the user saved first...
that.heartbeat();
The normalization is done before merging the configs together. And the validation is done after the merging. Your sentence is not enough as it does not talk about the place of the merging in the process
Will this work? If these are task objects, then we can't say `form.task` and get that child form. Perhaps we should be iterating over `form.tasks` instead and passing that value in as the `form` variable? Also, I think this should be a `tags` property - in this example, we have a `TaskType` form, which has a `tags` collection property. But perhaps I'm reading this example, wrong - let me know!
Cool, will give this a try