labofsahil/github-event-dataset-2012 huggingface dataset:
namespace LeechCraft
{
namespace Azoth
{
    AccountsListWidget::AccountsListWidget (QWidget* parent)
    : QWidget (parent)
    , AccModel_ (new QStandardItemModel (this))
    {
        Ui_.setupUi (this);

#ifdef ENABLE_CRYPT
        Ui_.PGP_->setEnabled (true);
#endif

        connect (&Core::Instance (),
                SIGNAL (accountAdded (IAccount*)),
                this,
                SLOT (addAccount (IAccount*)));
        connect (&Core::Instance (),
                SIGNAL (accountRemoved (IAccount*)),
                this,
                SLOT (handleAccountRemoved (IAccount*)));

2012/5/7 Georg Rudoy <
reply@reply.github.com
>

>
> Плохо передавать уже интерфейсы прямо в сигналах, ибо сигналы не умеют в
> неймспейсы. Поэтому в азотхе, в частности, везде передаются QObject'ы.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/0xd34df00d/leechcraft/pull/113/files#r780380
>
Ruby methods actually often don't an explicit return statement.  The method returns the value of the last statement executed by the method.
I don't think we should break b/c here.
Well, I'm concerned that this commit 28dc6ba4fcf77f0f7c11490a6459a0b9b23356b2 got rid of the concept of "sanitized" names.  

Spaces and special characters should be handled by default, so that "The Status" would make the route: "the_status". 

We could also add an "as" option:

```ruby
ActiveAdmin.register_page "Stâtûs", as: "status" do
end

```

Would you like to work on this in a separate pull request?
Crap, hold-over from my s/"/'/ that did the damage previously.
This should be a true false (0/1), not a date.
Mircea, shouldn't we have a L1NonTxConcurrentDistributionInterceptor as well?
Yes, sub-millisecond timeunits are not too useful for expiration and idle.
not really. I guess it doesn't follow my patterns of using static methods (factory methods, reuse stateless code) and I though that you made the effort of writing one more word in declaration with a reason - I din't see the reason so I asked :-)
I don't there's a need for the instanceof...

notifyCacheEntriesExpired() could take a Collection<CacheEntry> 

If you look at notifyCacheEntriesEvicted(), you realise that the only methods of the InternalCacheEntry used are getKey() and getValue() which are present in CacheEntry interface.

Please fix both notifyCacheEntriesExpired() and notifyCacheEntriesEvicted()
I can see what you're trying to do here, but wouldn't it make sense to have 2 managed attributes setters instead?

I mean, what do I pass in as afterFailures if all I want to change is minTimeToWait?

    setTakeOfflineAfterFailures(String site, int AfterFailures)
    setTakeOfflineMinTimeToWait(String Site, long minTimeToWait)

This is one possible option, but I'm sure there're others...
why static? :-)
I'd rather we delete this completely, I'm seeing 1000s of instances of this message during the test suite :)
Thanks.

My guess is that it will go into 1.3 (or whatever the new version is called), since 1.2.1 should be for critical bugfixes.
Please can you align src: with frame: above and the rest of the object too.
This doesn't work for me as written; adding the rule to the end of the CSSRuleList does work, though:

```document.styleSheets[0].insertRule(rule, document.styleSheets[0].cssRules.length);```
Quirk of csv/parse-csv
I don't feel like it is a hack to have it activate on a trigger run from tabset. But it doesn't feel right to add a trigger just for this purpose.

If there is a way of getting this to work for IE, I think we can do this...
When you're removing a page now (from draft+live), it shows a readonly version of the same, with the tree staying on the right. So a hybrid between the "edit page" and "page history" view actually. Strictly speaking that's a bug (undefined view), but to users I guess it still makes sense. The actual problem is that the tree doesn't visually indicate that the page has been removed, but that's already logged in trac. From a PJAX perspective, the server can choose to send back the "Content" fragment, which replaces both Tree and CurrentForm.
Thought that option was to override default_format if defined, but if it doesn't work like that then leave it as it is.
Good point, will adjust that when I pull it in.
My eye keeps missing the difference between `EditorToolbar` and `EditorWithToolbar`, which leaves me *really* confused. Maybe `EditorToolbar` can just be `Toolbar`?
It also looks like `class="deki-transform"` isn't necessary for syntax highlighting:

    mysql> select count(*) from pages where page_text like '%function="syntax.%' and page_text not like '%class="deki-transform"%';

This turns up 493 pages.
I found a nasty race condition in my code today, and I see you're suffering from it too. You should change this check here to: `OXT_UNLIKELY(pool == NULL || process->detached())`
Should the type for parameter ``$tableGateway`` not be "null|TableGatewayInterface" instead of "TableGatewayInterface"?
这里可以考虑一下。如果文件非常非常大，内存会吃不消。只读取文件的前面几个字符就够了。
I am not able to run KS on Android emulator (Don't know reason but it is not running)
But on BB simulator for me also that button is not switching on. 
I set log and saw 'Switch Value = undefined' 
I will raise a bug on it
Preloads all of the tmx/image files. For some reason tmx can only be loaded on firefox when not hosting the code on a web server.
Entweder die Repräsentation oder der PK. Letzteren könnte man natürlich als String übergeben. Für mich wär beides OK...
This should probably be put into a separate function (socket emit) so that it's a bit easier to read
What is the benefit of this? Seems like load_from_database and replace are identical.
why travel the DOM?
document.getElementById('system-message-container').innerHTML='';
Need a new line before the comment otherwise the code style checks will complain.
Ok, I'll move this up into a decorator.
What do you suggest here? Remove `ready` events from panels that depends on map panel and do initialization from there?
But if i have a dict how can get right request_string? Now i have with number of array..
You also get passed `user_id` and `screen_name`. Is this all information you can get from Twitter? If it is, then we do not have to refetch it in authentication.
Or don't. Forget this one.
Still, why hard-coded URL?
97%? Why exactly this value?
`data['panels']` -> `data.panels`, and you can fix this everywhere, so indexing by constant string should be changed to data.index.
So I remember from past that there are some issues with checkboxes. So because you can change them also through keyboard (navigate to it with tab and press space). So you should make it work also with that.

Anyway, make it work with mouse and keyboard.

Hm, or it was maybe input box and keypress and change. :-)
Done.

On 07/03/12 17:14, Joe Groff wrote:
>> +//	GET_FLOAT_WORD(is,t);
>> +//	SET_FLOAT_WORD(t,is&0xffff8000);
>> +//	u = t*lg2_h;
>> +//	v = (p_l-(t-p_h))*lg2+t*lg2_l;
>> +//	z = u+v;
>> +//	w = v-(z-u);
>> +//	t  = z*z;
>> +//	t1  = z - t*(P1+t*(P2+t*(P3+t*(P4+t*P5))));
>> +//	r  = (z*t1)/(t1-two)-(w+z*w);
>> +//	z  = one-(r-z);
>> +//	GET_FLOAT_WORD(j,z);
>> +//	j += (n<<23);
>> +//	if((j>>23)<=0) z = scalbnf(z,n);	/* subnormal output */
>> +//	else SET_FLOAT_WORD(z,j);
>> +//	return sn*z;
>> +//}
> Can this commented block just be removed?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/jckarter/clay/pull/235/files#r529448
This is wrong, per my comments in the Jira.
Indeed, you have greatly reduced the leak. But every call like `jQuery('<tr class="' + ( new Array(64) ).join("nocache ") + '"/>')` will leave `safeFragment` with another child: the initial value of `div` before its descent (in this case, a `<table>`).
Is there a reason the scripts can't be moved into the PostGIS prefix?
I would also call this function `_should_retry`, otherwise LGTM.
Too many line breaks make the code very unreadable IMHO :(
Oshi, both should be false. Rebasing again.
Wäre es nicht einfacher anstelle des zusätzlichen JSON Feldes "unknownbeertype" einfach gleich -1 als breweryid mitzugeben? Und dasselbe bei beertype.
change suffix to .nt
Here you're setting the default username and password to postgres. This will break other installs. Development versions of postgres, like postgres.app default to no username password. 

You'll want to set this in your .env if those are you postgres credentials.
these are just details...
```
my ( $port, $svc, $logger, $np ) = @_;
```

also $logger can be re-fetched instead of passed around
year 2012
@michaelficarra done.

@satyr  I'm going to leave that for a future commit and pull request. I'd like to get this merged after 8 months of waiting rather than complicate the pull request.
Why is there a while loop in a header?
yep the bootstrap-* i want to know if we can use bootstrap-min-* instead
Handling bounded support would be very useful. Having other continuous kernels besides gaussian that aren't fundamentally different (like Epanechnikov) would be at the bottom of my list, it's pretty much unimportant for the result of estimation.
Wasn't this already implemented?
That's a good point, we don't want to rebase this PR. That should happen on a new PR, otherwise many of these comments will be lost.
no camel case
I use the standard settings: org.eclipse.jdt.core.formatter.lineSplit=80.
Is this just an alpha channel? The transparency, as a value between 0 and 1?
I purposely did not bring in any aop specific code in.  The setter will be added once the event logging branch gets merged in.
sry...
should i fix it end commit again?
Good point.  This function isn't that much of a tall pole (using the lookup table is the more common code path), but that's a good change to make nonetheless.  I'll do that.
Thanks Mike, sigh!  Pushed!
ok,done
Suggest deleting this class in this pull request.
What do you think about calling `app.start(ImmutableList.of(testLocation))` instead of cassandra.start?
Makes no difference to behaviour, but I think it makes the look and feel of tests a bit more consistent.
Hmn, its a UTF encoded file, and that mu character seems fine on my terminal in the local checkout:

 fractionSizes = ['y','z','a','f','p','n','µ','m',''], 

~/lab/ruby/rigse » file ./app/assets/javascripts/flotr/flotr.js            asset_pipeline3 ✘ (ruby-1.9.3-p125@rails3_portal)
./app/assets/javascripts/flotr/flotr.js: UTF-8 Unicode c program text, with very long lines, with CRLF line terminators
I can't create a fake admin without to validate it first, so I decided to use the same. But, I can change the test
An improvement, although the even-better alternative is:

```python
if create_user:
```
Have you tried that in the buildservice? I am wondering because asfaik json is not part of the ruby1.8.7 standard implementation (it is for ruby 1.9.3). Moreover, I am not sure you even have ruby 1.8.7 installed in the kiwi containment. You should add a "BuildRequires: ruby" in the containment-rpm.spec.in.
Concordo com o Fábio.  E só para acrescentar, o settings.py é versionado e nós não queremos expor nossas credenciais de SMTP para o mundo.
Why have you changed that, I like the previous version better.
Where is the actual test request made?
This module name creates lots of confusion, it should be changed to show its purpose. Suggestion path resolver

It would be better to remove this module from webinos repoistory and submit to npm, a separate repository in webinos github account.
if it also stops enqueing if there are more than enough jobs in JOB_STATE_DONE right?

Can you toss in a value error when break_when_n_done <= 0  ?
I see you define that function as `continu`, why not just make it more clear by making it something else? that way it doesn't look like a typo. proceed?
Yeah, this is just commented out bits. Probably shouldn't have touched them though.
Here you're calling ``removeStyleClassName`` and ``addStyleClassName`` which are defined in ``lively.morphic.StyleSheets``. However, ``lively.morphic.StyleSheets`` is not a dependency of ``lively.morphic.Widgets``.

This causes some headaches if an error is thrown during the loading process of a Lively world. When creating a new error box in ``setStatusMessage`` a Button is created which also immediately invoked ``changeAppearanceFor`` even if the ``lively.morphic.StyleSheets`` module is not loaded (because it only gets loaded as a dependency of ``lively.morphic.StyleSheetsHTML`` which in turn is part of the default ``localconfig.js``.

So maybe we should put ``lively.morphic.StyleSheetsHTML`` as additional dependency to ``lively.morphic.Widgets``?
The default is a symbol in attributes/server.rb and how it is/was in upstream.  Want me to change them all to strings?
does this get called anywhere in the framework? would just need to make sure to override if so
Какая-то не такая проверка. Запускай перед отправкой, а?
Not sure if we should start adding bugs because the page is still under dev
Use docstrings to describe a class
I can put it in a css class. The problem is that I don't have SCSS setup yet. I'll add a new css file to base.html called not-implemented.css with a not implemented class. Sound good?
cast needed.
Could you please add a docstring that explains what is the content of the dataset and a link to a reference (URL of a website or a paper) that further describe where this data comes from?
Foobar? :)
If I rename the variable to `fit`, the code becomes confusing as the method is itself called `fit`, so please suggest a better name. And do you suggest we rename `train_wrap` and `csr_train_wrap`, whose names currently describe exactly what they do (wrap Liblinear's `train` function)?
Why was this necessary? Last time I checked the tests ran fine on windows and using DS tends to break more stuff than it fixes.

Did that change in the meantime? :)
event if it comes with NativeControlObject.h I would love to have self-sufficient header files
Liste les enregistrements de la quarantaine
Lines 30-33 look the same as 21-23.  Shouldn't these be hoisted out of the conditional?
isDataset and isRecord didn't catch the case where the count is "LEFT.childDs". This one does, and doesn't seem to catch other cases, so it might not be the best check, but works as we need. I might not have reviewed all cases (and compilation regressions might not have them all), so would be good to have something more concrete in here.
Oh, I see! So, the loop is just to try again in case of error.

A few things I find problematic in this approach:
1. If the error is permanent, you'll busy-wait forever. I think errors are errors, empty queues are empty queues. If you can recover from one type of error, do so for that type of error only.
2. The loop is part of the INotify logic, not the monitor directory (what misled me into interpreting it wrong), so it should be moved inside the INotify call
3. Recoverable errors (such as the ones you warrant a retry via the loop) should be contained inside the INotify call, and not leaked via return value
4. Using -1, 0 and 1 as return values with comments is, at least, misleading. Having an enum with the codes is clearer on intent (though this will turn out irrelevant, see below)

However, the most important thing is to separate recoverable errors from fatal errors. From my point of view, INotify should be self contained and there are only three - distinct - exits from that function:
1. Throw an exception if you find an irrecoverable error, or time outs, or too many attempts
2. Try again, inside INotify call, if you find a recoverable error
3. Return (void) otherwise, since the return value is only for error handling.

If the callers are not prepared to deal with exceptions (I'd be surprised), you can return an enum (preferably an already existing one for error codes), or perror, or anything else other than magic integers.
Just this else branch?
Please move it to `MercurialSCMTest` just in case - will ensure that this scenario works sanely regardless of what options you configure on the Hg installation.

Best of course is to temporarily comment out/revert the changes in `src/main`, then run the new test and confirm that it fails in at least one of the test suites. If it only failed in `SharingSCMTest`, that is of interest (possibly means anomalous or buggy behavior of Mercurial with this extension loaded), but we are still better off being sure that it continues to pass in the future in the other suites too.
I am not sure why we should bother with this.. is it intended as an optimization? Since all pages call this it's going to be in cache, using a more specific URL like this is likely to generate more roundtrips.
Excellent! This works just like the other models. Well done.
trailing commas in class definitions dont work on 2.6.x
Run `cfx docs` and look at this page ;)
whats the difference between these two. They look mostly the same.
Brought these up to style.css so other widgets can use this and it's more consistent
We should try to make controllers relatively alphabetical.
`first` would return `nil` for an empty collection
Please use OptPath instead, which will validate the file path for you.
So it would be sufficient to implement the moments functions in Cython.
```sage: eps = var('eps')
sage: sigma = 1
sage: f(eps) = 1/(sqrt(2*pi)*sigma)*e^(-eps**2/(2*sigma^2))
sage: plot(f, (eps, -10, 10))
<html><font color='black'><img src='cell://sage0.png'></font></html>
sage: integral(f, (eps, -3*sigma, 3*sigma)).n()
0.997300203936740```
So to cover 99% of the distribution we need to "go" 3*sigma in each direction from the centre pixel -> block_size = 2*3*sigma + 1
Let's see if @tonysyu has an opinion, then.
I think this is the behaviour we are looking for.  ::-1 reverses the order, slicing returns a new object.
np.ones((3, 3)), then we evoid the latter assignment of H[2,2]
Why not use the `parse` method inherited from `ActiveRecord` to be consistent? (you'd need to replace the constructor with the `populate` method for this to work)
Yup, good point. This relates to your comment from above, in the controller specs, where I use `matcher =`, right? If so, 100% on board.
You don't need to pass this in - `value_outside_of_array` can just use `@array`.
Get cache does a request, but later on, you use opener to call the same url again.
You can pass along params in getCache which uses self.urlopen
Vertical layout has one child per row
Change the => to {} and use a little more room.
These comments are very helpful. You should just use "//":

/** or /// is for docstrings
/* */ is for commenting out code
// Is for most normal comments.
this was based on the assumption that the browse button COULD return either a proper windows style path (c:\something\)  or a unix style path /home/rob.   After today's test, it would SEEM that it only returns unix-style paths, but, I'm not willing to bet that it's the same way for the dstore subsystems for example. I can't guarantee it. So I prefer to leave both in. 

'sep' is the separator we'd LIKE to use.  However the provided path may contain either separator, and we need to be prepared to handle that.
@gnarf37 @scottgonzalez @jzaefferer Now, on config.json, we are able to set jqueryUi as above.
Extra empy line
What is the purpose of this line? I confess that I'm surprised that it compiles at all, since it's before the braces.
Ditto. I'm pretty convinced this pattern isn't working out.
Actually, it looks like probably nothing.  I'm taking them out, then.
for some reason these got removed on the merge, let's add them back :)
wouldn't it be better generating one if none is available? maybe making it easy evaluating if its a stored, generated or set one?
It seems that the Before method isn't needed?
You can also remove the id columns in the patients table and in the registrars table. IDs will be generated automagically by ActiveRecord.
you should probably add the annotations allowing the AnnotationReader to validate annotations as of Common 2.2 (see the ORM or @schmittjoh's bundles for instance)
I found out that Azure server team actually doesn't appreciate the NS we have for the attributes, the server would return 500 error.
Drop the else, as the previous if is returning.
Also: if<space>(
You can remove these comments now?
`.prop()`
I think we can use ``private`` here as the class is not designed for extending.
почему? Сигнал вызывается уже после того, как таб добавлен же.
Totally agree, there's an openstanding ticket assigned to me that should take care of this in the next iteration.

https://jira.sakaiproject.org/browse/SAKIII-4759
No such thing as an `int` in Javascript -- it should be a `Number`
Let's make this handler a separate function so all this does is just set the binding
What about N32? Or does O32 imply N32?
Maybe `nothrow` too?
OK; maybe just name it deleteAsBatch? Defer to the next milestone if you prefer.
As with the existing LoggingObserver, this line is in place to ensure that _need_update() returns True when first called
Same as above...

```Dir["#{fixtures_path}/modified/*.conf"].each do |file|```
I believe you have a parse error here. Did you run the tests?
wrong indent in this file
`imagine, similar flow would` -> `imagine, a similar flow would`
`POST action URI would be` -> `The POST action URI would be`
`This time request would` -> `This time the request would`
`would save edit article to` -> `would save the edited article to`
there is always a parent. (at least after saving). assigning a different parent will result in a move operation
The MSErrorBlock is the existing one. It uses NSError though. Does Cocoa ship something like MSErrorBlock we could reuse?
Ok, then shouldn't the implementation be on the base, too?
Can you move this file to the root dir again?
should be path.existsSync, yes? pathUtil isn't defined.
It was ALREADY in there :/  I just increased the chances.
On Mar 6, 2012 12:34 PM, "CIB" <
reply@reply.github.com>
wrote:

> > @@ -97,7 +97,7 @@
> >                       id = src.wear_id.registered_name
> >                       if (src.wear_id.PHOTO)
> >                               photo = 1
> > -             if (id != src.real_name && in_range(src, usr) && prob(10))
> > +             if (id != src.real_name && in_range(src, usr) && prob(50))
>
> prob() in examine is a bad idea, it'll reward players who spam the shit
> out of the verb.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Baystation12/Baystation12/pull/627/files#r524804
>
Let's avoid the extra parens around `file`.
(see above about limitations) - this is an application that should take less than 1 day, but:

The primary use for this limitation is as a [control](http://en.wikipedia.org/wiki/Scientific_control) in the process, allowing us to observe a developers raw JS and jQuery skills.
This (and `leaflet.zoom.scss`) shouldn't really be here should it? It's a local asset with no upstream vendor... Also shouldn't it be `.css.scss` rather than just `.scss`?
Intended to remove the onus for enforcing the defensive purpose of IPA from the inventor (or worse heir) and make it automatic. This leaves the court to decide if Company or Assignee violated the IPA clauses in Section 2; if they did use patent offensively, that triggers automatic sublicense to cure infringement. Inventors don't have to do anything or be involved.
Our initial idea of curing by making the patent "public domain" doesn't seem viable once a patent is issued, so the sublicense was my next best suggestion.
I tried this out against my dev-gaia email and noticed that we can get here with an object with no `name` property. We should add something like so:

```
    else if (!pair.hasOwnProperty('name')) {
      addrstrings.push(pair.address);
    }
```
The country/region portion...
They are listed in the table in the link. If we add an example of the hex format, then we should add and example of the color name format. Personally, I don't think it's necessary.
Now the values are rendered in custom adapter (EventCursorAdapter)
BUG : this is not a pom => remove <type>pom</type>
Could you explain what you mean by "aToBobj is not applicable to the argument A"? I thought you meant that `aToBobj(someA)` would not work but see the following REPL session (2.9.2):

```
scala> class A
defined class A

scala> case class B(value: String)
defined class B

scala> implicit object aToBobj extends (A => B) {
     | def apply(a: A) = new B("obj")
     | }
defined module aToBobj

scala> aToBobj(new A)
res1: B = B(obj)
```
curious why we grant all ec2 access to any instance?
where is netstatus_get_default_interface and netstatus_get_interface_details documented?
I can't seem to find it here - 
https://bdsc.webapps.blackberry.com/native/reference/com.qnx.doc.bps.lib_ref/com.qnx.doc.bps.lib_ref/topic/netstatus.h_functions_overview.html
We could create a simpler dialog type if you want?
by the way, this can be written as `for key, value in result.items()`
Wait... do you want this to be an OrderedDict, or a namedtuple? You said below that you want the present/expected and for/against stuff to be namedtuples and I already implemented that :)
is "select count(id)" more efficient?