labofsahil/github-event-dataset-2012 huggingface dataset:
Tic-Tac-Toe submission for Bobby R. Ward

A wxWidgets GUI is included for ease of use.
Testing requires nose and the lint script requires pep8 and pylint

Thanks for your time.
Add latin characters to regex for autolinking hashtags

This patch adds Latin characters needed for hashtags in several languages with large, active Twitter communities, notably Vietnamese, Romanian, Polish, Czech, Croatian, and Turkish.  The only tricky thing was how to handle the "IPA Extensions" Unicode block; I chose not to add all of the letter characters from this block since the majority are only used for linguistic transcription - a handful of characters, however, are used in standard written alphabets in Africa, e.g. ɓ, ɗ in Hausa (~40 million speakers), ɔ and ɛ Akan (national language in Ghana) and Lingala; 12 in all.
Mle be less harmful
Merged with beardedstudio and added select slider from filamentgroup

See http://www.filamentgroup.com/examples/slider_v2/index.html

Specs are included:

kmandrup:jquery-ui-rails-helpers (master) $ rspec
...................................

Finished in 0.03009 seconds
35 examples, 0 failures
Building block: List

This is a first version of the Lists building block for Gaia
Lettura xml

Implementata lettura XML quasi completamente
(i sensori vengono ancora ignorati)
Minimalizer

Standalone master now runs on FreeBSD.
Development

Signed-off-by: sesas <gabreyla@gmail.com>
less boilerplate

o hai. i made things less boilerplaty
Use qtile table structure
Shared elf

 - several ELF fixes for shared library support
 - add -shared for ELF platforms
LPS-22424 Permissions to edit/delete comments allows User to edit/delete comments of other users

User shouldn't edit/delete comments of other users even if he own edit discussion and update discussion permission. But User should edit/delete comments of himself. Admin should delete comments of other users , but he shouldn't edit comments of other users.
Watermarking article

I am going to merge this without review, since the script works and it doesn't touch any other area.
Instructions for Fedora in README.md updated

My package of your extension has just been pushed to the RPM Fusion stable repository and is now available to every user of one of the most widespread distributions!
Time to celebrate and to update the installation instructions! :-)
Fixed group spawning by keyboard..

Fixed group spawning by keyboard, when only one group present in window.
fix imager.example.js typo in filename

add letter a between x and m in sample config imager filename.
Pulled fresh from posabsolute repo and merged with pretty select libraries patch

Pulled directly from posabsolute repo + Merged with pretty select support with demo for chosen and selectbox libraries
Add address and telcoms to patient record

Added address and telecoms to record model. Added support for address and model to C32 and HTML exporters. Added tests.
Adds the ability to use the databaseChangeLog top-level element as

a property instead of just a method call.
AA Boobies

Hacked some AA capabilites, sorry for my bad python (not a python natvie speaker)
Next bb10 locale

Added API docs for BB 10 Localization.
jQuery Validation Double Click Issue

Fixes #684
Catalan Translation complete

Hi,

I've updated the Catalan translation and completed all the missing strings. 

Thanks!
new config-dir option.

I added this option so I could migrate my testing environment. Any comments?

> adding an option named config-dir. this adds the ability to attach a config directory to kohana before you run your migrations.
> 
> if you have directories set up for different environments like application/config/testing or application/config/staging, use --config-dir=testing to the cli command to run migrations on that config set.
Botoonly

Settings compatible with boto (environmental variables), rename, virtualenv support, python project directory structure, switched to tabs, syntax cleaned up for 80char width.
Removed compressed js files.

Replaced html5_init_uncompressed.js, html5_worker_uncompressed.js, and navmode_uncompressed.js with html5_init.js, html5_worker.js, and navmode.js respectively.
This removes the all compressed js files with uncompressed ones.
doc: removed wrong cut & paste
Some new icons for Fuzzytime

Hi,

here is where I got so far. These two icons are in the Inkscape-editable svg file. 
Each border is it's own layer, so it's easily editable.


I edited only the `chrome.manifest`, but not the `chrome.jar.manifest`.
Therefore some rework (for e.g. at buildscript) is needed after this commit.
fixed for commandbook 1.8, componentizing
Added TargetApi annotation
ReadMe change per your request, "Add hints about Mac compiler install and Ruby build"

Hi Ward - I added hints to the ReadMe file per our email exchange today.  Title: "Add hints about Mac compiler install and Ruby build"
Fixed composer require function
Provided a Project Explorer context menu option to run Dtangler for projects.

Now we can run Dtangler for projects are imported to the current workspace. Single jarfiles analysis will be added as option later.
Get latest from my dev branch.
add version.php

you have to bump version.php in order to properly trigger xmldb upgrades when manipulating settings (adding/removing, modifying...)
LPS-24699 Plugins Parallel deployment
LPS-24090

New and improved CDN configuration
Fix for #16

This is what i'm proposing we add to support an optional jsvc user arg
INT-2427 Fix Types of AMQP Outbound Endpoints

Logic for returning gateway Vs. channel-adapter was
reversed.

@olegz
Update shebangs

Updated shebangs in Python 2.x files to call Python 2.x, instead of relying on the system's default Python.
Remove Oracle JDBC driver

Show instead a readme containing where it can be downloaded and explaining the reason we can't provide it
Enable JSON::PP feature with JSON::XS backend

The escape_slash method is only supported by JSON::PP, the environment defaults to JSON::XS if installed.  This change overrides to enable JSON:PP.

Also, added a version in the DBI plugin to prevent an error when Dancer can not determine the version of the module during startup.
Update Menu/DefaultMenuBuilder.php ...

... to extend it easily, without redefining service or service's parameter.
Add Italian translation

I translate the string.
Updated the JS to work with my roundcube 0.8.1 installation

Added a conditional for compose-body/composebody because it didn't work for my installation of roundcube (0.8.1)
Added german "beigefügt" and "beiliegend" to the keywords
Scale up the QR code image to 4 pixels per block.

 Fix the issues #1.
I'm porting CM9 to a different device, I was looking around on the TDR github for more specific changes how TDR got HDMI working, besides knowing they got it from a device in Google's android ICS source. TDR has a a lot of commits to look through, any help on this matter to make this process go faster?
w000! Out of beta!
这种纯蛋疼的标准就是拿来被打破的么。
This script also replaces the sequence definition lines in the fasta files with 10-character codes from taxa.csv
s/obtainedusing/obtained using
Should you remove the cancel transaction, don't other forms use that?
It's optional if the options[:views] is passed in.
In the future, we could add support for something other than minifiers & compilers and I don't sure they'll be using `type` prop.
лучше не сокращать переменные, очень сильно падает читабельность кода
From the page you cited, "Even better would be if Node.js supported Asynchronous Module Definitions, because those are much better suited for web browsers – no need to extract the names of required modules from the source code to preload them."

It looks like lobrow is using the 'grep Function.toSource() for require' solution.  (FWIW, Function.toSource() doesn't work on a few oddball browsers, like the PS3 and Opera Mobile browsers.)  If you're going to do that, requirejs is probably a more mature solution: http://requirejs.org/docs/commonjs.html#packages

I took a look at the CommonJS shims used in Narcissus -- and they're not targetted at the browser.  The shim is https://github.com/mozilla/narcissus/blob/master/spidermonkey/init.js and it expects to have a synchronous 'read' method available to it, which presumably is supported by the non-browser spidermonkey shell.

I can look at rebasing the AMD changes on top of Narcissus HEAD, using the simplified wrapper in http://requirejs.org/docs/commonjs.html#manualconversion.  That might let us get the benefits of AMD for the browser while keeping the Narcissus patch as small as possible.
the makefile is attempting to install gc_cpp.h and gc_allocator.h twice. the following resolves - am__append_6 is only used once.

(note the tab-characters for indentation)

```
diff --git a/Makefile.in b/Makefile.in
index 09dea13..1f4356e 100644
--- a/Makefile.in
+++ b/Makefile.in
@@ -108,7 +108,6 @@ check_PROGRAMS = gctest$(EXEEXT) leaktest$(EXEEXT) middletest$(EXEEXT) \
 # C++ Interface
 # -------------
 @CPLUSPLUS_TRUE@am__append_5 = libgccpp.la
-@CPLUSPLUS_TRUE@am__append_6 = include/gc_cpp.h include/gc_allocator.h
 DIST_COMMON = $(am__configure_deps) $(am__pkginclude_HEADERS_DIST) \
 	$(dist_noinst_HEADERS) $(dist_noinst_SCRIPTS) \
 	$(dist_pkgdata_DATA) $(include_HEADERS) $(srcdir)/Makefile.am \
@@ -579,7 +578,7 @@ include_HEADERS = include/extra/gc.h include/extra/gc_cpp.h
 
 # installed headers
 #
-pkginclude_HEADERS = $(am__append_6) include/gc.h include/gc_typed.h \
+pkginclude_HEADERS = include/gc.h include/gc_typed.h \
 	include/gc_inline.h include/gc_mark.h include/gc_cpp.h \
 	include/weakpointer.h include/new_gc_alloc.h \
 	include/gc_allocator.h include/gc_backptr.h include/gc_gcj.h \
```
The exclamation mark before BIND is still a typo?
You should remove the trailing $, in order to allow lines like `include = something # comment`
+1 for bump
This is done by the new commit in https://github.com/m-stein/genode/tree/69-Generic_MMIO_framework
properties solda iyi bence. sag tarafa zoomla falan ilgili bir kac tool koyariz. ancak minimum size i biraz kulcultsen guzel olur 800x600 bizim ekranlar icin buyuk biraz.
I noticed my post had bad html ;)  I updated it so it might make more sense now :)  thanks!
Congratulations @saturnflyer :-)
This is technically a race condition; however, in the course of normal gameplay we will not update another avatar's SDL.
BTW, for non-core libraries, can't we just continue loading, rather than exiting?

-viral

On Jan 28, 2012, at 1:26 PM, JeffBezanson wrote:

> It builds now. Now I get a run time error:
> 
> ```
> /home/jeff/src/julia2/julia/lib/libarpack.so: undefined symbol: dzasum_
> ```
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/JuliaLang/julia/commit/2029caaeab85b17e31d0cea53738e47265c05d8b#commitcomment-910574
I think there's some value to being able to generate the absolute tersest human-readable ASCII representation of data. Saving a byte for every value adds up for large amounts of data. I'm going to leave it. Are you talking about merging all of these into a single function that contains the call to the grisu library? If so, that's a non-starter because I'm going to have to write a fair number of these to implement printf formatting and there's no way they're all going to be one huge function with a bazillion options. Once it all works, I can figure out how to avoid the `ASCIIString` allocation and substring copy. The most obvious thing would be for all of them to share a byte buffer and just have the core call return how many digits there are (which is what it actually returns). On the printing side, I need to expose a write method for printing part of a byte array without having to create a subarray copy.
dis tu pourrais exporter ta base entire stp?

Le 9 fvrier 2012 10:23, Ludovic Lefebvre <ludovic.lefebvre@ensc.fr> a
crit :

> oui github est trop fort ^^
>
>
> 2012/2/8 Olivier Hoarau <
> reply@reply.github.com
> >
>
>> Oh mais a a march !
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>>
>> https://github.com/Olwaro/Web-loto/commit/e4f7fd73858a8da822932954da3734281399e72b#commitcomment-946497
>>
>
>
>
> --
> Ludovic LEFEBVRE.
> Elve 1er anne ENSC.
>
>


-- 
Ludovic LEFEBVRE.
Elve 1er anne ENSC.
On icc and RS like in cata, id is on boss, not instance. And shared even on 10/25 both normals.
Android app touchquode pro. 9 bucks you get git push and code editor
Holder reapplyed only if holder absent on target. if holder found - used another case. continue not needed - this finish of cycle.
BIG THX to tibbi that was the mistake^^
I think the issue here is **why** `widget` is coming in as `undefined`.  Rather than uncommenting these lines, I think it's better to fix the situation you referenced in the *HOWEVER* comment.

Fine to check this in for now, but please open a bug and track this down for M5, if you can.
I feel like we are going in circles here. `extractUrlBase` was with dot before and for some reason it was changed:

https://github.com/mrdoob/three.js/commit/81d338e915d569271daaca40b156f46fda96acb5
на первом этапе считаем, что никакого БЭМ-проекта со структурой нет, просто сервим все статические файлы от корня

в последствии конфигурация проекта должна будет храниться в папках .bem/
OK, looks good
В планы -- здесь нужно будет нарисовать стрелку
I love spelling nazi's :) 

The ssh_options should be read from the definition , it should not be hardcoded in the test I guess. Let me check.
Deleted a semi-colon. What do you think?
gtk on windows would be tasty :P
Tu va vraiment coder sous eclipse ?
I agree semantically but the h1 style here is too big for the length of the URL that gets used...I could just restyle this h1 though
good catch, Thanks a lot.
Additionally we need to write a test for this output provider too.
so locale is guaranteed to contain only chars that are valid in an etag?  i.e. no quotes?
0_о как удалить эту правку... блин чёзана
"den" to "din"
There are cases according to the [spec](https://github.com/mustache/spec/blob/master/specs/interpolation.yml) (in the section on sections) and the [mustache(5) manual](http://mustache.github.com/mustache.5.html) (in the subsection on "non-false values") where a string can be a valid element of the context stack.  It can come up, for example, in cases like this:

    template = '{{#person}}{{.}}{{/person}}'
    context = {'person': 'John'}
    expected = u'John'

Yes, I think there should be a more general solution to the problem.  In the section on interpolation, the [spec](https://github.com/mustache/spec/blob/master/specs/interpolation.yml) makes a distinction between two cases: hashes and objects.  Because of the case you raised, we don't want to consider strings "objects" -- even though in Python, just about everything is, strictly speaking, an object.

I think a possible solution might be to draw the line at built-in types.  Anything that is an instance of a built-in type (an integer, string, list, tuple, etc.) could be considered, for Mustache purposes, not to be an object.  On the other hand, instances of user-defined classes, for example, would be considered objects.
I was unable to get the mysql gem installed using the mysql_config for MAMP. Ended up being the wrong version of mysql and I would have needed to compile a different version by hand with some specific options. I felt like I was just going deeper and deeper down a rabbit hole I had no interest in. Plus I've been wanting to learn more about Node.js so I thought, what the hell, let's do this in Node.
Possibly. Could you raise an issue for that, and I can look into it independently? It will differ depending on the OS.
debugging remnant?
that's true!
i'll redo it to make the server load the emotes, and permit only emote ids.
I think it would be much better to set the categories auto-on from PHP than javascript.
javascript should not be assumed to be available... so set them as checked before the server sends the content... not after it is rendered in the browser.

I will handle this when I write the code to persist the search
Obrigado Maury.
That's good to know about pageSize-0 :). Yes, the goal is that people can take code from the kitchen sink and modify it to their own needs...that might include pagination. Thanks...great to have it added to the repository.
Don't hard-code the number of jobs here. Please make it a constructor parameter of the estimator. Also have you tried to make some empirical timing evaluation? Does parallelization improves the runtime on medium to large datasets?
Bug fix. - ReleaseOutside
we shouldn't encourage this in written code.  convenience in live coding is one thing.  But, documentation and real code should have explicit imports.
opravil jsem (chtel jsem si zkusit editaci pres github :-P)
为啥呢？
Actually, AFAIK (from my work with file manipulation in FireFTP), it's impossible to do this kind of thing on a separate thread.  I've tried and it'll make the browser crash b/c it's not thread-safe.  I'll remove the callback and make it sync.
Yes, we should take care of those too, I simply forgot about them (I'm still not used to this Store/Query split).
https://github.com/modxcms-jp/evolution-jp/commit/cc85fab3f8f43e36417831d5528762ff7f12abeb
For example.
Good eye, thanks again.
An enum that only encodes two values called TRUE and FALSE isn't adding much. Perhaps call the enum WriteType and the values SYNC and ASYNC. That way you don't need to read any documentation to figure out that `write(data, WriteType.ASYNC)` is an asynchronous write.
Same refactoring issue
replace == NPC_GB_KORKRON_REAVERS)
wit  == NPC_GB_SKYBREAKER_MARINE)
I wasn't sure where to put this, but could you allow tp command to work with player nicknames?
lol, i forgot I wrote that. I really have no explanation; it felt like I made 50 commits on the most trivial script I may have _ever_ written.
add comment to say it's for all the tests
This still uses Haystack.
Yes, looks a sensible change.
`delayIn` and `delayOut` are not supposed to work with tipsyHoverCard, which is only *based* on tipsy (but does not aim to replicate all it's functionality, e.g. `trigger` is always "hover")...

If you want to make it work on the other hand, I would of course accept any sufficient pull request.
헉 복사하니까 괜찮아졌어.. 받침이 다 옆으로 떨어져 나왔었는데!!
Might be slightly better to use "protocol ID" as per spec.
you need a colon to do the magic.
Thanks zzzzzzzzz, I fixed this in the latest development version.
Florent
On 8 March 2012 20:24, Ross Patterson
<reply@reply.github.com>
wrote:
> @lrowe
>
>> The overlays open inside the toolbar iframe
>
> I actually kinda like the way that the iframe inserts it into a modal overlay in the parent frame because then things like TinyMCE can reflect the theme's styling regime.

The whole reason of the toolbar/cmsui project is to simplify theming
of Plone by isolating the editing and administration interface CSS and
JS from the content. This is achieved by loading all of the
admin/editing interface in a separate iframe, overlaid on top of the
content page.

For current AT/Dexterity rich text fields, TinyMCE has it's own iframe
for editing anyway (to which you can add a supplementary css file if
you so wish) - so there is no change there in that respect. Deco is in
place editing and so makes certain demands of your content theme for
it to be compatible, but even this is much less than the current
situation where you have to deal with a huge amount of Plone CSS/JS to
get a working theme for editing/admin.

Laurence
Unrelated to this commit.
raíz -> raiz
In that case I will modify the FullTermParser to ignore them.  Assuming this doesn't break anything, I will make the changes so that only short ids are returned from the hierarchy service and the resources are constructed afterwards.  Given this change to the FullTermParser, I should also be able to replace the TermParent service with the concept neighbourhood service as we have discussed.  I suspect these changes will take me at least the rest of the day, but they should wrap up issues 24, 25, 26.
There is no v1.0.1 tag in Kiwi repo
All best DB for wow-enulation community, have UPDATES and FULLS. It means, that your DB is temporary slag
@wrwrwr You're right it was failing for SetField, is `convert_value_for_db` something specific to mongodb-engine?
Another good catch!
Where can I find the binaries for the Triumph after getting the repo?
Why don't you use JQuery AJAX methods like $.post(...) or $.get(...). Using the library functions will allow you to do cross-platform compatible and simpler AJAX calls. If you don't want JQuery, then you will have to instantiate your HTTP request object as such:

```JavaScript
if (typeof XMLHttpRequest == "undefined")
  XMLHttpRequest = function () {
    try { return new ActiveXObject("Msxml2.XMLHTTP.6.0"); }
      catch (e) {}
    try { return new ActiveXObject("Msxml2.XMLHTTP.3.0"); }
      catch (e) {}
    try { return new ActiveXObject("Microsoft.XMLHTTP"); }
      catch (e) {}
    //Microsoft.XMLHTTP points to Msxml2.XMLHTTP and is redundant
    throw new Error("This browser does not support XMLHttpRequest.");
  };
```
Unless Field instance is cached, this doesn't make sense,
as Class::getField always returns new instance.

And in this case we don't cache field. ;-)
works for me for the limited cases I've tried. Did not have time to create an expansive test, but have more time now.

Can you help me by posting something which fails?
Right, considered that it might be better for users to see a short, memorable OG server url that they could possibly retype later on another device. or tell their friends. But will do bitly if you still prefer?
Well done Andy. You have indeed been impressive.
But we also need to check whether most regressor implementations don't do this automatically. I think it's the case for Lasso / ElasticNet and LARS.
Google Chrome's stutter once every second for about a few milliseconds should be dampened now. By making dummy canvas calls in EVERY setInterval, rather than on-demand, we can avoid Google Chrome's stutter bug!
oh sorry. I got confused because he has write access to the repo. But well, it is the same for me on KnpMenu :)
thanks jim!
This seems to fix the error I was getting as well. The demo shown here has an extra "pxToEm" function added to the Number prototype that is not included in the code in this repository: 
http://filamentgroup.com/examples/equalHeights/

Original error:
currentTallest.pxToEm is not a function
/includes/js/jQuery.equalHeights.js
Line 26
accidental double space
Why is this included in the pull request?
Главное, дабы несколько раз eventRefLink не вызывался.
Окау.
Желательно, дабы все объявления были в начале функций.
Вопрос религиозный, вощемта.
К тому жо, rNum уже объявлен выше.
Awesome thanks. I have never used javascript objects before (I rarely using javascript), so I didn't know how to get the number of keys.
ok, that's fine then, I just wanted to make sure since the commit messages on IRC came through without a name on them at all. It's probably just a quirk of github's web interface editing then. No worries, thanks for checking!
Thanks

Yeah

way cool

i just pushed the first push : 

https://github.com/kathyonu/sequencers_prelaunch

It has none of the tutorial additions in it yet.

Will

On Feb 22, 2012, at 2:34 PM, Daniel Kehoe wrote:

> Thanks for your contribution. Congratulations on your first pull request!
> 
> -- Daniel
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/kathyonu/rails-prelaunch-signup/commit/0a05e4a8f6a6bc3a214e3601e68d0b498f66ecd9#commitcomment-998387
Oh, I was getting an error and I solved it changed date format, idk
this was also my bug report.
notification auto brightness fixes.. power widget also
yes, head please
Hallo;)
Please say me Why I can't add your modified application in GalaxyTeam and use their for cooper?
@Mook, I'd appreciate your input here... We could possibly do this more elegantly and put a variable here, and have the build process somehow replace it with the found GST_PLUGIN_PATH from build.sh that gets exported to the build system? That'd be a lot more efficient for the startup process...

I realize this step isn't always necessary for gstreamer to work, but it's really a good precaution IMHO.
Wouldn't it be easier just to switch to `mysqli_fetch_assoc`?
Not sure we have to special case the one element list since the behavior is the same?
"Common widget properties"
"currentEvent" vs "selectedEvent" might be better naming.
Just for the namespace
Now AMQP, but not clear on what I would do.  Pull the settings out into an object?
Sure - it moved it into a file next to HTTPServer.scala. Uhm, how do I trigger compilation of the example? I tried action "compile" in sub-project "akka-docs", but that didn't seem to compile the example code.  Is it ok if it's just compiled to check it (everything in abstract classes), but does not really run and do something? Otherwise this will need test data, etc., making the example a lot more complex.
Because that's how it is in NettyRemoteTransport (it is a class as well)  
and I thought there is a reason for that which I just don't understand.  
Check NettyRemoteSupport.scala line 345. I am not saying it is right :) I  
will fix it anyway, but should I fix it in NettyRemoteTransport?
Avoid this branching here by concating the empty list in case it's not defined and a List with the one item if defined.
wow, more than 120 chars?
Hmmm, "sameAs" would sound and look a bit nicer, wdyt?
The child tells the parent that it has terminated after it has run its postStop. And the parent proceeds to run its own postStop after all children has told it that they have terminated.

Does that answer your question?
Can't see why. In normal mode it will take a while for the real FD to detect failure.
scala> Class.forName("[B")
    res3: java.lang.Class[_] = class [B
    
    scala> res3.isArray
    res4: Boolean = true
why not apply the same niceness as above?
fantastic.  thank you!
That's rewrite all strings
Remove the empty extra response_code entry.
If you do make this change, make sure the criteria is valid mongo syntax for specifying a regular expression.
we want to view the `CheckBoxGroup` as an atomic widget, so you should be able to just call:

    self.checkBoxGroup.on('change', /*...*/);

otherwise, in the case where someone called `myMultiSelect.checkBoxGroup.setValue([1, 2, 3])` three change events would be detected instead of just the one.

additionally, if all you're doing is forwarding the change event, you should call `evt.stopPropagation()` and `evt.preventDefault()`, otherwise if the parent of a `MultiSelect` widget registers for change events it will get two, one for the `CheckBoxGroup` and one for this as well.

these change events should definitely be unit tested as well :smirk:
maybe roles_set_card would be clearer name?
I had the idea of making it create!, but that seemed to conflict with the superclass.  Probably not a great idea.
Fix IntelliJ
duplicated code...the constructor should take a...(name: String, cb: (List[Timestamp], Option[Exception]) => Unit, nb: Int) => Unit in param and call it...the two values can be getNextTimestamp and getNextSequence
requestextradata missing
It's back from the dead! shall I stick it in?
Can we move `_upload_to_hdfs()` into `mrjob.fs.hadoop`? Maybe for later?
Why is this indentation changed?
See the array form of the output from the scan method for how this works.  We are just iterating through this array of matches, an array of arrays.
The problem with this API now is that we can't tell outside this context what went wrong.  I think it should just add errors to the account (User) object (acct var in this code).

It is nice and simple, send in an arg hash, get out nil or a user.  I guess if we put the errors on the object, we have to always return an object if we find it, and give the 'can't find' message on nil as we did before.
please change, variant with with _self seems to be shorter in this case
if there is only one cursor that is most of the time it is better to apply changes to it otherwise whole editor is being redrawn all the time
when there are many cursors applying changes to the whole cursor layer is better
All the C-style comments in this header have suffered from the reformatting I'm afraid.
How about a shorter version?

    compare(A = {_,_,_}, B = {_,_,_}) when A > B -> 1;
    compare(A = {_,_,_}, B = {_,_,_}) when A < B -> -1;
    compare({_,_,_}, {_,_,_}) -> 0.
... each catalog on the way ...
Are you sure that this piece of method is in sync with the code in master?
On iE7, I think though would break, since there's no XDomainRequest object.  While we don't support IE7, I'm not sure we should try to break something that works for it.  Why not just check for both window.XDomainRequest and inOrigin and !file on one line, then go back with the previous ActiveObject checks?
I see, thanks.

My Objective-C knowledge is very on the surface. I'm coming from a 5+ years background of C++ mostly, and I've picked up some Obj-C based on what compiled and worked. I'll have to dedicate some time learning it more properly one day.
FWIW in the future, you can still edit PR's after you've opened them.
Overrides of `resignFirstResponder` must invoke `super`. I don't know what functionality you think is being replicated, but you need to invoke it.
These methods are explicitly to save the order of notifications and delegate method calls. A simple `[super becomeFirstResponder]` wouldn't cut it when the delegate/notification order is all out of whack.
You should keep the preferred order used at line 56 here.
This state is used later in `node.cc` to check if handle is reffed.
Can you please negate condition to reduce amount of repetitive code?
Maybe use `== -1` here. I get too many questions from people asking what `~foo.indexOf(bar)` does. :-/
Isn't this another `this`?
A repeat counter should be put in here. If you'd rather keep play as a privileged function that's fine.
That link is not wget-able, but this one is: http://gdata-python-client.googlecode.com/files/gdata-2.0.17.tar.gz
Why is the logger required here?
The problem is that `value_for_collection(element, value_method)` calls `element.last` by default, which is incorrect when there's a third value being passed through
OK, I agree. There is an confirm  about? So i close this pull
I don't think we need to deprecate it here
I agree with you. I'll change this.
I think the blank line should be swapped - assertion should be right below assert_raises block).
There are blank lines dotted throughout this method, this just seemed readable to me. Are there guidelines on this type of formatting?
Makes sense, fixed differently.
`\u2603` would have been my choice, but otherwise this LGTM.
That is a bit melo-dramatic.    However, this is not general "control transfer" it's early exit from the function because of errors.   It's semantically contained and easily identified upon reviewing the code.     The point of any software engineering "ethics" should be to make code easier to write, read, and maintain.    You always have to balance many things in that process.    I find that zealous over-adherence to any one principle typically causes unintended effects that make things more difficult. 

In this particular case, do you have an alternative suggestion that does not require removing the benefit of the macro in encapsulating code that is identical?
Regression test for what bug/ticket?
As shown by the missing fails, split this method up and just use expected=IllegalArgumentException.class.  Even with the method boilerplate, should cut down the amount of code, and is much easier to read.
oops ignore sorry
Doesn't really matter (as you note, `db` and `da` don't change, so no more than one of `db == da - 1` and `db == da` will be True).  But I'll change it to make the code clearer.
And this seems appropriate only for Gaussian beams. Can it be used for the geometric rays?
We'd have to compute the hash somehow, e.g. with `hash(frozenset(dic.iteritems()))`, and either put the calculation in _hashable_content or create a subclass of dict.
On Sun, Mar 25, 2012 at 9:28 AM, Matthew Rocklin <
reply@reply.github.com
> wrote:

> >
> > -        if 'multiple' in flags:
> > -            del flags['multiple']
> > +        # roots doesn't like Floats, so replace them with Rationals
> > +        float = False
> > +        if any(v.has(Float) for v in self):
> > +            float=True
> > +            self = Matrix(self.rows, self.cols, [nsimplify(v,
> rational=True) for v in self])
>
> actually, ```self._new(rows, cols, data)``` is the preferred method for
> this. It's possible that the subclass of Matrix won't have the


I made this change. Also, it would be nice to know if some of the annoying
imaginary residuals appear less often for you.
Actually, in light of discovering that ``Rational``'s are only created for numerical division, you should test it explicitly using ``Rational`` and giving it different numbers, e.g. ``x**Rational(1/2)``
I've removed that commit.
But multiplicity > 1 is fine UNLESS isinstance(root,RootOf), isn't it?  Would

    assert not isinstance(root,RootOf) or multiplicity==1

put before the inner for-loop be better?  Or maybe the assert should be removed completely and tested for someplace else?
No, clone is need, because the CSort and CPagination will change rthe command.
edge_list = [link_to_pydot_edge(link, node_dict) for link in link_list]
Ya, I think it's much prettier than the .assertRaises(exception, method, args, kwargs) of unittest
Please use the old Hash syntax here. Janky should work on Ruby 1.8.7, although this may change in later versions.
Void function can't return a value (atleast not in msvc)
We need to think about refactoring this - a tiny malloc on every node overshadows any possible gains from parallelism
Write what are the options, i.e., 'true' or 'false' only
 or add more 't' 'f' 'y' 'n' 'yes' 'no' ......
In addition to checking for null strings, should we not also check for empty strings?
These methods tend to be very difficult to test, due to the large number of dependencies involved, since it is working against the differences store. I'd prefer to evolve the shouldTileEvents tests when the rest of the zoom level functionality is removed.
```
pulls = Retryable.do { api.pull_requests.pull_requests(api.user, api.repo) }
pulls.detect do |pr|
.....
end
nil
```
No space before comma.
And what exactly is the pro of doing the global one file/function earlier than in the place they are needed?
how about:
ROLE_DESCRIPTION_FORUM_STD_NO_OTHER
?
Please switch to // comments.
I'm not sure this comment is correct. Should probably be something like:
`// Whether or not the user is able to read all forums`
`{` on next line as per coding guidelines
Add an empty line after this one.
See line below.
`smiley_text ` is called every time before rendering.
Hm, we never disconnect the agent. Doesn't this mean process will just continue running?
Oh, nevermind on the first suggestion :facepalm:
Hm, this class should be inherited by X11Window and Win32Windows I think, so that it would be platform-independent.
how about replacing this by string.char?
Only this 2 assets? Better as option to be open for more asset types
Broken indentation is now visible here
please add a descriptive comment -- why do you have this method? what does it do?
add semi-colons at end of lines
'see above' isn't much use if I enter ```help(iris.load)```
Sure, I just copied that directly from ``iris.analysis.cartography.project``. There are a lot of lines that look like that in ``iris.analysis.cartography``... I guess I'll change this one, may as well improve as we go forward.
Are you sure...
```python
>>> import iris
>>> import iris.tests.stock
>>> cube = iris.tests.stock.simple_2d()
>>> cube.add_aux_coord(iris.coords.AuxCoord([ -7.5,   7.5,  22.5,  37], units='m', long_name='foo'), 1)
>>> print cube
thingness                           (bar: 3; foo: 4)
     Dimension coordinates:
          bar                           x       -
          foo                           -       x
     Auxiliary coordinates:
          foo                           -       x
```
Comment change:
 * "... N.B. This could easily ```be``` changed to ..."
这个属性名是不是太长了？
I've reverted this change
When creating asset, the first asset status will be added into the history
>>> class ArticleForm(ModelForm):
...     pub_date = DateField(label='Publication date')
...
...     class Meta:
...         model = Article
btw, it's not very great to use vars after callback. the idiomatic node way is to `(port, path, base, callback)`. I guess `base = ''` ain't needed at all because we're already defining it in config
Found it later on, please ignore.
Please do not use CamelCase for variable names.
Also fixed this in `f660886`.
It used to be called alteration.py, oddly enough, but the idea is that this will eventually replace creation.py so it doesn't really have to parallel it. I didn't want something too long, and I think what it does is immediately obvious once you open the file and see "SchemaEditor".
I don't understand what you're trying to say. Care to elaborate?
Oh, I thought the whole point of Pull Request #642 was that they all had `.getBoundingClientRect()`, should I do `$(container.firstChild).offset()` instead?
`jQuery.fn.width`
How does this event get triggered?
Local variable transval is now superfluous and can be removed.
Can we set choices=EXPECTATION_UNITS here and for repeat_unit?
Assuming we keep the methods we have now, and don't fold `create()` into `save()`: I'm not sure I understand what you think is the alternative here. Can you elaborate?
I think that would be possible, @jacknagel.
So I have to make xcode_version behave as before (where is goes into the "rescue" part and guesses "4.3" based on the clang version ... hmmm).
Should I use `ENV.j1` or `ENV.deparallelize` ?
We're reverting back to the *original* gollum method here
Should read 'high_pass'
Instead of this you can use an eunit fixutre with `{setup, ...}`.
Happy to make it ask if a value isn't provided?
you could use the `_check_roundtrip` function here
If the defaultLanguage is different from the language, we are basically walking the whole components hierarchy twice (inside loadFiles).
Plus, there's a small bug that shouldn't get passed along: in the second loadFiles call, i think ``configuration.defaultLanguage`` should pe passed.
Could you declare this as 
  SetBinding<T> extends Binding<Set<T>>
?

Then you can make contributors typesafe.
This brace looks oddly indented.
options.into
Should this have `copy` = `false`?
Don't need the `if`, just do `Batman.currentApp?.fire ...`
@smurthas just for require or all strings?
Should this line be "return $@"?
This all looks good.  But would you mind changing 'doc' to reference editor.document?  I should have caught that in the original code review... (and I sure hope it's a harmless change).
But is that the same both before and after this change?  If so then it should be safe to remove...
Can we add a comment here about how this is used? I assume this indicates that a german localization exists.
enabledProviders is a variable declared as an empty array, so it will always be truthy, I think this should check:

``
  if (enabledProviders.length === 0) {
``
Shouldn't the line above include some params passed to confidence_interval?
Missing an actual test here?
Any reasons for this being moved into IntegrationTests? I guess they should be in the report-specfic Dashboard/DSR/FCTIntegrationTests files ...
HasJob needs hgapp style layout, which places sass files under static, not at root.
Do we always get e-mail?
Why not `request.POST.get('next')`?
No, this should be an argument for `LimitedDateTimeField` at initalization. I do not understand the need of `FUTUREDATE` setting?
```javascript
draggableType = ( dropped.element || dropped ).getAttribute...
```
I don't think so, seeking seems to occur from setting the currentTime (either programmatically or through a manual seek) and playing handles when it is playing so I think we should be fine
Actually, it might be better to do this here instead of calling it in `default.js` since all track event editors should be breadcrumb-able
Pretty sure you stole this wrapper from Kate :P. Do you need a reference to `jQuery`?
it's actually a duplicate defintion of "duration", if you look lower in the file I just used that
If that's so, let's revert this.

On Sep 6, 2012, at 12:26 PM, José Valim <notifications@github.com> wrote:

> In lib/jbuilder.rb:
> 
> > @@ -131,7 +131,7 @@ def attributes!
> >    
> >    # Encodes the current builder as JSON.
> >    def target!
> > -    ActiveSupport::JSON.encode @attributes
> > +    MultiJson.encode @attributes
> 
> Doesn't ActiveSupport::JSON delegates to the MultiJson backend since Rails 3.2 (or even 3.1)?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
The same things appliy to the other controllers.
A comment describing what the preprocessing does would be great
Whatever filename is used to persist the static HTML into should be persisted in the DB, e.g. as an attribute of the Page model? Due to my lack of knowledge of how to do this I didn't attempt it. I would guess the steps would be something like,

1. Update the Page model with a migration script that adds the static_file_path column (null values should be acceptable).
2. With a Page object reference, assign a value to static_file_path and save to persist this attribute into the db.
3. From other filter methods, e.g. destroy_file_on_disk, read the static_file_path column to determine where the file is located that should be remove.
Thinking more closly on this, it should probably be something:

zfs create share/tests/hostA
zfs create share/tests/hostA/device_1a
zfs create share/tests/hostA/device_1b
zfs set shareiscsi=on share/tests/hostA       (this will be the target name - the 'container' for the lun's)
zfs set shareiscsi=on share/tests/hostA/device_1a
zfs set shareiscsi=on share/tests/hostA/device_1b

This _should_ give something like:

debianzfs:~# cat /proc/net/iet/volume
tid:1 name:iqn.2012-04.com.bayour:share.tests.hostA
        lun:2 state:0 iotype:fileio iomode:wt blocks:31457280 blocksize:512 path:/dev/zvol/share/tests/hostA/device_1a
        lun:1 state:0 iotype:fileio iomode:wt blocks:31457280 blocksize:512 path:/dev/zvol/share/tests/hostA/device_1b


How does this look? But that will require quite a lot of work. Currently you can't set 'shareiscsi' on a FS (which 'share/tests/hostA' would become).
Shouldn't this be `(:asind,:asin)` rather than `(:asin,:asin)`
Also  doing :condition=> .. is the old way..

Organization.find_by_label(..) is preferable
This is useless as it is already done by the parent class before calling ``write``
Why don't you want to use LockScreen.unlock()  here?
Triviality: Missing `object` base class.
when you say "doesn't work," you mean it throws some exception, or it just says it's True when it's not?  (and is this in one of the tests somewhere, or did you try it at the command line or something?)
I've taken your advice and make this async + idempotent.
Seems redundant and safe to remove, as Bind() is doing that check on ``inaddr_any`` / ``in6addr_any``, as you pointed out.
but between concat there should: `$plugin . '.' . $location`
explode always generates int-indexed arrays
> Perhaps you should just provide your name and not the email?

Or write the email statically like that or create a new email address just for Firebug?

Sebastian
Should be good enough for now...
This is *fantastic*, and the implementation looks great.  I just think it needs mentioning somewhere in the docs, though...  Otherwise users are never going to find out about these things.  It could be just a paragraph pointing out their existence, but I think we do need something in the docs about these guys.
Let's name this `init_notification_widgets` to be consistent with the rest of the JS code style.
any comments would be helpful
There are two simple rules about hashCode/equals implementation from java language specification.

1) Equality for objects <b>mean</b> that they should have equal hash codes
2) Inequality for objects <b>doesn't mean</b> that they should have unequal hash codes

You can read more about it in <a href=http://www.technofundo.com/tech/java/equalhash.html>this awesome article</a>.

So we can't add unequal tests here.
https://github.com/kaltura/mwEmbed/commit/df3a0a7886e0ad7617bb5d653085ae79edcf4719#L1L148
That looks about right. There is no avoiding having to give both the OID and the path since you need the path to check filter application rules.
It might be nice to call `git_tree_lookup` before calling `git_packbuilder_insert` here, so that if the tree lookup fails because the user accidentally passed in a non-tree, the non-tree won't have been inserted into the packbuilder.
Also, `no-reply` would be more appropriate
Not sure. @craigcook said it needed to be copied here instead of using the one in the base template for some reason.
Oops - thanks :-) Still not a fan of the need for "git add"...
It was a necessity. The only reason using cache worked for progress before was that it was accessed very frequently. With these changes, it'd only get accessed when a chunk started getting indexed and then at the end 20 minutes later. By then, it was out of cache and everything was busted.

Having said all that, this should never have been in cache--it should have been in redis. cache is a terrible place to put scoreboard things.
This will throw an unhandled error in case of bad data (non-existent tester id).
``($versions != null ? $versions : null)`` === ``$versions``

This is strictly equal.
s/1/keyerror_on_save_processed ?
That's only if you have multiple exception blocks.
This method does not need to be part of the public API, there is no need (when setting a config parameter) to be able to convert it, it's enough for us to be able to do that when we read it, and we can use a more specific type than Setting. Setting does thus not need to be generic either.

Validation however is sensible to have in this interface, since we might want to validate the parameter when it is being set.
It basically just that tools that are sinatra-focused use the sinatra one and tools that are rack-focused use this one.

The listing of tools that use the config option is given--did you notice that?  Is there a better way to explain that?
this raises an exception before my change, works just fine after the change
@stof yeah upgrade then login
one thing to note .. because of ``prependExtensionConfigs()`` it would be fairly easy for someone to decide if they for some reason do not want some extension to be able to prepend in the rare case when its not wanted. it would be harder if it would be a compiler pass unless we add some method that can filter the array of prepending extensions.
typo => than
Can you do the same for the other exceptions? Thanks.
It should not be renamed. You are setting the logger for this channel, not adding one (as you can only have 1 logger per channel)
It's something that was already in sympy-bot, so I didn't modify the logic with this pull request.

I think it's better rather than to try each second, because it would overwhelm the server. But maybe we can add some max time. But the way I think about it is that in 10 tries (that's a lot!), the time gets from 1 s to 17 minutes. In 15 tries, it gets to 9 h. And then, if the server is really still not working, you don't want to keep trying anyway, as it is faster to rerun the sympy-bot. So I think that this solution servers its purpose.
@tzcoolman, @arvestad, please, have a look at this one... why error_rate gets corrupted? For example (when you comment the error_rate=0.0005):

<code>ERROR RATE: -36893488147419103232.000000
</code>

Instead of a normal initialization with:

<code>ERROR RATE: 0.000500
</code>

Am I missing some global variable that messes up that value when build() is called and when it reaches that point within the function?

Any hints are welcome!
Gotcha, just checking.
Interesting, the variables behind the method call? I'll make the edit. I assumed indention would be relative to the method being called.
I don't know if I like breaking lines in html/erb. We should check other projects and see if they do this. Don't let that hold up a merge. I just wanted to make a note so we don't forget...