labofsahil/github-event-dataset-2012 huggingface dataset:
3.1 i2420

enum open and vlabel
Fix Arcane Reforge

Implement missing GOSSIP_OPTION_REFORGE 
Fix REFORGE_ENCHANTMENT_SLOT id in EnchantmentSlot
Django 1.4 Time Zone Support

I've updated these to use a now() function. This attempts to first import the now() function from ``django.utils.timezones`` and if ImportError occurs it will use datetime.now().
Limit gestureRecognizer:shouldRecognizeSimultaneouslyWithGestureRecognizer: slightly

The current implementation of <code>gestureRecognizer:shouldRecognizeSimultaneouslyWithGestureRecognizer</code> is somewhat destructive. If you apply other gesture recognizers that interacts with the gridView (and the internal <code>UIScrollView</code>, the results can be a bit strange. Maybe KKGridView should only return the non-standard <code>YES</code> when it's the long press recognizer that's trying to be recognized simultaneously?

...either that or I can just subclass for my implementation. Let me know what you think :)
Names splicing functionality

I've added functionality for slicing names with usual haskell syntax (just add _ prefix to the name). Look commit comment for example.
Fixes phpunit-selenium issue #72: ReflectionException when running selen...

...ese test.

This also applies to magical methods which have no accessible reflection

Fixes issue #389:
https://github.com/sebastianbergmann/phpunit/issues/389
дз5
Visual studio2012

I added support for Visual Studio 2012.

If you want, feel free to pull the change into the master branch and update the VSIX on the Visual Studio Gallery.
Assorted language tweaks, list of exam specifications
Don't install passenger on each run

This PR checks for the passenger version (by running the `--version` command) against the attribute-specified version.

Favored over #6.
Use qtile table structure

This switches owl_viewer Rails app to use the tree-like table structure.

Some things to note:
- I upgraded to rails 3 because I didn't know how to install rails 2's dev server.
- The ranges are broken into a list of integers so we can filter out tile ids we don't need to check. This could probably be reduced into ranges again to make the SQL shorter and faster.
- The "tiles" API has to search through all the tables to find changes for the given changeset. I'd love to hear any suggestions to speed this up so we don't have to look through the whole database.
fix spelling mistake for config value
Use ccflags instead of cflags

config cflags key was used, but it is defined nowhere. Nor Config.pm defines it, neither any code in the module. I think it is a typo for ccflags (as this PR changes), but please, confirm.

Meanwhile, checking if this fixes the problem I reported on p5p: "ExtUtils::CBuilder - cpp flags about archs"

Will report soon.
Clickable email addresses in context menu

Clickable email addresses in context menu (just like urls). This is in regard to issue #201

Also corrected some small typos I noticed.
Fixed a spelling error

Signed-off-by: DrDephobia <drdephobia@gmail.com>
Marx 2979

https://livefyre.atlassian.net/browse/LF-2979

Wasn't quite sure why the other commits/pulls that I made are showing up when I review the files changed, but I'm going to see what this pull would do.
Patch 2

social buttons added
Fix possible container recursion.

Container.add() now checks if the object to be added is the add 'host' or an ancestor of the add 'host'.
1.1.x - fixed broken docs link

There! now its ok... ignore the previous one.
Cioppino text cleanup

removed reference to bin/toplone which has been removed (jan/8)
Clean up preface.
Other minor cleanup.
A couple of customization tweaks

Hey, great widget.  I needed a couple of changes to fit the L&F of our webapp; figured you might want them.

First, I removed the inline styles from the <div> wrappers around the range inputs, putting them instead in the CSS.  Much easier to override that way (which I happen to need to do).

Second, I removed the hard-coded "btn-success" style from the apply button.  Instead, I've allowed the user to pass in the class as part of the initialization options.  Again, my need for this was to use btn-primary (due to color scheme) instead of btn-success.
(1_8_X) [TIMOB-7644][TIMOB-7152] iOS: Memory is not being released when a tab is removed from tabGroup

[TIMOB-7644](https://jira.appcelerator.org/browse/TIMOB-7644)
[TIMOB-7152](https://jira.appcelerator.org/browse/TIMOB-7152)
[TIMOB-7714](https://jira.appcelerator.org/browse/TIMOB-7714)

Test cases in JIRA.
FR: Search for 'Proxy' and 'Controller' in Instruments for [TIMOB-7644](https://jira.appcelerator.org/browse/TIMOB-7644) and [TIMOB-7152](https://jira.appcelerator.org/browse/TIMOB-7152).
Support for additional parameters


The additional_parameters parameter can be used to pass additional flags as command line options to the program configured to be used when sending mail, as defined by the sendmail_path configuration setting. For example, this can be used to set the envelope sender address when using sendmail with the -f sendmail option.

cf http://php.net/manual/en/function.mail.php

Signed-off-by: Matthieu Baudoux <matthieu@hypernovae.be>
Added functionality & password reset controller.

Start adding the send email functionality to edit user feature, updates routes, & create password reset controller.
Added HMVC, General Cleanup

Hey Mike, added HMVC and some general cleanup.

I added an ignore for the entire cache directories so that locally generated cache files don't make it into the upstream.

A few nick nacks (presence of .DS_Store being removed, a .swp file being removed) I think are my fault.  This commit should resolve them.

Thanks!
Updated a wrong link
Add documentation on how to override class metadata

There might be a need to refine wording a bit.
This fixes #69.
Fix a few typos

Just noticed a few typos while I was reading through the README, and thought I'd see if I could pitch in. Thanks for the wrapper!
Date Range and Site Insertions
Prevent possible null pointer deref in _d_arrayappendcTX

This was a bug raised by dsimcha in gdc several days ago which appears to be a concurrency bug within druntime, and was found to be reproducible in gdc and ldc, but not dmd for whatever reason.

original report: https://bitbucket.org/goshawk/gdc/issue/295/null-pointer-deref-in-_d_arrayappendctp

I've done the most obvious thing and added a check for null pointer, which fixes the issue. Can you check to make sure it makes sense to do this?

Thanks
Changes to Config.php

Added the ability to flick between min css and normal css just by changing CSS_MIN to false etc. Saves messing around changing the file path especially if you have the css pulled in on more than one part of the site. 


main site & blog etc
mentioned that North* still exists as a product generator

...and would you mind cutting a release? (you may want to bump the version too)
including a set_relationship

Here is an example of adding a contact to a target list:


		$c->set_relationship(
				'Contacts',
				$contact_id,
				
				'prospect_lists',
				$target_list_id
		);
Textarea expanding issue

Fixed Issue #50 and #51.
Change pairing sessions sections

As the Tracker task says, instead of having the pairing sessions page be this:

1. pairing sessions I own
2. pairing sessions where I am the pair
3. available pairing sessions

I'm changing it to this:

1. my commitments (sessions where I'm either owner or pair, excludes past sessions by default)
2. my open pairing sessions (sessions I've posted and am waiting for bites)
3. available pairing sessions
Eagle binary format: an initial commit

This commit includes:
- separated reader and writer;
- some tests for reader.

The main goal -- is to get a confirmation if I got it right
(I decided to duplicate constants finally).
Ошибка в 3 лекции, связанная с площадью треугольника.

Пофиксил ошибку с площадью. Ранее были неверные рассуждения по поводу складывания ориентированных площадей и выражения площади через векторное произведение.
Develop

These are the changes made to release-1.5 to fix migration.
Intranet support added

Intranet support added
sRGB option

This fixes #19 following @rma's suggestions.
Implémentation nbLocations

Et hop, les modifications pour prendre en compte le nombre de locations dans les classes Vehicule et dérivées
added stop/start for web server

This is the first time I used github so I'm not sure if there is some other preferred means to propose a possible change to you.

I made a change to server.clj for both start and stop functions. Among other things, this lets you shut down the web server without having to kill your REPL. 

I haven't been using Clojure for too long so you might have a better way but I wanted to suggest the idea if you don't want the code as is.
Ruby 1.9 compatibility

When using this extension on 1.9, radius_tags.rb would give SyntaxErrors due to deprecated case syntax.
Please pull and release a new gem version.. thanks!
Added tests for null, anonymous and iterator models.

These tests are dependent on changes to View class as made in https://github.com/WebApiContrib/WebApiContrib.Formatting.Html/pull/2 for WebApiContrib.Formatting.Html project.

To get some of them to pass, pull request https://github.com/Antaris/RazorEngine/pull/73 fir RazorEngine needs to be accepted. However, it would be possible to work around the iterator model bug in RazorEngine.
Shouldn't this be _step=60, to match all the other step sizes below?
A parameter set data get call should be used here (and below) as shouldn't be getting at the distributed vector data directly. This is because we need to abstract this to allow for multiple types of distributed vectors.
Он вообще должен ходить по стеку контекстов наверх, если в текущем контексте не найдена нужная переменная, но был баг, которые это мешал делать, но месяц назад его исправили и я его к нам утянул, всё работает. см. d6367d8b1d55f7f3e81abb3d7916138fd4b5ac37
dal bych bez komentare, pripadne pouzil syntax //
I don't know if it's common but it's the approach we use in node.js these days. In six months time, I got almost a dozen bug reports that turned out be libc/kernel/headers mismatches. Doing the syscalls directly sidesteps those issues.
Link to that Hacker News post, and if you give stats it will make a stronger point (if you're allowed). Can also say something along the lines of taking the initiative to propose then implement it?
Should be "Copy" not "Move", right?

and default=False
If `per` is too terse I'd probably fall back to `perPage` but I **really** like the single-word attributes, it feels so clean!
![Oh no you didnt](http://iruntheinternet.com/lulzdump/images/gifs/beyonce-sexy-dancing-hot-bootylicious-13308047586.gif)
@craigh - ok but I thought you added/amended these classes as part of the category attribute stuff in `release-1.3`?
What's missing: an index for the column `key`.
I don't mind doing that together, but this must be covered in the tests before proceeding with the fwd port.
Hey how about this

    class Module
      alias :______________________________PRIVATE______________________________  :private
    end

    class Foo

      def hello
        print "hello"
        world
      end

    ______________________________PRIVATE______________________________


      def world
        print " world!\n"
      end
    end

    Foo.new.hello
    Foo.new.world # => dissed!
Any patch that improves or maintains existing functionality and net removes code is great for maintainability. Nice.
текущий мастер 4.0
@captn3m0  I discussed with neeraj and nisha... The suggestion was that instead of multiple repo we can go with multiple branches and then we will merge afterwards.. I know ideally we should have multiple repo.. but here it doesnt matter much as the code base is not that big.. and we can manage with multiple branch..

Second I dont want the redmine issues here.. I will put the fresh issues along with milestone myself.. once i plan it... 
And lets put up an expectation that if there is any issue u or anyone want to file.. u first have to notify in a mail ccing each of us. and then we will decide and reply if u can move forward to file it or not... I want the team to control what it can do or not.. as we have very less time.. and lots of work and i dont want any more chaos which happen when arbitrary issues occur.. and when filing issue u have to mention which version of Echo. And please dont file any feature request issue for version 1.
I thought its easier to grep in the log for all zeroconf related stuff if I keep this the same for all functions.
Fixed by commit c5339d29603bb34d9a62c4f398d5671aa883b5e6
Please, fix the indentation here. Should be like
<TAB>if (has_gtk) {
i'll delete it.
This test is kind of the point of the change. After this change goes in, if XPath stops returning a type that works as an argument to ElementMaker, it'll break a bunch of code, so ElementMaker will need to be fixed to handle the new type.
Cool! I've never seen `Kernel#__method__` before.
This is also a private member, add private prefix. On the other hand, I suggest you init this member in construct function for it is not a static variable
Unneeded paretheses
round 2 it will be cool to add an option / auto-do for no main -> drop to scala shell or bean shell
niCkel 
Le 26 janv. 2012 à 17:24, jgFages a écrit :

> nikel
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/cprudhom/Galak/commit/945e899eeda57d65b34025636de799ff81159ae4#commitcomment-904679
I was inconsistent on this.  I started just adding ret == -1 || and then started replacing the check of the NULL ptr later.  I'll move to more consistency and just check the return value.
ça parait sympa ces modifs ;)
lol? Should that be a setting?
just found this where is the ccdb forums or such?
Вообще я сам любитель по привычке   UserGateway gateway = new UserGateway(conn);
но правильнее следует как я описал выше ведь фишка в чем)
получается ты можешь присвоить папе любого наследника, а UserGateWay только его ну как-то так

ну эт так на заметку
Yup. I remember dealing with shadow texture in the icosahedron vertex color examples not appearing when `WebGLRenderer` had (or didn't have) a `clearColor`.
LOL @ "Easier Debugging".
Yeah we havent quite solved the test suite problem yet.  Maybe add
threadcount 1 to the pom under surefire if not already
On Jan 8, 2012 12:22 PM, "Andrea Turli" <
reply@reply.github.com>
wrote:

> I've checked target/test-data, now it's ok but for some reason 2-3 runs
> were blocked.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jclouds/jclouds/commit/a0e038d9dfb37c641ad929bed2466b9e9d2c6d6f#commitcomment-847953
>
This not crash, just kill -6 mangos-worldd while issue with load happens.
так я как раз в dll и посмотрел. если не верите - есть dllinfo или file. чистый .NET, причем pe32x.
что работает - я не сомневаюсь. но что лучше чем -DUSE_STD_MALLOC - не верю, причем совсем (и убедить не выйдет. если паскаль я знаю хреново, то асм - пока еще хорошо).
Я уже не раз говорил, что надо делать нормальную структуру обновления базы.
Должны быть кумулятивные файлы и отдельные обновления.
Внесли изменение - сделали отдельный файл.

В итоге можно будет нормально обновляться с 0 или с какого-то момента, а то сейчас приходится все файлы шерстить на предмет ошибок и лишних запросов.
do you need only this info right?

2012-02-15 17:57:43 ERROR:Spell::CheckRange() player Player Siilvanass (Guid: 150341979) not shot player Player Rjcv (Guid: 150190264), reason: strict 1, delta1 11.982013, delta2 -nan, delta3 10.599012

I'm do a txt with this info of 24h. tomorrow I'll send you
This should be 'orbitephem1_z' (not y), right?  I'm surprised this was giving good results.  This mistake shows up in a few places.  I'm fixing it now.
Unit tests would be great but first I'd have to understand what these things did :)
I think these tests look good, but I'm not sure a 404 is the right answer. I think a "that refund has already been processed" kind of message would be better.
And if $plugin is not an instance of ezfIndexPlugin ? please, log an error so that the developer has a clue of what is going wrong.
Явно отступы неверные.
Whoops, forgot to note this in the commits. I don't know why i had that set to OR instead of AND, but the result was that permissions were all screwed up. If someone's name doesn't match the whoset in their profile, AND their nick isn't a botmaster, then go on to do the error.
Don't do this. The arguments are processed inside the function, and the user can provide an arbitrary number of DOMElement arguments.
Default value should be true xD
In my OpenOffice 3.3, your updates to Tilde Documentation.odt (which I like), seem to ruin the curved lines illustrated in the doc/assemblies.png.  The lines all appear as vertical ones descending from the top.

Do you see the same thing?
Can we get this upstreamed?
I'd suggest letting this feature in as they could come back in no time.
I can fix it later today.
Because there is no reason for me to remove it
Hi,

I get an exception in my code after this commit. My code:

@Entity
@EntityListeners(SomeEntityListener.class)
public class SomeEntity extends TimestampedEntity {
   ...

So, I think this commit may have introduced some unintended side effect. Stack trace:

javax.persistence.PersistenceException: The class com.somecompany.SomeEntity is not an entity.
        at com.spaceprogram.simplejpa.EntityManagerSimpleJPA.ensureClassIsEntity(EntityManagerSimpleJPA.java:183)
        at com.spaceprogram.simplejpa.query.QueryImpl.init(QueryImpl.java:352)
        at com.spaceprogram.simplejpa.query.QueryImpl.<init>(QueryImpl.java:91)
        at com.spaceprogram.simplejpa.EntityManagerSimpleJPA.createQuery(EntityManagerSimpleJPA.java:492)
        at ... (my code)

Cheers,
Christian Tellefsen
Oh and you should be using it too, rather than google analytics, ya know, self hosted stuff ftw and all that ;). Wouldn't have a problem with removing google analytics ;)
Hey, I have just tested this and it looks borken to me. The stage0 background appears as a plain still greyscale gradient, and stage1 is just pitch black. Is it supposed to be like that?
Thanks Tsuna, I'll get all the changes in as you said. I'll make it a member of the TSD class.
This is highly toolchain dependent, but normally, no, not if quantlib is
a dynamic/shared library. The DLL/so will have a dependency on
boost_thread, but the user of quantlib shouldn't have to list that directly.

If they use the static library it really depends on the tools. Libtool
should probably figure out the dependency on its own. In any case, this
change eliminates the biggest need for a static library. People should
just use the shared library -- since the default shared library now
offers custom sessions.

On 02/12/2012 03:33 PM, Luigi Ballabio wrote:
> Oh, I see.  But wouldn't one have to link anyway just because one _uses_ the singleton?
> As soon as I call `create` and `SingletonInstanceLoader` is executed, I'll have to link to libboost_thread even if I'm not using multiple sessions.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mortoray/quantlib/commit/dbf6b23429b21cccb0982d6f3e5c13f5860842e1#commitcomment-957764
> 
> 


-- 
edA-qa mort-ora-y
-- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
Sign: Please digitally sign your emails.
Encrypt: I'm also happy to receive encrypted mail.
Hoah! 

Feedback is welcome, @carasuca @maczekmaczek @themoob. 

Don't call me too early ;)
I must say, your implementation is much nicer than mine :-)
TODO ^ (and fix the test that'll break)  :)
Please post this as a documentation bug to https://github.com/rspec/rspec-rails/issues. Thx.
FYI. You can have a shorter syntax for this one. This is an advantage of using builder pattern.


SearchQuery.QuerySpec.newBuilder().
    .setQueryString(queryString);
    ..setCategory(category).build();
When I try to inject a `describe` function, I get the following error:
'Injector already created, can not register a module!'.

That's why I've asked how this went unnoticed.
The patch is missing proxy support. You may want to take the new patch i sent you replacing this
Esta clave me la ha dado Google a partir de un fichero que tengo en mi instalación del Android SDK. No sé muy bien cómo podemos compartirla, a lo mejor si te paso el fichero en cuestión ....
I would prefer to use C**D to express exponentiation, since C^D already means XOR in tenyr, but I appreciate this code. Thanks !
Great work! Does this fix issue #200 ?
lol. go to settings > location. (this is in froyo though, might be slightly
different in gingerbread).

On Sat, Feb 4, 2012 at 9:27 AM, legendaryfox <
reply@reply.github.com
> wrote:

> how do i enable them? >_>
>
> On Sat, Feb 4, 2012 at 10:26 PM, susanafong <
> reply@reply.github.com
> > wrote:
>
> > Probably because you don't have any location providers enabled (I added
> > some code to check for that, but it's still buggy).
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> >
> https://github.com/legendaryfox/bamft/commit/f5bb365c9abe8d5a2b31363e170ddefcf2b551bd#commitcomment-933043
> >
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/legendaryfox/bamft/commit/f5bb365c9abe8d5a2b31363e170ddefcf2b551bd#commitcomment-933046
>
Sorry, my patch for 2.6.35 isn't available on github, but only on
http://sven.killig.de/android/N1/2.2/usb_host
Line notes! This will come in handy!
不必考虑鬼才，因为这个不是防改判定，而是防该判定换好牌。
换言之看看有没有给敌人送桃送闪的危险性
You are right.

I was just basing my thinking on node-semver about incrementing

https://github.com/isaacs/node-semver

The dash represents the build, be it numeric or beta or rc.

In this case, I probably should have just bumped to 0.3.6 as a patch.
I do, but not on this box, so I have been fumbling around with the Web UI
this weekend. Editing the master branch is relatively painless.

I think you have to initiate a "pull" for me to be notified and after that
it is easy.

On Sun, Jan 15, 2012 at 6:22 PM, Paul Connolly <
reply@reply.github.com
> wrote:

> Do you have git setup via command line?
>
>
>
>
> On 15 Jan 2012, at 22:58, David Mark
> <reply@reply.github.com>
> wrote:
>
> > When trying to merge, keep ending up on this page, but only has a
> "comment" button. :(
> >
> > ---
> > Reply to this email directly or view it on GitHub:
> >
> https://github.com/gibletto/My-Library/commit/f11ea1fe7f729a649cac003c482cae4e4f16f950#commitcomment-869380
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/gibletto/My-Library/commit/f11ea1fe7f729a649cac003c482cae4e4f16f950#commitcomment-869413
>
I think you'd need an IBuildStatus instance, and that class already exists, and `getURLForThing` already supports building a URL for that.  However, you only get build dicts from the DB, so I think this is OK for now.  The work on putting status into the DB will change this.
I'd be happy to accept a pull request for the `remove` part of this :-)
hmmm long time ago 
- with xz support my boottime was +5 sec ore more
- with lzo support maybe -1 sec.

the image will be around 15MB smaller with xz support and 10MB bigger with LZO support

but test byself :-)
After thinking about this a bit more, I think we only want to supress scrolling if the user's touch has not exceeded the vertical threshold. Once they exceed this, then we should allow scrolling. This will allow folks to use swipe on longer pages.
Gotcha. So is the issue that touch events from the swipe aren't being dispatched on `map.parent`?
Remove Create from values
Will move this into arguments passed, soon.  Just testing now.
needed to add  this._finalizeDrawingPath(); to _enlivenDatalessObjects to update canvas without clicking. see: http://jsfiddle.net/rkP3p/
canvas will not be updated until clicked.
Fixed issue where SMET_Request__c could not be owned by a queue - the trigger attempted to copy a queue ID into a user lookup field.
Good job!

Look OK to me. I'm merging it!
Ugh, I guess I have to turn off import fixing for on-save...
You don't need to do that. setNums sets the "ivstat" property of the sender() of a slot, so you can guess which stat was changed. Since labels are never used to emit signals, that one setNum is useless. Since only the setNum below is useful, you can drop the macro and write the code directly.
Subdomains are not that corner case feature of the web... The reason why it is not needed by more users, is because there are no extensions, which utilises this feature, and the reason for that is... the feature is missing. The duplicate content problem is caused by missuse of this feature, you can achieve the same problems by messing around with include CEs.

There is no way to overwrite the getRootPageFromUrl-method. And there is no way to replace the setCookie method, too. And the getPageDetails method, too. I would like a hint how to integrate it flawlessly as an extension. I have thought about it, and no way has come to my mind.
ah, now i see. 
I actually was wondering what was the difference, but thought of having them in the same style as in Neocons etc. 

thanks
On 20 January 2012 12:13, Dag Sverre Seljebotn
<reply@reply.github.com>
wrote:
> Hmm. I find this aspect of NumPy to be a little confusing, or perhaps "too much convenience"... In NumPy one can also do
>
> a[:, :] = b[None, :]
>
> or
>
> a[:, :] = b[:, None]
>
> (henceforth "newaxis-notation"), where None inserts a new 1-length dimension. This is what I usually do (as proof, I can't remember which of the above corresponds to a[:, :] = b[:]).
>
> The a[:, :] = b[:] notation is truly useful when one does not know the number of dimensions; but that is never the case currently in Cython.

The broadcast applies leading newaxes dimensions, i.e. b[None, :].
This corresponds to broadcasting a 1d array along the rows of a 2d
array. I agree that is is implicit and maybe confusing for new users,
but I like the compatibility with NumPy here.

> BTW, do you support newaxis-notation in Cython now? It's more general than broadcasting so should perhaps be implemented first anyway. Provided that we do want to support the more generic newaxis-notation then the a[:, :] = b[:] notation could perhaps be delayed until the question about whether we want it or not is settled -- it's easier to add it than to remove it.

That has been on my todo-list, it is not yet supported. I think it
should indeed be None, as that would be compatible with both NumPy and
our memoryviews. e.g. view.newaxis could also work if it converts to
None in other contexts, but then you'd expect numpy.nexaxis to work as
well, but Cython would try to convert that to an integer for typed
indexing.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/markflorisson88/cython/commit/31b2d0cef7ae404eb92cfaaf4f50210e65f07dc4#commitcomment-885558
8+6*4+1

Save bytes, please :3
ok, bcs I have old core and some issues after fixing are not closed :) anyway thanks
error at compil
to fix it change "&mCreatureQuestInvolvedRelations" to "&_creatureQuestInvolvedRelations"
UPDATE smart_scripts SET action_param2=2,action_param3=0 WHERE entryorguid=4961 AND id IN (1,2) AND source_type=0;
Is old but...

DELETE FROM `creature_ai_scripts` WHERE `creature_id`=@ENTRY;

Not define "@ENTRY" before, define after, not it's wrong?
I'm sorry, I thought I did. Well, I was wrong.
I went ahead and pushed `g/.*/d` to my repo. Should take care of any/all cases.
вот это совсем лишнее  - ты добавляешь user_id в таблицу users. Но у тебя уже там есть айдишник - поле id !!!
One thing we need to consider is how a volunteer should be promoted. Should we do it via update from the pop-up volunteer form or should it be done somewhere else? Either way, password needs to be generated and email sent to the newly promoted HC. Du has implemented part of those functions in the pop-up volunteer form, but sending email still needs to be put in place.
@metaskills looks like I was wrong - I'll take a look to see how this actually worked before and see what I can do. A pull request would be appreciated.
Yep. Not supporting a particular script loader will definitely make it easier for *all* of them to work properly. 

I apologize for merging the support in the first place.
Maybe I am a little too anal about extra beans, but can we not avoid creating the factory bean definition if the context already contains a bean definition for the input channel?

I realize it will only help if the channel is defined before the adapter, but...
Probably. Here I was mainly concerned with git.
Ok. I delete it too.
I've sent a pull request that fixes the same problem to original developer.
https://github.com/iaian7/mediaboxAdvanced/pull/14
Would you like add some comment to this pull request to point your change?
Hi, could you tellme what is this change for please?.
> Yes, Clay used features of numpy 1.6 that are not in 1.5. Jenkins also
> complained. I am on it.

Should be fixed
should this be..... get_descendAnts?  (note the 'A' instead of the 'e')
...because things like this happen.  Do we want to return EXIT_SUCCESS in this case?
Homework 3
==========================

**+2.25 points**

One general note is that in general python filenames don't contain capitals or hyphens and don't start with a number.  They also have as few underscores as possible.  Not really an issue, just a note.

Now onto the programs!

## prissybot.py
Good job overall.  I definitely felt insulted and had flashes of many an evil robot from my past.  There are a couple of places where 
your formating could be clearer but overall good variable names, division of sections, etc.  Also, there is no reason to mix and match commas/concatenation and formatting.

```python
# not great
print p+"Oooh! Look!  Look at the human! %s can make phrases with" %(name),length,"characters."
print p+"Do you know what that %d divided by 17283 is,"%(length),name+"?"

# better
p = "PrissyBot"
print "%s: Oooh! Look!  Look at the human! %s can make phrases with %d characters" % (p, name, length)
print "%s: Do you know what %d divided by 17283 is, %s?" % (p, length, name)
```

Anyways, for insulting humanity, I give you the [HK-47](http://youtu.be/Vg1gTas7OAA) award 

![award image](http://old-republic.com/images/kotor/characters/kotor_hk-47.jpg)

## asciiart.py:  
Only criticism is the `raw_input` for the character could have a space after the "." or a ": " instead.  Otherwise well done on the arrow.

## FractionReduce.py:  
I have nothing to comment on, works as it should.

## ConversionTable.py:  
First off, the table works like it should.  My only issue is with the formatting.  I'm not crazy about the underscores for the row markers (I personally prefer hyphens.)  Also, [repr](http://docs.python.org/library/repr.html#module-repr) is different than [str](http://docs.python.org/library/functions.html#str).  You probably wanted str.

Second, although every line here works, they are long especially since they can be written more cleanly and shorter.

```python
# long
print repr(second).rjust(12),"|",repr(bin(second)).rjust(12),"|",repr(hex(second)).rjust(12)

# same, cleaner(?)
print "%12s | %12s | %12s" % (str(first), bin(first), hex(first))
```

If you wanted to left justify instead, you can use `"%-12s"`.  You can find more information on string formating options [here].(http://docs.python.org/library/string.html#formatstrings)


Finally, you should look at formatting tables this way:

```
| **Test** | **A** | **B** |
|------|---|---|
| abcd | 1 | 3 | 
| abcd | 1 | 3 |
| abcd | 1 | 3 |
```
which looks like this when in github

| **Test** | **A** | **B** |
|------|---|---|
| abcd | 1 | 3 | 
| abcd | 1 | 3 |
| abcd | 1 | 3 |
Blended commit with the https://github.com/silverstripe-big-o/sapphire/commit/6433ddcc34e95b0cf92d278ec3ccc6cd838c6cdb ?
y u no werk http://puu.sh/k4JM
It doesn't work cus permissions are wrong at the moment.
On Jan 7, 2012 9:25 AM, "LeandroMedeiros" <
reply@reply.github.com>
wrote:
>
> Great Work! Credits for you both!
> I'm compilling right now, let's test ;P
>
> ---
> Reply to this email directly or view it on GitHub:
>
https://github.com/fosser2/android-tegra-nv-2.6.39/commit/18c3c5e5482ca4b10ab6334f2e631d0c623beb0a#commitcomment-846164
This change could be dangerous, it breaks backward compatibility.  I know we have been treating pyUtils as a pool of useful functions and just changing and updating as we can and with little regard for backward compatibility, but is that still wise?  Maybe it is time to start exercising restraint.  PyUtils is in allot of our code, and code that in some instances will no longer be maintained by DGI.
This follows all other release rather use device model use the code name.
On Jan 9, 2012 6:29 PM, "Takazumi Matsumoto" <
reply@reply.github.com>
wrote:

> Is there a reason for the release name change?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cmxoom/android_device_moto_wingray/commit/33a1baa78f6e5fffffced14a0f166e93dc57e5e3#commitcomment-850428
>
"add Gingerbread style text select handler" isn't compatible :( I think we have to take a look at "core/java/android/widget/TextView.java"
That's a feature, not a bug. I need the alarm clock to wake me up in the morning, and the standard ringtones simply weren't cutting it...

;)

Actually, I forgot to include the required AudioFilter.csv in this commit. It's here: https://github.com/psyke83/android_device_huawei_u8160/commit/2afac4745c373033be4c0826df2ee131fd8d16c1
Removed this annoying 'feature'
А шо не воркает? Опиши проблему.
So like i ve already said in a lot of issue, this bundle follow the master
wainting the 2.1 because i ve no time to maintains the 2 versions

Cedric

2012/3/3 Adam Kuśmierz <
reply@reply.github.com
>

> No, last stable (v2.0.11)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/commit/b6a297b201ca3a9046b3a2633aaeff4311fa6ef7#commitcomment-1038615
>
The compilation fails to work on the libandroid_runtime.

Try this, clear out and compiles libandroid_runtime alone.

This is what happens
http://pastebin.com/zJtUyRWP

Note: releaseWakeLock function, acquireWakeLock, uevent_next_event, uevent_init exists in the project, this is not the problem.

Now with the change
http://pastebin.com/VhT6HfDU
All correct

I do not intend to change it, you just notice the problem, maybe your computer behaves differently than mine to interpret ifeq.
Regards
Good work
Ничего военного. Добавляет лэзи линку в хэши лэзи линок.
'sampleCharValues -- renamed (used to be called 'scv');' can go
also, please add this var to the var list below
Yeah, that's already handled. The recursion comes from asking for the
module that's being requested.

Also, this is really "not new". The thing that brought it out was a
new build with a bad copy/paste by me. Putting 'yui-base' in the 'use'
for 'yui-base'.
jQuery UI version is buggy, See - https://github.com/xuqingkuang/jquery-ui-dialogbox
By using PartialFunction composition?
lol, epic typo
I'm not sure I understand, ScatterGatherFirstCompletedRouter only supports ask
> ok, let's see if I understand.
> The configured value should be `smoothing-half-life-duration` in unit  
> seconds.
> halfLife = smoothing-half-life-duration / collect-interval
> N = 2.8854 * halfLife
> alfa = 2 / (1 + N)

This seems to be correct, but wait for Roland's answer :) Just a final  
nitpicking, because I am a soulless bastard: add comment that it takes  
about 4 half-life to drop below 10% contribution, and 7 to drop below 1%.
no, initChild is used from two places: one which cares about uid and one which does not
yup, ditto
Won't this (and `query`) break anyone that's not using keyword args?
This if can be replace the else as an elseif, no need for double nesting.
But now I see that this visualization is inside of a test... Should the tests display anything in the first place?
It raises something like: 

TripWireError: We need package matplotlib.pyplot for these functions, but ``import matplotlib.pyplot`` raised an ImportError
What type of file?  .npy?
Pyflakes! Will fix that too. THX!
I'm going to leave these for now. They are used in a number of places, including okaara, and will need to be addressed more across the board as a whole in a future story.
remove this flag from cli. This will go away from the plugin once I merge v2_metadata branch and we default the generation to snippet based.
I'm using 'is' here (and a myriad of other places in the dispatch system) because this field is always assigned to a memory-resident constant. Anytime I can't guarantee string fields are going to be the same objects, I do use '=='. However, I tend to use 'is' as much as possible because the comparison is much much faster.
Yeah, I think this one should be OK.
Maybe change this to "Error while loading pollers: %s" % str(e)
You could merge DtoToJsonConverter and JsonToDtoConverter classes in one class QueryTermConverter with marshal() and unmarshal() methods that does the conversion between a ES JSONObject and a QueryTermDto
Why does this return `Univ`?
Why not a fromJson inside TaskContext instance instead of creating a new TaskContext and then copying data? The TaskContext should be able to change itself from json?
Oh, true, because I have to check the existence in the locked part  
anyway...
Also, I only see the version support in the enrollment confirmation request. Any call should be json based if the client supports it, not just the enrolment.
Enyo house style is to have a space after the "if".  I'll fix this in the merge.
perhaps I should leave these in, in case we figure out a way to make the prefetching better on BGQ, they are undefined above any way so they will be of no consequence
maybe it makes sense to namespace this under histogram? So that it get's added to something like `current_timer_data["histogram"][bin_name]`, would make the namespace a bit more hierarchical.
`CFIndex` is already signed. This should use `labs()` or `llabs()` instead of a cast.
This seems redundant with `TUINSWindow`'s responder notifications.
Alright, I can do that then.
Ok, makes sense that we don't need ordereddict for python < 2.7.

I opened a PR against your repo Satra: https://github.com/satra/nipype/pull/14
Asserts are useful for a particular style of design by contract programming.   This approach is used occasionally in the numpy code base so I don't see any problem with it.   Also, python -O turns all asserts off so it has some built-in support to the language.
I don't think crediting this way in a comment makes much sense and is more confusing than helping. Code comments should explain things.
Right! Plan to learn from my "put it all together" experiment.
заменить пробелы на табы
Last time I left it private, and you complained that I was only changing the indentation. :) This is just the way it was in my original implementation, I have no preference between them.
ruby-debug again
Then remove the `nil?` check?
I agree. I also searched *runner* method in rails/railties.rb :-)
I'm going to fix it.
Thanks.
I don't think it's necessary to talk about "config/routes.rb".. Only routes should suffice. How about showing the old way before?

```
Support unicode characters in routes, so instead of manually escaping:

    get Rack::Utils.escape('こんにちは') => 'home#index'

It's possible to write the unicode route and Rails will automatically escape it:

    get 'こんにちは' => 'home#index'
```
I *think* it might be better to not depend on `render :layout`, since the template is so small and there's no reuse, we could just add the layout content here. Wdyt?
Sorry for nitpicking, but let's move this to `_render_template`? This logic only makes sense when rendering a template. :)
Crashes for me in unpatched 2.10.x
Just a random question: Is anyone paying attention to these versions and changing them when the serialization of classes change?
Let's please assume this didn't happen
Yep, it's pretty loose as it stands. I was trying not to rely on that quirk, but that at least warrants a have been commented.
I think this one isn't done until you only have to call "setAttachments tree.attachments" from one place. We should especially resist having multiple calls to it when one is nested inside another one...
Vlad, you are right. Both the flag and annotation gets pickled. Once you check the flag is set you read the annotation. The point of doing that is to have very quick check (by using a flag) for short-cutting. Does it make sense?
Not sure if this renders correctly now. An issue is that ``*`` is reST markup, so when that's used as math it should preferably be enclosed in double backticks.
I told you the benefit.   It prevents writing a lot of similar code over and over again.   I'm not convinced that being able to "follow the flow of control" actually matters when the flow on error is out-of-the-function.   

Any desire to "break" a habit requires first seeing the need to.   You do a lot of complaining about style, code re-writing, but not a lot of demonstrating a better approach.   This rarely works and generally alienates the people you are purportedly trying to "help".   It also takes a lot of time and resources -- resources that are already in short supply.    You have to also look at how often someone will care about a particular piece of code.    The fact that this bit of code (it's supposedly ugly macros and all) has not been really touched in 10 years says something about the need to spend time agonizing over how it gets written. 

I'm left to do the work of inferring what you would rather see.    Are you preferring to see something like: 

if (!check_func(fcn)) return NULL;

where check_func is a regular function?  How else would you do things?  

-Travis
Does this compile to correct html (as an indented code block, etc)?
It doesn't, though.  It just wraps integer literals in Integer.  `int(1)/int(2)` will still give 0.5 (instead of 0).
I've fixed those occurrences of this habit about which I had no questions :-)
@smichr ?
On 12.06.2012 19:08, Sergiu Ivanov wrote:
>> +
>> +    No checks are carried out of whether the supplied object and
>> +    morphisms do belong to one and the same category.
>> +
>> +    Examples
>> +    ========
>> +
>> +>>>  from sympy.categories import Object, Morphism, Diagram
>> +>>>  from sympy import FiniteSet
>> +>>>  A = Object("A")
>> +>>>  B = Object("B")
>> +>>>  C = Object("C")
>> +>>>  f = Morphism(A, B, "f")
>> +>>>  g = Morphism(B, C, "g")
>> +>>>  d = Diagram([f, g])
>> +>>>  Morphism(A, A, identity=True) in d.premises.keys()
>
> `d.premises` gives
> ```
>   {IdentityMorphism(Object("A"), ""): EmptySet(), IdentityMorphism(Object("B"), ""): EmptySet(), IdentityMorphism(Object("C"), ""): EmptySet(), Morphism(Object("A"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g") * Morphism(Object("A"), Object("B"), "f"): EmptySet(), Morphism(Object("B"), Object("C"), "g"): EmptySet()}
> ```
>
> `d.premises.keys`:
> ```
> [Morphism(Object("B"), Object("C"), "g"), IdentityMorphism(Object("C"), ""), Morphism(Object("A"), Object("B"), "f"), IdentityMorphism(Object("A"), ""), IdentityMorphism(Object("B"), ""), Morphism(Object("B"), Object("C"), "g") * Morphism(Object("A"), Object("B"), "f")]
> ```
> In my book that qualifies as too long :-) However, I can try to break it down into lines so that it gets more readable.
>

you are right again
This line looks longer that 79 characters. Probably need to make it two lines.
How do we grab out the coefficients in a general way? I think that counts both returns the coefficients and the key as a tuple. The only other way to do this that I know of is to assume that something like as_coeff_foo works.
This sentence seems to be missing a verb.
But in this pull request it is spelled right. Either you corrected it or else you were testing a different sympy.
Burada ne demek istediğini anlamadım. dateString hiçbir zaman 'GG/AA/YY' gibi bir değer almıyor zaten. Bahsettiğin eğer dateInputArea'nın değeri ise, orada 'GG/AA/YYYY' yazılı iğken focusIn yapılırsa, textarea'da hiçbir şey göstermemiz istenmiyor. Bu kontrolün amacı, iki karakter girilip focusOut olduktan sonra tekrar focusIn yapılmış olma durumunu dikkate almak.
So, what happens now is:
if `BAR_WIDTH` is set to a negative value, the width is adjusted in relation to `OFFSET` so that it fits the screen.
if `BAR_WIDTH` is set to a positive value, we assume the user knows what he's doing and thus it's up to him to make the correct calculations.
This *appears* to employ a two-space indent.  Are these perhaps "tab"s?
len(thread_list)==0 ---> len(thread_list) == 0
Removed in commit a089006
Instead of failing we should probably just allocate locals/tmps on the stack. I only want to implement it before we have proper tests for the existing functionality though.
Should this not be 2.0.1?
By changing this to a text input, you lose the ability to choose `scan-completed` and `scan-failed` as valid pre-canned (non-JoSQL) rules.
Please rewrite this and all following sections to be: 

1. Less tautological. The current forms are vague, and do not add value to the documentation. 

2. Less passive.

3. Technically correct. The connection pool is something that the mongs has to connect to the members of the replica set. These don't have that factual point correct.
@graydon this was one line that I was unsure about.  It seemed like it could be a bug...? But prob not
Remove this property. Use is_active() instead and make it return a boolean.
Doc block, not comment. Should be `/**`
Hence the reason this bit needs rewriting. ;)
If you rename the file to just includes/akismet/akismet.php and use the class naming guidelines to name it `phpbb_akismet` (or just put that class in the includes/ directory like `includes/akismet.php` and name it `phpbb_akismet`), it will be autoloaded for you and you won't have to manually include the file.
Empty line before here?
Dependency: https://github.com/phpbb/phpbb3/pull/996
Per IRC, all js should be removed from this pull request. If existing js breaks with new backend code, remove existing js also.
For backward compatibility, we still keep these around.
As seen above, i quickly copy-pasted this. I'll make sure to check out the alignment more.
Should change this line to say it was corrected or something
there's lots of old code that you commented - please remove it rather than commenting it (we can always find it in the source control history if we need to re-add that code)
Thanks @tomgreen66
On Thu, Aug 30, 2012 at 11:40 AM, goodfeli <notifications@github.com> wrote:

> In theano/gradient.py:
>
> > +        accounted_for.add(var)
> > +        if var.owner is not None:
> > +            node = var.owner
> > +            if node not in accounted_for:
> > +                accounted_for.add(node)
> > +                for i, ipt in enumerate(node.inputs):
> > +                    if ipt not in var_to_node_to_idx:
> > +                        var_to_node_to_idx[ipt] = {}
> > +                    node_to_idx = var_to_node_to_idx[ipt]
> > +                    if node not in node_to_idx:
> > +                        node_to_idx[node] = []
> > +                    idx = node_to_idx[node]
> > +                    assert i not in idx
> > +                    idx.append(i)
> > +                    account_for(ipt)
> > +
>
> Actually there would be a bug in that.
>
>  x = T.matrix()
> y = x + x
> y.owner.inputs.index(x)
> 0
>
>  node.inputs.index will only return one index, even if the variable
> appears in the inputs list multiple times. So this would discard some paths
> that the gradient can flow through. Is there a clean way to get v to map to
> a dictionary mapping nodes to index lists?
> I also don't want v to map to a list of (node,index) pairs because I would
> need to postprocess that later to detect repeated nodes. I only want to
> process each node once per variable in grad_dict.
>
Good point, what about `{v: {node: [ii, for ii, vi in
enumerate(node.inputs) if vi is v]} for node in nodes} for v in variables}`
yea I noticed @rzezeski made a similar comment on the original implementation. Figured it best to not change it over as part of this PR like you said.
Yes - thanks!  The consequence of not having contributed code in months!