labofsahil/github-event-dataset-2012 huggingface dataset:
Good enough :tm: :) works for me. 

Sent from my digital shackles. 

On Jun 10, 2012, at 10:42 AM, Tim Simpson<reply@reply.github.com> wrote:

>> 
>>     @property
>> -    def links(self):
>> -        return self.server.links
>> +    def nova_client(self):
>> +        if not self._nova_client:
>> +            self._nova_client = create_nova_client(self.context)
>> +        return self._nova_client
>> +
>> +    def update_db(self, **values):
>> +        row = DBInstance.find_by(id=self.id)
> 
> I'd like to make this function atomic somehow (I should have put a todo here) so that only the fields sent to this function are updated and not everything else. So if two of these come in from different threads changing different values, in the end both changes get saved instead of having the last one to hit the update statement win all as is the case now. 
> 
> I'm not sure how to do this in SqlAlchemy, but saw Nova was doing something similar to this to update instance info in the compute manager and figured if it was good enough for Nova we could use it in the short term. 
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/110/files#r957045
i hope this applies to all of these types of exceptions in any case... :crosses fingers:
+1, can be deferred if needed - if you defer, just make a TODO for it.
Right, I know the pattern but it's not always suited, and I think this is a good example of such a case.

I prefer to avoid duplicate initialization of the index as it encompasses lots of IO, and if another thread happens to come here it would better block and wait for the running work to be finished, rather than spike more IO work (which is unneeded as the putIfAbsent would fail the check in such a case).
Blocking is fine, the CPU will be free to do more useful stuff in the meantime (context switching to something different)
I can't see the problem.
What is the problem?
`data_transform` maps the data coordinates to physical coordinates.

`patch_transform` maps the native coordinates of the patch to physical coordinates.  For example, to draw a circle with a radius of 2, the original circle goes from (-1, -1) to (1, 1) (i.e. radius == 1), and the patch transform would scale it up to 2.
@yurenju Have a look at test_browser_wifi and see how we switch on WiFi at the start of the test.
Are you sure we need to duplicate these lines? It has been a while since I wrote this code, but I think the `$curl` here comes from our `Request` class, which means it passes trough the `_build_easy` there?
typo: cnstraint -> constraint ;-)
It's a desktop-mobile hybrid example, so maybe it's better the way it is.
thanks for the review, I refactored it.
Please use consistent style with the rest of the RH benchmarks: function Foo(n) { (don't wrap for braces)
@arnodeclercq Can we intend everything with 4 extra spaces when it's inside `@media print`?
This should make it easier for people to distinguish this from the rest of the file.
This should probably be 

```js
$(window).on('resize',  function() { relocator.resizeFunction(); });
```

for consistency with the way that other events are bound.
I like the idea of minimizing the impact of functional changes and extra stuff that's required in newer browser, so I think I'd like to add this. This has been added in e2dde82.

For future reorganization, it might be better if this sort of thing was detected during the parsing of a user agent within UserAgentParsing and then set in some sort of UserAgentOptions object that hangs off the user agent and can accept arbitrary aspects of the browser. On the other hand, maybe we won't have enough of these to make that worth it.
Missing the license header.
Maybe this note should be up under the axis options?
Was this debug code?
Are you sure ``$validator->isValid(1)`` has the correct type as "Boolean" is not a sub-type of "Boolean"?
Comment before line
¿Qué ha ocurrido aquí?
использовать jQuery или вынеси всю работу с ajax в отдельный файл
Fixed! =)
отступы блеать
Good. It is good to report things so that they can fix things. :-)
You owe me 10 bucks!
Bind to both `keyup` and `change`.
If we are precise, you could simply render this in template. ;-) But OK, because later on users will be able to move between nodes is good that we have this in JavaScript. ;-)
Sorry, ignore that. `attr` should be used when creating DOM in JavaScript.
So in the first case, it doesn't matter if station_id and timestamp appear twice in get_or_create? I thought this is not possible, that's why i split it into two parts.
I know. My idea was to find a particular station with the newest timestamp, and if it fails, it would save it and push changes to the server. So pushing happens only if timestamp is changed. If I use upsert, it will update if it finds the object (which is pointless since timestamp is old and so is the data about free spots etc) or save it. So I would have to know  which action happened, update or save. to know if pushing needs to be done. Is there a way to retrieve this info?
I would use jQuery, so, `return $('<a/>').prop('href', href).get(0)`.
Never ever catch all exceptions. Only catch those you expect. Only catch `smtplib.SMTPException` here.
This not good. This does not exist anymore?
I supposed. But then I'd feel compelled to write a unit test for it, and we've made it so long without doing any of those...
Couldn't readFully just be:
public void readFully(....)
{
    int r = read(....)
    if (r != length)
        throw new IOException()
}
const Unit * source = ToPlayer();  -> Unit const* source = ToPlayer();
pls initialize
ok fixed in next commit
Yes, it will override the default one. I was wondering if we should delegate to the default one as well (it just logs an error to the console).
Ask Spiffy. Tbh I'm not sure why he did that. He assumes that the user will only use DateTime object, but what if the user stores timestamp ? On the other hand, it's quite convenient as when using Doctrine, we mostly use DateTime to store dates...
You probably want to only change the Forge API version on a branch called 1.0.2, make HEAD target Forge HEAD :-)
I hadn't used SequenceFile.Metadata facility before.. Nice to see this example!
kojnhbh
You added some unnecessary space here at the end of the line. Please remove.
you are changing the existing default behavior here, right ?   Shouldn't we have True as the default value ?
I am not sure it is a good idea to inherit from GetDeploymentOptions. I think in object orient design paradigm, Object A inherit from Object B is not only because they have similar structure, more importantly, they have a parent/children relationship. For example, it would make sense to let Volks Wagen Jetta and Honda Civic to both inherit from Car class, but it would not make sense to let Toyota Prius to inherit from Toyota Camry because they are really sibilings. This is what I feel here, GetDeploymentOptions and UpgradeDeploymentOptions are really sibilings of each other.
if (diff != 0)
      this.change_overlay_size(diff, diff);
That map function returns an array which I want to "append" to `ret`.
Missing semicolon. The 'g' flag doesn't matter if you are using it only to test for a match. But i would actually suggest to not use reg exp here. If the search term has special reg exps chars you will get an unexpected behavior (also it's on the TODO list that we will have to support for all these filters and searches ignore case and reg exp).
Should this not be "Show parsed requests and response headers"? The response is not actually parsed really, that would be a DOM.
I'm not sure what this does :) but maybe an id could be generated once and cleverly passed in using a vertex attribute, namely one of the RGBA components
Default construct for `vector2` initialises components to 0, so this isn't technically required. It doesn't hurt though.
good, I forgot about that
`File.open` preferred.
OK, then take that value from locale ([current language](https://docs.djangoproject.com/en/dev/topics/i18n/translation/#other-tags) Django is rendering the template). It seems that for API it supports much less languages than otherwise, thought.

Also implement local search as shown in [this example](https://developers.google.com/web-search/docs/).
shouldn't this use the one from the util?
Can I haz some ufos?
>Doesn't read like it's creating a CorrFunc using the given image as the definition. It seems more like you would get the correlation function of the noise present in the image.

This is actually what happens, it takes the image and then calculates that image's correlation function to make the `CorrFunc`.  However, if you think this method is confusing and doesn't add value I'll happily remove it.  It's a pretty niche application, I admit.
TestCase imported but unused. Is this one of those magic modules that gets imported for its side-effect?
i don't think it does that, does it?  if requests are 10 / 11 / 10 -- then we get minInWindow(down)=10 and maxInWindow(down)=11 and thus stable = false.  if i understand correctly then simply doing maxInWindow(down) gives the stability we want.
do you really think we need separate css file for this?
Btw, in distribution all csss are packed in one file (that is cached on client side for a year)
Perhaps we should reuse this css (anyway it's in browser cache, so no overhead really small overhead for a browser)
Could we get three contexts for the .move test, one for before, one for after and maybe raise an error if the position is not before/after?
ok, but It was like this due to devise
Active Record should not be a dependency
Also, why should "1" be not valid?
IMO it should just use casting rather then checking explicit types.
`test_entities` appears to be unused.
Because I can test the behavior on any version. Added a comment.
Can you add this above where all includes are.
See comment for #get_random_quote
Incearca te rog daca merge sa ai in fisierul config o linie de genul <code>   # ceva</code>. Cred ca trebuie facut un trim la whitespaces atunci cand citesti o linie din fisier.
I stand corrected! Cheers


On 7 November 2012 10:04, Sergey Bogdanov <notifications@github.com> wrote:

> In src/ServiceStack.Common/ServiceClient.Web/ServiceClientBase.cs:
>
> > @@ -904,10 +904,8 @@ public virtual TResponse PostFileWithRequest<TResponse>(string relativeOrAbsolut
> >                      {
> >                          outputStream.Write(boundary + newLine);
> >                          outputStream.Write("Content-Disposition: form-data;name=\"{0}\"{1}".FormatWith(key, newLine));
> > -                        outputStream.Write("Content-Type: text/plain;charset=utf-8{0}{1}".FormatWith(newLine, newLine));
> > -
> > -                        var value = Encoding.UTF8.GetBytes(string.Concat(nameValueCollection[key], newLine));
> > -                        outputStream.Write(value, 0, value.Length);
> > +                        outputStream.Write("Content-Type: charset=utf-8{0}{1}".FormatWith(newLine, newLine));
>
> According to http://www.ietf.org/rfc/rfc2045.txt (section 5.1):
> Content-Type should be:
> content := "Content-Type" ":" type "/" subtype
> *(";" parameter)
> ; Matching of media type and subtype
> ; is ALWAYS case-insensitive.
>
> which means that type is required, but parameter (charset) is optional. In
> w3 17.13 stated that default value of Content-Type is text/plain but since
> we need to specify encoding we have to explicitly set it to text/plain +
> charset.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ServiceStack/ServiceStack/pull/327/files#r2053354>.
>
>
Ok thx. I would merge tomorrow. THX for your work!
i dunno, ty
Пробелы везде.
Woops! Find and replace fail
Should say "works *only* in library mode" as we need to emphasise differences to carmart.
Yes, curious though, why?
need to clear a->a.objectItems
FYI: `trucate_words` has been replaced with the `Truncator` class.
Need info: Why wouldn't you put this in base Wallpost?
xbmcs configure needs a change in any case for passing of those flags with intree building.
pourquoi mettre une dépendance sur le format xml ? jusqu'à présent on passe par des tableaux
Здесь, конечно, интуитивного мало, но указывает он на инстанс блока. Переделаю, для читабельности.
Input validation should be in `fit`, not prediction code.
I think that it's OK, what bothers you?
Unlucky people with n_samples = n_features will get a warning even if their code follows the new API. Shall we update the warning message to take this into account?
no there is a bug: it should be `n_features / 2.` otherwise it breaks if n_features is an odd integer
I don't really get this early stopping condition. Could you explain and maybe add a comment?
What do you mean? There's a comparison of `ad` and `sd` below.
Changed to throw an exception
Need to prepend the value of args[0]
I couldn't think of a good name.

Implementation wise they might be a special form of a dataset, but language wise they are different.
This comment seems out of place, now
It should, when I get to that.
When other clusters have their own non-legacy names, we might want to add a hash with the names per cluster and remove the ```if name eq thor``` check.
There is no point in having a critical section, as initialisation code is not multi-threaded, nor re-entrant.

I also don't remember seen static objects lying around. Often, what we do is having a static pointer and a singleton extern function that initialises it on first-call. However, if you do follow this route, then the critical section becomes important.

Not just this avoids problems in initialisation, but also reduces memory foot-print and start-up time (even if just a little).
And these cannot be combined... maybe that's the design?
Why fixed size?
indexCapacity may still be < maxRows as a result.
@peterbe sure it looks fine but I just went with what prod does
Actually I don't need those functions. I've just added them when doing my first git experiments. They are in MsSql2005Dialect and maybe they would even work in earlier releases. I'll drop that change out of the file. It should be handled separately.
Lines too long for you, eh? Fair enough. :P
Maybe this is a dirty hack...?
Ok, yeah this one makes sense as a default.
Shouldn't be committing the .architect file. Aaron had added it to .gitignore in some other examples, which is a good idea.
From a casual glance, it seems like this scenario is already handled well by the ``Table`` class.

     @property
      def sequence(self):
            return (self._sequence if self._sequence is not None
                               else self._meta.sequence)
    @sequence.setter
      def sequence(self, value):
          if value:
              value = Sequence(value)
              value.expand(self.base_columns.keys())
          self._sequence = value
ah, got it. we should take out the commented line regardless though
We should still have an agent here. I believe the Android lib subclasses `CMWebService` anyway, right? So it can override this method.
Okay, good to know. I will add in the test case in Java code later on when I got time. Thanks
Though we move the everything to init(),  I think it is better to keep this._super() in  this.ctor() function. Keep it the same as parent class.   Is it needed? What do you think?

```js
 /* void constructor */
    ctor:function () {
        this._super();
    },
```
It will be a good example and code style for developers.
Please remove all commented code that we don't want to use. The try catch here does seem correct to use. I suggest also printing out the exception so that the user knows exactly what the issue is.
Consider @Logged
use pixels instead of ems here
This approach breaks if we later need to add a new field to the book. You should use `assoc`. No points from this one.
I'll just check with them :) and will update pull request, really is a lot of code copied and pasted along lot of modules!
Kinda hate \n as a separator.  Something wrong with a regular semicolon?
The same as with other ports loops. If there can be multiple INSTANCES all the ports should be reported. If not, this loop isn't needed.
Ok I'm no 100% sure on the report_note thing.  I believe I have fixed this, can you please double check for me.
Localized, see above.
I'm not even sure what this exception is trying to catch here...
Styling should be done in CSS.
Guess we're not replacing them!
Please rename the variable user_id to login_name
There's no reason for `transform` to be put into a variable here.  Just pass `this.transform` directly to the function.
Perhaps say "longitude, latitude, and height" since that is the order in the constructor.
1. As with a related comment, maybe it would be better for 'mat2' and other types to be a constant defined elsewhere, like in a UniformType object.  @pjcozzi ?
2. Use isArray(uniformValue) instead of instanceOf
Instead of using "private" functions in the `PolygonPipeline` object by prefixing them with an underscore.  Declare them above `PolygonPipeline` so like are local to the file and accessed through a closure, not the object's prototype.
Actually, I just reread the code - it's called twice. It's all about getting the catch block in there, and repeating that offends my sense of code cleanliness. :-)
Ah, gotcha. I'll change this. Thx
So, from a doc point of view, all Ti objects inherit from Ti.Proxy, which defines the methods addEventListener and removeEventListener.
What styles shall we use here?  [web-base-template/themes ](https://github.com/jquery/web-base-template/tree/master/themes) should help me?
Why was this uniqueness constraint dropped? Inversely, why do events need unique titles?
OK.  I made a handful of exceptions here so that variable names matched precisely those of the research paper where the algorithm is described; I'll correct them.
I think we could use `is(Type _ == Algebraic!(T), T...)`
Cool, now it's working for me, too :)

The Eclipse compiler still doesn't like the test, though. I've added a type parameter `T` to the test (https://github.com/gunnarmorling/hibernate-validator/commit/c75f15276f82f9a195f5a6fe7909d6b841b24f7f) which makes it also compile in Eclipse.
Can you justify why you run server commands then local commands?

Would it be better the other way 'round?

Would it be better to process the two lists merged together and sorted by date?
pls stay with the format 
+ ...
+ ...
+ ...

instead of 
...
+
...
+
...
why not naming it proxyFactory?
I use it 3 times here though, shouldn't I declare it?
You should also have spaces between language constructs (`if`, `foreach`, `for`, etc.) and brackets that follow them.
I also spotted a few other uses of `$var='value'` and `foreach ($array as $key=>$value)` where the same applies.
We won't be able to do awesome CSS3 transitions then ):
Not sure how to have Chronic parse it?  Would it do so automatically?
yeap, done, try again; I think it's all clear now.
I think we should add the ``setSerializeNull`` method to the VisitorInterface, and also the serializer. 

This would not only make this test easier, but also the usage in general.
no, this is just a dummy thing to be able to create a final matcher which extends UrlMatcher which has the route collection in the constructor. but maybe we would better implement a constructor on our finalmatcher UrlMatcher class that does not require the collection argument.

we later overwrite the collection when doing the match.
@m5o @trisweb - Bootstrap defines this as taking a value [0,100] when the CSS spec takes a value [0,1]. We redefined this (#52) to the CSS spec since it makes the most sense, and to fix potential conflicts with Compass. Thoughts? I've gotten used to the [0,1] behaviour.
how do you mean? inside of the repo?
Are these defined anywhere?

Feels like a regression because I've seen a comment from Chris about this before.
That's cool with me
Could we avoid defining this twice?
Weird. I've always seen cases outdented because they're labels. Indenting the makes the flow less clear IMO.  I think outdenting is consistent with the classic style layouts as well. But as long as the code is consistently formatted I'm not going to quibble about spacing.
pfft unclear, will break it out...
Can you format this constructor? Long lines are hard to read in GitHub
I did, but this seems inconsistent - no other SI headers have '-' in them
You missing the point. It is definitely not an exception. You simply used Expression to override the conventional mappings. That's all.
I agree, i'l close it
add @link for TransactionSynchronization
Use this.log.
Well, that line was automatically generated by Rails. I'll ask Matias about this.
Can you edit these files to move the sections down and add a single header? There's no reason that "Client Side Scripting" should show up on the dev TOC.
indentare, vă rugăm :)
Can you be any more specific than result and data as variable names?  If you can, that would be great, but if that's as specific as you can get, that's fine too.
Do we not need the starting index or a unique id for this?
Ok then, I'll try something else :-)
I disagree - I think the SimpleObject representation of an object should have a predictable set of properties, even if some of their values are null
Each of these comments could be refactored later. Would make it easier to read. #later
ba zepto dg nemikhad -moz- ya -webkit- bezari
This looks wrong to me. It will output it even when it is not exposed now
Or just import everything from ``numpy.testing``.
Just fixing a typo in the comments
Aivan, nyt tajusin, että toi on järjestetty. En varmaan tehdessä ajatellut sitä tarkemmin.
Please, note that condition to open link in a new window is inside the condition that menu url is not blank. It is because of the fact, that if you insert blank url, but set "open link in a new window" to "yes", then when you click on that menu entry (even though it is not a link), it will open new blank tab. Thus, we need to set onclick even only if there is a link present.
yep loll
@dylin Perfect.
@nukulb r+ for SCM
since each stack has its own table this is not needed.
I think the best place for the comments links is immediately after the diary entries link. I agree with your comments about how long the list is getting but I know other people are already working on redesigning that page so I suggest we don't worry about that too much for now.
Kinda. I think it is hard to decouple the CSS and HTML. So made a basic easily restylable verion based off SCCC.
I don't know, to be honest. I always reset the mock in a test before I use it, instead of in setUp(). I noticed that with Mock, somehow it kept its state throughout tests. I had some failing tests because of this. Maybe it is a bug in Mock, I don't know. But just to be safe, I now always reset_mock().