labofsahil/github-event-dataset-2012 huggingface dataset:
hehe. never trust the IE gc.

On Fri, Aug 10, 2012 at 1:33 PM, gonchuki <notifications@github.com> wrote:

> In Specs/1.4client/Element/Element.Style.js:
>
> >
> > +		it('[beforeAll]', function(){
> > +			element = new Element('div');
> > +		});
> > +
> > +		it('should have a non-null margin-left', function(){
> > +			expect(element.getComputedStyle('margin-left')).not.toEqual(null);
> > +		});
> > +
> > +		it('[afterAll]', function(){
> > +			element.destroy();
>
> you are right, should I null it anyways or we trust the gc to go over it?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mootools/mootools-core/pull/2403/files#r1354238>.
>
>
Probably he uses it on textnodes as a generic?
Type match is not enforced, otherwise the "mark all support topics read" time is ignored because TITANIA_ALL_SUPPORT does not match any topic type.
why this shortcut to display help ? Maybe an icon  will make this help more usefull (a keyboard icon in the board ???)
Adding the `:line` value shouldn't be necessary anymore, right?
i think twig for example understands isPublished too. you would write document.published . the member variable would consistently be called published, not isPublished. but i don't have a strong opinion here, we can leave it as is.
this one is intentional, it's supposed to be "memoize" :-)
Check for `username`, not for `User` (`User` is document name, we want to see if error is about username field).
Is is really necessary to use `set` here? Something like:

```
models.BicikeljStation.objects(station_id=station['station_id'], timestamp=station['timestamp']).update(upsert=True, **station)
```

does not work?
And name variables better. This is not XML. This is URL. ;-)

So:

```
horoscope_url = 'http://findyourfate.com/rss/dailyhoroscope-feed.asp?sign=%s' % request_sign[sign]
horoscope_xml = urlopen(horoscope_url).read()
```
Please add assert that `user` is authenticated. See `TwitterCallbackView`.
Why? removing those empty lines? We have them to separate import sections (system modules, django modules, library modules, our modules).
Ouch! My bad on this one. I reverted this back to the original implementation.
This should probably be a sphinx reference, so it points to the correct version of the docs.
another change example to avoid
I'm not sure what you mean. They are always visible.
`.querySelector` seems like a better fit?
If we want to define None as use default and require explicit ``no_checking``, then we should change the meaning now to avoid having to break backwards compatibility.
The version check should be done using the version of Common, not of the ORM, as this does not depend on the ORM
what is the purpose of this? to create stars?
Right.  Well, I think it's okay that we mention at the top that there's a C++ class that is analogous to this, but at the place where Barney commented, I suggest that we change the sentence to say "The HSMShapeData object can be initialized completely empty, or can be returned from the routines that measure object moments and carry out PSF correction."  I agree that mentioning it can be initialized from a CppHSMShapeData doesn't make sense (even though it's true) since we are trying to keep python users from accessing them directly.  Does this sound like a reasonable solution?
I didn't check the accuracy of this approximation (or gamma), but it would be worth adding at least a comment about how accurate it is with the current limits of 0.99..1.01.  And see if it might be worth dropping those limits to 0.999..1.001.  This isn't a time-critical function, so this is only worth doing to avoid the division by zero, not to speed up the calculation by avoiding arctan and such.  So it doesn't hurt to go more accurate than we think we need for these.
The dx=1. seems to be left over from an earlier version.
can you undo this?
apparently not
http://localhost:8080/job/pkg_opro_remote_all/compiler=system,packages=camlmix/9/console
```
. install -m 0755 camlmix /x/pkg_opro_remote_all/compiler_system-packages_camlmix/system/bin/
* install: target `/x/pkg_opro_remote_all/compiler_system-packages_camlmix/system/bin/' is not a directory: No such file or directory
* make: *** [install] Error 1
```
So does this mean exams can't be children of videos, in button mode?
I would feel much happier if you did tests here similar to what I did in the repave script -- are we double sure that we're not connected to a prod database?  Anytime we have a script dropping a database, we'd better be damn sure it's save before actually doing it.   Scary!
I don't stop caring about precision at 1.0 - let's handle some intermediate cases:
if (value < 10) return value.toFixed(2)
My original rounding code in tooltip used logs to generically get 3 significant digits, something like that would work as well.
Missing type casting/validation
Not to be nit picky... but I will anyway :P

`if (options.logger) {` would match the format used in the code.
Could add a link to EPiServer's add-on page (http://www.episerver.com/AddOns or something?)

it's --> its

*Comment:* Installing Wordpress plugins is a thing editors are often afraid to do, so I guess Add-On store might face similar "problems" ;)
this experiment is no longer used right? delete it?
That won't really work, because this commit removes all compatibility with pre-209 versions, not only in the version/verack message but also other fields.
Please make the example code more general, e.g. 
>  -brainsToolsDir /IPLlinux/raid0/homes/jforbes/git/BRAINSStandAlone

should be

>  -brainsToolsDir /BRAINSStandAlone/build/directory
Will this directory (/var/log/rudder/compliance/) be created by the logger in rudder-webapp? If not, it should be added in packaging.
Removed with https://github.com/cwolferh/thor-cli/commit/aad8de781a4aca85d7c5dadd5232a324f2549548
makemake currently assumes there are no comments following #include lines, so actually it would break with any kind of comment. The comments in your sources (regarding for what symbol is the #include is there) are likely to get out of date quickly on any change that adds functionality. Just drop them.
This is my favourite correction :--)
Раз уж зашел разговор о XmlBinder, то у меня там, если я все верно понял не работают только особенности варианта, которые я так понимаю не нужны в задании UserList. Итого вопрос - можно ли тогда просто затереть обработку этих особенностей и замержить в директорий?
(long) 0 == 0L
I think you have left some debugging in here with these two `print` statements!
I guess you accidentally put this here?
good idea. so I'll probably move the code that is here and leave the rest for later.
There should be spaces before the ':'.
please call that `dgemm` as we might want to add support for `sgemm` (single precision floats) in the future as well.
what is the license of these data if any?
I would replace "*that*" by "the ordering of the dataset"
Missing apidocs.
Remainder of rebase...
Can poolName be null? If so, something like:

ModelNode result = context.getResult(); // initializes it to "undefined"
final String poolName = componentType.pooledComponent(component).getPoolName();
if (poolName != null) {
    result.set(poolName);
}
good catch..
I suppose if you'd rather wait=0 could just be considered timing out the reload before it even started.
Yes, it will return NULL.  The setCsvOptions() will check. If it is NULL, the setCsvOptions() will not set @csvEscape.
Change to long name and be careful.
I think you should have an extra:

```python
elif isinstance(value, unicode):
    value = value.encode('utf-8')
```
I think we decided to use following forms instead:

```js
rules.forEach(function(rule) { /*...*/ });
Object.keys(rules).forEach(function(name) { let rule = rules[name]; /*...*/ }) // objects
```
re.sub() didn't learn a "flags=" argument until python2.7, so this throws an exception on python2.6 and earlier. I think that's ok, this tool is only run on special occasions (to update the rules file), but it might be worth a comment up above to make that clear. Maybe use "python2.7" instead of "python" in your Usage example on line 15? (I know, it nudges the line length just over 80 characters, that bugs me too :)
`params.first` is the json object
can you normalize these? (provides a fraction better performance)

normalized form here would be:
(QModelIndex,int,int) (yes, without spaces)

see also: http://marcmutz.wordpress.com/effective-qt/prefer-to-use-normalised-signalslot-signatures/
Good suggestion. Did that, but it will be in next pull request then.
almost but not quite, as disgusting as it sounds, normalization requires you to remove the spaces too

so really, (QModelIndex,int,int)
target user
Is it used after that?
The version field can be deleted (svn related, not needed anymore)
Yeah, good point although it can be also an internal URL and that's the reason I put it as a variable/option. My fault anyway, a bit lazy these days, the best approach would be to generate a dummy java applet within the Metasploit web server and then point to it. Is there any Metasploit library/class I can use for this purpose? Sorry, it's the first time developing a metasploit module.
Should this be != 1?
Missing commas "These constraints are, in typical applications, checked on several places:" and also s/on/in/
I'm not sure that this improvement is general enough.  It is still hardcoded to default to the billboard collection's specific needs.  Perhaps we can move that logic out to the billboard collection?
Let's keep it as is.
なるほど！
ありがとうございます。
Try and catch blocks should have '{' on the same line according to our coding standards (You do this is in a couple of places here.)
Changes default behavior. Is there a particular reason this needs to be set to word wrap.
Typo `When the paging control is enabled on iOS or Mobile Web,`
Again, it is not showing up for me in testing.
From IOS docs
`However, if the user pulls down the banner to reveal the notification center, your app moves to the inactive state just as if an alert-based interruption had occurred.`
No strong feelings either way...sounds good to me.
This PR looks good, though I'd like to clean up these properties by:

    // at the top
    var postRefresh = function() {
        this._refreshSections();
    };

Then in each prop, just set:

    maxRowHeight: {
        post: postRefresh
    }

Because the post function (as well as set/get) are internally apply()'d internally (thanks to mixProps()), you can use "this" in a function at the top.
Yes there is a parity issue on Android with the SATELLITE_TYPE showing roads and names instead of the expected satellite images only seen on the other platforms.
why not nokogiri?
you are still forcing a version instead of allowing a range, meaning you are forcing the user to use this specific version. Thus, this tag does not exist yet: http://packagist.org/packages/doctrine/mongodb-odm-bundle
Add { } please =).
não seria melhor incluir estas definições de pluginManagement no parent?
У нас, кстати, есть ещё `debug.js.coffee`, его тоже можно использовать.
Maybe write about the phantomjs dependency
Since it's a public function, input should be validated.

g_return_if_fails (executable != NULL && *executable != NULL);
g_return_if_fails (title != NULL && *title != NULL);

Unless of course they are allowed to be NULL or empty strings, in this case it's good to make a comment on what you consider valid inputs
Is inserting `assert(translatedLower <= translatedUpper);` better?
`hasLength` returns false for UTF-8 strings
This code is giving me trouble, but it _was_ preventing from compiling other, otherwise valid code.

Your solution seems much better. Thanks.
Well, that may be the intended purpose, but it doesn't compile. I didn't care enough to investigate further.
`getAccountRemovalAllowed` is permitted to throw on a network error "if the authenticator could not honor the request due to a network error". Think about whether we care enough. I'm erring on the side of "we don't care if it fails", but you might disagree.
This is potentially a regression, because now "term" and "scheme" are in a different XMLNS, the default one, instead of the "ATOM" namespace.
Why a package.json? This is not a separate module?
Can you try and parse the version string used in the source file name from `version`?
It's better to only have the version string in a single place imho.

Suggestion:

```python
(vp, vs) = version.split('_')
altver = '%su%s' % (vp.split('.')[1], vs)
sources = ['jdk-%s-linux-x64.tar.gz' % altver]
```
Именно что! Сэкономишь один лукап и один `toLower()`.
Single quotes, here and beneath
И чо, `NotHiddenColumnCount_` не может быть равно нулю?
Can you inline this code into the setScss() method? 

I know it is a bit slower as we have to make the check each time, but it isolates the code that is necessary to cope with the compass bug and allows us to remove it more easily.
This was a leftover from an old version of the code. It is indeed redundant here.
Not sure what you want to do about it ;)
Go ahead; from my earlier reply on the other commit...

"I agree - but would rather not do it in this sprint - maybe open a new JIRA and make it a sub-task of 1819?"
CDH4 is out of beta. CDH4B2 is beta 2 of CDH4. The correct version is 1.1.0-cdh4.0.0. Thanks for the pull request.
right now you can drop any file into the appropriate directory and it will be used as a fixture.
You could use jQuery to filter and extract this information. It looks nicer. :-)
Also the quotes of DRY are wrong must be `
Sorry to bikeshed, but I think `attr` is rather appropriate since we are testing against `attributes`.
普通に Some の値をパターン変数に束縛して渡せばよいのでは。
case Some(x) => mpService.search(x, page, sort) みたいに。
Multi-word triggers are not currently supported. To got the same functionality you can use "days" and in your sub, something like: s/^between//;
todo later: missing doc block
Still need to remove "the" from "Take the advantage". Also, "Demo Account" should likely be lower cased everywhere.
You're assuming X and Y are WGS84 degrees and Z is WGS84 height, but it's not necessarily true.  The actual projection is I guess in the PRJ file with the shapefile.  Supporting all the different projections would require that we use GDAL (an ugly dependency in my opinion), but we can at least be clear about which ones we do support and let the user know if we don't know how to interpret their shapefile.
Yah, I thought that multiple prompt can be implemented with writing an overload version of `.prompt()`.

However, to tell my opinion, I expect `.prompt()` to prompt always.  I would like to `.fill()` options if it's not supplied with command line option.

I mean:

```javascript
program.prompt({
    foo: 'please input foo: ',
    bar: 'please input bar: ',
    baz: 'please input baz: '
}, function(obj) {
    // ...
});

// I expect above code to always prompt `foo`, `bar` and `baz`.
// On the other hand,

program.fill({
    foo: 'please input foo: ',
    bar: 'please input bar: ',
    baz: 'please input baz: '
}, function(obj) {
    // ...
});

// I would like to fill `obj.foo` if `--foo` option isn't given on command line option, so as well with `bar` and `baz`.
// In other words, if `--foo` option is supplied on command line, I don't want to prompt that 'please input foo: '.

```

Does it make sense?
I afraid that it might be enough to have multiple version of `.prompt()`..., but I often want `.fill()` feature.
That is the only way exposed through RDFE. Matthew mentioned we should not really need this operation assuming the runtime does not have bugs, but we decided to leave it in the CLI for situations where we need to provide customers with a workaround. We don't expect folks to use this function in the regular course of working with zumo.
would doing a ```next unless user_identifiers.any?``` make sense here? also could use ```user_identifiers.first```
Use a class directive to have clickable links.
Can we name this to something more understandable, get_next_questiongroup seems to me like a function that will return you the next question group based on your current position in the questionnaire, but you aren't returning a questiongroup, you are rendering a form and returning a HTTPResponse. MAybe we could consider handle_next_questiongroup_form ?
formatting issues here.
formatting is really off here
Come on bro, no credentials.
I'm not 100% sure yet what this should look like to make ActiveSync happier...
the mirror function does not remove files present in $to but not in $from, but who cares? The folder is quite always created
Maybe it could also handle xml, and just serialize the pager.
Hey, that's probably cleaner.  I'll do that.
ok, but don't forget about it
Yes, it should be false by default.
`var gid = $sn.getAttr($text, 'gid');`
change to something like dev
The point of this exercise was to write `boolean` yourself. Therefore using the already defined `boolean` is not allowed.
Mettre le "pas" avant "volontaire".

Sorry...