labofsahil/github-event-dataset-2012 huggingface dataset:
align with (only-in ...) above?
Yep totally make sense
Make sure indentation is consistent. Sometimes tabs are inserted when spaces should be used.
createMap sets `__proto__` of the object to null
so that `toSting`, `__proto__ `etc. are excluded from tagMap
Shouldn't we log something like "older than" or "insufficient" instead?
Please don't call this variable `mCurrentScale`, which I find rather misleading. It is the scale at the time the gesture started, so something like `mStartScale` or even `mGestureStartScale` would have my preference.
Ah ok this isn't working as it appears the function needs to evaluate each argument before it tests the condition, which is breaking on nth($padding, 2) if $padding is only 1 element long
Can we add `jsonp` with `application/javascript` as well here?
Running the event loop recursively is generally a very scary thing to do, since code is rarely written to be run-loop-reentrant. It'd be far preferable to split up this handler over multiple received events instead, if possible. I realize that's a lot of refactoring, but the end result will be much safer to use.
Yep, this one should be preferably left private, but it's a mirror of CAConstraintManager (I think that's the name), and it should be fine if used by a user. The category on TUIView is an ease-of-use kind of thing, to mimic Auto Layout.
`data_[body_offset_]`
How do you know this won't wrap around?
I think we should return ByteBuf to allow method chaining
It would be great to document the context here, i.e. where this is currently being used, how it can be used in other components, and what the WFS schema needs to contain for this to be useful.
Didn't think of that, over engineering it. WIll remove thanks @smalyshev & @nikic
Since this is an internal syntax change, it doesn't need a changelog entry.
How about `options.delete(:serialized) { true }`?
Should we delete similar commented lines too?
Sorry about that, unneeded artifact from sprockets `2.6`
Yeah, let's do that in a separate commit.  What you have now is exciting and a good first step.
On second thought, I could remove these lines and have no output at all. I'll push a new commit if this is preferred.
Is this renaming OK or is there a better name?
Since `AnyRef` specialization is not really useful yet, I think we should revert or disable it. That seems the only thing preventing the name be known at cloning time.
```
A dummy value to denote cross-stage path-dependent type dependencies.

For example for the following macro definition:
{{{
class X { type T }
object Macros { def foo(x: X): x.T = macro Impls.foo_impl }
}}}

The corresponding macro implementation should have the following signature (note how the return type denotes path-dependency on x):
{{{
object Impls { def foo_impl(c: Context)(x: c.Expr[X]): c.Expr[x.value.T] = ... }
}}}
```
This didn't really achieve anything, as the previous rescue would complete the request w/ a 400 (malformed) and an empty response body. I believe a test covers this.
On 11.06.2012 15:23, Christopher Smith wrote:
>> @@ -0,0 +1,553 @@
>> +.. _polys-internals:
>> +
>> +===============================================
>> +Internals of the Polynomial Manipulation Module
>> +===============================================
>> +
>> +The implementation of the polynomials module is structured internally in
>> +"levels". There are four levels, called L0, L1, L2 and L3. The levels three
>> +and four contain the user-facing functionality and were described in the
>> +previous section. This section focusses on levels zero and one.
>
> focusses ->  focuses
>

done
No. We've changed the class definition

    MatAdd(MatrixExpr, Add)

to
The comments should be removed, and instead put into the git log for the appropriate patch.
Why "active"?
I think it's `->get('doctrine')->getEntityManager();`
Some additional syncronization
rm 221
`tal:define="tags context.tags"` is shorter
sizeof(*buf)
Yes, we should definitely look into Oracle support for the agent module at some stage - I'll create a ticket for this.
My fork is here: https://github.com/keeganbrown/cycle. Still working on it, so use at your own risk.
// space
Hmm. Maybe the comment should just go. I don't see what the comment is for.
I agree entirely with Igor, there is a reason we use dependency injection and not a service locator.

It's not as simple to find as you make it out to be, as you might be passing the container on to something you use which then can access any of the other services too, etc. So it can really be close to impossible to correctly identify them.

It's also about making code reusable, not just about testability, and having more usable documentation.

So in general do not pass the container if it's in any way avoidable.
waring -> warning
That might lead one to believe that all ODBC connection strings need to go there. For MSSQL via ODBC, the custom_dsn value isn't needed since the test framework already knows how to handle those. It's only used for when you need to provide something that phpBB would not have expected. (i.e. An ODBC string for Firebird or the PHP MSSQL module)
nit: looks like some sort of spacing issue.
Hmm, I'm not sure. I think there is still ambiguity here. At first glance I am not sure of the differences between `reset()` and `rewind()`. 

Also, what's wrong with `if (this.repeat-- > 0) {` ? :( I really like it.. I guess it's not very intuitive though.
lol, I'm not very good at regex in general, so I escape anything that it suspicious. Feel free to clean that up.
Use a static Map for this so you can just do `reasons.get(reason);`. Look at the other enums for how to do this (use `static { ... }`).
You could, but you probably don't want to change the license owner

It's cool to add yourself as an author if you've made large contributions that you want to be held responsible for
Yeah, it should be stated in there if we really want to impose it.
However I suggest we stick to the global standard.
@delallea @goodfeli yes I think `__eq__` and `__hash__` are both necessary, good catch.  Also, pep8 calls for 1 space between method definitions.
Not important and no need to change anything, but just in case you wouldn't know you don't need the double parentheses. You can use:

```
raise AssertionError(
    "If you're assigning to a DisconnectedType you're"
    " doing something wrong. It should only be used as")
```
So I should do a transfer for comparison?  I can also do it on the GPU.  I would do the .all() on the GPU too, but I can't do reductions yet.
Yes, I am finishing up on making the tests pass with it on by default, then we just have to decide if that is what we want
```js
exports.baseTechPath = require.resolve('./ie6.css');
```
don't remove, this is correct see 2b573087
Maybe called this `supported_useragent?`
do you have a version.rb in any other place ?
`if (jQuery.cssHooks[ fx.prop ])` => `if ( jQuery.cssHooks[ fx.prop ] )`
I didn't write it either, but it seems it's removing the duplicate from ret, which means `n` needs to back up one after the removal.
I don't have a good sense how expensive it is.  I just thought it'd be nice if we could avoid when it's not needed.  May not be worth looking further.
If I understand the code correctly, Meteor.accounts.oauth._requestServiceName returns null if this isn't an oauth request, and the next line will try to access Meteor.accounts.oauth1._services["null"] which wouldn't exist, which would cause us to correctly call next().

This works but is confusing. Can we put an explicit check such as:

// Skip everything if this isn't an Oauth request
if (!serviceName) {
  next();
  return;
}
There where a few other places in need of similar fixes in this pull request.
+1 to rolling it into LocalizingClient. However, I don't think that making a factory would work, because in the tests we get a instance of a Client subclass by specifying a class, which the TestCase class initializes on it's own. I assume that tweaking this would be more trouble than it is worth.
Hmm. Are we considering every installation that has tools in /usr/bin to have the CLT installed?

Doesn't this mean that, for example, Snow Leopard/Xcode 3.2.6 users will now start seeing `Xcode: 3.2.6 (CLT)` in their `brew --config` output?
Weird; I guess I'm just used to self-bootstrapping things. Seems fine though as long as it works and we ship bottles.
Well I also didn't expect things like `brew --config` to take
```
real 1.57
user 0.63
sys 0.20
```
and because I don't have knowledge of how things work or how many hats you own, I asked.
I tested *a lot* of builds against superenv. I must have built 50% of formula the last week or so. Just to reassure you.
whoops, this shouldn't have been committed.
Ideally it would be like LISP and a :as variable could be assigned to the set of kwargs. But it's Python. Whatever. We'll eventually end up with LISP again.
(I had to edit that prior comment, so if you're reading it via email, it won't show the update)
I'm of ambiguous mind on this.  The "upstream" graph (so to speak) is the root from the perspective of this generation, so I think it's valid, though I think it's also annoying. 

I was thinking about augments, but that's more of a method name, not "the thing that is done-to".  I think we need a noun form here.  Just not great choices.
no unitary . . . just 'binding' ?
Specifically, what does the "mojo" plugin do?  Can we add a comment?  The ideal is that someone could come to this file saying "I want to change how the JUnit build does X", and without knowing all the jargon (wagon, mojo), could find the right place to start looking.
I think, in general, we should try to get away from using variables like 'k', 'v', and 'pk'. Obviously everyone has heard this argument before, so I won't go into it here, but it makes everything more readable and we don't sacrifice code size.
Formatting is off, tabs, not spaces
Can't hurt the server - the Vector3 values generated when they are fetched (they aren't the internal values in the map), so the array and its contents are only for the event (output only)
This is ok for now.
it wasn't showing the border
New externalized strings  ``1 JSLint error`` and ``{0} JSLint errors``
Yup, made that comment more clear.
Might want to add a comment here noting that extensions may add additional constants later.  I was confused at first as to why these two are declared here but the FUNCTION_LIST_FOR_FILE and JAVASCRIPT_* ones aren't.
Our style regarding key names for objects is to not quote they key name unless it's required because its a reserved word or has special characters. Thus, you can remove the quotes from attriboption, global, and type. I think it's fine to leave the quotes on the tag names wince some of those could cause problems.
Should have a result.reject() case for extensions without unit tests
I see the beginning of a `/*` here; code can probably be removed.
Inline require's??
I'd be tempted to make the default to 10, so you at least create a valid character.
Same thing. Also, you can pass an expression to AssertTrue:

```python
self.assertTrue('attack' not in column_names)
```
missing space around operator
Why not pass the whole row object?  It would give the callbacks more flexibility as to what they can do.
shouldn't this be `stop-and-frisk.json`
This is wrong, you must have a bad rebase.
At 205 below, you create a new LatLng object with these scalars; let's just do it here so we don't have the overhead on every use:

```
_cachedGeoCode[ options.location ] = new google.maps.LatLng( options.lat, options.lng );
...
       var cachedLocation = _cachedGeoCode[ options.location ];
        if ( cachedLocation ) {
          location = cachedLocation;
```
I see what you're saying.

This needed to be px and not % because otherwise when you start to scale up or resize, it'll start to overlap over the left of the main container.
Also maybe a comment on these if/elseifs such as if (op === 0x40) { // push n bytes
Space at the end of this line
I'm just looking at the patch right now and haven't looked at the actual call graph, but it looks like this needs a lock.
Our style guide says no spaces between the')' and the ':'.
This URL looks incorrect, shouldn't it be `http://localhost:8000/api/v1/video/2/`?
The simplest, just to test and to get a proof-of-concept to work with, was just to add one target (with LUN=0) for each target. And each target is one ZVOL. I was also unsure on why/where to use LUNs at the time, but your (patdk-wk?) comment seems resonable. But considering that we can create a ZVOL with a colon, we could use that as the LUN.

I.e.:

  zfs create -V 15G share/tests/iscsi2:0
  zfs create -V 15G share/tests/iscsi2:1
  [etc]

and then use that as the LUN.

OR, I set the LUN to the same value as the TID instead of hardcoding it to '0' (the LUN that is). Might be simpler...

So techically, it's possible to do both (10/3 or 30/1). Actuall solution can be done in a number of ways. Even at the same time!

And you're right about loosing your targets if your restart ietd. A reshare would fix that, but... Do we really, really want/need to modify the ietd.conf?! Since I'm running a custom script after the target have been 'share'd, one could always document this behaviour and recomend adding that code to the script... ?
I suggest filing a separate issue for this. Seeing the build failure would be helpful.
I could not reference external resources successfully so I ended up with a data uri.
I changed the names to something like "player-contorls-button", I also think it's better no matter the names are temporary use or not.
This n_times attribute would be redundant if this class grows a `__len__`.
Although I cannot point you to the code line where this happens, I am pretty sure that blank lines will be taken out at a different position. After all, the `BaseInputter` doesn ot do anything like that in `process_lines`. According to the code comments in `BaseInputter.process_lines` the `process_lines` removing empty lines is not its intended use. From `BaseInputter.process_lines`:

        This routine is not generally intended for removing comment lines or
        stripping whitespace.  These are done (if needed) in the header and
        data line processing.
Sorry for being slow, I see what's happening now and I agree with your original approach.
done!
On Aug 20, 2012, at 8:18 AM, jao <notifications@github.com> wrote:

> In README.rst:
> 
> > @@ -153,5 +155,15 @@ of the source by installing `Sphinx <http://sphinx.pocoo.org>`_
> >  
> >  Then launch ``docs/_build/html/index.html`` in your browser.
> >  
> > +How to Contribute
> > +-----------------
> > +
> > +How to contribute to BigML's python bindings:
> I would delete this line, it just repeats the title
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
It would be could to have tests that use the locally generated predictor and compares its results with the remote ones.
I did a compare and you are right, mostyl indention changes.
This line seems a little garbeled and misses an ``;`` at the end, should be more like:
``<string notr="true">background-color: qlineargradient(x1: 0, y1: 0, x2: 1, y2: 0, stop:0 #F0D0A0, stop:1 #F8D488); color: #000000;</string>
Much better?
You are right, should be integer. I've updated the rogue int types.
The `Log` is in the same namespace, so you don't need to declare the `use` on this case.
Thrown when a new object is drawn.
Ok, we solved it together with @samandmoore on jabbr... basically we had to change Bootstrapper to run on PreAppStart (was running on Post) and add the Order attribute (first Bootstrapper, then FedLogin)
Please remove the extra spaces inside the braces (same on line 30)
There is no need for both the "=" and "::" expressions here. Just use a single "<<" binding.
remove namespace, we ``use`` this class.
Use initWithData:encoding: after excising malloc.
Yes, because it *shouldn't* (and doesn't) pick up this one.  It's a base class for real tests to inherit from, and won't work without subclassing.  It's the same as `clienttest` in IPython.parallel.
Would this be prettier? (BTW - completely untested so probably won't compile as is...)

```c
#ifdef GIT_THREADS
#define GIT_PACKBUILDER_MUTEX_OP(pb,mtx,op) do { \
    int result = git_mutex_##op(&(pb)->mtx); \
    assert(!result); \
    GIT_UNUSED(result); \
} while (0)
#else
#define GIT_PACKBUILDER_MUTEX_OP(pb,mtx,op) GIT_UNUSED(pb)
#endif

GIT_INLINE(void) git_packbuilder_cache_lock(git_packbuilder *pb)
{
    GIT_PACKBUILDER_MUTEX_OP(pb, cache_mutex, lock);
}
```
It looks like a pretty good fit, yeah. It's on my list to shrink the clone code by using that once they're both merged.
Trailing whitespace slipped in..
It passes all tests. Besides, in the old code, it left some meters being null when there is setup failure, which will cause NPE eventually. I think it's better to fail earlier than throwing NPE some time later depending on the execution path.
The following few lines *should* be superfluous: I believe we have deactivated default acoustic models per language. That matches the fact that CARTDurationModeller and CARTF0Modeller are deactivated above (empty line before their declaration).

The corresponding files at lib/modules/... are most likely not required either, or at least shouldn't be...
Ryan Merkley (my boss) wanted to ensure that the twitter and facebook links point to moziila opposed to firefox....
`'use strict'` is missing, you should run this file through JSHint: http://www.jshint.com/
Even if it were to kick up an error, we should still let the user ask a question. If not, we need to file a bug for that.
Don't use error.__class__; you already know what the class is from the line above: UninstallError
You should probably also preserve the original error message (e.g. str(e))
(This applies to all errors you handle in this manner)
@peterbe hmm good point I can add the , so it's actually a set, but wouldn't """if 'update_adu' in 'update_adu'""" work? Assuming any new entries were separated with , of course...
Is this realistic? Ideally, we should use what Mozilla uses in production I think.
I guess I'm not 100% sure why this is needed here. If the top navigation is already doing this, why do we need to do this here?
Questionable punctuation.
Inline doc overview of what this class does? Explain it to me like I'm five ;)
meaning it should not have been copied at all but replaced by ``{@inheritdoc}``
fair enough. Can you add the references to the PHP bugs before I merge?
If we eventually introduce FormBuilderInterface, this interface must feature a `setParent()` method, so we can just as well leave it in.
No, there is only 1 error handler in PHP. If you register one, it replaces the previous one (and returns it)
Not sure why you're changing this here either.
(for future performance optimization) - we should try to use `ChannelBuffer`s as much as possible throughout as it allows us to reduce the amount of copying we do.
Might steal some of the like button js for this.
New hash syntax preferred. Single quotes in ruby.

Also the "render" is implied, so we've been removing it, as here:
https://github.com/yammer/sched.do/blob/master/app/views/events/_form.html.erb#L14
akhe to w3 neveshte zamani ke mikhan be div background bedin ya az ye be chanta tag img mikhan container bedin az figure estefade konid
bashe div esh mikonam :)
Correct these global consts
Surprised that powershell doesn't have something like this
Hmmm. It was for me. Will re-check and let you know.
I guess at this point, if it existed, we would want to say something like:

```
Exposes a :class:`WmsFactory` which can be used by implementers of the :doc:`image_factory_api <image factory api>`.
```

That single point of documentation can then be used to explain how such a thing gets used (i.e. in matplotlib its ```GeoAxes.add_image```).

I'd like to get that going, so I'll have a pop at making the appropriate docs and getting it all merged separately.
What's the thinking behind automatically updating a user in zendesk?
If the app descriptor is part of this RESTful URL, we should make sure it indeed matches the app descriptor in the manifests (the app's `id`).
Spun up a Centos 5.7 box today and noticed it only has `/etc/redhat-release`, no `centos-release`. (The file does contain "CentOS" though.)
you can just specify bash here and not have to include line 71, interpreter 'bash'
Sorry again, but now the comment isn't vertically aligned with the other ones...
IllegalArgumentException already derives from RuntimeException. RuntimeException is the root class of all unchecked exceptions.

My intention was to suggest a new hierarchy only for the NDEF encoding/decoding purposes.
Why should this example use the `chosen.js` and `chosen.css` from the gh-pages branch? The gh-pages branch uses it's own versions of chosen that are not tied to tags/releases. Therefore they do not .work as demonstration material, as they are possibly buggy. For me it feels wrong to have them there altogether, since they are duplicates of those in the master branch. I think it would be better to remove them from gh-pages and reference the tagged versions of the master branch, instead. Shouldn't the latest tagged/released/stable version also always be published at CDNJS and in turn be used whenever possible, including the example in master and gh-pages?
Hi Zearin, everything you are saying is correct and I couldn't find anything related to the use of `==` or `is` for comparing numbers.
Thx for having awesome argumentation!
Automatically setting up the ActivityFeed to be the user' actions  TODO if the activity feed db can replicate, this will be replaced with an activity feed widget showing the teams actions
Problème de réseau chez moi, j'avais commenté temporairement mais j'ai gaffé en le laissant, je vais corriger cela ce soir !
This new attribute should have type="xs:boolean"
Duh, ok, I get it now.  Sorry, brain fart on my end.

Many thanks!
After puzzling over this for a while I've realised that we don't need the isFailedIcon logic at all. All that matters is that an entry exists in the icons object store for the iconUrl and that it hasn't expired. The only time you need to use the "failed" property is when drawing the history list so you know to display the default icon instead. Removing it.
This is getting a bit out of hand. Let's not use `handleEvent` for the clicks events and add them like this instead in the `init` function:

```
this.closeNode.addEventListener('click', this.closeWindow.bind(this));
...
```
The keyboard is there on homescreen if you press holdhome.
Consider to blur() somewhere in your code path.
I swear I added this strings and the PR was merged ... :S weird
the recursion here saves us additional lines of code and array slicing
(non-blocking)

Listen for `mozvisibilitychange` event and don't update the clock if the app is in background.
on second thought I have two objections:
* links starting with '#' are still used and could mess with the panel navigation system when clicked. If there’s no URL, why don’t you remove the href and data-href attributes instead?
* why do we need *both* href and data-href attributes here? wouldn’t href be enough?
same here: unless I miss a point, I don’t think we need any specific rule for `p.message`.
Why do we need to `waitForNextPaintAndBackground` for the painted frame? Background will not be visible to the user if the frame is painted right?
Ah. Got it. Makes sense.
Alright, so the maxItems is just a hard limit for RefusePuts policy. And otherwise it might blow that cap under pressure for short periods of time.
nice -> thanks for looking into that again! It also works on my machine like this now! :wink:
```js
return (options.namespace || '') + name;
```
I took the following to mean that a function handler won't require a {path}

"If a function is used, there are no limits on requirements on the content of the path..."
This is in fact the idiom I was taught to remove from a STL container while iterating it. A simple erase and subsequent iterator increment might operate on an invalidated iterator. It's not incremented twice though, depending on whether we found the element to remove we increment it while erasing, otherwise we increment normally.
It's not clear why this would succeed or not, and since the called method handles this for us automatically, I'd just remove this check and just do the `Nginx.reload()` every time.
I don't see what you mean
```
unless g.REPLY
  if g.BOARD is 'f'
    filetag  = QR.threadSelector.value
      threadID = 'new'
    else
      threadID = g.THREAD_ID or QR.threadSelector.value
```
?
actual_time ? spelling...
My replacement for the `FilenameCleaner` class
It seems like a wtf and dilutes the power of this test; to test if a file is *really* a regular file, you'd have to test both `-f` and `! -L`.
Probably it make sense to also support generators. but appart from that?
Pierre


On Sat, Sep 15, 2012 at 3:11 PM, Tom Prince <notifications@github.com>wrote:

> In master/buildbot/config.py:
>
> > @@ -674,6 +676,15 @@ def __init__(self, name=None, slavename=None, slavenames=None,
> >          self.nextSlave = nextSlave
> >          if nextSlave and not callable(nextSlave):
> >              errors.addError('nextSlave must be a callable')
> > +
> > +        # we support the previous nextSlave API
> > +        # this is temporary hack until we use a major release
> > +        # to really make a clean API (we first need data API to land)
> > +        if nextSlave and (nextSlave.func_code.co_argcount == 2 or
> > +                          (isinstance(nextSlave,MethodType) and
> > +                           nextSlave.func_code.co_argcount == 3)):
> > +            self.nextSlave = lambda x,y,z:nextSlave(x,y)
>
> This hack won't handle callables that aren't functions.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/buildbot/buildbot/pull/523/files#r1613614>.
>
>
This line re-introduces the ObjectEncoderDecoder to the map of buildInEncoderDecoders so it properly encodes/decodes simple JSON Object types
Yea , would get it corrected
because https://github.com/gitlabhq/gitlabhq/blob/master/app/models/user.rb#L10
Before you ask, this is repositioned because `.file .content` (formerly `.diff_file_content`) was defined twice, so merged that, but not sure where it is user, so thats why I added the `// REVISE` comment.
oh well, all the tests passed :)
It looks like the reset of java.io uses FileNotFoundException to cover both missing files and inaccessible files.  We should consider doing the same.
Also I'm little unhappy that we need `pbService` both here and in `private-browsing/utils` is there any chance we could move all uses to utils and call them from here instead ?
hmm when I change this I get 96/101 tests passing on tabs for Firefox, so I'll leave it for now.
The previous code explicitely specified `0` as singleton selection rangeNumber.
With the new code, it looks like it will be `undefined`.
It might be a good idea to be explicit by doing this:
`var selection = new Selection(0);`
could you write utility function like `isMatching(tab.url, this.include)`  instead, that way you won't need to beak and it would by way easier to follow. It took me a while to figure out what was going on here.
I'd like to get `to a bar`, but I think the value need to propagate `to bar` ;)
Is `doLogin` only called in code paths that guarantee there was no user interaction?
Personally I would say we should remove `hidecolumnpicker="true"` and let the user decide which columns he is interested in.
Awesome! Add ..."that are not resolved and that have some scrum data in the witeboard"...

This is a great place to remind them of that.
ここは、単一のファイル情報を取るだけなので、元のままでいいと思います。
Again, do not change my doctype.
delete this few line, push to your master then i'll pull them in to my branch.
This doesn't seem nearly as informative as the previous warning.  Can you retain the extra details?
@kbarber I'm getting a failure due to puppet trying to backup files that it's deleting (and the file is already in the filebucket from a previous run). We should probably set backup => false or why not just `rm -rf '/etc/puppet/modules/*'`, etc?

<pre>
Begin suite/tests/modules/install/already_installed.rb

puppet module install (already installed)

  * Setup
Stubbing host forge.puppetlabs.com to IP 192.168.101.10 on machine puppet

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet resource host forge.puppetlabs.com ensure=present ip=192.168.101.10
host { 'forge.puppetlabs.com':
  ensure => 'present',
  ip     => '192.168.101.10',
  target => '/etc/hosts',
}

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet apply --verbose
No LSB modules are available.
No LSB modules are available.
info: Applying configuration version '1338486577'
notice: Finished catalog run in 0.03 seconds

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet apply --verbose
No LSB modules are available.
No LSB modules are available.
info: Applying configuration version '1338486584'
notice: /Stage[main]//File[/etc/puppet/modules/nginx]/ensure: created
notice: /Stage[main]//File[/etc/puppet/modules/nginx/metadata.json]/ensure: defined content as '{md5}25f7ff8ccbfe7cf4eeca8012de33828e'
notice: Finished catalog run in 0.03 seconds

  * Try to install a module that is already installed

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet module install pmtacceptance-nginx 
Could not prepare for execution: Unknown destination type telly_prototype_console
Exited: 1
Unstubbing host forge.puppetlabs.com to IP 192.168.101.10 on machine puppet

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet resource host forge.puppetlabs.com ensure=absent
notice: /Host[forge.puppetlabs.com]/ensure: removed
host { 'forge.puppetlabs.com':
  ensure => 'absent',
}

puppet $ env RUBYLIB="/opt/puppet-git-repos/hiera/lib:/opt/puppet-git-repos/hiera-puppet/lib:$RUBYLIB" PATH="/usr/bin:/opt/puppet-git-repos/hiera/bin:$PATH"  puppet apply --verbose
No LSB modules are available.
No LSB modules are available.
info: Applying configuration version '1338486593'
info: /File[/etc/puppet/modules/nginx]: Recursively backing up to filebucket
info: FileBucket got a duplicate file {md5}25f7ff8ccbfe7cf4eeca8012de33828e
info: /File[/etc/puppet/modules/nginx]: Filebucketed /etc/puppet/modules/nginx/metadata.json to puppet with sum 25f7ff8ccbfe7cf4eeca8012de33828e
notice: /File[/etc/puppet/modules/nginx]/ensure: removed
notice: Finished catalog run in 0.04 seconds
Error: suite/tests/modules/install/already_installed.rb failed in 24.21 seconds
</pre>
Similar as to the last one. I don't have a better wording suggestion on this one though.  The "now" time is within the range, but the weekday does not match, therefore the schedule should not match.
I think MIT-License should be fine, too - sorry for that.
These should probably go in spec/ruby/optional/ffi/struct_spec.rb for now. Maybe it's a good idea in general to reorganize these specs based on methods, but that's another step.
How about instead of replacing stderr and stdout, we just request the IPython folks add a 'metadata' attribute, which we can then just set.  I inquired about this here: http://mail.scipy.org/pipermail/ipython-dev/2012-June/009828.html
Generating numbers that are much smaller or much higher can be done by dragging northwest for much smaller or northeast for much bigger - this loses accuracy, but if you need to get -400, you probably don't need granularity of 1 for experimenting.

I'm fine with switching 10.0 -> 2.0 or some other smaller number.

Switching 100.0 -> 20.0 makes the horizontal track where the granularity is 1 a little small for my liking - there are a _lot_ of places where adjusting by 1 is probably what you want to be doing.
it should be ``static public`` according to the CS
I believe that the db:schema:load is a no-op, as we're in Mongo-land?
Don't use `import *`.  Just import the names you need.
all but ``node`` and ``nodename`` point to documents or collections of documents. ``nodename`` points to a scalar and ``node`` points to a low level class instance of a PHPCR node.
This should actually become a plug-in for the Security component or better be removed at all, since it is only necessary for running the Integration Test Suite.
any explanation on why utf-8 rather than utf?
```
author_name=getattr(item, 'author')
```
(if None value is correctly handled by feedgenerator)
self.logger -> self.log
Could you give a little bit more documentation about what this really is,
and how it should be used?
why call a method?
I ran into this issue again on the next feature and decided to mock the model's save function and assert that it wasn't called, so I'm going to do that here as well.
The smallest of nits :-) Can you provide a more robust comment? Perhaps add a comment to the pattern that you took the idea from, was it Cédric Beust TestNG post?
Actually, none of the forms work for groups is necessary since its already cleaned within views. Just updated the pull with set_membership line and clean_groups def removed.
In case it looks weird: that's to avoid PHP short-tags problems. (There may be a prettier solution.)
For clarity, I would this amend to "...all the fields associated with a work in the database, including summary, notes and tags, but not the full work text."
We probably want to keep this in line with existing naming standards, so invite_request_declined.html.erb?
можна з конфіга зразу тягнути
FYI, you can do:
```
response = self.client.get(reverse('manage:users'), {'page':  5000})
```
I think.
it will be modify on next pull request, thanks!
That looks like it must have been fun :)

Not necessary to ship, but eventually this should probably live in utils/stat.js.
You should include the base64 code in the `create_subject` method. Rename the method to `create_base64_subject` and return the urlsafe encoded.

That way we can just set `encoded_string = N3Metadata::create_base64_subject(...)` and call it a day.
You can use a jQuery alternative for DOM manipulation in production. This is quick-and-dirty.
Probably I'm not sure if we will be deleting UpstreamConsumers.
You're right.. there's a problem with that as well...


So I'm thinking ow the $arr property should not exist at all. The `->setParts` and `->getParts` methods should immediately update the `->value` property.
This also means that the `->serialize()` method is no longer needed.
It can still run endlessly, if an interval is used.

So, the behavior for this combination is still incorrect.

If the BYHOUR specifies:

1, 3, 5, 7

And interval = 2, the outcome will still be:

1, 3, 5, 7

While the expected outcome is:

1, 5


A second potential problem is that 'H' is used, which can have leading zeros and may be interpreted by PHP as an octal value. 'G' is correct.

So ideally a bugfix for this is also accompanied by a test that identifies this problem :)
не круто. можно затайпдефить внутри класса и реализовать внутри класса
These two functions will not work in a distributed manner. It is probably easiest to create a vector and then project
Fixed.
PS: The second block depends on the first one, that's why I had split them. But, included in the above var block.
You have to be logged in to be able to **delete** a conten**t** item
> Will do! It seemed strange to me as well, I just didn't want to change more than what was necessary.

Hm ... now that I look more carefully - you did change more that it was needed.
The old code only modifies `$uri` if needed, while your code will always re-assign its value (even if it will assign to its own value).
согласен, прост этот код писался на коленке=)))
Отрицательное число пройдет проверку.
Hello!

Полностью проект залил.
Там и полностью весь проект, и build собранный, и .xib в под-папке.
`ids.flatten!` is required here otherwise specs will fail.  This is because we will end up in the rescue block (which is correct) but `ids.size` will evaluate to 1 because `ids` will be something like `[["some-slug", "another-slug"]]`
@gtanner (cc @dansilivestru) LOL.... :-P.. I knew there would be some scolding, but also the opposite, at having remove that epic, horrible hack. :-D

Also, in retrospect... I realized my performance hit mentioned in the commit message is probably an actual increase LOL.. given we don't... you know, copy tons of files every time!! :-O (says the guy with an SSD...)
This and subsequent doc blocks are missing indentation.
We should document the argument count limit here.
I'm not sure if it is necessary. Why would a user want to click on the plot and not open a new figure? If a user doesn't wan't to open new figures they don't need to click on it ;-)
see recent commit


On Fri, Dec 7, 2012 at 10:26 AM, Alexandre Gramfort <
notifications@github.com> wrote:

> In examples/plot_topo_compare_conditions.py:
>
> > +# Create epochs including different events
> > +epochs = mne.Epochs(raw, events, dict(audio_l=1, visual_r=3), tmin, tmax,
> > +                    picks=picks, baseline=(None, 0), reject=reject)
> > +
> > +# Generate list of evoked objects from conditions names
> > +evokeds = [epochs[name].average() for name in 'audio_l', 'visual_r']
> > +
> > +###############################################################################
> > +# Show topography for two different conditions
> > +
> > +layout = read_layout('Vectorview-all.lout')
> > +
> > +pl.close('all')
> > +pl.figure()
> > +title = 'MNE sample data - left auditory and visual'
> > +plot_topo(evokeds, layout, color=['y', 'g'], title=title)
>
>  I'm addings some cases + guessing. If colors is str, always use that
> colour, if None, cycle through list of colors. Is there btw. a prefered
> order? 'w' 'y' 'g' 'r' 'b' ? if not I would just take the COLORS list
> specified in viz and prepend 'w'
> itertools.cycle on ['w' 'y' 'g' 'r' 'b'] is good to me. indeed. I can
> reproduce the swiping… darn...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/229/files#r2346337>.
>
>
Please use Jslint to fix the style of the code.
Consider adding static factory methods to make this code easier to read.
Wrap the second item around ``_()``
`zstyle` contexts should being with `:presto:`, not `:zsh`.
it could also be an object implementing an ``__invoke`` method (which is what closures are in fact)
请删除空格
Totally missed it. Ignore me then. :)
I added an abstract class `UntypedActorWithStash` that can be used from Java in a way analogous to `UntypedActor`, but with the additional stashing methods. There is an example in the JavaDoc for the class. I also added tests for the Stash Java API.
Must have been a relic of a longer call statement. But it's fixed now.
use argstr="-R %s' in the trait
Или просто
```
if (!UnityDetected_)
    return;

if (aep != ActionsEmbedPlace::ToolsMenu)
...
```

Ну, чтобы не оборачивать все в if.
Да, это 6ыло 6ы здорово.
¿Este se identificó _dentro_ del sprint?
default should be true
should be on top of the class
In the readme under build - configuration you can find the instructions what to do to avoid build issues and to add it here. The maps api key is related to the applications usage of google maps and if you want to test this you need to register one for you personally
Ennen tätä pitäisi vielä tarkistaa onko vain toinen sekvenseistä tyhjä, ja palauttaa `false` jos on.
Any reason for changing the copyright of the file?
Not a complete redesign by any measure, I think just creating the InterceptorChain with a null firstInterceptor and adding all the interceptors only after registration should suffice.

InterceptorChain.addInterceptor() does look a bit dodgy, but I think it should work even when there is no interceptor without any change.
Mircea, are you sure this is what's happening in ConcurrentCacheManagerTest? I looked briefly at the test code and it seemed to me like each configuration was a completely new instance - so the TransactionManagerLookup wouldn't be shared.
Also, javadoc explain when/why to use this method would be useful.
Docstring still needs work to explain exactly what warn and force do.
Also, is there a reason we are not waiting for the home screen to be displayed (or rather for the Clock app to _not_ be displayed)? Perhaps I added those conditions to my test because the emulator was slow and you are not needing them on the actual device?
Methods within classes should be documented using our [commenting standards](https://www.plymouth.edu/webapp/mis/wiki/Comments_and_Documentation).  See also: http://www.stack.nl/~dimitri/doxygen/commands.html
Fixed, forced pushed
@jsloane Can we combine this `if / else` with the `if` above?
In both cases we're checking whether `selected` exists.
ahh it is the multiclass one right? but isn't libocas then using
1...<nr_classes>? so this should rather be +1 ?


On Thu, 2012-08-09 at 05:15 -0700, Sergey Lisitsyn wrote:
> In src/shogun/lib/external/libocas.cpp:
> 
> > @@ -1170,7 +1637,7 @@ ocas_return_value_T msvm_ocas_solver(
> >    /* initial cutting plane */
> >    for(i=0; i < nData; i++)
> >    {
> > -    y2 = (uint32_t)data_y[i];
> > +    y2 = (uint32_t)data_y[i]-1;
> 
> IIRC It is different in libocas and shogun so every data_y[*] should
> come with -1
> 
> —
> Reply to this email directly or view it on GitHub. 
> 
> 

-- 
For the one fact about the future of which we can be certain is that it
will be utterly fantastic. -- Arthur C. Clarke, 1962
Oh, the scss was merged from 3.0, but the css wasn't (and couldn't have been, because it wouldn't have been able to compile). Thus the compiled output lists all the changes made in the fixed merged scss. :)
Думаю, если в ```current_user``` булевые значения не хранить, то всю эту конструкцию можно было бы превратить во что-то попроще, наподобие:
```ruby
def signed_in?
  current_user
end
```
<b>почти</b> эквивалентно тому что выше
You are best to skip ahead to cache_pool.h before looking through this file.
What if asserts are compiled out with NDEBUG? Would you rather just exit here explicitly?
Might make sense, I'll have a look
There's a typo in "guidlines".
All the same comments as for the above similar code apply.
Can you put this in the function docstring instead?
Did this package change with newer versions of Debian (in which case it would be better to have a switch rather than changing the package name)?
@gavinking thanks for confirmation
Здесь и далее лучше заменить строки ('admin') константами (какого-нибудь config файла на данном этапе будет достаточно).
Это поможет поддерживать проект в будущем при изменении параметров.
i am fine introducing the concept of transformers.
but i prefer that we leave them out of the service configuration and as you point out simply add a section in the README with this example and how to define a custom loader service that uses a custom pdf transformer service.
Good point. I'll let Vince know.

Looks like the `:` should be a `;` in the entity name.
:+1: on the shorter title ;)
Should we find a better naming ? I think it's not so bad...
You mean in facebook callback view, when caling authenticate? I already did that.
This regex should match the regex of the username form. Could you maybe move this regex to some top-level constant?
Ok so I have to fork 3 repositories, fix their code and make pull requests just to get date working in that format? :-) I agree it's a good exercise but I don't agree that's how it is in real life. In real life I would be already fired for spending that much time for such a simple problem. :) I don't know even what was wrong with he format we were using before... But ok, I'll fix this. Tomorrow.
This string can (and should) be localized. You probably do not want to use it as form input name.
`columns`!!! Decide how you name this. What is `columns`? List of columns? Or number of columns? Name it the same in caller and callee.
{% for user in online_users %}
            <li>{{ user }}</li>
        {% endfor %}
why do we need this? how do I update userlist?
I think all views in `django.views.generic.edit` are accessible also through `django.views.generic`. Try it.
Or, of course, you can check the stacktrace yourself and see where is that called and why it gets through. It seems they opened some new corner which calls/assumes relational database backend.
*wlan slovenija* is name, this is not translated.
`(prefix+'etc').mkpath`
It will compile with a "wanky" patch. It actually works but I can't guarantee that I will maintain this patch or that it will work with HEAD.
I'm guessing you're using tabs instead of spaces. All of our HTML should be
written with two space indentation.


On Sat, Nov 17, 2012 at 4:27 PM, bkgomez <notifications@github.com> wrote:

> In fullhouse/templates/dashboard.html:
>
> > +		</select>
> > +	      </p>
> > +	      <p>
> > +		<label form="duedate"/>
> > +		Due: <input type="date" name="due" class="due-date"/>
> > +	      </p>
> > +		<label form="assigned"/>
> > +	      <p>Members: </p>
> > +	      <select multiple="multiple" name="members" class="member-list">
> > +		{% for member in user.profile.house.members %}
> > +		<option> {{member.user.username }} </option>
> > +	      </select>
> > +              <p>	
> > +		<input type="submit" class="addtaskbutton" value="{% trans 'Add Task' %}" />
> > +	      </p>
> > +            </form>
>
> Oh my lord what the hell. It looks right on my end but now it looks messed
> up.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160555>.
>
>
Any reason for this and the one below not to be removed then?
I always forget about `presence`, nice.
and it would be readable?
sLog->SetRealmID(realmID) is already being set, but it doesn't update the existing appenders with the new realmid, so they have to be destroyed and recreated with the correct realmid
UpperCamelCase here
test folder does not exist
`collectionPersister` is from the main side but the type is from the reverse side. In other words, if the collection is a one to many, I know I am looking at a many to one on the other side and I do narrow the search.

Yes `MapType` is a subclass of `CollectionType` in the Hibernate hierarchy.

I will add some comments to clarify that.
fixed in 785d0a1
It violates E_STRICT to override a base method (`setUp()`) with a different signature (`setUp($proxy)`). Because the parameterized method is not directly called by external code, I just change the name of the method to avoide the override conflict. It might be cleaner to seperate `setUpProxy` into two method, one `setUp`, which owns basic setup code, and one `doStuffWithTheProxy`. Thoughts?
Move the bracket up on the previous line.
Can remove the parens.
The 120Kb usage only occurs if the second parameter is `true`, but we all call it with `false`.  Although it is arguable if it is better, it seems your guys feel more comfortable with the call twice way.  I pushed a new commit.  Please review.  By the way, I think the real benefit of this change is that all `kTISKeyboardLayout` items which are more likely to be chosen by users appear naturally first in the list.
This one can't have the same name as the old field with a different meaning! Change this to "rqServerLocalHostname" or something. In general for compliant clients this field is not even used.
Oh, right I was still in the "plugin world" where `/completions` contained only additional completion files. I'll fix that.
I meant correl_data = correl_data = .....
(base - это самый простой, базовый аплод файлов через POST)
I'm not so sure I agree with the renaming of this method. It's not the *mulberry root* (e.g. where you clone Mulberry to) but the root of the application that you are running the command under. And if you're in a subdir, it goes up up up until it finds the actual root.

I don't think 'get_app_dir' is better, either, but if we're gonna change it, I'd prefer it actually is what it does. Maybe find_application_root or something?
Which editor are you using? In general do not use tabs (but spaces) and set to use 4 spaces everywhere.
That's supposed to have --- under it