labofsahil/github-event-dataset-2012 huggingface dataset:
Makes sense. I added this check. `stash()` now throws an `IllegalStateException` if the same message is attempted to be stashed twice.
no need for ``else`` here as the ``if`` returns
Sort of like a variable, when not ```None```, gets presented in an ominous red glow? :)
Globally, for consistency with other Phrasea Objects, why use ``getTaskID`` instead of ``getTaskId`` ?
Doesn't the space need to be here? If `detailedOnly` is false, then last thing appended was the list id, which doesn't have a space at the end. (This is only a question; I don't know much SQL)
If you use Project -> Properties for the c:geo project and afterwards select Java Code Style -> Formatter, isn't that set to have project specific settings with a profile called "cgeo"? Normally those settings should be shared and be applied automatically. If not, we need to check this.
Export?!
there'd be a lot of stuff needing lombok in that file alone...
Sounds good to me. Something like _is_valid and _is_valid_user_name?
Yeah sorry, hadn't been integrated yet. 

It's now in though https://github.com/infinispan/infinispan/commit/d9cefba2acb771b61f8b21df50a8935744379da8
201Z ??? I guess it should be 2012 (with a two not a letter Z)
This approach just provides some additional backward compatibility for installations that don't have any sites; if we make sure a site is always present (through `alchemy:upgrade` etc.), we can probably simplify this.
docstring. also, does the axes.texts list need to be cleared?
If so, I'll undo these changes and leave the typos in the history.
hi @teodosia 
this is the print i left behind for debugging purposes (if the list is empty when you expect it to be full, you probably have an issue with FF version vs selenium version).
Thinking about this, the error object will tell you what the problem is, as there could potentially be multiple causes. This is a standard line that we use everywhere, and all reason as to why it could go wrong will likely be out of date too soon.
Can you put the url between (single) quotes? Makes it easier to spot the URLs in a CSS file.
"use" should go on every line
We split out the Lookup to have a separate entity, that we can reason about. If you think in the terms of a LookupTable it does not care when a row was imported. The method on the lookup should have a more intention revealing name.
good point, I'll change that.
Ah, nevermind that, I missed the expectation on driver. That's fine.
Agreed, and I will take a look at splitting this into two different functions that have the two different loop orderings.
I don't know if this is the best example for an array property ...
Oh, I didn't know we were so anal about that. I thought you meant there was actually something syntactically wrong with the query. I'll "fix" this (and then others) when I get around to it.
And with anchor type, and dummy function, you could do nicer

use the convention to append a "_" to show this is a "technical" function, I think this is the rule in gobo for instance
so it would be

context_type_: FILTER_HANDLER_CONTEXT 
      do
	  	(create {EXCEPTIONS}).die (-1)
      end

This way, the compiler accepts this declaration using attached type.
Indeed the type checker is quite smart, and it knows the "end" of `die' will never be reached then same for context_type_, and thus it can accept the attached result type.

in fact, you could do the same with any feature with a "False" postcondition such as


context_type_: FILTER_HANDLER_CONTEXT 
      do
	  	never_returning_feature
      end

never_returning_feature
	do
	ensure
		False
	end
- Are you sure that the property "filter" exists? Did you maybe forget to declare it?
- Are you sure ``$this->connection`` has the correct type as ``null`` is not a sub-type of ``resource``? Please also check line 61.
或者callback(e);传递给外部。不能就这么吃掉。
Gibt die wirklich auch das aktive Rating? Das gibt doch einfach irgendein Rating?
PRIMARY KEY?
You didn't merge it well!!!
Add it to requirements.txt ?
It was incorrect, no? Because the field is blank after reload. So there is nothing anymore.
Are you sure you cannot solve this without `!important` flag? As we have all CSS in our hands, this is strange to be needed.
Indentation. Every block indent.
More one thing.

I have an ancient laptop that either does not have a dedicated video board.
It does not run the Windows version cause we use some stuffs not avaliable
on the laptop.
So i added a Compilation Symbol (REACH) and made some changes in the
monogame to run on this laptop.

Should i send this to the repo, or monogame will only suport Hidef profile ?


Thiago Dias Pastor


On Wed, Feb 29, 2012 at 6:34 AM, Christopher Chambers <
reply@reply.github.com
> wrote:

> > @@ -65,21 +65,49 @@ internal void Apply()
> >               {
> >                       GL.BindBuffer(BufferTarget.ElementArrayBuffer,
> ibo);
> >               }
> > -
> > -             public void GetData<T>(int offsetInBytes, T[] data, int
> startIndex, int elementCount) where T : struct
> > +
> > +        public unsafe void GetData<T>(int offsetInBytes, T[] data, int
> startIndex, int elementCount) where T : struct
>
> Hi Thiago,
>
> For GetData\<T\>, could a single call to GL.GetBufferSubData be used,
> rather than having to loop and copy data?  This would allow you to remove
> the 'unsafe' keyword, which doesn't match the XNA API.
>
> I know GetBufferSubData is not available in GLES, but I don't think
> MapBuffer is either, so we'd need a different solution for the mobile
> platforms either way.
>
> What do you think?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/mono/MonoGame/pull/324/files#r500561
>
that's a good change, hadn't thought of approaching it this way. Will make this change.
I think this is redundant here because you already do this check in `_calculate_actions` :)
That's why I said you didn't need to change. I'm just telling you that the sizes you assumed are wrong, should be 7+2 or so

it's minor issue..
Hm or remove completly. The english texts are okay and I only added this part of the script for the german users of our project.
Can we move this to an initializer?
Not a major issue, but g_return_val_if_fail() should be reserved as an alternative to assert().
According to GTK docs the function should be used to test "if (programming error detected) return".
This static method has the slight unfortunate consequence that a user might occasionally write

`d = Dendrogram()`
`d.compute(...)`

And lose the result of a (potentially lengthy) calculation.
Also here, this is not a `Db` :)
Bind 9.7.3
I changed it so that the logic is left up to the PathFilter, and I added some convenience PathFilters for accepting directores vs files and for composing those with other filters. Seems a little bit cleaner this way.
yes but "personas" doesn't make sense. maybe it should just be `bidpom`
"decode"
I'm not 100% sure if I understand the question but you're supposed to set it in the module's per-server config in your NGX_HTTP_SRV_CONF command handler. ngx_pagespeed doesn't have any such commands right now, they're both main|server|location.
It only makes sense when you have committed _after_ a branch (local or remote). Usually you shouldn't have any commits after a branch (because this branch is considered as merged).

Here is the result if I have 10 commits after the branch test:

    heads/test-10-g196880a


So it's the branch name followed by the number of commits followed by the short commit id.
I thought you wanted relative ones? Or was that in the past?
This  referred to the the ``__init__`` method of DiscreteMargins
Can you do the same in swift_required_pear.php as well?
i need to examine this class in more detail. i have a working version in a client project. i am wondering if we should provide a fully working default implementation or if we should just offer some abstract classes.
I see, but afaik people usually chain methods as much as they need without adding any unnecessary parenthesis.
а это не ты хотел объединить настройки

    PYUPLOADCARE_USE_HOSTED_ASSETS = False
    PYUPLOADCARE_WIDGET_URL = 'http://path.to.asset'

? ну в смысле какой в них и правду смысл, если Тру в первой это все равно что значение по умолчанию у второй?
该命令定义了系统各方面功能，比如（包括？）nginx允许使用何种方式来做连接处理。  这样？
I know, I don't know why I didn't do this about six months ago
In spanish we use "desvanecer". Google translate gave me "dissipar". If the word is correct you might use it.
I think this line is worth repeating everywhere in case people open up a file out of order.  But if you think it's too repetitive, we can just put it in the first one.
And I would say an exception, not a warning, since it's almost certainly an error that the user should immediately fix, rather than continue on doing something that isn't what they apparently wanted.
I think that was because we thought that people mostly specify Sersic profiles based on half-light-radius.  I've never seen anyone use FWHM in practice.

On May 9, 2012, at 12:35 PM, Mike Jarvis wrote:

>> @@ -202,16 +204,17 @@ def __init__(self, beta, truncationFWHM=2., flux=1., re=1.):
>> class Sersic(GSObject):
>>     """GalSim Sersic, which has an SBSersic in the SBProfile attribute.
>>     """
>> -    def __init__(self, n, flux=1., re=1.):
>> -        GSObject.__init__(self, galsim.SBSersic(n, flux=flux, re=re))
>> +    def __init__(self, n, flux=1., half_light_radius=None):
>> +        GSObject.__init__(self, galsim.SBSersic(n, flux=flux, half_light_radius=half_light_radius))
> 
> Why not fwhm here?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/GalSim-developers/GalSim/pull/140/files#r795996


-------------------------------

Rachel Mandelbaum
http://www.astro.princeton.edu/~rmandelb
rmandelb@astro.princeton.edu
Maybe use classList.add/remove here?
bad styling.

  on-line function declaration suites for those places where in scala you'd use closures with one statement in it, like map, foreach.
callbacks declaration is not the case.
다른 부분에서 'swipe'를 `스와이프`로 했기 때문에 그에 맞추어 변경하겠습니다. :)
@user_configがnilのケースを考慮できていないので、ログインユーザの config がまだ存在していない場合にこのコードだと怒られてしまいますねぃ
The reason we don't want to pass the painter is we want to be in charge of painting the icon ourselves, like we do for the subscription and type icons.

Would a playlist that is collaborative have per-updater icons? Or will a collaborative playlist just show a "generic collaborative icon", like the synchronized icon on the right edge? If it's not updater-type-specific, we don't need to add a collaborativeIcon() pixmap at all--the playlist item can simply cache a QPixmap (either self-drawn or from a file so J can play with it) and if it finds a playlist that has collaborative() set to true, paint it.
absolutely! I didn't know if "moved" was used elsewhere so I created a new one
`assert_expected` and `self.assertEqual` are not quite the same.  The difference lies in the message that you get out of them

`self.assertEqual` tests for equality and optionally allows you to add a message on failure, but you've got to create that message yourself every time you use the method.  `assert_expected` creates a standardized, easy to parse message for you.   

In many cases, I'm comparing dicts or lists of many elements.  During problems, I want to see how expected varies from the actual.  `assert_expected` gives me the two vars printed aligned on separate lines.  It makes it simple to scan down the parallel lines to where the differences are.  

On the other hand, if I were to use ``self.assertEqual`, I would have to format the string each and every time I wanted to use the method.  Call me a lazy programmer if you will, but I'll take my automation over manual work every time.

I could compromise here.  Since `assertEqual` is a method of `unitTest.TestCase`, I could use my object oriented prowess and override it to make it behave like I want it to.  However, the deadline looms, so I'll defer this until later...
Why do you need to mask them out? They ought not to be drawn in the first place.
Ar trebui facut si un trim la path.
Set the expectation on `StatsD.write` like the other test methods?
в прошлый раз отсутствие этой строчки вызвало серьезную проблему. на всякий случай. Ладно, сейчас уберу.
Вот представь себе, что у интерфейса объявлено, что он выбрасывает IllegalStateException при определённых условиях. И в вызывающем коде есть catch (IllegalStateException). Это должно работать даже если туда подставить прокси.
Область видимости этой переменной слишком большая.
Through various stages of refactoring, `World` became less and less aware of Blind's game rules. However, these tests still sort of cover both generic functionality and game-specific rules, and need to be cleaned up.
Ah well, thanks for checking. Looks intuitively "wrong" to me, but not a big deal.
Use `ln[0].ord`
true, and I can't see any problems as such with it, I'm just saying ;)
The point is that Markdown is not well suited to dump values. The markdown rendering breaks when the value is an array, and is not always clear for others (some values cannot be distinguished when casted to string)
userName should be personaName, as it's the personas name the request uses.
How do we validate the possible options for `:framework`? I don't want someone typoing something like

    rails g simple_form:install --framework fundation

And getting back an error saying that a template or readme does not exists. I think it'd be good to ensure the available options with `class_option`, if thor does not allow us to do that, we can do that ourselves :).

Wdyt?
@bodepd Not sure I see the objection with this, but I may be thinking about it wrong.  What we're doing here is letting the params class find out if hiera is being used and then settings defaults around that.  However we are enabling a user to turn off hiera in this instance rather than forcing them to swallow it by setting hiera_enabled as a parameter rather than reading it straight from the top-level variable.

Did I miss the point on this one?
I guess it would be useful. Will add.
exact solution would be more appropriate
If this is really debug code, I would avoid it in the trunk
this seems a little odd to me. wouldn't the `item` that you pass in here be an instance of `Equipment`? or would it be a name of an item? in any case it seems to me that this method might rather belong to the `Character` class?

Maybe what you mean is to have a method `use` without any arguments that is being called by a `Character` instance that "has" this item/equipment?
Alright, fixed.
:) I probably need to help you with some of the tests, huh?
Not required for a constructor without parameters.
The `_` is used to denote a private function. If you're sure it is ok to be public then please remove the `_`.
Why it is not: @user = User.new(params[:user])?
http://guides.rubyonrails.org/form_helpers.html#binding-a-form-to-an-object
Then I can't pull it until Gavin gets his linux box working....
I thought about that too, but it sort of seems like hidden behavior to me.  I'd rather add a --no-reload.
I think this destructor was introduced when you had removeRowBuffer call in here.
so don't need it added anymore
I think you can drop this test actually. The code in here is nearly the same as in test_views.py and that's sad. 
And we don't have to be purists in that test_models.py gets full test coverage of models.py alone because from now on the code you've added to models.py is just one line.
Re-write all this with:

```python

if form_selection != 'by_report_type':
    report_type = [hang_type]

adubyday = api.get(product, versions, os_names, start_date, end_date,
  report_type=report_type)
```
why are you adding this? is it used?
Reference these based on current namespace and/or imports.
The `TypeSerializer` is not used anywhere. Can it be removed?
+1 for linkToMethod
RLY??
Doesn't git use ``...'' for quotations?
Nevermind. I got it.
i had them as lower case as they are sourced form the config file, different from true constants, I can change them simply enough
I have modify this.The force default value is false.
don't need 'Version'
Should also ```register_autofilter_ports(3790)```, which as @bperry-r7 pointed out is the default for Pro.
We could add `mode` and `cval` parameter here. Also, we should keep `output_shape` rather than `out_shape` for consistency with all the other transformation functions.
This should be a `module`, not a `class`.
According to http://www.python.org/dev/peps/pep-0008/#inline-comments
Inline comments should be separated by at least two spaces from the statement.

PEP8 will return ``E261 at least two spaces before inline comment`` before I fixed it.
It's important to have as much consistency as possible between the JSON version and the javascript version especially if someone is trying to copy our examples and build off of them in their own files. Perhaps it is best to wait for a couple other opinions.
Good change.  I looked at this function pretty recently, and thought we should get right of using `arguments`.  This is very early code.
retain is only done when view is attached. Otherwise it is  assign
group vars.
can someone test this?
This mechanism with removing and deleting widgets and accessing them by index is error-prone. I would recommend to just have a hash of details widgets, with the link number as key and the widget as value. Then call show and hide on the widgets to show or hide them, but don't delete them. This should make the code simpler and more robust.
и эта тоже выглядит издевательством
I would just add a step here about team review, or test of the staged artifact.
I'm not against adding the basic fields to $info. Only calculating time to password change for others than the logged in user.
Why not change to 

 if($_SERVER['REMOTE_USER'] == $user && $info['expires'] && $this->cnf['expirywarn']){
This is actually a pretty good function, wonder if it should be published.
This opEquals DDoc seems to be opCmp. Following opCmp DDoc too.
Hi, Arjen!
Well, won't it be backported into Spring 3.1 ?
In the Spring Integration 3.0 we plan to add URI Templating for `<ws:outbound-gateway>` to support all Spring WS transports: https://jira.springsource.org/browse/INT-2720
But this release will depend on Spring 3.1 ...
Or do you propose to port your changes into Spring Integration for backward compatibility? ;-)
I probably shouldn't have upped the version here, but we can just keep this until we tag a new release.
prob don't need .. here
> I was organizing TestClientsEngineStage by grouping mock servers, delegates, tests, etc & adding a comment for the group. Is it just the @author that you prefer I remove? Or the entire comment?

The `/**` implies a Javadoc comment for the next piece of source. What we really want is Objective-C style `#pragma`s to categorize the source, but in the absence of those just kill the block comment entirely.
Do you ever expect a null to be returned? If not, one idiom I like is to have ``handleException`` return the same throwable that it throws, so you can do this:

``
} catch (Throwable ex) {
   throw handleException(e, "...");
}
``

``handleExeption`` itself should throw rather than return the exception, but this way it's clear that a null can never escape the method.
The reason I do not json_decode() it here is because the application using library should do the decoding of the data calls return, is there an internal method that fails due to this?
Hmm.  It is the same issue, since we'll need all the specialties to make the dropdown menu.  That sounds like a good idea.
Can just change hwprobe to class_check and import the class without having to update code in the module. (but not nack worthy)
I would prefer to use pure json messages here
does not seem to be done
Had some trouble in getting annotations to load. It seems that I didn't clean up enough.
Wouldn't this potentially happen for all locals, not just es_419?
some whitespace problem here?
as disscussed, just enahanced the logic in the ReplInterceptor to force async repl for TO caches.
This method should have a docstring, and will need to take the map as an argument once its not a `self` var
Fix spelling mistake - it's `Existing` not `Existen`
Using the sites (http://en.cppreference.com/w/cpp/io/basic_ostream/operator_ltlt) and (http://www.cplusplus.com/reference/iostream/ostream/operator%3C%3C/), I believe we are missing:
- long double
- signed/unsigned char
- const signed/unsigned char*
- std::ostream, std::ios, std::ios_base (instead of the typedef std_stream_manipulator)

And therefore do not require:
- std::stringstream
- std::string
- std::string*
I wonder if we should make this a generic class

ExpressionDelegateFactoryBean, with setAttributes(), addAttribute(), getAttributes().

That way, the next time we need something like this, we can reuse it, rather than inventing another one of these with different metadata.
I found that docbook renders better (margins) when you do: 

 ...used by the <classname>ImapMailReceiver</classname>.
</para>
<programlisting ...
</programlisting>
<para>...
If ~~however,~~ multiple entities are ~~selected~~ returned, an
OK; I understand what Artem is saying...

By using the same evaluation context for the attribute to be __set__ as for the attribute to be __retrieved__, and now that the evaluation context has a bean resolver, this has the side effect of allowing an &lt;enricher/&gt; to set __any__ attribute on __any__ bean, rather than limiting it to set properties on the message itself.

If this is OK, I will merge, but if it's an unintended side-effect, we should consider using 2 evaluation contexts.
Good points. Regarding context-refresh vs. late phase though, we could simply make the default phase value MAX, right? (still exposing the setter but with a good javadoc comment explaining why it's MAX by default).
Instead of calling this in every function, maybe we should store the moore parameters as an attribute to the Run class. Or you can add benchmark_to_moore as a method to the Bicycle object in BicycleParameters.
"StyleGuide).A very " We need a space after the period here. :)
This doesn't seem to work on my test system. From the puppet resource docs:


group
=====
Manage groups. On most platforms this can only create groups.
Group membership must be managed on individual users.

Seems like we need to handle this differently. Perhaps add this to https://github.com/theforeman/puppet-foreman_proxy/blob/master/manifests/config.pp#L2 ?
where are the coding guidelines followed
Also `public` keyword is useless in Interfaces =)
No need to re-derive `fname` here, I have already calculated this above as `inputSource`.

```coffee
runMain input, js, jsAST, inputSource
```
the line is too long.

Try to wrap to 80 chars (PEP008)
Fair enough, I'll make that change.
according to [phpcs](https://github.com/opensky/Symfony2-coding-standard) this is CS compliant now.
line change between ) and {
I think I know what these values mean, but the description doesn't really tell me.  What does it mean for the data to be shared?  What does it mean for it to be private?  I assume `shared` means "overlay this document on top of the parent document" (i.e. composite collection)  and `private` means create a separate document (i.e. a sibling dynamic object collection).  Is that correct?
Yes, I hate Python properties. How would a common getter/setter work out which property is being accessed?
Isn't it just temporal fix? layout subviews should be called in willAppear, because still bad layout is flashing firstly.
What might be the cause of different behaviour in iOS6? I noticed that problem occures only while building project using Xcode 4.5 (or rather iOS SDK 6.0). If you build using old one (i.e. 4.3.3) it still works fine, even if iOS 6 device.
you still have an issue here: the argument should not be optional as the default value breaks the code
@Koc Circular reference issue AFAICS (btw, the real place to solve it would be the helper, not the Twig extension as the helper would probably break because of a circular reference if you activate the PhpEngine without activating another engine)
<string name="hello">Maclaw工作組- 相機 - Beta 1</string>
Поехало выравнивание. Табы+пробелы?
Yes:  https://github.com/scikit-learn/scikit-learn/blob/master/benchmarks/bench_plot_svd.py

If you want to add a benchmark for arpack on sparse data, please feel free to do so as well :)
baba reset markazi !
Should this be ```conf_db```?
True, but it has a custom getter (lazy).
So you would @property/@synthesize in the .m but still provide the getter?
Los enteros se normalizan a Integer o LongInteger según dice acá: https://docs.djangoproject.com/en/dev/ref/forms/fields/
igual los integer aceptan 32 bits -> [-2147483648,2147483647] (http://infohost.nmt.edu/tcc/help/lang/python/2_6_3/reference.pdf)
Ahora hago las properties.
You forgot to change this one. Did you actually test this pull request? Please do.
Typo "can be a" -> "can be"  (sorry didn't notice this first time :( )
We should explicitly state that modules cannot replace the root controller, and should not implement this protocol. It is only documented because `TiRootViewController` conforms to it. Note that this may change, if we offer other root controller configurations and make them extensible.
This source link is still wrong, no?
Yeah, in fixing the docs, I noticed that I'd talked about a fictional "trace" call in order to explain how closures behave.  There was no trace call when I wrote it, but now there is.  So to eliminate confusion, I renamed it.
Just some formatting standard we follow - The *catch* is on a new line. Another example are if-then-else statements:

    if {
        ...
    }
    else {
        ...
   }
Хотя, учитывая то, что мы инициализируем переменную в самом начале функции, может GC и не будет тупить, сохраняя значение переменной, вплоть до следующего вызова функции (такие вот оптимизации у JS-движков).
you could make the alias private to allow more DIC optimizations as you don't need to retrieve it at runtime:

```php
<?php
$container->setAlias('fos_comment.markup', new Alias($config['service']['markup'], false));
```
ok so I assume when you make the sourceOrigin *, it doesn't cover local://
Is my understanding correct?
Tehtävä piti tehdä yhdellä vertailulla. Haettiin siis jotain tämmöstä:

~~~clojure
(<= jotain kamaa tänne näin)
~~~
You should rebase on master i think, i've merged that today