labofsahil/github-event-dataset-2012 huggingface dataset:
there are more examples that have this stylesheet, should remove from there as well:


bart-van-den-eijndens-macbook-pro:openlayers bartvde$ grep -lr "../theme/default/google.css" examples
examples/all-overlays-google.html
examples/canvas-inspector.html
examples/filter-strategy.html
examples/google-v3-alloverlays.html
examples/google.html
examples/kml-pointtrack.html
examples/kml-track.html
examples/osm-google.html
examples/wfs-filter.html
@elemoine, we have a use case to display time animated data and while the tile manager img element caching helps, it would be even better if we could buffer the tiles and then just use the memory cached img element when playing the map or jumping to a location.

There are a number of benefits to this patch without the time animation use case. Panning and zooming are far faster and the performance is much better.
Why are we dropping these pre/post conditions?
Here it can be interesting to specify what is the license you would like to use
I can't find in this method where the `widgetId` gets defaulted to a random id. Does that happen somewhere outside of this method?
Ahh good catch, I meant to refactor this somewhere else at one point.
I swear I removed this when I found it.
Using the DOMDocument class for XMLParsing. See http://php.net/manual/en/class.domdocument.php
Now that this function isn't called with the serial lock held, what prevents a race here?
There's one tab to much
I use Traversable[T] instead of Seq[T], because it is more abstract. Traversable is just any collection. Seq needs the selection to specify an order. So for example Map doesn't inherit from Seq, but from Traversable. Using Traversable and not Seq allows us to accept any kind of collection of types here.
``$GLOBALS`` does not seem to conform to the naming convention (``^[a-z][a-zA-Z0-9]*$``). Please also check line 30.
- Are you sure the method ``lock`` can be called on ``$this->store`` (of type ``object<Symfony\Component...l\HttpCache\Store>|null``)?
- Are you sure ``$this->request`` has the correct type as ``object<Symfony\Component...oundation\Request>|null`` is not a sub-type of ``object<Symfony\Component\HttpFoundation\Request>``? I also found this on lines 68, and 73.
file allready contains folder
i guess i can do that. i was hoping it was the same branch, so the new commits would show up on the same PR
the fix is for the prepare subaction, which uses the OutOfKitchenError. So am subclassing that class. Also in my code, i can differentiate between errors from knife-solo and otherwise
Seria boa ideia trocar onde diz semana por data, visto que este método é mais geral.
oh, you mean the timedelta, don't you? Yeah, that should probably be an argument.
Oops, thought this was straight-forward but now `:annotations` needs to be supported with `Report#to_annotations`, etc. :(
I might point out that you almost always want ResultsReaderXml.
Why not:

```
objectify.parse(urllib2.urlopen(BICIKELJ_STATIONS_URL))
```

You don't convert to string and back in this way?
No no no. Please use verbose (long, spelled out) variable and function names. Also you can simply define methods on the class, no need for global functions?
ul/li list would be better.
Probably better place for all this would be in `hydrate`, not `ob_create`:

http://django-tastypie.readthedocs.org/en/latest/resources.html#hydrate
IMHO it's more complicated with the save_sources flag, I would use Pkg::SourceSaveAll() directly here, one extra save in some cases does not mind. (USB installation sources are really rare I guess.)
Again, I want to see how this performs. Should be benched against a high number of elements
Ah sure I will add that... Mhh before I answer the second question I'm going to write some tests ;-)
why?????
We might want to print a warning here so that we know if any gateways get completely missed.
done - committed - pushed
yes. I mean: if it's commented you likely want to remove it before we merge this pull?
I have no idea why the schema is different for you guys. Maybe Rails changed this recently?
You just renamed `resolve` into `fulfillment`, is that it? This doesn't improve (or degrade) anything.

Why not something more descriptive, such as `asResolved`?
Wonder if there's a reason for the gradient? Also prefixed "90deg" is unprefixed "0deg".
I agree, however, the reason I pulled this file in is that I made a few minor changes (we use strict compiler settings, and fix for a segfault on the first request, when all memcached backends are down). 
If these changes could be merged with the original file in mod_pagespeed, we could remove this file.
the return value may be NGX_ERROR.
The style used in the rest of this codebase for multi-line blocks is to use `do`...`end` instead of curly braces.
For constant strings use `'`.
Mutator seems wrong?
What if it isn't a MimeMessage?
думаешь стоит оставить hw=*? я сам сначала не понял, что это про дорогу...
Their minified ones?
This was changed.
Both good points, which I just fixed.  (Will push a few commits including that one very soon.)
Why set provider=spec? It will either be ignored or the value reset if the if-else below, so setting it here is just confusing.
I agree with @mfournier . Also, as far as we're concerned, we should try to migrate to puppetdb everywhere, that would be more useful.
Sorry, I don't understand this.  The test says that it is starting w/o any subscriptions, the above line adds 2 subscriptions yet the action is to only add the 1st.  What am I missing?
This URL has been changed : `layout=box_count` -> `layout=button_count`
むむ。どの URI に何のリクエストを送った時に routing error になりますか？
Shouldn't this be JPA2?
Ahh!  It's the d/2 being squared that creates the 1/4.  Thanks!
That's a smart thing to do :) I'll give it a try. Maybe this is a bit too smart for would-be plugin developers though?
Added my own props
This AdapterFactory is currently inside of the pipeline package, therefore it should not do things with the datamanager
Looks like there's such a beast in ```ActionView::Helpers::NumberHelper```

```
(using Homerun) #> include ActionView::Helpers::NumberHelper
=> Object
(using Homerun) #> number_with_delimiter(10000000)
=> "10,000,000"
```

That makes more sense to me.
Выше проверяются именно такие случаи, и на этом входе программа выдает 'Incorrect input'
Can this inherit from ExpressionException?
Learn something new every day, and it's a quick fix. :)
Comment nit: s/paypall/PayPal
I've styled things that are not implement in red. Let me know what you think of this.
This is just fine off course. In general I would recommend using the `str.format()` call in Python. Also applies lo line 51
I hoping @vdboor might have some ideas on how to make this work. Since this is only useful for development, I've disabled it.
This page should not be removed.
On Tue, Nov 06, 2012 at 05:54:25AM -0800, Olivier Grisel wrote:
> I think adding y=None to all unsupervised model with a transform method was
> required when used in the early stages of a pipeline.

Good point! That's the reason why.

Anyhow, I think that it is good policy to be as flexible as possible, and
thus to have both 'y=None' in the fit of unsupervised methods, and
GridSearchCV work in the absence of y.
n_samples, n_features = X.shape
On Thu, Nov 15, 2012 at 12:02:37PM -0800, Andreas Mueller wrote:
> I think it would be good do have some sort of default. Usually there are no
> mandatory __init__ parameters.

+1, or 'None', and a heuristic to guess.
Hmm, I tried first with dir=rtl and it ditn't seem to work. Now I realize I might have had a wrong selector (on #main instead of body). will fix.
There is no path called jboss.domain.temp.dir on the server hence this change to the config to avoid missing dependency
Despite these calls, clicking #openButton while the popup is still open causes it to open and close immediately. In other words, click on the first button to open it, then click on this button, and it'll go through to show and hide animations and end up not visible.
No. The spec for coff is very specific - 4 bytes for the time. Using time_t makes it variable.
You're correct.  I don't know how to achieve the goal of streaming input/output and saving state.
I know this is not all your code, but could you make the change to use the prefix underscore instead of the suffix underscore in TiUIObject class please?
I'd actually prefer that it crashes instead of returning undefined. Hiding NYI (meaningless without context) in the description isn't good enough for production but for now I guess it'll be fine.
Remove 1st logging.
Are all the properties supported for the change event?  Somehow i doubt it, so it should be specified in the doc if properties are not supported.
Should consider dict + dataset too.

I think I can probably assume/assert that left is a dict though.
Should also check the new new whole record flag, and not use if set.
Is this right? Now, the text is butted right up against the edge of the box. Perhaps just reduce the padding, rather than eliminate it?
For safety the plugin always spells out the long form of every option (`--branch`).
Let's wait merging this untill https://github.com/xavi-/node-copy-paste merged our pull request and pushed this to npm so we can just replace this line with

```
"copy-paste": "latest"
```
Somewhat, but let me see how hard it turns out to be in reality.
The extra spaces will be removed in the `trim` call below. But this could be changed to use `formatteddelimiter()` so that the code matches.
I'm +1'ing that.
Here in composer/master: https://github.com/composer/composer/blob/master/tests/Composer/Test/Repository/VcsRepositoryTest.php#L13
Can also be a View\Variables object I assume?
I think you can remove this line... it not really matter and it is no needed implementation detail IMO.
You only want to produce invalid/empty content, no need to check method parameter then.
we could use linkToInstance or linkToMethod(on(Mycontroller.class).foo(1) ), respectively. The mechanism I am using here needs Object since the recorded method can return any object. So I cannot make the signature any more specific than it is. 

Best regards, 
Dietrich
-- 
Diese Nachricht wurde von meinem Android Mobiltelefon mit 1&1 Mail gesendet.



Ricardo Gladwell <notifications@github.com> schrieb:

In src/main/java/org/springframework/hateoas/mvc/ControllerLinkBuilder.java:

> +	 * personResource.addLink(address); > +	 * </pre> > +	 * > +	 * The <code>linkTo</code> method above gives us a > +	 * <code>Link</link> to the person's address, which we can add to the personResource. Note that the path > +	 * variable <code>{personId}</code> will be expanded to its actual value: > +	 * > +	 * <pre> > +	 * http://www.example.com/people/42/address > +	 * </pre> > +	 * > +	 * @param method representation of a method on the target controller, created by {@link #methodOn(Class)}. > +	 * @return link builder which expects you to set a rel, e.g. using {@link #withRel(String)}. > +	 * @see #methodOn(Class) > +	 */ > +	public static ControllerLinkBuilder linkTo(Object method) { 

This Object type method parameter might be ambiguous: what if in future we wanted to build links from instances of Controllers instead of their class (for example, ControllerLinkBuilder.linkTo(this))? How would the API be able to distinguish between an Object-as-method or a Object-as-instance?

—
Reply to this email directly or view it on GitHub.
This Object type method parameter might be ambiguous: what if in future we wanted to build links from instances of Controllers instead of their class (for example, ControllerLinkBuilder.linkTo(this))? How would the API be able to distinguish between an Object-as-method or a Object-as-instance?
Tämähän menisi myös näin:

~~~clojure
(cond (== 1 size) x
     (== 3 size) (+ x z)
     :else nil)
~~~
use the methods from Post::File when possible please
Oh, a Sublime crash... That's sad indeed. Now I see the problem, and this looks like a nice way to handle it.

By the way what about the scope of patch_source? Do I need to unpatch it if the user presses undo, or its scope only covers the immediately following command?
what about `[base_description, role_description].compact.join(' ')` (unless one of them could be an empty string...)?
maybe this could go into a partial?
Indent two spaces.
I'm happy to see a DOC_TBA get replaced with doc, but make sure to add param and return statements as well, maybe throw in an example, so that the doc is complete.  On the other hand, I'm not even sure why this function exists, especially since it just passes values down to the TextureAtlas, I vote with remove this method all together.  What do you think @pjcozzi ?
I'm not sure about the name of this function (perhaps I named the original version?)  This is really a side-of-plane test, right?  Its result is above, below, or both.  Do you think these semantics are better?  What is most commonly used?
Definitely needs an `@example` or two.
I screwed up here, fixing soon.
I've been reluctant to add api-specific tokens to scribe, this is falls out of the scope of the library.
On 10/15/2012 7:53 AM, Greg Thornton wrote:
>
> In lib/cached_shell_redirector.rb:
>
> > @@ -104,7 +104,7 @@ def close_write
> >      # Ruby 1.8 define_method doesn't work with blocks.
> >      def method_missing(my_method, *args, &block)
> >  	# Only handle IO methods!
> > -	if IO.instance_methods.index(my_method.to_s).nil?
> > +	if IO.instance_methods.index(my_method.to_sym).nil?
>
> It could be related to Ruby version differences, but 
> |IO.instance_methods| returns an array of symbols for me, not strings.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/kubitron/redmine_git_hosting/pull/80/files#r1842731>.
>
Hm... My version of ruby returns strings.

Which version do you have?

--KUBI--

-- 
Professor John Kubiatowicz
673 Soda Hall
Berkeley, CA 94720
Could you please use `@sendfile` as per http://docs.puppetlabs.com/guides/templating.html#referencing-variables please? Thanks!
setDocumentElement is not needed?
I'm not aware of any official policy. I generally add my name to anything for which I've made any contributions which seem significant enough (though that's obviously vague and subjective). Some people do the same, while others almost always forget to add their names (e.g. I think that I'm the one who added Kenji's name to std.conv in spite of the fact that he's done a ton of work on it). In general, I'd say that you should add your name when you feel like it's a large enough contribution that it makes sense to have your name on it. If anyone disagrees with you putting your name on a module, they can say so. You've added several functions, so I see no problem with you adding your name to the list. And if they're unoriginal enough that someone can argue that they're not really copyrightable, then oh well. It's Boost, so the license is already about as unrestrictive as a license gets anyway, so I see no reason to get super picky about it.
s/convertable/convertible/
Similar to my previous comment, please give these more descriptive names. Using i for a counter in a for loop is one thing. Using a bunch of different single variable names like this is another. It makes the code much harder to understand. They should be something more like

    immutable strLen = s.length;
    size_t wordStart = 0;
    size_t currIndex = 0;
    size_t lookAheadIndex = 0;
Can we remove the comments?
add a note that warnings are now correctly emitted? :)
Nit: For forms and tabs, we moved this inline.  Saves a file.
Actually two words — "should I back off? My backoff is set to 10", just like "can I log in? My login credentials are".
FYI: this is being addressed in a [separate pull request](https://github.com/ps3mediaserver/ps3mediaserver/pull/85).
Doesn't jQuery let you do things like `$('.drug').trigger(......)`?

This will enforce that `drawGraph` occurs iff `reload-drug` is triggered
change `@link` to be `https://github.com/windowsazure/azure-sdk-for-php`
Regressed.
Yes as long as they're using the new params then the behavior should be the same.  Although, "open_postgres_port" really has no effect when by itself. It really needs the listeners set, too.
So you rely on the "read" flag in order to make sure that  the write skew check  is only performed on the  entries that were read both read and written to.
This logic doesn't seem to be correct in the general case:
Object r = cache.put(k,v);
//do something with old read value.
now for this example the old value of k is read implicitly. This situation doesn't seem to be covered by the code, i.e. WSC won't be performed in this case and it should be. If I'm wrong ignore the folowing :-)

IMO this is bit more complex thing to solve, and should be thought of for the general situation as well, not only for the TO. I'd rather drop this functionality for now and I'll add it back in ISPN-1841, together with some heavy unit testing.
Are all arguments required?
Пробелы вокруг угловых скобочек тут лишние, можно все поубирать.
`result.num_cases` isn't relevant for this test
Oleg, I believe we agreed the existing "should delete" and "mark read" are orthogonal to transaction support. In other words if they set should delete the message won't be available any more in the transaction synchronization.

Did you change your mind?
that's why I also mentioned: System.identityHashCode(session)
We can if we have the need outside from where it is used now. Do you have any use case?
This direct link style was explicitly preferred by the content editors over the creation and management of fwlinks. The links point into specific topics in the reference content directly, which are not likely to be migrated into the WindowsAzure dev portal.
There's 1 reason... styling. Makes it more coherent with other languages and modules.
I was getting an matching error and it was making the puppet run fail. I think I may have an old version of stdlib? Perhaps we could put separate templates for each version instead? That would also give more configuration options down the road.
Yes, by the way, the way the plugin works depend on TT : it uses coderefs in the tokens hash, for instance Template::Tiny which is the default one in Dancer 2 does not support that.

Maybe the plugin wants to document that?
Could you instead refactor it so that the logger callable COMES from the service locator (instead of having SL in the logger)?
Change to _Jenkins_ Port Allocator Plugin?
Minor little thing...

This might be better as an instance method on Types.  e.g. `factes[:types].imminence_slug` (or a better method name that I can't think of right now.)
Can you add the phpdoc ?
@fabienrohrer 
This is the sign mistake I spoke to you this afternoon.
Should we put some decepticon stuff in here as well?
I didn't realize the client now ended this URL with "tokens"... that's kind of odd. Any change we can change it?
I feel like the target shouldn't be dependent on the passed in data, but instead on the parent it is nested under.  I think it might be worth changing target here to be -1 or something nonsensical and ensure it is still 2 below as expected.
This will not work to check the current difficulty level
For consistency reasons I think it should be on the right. In all other cases the sidebar is on the right, its a bad user experience to move common elements around.
Talk to Pete. I don't know how CDI can do interception on static methods.
Each iteration computes a weighted square 2 norm of Q[i, :] with the dimensions weighted by the vector v_prime. E.g.

    diag[i] := \sum_{j=1}^n v_prime_j Q_{i,j}^2

It could be rewritten as an unweighted square 2 norm since the elements of v_prime are non-negative

    np.dot((v_prime ** 0.5) * Q[i, :], (v_prime ** 0.5) * Q[i, :])

How would you compute it without the for loop?
I used htmlentities because it is sufficient for the JS function name and seemed also safe enough.
However, after analysis, I could not confirm the actual threat, so I think you can close this PR.
sure ... I forgot for that ...
To be consistent with other event emitters this event should just be called `error`.
cambiale el orden a inicio y fin.
tipo DateTimeField. es fecha y hora
podrías agregar algun test, para variar :P
Sorry, will fix. I only tested in 4.3.
на итераторы пока забил или вообще забил?
неплохо бы было бы хотя бы сделать всякие insert и erase.
Так как хочется уметь делать групповую операцию не плодя много версий, которые сразу удаляться
Definitely does not *create* the instance.  Just retrieves.
Yup, a blunder... thanks.
I never knew about toContain, nice find
you need a find a better place to do this, this was ok if you were waiting for a failed request but since you are not, this is the wrong place for it. 
@jeffheifetz @rcruz @rwmtse  might be able advise a better place to put this
I guess it doesn't matter, since they will exist, but in this pull request they're dirty code
Move this logic to a unit test on UserProfileService and mock the userManager to have test-defined users searched for.
could you do the same one with assetic less version updating https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/blob/master/Resources/public/css/all.css
This was why your redrawCheckbox test was failing, because you were just calling the function and it was setting the value of redrawOnMove to false implicitly. I've changed the call but achieved the same behavior, and now your event handler on the checkbox can work as intended.