labofsahil/github-event-dataset-2012 huggingface dataset:
You should really look at phpBB's code. :P 
"A" and "The" are used all over the place.
This function should also use generate, in general try to avoid using abbreviations unnecessarily.
Can be made prettier by creating the request and adding the attributes after that.
$user is injected on every use, I just dont want to change the order of the arguments and still allow being able to just use the default params
How did this name come about? At most this is an insertion point, not a link. I'm guessing there is no requirement that inserted content be a link to anything, much less to index. If "index link" means "link to index".
Should be disabled (0) by default, imo. When a new addition is added that changes the existing functionality, it should be as unobtrusive as possible (i.e. current functionality should be retained by default, and then un-required changes/improvements should be configurable via the ACP).
For what is this blank line here?
its actually declared in the big var above.
I assume we don't want to bump the bundle version if other stuff in the repo which is not part of the bundle changes.
Not needed, since overridden by admin defined Formulas.
> CppUnit::TestFixture is inherited from TestCase

Vice versa according to the class diagram, hence the question
This should be in a header, right?
Why do we need "default:" here?
- Swap _TModel_ with _var_
Please use "buildInputs = [ autoreconfHook ]" instead.
The type here should be int
When pull from your gpu_conv_faster branch and try to run this, it fails because the device number has already been extracted as an int.
Removing the subscript operator fixes it.

In theano/sandbox/cuda/\_\_init__.py:298

```python
elif device.startswith('gpu'):
    device = int(device[3:])

...

use.device_number = device
```

I hope this is the correct place to make this comment, this is my first time using github (or git).
It took me a while to parse this.  A comment would help:

   # CudaNdarray.take only supports ndim <= 3
Ok I'll do that in another PR
":" missing after function declaration
You changed the fct name. I didn't saw this. You need to have both fct I think. I'll fix it.
Is this copied from somewhere else? Why is it needed?
Any reason not to suppress CloneNotSupportedException. Will super ever throw it?
don't validate in __init__, use def clean

class ContactForm(forms.Form):
    # Everything as before.
    ...

    def clean(self):
        cleaned_data = super(ContactForm, self).clean()
        cc_myself = cleaned_data.get("cc_myself")
        subject = cleaned_data.get("subject")

        if cc_myself and subject:
            # Only do something if both fields are valid so far.
            if "help" not in subject:
                raise forms.ValidationError("Did not send for 'help' in "
                        "the subject despite CC'ing yourself.")

        # Always return the full collection of cleaned data.
        return cleaned_data
*Please merge with upstream allegro repo first*
just return true / return false please.
It does now - have just fixed it.
Wait, no, it's fine there - at worst it'll set it to null when it's already null.
@dfreudenberger because of a crappy decision at the beginning (we have the same issue in collections). It can be changed in the EntityManager and the Connection (and will be needed if the interface enforces no typehint) but cannot be changed for collections without a BC break unfortunately.
Lot of logging for test has been left around (cargo culted from other code in fabric)...
Let's move the mnesia stuff to app:start()
Thanks, I forgot to remove the log level option.
Is there a particular reason why debug is always on?
This is a change in API. For instance, .promise( true ) in particular does not yield the same result.
Sorry, I will revert cdf to yours version. In the past before I don't know why you implemented your own version
Does it make sense to have a list of outstanding tasks  here?
Well, that's lame. It bugs me when people use tuples like this, but that isn't something that we can fix here.
This whole block would just be better as 

    RHC::RerunWizard.new(config.config_path).run ? 0 : 1

Also, RerunWizard is a wierd name, makes me think I'm watching television.  RerunnableWizard or ReentrantWizard would be more appropriate (use adjective or verb instead of two nouns).
You need to be resetting the storage user - I realized this was a bug in our code just recently.  Change I am making:

    Before('@geared_user_required') do
      $old_username = $username
      $username = "user_with_multiple_gear_sizes@test.com"
    end
    After("@geared_user_required") do
      $username = $old_username if $old_username
      $old_username = nil
    end
I was trying to look for places where this might happen and it looks like I might have missed this particular case.  I think that was the only one.  But I think it's more convincing to just return if <= 0 rather than continuing.
I think this needs to pushed down into `_handle` because this level makes it really hard to tell which middleware the `MiddlewareControlFlag` was raised from.
> You can not change file once it is committed

Why? Of course I can. The sentence "you can’t modify a file with information about the commit after you’ve committed" makes no sence! How git supposed to modify a file with information about the commit _before_ it was actually done?
How about just returning a DatetimeIndex instead of a raw numpy array in this particular case
Good call.  Agreed.
Ugh.  I agree.  Please contribute a better name. :D  

Seriously, I'm in.  I'm just having total naming creative block. 

And i'll try to keep that OCD rule.
Check if the nick of the user who joined is the same as the nick of the bot here?
Can we use PE like :  `dep=${vdep%%_*} ver=${vdep#*_}` rahter than cut here?
Here, and everywhere, I'm trusting you on the typing.  I'd recommend you reach out to the junit and hamcrest lists, to get people to compile this with their favorite Java SDKs, to make sure that the generics match up on all compilers (it could be that everything's just that much more standards-compliant than the last time I tried it.  If not, there's bound to be someone broken, it seems).
More code repetition, see @wence- earlier comment.
Two things:

 - Both `readyToRender` and `waitForHTML` can be redefined or in the case of the latter just set on the prototype in client code. `waitForHTML` means "this view will populate it's node with the string returned by the `html` accessor" . In the event that you defined a lazy accessor for `html` and your view didn't have a `source` property, you'd have to set `waitForHTML` to true in your code, yes. In this sense I think `waitForHTML` is the right name.
 - `waitForHTML` is inspected in the code for `View::render` to make the decision about weather or not we should populate the node with our `html`. In other words `waitForHTML` indicates weather or not this view is template based or if it expects to be given an already existing node replete with children and jQuery. In this sense `waitForHTML` is a crappy name.
Probably want to check `response.statusCode !== 200` here.
Doesn't need to be in this commit and was probably inserted by your editor. Would be fine as a clean up commit on master outside of this topic branch.
Just to make sure I understand -- so "before.jstree" is a generic event that's fired immediately before every other jsTree event?
No, it's fine to leave it in.
And same for the next two tests also (relative before/after).  I think it'd be better to leave all these the way they worked before -- each creates its own top-level menu to play in, and adds all the items it needs/expects for its specific test setup.
Is there a name that makes it more clear that the outcome is completing the operation rather than canceling it?  Maybe something like forceCommitRename() or forceFinishRename()?

Also, we might want to go with something like "FileNaming" instead of "Rename" since it's not really _re_naming an existing file in this case.
Oops. Missed this during the rename. Thanks.
Yes, I understand that it won't happen as your code is currently written. But, eventually someone else may come along and change your code, or use that function for some other purpose.
Ideally the segmentation would be parameterizable, so e.g. you could segment the CSS selector search mode on dot, space, #, etc.

For now though, you could just add a comment saying this block is special-case code for handling paths.
using elifs instead of continues here might be a little cleaner and more maintainable
Probably also a util for this, no?
同 foto_top
用一個就可以了
Views or signals? In views you just raise exception and with `DEBUG` set to `True` it shows you technical response.

In signals exceptions are not propagated (I configured like that in case of passthrough view).
This check is basically there to say "Is start invalid and was it provided?"
Can you remove references to `style`, and use CSS classes instead? Want to avoid storing magic numbers/style within code like this.
Just curious: why do you attach `alreadyFired` to `runCheckFn` instead of just making it a flag variable inside this loader function?
Where does this get used? I'm assuming something in the photo/zoink editor will eventually use it, right?
该方法实现有问题，参数不完整，请严格按照API定义实现

http://docs.qiniutek.com/v3/api/io#upload-token
Fixed!  Good catch.  (Sorry for the delay, was out of town on vacation.)
Please add a comment mentioning that ActiveMQ is to blame for this.
This line is the only place where we actually test the merge, right? Could we do something more rigorous than just checking that the file size is larger?  I would think we could use a similar technique to that suggested for the Agi-IQ-only test above.
Two things:

1. `s/.bash_it/$BASH_IT/` so its relocatable.  Since I just merged the other PR about this, it's sticking out. :-)
2. What about adding a check for environment variables of the same name and not overriding them?  That gives the user a chance to set them in `~/.bash_profile` to be custom.
One more question: Why is `zip_safe` set?
nexus runtime means ' what common bits does nexus runtime need at runtime'
probably should also include rar
I have been thinking about this and my opinion is just to store "once" scripts there, since katello-upgrade log is not being deleted (it's beeing appended only) and we have also time/date markers there. Therefore the log file is what you are looking for. If there are no objections, I would keep the current approach.
Sort of - I just sent an email about it. I'd like to move all of the custom headpin stuff to headpin/katello-cli-headpin and change the build process a bit for building that.
Is this typical for community projects, or would it be better to point to katello wiki? The IRC channel could be pointed out prominently there.
Okay. I added names for some of them because (I think) you need names in order to do the weird template-y url stuff, but I'll look and see which ones are actually references.
I added a comment on this line because the KeepAlive name itself was bothering me due to its new implementation mechanics.  I couldn't think of something better though (hence the comment).
Typo: modalbox-zentyal.js
Should we log some kind of error here instead of silently failing?
Changing keyword to `multiprocessing` does not work because it clashes with the `multiprocessing` module that I use for multiprocessing. Any other ideas?
https://github.com/axsh/wakame-vdc/commit/0ca1fdeb336c8714248aff6d6d6229ce9e5f3c24
because I wasn't ready to add the idea of the prereqs phase to the test runner for this PR. should be removed.
If I would move this to handleURI, where I have an unprocessed QString instead of a pre-processed QUrl with "//" removed from "bitcoin://" this would bloat the code a little, so I don't like that idea.
Small typo /unkown/unknown/ :)
Why the extra new line?
You could refactor this test to use a dataProvider.  It might make the test clearer.
@fredj, has this change been made upstream in the sandbox?
Line's too long.
You use `()` here and `[]` for `anchorOffsets`. I think we use `()` elsewhere.
For more flexibility I would make `cgxp.FloorSlider` a subclass of `Ext.slider.SingleSlider, and possibly add `cgxp.FloorSliderWindow`.
I don't understand the description of the `maxMeanAll` option.
Yes. Under our copyright we can add a comment to indicate that the code is inspired from code in GXP from the Open Planning Project.
There is a Migration and a Model with the same name `UserStatus` so including the `JabbR.Models.Migrations` as a using statement won't compile. Rather than trying to change the name of any files I opted to just fully qualify the type.
you mean to don't have it as field at all...nice catch
tone it down a bit. We're not selling Shamwows!
Was the intention for non notification things, such as the web and api interfaces, to be moved into gateways? I thought it was just for things responsible for talking with external notification systems.
line 240 above this one, there is a
```
if 'rep' in h :
   continue
```
Maybe we should change it for `'rep' in h or 'recall' in h`.
attachVolumeToServerAsDevice(volumeId, serverId, device)

^^ helps disambiguate various string parameters
You can avoid the cast below by writing the method like so:

    - (BOOL)shouldUpdateCellWithObject:(NISegmentedControlFormElement *) segmentedControlElement {
It is now possible to use it like this:

                    $table->increments('id');
or
                    $table->increments('id',3);
or
                    $table->increments('id')->length(3);
or
                    $table->integer('id',true)->length(3);
or
                    $table->integer('id',true,3)
or
                    $table->integer('id')->length(3);
or
                    $table->integer('id',false,2);

I tested and worked correctly every time.
On Fri, Dec 21, 2012 at 7:16 AM, Benjamin S. Kirk
<notifications@github.com>wrote:

> This in particular looks like a lot of code duplication - can we do
> something clever with templates instead??
>
> Similarly with the ProjectFEMSolution vs ProjectSolution classes - could
> their common functionality be in a base class or templated class?
>
It is quite a bit of code duplication. I'm not sure templating will work in
teh ProjectFEMSolution case because the algorithm is slightly different
because of the construction of the FEMContext object (in particular, the
element and variable loop order is switched to avoid superfluous element
reiniting). Nevertheless, I'll look into trying to remove some of the code
duplication.

However, at a higher level, I've been silently wondering whether we can
encapsulate these element-local solves better because very similar
algorithms appear in adaptivity, Dirichlet constraints, periodic
constraints, etc. In particular, HCURL/HDIV continuity has been on my TODO
list for awhile and part of that was figuring out all the different places
that would have to be put in. Would be nice to only have to do it once.
Would that be better?

git_object_type(object) == GIT_OBJ_BLOB -> GIT_EINVALID
target_type == GIT_OBJ_BLOB || target_type == GIT_OBJ_ANY -> GIT_EAMBIGUOUS
Ehrm... I meant that this is supposed to be `git__free`...
But our current `git_clone` API does neither allow to give a remote name nor to specifiy a refspec directly. Thus, the remote is always named `origin` (currently).
Capital I \</wenzel\>

Also I think this comment would be more useful before the `if force_direct` part as it helps explain why we're setting download_link_direct.
Can you change this to `r'^foundation'`? And remove the `foundation` prefix in `apps/foundation/urls.py`. This will avoid a bit of namespace pollution.
I will change but curious, what is the reasoning behind it?
awesome, thanks in advance. This should really be a dictionary with just the data that you use to build this :)
origin isn't defined here IIRC.
Well, what we really wanna do here is to check if the destination folder is not existent. We do not want to install Firefox to a location which already contains other files and sub folders.
This is probably fixing a bug that I think may be fixed or partially fixed upstream.
Shouldn't that be `super(CherryPyStandAloneApplication, self).__init__(*args, **kwargs)`
Massive difference in whitespace.
Ugh, this is quite ugly. But I assume there is no way around it. Is it possible to "selectively" enable globals just for the thrift tests?
No, since the rest of the N64 time code doesn't work in general. It's just faking values for days/months/years. (and, according to the comments, it looks like it won't ever?)

I'll add that error for the other two ports though.
As long as the lever is attached to the block the IC sign is hanging on, levers on the ground will be triggered.
typo: ...OptionWasRead
Thanks, stof. Pushed a fix.
As said before, does it make sense to leave in buildViewBottomUp for BC if users have to adapt their types anyway? (signature of buildView and getParent was changed also)
You should also try to use iconv if it is enabled and mbstring is not.
Can you make it more clear?..
增加baidu.plugin和baidu.plugin._util_名称空间
What is endPoint ? To document here and below in the specializations
I don't understand that API. 
First, the result should be a Box, because updating could fail.
Then, why the old ID is needed by the caller ? What happen if the caller give some bad id ? 
I thing the user only want to store some id (and maybe be able to check the result).
So, something like: """def updateLastId(newId: Int) : Box[Int]""" seems cleaner.
This whole file is the old way.  We've revamped the testsuite, you need to update and do it in the new style (which is much simpler!).

And it's clearly not the right version -- this tests oiio.ico, which is not the right format and obviously doen't match the ref/out.txt test output you checked in.
Is the result always well ordered?
Hmm? how? O: Isn't player the SpoutAPI player?
i think "ALL" isn't a good name convention. all could be all ;)
I see this piece of code everywhere, why not add a static method in eZTagsObject, something like showHiddenTagsEnabled() to have it at one place only?

Also, if the setting is commented out by default, you should use hasVariable in order not to fill up error/debug logs unnecessarily
How did you test this? I've never localized a manifest before :) only text in the  application.
shoud be replaced with:
subsystemRegistration.registerOperationHandler(DESCRIBE, GenericSubsystemDescribeHandler.INSTANCE, GenericSubsystemDescribeHandler.INSTANCE, false, OperationEntry.EntryType.PRIVATE);

as you no longer need to provide it yourself
This is because one of our features that is automatically tested in our test suite.
The feature is available to our users/customers too. 

To be concrete we're extending one of Apache CXF WS security classes in our endpoints
and the extended class uses @Resource based injection. We have been handling
@Resource injections on our own before this PR. With this PR in place,
we need to jandexize all the modules where we want to see injections.
(IOW leaving AS7 to take care of injections - no more hacks in our WS components).
My guess is that you needed to disable draggable for this to matter.
Thanks for the pointing out!
well, how that worked before...
Don't know why we are searching for those 3 spaces, is not simplier to rebuild the content we want to display?
+1. Fixed ;)
typo -> "openTrustedUI"
I don't understand the logic here. So hardware volume buttons will only be hook into 'notification' (ringer + notification) and 'telephony' (oncall), but not 'content' channel? I cannot change volume of the music Music app is playing with hardware volume buttons?

Also, with the removal of music channel, how would Music app getting background playing permission.

I need to be informed in order to do the review.
Yep, you are right :)
I would prefer use the 'load' method (that is the one that refreshes the list) and checkEmptyList always there after finishing,  instead of modifying 'refresh' method, that is only refreshing 1 specific contact. What do you think?
Why does classList.contains fails? Is it because the classname is a composite of message? something like foo-message or message-foo?
The fact that it works in one app is encouraging, but doesn't mean it absolutely won't cause regressions in a different app.

@cgjones do you think this is likely have a big enough performance benefit that it justifies the risk of change?
It was named this way because the file was copied from a `background.html`. Let's correct it.
Already discussed in IRC. But we should address this later.
nit: does the important is really needed?
Remove unused code please!
Just curious. Is it possible to add callbacks as parameter to the openDialog method so you may know when it is close and do what you want instead of assigning directly a handler to the onclick of icc-stk-app-back?
Resolved - this are now lazy loaded.
I'd like to take credit, but I lifted it from StackOverflow: http://stackoverflow.com/questions/436159/how-to-get-all-subclasses
There's quite a bit of duplication here (e.g. `feature.name.split("\n").first` shows up 3 times).

Can you pull out some helper methods for this kind of stuff?
> Just watch out that you don't repeat so much code twice. I think it should be one method with two different cases in the end depending on whether its OutputForm (-> string containing "_") or another form (-> SubscriptBox).

I was worried about this and didn't like my solution much mostly because of this.

The problem mostly was that I didn't know about OutputFom until now.

I do now and the last commit should fix it and should make this branch ready to merge.
is this safe even when the function is not in the `converted_functions` kwarg? I would like to see a test case involving `Derivative` with a function that has attribute `HoldAll` where its arguments are not touched when converted to sympy through e.g. `Apart`, but only when `DSolve` is applied.
Same with most of the selectors earlier in the key handlers.
Does the inspect return nothing / empty string? Curious if that's what you are trying to convey.
can also probably just be ChatManager
is there a specific reason why? because you will in 99% of the cases do !Server.isWhitelisted(player)
This docstring is now in the manual, so it shouldn't be replicated here.  The manual may require some adjustment for this new addition.
Noted. @EvanDotPro, is it not possible to pluck the cert from the connection in use itself? There then wouldn't be a need for the second connection.
Great job tracking down the correct validity attribute!
Well, since you changed other code as well (debugging) these values do not reflect the bit field changes. I will check this for myself. Stay tuned.
shame on me.
content[CUT, 2]
I see ... nice trick ;) ... but I'd rather see the `(function() {})()` style ...

* it makes it "clearer" (through the extra `)`) for the reader that the character in front (i.e. `!` or `(`) is related to the invocation
* it avoids "overloading" `!` with another "meaning"
oh guys. Please split this one.  80 chars per line recommended
Completely agree, this is a horrible solution and :baby: may be hurt because of it, but it's a quick fix for this campaign. 
Did try to do it properly but that template doesn't have the row info inside it so don't know where in the loop it currently is.
we use:
typedef struct Name_ {
    ...
} Name;
A manually constructed object with, for example, DB=False, would have unexpected output. Adding "if y" would avoid that.
`<l:hasPermission permission="${permission}">` is redundant after this test, since if you have administer permission on all of Jenkins certainly you have it on the view too. But I do not think you have adequately explained what the purpose of this pull request is. The only permissions you should need to edit a view’s description is `ADMINISTER` on that `View`.

Anyway the `<l:hasPermission>` in Jelly just shows or hides the link to `/editDescription`; a user can always manually jump to that URL, and edit the description if they are authorized to do so. (`View.doSubmitDescription` actually checks only `CONFIGURE`; `AllView` checks `ADMINISTER`.)
code cleanup of transitional code for issue #1592
do you want an empty string here, or a null?
again, why comment?
Should this query have an additional sanity-check in the where clause to ensure the `worker_id` matches, so workers can't mark rows complete that they don't own?
I don't think this change is correct; the argument is `return_type` with lowercase "t".
What about filing a bug to let `nightlyApp.oldUpdateTitlebar` a getter / setter, like here in `suite.js` (and also with the other checks)?
Yes. The img can be removed. It can be called from the css. But, the problem is, we need to add in extra tag for each page just to call different image for different pages. Remove the h1 tag only, perhaps?
none of the ``else`` are necessary
This is going to be a dumb / naive comment / question.  The implication of the addition of "environment" here is that the master can distinguish between plugin directories for different environments... how is this done?  A different "modulepath" per environment?  Is this something we've always supported and it was just determined implicitly before, whereas now it is an explicit part of the request?  Are there any security implications from allowing the agent to specify this explicitly?  I presume that the worst that could happen is that an agent could be hacked to retrieve plugins from a different (but existing) environment, which is probably more of a shoot-yourself-in-the-foot situation than a security problem.
Why `nohup`?  This is a smell that Puppet isn't behaving properly.

Also, this appears to start a daemon process using the `--no-daemonize` option, which is surprising.
httptester doesn't use the new generic view structure, so maybe we should include an example like Kannel?
Good idea...I remember there was a place you did that previously.  I think we may want to look into extracting that into something common (like a mixin) so we can just say `method_from(@object, method_name)`, which is more expressive/less noisy, plus it could potentially be faster by caching the result of `::Object.instance_method(:method)` since that is essentially immutable.  (Although a user could of course redefine it but you'd have to be crazy to do so and all bets are off when you are doing invasive things like that).
Why not allow GET as well?
kind of ugly, perhaps accept an Option[Int] param that prints progress after count reaches that value
nvm it's not
Basically, `devhq_header` needs to be full width, whereas container is only 960px wide. Really, `#container` should just be a class...
Remember change underscores for slashes
@Leftbrained The test was good, only the implementation was the problem.
"widget.ui" seems a little bit over designed, what do you think?
This should be `data-title` or `data-status-title` not `original-title`.
Good grief. People actually use this :)
Good point.  IsNullOrEmpty seems safer.
testing code commenting :D
Yeah for now you can trigger a reload at the next page load using the same technique used on the package creation (some hook that triggers a click of the "Force Update" button in JS). Once upon a time we'll have a proper job queue.
@FabioBatSilva maybe it's too much of an edge case :) No worries, just ignore it... Bundles extending other entities should be aware of what problems they're running in :)
Yes, copy & paste ...
See note re: `before_filter` refactoring.
There might be directory traversal attacks that bypass this replace filter (eg by putting  a NUL char between the two dots). I think it is safer to use php's realpath() function to check if the requested file is within the root of ez
This is already [mentioned below](https://github.com/ezsystems/ezp-next/pull/128/files#L14R18)
+1 to this as well. `label` and `data` are getting leaked as globals.
this is taken care of in the consumer of this.
I will update the contract in the javadoc but you don't need to send null. Just the fields that are defined
Should probably throw a TODO: or FIXME: in at the top.
this should not be removed
add whitespace padding: `})( Popcorn, this );`
This sort of thing tends to be cleaner if you do something like

```ruby
@response = parse_logoutresponse(@raw_response)
```

and ignore class state inside the parsing function. That approach might help with the general ugliness of the `parse_samlresponse` method. Don't write methods with side effects when the side effect free version is just as easy to write. :)
using `ComponentManager.get(ContentHostingService.class)` would make refactoring easier.
`no` instead of `false` please
Yes, we can. But may I ask why? I thought `||=` are idiomatic ruby way.
I think its's a good idea, at least on internal tests. It shouldn't be required for users, though.
If Image just contains a pointer, no need to have a pointer receiver. I think "func (buffer Image)" would be more appropriate, but await your comment.
I apologize, I Know almost nothing about how Snap works yet :) Tomorrow I'll dig inside the docs and I'll write a cleaner error handling :)
Please be patience :)

Cheers,
A.
There have been a few requests for special support for classical works over the years, but I think the best mapping we have for now is book.
Please use a lowercased boolean: ``true``
would rather not tie UI logic to library code
Could be accomplished with intval($_POST['ch'], 8)
backticks around `inline code`
Oh yeah, I'd forgotten I'd changed ``socket`` to ``legacy``.
only tiny complaint:

```js
js.write(umd.join('\n'));
```

should be fine, maybe ditching the last `\n` in there
inline или в c++
this should be moved to its own file
Kun je je indenting hier fixen? De rest van de code gebruikt de stijl om de control-statements gewoon op losse regels te zetten.
I think you might be able to make this method a lot faster by just caching the widgets during __init__ and yielding them for a list. Haven't checked closely enough to be 100% sure but it's worthwhile thinking about when one goes through the config. Perhaps more importantly, it might show a large class of errors on start-up rather than when a page is rendered.
Hmm... if it's that much slower then perhaps both cases should be written using `into`; I'm a little surprised that it'd be that much slower!
Yes, exactly. If you look at the particular implementations of `compile-term`, you can see that none of the maps returned actually has a `:joins` key.
We're available sure but not at beck and call. Not sure how to phrase that.
Thanks kannan, It is fixed now,
Still needs to be below the next statement.
http://www.postgresql.org/docs/8.2/interactive/ddl-constraints.html#AEN2058
The public alias should not be exposed when a user requests a user he can see, otherwise he'd be able to track back the user once he hides his profile.
In the latest versions of Chrome and Safari this animation happens even when the page is already scrolled to the top (it jumps down first and then scrolls up). I suggest we leave out the animation and just reposition to the top of the page. What do you think?
Fixed this and other instances.
please prevent this duplicate code
We also need translations in the yml file for global.waiting & global.requesting
Should this be un-settable for now?
It's a suggestion for certain formatting/style guides in research and other published works, two spaces are placed after periods to make it easier to read and see the distinction between sentences. It is optional in most modern style guides, but you likely see it in text like this because it was probably drafted and revised in a word processor by someone that has to use MLA or APA formatting on a regular basis.
В том то и дело, что по условиям задачи колличество элементов строго определено, но универсальное решение было бы предпочтительней. Сам я так же задал длину массива константой, на что мне как на недочет указал мой проверяющий.
Скобки умышленно удалил. Не знал что это по этикету положено. Спасибо, буду знать.
MUST
Don't push this file
Same question here as above re: ClassMetadata extending ClassMetadataInfo.
Både collapse og transition finnes som del av twitter-bootstrap-rails-gemen vi bruker. Prøv å slette de, ting skal fungere fint uten.
You mean we should not add en entry to jump history on toc entry jump?
maybe a short comment how the fmin / fmax works would be helpful for novices, e.g., that they will be zipped and constitute a series of frequency lower and upper bounds...
equal to
I don't like mention to p-values here. Just say 'score'. even pearsonr does not give proper p-values
due to correlated samples on the timeseries
I think the method name should be isDuplicateConceptName
Because CohortService shouldn't be writing anything to the DB, why would it?
Sure. Should this be instead of - or in addition to - defining the acronym inline? Normally in a technical document, I try to spell out the acronym on first use in addition to a definition of the glossary, but I'm not sure how the hypertext nature of this document may make that unnecessary.
Same here, I think you want to write `def unbind(key: String, value: B) = self.unbind(toA(value))`
This isn't clear on what it's doing. Can you make it clearer or add comments to explain what happens?
Something similar, yes :)
I still don't think it needs to be a list of hyperedges.
In the documentation we use a 'line break point' around 72-80 characters, it don't look nice if a link is much longer. If you place it in a new line it looks a bit nicer.
They are used for hyperlinks:

    This is some text with `A hyperlink`_ and you can refer to a doc article: :doc:`/cookbook/templating/twig_extension`

     .. _`A hyperlink`: http://symfony.com/
"... through it for duplicate ..." (Drop comma.)
done, also changed the name of the class to CamelMessage, it does make more sense.
I suspect this shoud be static, as well as have an error return.
A space would help here, `Nk` -> `N k` (or `N * k`).
hey, these docstrings are looking good.
You should remove this, because there is self.subfolders list in main class.
Missing covers annotations for new tests
Depends on how much blocking is going on, this is not high performance computing :)
Maybe add a little benchmark script if you want another number.
Missään ei varsinaisesti taata, että `vals` ja `keys` palauttavat arvot ja avaimet vastaavissa järjestyksissä. Jos haluaisi ottaa tämän huomioon:
~~~clojure
(apply concat (map (fn [[what times]] (repeat times what)) a-map))
~~~
This should be just `Class<?>` and not `Class<? extends Object>`
+1 for this exception class, a more explicit way of informing the user
think the non tx context is the way to go here, bit I don't see the correlation between this and the ClassCastException shown in the exception's stack trace - can you please elaborate a bit why is this fixing the problem?
Would be nice to have an UnitTest describing the problem as well.
Shouldn't be using the metadata repository to check for annotations, rather than inspecting the class directly?

IIRC, the metadata was compiled at build time to enable fast look ups and avoid this kind of inspections.

Check how it works in the core/
Yes, but doesn't gdc has the -lib flag?
This should be marked `copy`: https://github.com/github/objective-c-conventions#declarations
Looks like it was for an older interface or was a copy&paste error as it doesn't apply to this class.
Seem to have inadvertently deleted an 'a' here...
Sounds like you are turning over some stones and finding some interesting "features". It might be worth putting some comments/attribute docstrings in ```matplotlib.cm``` at the ```datad```, ```_cmapnames``` and ```cmap_d``` variables. Additionally, some of these inconsistencies might be improved by making the default colormaps go through the ```register_cmap``` function rather than have them modify the data structures directly (but that is a bigger change, and I can't guarantee that it is entirely sensible).
Thanks.  I included this in commit 92575e2 below.
I know its picky, but would you mind putting 1 newline between methods and 2 newlines between top level classes/functions.
Don't you mean test_launch_app here?
The specific JS logic here should be it's own function in GaiaData
А, ну да )
@willdurand  i don't understand, Vendor are the same in french, like in http://symfony.com/fr/doc/current/cookbook/workflow/new_project_git.html
here ```=implode(``` should go on the same line as ```$create_query``` and every other line should go 4 spaces back
Remove the debug statement.
Does my original test code in #318 work now?
Any reason not to just use unpack-chunk-location and unpack-chunk val separately? Seems like this is repetitive.

```clojure
(def chunk-locval (juxt unpack-chunk-location unpack-chunk-val))
```
@dooremont Could you translate the name & description to French? We shouldn't be adding `french` to it.

There are quite a bit of config.json files that still need to be changed.
style: please move bracke to line 25
I think it would be great if there was more space in the code above, so it's easier to read, what do you guys think? We really need these coding conventions otherwise I see we will have a lot of arguments :)
Aha, já si říkal, proč tam ty parametry jsou.
Noo, ale tu mezifunkci bys měl podle mě udělat spíš ty, ne? Že ten tvůj timer bude volat nějakou temp_schedule (struct timer *tmr, void *data) { schedule (); } .. aby se pro ostatní, co byli zvyklí normálně volat schedule, rozhraní nezměnilo. Můžeš to tak udělat? Přijde mi dobrý se snažit zachovávat původní rozhraní, i když samozřejmě to můžu udělat jak píšeš.
so is this set to debugging now? does this change how we need to push to production?
Why wouldn't a failure to close the cursors result in transaction abort, not transaction commit?
Still one space too many.
> IMO, we should remove this permission and implement a board setting to allow admin to global disable drafts if they really need to.

Which is exactly what I wanted to point out ;) It should be possible to disable them.
The assignment to ``$_SERVER`` seems to be dead. Are you sure it is necessary, or maybe you have a typo somewhere?
Are you sure you want to override the parameter ``$className``? This often makes code less readable.
That's just for methods that return booleans. Something like ``hasRenderErrors()`` would be better here.
Accepted and made the following change

var test = e.rowData.test;
var dataTitle = e.rowData.title;
if (isBlackberry)
{
	test = data[e.index].test;
	dataTitle = data[e.index].title;
}
if (typeof test != 'undefined')

When Alex finishes work on RowData and it will be committed the code should be reverted back to work similar for all platforms
Geez, I know that you're trying to make the dragon sound a certain way By Typing. Like This. But. It is Super. Annoying.
seeds.rb is for data that is required for any instance of the application, i.e. if we deploy it, and then another department on campus deploys it, the seed data would provide whatever initial data they both need.

In this case, I think you want "test data". Check out: http://guides.rubyonrails.org/testing.html . We can go over testing in more extensive detail on a Tue or Thurs if you'd like.
you should provide a branch alias and use it in the doc instead
Only displayed in the admin, doesn't need to be localized.
Static string, `'`.
This moves or it copies?
This could be better written as:

```
layout.column, layout.order = column_ordering[panel]
```
I really didn't like that we now have `slug` everywhere in templates and that keyword in `urls.py` has to be named `slug`, so I updated `mongogeneric` `DetailView` to Django 1.4, where you have `slug_url_kwarg`, so just add:

```
slug_url_kwarg = 'username'
```

also to our `UserView` and you can have usernames all around. :-)
This is point on map's right edge. It is a helper to determine distance from the centered node. I am just wasting time here, I have spent 10 minutes brainstorming what should more verbose name be :). I am not good at this.
`farRightEdge` ?
`object` or `obj`?
Ahh, interesting. I must have been using a crappy Markdown parser before.
are begin/end for rendering? than i would call these methods: renderBegin and renderEnd.
Does Axcess support a Return? If not, does this even need to exist?
const Position protodrakeCheckPos = { 206.24f, -190.28f, 200.11f , 0.f};
spaces, at * and inside {} and unneccessary ; after closing }
Why this change? In Doctrine bridge they're also expecting the option to be the em name, and they're getting the manager from it :

https://github.com/symfony/symfony/blob/master/src/Symfony/Bridge/Doctrine/Form/Type/DoctrineType.php#L117
Done: 75cf3ba
What icon should i use
I prefer test_xxx, e.g. test_sweep_alpha  so it's easier/faster to see that this is a test method
code duplication ? if this is the same, we can just import it from the slsqp version
What do you think if we move this fallback to the default locale into `convert_locale` method? Maybe `convert_locale` wouldn't be such a good name then, but I think this method is used in a couple of other places where fallback might be useful as well.
Laziness? Where does it go? :)
It may make that initial rebase step a little tricky, in that we may have to do something like a manual merge conflict resolution.

But deleting directories won't actually cause problems of that sort with git, because git doesn't deal with directories at all; not having any files in a directory is the same as deleting it.  That's the upside to the various sometimes-it-ignores-directories-we've-deleted problems we've had in the past.
Sorry, I AM a pain!
It helps with consistency in this case!
Rather than rewrite _parse_sizes just to change the error message, we could instead pass a string argument to _parse_sizes the would be used for the end of the two error messages:
```
RadialProfile._parse_sizes('first component of the DoubleGaussian',sigma1=sigma1,...)
RadialProfile._parse_sizes('second component of the DoubleGaussian',sigma2=sigma2,...)
```
Likewise, the other calls to _parse_sizes could specify the actual profile type that caused the error, rather than just calling it 'this object'.
I think I might prefer GaussianNoise for the name of this.  Deviate is awfully similar to deviant...  

I know we already have UniformDeviate, but UniformNoise doesn't work there as well, and I don't have any other suggestions for that one, but if someone has a suggestion, I wouldn't mind changing that name as well.
pushed a minor typo fix here
Should this pass `true` for dryrun, when calling parse?
I'm not convinced by the hierarchy of AbstractPortableTemplateBuilder, PortableTemplateBuilder and ReusableMachineTemplate.
Not sure I have a particularly better suggestion, but it currently looks quite complicated. 
And feels like the "is a" relationship for "ReusableMachineTemplate is a jclouds.TemplateBuilder" fails, because you presumably shouldn't call build() on it, or attachComputeService. It's therefore not substitutable - you shouldn't pass it into a method that expects a TemplateBuilder.

The "abstract" doesn't feel like it's particularly abstract: it could have the build() and method itself, if desired (e.g. overriding newJcloudsTemplateBuilder in the sub-class to set extra fields on the Template).

Is it worth us switching to jclouds 1.5.beta so we can get Adrian's new TemplateBuilderSpec?
Or is that going to cause yet more pain with Whirr wanting to use version 1.4!

So in conclusion, not sure what to propose in terms of changes to this code (as I'm not sure about all the use-cases you're thinking of), but just want to raise it as worth taking another look at.
i think that fails if we enter, say 128 as byte technically in java is -128..127 (ffs)
This should be replaced with `current_active_project = Admin::Project.default_project`
If that method doesn't work in this case lets figure out why and try to fix it, or add another method to Admin::Project that does work.
Hmm, I not know even. Return code seems to have changed.
"it" on this line seems to be referencing "the value of the named property". Referencing the value of the value isn't incorrect, but it's weird. I would just go with "otherwise, return it".
Why do we want to use a `phpunit` collection?
여기에도 버튼이 들어있네요 :)

다음에 작업하실 때는 git branch 브랜치명으로 새 브랜치를 만드신 다음에 하시는 것도 좋습니다. git checkout 브랜치명으로 이동할 수 있고요.

나중에 푸쉬를 git push origin 브랜치명 으로 푸쉬한다음에 작업하신 브랜치에서 pull requeset를 던져주시면 됩니다. :)
you can remove `EMAIL_PORT = 587` and `EMAIL_USE_TLS = True` from this section, don't you think?
This is the blacklist plugin: https://github.com/bergie/hallo/blob/master/src/plugins/blacklist.coffee
I don't think an empty blacklist would work.
I wasn't sure about this. Basically, as I was going through cleaning up the GIT_UNUSED stuff, I noticed that we were calling `free()` directly here instead of using `git__free()` and I assume that is just a bug. So since I was touching the file anyhow, I figured I would fix it.
Changed to only GET
I don't really understand... a example would be great
Compiled and tested. It didn't seem to make a change:

http://imgur.com/I6zOL

(I'll revert and recompile just in case)
Can change it up if you think this is just arbitrary
我觉得在一个项目里最好是不改变风格的：）还是用def joined(member)好一些吧
Single line formatting is preferred for this.
Подчёркивания в названиях недопустимы.
Missing an opening parentheses. Also, this should probably be one sentence.

    Here we use the jboss-javaee-6.0-with-infinispan stack and the jboss-javaee-6.0-with-transactions stack (you can read this as the JBoss stack of the Java EE 6 APIs, with Infinispan and Transactions)
we can add more screen shots  for Hypno 

The issue here is that Checkers is not our app so we can't control anything with it... and if the tests change anything in it the owners might get upset..,.

Hypno is our app so we can use it in our tests...
we could check for it in configure, whether it's needed for the platform it is built on. but i don't think it's needed at all here by this add-on. any idea @margro ? anything dlopen'ed in here?
Can we assert both files?
So...much...code duplication.
This comment is now out of place and should be moved to line generating the data array from a binomial distrib.
This PR only implements a Bernoulli RBM iirc (only binary units), so you can't whiten the input.
We could use the MinMaxScaler now.
right - the magic number... - the problem here is that we don't know in advance how big (=number of nodes) the tree will be - simply allocating the amount of memory for a complete binary tree is too wasteful for large ``max_depth``. So, if we need to store more nodes we dynamically resize the arrays (we double the size each time we overflow). 

I'll add a comment
Maybe you could also store the CVed `best_n_estimators_` value as an attribute too?
I still don't get why you need this attribute since during the fit you can pass active_set_init that should serve this purpose in the path function
```'wallpapers' => $pagination,```
wtf? oO
changed to use a non-specific 1.x version
Should have only 4 '#', 5 makes it look like a sub component of iOS
hash is a builtin function.
And \r, which we will get from the stupid windows users.
Ok, time for a new CF ticket about the crappy default AZ "algorithm" for multi-region
I added a kernel here in case you didn't know how it was implemented. This also lets you play with values. Since it's a signed int, you can give it negative values. Using ints for your filter values lets you keep the cool effects, without having to check for overflow like you'd have to with floats.
It is only optional for actions.  I suspect it would be clearer if it was required for that as well.
Yes, that was the thinking 32bit systems not the priority and makes things simpler cleaner overall.
Moving the Macro into the IF block is unneeded as the Macro will only run if called, and the only calling cmake code is in the ADD_SUBDIRECTORY calls in this file which are now in the IF block.
The strcmp is expected to fail unless logfile_extension() returns "aizenauoizenucaozen.tmp"
This slot needs a better name. Something like "library_directory" or something.
I don't know, but I'll ask Franklin to comment on this.
use ``assertFileNotExists`` so that PHPUnit gives a useful error message when failing instead of saying the boolean is wrong
const for standard autoloader.
fixed in next commit
it's my first time coding with Python, could you explain to me the role of the FUNCS array?
It will throw a NumberFormatException, but I can't see how we'd get in that situation - an Http header without a value is a violation of the Http protocol. We get that, the service is seriously borked and needs to be fixed.