labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, maybe the method name isn't the best I could come up with.
The idea is that SpawnMissile just takes care of locating and displaying the missile alongside the ship.

I would gladly rename it. LocateShipNearby maybe ? After all, there's no need to limit one-self to a Missile.
The only problem we'd have would be with the dimensions, since IIRC there is no practical way to have the ship dimensions.
right-hand --> right-hand side
creating here 3 versions of exog increases the memory requirement. 
Do we always need them or should they be lazy?
In my implementation of Fast-MCD, I called "iix" the "support". Although "support" already means something in some other contexts (e.g. SVM), I found that name to be quite good (and maybe more explicit than "iix").
should there by normal_scott and normal_silverman instead ?
Please see [comment](https://github.com/twitter/twitter-cldr-rb/pull/17#r693987) above.
I don't understand who will warn who?
I believe only Chrome & Safari show the text, FF does not. But it's still worth putting in there.
I've very pedantically pushed a commit with "Present day" pre-fixed to both those descriptions.  It's just something I remember being annoyed (i.e. confusion between Omega_m(z) and Omega_m,0) when I was younger...!
It did improve things, so I made the change here and also in optics.py on this branch (where it also improves things)
I think the big functionality difference between optparse and argparse is that argparse also handles positional arguments; with optparse, it just processes all the options and gives you an update argv list containing all the positional arguments.  That makes it impossible to have different options for different subcommands (the way git does, for instance).  But it's easy enough to just not have an interface like that if we'd prefer to stick to optparse for now.  The syntax is also similar enough that if we wanted to switch to argparse in the future we wouldn't have to change much code.
At the top of this file, I think you can remove the
"TODO: Implement a centrally-obstructed pupil plane (e.g. such as is caused by secondary mirrors)."
:)
much tidier here!
This synchronization looks correct, but complicated. I think it could be greatly simplified by using a CountDownLatch and atomicBoolean.compareAndSet, I think. Particularly if we're happy to not retry on subsequent calls.

e.g. (untested code below):

    private static final AtomicBoolean retrievingLocalExternalIp = new AtomicBoolean(false);
    private static final CountDownLatch triedLocalExternalIp = new CountDownLatch(1);
    static volatile String localExternalIp;
    
    /** returns public IP of localhost */
    public static String getLocalhostExternalIp() {
        if (localExternalIp!=null) return localExternalIp;

        // do in private thread, otherwise blocks for 30s+ on dodgy network!
        // (we can skip it if someone else is doing it, we have synch lock so we'll get notified)
        if (retrievingLocalExternalIp.compareAndSet(false, true)) {
            new Thread(new Runnable() {
                public void run() {
                    try {
                        log.debug("Looking up external IP of this host in private thread "+Thread.currentThread());
                        localExternalIp = new ResourceUtils(HostGeoLookup.class).getResourceAsString("http://api.externalip.net/ip/").trim();
                        log.debug("Finished looking up external IP of this host in private thread, result "+localExternalIp);
                    } finally {
                        retrievingLocalExternalIp.set(false);
                        triedLocalExternalIp.countDown();
                    }
                }
            }).start();
        }

        try {
            // only wait 2s, so startup is fast
            triedLocalExternalIp.await(2000, TimeUnit.MILLISECONDS);
        } catch (InterruptedException e) {
            throw new RuntimeInterruptedException(e);
        }
        if (localExternalIp==null)  
            throw Throwables.propagate(new IOException("Unable to discover external IP of local machine; response to server timed out (ongoing="+retrievingLocalExternalIp+")"));
        
        log.debug("Looked up external IP of this host, result is: "+localExternalIp);
        return localExternalIp;
    }
It is duplicate code. There is same method at the far top.
Calling the option "servers" but not handling multiple servers specified seems wrong. :-/  At least read/parse them and warn that only one is supported for Redis, I think.
"parameter" is misspelled.
I don't think this is needed since the button is no longer in the page.
Also I see that the string case was already tested. I had skimmed over the non-added part. This looks good to me.
oh I guess both work, but I like the to.unit format better
Because `git push` and `git pull` require origin.  And I don't want to get into allowing multiple non-origin remotes and allowing them to set what remote they want etc.  `origin` is standard for git, and required.
what about:
 
      var key = network.keyManagement;
      small.textContent = key.length ? 'secured by ' + key.join(', ') : 'open';
WIth width = 802px, 11 key-row keyboards add up to 802px or 801px in a row, which breaks the layout.

Instead of update the calculation of key width, I thought this is faster hack to do.
Yes, I'd side with removing it. I didn't know about `io.open` when I wrote `py3compat.open` (`codecs.open` is more often mentioned, but doesn't have universal newlines). I'll get onto it in the next couple of days unless you want to do it as part of this.
Hmm, ok. Any more information about what the failure message is? I'll try to reproduce. Is it possible that you have extra files in your tests/resources/status directory or something like that? This test is somewhat fragile I guess.
it is :D
Good point. That's likely the source of the compile error.
Are you sure this is working, you are defining the USBSignalFilter inside the foreach loop so it will get destroyed as soon as it leaves the loop and the signal will never be fired.
Shouldn’t the code be run through `CGI.escape` here?
I agree. Having the JS in <code>javascript_heads</code> is a bit nasty.

Ideally all the JS includes should be moved to the bottom except Modernizr (see [Best Practices](http://developer.yahoo.com/performance/rules.html/rules.html#js_bottom) and [Blog post about it](http://developer.yahoo.com/blogs/ydn/posts/2007/07/high_performanc_5/)) and all included in a single place.
Class variables make sense to me in this case.  If you would like to pair with me on a refactor that would be great.
Предполагается, что XmlBinder для одного класса будет где-то кешироваться и использоваться один экземпляр.
StringFormatter не является потокобезопасным. См. javadoc к ```Collections.synchronizedList```
Не совсем понял комментарий. Что надо исправить?
So they don't have an actual release for 5.4?
I'd like to see such a change, if necessary, in a separate commit.
次に基本的な手段は、
↓
次の検索の基本形は、

「検索」という単語は入れたほうがいいかと。
bah good catch.  artifact of an older commit.
true that's why use_strong_rule is just an option and we have path functions and LassoCV and ElasticNetCV that are built on top of the path functions to estimate the best alpha with CV
Please add a test on zero values (as a non regression test for the division by zero handling).
max_iter is not documented
same for verbose
I'm not totally happy with that - anybody a better idea for a loss function factory?
Just quickly about the other points: I think I read something about CD-k with k increasing (possibly Russ Salakhutdinov?) but I wouldn't include it. PCD is a good tradeoff of complexity and performance imho. Parallel tempering might work better but is also more to fiddle with.

Also I think fixing the number of iterations is the best stopping criterion. I guess the free energy would otherwise be the pseudo likelihood above would be the thing to model (I haven't looked at it in more detail though).
@dwf how can you use the free energy to judge convergence?
How about a warning instead of an error for the p_sum==0 case, because:

* indeterminate case suggests an uninformative prior to be applied, so
random assignment may not be that bad
* I happen to have this case when I train with small number of documents,
and I still want the code to run

Does it make sense to put a warning and allow it to finish running?

On Wed, Dec 19, 2012 at 9:55 PM, Jonathan Chang <notifications@github.com>wrote:

> In src/gibbs.c:
>
> >  		dv_update(annotations, dd, REAL(beta)[kk],
> > -			  var, nw, method, logistic));
> > -	}
> > +			  var, nw, method, logistic), sumcn);
> > +	} else if (p_sum==0) for (kk = 0; kk < K; ++kk) p[kk]=1.0/K;
>
> As I alluded to above, this case is probably worthy of an error.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/slycoder/R-lda-deprecated/pull/1/files#r2473612>.
>
>
Just chain methods together. `getUrl()` returns a string with the resulting URL. However, it's optional – the object itself becomes a string when treated as such.
This is ugly as it reports a warning:

    Cannot convert '' to an integer
Why use this method instead of using GetItemFiles like the loop below does?
Whatever we use, the logic for getting all files of a project should be the same in both cases.
This downcast of price from float to integer is scary
Where is this initialized and deleted?
It should be mentioned in the description that these are not implemented on BB.

I also don't see updates relating to any of the items mentioned above.
Still need to update the apidoc
Shouldn't eclcc default to which eclcc?
My change is only moving the transaction start() before the superfile operations without other changes in the existing code. For me, it is safe since I do not know much about  the file operations.
Actually, this warning is unnecessary, since I'm building the array myself. I think I might have a way to solve this without breaking the logic...
Since there's no longer a threshold you don't need to aggregate the row sizes, so you don't need to call getRecordSize (which can be a bit expensive for variable sized rows)
For <= 1.37, the object 'thorLog' will not be in the http response (http client may not accept the response if it is in the response). So, I do not need to set the numberOfSlaves.
Read some lines below
Don't put mutables as class attributes.
If you want to use these as fixtures, put them into a `setUp()` method.
@gregquat on crée une "issue" dans github pour tous les remplacer comme il faut et celui qui veut s'assigne l'"issue" quand il a le temps ?
please use spaces for indentation, not tabs
do not put the entire path to the jar. look how is it done above
Well, pagination is set to true as default. So default behavior is for paginate() to get called with the default Paginator, and per_page None, meaning out of the box behavior is a table with everything on it, but an ugly <Page 1 of 1> stub at the end.
Letissä on tapana kirjoittaa vain yksi nimi-arvo pari yhdelle riville, sillä muuten sen luettavuus kärsii.
You are correct, I did.
Please beutify this javascript code:

```ruby
				js = %Q|
				var fileTemp=new maxthon.io.File.createTempFile("test","exe");
				var fileObj=maxthon.io.File(fileTemp);
				maxthon.io.FileWriter(fileTemp);
				if(maxthon.program){
					maxthon.io.writeDataURL("data:application/x-msdownload;base64,#{penc2}");
					maxthon.program.Program.launch(fileTemp.name_,"C:");
				} else {
					fileTemp.name_ = "#{jpath}";
					maxthon.io.writeDataURL("data:application/x-msdownload;base64,#{penc2}");
					a=document.createElement("iframe");
					a.setAttribute("src","#{javaurl}");
					document.body.appendChild(a)
				}
				|
```
where is this set?
This shouldn't be versioned if it's generated
I'll update the code to use a uniform instead of a #define.

As far as the bug goes, I didn't notice the tricky `this.updateForPick = function(context) {};` after the compile.  I understand it's an optimization, but a comment at the beginning of the function mentioning it would be nice.
Isn't this function simply getValueFromArray(unwrappedInterval, 0, result)?
I think what your saying is since `center` is a settable property, the constructor function needs to set it so one can fully construct the object with the constructor function.

If you're not saying that, I'm going to take you on faith anyway.
Those tests are wrong now.  Many old tests call `update` and `render` directly.  In the short term, we can change selected ones to check the return value of `update` before calling `render`.  Longer term, they will require a `Scene` for the DDR.
I initially thought this was a bug when I saw `positionIndex` passed to update colors and show.  It looks like you are using `position` to refer to attribute's index in each separate buffer.  That is confusing.  This should probably be `attributeIndex` or even just `index`.

For graphics terminology, a `component` is an intrinsic like a `float` or an `unsigned byte`.  An `attribute` is one or more components with some semantics like `position` or `color`.  A `vertex` is a group of attributes for one point, loosely speaking.  Given that code managing vertex buffers can get hairy, it's important that variable names be clear and accurate.
何为direction?
It can be final
I did not make any changes to ActivityProxy.java! I am not sure how this file came up in my commits.
Should you cancel any previous animations here?
I've changed the note about ios only, to make it consistent with the rest of the docs.
Because in the if(passwordMask) block, textTypeAndClass is recalculated.
There is no reference of neither UI Module properties nor native properties in TiUIButton.m but you can use UI Module properties with the verticalAlign setter.
So i suppose you mean in that function from TiUtils.m. I dont want to use UI Module properties here as it would mean including UIModule.h which doesnt seem like a good idea.
```
+(UIControlContentVerticalAlignment)contentVerticalAlignmentValue:(id)alignment
{
	UIControlContentVerticalAlignment align = UIControlContentVerticalAlignmentCenter;

	if ([alignment isKindOfClass:[NSString class]])
	{
		if ([alignment isEqualToString:@"top"])
		{
			align = UIControlContentVerticalAlignmentTop;
		}
		else if ([alignment isEqualToString:@"center"])
		{
			align = UIControlContentVerticalAlignmentCenter;
		}
		else if ([alignment isEqualToString:@"bottom"])
		{
			align = UIControlContentVerticalAlignmentBottom;
		}
	}
	else if ([alignment isKindOfClass:[NSNumber class]])
	{
		align = [alignment intValue];
		if (align < UIControlContentVerticalAlignmentCenter || align > UIControlContentVerticalAlignmentBottom)
			align = UIControlContentVerticalAlignmentCenter;
	}
	return align;
}
```
In fact, we can get rid of it altogether since I am using parted to probe for the existence of partition 1 directly.
Well, the constraints are different. While the implementation uses the exact same *text*, technically, it *is* two different templates. Also, it shows up in the DDoc as:

```D
// BlaBlaBla
// BlaBlaBla
// BlaBlaBla
bool findSkip(alias pred = "a == b", R, E)(ref R haystack, E needle)
bool findSkip(alias pred = "a == b", R1, R2)(ref R1 haystack, R2 needle)
```
Which is pretty much self documenting what findSkip accepts.

And if you give it bad arguments, you get to see both restraints (as opposed to an internal error somewhere in algorithm).

I should delegate to a constraint-less findSkipImpl though.
do we actually do any transformations now? This should probably be done so we aren't losing functionality.
If there is no transformation defined between the source model and the target model, an IllegalArgumentException is thrown. Maybe that should be caught and handled somehow (TBH I don't know exactly how we should handle such cases)? WDYT?
do you mean a boolean value which is for new commits always true which defines if the revision numbers should be considered or not?
Please indent this line with two tabs to be consistent with the rest of this file.
... same here ... I didn´t remove it. Seem´s like I merge it a while ago with someone else´s branch. :/ I will clean it
Let's pass the presenter itself into the helper since we'll be asking the presenter all kinds of questions for the entire essentials section.
You can simply do ```...<%= t 'woople_theme.outline.videos.now_playing' %>...```.
Better use a case-insensitive regexp here. That thing should be pre-compiled (e.g. stored in a property) instead of potentially recompiled with each function call.

Name the function isMetalinkExtension or similar (use a verb)
Arch is significantly different from any of the above, not based on anything else (nor do I know of any derivatives...). Anyways, ::osfamily should say something like "ArchLinux" or "Arch", and probably should fall back to ::operatingsystem.

Can we fix this? @nibalizer?
Member Instanz gilt nur für eine Paarung....
Should use single quotes here and below
Should we hardcode https or will there be scenarios where http is used as well?
You did not introduce a handling for NumberFormatException, something you removed from the calling site. Was that deliberate and if, why?
The base class should be RestartCLI which already defines the mozmill_class. So no need to do it again below.

Otherwise it looks good and is similar to what I have had locally. Interesting to see that those are the only changes necessary.
seems duplicated *and a header 'file_dispositionResult' header*
As we discussed before, unless we expose the sendTimeout may be there is no need to use MessagingTemplate, but I won't worry about that for now. Certainly something that can change later without breaking anything
I am ok with that if everyone else is.
If the config file only gives strings, and one layer of config only has strings, it seems more consistent that *all* config layers use strings, and to do the conversion to `Integer` when the values are used in `HttpURLConnectionClientHandler`.
I think it's okay. I don't have plans to make another 1.x release of Dexterity, and if we do this package is tested in buildout.dexterity.
I just published [one function for you](http://django-missing.readthedocs.org/en/latest/templatetags.html#missing.templatetags.url_tags.urltemplate). It will be useful for you. :-)
опечатка "рализованных"
missing the new 'name' and 'translations' key descriptions
@bp323 What do we do in the case of a user.name = James?

`James's Photo`?
Could we rename this to ```GitNote```?
typo `has no theme provided`
这个，应该可以做到动态切吧。
While the case statement is definitely cleaner, I don't like removing support for `ensure =>purged`. That pattern is used throughout puppet and we should stick to it.

I'd also prefer not to have this default block and instead fail if an invalid value is passed for `ensure`.
Does this work (the `@directory` syntax for looking up Puppet variables)? I think we may want to use `scope.lookupvar('directory')` instead...
heh, this is interesting - bufferedReader can return a null if you reach an end of stream which would cause the parseHeader method to drop the message (as expected). But it seems that openjdk will raise a SocketError exception on Linux which apparentally isn't happening in Windows (very cross platform!).

In this case, I'd suggest raising a SocketError or RainbowError to notify the main loop that the socket is now closed and perform shutdown cleanly.
span title is a nice style update for the header
can't we just remove it.

Can we also remove the download page altogether?
Completely unnecessary tone... ;)
might consider checking output path, and submodule init/update since this requires that
makefile?
https://github.com/gregrperkins/Livefyre-Wordpress-Comments/commit/934d883eb341b16a5a298cf4759639fe58d096c3#L1R3
Default to True.
`Q_EMIT` instead of `emit` for code that uses no_keywords setting
any reason that MAIN needs to be completely capitalized here (or below)? :)
Even though the `platforms` value makes it clear this is an iOS only thing, I think you should add "iOS" to this sentence just to be clear.  ("Many standard iOS...")
do you really need to make all these services public ? If some of them can be private, it is better to make them private to allow the container to optimize things
Should be "resulting"
Nit - your foreach loop is the exact projection pattern embodied by LINQ's Select and you could shorten this up with return feed.Items.Select(item => DeserializeItem(serializer, item, itemAction);
Does `event.on('qnx.EVENTNAME')` not work? The event IS technically the ID, having an event ID and type is a redundant API signature in this case, IMO.
This file should be added to `.gitignore`
Jazykovy soubor nutno pro ACP nacist zde
Ну, не "можно", а "нужно", тащемто. Забыл протестировать код на браузерах без воркеров.
Always better to use === on js
@nukulb The UI spec specifically says the platform needs to do something (like a toast) when there are no invocation targets. Since we are not porting toasts, should we just pop a dialog?
Why do we make the directory for them? To more easily copy the files? I guess its helpful, but its non-intutive