labofsahil/github-event-dataset-2012 huggingface dataset:
Updates for API v9

This branch updates things to work with API v9 which was recently released. It breaks backwards compatibility and will not work with API versions <9.

For both the data-loaders and the demo I moved the settings to an included file `config.php` which needs to be made by copying `config.sample.php` and entering your own credentials.

This release also marks the start of a better approach to the whole repo in which we'll be making branches here and then merging back to master as appropriate instead of @TeamOneJ pushing from his fork. 

I will also begin tagging releases. My tagging plan is to have them all be 1.x.x with the first .x denoting which version of the API it is targeted for and, if necessary, the second .x shall be revisions within that targeted version.
trackball wake and trackball navigation for legacy device like nexus one
allow-tags-with-empty-values-10

gridaphobe: **approve**

Allow empty tag values in the popup (convert to null).

Fixes #10
Actualización de reportes

Nuevo reposte agregado
add custom form types to access container
ENHANCEMENT Configurable source paths for Solr config

- Made Solr_Configure task output more expressive by exposing the directories involved
- Documented configuration copying process, and how to override it
Fixed typo in usage doc
Allow type as scope

I needed to be able to use the _type column as scope to give each class in an STI table it's own acts_as_list scope, which this patch enables.
nice url & icon for iOS
Move OK counties to prod
Update bitstream loading progress less frequently

Changed the hardcoded update interval for _x6500.progresshandler()_, to reduce the amount of messages during programming. Maybe there's a way to display that progress using the curses interface...
Added order and defaults
Fixed typo in configuration file (routers -> routes)
Improvements of Transactions benchmark

some are trivial, some are important.
Main changes:
 - warmup should not be different, just take out of stats the first time slice of 10 seconds
 - reduce number of callables being used (reuse them in a loop)
 - don't start all writer threads before reader or vice-versa as they are not going to run in parallel but are enqueued
Remove D2 features from the D1 spec

D1 doesn't have const, inout member functions
D1 doesn't have @attributes
D1 doesn't have typeof(return)
D1 doesn't have lambda expressions
Adding support for the 480x800 devices

Might need more adjustments in the pic sizes but seems to fit well on my screen.
No problems that I've encountered

Change-Id: I152b8d6bd655de575aa9c4b0875db8334c910355
Update res/values-fr/strings.xml

et voilà... First read, adding missing translations, some modifications done in french text.
loading phpspec from composer
create.gsp should post back to /create

Posting back to /create doesn't do much. You should use the save action to store new instances in the db
Minor fixes.

Two changes:
  * Prevent Fieldset decorator from giving element attributes to the
    surrounding fieldset.
  * When the element's 'required' flag is set, the appropriate HTML5
    'required' attribute is added.
update omx mm-video
remove shorcuts.js
２訳完了しました

本家にマージをお願いします。
今回は作業が大変遅くなりすみませんでした。
よろしくお願いします。
Relecture du chapitre 00

Il y a-t-il une notre de prévu pour les traducteurs et relecteurs de la version française ? ;)
Made required changes in authentication.rb and curriculum.rb for creatin...

...g a proposal using ks-embedded
Yes, I understood the same think when I first viewed the code,
but since I wanted to commit any changes without rebooting
then I needed a sub-menu to be able to change accordingly(dynamically).
Thus I kept in init_menu only the items of main menu.
Reviewers: Is this fine? :/
Looks good. The predicate name `CCastable?` should be capitalized, though. Other useful conversions would be to cast any `Pointer[T]` to `RawPointer`, and to call `cstring` automatically for `Pointer[CChar]` arguments.
Maybe a todo should be added telling to handle flipped tiles, also. Apart from that, I guess you can push it :)
Just to keep it consistent across the project, I am going to alter these in the commits to be this instead:

```ruby
helper 'spree/promotion_rules'
```
为什么没有必要....
Dekuji za rychly bugfix. [ http://forum.nette.org/cs/9984-vnorene-komponenty-a-jejich-parametry ]
Rails doc http://apidock.com/rails/ActiveRecord/Persistence/update_attribute: 

> Callbacks are Invoked

If you want to avoid callbacks you can use this:

    t.update_column(:state, 'approved')
omgwtfbbcode i like.
Hint: if you write such *intentionally*, write down also the **intention** you had in your mind. Others may not see it (or even you may forget). Thx
Was the issue that this didn't compile in MSVC when using InputCommandMapper subclasses which also had templates?
I can put the checking code here and actually my 1st rewrite was like that.

About inlining, it was like that in my 1st and 2nd rewrite and then harrah said ``In general, keep methods small and make them a combination of other well-separated methods. For example, the Git resolver implementation should have the branch checkout code as a separate method.`` so I used small methods like in previous RetrieveUnit.
Yup, will fix that
is this intended? with my scripts tinyxml still needs to be in tinyXML/tinyxml.h. I can change it if needed but currently it won't find the header.
Firefox 3 will actually remember certificate serial numbers and reject a new certificate with the same serial number as a previous certificate (coming from the same authority, that is), so make sure you’re keeping track of serial numbers. 
Source from: http://www.devklog.net/2008/05/25/generating-ssl-certificates-that-will-make-firefox-3-happy/
why this come-back?
Apparently URI.encode_www_form is a 1.9ism
looks good, i'll merge it in, thanks!
Of course there is no useful purpose of focusing a non-visible element. The issue I'm having is that this is crashing IE<9 because jquery mobile attempts to focus hidden elements somewhere. (I understand this is not the fault of jquery) I was just curious as to what purpose the offsetWidth condition served.
These commits don't exist.
Hi,

> That's what I get for testing on one machine and not pushing the test branch to GitHub and then submitting from another box.
> 
> Lesson learned.
> 
> Could you please release new packages in OBS with this fix?

will do that as soon as I get a clue on 743159

Regards,
Marcus
-- 
 Public Key available
 gpg --keyserver gpg-keyserver.de --recv-keys 0xCCE3C6A2
 -------------------------------------------------------
 Marcus Schäfer (Res. & Dev.)   SUSE LINUX Products GmbH
 Tel: 0911-740 53 0             Maxfeldstrasse 5
 FAX: 0911-740 53 479           D-90409 Nürnberg
 GF:  Jeff Hawn,Jennifer Guild, Felix Imendörffer                  
 HRB: 21284 (AG Nürnberg)       Germany
 http://www.suse.de
 -------------------------------------------------------
merged should be the other way arround
it's same as my implementation except some exception processes,
as the return value, I prefer mnesia style {atomic, Val} | {aborted, Reason}, and use abort(Reason) to rollback the transaction.
For module style, do you mean you prefer the stand alone module?
Sadly I fear that this will break a lot of other dependent code potentially outside of urllib3. I'd like to avoid returning the path if we can help it. I'm not sure if this is necessary for the proxy tunneling fix, is it?
For now, treat a "date" as being 12:00AM UTC, since there is no way, currently, to have a "timeless" representation.  We'll will likely have to do some additional work in this area.
I tried without it, but it didn't make it executable.  I'm not sure why.
This looks really like a hack and it's not even documented why this is not setup in the theme layout files but instead hardcoded here.
Why the || here?  You don't check for NULL output elsewhere...
A little more clarification and a typo fix.
że jaki language? :D
I actually didn't look that far up the code. Certainly this code be turned into inlineCallbacks style code.
For "Fire!" i think the best way it use spell_linked_spell.
But i start to make a new version with EffectMask =)
this bit should be squished (no big deal)
I do not use this feature, but the experience came to me one question.

Imagine the following:

1) The user writes message.
2) Erases and does not send message.
3) He writes message.
4) Delete again and does not send message.
5) And he writes message.

Does in those situations where there is no items of one or other of the notifications think should not be notified again.

But it's my opinion.
@Reamer, i export this dump with gdb.
First Dump: I don't understand.
Sec Dump: for logout player and clear corpse happen.
Thats just a copy and paste from Weds when I had no time to worry about such things, but I've got time now.

/me worries.
I have only tested on Linux so far.

On Tue, 28 Feb 2012, quentinmit wrote:

> Is this true on Linux or OS X or both? (Also, this comment does not appear
> on the web)
>
> On Tue, 28 Feb 2012, Keith Winstein wrote:
>
>> For some reason the server is not quitting on SIGINT or SIGTERM (as it probably should and did before this commit).
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/quentinmit/mosh/commit/10f781dd5aaa0fd30696711429533901cd090245#commitcomment-1024587
>
`takebuf` (in sys.c) already does the first part, except in small data cases. `readuntil` does this in all cases. We don't have the one-pass encoding check yet of course.
> There is no source-file-name standard.

Exactly my point! There are just conventions used. And many compilers also default to certain extensions, but also allow overriding them. Eg. MS CL compiler allows to define C/C++ code with options `/Tc` and `/Tp`.

> We have --std=c99 and --std=c++11 for CLI already and in standards.h.

These are global, not file per file. Which is a problem already in project mixing C and C++?
Just delete that file from project.
https://github.com/fallenangelX/Infinity/commit/ef2ca20e16b8031e14a6a74881d1304095b5a3d9 there our a few more
Why change this?  Wasn't `richText` correct?
The dependency for actionpack 2.3.14 is `rack '~> 1.1.0'`
How is it, or why are you using 1.2?
should simply be ``if``
Looks nice. I've only gone through the gist, not the detail, but I like the structure (e.g. putting it into misc as a private method was a good plan).

And you used a decorator! I've still never actually bothered to use on in earnest, so that might be the first one in the project! Actually I have used `@class_method` but that barely counts. ;-)
Yeah I know I know:)
On Mar 7, 2012 1:38 PM, "Moritz Peters" <
reply@reply.github.com>
wrote:

> npm publish? :)
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/teknopaul/dom-js/commit/3ea432704c2d266cdeb61f14d1eea2585d584e6b#commitcomment-1053029
>
Danke dafür. android:showAsAction="ifRoom" ist ein feature, was den button automatisch correct anzeigt?
I believe I have fixed all the bugs I found that I created when adding things. and have committed everything including the two joda projects in the different repositories so that way they can be downloaded and maven may be used to install them for AutoGroup to compile against, and since i changed their package name chestshop can't complaind about not finding them in AutoGroup's  jar. If you wish to try it I uploaded it as version 1.2.2
It definitly should cause the warning at line 314 "warning("Unexpected File Position 0x%03X At End of Save/Load", s.bytesSynced());" to be trigged all the time I think.
Actually in the XCSoar project we are aiming to be pretty much backwards compatible. I haven't tried your application lately, but I don't think there is much that needs to be done.
+1 I think
>you actually have to use the file input

But I wrote `use Shift+Click on the file input`.
I can't see how I could make it more obvious.
Whoops, good catch. Yes, it should be.
https://github.com/codels/TrinityNya/commit/c2591afcb83a01517e9da06cbb1e917336397595
Yes. Avoids project root directory = workspace root directory
There's an all_files on a version which is faster.
On 9 February 2012 10:18, Erik Ogenvik
<reply@reply.github.com>
wrote:
> With C++11 you should also be able to add "constexpr" to them to make them even more usable at compile time.
>

I also found that the c++0x cmath header contains a bunch more useful
function, all of which are overloaded for different numeric types, and
will allow us to get rid of the iround header in Mercator.

The trouble is as always waiting for the fringe compilers to catch up.
Do you know if Peter actually uses VC++ to build his plugins, or was
he using that as a portability exercise? I spent a couple of hours
trying code in Visual Studio Express last night. It's like having my
teeth pulled.

Al
-- 
Alistair Riddoch
alriddoch@googlemail.com
http://alistairriddoch.org/
No, it's not mandatory, but it means next to nothing anyway. Erasure cleans it after compile, and it doesn't hurt as it is.
I experimented with fn map destructuring earlier in my clojure learning trajectory but was warned it could impose some overhead on function calls--should I be aware of any performance tradeoffs here?
Was adding this section in this commit a mistake?
в наследовании, насколько я помню, родительские классы отступают не на 2 таба, а так, что бы все они были почти в столбик.
What if i want both timers or none?
I'm not sure if parent version can be also inherited. It may be interesting.
The URL won't resolve that way, the test was wrongly tweaked to make it accept the wrong path.  (fix in progress)
Is there a way we can add this domain label to the `<g>` of the node it's labeling? That way it could float around seamlessly as the node's position changes, if the user is dragging it around...
uh k, yeah, thats wat i thought :)
Added this for future maintenance, in case I add any code that uses xpath before this and forget to clear it.
Convention is to have the `Actor self` parameter first, consider?
dont have them any more (reformating ffs \o/)
I tried adding all the styles and js libraries from openemr/interface/patient_file/rules/patient_data.php no luck.

I am at a road block on the patient select popup, I tried looking at the code in the interface/calendar/add_edit_event.php file as it is the closest to the same behaviour (a dialog from a dialog) and I still can't figure it out.
Also,
In the versions.php file, increment the $v_database by one.
`quiet = False` is already defined in `ServerAdapter`, you don't need to redefine it.
I think each one should has your own .rspec file(added on .gitignore) with his preferences!
Github is cool :)
:fist: pump!
Добавить в качестве callback событие self.evAfterHide
The view raises `ValueError: new type not compatible with array`. if the array size is not a multiple of the number of columns.

I'm not sure what shape  `astype ` produces

```
>>> np.random.rand(10).view([('var1', float),('var2', float),('var3', float)]).view(np.recarray)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: new type not compatible with array.
>>> np.random.rand(9).view([('var1', float),('var2', float),('var3', float)]).view(np.recarray)
rec.array([(0.15397837174315598, 0.21467949782036444, 0.016512595780767358),
       (0.015503599369291909, 0.11628127486111539, 0.26997851133311479),
       (0.61785843029323961, 0.60289240216246176, 0.51571919054757442)], 
      dtype=[('var1', '<f8'), ('var2', '<f8'), ('var3', '<f8')])
>>> np.random.rand(10*3).view([('var1', float),('var2', float),('var3', float)]).view(np.recarray)
rec.array([(0.4081253753282259, 0.035102342238546358, 0.32917336153368315),
       (0.62606548963735009, 0.82254839219445808, 0.13620959444280789),
       (0.31829809858622227, 0.44701135079854948, 0.71486637642908524),
       (0.36842888561315623, 0.79985475284331509, 0.053331058711121537),
       (0.75231868036239102, 0.34707893285415359, 0.76609368142257916),
       (0.3035818001799242, 0.81478455939725969, 0.15414633233954644),
       (0.55344597724915801, 0.08579456310198208, 0.04221145035749907),
       (0.77815842073057784, 0.25667010682073521, 0.044705375149677318),
       (0.23536685243605959, 0.29203742384868137, 0.85685969536488482),
       (0.78389988871810623, 0.8198833992097837, 0.23717752864730157)], 
      dtype=[('var1', '<f8'), ('var2', '<f8'), ('var3', '<f8')])
```
Same comments as edit.
I shouldn't matter but use >>> instead of >> for clarity
``` javascript
if (e.keyCode === 13) { 
    toggleFeatureExpansion(e);
}
```
^^ Credit Whore :)
Because the first pass was to replicate the existing functionality, not augment it. And none of the existing code in MT would use the delegates anyway, they call check IsDebugEnabled, etc. before formatting complex debug values.
UPDATE quest_template SET Flags=12, SpecialFlags=2, RequiredNpcOrGo1=0, RequiredNpcOrGo2=0, RequiredNpcOrGo3=0, RequiredNpcOrGoCount1=0, RequiredNpcOrGoCount2=0, RequiredNpcOrGoCount3=0 WHERE Id=24851; << this is hack.
vmaps 3: 569MB 8871 files 
vmaps 4: 560MB 9876 files
haha i get your point :) but still they wont remove it untill its 100% blizzlike so i guess you will have to w8 then :D
Please add a JavaDoc that briefly explains what this method returns.
ok, pero va con core incluido ;)
Sure. @josevalim @carlosantoniodasilva can you release it?
Thanks.  It isn't ready yet, but I will let you know when...

On Thu, Jan 26, 2012 at 1:49 PM, nickreich <
reply@reply.github.com
> wrote:

> Thanks for working on this, scottyaz. Let me know if you want me to pull
> your changes into the main trunk.  I've been looking for an excuse to build
> a new version of this and make some other small changes.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/scottyaz/coarseDataTools/commit/27c32ef19533f85abc30a0778020fe30b2ac5e3f#commitcomment-905271
>



-- 
andrew azman
+1.410.929.2482
Explique as razões.
Vou dar uma olhada. Não estou conseguindo pensar em um exemplo no kanban. Alguma ideia?
is get_current_screen() a part of the screen api?
Recent pipeline changes were to make the process "non-hacky" and to workaround google chrome's requestAnimationFrame timing issues.
use this method to get the application dojo.Evented instance. When the view class changed to object and remove scene type, this responsibility will move to load controller.
Assert ?
token_get_all() is slow on so many files and the results are harder to work with. preg_match() is much faster.
there is a probleme when i add ur code :
You have requested a non-existent service "solarium.client".
Who knew? :)
Damned if you do, damned if you don't. With other repositories people complained we left them in place.. Sorry I broke yours!
Ъб мать, 
function() {
			readViewedPosts();				//Log('readData');
			forAll(doPostFilters);			//Log('doPostFilters');
			if(Cfg.delhd == 1)
				{ forAll(mergeHidden);		}//Log('mergeHidden'); }
			if(Cfg.expimg != 0)
				{ forAll(eventPostImg);		}//Log('eventPostImg'); }
			if(Cfg.expost != 0 && !TNum)
				{ forAll(expandPost);		}//Log('expandPost'); }
			addLinkMP3();					//Log('addLinkMP3');
			addLinkTube();					//Log('addLinkTube');
			addLinkImg();					//Log('addLinkImg');
			saveHiddenPosts();				//Log('saveHiddenPosts');
		}
не вызываются! Вы работу с куками тестировали вообще?
Try not to mix C-style I/O (this function) and C++-style I/O (the next one).
Hello Quentin!
Catalyst::Test is used to get context object for making connection with DB by use schema generated by Test.pm
In your example they also have different parents, the difference is that you're talking about ```Theme.ActionBar.InternalBase``` instead of ```Theme.ActionBar```. But why do that? Why create ```Theme.ActionBar.InternalBase``` and then a "new theme" and inherit from that ```InternalBase```? What's the purpose of that?

My point is, the InternalBase is redundant. I can't see the benefit in that and that's what I'm trying to understand...
Thank you for your addition. I can imagine that there are people who want to see the `+filters` as separate e-mail addresses. I think this would be a great addition if it's supplied in a separate form that extends the `SignupForm`. This way people can choose to use this extra validation by using this form.
Yeah!
I removed some files references.
So, device will write after vendor... all libs will subscribed.
I will remove these lines soon!
The p tags are only important if you're actually looking at the formatted javadocs?  Are you doing that?  If not, and you're just looking at the comments in code, I think you're fine to drop the p's
It takes a lot more than a result limit to keep chuck norris out
Yup, that's a totally different error. See https://github.com/bterlson/redmine-status-updates/issues/7

Then you'll get two more errors once you fix that, which are totally unrelated to either of these issues. I fixed those in my `status-create-fix` branch.

FYI, I ended up getting all these issues fixed and got it working, but once I did and tried it out, I determined this plugin still isn't as good as the [redmine_user_status](https://github.com/bthesorceror/redmine_user_status) plugin. And the author of that one pulled in my pull requests right away, so I'm switching back to that one.
@emmanuelbernard Ah ok, I see what you mean. I tend to think that we should have approaches #2 and #3, though there also might arise some issues when mixing both approaches (e.g. if the validated method itself has a single parameter of type `Object[]`).
Why is def run() returning this?
That is pretty cool. I'll have review more.
No, last stable (v2.0.11)
1. В `ctx` по умолчанию у нас сейчас всё. Вот когда вычистим, тогда и.
2. Не `plan`, но `arch`.
Could we check if this is actually necessary?
Because i'm a goof and used the wrong one. Should that be Error or Warning?
Isn't this assert wrong? Shouldn't it be:

   assert 'Location: %s' % dist.location == lines[3]
I see what you mean about the efficiency of passing around the match result.  There's really no way to avoid the array creation in this context short of not using regex.  It seems like bit-wise operators could simplify this.  

How much of an influence does the underlying regex implementation have on the public api?  If an alternate approach can be used, would you still prefer to pass arrays around?
Fixed as john's comments
Thank you for renaming these files to reflect the name of the class!
Yeah, sorry the issue is with smart losing focus, so a couple lines above if it still thinks rs isn't ready (even though it is logged in, smart just lost focus), it shuts down. this will attempt to flash smart if it has been 1 minute which always fixes it when this happens.
I approve of this TODO.  If we're storing the name of the config in the container, we should be able to have tronfig send the name separate from the config chunk, which would remove the need to store the config_name in the config
members will never be null, that must be part of the MembershipChangeListener contract (i.e. tested elsewhere if needed)
in practice that would be strange, but what type to use then? does it matter? isn't scala.Iterable thread safe?
should I have to do .toIndexedSeq or something before doing ++
Evernote was unable to submit your note for the following reason:
The storage limit for this account has been reached.

Original message information:
From: Roland Kuhn <notifications@github.com>
Delivered to: thanhbv.fa84f@m.evernote.com
All recipients: akka/akka <akka@noreply.github.com>; giabao <giabao@sandinh.net>; 
Subject: Re: [akka] Fixed handling of unbound transport errors (#970)
Sent: Fri Dec 21 10:55:10 PST 2012

To prevent excessive emails, you may not receive another error reply
for the next 360 minutes.
Opened a week ago: http://www.assembla.com/spaces/akka/tickets/2322
you might want to extract this to a method used here and from preStart
was it intentional to use different delays?
Soooo this is UNGUARDED and shouldn't be called in more than one thread at one place?
Don't these messages have something in common?
Moved to Card model
Missed that, never mind. Looks good to me. We'll have to decide wether this is for 1.2.2 or for 1.3. Thanks!
This signature does not match the API
Nop! Use something like:
```
extent = AliasProperty(_get_extent, None, bind=('orientation', 'size'))
```

Same for all the others + add documentation
These kinds of diffs happen when the line ending in the file changes. IIRC, most, if not all file line endings in this project are unix type (LF). When making changes like these, make sure that the line endings do not change.

When we setup support for a new .net/mono version, we define the symbols for all .net/mono versions up to the latest version (examples can be [found here](https://github.com/nant/nant/blob/master/Makefile) starting at line 43).  So if the code block is already set for .NET 4.0 and still works for .NET 4.5, this line does not need updating.

Also, I don't see the "NET_4_5" symbol defined anywhere.  It should be defined in the Makefiles in the project (ie: Makefile & Makefile.nmake).
Might want to add a section about our consumer management as well.
Yes, you definitely did mention you weren't fully committed to this in the demo, I remembered that when I saw it. I also don't have a great idea for a name either. Maybe something like "unresolved" or "unconfirmed"? What's happening is that the agent isn't fully up to date on what the user wants it to be, which means the agent is in a _____ state. There has to be a word for that, but damned if I can think of it.
I went back and forth in my head as to whether or not I liked the auto publishes being tagged differently, but I think I do. Could we tag them twice, to mark as both a publish and an auto publish? I may be going overboard, but fundamentally it's still the same publish and if I wanted to see all publishes it should show up (again, assuming that sort of view would be valuable).
NVM, 6 is al RW, niet RO. my bad.
I don't think there is a generic way to make that happen. Fundamentally, there are cases where the Tuple sizes won't be known (ie when there is no Schema to work off of), and appending is the only way to do it (AFAIK, could be wrong?).
Actually I firstly wrote to return from within each case, but when I saw the original code it was written in such way (returning at th end), so I followed the way.  Okay, I’ll change this anyway. Thank you!
Hey, I'm a bit uncertain how this would work with the HTTP clients. The methods accepts an array, sets the `@urls`, and the returns a random item from the array, correct?

I think we need much more robust implementation, as outlined in #162.

What I like about the implementation is that everybody can simply stick this:

~~~ruby
module Tire
  class Configuration

    def self.url(*values)
      values.flatten!
      @urls = values.map{|value| value.to_s.gsub(%r|/*$|, '')} if values.any?
      urls.respond_to?(:sample) ? urls.sample : urls.choice
    end

  end
end
~~~

into their `lib` and they get the behaviour. I'm leaning towards closing this and focusing on the more robust implementation with round robin pooling, graceful handling of failed IPs, auto-discovery of cluster nodes a la the Perl client, etc. What do you think?
Remove this line please
`is_safe_preserved` is wrong English. I know you wanted to be in line with `is_safe` but it doesn't work here.
I don't really like that, those all are keywords; and I really don't like it when I see the next one (see below).  If you want more keywords sets, update the Scintilla lexer to have more -- or ask a Scintilla guy to do that -- but don't hack so hard.
semi-colon? are you for realz?
Okay--done in next commit.
What that?
Unindent a bit? Should be below `[`
Ups. Já si to prvně vyhledal a pak spolehlivě zapomněl.
Actually I've kept the ability to store page and resolve files at the root of  `MARKDOWN_DIR` so it shouldn't break.

I'll add more tests though.
Subbu, where else does this literal getting used? Can removing ^ break some other places? Nothing breaks in make test at least.
почему это все делается в MainWindow? нужно сделать отдельный класс
Is this actually needed? Since this is under the postgresql adapters test, won't it only be tested on postgresql?
Yes, in case someone tempers the cookie or, more important, you changed the secret because you updated a Rails version or the previous one was "stolen", you should be able to change the secret and we will simply discard the invalid cookies.
Only to simplify
Typo: `.` instead of `,` after `NEW_CONST`.
boolean --> bool

indepentent --> independent
These are the env vars set by heroku. An option would be to implement another redirection which sets `ENV['API_PORT']` to `ENV['PORT']`, but I don't think this is useful. The same goes for the redis and rabbitmq vars.
I am not sure if this is actually required any more.
Is there any reason this function has to be nested? This should generally only be done when it needs to access variables from outer scope (sometimes also called closure).
Another problem is that all constructors are inherited by principle:

```
def constructor(self, *args, **kwargs):
   # some manipulation with args
    if true:
        return inherited_class_obj
    return super(...)
```
or
```
def constructor(self, *args, **kwargs):
   # some manipulation with args
   return inherited_class_obj
```
thus, we can't correctly implement unimodular key argument check.
Should we move our discussion to the SymPy mailing list ?
If you mean Add.make_args((((1 + xin) ** exp).series())), it gives an error.
Actually, this returns an error for me when I try it

`AttributeError: Matrix has no attribute free_symbols.`
One way to do it:

```py
In [57]: ZZ.invert(5, 7)
Out[57]: 3

In [58]: ZZ.invert(5, 6)
Out[58]: 5
```

There ought to be a way to do it using `FF(7)`, but I couldn't figure it out if there is.
Why is this better than the previous? Sorry if it is obvious. By the way, it may be a good idea to add the explanation for this to the commit message (rebase -i probably?)
Alright, this is as far as I got today.
Won't `task.course` fail because we've just deleted the task? Anyway, we should be using `@course` instead.

```ruby
if task.destroy
  flash[:success] = "Task removed successfully" 
else
  flash[:error] = "Sorry, couldn't remove the task"
end
redirect_to @couse
```
`@deprecated since 1.1.7`.
I just hate next part of code. Why do you need this climbing and jumping with _code?
Good technique, will work both for windows and for linux.
Layout достаточно наследовать переменную и не надо писать путь он сначала ищет у себя потом в темах потом обычные вьюшки
I'll see about implementing this.
If someone goes through and checks all the boxes individually, this code won't check the toggle_checkbox right?  (but I think it should). 

Would the following be simpler and keep the toggle_checkbox in better sync?

    $('.pq_checkbox').live('click', function() {
      // Set the select all button to checked iff the number of checked boxes equals the number of entries in the list
      tog = $('.pq_checkbox:checked').length == <%= entries.size %>;
      $('#toggle_checkbox').attr('checked', tog);
    }

I didn't look at the rest of the code so I may have gotten the polarity wrong on ````tog````.
This seems to have some extra indentation.
Ok, maybe just a comment to the effect that the arbitrary selection of transitions is intended to be representative but not exhaustive - might help a maintainer if a refactoring later breaks something either covered or not covered by the test.
I'd refer to it as version 1 (i.e. `{v:0}` and then not use the json any more.
Should this use iterator() instead? Seems like it could be a large result set.
turn it into a function and pass the url and the message_text as parameters? the rest seems to be the same
Do browsers still not understand hover on elements?

If whatever browsers are supported by this pr do, add a class instead and specify styles in our css.
We use else if, not elseif.
Unique is the wikipedia definition. ;)
What about: "Gravatar is a service that provides you with a global avatar. For more info please visit www.gravatar.com."
Actually 18. I'll try to do what you suggested.
I'd suggest using s_ as prefix like we have on template vars
Nu mi-e clar dacă ai vrut să le aliniezi, dar vezi că nu a ieșit ok. :) Asigură-te că ai doar spații. Probabil e mai simplu să lași doar un spațiu după „:”.
Well, maybe. Maybe it should be transformed to a skeleton for makeing new tests.
Since the work of the loop is already done is the intent here really to break rather than continue? I know jslint often complains about 'continue'.
Opening curly-braces should be on a new line. This should be applies to all opening-braces in this file
Yep, using `--disable-kmods` doesn't generate build targets for the kernel modules, as we're doing this by ourselves later.
Ah, sorry. I mis-read it as the encounterType name
could increase readability by writing 
```DrugInventory created = service.saveDrugInventory(drugInventory);```
instead of instantiating the variable above and then saving a value to it below.
What if I have a masked array with no missing data? I'd still like to be able to check the monotonicity.
*rolls eyes*
Consider adding a case for 4 here, as mentioned in the code review of the kernel.
What is the best way to integrate a file like this into Theano's tests?
What does making these public do?
The index count node is the child of AbstractScanPlanNode that has the predicate by default. AbstractScanPlanNode maintains the predicate info and serialized it into EE.  It runs in catalog compile period, does the extra predicate information cause performance problem? Should I add a flag into AbstractScanPlanNode not to write predicate info down to JSON string?
Most of the changes here are to eliminate the redundant dependency on ee of things that already indirectly depend on it via voltdbipc.
Checking the build state of an up-to-date ee just seems to take forever, especially on my home machine, so I am trying to avoid doing that twice.

The reee target was thrown for  similar reasons, as a way to quickly just recompile ee without its dependencies -- slightly risky, slightly fragile, very obscure -- I hadn't planned on checking it in.  I just got tired of waiting for all the churn when a C++ compile failed and all I wanted to do is retry the compile with some patched source files.  I lost sight of this change within the other changes -- I don't know if there's a better way to do this -- or if there is some kind of fixable performance bug in how ee's dependencies get checked -- it's distractingly slow.
use .split() not .split(' ') -- then it works on any whitespace and any amount of it
Oh, sorry for that, we remove those random debugging prints
I'd lean toward yes.
I think the `IndexN` calculation could possibly be made more efficient
by making use of the techniques found in `riak_core_repair`.  It would
avoid a `div` and `rem` operation per key which could maybe make a
difference in the tight loop of vnode folds.  Anyways, I realize the
focus right now is to make it work first, worry about optimization
later.  If I have time I might try taking a closer look at this.
Why do expected string has a whitespace before "Hot Potatoes!!!"?  Only a string without a whitespace is inserted.
You are right. I will rename it.
Sure - I don't know why I did this - the compiler optimizes it out anyway - there wasn't really any reason for me to do this.  I'll change it back.
Fixing the tests is the correct thing to do. It sounds there is some other problem if unmanaged models try to create themselves...
IMHO nullable columns should be avoided where possible. I suggest defining this property as follows:

    hazard_curves_from_gmfs BOOLEAN NOT NULL default=false
Rather then asserting that we have found 1 or more errors we should probably assert the exact amount that we expect. This way we are asserting the exact way that we think the parser should be behaving.
The more pythonic way of doing this is to create a list of all the things, then use join

x = []
for conn in self.connections:
    x.append(conn)
return output+"".join(x)

The problem with doing it your way in Python is strings are immutable. You're creating a brand new string on each iteration of the loop. At this way, you're creating only 2 strings instead of N
That is a solution, but I have a feeling changing the behavior for contains for unquoted values could bring on many bug reports for when a var is concatenated with `:contains`. I'm doing some exploration.
Let's just use Sizzle instead of window.Sizzle.
Please use Oxford commas:

> ... an elegant, open source, and object oriented ...
You're right. Plpython has a lot of string utility, and easy to use. However, for the historical reason, we use the PL/PGSQL. We have some discussion on this issue in the meeting before.
We could add a YAML comment.
What is the name of this ".template" technique...?

```
            = inMetric(inMatrix.col(i), inVector).template getAs<double>();
```
I've surrounded everything by quotes to work around some weird issue where it started to interpret @ differently. For Windows we need the newer 0.6.7 instead of the older 0.6.5.
I really prefer variables being declared on top of scritps :)
Yes, I thought to that. I just wanted to have a first valid draft before doing that.
It can be writtten little bit simplier:

``` javascript
opts = _.defaults(opts || {}, {kind: 'generic'});
```

But then again, I'm not sure why we need explicit kind/type
Does this mean a router can start receiving messages before it's redis connection is available? How much of an issue is this?
Perhaps, `Remove cached RedditContentObjects by URL.`
This listing of branch "MASTER" is strange here. Are you sure?
Indeed. I think PEP 362 is aimed at this sort of thing - you'd use `**kwargs`, and construct a more meaningful function signature for introspection - but that's still a work in progress.
Make the right-hand side simply thing.display_score.
also, if you like this pull, I think this line should be changed to
```ruby
  new_file = File.join dir, rest.join('-')
```
I was going to (and still can) make a separate pull request for this, but it didn't make sense without this one.
getElementsAnnotatedWith() only returns the elements being handled in a particular round of processing. If we delay this until the very end (processingOver()), getElementsAnnotatedWith() will return an empty collection. Instead we're building the set from the first round so that it's complete by the final.
git-fail
This async chaining feels a little weird because, assuming `FILE_ADD_TO_WORKING_SET` works correctly, `openAndSelectDocument()` should be guaranteed to fall into the synchronous case (file is already current, we just need to force sending a notification).  Is there a clean way to factor out lines 123-127 so they can be invoked here without having to call the sometimes-async `openAndSelectDocument()`?
Minor nit: I'd like to see an empty line between this block and the next one for readability.
There are a couple other exported functions below that start with an underscore that should also be moved up here under this comment.
While I appreciate your enthusiasm for documenting code, but I think its a bit superfluous here. A lot of the lines are self-documenting, so documenting them isn't necessary.
Agree; I screwed up here. I felt that way after the first commit, but haven't bothered to double-back to fix it.

Since DygraphOptions class is private, then these are default by their nature of never having their object exposed is my opinion.
So, let's go over a case: Let's say someone had:

Y3: { axis : {} },
Y4: { axis : 'Y3' }

Now suddenly they move that to 

series : {
  Y3: { axis : {} },
  Y4: { axis : 'Y3' }
}

That's just wrong. Fortunately, when axisToIndex_ is called, it'll throw an exception. Meaning that they won't have a silent error.

I wonder if axisToIndex should have a special if typeof(axis) == 'object' to prevent that warning, strictly to prevent migration errors. Your thought?
I don't know if all() is always evaluated lazily (it might be)  but for performance safety's sake I think it'd be better to just us: User.objects.count()
wouldn't it make sense to leave the test so that it verifies that future versions of the paranamer don't break again?
The ruby convention for sample output is `# =>`

``` ruby
user_agent.browser
# => 'Chrome'
```
I don't see how this can be run automatic
Use reST markup for comments (and links).
There are a few of these you can probably get rid of.
Instead of using `git show`, you can use `git describe` which output looks like this: `v0.4-81-g24d7d1b`

It gives you 3 pieces of information: the tag that the commit is based off, how many commits have been made since that tag, and the commit sha.
Can you turn this into an actual `?` conditional statement? This looks a bit hacky.
Why not early return first?
Same as https://github.com/mozilla/butter/pull/1236/files#L0R18
It's most likely relative to `templates/` right now, which will be the default setup for those who don't care.
I was actually wondering about that return statement myself, but I just kept it in from the previous revision of the code. I'll remove it shortly.

As for the optional callback, you make a good point. I'll fix that as well.
you don't need this block
This comment could do with a tweak to remove reference to `@ignoreMouseup`.
Ok, this sounds like a good solution.

You can write and use generic steps, but you just have to make sure you test exactly what you specify in the step.  So 'there is a new app' should literally just verify that there's a new app that's been created since the scenario was started.  It doesn't really matter *how* that's verified unless you specify how it's verified in the step - s what you've done is a very reasonable solution because you've chosen to define both what is verified and how it is verified.
Right-o!  Project-wide, the following lines would need to be altered:

```bash
lib/helpers.bash
42:        if [ -h $BASH_IT/plugins/enabled/$(basename $f) ]; then
72:            if [ -h $BASH_IT/plugins/enabled/$plugin ]; then
117:        if [ -h $BASH_IT/plugins/enabled/$plugin ]; then

plugins/available/z.plugin.bash
18:if [ -h $BASH_IT/plugins/enabled/fasd.plugin.bash ]; then
```

If it is easier, I can make those edits.
Hmm not sure anymore:

```
144         while True:
145            if not poller.poll(timeout*1000) and tried >= tries:
146                raise SaltReqTimeoutError('Waited {0} seconds'.format(timeout))
147            else:
148                break
149            tried += 1
```

if 145 match the condition, it don't reach line 149, if it don't, so it match the else in 147, it don't reach 149 either.

in all conditions 149 is never reached
I like this change. It reminds me of an anecdote from a lisp-programmer who traced a parameter through a call-stack of 20 so routines, just to find out it was never used. The programmer proceeds to ask the author of the code, what it was for. Just to find it was put it in "just in case", as a reserve.
static -> statically
Tady a u následujícího bych použil "komentář", "poznámku" bych nechal výhradně pro poznámky pro překladatele.
I think there are few points here:

1. HTTP allows you to do a HEAD request, which essentially is a request for only metadata. So I think there is a good reason to have a special metadata request.

2. Should the metadata come in the form of a graph, or should it be hard coded?  I had thought of encoding some hard coded metadata in a [Meta trait](https://github.com/bblfish/banana-rdf/blob/1320b160f1dff9c7739e581aaf92b372590641d9/plantain/src/main/scala/plantain/PlantainStore.scala#L42), that could be added to any type of Resource. 
   * HTTP Metadata is very flexible, and most of the elements are usually optional, so all the fields would have to be `Option[T]` (indicating once again that we are in the open world space).  
   * Because of this flexibility it also makes sense to have the metadata as a graph, or even perhaps as name/value pairs as the subject is always the same
   * One could also think that this indicates that one should create something above a pointed graph, namely a MetaGraph, which contains metadata about a resource, and within it a graph. ( I had suggested something like that in Clerezza 1.5 years ago). Pretty much all data served by the server is of that type: what the server knows about something and the content of that thing. (This is equivalent to an Atom Entry if you want )

In any case in the test I have the meta data contain the access control rules, which I would myself argue is wrong. ( I was more trying out the Free monad there to see if and how it worked ). The meta graph/attribute-value-set should contain a pointer to the resource that is the acl graph. 

So yes, it is not completely right, but we need to decide on how to do these things that are needed.
but the main concern is about readability
Stranger huh
Bug introduced with Benefits + Credits fixed with 5e4dd0c.
That's a good technique! And I apply it in the last commits.
By the way, are this kind of utilities functions centralized managed in Gaia?
sorry, for consistency with your code this should be `var Term = function term_constructor(phrase, freq) {`
Not necessary, at least not yet. Places can be added and updated (when the title of a place changes), but visits by definition are historical as soon as they are created and therefore should never need changing.
You may want to check if ``allclose`` works with Numpy 1.4.1. If not, I usually use ``np.testing.assert_array_almost_equal_nulp``.
If you write these as e.g. `` `~astropy.cosmology.kpc_comoving_per_arcmin`  ``, it will only display the function name in the docs, but it will appear as a link to the appropriate function reference.
Yes, I don't consider Windows to be a proper OS ;)
No need to remove it everywhere, but also no need to use it in new places IMO.
fValid is always either true, or this loop breaks (condition for the for loop includes fValid) and goes directly to the boost check.
Swallowing exceptions is a bad practice, you should at least log an error.
``@static`` is useless IMO
Why implement the constructor if you're only calling parent?
what cb stands for? maybe make a comment?
This whole system around new vote events needs refactoring (per our conversation).
Can we come up with a better name?  I haven't read the function, but my first guess would be `start` and `end`?  Or `spos`, `epos`? Or..
There are a number of situations where it helps end notification cycles. `traits` proper lets you configure this on a per-trait basis, but this is not used especially often.
you are right, I'm going to fix it
should probably make DELEGATE_MAP = ImmutableMap.builder().putAll(USER_DELEGATE_MAP, ADMIN_DELEGATE_MAP).build, then use that here
You are right, I was a bit confused with the difference between the method(s) and dynamic properties, but I've read the docs more thoroughly.
Wouldn't it be better to actually feature detect mediaqueries. Like for example how they do it in modernizr https://github.com/Modernizr/Modernizr/blob/master/modernizr.js#L146
use ``url('privacy.policies.firefox')`` (I think)
__whimboo__ wrote:
> Hm, what is that for?

Just a rewrite of the result condition: it screams if somebody modifies the loop condition too lazily.
Rolled back, already.
Would like to test the "Good catch" above:
> ... That one could have been caused some of our nasty to reproduce waitFor bugs when the while loop didn't check the result before the next interval is fired.

But - as You can see - I'm not sure how.
should these two not be grouped in a transaction?
`processorId` should be `processor_id`
is it ops_managed or it_managed?
On Fri, Dec 21, 2012 at 04:35:58PM -0800, Rob Hudson wrote:
>Signals? I think this handles that...
>https://github.com/mozilla/zamboni/blob/master/apps/amo/models.py#L350

Signals, yes. When I used .update() rather than .save(), the 
_current_version field didn't get updated. Although, with the 
addon.update_version() line it probably doesn't matter.

-- 
Kris Maglione

Perspective is worth 80 IQ points.
	--Alan Kay
right too. Both tags are unclosed actually
I like this a lot but yes, the state field type all in red needs to be fixed to merge it in. Possibly a BOM or permissions issue.
Could you add these as PHPDoc comments please?
example: 

    /** @var array Caches modules that exist */
open a new ticket to track this one. DSP-1036 is created