labofsahil/github-event-dataset-2012 huggingface dataset:
perhaps rename it to munin-basic or something to avoid a clash?
2 lines in one?
Could you split them please, and use spaces around "="
> regarding contention: a forums poster reported 60% of his application time spent on this lock.

wau
This is too specific to this case to be called "isOrdered" and it seems (by the comment) that it won't be anything other than boolean, so I think we're safe in doing that.
I wonder if it would have been better to emulate other file systems and to have translated "esp/files" and "esp/files/" both as requests to iterate over the directory entries?
Doesn't the `browser_signature` also need a `|urlencode` thing?
Same, use `$.each`
Toby,
We do set it to the new value when configuration change.
There are only 2 scenarios that lazy is gonna be used:
1 - on first invocation.  I could be eager and add that to the constructor.  But then we have the second case
2 - after clearCache().  I could fetch this eager when clearCache() is invoke, but it felt better be a bit lazy.

Now, when you create/delete/update any role/priv/user it is eager, so before the operation lock is released I replace the collection with the newly available data, on that scala like approach we talked about.
Ok. This isn't PHP, and isn't consistent with existing JS coding conventions or our other usage in the node files. Shouldn't do it here.
A quoi sert cela?
Yep that's right. I'll change it in all the loaders.
This test (and the other 2) depend on a connection to ldap.berkeley.edu. You don't want the test to fail just because that machine's unreachable. So you should wrap the ldapService call in a try/catch and catch IOException. On IOException, just log the exception as an error. See CanvasProxyTest.getCourses() for an example.
The "completed" property is left over from working with the dummy data. It turns out the Canvas API data doesn't return the completion status for assignments.  I am modifying some of the data as it's processed for demo purposes - e.g. ensuring that we always have at least one assignment marked overdue, and at least one marked completed. Post demo, we'll need to work out how to proceed. I have feature requests for the upcoming API already logged with Canvas. 

However, the glitch above is an actual mistake since this is not the "faking properties" part of the code :)
duplicated code. This should definitely not exist.
```data/wordlists/sap_default.txt``` doesn't exist and isn't part of this pull.  Did you mean to include it?
Yes, you are right. Never realized that, to be honest. You should have told me that before... ;-) I have used that style for all contributions so far.
Remove one whitespace after full stop
Why not make the check as strict as it can be: [-1, 1] ?  Otherwise, it is very hard to justify this: "we check that the floating point numbers are in approximately the correct range, but you can switch off that check or specify a more stringent test by changing frange."  If we use [-1, 1] we are guaranteed good data, and if it fails the user can choose to loosen requirements.
Why would this raise an exception? What use case is being tested here?
You may want to check if _table is not nil here
Okay, no problem.  I'll update all of the tests to try and be more descriptive.
@mramato That's a pretty rare use-case.  If someone is pre-baking a texture atlas, e.g. for UI elements or because they know all possible images up front, they pretty much always bake everything.  Perhaps a use-case could be that user wants to prebake the most common images and dynamically add the rest.

Maybe we can just change the design of this function and also add support for "sub-regions" (and probably rename them) to texture atlas construction.  Instead of taking an index, perhaps it can take an image.  That should be much less confusing - so the function really adds an image with sub-images.  That could even be an optionally argument to `addImage`.
I'm lovin' the unit tests!
Is possible to have ErrorRoute.DEFAULT instead of ErrorRoute.getExceptionAttrName()?
Technically, it's not a JSON object, but a standard object. The difference being that JSON doesn't allow for titanium proxies, while this should. EG, myTableView.applyProperties({backgroundColor:'red',sections:[section1,section2]}) should work, despite section1 and section2 having no JSON representation.
Aqui não acho que faça sentido existir essa classe. A única coisa que ela leva em comum para as subclasses é uma variável que contém o valor da área. Além disso, a variavel se chamar getArea é confuso; inicialmente, eu passei os olhos e achei que era um método!

Era melhor ter feito aqui como no exercicio de herança de vcs, em que esse comportamento esta isolado em uma interface. Apesar de existir uma composição, ela está meio "torta", com essa questão de salvar os valores nessa variavel herdada.
Obviously urllib2 cannot handle utf8 strings. Consider the following example (python 2.7):

```python
>>> wikilink = u'Höhenradweg Musterhausen'
>>> import urllib2
>>> urllib2.quote(wikilink)
/usr/lib/python2.7/urllib.py:1268: UnicodeWarning: Unicode equal comparison failed to convert both arguments to Unicode - interpreting them as being unequal
  return ''.join(map(quoter, s))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/urllib.py", line 1268, in quote
    return ''.join(map(quoter, s))
KeyError: u'\xf6'
>>> urllib2.quote(wikilink.encode('utf8'))
'H%C3%B6henradweg%20Musterhausen'
```

According to http://bugs.python.org/issue1712522 this is fixed in python3 but not backported to python2. Are you using python3 on the production machine?
With the proposed class design, this would become
    
    m_backend->addPackage( iter->name() );
Please use string interpolation here.
I don't know how good your minifier is but `0px` should just be `0`
You will probably want to define a name() method that returns "general" for this class
Impossible. It is used in `_initializeFrom` and `emplace` overload for a static array.
It should be "scheduled for deprecation" rather than "deprecated." Also, if we're just moving the module, it should be more like std.contracts was before we removed it - that is, it should publicly import std.crc32, and have its documentation (and the pragma) telling users that it has been scheduled for deprecation and that they should use std.crc32. There's no need to have any implementation in crc32 if none of the implementation is changing.
I guess it's in case somebody implements `int opEquals()` ? Seems legit to me.
Change the position of equal signs
Can we link issue numbers here too?
the placement of the comment shouldn't cause the ``else`` to be moved to a new line
I have 4,000 form entries. We fetch 5,000 history items at a shot. Let's bump this to something much larger — 2,000, say.
Use tabs as separators, not spaces.
This seems magical. What is the 10 for? Maybe we can make it a little clearer in the code.

-C
Looks like myhostname should be the fully qualified hostname (http://unix.stackexchange.com/questions/58243/which-name-should-be-used-for-myhostname-in-postfixs-main-cf), which is often not what is returned when you execute hostname.
Can you try and use the `--with-bam-libdir` option, without using this patch?

Just set `configopts = '--with-bam-libdir=${EBROOTBAM}lib' ` in your easyconifg, and remove the `patches` line.

Let us know if that works, and if not, how it fails.

I'd prefer avoiding a patch if it's not really required (since that's an extra dependency for the build to work, next to the easyblock and the easyconfig file).
<updated_at>
We can't be getting rid of a public function (even if it probably doesn't get used elsewhere) without a deprecation notice and an addendum in the api_changes.
something that i liked since i started using backbone is that it offers this.$ which is a shortcut for running jQuery in the context of the view you're dealing with.

Not so important in this case, but maybe it would be nice to let Khan have its own scoped jQuery that automatically either runs in the context of the `problem` element or in the context of the page chrome? i dunno, maybe that's superfluous, but since i assume that adjacent cards may be similar in domishness, it would prevent accidental jQuery (".sel") from maybe clobbering other exercises that are queued up? (if that's not how things are, feel free to ignore this)
`self.assertEqual(self.model.Suite.objects.count(), 0)`
Done!, we leave as property, I think is better.
I'm still curious as to whether initializing `attr` is actually necessary? It probably doesn't hurt either way, though.
Dedent what exactl? The entire case blocks?
This link doesn't actually work.  Did you load it up and test it?  The link has to be settings.htm.  (make the page name lowercase to fit in with everything else, and name it .htm to make sure it goes through spring)
Good evening!
I don't think SONAR is panacea. I'm more about consistency: throughout Spring JavaDocs have full Class names,
 if they are mentioned only in JavaDocs...
Probably should update the id to be ...-gateways.
OK, let it be.
I just think about DRY.
Does this take out a square chunk of board, or a chunk of board with a radius?
No need for quotes around ``type`` and ``button``.
Which is  will end up with `null` as return var, but i can explicitly return `null` to make it more clear.
It could also be a ternary one-liner, if one wanted to save on line numbers. :-)
What about using `asset-path("path_to_font")`? Your solution doesn't with s3
Could you drop these changes from this pull request as they are unrelated?
Ah ok, got it. This should be changed.
Most of the changes here is just switching to using on/off rather than bind/unbind.
This check definitely looks like something that deserves a clear name to indicate what's going on (and what is filtered out).
Not sure about `issubclass` tests either.
You can also iterate resource and amount together like this:
```py
for res, amount in production.get_consumed_resources().iteritems():
```
Ah, would you mind prefixing full-line comments with two semicolons? Emacs is much happier then :-) Also see http://mumble.net/~campbell/scheme/style.txt
Es la forma mas limpia de agregar las cosas indespensables.
El anfitrion y la sala tienen que pasar por las mismas validaciones que el resto de los requeriemientos. Ejemplo: si el anfitrion esta coupado la reunion no se hace, etc... De otra forma nunca hacemos estas validaciones!!!
Error should be the first parameter to match node conventions.
These beans are @Dependent scoped, which means if you passivate the bean which injects them, then you must also passivate the e.g. Validator instance. To ensure that this is possible, the e.g. Validator bean must be passivation capable. Of course it is, because behind the scenes you just (re)connect to the service
arf, this used to be more complicated.... thanks for spotting it :)
extra space in method sigs from where String suffix was removed
Hmm .. I'm not sure that this solution is safe for every amount/combination of tabs and/or whitespaces.

Why not just sort -r -g? I agree that -g is missing and that this leads to wrong results!
But -g solves the problem for me under both OSX and Ubuntu 11.10.

Thanks!
I've checked the log of this file and found only one change [`Port of Hibernate read-only entities feature, including tests (NH-908)`](http://nhibernate.svn.sourceforge.net/viewvc/nhibernate/branches/ReadOnlyEntities/nhibernate/src/NHibernate.Test/ReadOnly/TextHolder.hbm.xml?view=log) by @julian-maughan
return "%s %i" % (self.servicio.nombre, self.numero)
This call is incorrect.
Should be a readonly property.
The `<TiBlobTypeData>` does not appear to be displayed in the generated docs. Another bug?
example of hex format would be helpful
typo here, definE
I haven't deleted this code, but I think it's duplicated...
add an audio with the id audio_instructions_player_source  in tcpp tdfp and tdfm and have it auto play when the page is loaded
```html
<a href="<!-- IF sn_user_menu.URL neq '' -->{sn_user_menu.URL}<!-- ELSE -->javascript:void(0);<!-- ENDIF -->"<!-- IF sn_user_menu.EXTERNAL --> onclick="window.open(this.href,'_blank');return false;"<!-- ENDIF -->>
```
OR
```html
<a href="<!-- IF sn_user_menu.URL neq '' -->{sn_user_menu.URL}<!-- ELSE -->javascript:return false;<!-- ENDIF -->"<!-- IF sn_user_menu.EXTERNAL --> onclick="window.open(this.href,'_blank');return false;"<!-- ENDIF -->>
```
Senky solution is also good.
One of success or fail should be called here
How does this loop end?  It seems like there's no break statement to exit the for (;;) loop at line 149.
Did you make a change to fss-layout.css intentionally? It's generally not a good idea to fork code, as it will be a bigger issue when trying to upgrade in the future. It's unlikely that when we update this to a new version of infusion that we'll remember that this was changed. If you need to make this change you may want to file a Jira against fss-layout.css and override it in the studios own style sheets with a comment about why.
Maybe--I can go either way on this... it wasn't necessarily my idea to change the numbers in the first place... your call.