labofsahil/github-event-dataset-2012 huggingface dataset:
yeah - if you want to put the GitOid type on the delegate signature, I think you will need the ref keyword (as the native function is returning a pointer to a struct, and not the struct itself). If you do not want to enter the types in the lambda, the other option is have IntPtr's on the delegate signature and marshal the structs manually inside the callback.
should be smethod, but I like sort_method better.. so I vote for changing smethod to sort_method
personal issue: "swagger" is not a positive word. If you have swagger, you're an egocentric person.
This cache is not portable -- Windows users (like @pomax) will have to change the path.

Why not use [Local-memory caching](https://docs.djangoproject.com/en/dev/topics/cache/?from=olddocs#local-memory-caching) here?
Shouldn't this check that the person count has been incremented by one
Also it would be nice to check that the created person's age is correct
Did you see my comments? What would be the behaviour when updating the patient because this will happen on every update of the patient
curious for some pro/con on having overrides accept a default/missing arg.
I'll see what I can do to make it optional, apt is already there.
> +Good, fast, or cheap (pick two): it may be the Operations professional's

Is it worth making a citation to the
http://en.wikipedia.org/wiki/Project_management_triangle ? I think this is
a core concept that we might need to explain further somewhere.

----

Project Management is a topic under soft skills. I moved the Agile headings
under Project Mangement but all of that needs content. Should this go there
instead of in the top section?  I put it in both for now.

Carolyn
Please leave specific imports instead of "*", as there can be class name collisions this way.
Sure, my lord.
Do we need the slug check here? If the view is the page view then I think we've matched a valid page.
Please add another paragraph as how different this is from twig and in which cases is this more convenient, or if this is a default case. A clear complete discussion on this can be most helpful. Thanks!
``--dev`` has nothing to do here. Its only effect is making composer install the packages listed in the ``require-dev`` section of your root package.
compatible with翻译成兼容
或者对应（意译）
Why all the HDFS copy-pasta when the local filesystem source and sink don't even use HDFS?
Now I've seen the same string quite a few times, please use the Config library's "fallback" mechanics.
This class needs more love - if I only had time...
cond is False can't happen, it's taken care of in `__new__`.
perhaps pass down `config` into the constructor so that local overrides are possible in the dispatcher config section
Порядок инклюдов.
В частности, qjson должно быть после Qt и до util.
This might be easier to read like so:

```
Vimrunner::Vim
  #gui
    returns mvim on Mac OS X
    returns gvim on Linux
  with a +clientserver vim
    with -xterm_clipboard
      on Mac OS X
        uses mvim for the server
        uses vim for the client
      on Linux
        uses gvim for the server
        uses vim for the client
    with +xterm_clipboard
      uses vim for the server
      uses vim for the client
  with a -clientserver vim
    on Mac OS X
      uses mvim for the server
      uses mvim for the client
    on Linux
      uses gvim for the server
      uses gvim for the client
```
¿Este es (o no) el mismo que el de la línea 73?
We need to assert that the component returns an error (which is currently does not) rather than this arbitrary method call. We need to rescue like we do for OptionError.
Please have a look at all these template and do a quick clean of these document.getElementById, width and offsetWidth detection
I strongly suggest doing this pull again
It can't since the methods are static :-(
@vblagoje I don't buy your argument, you could use atomic maps to group the keys. So, given a task, here's the keys that belong to it. Just like HTTP session replicaction works: 1 session -> N key/value pairs. Clean and simple, and without a million of caches :)

@danberindei Indeed, I remembered something about jira specifically for that, thx for adding the link. IMO, that jira should be implemented separately.
Why not assign this directly into `*error`?
You can shift this comment to the left a little bit.
Tiny nitpick on something you haven't written: we should not test for an emtpy list using len(list) but just list.
An empty list evaluates as false in python.

Maybe you can replace:

    if autolim and collection._paths and len(collection._offsets):

by

    if autolim and collection._paths and collection._offsets:
Inheriting from object, while probably benign, will change behavior for those inheriting from these classes under Python 2.x.  This should have an entry in `docs/api/api_changes.rst`.
Really? My mistake then. :)
I think it could be but there are a few other places in which this exists so we should do it as a separate pull.
Very thorough, and all sounds good to me. I'm happy with option 1 (no refactoring in this pull, but file followups). These followups can be discussed independently, but I like the proposals in your comment.
Break these locators out into private variables outside of the test method
Weird sentence in French...
How about using `running: !appWindow.fullscreen` instead, and removing the start/stop logic in the mouse area (leave restart)?
Too much ternary if; put all of these in an if (fitsVertically) { } else { }
It will not work on http://openlayers.org/dev/examples/
The WMTS server from Vienna should be used (see examples/mobile-wmts-vienna.js)
I'm sure BC is not the problem if we want to create really usefull things, support old thing always expensive. As I said before, the size and performance is impotant. I have a project with billions of logs for analyzing purpose, and additional payment for useless bytes looks bad.
I like this name better than the one I saw you post in Team. :thumbsup:
Or what that comment means -- what was changed?
theres an image upload bucket variable you should use
could you send these patches separately next time? though ok for now
The problem with javascript is often people find and alter the piece they are interested in, and ignore the wider picture. It's nice to know what code is doing without having to jump all over the place to find out.
I've added some inline comments around this, as its not very straightforward to read (see merge commit).
Using `decodeUtf8` is actually unsafe, since it's a partial function. Can you use something like `decodeUtf8With lenientDecode` instead?
is there any way to non-inline this javascript, even if it has to be an additional js file? :)
Are you sure the method ``setParent`` can be called on ``$contentPage`` (of type ``string``)?
Are you sure that ``$options`` (of type ``*``) is traversable? I also found this on lines 121, and 158.
I don't know what to put for the 'filePath' as we don't have any other information other than the contents of the stream.
ScriptOrder itself should accept a file, no need for this second argument
I've always liked hidden args for internal use especially when it avoids an extra (interal-)global variable, but I guess that's personal preference.
Is there any way you can change this to a single HTTP request?
Fair enough - lgtm.
The updated commit should fix that.
We should return an InvalidArgumentException in this and the next case.
Currently just the csv datastore uses this. I use a different csv parsing strategy in the importer case, namely that I return the attributes without trying to guess the geometry from the column names. Guessing was problematic from geonode's perspective, due to the original column information being lost once the guess has been made. All that we get back is the feature type that was generated, and we wanted to keep all the parsing on the geoserver side if we could. For that use case, handling that completely in the transformer worked out better because it allows for a ui to give the choice to the user.

That being said, this is our only production use case, afaik, for using any csv parsing strategy at all, so maybe I should just change the default strategy that the datastore uses and not have to introduce this context parameter. The down side is that in a dev setting it has been useful to have others import a csv file through the importer ui and have it guess the parameter names.

This could also just be the wrong place to signal that parse intent. Thinking ahead, we may need to have some way to pass down information to the parsers to alter their strategies, even with csv. I can imagine a column that represents an address, and a parsing strategy, allbeit a slow one, could be to geocode the column to generate the geometry. Maybe for kml we'll have some variations too, and it could be useful to somehow pass that information, possibly from a decision driven from the ui, down to the parsing.
I'm not sure I understand why the group needs to be cloned.
Good idea. Apparently I forgot the Rails 3 test file anyway...
Decide, do you want empty lines or not after super call.
Why not `BooleanField`, too?
(Untested code. But to give you an idea.)
But this should definitely not be committed!
Do we need an access to direct messages? If not, then the problem is solved. Twitter applications can have three level access: read, read&write, read&write&access direct messages. I read on couple of forums that Twitter always wants user to reauthorize when an application is demanding 3rd level access. So if we only want:  read Tweets from timeline, see who they follow and follow new people, update their profile and post Tweets for them, then authorization is done only for first time.
Space around `=`. Name variable the same as key.
It's certainly possible to build an up to date 32-bit pypy from source, the problem is that doing so requires 4GB+ of memory at a minimum and takes on average upwards of an hour. Obviously that's not a deal breaker per sé, but yeah it's generally recommended by the PyPy devs to stick with the binaries where possible.
just confirming

replace
  target='_blank' if thing.newwindow else '_parent'
with 
 target='_blank' if thing.newwindow else ''
-chH.PSendSysMessage("You changed [%s]'s display to %u.", target->GetName().c_str(), display_id);
+handler->PSendSysMessage("You changed [%s]'s display to %u.", target->GetName().c_str(), display_id);
Backticks `
The tuber gotta disappear for a while. It seemed to be the easiest way to make it not trackable for the boar and invisible to the player for the next 30secns (Will be "respawned" in UpdateAI).
Yeah, that is my intent.
On Nov 10, 2012 2:34 PM, "Moxie Marlinspike" <notifications@github.com>
wrote:

> In src/org/thoughtcrime/redphone/call/LockManager.java:
>
> > +
> > +    accelerometerListener = new AccelerometerListener(context, new AccelerometerListener.OrientationListener() {
> > +      @Override
> > +      public void orientationChanged(int newOrientation) {
> > +        orientation = newOrientation;
> > +        Log.d("LockManager", "Orentation Update: " + newOrientation);
> > +        updateInCallLockState();
> > +      }
> > +    });
> > +  }
> > +
> > +  private void updateInCallLockState() {
> > +    boolean horizontal = orientation == AccelerometerListener.ORIENTATION_HORIZONTAL;
> > +    if (phoneState == PhoneState.IN_CALL && !horizontal) {
> > +      setLockState(LockState.PROXIMITY);
> > +    } else {
>
> Yeah I'm just thinking the check should be removed all-together then and
> made an assertion (implied or otherwise).
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/WhisperSystems/RedPhone/pull/60/files#r2091610>.
>
>
See issue opened on php-lib to avoid these changes in future:

https://github.com/micheldumontier/php-lib/issues/12
Is this kind of configuration new in `Zend\Di\Di`?
More code!
```php
if ($user = $this->userIdentity()) {
    echo 'Logged in as ' . $this->escapeHtml($user->getLogin());
} else {
    echo 'Not logged in';
}
```
FWIW: we might want to add https://github.com/brightbox/object-factory in the future. Me and @mseckington have used it a _lot_. It's a bit like FactoryGirl meets rspec mocks.
Looks like yes, I should.
Change it by :

$container->setParameter('lexik_paybox.transport.class', $config['transport']);
This seems to need better documentation. It finds the number of "loop begin" instructions.

So the program "[[[]]]" has 3 nestings.

This program: "[][][]" also has 3 nestings. Are you just trying to find the number of loops? Or the max nest level?
the point is that when this gets added to the repo everyone else will pull your changes and everything in this needs to be commented properly so there is no question as to who did it and what it is there for. I figured out it was for testing, lets not leave it to chance for everyone else to figure out though.
Add ``// @todo use XPath to get element type.``
Yes, that can wait.
Maybe we should drop the ngx_http_pagespeed_ prefix and replace it with e.g. ps_ someday.  (But not today. Just food for thought.)
Oh, right. Hmm, I'm not sure what we should do here. I didn't check what Stata does. My inkling is that if it's 'all', we should just return the marginal effects at each observation and not worry about computing the standard errors. If users want to look at more detail of a specific observation, they can. What do you think?
For readability I would name this ``bandwidth``. Or if you really think that's too long, ``bw``. With multiple one-letter variables things become hard to read quickly.
I meant streaming in the sense that you're not loading the entire table into memory.  Instead, you're opening the file with `open` and reading and parsing each line individually.  The line `parse_trie(load_collation_elements_table)` doesn't pass the table to `parse_trie`, it passes an instance of `File`.
Why parenthesis around `GENDER_CHOICES`?
Please spell comments out. *Auth* -> *Authentication*
you're 100% correct
If you put this in TNArchipelLibvirtEntity, each entity will be able to manage it's own connection independently.
Are any of these classing going to be annotated with the event logging annotations?  If so, you need the aspectj configuration here.  See some of the other files in this directory for how that looks.
Using "Desvanecer" for fade everywhere;
I've edited this paragraph a bit to include mention of the `Add` (there being only two compounds seems to make sense to mention both) and also added a paragraph break - this is purely subjective, but I think whitespace between different sub-topics is useful.  Fr me it makes it just easier to read.  Re-edits welcomed.
can you make this protected?  also s/They/The in the comment
how does it know in which path it has to install the package ?
ok it seems to be using default ocamlfind directory, so it should work fine. Not sure about the non-standrad installation of the C stub, though ...
Oops, it's the third frame!
Let's address this in a subsequent pull request.

I'd suggest something similar to DeployCommand.waitForAppStarted, but where it's waiting for either the app to be unknown (is that a 404?) or for the app to report a status of ERROR. And like the deploy command, it could write out the app status changes that happen while it's waiting.
Could hopefully use `Collections.emptyMap()` or `ImmutableMap.of()` here. But not worth changing...
J'ai vu global license utilise aussi, t'es sur de toi ?
음 넥서스에서는 Play 스토어도 아니고 Play 마켓으로 뜨나요?
아 그리고 런쳐 아이콘에서는 Play 스토어로 표시되지만
영화/도서 구입 화면이 표시되게 미국 캐리어로 설정하고 프록시를 통해 앱으로 들어가면 제목 줄에 Google Play라고 뜨더라구요~
okay, I've adapted your code and updated the branch. That's way much clearer like this, thanks.
The correct locator is 'edit_profile'.
I'm sorry but i don't understand what i need to do...

Do i need to move this portion of code in a model ?
Do i need to use this line of code :
```ruby
@authors = Refinery::User.joins(:posts).select('id, username, count(*) AS post_count').group(Refinery::User.arel_table[:id])
````
If yes, i have an error : "Association named posts was not found; perhaps you misspelled it?"
Acho que deviamos parar de definir estilo no html, somente no css
add_cc and add_bcc are not aliases, they add to separate recipient lists and do not allow devs to attach names (just email addresses)

I'll add some examples for add_to, thanks.
ok, seems those things where all obsolete except for parentElement: body which makes the editor float or not.
I guess it is unlikely that we can have an alert for product without a stock record but it might make sense to check for ``StockRecord.DoesNotExist``. What do you think?
This behavior doesn't happen to me on the current HEAD of master.
Too quick to commit it seems - i'll look into this. Thanks
This function already makes a lot of unnecessary assumptions about presentation. I think it's fine with these modifications. When rollouts is used as a library they can just used the windows directly instead of calling into this presentation fn. Rebinding out is one line of code.
Can you please use `{}` ?  And this should use `inherits`
So, what are you saying? have another ctrl method used for "new injection" results?  say a little more about what you're envisioning
this line is redundant if you extend OBController
nit: 

      result.push({
        phrase: phraseList[i].phrase,
        prefix: queryList[j].prefix
      });
visibleBounds
Is there a reason you use a class here? I've updated your code to use just a function but wanted to check I'm not missing something before committing https://github.com/Stuk/sublime-edit-history/blob/pref/Edit%20History.py
Should the StoreType enum really be in an impl package? The public API (Store.java interface) returns this on getType()
what about ```null``` ?

The Pipe IMPL does this:

```
if (filter == null) {
            filter = new ReadFilter();
        }
        final ReadFilter innerFilter = filter;
        
```
right now I just used it in tests, I just thought that group.name will be much more clear then group.lastname for new contributors. 
also will be good to to find out place where lastname in group is used and replace them by name. I can do that.
А можно ли потребовать это от пользователя? Просто не ясно f или e...
Ни интерфейс, ни класс - не private. К тому же, тебе надо их вынести в отдельный пакет, чтобы правильно проверить. Тут ведь они в одном пакете с ShardingProxy
Кажется, тут надо проверять, что charAt(1) != '-'
String.IGNORE_CASE_ORDER
Вообще, это не очень правильно, когда пользователь реализует интерфейс Runnable. Что можно подумать, если в коде встретишь user.run()?
Search.__init__ sets search_page._page_title to the expected result ... you could use that here.
This isn't related to the user, it's the API credentials. I would create a new section that looks like this:

    api:
        key: <value>
        secret: <value>
I've removed the need for this class, and removed utils.cpp from my files.
Try `{{ q|default_if_none }}` instead
`DecistionTreeClassifier` and bagging ensembles of them.
I thought there was a better idiom for this but I can't remember right now :-/
I think you haven't addressed my comment here yet (it should say what the option does, not what should be passed).
use array2d from utils/validation.py
0.15 not 0.25 :)
@glouppe I coded up the feature importance procedure from ESLII (2nd Edition). Your argument sounds convincing though - I'll look into the formulas more carefully - maybe I missed a normalization step. Could you pass me the reference for the 'gini' method (is it Breiman's random forest paper?)
If you want you can delete everything relating to prodLDA; I haven't found it useful at all and it just makes the code more complex.
I'm not sure just wanted to check your opinion. For instance if I have a slow connection and click a link it would be good to get some sort of feedback that my page is loading otherwise I might click the link several times thinking nothing is happening.
Is
    [^ \t]
needed in this part?
    \\{(.*[^ \t])\\}
According to the example, it can contain spaces anyway. Is there any need to have "non-space-character}" to identify the end of the alias?
I've reverted that portion in 99a1daa
Also, `query_values` can be nil, so I'm going to fix that in my merge.
I get the feeling that these will always be read write..
I would just remove this line now. I think it’s something that we had when we only supported Public, because it was confusing to users that private didn't work. Now that it all works, it's what people would intuitively expect, and is not worth calling out IMO.
- Use `<classname>Coordinates</classname>` so that formatting makes it clear it's an actual class name.
- Should we use fully qualified class name in this context? Not necessary if you provide a small example; An example would be better to keep the text better readable.
- whitespace at the end of sentence?
Typo in old commit, glad I have tests now. ;)
Just noticed, this logic breaks down, if numRows < nodes..
i.e. lastRow can go negative

Need to check for that.
Please don't remove the upper bound as it would mean we cannot tag safely anymore
```itertools.groupby``` makes it clear where this functionality comes from (given the length of this file)
Any reason not to match above? Using ``password_changed`` here too?
As above:
So something like: 

```
ensure_resource('package', 'pyssf', {'provider'=>'pip', 'ensure' => 'latest', 'require', 'Package['python-pip']'})
```

would be acceptable?
Ok pour l'insécable, ça fait 1 an que j'ai arrêté de m'occuper d'IE7 et IE6 :)
Why did you add "intransitive"?
Yes, this algo is here to handle other than a flat list of arguments.
you should keep ``$this->fail()`` here (it won't be executed anyway as ``markTestIncomplete`` uses an exception). and marking the test as incomplete should be done at the very beginning IMO
can you wrap the lines please ?
You might want to take my motorcycle loving code out LOL
We do not want to change the behavior. Also, the emulated env variable still count for the emulator.
Thanks, I needed someone to look through all these old commits with me. Sorry they're such of a mess.
I'll get to these soon
Changing the default to false is also fine. The front-end developers will just live with creating a file in their properties directory. It's no big deal, they're used to that sort of thing from OAE anyway.
Is this only used on GameDay right now?
http://abstrusegoose.com/467
`contains?` returns a boolean, so no `if` is needed.
All of the of for you.
false is not the same as 'false'
header ro hazf kon va be nav hatman class bede
shayad nav dige ham dashte bashe yek site
Document each field.  Get doc for this `struct` from the [wiki](https://github.com/AnalyticalGraphicsInc/cesium/wiki/Fabric).
Depending on who goes into master first, this is going to have merge conflicts with my jitter changes, which introduced a `us.update` to handle all camera/frustum changes.
i was actually just about to recommend that. changed.
What is slightly - and does it matter in the big picture setting of this code?  If so, keep it.  If not, the more concise version is probably cleaner - I suspect I am inconsistent with it elsewhere in the code.
Given that the rest of the code uses `AnimationCollection`, why break the abstraction here and go behind its back to `Tween`?  Why not add something to `AnimationCollection` instead and use it here?
The used pattern is to generate component id using `heat.exe`
#RGB definitely works in our android implementation.  Not 100% sure about alpha, but i believe so.
Mobile Web does not support up or down because none of the other platforms did at the time it was implemented. We should probably file a parity ticket.
Must be specified at creation.
This should be a function declaration, not a function expression. Also, local identifiers shouldn't start with an underscore
i have no problem with renaming that. will do
I like error 37, can we change it though to include a description?  IE: "Error 37!!! (window already exists in stack)"
The methods work on IOS. However there is a clash on the animate property with the `animate` method on View. So the property probably will have to be renamed. Will set up a new PR and update docs accordingly. This is however outside the scope of this PR
FPO....
I thought the header was Cache-Control? Are HTTP headers case-insensitive?
Same as above, `proto` should be `this`.
Do you really mean to cause the IPv4 hosts and IPv6 hosts to log to the same access/error log files? I'm not familiar enough with apache, but I expected each vhost to have it's own log file.
Changed in 7c39ba8cf534432881178e9f88d3130ca532ad64
Why not  ```if metadata.any?``` ?
hmm .. i keep wondering if there is a better pattern to be able to add things on top of a service definition and then alias the new version to the old service name ...
I added another 'if' at line 28 to only display the current items in the breadcrumb
Ok, I saw KnpMenuBundle has been updated. Above issues have been resolved, thanks.
Changed that to "Container with data."
@ogail says "yes"
It sounds like Oases is borrowing some of the source files that Velvet provides. In that case, it would make sense to add the Velvet tarball to `sources`.

However, then it wouldn't make sense to still list Velvet as a build dependency (because then you require that's it also built by itself). 

Is this documented somewhere? Can you point us to it?
Why shouldn't it be no lines separating them ? It may improve readability to regroup the `use` by vendor name, separated by a line.
Didn't we have a separate function for this before?
this is logically equivalent with if (txInfo.isMarkedForRollback()), which is easier to read. if you change this then the result of txInfo.markForPreparing(); can be ignored/removed.
Right, a correlation-key cannot be a requirement for the gateway to work correctly. It can be required for optimized behavior, and the log message can indicate that. Obviously, if someone does not have a consumer that would support a correlation-key (those that always pass JMSMessageID back as JMSCorrelationID), they still must be ABLE to use our gateway and not have reply Messages lost or going to the wrong consumer.
The bottom line is you can't use arbitrary threads in WAS, especially when using container transactions.
New files should have 2012
Can you indent the new `@return` descriptions to match the indenting of the existing `@param` descriptions? It is not a big deal, but it helps keep the code comments looking clean.
This is redundant. It's probably better to use a condition here, rather than attempt to to do 4 redundant subs on a string.
I think the change by @Freeaqingme reads fine.
Please test this method
Great! It makes no sense indeed to test py3k with Django versions that do not support python 3.
Hmm...IE has supported `Array#push` since version 5.5; 5.0 and earlier didn't implement most of the ES 3 array methods. I can revert this if you'd like, but it's worth noting that `push`, `pop`, `shift`, and `unshift` are already used throughout the library.
I don't believe so.  We just need to the delegate to nil in case their is an outstanding async operation happening with the webview, we don't want it call back on the delegate (which is the instance just now being deallocated).
The syntax for raising the error would be:

    raise ValueError("Command failed ...")
Virer la majuscule.
On 10/29/2012 03:40 PM, Jim Porter wrote:
>
> That's not true, unfortunately. Gmail uses "Mail:DEFAULT" as the 
> CollectionId for the inbox.
>

Ugh.  Then this is fine!  I'll ask your advice on what to do about that 
for my implementation on the bug.
what is the idea behind this if statement?
As opposed to properties, these should likely be documented as events and include the expected signature.
Terrible hack #2
`var i_gid = $sn.getAttr($(this), 'gid');`
I actually meant that we should be pulling this from the JSON if it exists and having defaults otherwise. But seeing as its not being used atm, it'll just be done when and if we ever truly need versioning
After some investigation we can do the following on the overlaywebview:

1. Keep the "viewport" tag
2. Add the following call to the webview: webview.setDevicePixelRatio : 1
They sure look like constants aren't they? You guys don't alter them during the course of the context menu.
Are voided/retired items exported?
I think we could use date_format and datetime_format to speify if medium or short http://php.net/manual/fr/class.intldateformatter.php when we choose localizeddate.
ups, actually this is coming from a second change i am in the making. the route does not yet exist in the commited code!
I'm a little worried about where SituationQueryBean is headed in terms committing to an API.  Part of this is on me for not specifying this bean more clearly from the start.  I've got some thoughts on improving the interface which we don't necessarily have to address now, but I'm wondering if you guys have any flexibility on making changes here?
if you close the connection, maybe you can do a context manager that allow you to do:

with msql_connect(...):
   cur = conn.cursor()
   ...
if window.console?