labofsahil/github-event-dataset-2012 huggingface dataset:
Multiple message templates

With this modifications you can specify multiple message templates. So, you can specify emailer part like this one:

<pre>
user:
  subject: Subject1
  from: my@organization.com  
  recipients_field: email
organization:
  from: no_reply@organization.com
  subject: Subject2
  recipients:
    - my@organization.com
redirect_to: /about-us
</pre>

Each top level key (except :redirect_to) in this config means some custom email page part (like "email_key_html" for html parts or "email_key" for plain parts). So, for this configuration we must specify "email_user_html" part and "email_organization_html". With this feature you can specify as many config parts and templates as you want.
added new datepicker type

Added new admin form type which uses jquery datepicker for date selection

configuration section:
sonata_admin:
    form_types:
        datepicker:
            date_format: d.m.Y
            locale: ru

date_format - php format for field value (default Y-m-d)
locale - datepicker locale (default en-GB)

use example:
    protected function configureFormFields(FormMapper $formMapper)
    {
        $formMapper
            ->add('title')
            ->add('startDate', 'sonata_type_datepicker' )
        ;
   }
This is needed to share the twig-extension
Use self.table_name instead of set_table_name to fix Rails 3.2 deprecation message.
Handle foreign characters by using unicode, option to write to output file

This replaces all prints with xbmc.log calls with different severities. Also did some cosmetics like renaming paths and shares to sources and some consistency with string quotation and building. Handle foreign characters by using Unicode and added ability to write missing movies to an output file.
minor enhancedment

display empty string when link to maven repo and site is not availabel
Исправление анимации

Теперь как? У меня альфа-версия хрома, там вся анимация через жопу работает, проверить сложно.
Restoring original format on language files
Added package.json

Took a look at this tonight and it seems pretty useful... I might dig in to it a bit for my own needs. :)

Made a few small additions

* added a package.json to include dependencies
* wrote a silly integration test to prove it works
* changed readFileSync to readFile

The integration test is really an experiment on my part. It starts a webserver listening for posts at http://localhost:3001/example and when a request is recieved verifies "headers" and "headers.return-path" are present in the json posted to it. 

The package.json allows one to just type `npm install` from the project directory to gather all the dependencies.

Good stuff, keep it up. :+1:
Select widget

This is a fix to make the select widget multiple.
Native dialogs to HTML5

Removed the native dialogs in place of using the HTML5 Dialogs
Adds support for default picker

Fixes issue blackberry/BB10-WebWorks-Framework#150
Fixes issue blackberry-webworks/BB10-Webworks-API#27

Functional tests are here - 
test/test-app/manual/html5/input.html

From the test-app you get it by
Manual --> html5 --> Input Test
Adding proper setter for ContextMenuCancel event that was missing from t...

...he client webview since it was referenced from WP fixes PR: 249122
Modified AccountController.SignIn methods to be virtual

I wanted to override the AccountController.SignIn methods to provide
additional behaviors but still utilize the logic in the standard
methods.
deleted var set false in nodeconstructor and added to query in repositor...

...y
resolving errors described at http://tiny.cc/qsd8mw

Change-Id: I596815a6a2de07bc9c3bc511acec025ff9e5e974

Relevant pastebin showing the errors I received is located at http://pastebin.com/Jsm0T8PR

After making these changes, everything compiled without error.
Message is now shown to the user who sent it

ALL the merge requests
timob-10527: KitchenSink: Mobileweb : Unsupported tests for kitchensink for mobileweb should be removed or grayed out

http://jira.appcelerator.org/browse/TIMOB-10527
Added ReadMe, License, and Notice files.
Spelling, grammar changes

Some spelling and grammar fixes, a few word choice changes, and one or two sentence-level changes for improved clarity/accuracy. These go up to (not including) Section I.4.
Changes to service start/stop infrastructure
Building on Ubuntu Oneiric

Hi Gabor,

I've made a small change in the SConstruct file to build on Ubuntu Oneiric (32-bit). I also added a list of build time dependencies to the README file. Would you like to pull the changes into your repository?

Maciej
add lines

add lines that were meant to be added
added misssing MagneticFieldSensor

cc @Smuldr
German Translation for QuickApps-CMS by Oliver Strixner

v0.0.1 First test infrastructure, change foldernames
and doing some translations
fix voip

Please also change the kernel config like this:

#
# MSM SoC Audio support
#
CONFIG_SND_MSM_MVS_DAI_SOC=y
CONFIG_SND_SOC_MSM8X60_PCM=y
CONFIG_SND_SOC_MSM8X60_DAI=y
CONFIG_SND_SOC_MSM8X60=y
# CONFIG_SND_SOC_MSM_QDSP6_HDMI_AUDIO is not set
# CONFIG_MSM_8x60_VOIP is not set
CONFIG_SND_SOC_MSM_QDSP6_INTF=y
CONFIG_SND_VOIP_PCM=y
CONFIG_MSM_8x60_VOIP=y
updates

This pull request includes:

1. Remove project files from git.
2. Use Flash's built in JSON (only available in latest Flash versions?).
3. Use xmlsocket instead of http for fetching crossdomain.xml (suppresses error in Flash 11).
Adds "theme" attribute to pages as string

After spending some time trying out integration with the themes-for-rails gem, I've come around to think that's not the right way of approaching this story. First, tying themes in to the Lightweight engine is problematic, because it's non-trivial to make the engine theme-aware. Second, conceptually it feels like the Theme as the gem understands it is too heavyweight a tool for this story.

This approach, on the other hand, is very lightweight: it just adds a custom class name to the div containing the page content. This allows a designer to provide custom CSS for different page layouts by targeting that container class, essentially namespacing their CSS. The question, which I suppose I'll need to put to Ethan, is whether this is *enough* or whether we also need to provide a means of adapting the HTML depending on the 'theme'.
Folder option and include path fix.

I added a way to designate a sub-folder for text includes. Also, there were invalid module id's being generated for included text files, so I converted all \ characters to /. This also matches the way that module paths are normalized by require, I believe.
I've added code to allow for alternate download locations

This will allow the user to house the plugin zip file on any server, including Amazon S3, Rackspace CloudFiles, etc. This update includes modifying both the wp_auto_update class and the update.php file for the API server.
COLORADO-2535 modifyDatastreamByReference Error

We were calling modifyDatastreamByReference and sending it a URL
that didn't necessarily exist depending on your drupal configuration.
I modified it so that it creates a file in a publically accessable
location. That way we can ingest it. Also added a line to delete
the file when we are done with it.
Removed test database named and replaced with filler.

Last commit had a database name filled in already. Swapped it out with filler "MYSQL_DATABASE"
Added imagefield_crop module for testing
Old version just for testing
Added getMostRecentVideoFeed() to return the most recent youtube uploads
Trad dairymainhouseinterior

J'ai eu l'incroyable génie de ne pas pusher à chaque modif du fichier... tu devrais moins te prendre la tête pour la relecture. :)

A plus tard.
add perl control test
Relecture du fichier colo.xml
unrelated.
Yes, I tested it. 
It caused HTML validation errors before.
Is this fix tested and working. I want to release today if so
Just scanned it for obvious things. I'll let the Akka Team do the 'real' review :) No point to punch lamas :)
@dreamr

> Santa kills 3 kittens for every vulnerable model

In this case there was a dead octokitten.
NICE CATCH! I learned something today!
```
Returns the shortest distance between self and o.

If o is a Point then self does not need to be convex.
If o is a Polygon, it and self must both be convex.

```

(Leave a space and write `convex` not `complex`; user periods at the end of sentences.)

I think the "how it does it" is not necessary.
I don't think this works. `twig_convert_encoding` accepts only strings, doesn't it? But `$values` is an array now.
再 1e7afb5 里删掉了 Trans2.java
Back to real life:  If your rails app is cracked, your customer WILL NOT ask the cracker for consulting or a vCard, they will call you and blame You.  This guy did nothing wrong, he raised a red flag, got called a troll for his troubles, then demonstrated the problem, and is now getting beat up.  He could have done so much worse...
![](http://i.qkme.me/35mqm1.jpg)
Yes, the problem you are trying to figure out has been in master/trunk before we even moved to Git. I was planning to work on it as it blocks scalagwt development (i'm using @magarciaEPFL's jdk2ikvm) but I got distracted by all sorts of things happening at the end of the year.

Me and Martin wanted to fix it here: https://github.com/scala/scala/pull/1 but I never had a chance to test it. I'll have time to work on this stuff on Wednesday. Let me know if you fix it before that time or you need some more info.

PS. This issue is particularly annoying due to the fact that I don't know how to write test-case for it. I'm not used to fixing bugs without tests being written first.
It doesn't.

My setup:
Linux 2.6.38-13-generic #54-Ubuntu SMP Tue Jan 3 13:38:12 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux
Ubuntu 11.04
git version 1.7.4.1
Finalmente l'estrazione dal db funziona.
haven't touch it for a long long long long lllllllllllllllllllllllllllllllllooooooooooooooooooooooooonnnnnnnnnnnnnnnnnnnnnnnnngggggggggggggggggggg time
****

对于这种具有明显扩展性质的东西，不要放到主仓库

建议新开一个Repository
ok, I'm good with that then.
There is an accessor in the VM for the MM already (`getMemoryManager()`). The `VMAllocatedList<T>` class is a convenience, targeted at all the Implementation's to come. If you have a solution that allows one to have at his disposal a class with the same interface as `VMAllocatedList<T>`, but without requiring to write that `VMAllocatedList<T>` in a closed way, I will be happy to change that.
Honest question: Is there a good reason for this? 

Rails provides a unique name to each version of the precompiled assets. If the file name is the same it's still the same file, regardless of the last modified date. There are many reasons why the Last-Modified date may differ. Removing it would completely eliminate the validation requests, relying on rails to simply provide a newly-named file when the asset changes.  What am I missing?
I believe the replace uri problem is solved on faraday master, they are already calling dup in the object before changing it:
https://github.com/technoweenie/faraday/blob/master/lib/faraday/request.rb#L54
I don't want to look fussy, but generally size() != 0 should be faster than !isEmpty() because of the applied negation ;)
I was tackled same works at the same time ;-)

BTW, when I executed the following code, I always get Fixnum. typecasting?

```
Company.joins(:contracts).select("developer_id").first.developer_id.class
Company.find_by_sql("select * from companies c inner join contracts co on c.id = co.company_id").first.developer_id.class
```
This should also be removed in phprojekt/application/Core/Sql/Db.json
Yeah, I was referring to autoloading. I know that makes some things less pretty, but much simpler when it comes to loading. Any other suggestions? We could, of course, add very generic and important classes like Cache_Filter and the interfaces, to src/Cache.php. We need them anyway and that way they'd be out of that directory.
(Sorry, I haven't been keeping track much of scipy pull requests recently)

Just a quick browse through the algorithm, i haven't looked at the binning part yet.

Overall I think this looks good. I remember parts of it from np.histogramdd, and bincount should be reasonably fast. Did you compare notes with ndimage.measurements? The c code there has been replaced with a similar code, but I don't remember the details.

The statistics part could eventually be replaced by a groupby functionality in numpy, A (visual?) separation between the binning and the groupby parts might be useful later on. Or maybe it's already separated enough.

The tests are drawing lots of random numbers. The random numbers should have a fixed seed. One possibility would be to put all tests into a class, so that the random numbers could be reused (generated once in a setupclass ? method)
Can some random number tests be replaced by just a simple explicit example? e.g. hardcoding one draw of the random numbers with a smaller number of observations (which could be manually verified)? 

Are the binning and binning options the same as for histogram? I didn't check the details.
Yeah, I was actually thinking about redoing the option code. Specifically I wanted to move options into the DSL and auto-add option accessors a la imagemagick.rb, which could potentially be more complex to allow for returning values out of `--foo=bar` type options.
# Nope

Failed like champ.
Looks like bases can now be removed from both method sigs and flowing back from there the calculation of the base sets can also be removed.  Minds cleaning that up?  Otherwise - this is great
Okay Some many Sh*t I have just check what can conflict with my previous changes and found none! So you are good to merge the Sh*t
A few comments:
 * DataInputStream.readLine() is @deprecated and you should better use BufferReader.readLine() here if you really want to read a String
 * Wouldn't it be more efficient to change ErrorReply to use ChannelBuffer in the constructor and then use "ChannelBuffer.readBytes(ChannelBuffer.byteBefore(ChannelBufferIndexFinder.CRLF))" here ? This would save you some copy and converting. See the ChannelBuffer apidocs for more info
To make this work for me, I had to disable docs on libvbucket by doing a "--without-docs".  Otherwise, it depends on pod2man, which brew couldn't find (and if it does exist, it probably brings a ton of Perl in).
Async.Delay
This is an internal API method, it  shouldn't be exposed here.
I added some explanation in e00df9b2a88cb9e692ee102ff33f1ffe62c4dd22.  Let me know if it's clear enough.
I was only thinking length checks, nothing fancy.

If you try to stick something too long in the database it gets truncated, and that can cause confusion / anger. For example, if the migration would say that :length => 4 on the city (unlikely, but still), I'd write "Boston, MA, USA" and get back "Bost" with no warning whatsoever.
Wow, thanks a lot for doing this.

If you have any questions related to fuse4x - let me know.
It is my patch.
I mean it's going to be turned into a string sooner or later. Better to discover that you stuck something in there that can't be turned into a string immediately that finding out at some later time, no?
Hab doch extra ne mail geschrieben ;)
this needs to be able to support reading from both gzip'd and non-gzip'd files since people will still have the old files in the directory.
You need to add `:foreign_key => "parent-id"` with the dash in there instead of the underscore (Then you will see all the breakages). The problem is that we need to have both accessors for the aliased attribute name *and* the original attribute name, plus all the dirty accessors... (See https://github.com/mongoid/mongoid/issues/1571) No matter how you look at it something is going to break with that dash in there without using define_method. (And yes you are correct can only use #send in these cases, but Mongoid internally only uses send to do the auto-setting.)

The problem with underscore is it only handles the dash... If other special characters are in there then they are not covered and this blows up again, and there's an entire slew of unknowns around this that just sending the specialized stuff to define_method can cover for now... 

In the future I may just not allow this since it's non-standard Ruby, and it's non-standard practice for field/column naming in database schemas (underscore is the norm)... I think given that the data is binary JSON in this case though it might be a bit of grey area... But my ultimate wish is that no one would name their fields like this and I could just prevent it completely. :)
If I ever want to cry myself to sleep, I'll read your code reviews
Can you elaborate a bit on what the pyparsing bug is?  Do you mean it requires *at least* 1.5.1, or exactly that version?
Thats fine. I've incorporated your changes into Rio 5.0, will push shortly

On Feb 24, 2012, at 1009AM, Dawid Loubser wrote:

> It's the convention we typically use, helps new developers off the bat (nice signal-to-noise ratio, and no need to try to cover everything conceivable in a single ignore file. Anyway, this was a minor detail committed with the rest :-)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/DawidLoubser/Rio/commit/8a5f7bc959cd13a9b0764dcbcaf525cf5b4b15ac#commitcomment-1007873
Hello, SilverIce, a very interesting work you have done.

Here's updated `transports` table, you seem to forget add it to this branch

http://paste2.org/p/1933762
I've never seen any debian package 'file names' that include the epoch. Apt-get also doesn't handle things well, it converts ":" in the filenames to %58 url-style encoding. Much confusings occur.

If you can show me a few debian packages with the epoch in the filename, we can revert this change or alternately make it a flag.
As far as I know "default-key" may not contain spaces.
Likewise, this should be `saved_prefs` instead of `old_prefs`.
Why bother checking if `hasOwnProperty` is available if you assume “modern standards support” anyway?
delete this or exclude from merge, been done already
done in 9e01d2600f2dd443ac0d148f9d60187d86714787! thanks!
Definitely something to investigate. Adding that to TODO.
Yes, the above comment was just for reference.
See issue #16
@brentonashworth This is fixed in https://github.com/fogus/marginalia/commit/8780370415c2f8c2efcab4f29285cce95eaae507.  Thanks the the report.
On 01/06/2012 07:15 PM, Yaroslav Halchenko wrote:
> ah -- right .  I guess it might be worth adding a comment like
>
> ```
> # if you run fail2ban under dedicated FAIL2BAN_USER, and have added static firewall
> # rules to initiate the corresponding chains, please create a iptables-xt_recent-echo.local
> # file containing the override:
> # [Definition]
> # actionstart =
> #
> ```
>
> I am yet to check either it would be in effect ;) but overall that might be a better strategy than modifying .conf file which would cause interaction any time we change this .conf and thus changes would conflict with user's changes in this file
>
> agree?
Agree.
Probably [nothing substantial](http://jsperf.com/captured-vs-not#chart=bar).
I don't think scale-9 is possible in the GPU without some special processing. You'd have to split the texture up into the 9 slices and then scale those properly in the runtime, which seems like a pain. I'd avoid it unless absolutely necessary.
I would say, "A user with that email address does not exist."
@indexzero Thanks. I feel a little silly now!

I'll whip this into shape tomorrow morn.
Hmm... Chyba emulator ma błąd, bo przy próbie powrotu ekranu do pozycji pionowej ekran dale zostaje poziomo... Nie wiem skąd wzięłaś ikonki, ale na moim wyświetlaczu wyglądają świetnie (uruchomiłem aplikację na prawdziwym telefonie a nie w emulatorze, bomba!!!)
Esto es tuyo, prefiero que en el oficial quede...
Thanks for highlighting mongolite. We really like the validation capabilities of mongokit :-)

I don't see the behaviour you state - CustomTypes behave differently than normal types in the structure.
eg:

    import mongokit
    c = mongokit.Connection('localhost')
    db = c.test

    @c.register
    class Test(mongokit.Document):
        structure = {
            'foo': unicode,
            'bar': unicode,
        }
    
    db.test.insert({'_id': 1})
    
    doc = db.test.Test.get_from_id(1)
    doc.validate() # <---- exception at this point
    
vs

    import datetime
    class CustomDate(mongokit.CustomType):
        mongo_type = unicode
        python_type = datetime.datetime
        def to_bson(self, value):
            """convert type to a mongodb type"""
            return unicode(datetime.datetime.strftime(value,'%y-%m-%d'))
        def to_python(self, value):
            """convert type to a python object"""
            if value is not None:
                return datetime.datetime.strptime(value, '%y-%m-%d')
        
    @c.register
    class Foo(mongokit.Document):
        structure = {
            "date": CustomDate(),
        }
        default_values = {'date':datetime.datetime(2008, 6, 7)}
        
    # insert a foo document without this field
    db.test.insert({'_id': 1})
    
    doc = db.test.Foo.get_from_id(1) # <----- exception at this point
    doc.validate()
added -e switch to echo to allow interpretation of \n
No, the browser side of this is not yet implemented...working on that today.

On Tue, Feb 7, 2012 at 6:09 AM, tburnett
<reply@reply.github.com>
wrote:
> This is exactly what I've been waiting for.  Do you have a notebook example?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ellisonbg/ipython/commit/461c873319128dceae422030c1bb44bf6dbd40fd#commitcomment-941038



-- 
Brian E. Granger
Cal Poly State University, San Luis Obispo
bgranger@calpoly.edu and ellisonbg@gmail.com
Why not just use my [patch](https://github.com/gitlabhq/gitlabhq/issues/324), really?
The original behaviour of throwing when trying to compare arrays was informative if slightly inconvenient. i'd worry that this change will cause unexpected results and provide no feedback

I think i'd be in favour of keeping the original behaviour, and as arrays are a "special case" in CF, creating  assertArraySame / assertArrayNotSame methods, and instructing users to use these methods in the throw message.
I made create-templates so that it can be invoked from javascript code or the command line, and did not change any semantics.  We should consider re-working how templates are generated and included.
Similar functionality for growth and lighting checks would be great, too!  Both of these hit the server pretty hard don't they?
I'll just have to nullcheck ItemStacks?
Ideally I guess the 2 commits should be separated, but feel free to submit the 2 of them in the same pull request, as I mostly reviewed them both and have no further objection.

Suggesting hitting `+` seems perfectly fine to me
You can just use : Tools::getValue('id_category')
??
Предлагаю iconsets/roster/mrim2006.jisp выпилить.
Please alphabetize these lines, and add " - Contributed by AdmiringWorm" to the end of each game you've added
I've also thought about this optimization. This is great news that there is already an implementation of ICV-coalescing. I would be very happy to see this as part of the asynchbase client and I'm sure that I would benefit from that too. Is there any chance to provide a first patch for tests or is it completely decoupled from the current asynchbase trunk?

On Mar 7, 2012, at 10:06 AM, Benoit Sigoure wrote:

> Would you benefit from increment coalescing?  I've implemented this in another ICV-intensive application that uses asynchbase and I'm able to reduce the number of ICVs by over 10x for my workload.  I've been thinking about porting that code to be part of asynchbase itself.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ylangisc/asynchbase/commit/4bb9a364c46fcd4600c707124a0ce140979dbcf6#commitcomment-1052386
>
use xla() function
i'll implement test to this implementation shortly
You should merge this commit into the commit adding the debug line.
The idea is to not have to ever see the debug output line.

You can use something like "git rebase --interactive HEAD~10" to rewrite your history.
please constants instead of raw numbers (like gtk.event.button.whatever instead of 2)
https://github.com/simplexo/Infinity/commit/0c09bc62ae996a0426f8636abc1feefe06f9302d  here spell sniffed and sought out spell research for svala
It could be renamed to project instead of init, and substitute on the whole class the call @project to project referring to the method ;p
You're returning here without dropping lock. Use a Mutex::Locker instead?
Are you sure? I asked on IRC about this, and general consensus there was that the designer framework had been pretty much abandoned along with greenhouse. It seems silly to require support for something like this instead of actively removing the need to keep it around.
于 2012-2-25 0:15, liu fengyun 写道:
> better reformat the readme using markdown:
>
> https://github.com/github/markup#readme
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/sleepcry/Busecretary/commit/c7225b64b327055a36e29c75cb296bfe5e0bd8e3#commitcomment-1008169
they dont say how to install in windows
i`m under windows and downloading photoshop,as you just proposed, to do 
some art design myself.


-- 
Long Live and Prosper
Yours
Liuq
Do we need to use a mock instead of a stub here?
From http://www.rossde.com/PGP/pgp_signatures.html:

"Line-end indicators are set to CR-LF (carriage-return and line-feed, hex 0A0D), replacing other line-end indicators such as CR-CR-LF or merely CR."

This may or may not apply here, but I think it would be easiest to have Libmacgpg convert
each \r\n first to \n and after that all \n to \r\n.
Or walk through each byte of the data and check for \n not preceded by \r.
Thanks for letting me know

On Fri, Mar 2, 2012 at 7:12 PM, samtsai <
reply@reply.github.com
> wrote:

> Turns out it was a perms issues. I fixed it by removing .npm in my user
> directory and re-tried npm install and it's good.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/LearnBoost/socket.io/commit/5312e154b304588230deaefb0bf2a5730f07cc39#commitcomment-1037575
>



-- 
Guillermo Rauch
LearnBoost CTO
http://devthought.com
Thanks. But, are you sure I don't need to do anything special in the way I generate code that calls `_cxa_find_matching_catch`? Right now, I'm just textually emitting a JavaScript function call. Unless emscripten then goes and parses the JavaScript which I emit, I don't see how it could work out that it needs to include the library function.
Is this needed for stable builds or just head builds?
Fixed both, thanks.
Oh, by the way, after you rebase you will need to `git push -f` the next time you want to push your changes, since rebasing is history-rewriting.
Check out commit 7447059b14a2e3a14db6d30319c4ce03105ef584, hopefully this will fix problems I might have introduced by being a less invasive change.
Tu as oublié le bouton Pause pour le joueur violet il me semble ;), et tu les as pas colorés ^^
I mean instead of this `<span class="b">WooCommerce %s</span>` - `<strong>WooCommerce %s</strong>`
Hard core :D
BabySmyle - нужно было BabySmile
On Thu, Dec 29, 2011 at 7:02 PM, Lars Immisch
<reply@reply.github.com>
wrote:
> Good idea. We should split out the timer stuff. That will make it much clearer.
>
> In the end, Arduino may be pretty empty. I like the PIN aliases, and a rudimentary init should stay.

In my previous version I had some cute startup stuff that let you
customise the init but still kept the generated code minimal. We might
want to take a look at that...

>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/larsimmisch/arduino--/commit/67f821a469f5447a7bd1d2b52c133e591901ecd3#commitcomment-826765
@ornicar- do we need a new integration branch for the v3 stuff or do you want to just use master?
not sure yet, I'll come back to that :)
Go ahead, demote me. There WILL be a mutiny.
It's not correct to say that using an ID is simply "the right thing". The ID attribute is only there to allow you to set a unique identifier on an element - http://dev.w3.org/html5/spec/Overview.html#the-id-attribute - and most modern development practices shy from styling elements based on their ID - http://oli.jp/2011/ids/ ...so it's staying as a class.
Dan,

I believe I only updated the website one, I was not aware there was an
email one! Do you want me to change both??

Laura

On Sat, Mar 3, 2012 at 5:03 PM, Dan Gooden <
reply@reply.github.com
> wrote:

> Hi Laura, there are two functions that generate recent activity
> statements. One for the website, and one for the email (they should really
> be refactored into one!).
>
> Did you update both? or just one?
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/ldow86/sharedearth-net/commit/22253b3aeb4bfafd005410504c4283f586667144#commitcomment-1039533
>
Are you using this day to day? Can we get it merged into Bushido's repo and published as a new gem version?
It would be better to put Mojang to the top.
yep i know, this is in development.
@kaelima It's not that. I'm just saying it's rude to randomly rename / remove public functions like that!
@sharq maybe, we will include only what can be usefull, we want a clean database with clean content.
thanks. i resolved conflicts manually and pushed my master branch to github.
github is powerful, but can be complicted. i'm still learning too :-)
The fact that you wrote these comments waste more "hard drive" bytes than the code itself.
np ;D
if you didn't notice, it is alphabetically, not random
That's weird. On my box it compiles clean. Can you try to rebase to the most recent master?
We don't need to bump every version - the twiddle waka (~>) mean latest version in this tree. 

So 3.2.0.rc2 will request 3.2.1 when it's available
Just a quick check in pry:

```ruby
» pry                                                                                                    ruby-1.9.3-p0 
[1] pry(main)> '1.5'.match(/(\d\.)*(\d)/)[0]
=> "1.5"
```

I don't see any problems.
i just had a look at the collection and can not explain this strange behavior, yet
to find this out is some perfect occupation for a public holiday we have on Friday :-)
If you know more hg than I do, I would appreciate it if you could verify that el-get's hg method actually follows this rule. I suspect it doesn't.
Ok merci.
Looks like a fix...nice find @danielduner
This commit close #1 issue.
Currently, _get_highest_rid_to_ack() adds stuff to unacked_responses (has a side effect). This seems to be totally off. The name doesn't suggest that it does something that mutates internal state.

Additionally, there is also a bug where you might not return anything!
@simi It is https://github.com/resolve/refinerycms/blob/master/Gemfile#L72
@samvermette Bah la réduction du noise dans le code. La même raison qui me fait adorer Ruby et CoffeeScript, par exemple!

C’est vrai que je parle à un NSCoder qui en voit de toutes les couleurs côté syntaxe… ;)
O "emergente" se refere ao título do trabalho. Acha melhor colocar "Mas TDD é uma técnica emergente de desenvolvimento de software? Não e sim."?
> About fucking time! :)

@FroMage Your mother never told you not to swear?
so many extra checks for nothing.
perfect. Worked
I think I should add a few comments, but so far this is pretty nice, even though too much Java gives me a headache.
When you have that check, then I'll update the test to try double-opening a fractal tree once in a while. Just to make sure it fails.
`=>` is a [C# lambda operator](http://msdn.microsoft.com/en-us/library/bb397687.aspx) so it's common for many people.

Type of expression like `a => a` is inferred from using.
    auto func = a => a; // error
    int function(int) d = a=>a; // compiles

If it is passed as `alias` template parameter, the expression is passed (without instantiation). One can call `t!(a=>a)();` or `t!((a){ return a; })();` where `t` is:
    void t(alias f)() {
        pragma(msg, typeof( f(1)   )); // prints int,    like (a=>a)(1)
        pragma(msg, typeof( f(1.0) )); // prints double, like (a=>a)(1.0)
    }
If you change method-signatures, always keep the old ones (and just deprecate them)! We don't want to break plugins that were using this method!
yo va a ser dificil pero giresse esta entrando mucho, le dire que te busque
You're right. Let me fix that.
Thanks. there are a few test cases i want to run this through first. I'm in the middle of a fairly big refactoring and will get to this after that.
since 12.1
`TBL_KEY`?
core of 430 can available now?
Can't we do this kind of message passing with either intents or content uris?   Do we really need to use servers and sockets for this?
Should have the description for this action.
Hi Mitch,

Command+I applies your indentation preferences (converting tabs to spaces, or vice versa - whatever you've specified).

Yes, the key is to be consistent.  Where tabs break down is when you try to align things beyond the first indentation.  Sometimes you may want to do

function(Object arg1,
         Object arg2,
         Object arg3)

This is impossible to do with tabs as how a tab is interpretted on each system is variable (I may have tabs set to be displayed as 4 spaces, you may have it as 8).

Until this year, I was always a fan of mixing tabs and spaces (tab for the initial indentation, and space after everything else).  This works for someone who notices little details like me, but in practice have found most others are not that attentive (which is fine - it's not worth the time correcting people on).  Using all spaces is the common denominator so that all parties can be happy.

Let me know if you have more questions about this, but hope the above makes sense.

Steve

On Jan 7, 2012, at 5:41 PM, Mitchell Loeppky wrote:

> So what does command-I do in eclipse? Does it see that I use tabs and then
> make it consistent? If I used spaces would comd-I convert everything to
> tabs?
> 
> In article 1 it says that the key is just to be consistent so what is the
> point in changing to spaces. This particular piece of code was mostly
> copied from a GWT example and therefore inconsistent.
> On Jan 7, 2012 5:21 PM, "Steve Loeppky" <
> reply@reply.github.com>
> wrote:
> 
>> Pros:
>>       It enables someone to have more control on how their code is
>> displayed. You may like tabs 4 spaces wide, and I may like it 3. By using
>> tabs for the initial indent, we get an abstraction to be able to configure
>> this.
>> 
>>   Cons:
>>       A programmer has to be more conscious of what they're doing to
>> determine whether they should be tabbing, or spacing. Jeff Atwood believes
>> it combines all the complexity and pitfalls of both approaches.
>>       In practice, haven't found the necessary attentiveness for this
>> being present by default.
>> 
>>   Relevant articles:
>> 
>> http://www.codinghorror.com/blog/2009/04/death-to-the-space-infidels.html
>>       http://blogs.msdn.com/b/cyrusn/archive/2004/09/14/229474.aspx
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> 
>> https://github.com/BigLep/ChuckanutBayFoodsInternal/commit/076fd524cba98ddfd73928c5a265d9ed1bd4ebee#commitcomment-846963
>> 
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/BigLep/ChuckanutBayFoodsInternal/commit/076fd524cba98ddfd73928c5a265d9ed1bd4ebee#commitcomment-846979
@schmittjoh because the solver needs to know the constraints to be able to solve them. And packages provided by additional repos cannot be registered in the solver before you know which packages should be installed (i.e. after solving the constraints)
I would suggest to rather use a class instead of an id so the html won't be invalid when somebody should use the plugin more than once on a page.
...and BTW thanks for your work! - i used your tests when doing PostgreSQL dialect :)
Cierto, parece ser que lo han optimizado un poco, lo haré en el próximo commit :D
Just curious - why are you using SimpleMarker instead of google.maps.Marker?
Understand, I will change this code
ok, i understand the way you did it against other micro-framework.
Thx
Don't use such abbreviated variable names - we're not short of space here, so let the code document itself correctly.
'non-significantly differentially expressed' to 'non-differentially expressed' everywhere - thanks.
samples or factor values?
the point is to be clear that the "mechanism" can actually cause a test failure, since you're doing an explicit assert.
For some reason I always like to use named functions for recursion
I don't think this logic should be baked into this view.
Shouldn't the 2nd argument to Message be a default error msg?
or `scala.util.Random.shuffle`
One liners please
We asume that breaker round trip time is longer than duration of calls, which I think is a practical assumption, which simplifies the implementation a lot. Just saying.
Looks good!

So what happens if I stash too many messages? Are there any performance Implications? What happens if my actor is restarted?
I need more detail to construct a test.
Hmm, I wonder if theres a way we can split this debug info out when chef crashes. Thats mainly why I like it going to stderr.
do we have anything else we can do?

example do we know ythat we only want to do things when a certain action is being used so we could do
    class my_plugin(...):
    
        def before_view(self, data_dict):
            # if search don't do any work
            if c.controller == 'package' and c.action == 'search':
                return data_dict
            # do some work here

we can get `c` from `ckan.plugins.toolkit`
The extension shouldn't have to know the option is called "metadata". If we rename this (which we should, it's wonky), each extension will have to change it. Better to have a constant in the same module as the UnitAssociationCriteriaCommand for what the option represents and let the extension use that for the look up.
I doesn't look like there were corresponding updates to the docstring for the new arguments.
See https://github.com/wagn/wagn/blame/develop/lib/wagn/cardname.rb.  Basically nothing after 10/20 is represented in gem
I'll check, could be a mistake
and SAFE_RELEASE(wmMediaProperties)

and a few more, and same comment for the other 'return false' statements earlier in the function.

might be worth restructuring more like a try {} finally {} to be sure to always SAFE_RELEASE everything, without c/p?
or a goto
Than each time a class is instanciated the wrapper instanciates a new function per each public method so created functions at a given time is equals to (class instances * class public methods)
Don't use path.exists.  Use fs.stat, and verify that it's actually the sort of thing you want it to be.  (Ie, `st.isDirectory()` etc.)
this case is the one where Behat is installed as the main package, where we know that the vendor dir is not renamed
All the `.h` header files don't need to be listed here (I guess it doesn't hurt anything though...)
Try changing this line to get advanced optimizations to work:

```js
priv['core'].loadScript = function (def, success, failure) {
```
This does not solve the problem with performance. The Twig_Error_Loader exception is too slow. We must have a method, which does not throw Twig_Error exception.
TUIViewController needs to be imported for the view controller methods we call.
Change the else if into an elif, saves a level of indentation.
Well, lets do it this way then `options.fd !== undefined ? ... : ...`. I really dislike `in` keyword.
"if"
Well, you store the commit message that the user entered as member already. The only thing that the beforewrite handler needs to do is

    options.params.handle = this._commitMsg;
    delete this._commitMsg;

The rest can be done in beforesave.
seems  you didn't try to test it, here miss a ';)',  please make sure the tests passes before push :)
это опять глюки с discussion
if `avatar_user_method` just returned `:email` by default then you wouldn't need the conditional logic here at all :-)
Why is this flattened?  Are we supporting `Model.select!([[:foo], [[[:bar]]]])`?
Perhaps its a difference of milliseconds, not visible in `#inspect` output. Could see with `#to_f`...
Instead of calling [] on self will be better to pass just value in the false part of ternary operator
Can we somehow ask this information to the `lookup_context` instead of guessing when comparing formats?

For example, whenever the formats= method in the lookup_context is invoked and we do the logic you mentioned, we could set a variable named `@html_fallback_for_js` that we would simply ask here.

I really like your solution, thanks for the work!
Sorry about that. My bad. Thanks for finding it.
Is there a Rails 3+ on symbolic Hash formatting?  Should this remain as `instance_writer: false`, or does it not matter?
1) I believe it's a guideline, not a standard, and 2) ಠ_ಠ
Sorry, I didn't quite understand what you meant here.  Did you mean you think the previous was correct and my change is wrong or that the previous was wrong and my change is correct?

Thanks
Oh, really, to my shame, I somehow missed De Morgan here.

> Is this being done for some compelling reason? It seems like a really bad idea.

Going to take some time to study.
I was thinking of going the other way: -Xfatal-warnings
Thanks for all the help. It passed the doctest now. Hopefully it goes smoothly from hereon. Thanks a bunch again.
the word "compare" is missing?
Changed to suggested use of symbols.
@smichr, as I mentioned earlier, these files are not part of my code (this is the old plotting module that you are looking at). I have indeed moved the files, however I have not done any changes to them and I would prefer not to touch them in this pull request as this is unrelated work. Though, it is true that there are issues to be fixed in the old plotting module.
Yeah I was also wondering whether to include this as a comment or as part of the docstring. What do you think about rewriting the docstring as:

Generates the dihedral group of order 2n, Dn, *yep it should be just Dn instead of D2n*. The result
is given as a subgroup of Sn, except for the special cases n=1 (the group S2) and n=2 (the Klein 
4-group) where that's not possible and embeddings in S2 and S4 respectively are given.
perhaps this should go to the end of the file as in intervalarithmetic's test code.
My bad, the problem was somewhere else. Got why you raise the exception.
buna gerek yok.
goog.require('tart.StateMachine');
you have to empty the element first.
Explicit space after `for`.
Why this one is empty? Short form will not work with this translation.
should be `borderColor`
Superfluous cast and sizeof.
Move the `style` to the styleshet.
I think it's fine to remove this test, since the value it's checking is kind of ridiculous.
Don't need this line if you do the changed above
sharded, not sharding.
I'd suggest emphasizing that the connection must be available in *both* directions: from m1 -> m2, AND from m2->m1.  Network topology and/or firewall issues mean that you need to check going both ways.
Where can 'RETURN_TO_BREAK' be used? Or if it's fon't used, I will delete it.
(the other language key is `'RETURN_TO' => 'Return to',` for information)

* styles/prosilver/template/jumpbox.html

`<p></p><p><a href="{U_VIEW_FORUM}" class="left-box {S_CONTENT_FLOW_BEGIN}" accesskey="r">{L_RETURN_TO} {FORUM_NAME}</a></p>`

`<p></p><p><a href="{U_INDEX}" class="left-box {S_CONTENT_FLOW_BEGIN}" accesskey="r">{L_RETURN_TO} {L_INDEX}</a></p>`

`<p></p><p><a class="left-box {S_CONTENT_FLOW_BEGIN}" href="{U_SEARCH_TOPIC}" accesskey="r">{L_RETURN_TO_COLON} {SEARCH_TOPIC}</a></p>`

`<p></p><p><a class="left-box {S_CONTENT_FLOW_BEGIN}" href="{U_SEARCH}" title="{L_SEARCH_ADV}" accesskey="r">{L_RETURN_TO_SEARCH_ADV}</a></p>`

 * styles/prosilver/template/message_body.html

`<!-- IF SCRIPT_NAME == "search" and not S_BOARD_DISABLED and not S_NO_SEARCH and L_RETURN_TO_SEARCH_ADV --><p><a href="{U_SEARCH}" class="{S_CONTENT_FLOW_BEGIN}">{L_RETURN_TO_SEARCH_ADV}</a></p><!-- ENDIF -->`

`<!-- IF SCRIPT_NAME == "search" and not S_BOARD_DISABLED and not S_NO_SEARCH and L_RETURN_TO_SEARCH_ADV --><p><a href="{U_SEARCH}" class="{S_CONTENT_FLOW_BEGIN}">{L_RETURN_TO_SEARCH_ADV}</a></p><!-- ENDIF -->`

* styles/prosilver/template/search_results.html

`<p><a class="{S_CONTENT_FLOW_BEGIN}" href="{U_SEARCH_TOPIC}">{L_RETURN_TO_COLON} {SEARCH_TOPIC}</a></p>`

`<p><a class="{S_CONTENT_FLOW_BEGIN}" href="{U_SEARCH}" title="{L_SEARCH_ADV}">{L_RETURN_TO_SEARCH_ADV}</a></p>`

* styles/prosilver/template/ucp_pm_message_header.html

`<!-- IF S_VIEW_MESSAGE --><a class="{S_CONTENT_FLOW_BEGIN}" href="{U_CURRENT_FOLDER}">{L_RETURN_TO} {CUR_FOLDER_NAME}</a><!-- ENDIF -->`
This does not look like a pleasant to use api. What is in user_id_ary and username_ary?
No dash in override..
This language file should be removed from the `mods` directory. Can be merged into `/langauge/en/search.php` and `/language/en/acp/search.php` depending where the string is shown.
Please use a data provider and move user ids, parameter value and expected output out of this function.
Just for our sanity debugging stuff for the near future do you mind adding:

    JSON.stringify(data) ..  '\n'
Okay. Then you should fix the misprint :)
Тук бих го написал без къдравите скоби, ето така:

``` ruby
link_to image_tag('rss.gif', :alt => alt_text), url, :id => 'rss_feed'
```

Донякъде е въпрос на вкус/конвенция, но мисля, че повечето рубисти гледат да ги избягват, когато хешовете са последни аргументи.
just kidding... done.
`data:image/git;base64,R0lGODlhAQABAIAAAP///wAAACH5BAEAAAAALAAAAAABAAEAAAICRAEAOw`
The ItemStack "ingredient" is declared as a parameter in this constructor, but it is never used within the constructor to set the class-wide level ItemStack "ingredient"
I didn't realize I spelled it differently. I was trying to follow the plyr lead. We should probably change it. I'm not sure I like the name anyway, but I couldn't think of anything better. We're making a new DataFrame from columns of an existing DataFrame. Maybe we just overload DataFrame. Any better ideas?
I think colnames should be parametric. Ideally any type, but at a minimum any String type, which will allow for UTF-8 and such for international use. How about a constructor of the form DataFrame(::Type{T}), and a 0-argument version that calls that with ASCIIString?
Prefix with "this."
Type and members are missing XML-comments
Perhaps this could be refactored a bit so it's easier to follow the logic
I will fix this in my next pull request.

On Wed, Apr 18, 2012 at 1:11 PM, nathanleiby <
reply@reply.github.com
> wrote:

> > @@ -0,0 +1,15 @@
> > +Ext.define('RaxaEmr.Screener.controller.DocList', {
> > +    extend: 'Ext.app.Controller',
> > +
> > +    refs: [{
> > +        ref: 'docslistGrid',
> > +        selector: 'docslistgrid'
>
> style... we are using camelCasing throughout. Doesn't need to be fixed
> immediately, but FYI for future commits. We'll have a code-cleanup phase in
> 2-4 weeks in which we'll address style issues / coding conventions.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/Raxa/Raxa-JSS/pull/18/files#r699796
>



-- 
Vikas Singh
Senior Undergraduate
Department of Computer Engineering
Institute of Technology
Banaras Hindu University
Varanasi
Ph: +91 9838438152
no need for "ind"
How about a single ```file.read()``` or possibly ```file.seek()```, rather than the overhead of looping through and unpacking values you don't need?
Why numpy? I feel like the more things I install with pip, the less benefit I get from the OS package manager. If the required version is available from apt/yum/... I think we should encourage people down that route.
Ok, someone else more familiar with the code should look at it. My feeling is that the gradient should be cut as soon as a None is found, so that grad() would never be called with a None as input. I find it weird to check for None in this Op while other Ops seem to be assuming their input grad is not None...
This is forbidden because scipy.sparse do not implement this feature for sparse matrix.
provide better verbose_name
that code checks only if spellId is 0, not if it's -1 or a nonexistent spell != 0 . Even if incomplete, I tell you what I see wrong so you have all the time to fix it before pushing to the main repo.
Umm.. Has this class been moved into mods/_standard/basiclti/classes/AContent_LiveContentLink.class.php?