labofsahil/github-event-dataset-2012 huggingface dataset:
IPv6-support

* Added IPv6 support (incomplete)
* Only set last touple of IPv4 to null. This is fine even for german
concerns.
Allow for special "reserved" parameters.
V2.1.x debian build fix

Commit 2e71dd9 can be applied cleanly to v2.1.x branch to fix debian package build issue. Only the commit message is different, refers to libltdl fix in their respective branch.

Commit 5c82368d changed src/modules/rlm_sqlippool/Makefile.in to
fix libltdl issue. This commit adjust
debian/patches/rlm_sql.libs.diff to match that.
I was using the service to xml

When i posted xml i got back an xml representation of the params hash.

I instead felt that the body that was sent should just be sent back.

Commit message:
XML is sent it just echo's back the body that way you can prepare a
document have it post and then check to make sure you got the same
document back.
font dragr integration so you can drag and drop any font into another window for easy testing

I think this will be a great feature to have on your website it requires minor additions. Basically this pull request enables a user to drag any of the fonts into another browser window and have it render on the fly.

* Added some `data-font-weight` attributes to the anchor tags
* Super tiny js file (0.9kb) to enable Drag and drop into a website with font dragr bookmarklet loaded

You can test it out by going to the [bookmarklet test page](http://dev.fontdragr.com/bookmarklet/) or by adding the [bookmarklet](http://fontdragr.com/) to you bookmarks and loading it on any website and then open up this hosted version of [beautiful web type](http://dev.fontdragr.com/bookmarklet/demo/) and drag some fonts.

If you're still not convinced watch this screencast demonstrating it working! Oh and it works in IE6+ too!

http://www.screenr.com/pmVs
fix pitchfork addres match
Update omnigraffle doc
Fixed redirect ISLANDORA-566
Updates for 1.9

- Updated to the latest version of the code.
- Eliminated warnings.
- Changed some code formatting.
- Updated the api for messaging changes and to complete removal of unused methods.
- Added a new example.

Reviewer: @aogilvie
Add elasticsearch-test mention on client & integrations page

Hi,

Just an addition on the Client & Integrations page, as requested on the mailing list.

https://groups.google.com/forum/?hl=fr&fromgroups#!topic/elasticsearch/Y08T-dfov4U
Optimized Doctrine2 Annotations

This pull-request consists of some commits, enhancing the Doctrine2 Annotation exporter.

I've added an option (`useTabs`) to use tabs instead of spaces for indentation. When this option is set, the `indentation` is ignored.

Comments for tables and columns are written to the PHP-file.

Fields for relations are not written to the class any more. If you've got two classes, articles and tags, and the tags have a relation to the articles, I think it's better to use the `setArticle()` method than to have the `setArticleId` inside the Tag-entity.

Similar to this feature, I am no longer generating entities for many-to-many mapping tables. As far as I understand Doctrine2, you do not need an extra entity for these tables. Just use the add*-method of entities belonging to the m2m-relation.

The rest of the commits add features or options that you can set using the comments on tables or foreign keys in MySQL Workbench. These features are explained in the Readme.md. Most of them add options to the relation-Annotations like cascade, fetch and orphanRemoval.

But there is also a comment for tables to mark them, if they were incorrectly regarded as m2m-mapping tables. It is also possible to make relations uni-directional, if you want to keep your models cleaner.

I hope you like the changes :-)
check if inherit perms set before checking parent
Prevent PHP Fatal errors when the class is included multiple times

Hey,

Really dig your plugin, and use it in a couple of my own plugins. The problem arises when you enable two plugins with the same code, it starts throwing Fatal errors. So to avoid that we need to check if class exists or not.
Add curl timeout

For not blocking a script if Brightcove API is slow to respond
(Minor) Corrected title

Changed title to say "Frequently Asked Questions" instead of "Merchandise and Donations".
freeradius2 updates pkg v1.5.0

- Changed: handling of cert-manager selection in GUI
can't get the parent in a polymorphic_belongs_to (rebased)

See https://github.com/josevalim/inherited_resources/pull/158
normalize spaces in metric names

It can happen that names in the collectd world contain spaces,
which carbon doesn't handle well, replace them with underscores.
Move "Pair Device" into Firefox Sync settings, per design review.

Show one-account-supported alert when attempting to add more than one Firefox Sync account. Pair Device can be accessed from Firefox Sync settings in Sync account menu. Quick fixes for VKB problems. Added some strings (will file localization bug next).
SyncedSideBar plugin

Sublime Text 2 plugin to sync project sidebar (folder view) with currently active file.
Details: http://www.sublimetext.com/forum/viewtopic.php?f=2&t=4080
Add an :issues command that provides a link to the issues for the specified project.

I'm not sure how to test this... could we add some details to the README?
update uninstall so it does not remove the related collection of cmodel

ticket https://jira.duraspace.org/browse/ISLANDORA-651
new pull request to get latest changes

added in child objects (certifications, education, skills, etc) to more
closely reflect the current full profile.
Some changes I found while reading

- Typo fixes
- Minor code example errors
- Added section links to all #id'd section headers, also added a little style flourish so they only show on hover, just thought it looked nice :)
Now works in django 1.4 (admin site)

Now works in django 1.4. There was a error with the administration.

The server did not find the static file (before media file) like
/admin/css/base.css
Fixing bugs (loadErrorPage was not actually loading the page / loadRemot...

...ePage had the wrong assert)
Remove assumption that the key is always Base64 encoded
ENHANCEMENT: Workflow-definition buttons are shown disabled if current C...

...MS user does not have canEdit() perms on a workflow-definition (Issue #59)

MINOR: CSS tweaks to ensure button hover and focus states do not show when buttons are disabled
LPS-25401, Team Permission missing from action button
Created my own typo while fixing yours!
Fixed and added some links

I fixed some broken links in the impressum...
Pull Request - Member Add
Дз5

пока только 3 из 5 вариантов
Version used in the IRC Bot

Added basic type system, arithmetic, assignment, and functions.
clean-up-old-links (apply in master from clean-up-old-links)
Use a different ID in filter input in modal

I got some conflict when working with a js lib that need unique id, and btw, it's more "the way to do" (html related).
command definitions
Implement the basis of a Coq implementation.

This provides a very simple 101companies implementation, built off of the Haskell code more or less. Next up is to add some dependent type rules to the company making sure managers earn more than their employees by static verification.
fix SASL to work with multiple nodes

sorry about the number of commits, I'm really clumsy with github, git in general...
please let me know what you think...
Bug fix for drop text not updating on item creation

New entries created via the ajax stage were not having custom drop text applied until page reload. This commit adds the data-drop attribute for the item after successful creation.
Fix ZTS build

The current version can not be compiled with ZTS without this patch.
Added composer.json

Again, please be sure to add this bundle to packagist after merging this PR.
Release/tucker

- Updated Contribute and onemenu template.
- Changed projects list number on CMSTag Plugin
Added collective.gazette repo
Add the possibility to get new updates of bootstrap directly from the official bootstrap repo git

I've searched how to do this, and i've only clone as submodule the official repository and adapt the assets, I hope you will accept my modification and enjoy it :).
Add compatibility for rails 3.2

This should be just a matter of changing dependencies. All tests continue to pass.
Pull Req: Removed extra slash from file shortcuts

@haahmad @themainguru
Please confirm this fix is ok.
Implemented handling for Unknown Protocol. Invoke related

Unbound Invocation for links that have mailto: or tel: or something similar to that.

To test this you can put an anchor tag like shown below on your page.
`<a href="tel:647-274-5208">phone</a>`

You can try other links as well like mailto: etc...
If you try a link that is unknown, then nothing should happen.


@tohman review?
@ishneur test?
send a 'title' argument

This will be ignored by desktop for now, but will be used by android's notification when receiving a pushed tab.
Added oAuth support for Android: Fixes issue #56 (same error, different cause/bug)

Fixed the "access token must be used to query information about the current user" error for Android in Phongap1.4.1. The new SDK is expecting "authResponse" but getting "session" from the plugin. Not sure what else might need to be returned (i.e signedRequest???) but it seems to work for FB.api('/me') calls.
Removed session fixation

Hi,

Please check my commit.

It was creating a new session id on session_start().

I have passed a new variable for session id.
If it is there it will not create another session id.

Thanks & Regards 
Manu
Minor changes to definition of Anonymous Network

Changed capitalization and wording just a little.
Feature docedit

FIN SKIT
rowend/8-admin-feed

Translate admin feed folder
Roman Numerals kata
Remove mention all from integration notification
Added callbacks for URL changes and for when the browser is closed
Update Camera/Files/res/values-ru/strings.xml
Added documentation for option graph-xaxis-labels-enabled

See PR #15
Blackboard communication

Adds the blackboard clients for c++ and java. Also includes java in the build system. Libjson added, doesn't need review.
resize andromadus icon to be aligned with others
Fix crash on open with some keyboards (e.g. Swiftkey 3).

cherry-picked from http://review.cyanogenmod.com/#/c/21483/
Fix for extra char at end of line

The problem occurs when two combinations are used in same String.  no more extra (Chinese  ) at end of line.
Awesometastic, published: http://search.npmjs.org/#/bean
works? :O
Doh!!! Sorry guys - first push here - forgot to rebase.
I do. Just saying.

I can't edit it in Simba, remember? Or else git thinks I changed
everything.

On Sun, Jan 8, 2012 at 7:08 AM, Merlijn Wajer <
reply@reply.github.com
> wrote:

> Nava2, you need the Delphi syntax mode. And you should always check if it
> compiles...
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/SRL/SRL-5/commit/5e1a48e2e0e357e84c913b784a138cef863fc4a9#commitcomment-847422
>
1. your may try check procEx = PROC_EX_DIRECT_DAMAGE instead of procFlag = PROC_FLAG_ANY_DAMAGE
2. i'm planning rewrite procsystem, and send in proc methods DamageInfo structure instead of uint32 damage. but have very small time...
Nice de-coupling of the different components of the page. :)
。你喜欢打两个空格？
  
 
------------------ 原始邮件 ------------------
发件人: "jybox"; 
发送时间: 2012年3月2日(星期五) 晚上11:42
收件人: "whtsky"; 
主题: Re: [PBB] 支持自然换行语法 (f97f33a)

 
表示不怎么喜欢这样......

--- 
Reply to this email directly or view it on GitHub:
https://github.com/whtsky/PBB/commit/f97f33a8a849ead683c4d293355d254f261801db#commitcomment-1035948
should this maybe be specific (or somehow tied to) server instances? so that on close the cyclic timeout stops
Since the constants are now part of the main class (which they probably should have been from the beginning) we can probably drop the pseudo namespace (ie, SP_PRECOMPUTE instead of STASH_SP_PRECOMPUTE). These constants aren't just used internally, but are passed to the stash "get" command to tell it how to deal with cache misses.
Please change in alfred/meta/model.js  lines 158 - 171 
from 

```javascript
Model.prototype._delete = function(id, callback) {
  this.collection.destroy(id, function(err) {
    if (err) { self._error(err); return; }
    callback();
  });
};

Model.prototype.delete = function(id, callback) {
  var self = this;
  this.promises.done(function() {
    self.delete(id, callback);
  });
};
```
 to 


```javascript
Model.prototype._delete = function(id, callback) {
  this.collection.destroy(id, function(err) {
    if (err) { self._error(err); return; }
    callback();
  });
};

Model.prototype.delete = function(id, callback) {
  var self = this;
  this.promises.done(function() {
    self._delete(id, callback);
  });
};
```
 Underscore for _delete should be added at 168-169 line.
Shouldn't another config variable be used to silence the sprocket warning ? The current naming & behaviour is a bit confusing.

```ruby
config.assets.logger = nil  # currently logs using Rails logger
config.assets.logger = false # currently silences log output
```

These two options should ideally behave the same? Also I expect *config.assets.logger* to be used to define a Logger and not influence 
whether the logging will happen or not.
Cool, I think it's better to leave the default conflict commit message, where you see which files were conflicted.
A扁：已經退流行，不是常用詞
加權指數：我想通了（喂
的話：常用語助詞
@luislavena I had to add this if and the ENV['CROSS'] above to make rake not output a bunch of warnings on every run. If there is a proper way, please let me know how to configure it without having all the warnings.
This line confuses me, it looks like "type" can either be a scalar or an array.    I thought to rename it to "types" and have it always be an array.
pls fugbix pyto: Coipyright -> Copyright
this was regrettable.
You may be right, but it seems hard to guarantee an exact value is hit without creating stuttering. We'll see if this is a problem with real animations I guess.
I don't think so; technically it's a misuse of CFLAGS and CXXFLAGS. The build should respect CPPFLAGS correctly.
yeah ignore this.
I think it can happen in the error case, where the form inits with existing previews. Not sure though
thx for answer, i hate small diffs xD
Like above, maybe movn should be changed to movn_ to mark that calling it directly may be a bad idea. Same for every similar instruction.
Github is becoming really interesting place! :shipit:
Use `this.subscribe(topic, function)` so that the subscriptions are cleaned up on widget destruction.
Hmm... I have to think long and hard about whether or not to merge this in.

On one hand, I really like the cleanliness of being able to do "case String" and "case Collection".

On the other hand, I am not sure at all about pulling any methods out of the "evaluate" method, because it obscures the recursive nature of that method. I did it for runProcedure(x,y), but under heavy doubts.

Any comments on these thoughts?
This ```err``` variable isn't used anywhere. Should it be logged?
Sql tables need to be placed here:
sql/database.sql (place it within the tables in alphabetical order)
sql/4_1_0-to-4_1_1_upgrade.sql (see the #IfNotTable product_warehouse for example of how to add a table; note these are functions and can see top for their descriptions)

Above will make it much easier to review/test/commit.

Also, do not set a charset (this is left up to the user) and use the MyISAM engine (at some point, plan to upgrade all database to InnoDB, but haven't done it yet).
```php
cot_inputbox('text', 'rstructuretplquick['.$structure_id.']', $row['structure_tpl'])
```

I'm afraid this will block other TPLMODE options, because if rstructuretplquick isn't empty, it will be of higher priority.
Clever. Took me a read-through to get it, but quite clever.
Correction: the problem is in

* src/config.cpp

not

* src/phantom.cpp
you could use `eq_`
Could you explain a bit more what it does?
Can `hex(i,n)` and `show_trailing_hex(i,n)` be merged somehow?
or why doesn't Workbench.unload just call Runtime.unload?  I think this is what I had done for run()
я подумал на досуге и решил что был не прав, попросив добавить этот файл в гитигнор.
Причина: ведь если этого файла не будет в приложении то не понятно на каком руби его запускать.
Откати пожалуйста это изменение :)
соблюдай отступы, чтобы код был читабельней
Where is the patch from? Can it be sent upstream?
// alternative model name to be used in forms
yo tb lo test pero no funciona falta el sql
@wagenet I guess this is less expensive as long as you don't need to test DOM interaction? There are probably a bunch of tests that don't really need to append and could use this approach instead. If you set foo to something else, does it update the element even though it's not in the DOM?
Should be `Piecewise((Piecewise((x,x<0)),x<0))` (the inner call should be a tuple).`
Yes, this is cleaner than inheriting from list.
I think Fred's question referred to test_linear_cg-- since there was a broken import in linear_cg.py, the buildbot should've said something was wrong with pylearn2. Maybe it did and we just didn't notice?
Any reason dstFormat isn't a const reference?
реализацию лучше утащить в .cpp-файл, даже если она пустая
Why is the org.hibernate:3 dependency not optional anymore?  We don't create that module in the AS build, so this seems wrong.
Not sure if your serious or not :P
fixed the flame
Do we need all that commented out code, starting at [L139](https://github.com/acarrillo/Cribble/blob/master/src/graphics.c#L139)?  Remember that, since this is all under supreme version control, it is okay to delete code, since we can always get it back later.  It looks like that code is for the color mixer, which we are no longer using.  But if we ever do want it back, we can always revert or patch that old code back in magically.  Such is the magic of Git version control!
New version of this patch sent as pull request #189.
It is so cool that it had to move into its own library :)
Also added partials to data object
This is a cucumber-specific file, right? I think the file name should reflect that (and it currently doesn't).
oh btw, the move to make the socket param public isn't related to zmq-3.1.1.  Instead, I had an issue where i wanted to tell the ZMQSocket something directly, and couldn't do so because it was private.  I don't feel strongly on this item, so feel free to revert it.
@jasoncale I'd appreciate it if you had a quick check to see if this looks right.
I usually avoid checking in derived files by using

    git add *.tex

rather than

    git add *
Good idea. I usually do 
'''make realclean'''
'''git add .'''

I clearly keep forgetting the first step/
See PageService.
thanks, cleaned up in d247120b3af0bdf914baa2751d1887dff0a57370
I know there aren't any tests for this middleware yet, but could be cool by starting adding some at least for this new feature.
"list of calendar-homes for each user" sounds great
sorry, did not want to criticize you. before it was broken and you fixed it, so definitively an improvement. what i meant was to not have the instanceof check here but tell the properties inside the node. i did this in ed15e33
Hi Rob,

thank you for the report and the proposed patch.  I stared at SPLIT-STRING for a moment to determine whether it can be fixed so that we don't need a special-purpose function, but then decided that the easiest way would be to remove SPLIT-STRING and use CL-PPCRE:SPLIT instead, which has a LIMIT keyword argument that can be used to fix the problem, too.  I pushed that change, if you could try it, it'd be great.

Thanks again!
Hans
looks good! thanks!
Any key? Suena interesante, sobretodo para pantallazos y cosas por el estilo. Por cierto, sobre la nomenclatura... yo lo haría al estilo bool isAnyKeyUp?
There's a git conflict in here. Please fix this.
just `$rootScope.$evalAsync(callback)` should be fine...
Убрать генерацию объекта JQuery из объекта JQuery
*throws the cat a ball of yarn*
Yeah sorry, this is corrected line a later commit, the "unsupported tags" commit: https://github.com/gotosleep/DTCoreText/commit/1294b6ea0ae0903023dc0eeaa01b2ff1fac86afa
Yes, I always use collectiveidea. As far as I understand this is the official repo now and tobi is no longer maintaining his. Confusing though I know.
what is array(5) doing here???
Shouldn't we be able to add two media's with the same url?
А ну его к чёрту... >_<
во, нашел
https://addons.opera.com/ru/addons/extensions/details/noads-advanced/1.3.4/?display=en&reports#r23815
Sieht mir nicht nach newbie repack aus... aber ist jeutie so gut?
Damn, I fumbled my git commits...
It may be, but this is not an issue from the script point of view. :)
The missing spells issue should be solved on the core side anyway. The important thing is that when we have this feature in core (or as a custom patch) the script will work as intended without any changes.
I don't really know if this correct or not, but I guess it looks okay.
I usually justify dependencies with: "If I copy just these two functions, and then the next guy copies just those two functions, and the guy after just the next two, eventually we have the whole library copied in the project"

That said, this is not my project, so I will go do it your way.  Updated patch in a bit. :-)
Oh my bad well I didn't read through everything going on here so that's what I get for assuming.  Regardless I was having an issue with the theme I was using that was related to this change, and implementing the fix you had done for the other themes fixed my theme as well.
Its a temporary change. Logging in the web apps is not working correctly. We need to fix the logging in the webapps and then will replace all the system.out.println to use proper logging
I'm on the fence here - RT.list and RT.map are very clear and it seems like reusing the names does no harm.
Ah, hang on, in both cases, attr is a link counted reference to root->queryPropTree("Attr") (see loadAttr())
So it although it does attr.clear() - it will still commit the changes to root, via the changeMode in unlockProperties

confusing..
great work! does this mean updates from 2.7 and 2.8 will be included in future aswell? would be awesome!! :D
Частично это (хотя когда предложит ли Опера когда-нибудь посайтовые фильтры это ещё большой вопрос), но более серьёзной причиной была необходимость перелопачивать код и перепроверять все сохранения-загрузки.
Could not find nor verify:
CMSG_AUTOEQUIP_ITEM_SLOT
CMSG_CHANNEL_KICK
CMSG_CHANNEL_MODERATOR
CMSG_CHANNEL_MUTE
CMSG_CHANNEL_SET_OWNER
CMSG_CHANNEL_UNBAN
CMSG_CHANNEL_UNMODERATOR
CMSG_CHANNEL_UNMUTE
CMSG_DECLINE_CHANNEL_INVITE
You forgot a "}" below ;)
There is a mistake, see on https://github.com/TrinityCore/TrinityCore/commit/9d185c5d6657c74b24041356916d095ddb6f0d73#commitcomment-933161
subtype requires use Moose::Util::TypeConstraints;
Looks like a typo :)
But this then won't work with the current 1.4.1
This causes an exception, which triggers HTML mode, and mistakenly interprets the XML file as HTML.
The exception happens in seeXMLElement
    logging.debug("ADDTAB " + match + " " + urllink)
TypeError: cannot concatenate 'str' and '_sre.SRE_Match' objects
parents might also need to get merged into the original resource.
Does this solve the IE9 width problem? The show() was before the calculation because I assumed I had to make it visible befor width could be calculated. That used to be a problem in certain (older) browsers.
I'm done for the night, but if you want to change it to a set please feel free.
Second command line looks correct for me. `-subfont` is not needed in `-ass` mode. And font is actually the same `/System/Library/Frameworks/JavaVM.framework/Versions/1.5.0/Home/lib/fonts/LucidaSansRegular.ttf` in both cases.

I think the problem may be with font name. SSA styles use font titles like _Arial_ not exact file paths. On linux systems we can grab list of registered fonts  with `fc-list` command, but I don't know if it works on Mac. Try to enter font title like _Arial_ in GUI and also enable fontconfig to see if it makes any difference.
そんな罠があったとはｗｗｗｗｗｗ
まじお恥ずかしいところを見せてしまって、、、なんというかもうアレですね。
All in all I dont think you should have committed this code, search is no longer working. I thought we said I was going to review this before you committed.
The strange this is that the error only happened for @clintongormley on a "second" run. But, yea, I can see that there is a mistmatch between the code that writes the pid from teh actual process, and the one in the script. I will fix the code, no reason for the `\n`.
Homework 1 is all set.  1pt
Ok, I imported them into the database. I'll switch to the user_data and documents databases soon so you'll probably have to re-register.
hehe it was weird, a page so long :)
Hey roman, u didnt get back to me yet, about Galaxy Tab 7 GSM support (pretty much similar to vzwtab, just ril stuff differs - n the kernel)
Hi Tolgor,

The 4.5.1 requirement is just for building - it's not a framework requirement. There are various SDK bugs with regards to asdoc building in 4.0.
The behavior for me is still the same. Taking out the "hacky line" does not change anything. I am forced to use "@import active_scaffold" instead of "*= require active_scaffold " and rename my manifest from scaffolds.css to scaffolds.css.scss. I did not think of this before, but it seems to make it work. But I think this will require everyone to use Sass, because if you don't use Sass the active_scaffold_colors file will not be a part of the asset pipeline.
The description could use more effort.  For example, explaining what kind of info it actually gathers.
@wallern:
I try to search numbers (ex: 12) with this analyzer, it works for me.
Can you give me an example to reproduce this issue?
1. Посмотри на Data.List.splitAt.
2. take i xs накапливается в виде thunk'ов.
3. Зачем делать split одного списка, если можно наделать генераторов по числу потоков?
pourquoi get_competence2 ?
its flaky and inconsistant

On 1/20/2012 5:03 PM, James Adam wrote:
> If I'm correct, printing the last byte here (10) will insert a newline. Is that intentional?
>
> In my own exploration, even removing this byte (so this becomes `writeBytes(27, 45, 0)`) will result in subsequent text being printed on a new line. Any ideas?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/adafruit/Thermal-Printer-Library/commit/55a6db28518398ae06cfd78f2d4b2377e0c685e9#commitcomment-887460
As this is for Spring Data Repositories, why the reference to 'service', in particular in the usage {"link": {"rel": "service",...

Shouldn't this be prefaced by saying, assume PersonRepository, AccountRepository Spring Data Repositories exist.  Then a request without an accept header to the root will return
<pre>
HTTP/1.1 200 OK
Date: ...
Server: Spring Data Services Web Exporter/1.0.0.BUILD-SNAPSHOT

http://localhost:8080/baseurl/person
http://localhost:8080/baseurl/account
</pre>

and with an Accept Header something other than "rel:" "service" would be used...

At a high level i think the mapping looks good but we need to drill in on details.  The discussions I remember floating around were

1. A get on /baseurl/person/1 would return the non-collection elements marshalled into JSON but any relationships would be represented as links in the body (not header - also not a fan of putting it in the header).  Basically a discussion on what our 'default' representation should be with the assumption that the primary consumer of the representation is the browser via a javascript library like jquery.

2. A put on /baseurl/person/1 would not allow only the update of the entity corresponding to the ID=1.  If the JSON body had ID=2 that would be an error and the update would fail.  

Jon, can you add a separate section for action under /baseurl/domain/id.  for example, what are required formats, what error codes can be returned for each request.
I'm guessing it's for any plugins that will be an "addon" to TSN.
Ignore cache uploads please
Do you mean that the code itself is wrong for calculating age?  It looks like that way to me... which makes this test worthwhile!
What file does this try to open? What does the 'w' do?
set_tunnel was added in 2.7. In 2.6.6, it's called _set_tunnel and isn't part of the public API, but may still be usable. In 2.6.1 it doesn't exist at all. This could be handled like this:

        if hasattr(self, "set_tunnel"):
            self.set_tunnel(real_host, real_port)
        elif hasattr(self, "_set_tunnel"):
            self._set_tunnel(real_host, real_port)
Gah, didn't notice that :)
Why is thin needed at runtime?
Is the function getMatEntry() necessary? When is it used? If not needed, I'd like to remove this function because Lis doesn't provide this functionality.
I'd like to understand it more, can you add some comments to it next time you're in this code, if you can even remember what it's doing :P
Can it reference to BCommonProperties.icon?
Please fix comment alignment.
No need to redo the reduceLeft for every r, right?

    for(c <- Option(config.reduceLeft(_ withFallback _)); r <- role) _nodeConf += r -> c
please don’t use default arguments for internal convenience: they are a PITA when refactoring (due to missing errors), and thus should only be used in stable user interfaces when they make the API _that_ much more slick.
The logger is only created once, right?
Is there an elegant solution for this problem or we should drop it on the floor like we do now? Poor java guys...
but this class already adds children using InternalActorRef assumption, so it is very good to blow up if this fails
asInstanceOf ?
Nice catch dawg
Semantic change. This will allow non escaped spaces in the definition. I'm not sure if lex/flex allows that. One problem I can think of is a dangling space at the end of a definition. That may produce some headaches when debugging as it is kinda invisible in some editors.

Or please let me know the reason for this change (the '\r' part is OK).
Move to pack
indenting is messed up here as well
vectors.
I'm not sure this will work.  the two are different in that one has filters.  depends on rails implementation.
I think so. getTimeline orders by timestamp asc.
Don't like this class having a lot of stuff duplicate from static. Is it for app-cluster4 to merge the 2 inside one mega cluster test driver? ;)
I missed it was commented out.
we can just test a few simple contracts:
- set(), get(), isNull() are consistent
- append(), get(), size() are consistent
- tuple generated from schema works with the corresponding set()
- equals()
- ...
some negative testing:
- type validation
...
Yep realise that now. You could always turn that into a pane.method so it only has to be declared once.
there's a dot and a comma here that aren't right.
Yeah, Timeout.timeout is bad (see Charles Nutter's [explanation](http://blog.headius.com/2008/02/rubys-threadraise-threadkill-timeoutrb.html)), which is why everywhere else in the codebase I chose to do the timeout maths.
init is a better wording if you want to initialize something. I would use init to create a config set or something like that. run_once is ment to be used to do exactly that, run it once. I dont think it is an good idea to rename it.
ditto for above
Please fix the visibility of these headers.
Casting should be a warning sign that something is amiss. Here, why is this an `int`? Why not make it match the type of `elementCount`?
[6cc0b22](https://github.com/r7kamura/kwsk/commit/6cc0b22462616d77779b9735b42bf392b7f74f97)で修正しました。
Probably I missed something, but I don't see a closure here since you don't import some external variables. This is just an anymous function set to a variable. And I don't see why not to create a simple function.
Ah I see, thanks Ben!
Let's hold off on this edit until we actually commit the C# sample project.
This seems inconsistent; the process methods take ref float, while this one takes float[].

My gut reaction is that it makes more sense to use float[] in the public API and do the conversion to ref float under the hood.

(Also, I'm slightly concerned about bounds here; will C# behave well if you pass an array that's too small?)
Do we need to specify states here in the config?
ok. I'm fine with that.
The 'u' prefix won't work in Python 3.x, you need to use the helper function asunicode

    from numpy.compat import asunicode
btw, own_fid is somewhat an incorrect name here since we are not operating on FIDs but on handles, so own_fh or own_file (both used elsewhere in numpy) might be more appropriate
Thanks, I learned about that function from SessionHandler->close() which deliberately doesn't check for failure, as it could cause potential issues. Amended in my latest commit.
Correct to "you may *have* specified an internal node"
у меня в комментах неверное отображение было - я не понял из-за этого. там показывает 1500-какую-то строчку
этот закомментированный код как раз тот, который вы сказали переместить на вики как второй способ
I'd only do one iteration to convert the Arel node and Remove the ASC/DESC modifiers. WDYT?
```irb
>> module Foo
>>   extend ActiveSupport::Concern  
>>   module ClassMethods
>>     def foo
>>       bar
>>     end
>>
>>     private
>>
>>     def bar
>>       "bar"
>>     end
>>   end
>> end
=> nil
>> class Baz
>>   include Foo
>> end
=> Baz
>> Baz.foo
=> "bar"
>> Baz.bar
NoMethodError: private method `bar' called for Baz:Class
	from (irb):19
	from /Users/rafaelmfranca/Projects/github/rails/railties/lib/rails/commands/console.rb:78:in `start'
	from /Users/rafaelmfranca/Projects/github/rails/railties/lib/rails/commands/console.rb:9:in `start'
	from /Users/rafaelmfranca/Projects/github/rails/railties/lib/rails/commands.rb:71:in `<top (required)>'
	from script/rails:6:in `require'
	from script/rails:6:in `<main>'
```
An array on PG can be of many types. Integer, String, so the column type is still Integer, String or primitives types but with array support.
I agree with @carlosantoniodasilva, adding one more case doesn't hurt.
Good idea. Changed it to `updated_count`.
``import numpy as np`` is not necessary to include in any example, it's assumed to already be available.
This looks like it will create an html element with the id `<tt>blah</tt>`. I don't think `tt` tags are supported in html 5 anyway, so maybe just remove the tags here and style these anchors with `font-family: monospace`?
Fixed (should be 0.11). The comment about granularity holds anyway.
I guess you are right. We could put an argument to binomial_expand to get the desired variable to expand the series in, but then that would be equivalent to what you are doing with series. Though, as you suggested, we should put a note in expand_multinomial to get the binomial expansion for non-integer exponent when required.
In still have the bug, as of Friday's last commit.

    sin(x-y).expand(trig=True)

AttributeError: 'Mul' object has no attribute '_eval_expand_trig'
> puts them there right away

But in the loop if it's a gamma you wanted to put it in numer_gamma, didn't you?

```
elif isinstance(arg, gamma):
                numer_gammas.append(arg.args[0])
```
> I've changed this to `modified version of`. 

I like the new formulation, thank you.
Well, I had the impression that anonymous morphisms, which are instances of `Morphism`, could be useful.  I'm not that sure now that you've asked.  I use anonymous morphisms rather extensively in tests and examples, but I can't say whether it's such a good idea.  Moreover, I start to think that anonymous moprhisms are bad for the same reason as anonymous objects are bad.

All right, I'll redo it right away, thank you for changing my mind :-)
* PEP8 explicitly warns against aligning things
* We're more likely to change the system entirely than to move the handlers.
Should the conjugate go away here because x is real?
Maybe this ahppens at some other place in code.
If the functions have been simplified, then the fingerprints will be unique as per our algorithm. In that case, checking the keys and the variables they correspond to would be enough, as we have established that varibales with same fingerprints are interchangable.
Cool, didn't know about this.
Good question. I matched it with maven-gae-plugin: https://github.com/maven-gae-plugin/maven-gae-plugin#maven-gae-plugin
Added one issue for checkstyle and one for removing @author
and missing empty line before the return statement
`quote.subtring(quote.lastIndexOf(' ', 180))`
Oops, copy/paste mistake I missed ;)
Removed require 'params'
ok, agree (forgot it's a test).
I kinda like the simplicity of introducing a wrapper, but the one thing that changes is that right now, both the query/time and the query/wait are based on the same startTime.  If we separate them, then those startTimes will also have to be separate.  Probably minor, but worth it to do it the way it is done, I think.
this indentation pattern is novel and unlike any other occurrence in the manual. Why didn't you follow the indentation pattern used elsewhere? If that form doesn't have any rendering problems, we should continue to use it.
does INCLUDEJS work for template inheritence
@callumacrae Hum... got the main idea, yeah...
@imkingdavid 
When the changes come from my initiative, I know well what to write (check my private forum feature commits, pls). When it's this stuff.... I don't really know how to verbose them. It's just a bunch of minor changes that I don't know how to describe.
If you return $path here, this function would work correctly regardless of whether $path had any variables, no?
Should probably be calling the `send_status_line` function here.
It's `onclick`, not `onClick`. You shouldn't be using it, though (seperation of logic, all that).
It's the old syntax.

On 13 Nov 2012, at 09:47, brunoais <notifications@github.com> wrote:

> In phpBB/assets/javascript/autosave.js:
> 
> > +
> > +	var key = window.location.host + window.location.pathname;
> > +	// Extract the f=, t= or p= from the URL and separate with underscores
> > +	key += window.location.search.replace(/.*?\W(f|t|p)=(\d)+.*?/g, '_$2');
> > +
> > +	// Just store a variable without the creation time for convenience
> > +	var key_no_creation = key;
> > +	key += '_' + $('input[name=creation_time]').val();
> > +	
> > +	// If we have data in localStorage when the page loads we can assume it's
> > +	// ok to load their autosave
> > +	var data = {};
> > +	for (var storage_key in window.localStorage) {
> > +		var explode = storage_key.split('_');
> > +		if (key_no_creation === explode.slice(0, -1).join('_')) {
> > +			data[parseInt(explode.pop())] = window.localStorage[storage_key];
> Because it's a parameter in the object and not a method like I find in the spec.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Fair enough.

@callumacrae It was a couple of weeks ago. ;)
Add a blank line for readability. This looks like one statement.
This is just a single bit, so maybe `// Set the user_message_rules bit`.
LEDGE -> EVENT
p/phpbb3#24
Всички тези тестове си струва да ги групираш в един `describe`, който да бъде "(showing faculty number)"
yeah I think that's a good idea. I'd rather we were preemptive on it =)
ok.. I will do that..
remove commented code on ln 22 and 46
已经是Bar图了。
Isn't this code point equality check redundant with the first iteration inside the recursive compareAt?
-- I think that it just interferes with the correct handling of some edge cases (e.g. 'X%_' LIKE 'XY' ) and should be dropped.
Might be useful to add the ENG- number here so people can look up the bug?
why is this assert disabled -- it seems reasonable.
Put the { on the same line as if
Never use LookupEntry() when looking for spells, use LookupEntryForced() instead and check for NULL. Just for your own culture, check what dbcSpell.LookupEntry(-1) returns (or 0 or any other nonexistent spell).
Could you use `forEach` here? This would be consistent with other source code.

```coffeescript
@testFiles.forEach (file) =>
  mocha.addFile file
```
>Also, as the set of reserved tasks includes active tasks as well, terminate works as expected (e.g. terminates active tasks)

Still, you shouldn't terminate a reserved task.  If a reserved task is revoked it should be dropped _before_ any pool worker starts to execute it.
I am going to nitpick, but the rest of the code tries to use a convention where there's a space after `if` and between `!=`, so `if (version != null)`.
please use ``null !==`` to compare the value with null, to be consistent with our coing standards
yeah, this sounds like a plan.
Ah, that's true, Octokit `1.0.0` will be installed when you next run `bundle install`.  
But I still think it's a best practice to guard against unintentional updates (e.g. when `1.1.0` is released.)
Never mind though, I'll do that when I test it out.

Cheers
I don't like long class methods especially when it is the only method in a class. I think they are not very OO. I would instantiate this class and divide `generate` to smaller methods.

Also I wonder if `fingerprint` without `api_key` would be good enough. What do you think guys?
For the sake of clearness I would merge the first two if hc.ground_motion_fields statements
Why do you need this? We don't use `checkOption` for globals.
Y U NO CLOSE? Fixed... xD
This makes sure we don't send temporary Wix installation files over Git.
Don't chain commands in shell invocations, and calls to things like `cd`, `mkdir`, etc. should be done using the included FileUtils methods rather than shelling out. e.g.:

```rb
cd 'foo' do
  # do some things
end

cd 'bar' do
  # do other things
end
```
oops, I missed with my dupe pull req. ignore my previous comment. sorry
Why aren't we storing this in an `X-Riak-Meta-Version` header? Seems to be recommended for [storing meta data](http://docs.basho.com/riak/latest/references/apis/http/HTTP-Store-Object/)
В данном случае, например, подошло бы «После тестирования ветки с исправлением сольёте её обратно и опубликуете исправленную версию веб-сайта».
I don't remember ever coming to an agreement on that one
evtl.. hab ich auch schon überlegt. allerdings sollte das dann imho ne 1:1 reimplementierung von FB:: sein.. spricht auch nix dagegen imho. Die nativen methode von Firephp bleiben natürlich ungecheckt.. aber das muß dann auch jeder selber wissen wenn er die verwendet.
Might not want to check this one in...
Reading this back, I think a better check would be to ensure each of those templates are included in .templates rather than asserting equivalence.
it wasn't my function, i just moved it to pageStack.
so, i agreed about suggestion.
Even if it's a global parameter, I think it's important to mention '--blog' here.
Why wrapping `return(dirname == path.dirname(f));` into parenthesis?

Also, might seem silly because you don't compare against falsy value but could you do a strict comparison instead `===` ?
Should use C++ style cast here
I would prefer to pull this change in unchanged.
@z-index-brackets-results-panel was 19 and @z-index-brackets-triangle was 10, and now they are both 16. Is that ok?
Safari uses Visualización
It would be more consistent if ``_findCommentStart()`` returned null if not found the same as ``_findCommentEnd()``.
since @fjakobs added the error handler to the server middlewares, it needs to stay at the last position in the stack, since it is special. See #L102 .
should reference a constant for NOT_FOUND or something?
i wonder if this whole thing would have been cleaner and easier in elastic...
Yeah I'm going to just leave this function here since all the others call it. There may be other things we want to add to the shared base context.
This change was just so that the only place where the `:` character is referenced is in common - plus, it seems like the right thing to do
Should not bother creating a Meta object (or dict, as the case may be) unless _grid_callbacks is not empty.
`get_grid` should be removed now.
I really don't like this approach :-/  What about doing it somewhere in `panel.r`?  The panel object is supposed to coordinate the coords, scales and facets to figure out position info.
Yes, sorry. There are some things which are hard to delegate because you spend more time delegating than doing yourself. This showed like one of those. Sorry for all the work you put in. Just do something where you have more free hands and can express your ideas freely. But still concentrate only on one feature. If you find some other thing to fix, do it in another branch, another pull request.
Why both `div` and `ul`? Why not simply `<ul class="user">`?
Or just remove the comment. :-)
I did some minor fixes but I'm not planning to restructure it much more. In my opinion this version is better than previous but if you do not agree, let's just agree to disagree, close the pull request and forget about it. :)
It's already in wrap.start
he put it in a for loop, I guess when you comment on the diff on github the lines move/disappear once changes are made?
You need to put the callback and early return if the update fails; might just be easier to leave your added code as the .success() promise and add the old code as the .error promise.
Actually, it is alright to do:

target && target.removeChild();

here.
you could do if ( !e.dataTransfer || !e.datatransfer.files[ 0 ] ) { return; }

and then set file after this if you need to. This way we can avoid two if statements
We don't need whitespace at the beginning of any scope, right?
I think much of this block isn't required anymore, since CSS is provided by the tabzilla stylesheet. Just keep the pieces that place the tab where we want.
You can probably place the majority of this in the layout, and only manipulate the `innerHTML` of the `autosave-name` span, right?
This isn't necessary here. For now, let's just make template authors deal with `butter.project.customData` manually.
I hadn't noticed that before - good job in removing it :)
no need to declare and assign separately. it's not plain C :)
Can you rename `urls_or_config_hashes` to something less verbose? I think `nodes` is fine, as it is `#add_node` that is finally concerned with what to do with the contents.
> The documentation you referenced seems to indicate a nil reply means an empty list for a "bulk" section, not a "multi-bulk" section. In a "multi-bulk" section the nil reply means a timeout (for a blocking list action) or an aborted "multi...exec" block (my use case).

I stand corrected. I had another look. I was thinking of -1 _in_ a mulit-bulk reply (so that would be a bulk reply). So I think this would be OK, except that the error condition could happen in other circumstances as well. If we look a the example from the protocol specification; 

    C: BLPOP key 1
    S: *-1

In this case the multi-bulik -1 is an indication of a timeout. So if we change the name of the Err to something more generic, I'm ok with the the named Err.
Remove the _ if you're willing
Inadvertent and reverted.
Good catch, with @iNecas dynamic directory line this shouldn't be needed anymore. I'll remove.
no because it'll make it easier for rx to grade.
I think this is describing 'index', show is to show further information about one particular unit. For our purposes, I don't think we will need the show action, since all the information about a particular unit is already viewable in the index table.
Added issue number to comment
(here are other instances of this, but I will not point them all out)
``__all__`` should probably be defined to prevent things like ``izip`` to end up in the API docs
Maybe put this in an "Examples" section instead?  E.g.,
```
Examples
--------
This function is a context manager, and should be used as e.g.::

    with get_readable_fileobj('file.dat') as f:
        contents = f.read()
```
`rt:stop_and_wait/1` will also work.
Though it looks like it should probably never happen, the possibility of an infinite recursion here scares me.
I had to smile ^^, sorry.
Can you add braces { }, it's easy to introduce errors this way, and it looks a bit confusing with the comment
I suppose you could do

    ifdef STATIC
    LIBS += -l dl
    endif
What is the guideline on the decision bootstrap/core ?
We might want to move the rest of the Cache stuff there, too, then.
/heirarchy/hierarchy/
``callable`` is a type, nto a class, so it should be lowercased
maybe it'd be better to use "user != group_user" instead of asking for their ids. That allows the user class to figure out how it checks equality, and certainly an active record class will do it using id by default, so in this case it'd be identical.
Not with weakmaps, no.