labofsahil/github-event-dataset-2012 huggingface dataset:
Add a /api/0.6/user/NNNN API call

This branch adds the `/api/0.6/user/NNNN` API call for requesting public information on a specified user in machine readable form. It is based on the existing `/api/0.6/user/details` call with some extra details added. Example output as follows:

```xml
<?xml version="1.0" encoding="UTF-8"?>
<osm version="0.6" generator="OpenStreetMap server">
  <user id="1" display_name="TomH" account_created="2012-09-19T18:33:34Z">
    <description></description>
    <contributor-terms agreed="true" pd="false"/>
    <roles>
      <administrator/>
    </roles>
    <changesets count="0"/>
    <traces count="0"/>
    <blocks>
      <received count="0" active="0"/>
    </blocks>
  </user>
</osm>
```

The extra fields have also been added to the `/api/0.6/user/details` call, which also retains some extra information (home location and languages) which are hidden from this public API call.
1.8.0rc1 release prep
Fixed License and Copyright

Fixed License and Copyright
ruby string methods

basic string methods for working with georgian language
For Issue 999:

The progress bar in dark mode is not displayed appropriately, as described in issue 999. I made two minor changes on UI, one on deck select table and the other on deck view dialogue, so that progress bar can be displayed in dark theme.
Fixed the github fork me icon
Proxy support

Now with less tabs and more spaces! Woooooo!
Integ b1 b2

Branch1 and Branch2 changes
[TIDOC-389] APIDoc: UI.ActivityIndicator rewrite
Config especial para o Curto Café

Adicionei as informações de Ip, DNS, host e subnet manualmente, já que DHCP não estava funcionando.
bug in determining if iteration is between given dates

fixed bug: if iteration is spread over 3 months, then although the report date is between start and end date, it is calculated as being not in range
passion: add settings overlay for dock settings

Please pull the overlay into your passion tree. 
thiasB (xda)
Automatically add client_id to login url when AppId set in FacebookClient

Fourth times a charm. :)
Issue 47

Seems like kevin had some old commits on master, which are not part of unstable and are part of this pull request. Let me know if I should try to remove those commits from the request. Thanks!
LPS_24760 Persistence beans have an incorrect state when using the setup wizard

@shuyangzhou, can you also please review this? thank you!
Added UTM conversion test to 'testNav'
Add actual vhost.pp configuration for 11.1

This previously put the new init.pp as vhost.pp, and no vhost
configuration was actually created. This file already existed
in exercise 12.4, so I just copied it.
Rakefile update

Updated generator task to act like sproutcore-statechart. Added a task
to clean up build artifacts.
Added code fencing

I copied the output of GH's fenced code, but also added a `lang=""` attribute so you can color the code with whatever script you want. I also did some basic tests and everything else seems to work fine with the other ways of writing in code (like single ` and 4 spaces).
Added space for networkX graph. Can be loaded from a file.

This does not update yet, but that's next on my to-do list.
Feature/sortable inlines

This feature adds drag and drop sortable inlines. To utilize this feature a model must have a position field such as the following.
`position = models.PositiveSmallIntegerField("Position")`

The admin for such a model is as follows.
```
class MySortableInline(admin.TabularInline):
    model = MySortableModel
    extra = 1

    sortable_field_name = "position"
```

The `jquery.sortable.js` file is 3k uncompressed, and could probably be minified down to 1k. A custom jQuery UI build with only the sortable plugin comes in at about 40k.

If this implementation is acceptable I can update the README with documentation about this feature.
Ipn info

This displays in the admin interface what I consider to be some of the more important info from the IPN notifications.
Update for Java.test to not sign the build

Fixes #9
Declear FOSRest dependency.

There is an undecleared dependency to the FOSRest library, here's a quick patch to fix that.
Portuguese and German localization

Hello,
This is the up-to-date translation into portuguese (This time with the right files...), also added the translation of your newest features into german.

David
Sdk download + integration test

Added automatic sdk download from github and integration test
Update README with note about GCC for OS X

Some users need to set environment variables to use the correct 
version of GCC to build native versions of certain gems, including
but not limited to rails and nokogiri.  This would just add a little
footnote to the README to include instructions for Mac OS X users.
Update Resources/templates/CommonAdmin/ListTemplate/actions.php.twig

Since you may add custom actions in your generator - their translations should reside in custom i18n_catalogue (or default "Admin"), not in common catalogue "Admingenerator".

Also, introduced translation of action.confirm message.
Test fixes

fixed two issues in ProxyQueryTest, 
fixed namespace in PathInfoBuilderSlashesTest and ListBuilderTest
Gets rid of false positives inside lengthy error messages

The default file regex pulled up some false positives inside verbose error messages, resulting in bogus file names.
New post

I think this pull request is how I get my post over to your repository.  One potential problem is that my computer saved the file as "filename.markdown.txt" when it ought only to have been "filename.markdown".  I'm still not entirely certain how to change the file type because OS X can be a little funny about file types.

This time, I put it in the "posts" folder.  Is that where you'd like me to put them?  or is "drafts" more appropriate?
Fix generation of desktop file
Twitter theme: few little improvements

Hi,

I tested the _feat-twitter-theme_ branch and I must admit that you've done a pretty good job, congrats! The only modifications I made were to change/add icons. I think it's even better with these changes, but I'm aware that it's very subjective. Feel free to test it and accept it or not :)

Moreover, I found a bug with collection forms. You just can't add new items. It seems that the JS part of the collection type generates correctly the fieldset containing the new item, but this fieldset is empty.
Fix for #28

This fixes bug #28. Updated, original request #29

Changed some lines of code to let the user delete the playlist-folder via context menu. Fixed comments and style in LukasSkywalkers commit.
Thanks for letting me know. I've fixed it.
This change is wrong.
Fixed https://github.com/rails/rails/commit/92a7296a5deb109d054b32d96fa1a74f0859a81b
If PHP code is producing errors with register_globals on you are terrible terrible programmer. If you are using magic_quotes you are simply stupid.
This should not really happen, ExplicitThis is not allowed on MethodDefinition.
Haha, I completely forgot about that. Alright, I'll revert this next chance I get. Until then, you can always use ^[q by hand.
It's really hard finding a good spot for this. The window manager will usually take all the good keys. They don't get sent to the browser. 

Alt-` might work. It look like xterm sends `à` to the terminal on alt-`, but that's without 8bit input. It also has a place in tektronix emulation but we don't have to worry about that.

That also reminds me. alt doesn't currently send escape for 0-9, only a-z. I should add that too.
As long as this branch is not Beta, would you mind changing the BUILD constant to something like `pre` or `pre-beta1`? This should be done for all non-release versions (beta, RC or final) to differentiate trunk "builds" more easily form release builds. 

The MooTools project sets its equivalent value to `%pre%` (AFAIK) and updates it to the latest commit ID on every release (and resets it immediately after the tag has been created and the files were released). 

If you're concerned about ER compatibility, would you mind adding an extra check somewhere (might be enough to modify rep_base or rep_client) so that this isn't a problem?
there is close-to-complete overlap with the Deadline class, but that one also is accessible with a new DSL:

    import akka.util.duration._
    val deadline = 10 seconds fromNow

so maybe it would be best to just merge the two?
It is really disappointing that here goes discussion about his personality and some even try to judge him instead of focusing on the problem he showed.
I didn't do it it was Breu... I didn't even look at it :)

Regards,
Jason Cannavale
(210) 312 - 4977
jason.cannavale@rackspace.com


On Jan 10, 2012, at 19:31, "Justin Shepherd" <reply@reply.github.com> wrote:

> OMG.. i am yanking your commit rights to this repo.. that way you have to pull request and I can DENY!!
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/rcbops/kong/commit/544c27f5524fd1dc734f06afaa3c7f96815ee14f#commitcomment-855415
Is this working as expected? Can I "merge" it in? :)
resolved after updating from bn 0.4 to bn 0.5 the original error was:

```
AttributeError: 'module' object has no attribute 'nansum'
```
this should be something along the lines of URIUtils::GetFileName(strFile).CompareNoCase("index.bdmv") == 0, no?

(Sure, unlikely to have it anywhere else, but I've seen sillier things occur...)
Thanks Atry.
No, not really, I just tried to pass the specs :) So we would move validation to "to_der", just before the point where the value finally gets encoded?
Official thanks for @butu5 for all his help with both Driver design & Code.
This seems good, but can you just pass the event in here?

    return target.send(actionName, event);

Now there's no reason to send it wrapped in an object.
Already did :) 4945208a2965c2772bb3910a7528cb9dc19c37c2
per the mailing list it's already fixed upstream -

```
git clone git://github.com/ivmai/bdwgc.git -b release
cd bdwgc
git clone git://github.com/ivmai/libatomic_ops.git -b release
autoreconf -vif
./configure --enable-cplusplus
make check
sudo make install
```
sigh. done.
Thanks for this.

wrt bouncing, the idea is sound but its just too much in 85.
My assumption here is that the toString is purely for debugging.
i do agree on the fact that the network traffic limitation may or even is on the wrong level. What would you propose instead?
Call the field's `setValue` here instead of building up an object to pass to `setValues` (which will iterate through the fields again).
for some reason this changeset introduced/uncovered an error where barSettings is not defined is being thrown from https://github.com/ajaxorg/cloud9/blob/07057304faf019b395b5da634e87747f09d94bc8/client/ext/settings/settings.js#L83
This is exactly what I wanted to do with the original repo. Maybe make a pull request?
`:arg ids`
Venugh, what the source? anyway I agreed with you, if someone copy-paste a code he must write author
I was toying with adding an explicit call to `objc_autorelease()` (which would have required declaring the function since it's not in the headers), or passing it through a local `__autoreleasing` variable, but after asking on the clang users ML at work, I was told neither of those approaches were guaranteed to behave correctly and that the only way to do this was to turn off ARC.
Can probably be `val ans = cp.flatMap(_.metadata get Keys.analysis)` if you prefer, although this is fine.
looking at the helper, is there any reason we can't define "iterate" and "generate" as protected methods and set "page_groups" as an instance variable for the duration?  I think that would do away with the inline definition of procs and such, but maybe there's an obstruction I haven't noted
this is kind of unfortunate.
Hi @andreasgal - could you explain a bit more fully what the problem was with IndexedDB? Should we not be using it elsewhere in Gaia either?
seems a little inestable in this version :S

I'll send you the crashlogs!
yes, on pve the resilence doesn't exist :S
this broke behavior of `any_of` for me. note below that in 2.4.1 the `$or` clause gets split into two separate hashes now, which is a totally different query from when it's one hash.

2.4.0:

~~~ruby
1.9.2p290 :002 > Contact.any_of [{a: 1, b: 2}, {c: 3}]
 => #<Mongoid::Criteria
  selector: {"$or"=>[{:a=>1, :b=>2}, {:c=>3}]},
  options:  {},
  class:    Contact,
  embedded: false>
~~~

2.4.1:

~~~ruby
1.9.2p290 :001 > Contact.any_of [{a: 1, b: 2}, {c: 3}]
 => #<Mongoid::Criteria
  selector: {"$or"=>[{:a=>1}, {:b=>2}, {:c=>3}]},
  options:  {},
  class:    Contact,
  embedded: false>
~~~
Yup. Both on my blog and the InfoCamp blog. I always thought (maybe @onli could chime in on this, although it's not really a discussion for GitHub) Bayes was supposed to be used as an "addon" to Spamblock.

Anyway, I applaud Commentspice as an "alternative" and have added your code to 2k11, I just didn't commit it to the s9y repo yet. This will go into the s9y repo with the next 2k11 update or in case of a 1.6.1 release.
You probably want something like ExtractFileFromPath here.
you can use `eq_`
更新测试一下吧。
My level of pythonis very very limited :p
Awesome work - can't wait to try this!
good point... I'm sure it will work in windows and linux. Never tried in Mac... do you know if it's supported?
I think you want something more like (produce-digest digest :digest seed :digest-start (* i 32)) to avoid allocating.
It is white text on black background. White rectangle is cursor. Try input some value.
btw - bordered text is no nice - i did not add it in bitrate viewer too.
I guess maybe personas weren't excluded before. but they will be included when the include personas setting gets turned on.
Ah, this seems to have som implications. Didn't look into the details, but there was an "out of scope" exception in IE7/8. Gonn ahave to look a bit more into this then.
DDDDDDDDDDDDDD
How about:

```javascript
// 1
parent.convertFromLocal( child.position );

// 2
parent.convertToLocal( child.position );
```
With this commit, the boost library is now required to compile the project.  It is available here: http://www.boost.org/doc/libs/1_48_0/more/getting_started/index.html
да, вполне
That's because you need to use CMAKE_BUILD_TYPE as well as CMAKE_INSTALL_CONFIG_NAME. I can't see a way of getting around this?
I'm not a huge fan of nesting control structures but if you must then perhaps name this ``_notify_if_failed`` so it's more obvious
ыыываывааываыв
Niiiiiiice..... I'm not super awesome at figuring out how to build it. When will the straight-up js versions (in dist/) be available?
GPL and MIT.
因为www.zlib.net无法访问。
I was reading up on JAVA and they were saying that the fewer catch's you have, the faster your code will run, so that's why I was checking for null and letting the caller receive a false from the method.
We only want this to occur when users click on the toggle-all  checkbox right? That's why I am refining this here.
Merged as a part of other PR
You have to check for ```options``` before evaluating ```options.noAbort```, because ```options``` may be undefined here.
I dont think we need this line.
Something in this or the commits before (relating to ItemStack) has broken WorldGuard.  The relevant bit of WorldGuard that is having problems, is:
```int slot = player.getInventory().getHeldItemSlot();
            ItemStack heldItem = player.getInventory().getItem(slot);
            if (heldItem.getAmount() < 0) {
                player.getInventory().setItem(slot, null);
                player.sendMessage(ChatColor.RED + "Infinite stack removed.");
            }```

It is throwing an NPE there when it calls getAmount().  Is this something WG will need to update or is it a problem in Bukkit?
:o: Please consider using EventEmitter2 as the base of the rebus object. It will play nicely with other node.js concepts.

Other than that it looks good.
this line can now be removed
I wonder if we should allow codebaseGenerator to return a deferred.
Thanks, confirmed that it still works on my dev environment. nice job!

On Sun, Jan 29, 2012 at 11:45 AM, susanafong <
reply@reply.github.com
> wrote:

> Also changed variable names in BamftActivity to CamelCase
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/legendaryfox/bamft/commit/018537e938527ccec62534bba04b849cdcce47bc#commitcomment-912263
>
No problem - Had some spare time this morning and thought I might as well get your changes in :)
Any reason to keep the init versions of these?
Hacks, makin' life easier since origin of time axis.
Since we're now including 2 config files, using variables instead of constants might make sense - so that the 2nd config file changes a value set in the 1st one...
please don't add includes that are not necessary in header file.
do instead:
class vtkMRMLFiberBundleNode; 
...
Да вообще холиварная тема.
This seems risky. You don't ensure that the content actually is of the form application://<foo>.desktop. Use a combo of strstr and g_str_has_prefix / g_str_has_suffix
I think rbx-18mode is running under rubinius 2.0
Refactor: keyFor is silly. Should have a method that sets Results instead.
@Marak you might wanna look at github.com/pksunkara/nosqlite
Nice! Go on!
If you want to switch to using linux, uncomment the //#define LINUX
I don't see any place that redis is actually being connected to. That may be by design, which is fine.  Am I missing something though?
musst du bencoded zurueckgeben?
Yes, a class hierachy is the OO alternative to a type code.

For each field that needs to be validated you can just instantiate the appropriate type of validator. The validator can probably be made capable of generating sensible localised error messages.

My thoughts on a custom widget were to create a new abstract subclass of gkt.Widget which wraps another widget and provides validation state and validation events. Then there will be very minimal wiring together for the UI - there just needs to be the same validation state change listener added to each widget which will add or remove an error message from the UI. As you point out though, either there will need to be multiple error messages displayed at a time or, while errors still exist, previous error messages need to be restored after errors are resolved so this approach requires a bit more thought. Maybe the validation state change handler needs to iterate over all known inputs and revalidate them when one of the inputs validation state returns to "valid". The Eclipse IDE presents input validation messages in this way (one message per screen, updated as you type, relevant to the earliest invalid field on the screen) so it could be worthwhile looking at the source code to see what pattern is used there.

Whether we use custom widgets or not though there will still need to be the string input validation type hierarchy so I suggest going ahead with that and seeing how it looks.
Why not use the new ActiveSupport::Concern?  It gives this stuff for free.  http://guides.rubyonrails.org/plugins.html#add-an-acts_as-method-to-active-record

# yaffle/lib/yaffle/acts_as_yaffle.rb
module Yaffle
  module ActsAsYaffle
    extend ActiveSupport::Concern
 
    included do
    end
 
    module ClassMethods
      def acts_as_yaffle(options = {})
        # your code will go here
      end
    end
  end
end
 
ActiveRecord::Base.send :include, Yaffle::ActsAsYaffle
Yes, exact same issue. I knew it was there in issues but couldn't find it yesterday
OK, OK. Acá tenes mas para probar =) Estoy hecho una maquina =D =D =D 
https://github.com/Shelvak/print_hub/commit/7209974d336c44a2b5580e9dfa28a7fe7de6e6fb
I don't feel totally comfortable ignoring Windows-Eyes as those users may need the text to read more than anyone. The market share may not be significant but it is a market with few other options.
Looks like the o variable is referenced outside of the if() statement it is defined in, which can cause the nestedSortable to break if the .scroll property is set to false. I should have split this into a separate commit.
Time ago, the function was called GetCreatureTemplate(); and someone change to GetCreatureInfo(); 
Why we go back that?
No need to introduce a deprecated variable. ;-)

You're probably emulating many of the protected or public variables that are found all over the PMS code. The reason those variables are marked as "deprecated" is that their public getters and setters were introduced later; plugin developer could be using those variables.

By marking them as "deprecated" external plugin developers are warned to stop accessing the variable (Eclipse will display the variable in strikethrough font and generate a warning), because somewhere in the future the variable will become private. So it's in there to allow developers some time to adjust to the new code.

This, however, is new code, and you nicely provided a getter and setter. No danger for losing backwards compatibility, no need to warn people.
Simply make the variable private. Force all code to use the provided getter and setter. :-)
Also use SMART_EVENT_LINK =)
sorry, pc got freeze, don't know, why commits has been overloaded...
i had full t9 with raid gear on every slot and still every dungeon was blocked. even the 3.0.3 dungeons which has no itemlevel requirement
Dette skulle ha vært 5 px
Need to check each function in turn.
I think perhaps it would be better to put the `isInterface()` check inside `isInstantiable()`

Can you check if that works and update your pullreq if it does?
Aren't you missing Category here?
Suppose it should be "Category.column_names.include?("ancestry")"
https://github.com/plone/plonetheme.sunburst/commit/af8d7984e75f141e38acce9664b82b2b5fed7e15
Same as before - this shouldn't be in BattleManager but in Utilties
yeah, it was a dirty hack - I'll do it properly instead ...
Shouldn't this condition come first?
Can't you use flake8 directly here, instead of pyflakes +pep8 ?
I forked and cloned the repo and then tried to run the tests. I got 2 failures:
   "filters date.should accept various format strings failed (3.00 ms): Error: Expected 'Sat, Sep 3, 2010' to equal 'Fri, Sep 3, 2010'"   
and
   "filters date.should accept default formats failed (2.00 ms): Error: Expected 'Saturday, September 3, 2010' to equal 'Friday, September 3, 2010'"   

My local timezone is +8 (Beijing). I set my timezone to -8 (PST) and it solved it. I figured the getDay function was using my browser's timezone instead of the specified offset.
You also have to keep this so some of the tests don't break.
Sehr gut, finde ich auch noch mal klarer.
I think I added a class last night called StrUtils, perhaps that stuff should go in here too?
change '&' to '&amp;' to complete w3c validation. also change .js file, to make reverse replace.
oh, also, an example of it in use: https://github.com/koto/phar-util/blob/master/phar-build.php#L175
What if a deployer puts spaces in between for readability? Perhaps you should add a .strip()
Ha, funny. Good catch.
will do that. but for now the resolver tests produce fatal error, i first want to fix them :-)
Alla fine hai trovato il perché non sono riusciti a farli comprensibili a priori?
Can't open the zip file..No clue why. Think you need to install gzip or something
Right, that sounds like BVAL-208. Btw, there was recently a post on the forum where the user could have made use of such a group translation.
Vendored gdata will be only used in spec.
gemspec does not support Bundler syntax.
No. I am informing you of these things so you start seeking a second opinion before approving your own submissions. The commits to fix recent code speak for themselves. I am just offering a solution.
Merge pull request #94 from swas/master
Fixed up correctly .foo extensions to work when get arguments provided
This looks much better to me.
Thanks for the contribution! I'm curious to know if you are using globusonline, or just using verified_https.py. I could fork it off into a new project if there is interest.

The above patch doesn't work with 2.6.x, and there are some minor style issues. I'll accept a pull request if you make the above changes. Thanks!
Good Job! Keep it up! By the way, I can help you to do it better. If you do not mind of course.
Check against any of the popular code standards and you'll see that most have a note about not abbreviating variables. It's basic best practice.

A name of `$github` or `$github_domain` would suffice here, and would not make any lines go significantly over the (loose suggested) limit of 80-100 characters (the fact that you've stated a range, means it's not set in stone).

However, since this plugin is all about Github, and since Github aren't going to be changing their root domain any time soon, I think this extraction of the domain is actually pointless and hurts the code by being less readable., not mention the (albeit extremely tiny) performance hit by now having several strings in double-quotes instead of single. You're not making the domain filterable or doing anything special with it, so there's no need to extract it. A more likely candidate to extract would be the project name portion of the URL, as that is more likely to change than Github's domain.
```javascript
queue.drain = callback;
```

If you use `drain`, the build will not continue until all the items in the queue are complete (so things don't get out of sync) :)
This line was taken from the phpbb_style class. Basically, $user->theme['style_parent_tree'] is apparently a string of slash-delimited style names, so we get them in order from top parent down to the most recent parent

That being said, I am not sure how to make a tree of styles using style inheritance, so that will still need to be tested. What I do know is that it worked for getting the style in use as well as the "all" files.
On further consideration, I think this is assuming too much. I'll take it out.
Sold! You had me at "¿".
I worry about this using a "private" API which is not documented. I'll have to check to make sure it is okay for us to show using such APIs on the website.
That wouldn't work, since all the changes would get nuked by the `updated -f` and `reset --hard`.
So the ticket includes which classes need to be converted?
How can this be right?
or instead do:

  case Terminated(`guardian`) => ...
  case Terminated(other) => terminationHooks -= other // no need to check presence before removal.
boundedness does not change ordering AFAICS
Add type annotation to not-private members.
you can still create a ticket if you fear that you would otherwise forget it (I know I would)
From a quick glance, checking the uses of "Status" trait it should be fairly quick, just remove Status, Fail and Invalid, see what breaks, introduce InvalidAssociationException
_validateNotNull checked only empty string values and allowed to save NULL values without error message in case when form field return NULL value (for example DatePickr).
why not just add a `%s` to the beinning of this and remove the need to enforce that the client specifies that on the command line? (they prefix because exactly that, just a prefix).

`format = fmt.Sprintf("%s%s:%s", c.prefix, stat, format)`
i was leaning towards forcing the user to embed the hostname if they wanted it in there rather than trying to bake that functionality in... thoughts?

in our case we have this nice "short_host" helper :)
I would separate out the do-repos from the `setup-datomic!` function
datomic.clj line 32
I'm using it with update-in
Good spot.  I'll fix that.

Do change it here though outdated documentation is not an excuse -  This has been repeatedly brought up in pull reviews

thanks
Given that a user of the ProgressReport class will likely do:

progress_report = ProgressReport()
progress_report.build_progress_report()

Naming the method "build_progress_report" seems a bit redundant.  ProgressReport.build() seems sufficient.  But maybe that's just me.  No real objection here.
just declare these where you assign them and drop StringUtils::EmptyString
Thanks - have added a commit on top to take care of that.
Ook hier, spatie voor en na =-teken.
I don't understand this line. `len(self._pool) <= self.min_conn` is ok. There's no need to limit the connection that are needed to reach the minimum, but what about `timenow > timeout`? Are there cases when this is true? W

hy would the creation of connections be delayed? To run a batch of queries the same amount of connections are needed and sometimes new connections need to be created for this.
Backticks for fixed width text. Use with command names.

```
After that is set up, `cd` into `emrio` and run:
```
Do we really need this enum ? You can use the TransactionAttributeType annotation directly in your plugin
How about "Toggle background color"?
Hi, is it in fact a typo? :) Cf. http://en.wikipedia.org/wiki/Operating_system
OK, that makes sense. Was that changed when bin/behat was refactored? The projects I have, have this file in bin/behat itself.
Besides, `node.log` doesn't relate to `node` itself, but to the app.
This class still needs to be renamed to `TUIActivityIndicatorView`.
"Each guard" => "Each Guard plugin" (for consistency with the rest of the docs)
"onw" => "own" ;)
OK, can do that.  But how do you explain line 3 of ingo/lib/Script/Maildrop.php:
https://github.com/horde/horde/blob/3375cf1e6e2bfe152c884c4a9eee50f5ca7bc2a6/ingo/lib/Script/Maildrop.php#L3
tabs were included. Fixed.
I'm just following the style of this file. Most methods that start with a _ are public.
The 'N' seems extraneous.
тут сделаю
Couldn't this simply be `self` instead of `Rails.application`?
Done in https://github.com/0x000000/rails/commit/861630777affc03428816e89dd294f0515bbb976
@steveklabnik I don't plan on doing additional work on this patch.  If that means you won't merge the patch due to style issues, that's fine with me.  I'm not an ActiveRecord user, for obvious reasons.

@evanphx most callers are going to call hash_rows, which is going to use the columns as hash keys (https://github.com/jeremyevans/rails/blob/3aef5ce9b35a4659379201eb6bb1dba355a83ba4/activerecord/lib/active_record/result.rb#L57). It's possible to move this map call into hash_rows if you are worried about affecting the other cases.  Also, this should not allocate any additional strings, since String#freeze just modifies the object's flags in-place, it doesn't allocate another object.  However, Array#map does allocate an additional array.  The only situation where this change allocates more objects is in the case where 0 rows are returned or hash_rows is not called, in which it allocates 1 additional object (the array).  If a single row with a single column is returned, this will allocate the same number of objects (an array object instead of a string object).  In all other cases, it should allocate fewer objects (columns * rows - 1 fewer).
@carlosantoniodasilva thank you for reviewing. I'm going to fix and update it.
ah, yes! i had copied/pasted an example from traversty that included it and i didn't catch the semis removing!

:beer:
You're right, I should at least flesh out the comment (I hadn't realized the PSA when I wrote this patch) but how about we find a second place where we need that method before I write it. It's not that I don't think there is one, but I don't know where it is and in the absence of that understanding I'd rather keep the code localized.

Another complication is that dealias goes "all the way" (modulo widen) which means alias -> alias -> type skips the middle. If we're returning an iterator of types I feel like we have little choice but to do single-step beta-reduction here. In fact, I suddenly realize that this method also requires that, doesn't it @adriaanm ? Otherwise we are potentially blowing right past the alias with the matching type parameter arity, even now.
confusing --> could also be interpreted as "annotated with @specialized"
Ok, `ControlThrowable`s certainly make sense to catch in certain contexts. I'm wondering if `NonFatal` is the best name for such use-cases.
I think this is being done in a couple of places -- it would be good to unite them all at one point.
We can have ARTIFACT and isArtifact in our codebase and expose isImplementationArtifact in the API, okay?
It is strange that `CoordSystem` can reach into and affect a `Patch`

Also, apparently Patch is mutable? This is contrary to general SymPy practice. 

I wonder if there is a cleaner way to do this. Nothing comes to mind.
docstring + doctests missing.
Is this the right thing to do if  `is_real` is None?
Should it be in a finally block?
I am just returning a Tr in case I don't find _eval_trace. So, I couldnt possible do a doit() on Tr again.
Please leave the single space.
btw, ordinary Bell polynomial existed and in the mpmath library.
`return (string)$this->value;` is enough.
what if we've logged more stuff while the request was hitting the server?
Am 20.04.2012 21:24, schrieb Gustavo Boiko:
>>   	for(int i=1; i<passNum; ++i) //progress pass, the offset start from 1 as it is 0 based.
>>   	{
>> -		if(scene->getSignals()&  Y_SIG_ABORT) break;
>> +		passInfo = i+1;
> Just being the boring reviewer here: can't you increment the passInfo after the sigabort handling? then you don't need to decrement it there
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/YafaRay/Core/pull/12/files#r715193
>
Hello Boiko,

i have updated my branch and followed your suggestions:
https://github.com/OlafArnold/Core/commit/144c1642a4c9b5bbda2d405e324448962793bbdc

Bye, Olaf.

-- 
Olaf Arnold
(Lokaler Administrator EDITH und Vetretung Dienstplaner MFL)
VB Oberbayern
Produktionsplanung (P.R-BY-OB B1 Ar)

DB Regio AG
Bahnhofstrasse 4, 83395 Freilassing
Tel. +49 15158408189, Fax 08654 471-512, intern 9636-
Mobil: 0151 12541 372
I guess that if the pipe is broken, the return code should be 1.
Is "returns" magic or convention? Forcing all descriptions to start will "returns", particularly as one is trying to be concise, sometimes leads to awkward wording.
this should be resolved now in this request.
by Silex, not by Symfony
should be `setApplication` as it is the application you are setting and not a `Container`.
You mean `addListenerService($eventName, $id, $method, $priority = 0)` ?
FreeBSD's cut does not have long options, therefore common for both are the short ones.
The check with `config.semaphore` is indeed not pretty, but I wasn't criticising that. I mean the `setTimeout` to try again every 13 milliseconds. Maybe it can use `onload` as well? Or one QUnit's own callbacks?

QUnit.load starts with running callbacks for event "begin". To ensure onload is finished, maybe try this:

```js
if ( config.semaphore === undefined ) {
    QUnit.done(function() {
        // This is triggered from top of QUnit.load, go async to let that function return first
        setTimeout(function() {
            start( count );
        });
    });
}
```
It is a bit more code, but should be a lot more precise and less "polling".
I'm using ScannableUtils.generateDigest in the production code, but the helper function in the tests was more to verify that I've got the logic correct in the tests (and by implication, the code).  Hopefully the naming's clear, but if not, I could rethink it's use.
What kind of Rules do we have?  I believe they are all LoadRules, no?  In general I don't like instanceof checks, they are often used as a replacement for polymorphism and if you can't do what you need with polymorphism, then it's a good time to step back and double check your abstraction, 'cause it's probably in the wrong place.
So, what is it compared to?
I note that `integer_type` has `integer`; so it's probably not the lua `type` function....
Ok done ;)
Per-row event again?
I don't understand what you mean here, the event is to inject additional "editor options" _(like BBCode enabled/disabled, smilies enabled/disabled, etc)_. So the name covers that pretty well.
It doesn't call the lang function, it does an array lookup.
Newline missing between those two functions
Yep, I keep forgetting. And they obviously highlight the same.
The code to read and retrieve extension meta data really belongs into extensions manager. The ACP should just be assigning it to tempalte variables.
what do you mean? just remove this check?
oops, nm
no that's json_decode() your talking about :P
Can we do this intuitively?  (i.e. without globs?)
if there are no children, the `reduce` call will have no effect. In that case, the method will always return `top: 0, right: 0, bottom: 0, left: 0, width: 0, height: 0`.

The code for empty objects already lives in `DisplayObject.prototype.getBoundingBox()`.

Adding a guard here should do the trick.

```js
if (!children.length) {
  return DisplayObject.prototype.getBoundingBox.call(this, transform);
}
```

And a unit test for this special case might be handy
We could check that here and report it via warn().
Yeah, it was a pain to me that the testing framework didn't have it and that I needed to write it myself :(
debug is never default I think
No, it does nothing.  I expect that's just an artefact of the copy/paste from the original, where int is required when passing to the underlying C function.
Just noting that google-gson has Apache 2.0 license, and is on Maven Central: http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22gson%22
Also, the Google Visualization API is requiring a 2D Array: https://developers.google.com/chart/interactive/docs/reference#google.visualization.arraytodatatable
you could pass this array you should pass as an argument to the above listeners. 

Please name your variable something specific, related to its use. One letter variable names like ```a``` are confusing.
Piyush - in printcard.js, there are two functions -- storePatientDataFromReg and storePatientDataFromSearch. For printing to work in both new and returning patients, do you need to add oldPatientIdentifier into both places?
An example of mocking can be found here: #229
I believe this is no longer the case as ```make_coord``` is now called if and only if the derived coord metadata matches the query.
Was "Op contract" removed by mistake?
"should not be allowed"?
Get it, Done.
Lies?
please provide comments for border case scenarios
*never* do "import *"
Is this size in MiB as everywhere else?
Don't use ``has_key``, use ``in``
Useful question (repeated in a couple of places) on the guarantees that are made about the state of current_node_.
Please create get_current_modifier_linkable_list() or similiar function.
The query.select_related has numerous states: it can be False (for no select_related), it can be True (which seems to indicate that depth based select_related should be used) or it can be a dict (for user-selected restriction). I think here field_dict could end up being True here.

In the long run it would be better to split the query.select_related to two variables: one would be query.select_related = True/False, another would ne query.select_related_restriction = None or dict_of_asked_selections.

For the time being maybe
   field_dict = isinstance(self.select_related, dict) and self.select_related or {}
would do? (naturally, not tested).
Wait, just got it to work... let me see how can I add the commit to the pull request...
Ok, I agree. LGTM!
Currently lambdas.cpp rewrites NameRefs to captured variables. Instead of doing that, it should set up the overload's body as being the lambda's body preceded by a binding statement to bind the captured variables locally. Then the rest of the body can remain identical.
`"interactive") {` => `"interactive" ) {`
We don't want to remove escaped trailing whitespace, though:
* `document.querySelectorAll(".foo\\ ")` => `[]`
* `document.querySelectorAll(".foo\\")` => `SYNTAX_ERR`
I have made a pull request to correct this #362
What do you think about adding operator= for value copying? or this has been done somehow thru copy constructor?
Is this code copied from accounts-oauth2-helper? If so maybe instead put this in a new packaged called accounts-oauth-helper? (Not totally sure this is the right place, feedback welcome)
OK, I think I get it now. The start and end are dates independent from the actual data being passed so the exact time may not align.
Nit, but can we shorten these lines up at all (without matching incorrect things, of course)?
Indentation here, also, `dep.lang_list.each do |lang|` ?
Shouldn't this use $datadir 4 lines up? Or use OC_Setup::protectDataDirectory().
The comments is a bit outdated too, as it doesn't really test anything.
should we have this code in handle_item_list() as well?  It seems like its possible for this to happen there too.  However, I'd be worried about performance if we did it for every new item.  Maybe we should just stick with this, then add a note to investigate more after 5.0?
looks like this could be a static method
Added a solution to this.
Hmm ich seh grad, dass das result gar nicht spezifisch fuer eine instanz ist. Daher koennte die methode komplett weg oder?
I think this would incorrectly urlencode the domain component of a URL with internationalized domain name, wouldn't it?
This should be zip_bins in accordance with pep8.
around an existing binding.
On Sat May 05, 2012 at 01:16:42PM -0700, David E. Wheeler wrote:
> > -        username|user|u=s
> > -        host=s
> > -        port=i
> > -        sql-dir=s
> > -        deploy-dir=s
> > -        revert-dir=s
> > -        test-dir=s
> > -        extension=s
> > -        dry-run
> > -        etc-path
> > -        quiet
> > -        verbose+
> > -        help
> > -        man
> > -        version
> > +      plan-file=s
> 

> I would rather that indentation always be +4, rather than +2 in some
> situations. I think I started that after reading PBB.

Can't seem to set this definitively (as in always forced to the right
indent) but with the --notrim-qw option perltidy will leave multi-line
qw// contents alone.

-- 
Mark Lawrence
Well, this makes sense when you don't know where the variable is coming from. Here, I create it safely just few lines above.
腾出手来可以考虑诶~

On 2012-3-31, at 上午1:20, Yihui Xie wrote:

>> @@ -200,6 +200,13 @@ $ which R
>> $ echo $PATH
>> /usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
>> ```
>> +当然,奇葩的make -j8可以不用安装(sudo make install)就能使用R,你只要在`~/.bashrc`后面加上PATH就可以了. 更改`YOU_R_PATH_Directory`为你的R目录,复制如下内容到你的Terminal中:
>> +
>> +```bash
>> +cat >>~/.bashrc<<EOF
>> +export PATH='YOU_R_PATH_Directory':$PATH
>> +EOF
>> +```
> 
> 要不你包养一章呗。
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/yihui/r-ninja/pull/2/files#r629516
Yes ...

-----------------------------------------------------------------
    Dr. Gihan R. Mudalige
    Research Associate
    Oxford e-Research Centre,
    University of Oxford,
    7 Keble Road,
    Oxford. OX1 3QG
    Tel: +44 (0)1865 610787
    Email: gihan.mudalige@oerc.ox.ac.uk
-----------------------------------------------------------------


On 17/05/12 15:00, Florian Rathgeber wrote:
>>
>>       if(my_rank == MPI_ROOT)
>>       {
>> +      //printf("\n\n___________________________________________________\n");
> Could this line be remove?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/OP2/OP2-Common/pull/18/files#r838471
`s/Storage/Server/`
Nit: please remove that whitespace.
Looks good except one thing -- can you change Ctrl to cmd for mac shortcut? It's clearer if the intended key is Command and not Control.
Added a unit test for one case of this in the new Document-test spec.
Autoconnect is triggered if it is enabled and the session storage (see above) is set.
Nice suggestions. I added these two variances in the same it().h
Yes, it's more of a general best practice. It looks a little cleaner, too :)
Want to disable this so we don't have the overhead for the time being?
This was a bogus dependency. array_ops was used in the install-check, and only there. That is, there was never a need to declare this dependency.
Wow, I completely missed the function!! Thanks!
The logger is never used.
I left the paramString addition in a comment but it could easily be removed altogether.
Completely remove. Remove also `database_file` variable.
After first look of Post fields which facebook uses they have something similar, but I named it wrong. They have updated_time (The time of the last comment on this post) and that might be useful. Should I delete it or rename?
We're already logged in at this point, so I don't think we need this condition in the if
It's something that has to be manually changed to the correct value to wherever it's hosted. AFAIK this is how it was before.
Do an explicit check for === 0 here, not a falsy check.  It makes the code a lot more readable.
I assume this is for debugging purposes :) When it's complete, can you remove any console.log messages on the frontend?
are they there for a purpose? doesnt redirect always exit in those cases anyway?
Because this method modified internal state of the component.
这个改成ngx_uint_t? 一般用作标志的都是无符号的。
Tämä vain pois
Something more descriptive?
???

Moje výtka je vůči " se soubory projektu".
Ok, men `ARTGC#lab_groups` är alltså inget värde som "lagras" eller finns i databasen, utan rails har gjort så att ett anrop av `ARTGC#lab_groups` kollar igenom alla lhgs och utav de som rättas av assistenten så returnerar den labbgrupperna av lhgs och sen `unique`-ar den. Det är ungefär så det funkar va? Då ser jag inte heller någon anledning att ta bort den, tvärtom, den verkar bra att ha.
As a note for others looking, if your XCode app has spaces in its name (ie you have multiple versions of XCode, like "XCode 4.5.app"), this require path should be: 

```
  s.xcconfig     = { 'HEADER_SEARCH_PATHS' => '"$(SDK_DIR)/usr/include/libxml2"' }
```
Updating both message_for and pending_message_for using commit 6e481e64be63a0482858f508b57fec4e820ee6ae .   The new names will be message and pending_message.
I suggest "Subscriptions are not Current due to insufficient subscriptions"
Maybe this should be 'The duration before timing out' ? or something similar
Sorry, should be 

    if (!this.hasPincode || !this.conn.cardState == 'pin_required') {
      console.log("Can't unlock.");
      this.reset();
      return;
    }
should be ``plt.savefig(tmpf)``
I found this confusing, with debug enabled, 4 nodes and a target_n_val of 4 this prints

    [debug] claimv3: Not enough nodes to run (have 4 need 4), diagonalized
Can you put this backcompat bit (which is repeated lots) into utils.js and just have that loaded by bin/haraka ? (if it isn't already). That way the rest of the code gets it automatically without having to repeat it all over.
Any reason to do this instead of just using `@video.comments`?
Woah there cowboy. `number_to_currency(v.price)`. Rails is your friend here.
In my opinion, these should be on the User or Video models. For example:

```ruby
if current_user.bookmarked? @video
```

or

```ruby
if @video.bookmarked_by? current_user
```

What they're really doing is checking for the existence of a Listing record in the database. Helper modules (at least in my understanding) are more for helping out with complex view logic than getting information about models. 

Now we obviously don't want to be hitting the database for every one of these calls, so we're probably going to want to do a slightly bigger query at first and bring all that data in at once. Then we can add some method in one of those models that quickly checks whether such a Listing exists.
Exceptions should include a useful description based on what the failure is.  Perhaps 'Invalid encryption scheme chosen'.
should be `foreach (`
Good catch. Unintended (vim) "to_lower_case" change.
switch labels are never represented as integers in the admin interface, if you give this to a user, they will not be able to call `SwitchPinger_ping()` with it or `RouterModule_pingNode()` or compare it to the result of a `RouterModule_lookup()` so it is useless. You need to use `AddrTools_printPath()`.
Could you fix this? Otherwise I'm not sure what this might do to the history. Thanks.
This has been tried before and it's breaks urls.
Hy @guilhermeblanco,

I didn't know @asm89's patch, yesterday I talked to him about my patch.
then we discovered we were in the same thing. =)

Please take a look again I added support for __set_state

@asm89 maybe we can work together in this path or in your implementation
Geef Kernel mee aan constructor
I namespaced options with a devtools. prefix butI find devtools.github.user a but verbose and think that github.user is explicit enough (and reusable enough) to be used as is. Let me know what you prefer and I'll change it back if need be.