labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, I was thinking of a case class. The advantage of an implicit over a val is that you can set it once in, say, a trait that you mix into every Job (which I suspect most organizations have), and then just use all of the DailySuffix, DailySuffixCsv, DailySuffixMostRecentCsv, etc etc, without having to have subclasses of each and every one of them.

You could also have a trait that you mixed into every instance of those you created, but again it's a little more awkward.
Document why you've set this constant to this value.
I'm quite sure it's part of the standard library. I found it here: http://docs.python.org/2/library/xml.etree.elementtree.html

Does that mean it's part of the standard library?
Should be a docstring
This should be private, since it never should be called outside this class. 

Python doesn't have real private functions, but instead uses name-mangling to simulate it. Class members prefixed with two underscores (__), have their name mangled so they can't be called outside the class without some extra effort. 

So, I'd rename this __replaceVar(self, string), and update the docstring to show it is private.
всегда для нового? а как ршибки валидации смогут показаться?
Did you mean "should be None"? If so, how would gain=None be treated in the code? Poisson noise from the object flux would be ignored?
You can use `textwrap.dedent` here to make this a bit more real:

<pre>
   svn_info_stdout = textwrap.dedent("""\
                        Path: .
                        URL: https://svn.example.com/svn/trunk
                        ...
</pre>
it is to prevent testhooks to work in a prod environment. I actually did not yet implement the check in testhooks.
This should probably be changed to test `.force` instead.
btw, you should sort the use statements here
Okay this was do to a mistake in JTS.transformTo2D( env, crs, lenient, nPoints ) - messed up the calculation of the right edge, resulting in the inconsistent images shown above. 

WIth this issue fixed the bounding boxes are not exactly the same (which is good) but are very very close (as expected). Visually I cannot tell the different between them - hopefully visual diff agrees.
It is not necessary anymore. I'll remove it.
It's actually better to run zeus outside of bundler. I quote from zeus README:
"Q: "I should put it in my Gemfile, right? A: You can, but running bundle exec zeus instead of zeus can add precious seconds to a command that otherwise would be quite a bit faster. Zeus was built to be run from outside of bundler."
In the future, could we get versioned releases of this instead of whatever this is fetching? Don't worry about this for now :)
> hum ok, so it is usefull. The name sounds really weird for me. The word lambda is about really different aspects.
> Did you had any alternatives in mind? freezeFunction, protect, immutableFunction, deepFreeze ...

In some functional languages lambda is used for pure anonymous functions. I was also considering `f` and `defn`. Maybe `prototypeless` or `frozen` or `iced` (implying that it's frozen even deeper :D).

> Last question: what is the point of freezing Function.prototype then?

It does not needs to be last. It's basically for the same thing:

```js
const { Loader } = require('@loader')
Object.getPrototypeOf(Loader).prototype.message = {...} // Equivalent of Function.prototype.message = ...
```

In fact I've realized that we should also freeze `Function` :)
`vtype` isn't used anymore.
Do we also want to call the count setter here?
Is this supposed to be hardcoded to your page? This file is appViewer.js
This, and the change below aren't really needed, since these bodies should never be seen by the parser. In fact, these should probably just be defined in `interceptor_spec.rb` now that I think about it.
Yak to shave later.  Not a blocker for this pull request, traced back to 121d2911
@vrthra did you mean to change this?
Why do you generate a second EditorView instance (the other one being instantiated in EditorController) here?
```py
buildout_cmd = ['./bin/buildout']
if options.config:
    buildout_cmd.extend(['-c', option.config])
subprocess.call(buildout_cmd)
```
that's what I thought as well
Not entirely sure what happened here; it stayed changed even if I removed line 210
Indeed, changed that now
Class name is incorrect
Just add a new commit, also you should use a custom branch rather than master branch.

http://blog.evan.pro/keeping-a-clean-github-fork-part-1 by courtesy of @EvanDotPro
Please check if you cannot reuse the setOptions function of the base class here.
we should check for ```is_null($name) || (is_string($name) && 0 === strlen($name))```; the idea is we don't want an empty value.
I think that the correct package name should be `zend-search-lucene`
The same observations about phpdoc syntax as above
That's true, and `new RegExp("[\\s\\S]{" + i + "}" + atKeyFramesRegExp.source).exec(token)` wouldn't exactly be ideal, either.

Of course we could also go all low-level and scan through the string one char at time.
This is the default intelliJ setting.
_created does not exist.

VeryVerySpecialPythonFileMod - Traceback (most recent call last):
  File ".\python\ErcanaCitySilo.py", line 217, in OnGameScoreMsg
    if not msg.getName() in self._created:
AttributeError: ErcanaCitySilo instance has no attribute '_created'
Maybe just $counter?
Hmm, good point. Let me play around a bit.
That's fine for now. Hopefully soon there will be a cleaner solution using Dart Pub.
Slight nitpicks: "a URL" instead of "URL" and "the colon" instead of "colon"

Rest looks good, thanks for opening this PR!
@harjis Käytetään Pupen olemassaolevaa funkkaria
you should use ``count()`` instead of ``sizeof`` which is only an alias
Doh! Should have thought of this. :+1:
(continued from above)
This is actually extraneous, express will know to send it as JSON.
Should be: `trackMediaElement( mediaElement )` (no space before first paren)
remove extra line, please
Should point to `"../lib/johnny-five.js"`

:)
This should be cut-and-pasted from the cases below it that do the withFallback, rather than from the case above it which is the special "first" case. However when adding the third copy of the code block it's probably time to make it a loop instead of copying the blocks.
Note: this part is removed, because the spec doesn't require this.
I know it's not your code, but I'm not exactly sure where ```downloadable:true``` came from here. We don't have any code that checks for it and I'm not sure what it's supposed to do.
This could just be my unfamiliarity with MVC, but wouldn't this code be better-placed as a method on User? Similarly for the SQL in the call to `find` below.
I'm curious.. why `vmouseup` not `vmousedown`?
Suggestion taken and I also added exception handling for the case when improper search parameters are specified.
Still need backticks around io here.
Hey, you don't have to copy all the documentation from ``bind``.  I think that's self explanatory.  I would just say 

```
unbinds an event to mousetrap

the unbinding just sets the callback function of that keycombo as an empty function
and deletes the corresponding key in the _direct_map dict
```
Don't change the size, the Dynaforms's title can be a long string
Oh, it's commented out... sry :)
Awesome, thanks for adding this. Do we need one for SHA512 too? Also, this looks like this is exactly the same code as it would be for SHA-1, can you just subclass that built-in hasher? Just wondering.
I know this works now, because we'll iterate over the keys, but it would be better to explicitly send the keys in case the API changes in the future.
I'm not sure what the Puppet Language "spec" says on the matter, but is it really the case that we can get both `=` and `==`?
We could add the protocol to the baseUrl?
We could make limit optional and just default to 10.
I'd add it now with the note that packages are slightly outdated. It's one of the two most popular options on Windows, and until recently it was the only one for people outside academia. If they don't update in say 6 months we could still remove it then.

I'm also missing Sage, or did you leave it out on purpose?
What is the default array `[]` used for? Does it make sense to call this method without any values to fill in?
should retain the spacing here.
We don't need to have this notice in bold.
3.0

Emailing from my iPhone 4S like a BOSS!

On 30 May 2012, at 16:35, madhazelnut<reply@reply.github.com> wrote:

>> @@ -120,11 +130,20 @@ function create_captcha($data = '', $img_path = '', $img_url = '', $font_path =
>>        // -----------------------------------
>>        //  Assign colors
>>        // -----------------------------------
>> -        $bg_color    = imagecolorallocate($im, 255, 255, 255);
>> -        $border_color    = imagecolorallocate($im, 153, 102, 102);
>> -        $text_color    = imagecolorallocate($im, 204, 153, 153);
>> -        $grid_color    = imagecolorallocate($im, 255, 182, 182);
>> -        $shadow_color    = imagecolorallocate($im, 255, 240, 240);
>> +            if(!is_array($colors)){
>> +                $colors = array(
> 
> I think I found it here: https://github.com/EllisLab/CodeIgniter/blob/develop/user_guide_src/source/changelog.rst
> 
> What version should I add my updates to? 2.1.1 ?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/EllisLab/CodeIgniter/pull/1405/files#r900862
Add a space here?
This section is redundant.
В метод mutate мы передаем значение типа integer (-2,147,483,648 – +2,147,483,647) Мы по условию задачи не можем передать дробное число. Соответственно нам нужно только проверить значение percent на соответствию диапазону от 0...100, что я и исправил в последнем комите.
Для сравнивания строк лучше использовать isEqualToString
Не вижу реализации этой функции
Не согласен потому что нет точного определения мутации, где "Х" в следствие мутации определенно должно стать "!Х". Хотя если подумать, скорее всего должно быть именно так, а не как я предположил.
Так у вас в данном случае округлять даже нечего! Просто берется х и все. Вам говорили именно про округление, судя по тому, что вы написали. Так что смело оставляйте изначальную запись, она пускай без округления, зато корректно обрабатывает процент. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
насколько корректно это сравнение? вы сравниваете два указателя.. меня это очень смущает. на мой взгляд разумнее воспользоваться методом
 - (BOOL)isEqualToString:(NSString *)aString
как вы считаете?
Такое выражение избыточно. Достаточно просто nucleo == [n nucleo]
магические числа...
thx, update it
Does this mean that the psuedo-google skin won't work anymore?
I would comment out or disable the account/verify_credentials.json call by default:
* This call is not necessary for login. Applications get Twitter User Ids in the previous step as part of the OAuth workflow. One HTTP Request less makes login really faster.
* OAuth calls are [not rated](https://dev.twitter.com/discussions/1196) and allows thousands logins. However the [account/verify_credentials.json v1.1](https://dev.twitter.com/docs/api/1.1/get/account/verify_credentials) service has a limit of 15 calls in 15 minutes. This can break gae-boilerplate twitter login unexpectedly.
* No need to maintain twitter API versioning.
This can use `-collect`, which has been merged into master.
Hmm, yes you are right.
Wrong description. 

It will actually allow scrolling when the events "swipeleft" and "swiperigtht" are unbound. As per mentioned issue.
>The library doesn't have tests that change the index, does it?

Nope. I've added one as part of the pull request :)
The `%` interpolation operator wants for some spaces too. As above, since there's only one thing interpolated, I would write it `'shakes/%s' % shake_id`.
use: `scipy.sparse.issparse` and convert is to csr if needed with the "tocsr" method
At some point we decided to get rid of the spaces, so I was doing this for consistency. If our docs aren't totally messed up it must be working...
Yes, I'm working with another branch now, forget this pull request please.
Maybe we should use just `send` directly? I don't even think we should use `respond_to?`, because if someone supplies an attribute that does not exist it must surely explode.
This looks like a no-op.
Do we still need popup.js?
损失透明度可以获得更高的图像质量。
Params is never defined here.
This pull request is sooo refused. Seriously, you renamed my variable names? And this?
That doesn't sound good
Yes, I fixed it (see below).
We'd like to retain Python 2.4 compatibility for now, so conditional expressions shouldn't be used.
how about now?

https://github.com/ssikka/nipype/blob/08001ab1b24e96184962d56d8ed94c1fb6fb060c/nipype/interfaces/afni/preprocess.py
On Fri, Mar 2, 2012 at 11:22 AM, Satrajit Ghosh <
reply@reply.github.com
> wrote:

> >
> > +
> > +class To3D(AFNICommand):
>
> note the blank lines and the use of `=` for underlining.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/nipy/nipype/pull/278/files#r512055
>
Хз. Мне проще сделать проверку, чем думать: может или нет
Well, its a problem now that all resources are registered on all nodes.  Its like every node has a duplicate, and since stonith doesn't check for duplicates except by name, operations sent to both Fencing and FenceingDup are not serialized on a node.  This was causing problems where on the async 'list' query we'd attempt to contact both devices at the same time on the same node.   Depending on the order, the second agent to try and connect would crash.  We'd try it again and everything worked fine, but having the duplicate isn't necessary anymore.
¿El DataType no nos mete validaciones en JavaScript, verdad? ¿Qué pasa si se ingresa algo incorrecto?
XD Gracias, me pasa por hacer las cosas apurado.
Why do you prefer new object assignment than in-place update?
Note that `optparse` was deprecated in Python 2.7, and is replaced by `argparse`. No need to change this, but good to keep in mind for future scripts.
Updated, logger is now required
This constants should probably be renamed as ``STATE_*``
probably better to use new `CLI`
i think we need to acquire self.cond here because we're accessing the shared migrating_instances data structure.
I think it should be automatic, I'll roll that in.
Yes, yes it should. This is concerning.
You're correct. I'm not sure why I always pull out the else ifs in these situations. I guess to me its clearer, but since the last part isn't in a giant "else:" block even that is debatable.
the only thing that's needed from the InboundInvocationHandler seems to be commandsFactory.initializeReplicableCommand(cmd, true) and you already have a reference to the commandsFactory. Or do you need the locking as well?
Why do this? Can't each node create the cache? What's the rationale behind having an RPC to create a cache cluster wide?

Again, see Hot Rod topology cache for such an example.
Explicit synthesis should be unnecessary.
I don't understand.  matplotlib.pyplot does a `from matplotlib import cm`, so plt.cm.rainbow should be ok?
It would be quite readable as two lines if the second line started with "if".
I have made an effort to browse the revision history. It seems that back in 2004 the `rc` function in `pyplot.py` (which was actually was located `matlab.py` then) did something more than simply call `matplotlib.rc` (actually it was calling `draw_if_interactive()`). However, over years this additional call was removed and now it seems perfectly safe and proper to make it just an alias to `matplotlib.rc`.
s/*self._header_loactor/*self._header_locator
Can you make this a python property, python named (so without `get_`) and make the name imply that it's returning enabled? That would follow existing style a bit more closely!
А вот этой функции честно говоря не знал и не нашел. Сделаю через неё.
@pupkinV Тебе не кажется что это приведет к неправильному использованию этого класса ?
Хотя я пока ни в том ни в этом не уверенн, можешь привести пару примеров где это нужно?
I think this would look better with each parameter on a new line...
much clearer set of options
Yes.  Definitely.
I assume this operation is idempotent? The repo conf generators get run on every boot.
yes do it this way
That's weird, my editor is showing soft tabs of 4 spaces for this file.
Why is the include required in the first place actually? Its just used once, right? In general, isolating reusable bits into includes is good, but it also makes templates harder to follow, so we shouldn't do it just because we can :)
Minor thing, add a space here right before the opening bracket, should look like:
$resourceTypes = array_map(function($row) {
Place "else" in this same line, always
Já myslím, že ne, od toho je tam ten ipl_t state, který si zapamatuje stav interruptů před voláním query_and_disable_interrupts () a k němu se to pak v tom conditionally_enable_interrupts (state) vrátí. Jestli myslíš tohle.
Jinak oprava dobrá, ale furt mi ten sleep nefunguje. Zase jen blokne aktuální vlákno, přeplánuje na další, který po chvíli zavolá yield a pak se to hryzne. Bohužel bez debuggeru se mi to ladí dost blbě, nechceš na to kouknout? Nevím ani, jestli je chyba někde v těch timerech nebo dělám něco špatně já.
I don't know Arabic, but this change causes the test cases failing. Can you check the test cases are correct or not? (see the mail you received from Travis CI)
It could use some parenthesis or maybe be split into multiple lines. When reading compound `unless` statements, just remember [De Morgan's law](http://en.wikipedia.org/wiki/De_Morgan%27s_laws).
got it, this LGTM then
Great catch! I will reissue request.
again, ``` -> `?
It looks like this method is not actually used in the real code, correct? If not not, I think it would be removed as `#keyword` just returns `nil` when it doesn't exist, and that `nil` is discarded by the `Array#compact` call before appending the message.
This is the name that is used in `require()`, right?
I don't know. I asked on javascript.ru to help me to "decode" mootools code into pure js , and professional js-coder helped me.
I also offered innerHTML = '', but he sayd "removeChild() would be better" ....
Mixing return types here, probably not the coolest thing to do. perhaps better to just return None.

Toyed with the idea of returning the path instead which could be stored along with the request dictionary and returned with each response and form a kind of look-up but this kinda complicates things a bit and not clear to users.
Are we just trying to make the tests less noisy here? If so let's send the log to a file during testing instead of introducing this sorta stuff into a test function.
since the method is not implemented this wont work, also the comment should be available in the javadoc..
This is probably giving you a raw-type generics warning.

Suggest replacing `Class` with `Class<? extends InputKind>`. This will create some ripple effects though that might be a little tricky...

Since this is SDK-internal for now, I'll leave it to you whether you annotate this type fully.
Is that `|| Array.new` necessary?
You can probably just leave to default `get_random_string` letters to chose from? Or you think we have to use `string.letters + string.digits`?
It would almost be if you would not change it to something wrong. ;-)
I have already done so (the second part). Check tasks.py. Good?
I will add panel class and leave the rest for UI designer.
I would again verify here that user is authenticated and that is staff.
Where do you set next? Why it would be configured? This is your job to set it.
You need to change "...nibawem" to "...niebawem"
No, I mean just kolmogorov.gif.  I thought it'd be able to be seen as doxygen and so we needed it in the tree, but this part is out of doxygen, right?
So are you saying that this change is necessary to make the GC work properly? Or is it just not needed? As I understood IE's garbage collector, it is a bad idea to leave an element like elem in a Javascript object because elem won't be seen as unreferenced.
Add new line add end of file -> pep8
Isn't it rather `unexpected`?
I cannot (and will not) accept this solution.
main_app should not be needed _in_ the main app, this is a helper for accessing the main app from _within_ the engine.  Did you hit some issue here that required it to be in the host app?
Perhaps it should return IEnumerable instead of IList to prevent users from calling add and think they have added an application.
Same goes for the other resources.
Please keep package __init__.py files empty; can this go into a separate module within the refreshbooks.transports package? (Non-empty package init leads to some unnecessarily-confusing behaviour around package and module initialization. It's harmless, but I don't need the baggage.)
I don't believe we can rely on `Array::some` in older IEs, which have to be able to run the compiler. Correct me if I'm wrong, and I'll gladly change it.
oh, I see, it throws type errors. This should work.

```
some = if Array::some
  [].some
else
  (fn) ->
    return true for e in this when fn e
    false

some.call [1, 2, 3], (a) -> a is 3
```
And should we add a note about the existing DOM elements in `allParagraphs` get updated live?
mostly won't work.
Cool. So this can be omitted
Receiving an error here:

  File "build/bdist.macosx-10.7-intel/egg/circus/plugins/resource_watcher.py", line 35, in look_after
    max_cpu = max(cpus)
ValueError: max() arg is an empty sequence

This seems to be a the case if the process was stopped on purpose by the user.  Do we have any way to figure out why a process/watcher is down btw? Anyway, I guess I'll add the "if cpu" toggle here as well.
I think I just left that as is, but fixing now.
Maybe you haven't gotten around doing it yet, but this should be "document_list".
Use `var` instead of `const`. BTW, all new files should be written in strict mode, so add "use strict".
> It may be not atomic in some OS[1].

That's only applicable to non-aligned or non-natural size stores. `lock_ = 0` is neither.
As I commented on that bug, I don't see why this is a problem. If the user thinks the cache uses too much memory, he can always change the value in the settings.
Apparently removing this means that you need to adjust currentmodule to point to statsmodels.robust.robust_linear_model or else RLM and RLMResults are not picked up.
correct .. however we then need to adjust the config

```
symfony_cmf_create:
    image:
        model_class: Symfony\Cmf\Bundle\CreateBundle\Document\Image
        controller_class: Symfony\Cmf\Bundle\CreateBundle\Controller\PHPCRImageController
```
The popup shouldn't close when okay is pressed in case there is an error
I think this is ok.  People probably won't call this more than once typically.  If we have some use case where we want this to be efficient for subsequent calls, we can initialize `_re=0.` in the constructor.  Then here we can write:
```
if (_re == 0.) {
    _re = ....
}
return _re;
```
> Just to be clear, I'm not going to do any editing today, so you won't clash with me.
> 

Okay, so tonight I'll fix the typos and will continue to post more substantive things here.
If the error occurred in the boilerplate demo, then I have an explanation: in the boilerplate demo, the global jQuery and $ variables are actually a fake jQuery that looks like this:
```javascript
$ = jQuery = {fn: {jquery: 'Fake jQuery to catch accesses to the global object'}};
```
So, there is now browser property here, which explains why in the error jQuery was defined, but jQuery.browser wasn't.

This would mean that we have here an accidental access of the global jQuery object, which should never happen. If that's the case, then somewhere the sanitize object is instantiated with the fake jQuery, which should be fixed there.
Do we really want to do that? I think that is surprising behaviour. I might want to have a value that is a collection, but it won't let me. Changing behaviour in the method based on instanceof is not what I'd expect, so I'm guessing not what lots of other developers would expect.

I'd have a separate method for applyValuesToMap if you really want this convenience.
let's hide this behaviour inside the base class by defining  an abstract method for command behaviour, a method that gets called from BrooklynCommand.call.
From the PT story I think this text is supposed to be: No Materials assigned to this class.
ouai mais c'est la que le bat blesse, c'est vraiment pas credible. Tu devrais rester du point de vue du consommateur et analyser en quoi le modele actuel est pourri et ne correspond pas aux attentes et aux habitudes des gens, mais de ka a recommander un modele c'est too much
Vire le _of compouters_
Working version:

```javascript
_.isFinite = function(obj) {
  var value = (0).valueOf.call(obj);
  return typeof value === 'number' && isFinite(value);
}
```
I added changes to them now.
.details-cell span {
    max-width: 465px;
}

465px is too much in this new layout, I suppose
I think this actually needs to be split into separate tests for different scenarios:

* Test where Git is found in %ProgramFiles%\Git\bin\git.exe
* Test where Git is found in %ProgramFiles(x86)%\Git\bin\git.exe
* Test where Git is not found in either location

As it is, this no longer verifies that Execute() fails if Git.exe isn't found.
This shouldn't be needed. Yum should resolve this dependency.
Seems sensible - do add the codes to php_mongo.h though, there is a whole list at the bottom. You might have to add a number to them all (like 20 or so) to not conflict.
troca o idAeronave  para aeronaveId
We need <code>self.click_options()</code> before this.
@mappings = {}

etc, with the next line
Smart. But is that you?
That's a good idea. I've opened a Jira under Python for it: DVPL-1069.
isOperator (and therefore operators) is used outside this method though (in appendOperator)
I think we're fine to use a simple `@todo.get('done')` here; the XSS vulnerability only matters if the user can enter the text and put something malicious in there. Since the `done` property is a boolean, we should be fine.
The coding standards are to use camel cased names for variables.
ident is differen than the rest for next few lines.
The original, before your comment was: if((ishuman(occupant)) ... ), which is correct syntax.
you should change it to ``<2.3-dev`` to allow people to use KnpMarkdownBundle with Symfony master
The mpu6050 driver doesnt have that feature. It should be updated to the current state of the mpu6000 so they match.
It coerces a class into a class which is an array of that type.  It is so the user doesn't have to know ahead of time if the a method will under the hood use a single instance or a collection of a type.  They just have to care about the generic type.
Those two should probably go into `javascript_heads` (and the jQuery above too, though not part of this PR). I'm not sure how much I like generating one big blob of js-related stuff in a helper as it removes it from the layout (you can't look at the layout and see there's jQuery and modernizr and whatnot but have to hunt down the `javascript_heads` method…)

@meineerde what do you think about `javascript_heads`?
What does the ```\&``` do here?
Зачем этот if?
Did you forget your return statement here?
This is another line items
change was done everywhere
I misunderstood what you meant. Since you're talking about extraction mechanisms, it may be clearer to say "in a single transformer".
how about `embedder`?
Shouldn't this copy be made outside of the for loop?
> with the "+" marker, I can see no difference. Does it have any effect?

You're probably right. Sorry for the noise
What's not pep8 here?
I think it would be good if we handled this in a consistent way across classifiers.
Usually I rooted for ``self.classes_, y = unique(y, return_inverse=True)`` and I used that in a lot of places.
We probably should put this in a different method, to ease reading and to ease the reusability of thi sparticular thing.

Also, I'm wondering why debts is a list and not a dict, as we're putting dicts in them with only the person and a balance (which could be key and value)
Yes, you are right, I have made a small re-factoring there..
Please use `$p= new Properties($name); $p->load($this->cl->getResourceAsStream($name.'.ini')->getInputStream());`, `fromFile()` [is deprecated](https://github.com/xp-framework/xp-framework/blob/master/core/src/main/php/util/Properties.class.php#L155)
Sure, for future, but not now.
Y u no like Snow?
active being removed?

maybe your development database wasn't migrated?
Then, while this makes things not fail, it's not right.  The entire "nothrow pure {}" shouldn't be included at all.
Wouldn't we also need this in tibbapp's common.mk?
Shouldn't this use a QTime instead?
Add this file(.project)  to .gitignore.
(All) These tests should now be using memsize_t for nextSize and requestSize.
since we've lost our content, seems like an exception would be better.
Not sure why this was requiring physical deletion before...
Takes a `Library Specification` object and pulls/updates dependencies. Other setup moving parts could also be shoved into this method.
I think he's referring to the last

621: if (build['build_type'] == 'Nightly'):

No need for the parentheses.
Wow, that's some really interesting behavior I haven't encountered in Python yet -- thanks for the TIL =)
Yep, you're right. The parser for matching parentheses is not perfect. But then the MsSql2005 dialect suffers from the same problems because the method is a copy from there to get a quick start in trying that out.
Also when using direct SQL queries, eventually contained comments with parentheses in them might lead to incorrect results. Are there more such simple parsers somewhere in the NHibernate codebase? Maybe this aspect could be covered more centrally.
Do you know if $module_name (two lines below) and $caller_module_name is top scope variable? I can't easily discern by testing 2.7.x and 3.x.
Same as above for these two.
Could we try using `return self.is_element_visible(self._logout_link_locator)`
Maybe also give an example of a deep include here as well?
Tu caches les erreurs? Il se passe quoi si le gars soumet avec un champ vide?
Still hasn't been fixed. :)
if ret =~ /timed out/
Stack-based buffer overflow in the BCAAA component before build ....
I read this code again now, and it confuses me every time.  I can see the intention, but that in-place modification of preferred_plugins... That affects subsequent load operations.
I will add a reference to http://en.wikipedia.org/wiki/Wavelets. Levels refer to wavelet levels, and one would really need to understand wavelets to some extent to grasp what a level means.
See: http://sepwww.stanford.edu/~morgan/texturematch/paper_html/node3.html
A cleaner way to do this is to do `@errors = Array.wrap(@errors)`.
i was conflicted about the idea of putting an assertion in a page object...seems like bad practice. on the other hand, it's just a different way of throwing an exception...
Unless I'm missing something, block comments don't nest  very well.
Since this is a C-style `struct`, I believe the term is `fields`, not `member variables`.
We should probably format this across multiple lines.
同上，无论是鼠标按键的映射和键盘按键的映射都只需要存放到同一个map里。
Since:  "2.1.0"
"[WARN] Couldn't determine the appropriate text encoding for: %@. Please report this to Appcelerator support."
Should not take an argument.
Indent 2 spaces more
Erm, wut? Is this idiomatic style, stuffing things in a tuple and then pull it out again like this?
Maybe instead of just returning false here (thus ignoring all other tenants), a continue would be more appropriate here.
Looks like you've figured out to automatically text wrap with code. This functionality could be very useful to include in sympy.physics.mechaincs. Let's talk about how you do this? Maybe the functionality is already in sympy.
Was it intentional to change appendfsync to everysec? Didn't see it in the pull request description.
We should add a blank line after each and every `modelVersion`.: http://maven.apache.org/developers/conventions/code.html#POM_Code_Convention
Я запутался, у них неймспейсы в эвентах с конца что ли?) Расскажи поподробнее, или дай ссылку почитать?  

--  
Valya
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Thursday, 13 December 2012 г. at 17:09, Roman Pominov wrote:

> In app/assets/javascripts/uploadcare/widget/dialog.js.coffee:
> > > - close: -> @content.fadeOut('fast') > + @switchTo(@widget.tabs[0]) > + @content.fadeIn('fast') > + $(this).trigger('uploadcare.dialog.open', @currentTab or '')  
> А вы потом это как-то используете, то что получается красивая структура внутри?
> Вообще не страшно конечно. Просто в jquery придумали точки использовать для другого, и может кого-нибудь смутить, кто будет вашим виджитом пользоваться. Или это какие-то внутренние события?
>  
> —
> Reply to this email directly or view it on GitHub (https://github.com/uploadcare/uploadcare-widget/pull/9/files#r2406874).  
>  
>  
>
вот тут ты и мог заметить странное не соответствие. чтобы узнать количество все пишут size.
count --- это "посчитай"
This is a good test of the sorting options on mobile, but it might be better placed in a filed called test_sorting.py.  For desktop, we have sorting tests in test_search.py.
throwIfFontPathCouldNotBeResolved
That code was actually smarter than what I had in mind. Well played.
Same note about line length. I don't see any reason to use newlines when the statement will clearly fit on one line with plenty of room to spare.
He would have to change the assert to `fullyQualifiedName!(Outer.Inner) == "Outer.Inner"` which means the module name isn't encoded since it's a type nested in a unittest. He's explicitly testing qualification of types within module names so I think this is ok as it is.
Make sure the header follows the convention used in other druntime headers.
ingenious
What's with the 'minus'? It's called a dash in non math afaik.
The purpose of picking `MINIMUM` was to describe an absolute lower bound.

Might I suggest either `MULTI_DEVICE_INTERVAL_MILLISECONDS` and `SINGLE_DEVICE_INTERVAL_MILLISECONDS`, doing away with `MINIMUM` altogether, or adding `Math.max` to `getSyncInterval`?
what does this mean? No input/output params?
I can rename it. Though the correct name is upper-case and IIRC there is something in the phpBB coding guidelines about using upper-cases with regards to names.
I can be mistaken though *(didn't check)*. So if it doesn't it can indeed be lower-cased.
Although besides `__call()` probably nothing should call that member

Sent from my iPod

On May 3, 2012, at 4:55 PM, "Michael C."<reply@reply.github.com> wrote:

>> + * @copyright (c) 2012 phpBB-Blog
>> + * @license http://opensource.org/licenses/gpl-2.0.php GNU General Public License v2
>> + *
>> + */
>> +
>> +/**
>> + * Cache object
>> + *
>> + * Internal caching object, is responsible for all cached information,
>> + * this object wraps around the phpBB caching object and gracefully falls
>> + * back to that if a given method isn't found.
>> + */
>> +class phpbb_ext_blog_blog_objects_category
>> +{
>> +    private $db;
>> +    private $phpBB_cache_service;
> 
> +1 to ikd
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/phpBB-Blog/phpBB-Blog-for-3.1/pull/66/files#r768408
Porque esse assert de novo?
Same thing about docs here.
il me semble que c'est drôle comme constructeur... ça devrait pas être interne ? on devrait pas avoir une méthode pour faire ça à la place ?
genre getApplication().createWindow()... il me semble que c'est un leak de voir le MC à cet endroit là...

Qu'en penses-tu ?
Please add this model to this pull request.
the leading ``\`` is missing (or even better relying on the use statement)
you misunderstood me. i prefer having it in the .travis.yml file so that its clearer what travis needs to do by just looking at that file.

the disadvantage is indeed, that this makes it impossible for others to just run this command to pull in the composer dependencies. however i expect users to eventually have global composer installs etc.

so imho its better to move it to the .travis.yml like we have done in most other repositories.
is there any reasons this cant just default to 127.0.0.1?
колонка для очереди
i guess also, binding events to `Khan` directly frees you from having to pass it as an extra parameter to `$.triggerHandler`, right? so that all handlers that run in the Khan context... the khantext?
new ImportResult class handles this better now.
Type is backwards.
Django model managers have a `get_or_create`  method that can replace this entire try/except block, and has slightly more advanced handling for race conditions.
@croby I don't think so, as you're saying we usually use `document` for this so I will change it to that.
This template is REALLY hard to read and follow the logic.  I would prefer we use a number of smaller sub templates than doing logic like this at the head.   One template for output mappings?  One for numeric concepts? One for coded?
SONAR does complain about unused imports and flags it as a violation if i am not mistaken.
I don't think we need the word Adapter here.
See comment on the other repo - do we want to remove these?
The method doesn't belong here.
I agree it not being the prettiest solution, but it would require us to move the ip to the params as well.
You can simply define global error handler. [See](https://github.com/wlanslovenija/PiplMesh/blob/master/piplmesh/frontend/static/piplmesh/js/common.js).
Don't concatenate static files URLs manually but use a template tag.

I recommend you check [this base file](https://github.com/wlanslovenija/PiplMesh/blob/master/piplmesh/frontend/templates/base.html) for some ideas.
and docblock!
Probably an inadvertant file save in vim :)
we should run phpcs_fixer to avoid disparing CStyles
Please undo the reformatting, I just spent 10 seconds looking for any non-formatting changes in the first hunk, and this one makes the code unreadable. Ctrl+Shift+F is **not** your friend!
In general I simply add `.*`. :-) (You can always force to add the file if you really want.)
I actually just use it to access query, something similiar like https://github.com/FriendsOfSymfony/FOSRestBundle/blob/master/Request/ParamFetcher.php

However domain specific objects instead of the generic object.
Could we re-use the 'name's for this? Adding 'Mixed' of course, but i'd find it confusing to have two descriptions for the same mode (one in the settings and one in the loading screen), especially if i want more of one of them and cannot find the corresponding section. Also it saves two strings :-)
I wondered that myself, I will change it
it's not a  Role FMPOV
It's for the h2 title of the html log. See e.g. http://dragonfly.opera.com/app/stp-1/logs/opera-dragonfly-1.2012.07.16.1-release-task-9-dfl-3275.a1be3aa780df.html. There is still a hash link. But perhaps we shouldn't do that? Keep the whole tag intact and make the whole title one link?
why was this test commented out?
I am not sure about all cases either, but the UPDATE action for an onDelete event sounds like something we might want to check for. In this case the information for this entity is still in the index and searchable, but we cannot retrieve anything from the db.
What is this?  Why would anybody want a "unique result code"?  Again, I think you need to look at our own examples of use.
If *.class is always added and executed as the first filter: Which other 'resources' would you like to include that needs instrumentation?

If everything is included by default, wouldn't exclude be the only option?
This is broken. You cannot assume that all encoders are implementing the DecoderInterface as well
Why the block comment? The method doesn't seem to do anything.
Можно реализовать конкатенацию картинок с огг-файлами, как сейчас это происхождит с рарджпегами
This is a little weird.
What's wrong with `expect(logger.warn).toHaveBeenCalledWith..` checks? It's kind of weird to validate each call by index [0,1,2 etc].
please remove i
also these obviously need to go in the webplatform
Uhh yeah, you're right -- it doesn't make sense. What I meant is that all Citibank debit cards have a Mastercard logo which means they can be used either a debit card or a credit card. You already mentioned that though so I guess I should take it out.
Other license