labofsahil/github-event-dataset-2012 huggingface dataset:
If you still want to change this file you cold if the pep8 w602 error 
http://www.python.org/dev/peps/pep-0008/#programming-recommendations
`compatibility_type` is the kind of message raised during the compatibility bump. `traverser.err.*` is the kind of message that's raised all other times. The only kind of non-`compatibility_type` message that affects the bumps is `traverser.err.error`.

`traverser.err.notice` should be used for anything where the message is fairly harmless (e.g.: the return value of a function changed or there's some kind of new, perhaps unexpected behavior). `traverser.err.warning` should be used for anything that would cause breakage going forward. For instance, if functions are removed. In that case, you're guaranteed to end up with breakage because all add-ons that use those functions will throw errors.

All compatibility messages are still shown during standard validation, but during standard validation, their `compatibility_type` is not used.
Add a new line here for PEP8 compliance
just change `Method:` to `APIMethod:` ?
What's wrong? $a equals $a in this case, because of $do_it is false. This is only remake of the existed function compat, with similar test case.
This should probably be EMI-only.
Is this a hack? If so, please mark it as such.
If editors is an Array, are we better off doing `if (editors.length) {`?
In this case the openDialog variable for the initializeAddPeople function will be undefined, but that's not really an issue.
What is a editquestion Set URL?
It would be good to explain how you give access (through what interface) to these folders (i.e right click folder in explorer). Leaving things to the readers discretion always allows for miscommunication.
Are you sure you want to override the parameter ``$predicate``? This often makes code less readable. I also found this on lines 124, 126, and 129.
if delay is the only option, why is this an option hash?
The issue why I did this was the height and width = 'auto' properties. For BB now this is not supported and instead of setting w/h as 'auto' I tried to reset for BB precise values. 
But I modified the code in other way after your comment - more logical in my opinion. Example is below.
a.add(Ti.UI.createLabel({name:"label a",color:'white',text:'A',height:(!isBlackberry) ? 'auto' : 40 * scaleY,width:(!isBlackberry) ? 'auto' : 95 * scaleX,font:{fontSize:48,fontWeight:'bold',fontFamily:'Helvetica Neue'}}));

So vars aLabel, bLabel, cLabel, dLabel are not used any more.
when width/height auto will be supported even this changes should be reverted back. (added comment)
break up the line so it's not so long across :)
also, missing the <a href=''... part
Don't worry in that case
Can we solve this more elegantly with late static binding?
Por curiosidade, este valor mágico é o hash (MD5?) do quê? :P
Haha, oops I don't even know what I'm doing.

Should be

`node_type? out, :call, :attrasgn`
Write variable names out, fully.
No, is OK. But just then code bellow is wrong, `len(response['objects'])` is probably wrong as it is just one object in response?
What about: Repeat password ?
Google api also returns profile picture url among other things. Can i define it in user document?
Arguments in anonymous function.
Sadly, JavaScript does not allow comma at the end like Python does.
What is difference between your `user` and `request.user`? You already have user object in `request.user`. Why fetching it again?
That is not good, we need it like that to prevent exception:
"ValidationError, Exception Value: Errors encountered validating document: gender: Value must be one of ['male', 'female'] ("gender")"
Should we put `['*']` as a default audience? Maybe this is too permissive?
Why you no use one space only! maybe!
Yup, turns out just `#application_id` returns the default. Fixed.
http://developer.android.com/reference/android/os/PowerManager.WakeLock.html#setReferenceCounted(boolean)
yeah, this is what I was referencing when talking about the Twig testsuite.
```
change 'see pf::SNMP::H3C' to 'see L<pf::SNMP::H3C>'
```

with this you get a nice hyperlink that works fine on our published documentation on the web site
This should be the first method in the class, to indicate it is the main thing that users should call.
Not your code either, but we should have one var per line.
Not particularly familiar with ShutDown() code, but in ApacheRDF we do StopCacheActivity() and destruct the fetchers here as well.
Can we just completely remove this since the page doesn't exist anymore?
Combined the ifs.

Afaik `$PROJECT_HOME` defines the directory in which `mkproject` creates new projects (see [Project directory management](http://www.doughellmann.com/docs/virtualenvwrapper/command_ref.html#project-directory-management).  It's not related to the virtualenv management features like `workon` and `mkvirtualenv`, and not mandatory to use `virtualenvwrapper`.  It don't think that it makes sense to check that variable.  I'd guess that most `virtualenvwrapper` users don't have that variable set.
column_stack would require less thinking

(I didn't see that tydat is 2d even if univariate.)
no brackets around here. It's correct, but looks unusual
That part was accidental actually. git:// cloning was broken this morning, but https:// wasn't. Happy to change it back.
oops! sent a new PR to fix this
BALEETED!
You should really learn to use class-based views.
Format docstring correctly. Use reStructured markup (for links, for example).
Suggestion: this and the next 3 lines can be reduced to: [constructor: (@name = '', @roles=[], @collection=null) -> ] with nothing in the constructor body. It seems pretty common and shouldn't reduce readability.
Did you do this so that it's possible to do ```galsim.NFWHalo()``` instead of ```galsim.lensing.NFWHalo()```?  Or is there some other magic that is permitted by ```from lensing import *```?  (No objection, just wondering about the details.)
The reason it is useful to have fwhm for everything is precisely the use case shown in script 1 of MultiObjectDemo.py.  We make galaxies that are a specified resolution.  So the size is a specified multiple of the PSF size.  We could get away with specifying that in terms of the half-light-radius, but then we'd need to use half_light_radius for the PSF as well, rather than fwhm.  Or add a getHalfLightRadius() method to know what is is for the PSF after specifying the PSF in terms of fwhm.  (Probably useful to add that functionality anyway...) 

But mostly I thought it would not be hard to add, and since it is plausibly useful, we should do so.
Hold on, is that even possible within SBProfile right now?  I mean, instantiating an SBProfile and setting params later?  They all have setFlux() methods and I see that SBMoffat allows you to set FWHM or scale radius, but otherwise you'd have to write setSigma(), setHLR(), setFWHM() for the various other SBProfiles, yes?  (Not that that would be too hard, I guess.  But my point is more that what you're proposing doesn't just require that we don't throw an exception when no radius is given, but also that we change the guts of SBProfile to allow it to initialize to some default size and then get changed later.)
Suggest:
```
use_shear = galsim.Shear()
```
What's with this 'foo' thing?
Put back in the "JBoss web application server" javadoc comment
the 'newDefault' stuff has been moved to the caller code. They are worrying about where the properties come from and the CredentialsFromEnv is kept 'dumb'. It just gets properties.
Same issue as above. We need to understand this problem better. In this case the content is always from the database. So then it is either a database configuration problem, or when the content is stored it wasn't converted correctly.
ca fait un peu bizarre `gurus`, ca peut avoir une connotation tres negative je trouve (du genre secte,...)
I think the use of `memmove`, `malloc` and the `ptr` can be replaced with just taking the address of `point` and passing it to `dragImageForRowsWithIndexes`. The docs says that the last parameter is an in/out parameter: [NSTableView](https://developer.apple.com/library/mac/#documentation/Cocoa/Reference/ApplicationKit/Classes/NSTableView_Class/Reference/Reference.html). The original code is just a workaround for Java's lack of pointers. On the other hand I think the code is filled of cases like this. We should probably take a good look at all uses of these C functions handling memory. So this can perhaps wait.
haha, yes. Will change.
My proposal would be to add an options hash to init method, since most of the time, no_subdomains will be false.

Example:

  # new param for method definition
  def self.init(config, s3, encryptor, options={})
  end

  # normal code does not pass options
  S3Helper.init(....)

  # tests pass an extra option
  S3Helper.init(...., :no_subdomains => true)
Other places we have checked `RUBY_VERSION <= '1.8.6'` to make this code easier to find and delete ifever/whenever we drop support for 1.8.6. Thoughts?
can't you collapse this to

    elsif comma_position and comma_position > (value.length - 4)
      [...]
    elsif comma_position
i thought this will fail on windows?
I agree that it is a bit off topic. I should have put it into a separate pull request or commit. But it was deliberate. I was running the ``show_urls`` command provided in ``dango_extensions`` which calls ``__name__`` on views. This made me realise that having ``__name__`` as a method causes problems.
As you have already merged the changes in, would you like to get something like this in a separate pull request next time?
formatting is different
Save kittens! Align text!!!
That is pretty much what I do.
IMO I think the makefile should be at the top level directory... rather than hidden in the assets folder
And when it's double digits, it shows as 

    4:010:36
...because of the add_zero adding a zero at all times.
We could. 

I believe our thinking is that the StepRegistry is an internal component that the end-user does not care about. I am fine making it a first-class citizen if you believe that's the best way moving forward.
You can accomplish this by doing

```ruby
model_for(resource_type.to_sym).new(r.attributes, true)
```

I think.  That additional true tells it it is persisted.  This is probably better than hooking into the internals of the object.
Метод можно сократить до ```return s.length() > 0 && Character.isDigit(s.charAt(0));```
you should be able to use the assetLoader here and get the tree asset thats been preloaded.

```javascript
_tree = assetLoader.get("images/tree.png");
```

you'll have to require the assetLoader, though...
For my own edification, what exactly does this line do?
"This tutorial" is too generic for the PDF version.  Likewise for "The tutorials on this page", below.
Can you change this to require rhc/json and swap ActiveSupport::JSON.decode to RHC::Json.decode?
I think this is why you were having problems. Changing this to createCipher fixes it.
Check out http://api.drupal.org/api/drupal/includes!common.inc/function/drupal_goto/6 for the redirection.
>    On 09/23/2012 08:00 PM, Gael Varoquaux wrote: 'R' is a single-letter non
>    descriptive name. It would be much better to find a better name
>    This should go on twitter ;)

:).

Actually, it should be named 'residues', here :P
Space after the closing square bracket, please :)
I'm confused by this change, is that something you did?  or something that comes from changes done by appcelerator?
JP suggested we should leave it as is.
Ignore this comment, the tool has no way to preserve formatting on all cases.
Agreed, I'll change
I can't tell is this is equivalent, or better than the code you removed from the body of the function below... :(
REST based transactions are more lightweight.
indentation is slightly more than normal
OK. I was thinking that I may extend this function to check other input (such as cluster) in the future. I guess I may create a different function, if needed, for other input.
I'd prefer to have a single name for the bundle file, despite its location (master or slave), but I will leave the local cache one as "xfer.hg", which will imply updating the existing unbundle calls to use "xfer.hg" also.
Examining the code, I think this is because when sharing is enabled, rather than `hg clone -U -r nonexistent $remote $ws && hg -R $ws up -r nonexistent`, which would fail in the clone, we do `hg share -U $remote $ws && hg -R $ws up -r nonexistent`, which succeeds in the share but fails in the update.

The patch here is good because it checks the update, which could in principle fail even without sharing, just under weirder circumstances that might be hard to write a test for: e.g. the repo contains some files with paths forbidden by the local operating system. Keeping the test case in `MercurialSCMTest` will ensure that behavior stays good even if the `clone` method is rewritten for some reason.
I'd much rather have override default to False. No need to make people think about it in except in the very few cases where it actually matters.
I think this whole bit could be more concise and have less magic numbers.. I'd set 8 as a variable, and either use a loop or do something like:
```
total = 8
formatted[report['report_date']] = [[0 * (total+1)]]
```

And use the "total" variable instead of harcoding 8.. need to think about this more though, probably could abstract further.
please set default as `null`
It might be 'undefined'. The best way to check is just
  if(androidVersion) {
    ...
  }
I ended up using `sections` and removing `range` thanks to following suggestions :)
Please fix the comment, or at least note that it is wrong.  I do strongly prefer documentation, even historic documentation, as long as it is clear what it means and/or where it isn't true any longer.  Better nothing than surprisingly wrong though.
is this url option right? isnt it /people/:id/photos, or something like that?  

Honest question, I dont actually know myself /cc @danielgrippi
No Javaskriptiä kun on tullut viimeaikoina koodattua niin tulee melkeen automaattisesti.

Musta tuntuu että aika tiuhaa koodia on ollut näissä, itse yleensä kirjottelen aika harvaa. Koittaa jotenkin ryhmitellä rivinvaihdoilla eri toimintoja.

Mut melkosia ikuisuuskysymyksiä
In this case how do you overwrite, or correct a fumble-fingered mistake? Perhaps it should be azure site config set <key> <name> instead, which does an upsert (update if there, insert if not)?
Le module est déprécié, j'ai du utilisé http://compass-style.org/reference/compass/css3/images/ à la place.
for all these page verifications define a method in the page object that checks the data is there. You did the assert correctly here.
Seems like we duplicated this, I removed this instances and updated it below.
I do have the CSS linter enabled in Sublime, but it highlights so much of the stylesheet that I had the impression it wasn't being used by others. I'll start watching that more carefully.
Just curiosity, why use a unix TS instead of a date field value in the DB?
btw, Buzz had some weird issues a while ago because they were reusing the same curl resource for several request. I don't remember if it was general or only for some PHP setup. But Buzz switched to create a curl handler per request instead to fix it.
So reusing the curl handle should be done carefully.
typo

also, if not success

or

if !success
In here, you should take advantage of send_request_cgi().  Here's exactly how to do that:

	res = send_request_raw(
	{
		'uri'          => "/en-US/api/search/jobs/#{job_id}/result",
		'method'       => 'GET',
		'cookie'       => @auth_cookies,
		'encode_param' => 'false',
		'vars_get'     => {
			'isDownload'     =>'true',
			'timeFormat'     => '%FT%T.%Q%:z',
			'maxLines'       => '0',
			'count'          => '0',
			'filename'       => '',
			'outputMode'     => 'csv',
			'spl_ctrl-limit' => 'unlimited',
			'spl_ctrl-count' => '10000'
		}
	})

If you don't want the API to automatically escaping the parameter name & value, set encode_params to false.  When I tested this, I didn't have to do that though.
I think "Lotus Mail Encryption Server 2.1.0.1" is a better target name in that case. At the end, as far as I understand the module has been tested only again this version. And there isn't any version auto detection capability.
I did this myself so far. I would be totally fine with your dissertation as a reference although it is not exactly the same as the implementation we use... but that is OK I think?

Or do you want to include the equations directly in the doc string?
two spaces for indentation in html
Matt, I mostly agree with you.  But:
* This pull request is huge already.
* Pull requests need to be smaller and more incremental, not bigger.
* We need to resist the "well I'm in here anyway, I should do this this and this too" because that leads to bigger and more complicated pull requests that inevitably are reviewed less thoroughly as a result.
* This pull request does no harm as far as I can see.  Better is better even if it's not perfect.
* It is extremely painful that we do not yet have this branch in master.

That said, I agree that documentation should have been written during development of this.  In particular, adding new DOC_TBAs (like the one below) just seems wrong.  If Dan or someone can clean this up quickly, I'm all for that.  If it delays merging it even a few hours, I say we merge this now and improve the doc in a separate pull request.  I think that's just being pragmatic.
I'm surprised nobody commented yet that it should be spelled `TextureAtlasBuilder`.
You can get ride of the clone calls here and just pass white and black directly.  There's not needed now that it's a function.
There is a known breaking issue on iOS where %-sizes for `Ti.UI.ImageView` width/height refer to the percent of image size, *not* the percent of parent size.
Well, the next couple of lines of code need self.exts not to be None, so I have this check here now.
@scottgonzalez @jzaefferer Is it ok the above solution?

I need something to manipulate (encode and decode) the querystring and I found nothing, but encodeURIComponent and decodeURIComponent. I found no known lib to handle that in the client. So, I have ported the node's querystring into the above.
Why? The extra `=` is not needed here.
.user files shouldn't be in version control; they're specific to your Creator setup.
temp is a bad variable name. Rather name it line or something like that.
`bind` is not a great name for a javascript function.
Not a biggie but better put there str!= null && !str.isEmpty()
`S.stringof` should be outside the string literal.
Hmm. Yeah, it's already in a branch which is an optimization, so that makes sense. I should have thought of that. Will fix.
This is presumably configured by reopening the class in the applications themselves?
You missed one!
We really need a self-contained client library, don't we?
return something from this function as well as choose to use \PSU if you are going to be using that.
it should be:

critical "Unexpected HTTP response code:#{res.code}"
FYI, I wrote a module to wrap this up a bit more: https://github.com/smurthas/Request-JSONStream/blob/master/test.js#L30

No need to change now, but in the future.
Damn me and my spacebar-thumbs.  Good reformatting!  Did you use a tool to detect these or did it all by hand?
create it in the constructor and return the instance here
verify I read this correctly, this is for the warning on contract expiration, as opposed to cert expiration as in _get_entry_image from MySubscriptionsTab
When would this already exist?
upps of course ...
Есть впечатление, что ты натрахаешься со стратегиями владения. Оберни это в shared/weak_ptr.
rob: we can do this like the `track` command does it
Can we chain these
@bp323 sure, should be fixed in a follow up commit - I also had to change the place where we call `positionInserterBar` since it didn't always work in FF.

Would love to see this as a `postion:relative` though, probably in a future release.
Shouldn't happen, but check for null beanFactory?
I don't agree; they are __different entities__; the name of the header (content-type) and the name of the schema attribute (content-type). You are mixing concerns by using the runtime constant for the schema attribute, just because they happen to have the same value, doesn't make it right.
Need to document this change on the wiki (Migration Guide page).
Added this check
I would remove `== 'true'` because accepting all truthy values is ok IMO.
I think the correct method is getArrayCopy() if we want to dump a copy of the ArrayObject
опечатка, должно быть "являются"
This will work, please fix, and close this pull request.
I agree, let's not worry about the getErrorNum stuff in this pull.  As for catching, it's the responsibility of the developer to add the try-catch.  It's just something you need to learn to handle when working with an OOP framework.
time is a built-in (right?), put it in the group above with the other built-ins.
oh ok, well it would only be 0 if the array contains no data and an array shouldn't be empty in the collada file as far as i can tell. There is no src attribute or anything like that.

Just add in a check to see if array is zero? Would that work?

```actionscript
if (array.array.length > 0)
{
	if (array.array.length < (offset + (count*stride))) {
		if (array.array.length < (offset + (int (array.array.length / stride) * stride))) {
			document.logger.logNotEnoughDataError(accessor);
			return false;
		} else {
			count = actualcount;
		}
	}
} else {
	document.logger.logNotEnoughDataError(accessor);
	return false;
}
```
just assert for the instance['name'] equal to 'test_SERVER_ERROR'?
I don't think we need this file, people can just change the file from "autoload.php.dist" to "autoload.php" in the phpunit.xml file, no?
`blog_archive_dates` collects the dates for the archive. `ArchiveWidget` takes those dates as a parameter, groups them, formats them as links and then renders a partial.
Incorrect HTML here, missing a closing <div>
Might as well check number of elements in the stack entry here
remove type for now, add when needed only
If path is a filesystem path, please use FileSystem.Combine instead.
do we support "platforms: [iphone, ipad]" for parameters in yaml?
typo "you can you can"
I think pagesInDom above or locsInDom below make a good argument for dataDash operating on arrays of elements, it makes for a natural interface when most selectors return more then one item anyway.  It does have the odd problem of requiring you to get single items out of the array when you want just one, however you have the ability to do powerful things to ranges of selectors, and not worry what happens if there are different elements in the dom then you expected, you can just always get the first or last item, even if you get more then you expected you get a sane result.
And it was tried, but i think we need a voice chat so Gary and I can go over every solution we tried.
Sorry, never mind.  You are actually adding a test.war to the ear. (thought you mapped the one the ServletProtocol adds :)
Menkööt tämän kerran, kun on sentään `reduce`:a kutsuttu.
If the contains function is just used in unit tests, you should use the built-in matcher in Jasmine:

expect(x).toContain(y); passes if array or string x contains y
Oop.  My bad.

I think it'd read cleaner across two lines.

    request = self.create_request(...)
    self.request = request
This runs on the client so why do you need to require dialog, why not just do an alert?
Just curious?
What about always having file access turned on?

This was added for that reason I believe.
Settings is optional according to the API docs - https://bdsc.webapps.blackberry.com/html5/apis/blackberry.ui.dialog.html#.customAskAsync