labofsahil/github-event-dataset-2012 huggingface dataset:
Next next invoke config

1. Modified the config parser to parse invoke targets.
2. Modified the native packager to generate targets in the native packager.
3. Updated the version of the packager tools.
4. Added unit tests for invoke target.

The issue related to this feature is:

blackberry/BB10-Webworks-Packager#111

The packager tools dependency had to be updated so that the native packager would recognize the invoke target elements in the descriptor and produce correct invoke target entries in the bar manifest file. The version of the updated native packager is 1.5.

If you want to grab the updated packager tools you can do so by grabbing the artifact of the ci build for this feature branch and copying the dependencies folder of the artifact to the project root directory.

Note: The updated packager tools does not support the signing of applications. Therefore this pull request should not be merged until signing is supported by the signing authority.
Changing default build.xml for better clarity
added spec/factories.rb

previously this file was missing, and the tutorial linked here for an example.
from default cursor:move

from default cursor:move to default
Add system_timer dependency when using Ruby version < 1.9

A recent change to the `Instrumental::Agent` (on 5391f6e) requires the `system_timer` gem while using a ruby version less than 1.9.  This commit updates the gem spec to reflect that requirement.
WebAOKP

WebAOKP selection.
Should double check the password field  ...

... since the post value is a string and will always return TRUE on isset(). This fixes a bug which always updates the password on update_user even when no password is filled in.
ModelMixin's lazy model_instance and IsModelInstanceOwnerOrIsAnonReadOnly permission

Instead of loading the ModelMixin's model_instance in the .get() .put() and .delete() functions, I moved it to a lazy (hits the DB only once) model_instance property. Then it can be called elsewhere, and earlier, such as the auth/permission phase, where the ability of accessing the model instance data can provide more granular checks.

As a first step (I'm preparing the implementation for OAuth 2.0) I've added a IsModelInstanceOwnerOrIsAnonReadOnly permission, which only allows write access to the model instance's owner (assuming the model implements a .get_owner() function). Based on this, we can implement a resource/instance authorization system between resource owners.
zh.yml file updated on 05-31-2012 

New Simplified Chinese translation for the plugin on 05-31-2012
Optiml alpha eclipse

The reorganized version of lms that can be imported into eclipse.
Add 'related_results' api, exposed tweet entities, update Tweet members

- exposed 'get related' tweet api result set, see http://groups.google.com/group/twitter-api-announce/msg/34909da7c399169e
- exposed include_entities parameter for timeline/search api
- exposed embedded retweet, user profile json to tweet object
- updated tweet deserializer logic to handle embedded json tree
- added proper equals/hashcode to core tweet, user profile objects
- tweet object is now serializable
Fix index out of range in selection when there is no selections

Not sure what's going on with indentation or diff algorithm.
Password Archivable Bug fix

I am working with a client where security is very high on the priority list and I must say that devise_security_extension addresses many of the concerns extremely well... nice work!

While using and learning about devise_security extension I came across a handful of bugs. I did my best to document them clearly in each of the commits. The main bug that I came across was the password archivable module allowing the most recent password to be reused a second time. This exact case is validated in the secure validatable module, which likely masked the problem. I therefore created a fix for the bug in the password archivable module and removed the current_equal_password_validation validation from the secure validatable module.

Let me know if you have any questions and thanks for all of your great work on the gem.

- Jed
Various fixes for Unicode issues + initial implementation of frequency per 10000
Corrected "Home" translation to real context.

 Previously was literaly "homepage".
Added support for Func parameters and added tests

Using a custom Ninject module, the bootstrapper is now able to resolve
Func<T> dependencies. Also added a test project and a tests that makes
sure that the INancyEngine can be resolved from the container.
SOCIALFB-51: Added commentCount to Post
2.5.x

Some changes in the cheat sheet, see bug #3108
Draft code for the "/app" page.

As I was assigned, issue #26 asks for a /app page where registered projects are listed.
Besides I did not finished, I would like to submit this code as I practice the workflow. As a simpler task, it is easier to get some code review and feedback.
Composer json fixes

* Updated composer.json to be able run tests using composer. (changed Tests/autoload.php too a little bit)
* Added paragraph about running tests to README file.

License of the code: MIT

Travis CI status: [![Build Status](https://secure.travis-ci.org/l3l0/AdmingeneratorGeneratorBundle.png?branch=composer-json-fixes)](http://travis-ci.org/l3l0/AdmingeneratorGeneratorBundle)
Removed a hard-coded reference to PHP binary.
Update build.sh

Changed to $HOME
Added javascript tests into the Rails app.
Incorporated Rod's changes and re-arranged headings

I've got a few javascript things to drop in over the top yet, but I offer this pull request for your consideration in the meantime. It includes the changes from Rod's pull request, plus a bit of re-arranging.
Here's the revised patch and semi-integrated test

* Adds test for Quoting module's #quote and #quoted_datetime
  methods when integrated with the adapter
* Includes test for ActiveSupport::TimeWithZone wrapping Time and DateTime
Remove trailing whitespaces

Just removed some trailing whitespaces
Update Contacts/Files/res/values-ru/strings.xml
Copy

updates srtings.xml
ifdef for sch-i1500 devices
Разделение scriptCSS на части

Deleted
CSS3 анимация.

Собственно само изменение алерта можешь сделать по вкусу.
Added Boundless Learning to the list of companies hiring

Added us to the list and fixed a typo elsewhere ;)
support delayed response
Trad env_manorinterior_act4return_english
Trad "env_marshhouseinteriorroom_english" et "env_rooftop_english"

My bad, j'avais linké la mauvaise adresse pour le push...
Ca devrait être bon maintenant.
Keep head repo up to date (Version 0.6.0).
Mute

M now mutes the sound in the game. It doesn't work in the menu but I can't quite figure out why.

I think it is to do with the InputHandler not being used on the menu. I tried putting it in the KeyPressed(KeyEvent e) function but couldn't get it to work properly.
Now works with python 2.x; fixed multiline q/a; simplified code
Small fix for se_default_search
Looks like you wanted to use this in OSDMonitor::get_health
@hh just fixed some issues in veewee 03 master to make the scripts work again for windows. Hope this helps you go through with the merge easier.
Thanks.  I'll use the (?u) syntax inside the regexp.
Was this meant to hit master?
Sooner this day, I would have complained about the indentation. ;)

Looks fine. Yet, I can see it's reviewed already.
不符合xhtml标准阿.......
I think this change causes this problem: https://bugs.launchpad.net/ipython/+bug/383554?comments=all  that bug makes ipdb almost unusable on windows - every time you hit a new set_trace within ipdb, ipython breaks.  for debugging django it's a huge pain.

In my local copy of ipdb I have commented out this line:

    io.stdout = sys.stdout = sys.__stdout__

which fixes the problem.  Maybe there's a way to just not to do that line if it's already done?
I meant that if you use the code I posted.. then you don't need the extra events... not sure I understand what you mean with level of interactivity...

About the focus event... why would you bind to the focus event at all... 

at first "focus" - when you focus the table field it's empty so nothing to be done there... 
any next focus - in the case that you typed a custom value for the model... and then just focus the table field your code would change the model... this maybe is not intended... that's why my point that "focus" should not be used
不曉得為什麼上一個回覆用英文... ORZ
P.s 這commit更改使用者習慣：
當 常用輸入法(s) 和 symbol table 被設成 in cycle 時，使用者若習慣使用`<Ctrl><Shift>`開啟 symbol table，
在這commit以後，若 symbol table 已被啟動，使用者剛好又使用彈出式選字視窗，out focus 時 symbol table 被暫時隱藏，使用者將無法單擊`<Ctrl><Shift>`開啟 symbol table ，而需重新循環。
雖然這個使用者習慣我完全不鼓勵 XD
Unless you elaborate it will sound silly to talk about the length of a point. See my previous comment.
Excellent. Great job.
That's right. Perhaps it's meant to be `==`?
`#!/usr/bin/env python`

would be nice.
Hi, have you read my reply on gunship topic of TC forum? http://www.trinitycore.org/f/topic/1005-gunship-battle/page__view__findpost__p__35499
Sure I'll do that.

On 20.01.2012 14:43, Christopher Smith wrote:
> They pass. Could you run the rest of the test suite, though? I think
> this is a pretty sound change but you know how it goes.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ness01/sympy/commit/6f9811058b0c0f560d8e92cd885bb80045b48e0c#commitcomment-886208
>
Nope, this wouldn't work. (You can make multiple listeners of one type, with different priority, linked to one class)

I'd rather do:
unregisterEvent(Type, Priority, Listener)
and the 2nd one is fine:
unregisterEvent(Plugin)

Also, it would be nice to have:
unregisterEvent(Type, Plugin)
unregisterEvent(Type, Priority, Plugin)
unregisterEvent(Listener)

:)
Any idea how to get the access token of the instance? I noticed this error as well but decided to rush in putting the token in a session once logged in (the project was timely and no longer active)
dimesion --> dimension

The `np.` prefix can be left out of the type specifier everywhere
I'd guess this is the first "resource".

I think of `@prop` being similar to a ruby instance variable, and in ruby, you can't use `@` on its own--you still have `self`.  Maybe that's not a valid argument.

Most people I know who are writing coffeescript seem to agree that naked `@` feels weird (which also, isn't a great argument).

So ... "because it feels weird"

Would love to hear @maccman's reasoning.
I would suggest they're reasonable (e.g. roughly the same as time.time()). inode you could probably ignore.
Interesting, since it is only a short way of starting compiling.

Does exactly the same as Visual Studio would do.
I don't care about the `<?xml ...` line, but I would care about interoperability with standard XML tools, not that I have some need for that ATM. So you could at least claim to implement some "subset". :-) But documentation should suffice at this point.
why not include the DEBUG code? limits does...like I said, I found it instructive to watch this process.
I used your previous implementation (v0.5)
It would be nice if we could use names that follow the native/host names. Perhaps have a default namespace of "API" or "JESSIE" and then an option in the builder to override to a name of your choice.

Although with that said it is nice have the code as simple as it is currently.
Because getComputedStyle is on the global scope in most browsers.  So doing
something like:-

if (getComputedStyle) {
...
}

...would be a potential disaster.  The local one may be undefined (due to
feature tests), but the global would resolve.

On Thu, Feb 16, 2012 at 10:27 AM, Balzs Galambosi <
reply@reply.github.com
> wrote:

> Why is it called `getStyleComputed` instead of  `getComputedStyle`? It is
> 1) inconsistent with the vanilla DOM Api naming 2) inconsistent with other
> function names inside this collection. Other functions would look similarly
> bad `getSizeInner`, `getSizeViewport`, `findStyleProprietary`.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/rassie/jessie/commit/a0cbe2328294127dbd7bd44653b81d2a5ac55a84#commitcomment-974057
>
You could use Collections.<String>emptyList();
done goofed, i wasn't supposed to do that
does fill make sense for lines?
Yes, this is new info to me as well, thanks! There are a few places where we could use this
Skapade en branch enligt *git flow* vid namn `feature/sti`. Gjorde en test-push, tar och jobbar lite på branchen i fråga.
@richarddong sorry之前没有注意到这个评论。route里是这么写的 应该是因为动作是search这个controller里面的search_news_by_tag吧。
не, там она. тут стейтов неограниченное количество, но некоторые из них при дропе создают новые, которые дропают текущие и дальше в цикле :) разберемся, это не проблема, главное что базовый код неожиданно почти сразу заработал...
We've committed to using XUI. Yes, in some cases you might be able to not use it. But that's not what we've decided to do. We're using a standard, stick to it.
Euh, why should it thrown an error? I don't understand why people use the emailaddress as the name. See: http://forkcms.lighthouseapp.com/projects/61890/tickets/241-frontendmaileraddemail-bugs-out#ticket-241-11
why? you'd recompile it, put in the root folder, commit your changes (without changing *revision_R2.h*) and use smth. like:
    $ git pull origin master
    $ ./git_id -l
    $ git push origin master
I'm not excluding the old tests though. It seems like we should include old tests for now (unless there are conflicts). That way if we touch any shared libraries we will see test coverage in CI.
Wow. That's pretty convincing! Albeit, for some reason, when I fly around the audio speeds up/down. But if you don't move it works really well.

Yeah, we can probably upgrade `misc_sound.html`. Or duplicate it and do `misc_sound_chrome.html`.
The original code is not Duffed, this is alRaidCD:
https://github.com/Allez/alRaidCD
http://www.wowinterface.com/downloads/info18062-AllezUI.html
by Allez.
This image is provided by travis-ci
Thanks for your feedback.

No problem re tests, I can add some.

About the rules using comments: I like the idea. But in a project I just did with this, that may become very unwieldy as I added about five top-level selectors plus four media queries with 13 selectors total.
Its a list
the tool tip is separate. It still shows.
you might want to try rewriting this using the --porcelain option. it is designed to be used in scripts and is guaranteed to be stable.
Коммит ты делаешь для того чтобы записать изменения в папочку .git а сами изменения улетают на сервер только после push
Ok, LGTM. The patch is missing a signed-off-by: line, so please add one on v2
EH ta brabo
It should just be this line only that gets skipped, the others are required.
You should add kind of "plugins" to allow third party contributions. It could be fun to trigger a Karotz (~ Nabaztag) for instance :)
Please read this here: http://forums.spout.org/threads/restriction-of-spout-in-plugin-names-and-namespaces.958/
In the end, I've [changed rummager](https://github.com/alphagov/rummager/commit/70501f5e5844b830ca0b15a12cddfe926cf382ba) to fix its rack version to match the one supplied by puppet and the one fixed in the whitehall app Gemfile. This seems to have solved my problem for now. Since rummager's dependency on rack is only indirect i.e. via rack-test, I assumed that dropping the patch version by one shouldn't matter, especially as the tests all still pass.
Same thing for the phpDoc
On a slightly moar serious note,

it hasn't been fully actualized yet, but the point of the EvilQuery classes are to encapsulate the places where we do way too much ridiculous stuff with sql to deal with the fact that we have a nontrivial amount of users,

If someone (especially you miss @sarahmei ;-)   )  can think of a better (moar fun) namespace for this stuff, they suggestions are welcome.

EvilQuery came out while I was trying to encapsulate what I was doing while fighting a fire. EmoQuery is a silly play on that. Anyone got a better idea?
What is the current status regarding automatic driver instantiation for PCI device drivers?
Very cool.  My main desire in a fork is two things: renaming the packages and economy support (through vault).  One of my plugins has to use the DC api, and it has to support the old version and new version (which use slightly different calls).  So in my owner clone of your fork, I refactor the package names so there is no conflict.  As for economy, we'd love to see drop chests cost money to make.  Vault is supper easy to support, so it wouldn't be that big of a deal.  I could always insert it myself, but you seem to be doing great stuff, so I'd hate to have to fork it again.
What's this usage of `share`, can you give a example on this?
`it "duplicates..." do`
Notice that this is called ".rightBody" but is on the left.  Maybe it would be better to name your styles for what they are "menu" or "body" that way you separate what it is from how it looks.

Everything works this is just a comment on naming.
@mattwynne do you want me to just add those failed assertion tests or replace the "positive false" assertions?
yeah, and vars aren't currently doing much at all. In the 0.2 version we'll look into how to deal with vars (and bindings).
Note that this works because event dispather extends channel, as seen here: https://github.com/pusher/pusher-js/blob/master/src/pusher_channels.js#L66
A plugin can easily do the permissions check itself by throwing a block break event. But it couldn't get the dropped items without a self-maintained list or calls into the CraftBukkit/Minecraft code.
blood tap can be fixed by merge TC this time
That sorted it. Thats a great addition and looks so much neater. 

Just wondering now if the Byte Sizes implemented into Gugahoi's Search module can also be used in the Disk Space module to correctly identify KB's, MB's, GB's and TB's...
Most definitely yes. I'm not sure if we should write it in a system-wide path, though. Isn't it possible to install the binary in some autotest controlled location?
You don't need to initialize this twice.
contains does a partial match. the annotations uses equals and which is why the tests were failing until i made the above change
This was another type-checking related fix.
indeed. there are quite a few things i need enums for
I prefer creating the flag or switch in a test, but when I do mock it, I put a comment after to specify *which* flag it was I was intending to mock.  Good for when you search/remove flags or have many flags at the same time as mentioned.  I think it'd be handy to encourage this behavior...

`@mock.patch.object(views.waffle, 'flag_is_active')  # for flag 'waffles-are-yummy'`, e.g.?
I'm hoping there would be less need for things like `//{{{` with the new JavaScript rendering features.

In terms of compatibility I just meant ensuring that there is no ambiguity with existing syntax.
Attention, il peut y avoir un retour à la ligne. Voir par exemple [océan](http://fr.wiktionary.org/w/index.php?title=océan&action=edit)
Fixed this and made hooks more consistant in latest commit
It's possibly good idea. We need to test it.
Is there an issue filed?
I'm sure I must be missing something. Tried again, this time I installed Ruby 1.9.3 through rvm and started again from scratch.

I think I'm on the correct commit - after doing a fresh pull from your repo, I'm checking out commit c97608ab73596118326dc6160e5423c64144361e before adding my parse_resourse.yml and running the tests as seen here:

https://gist.github.com/f1b2a195ddc1d1cfd80d
Comment also created by PyGithub
seems to not work with firefox 9.0.1:

i get a white background here:
http://test.springrts.com/phpbb/
Fixed in 0.8.6-SNAPSHOT. Will release 0.8..6 tomorrow just for you :-)
I don't remember off-hand, but I'm pretty sure I did this because it was causing errors somewhere in Haystack because those instances don't have any of the data that Haystack expects.  I don't think we can /totally/ disable the indexing; otherwise all the view tests will fail since they depend on the index.
> when was the last time you used an 80x24 tty?

Let me think: I use them daily. 80x61 right now. ;-) Sorry, but I'm on a windows machine, which has 80x25 per default. And I like to work with two tty windows side by side on my notebook, so 80 columns is my standard. But I'm not sure, if that's normal.
that makesme crazy
core: fa546d1428989a4e3eb17ad60be34ab3f5f08ab8 ( latest )
again this startup error
2012-02-22 19:16:18 Loading Conditions...
2012-02-22 19:16:18 ERROR: SourceGroup 17465 in `condition` table, does not exist in `creature_loot_template`, ignoring.
2012-02-22 19:16:18 ERROR: Item condition has 0 set for item count in value2 (0), skipped
2012-02-22 19:16:18 >> Loaded 7059 conditions in 85 ms
J'ai testé et c'est très cool, j'ai vu qu'il n'y a plus le problème avec les petits poids. J'ai testé l'example donné dans la page que tu m'a passée et j'ai remarqué que les contraintes sont pas du tout elastiques, ça serait pas mieux qu'elles soient peu élastiques pour les polygones, ou enfin de choisir xD
Après j'ai remarqué des collisions étranges xDD, je pense que c'est à cause de la masse de certains points, mais je ne sais pas, j'ai pas trop regardé encore ^^
Branch takes a selector, so you could do $qp->branch($k)->count() > 0. I don't think that'll make a performance difference, though.
@shlomi1515 have you tested the SAI I suggested?
The problem with waiting for cast before summoning is still open as far as I know but since all *_scripts have to be replaced with SAI there is no other option I'm aware of.
completely Disagree with this Enrage script method
Hi Tim, I now really think that we should add a MAXAPI and MAXAUDIOAPI macro to jamomalib.rb instead of cluttering the yml files with specific windows and mac paths. What do you think?
Is this necessary? It becomes kind of annoying having to hit enter ater the message displays. Most other compiled languages do not report anything in vim, just errors if there are errors.
Were not

On Feb 5, 2012, at 10:53 AM, Wouter Gerarts<reply@reply.github.com> wrote:

> why remove the licence? o.O
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ForgeWare-Inc/ForgeCraft/commit/55e50622f8a4907c1a8b637b5f7f1f0de4215c70#commitcomment-934310
Why did we add a README file?
Why is it blank?
I just remembered something. Do you have the module info in the msm_rmnet.c?
I'm talking about this:
https://github.com/marc1706/desire_kernel_35/blob/053135b8f7741cbef46b0bb084f691497d781c06/drivers/net/msm_rmnet.c#L687

If you don't have that you need to change EXPORT_SYMBOL_GPL to EXPORT_SYMBOL.
This line is not needed, can you please remove it?
problem is not in sqlb!

we cant also use slub or slob

the same problem will happen!

so update is good, we need to find out why only SLAB is good to be used!
This PR introduced a validation error here. It should be:

type: [Array<Titanium.UI.View>, <Titanium.UI.Toolbar>]
你们赢了,github还能这样子玩=,=
Hmm noted. https://github.com/jshiell/checkstyle-idea This should help me not commit garbage.
Hey Kroc, I will contact you tomorrow. I've been using NNF for some internal stuff at work and I'll be glad to tell you about our usage so far. :-)
Make `INLINABLE` to get rid of the `Eq` dictionary overhead.
კარგია , მადლობა :)
How does this code handle grouped Datasets?  It looks like only the first group will be sorted.  I haven't tested.
How about `logger.warning`? :)
Some stupid comment.
Line looks misaligned?
Оченно жаль.
Йоппера всегда вперде планеты всей.
Hi An,

I got it to work. I think the reason for failing it was that I had to much messed around with the original Dungeon example, different phonegap versions and so on. After resetting my device and installing a new clean build of your example everything worked like a charme. You did a really good job!

thanks,
Ulli
Yes,initramfs allow "none compression" or "Gzip compression",kernel compression type can be customized,default is Gzip.

root@xiaolu-ubuntu:/home/xiaolu/ramfs/i9100# ./repack zImagexxkpa xxkpa.cpio new_zImage
[I] ---------------------------kernel repacker for i9100--------------------------- 
[I] Extracting gzip'd kernel from zImagexxkpa (start = 17972) 
[I] Non-compressed CPIO image from kernel image (offset = 192788) 
[I] CPIO image MAX size:2014720 
[I] Head count:2207508 
[I] Making head.img ( from 0 ~ 192788 ) 
[I] Making a tail.img ( from 2207508 ~ 9858276 ) 
[I] Current ramdsize using cat : 2422272 with required size : 2014720 bytes 
[I] Current ramdsize using gzip -f9 : 1374307 with required size : 2014720 bytes 
[I] gzip -f9 accepted! 
[I] Merging [head+ramdisk] + padding + tail 
[I] Now we are rebuilding the zImage 
[I] Image ---> piggy.gzip 
[I] piggy.gzip ---> piggy.gzip.o 
[I] Compiling head.o 
[I] Compiling misc.o 
[I] Compiling decompress.o 
[I] Compiling lib1funcs.o 
[I] Create vmlinux.lds 
[I] head.o + misc.o + piggy.gzip.o + decompress.o + lib1funcs.o---> vmlinux 
[I] vmlinux ---> zImage 
[I] Compiled new zImage size:5119280 
[I] new_zImage has been created 
[I] Cleaning up... 
[I] finished... 
root@xiaolu-ubuntu:/home/xiaolu/ramfs/i9100# ./repack zImagexxkpa xxkpa.cpio new_zImage lzo
[I] ---------------------------kernel repacker for i9100--------------------------- 
[I] Extracting gzip'd kernel from zImagexxkpa (start = 17972) 
[I] Non-compressed CPIO image from kernel image (offset = 192788) 
[I] CPIO image MAX size:2014720 
[I] Head count:2207508 
[I] Making head.img ( from 0 ~ 192788 ) 
[I] Making a tail.img ( from 2207508 ~ 9858276 ) 
[I] Current ramdsize using cat : 2422272 with required size : 2014720 bytes 
[I] Current ramdsize using gzip -f9 : 1374307 with required size : 2014720 bytes 
[I] gzip -f9 accepted! 
[I] Merging [head+ramdisk] + padding + tail 
[I] Now we are rebuilding the zImage 
[I] Image ---> piggy.lzo 
[I] piggy.lzo ---> piggy.lzo.o 
[I] Compiling head.o 
[I] Compiling misc.o 
[I] Compiling decompress.o 
[I] Compiling lib1funcs.o 
[I] Create vmlinux.lds 
[I] head.o + misc.o + piggy.lzo.o + decompress.o + lib1funcs.o---> vmlinux 
[I] vmlinux ---> zImage 
[I] Compiled new zImage size:5472444 
[I] new_zImage has been created 
[I] Cleaning up... 
[I] finished...
I am uploading it first before changing anything
None of the above is needed now that we're using the ListView below
Nothing huge but the link isn't right.
!! :)
When you gather information about constraints, you at some point need to load the composer.json file. At this point, composer also sees that there is a repository defined in the file and could add it to its internal list. I don't see how composer could solve constraints without parsing the composer.json files of the different repositories that makes no sense.

Is/was there a ticket on the composer repository for this?
Added a test for fetch() and made small recommended fixes.
I see. I still think all the `validateXy()` methods should be directly on `Validator` :-). I'm going to add a remark to the section that there is an ongoing discussion regarding this matter and will ask for other opinions on the mailing list.
i have to create a mvc in php plz help me what i can do,your cms mvc more difficult to learn rpl a s a p
This is not the Kobayashi Maru. This test was here for a reason. As is now, these assets would have no chance of coming from the pipeline since they do not include the real URL. I'll look over some of your other commits to see what broke.
it might be useful to directly reference the relevant code snippets. You can extract src code using the openengsb-maven-plugin and include it afterwards into your documents. For an example see [1]. While the parts in pom.xml are rather "black magic" [2] the usage in the src should be quite clear [3].

[1] https://github.com/openengsb/openengsb-domain-test/
[2] https://github.com/openengsb/openengsb-domain-test/blob/master/pom.xml#L87
[3] https://github.com/openengsb/openengsb-domain-test/blob/master/src/main/java/org/openengsb/domain/test/TestDomain.java
[4] https://github.com/openengsb/openengsb-domain-test/blob/master/src/manual/docbook/org.openengsb.domain.test.xml
will we put it again in systemui ?
for -> with?
props[p] can get what you want
"editable" can be inherited from SimpleListItem
can't this null pointer if getUsernameAttribute is null? You should check it the oppositie way or better yet don't use a default attribute.
any reason not to log that stack trace?
I don't understand why this constant is protected whereas the others are private.
I saw that many times : why using this extra property ? ```transient``` is not enough for your needs ?
If we do that it would be possible to serialize that case with a class name
string. Perhaps not important.

On Tue, May 15, 2012 at 4:16 PM, viktorklang <
reply@reply.github.com
> wrote:

> > +      case Left(e)      => throw e
> > +    }
> > +
> > +  protected def deserialize[T: ClassManifest](data: ByteString, clazz:
> Class[T]): T = {
> > +    val bytes = data.toByteArray
> > +    serialization.deserialize(bytes, clazz) match {
> > +      case Right(x) if classManifest[T].erasure.isInstance(x) =>
> x.asInstanceOf[T]
> > +      case Right(other) => throw new IllegalArgumentException("Can't
> deserialize to [%s], got [%s]".
> > +        format(clazz.getName, other))
> > +      case Left(e) =>
> > +        // Fallback to the java serializer, because some interfaces
> don't implement java.io.Serializable,
> > +        // but the impl instance does. This could be optimized by
> adding java serializers in reference.conf:
> > +        // scala.Function0 (the creator)
> > +        // com.typesafe.config.Config
> > +        // akka.routing.RouterConfig
> > +        // akka.actor.Scope
>
> Yeah, we could always create our own Function0 for the class-case
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/akka/akka/pull/452/files#r823633
>



-- 

Patrik Nordwall
Typesafe <http://typesafe.com/> -  The software stack for applications that
scale
Twitter: @patriknw
cool, should at least test for error status and, if possible, error message
Sure, I'll add a test for that.
will this patch still ignore .git files by default?
_trigrams is and instance of FreqDist. That class has a custom .keys() method that sorts the keys by values before returning a list of them.
This means the first member of this list will be the one with the highest frequency and (as a consequence) highest probability.
jdob, I thought of that, but in order to truly remove cross git repo dependency, we will also have to copy all the files in platform/selinux/server in pulp_rpm as well. Do you think we should do that?
Typo: queue -> queued
This should not be needed anymore, is it?
I'm working on updating this and will comment on the other pull request after I post it to that branch.
Of course - will be fixed by removing { and }. Thanks!
Somewhere, not sure if in the docstrings, the design decisions for some aspects of this class mus be documented: for example: why *fields['corpora']* must be callable.
Hay que quitar esta constante de Unknown....
We dont need this. It's already available through ```resources :experts``` (Line 12).
I think this process should block (maybe an option in the task like isSequential) until the response is received. For some use cases, we can not process the next task until the current one is completed as we can not be sure in which order they will be executed and maybe they have to be ordered sequentially,
I don't understand this. Unneeded?
This way we are also ignoring the native code, I think we should move the compilation out of this directory instead.
I deleted the staging repo commit. Now I'm wondering how wise that was, removing a commit someone fetched already.
so the construction of this class can only succeed if besides the current thread there is one available for running the manager actor; are you sure that this cannot possibly fail?
yeah I think that makes sense.
Good spot… I have a nasty habit of not using that syntax.
We probably want to check the number of arguments using g_strv_length(values) rather than doing it one by one. Then we can have something like "Specify a package name and data type, e.g. 'colord url'"
you already check it in your $myMainConfigFileName
That should be `mode_`.
The fast path is handled a few lines up. If this code gets executed, there's at least one removeListener listener.
Style: extra newline.
You probably want `LIBS = $(LUADIR)/src/libluajit.a $(LIBS)` - doing it the other way around will break on systems where `-Wl,--as-needed` is the default, i.e. most modern Linux distros.
Your suggestion does not work. It needs another pair of braces around it.

The code was originally written as:

```python
constants = {"ROSE_ORIG_HOST": socket.gethostname()}
for k, v in constants.items():
    pass
```

Therefore, it looks as it does now. If there are more automatic constants in the future, I'll pull the `dict` out.
Is this really the intended hash code?
OK, I see what you did.
I did it this way so that the hosts will be printed in the same order as the provided arguments, and also what I thought would be a nice-to-have feature, case-insensitive matching. Thinking further, it seems case-insensitive matching wouldn't make much sense.

How about adding a ``get_host`` method to the ``Configuration`` class, and then doing the following:

    for h in args.hosts:
        host = args.config.get_host(h)
        if host:
            print host
        else:
            print >> sys.stderr, "host '%s' not found" % h

I can submit a different pull request for it if you like.
Is it really necessary to call handleDelete again here?
Spaces ;)
Agree. In this case I think we should revisit qiime/util.py and move some functions to qiime/stats.py, such median_absolute_deviation(), IQR() or matrix_IQR()...
Sorry about that, it just seemed like a good thing to do, since those methods are only used internally. I've removed the `private`
It's required here, I'm getting two failures after of delete it:

1) Error:
test_permitted_attributes_can_be_used_for_mass_assignment(ForbiddenAttributesProtectionTest):
ActiveModel::ForbiddenAttributes: ActiveModel::ForbiddenAttributes
    /Users/guille/code/rails/activemodel/lib/active_model/forbidden_attributes_protection.rb:8:in `sanitize_for_mass_assignment'
    /Users/guille/code/rails/activerecord/lib/active_record/attribute_assignment.rb:36:in `assign_attributes'
    /Users/guille/code/rails/activerecord/lib/active_record/core.rb:188:in `initialize'
    /Users/guille/code/rails/activerecord/test/cases/forbidden_attributes_protection_test.rb:32:in `new'
    /Users/guille/code/rails/activerecord/test/cases/forbidden_attributes_protection_test.rb:32:in `block in test_permitted_attributes_can_be_used_for_mass_assignment'
    /Users/guille/code/rails/activesupport/lib/active_support/test_case.rb:70:in `assert_nothing_raised'
    /Users/guille/code/rails/activerecord/test/cases/forbidden_attributes_protection_test.rb:31:in `test_permitted_attributes_can_be_used_for_mass_assignment'

  2) Error:
test_protected_attributes_cannot_be_used_for_mass_assignment(ForbiddenAttributesProtectionTest):
ActiveModel::ForbiddenAttributes: ActiveModel::ForbiddenAttributes
    /Users/guille/code/rails/activemodel/lib/active_model/forbidden_attributes_protection.rb:8:in `sanitize_for_mass_assignment'
    /Users/guille/code/rails/activerecord/lib/active_record/attribute_assignment.rb:36:in `assign_attributes'
    /Users/guille/code/rails/activerecord/lib/active_record/core.rb:188:in `initialize'
    /Users/guille/code/rails/activerecord/test/cases/forbidden_attributes_protection_test.rb:51:in `new'
    /Users/guille/code/rails/activerecord/test/cases/forbidden_attributes_protection_test.rb:51:in `test_protected_attributes_cannot_be_used_for_mass_assignment'
Notice that the `phrase` was being escaped before used in the previous code, with your changes it no longer is.
Why you removed the `polymorphic` option? I think that is better use it instead of the `column_exists?` call.

WDYT?
If there's another intentional difference I'm not aware of it.
I think that this adds even more unnecessary confusion and additional compatibility issues when people's perfectly fine `coll.par...` stops working in 2.10.0.


Why not just `override def par(implicit ts: TaskSupport = defaultTaskSupport): ParRepr`?
Unit triangular is a triangular matrix with ones on the diagonal.  It's not a commonly used predicate outside of numerics.
if you use `uniq` you will retain the original order and still get no repeats. This may be more canonical.
This change only seems to complexify the code gratuitously.
I think this might also confuse the test runner, so it should probably be renamed.
https://github.com/sympy/sympy/pull/1535 Just changed this a bit. I don't remember exactly what it did (maybe made it private?) but I do remember that it was changed. You might want to check that out. Users really shouldn't be using mat. 

Also, this is the way it was before. Now it's a property rather than a variable attached to the object at creation time (see Ln 42 in this file). I consider this a separate issue to this PR
perhaps `whlie len(lines) > 1 and lines[-1] == '\n' and lines[-2].endswith('\n')` would be better.
Ok, made suggested change--it is easier to read.
You could get away with a ```reduce``` here.
It was discussed whenever it happened.  So if you dig around, you can probably find it.
That's because an object is essentially a name, so an object without a name is essentially nothing.  On the other hand, a morphism, is an arrow between two objects, which may also have a name.  Anonymous morphisms are useful when talking about identities (doesn't matter which name you chose, of course).  Also, `Diagram` makes some use of such Morphisms: when constructing all possible compositions, the new composites are anonymous and are unique.

However, I fixed this:
```
In [5]: a = Morphism(A, A)
In [6]: a == a
Out[6]: True
```
which used to be `False`.
Note that this line is removed from the code for week2 since computing the order for something like SymmetricGroup(1000000) at initialization proved to take almost all the time. Instead, I changed the code a bit so that the order is computed as factorial(n) only when it's called for explicitly. The same applies to AlternatingGroup.
could you change 'mass center' to 'center of mass' throughout?
`c, nc = self.args_cnc(cset=True, warn=False)` can be used and the `_ncsplit` deleted.
Can you put a simple doctest here showing how to use the class?
This section also needs some identation fixes
that seems dangerous and I'd say it's wrong. The parent is only pending if it does not have an ID, from what I have seen so far.
Yes it should, my apologies, I guess I fat figured this one. The test does still pass so I'll also amend the test to properly test these arguments.
Print лишний
This function isn't exported, is it?
No, it holds the old value for restoration at [L479](#L0R479)
Also please don't use `\` to break strings. Use parenthesis at the start, eg

`label=('Receive...`
please change machinevolumeCollection to machineVolumeCollection
One per line?
Use `typeof var === 'undefined'`, as what you're doing will throw a ReferenceError.
Isn't this done in common.php?
This should go into the schema changes section?
L_PURGE_EXPLAIN
I'm not sure I like "Site home". Perhaps "Main website URL"?
I'd say it depends on what errors can actually occur.
err isn't defined in this closure
Shouldn't we be using NewHistory?
Din moment ce e vorba de un bonus, cred că score(qotd-ok) trebuie să se întâmple mereu. Doar acordarea bonusului se întâmplă condiționat.
Yep, it's probably a gcc quirk. As said here: http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2007/n2431.pdf
```cpp
if( nullptr ); // error, no conversion to boo
```

So probably I'll change to
```cpp
return test == nullptr ? 0 : 1;
```
Это блок с получением всех баннеров я бы вынес в самое начало, до всех проверок. Если они не загрузились, значит что-то не то и продолжать не имеет смысла. Заодно можно не пытаться обойти ошибку с помощью $units = array();
I'd give this a better name.

pykeg/web/wsgi_app.py ?
No, I dropped them.
Also, instead of hard-coding in these drugs you should issue a GET call to get all the drugs for this store. Your GET call will look exactly like the call in screener -- check out PharmacyForm.js and DrugStore.js in src/screener/app/view
Yup, my mistake. Nice spot. 

I'll correct it in a PR update.
minor, *"these"* feel free to ignore!
变量名小写首字母。要不你会覆盖前面的定义的。
This is only useful for tests. I don't know a better way to do this.
For the sentence that starts with "So  it takes..."

"It takes as parameters the lists of input and output Theano variables, as would be provided to `theano.function`, and a list of real values to pass to the compiled function (don't use shapes that are symmetric, e.g. (3, 3), as they can easily to hide errors)
Not true for new c code, we can reuset the theano/scalar.{Mul,Add}.identity properties to have generic code for this. I talk about this in another comment too.

Making the test generic should be hard. it have been done for the CPU CAReduce function.
Another option is to modify this Op to return a Generic() type instead of an Generic(), Ndarray() tuple.  Is there utility to exposing this "not-yet-ready" data buffer as an output?  If not, then it might be better to return a single output of type Generic(), which is in fact a list of two things: `[request, un-initialized data]`
Fewer arguments, but less explicit about what the method needs. I like knowing what information is required to do something. It tells me something about how the method does its job.
IMHO, single-letter variables are only OK for very short block scopes, and this case is a stretch.
you don't want to call out bitcask explicitly here?
Как именно используются? Не хватает документации в этом месте.
One thing about a key is that it's not always 1:1 to an Attribute. A key would probably be a Header or Header-like object that contains one or more Attribute objects. I don't know if we should necessarily assume it's a property of an Attribute.
could you replace this number with a named constant?
Splitting the caches means no more need for `.indexOf`!
Uh, of course... I was mislead by that the referenced sub will return undef, but that was also wrong... Lets see...
OK. You are the boss.  I'll change the rename back. Even I don't know the differences. But if there is no good reason for other functions' name, I refuse to change. 

About the script. I can just generate 95% percent of the codes. About half of the files need manual adjust. So It's meaningless to make it into the repository.
Add the subroutines for data sampling.
This is an insertion sort, which may become a bottleneck.
I have no definite answer yet. But I want to mention that for some modules initialization (at least in the first iteration) will depend on the first input tuple and cannot be done before seeing this.
Changed and updated.
This can be converted into a `fails_with :clang` now
Make that I should since I wrote this :)
I'm not a huge fan of adding docstrings for overridden methods if nothing has changed. I'd rather leave it up to the developer to check the base class (which admittedly didn't have docstrings until a few minutes ago) which is the definitive guide to what the method should do IMO.
Same as before, only `@constructor` is needed
flip lines 49 and 50 ? keep field list, parameter list, and assignments in the same order for consistency!
Du hast Recht.

also... abgemacht.
I'm not sure I buy this, either. Basically, what we're saying is if you explicitly don't want jQuery in your project, we're going to go ahead and include like a third of it anyway. "Why," a curious user may ask? Well, dear user, because we use `querySelector` twice in the framework (one of which could be replaced with a `document.getElementsByClassName`), and therefore need to burden you with this extra 4kb dependency.

Counterpoint: It's 4kb, remove an image from your project.
Get in the habit of always using `===`/`!==`
Sure, but I don't see that making a large difference in how brittle it is. 

The setupAdMob function would just have 
if (this.AdMobAd) {
          // need to destroy and delete existing ad
          this.AdMobAd.destroy();
          this.AdMobAd = null;
        }
        
        this.AdMobAd = new toura.AdMob(this.appConfig.ad_mob.publisher_id);
        this.AdMobAd.loadBanner(this.appConfig.ad_mob.publisher_id, mulberry.Device.type);
in it, and then we'd be duplicating the isHomeNode check.
In the JSON file, I also see many instances of type: "flag".  Is that somehow different from "boolean"?  Of the "flag" props I recognize, all seem to take standard true/false values too... seems like we should offer hints for them.
If you look through DOMAgent.js, you'll notice that none of the strings have been externalized yet. Probably not worth externalizing just this one.
This loop has 2 other problems:

1. doOpen() is only meant to open a single file because it automatically makes the file the current file, so this creates a race condition to see which of the multiple files is selected. Which is also unnecessary processing. Async.doSequentially fixes the race condition, but we still don't want to select every file in the list one at a time.

2. For files *inside* the current project, they are not added to the working set, which defeats the purpose of opening multiple files. It looks like you only opened 1 file.

So, another code path is needed that opens files and adds them to working set, and then selects 1 file (first or last?). We don't want to update the existing code path because single-clicking a file in project tree should *not* add it top the working set.
It would be much simpler to use:

```
  return (b.specificity - a.specificity);
```
This loop should be wrapped in an operation() to batching
Usually when I see this pattern , the text is also changed to white.
When I look at the About dialog in this branch, this line says "sprint 15 experimental build 0.15.0-0". It looks like the html template adds the version at the end. Wouldn't it be more clear for this string to be the following?

"sprint {VERSION_MINOR} experimental build {VERSION}"

Not sure if the "about build number" can be added as well.
I believe jQuery always returns [] even if nothing is matched by a jQuery method (so that further chained methods don't die by trying to execute on "undefined"), so `if ($title)` will never fail--you'd need to do `if ($title.length)`. Or you could just leave it out, since `$title.text(title)` will simply do nothing if nothing was matched.
Huh, that's a really good idea. The latest commit has this change: https://github.com/ajaxorg/cloud9/commit/bbcd3021c247545a992273041f6cbe7b1680f7de#diff-2
can i do this in sprint 2?
You are correct. I did not realize there was was a semantic difference between alloc, and the default constructor.
Please do forms with ruby code -- not html :(
Nit-pick. I am sure readercity is a great site - The rest of examples are with `localhost:9292`, could you please change this?
The idea is that if the objects supports serializable_hash, or is an array or hash of those objects, then we need to build a new array or hash of the results of the serializable_hash calls. Otherwise, we want the results of to_json as our final output. The comparison checks if the unmodified object has been returned, and is an equivalent to the final condition in the previous code. You're right that this is unclear, and I'll refactor it to have everything inside a single serialize_object method or similar.

It is hefty for deeply nested hashes, but how else can we ensure they are properly serialized?
Instantiating a page object should not load the page. We should provide another way to load the page given a token, and return the appropriate page object. Something like the PageProvider would work.
Why the change from trackEventMouseUp to trackEventClicked?
This should probably only happen once per `onDragged` execution, right? Looks like it goes `onDragged->for each draggable->update->checkScroll->for each draggable->updateRects`. Hopefully, it's easy to remove the nested loop there.
Remove this hover style since it isn't doing anything
Let's stick with uppercase-only for environment variables
anyway we could do this in a callback instead?
Bracket spacing (I think).
I had to include this for the statfs, since it only works with posix fs currently.
You are correct. I'll fix that.
Kvůli jednak "položky", jednak přílišné délce: musí se v okně vejít na jeden řádek.