labofsahil/github-event-dataset-2012 huggingface dataset:
Display text in noodall component view

Ref Cutlets issue #14166, component admin views need to have *some* text, in order for the component table highlighting code it work in Noodall UI.
configure jsch to disable interactive authentication

jsch is by default configured with a set of authentication strategies, some of them require user interaction. By definition, jenkins jobs are non-interactive and can't use them, so authentication may better fail than showing a login prompt on console and fail on timeout.
Modelo modificado

Campos Monto Titulares, Monto Beneficiarios, Monto ISLR y MontoIVA agregados al modelo
Update pom.xml to include daofusion lib.

I couldn't build and deploy the project without the daofusion dependency in the pom
Allow data-uris to pass whitelisting.

Fixes blackberry/BB10-WebWorks-Framework#349
Jeffheifetz/whitelisting rewrite

After this change basic websecurity should be enabled. This should prevent iframes from accessing window.top among other things.

This should NOT affect navigation or api usage compared to before. However this will affect XHR usage. 

From now on <access uri="*"> will not allow cross site xhr access, this will need to be enabled explicitly with a specific <access> tag. Additionally an alert dialog will not pop-up however you will see an explicit error in the console of webinspector (generated by webkit). @kwallis @nukulb are these acceptable changes?

This pull request still needs working functional tests, but the code and unit tests are complete.
Return false if a record does not exist

Checks if `exists` key is set and returns false if the value of `exists` is false.
Initial GT-I5510 support without RFS
Added Register view, nixed the anonymous click handlers 

I fleshed out a basic view (email, password, confirm password, create acct button) for the Register activity, as well as set up a basic log + stub handler for the account creation button. I also changed the click listeners for the main activity to be explicitly defined rather than anonymous in-line handlers.
Reverted a the test to expect the hack from PLFM-1287
PLFM-1452: API to recursively apply ACL to an entity and all of its children
Add some comments and README.md
make sure there are no double slashes in the url by using an id with lea...

...ding slash removed
Repeat same address for collector until submit
Fixes for global.error and mounts monitor disconnection

This should correct the points you mentioned from the previous patch.
Update fr.lproj/Localizable.strings

Some sentences didn't make sense :-)
Express: Link to source on github.

Link to the latest commit on github at the bottom of the page on the express server. #148
Solutions for the exercises

Solutions for the exercises.
Delete404 fix

fix #63

Set `Content-Length=0` for delete
Add some submodule instructions

Add submodule instructions for people who aren't familiar with the concept
Fix html semantic

replace `<div>` by `<ul>` to fix html semantic
Enables web inspector if debug option passed as a parameter to packager.
Fixed issue with default Contact controller

Fixed issue with default Contact controller
DriverStationLCD message added
Fixes for attachment search

I made some fixes for the seach for attachments. Needed some small tweeks to get it working.

see https://github.com/sproutventure/search-everything-wordpress-plugin/issues/15
Document predefined Ddoc macro SRCFILENAME.

**This should not be merged until after [dmd #786](https://github.com/D-Programming-Language/dmd/pull/786 has been merged).**

And even then, it probably shouldn't be uploaded until after dmd 2.059 is released since it wouldn't work until then.  I really feel this repository needs to start using a [development branch](http://nvie.com/posts/a-successful-git-branching-model/).  Several people have been confused by the website having stuff that isn't true yet (especially with the changelog).
Fixed link to http://mindsaur.com/
Added ticket at http://trac.elgg.org/ticket/4325 and will remove the casting to null.
@swaroopch don't apologize, we need more nitpicking like this on the docs. You are encouraged to double-check us.
An entity is not defined parametrically.

"Return the set of free symbols in the Entity."
Ah, guess I didn't see that you were using javascript's type coercion to return 0 or 1. I'll pop that back in, but as the following:

```javascript
return (( n * p ) === k ? 1 : 0 ) ;
```
@tekknolagi I'm not sure how that would look to people who haven't followed everything @homakov has said and done these last couple days, like we since all this happened. He broke into the Rails repository and was not even suspended?!
@KenanY that had me laughing like a fool out loud. Nice
Right about `+=`, not the best option. Plain `<<` wouldn't work because `block` there (bad naming) is a set, so it would add it as an element. I'll change it for `merge`, which just adds the new elements to the original object.
Hmm, that might be something we need to add. Something in the context that indicates redefinable functions. Or maybe when we get a bit better error system that becomes a non-critical warning unless you say otherwise.
Should :someattr not be :attr (or the inverse) ?
When you have a nested resource you will generally want to load the parent resource first and do all your operations off of that object. I will usually create a `before_filter` to load the parent object for me.

``` ruby
# before filter
def load_open_mic
  @open_mic = OpenMic.find(params[:open_mic_id])
end
```

Within this method we can then just do:

``` ruby
@post = @open_mic.posts.new(params[:post])
if @post.save
  flash[:success] = "Posted!"
end

redirect_to @open_mic
```

I also left out the `deny_access_for_non_hosts` because this could just be a `before_filter` as well. Just have it after the before filter call for `load_open_mic` and `@open_mic` will be available if it exists.
I bet we can _ some of these instead of loc, just don't see how they'll change much.
`if 'i386' in Utils.cmd_output(['file', nodebin]):`? It doesn't look like you need a regular expression here.
There shouldn't be any models in the system without a created_at
field, if there are, that's a bug. Also, we're restricting to only
using these models for rankings:

["PredictionGuess", "Question",  "Idea", "Forum", "Event",
"PredictionGroup", "IdeaBoard", "Gallery", "Resource", "Classified",
"PredictionQuestion", "Card", "Content", "ResourceSection", "Article",
"Topic", "Answer"]

And I just double checked, all of those have a created_at field.

I just grabbed a copy of your rails3-master-merge branch today and I'm
diving into it. Let me see if I run into the same issue.


On Mon, Jan 16, 2012 at 11:49 AM, Joe Van Dyk
<reply@reply.github.com>
wrote:
> Do all 'item' objects that can be scored *always* have a created_at attribute?  The tests are failing here, because we are calling item#created_at on objects that don't have a created_at method.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/newscloud/n2/commit/524eb0dc0db4e8c2c67d54c957de78994730a37b#commitcomment-871860
GitHub's feed is one example -- this function allows `]]` to load more items from the feed.
the new configure / autogen.sh wasnt working for me: could not execute /bin/sh ./../config.sub
restored the old configure script, so new users can either use that one or run ./autogen.sh
okay. new syntax for plugins:

```coffeescript
module.exports = class UglifyMinifier
  brunchPlugin: yes
  type: 'javascript'
```
It also work for me on nvcc 4.0. What device cause problem? I tried GTX285,460,470,
same as above - keep BC
Shouldn't it be
"Gets whether to use vanilla (false) or exact behaviour (true)."
?
(first line)
Don't need to do the change, but for your knowledge this line do the same:

Py_XDECREF(%(Z)s);
you're right - switching to gemv_no_inplace
This patch is now applied.
This line causes an audit warning; perhaps we should special-case audit handling of `--devel` in my reworked options branch.
this is introducing a bug. `cursor` is the wrong variable name. `exec_cursor` is the correct one. this is the danger with trying to rewrite an author's intentional code because a tool doesn't like the style of the code.
| Or to put it another way: if the smoothmesh drawing code (/airmesh) happened to use triangles instead of quads, would you have kept the old interpolation?
I would have thought about the change a bit longer, but the used interpolation function is still less smooth, less common & very likely slower than the bilinear one. Neither is (/airmesh)  default enabled.
Also if I would rewrite Ground.cpp I would just use the bilinear interpolatiom, but I don't rewrite it ;-)
In the case here I didn't bothered with it before either, but now when it was bugged a change was quite obvious.
move myself* ;)
Have you run this / do you know that it works? I ask because it looks like it wouldn't have run before the change to tensor.concatenate.

It looks basically OK. I think it's ready to pull if you add some basic unit tests. Obviously the stochastic nature of the criterion makes it hard to test but you can at least make sure that the class can be imported and allocated successfully. That will protect it from getting broken by simple code refactoring in the future.

Lastly, once you get this working on an example problem, could you commit a yaml file so that people know approximately what hyperparameters make it work well?

Thanks for writing this.
aaaaah... what about this.state = state? (+if you use enum, you don't have to check whether it's in {0,1,2})
Heh, sorry to keep adding corrections.  After looking into it a bit more, those changes I mentioned are probably unnecessary.  Since you're already flipping through them, it's trivial to dump that static list and use the following loop instead:

```c++
for (int i = 0; i < plLocalization::GetNumLocales(); i++)
{
    SendMessage(GetDlgItem(hwndDlg, IDC_LANGUAGE), CB_ADDSTRING, 0, (LPARAM)plLocalization::GetLanguageName((plLocalization::Language)i));
}
```

I always end up trying to make things too complicated.  This solution looks much better than going through pfLocalizationDataMgr.
I knew I forgot to test a case…

That works for me. I'll add a unique_together field to ensure this.
Yes, good poitn
indeed, não funcionou anyway :p, o basecamp me retorna unprocessable entity, acho que tem algo errado com o json que eu envio pra eles :(

tenho que refactorar isso ai, deixar mais coffeish :P
its been a while since i've looked, let me check
The error handling cleanup seems good (I think), but can you put that in a separate commit,
since it is not related to the issue addressed in this commit? That would make it easier to
understand what you are trying to accomplish.
i added generic .* matching one commit later, since any file starting with a dot should be considered nogo.. these are purposely hidden system-specific files and should never be exposed to the user. Imho .* matching plus a short list of specific files (_readme.txt,...) seems a sane approach..
gui for now
Does ErrorNoHalt do anything else than notice? I've removed it only because it spams about errors to every player on server. It isn't good idea to send it to everyone.
The commit summary/title looks stupid now that it's public :)
can get rid of this and forward if you do the above
joar...aber eig könnte uns das egal sein^^
[DoctrineFixtureBundle's composer.json](https://github.com/doctrine/DoctrineFixturesBundle/blob/master/composer.json) depends on version `"*"`, but I know we probably don't want to do that. The data-fixtures package has two tags at the moment, `v1.0.0-ALPHA1` and `v1.0.0-ALPHA2`. Will composer correctly resolve `1.0.*` to the most recent tag even if the package's own composer.json file does not specify a version?
a bit lower is code:
http://img6.imageshack.us/img6/766/53705028.jpg

mby it can be moved up here, like?
http://img706.imageshack.us/img706/1844/43373227.jpg
Брр, что-то запутался. У нас уже есть координаты назначения m_targets.m_destX, m_targets.m_destY, m_targets.m_destZ - т.е. конкретно те, куда направлен спелл. Откуда взять Z еще правильнее?..
It was for:
https://github.com/mangosR2/mangos/commit/44cb4a9caeb9dcd840b7dee20c7926ebea6f0083

need to check if those spells still working properly
this crash is not on clean core, is it?
should probably use the `media` helper [like so](https://github.com/mozilla/zamboni/blob/master/apps/devhub/templates/devhub/builder.html#L21)
This was the only part where i wasn't 100% sure about whether I was doing the right thing
Isn't this supposed to be `split[0] = "/" + split[0];` ?
This messes up other plugins by transforming a command like `/yes` to `yes/`
Isn't TSS_TPM_TXBLOB_SIZE meant to be the size of data sent to the TPM?  But here we're using it in a test on the packet size between TSPI and TCS?
I think you might have fixed up these merge conflicts incorrectly. It might be better if you just do:

git remote add rapcad git@github.com:GilesBathgate/RapCAD.git
git reset --hard  76cbb6db4103a06713d2c0247092977f6a6e54da
git pull rapcad
git push --force origin

Or something like that
yep, that's sorted in a different branch, that's not ready to join yet.
This looks good to me
So you're compiling for Windows on a Linux ? (That's my plan)
Interfaces should begin with I.

Though in this case you'd be better off having a custom extension node and registering the URL directly in the addin file, instead of requiring a class.
sorry, the issue number is mistaken in comment. Proper number is #19 !
This name is a little confusing to me because it doesn't actually refer to a menu. In this case, you can refer to the "h1" you're targetting by using ".content h1" as the selector.
Copy paste error. Fixed in a later commit.
another likely conflict spot.
Taille des affaiblissements
Here too ret is not used.
I think this is the last place _print_spec was used with 2 arguments (i.e. with optional status argument). You can refactor it yourself or leave it to me after the merge, whichever you like.
This is a test only.
has been removed already
ideally just the sess one, we could move it out and hash the signed value, but then we have to do the same on the other end so we would have the signing overhead. The JSON cookie is used for cookieSession(), and in Express so you can do `res.cookie({ complex: 'obj' })`. It could be removed but then we would need another middleware just for that. Sometimes implementation elegance < usability IMO
Yep :P same commit for the same thing xD

Okay for the new permission registering system everything should be OK and
compatible with the other addon (well it's compatible since I tried with
WarpSign).

Oups ... I maybe have something to change ...

2012/1/30 Lathanael <
reply@reply.github.com
>

> hihi, we both noticed that :D
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/Belphemur/AdminCmd/commit/640cbf64dcdd42f68a17e3c3ef74f6cb90a62a0b#commitcomment-915020
>
`size ||= digits.length` is more compact
Humm, I'm not sure...

See:
My site are in http://localhost/site
The CI are in http://localhost/site and the "application" folder are in /site/application

It match with you affirmation? If yes, why not warned in the user_guide?
The answer is easier. If unspecified the browser follows the encoding the page is in. Unless you change that this will always be UTF-8 in Werkzeug since this is out default outgoing encoding.
It will break all the tests since `which pygmentize` returns empty string in Mac OS X.
I suggest a patch:

```diff
diff --git lib/rocco.rb lib/rocco.rb
index 510d7c2..ee79d11 100644
--- lib/rocco.rb
+++ lib/rocco.rb
@@ -51,7 +51,8 @@ require 'net/http'
 
 # Code is run through [Pygments](http://pygments.org/) for syntax
 # highlighting. If it's not installed, locally, use a webservice.
-if `which pygmentize`.include? "not found"
+pygmentize = `which pygmentize`
+if pygmentize.include? "not found" || pygmentize.empty?
   warn "WARNING: Pygments not found. Using webservice."
 end
 
@@ -161,7 +162,8 @@ class Rocco
 
   # Returns `true` if `pygmentize` is available locally, `false` otherwise.
   def pygmentize?
-    @_pygmentize ||= `which pygmentize`
+    pygmentize = `which pygmentize`
+    @_pygmentize ||= !(pygmentize.include?("not found") || pygmentize.empty?)
   end
 
   # If `pygmentize` is available, we can use it to autodetect a file's
```
Couple of small nits, if you can fix them then LGTM.
Also for the htmlspecialchars(xl()) function, note this can be reduce to just using the xla() function (optional since they both do the same thing).
In that case I'd pull the ECANCELED covers out of all the *_impl functions and just put them in the top level sections. Unless I'm missing something?
Will put it back when a user interface is done so it is going to be usefull
Just looking at it again, it looks like the problem in the first place was that someone accidentally used += instead of .=

What it was changed to looks cleaner anyway, though.
or can u come in trinity channel, is easier for talk
empty line? just wasting precious disk space!
otherwise looks fine, assuming it runs.  we should add an admin_socket teutholkogy task for it.. see ceph.git/src/test/admin_socket/*.  the existing suite job already generates an osd workload to check the objecter request state... we can check the osd requests at the same time.
yes i contact with you i need devs :P
I'll need to ask someone about this, I've been told there are mature Bundler-like dependency management tools in the Perl world. If this is the case, we need to detect and use them, just like we do for Erlang, Java and Scala (not yet released).
yes it's not as nice.

The problem with generating a hint (I thought about using <declared hint> + "-" + provide class name), the problem is that you cannot look it up easily anymore since you don't know the hint to use...
Странное что-то.
Nose si lo que has quitado no dará problemas en ICC.
I think for time-windows we can leave it to accept only lists, since otherwise the time-windows wouldn't be meaningful?
If you do `git show d46a1f5`, you will see that your file has Windows line endings. (They show up as `^M`, which is shorthand for `Ctrl+M`, which is another way of getting the carriage return (CR).)

Please get rid of them and make sure your editor is set to UNIX line endings.
see #117
wazzap! I see you are trying to fix some things in core, do you have issues like these:
https://github.com/ProjectSkyfire/SkyFireEMU/issues/88
https://github.com/ProjectSkyfire/SkyFireEMU/issues/84
https://github.com/ProjectSkyfire/SkyFireEMU/issues/112
https://github.com/ProjectSkyfire/SkyFireEMU/issues/117

I can't decide - use skyfireemu or arkcore, because if these issues related to skyfire == not funny.
I specifically did not make a synthetic example, and took left part content of one real file.

In ActionScript 3, the "include" directive (http://help.adobe.com/en_US/FlashPlatform/reference/actionscript/3/statements.html#include) works with files on disk (this is not the "import" directive). It is better to move it inside the class.

I can make an example for all options, ok?
Maybe it was just something wrong with my 1.4 upgrade - when I checked the
variable it was returning an array with 1 empty index - array ( 0 => ).
Maybe it was an error with my tax rates option. Regardless it was giving a
whole heap of notices about indexes not existing because it passed the if
($tax_rates) check.

On 15 February 2012 10:12, Mike Jolley <
reply@reply.github.com
> wrote:

> Hmm are you sure? I'm pretty sure an empty array will be false if placed
> in an if statement? Will check tomorrow.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/lukecarbis/woocommerce/commit/43036965a3db02616008471fe88f5ac5ce397e7f#commitcomment-967240
>
yes, but lose the comment.
yes... i'm sorry... but this bullshit.. any stable commit for 1 month) I should be happy?
:D thanks for remembering this ;]
Старайся писать аккуратный код. Это значит без лишних пробелов, пустых строк (конкретно здесь пустая строчка явно лишняя), соблюдай отступы (было выше в валидации).
для сведения: помоему если разрешить действие create то действие new станет разрешенным автоматически, так как нельзя делать create без new. Аналогично с edit/update.
Issue has been fixed in commit f5475ee5b5

( https://github.com/riftworks/riftworks-cms/commit/b952d40f061527d0a9c37d98669af0e10ed19af5#L0R29 )

Thank you for reporting. :)
Because Other projects steal our work then try and pass them off as if they did all the work.
I would move this error logging to the implementation of the supplier, and/or throw a typed exception in the supplier with the corresponding message here.
De vraag aan het eind van de zin werkt in het Engels, maar in het Nederlands is het geen standaard stijlvorm.
Looks good. Can you rename _DEBUG to plain DEBUG? It's the usual macro used for such cases.
Good catch! Thanks. I guess I was staring at it for too long :P

I'll change it to just be two assertions in a row after the setup.
¿No deberíamos droppear también el table services?
I think this will work fine as opposed to my other solution.  Will test it out this evening.
Why do we need this if here? Shouldn't we always notify when we are changing the status of the message to APPROVED (no matter what's the status before?)
nice. I wonder when I'll need a double octave rather than the octave tho
insn't it the default value for the type ?
Cool. I'll give it a try tomorrow on my desktop.
Glad you found someone who knows PowerShell better than me
Will do later today.
We usually only wrap the `foreach`-loop because it is the only thing affected by the transaction.
As long as I get some cool points and/or xp, by all means, fix any mistakes and improve where possible.  My two objectives for this patch were 1) getting row level th's to get their propers, and 2) try to make that happen with as minimum code as possible by using clever and/or standard CSS selectors.

As for the td+th, I rarely use the + selector, so when it worked in my one use case of the patch, it reinforced my misconception that the + means "and" not "and next sibling".  My bad.

As for the td:not, that was just sloppy copy/pasting. Glad you caught it.
Hmm, I'm not sure if that file is inherited after or before, I just took patches directly from cm/bravo couldn't cherry pick because files were a bit different.
why all the repos? place them in /parent if needed
May be it would be better to us os.path.splitext? =))
@chillu I spoke with @felipesilverstripe about how to implement the sync button. He suggested adding it to the "Actions" drop down menu in this design: https://github.com/silverstripe/silverstripe-design/raw/master/Design/ss3-ui_files-manager-list-view.jpg

That saves valuable screen space and puts the sync button in a logical place. 

I tried pulling in the LeftAndMain->BatchActionsForm(), but the structure is too different from how AssetAdmin->getEditForm() works. I eventually gave up and just implemented the sync button as a regular button. Could you please refactor it to be part of the "Actions" menu, once you have that working.
This leads to forced re-creation of the view on every screen timeout! So when you enter your PIN and the screen goes blank the view receives a reset and is recreated. So it never gets informed about the PIN-supply success! Then the device stays locked until SIM state changes and then displays pattern lockscreen...

behavior is described here: http://forum.xda-developers.com/showpost.php?p=22054917&postcount=434
Have follow the last install doc, and it isn't in it. (https://raw.github.com/gboudreau/Greyhole/master/USAGE)
Seems to work perfectly without it. We can have it or not I think...
I hope database fields will still defined in DataObject classes and not in a yml file ...
col_content = col_content.force_encoding(ActiveRecord::Base.configurations[Rails.env]['encoding'] || __ENCODING__.to_s )
Moved the lines from below the variable declarations to ensure the pickWinHeight calculation is correct.
Fixed in pull #79.
Required datatype missing (`string|bool` or `string|boolean`), along with an optional description saying under which conditions the string or boolean returns.
I have made fittedParameters output as a column vector as a matter of personal preference: this can be changed back if desired.
Can we have this file please :) It breaks compile
@ohadlevy - In other places it is almost always new && save, so that would seem more correct, I concur.
OK, then is this feature still needed? I also don't think it is a good manner. I think we can tell user the used times of resources, and it depends on he/she to delete them or not.
Please see line 114 - d = this.designRoot.
`return false;` ?
I see now. We can get the context easily enough for the current request, but how do we match that to the menu item. Sorry wasn't awake yet when I read the comment/code the first time.

Elgg normalizes all URLs so you don't have to worry about relative URLs. PHP has a function for breaking a url into its parts. You could grab the portion you need, use the str_replace method, and then explode and pop. That's a little cleaner than the for loop over the special characters.
Totally in line with you, but I will do that in a future pull request as Id' like to frame my work in 4 pull requests just to simplifiy tchats and feedbacks.

I sent an email proposing that : https://lists.wisc.edu/read/archive?id=18683349
I am not able to find where in code this collection is used. Could you clarify please? 

To restate my comment above, I believe this class could potentially benefit from a collection of "metadata/attributes/props" that might be gathered  and passed on by the actual handler. In case of ldap, this would be for instance attributes that are set to be retrieved, which could be used by the subclasses to further examine the account state.
I think this was a bad merge, I don't remember removing it.  I'll add it back
Yes, I used the assumption that it is called on startup. Not nice, will  
refactor.
Same here as with the Java example
You might want to put these messages inside the AMQP object, and instead of having the getInstance method, offer a java method:

    object AMQP ... {
        case object Start ...
        def start(): Start.type = Start
    }

which in Java allows for:

import static akka.amqp.AMQP.*;

if(msg == start()) ...
Future.sequence(activations.toList) map activationsPromise.success
better include full config path in the error message
You can use named parameters instead:

    def connect(from: Address, to: Address) = …

    connect(from = ..., to = ...)
"defn" is an fn composed with def; def binds something to a (namespace local) var.  Idiomatic would probably be to make your list of functions use "fn" for each instance.  "fn" is basically lambda, but you can also provide a name, i.e.
(fn a [x] (+ x 3)) 

So,
(def transmformations-log
 [(fn timestamp [m] ...)
  (fn date [m] ...)])
would be a more idiomatic approach (though I'm still not sure what you're doing here - need to read it more carefully or ask you on Sunday)
I think this one is wrong, «a pat on the back» makes more sense, also there is «turorial» to fix on this line, but all the rest is correct i think.
The indentation should be 4 spaces.
This seems like a blocker to me - not quite sure what the issue is though - I think we need to review the whole add dataset stuff at least for the new 3 part stuff
well this covers most auth checks, and did not feel it was harmful to add more.
For my OCD, can you do "Operating Systems"? OS's looks odd to me.
not used anywhere normal?
NEED A BETTER NAME, EH? What shall that name be? Lets not leave this in the final code.
The downside of using `Hash#fetch()` is that if someone passes in `:root_lock_node => nil`, it will cause `@root_lock_node` to be `nil`, which isn't what we want.

I would prefer if this was:
``` ruby
      @root_lock_node = options[:root_lock_node] || Locker.default_root_lock_node
```
should we be able to new TaskAcceptor? If not we, could make it a trait.
Please add a use statement for the class. And the typehint should be the LoaderInterface IMO, not a specific implementation
You should explicitly pass canary=canary here. Relying on arg order and names is a pain later on.
And once selected, print the according file. The order could be also implemented as a setup in a submenu (in addition to progressbar) to custimize the appearance
Good suggestion. That'd be faster, too. Will make this change and re-submit the pull request. 

Sent from my iPhone

On Jun 30, 2012, at 6:15 AM, "Ben Combee" <reply@reply.github.com> wrote:

>> @@ -255,7 +255,12 @@ enyo.kind({
>>        }
>>        // waterfall to my children
>>        for (var i=0, cs=this.children, c; c=cs[i]; i++) {
>> -            c.waterfall(inMessage, inPayload, inSender);
>> +            // Do not send resize events to hidden controls. This saves a *lot* of unnecessary layout
> 
> Rather than checking the message name, could we do this by checking a property on the event, something like "showingOnly"?  Then modify the source of the onresize/onpostresize to include this in the event structure.  This would make it easier to add more events like this in the future and also make this mechanism available to devs.
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/enyojs/enyo/pull/110/files#r1080588
@Dentxinho This was fixed already in c503287.
This should be `string.UTF8String`.
Asserts added!
I mistakenly thought the original line was casting another variable (not the argument). I agree with @jwilling and @galaxas0 that the argument should just be typed correctly and the temporary omitted.
Rename this variable to something more informative than `t`
Xcode did align them by colons...
`exec()` is async, so `try/catch` isn't gonna work here
Let's use paths relative to the package, e.g., "native/" + osName + "/" + osArch + "/" + library
Rather weird Twirl formatting here, though I can sort of understand what it does. I'd prefer something like

    @{gcs.map(gc => "%s: %s collections, %s ms".format(whatever))}

That is, keep it as a Scala expression rather than this mix of text and Scala which I guess is okay with Twirl.
I appreciate the idea, but the presented code would unfortunately not work. Such solution would be quite complicated considering prefix and suffix are deprecated. Let's leave this for the future.
You are right. Sorry to bother you. The patch is good to go. Thank you so
much for the implementation. I'll merge when get chance.
On Dec 2, 2012 6:58 PM, "Josh Susser" <notifications@github.com> wrote:

> In activerecord/lib/active_record/migration.rb:
>
> > @@ -554,6 +555,10 @@ def basename
> >
> >      delegate :migrate, :announce, :write, :to => :migration
> >
> > +    def fingerprint
>
> No other methods in MigrationProxy are documented. I'd suggest making the
> class nodoc, as it's an internal implementation detail. Or, since the HTML
> docs include the source of the implementation, the 1-line implementation is
> pretty much self-documenting.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rails/rails/pull/8399/files#r2288049>.
>
>
We usually recommend code snippets to be formatted in the guides and api docs. That's the reason I asked for the move. But I'm not necessarily bothered by this being here.
> to change the style after they merge it,

Naw, style changes are always asked for before merging, not after. We actually don't do style-only commits.
Sorry, :s misplacement. I have fixed it now.
Yes, but do you want one element of `collection` to affect how the next elements are rendered? Maybe just pass `:second` to the line below directly, without mutating `value_method`.
Q: Are we sure that underlying is not a JListWrapper?
So you're smuggling a MethodType past the typechecker disguised as a function value, then putting back the type? It seems likely to break if anything else touches the tree or acts on its type between point A and point B. Do we have some way of knowing that's not going to happen?

I was going to say eta-expansion is not reversible (e.g. f(xs: Int*) and f(xs: Seq[Int]) have the same expansion) but if you're carrying the original MethodType then it is - assuming it's still where you left it and in approximately the same condition.
What is the reason for adding this line? Is it to guard against the `iterator` being accidentally inherited from an incorrect supertrait?

Also, we should have the return types explicit in the public API methods.
Better to remove a few examples I think, this is quite long. How about keeping one example per point you want to make? So keep ``rvs1, rvs2, rvs3, rvs5``.
Also I like to indent to match the Google style for function calls, but it can lead to some awkward formatting such as:

```
      uint32_t newFirstPosition = QueueFile_wrapPosition(qf,
                                                         qf->first->position +
                                                         Element_HEADER_LENGTH +
                                                         qf->first->length);
```

In which case I just reverted to a simple 4 space indent.  it is inconsistent, but makes for more compact code.

```
      uint32_t newFirstPosition = QueueFile_wrapPosition(qf,
          qf->first->position + Element_HEADER_LENGTH + qf->first->length);

```

Do you think we should doggedly stick to the Google format in cases like this?
yep, you are right
This is a problem.

```
>>> E.n(1)
3.
>>> E.n(2)
2.7
```

nmag is suppose to be an integer and evaluating to 2 places will return an unrounded value. So perhaps this needs to have .round() appended.
That's not quite what I meant.  I was thinking more

```
  ,
 |
 {
 |
  '
```

Really, though, none of them look very good.  But we really should figure out what looks the best, and change the Piecewise ASCII pretty printer to use it.
This is wrong. Just knowing a real number is raised to an imaginary number, you can't say anything about the result. That said, your code does not check for this case.
So use the string.
valueOf() ?

```
+date;
```

is less quirky. But the proper thing is ```date.getTime();```
May be its good idea to change this to array_map() and avoid use $names array.
Yeah, that won't work. You'll let the build run in background and then redirect the output to `/dev/null` this way.
You're right, it's the standard to have twice indent for methods and ctor line breaks
On 5/1/2012 2:07 PM, Jeff Balogh wrote:
>> @@ -90,7 +90,7 @@ def message_validator(request):
>>           return request.errors.add('body', 'body', msg)
>>   
>>       # These are the keys we'll accept for messages.
>> -    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId'
>> +    VALID_KEYS = 'title', 'body', 'actionUrl', 'replaceId', 'cryptoBlock', 'site'
>> Granted, as far as I'm concerned, "site" is just an index value. I'm using the hostname of the remote service because I figured it'd be easier for folks to work with, but it could be any string value.
> If this is an index value for your use, why bother developers with it? You can use the push URL as an index and keep the protocol simpler.
>
I was able to drop "site" as a storage requirement.
Is it possible to make this a bourne (not bourne again) shell script? On e.g. FreeBSD, bash is not installed by default.
Superfluous switch.
This is redundent check since it happens automatically when you try to define new param in the next line
How is calling ```$object->changes();``` and then cycling over all the default values different from just doing ```$data = $object->toArray()```? 
In what case they would produce a different result?
what's the benefit of setting this inside the if instead of at declaration?  I think it makes the flow harder to follow.
This should be in core, not reference.
cross reference these to the man pages.
make the above change throughout.
This issue becomes nonexistent if I merge in develop because the copyright symbol was replaced, iirc, with (c) for consistency across develop/ files.
So you can restore the modules with the files. All information that might be different in the modules (like auth, language and their name), is in the info files. just "module enabled"  and module_display arent?
Processors don't exist anymore, they should be removed from the `services.yml` file.
You might want to put those three lines into method too. `$query_id = $this->store_new_rowset($rowset);`
Yea, I don't like it either. I noticed it just now when trying to get this PR taken care of. I mean it could offer some flexibility but I can't think of a use case for it being async.
Aici trebuie folosit castul normal, clasic.

Ceva gen: `player_dest.magic.cast_spell(spell, self, due)`
Please never use `using namespace' statements, it leads to a potential namespace collision. Use std::smth instead.
Hm, I think this requirement is not mandatory. We should stick to CMake 2.8+ as in CMakeLists.txt
It seems that `resolve()` is the cleaner API call.  Alas, I couldn't figure out how to change the `loader.calculate({required:required})` into a `loader.resolve()` call.
In this paragraph you reference `log` object, which is not defined, instead it shoudl be `logLevel` property or attribute. Remove the reference to time stamp.
Do not overload return, replace it with:

```
if (callback) {
     callback(y);
}
return y;
```
Nitpicking, but shouldn't it be "Most likely the Model, or a property on the Model, is null" ? :)
create ticket or remove
Don't change this line (keep pharmacy and outpatient here)
this should be return Util.getPatientIdentifier()
@esc24 
>If I understand correctly, the real problem is that cube.data is modified by the call to iris.analysis.interpolate.linear

The problem was an *Exception*, triggered by the final line of the test:
```
Traceback (most recent call last):
  File "foo", line 26, in <module>
    s = str(cube.data)
  File ".../python2.7/site-packages/numpy/ma/core.py", line 3531, in __str__
    res[m] = f
IndexError: index (19) out of range (0<=index<18) in dimension 0
```
This failure is the result of a shape mismatch so I don't *think* explicitly testing shapes will add anything.
You don't really use these initial values, so don't require them. Just have a no-argument constructor.
Doesn't look like a reasonable conversion in the JobRepository class.
which methods? If you're thinking of separate(), separate does NOT do what we want, thus the bug related to the need for this change.
Are those backslashes really needed?
I'd suggest:

> Although Django supports Python 3.2 and 3.3, we don't yet suggest that you use Django and Python 3 in production. Python 3 support should be considered a "preview": it's offered to help you start porting your apps for future Python 3 compatibility, but we're not yet confidant enough to promise stability in production.
This might end up being a hot path, maybe we should iterate through `arguments` instead (as per [this blog post](http://blog.izs.me/post/7746314700/benchmark-array-ification-of-arguments))?
I'd say it's more correct to print to stderrRawFile, since the message shouldn't be buffered.
Nice use of options to protect from custom converters.
I'm just really trying to help make this PR as small as possible to increase it's chances of being merged for 1.8, where we're really trying to focus on size. Sorry for the continual nudging and questioning each line.

Are all of these css properties needed? I don't think `padding`, or `border` affect the top and the left values here, do they? I mean, getBoundingRectClient() measures border-box size, meaning only margin effects top and left: http://www.w3.org/TR/cssom-view/#the-getclientrects-and-getboundingclientrect-methods
I just add some comments in the code about the standard output.
This file also should not be included.
"on the Tencent QQ site"
(include the word "the", and mention "QQ" again)
This line does not compile in Mono for Android.:

> Error	4	The 'this' object cannot be used before all of its fields are assigned to	C:\dev\GnomicDev\MonoGame\MonoGame.Framework\Input\Touch\TouchLocation.cs	145	35	MonoGame.Framework.Android

Changing to assign directly to `_timestamp` for testing.
Ignore what I said here--I'm wrogn.
Rather than weirdly break these if modifiers, just use a regular if..end block
Aye, ok. Apply same comment there :)
This is done in 772e4a1.
@rmocanu add newline at the and of the file
Are these styles needed? Shouldn't it match (thus, use) Webdix's CSS instead?
This line should become a `@deprecated` Javadoc tag
Likewise, this is 100, right?
Rename to `usesPassedInClassObject()` or something simular?
numberOfUse
VoxelBorder has nothing to do with that, infact that was written Wayyyyyy before VB. THAT is THERE (doesn't now...) because some long time ago the PlayerMoveEvent was executed by the Bukkit guys in the wrong place at the wrong time and if you MOVED the player from inside the Event's scope and they moved further than ~10 block distance it would 'kick them for "hacking" '

That, is the reason why the Teleportation was delayed by the scheduler
Are we normalizing this field to be all lowercase on the server? Because I think it actually comes in as 'application/x-mpegURL'.
I just saw the note in the description about supporting inline editors. It's still worth putting a comment here in the code as a reminder.
I changed _enabledAgents to _enabledAgentNames and added a comment to make this more clear.
I think you can call editor.document.getLine(pos.line) here so you don't have to touch the private code mirror var
The defaults object can use ordinary property names instead of a quoted string.
Updated to: _loadProject(files[0]).pipe(result.resolve, result.reject)
Using `key` like that seems dangerous but JSLint complains about creating functions inside loops otherwise. Is there a better way to do it?
This function should probably be named "appReady" to be consistent with the constant.
scrollDelta should be named something like scrollDeltaY or verticalScrollDelta
keep - erhalten oder beibehalten? - erhalten entspricht mehr "bekommen". Die sinngemäße Übersetzung im Kontext mag auch "aufrechterhalten" sein.
Commentaire de ligne
Can you do `5.seconds` here without it breaking shit?
This can simply be `!isset($config['output-html']) || $config['output-html']` - in any case you shouldn't check against `'false'` since the config is json it will handle `false` just fine.
it's an array. changed.
I can't tell for sure, but it looks like this is a server issue, not a client issue, so it should be a 500.
Can you just add a comment here about why you're including this?
I saw you put quotes around a urls above (https://github.com/mozilla/butter/pull/895/files#L2R118). We should probably keep things consistent.
This is intentionally different. If droppable is not enabled, the text should only say no image. If and only if droppable works, which means jQuery is available and EditorHelper is working, the innerHTML is changed to "Drop something here".
Yeah, that'll need to be added back.
keep line 23 intact, and remove the else branch
Do you mean for the path to be an atom? Or ETOOMUCHRUBY? :)
Ok, but I still don't really see why we're clicking the link.  A 'then' step should be just verifying something in the current page - if it was something like 'then the link to the database should take me to the database page' that might make some sense... but even then it ought to be something like 'when I click the link to the database / then I should see the database page'.  

If you just want to verify the link is there, use have_link.
@UtahDave any reason why we're not using `.format()` here?

```python
name32bit = '{0}:i386'.format(name)
```
Move out the logging to before the match?
U prohlášení jako "je zákazené", "povinné je" apod. by bylo dobré zmínit příslušný dokument -- HIG GNOME, Windows, OS X?

"Probíhá automatický překlad..." je bez diskuze lepší, opraveno.
I added assertions for ensuring that the view falls back to showing the speakers with the lowest possible character. I added a speaker starting from 'A' and tested that the view contains the name of that speaker. 

The only exception is when I check for a correct  `character` argument, in which I use `r` as an argument and assert that the view contains 'Random Speaker'.
What is this change for? Can it be part of a separate pull request?
remove Acme\Demo bundle
beshe ke ...
So it must be scaled with CSS as ```width``` and ```height``` are not valid attributes for ```<input>```.
Hate to be nit-picky but why are these still "inside" the content_search.scss or at least defined up above like the other colours?
This particular icon was when you hovered vs. when you didn't hover.  The slight 1px difference caused a shifting of the position of the icon on hover.  This was not the same as the bug that addressed alignment issues but was an issue I noticed when fixing the bug related to the missing details icon.
I can see it now there, I have missed it, yeah. Well, it's the puppet class that expands it, you could "require" this "generate-passphrase" exec class in both expansions if you'd put it in a common section, but this works too (with a warning message "Passphrase file was already generated, you can only generate once" in Puppet logs).

Works for me.
I'll update this to be more Ruby centric.  I tend to default back to more straight-forward programming without realizing it :)
It was there when I started. I agree though, we can safely drop 1.9.1.
This comment doesn't really match anymore.
Not sure what happened there...
Note that the NSAssert comes after this.  This code runs in addObject: which invokes setObjectID:forAttributeValue: at the end.
Can just use `np.asarray` here for this whole `if/else` clause -- that will support other list-like things.
Maybe these two should be configuration items, also? That way if I'm in the US I can easily change the primary to CfA and the mirror to CDS to speed things up...
I think if we follow the approach I described above of having base (abstract) classes, it should be in this PR because it'll be a fundamental part of the implementation.  Just have the base class have:

```
def  validate_shape(self, shape):
    pass
```

And just be sure to call it when the object is created. Then the subclasses implement as-needed.
Sure, that's a good idea -- will add to next commit.
We can, but in terms of readability I really like to see to which namespace a function or variable belongs to. But I guess you just want me "force" in a friedly manner to revert that change ^^?
I'm rather sure you can remove the above include, as win32_api.hpp get's included below message_queue.hpp.

interprocess/ipc/message_queue.hpp
includes: interprocess/detail/managed_open_or_create_impl.hpp
includes: interprocess/detail/os_thread_functions.hpp
contains:
<pre>
#if (defined BOOST_INTERPROCESS_WINDOWS)
#  include <boost/interprocess/detail/win32_api.hpp>
</pre>
There is an extra double-quote here; wonder why that didn't break my gitian build...
handleURLReply is only called with the request is "finished()" so the bytes available here is the total that is available as the connection at this point is finished.
Yes, I'll include such a comment later today, so we can get this in.
This should not happen, the is update should be propagated in the validation set
what do you mean? I just used the exact same string as truncate()
This should be wrapped in an `#ifdef` so that win users never even see the option and linux users don't have to be told about windows.
please remove ``@static`` for all methods
Doctrine also uses space after use.
Is this OK now?
with uppercase C
MIssing spaces around `!`.
anyway we can fix this later
Well, we'd have to look at the other places where this is used, but type seems a lot more logical at least looking at this code right here.
Are you sure you need to do all this path munging?  Couldn't you just use `normalizePath`?
@benatkin This should now be `[http://localhost:8080](http://localhost:8080)`
my bad! I'll fix it now!
seems better to correct CreateVolumeResponseHandler to not break on the tagSet element, than maintaining a sublass
With the side-band-64k, the server may send a line that's up to 65520 bytes. See the [documentation](https://github.com/git/git/blob/master/Documentation/technical/pack-protocol.txt#L407-410). We won't reach that amount all that often, but currently we require the whole line (data packet) to be in memory in order to parse it.
But I am afraid this will work only for N < 9 ;-)
Use $elem.data('poster') here.