labofsahil/github-event-dataset-2012 huggingface dataset:
On Sun, Jul 22, 2012 at 2:51 PM, oltmans <
reply@reply.github.com
> wrote:

> > +            self.session.create(name, args)
> > +        except:
> > +            print('**** ERROR in session.create("' + name + '","' +
> args + '") ***** ')
> > +            raise
> > +
> > +    def session_close(self):
> > +        self.session.close()
> > +
> > +    def tearDown(self):
> > +        self.session.close()
> > +        super(Test_Cursor_Random, self).tearDown()
> > +
> > +    def create_session_and_cursor(self, use_next_random=False,
> use_random_table_name=False):
> > +        if use_random_table_name:
> > +            rightnow = datetime.now()
> > +            self.tablename = self.tablename +
> rightnow.strftime("%Y%m%d%H%M%S%f")
>
> So this could be misunderstanding on my part. I used the random part
> because I thought if I open a table whose name is defined in "scenarios"
> then it could be possible that table was used in a previous test .


Every test_XXX function gets a new, empty directory to run in.   If you
open the same table, in the same test_XXX function, then you'll potentially
see overlap, but not otherwise.   (I usually just drop the object at the
end of a test function out of habit, but that's not necessary.)

--keith

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Keith Bostic
keith.bostic@wiredtiger.com
I'm uncomfortable doing a full pass over the entire cache to find a single page -- previous incarnations of this functionality made a specific request to the eviction server with a reference to a single page, and forced that page out of cache.   Obviously, that requires more infrastructure to negotiate between the requesting thread and the eviction server so they don't race.

My concern is a single file that's messing things up, inside a cache that's quite a bit larger.  In that case, we'll walk the entire cache, when all we care about is a single page in a single file.

Maybe there's a possible middle-ground: if we believe this only happens for a single-page file, could the forcing thread request a single file be reviewed instead of the entire cache?   That way, we don't need any particular infrastructure because the btree handle can't change out from underneath the eviction server, and that way we don't walk the cache, we walk a single file's pages in the cache?
request_var() ?
Are you sure ``$length / count($this->sources)`` has the correct type as "integer" is not a sub-type of "double"?
The IF-ELSE statement can be simplified to ``return $response->isSuccess();``.
Das stimmt nicht ganz. 'ch.hsr.bieridee.back' gibt es nicht. Die Backend Sachen sind direkt unter 'ch.hsr.bieridee'
I tried coming up with some alternative function names, but they were all bad.

I think since `remove` and `delete` have different number of arguments and no optional arguments, any confusion between the two won't be an issue: If you happen to mix them up, you will get a runtime error, and not accidentally delete the wrong type of thing.
use the new import syntax.

import 'dart:math';
Think you want to delete this line, or change the comment
You should be able to refer to this by the fully-qualified name:

`$host-git-repositories::gitrepo_dir`
Not sure, that's something that should be discussed though. Everywhere that these $db->getError msg/num values are - an Exception has already been thrown in the database. Andrew indicated these should be removed. I assume the thinking is that the originating process will trap for exceptions. 

I'll add a post on list - I think this needs some discussion.
Nah, onde definir a currency. Desculpa, esqueci-me de dar contexto. :P
Up to you (and it's not a deal breaker) but this may be more aesthetic as well:

``` js
if (hasBackgroundPositionXY && /^backgroundPosition[XY]?$/.test(property)){
    if (!result) return (property == 'backgroundPosition') ? '0px 0px' : '0px';
   return result.replace(/(top|right|bottom|left)/g, function(position){
       return namedPosition[position];
   });
}
```

Note, this would mean that L128 - L130 are no longer needed. Just need for you to confirm that the hasBackgroundPositionXY check doesn't break/filter something out.
1s sounds pretty long? can we drop it to 0.1 or so ?
I am not sure if it is OK to have a newline before `{% plural %}`.
Move `10` to some settings constant (`CHECK_ONLINE_USERS_INTERVAL`?).
This should already be gone :)
I would add here also `user.is_authenticated and user.is_staff` to the mix. Not that it is necessary, but to the reader tells easier when this thing goes through.
`columns[len(columns)]` == `columns[-1]`
Spinner removed.
Why are you defining constant variables you are using only once?
Because Django doesn't like other formats. I have a lot of troubles with sending array of arrays with post for example.
Didn't work with those.
This line. How do I remove it?
no, I'm actually passing in the emails to the template now.
Don't create a new struct, Position class should work
Yeah, this should be moving from tabs to 2-spaces.

On Sun, Oct 14, 2012 at 12:38 PM, Moxie Marlinspike <
notifications@github.com> wrote:

> In res/xml/preferences.xml:
>
> > @@ -18,42 +18,42 @@
> >   */
> >  -->
> >  <PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
> > -  <PreferenceCategory android:title="General Settings">
>
> Looks like this diff is only spacing changes? I think we should be living
> in a 2-space indent world, wherever possible.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WhisperSystems/RedPhone/pull/18/files#r1837738>.
>
>
@ametaireau but he tested "if args.command not in self.commands:" just before, so this line seems safe
Why do we need here a new handler? We already have such functionality?
take care of the connection and pipe_fd when destruction
use a single line with like this:
    if (timeout_event_ && timeout_event_->timer_set) {
该路径名可以随时 ->该文件名总是可以
I wonder what would happen if these p elements weren't centre aligned.

The page might not look right...?
Include stdout in IllegalStateException? Or log and then throw?
Did something similar: getCli()
Intent is that it should grow to just two, as that is the "max sustained desired". 

For example, I might want to only grow to a given size only if you consistently tell me to grow for 1 minute.
if the metric causes something like this:

assert initialSize == 1
load_sufficient_to_resize_to(100)
sleep(58*1000)
load_sufficient_to_resize_to(2)
load_sufficient_to_resize_to(100)

then 60 seconds after the start we'd grow to 2, because that is the highest sustained load. And then 60 seconds after the second resize_to(100) we would grow to 100.

We added this functionality for a customer proof of concept, because we wanted to avoid growing/shrinking too aggressively when it was taking a minute to bring up a new instance in the cluster.

---
I'm guessing the test failed because it's too time sensitive.
The test is also hard to understand, so I'll improve that as well.
Bah justement je donne mon point de vue. Je pretends pas avoir deja distribue du contenu mais j'ai un point de vue de consommateur.
It's a valid scenario - if the trace ends with a scope open (cut off too soon) or begins with a scope open (ring buffer was full and the open event was missing) it's possible for one to be unmatched. It'd be cool to show a little indicator that this happened, though, instead of broken bars. In this case, I'm seeing a weird unmatched scope that is certainly a bug in the wtf.analysis library that's causing a scope not to get all its events loaded.
Is it really good way to use javax.validation?
This comments should not be included right? So maybe do not show them in the code-example.. Its stated in the syntax section above already
You should be able to overload or derive a dictionary that used a custom IEqualityComparer to do case insensitive key comparisons. That way you could get rid of all the foreach over the keys and explicit comparison, and probably pick up some perf and clarity.
Isn't it `stdClass`, or am I about to learn that class names are case-insensitive?
мм. коротше ніяк? порадься з хлопцями
Since this will change default behavior of everyone using the library, I'd like to make them able to "opt-in". Could you update this code such that they call it like this, `Monday.parse("$1.25", :assume_from_symbol)` or something similar? Essentially they can pass that symbol (or something similar) instead of an actual currency and it will use your assumptions.
Just bundle install will do here. Using binstubs and setting the path are pretty specific settings, neither of which are common. I also tend to prefer a `rake setup` task, but this bootstrap script is fine. thx
Can we not use tables for layouts.
This was a bug in webwerkstatt, which i fixed today, which seemed to be already fixed in the core…

Am 21.11.2012 um 21:21 schrieb Robert Krahn <notifications@github.com<mailto:notifications@github.com>>:


In core/lively/bindings/tests/GeometryBindingTests.js:

>      test10UpdateOnConnectViaConnectionsSpec: function() {
>          var obj = {x: 23, connections: {x: {updateOnConnect: true}}};
>          lively.bindings.connect(obj, 'x', obj, 'y');
>          this.assertEquals(obj.y, 23, "not updated in connect");
> +    },
> +    test11GeometryBindingUsesConvert: function() {
> +        var morph = lively.morphic.Morph.makeRectangle(0,0, 20, 20),
> +            observer = {result: 0},
> +            c = lively.bindings.connect(morph, 'position', observer, 'result', {
> +                converter: function(v) { return v.x * 2}
> +            });
> +        morph.setPosition(pt(50,50));
> +        this.assertEquals(100, observer.result, 'converter not working');


A test for that was already added in #12d4b971

—
Reply to this email directly or view it on GitHub<https://github.com/LivelyKernel/LivelyKernel/pull/71/files#r2198945>.
That seems like a bad idea.  Because idGenerator and storeFactory should be left as final, and if this is private there is no way to set them to anything other than our defaults.
Maybe we should extract "UTF-8" to some constant.
This probably should be "unneeded". However, I would suggest "unnecessary", as I think it's used more.
В finally оно тоже есть. Но в catch происходит exit(1) и finally при этом не должен выполняться, вроде бы
И что в этом плохого?
Как-то это топорно. И неудобно.
А в связи с чем?
it looks like the server param no longer overrides the value of servers_real. is that intentional?
well in fact, int the original code written using Weaves and C, I had these functions implemented, but I am not sure that it has utility, so I did not reimplement them yet, I will remove these lines for the moment.
This reminds me, we should probably make sure all the plugins conform to the dtype standard given here: http://scikit-image.org/docs/dev/user_guide/data_types.html
h is sampled for the persistent gibbs chain. For computing the learning statistics, by deriving the log probability w.r.t. the parameters you get a difference between free energy derivatives. Which results in using the mean field of the h's (see http://www.iro.umontreal.ca/~bengioy/papers/ftml_book.pdf page 64). When computing the learning statistics, it's mathematically valid and better to use the mean-field because it adds less noise to learning.
Well if the function is in Cython already. But then it only works for a pre-defined set of functions, which is not really the point of passing a callable, right? If we have to hack libsvm any way, why not just include the kernels there?
The cython callable would help users that write cython, though...
The line below?
@amueller I personally incline to my implementation here a little as I just think ``x in labels`` thing is a little slow and it should be done for all ``y_pred`` and ``y_true``. Personal taste though.
`#{doh || 'something else'}` :p it's just javascript
Thanks for your suggestion.
In this case, using `(storage_class & STC_TYPECTOR) && !isThis()` is more short and identical.
This shouldn't need QString (or std:string), except when you need to use .arg()
Is this the right formula?  I would think it should be `max * numberPart / 100`
Use Object::Get instead of searching the list of properties:
```
Local<Object> propObj = propValue->ToObject();
propObj->Get("title");
```
rolled back to TiUIOptionDialog, TiUIPicker etc.
Yes in smaller codebases it is cleaner!
Rajouter un point virgule à la fin de l'instruction sql. et mettre VARCHAR(255), on aura de la place comme ça et ça nous coute rien.
Tu peux aussi renomer le fichier en changements.sql et mettre des commentaires en les commençant par -- 

Sinon a-ton vraiment besoin d'un chemin pour les docs ? On peut pas le déduire de l'étude et du type de document ?
You should open an issue if you want this to be considered/done :smile:
Shoudn't it be `ElementKind.INTERFACE` here ? `@Accept` is only used in interfaces. I think this change breaks the feature. Can't test your pull request right now, I only have http access.
`what are they used for ?`
I think this is saying that the number of buckets should be set so that one bucket can just be big enough to not spill.

I'm not sure if this is a good estimate or not.  It is a sensible minimum, but if the buckets aren't filled evenly, and no dedups occur then they're all likely to spill.  I suspect a larger number would be better.
This is 2-spaces indented when everything else is 4-spaces indented. Can you please move to 4-spaces everywhere?
So after testing it seems it was a mistake to not install it as a general module. Can you please turn it back to ``sudo npm install less``?
I think a showspinner method would be good here, just so if we change how it's displayed it's easier to keep the two methods in sync. Other than that looks good to me
Done in  10fe282b26
See this is why it would have been a bad idea to fake on the view level - this is a different bug that you've uncovered. The error seems to be because `artwork.images` is empty, so `artwork.images.first` is `nil`. That shouldn't be happening because that's exactly what `displayable_artwork` is supposed to fix. So if this error is still happening then it's a problem in that method that needs fixing.

Can you investigate by looking at the values returned by the various methods in that chain? (e.g. call `Artist.with_displayable_artwork` in a console and check that none of the artworks are imageless and same for `artist.displayable_artwork`).

This isn't a notational error - it's an algorithmic one. There's a logical flaw somewhere in the methods defined in the artist domain model.
never break backward compatibility ...
What? isn't this on line 68:

https://github.com/puppetlabs/puppetlabs-razor/blob/master/manifests/init.pp#L68
Why not use job.to_s.tuncate(60)?
empty elsif statement should be removed
I am not sure Chris will want to rename this option.
Probably. I think both are about the same.
Shouldn't be list of id's?
When referring to options, methods etc., mark those references up as code (or whatever we're using?), like "use the &lt;code>menus&lt;/code> option".
Shouldn't need this with WD but let's see how we go and we can remove it later when all POs have been migrated and we can run everything.
Since "test_instance_resize_up_success" has no assertions and is so small, maybe it could be merged into this method.
I thought for a rectangle the width and height had to be positive, non-zero numbers. My thinking was if the width and height are zero then its a point and if only one is zero then its a line. Now, I think making the defaults all zero is OK though.
I totally agree with you here; I just couldn't figure out how to do it in regex. My goal was to "make it work" and after getting that all integrated and with tests, I'll worry about the "make it fast" part.

It's not just Date with this limitation; for example, anywhere in Javascript:
var foo = +".5"  // turns into a number of .5
var bar = +",5" // turns into NaN

As far as I could find online, this behavior is the same no matter the system culture, but it's something we should probably get hard facts and a link on.
Consider naming this `getRenderState`.  It only creates them once.
I actually tried that and it didn't seem to work (though I'll admit I'm not CSS guru so it could have been something else I was doing wrong).   In general, we prefer clear code over concise code, though in this case your suggested is probably fine as well (assuming I could get it working).  I'll give it another try.
@IanLilleyT have a look - let's make sure `LabelCollection` is as simple as it can be before merging this.
This might not be a big deal (and could be reserved for future cleanup) but we can probably cache the result of `getScreenSpaceCameraController()` instead of calling the function every time, right?  Same for other calls to this function throughout this file.
Yeah, maybe.  Are you suggesting to promote this function to a function object that extends the Model class?
I will look into that, but I wonder whether that is a show stopper for this pull request to be accepted? Personally, I dont think that is a show stopper. If you agree with me, I can create an issue of your request so that we dont lose it.
dman seems to want some magical `.tag` files. Not sure where they come from. Ignore dman for now, I guess.
This looks reasonable. However, there may be other tags for `env` that are not set in chef. Needs a little consideration, since even chef call it `chef_environment`.
对齐……
Can you clarify this and/or give me a bug #? Does this mean that the we USED to do this, and the composite layout updates changed it? Or that we NOW scale based on the image size rather than the parent size?
Any reason this can't just be a private variable of the class versus passing it around as a function arguments?
Trying to be consistent.  Every method should have a javadoc even if it's a placeholder IMO.
when you split an unicode string, all string must be declared as unicode; please put an u before the " (u"")
This join isn't doing anything.
Right-to-left languages, mainly.
Please add some empty lines to group the code. This is hard to read and understand.
True.. but I wanted to be sure in case people move methods around after generation.
а то был же действительно баг который ты не заметил, с социальными штуками
Please don't use wiki as a source.  From the same source I found http://en.wikibooks.org/wiki/LaTeX/Advanced_Mathematics#Advanced_formatting where they do not include the dx at all, which is mathematically incorrect, and an example where they do use dx without the roman font.

The source I gave you is a source written by mathematicians and is not available for anyone to change without vetting the information.  The problem with wiki is that it is not reliable since anyone can change it.
I think it should be called with a clear reference to the application panel, it seems global to the shell
This needs to run during a `make as7` as well.
Because uid.system can't access /sdcard for some reason.
From your words I take it that currently emplace is broken and *if*/*when* it's fixed there won't be any bug with appender?
Hmm - I feel like I tried that, but can't remember for sure. I'll give it a go when I have a minute.
I see now, so I think we should be calling the other update() method (the one with 6 arguments not 4).  In this version we can get the TDRevision back directly, and not have to pass it through the doc object.  This will work more like the iOS version.  Also the update method you were calling is setting headers and response bodies that are not correct for the bulk request.  It may work because they ultimately get overwritten, but its clearly not the right way to go.  There are no good unit tests for bulk_docs so I'll let you know when I have something to test.
s/its/their/
We check for "force" somewhere else in the code (skipping backoff ourselves…?). Clean that up, too?
This has to be called after we have access to `Config::get('metadata.theme')` and before the page is loaded.

It could possibly be placed around https://github.com/anchorcms/anchor-cms/blob/master/system/classes/anchor.php#L49
We now have two drug forms per drug - one for each of these fields, each with its own submit button.

It looks like this is going to be a common occurrence. How about using the jQuery Form plugin: http://www.malsup.com/jquery/form/ ?
Is this really required? the global scope is `window` already...
Any updates ;)
So, the idea of the top-level class (`init.pp`) is to be a super-simple high-level abstraction around installing PuppetDB all on one machine.  It's always going to install postgres on the same machine as PuppetDB, so I don't think we need this parameter or the`database_host` parameter.

The idea is that if folks want something more complicated, they'd use the lower level `::server` and `::database` classes on different machines... does that make sense?  I'm open to feedback about changing that but I think that seems reasonable.  I also think that most of the rest of the parameters are probably OK to add in.
this is actually a question i had when i was considering this:

i didn't know if it would make most sense to attach the events to `body`, `Khan`, or `problem` and secondly, i didn't know if it was attached to a dom element, if it would make sense to use $.trigger or $.triggerHandler to have the event bubble up with 'problem updating' events listening to, say, the `problem` dom element and 'chrome updating' events listening to say, `body`

also, is namespacing the events crazy?
Sorry. I had made substantial "behaviour" changes and then remembered it was correct! I will fix that one.
This is a standard in many object-oriented languages, including C# and Java. PHP has a lot of old habits, including a complete mess when it comes to naming of native functions (some are abbreviated, others have underscore). However, native PHP classes (PDO, DateTime) have camel case has the default naming convention.
Bah, I can't get this on the correct line, but the error message below refers to the iframe dialog, not the window dialog.
I do agree with @christeredvartsen on this one, but I've attached a test that asserts the output for each attribute I've made support for.
While you are at it, could you add double colons here so that sphinx can render the following properly?
Could `signalfd` be considered `@trusted`?
Can you make the header follow the convention of other headers in druntime? See e.g. the `src/core/thread.d` header.
Shouldn't this be ${buildDir} rather than hard-coded?

http://wiki.gradle.org/display/GRADLE/Gradle+0.9+Breaking+Changes#Gradle0.9BreakingChanges-Project%7B%7BbuildDirName%7D%7D
Sorry, I was just responding after seeing the snippet in an email and thought this was in a *unit test*. So, ignore what I said completely... IllegalArgumentException is probably better.

Although interesting side note: using Assert in the first line instead with instanceOf might be even better. When it comes to unit testing coverage, direct invocation of Asserts within the mainline logic produces better results than having checks within conditionals.
Is not possible to have this?

```
_build/
languages/*/_static/
```
The factories are used for pickle's _Given a $model exists with $fields_ steps and so on, see comment below. It falls back to ActiveRecord, but that doesn't handle things like wanting to have for example a vanilla contact detail for a scenario, which would depend on a user that we don't want to explicitly create all the time. The factories create default values for their associations (using other factories) unless specified otherwise.
You probably want `version` to always be first.
As said, I was just trying to work on the FIXME 

> FIXME The List<VP> does keep a strong reference to the key ClassLoader, use the same model as JPA CachingPersistenceProviderResolver

The javadocs for _WeakHashMap_ says:

> The value objects in a WeakHashMap are held by ordinary strong references. Thus care should be taken to ensure that value objects do not strongly refer to their own keys, either directly or indirectly, since that will prevent the keys from being discarded. Note that a value object may refer indirectly to its key via the WeakHashMap itself; that is, a value object may strongly refer to some other key object whose associated value object, in turn, strongly refers to the key of the first value object. One way to deal with this is to wrap values themselves within WeakReferences before inserting, as in: m.put(key, new WeakReference(value)), and then unwrapping upon each get. 

Given that the list of Validation providers will reference their class loader it seemed to make sense to wrap it into a _WeakReference_. I see your point though 
that this means entries will disappear quickly from the map. On the other hand, does the current approach work at all? Are we not really after a soft reference map where the entries get garbage collected when memory is needed? I have another look at this (maybe I can actually test this in the HV integration module)
Can this be:

```ruby
class_or_item = class_or_item.roots if [Class, ActiveRecord::Relation].include? class_or_item.class
```
```ruby
remote_#{column}_url.present?
```

Good call on using super
should be `app = Rails.application`
I don't think you need to ship the entire license file like this, we've just been including the smaller boilerplate. See the appendix for the boilerplate in question:

http://www.apache.org/licenses/LICENSE-2.0
Actually it is the level's tick: The list "tickItems" is processed in the methode "tick" of the level-class: 

    public void tick() {		
    		for(int i = 0; i < tickItems.size(); i++) {
    			tickItems.get(i).tick(this);
    		}
        ...
Oh , ok ...
Here applies the same from the previous comment, we should test something like:

I am outside of the system or
I am not inside the system or
I am not signed in or
I can't start a new session
The `StartsWith` comparison here should use `StringComparison.OrdinalIgnoreCase`.
> Yes I agree that might help even more for particularly low rank matrices X. Not sure how it would work in general. I didn't realise sklearn had an implementation of that randomised low random SVD algorithm - very nice! Are there any benchmarks comparing fast_svd to other SVD approaches?

I'm curious whether this can work or not. I would tend to think it
requires an exact SVD. BTW, keep in mind that using fast_svd would
require an extra argument (e.g. named "n_components").
I would expect this to be false. Setting the next line to be true so as not to break the build for now.
Correction : the has_cell function should do that. My phone just placed this comment on the wrong line.
actually I think it should be retained in the non-profiled part and removed here, since you want to do the manifest-update in both development and release builds -- in development it would set debuggable to true, and in release, set it to false, plus do the versionCodeAutoIncrement.
I want to be flexible on the use of css classes. But `.center` we should easily be able to avoid!
INITIAL_SIZE is private, so won't show up in the JavaDoc unless you run with non-default options. Which you probably don't want.

Probably want to say, "with a default capacity."
Break this out into a const. No need to allocate a new array.
All files have been updated with the proper heading.
The whole point of all these methods was to avoid exposing this object, otherwise why have any of them?
seems like this should be removed.
This isn't being used.
Do you really need to put the session token in the request body?  It's already in the header.
Is there a type-o in the namespace for this class. I'd assume it should have been either fl- or fls-