labofsahil/github-event-dataset-2012 huggingface dataset:
Move sidebar JS to application bundle

It's required on the main page so will nearly always be loaded anyway.

Enclosed in anonymous function to avoid leaking `onclose` global variable.
Andre Kwok's pull request

"added akwok.png and updated i_completed_githug.txt"
Router fix

Without this fix routing fails if APPPATH is different from defoult.
Added option to enable certain locales

Usage:

```yaml
server_grove_locale:
    enabled_locales: [ "en", "es*" ]
```
This will display only the flag for the **en** locale and all the flags for the **es** locale
Added russian translation

ru.yml
#3450421,  xslt2: Relative paths for transclusion do not work

From Sourceforge's bug tracker: id#3450421,  xslt2: Relative paths for transclusion do not work

Feel free to integrate it, if you like it. :-) Hope this is correct.
Add a PublishedManager to ContentBase

This gives the concrete `Content` class in `armstrong.apps.content` a `PublishedManager` as well.
Syntax Highlighting + Gemfile + Updated README

Added:
1.  Syntax highlighting via `pygments`
2. Gemfile for easier install
3. Updated README for easier install
Consolidating schema definitions and data syncing methods
add group option to rlm_linelog

Hi - ditto for rlm_linelog, is this OK? Thanks - Matthew.

Allows the group to be set when updating linelogs, rather
than being fixed as the group of the running daemon.
Added custom parameters and drop target

Hey Aaron,

Love your drag-and-drop uploader. I added support for custom parameters (so I can send {'post_id': 12345}, for example) and an option for a custom drop target (I had a pre-designed div I needed to use). Feel free to take my (relatively minor) changes and thanks for contributing such an awesome and simple plugin.
SS3 support, by Shea, Mateusz, and Sam

This is working pretty reliably in SS3.  There is one known issue - that when you create a new importer you need to press Command-R to see it - but I fixed a number of issues that I identified in Shea's cut.
Fixed Admin Notification

Fixed admin notification - it wasn't using edd_format_amount()
Just little Updates

H todd,

i've updated the translation-support to the latest changes and added if is_search() to the loop...


Best regards
Fixes Multipart_params to work with json/javascript

Multipart_params is expected to be a json-string/object, putting it in "" does not work, same if you want to place a functioncall there which returns the needed data.

Same will probably be true for headers and some other fields, if I have time to test/use them in a real app I will create more patches.
jetz oba wirkli rss add^^
Logging between runtimes with custom levels is now consistent.

Timob 6532: When a message is logged with a custom level in v8, the behavior now acts like Rhino.
added delete_old_maileditems command

added ```delete_old_maileditems``` command with tests configured with a constant ```MAILEDITEM_MAX_AGE``` in seconds
Added call to super() to MyDashboard documentation since MyDashboard.children have not been populated

It should work better now :)
second iteration...

OK, so I pulled from the original repo and updated here.
(Some files were deleted).
I also made a minor change.
New posts with git gotchas

I shortened the post a bit, removing the company-specific references and anything that wasn't a "gotcha"
Fallback when PGP Servers timeout

When a PGP server such as pgp.mit.edu times out, and exception is raised which breaks outside of the 'for ks in keyservers' loop, so no further keyservers are attempted.

The code for this was repeated 4 times so I moved it to a function to shorten the code a bit. I included a try block inside the 'for ks in keyservers' loop so that multiple keyservers will be tried.
cleaning the devApps

some have been moved to examples, some have been removed completely. as examples, they may not be the best, but they're a good start.
Remove workaround for bug 2997
Fix for Bug #6472. Make RedBlackTree's removeKey work with strings.

http://d.puremagic.com/issues/show_bug.cgi?id=6472

I also made `RedBlackTree` final, which it should have been in the first place. That _might_ break code (if anyone decided to derive their own class from `RedBlackTree` - unlikely IMHO), but it was already decide previously that std.container's containers should be final classes so that they would be reference types and still be able to have their functions be non-virtual and inlinable.
Resolve issues with web server config generation. (Fixes #7750)

Since 973ef6340e we can no longer rely on the existence of the
APPLICATION_HOST config value.  In the end this is a good thing (the
correct value is detected by OMERO.web) but it does not provide
sufficient information to the OMERO CLI web plugin.  Here we are
explicitly assigning the default HTTP port to 8080 (valid only for the
'nginx' config generation) and we continue to allow it to be changed via
the --http command line argument to 'web config <webserver>'.
Fix help-command bindings in perl & ruby modules

If you need more info let me know.  Thanks.
Create derivatives prior to object ingestion

* Builds derivatives from pre-file-ingest hook
* Removes code used to pull image file out of fedora object since the
  raw upload is now available
* Removes islandora_basic_image_add_datastream since images are simply
  added to file-to-ingest array
* Uses SESSION and post-ingest hook to remove files copied in image
  derivative creation
some small change

added some logging on start and stop
+ set mongodb version to 2.2.0
ISSUE-16 ISSUE-57 using coverage to procude coverage reports (can be adapted to produce jenkins-compatible output later) and created unit tests for model entities. The same pattern can be followed for other tests

Initial unit tests (using sqlite)
Yiwei's request
Strict binding class mapping

If the option `strict` is true, and a binding class is specified but cannot be found in the bindings object, then it will throw an error. It's just a troubleshooting feature and adds to the size of the file, but I thought I'd submit it in case you might think it's useful.
Test pull request

Jeg har lagt til en fil som heter ny_test.txt
Added zig-zag pattern

I didn't knew where to add it in the list, so I've added it first so it will be easier for you to find it and rearange
LPS-24661

Problem is that we use the control panel layout rather than the current scope group to see if a user has the ability to access edit mode, so fix it by passing in the scope group and a null layout when inside the control panel and leave the logic alone when on a non control panel layout.
Fixed the issue with Radio and MultiCheckboxes

fixes #39 - Radio and MultiCheckboxes now get rendered with no breakline between each element.

Updated the git ignore to avoid commiting project files.

My earlier fix for horizontal forms had to be changed to initialize prefixes but not call the parent constructor until we had set the default decorators. File elements were no longer rendering due to this.

7e64dfc005c611f4e8f3f1595b299450aeca3ebb fixes #27
powerHAL: use code from power_tuna.c to fix screen off bug

This should fix the max frequency occassionally getting set to the screen off frequency.  I'm sure it came from someone else as I pulled this code straight from power_tuna.c which was modified by CM or other devs but I hope this will help.
pyramid: add cifs module
Add support for YouTube short URL format (youtu.be)

YouTube has at some point decided to offer it's short URL format (http://youtu.be/Wl4XiYadV_k) as the default share format. Because of that I thought it might be worthwhile to add support for it to this module too.
See if this fixes the problem with Linux based compile.

Reduced required precision for inverse statistic distributions, tried to capture out-of-bounds better.
Snippets for comment headers.

Might be useful for many developers.
Граница справа пропадает в хроме при переключении вкладок
Quick loop locking fix
Unit test overhaul so they don't bleed GLOBALs between tests.

There still seems to be some weird things happening forcing me to delete module cache more often than necessary, but overall I think this is a good first step.
Changed main loop to blink screen back and forth inverted for four secon...

...ds, then quit.
Profile improvements

This pull request includes a number of small improvements to the profile and its default settings.

To wit:

  disable story content type (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/3f759b0c3c03cbba7fc73f530b75c667106f7d0f)
  turn revisions on by default for page (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/3a0ce4531aca6be7956a0fff914c2d4c3fef0ba4)
  turn comment module off by default (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/dab6e47b98bbe94ea9c4335551464c2863c05dff)
  Date/Date API: enable date_api and date_timezone; disable user-configurable time zones (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/1a8d93fb1f3ec57a54e51b2b803b5e902b956857)
  File upload size per upload and per users (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/9d70588a7cff653c8ef21888efe9d3f6575ff409) 
  set error messages to only go to log (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/6808c2c752d53be0218b5ce04d01d9ad1ee32849)
  set user creation to only by administrator (https://github.com/SU-SWS/Stanford-Drupal-Profile/commit/83f21cf952f841679635b481aa05970dd565f2cb)
fixing relatives
Fixes to SRA1.3 exporter

simplification on the annotation requirements + finer tuning of ISA parsing, better representation in SRA
Lanado ics

Hey Pete,

I just wanted to show you some of the changes i've made if in case your intrested.

Cheers
Allow to set header for apexrest function.

I believe it's pretty common to set HTTP headers when using REST. I added a parameter to the apexrest function to pass in a map of header names to head values. What do you think?

Example:

forcetk.Client.apexrest('attach',
				function(msg){ alert('rest response'+msg); },//callback
				function(msg){ alert('err:'+msg.responseText); },//error
				'POST',//method
				e.target.result,//payload
				{upload_filename:"filename", upload_pid:"parentObjectID"},//paramMap
				true);//retry

Signed-off-by: Alex Berg <chexxor@gmail.com>
Dont build these sensors if fascinate

Fascinate uses slightly different sensors
Remove Linux distro logos from generic GDC instructions.

There are no distro-specific packages and all the logos made
the table look awfully croweded when browsing on smaller
screens (the cell had two line breaks).
removed unused classes

Removed files that are not needed for the box2d extension.
The most important pull request you will ever accept!
Changes 2/7 (Hotfix for 2.01)

Hey Fires,

Just a few changes in this one, just to bring the parts of the Modern Era that were still on 5s to the new system and little fixes as reported in the thread.  I think Im going to start on the ww2 units now; I should get it done by tomorrow.  Once we (read: you :D) finish the setup for the tech pages, I suggest we go ahead and knock out the policies; they have been sitting in my back pocket for months lol.  Thanks.
file added /  library modified

...Also added the example file in its incomplete state so there will be at the least a place holder for the file
Correct, Bruno, thanks. I don't know yet what users should have pre-publication access, but we certainly don't want e.g. reviewers. For the moment I've limited it to press editors and managers; see https://github.com/pkp/omp/commit/f56615dff1bbd37375a9ae864a7b818ba779087b
I fear this one broke the login, I can't login on my blog anymore since I updated.
Ok, thanks for catching this. I have just hard-reset my password from the console, filing it as 'need to check later why this happens now and again'.
yeah, altho it looks like i meant to call handle_write_error() here... looking
At the moment only Integer up-casting (and signed to unsigned and vice versa) is implemented. If more is needed, it should be enough to change the predicate to this function.
Can we have a new release with this fix? OmniAuth depends on this to be compatible with rack 1.4
Fixed in: 4faa4121520daeed54600d39f58b948d1872641d
Thanks, cheers !
eve.c:947
SYMBOL_TABLE is not capable for key handling, e.g. feedkey, and it relies on previous input method so in_method should not be changed.
eve.c:1075-1076
To skip symbol table when it is shown while presing <Ctrl><Shift> to prevent staying at symbol table.
He also invented swag, so you can blame him for all the crap you'll get in your SXSW goodie bag.
I don't think array_keys returns a reference; in any case, no need to use a reference here.
@drogus thanks for working on this!
You can't be sure how much time it will take, 25 ms is only a minimum that will be spent between two calls.
indeed.

To explain my train of thoughts.
I did step by step in the repl the different steps you described in the commentary of the function in order to own this code.
I noticed the question on the code.

I did not focused on this question at first and then noticed that the output of the block :

```
(map #(ns-resolve ns %)
   (keys (ns-publics (find-ns ns)))))
```

and the one from

```
(ns-publics (find-ns ns))
```

was similar.

I tested on the repl and it was ok so i removed the old code.
I think maybe 'failing' is a better name, for two reasons: 1) "unwinding" doesn't have as much meaning from a user perspective, 2) "currently_unwinding" is really long.
soga...
it's not really something I can `reverse`, so I went with the fragile method. I can strip the locale for added robustness maybe.
No idea either, but there are two more such lines that have mutated over the years:

`if (aircraftState != AIRCRAFT_LANDING && owner->commandAI->HasMoreMoveCommands() &&
(l < 120) && (deltaDir.SqDistance(deltaVec) > 1.0f) && dynamic_cast<CTransportUnit*>(owner) == NULL) {`

`if ((goalDir != ZeroVector) && (goalDir.UnsafeANormalize()).SqDistance(goalVec) < 1.0f) {`

AFAICS they were originally introduced in https://github.com/spring/spring/commit/beb580a243ae1b003a68ae5d7f161e466ccf9e15#diff-1 which apparently nobody reviewed for saneness.
^=O
.swap()이  콜백의 리턴값을 반환하는 형태로 변경됨
Does this still pass on older versions of Rails, though? We support all 3.x versions.
It seam to original test should have tested complex, but by error it didn't. Can you add those case?
nit: no apostrophe here.
Oh dear. It seems that I wrote over some changes. I'll fix that in a second.
Ok :) Will do that.
With that part moved, would that patch (below) be pushable?
https://github.com/Bertram25/mana/commit/cb15794bc7121604fcee52d1de5e7d3b4b86ed8c
`del` is such a weird thing to do that we should probably document our intent.
Because it broke the test suite. I don't understand how the tests have been rearranged, and I couldn't figure out how to make things pass again.
действия чарма спадают а вот панель залипает у хозяина петов (ханты , варлоки) серым и всё :( .
ERROR:World::Statistic AHBot 0 AuctionMgr 0 BattleGroundMgr 0 Begin 0 CLI 0 LFGMgr 0 MapMgr 992 MapPersistentStateMgr 0 MassMailMgr 0 Quests+RandomBG 0 RemoveAllObjectsInRemoveList 0 Sessions 150 TerrainMgr 0 UpdateResulttQueue 0 Uptime 0 WorldPvPMgr 0 

When MapMgr increase 500 = lag, and keeps with hight value on every statistic output until server reboot.
Время тоже желательно вынести
ich hoffe das mit dem Cmake passt so...

Ich kriege immer noch massig Fehler beim testen. Liegen alle an einem Fehler beim erstellen des .emp-Verzeichnisses. Hat das Problem noch jemand?
on players - not fade, tested. but on creatures - this impossible on current mangos aggro scheme.
Good point! I will review it
RE: msg, if the msg is 100k, the set member will be 100k.  If you use a hash, the set member will be 32 bytes not matter what the message size is.  You just need the set member to be a unique fingerprint of the msg.

RE: leak, the network could die and then when the srem fails, that set member is now there forever.  Redis would need to support expiring set members or we'd need to support a collection of sets, e.g. one set every 15 minutes, that we can manually expire the entire contents after so much time.
Correct. I'll clean up the redundancy here.
... and justification.
I think you should inherit from QObject as well.
Consider jQuery to be the canary in the coal mine. Messing with `Object.prototype`, or most other prototypes you don't control, has been proven time and again to be bad practice. The old code didn't check to see if it was clobbering someone else's (admittedly ill-advised) `.each()` or `.merge()` extension to `Object.prototype` for example, so it didn't play well with others.

http://erik.eae.net/archives/2005/06/06/22.13.54/
Nit: This is not quite true; you don't support floats, the calling convention is somewhat different, etc.

Please make sure (adding a test?) that this DTRT for negative numbers.
please check my pull request
Django's default file system storage doesn't allow you to put files outside of a root dir. So if you pinned it to /data then you can't write a file outside of that. That's great but it would be a huge undertaking to implement that in all of AMO since there are many root locations we use. The exercise is just to move us toward a cloud service anyway where we can safely use any filepath.
The only problems I've had reported were the MFC interoperability issues.  Then again - I refuse to accept reports of stability problems from anyone undervolting or overclocking.  Patches like this are exactly why - if the CPU is on the hairy edge of stability, load transients (like core power being turned off/on more often) are going to give the regulator problems tracking.  However eliminating 90%+ of static leakage consumption (AFTR powers off core but not cache) by completely depowering the core more frequently vs. reducing it a fraction of a percent by undervolting a few millivolts extra - that's an easy decision.  I need to rerun my tests, but with the patch, the penalty for incurring a permanent wakelock is now a bit less than 1.5%/hour - pre-patch it was around 2%.

Of course, if this is the final patch that makes sense.  I'm going to continue maintaining my GB kernels right up until I9100 source drops - probably a little bit beyond.  XXLPH is highly unimpressive, and I don't think Sammy will be able to polish it up to the point that GB is polished in only 2-3 weeks.
this seems a little too clever :) having a separate property name (instead of just `not foo.passed`)  makes usage unclear IMO
Excellent to see this. I had forked this repo for adding this very feature. Nice to see you beat me to it
Yes, and it would be great if it works. But, in my case, adding

3 => 'bureaucrat'

(where 3 is the group ID for moderators) to the array did not work. I have had to query the database directly and run some INSERT commands.
we should use cachify in communication_iframe.js /cc @ozten
Just had a look, there really seems to be no good jQuery support for text selection and contenteditable elements yet. The things that are closest to have any relevance are:

- http://archive.plugins.jquery.com/project/a-tools
- http://stackoverflow.com/questions/985272/jquery-selecting-text-in-an-element-akin-to-highlighting-with-your-mouse

I wonder, should we maybe move our DOM level text logic into a (well tested) jQuery plugin and open source that? With that we could keep Lively clean and profit from bugfixes from others...
Nope! I'm just uneducated.
Wait. It's not timer but interval. So your naming was better, but let it be more clear: extensions.memchaser.memory.interval. I suggest one more level because we probably will get more preferences for memory related code.
There are issues with this:

1. This should be optional. Perhaps turned on by default, but at least it could be turned off.
2. Any new code needs to be in the source files, not in the compiled files. The source file for this is `src/browser/documentup.coffee`

You don't have to compile it afterwards, I can take care of that.

If you don't feel like fixing it, it's alright I was planning on doing it anyway. I would just want it to be done right.

I appreciate your involvement though!
I know procworld, it's a reference in my job :) I saw the paper and it helps me to better understand the original one ( http://algorithmicbotany.org/papers/colonization.egwnp2007.html ).
Your screenshots look very good to me. I though about the cylinder problem you're refering to. And I think I will go with a more "conic" aspect of the thing. For each branch I think I will try something like "the more a branch as child node the bigger is its base radius". Like that you'll be sure to have a big trunk looking like it can supports every branches up to it. Now I have to test that thing up to know if it works for real.
Anyway I think I will try to find (when i'll got the time) how to make your project works on Linux and submit you a pull request.
Do you plan to work again on this 3D tree generator project ?
@pksunkara Is there a reason you want the statusCode handled on a different path if it's 422?

Also, I think having body to be null would be better, so people can do:

```coffeescript
if body
  callback body
```
nice design~
it's not public facing luckily so will push with whatever else forces 2.0.1 :-)
Coraz bardziej mi imponujesz, taka późna pora a ty ciężko pracujesz. Brawo!
intents are fucked up here quite a lot
class names aren't case sensitive iirc (unlike constants) - it only matters when working with autoloaders.
Actually, disregard my above statement. I realized that you have placed new settings here.
On ruby1.8.7mingw the original remove(entry) was returning a joined string of all of the names within the @entrySet variable and then trying to write the contents of srcPath to this invalid name.
ie:   file1.txtfile2.jpgfile3.jpgfile4.png
Test for wms transparency on IE9
I just couldn't confirm it was happening, all working fine here.
I looked in my console log, and I am afraid this code in XMLParser.m is not robust enough to handle the various date formats we encounter in real world feeds... I notice problems with dates like :

Tue, 17 Jan 2012 18:08:37 GMT
Tue, 17 Jan 2012 20:05:10 EST
Tue, 17 Jan 2012 17:39:55
2012-01-17T18:28:10.000Z
2012-01-07T02:08:39.758+08:00

See also the comments in source at lines 451-462...

It might be risky to get rid of the libcurl hacks... We can either copy and adapt the source of curl_getdate, or cherry pick on these commits : 

https://github.com/barijaona/vienna-rss/commit/05271534b00a46b9666a582014809f850905ec08
https://github.com/barijaona/vienna-rss/commit/cf1b1dac1fede5272040d1042d87810058a508fb
https://github.com/barijaona/vienna-rss/commit/03933d107e4d0345b89e4d00a000bd1786b19482

Basically, what I did was to transfer the call to libcurl in a library/framework compiled with 10.5 SDK.
Yes that is what I did wrong, I forgot to check where it was used and how, and should have made the right changes there too.
fixed (replaced the 1.0.1 file)
What were the reasons for pushing these classes to the common core trunk and not leave them in the Voldemort specific part? Granded some noSQL use JSON but that's not necessary the majority.

Likewise, stuff like URLs is not necessary for all NoSQL stores.
who invited this joker?
however you can dedicate a commit to a style fix. It helps the review and following changes back in history.
Your attribute values do a good job of describing the data in your representations. But you've forgotten to describe the state transitions (links and forms). You need to define `rel` attributes for the links in your application that indicate what kind of resource is being linked to. You also need to define `class` attributes for your forms that indicate what they do.
The Ajax includes arbitrary forms' report.php though.  One of those reports might use session variables.

+  if (!function_exists($report_function)) exit;

+  call_user_func($report_function, $ptid, $encid, 2, $formid);

+}
You are missing the second line...

It is important to get things right the fist time. Especially when yo…

…u will be staring at them for a while to come.
Well, the documentation is a bit of a mess as well, which is one of the problems I'm facing... however, I don't really feel like introducing incompatible changes.
The PAS propertyvalidation API returns strings. This keeping the to the PAS specs, not creating an entirely new API.
> Can anyone explain this change? What the reason to introduce truth and falsity?

Two reasons:

1. it was annoying to get the type `true->false` inferred for the expression `true->false` instead of `Boolean->Boolean`.
2. your `Boolean.instance()` method that you use for unboxing returns a `Boolean`, not a `true` or `false`, so javac complained.
Ah, that totally makes sense. Thanks! :)
Ulduar has only 2 difficulties. The so called "heroic" chests are spawned when hard mode is activated. And you can have both normal mode and heroic mode chests spawned in the same time -> see Hodir.

This doesn't apply to ICC, so I think the naming suggested is fine.
Не пиши "далее" и т.п. Лучше сделай заголовок "Анализ тестовых решений"
I don't have enough time today to reinvestigate the issue to the deeps and create testcases for it (it looks like it's no so simple to reproduce osgi-like environment in TC) to give you some points for this patch but it's definitely right thing. You see I had to create an opensource hibernate-osgi fork with only this modification to make possible to use it in our project.

I can try to explain the reason for accessing both classloaders:
Without this patch Hibernate **assumes** and **requires** that all persistent classes' classloaders have access to hibernate and proxy classes. But in some environments it's not logically right and persistent classes may have another classloader that knows nothing about hibernate imports.

At some point it does the following `JavassistProxyFactory.postInstantiate()`:
`factory = JavassistLazyInitializer.getProxyFactory( persistentClass, this.interfaces );`
with `persistentClass` set to my pojo class(loaded with another 'hibernate-less' classloader) and set of interfaces to support: [`org.hibernate.proxy.HibernateProxy`]

During this call creating the factory it uses my pojo's 'hibernate-less' classloader to load the following classes that are not expected to be known to it (I just put a breakpoint into my `findClass` to catch them):
org.hibernate.proxy.HibernateProxy
javassist.util.proxy.ProxyObject
javassist.util.proxy.MethodHandler
javassist.util.proxy.MethodFilter
...

To make possible to put pojo classes into some clean 'bundle' and make them not to depend on Hibernate and javassist I created a new ClassLoader with `parent` set to persistentClass's classloader and override findClass(). This findClass() uses platformClassLoader to load everything that wasn't already loaded and found with `parent`.

If I have time to reinvestigate it again I may provide more detailed explanations, but not today.
C++11 specifies override keyword. See virt-specifiers in section 10.3 [1]. But unfortunately trinity doesn't use C++11 yet.

[1] http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2011/n3242.pdf
Just checking it out for our test and development environment. Since we are using subversion, I am thinking about using git to store our builds, as we do not deploy the code directly on the servers. :)
> Are you saying that we should do the equivalent of
> alsoProvides(someview, IOverlayed) or similar (in zcml, with browser
> layer of course), and then in IPubAfterTraversal, if
> IOverlayed.providedBy(context) you set the header and change the skin?
> I just want to be sure because I like that idea.

Yes. Specifically you would use something like the following zcml to annotate the view class with the interface:

    <class class="plone.app.something.browser.SomeView">
       <implements interface="plone.app.toolbar.interfaces.IUIView"/>
    </class>

Then have subscriber for IPubAfterTraversal doing:

    def mark_ui_views(event):
        request = event.request
        view = request['PUBLISHED']
        if not IUIView.providedBy(view):
            return
        request.response.setHeader('X-Theme-Disabled', 'True')
        view.context.changeSkin('toolbar', request)
Sorry, it looks like I was wrong, for some reason I couldn't see any other cd commands with back ticks around them. I will merge this now - thanks!
Oh Yea Mego Commit)
I feel we should place `RequestFactory` in `Aura/Http` so will reduce the lengthy name . @pmjones can give more suggestions though. Also why `use Aura\Http as Http;` ? Can we make use of use statements and make the lengthy names shorter in class ? So an eg can be 

    $cookiejar = new HttpRequest\CookieJar($cookiefactory);

becomes

    use Aura\Http\Request\CookieJar as CookieJarRequest;
    $cookiejar = new CookieJarRequest($cookiefactory);
Vamos a resolver la generacion del JSON generandolo automaticamente desde una vista de Django, por lo cual, esto no seria necesario.
Adding the KeyError to the existing exception list displays an error consisting entirely of 'userid', which seems less helpful to the end user than no error message. But maybe I'm wrong about that.
There is no need to catch these in tests - let them propagate and JUnit will fail the test automatically and print a stack trace for you.
great stuff!  thank you!
I added the the build status image to the README.  turns out it is really easy.  just click on the the little gear icon in Travis and it gives you the right format to include the build images in markdown, rdoc, ...
Wow I only know a few things that can break loops: broken roller coaster rides, scissors, power switches...and they are all powered by possessed magical pixies that are looking to be destructive--to be the messengers of the Antiloop. But you...you are the king of magical pixies. You can break loops with your own two hands, loops that even possessed magical pixies cannot handle. You are the bringer of good loop breaking. You are the Loop itself.

I can only appreciate such greatness from afar.
Hello! This is the complete code needed to run the SolarRoboFlower as-is from the HackerSpaceShop.com
The SignalNode object may be over engineering, but we might as well do it 'correctly'. If we really want to be efficient, we can even use an enum for the 'name' field, as there are only three options, and thus three pointers to singleton string instances would be even more memory efficient... But in reality, a gain of some few kb is not worth our time :P. 
Good work on using urlencode, the previous method was rather silly.
The example in the documentation fails. 

https://github.com/probablycorey/mini_magick/issues/59

https://github.com/probablycorey/mini_magick/pull/79

https://github.com/probablycorey/mini_magick/issues/80

https://github.com/probablycorey/mini_magick/pull/81
On Tue, Feb 28, 2012 at 09:20:01AM -0800, Olivier Grisel wrote:
> I like @GaelVaroquaux suggestion too.

I'd even prefer my other suggestion. It's a bit more work :$

G
lol ur running way ahead of me lol i like it... k give me a sec to catch up and i'll also check the rest of the spells...chaka i know needs to go to spellscripts....and i suck at making spellscripts... /sigh
try this out with the show Alcatraz... some nucklehead posted a special as 'S01'.
It works fine. <3 First thing i do works =O
you should have made it a class where it has operator<
Looks like this may have fixed https://bugzilla.wikimedia.org/show_bug.cgi?id=33831 :)
Hi Jay and Mahmood

This should show you the principle behind Github...
CamelCase support
вот здесь, и еще в подобных местах, вместо этого ItemsOrders.where("item_id = ?", item.id) есть же ассоциации, типа item.items_orders
I did this to fit with password_test.rb
https://github.com/cfabianski/activevalidators/blob/master/test/validations/password_test.rb
Is there a need to maintain the namespaces? Our idea has been to get rid of them e.g. with

oaipmhTransformation = strip_namespaces.xsl

as it makes further processing much easier.
Update for Arduino 1.0
Why wouldn't it be is a better question.
Yes you can not construct an ssl::context without the io_service.
Surely you can kill these two overrided-abstracts, mm?
Nice :) Also: Burning the midnight oil again, dude?!
I'd use FILTER_UNSAFE_RAW because we don't want to remove any characters from a user's password.
I was actually going to suggest this as well.  This way if others copy the formatting of our EditorConfig files they might retain the descriptive comment.  I was thinking:

    ; This is an EditorConfig file.  More information at http://editorconfig.org

I like your suggestion better because it's more descriptive.  I'm going to use it for now.

I am planning on refactoring the introduction to the EditorConfig website to make it more concise (someone told me they didn't understand what the project was about after reading that section).  I may improve the wording further at that point.
`cell.dereference` je funkcia, treba ju asi zavolat ako `cell.dereference()`
Das Link ist Normal - also, Normales LInk.
Oh my bad. Should be fixed now.
Yeah, i hadn't forked this repo yet when posted this comment, but ended up doing so. :)
Heya
sql fix, your sql is wrong
DELETE FROM `spell_script_names` WHERE `spell_id` IN (48018, 48020);
INSERT INTO spell_script_names (spell_id,ScriptName) VALUES
(48018,'spell_warl_demonic_circle_summon'),
(48020,'spell_warl_demonic_circle_teleport');
Thanks. Did not test that. A-last-minute-change which does affect to anything. Yeah, for sure.
where?
https://github.com/Squadzone/android_packages_apps_FileManager/tree/ics ?
This change works great for me!
cat <<EOF
$0
Copy a remote Plone site database to local computer over SSH
Error in $0 - Invalid Argument Count
Syntax: $0 [SSH-source to buildout folder] [buildout target folder]
Example: ./copy-plone-site.sh yourserver.com:/srv/plone/mysite .
EOF
Aha, there is a `getCredits()` method — https://github.com/unl/UNL_Services_CourseApproval/blob/master/src/UNL/Services/CourseApproval/Course.php#L114

I suppose I could just use that. I'm not sure what it returns in the case where a course has no credits though.
@sebastianbergmann I'm not 100% sure. I think be put it into 1.2 because it has BC implications. I'll check if it has any impact on bigger test suites soonish
I'm not super happy about having to use "merged" scope for the flex-common rsl; but I get a ton of load errors (and the application won't work) without this change. See: http://forums.adobe.com/message/4053631 and http://www.adobe.com/devnet/flex/articles/flex-maven-flexmojos-pt3.html for background/explanation.
Why was this enabled?
No, they don't. They might line up perfectly well in *your* IDE, but that's missing the point. Again, check the WordPress coding standards which support my comment. And check the raw view of your file (View file @ ... linke, top right of this code block), and you'll see that they are still misaligned.
Comment on the use of -1e6 value by Atlas at experiment load time (to represent non-float expression values in the loaded data files)
Missing brackets in method definitions that take an argument :). Fixed in ccc2177f125541b7d849219c50b1dd60d3332858
The description of this test seems to be basic JavaScript logic, why does it not run in IE?
Is "continueToDeletei" a typo?
I think I would prefer to see the introduction of a new base class for this purpose as it's own pull request please.  Since the changes to add Background are not directly related to the addition of the Image widget, they really should be in separate requests.
Nowhere in code calls `getTickets()` AFA I can tell. The option does not require/have a cleaner. The class itself is marked as final so points of extensibility by deployers are also restricted. So, in that regard, no effects on the cache other that actual and explicit calls by the client, such as the SSO Sessions report below. 

As far as performance,  this is what EhCache says:

> The time taken is O(n), where n is the number of elements in the cache. On a 1.8Ghz P4, the time taken is approximately 200ms per 1000 entries. This method is not synchronized, because it relies on a non-live list returned from getKeys() , which is synchronised, and which takes 8ms per 1000 entries. This way cache liveness is preserved, even if this method is very slow to return.

The stats may be old though. I haven't encountered any machines with a 1.8GHz CPU in a long time :) Nevertheless, we might want to switch to getKeys() or getKeysNoDuplicateCheck().

P.S: `getKeysWithExpiryCheck` is already in use by `sessionCount` or `serviceTicketCount`
Very good point. I was going to add that and forgot about it. Will do now.
could save some cycles by omitting this if `intex != null`
type should be `immutable.Seq`, or `Map`
This will be used automatically if you use the factory method, don’t pass in a loader and the context loader is non-null.
this method can be used from user code, but if you think it's not possible to create/pass in a RichX I don't mind removing. It was there originally, but then things were not java.lang.Number
no: if the user changes this to `""` without enabling SSL, why should he reasonably expect it to work?
good catch indeed
I think the natAddress should be passed into natOK and not host & port
My understanding is that the doc is where it belongs right now; these classes are not `private[akka]` … (scratch head) … for some reason I cannot recall, but they are for sure not supposed to be well-documented public API. I’ll add a clarifying warning at the top.
May be, but it isn't what the docs for it says:

"There is also a release available if you are using sbt 0.12.0-RC1

addSbtPlugin("me.lessis" % "ls-sbt" % "0.1.2")

resolvers ++= Seq(
  Classpaths.sbtPluginReleases,
  Opts.resolver.sonatypeReleases
)
" - https://github.com/softprops/ls
This dance is repeated on line `534`. Could we refactor this?
yeah, but I mean here, when deserializing bytes to String
it looks like you do it in a different way in the scala sample (utf8String)
comment ?? string.Empty

Same idea for the similar others
This is consistent with above, but didn't you mean "octahedron_faces" here and above? 

Sorry that I am only commenting on typos so far. Looks great otherwise.
I've changed my mind :) You inherit from ProbDistI and it has discount() method, so having 'discount' attribute in KneserNeyProbDist would be very confusing. So IMHO it is better to 

a) make sure that KneserNeyProbDist.discount() means the same as ProbDistI.discount();
b) if they mean the same, use a function call to get the discount (so that overridden discount() method would behave as expected). I don't think it will have a performance impact worth worriyng about.
Components work. In hindsight, I kinda wish server-side had a different term from plugins since that's a nice easy to grok term that I'd like to use to refer to everything that's pluggable. But components works for me.
Shouldn't binding be plural in the URL? It is on the ones below.
I'm including this (transclude/include) upstream on a simpler request (isn't quite ready yet).  That will help reduce the distance to this branch as well.  I could pile some of the other incidental changes onto that branch to reduce it further, what do you think?
This might not work. av_malloc() is not defined in avfiltergraph.h

just do:
inputs = NULL;
outputs = NULL;
by the way there was a shameful bug, concerning the _super() and _superApply() methods... I committed the bugfix
Can you add `continuation?`?
Sorry I noticed one remaining problem with this part of the patch. This `source` path is "resolved", which means if `source` was relative it will now have the map path prepended to it. It is important that the resolved version is passed to the tileset because the map path will be forgotten later, which can cause problems on saving (where the relative reference is probably resolved relative to the current working directory instead).

This patch currently causes the unresolved version to be passed into `Tileset::loadFromImage`.
use immutable.Queue so it is apparent that it's immutable
Cool, my first "commit" to a component of symfony :)
I'm not sure it's required, but this should be `false` if I read AsciiDoc docs correctly
"source file" matches most of the other filetypes in the markup section, so I think its better.
Looks like this is constraining the values, not wrapping them.
There are quite a few places we convert from cartesian to spherical coordinates without using d3_geo_spherical. I guess maybe they're each different enough to not merit code reuse, but I wonder if there's a more flexible form (say spherical(x, y, z) rather than spherical([x, y, z])) that would encourage more reuse?
If it returns `FileTemplate` it should be then named `IFileTemplateFactory`. Otherwise it should return just `ITemplate`.
It could!
Correction: it should be a tuple of the defaults. The problem is that the length ``ndefaults`` is needed.  This happens in the fallback where the information is extracted from the error message.  I am thinking of returning ``(NotImplemented,)*ndefaults`` so the main code can just do ``ndefaults=len(defaults)``
Done.  (And a few others found too).
I don't know.  I mostly put the PRIVMSG function again.  Though, now that you mention it, it does kinda assume there's a name and a message attached.
Having said that, running it without parameters simply throws an exception (which seems to get caught somewhere, and the IRCd continues without issue), just like when an unimplemented command gets put in (which really should eventually return the proper numeric).
Unless I'm missing something I don't see the double free.  I am using it for two different purposes.  Allocated on 755 and either free'd on 758 or 786 and then re-allocating for another purpose on 760 followed by a free on 768 or again at 786 depending on execution path.  Perhaps I should be using a separate variable name to avoid confusion?
How about `.routeTable td`?
Ditto. Splitting out `keys` is less clear. The original is good, except for the short-circuit return.
#available_environments method return all environments (including any custom environments), where detect iterates over three default ones. 
Previous code was great to run 'rails console p' and have 'production' environment loaded into console.
But it would work incorrectly if you have custom environment called 'pro' and try to run 'rails console pro', it would load production environment, not the custom 'pro' one.
Wrong place for this comment(since this is in 2.10.x too)

Aren't $ in method names dangerous and not good?
This test looks like it would fix the long outstanding #1046 issue as well.
this looked dubious, but i know it's a callback. should we suffix all of these function pointers with Func?
While this is important to understand I think that this line will be confusing to non-experts.  I think this sort of thing belongs in documentation but not in docstrings.  If you really think it should be here then I think that this issue should be explained.
Will this return an int or a Rational? What *should* it return?
well...what is the first column of a columnless matrix suppose to return? I don't see why Matrix (and not list) should be allowed to reinterpret a request for a literal element that doesn't exist and just give nothing. That would be like [][[0] giving [] when it should not give an iterable, but an element. Perhaps you intend to use M[:,-1:]?

```

>>> Matrix(0,3,[])[:,-1:]
[]
>>> _.shape
(0, 1)
>>> [][-1:]
[]
```
There are going to be no issues, I think, because `s` is never modified directly.
This should be refactored. Bare `except` is almost never a good thing.  Do you really just want `hasattr("_eval_expand_trig")`?
why are x_i and y_i in braaces. Are these the generators? Before you referred to those as `f_i` and they appeared first in the tuple. Are they at the end here?  (This would be easier to talk about at a whiteboard ;-)).
Missing space around `+` and `-`.
I don't see much of a difference between the two approaches.
All right, I wasn't aware of that :-)
I think you should explain what the `__eq__` operator does. [I.e. to what extend does it flatten composed morphisms, use commutative diagrams, whatever.]
Why is the `as.numeric` necessary?
Yes, I saw all this... what I commented is that the official documentation does not mention it...

In any case for me it's OK if we use it
why do you define 3 additional cases? You could just use switch(strtolower(substr...)).
change to void if we're not using the bool
I'd probably replace that "if !klass && !block" with "unless klass || block", but that's just a style issue. :) I like what @eric suggests.
`version_compare` could be done out of the protected closure as that will never change.
I've no clue why this was originally excluded.
All sphinx constants should probably have a sphinx prefix.
Actually we can do something about this. First select a timezone at the index of the desired timezone, say, +3. Then select the desired timezone.
Is this function needed anymore after the refactor? It seems to just add a level of indirection.
Ok, I will improve.
Agreed, I added the namespace to pass the tests, but after I fixed the mojito mock to provide that namespace, I'm off the hook to use the full namespace here.
Correct. Sloppy dirty debug code.
`request.path` is not everything, what about possible query string (imagine person clicking login on search page)?
then let's assure that it doesn't leak anywhere higher (explicit __all__  where it gets imported for use etc)
You can't just leave a bunch of commented lines. Can we just throw an error if all three parameters are given?
Hows about some helpful blurb here, *"doh! too much cubage!"*
I can also just rewrite this function.
I think that deployment files for sample apps is considered "old fashioned". You might need to sync all of this with the latest versions of voter, etc.
Restore newline at end of file (many times / many files).
Should you trim before appending '/', in case they left a space before the comma?
This has to work without JS off too!
I feel kind of weird about keeping whether a user is online or offline in mnesia. Not saying we shouldn't do it, but we will have to be careful about catching when a frontend server crashes and making it so that all the users that were on that frontend have their status set to offline.
Yeah, we definitely should use `gen_server:call` here.
`// Support: IE9`
As it's not needed elsewhere, this static should not be in global space. It should be inside widgetClass(). (and if global, it ahouls be wraped into a function)
Ah, yes of course.  Makes sense.
no spaces + add missing semicolon
One reason I don't really want to do this is because I want the current_search dict to have the original textual values, since it gets used to rebuilt searches. I am going to leave it as is for now.
Кажется, оба варианта допустимы, ошибки нет.
Если уж менять, я бы предложил отказаться от «большого промежутка времени» и написать «в течение длительного времени» или «на протяжении долгого времени» — по-моему, так лучше звучит.
Three lines of whitespace here
I agree, but still, for msgId and for msgIdPlural, the following text is the same: "the message id for the translation". This is not right I think.
Ahh sorry about that, there was no reason for it I did know that I had.  I will revert.
“动态”是什么？
`actionIsInternal: true`
To be more consistent with "addEmbeddedStyleSheet" maybe this should be called "addLinkedStyleSheet" ?
``_initStatusBar()`` is only called once so we can probably inline it here. I don't think it's necessary to factor out into it's own function.
Seems like these offset computations could share code too...
You're dynamically changing display to 'inline-block', so you probably don't need to set it to 'inline' here.
What's the expected output for a path that's outside the project root folder? I could imagine this happening if the user sets their Brackets project root to a subfolder of their main root folder, but the HTML file there accesses assets like stylesheets that are in a different folder.
You can use "padding: 0;" as a shortcut.
http? Won't this fail in production which uses https?
Please move these style tags into the css :)