labofsahil/github-event-dataset-2012 huggingface dataset:
.htaccess improvements.

modified the form, htaccess file, wsgi_handler, and dispatcher for better default behavior at the database URL root, so that http://your_host/philo4/database_name defaults to the search form.
Fixing CSV issue

... , we should not support older ways. Added quoting too
do not validate 'decorative' inputs

#401
from Olya Levitskaya

sorry, I'm late...
Adding SQL for blog permissions install/uninstall and updated readme instructions
Add &url parameter also for Spot.

I add the &url parameter for Spot also. In addition, I change the method extracting main content from URL to ServerUtils.

Conflicts:
	rest/src/main/java/org/dbpedia/spotlight/web/rest/resources/Spot.java
fixed mms

changed the mms config to fix mms for VM phones
Fixed a few issues with JS

Fixed a few issues:

- Fixed tiny icon/shadow
- Incorrect map center on first load
- Spelling/Grammar
Placeholder images

:3+
Construct a full JenkinsJob instance form raw job names.
Do not make apps depend on rspec

After upgrading to 2.0.1 my rake tasks stopped working:

```
$ rake -T --trace

cannot load such file -- rspec/core/rake_task
.../gems/activesupport-3.2.2/lib/active_support/dependencies.rb:251:in `require'
.../gems/activesupport-3.2.2/lib/active_support/dependencies.rb:251:in `block in require'
.../gems/activesupport-3.2.2/lib/active_support/dependencies.rb:236:in `load_dependency'
.../gems/activesupport-3.2.2/lib/active_support/dependencies.rb:251:in `require'
.../gems/refinerycms-copywriting-2.0.1/lib/tasks/rspec.rake:1:in `<top (required)>'
```

I moved the `rspec.rake` to `Rakefile` to avoid this.
Merged master and updated jpeg method to handle params
List concatination -> Binary concatination
Some code improvements + unit test reproducing an unsolved bug.

Hello I fixed a couple of small issues:

* commit flavio/terminal-table@e6af300d0646ad736666: X,Y and I handling during column width
* commit flavio/terminal-table@658609bee746ce60a56e: borderless tables rendering had rendering issues. Rows containing separators were rendered with an empty line

I also found a rendering issue that I have not been able to fix. This is highlighted with a unit test provided by commit flavio/terminal-table@57615ad264f9b7859882. Unfortunately I have not found a solution yet.
Issue 6013 - private ignored for aliases

http://d.puremagic.com/issues/show_bug.cgi?id=6013
Elimino método utilidad del componente de menu

Los que no son miembros de venezuelacert, ignorar el mensaje.
Start of Multivariable analysis 

RGA and SVD with frequency dependency introduced.
bumped cabal dependency for unix for ghc-7.2

Would be nice if access-time could be built with ghc-7.2.

Cheers,
Sönke
What do you think about using gnome-shell-extensions team repository https://launchpad.net/~gnome-shell-extensions/+archive as official repository for Ubuntu?

Hi Christian,

I think it's better to use "gnome-shell-extensions" team repository https://launchpad.net/~gnome-shell-extensions/+archive/ppa as official repository for gnome-shell-extension-weather instead of yours private (ppa:xeked/gnome). Most Ubuntu users don't trust private repositories, also it would be easier to fix problems in packages for other gnome-shell-extensions team members, who really use Ubuntu, like me :)

I've already configured daily builds in ppa:gnome-shell-extensions and you have administration rights in gnome-shell-extensions team, so you can always do what you think is better :)

Btw, I also simplified the instructions for Ubuntu - key is automatically imported when 'add-apt-repository' command is used and mentioned compatible distributions, like LinuxMint and Debian :)

Please accept my improvements :)
Fix the help link.

If Redmine is installed in a virtual directory (ie, redmine), the help link will point to "redmine/redmine/plugin_assets/redmine_redcarpet_formatter/help/markdown_wiki_syntax.html".
Little bit of refactoring and a PublishedModelDetailView
Should this salt be put into the .env file?

If I'm understanding correctly, this "secret" var is a salt, so should it be put in the .env file?
I added my recently released Clojure v3 API library, tentacles, to the list of libraries
More Robust CI Script

Update spec/bin/ci to take several options for running rabbit and
running only one of the test suites at a time.
Store raw mention object in mentionsCollection

Hey everyone!
In this commit I made it so that we store the direct object that the user gives us instead of plucking values from it.

I also fixed a naming inconsistency between mention.name and mention.value.
[WIP] Some refactoring
merge in minor style edits

merge in minor edits; fix loading of google font
Improvements to handling of gallery pages.

* Detection of start page should now be handled on a per site basis.
* Imgur galleries now handle both styles of page hashes.
* Clicking on a gallery image can now go to the appropriate image in
the real gallery directly if supported.
Fix shortcut button /r/reddits

Per [beta thread](http://www.reddit.com/r/RESbetateam/comments/x0qwr/chrome_20_res_413_beta_c_shortcut_not_displaying/).
Client does not create a valid signature when spaces are in the URL.

I noticed while trying to register an ssh key with registerSSHKeyPair that the client was generating an invalid signature when a space was added in the parameters. The API is expecting spaces to be encoded with %20, not +. This commit replaces the encoded +'s with %20.
Payment History to WP List table Upgrade

I've upgraded the payment history page to use an instance of the WP_List_Table class.

This enhancement includes a payment history search, related to #518.

It also moves us one step closer to moving all UI elements to core UI #299.
Fixes to enable and enforce SharedEventManager instance

- Instead of using StaticEventManager, pull shared collection for event manager
  in module manager
- Instantiate and injected SharedEventManager in index.php (we can refine and
  abstract this later)
List new operations

Agrego las siguientes operaciones a la lista, list_take(...), list_take_and_remove(...), list_filter(...) y list_map(...)
Update to work with ejabberd 2.1.8

update mod to work with ejabberd 2.1.8.  At some point they changed to passing around bitstrings rather than lists
Upgrading to node 0.6.10
Changes for click handling

Here's my edited version of jquery.notification.js, for the world to see.
Fix for web2py version 1.99.4

The storage module is actually gluon.storage, at least in web2py version 1.99.4.
Fixed get_field_value call

The get_field_value call was done on the wrong object and caused an error as that object didn't have that method defined.
add recovery: enable building recovery

everything's working: adb, mass storage, charging, ...
There's missing the kind of adapter you're looking for when retriving images.
fix small typos in search preference page
add some examples of field creation and deletion to the update API
Changing default scope from visualforce to web.

This should have been in with the frontdoor change.  Web scope is now necessary for all apps originating from the web.
Issues with 0.5.0 in Rails

* Fix: `const_missing: uninitialized constant Delayed::DelayMail (NameError)` with Rails.
* Fix: do not run job when a Mongoid instance is not found within delayed_job arguments and `Mongoid.raise_not_found_error` is set to `false`.
* Prettified .rspec.
* Added integration tests.
Only show online player markers, discard others (like offline players)

This will help to prevent showing any markers that are not specifically "Online Player" markers (id: 4) from plugins where you can't easily remove them.
Integrated Disqus into the site
Labels etc.
added Configuration class to enable support for config:dump-reference
Core/Opcode: Resubmitted Pull Request from SkyFireEMU/4.3.x
Add missing jar so insight can be launched.

Fix build. Jar was not included in ivy.xml. I will create a ticket to improve support of missing jar.
FIX: keep the error and error string when a request produces an error

When a request produces an error response in the execute method it saves the error_code and error_string in the current instance but both get overwritten by the set_defaults method. This is a small fix to keep the error_code and error_string.
Use `user-emacs-directory` instead of `~/.emacs.d`

Like in other places, use `user-emacs-directory` consistently.
Lualatex statt pdflatex

Unicode ftw :)
Update native/SalesforceSDK/src/com/salesforce/androidsdk/app/ForceApp.j...

...ava

No idea how that space got in there. So sorry!
Change from deprecated dynamic property to .ext

Working through the book with the latest version of Gradle, I am changing code as needed for deprecations and errors.
Mylyn poc tests

Adding functioning POC tests for querying bugzilla repos.
Avoid capturing self, refresh data on becomeActive

- avoid capturing self in a block with asynchronous task
- in JobViewController show picker from tabBarController.view, so the cancel button won't ignore touches on the bottom of it
- some code reordering for easier reading
- refresh twitter and jobs when app became active
- more code reordering for easier reading
Test Fixes

portlet/assetpublisher
portlet/documentlibrary
LPS-15936 Activities Portlet feed to display bookmark entry from Bookmarks Porlet
Pulldom

...ady did last commit.
Mockups

Gráficos principais da aplicação mostrando interações de mouse over

[ ] Desenhar interface para operação de tecer
Small tweak in template password_reset_message.txt

Better to use the view name instead of the view path. That way if you have a custom password_reset_confirm view, you can still name it 'userena_password_reset_confirm' in your urlconf, and don't need to override the template.
Fix a few more indentation errors.
Convert the test cases that are used in our Fluidity branch to Flmls

All the test ufl inputs that are tested (using MMS, MES, or whatever) in our Fluidity branch have been converted to flml files based on the Fluidity schema. To create a new model, you open up diamond with the fluidity schema from the launchpad ufl-input branch, and for a field, change it's equation type to "UFL" and then type your UFL in the box in the GUI.

The changes that have been made (mainly to do with the testing, a couple of bugfixes as well) are:

1. fluidityequation.py - getFieldFromCoeff would return fields on the same function space as a given coefficient that were in the state but weren't necessarily used in the UFL and were not on the GPU, so they weren't suitable for being copied into new, temporary fields. This method now returns the name of a field that has the same function space and has been accessed, so will be available on the GPU.

2. Since multiple fields can be present in an FLML, there can now be multiple output files. The frontend now creates a folder for the output, based on the name of the input, and the output files are saved in the folder with the name <phase><field>.{cu,cpp} etc.

3. When parsing UFL files, the input parser creates FluidityEquation objects whose name is now "model" - previously the name of the equation was the empty string. This is so that an output file gets generated with the name "model.cu" in the folder named after the input file.

4. The autotester has been refactored to deal with the fact that there may be multiple output files for a single input file. In particular, the AutoTester class is now the base class for a tester, and the SingleFileTester and MultiFileTester are descendants of it.

5. It turned out that the exit code returned by autotest.py was not being set correctly, since a while ago - it would only return 1 if the very last test failed - it now returns 1 if any test fails.

6. Addition of the flml files for noop, identity, helmholtz, euler-advection, diffusion-3 and simple-advection-diffusion. Removal of the ufl files with the same name.

7. The expected output for each test case that was converted from ufl to flml has been changed to give the correct expectations.
Default grub

To reproduce:

1) Add some items in the boot menu
2) Make the last one as the default
3) Delete items before the last one
4) Synchronize

=> default option is not correct without this patch
Test Fixes - 01/25/2012

stagingcommunity/assetpublisher
stagingcommunity/blogs
stagingcommunity/documentlibrary
stagingcommunity/sites
stagingcommunity/webcontentdisplay
stagingorganization/blogs
Sorting change

Really small update. Sorting on yes/no didn't make too much sense in this context, so I changed the field to sort on the `order` column instead. This allows you to get a list of entries back where the box is ticked (e.g. a "Feature on the homepage" checkbox) but also sorted by the order in which they were featured. No need to cobble together a checkbox and order entries to achieve the same spunktionality.
delete the DS_Store file
otter: Readme - fix ICS reference

Quick readme fix
2.2 & 2.3 review

2.2 & 2.3 review
Update admin/es-ES.com_hwdmediashare.ini
Project compilation and Restkit Setup

Two commits to pull Restkit in as a submodule, project config so the unit test(s) run.
.NET 4.5 -> 4.0
Упрощаем prepareData
enable-flash feature should be ignored in access elements

Fixes blackberry/BB10-Webworks-Packager#181
Addresses #3

Reorganized into a plugin and proper extending
Failing test case test_with_content_type_args

Looks like whitespace error. Does this test case fail for you without this patch?
Dev

Fixing errors
last pull of this outdated app
added link_to in others fields

in partial _issue, added the helper link_to in project name and subject to issue.
Routing parameters in generator.yml object actions

These changes allows adding parameters to object actions. For example:

    # yml
      actions:
        list: ~
        special_action:
          label: Sample Action
          route: Some_sample_route_rule
          
          # added key
          params:
            # It is possible to send variables...
            id: "{{ Post.id }}"
            # ...or hardcoded params
            desc: Test description
Decorates the attribute classes with the AttributeUsageAttribute, to restrict usage of the attributes to their intended targets.

Although it's kind of subtle, I think it's better to have the attributes express their intended use as much as possible. This commit will cause compilation errors when applying the attribute to targets they're not intended for.
Updated github link for vim-kolor coloscheme.
Added more details to createEvent() readme section
Feature/jQuery QRcode

This removes the dependency of the ruby rQRcode plugin. It uses a jQuery plugin and includes the jQuery source to avoid the dependencies of a CDN.
Collapse transactions, loop sleep

I've made some changes to prevent the bot from lagging behind when there's lots of updates, by collapsing updates (grouped by market-currency pair) above a certain threshold. neofutur is running this in #bitcoin-RT on freenode at the moment, if you want to see it in action.

- Add option to collapse transactions when there are many of them
within one update. Prevents backing up of message queue, which
would make information outdated. Threshold editable as config
parameter.

- Add small delay of 10msec every loop iteration, to prevent 100%
CPU usage.

- Some extra documentation/error specification
Add extensible character sheet tabs

This code should allow extensions to easily define the character sheet tabs. By overriding the charsheet_main and charsheetmini_main classes, the two main sheets can be modified. Or these tabs can be removed via the CharsheetManager library, which also allows new tabs to be registered and ordered correctly.
Fixed memory leak

The image data is not released after object deallocation. This causes massive memory leaks. I added a callback function to fix this.
Closing the Application

Implemented Issue #3.
Fix bug in showPostPreview
Removed deprecated class UnicodeFileToHtmlTextConverter.java All you can...

... need now is class Text
Marissa

Added a shooter with three speeds and a shooting joystick
ソースコード実行結果をPharo1.3で確認。振る舞いの異なる箇所に訳注を追加。
Thanks Montellese. It is always good to get rid of hardcoded strings. Cheers
J'ai traduit le terme avec la signification d'origine. C'est bien de l'agent e-mail dont on parle ici, car les geek barbus utilisent généralement Mutt comme agent (et non comme boîte mail). Il y a peut-être matière à éclaircir cette partie, peut-être en utilisant le terme "client e-mail" ?
直接在页面js里维护input的name值了。因此这里的参数准备可以省去了。
Thanks, removed in 96a137a.
Is this exploitable? With a content-type of `text/plain` the response should not be getting interpreted as `text/html` or anything else.
Pattern like: "#{key}: #{value}" is all over the place and can cause escaping issues. My currency format is: "%n&nbsp;%u" and I see plenty of places, where escaping is broken. If we agree on some solution, I can try to fix them.

Don't care about escaping is really nice rails feature and rendering raw string undermines this idea.
"Most other commits in this branch may have no visible effects": the most visible effect is that they won't be reported by cppcheck as errors anymore (it might useful regarding http://qa.debian.org/daca/cppcheck/).

Agree with the bugfix.
Just for future reference, you should have used `==`, not `is` here, as `is` comparison of small integers is just an implementation detail of CPython, and would break in something like PyPy.
Up to you. What caught my eye is that horo has 2 forks only while YARD 72. Hope we won't end up in a situation like rspec-rails-matchers.
Are you using this function in the previous commit?
This generates pretty beautiful code, cool!
looks nice, i've been waiting for this for so long, good luck
shouldn't the parameter name be 's'  for 'skip'? Or even better 'sr' for 'skip resolver'?
As opposed to what?  How would you write it?
Tai oikeastaan cbNetworkin Logger-utilityssä minkä koodasin. :P
so testable!
is this really a warning? we expect to see this sometimes, right?
There is a bug, since you are storing the global in TmpVal, second parameter should be TmpVal and not getAccu()
fixed in this commit: https://github.com/kohske/ggplot2/commit/b67336d6123a16c4de0bb54c5d2d21267d20535a
Если это попытка сделать получить сообщения за неделю месяц и т.д. то неверно
Before merging, I'd probably change "destination" to an optional parameter and use the object_id if ommitted. I'd also prepend / in this case, in the same way __file works. Consistency is a very good user experience :-)
Good call, we will need to figure this out as well
Oh sorry that's what I get for coding late at night.
getTask() is always returning a non null?
Yeah thing is there were no "conlficts", Wlly merged some new code that was written over an old fork of mine, so basically, I had to push my changes again. Anyways, nice to see u around Raist.
>
>
>
> Until 3-4 months ago only old way was working. The callback URL should
> point directly to callback.php (
> http://your_playsms_website/plugin/gateway/clickatell/callback.php).
>

I was trying it yesterday, and it worked fine with the new method using the
variables, I could send text messages, and get the delivery notifications
just fine with the new url (
http://my_website/playsms/index.php?app=call&cat=gateway&plugin=clickatell&access=callback
)

I must say that my account is only a testing account at the moment, but the
behaviour should be the same.
Same here, this line can be removed.
oh I totally did this and with a transform - and then I got rid of it.
Gracias  sergen ^^ eres el mejor tio xD
I think this behavior is actually better, where it prints the stop value rather than the endpoint.
The Bool example is actually a really powerful argument for doing all arithmetic as Int64. It kind of distills the problem down to its essence. But Int8 is really not fundamentally different. Nor is Int32, of course, but the range is big enough in that case for it to be ok.
Because we only need the dateAdded field filled for tvshows and not for every path. But I can move it into AddPath and get rid of the XBDateTime.h include in VideoDatabase.h
stimmt...is ne Idee, hat ich wieder ne denkblockade;)
I believe `have_content` was meant, not `have_conteXt`
Do we really care about that?

What I imagine here is that you return True instead of breaking the loop, and false at the end
вроде разобрался ето спелл ачивки SPELL_ACHIEV_CHECK                     = 58630
но его нету в дбц
Yup.  Very sure.  Only exception is when you're checking the type something that is not in the local scope, like myGlobal === 'undefined' would result in an error.
hi, I have tried with view.on('didInsertElement', this, this. myDidInsertElement), and not worked, but as soon as I see the callback I tried view.on('didInsertElement', function() {}); and worked. Thanks. But in myDidInsertElement, I checked is the view was inDOM, and the result was confusing, because was in preRender....is this behavior correct??
couldn't you just `return` here?
Works well with 12.01. Thanks for the good work!
zeros_like will preserve F-order, whereas this changes doesn't, Maybe we want np.zeros_like(f, dtype=otype)?
How about (reinitialize-instance digest) instead?
you could have also used:
    print
In order to fulfill the new ARIA specs I needed to remove lazy refresh (on first usage) because otherwise I have no ID attributes to use with aria-activedescendant.
I think that 
```cpp
static JavaVM *gJavaVM = 0;

jint JNI_OnLoad(JavaVM *vm, void *reserved)
{
    gJavaVM = vm;
    return JNI_VERSION_1_4;
}

void Java_org_cocos2dx_lib_Cocos2dxRenderer_nativeInit(JNIEnv*  env, jobject thiz, jint w, jint h)
{
    if (gJavaVM)
    {
        JniHelper::setJavaVM(gJavaVM);
        gJavaVM = 0;	
    }
```

can be changed to
```cpp
jint JNI_OnLoad(JavaVM *vm, void *reserved)
{
   JniHelper::setJavaVM(vm);
    return JNI_VERSION_1_4;
}

void Java_org_cocos2dx_lib_Cocos2dxRenderer_nativeInit(JNIEnv*  env, jobject thiz, jint w, jint h)
```
What I don't understand is that the bundle seems to be installed correctly even when given an array.
I'm aware of this and I'm not sure.

I don't know what's a proper way to handle these. You definitely do not want to internally `clone` / `new` / `[]` / `{}` unless really unavoidable (i.e. when stuff created this way will stay; you don't want side-effect of methods spawning new objects just to be thrown away).

I solved it like this here as this is how we do this elsewhere, so at least it's consistent. And you are absolutely right that this could lead to some hard to track bugs, if you don't know what's going on. 

With this pattern it's always responsibility of caller to do a local copy of the value if it's not used right away:

```javascript
var relative = object.position.relativeToAbsolute( object2 ).clone();
var absolute = object.position.absoluteToRelative( object3 ).clone();
```

Or if you do this in a loop and you need to keep intermediate results:

```javascript
relative.copy( object.position.relativeToAbsolute( object2 ) );
absolute.copy( object.position.absoluteToRelative( object3 ) );
```

So far this worked ok, just you need to be aware it works like this (so far there was a single issue posted and poster figured it out himself).

I chose this solution as lesser evil because everything will eventually end up being called in the loop and will cause garbage collection hickups. 

If we hardcode cloning inside, there is no way how not to have it. While if we return internal variables, we still can achieve the same effect in other way.

Another option I considered was to have second parameter, which if undefined would lead to creating clone:

```javascript
var relative = new THREE.Vector3();

// this would use supplied vector
object.position.relativeToAbsolute( object, relative );
```

```javascript
// this would create clone
var relative = object.position.relativeToAbsolute( object );
```

But this is somehow uglier API (and inconsistent with how we deal with this issue elsewhere). Also if we switched to this, it would promote inefficient code. Everybody would end up doing clones everywhere simply because it looks neater.
well 226 + 30 = 256
on the other hand it looks like a security-calculation to not reach 100%.
Probably you are right. We could enable the use of 100% but it could cause overheating and hardware damage.
I will change it back.

Greetings ;-)
huh? fix or remove the test
Ik heb deze commit gedaan, ik weet niet waarom hij unkown aangeeft.
Did the above comment auto-submit?
We spent much time yesterday diagnosing this and similar errors (rake is also affected). I didn't realise that Rack was specified in puppet.

The plan is to fix it all to something sane: find a combination of Passenger, Rake, Rack, and RubyGems that actually works (because not all permutations do) and then make all the apps use that.
you're right.. the links should not be replaced or rex_getUrled in the first place..
i checked, and it doesn't happen in all installations.. however it _does_ happen in my main dev install.. need to drill down why..
Para que se pueda mostrar la foto del avatar de Twitter.
How about that we just use the symbol "-", not to translated the "dash" to the Chinese vocabulary in TEXT?
Ex. 而「-」表示沒有權限

To SPEAK, it's the job of the reader.
[jQuery.each](http://api.jquery.com/each) calls were being exited prematurely by `false` being returned since [CoffeeScript insists on returning something](http://coffeescript.org/#expressions).
How about:

    if not isinstance(s_bytes, basestring):
        raise TypeError("s_bytes is req
Would it be possible to make the used dump configurable, e.g. through an environment variable? Then we can use data similar to mini_dump.php in our service stubs and later run the tests against a Legacy implementation with exactly the same data.
Évite de versionner les pyc (et les binaires en général si tu peux), git n'est vraiment pas fait pour ça et ça va faire augmenter violemment la taille de ton repo. Tu peux utiliser un fichier .gitignore pour dire à git de ne pas versionner les pyc (mais il faudra que tu les git rm manuellement avant)
1. Ok, but it doesn't cause any harm and it would be resilient to a future change, no?
2a. I don't understand this... "get in fact local variables in function FSwitch()".  Where?  That argument goes into a:precmd and merely gets 'executed'.  The b: seems quite necessary.
2b. I also don't understand this... "may lead to error messages about non-existent variables in some specific cases when option bufhidden is set to delete".  Example?  The lifecycle of these variables is for the single 'execute' so the buffer can't go away during the time of the execute and, thus, bufhidden isn't even involved.
I don't think this is the problem, because getSecondLevelCacheStatistics().getPutCount() gives me correct numbers.
I actually didn't submit it to npm. I will asap.
Found it - your calculateLaunchSpeed() doesn't currently take into account that a "launch player" should also be a valid text!  "launch player" means the minecart should take off when the player hops into it. That part probably already is handled somewhere else in Afforess's code - but currently, calculateLaunchSpeed() only expects to find "launch north/east/south/west", therefore throwing endless amount of Java exceptions, when I write "launch player" to my sign.

I could try and create a pull request for you to fix this - but I think you can fix it far sooner than I can.
We are going through the contract to figure it out... I doubt it is right though
lockManager might be null
I didn't merge it in because I wasn't sure of the best way forward. We might need to discuss the details off line. Flesh out the details and merge it in when it's ready.

Sent from my iPhone

On 27 Jan 2012, at 18:50, Luke Page<reply@reply.github.com> wrote:

> Is it ok for me to pull your master branch into dotless/master? I'd love to build upon this rather than re-invent or go at cross purposes..
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/jamesfoster/dotless/commit/cfe17179d6819d3e3c4e9e2eb04748c0190ce83b#commitcomment-909226
I use ytdb so all bugs related to DB other than ytdb you ppl will have to resolve on your own.

I switched to ytdb when I started using rsa's fork. Back then, in UDB there were no spawns in instances like ICC or RS, Icecrown was missing phases etc. etc. It lacked a lot of stuff, I dont know how it is now.
And I agree to alexluana, UDB is more blizzlike, its team's work style differs from ytdb's.
But what the hell do I know about it?;p
Very nice, glad you fixed that again.  For some reason the white space and explode function did not get along well with setting Mode type and would throw error just like you didn't set it.
Since this is in a module called 'costs', it's a bit confusing to use the word 'error'.

I think it would make more sense to call these SupervisedCost and UnsupervisedCost. To avoid having more submodules we should put them in pylearn2/costs/__init__.py.
Thanks for that. I'll incorporate it when I've got half an hour. What were the problems that you encountered? Anything I should know about, or just that we should be delegating everything to the original security manager?
Removed in the next changeset.
Still missing in Avisota 1.5.1 stable	#27 please promote to ER
needs more p
I think I only added a new TID out of paranoia, to make sure that heim_release() works right, but I don't really need it, so I'll just remove the new tid.
Ok. Removed it from my master branch.
Try this on documents where hard-wrapped paragraphs are indented by four or more spaces, eg. as part of a nested list.
That still doesn't build.  CStringTCharFromWChar has an implicit conversion to LPCTSTR, but not LPTSTR, which is what ActionExec accepts.  An explicit cast is required to remove the const modifier.  It compiles in Unicode configurations because the macro just returns its input unchanged (hence "IfNeeded").

Generally it is preferred to limit each commit to one distinct set of changes.  (I guess you'd call it an "atomic commit".)  If you must commit multiple changes, the first line of the message should reflect it since that's often all that is visible.  In this case I was expecting only fixes for build errors, but there was also a change to behaviour.
"" looks wrong here.
It should be pre_text at least, probably plus some additional whitespace iirc.
Please use tabs here.
Add the `layout` parameter here:

`private static String format(ILoggingEvent event, Layout layout) {`
It needs 4 whitespace, you have two here.
можно писать без скобочек: delete stack;
That way does look easier to read.
这个时候还在为开源代码工作的人伤不起啊。
- Clients can easily scale their own apps without needing a deep technical understanding or writing any code.
Yes, following line allows empty p tags (&nbsp;). It may be that tinymce poorly handles convertion between the old behaviour, can you provide sample document?
memory leak => 메모리 누수 정도로. 윗단의 "메모리 관련 문제"도 누수로 통일하면 읽기 좀 편할듯
Sure, we can do that. In the meantime, you can test on your own code from
6.005.

On Sat, Feb 25, 2012 at 6:18 PM, jdhenke <
reply@reply.github.com
> wrote:

> No. I have not run anything. This was all just fixing compiler issues.
> Maybe on Monday we can setup some sort of testing data?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/uid/caesar-preprocessor/commit/2f682b0f3f2a294142c31b4f6ec6073d03534e65#commitcomment-1011621
>
Did you actually change anything in this method?
@mattleff @KendallHopkins I think I'm actually going to revert this b/c bootstrap uses ```@linkColor(Hover)``` in other locations such as for tabs/pills, and dropdown carets. The proper way to handle it would be to change the @linkColor var in variables.less, but this can't be done on a per template basis. wdyt?
Atm on trinity if you click "yes" to unlearn talent points after clicking "Unlearn Talents" and you have no used  talent points you will get error message "You have not spent any talent points.".

On offi when you click gossip option "Unlearn Talents" and you have no used talent points you get error "You have not spent any talent points.".

So the error checking should go before attempting to wipe not after.
thx, fabi!

How to calculate it?  :p
У тебя почти идентичные формы на странице edit and new - так быть не должно. Код не должен дублироваться (в частности здесь у тебя в обеих формах не валидный HTML: абзац закрывается в блоке semantic_fields_for). Если же формы должны отличаться то есть несколько приемов: 
 - проверять есть ли объект - f.object.images.any? - это для кода Line 6-11
 - проверять action_name (или даже controller_name)
 - можно просто передать переменную и в зависимости от ее значения менять поведение формы
Would you mind moving GraphSvgDisplay and related SVG graph specific classes to a subpackage, e.g. graph.widget.svg? or to graph.svg_widget?
Pet peeve. Use your own. Things like on (e) -> for the event callback is common practice, but ignore my other name changes.
@mdesrosiers right now building locally sounds like the thing to do.  I did find [this article](http://guides.rubygems.org/run-your-own-gem-server/) which describes how to run your own rubygems server.
I'm wondering if the differences in the font size might be related to the platform(s) we're using, or maybe the versions of showoff?  Or perhaps the browser?

I'm using MacOS X 10.6.8 with Chrome 17.0.963.56 and version 0.7.0 of the showoff gem.  Maybe I need to update my version of showoff?
Inconsistently calling `stroke` before `closePath` or vice versa
Of course. I wasn't sure if I was going to add more to it, but it doesn't look like I have much time right now so it's probably better to pull the small update.
Why you don't written it's device with my changes for latest CM sources?
I could do this (see code below), but it will only work with burp set to support invisible, I think this is because data is just sent without negotiating the ssl session first and for this it would be better to just use CONNECT

```
if (self.ssl)
				prefix << "https://#{hostname}:#{port}"
			else
				prefix << "http://#{hostname}:#{port}"
			end
```
Sorry, I was half asleep while I was typing the commit message.
Woah, completely missed this! I'll take a look tonight :-)
to then use the 4.32 and the db where the sack?
'rank', 'rank',
single quotes, geen double (double quotes = trager, PHP gaat die parsen)
How can I hide some datas in github?

On Fri, Feb 17, 2012 at 1:24 AM, Pieter Colpaert <
reply@reply.github.com
> wrote:

> What are these? Are these keys you should keep private?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/iRail/BeTraffic-Android-Client/commit/78ca9bd12b52c4dec55993710a7cc911123239b1#commitcomment-976061
>
Small things here.  I think the annotation should be named "Feature" to stick with existing conventions of the plugin.  I would make the 'feature' attribute named 'name' instead - again, keeping with convention.

Is there really a case for anything but a 404 response code?
Rogue line?
What, you don't like tacos? ;)
I only noticed it when I copy and pasted from the example.  The change works very nicely, if you're auto-completing users then you could include their icon and more details.
It would be worth updating this to explicitly state that all "callable" entries are allowed where you say "method-name".

It might also be worth creating an "extending the template engine" doc somewhere, even if it's just bullet points and links to the right parts of the API docs.  It may seem lower priority, but it's easier to spend 15 minutes writing some crappy documentation now, rather than forgetting what we're supposed to write up in 3 months time. ;-)
type attribute redundant in HTML5.
Ooo, interesting. I like it.
Hmm, yeah I guess not. I'll go ahead and remove that check.
If You mean that in the above case if dimensions are 1280x720 they are remapped to surprise 1280x720 :)

....
static const camera_size_type preview_sizes[] = {
    { 1280, 720 }, // 720P
    { 768, 432 },
};
....

which obviously is pointless, you are absolutely correct:)

it was for fast testing, i plan to remap to something else with almost the same aspect ratio.

in my testing overlay-ing to 1280x720 works, but can't focus properly... so further testing will be with something different (picked from a list of the "possible" values preview-size-values=176x144,320x240,352x288,640x480,720x480,720x576,800x448,848x480,1280x720) with about 1.77777 aspect ratio. If that helps focus (as wide 848x480 seems to focus very well), will dig around the same about still pictures (in my opinion they are all slightly out of focus)

of course (at least for now) all that is just speculation, based on theory that focusing is based on the contrast of the displayed "overlay-ed" picture
I'll go ahead and do this change, but keep the original value commented for reference.
Right now I am using jQuery's .submit() on a submit button and although jQuery's API tells me that I am not supposed to do that it works most of the time. I really want to stab myself in the face now.
Didn't liked that as well, thought it could be useful.
No exception handling - what happens if you upgrade your version of python or are using multiple versions of python for things?
I'd prefer if we used escaped double quotes here (instead of single quotes) to be consistent with the steps below.
I think we should protected against bad input here so we get `-1` instead of an error: `model && model._isYUIModel`.
@lsmith There's a plan to add the concept of row selection to DT, right? (Maybe it is already there?)
Rename "temp" to something more meaningful such as extendingProperties.
It seems that what you are doing here, is creating a brand new dialog EVERY time the user clicks the edit device button.  This is very inefficient and potentially prone to memory leaks or state confusion.  I would prefer to see a static dialog pre-created (or better yet, created the first time it is needed) and then update the text fields to match the current context.
val committed = available.collectFirst { case ("heap-memory-committed", b) ⇒ b }


Do the same with the ones above, a bit easier on the old eyes :-)
It works just fine, it's a very common Scala idiom
Can't we detect this so the need for the boolean disappears?
ok. I just took across the way headers was done by the dude before me. Will bring the amount of similar methods down there too.
table.get.get(resource) match {
  case Some(r) => r.isAvailable
  case _ => false
}
/^\s*\#?\s*require\s*\(?['"]sinatra['"]/ is more accurate and easier to follow IMO
But it needs a test if we want to keep it and use it.
oh, I get it.  yes that's better.  that kind of thing could go  right into develop if in its own branch
need whitespace after
Diff didn't pick up right, so here is a change.
We've talked about having the sphere be "immutable by convention" because we want to add a hash to it. Also because the sphere has @auto_attr properties, we'd need to reset them all if we changed the sphere, but I think there's a way to dot that.
And the code reads pretty well, btw :)
There are at least 4 calls to `$('#user_follow_button')` in this method. A more efficient method is to cache the element in a variable at the top of the function to keep the code dry.

```js
var button = $('#user_follow_button');
if (button.data('state') === 'follow') {
...
```
blank line below here for PEP-8
Agreed, not sure why I didn't.
The code must have changed since this comment.
I say we put this all off and put the data in a hash.  This current migration will have conflicts on several sites, so as you say, it's worth backing out until we're ready to do things rightlier.
Fetching all current data on a table. When it finished, it loops over and starts again with the oldest current data that is not null.
Is this broadcasted to the bindings elsewhere?
That's why I do Query(true) here. After this GetCurrentOutput returns the resolution the user has set to with xrandr. The assumption is that when a user uses xrandr he/she wants this new mode as the desktop resolution. XBMC would then switch to this new mode when going windowed.
Yeah, unrelated.  I can pop it in separate if you want.
Добавь еще один коммит с изменением на меню.
Hey Thomas! 

First, this is great! Awesome pull with tests :)

One comment - I think that this method should be split into two:

     getDirectDependencies()
     getEffectiveDependencies()

Where `getEffectiveDependencies()` actually checks the project building result and determines the dependencies from the built project meta-model. This would be more consistent with the DependencyFacet.

WDYT?
~Lincoln
This is weird. Also, this header should only be added on HEAD requests.
Why calling val.substring twice?
`[object Object]` ?
Min-width style.

You can test it you by running `make package` in cloud9 OSS and then starting the IDE in packed mode: sh bin/cloud9.sh -P
Not necessary, but it can be very helpful.
Can you split this into multiple lines? It's a bit confusing to read a ternay and an if on the same line.
That was the intention, but it was never implemented, and won't be as part of this pull request.
Perhaps you could honor the `animated` parameter and use a loop to push every view controller but the last with no animation, then animate the last one.
We should remove the explicit ivar declarations and `@synthesize` statements, and then this will happen automatically.
Is swallowing errors a good idea here? Shouldn't they be logged or emitted to somewhere somehow?
Oh I see - empty task string is not supported, but is still possible. Maybe we should physically talk instead of annoying Dave and Andy...
Could we move the code block added below to a separate method to keep this method simple?
True, that's worth a try. Would be cleaner for sure.
why you need locale.h?
это сокращенное PathStroker. исправить всё равно?
yeah, I agree it's kind of weird ... do want me to remove it? 

I'll fix a new empty line that just [got in] (https://github.com/rails/rails/commit/b2a59388b2ad281ccce1f72dd5fda09ca746dc32#L0R29) today anyway...
Could you please add a `# :nodoc:` directive here?
Yeah, seems better, thanks mate.
Anyway, don't worry, the middleware is not added if the config is `false`.
short-circuit return = :(
Good eye. Fixed.
Should not be `DISABLE`? Also put space after the `{` and before the `}` just like it was. The parentheses is fine too.
```
if ...
  super
else
  actual_last = ...
end
```

would be nice to get rid of the short-circuit `return` entirely
Forgot to add the inet/cidr one originally, I don't see it anywhere in the changelog now. Please correct me if I'm wrong
https://github.com/rlr/standup/pull/78 covers fixing the tag issue.
What the hell? Removing the `--break` option?!
right thanks
I'd maybe move these to inner functions to the test_* routines --- then you can use variables from the outer scope rather than passing them via yield (which pollutes the test output with unnecessary information, which, being arrays, doesn't print correctly).
It seems (I will return in the evening and check) that for  `trigsimp` to be enable handle with it , it is needed to implement such equivalence to be True.
On Thu, Apr 19, 2012 at 1:05 AM, Aaron Meurer
<reply@reply.github.com>
wrote:
>> @@ -92,3 +99,12 @@ def __init__(self, value=None, feature=None, last_supported_version=None,
>>
>>      def __str__(self):
>>          return "SymPyDeprecationWarning(\"%s\")" % self.fullMessage
>> +
>> +    @property
>> +    def warn(self):
>> +        from sympy.utilities.misc import filldedent
>> +        from warnings import warn as warning
>> +        msg = self.fullMessage
>> +        see_above = '\n%s\n' % filldedent(msg)
>> +        # the next line is what the user will see after the error is printed
>> +        warning (see_above)
>
> This needs to be changed to use SymPyDeprecationWarning.  By default, it uses UserWarning.

I changed that.
What are the limitations that you're running into?
This method is just a helper function in density.py. I have removed the _generate_outer_prod method. Therefore, this method can stay and continue with the same name.
Perhaps a better example regarding laziness is the following
```
In [4]: A = ImmutableMatrix(ones(3,3))
In [5]: B = ImmutableMatrix(3,3,range(9))
In [6]: (A*B)[1,2]
Out[6]: 15
```
or
```
In [2]: Transpose(MatrixSymbol('X', 3,3) * MatrixSymbol('Y', 3, 2))[1,2]
Out[2]: X₂₀⋅Y₀₁ + X₂₁⋅Y₁₁ + X₂₂⋅Y₂₁
```
These only compute what is necessary to get the answer. The sub-expressions `A*B` or `X*Y` are not computed in full. I don't think that FunctionMatrix (or whatever it will be called) is the only lazy Matrix type; rather I think that all of MatrixExprs are lazily evaluated. It could be that my definition of lazy is confused though.
I'd read over PEP8 and adjust many or your variable names. I don't think the leading caps (or caps at all) are preferred. mixedCase is potentially, but these stand out to me.
Undo the changes. Then

    git add Gemfile
    git commit --amend
    git push -f origin subcommand-fixes
This is not related. And its only first. Second is this construction should be rewritten:

```
$checked=is_array($select) ? in_array($value,$select) : !strcmp($value,$select);
```
tried taking it out. however the head_width and position are used for rendering.
I looked at checking the type and it started to get a little ugly.
so I've left the code in without the code_head attribute.

no longer a durationToGlyph metrics... 

I had originally thought about putting the function to draw the note head in this table... and then it would all be in place in case other note heads need to be manually drawn.
Can we move this match into a static function in CacheManager?
"should used" ?
this could be 

```javascript
// Nodejs lib.
var fs = require('fs');
var path = require('path');
var existsSync = fs.existsSync || path.existsSync;

...

// Where might a locally-installed grunt live?
var dir = path.resolve(gruntfile, '../node_modules/grunt');
// If grunt is installed locally, use it. Otherwise use this grunt.
if (!existsSync(dir)) { dir = '../lib/grunt'; }
```
Just to avoid having to declare this (and the above function pointer), I would suggest using `imp_implementationWithBlock` instead.
I'd just use `remove(name)` here. It's portable and less code.
It *probably* doesn't matter here, but environments exist that allow assignment to `undefined` (which is why the [jQuery IIFE](https://github.com/jquery/jquery/blob/master/src/intro.js#L14) uses it as the name of an unprovided parameter. This check still works under those circumstances.
link to your new capped collections page here.
"at the array level" is not a formulation we use.
Totally. Consistency is important... I just don't know which should be picked. Just thought I'd raise the issue :)
It would better look as enum
what is this even? 4.5? Stop living in the future.
Apologies for not having time to go verify this on my own at the moment, but while I'm thinking about it, is the checking to make sure this group isn't assigned elsewhere in the repository done inside Group.delete()?
I meant on things like `System.Func` and `System.Tuple`
Why is that? It's an HTML5 attribute as well, it seems.
remove commented code.
run jsbeautify.org
why did you remove the microloader?
no need for this file, I believe
As @pp-mo says, the text is still present but has moved up a few lines. (Currently to L301). Maybe it's not exactly the layout/order I'd have chosen, but that's always the way with prose. Looks OK to me.
If you wish to impose coding standards that deviate from the main Python community, you should publish them.
They are ``npy_intp`` already, which apparently is the type to use for indices, so using maxsize should work I think
Btw I don't think -1 is needed either (if we are using maxsize in other places, it may be better to be consistent and use it here too).
Actually, that would be the case only for floats, the gradient wrt int variable will still be floats.
Aack!  I would file that as a bug.  In the convention I had been using, functions that return status integers (like this one) are not supposed to allocate or free memory. They are supposed to assume their arguments are views.
so, how i can cut this line?
Is there a non-debug use case? If so, should use styles defined in a LESS file for the pins instead of hardcoding "yellow".  Same for border.
Yes, I commented it out temporarily because re-running the autobahn suite for each set of small fixes was taking too much time.