labofsahil/github-event-dataset-2012 huggingface dataset:
Unfortunately put(k,v) is final and so I cannot override it to apply the default semantics only in that case. To make the semantics clearer in the code we could probably use constants to  replace the 0s both in the constructor of a remote cache and in this code.
javadoc - might worth specifying that the first element in the returned list is the primary owner?
Don't worry about message compatibility right now, but there will be coming some tests that verify this, so might bug you about it ;)
you should add class constants for the strategies
Perhaps this could be a function in ```backend_bases```? ```pylab_setup``` pretty much does this for you. Something like:

```
import matplotlib.backends as mbackends

 # NOTE: requires a change from #1020
backend_mod, _, _, _ = mbackends.pylab_setup()


# NOTE: would require all backend modules to have 
# a FigureCanvas rather than say a FigureCanvasGTKAgg
backend_mod.FigureCanvas(self) 
                             
```
Tiny style point #3: it is preferable to use ```obj is None``` (PEP8: "Comparisons to singletons like None should always be done with is or is not, never the equality operators.").
I might be missing something, but this is what I was trying to avoid

    In [1]: opts = {'transform' : None}
    In [2]: if opts.get('transform') is None:
       ...:     print 'user set transform=None. Taking action'
       ...:     
    user set transform=None. Taking action

    In [3]: opts2 = {}

    In [4]: if opts2.get('transform') is None:
        print 'user set transform=None. Taking action'
       ...:     
    user set transform=None. Taking action

The desired behavior is

    In [5]: if opts2.get('transform', False) is None:
        print 'user set transform=None. Taking action'
       ...:     

IOW, `None` is the desired `True` condition of `get()`
thx stephend, I didn't understand the context of this line until you pointed me to https://github.com/mozilla/Socorro-Tests/pull/117#issuecomment-5054808
Decidi separar do `make all`, já que, muitas vezes há erros na compilação do documento e o diretório fica cheio de arquivos temporários.
to support fedora, we need when "rhel", "fedora" since its not in the rhel family
It's an old habit of mine from old days....will fix.
+1 on headers earlier. [Requests](https://github.com/rmccue/Requests) uses `$url, $headers, $data, $type, $options`, which is based on usage statistics. (It also means you can just do `Requests::request('http://example.com/')` and forget the rest, with GET implied.)
There are a couple of other ones in here who could benefit from it
Instead of searching the child elements can you add an id to the unordered list and select that?
when i saw this, i was assuming as well that this controls to what depth the objects in that property get serialized. the other behaviour of saying only serialize if not already that deep is rather tricky, as in the object graph the same object might be at different depths depending on where you start.

what if we would call this setting `OnlyAtMaxDepth` (or something nicer?) and keep `MaxDepth` for the other use case?
Wrong copyright probably
That note should come earlier, on first usage.
I don't see any need to extend DatatableSource, extending EntityRepository should be enough.
Well, except for the phpunit, which is obviously after phpBB in alphabetical order.
Well, it is documented in ```usage.rst``` ("Middlewares are fired just before the route callback, and even before the Application ``before`` filters").
My english is poor, so the wording may be enhanced, but it its documented :-)
I still need to think about a nano.
Yeah its pretty sweet to discuss code :D
When I tried this I got a fatal error saying that insert() requires two parameters.
Oh, that's a very good idea. Yes, let's do that!
If the module... ? Maybe "of the setting" ?

In my opinion, if "needs_trust" is False, we always trust user, so to the question "can creator be trusted", we should always say "yes" in this case
This is not used anymore in the app, I will remove that too.
Probably shouldn't capitalize the type as that's not done for any of the other types. Same for the line below.
Good call. I wasn't sure where it should live.
Может более организовано сделаем хранение настроек в zookeeper? Скажем в config/<class_name>/<variable>, например config/io_ring_sender_t/number_of_connections. И зачем префикс к именам добавлять? zookeeper api вроде само умееть относительно какого-то пути работать.

http://zookeeper.apache.org/doc/r3.3.6/zookeeperProgrammers.html
"Added in 3.2.0: An optional "chroot" suffix may also be appended to the connection string."
Yeah, this should be a separate request - it was late :)

I'll strip back the rest of this pullreq and rework it so you can merge these two - they're only here so we can store it somewhere anyway
I do something like:

```
    $.ajaxSetup({
        'error': function (jqXHR, textStatus, errorThrown) {
            window.console.error(jqXHR, this, textStatus, errorThrown);
        },
        'timeout': 5000,
        'traditional': true
    });
```
No, this doesn't help me because here we are using tastypie api.
This is my code and it doesn't work:
`{% url "api" resource_name="post" api_name="v1" %}`

I get ViewDoesNotExist at / error. So I guess I'm missing something. I found this here: https://github.com/toastdriven/django-tastypie/issues/409 but I don't really understand it.
You do know that in JavaScript you can define function inside of functions?
Yes, if you will be also setting read field. But of course, API should allow only setting read field. You might to configure other fields read-only.
No, this is not OK. You should match on the email in `user.browserid_profile_data`. It is not necessary that user has the same e-mail used for our portal than she is using for BrowserID. Check how other backends match.
It is OK, if we would see that it is not possible to do it otherwise. Not possible does not mean that there was a stupid mistake.

You can create another pull request with code which tries to reuse existing views. If we see that it is not possible to do it, we can get back to this pull request.
JavaScript escape!
This middleware is already there!
*settings* -> *account*?
I know that, thanks. I already had a branch for it (hotfix/dependency iirc).
Could we just iterate on rows with:
for name, hours in rows:

and eliminate the variable assignment (name = user) immediately following?
hi @jdeniau , this is only for the function, not the method.

the following `p` is for the method :)
Oh sorry, I see that it's moved above ...
Could you extract this round robin logic to a method and add a test for it?
I found /data/local/tmp will be removed after test case finished, therefore, I need to make sure /data/local/tmp is existed for the |echo| command in line 12.
Why $options = null here, when you don't have the = null anywhere else?
Yes. But i also think `call` makes more sense here.
Check for parent.
sure, I login to alot of el5 boxes with older zsh....made it return as they throw errors to do with add-zsh-hook....will look to see if i can get it working on el5s version of zsh
@sorin-ionescu :+1:
FYI, all of the datasets have this information

print sm.datasets.anes96.NOTES
This was a hack to work around the constant resolution in 1.9.1.
100 character wrap
And then you could just do `plyvars <- c(params$binaxis, "group")`
Make a sentence or something.
+1 on the SYMBOL_TO_STATUS_CODE stuff. I think I'm going to put this table into the Http gem ;)
That's what confused me, it seems to work fine so I thought it was deliberate!
Yes, very similar problem...
```
doxygen_tutorial browe$ more doxygen_readme.txt 
doxygen_readme.txt: No such file or directory
```
The devutils file was only ever a soft link to the docs one.
You're right.  That's a relic from how I was first trying to do this.  I thought I could have draw_stamp write directly to a sub-image, but there were issues with that, so I decided it was cleaner to just return the image created.  But you're right to notice that ix,iy,n are not used anymore.  I'll clean that up as you suggest.
Barney, do you want me to change that while I'm editing, or will you do that?
This isn't used anywhere it seems, and not convinced it's safe. It doesn't do type-coercion when the keys are strings.

We could enhance it to lookup the config key on the entity to check if a strongly typed key is already defined, so that we don't cause subsequent class-cast exceptions. Or we could just delete this, and add it only if it's needed.
i've no idea when TLS won't be available.  perhaps java mobile or other weird env... have never seen it happen.
i think best to warn only for now, if we encounter it then try to patch.
I tried that, and it didn't seem to work. Probably because it's implementing the interface, not extending an abstract class. It seems like an issue with the way field annotations are detected.
Shouldn't this rather be a switch/case statement? And again, just assuming that the only error (3) is for slaveokay/read-pref is a bit weird. We should really use the error_message() way of things here if the result is "3".
And I forgot to translate.
cool. All good then.
It seems like this logic might be better placed at a higher level, not as a core piece.
True, but the `url` ajax option is the value of `resourceURL`. I can definitely add it back in though.

`var url = this.resourceURL();`
Better to keep your changes of dummy bluetooth. source can be moved to above conditions, and targer_arch has to be removed.
Guess this is no good, should use the binary technique?
``->fixXmlConfig('view_directory', 'view_directories')`` to have the proper singular
Seems a bit like overkill to me. No?

There's already a link to the documentation in this and in the DocBlock for the setOptions method.
Deleted zip file, it was only to help the newbies to download the Eclipse project files for each of the OpenJDK projects and get started by unzipping the file without having to go through many of the steps!

README.md changed.
Also (which is why it sucks this has to be redeclared here... :/ ) we're on [2.2.1](https://github.com/AllPlayers/USARugby-Stats/blob/master/app/include.php#L18) (and now that I look - it looks like I forgot to change it in [index.twig](https://github.com/AllPlayers/USARugby-Stats/blob/master/app/views/index.twig#L16) bahhh!!)
I thought an error would generate an exception, so the if statement shouldn't be required.
What's the difference between satchel_ore_boxdm and satchel_ore_box above?
Enum-Member PascalCase
Я сейчас подумал, и там функция несколько отличается - в XmlBinder для меня "примитивными" так же считаются Enum'ы и String'и, чего не скажешь о ситуации здесь. Так что не буду изгаляться и оставлю как есть.
Если это суть задания почему в нем ничего не указано про случаи с private полями и про  reflection?
> But 99?
> Isn't there something like Number.MAX_VALUE ?

Sure, but 1.7976931348623157e+308 would not work as an index. 99 is 
more than high enough in my opinion but we can make it higher if you 
prefer.
We should declare another locator instead of using it like this. We use this one twice in this file.
Yes please add a test for the int case. Thanks.
technically a hash link #foo could point to an element with name foo e.g.

	<a name="foo">link</a>
This line needs to go. It was likely added by PyCharm. You can change this in the code templates settings.
This would change the opacity of the background not the control. If you want to change the background opacity, you need change the alpha of backgroundColor property.
I haven't Titanium supported locales list, also not found what is pattern for. That's why I leave it
The command after:
command = [packager, '-package', package, 'bar-descriptor.xml', '-e', savePath, projectName, 'icon.png', 'assets']
has icon.png which not exist anymore.
If provided blackberry with android, android related stuff not generated.
The same

```"terms" : "<p>Haz lo que quieres</p>"```
for human beeings decimal degree is much more readable
Shouldn't verbose keep timings, too?
We really should have an appendBool() to save code like this.
Should this be managed by a local nestedContext object, so that it is correctly reset if there are exceptions.
Why not require the bindings to be Strings?

```ruby
def path_variables
  path_spec.select {|p| Symbol === p }
end

def path_spec_satisfied?(vars)
  path_variables.all? {|s| String === vars[s] && !vars[s].empty? }
end
```
Есть же форма в edit для юзеров такая же О__о
В таком случае было бы круто сделать форму подключаемой через ```render 'form'```, и использовать для ```edit``` и ```new```
I think we can get some inconsistency between now, start and latency here. It might be better to pass "now" in milliseconds as a parameter instead of latency and then create DateTime based on this value.
Hard-coding it here doesn't seem right. How about omitting it here and using default value in jetty-esauth.xml?
Can you please separate the instantiation from the append call? Also I would say lets put the instance as a member on the class. Could probably be helpful later. Can be `self.report`.
Also, I can't use PuppetSpec::Files.tmpdir here because the spec_helper will delete the temp directory after each individual test, and my intent is to use one temp dir for the full suite of tests.
注意 <para> 的位置，不要塞進空白字元
South does not call the method `save` from the model, so this need to be:

```
def forwards(self, orm):
    for track in orm.Track.objects.all():
        track.slug = slugify(track.name_en_us)
        track.save()
    for session in orm.Session.objects.all():
        session.slug = slugify(session.title)
        session.save()
```
For the `not available` message - this is code from the POC CalCentral.
When does Spring throw the NamingException? And can you catch that instead of RuntimeException? I'm leery of converting every RuntimeException into IOException -- that will make it really hard to debug.
Point taken. I will push those comments in the Ruby code.
Not sure what are you trying to do in this block, sorry, but if you're checking if the dir exists, you could use the "directory?" method from the Post::File mixin
Just 1?  What's the rush?
This is wrong. `(Boolean) '0' === false`.
You shouldn't need to set the error handling mode here, it will be set to return error codes by default, which will get caught by python and raise an exception. If the exception isn't caught within the example then the error message wil be output and the example with exit with a non-zero status.
Only [`certifi`](http://pypi.python.org/pypi/certifi/0.0.8) which it [specifies](https://github.com/kennethreitz/requests/blob/develop/setup.py#L24), so pip will install it.
Renamed it to `surfaceHeight` since we already have a `height` parameter.
Core/Rectangle is x/y/width/height.
Facet culling is false by default, so this can be just `imageryLayer._rsColor = context.createRenderState();`.
Do you mean sort each command in the bin by shader, front-to-back, etc.?  If so, that is already a roadmap item so I would remove this comment, or if you want to keep the spot in the code marked, change it to `PERFORMANCE_IDEA`.
High-level `from` functions like this and `fromExtent2D` are OK for now I suppose but I'm not sure that they belong this deep in the Cesium stack.  In good design, a class is cohesion and all of its functions are at a similar level of abstraction.  This function knows about Columbus view, which is really a scene-level abstraction, not core.  I'm open to ideas for cleaning this up, but not necessarily right now.
Also, this function needs tests.
It's not actually a minimum version right now - we only work w/ Python 2.7 (Django
doesn't yet run on Python 3.x in a stable release, and we're not supporting older
versions of Python right now).

I could do PythonNotInstalled, MissingPythonPreReq, or something, but I'm not
sure min version quite captures the intent correctly.

> -----Original Message-----
> From: André Rodrigues [mailto:reply+p-1128552-
> b3cc6d540f20d658ea588ecf93d70a3c1820803c-320265@reply.github.com]
> Sent: Monday, July 09, 2012 9:47 PM
> To: Dino Viehland
> Subject: Re: [azure-sdk-tools] Dev (#283)
> 
> > @@ -1143,6 +1152,24 @@ internal class Resources {
> >          }
> >
> >          /// <summary>
> > +        ///   Looks up a localized string similar to Python 2.7 is not installed.
> Please install it as well as Django 1.4..
> > +        /// </summary>
> > +        internal static string Python27Missing {
> 
> Why not call it min python version or something like that instead ?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/WindowsAzure/azure-sdk-tools/pull/283/files#r1128552
> 
>
Do you think validating that the used variable is defined is sufficient?  We will fail in the ps script if this setting is invalid, which would result in the role failing with a pretty good error message.  The defined check makes sense, though.
s/convienience/convenience
We might want to refrain from commentary in documentation. Instead:

> The coverflow view is a container showing animated three-dimensional images in a style identical to the coverflow presentation used for iPod, iTunes, and file browsing.
For iOS, this is not only composite layout properties, but ALL properties (as any may cause a relayout, such as label `text`). We may wish to be explicit here.
Yeah I should probably fix the tabs background color for the Tab Host implementation.
I don't believe we can support this for action bar tabs since most of their styling
is done in XML.
It's possible that open can fail, even though we check to make sure the file exists (think permission denied). Should probably wrap in a try catch or do the open call in a separate operation.
Defining these here breaks modularity...

I.e., to add an MPI library of a different type, you'd need to adjust this `__init__.py`.

It's imho better to define constant like is done for `MPI_FAMILY` in the MPI modules themselves. Do make sure they start with `TC_CONSTANT_`, there's some magic going on there:

```
TC_CONSTANT_MPI_TYPE_OPENMPI = "MPI_TYPE_OPENMPI"
TC_CONSTANT_MPI_TYPE_MPICH = "MPI_TYPE_MPICH"
```

It also better to not use the same definition for `TC_CONSTANT_OPENMPI` and `TC_CONSTANT_MPI_TYPE_OPENMPI`.

You'll also need to initialize `MPI_TYPE` in `easybuild/tools/toolchain/mpi.py`.

In an easyblock, you can then just use `toolchain.MPI_TYPE_MPICH` and `toolchain.MPI_TYPE_OPENMPI`..
Aren't allowed values 'present' or 'absent', then, and not 'enabled' and 'disabled'?
jquery классно все запутали :heart: 

ок, давай так
Bad trose, no static paths
See my notes on `popFrontExactly`'s ddoc comment, as they apply here as well. Sorry, that I'm not really posting comments in order within the file (hopefully that doesn't cause you too much trouble), but I've been seeing things in both directions when reading through it.
Should this be loci.common.DataTools?  Or are these delegators intended be in the old packages, but import from the new ones?
Dan, I misread a bit what you commented on. I thought you wanted an option to choose whether or not to have Launchd as the default service provider. Now I see that you commented about having launchd run in interval mode compared to having chef-client run in daemon mode.

If you have a patch against my branch, I'll integrate it. For me, the interval mode suffices.
space this in
I'm not sure about this. I don't like the 127.0.0.1 as a default - I think this might get over looked by too many people. Similarly, people currently using this module and upgrade to this version will have made horizon listen on localhost only - unless I'm missing something? If so, I apologize.

As well, in my deployment, I have all traffic redirected from port 80 to 443. In my ports.conf file, I have 2 Listen parameters for 443 - wouldn't those need changed?

I think these edits might be better done though the apache module.
Here [Debian -- Package Search Results -- libopenssl-ruby](http://packages.debian.org/search?suite=squeeze&section=all&arch=any&searchon=names&keywords=libopenssl-ruby) you can find that **libopenssl-ruby** in squeeze is also provided by **libruby**
yes it does, because it's yml file, i guess RestYamlCollectionLoader loads yml files.
I think that `$duplicateDetectionHistoryTimeWindow` should have extra 4 spaces indent
`.add()`
`import_suites`
Seems like I couldn't comment on the jQuery UI file `jquery-ui-1.8.16.custom.js` so I'll do it here.

1. What has changed in this file? Can we update the `jQuery UI Build README.txt` file in the same directory?
2. Can you add the requireJS wrapper? (described in the readme)
Should be using `.off()` here
And these lines should be replaced by `case_version = CaseVersion.objects.create(productversion=product_version, case=case, name=new_case["name"], description=new_case["description"])`
Yes. Usually we just leave out the braces and indent the code within by one level.
Fixed to reflect where the code was relocated to. thief!
What I am saying is that I would not expect a method named "determine" to have logic in it that *creates* something. It's a minor issue since it's a private method, but I wanted to mention that it didn't feel right to me.
I like 'adapter' and we already have (with quick search) AmqpInboundChannelAdapter
we don't always want to use the ConversionService that is part of the BeanFactory. That is why we make a distinction with "integrationConversionService" as a bean name elsewhere. The main idea is that Converters used for configuring beans may not be the same ones you want to use for converting payloads etc. See IntegrationObjectSupport's getConversionService() method for a bit more context. It uses  IntegrationContextUtils.getConversionService().

One idea is that we might want to reuse  IntegrationContextUtils.getConversionService() here, and we may want to add an additional method that also takes a boolean indicating whether to fallback on the beanFactory.getConversionService() [side note: for 3.0 we might want to even consider that as the default in  IntegrationContextUtils.getConversionService(), but I cannot recall now why we thought that was not a good idea (to fallback) earlier.]
Which if any fields are required?
1. wrong functio name: `doRequest()`
2. are you sure, that the best way to validate data is execute the request?
(See comment below)
So for example here is left side.
NSNumber* showSplashScreen
there is no need to a temporary variable here
the ``action_label`` should be a translation parameter used  in the xliff file
I have some doubts on this line:
I don't think so because Mongoengine don't allow creation of references for non existing objects (eg. handling objects saves and then, referencing).
We should not affect this principle (cf. [mongoengine/fields.py#L789-793](https://github.com/MongoEngine/mongoengine/blob/master/mongoengine/fields.py#L789-793)).
Do not scroll if there is no next card
TraversableResolver
Missing `@list = ` here
Get rid of fraction part of seconds here.
If these commented out sections no longer make sense, delete them or annotate them as to why they have been commented out.
This file should be named `SimplePanZoom` for consistency with everything else.
Does this mean you're using a patched version of Leaflet? What's the schedule for a new upstream with the patches?
formatting is off for these method
Hopefully I'll be able to remove this code as I start improving the implementation of the persistence...
If you make NO_SOCKET_YET = 0 this could just be:

if sock:
   return sock, True
Change it to "reports_to"

Add a method to return the inverse relation (reports)
This is not description in fact :). It should look like this:

> Maps $.sn to $sn.
>
> Necessary until all files are rewritten
Are you sure it does not reproduce another problem? Why it was added here? What should it prevent from?
This should be a .hpp extension
needs a link
sit means it isn't being run
I'm able to render context menus above the invocation list right now since viewable invocation list evaluates to zIndex 20 on device.  Should this be 100?
Do we need this. I thought we were planning to get rid of requireWebview eventually too.
`None` feels better here to me.
Nitpick: `$(D null)`