labofsahil/github-event-dataset-2012 huggingface dataset:
Minor update to application's event aggregator

We've found the following addition very useful when using Marionette, and thought that it may be a nice add to the main project.

Currently it is easy to bind to the event aggregator so that it's awaiting an event called `save:images` but elsewhere in your app, you mistakenly have your trigger firing `save:image`. In large scale apps it would be nice for these "loner" triggers to be routed out quickly.

This update simply overrides the standard `trigger` method, so that it first checks the `_callbacks` object for the event that we attempting to trigger. If it can't find it, an error is thrown.
Add function to get value without placeholder [issue #30]

Adds a function to get the value out of the textbox without getting the placeholder value. Addresses issue #30.
Enhancements & Bug fixes

- Markers are now properly removed from the ```activeMarkers``` array when the ```removeAllMarkers``` function is called.
- All listeners are removed from a specific marker instance when it gets removed.
- The same handler function ```onMarkerClick``` is used as a listener to the click event of each markers instead of creating a new handler function for each.
- The html to be displayed in the info window is generated only before the window gets displayed instead of when the marker gets created.
Задержка появления превью
Add common mdpi and update
Haproxy module

completed 06 module
Change site.pp to master.pp

This commit changes the file site.pp to be master.pp.

This is being done so that other projects can simply copy in the site.pp
that they wish the master to use.
Missed files.

Oops.
hu strings
Improve 7 Strings, Add 2 Strings and I translate CTRL / Fn Keys

;)
Convert everything to use QNX build system

The following was also done:

A number of code changes were made to repress warnings from the compiler
Made JSON library a shared library between all extensions, use manifest to make it a dependency of your extension
Windows support is tested and working
Libraries have their symbols stripped for the smallest size
Native unit tests are working again
If no operator is specified, don't filter

Previously, "=" would be used as the operator instead. But if the user didn't specify an operator, it's safer to assume she doesn't want to filter at all.

This issue arose for me using with HTML `type="range"` input, which always submits a value (even if the user doesn't touch the input field--in which case the starting value is used). So, without doing anything, SonataAdmin would insist on only showing content objects who had that value = 0, forcing me to manually set the filter to ">=" on each page load.
added method to get last insert id

I needed this method in my project where I use your library, so I guess it would be handy to add it also to your code.
Lots of improvements

Settings page with ALL of the tidy settings
{tidy:options} tag where you could set settings per template
and some other small improvements
PLFM-1442: EntityBundle will return null ACL for Entities which inherit ACLs.
Adding myself as a supporter

Thanks!
add japanese locale file.

Hi,
I make japanese message file.
bug fixed: detection of device with the given serialId
Set CSS class on Body

These changes set a CSS class on the `body` tag. `stealth_on` and `stealth_off`. Thus everyone can style portions of their site according to the stealth mode.

I really need to practice on using git...
Added french translation

Hi,
i have translated your great plugin into french.
W3C Geolocation added to user account settings page (4rd attempt)

I added W3C Geolocation functionality to the user location map on the account settings page. This attempts to find a coarse location if the user has no home location set, so as to make setting the home location easier. If the location is already set, the map can be manually centered on the approx location by clicking a link on top of the map. When W3C geolocation fails (or is denied by the user), the page falls back to the old behaviour of showing a world map.
more fix kanjiclock

since we use this format :-)
add param number_format

<pre>
admingenerator_generator:
twig:
    date_format: 'Y-M-d'
    number_format:
        decimal: 2
        decimal_point: ","
        thousand_separator: ' '
</pre>

And also user mikroshka add some classes to templates for better css configuration
Rails 3.1 fixes

Please merge these changes (mostly openSUSE's Rails 3.1 fixes).

Alternatively, please add me (dlitz@dlitz.net) to the gem owner, and I'll upload a new version of migration_test_helper.
Add EPG NowNext
Moar fixes.

yarp.
Copy

Update strings.xml (Browser)
renamed packages so that they don't equal the vaadin-internal package and removed vaadin-JAR

As discussed on 
http://ondrej-kvasnovsky.blogspot.com/2011/08/how-to-add-vaadin-add-on-into-grails.html

When using same package-names as in Vaadin itself, there seems to be a problem when extending GrailsAwareApplicationServlet and registering the extending class as application-servlet.
So packages were renamed (to include grails in the name).

Another thing was removing the Vaadin-JAR from the plugin and adding Vaadin as a dependency instead.
printers_spa
Fix for issue# 8479.

> dmd and the spec do not match with regards to the list of named character entities that they support

rang and lang changed between HTML 4 and HTML 5. All the rest stayed the
same, hence why those two lines changed. There are plenty of additions
as well, but I left the table as it was (aside from the fix for rang and
lang) and just gave a link to the HTML 5 spec for the full list.
add notification for channel
Feature/project listing random 599

I think that's all for https://dev.imaginationforpeople.org/redmine/issues/599
Add Host IP to HostWaitMenu

The Host Wait Menu page now displays the hosts IP.
Implemented editing note.

Implemented issue #9.
New branch for 3.0 compatibility

Hi!

Thanks for your module. I hope my changes for SilverStripe 3.0 compatibility are correct. I’m not really a programmer, so I’ve not tackled all the TODO tasks you were mentioning.

Best regards,
Juan
Makes changes following the discussion with @rfk on #1

This implement what we discussed in the previous pull request for multiple application support.
Metadata version updated
Mute and FPS Counter

Sound can now be muted in game by pressing 'M'

FPS counter removed
Cross Translation Matrix

Ohh wow
Updated screenshots to Windows 8 RTM + bug fixing + other minor updates
Server 2 with capital S
should the game really be blocked from executing when rendering the effect?
Why not just delegate this to SubmissionEditHandler using the usual index.php entry?
` if event_code = options[:coupon_code].to_s.strip.downcase` will always be true, is correct ?
Раскрыл Parser<token,ast> в ITokenLexer<token> -> Result<ast,token>
W00t, awesome!
Bruk resourcebundle
s/ought to/needs to be. Just looked at the source and remember it does a lot more
nope: cf line 29 of settings.py.
My FallbackConfigParser doesn't do that.
Maybe some comment, describing this new mode of li(), like replace normal lui/ori pair with a LoadRoot() where that is feasible.
SPRING_DATADIR needs to support multiple paths, like the windows PATHS environment var

For example:
PATHS = C:\blarg;D:\blee;C:\yomomma;
Extrapolated ->
SPRING_DATADIR = %USER%\spring\Evolution-RTS;%USER%\spring\Gundam-RTS;%USER%\spring\Zero-K;

And so forth. The biggest issues is the fact that you guys keep removing control of spring's data dir from Content devs. In the process, you end up screwing us over.

By default, spring should look no further than it's current directory ( %CD% ). However, via environment vars or ini or however you like, we should be able to tell spring to look in any number of locations.
hand.gametype['base'] will either be "hold", "draw", or "stud". We should probably just say 'if hand.gametype['base']=="hold":' in this case & throughout.
The code style should be consistent, and our function style is like 
private void f () {
}
@giannis Can you see what impact (if any) this has on this branch: https://github.com/ajaxorg/cloud9/pull/1024

I was working on "search and replace in all files" and did away with the drop-down menu. Not sure if this affects your code or not.
The patch has been updated here:
https://github.com/Bertram25/mana/commit/cebe59b9ce9e026d64953ddfc5454f822037186e

(The hairDb instantiation and naming is left to be discussed.)
I think I like it :)

but I dislike that this way the response object doesn't follow the RAII principle... Would it be possible to leverage shared pointers here and thus get rid of the requirement to call void close() ?
It's a CMAKE :o
And also a trailing $ on the last one.
I was late for the discussion.
I think that consistent is too common word.
And I'm sure just now too that validates_with_method is required two values when return false.
How agly!
http://datamapper.org/docs/validations.html
Для доступа к ключам массива лучше использовать такой код $_SERVER['SERVER_NAME'] - использовать кавычки.
It used to be Post.

Also, in the brave new world of IndexMixin, we can have models register themselves and do more of this automatically rather that what we've got now.
Bonjour Regis, 
Je voie qu'effectivment je passe des paramètres dans le tableau qui 
sont déja $objet, je corrige. Alors juste htmlname dans parameters, ca 
passe ?

Le mer. 07 mars 2012 12:20:37 CET, Regis Houssin a écrit :
> bonjour florian
> $object est déjà passé en paramètre dans la fonction executeHooks
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/FHenry/dolibarr/commit/0b9b92dedd0f4af3e3f21f6e105299ce9cbd4480#commitcomment-1052827
compile on VC 2010, we have no support for vc2008 and i hope for compile on vc2009 :)
It should be localized
多余空格去掉。
Deleted event_gunship.cpp?
esto que soluciona
I thought about that but wasn't sure it would help. I'll try it though.
if Josh recalls correctly, the changes needed to make the plots actually populate happened over in the ReviewTab repo, rather than here in the main repo. we believe (nearly certain) this is the last commit on the abmplus branch that actually had to do with the review tab.
@Th0ar 有空把 'a', 'b' 去掉~
Okay krass, kann ich mir aber vorstellen, allerdings ist die Reihenfolge bis jetzt auch egal. (Außer für die tests:)"
oh...sry, is mir garne aufgefallen
Sorry - I got a bit confused.

mlist is something that has 'ids' in its first column?

These ids are sometimes zero or negative meaning they are - ? not present or not frames or something

Did I understand correctly that the algorithm is the same as the one below?

<pre>
ids = mlist[:, 0]
valid_ids = ids[ids > 0]
valid_order = np.argsort(valid_ids)
for i, id_no in enumerate(valid_ids):
     id_dict[i] = [valid_order[i], id_no]
</pre>

So the dictionary will have keys of ints = `range(len(valid_ids))` (so maybe less than mlist.shape[0]) ?
Probably, but i wanted to quickly way to make it work.
`install_github` ignores `branch` ...
I think original name is better than using local name. As most users is familiar with the US name of the movie.
I.e. Mästerkatten I bet you don't know what that is. But Puss In Boots you know.
Git Github... 全局搜索替换嘛
а ну CheckGOInpath тоже стоит на 1.
Там по сути то пашет, но если под угол встать, то уже не пашет. Напишу.
`required` is the default
disregard this - I think I'm good
Alright, then everything should be fine.
I'm not speaking english well, so please fix the name of variable if not good.
Cool. i will commit it!

-Vivek
The above change is a little suspicious to me. Could you, @UrsKR, verify that this will not cause any problems. I have tested Anathema with what I have found and did not find any problem.
maybe you could add a test for `from_email` as well?
i face some issues on CI 2.0.. about hosting problem.. i need some one help.. this is my first project developed using CI. so i face index.php issues in base URL.. ple help me...
Ok, I'll try to merge our `dev` branches ;)
Great! I have just commited fixes following your advice.
Seems we don't and they'll be kept for `apps` only. I'll wrap the markup in a template condition.
I would change this to single quotes and make the comment end with a period.
tnx kaelima
https://github.com/TrinityCore/TrinityCore/issues/90#issuecomment-2502613
also the rocket packs are never removed, however i see the code for that in case ACTION_DONE: _instance->DoCastSpellOnPlayers(SPELL_REMOVE_ROCKET_PACK);
voire mieux : 

int conversionIntToEPosition(int entier) {
return (entier >= 1 && entier <= 24) ?entier - 1 : 0;
}
That Firefox bug has since been fixed though I don't what version fixed it. What version are you using?
logic that uses this is in the JavaUrlHttpCommandExecutorService
Why not use EventEmitter2 and return that as the result of start.
You will not need to handle all addListner/removeListener crap yourself.

then instead of rebus1.subscribe you will have the regulr rebus.on('aaa.bbb.cc', function handler(args) { ... });
and for you will also have removeHandler and such ...

I think it will simplify this module a bit more.
This is realy weird
I got the same problem some times ago running the test suit on postgres.
https://github.com/doctrine/doctrine2/pull/194
@raykrueger Help me out here - do the following lines in resque-lock accomplish the same goal?

https://github.com/defunkt/resque-lock/blob/22f276b5e1cd322865d8db95d5c8834d40b7fac2/lib/resque/plugins/lock.rb#L58-62

Or is the on_failure_lock added for the purpose of sending exceptions like this one into the locksmith failure backend specifically?
s/don't use/ain't using/
Don’t worry about the travis build failure email, we haven’t had it pass once yet… :/
Definitely an improvement!  I wonder how hard it'd be to get the message to render in the normal site template rather than just as plain text.

Maybe it could do something like (similar to what pages.views.PageRenameView is doing):

messages.add_message(self.request, messages.ERROR, self.forbidden_message_anonymous)
return HttpResponseForbidden(self.get_success_url())
Just noticed this, you forgot to tag me
It totally should. Can you patch it please?
needs to be size_t, see countDescendants
I agree here, the multiple inheritance for testApp was surprising, I don't feel this is a good solution. A separate object would be better imHo.
Yep :-) Thank you :-)
Fixes the issue with dot files not loading automatically on startup.
In fact it never changes
Should only allow 'm' -- SMART heights must be in meters.
Nice pickup, applied here a8f09793ae4ce257f38df50b6e23f60c887e4a0a
I've fixed the default for unknown types, culminating in this commit:

https://github.com/Jermolene/TiddlyWiki5/commit/26b5fc4a6d2c9f6aacb0d11bfba3a334aeca3fc2

I don't like the look of the three backticks together, but do appreciate that '$$$' isn't ideal.
This should have been put in a separate branch... gah.
I'm using the download task on its own from an extension, so it needs to update the resource in this context, but that's fair enough I can check and update the resource if necessary from the extension itself.
But with this changeset the 'hash' property is not set on the resource any more, making this test fail:

https://github.com/okfn/ckanext-archiver/blob/master/tests/test_archiver.py#L171

Has the 'hash' property been deprecated?
FYI, I changed this back in the next commit. Sorry about that, folks!
@mike1072 tests seem to break here. @lekksi: Is the Agilefant repo hooked to the CI server?
autopatching 'model' should be 'anchor'
comment by Autor:
once there was also an issue when the whole site went to clean the cache.
It turned out that was not initialized property in the $ modx base_path (?).
In the snippet is this line:
$ path_to_cache = $ modx-> config ['base_path']. «assets / plugins / cacheaccelerator / cache /»;

In general, I did not understand thoroughly and then forced to set the full path to the cache. Otherwise, the class filecache considered the root of the root directory cache site))) Well, cleaning it))
It is also not clear how it gets to the root, and falls out with an error.
LOL. Nice ASCII art.
헛... 병합해버렸네... 괜찮을려나?

개발 및 운영서버 설정파일은 깃헙으로 형상관리 안하고 서버에 두려고 일부러 안 올렸는데.. ㅎㅎ;;

maven profiles 설정에 보면 어떻게 하려고 했는지 보일게얌!
It seems like you broke the N2x0 build. Will it build properly this change?
> Hm. Reading through this again, I don't think I made some sense, did 
I? ;)

Yeah, its alright now, as I did and do see the need to switch to a fast 
medium in a case like that. That is why I noted to think of the others 
and sum-up what happened, or describe the solution right afterwards.
And I just wanted to keep on track, that's why I asked! I definitely did 
not want to forbid networking like that.
So please take it just a small and friendly note to us all.
Glo-grue: Once a fierce predator of the dark, this irradiated creature glows brighter than a flame, begging the moths to come closer.
does this happen if the server doesn't have a default world setup properly for global usage?
Can you pull this & initParser up?
I notice you have stylesheets, but you don't include them in your html file. Why?
Il manque la réference, (le &)
i love you guys too :D
codestyle:

-- fix summon charger for horde
UPDATE `npc_trainer` SET `spell`=34767 WHERE `spell`=34766;
DELETE FROM `spell_required` WHERE `spell_id`=34767 AND `req_spell`=33391;
INSERT INTO `spell_required` (`spell_id`, `req_spell`) VALUES 
(34767, 33391);

-- fix summon charger for alliance
UPDATE `npc_trainer` SET `spell`=23214 WHERE `spell`=23215;
DELETE FROM `spell_required` WHERE `spell_id`=23214 AND `req_spell`=33391;
INSERT INTO `spell_required` (`spell_id`, `req_spell`) VALUES 
(23214, 33391);

-- fix dreadsteed for warlocks
UPDATE `npc_trainer` SET `spell`=23161, `reqskill`=0, `reqskillvalue`=0 WHERE `spell`=23160;
DELETE FROM `spell_required` WHERE `spell_id`=23161 AND `req_spell`=33391;
INSERT INTO `spell_required` (`spell_id`, `req_spell`) VALUES 
(23161, 33391);
That's a valid activity ID isn't it? What was wrong with 0?
I think this one should use the preference
Well... seems my changing the line endings made git freak out a little...
To fix the alignment just run this:

lcd1=i2c mw 40 00 00; i2c mw 40 04 80; i2c mw 40 0d 05

Instead of the full lcd1
I hope you didn't try to do it the easy way, and just implement it like LeeDroid.
Could you double check your keywords solution? Search seems to be broken for me now too...
Yar, can you do a check for env variable and/or determine something like this?

XCODEBUILD_PATH_43 ="/Applications/Xcode.app/Contents/Developer/usr/bin"

if [-d XC4_3BP]
    XCODEBUILD_PATH=$XCODEBUILD_PATH_43;
fi
I'll just use this variable in the `if` statement
Changes to htcleo_defconfig and devices-qsd8x50.c can be ignored when applying this to other repos
Rad 25-26 borde inte fungera längre eftersom $dsn är omgjord
EEPROM Boot up time: 6:51 AM
Boot up time: 6:54 AM
Launch Time: 7:10 AM
Power off time: ~11:25 AM
You ought to define different attribute values for differentiating "forms that add workouts" from "forms that add exercises". Remember that this metadata is intended to help a client understand what different controls do, and these two kinds of forms have very different effects.
find ich ne gute idee, schreib ich heute oder morgen mal
I'm using version 5.6, and am including the bootstrap-image-gallery.min.js file hosted at blueimp.github.com.  After this commit the image gallery breaks.  Chrome shows an error "Cannot read property 'prototype' of undefined from the bootstrap-image-gallery-min.js file hosted on your github domain.  As a result, the application.js file throws an error saying 'Object [object Object] has no method 'imagegallery'.  Was the image gallery intentionally removed from application.js?
this function has been added because when the "datacenter ChartBuilder" class was trying to set a dataset as a line to a specific Y axis, there was a param duplicated inside xml file (parentYAxis). So this verifying has been wrote to avoid this problem, allowing adding the attribute only if it does not exists.
this was just some mistake
Uploading all these config files to github is a bad idea, as they contain sensitive data.
Во сущности картинки этих данных нету, так как они не подтягиваются с БД.
Добавил передачу альбома во при просмотре картинки.
Правильный вывод:

{if $oAlbum}
{$oAlbum->getTitle()|escape:'html'}
{/if}
Esto es **MUY** pro para mi papá
Yeah, this is pretty much horrible. We can't use the poundpay gem with rails >= 3.1. Please merge!
Fixed in a new commit
I mean you don't close the DB in `handleRequestFailure` or `handleRequestError`, so if the `GET` fails the DB is never closed.

But my first question was: you're opening the DB in init, and closing it here. That's slightly confusing, because you make *two* HTTP requests. One has to read carefully to notice that the delegate for one is `this` and the other is `clientUploadDelegate`. At the very least you ought to cluster the `GET`-related requests together; you should probably also consider putting some of this logic in a separate method or two, and calling it from this generic-looking HTTP handler.
Some questions on this: 
* What is the "name" parameter that you're showing here and why is it being passed? Is that account name? Profile name?
* For the first code snippet (including the generation of a guid), is that meant to take place in SyncClientsEngineStage or is guid generation to happen in the GlobalSession constructor?
I think these should be up together with the original definition of created, and updated. To have it two places is confusing. I think we need to make this hand-made, and remove the auto-generated lines for this.
maybe mention Telephone.app for devs with a PBX (like us)?
Somehow I got it work, your code is correct. Thanks.
This line looks like it always overwrites the else if branches inside the loop, so the result will always either be 1.0f, or whatever value score was before the loop.
Now that the "Download" button is on the main page we only need one
that was the only thing i found, since you removed action_admin_init it was throwing up notice and sorry about new lines :) my laptop Komodo IDE had that "trim trailing whitespaces and EOLs".
done now: editorconfig/editorconfig-gedit@3a55181c28305f25d4daca620bd6e309141ba3a8
Superflous  `I`.
Sure, can be done with Shrinkwrap quite easily I believe. I will open JIRA for it. Thx for this suggestion.
Nope! Removed this on the develop branch.
I think taking in the `Class<T>` is cleaner and more standard in Java.
Can you try again? I have added even more details, and possibly a fix as I have a test which produced the same error.
Note the all the tests have @Ignore because the logic in the handlers need to be updated for them to pass, except for this address one that passes for other versions except 1.6
Yeah, I was thinking that as well.
I haven't actually tried building ics yet, but I've been running imnut's ROM on the MTD kernel for a bit. 
I was kind of surprised the nvdata stuff worked. The nvstorage kernel module has some ioctl stuff; I'm not sure what it actually does. I'm thinking maybe write protection?
Is not owned by Niklas Hofer, so can’t be present here in such fashion.
@ZyX-I did you see this one as well?
https://github.com/TwistedUmbrella/android_hardware_libhardware_legacy/commit/ae5ea2ad2e6686b442ebb5a0f025758e289f9027
Just a matter of making sure the original format wasn't lost in the transfer from bluedroid (:) to wifi (+)
I think I may have failed to explain myself clearly.  I'd like to move the discussion of Y.soon here: http://yuilibrary.com/forum/viewtopic.php?f=18&t=10636
My point was about the message itself.
I think it should never crash even in this odd use case (no mapping for ```BadCredentialsAuthenticationException.CODE```).
So this function is called from python.  I'm not sure I can do pointer arithmetic on that side.  Let me check
This name is a little misleading. It populates the lists of the objects in place, it doesn't return a whole new DAG object.
lots of repetition here, can we improve on that?
I started down that path but migration manager flagged that things like
copy() and unapply would no longer be compatible. I don't think any client
code should be doing that, but who knows. Should I go ahead and ignore
those?

Thanks
sRp

On May 8, 2012 3:28 PM, "viktorklang" <
reply@reply.github.com>
wrote:
>
> >        val socket = SocketHandle(socketOwner, ioManager)
> > -      ioManager ! Accept(socket, this)
> > +      ioManager ! AcceptWithOptions(socket, this, options)
>
> You don't need to do AcceptWithOptions, just instead of having the empty
Seq as a default-parameter, create a new constructor that has 2 parameters
and that delegates the empty seq to the 3-param constructor
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/akka/akka/pull/438/files#r790997
@rkuhn I'm trying this out, your definition:

    val decayRate = 1.4427 / halfLife
    val alpha = math.exp(-decayRate * collectInterval)
    avg = alpha * avg + (1 - alpha) * input

Your alpha = 1.0 - α   (α in the old sense)
Do you mind if I continue with the old definition of alpha, since that is what is documented at wikipedia?
http://en.wikipedia.org/wiki/Moving_average#Exponential_moving_average
Reason is simply to be able to point to the description.
Otherwise, do you have a better pointer?

In other words

    val decayRate = 1.4427 / halfLife
    val alpha = 1 - math.exp(-decayRate * collectInterval)
    avg = alpha * input + (1 - alpha) * avg
So this is jus t way to make the configuration of the build and tests from Jenkins more manageable and readable. We currently use properties to control a number of things in the build, and this allows us to read them in from a file as well as from the command line.

We also use properties to _communicate_ with the sbt-multi-jvm plugin, if we don't want to pull in the config library there as well.
and postRestart by default calls preStart.
yes, that would look better, thanks
because it was not documented that the result shall be lazy. done that now, also for `children`
Why can't you do:

    case class ConsistentHashableEnvelope(message: Any, hash: Any)

Same as we do in Broadcast(..). 
Seems tedious to have to define an envelope for each new message you want to hash.
`private[this] val refs` and `refs` generates same byte code as far as I can see.
It doesn't look like we use `private[this]` in many other places in akka.

`private val` generates something else (less efficient).

The naming convention with `_weights` because it's only used in constructor is not something I like, because that is seen from the outside (named parameters), but if that is how we always do it I will follow it.
perhaps the solution is to only ever send RDY 1 when you are in a backoff state until you completely recover (this simplifies all of the logic you're referring to)
This seems to work better that I expected.  Initially, I had an attribute *obj to hold the content Object, but now the Content is a Delegator.  It doesn't care what the type of the underlying object is, it just delegates all methods to it.  It seems  I just need something extra to access the class of the delegator.
Cleaned this up when I moved comment handling up to parse time for the transclude.  That was also an opportunity to streamline this code.  Now an instance method that sets @configs and @options directly and just returns the name.
I guess the template needs it as @subject?  But args[:subject] will feed the header, right?  Later the args handling is different.  Defer fixing 'til that's all merged.
Per discussion in IRC, the following is more accurate:
`Numpy array of values as input to a survival function for the given distribution.`
The following ones refer to a existing subscription. Unfortunately do browsers not sufficiently support HTTP, so I simply added the functionality as a part of the resource (./edit, ./mark_as_seen, ./delete) which is, however, not as restful as I' wish. One could argue that the subscription list resource (/subscription) should take control when to kill a subscription and not to let the subscription commit suicide,but that's just a discussion in order to deal with workarounds when having a not-restful clients like browsers.

On the other hand check_name is simply trying to figuring out whether a subscription name is vacant or not. So, there is actually no resource /subscription/{subscription_name} onto which one could perform that operation.

One could again argue that here a 404 Not Found might be appropriate for the JavaScript to determine whether the name is already taken or not. However, in case the name is taken, too much stuff would be loaded for nothing just to tell the script that the name is taken.
Yeah, this is the other place, should be the same code that decides on the submenu presence.  I think we need to address the authorized user's permission more generally so that it would be included by the account trait access.
The curly brace in our Scala code is usually at the end of the previous line. Maybe the code could be formatted better.

(I had to read the documentation of foldLeft to be sure...)
I think the idiomatic way to do it is initialize with _
Alternate is:

int updateRA = (message.GetSenderId() == GetID()) ? message.GetParam2() : (Video | Audio | Totals);
// update as per usual
right ok, we could change this.
я думал что генерация имени может измениться, поетому винес в отдельный метод
you have an extra empty line here
So I just ran this, and i was staring at the following:


artem@dev13:~/pg/emrio (master)$ python emrio/EMRio.py 
None


with no clue as to what is going on for 2 min until a report was spewed out.

Lets add some status lines like:

"fetching 2 months of EMR history from Boto.."
"..simulating.. cost"
"formating report.."

just so I know what i'm waiting for :)
Actually, why is `mMap` a member variable? Same question for `mRenderer`. I think they should just be local variables in this function.
Go up 5 mm after homing. Not sure how to check for different printer which need -5mm instead?
Indentation is weird across these couple lines too.
My logic here was that `TUITextRenderer` was the engine, and would not be directly accessed by the programmer, as `TUITextView` and `TUITextField` exist. This would have the more precise fine-tuned settings, and so I left it as `shouldRefuseFirstResponder`, in the event that the text engine would grow to do more with the renderer.
Yes to both. Teaches me for trying to be lazy with dictionary keys :p.
Incorrect indentation here and below.
I think @piscisaureus already pointed it out but these should go into SHARED_INTERMEDIATE_DIR.
Better, but why not create the array in ConvertList and return it?  That would eliminate another duplicate line.
Use startsWith instead of contains here
Will update ``_get_argspec`` so that ``defaults`` is always a list of length ``ndefaults`` so this logic will be removed.
`(int)` cast seems unnecessary here. (Also the else clause can be dropped.)
We can pass "args.logEmitter" and "args.parentEvent". I prefer lesser no of parameters.
звездочка
Это класс или структура, а они начинаются с заглавной.
это все заменяется одной строкой isCircle  = ((newSrc == newDst) && newSrc);
и, кстати, circle (который "петля") надо везде заменить на loop
Thanks - sorry I didn't get to this in time, busy with client work :)
String interpolation ftw!
I'd say this is obvious, so remove the assert?
factorisation->factorization
```
if len(args_list) < 2:
    return function
```
Ah, wait, my example was dumb. I don't think that this works.

```
In [2]: def foo(): 
   ...:     if False:
   ...:         yield 5
   ...:         
In [3]: q = foo()
In [4]: not q
Out[4]: False
```
It seems that it is especially meaningful: if Bool1*Bool2 == Bool2*Bool1 then Bool is commutative, isn't it?
The `order` argument was originally (I mean, currently in `master`) unused in `Basic`.  I wonder whether that's a typo.

> When you do that, keys will be order-sensitive:

Oh, I see.  Thanks for explanation!
linmom -> linearmomentum
`$this->_totalItems` would do it here too.
blank line required after return.
Same thing as previous comment. This will crash us since _platform will not be nil, but it will be a destructed value since you've given it an autoreleased assignment. 

do this:
```
_platform = [[NSString stringWithCString:machine encoding:NSUTF8StringEncoding] retain];
```

or this:

```
_platform = [[NSString alloc] initWithCString:machine encoding:NSUTF8StringEncoding];
```
since an alloc'd object has a retain count of 1. Unless autoreleased.
Need to fix indentation here too.
only newlines where needed and then use proper indentation.
$s = $s || \Core\Service\ValidationWrapper::hasFailed();
Ah, I see it in the test...
Maybe `indexJsonAction`? I think it's a good convention to follow.
Parenthesis not needed.
add execution examples:
        \bbfs>ruby -Ilib examples\tcp_example.rb  # to run with default parameters
        \bbfs>ruby -Ilib examples\tcp_example.rb --log_debug_level=3  # to see more log details
The arguments are reversed:
irb(main):007:0> File.join("tafty", Dir.tmpdir)
=> "tafty/var/folders/mt/q14d80rs3vg5xv2lyk5hy1p80000gp/T"
I don't like javawebscriptmaster.
However, physical filenames we generated should not be leaked to uploaders/downloaders either.
Yes, password reset
Run nginx or lighttpd instead of apache.
Make sure you use the array keys and use isset.
Should probably use the `sql_list_tables` method in `phpbb_db_tools`
I have to take care of that.
Shouldn't combining these be a better approach like I did with mysql plugin?
Do you mean I should use the reader instead of @prs in the edges
method or I should access it via prs in the spec and eliminate the
edges method entirely?

   ...scott

inter prospera et adversa and dragons never quit



On Mon, Apr 9, 2012 at 5:25 AM, Randy Morgan
<reply@reply.github.com>
wrote:
>> @@ -35,6 +35,13 @@ def initialize(options={})
>>        end
>>      end
>>
>> +    # Return list of edge names applied to this border
>> +    def edges
>
> This should be pulled off the prs readable attribute.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/randym/axlsx/pull/77/files#r659543
注册放在首页了，这个文件可以清理
Yep, that would be much preferable.
Also on Windows you need to add the folder containing OpenCV DLLs to PATH, i.e. put the line

    set path=${EXECUTABLE_OUTPUT_PATH};%path%

before

    nant -DjavaLibraryPath=${EXECUTABLE_OUTPUT_PATH}/Release buildAndTest
- Should be wrapped in curley braces
- Use this overload of the ArgumentNullException constructor http://msdn.microsoft.com/en-us/library/k8a0dfcy.aspx
Ticket: RAXAJSS-355
Review all new models / stores .. am I using them? I tried several but ultimately didn't end up using (except for observation). Right? So remove them from pull.
There is already a daemon user. Can you try that??
I think it doesn't have adequate rights to create new privileges, but we can use a PKI, where the server will know a valid client.
`` ` ` `` => `repr()` ... agreed. (If for no other reason than embedding a back-tick in a comment is a pain in the neck!)
Does this just test if randomFill() and migrateTable() throw?
targetTupleToUpdate should go before sourceTupleWithNewValues?
Drop this obsolete comment.
I don't disagree with your point. But this code would need more major changes since both verb_args and verb_opts are mutated here after retrieval. Let me know if you think it's important. I'd have to make copies or something...
This implicitly imports other models thus we cannot use it here. Read below for explanation.
use constants from Choices definitions
shouldReturnJobRequestContext or returnsJobRequestContext
it was a typo already fixed to job ads
But the models that are deleted are from a set list, and none of them are through models for M2M fields, so that will never be the case, or at least as far as I can tell.
Can we validate that verifyHost has been set to 0, 1, or 2 and if not, throw an exception?
Flip the quotes around: single inside double
If "sourceTable" or "attrColumn" does not exist, this line will be hit. 

Currently, we do not have the utilities of table_exist and column_exist. So I used this not so elegant way.
And if so, the right check is AggCheckCallContext.
weird comment to have in our lib code, probably better to remove it
Good point.   I will try to re-write the instruction.
Do you have a way that we should switch this instead? We could add yet another flag in the settings, but we seem to have a lot of them.
^Return subreddits whose display name contains the query.
No, tohle je testovani knihovny, ne webove aplikace.  Tady testuji, ze se ty funkce chovaji tak, jak ocekavam. Pokud chces testovat opravdu ten samotny web, tak potrebujes neco jako Selenium. A diky temhle testum jsem nasel spoustu bugu v mem kodu. Unit testy jsou hlavne na hledani regresi.  Samozrejme, ze se ti zda, ze tyhle testy jsou trivialni, ale jakmile po nejake dobe budes chtit udelat zmenu v jedne funkci, tak uz si nebudes pamatovat, kde vsude se vola a co presne ocekavas, ze vraci.
if `config` is present here it should also be present in the Base Store since all the implementations should be API compatible, and the API should be enforced by the Base Store.
I typically use hash.delete when i'm passing options to a method that contain options for more than one thing. It lets you pass the remainder of the hash to something else:

```ruby
options = { first_name: 'justin', last_name: 'mazzi', shoe_size: 12 }

class Sneaker
  def initialize(options = {})
    @first_name = options.delete :first_name
    @last_name  = options.delete :last_name
    @options    = options # Which in this case would only containt show_size
  end
end
For state I'd use absent and present to be consistent along all types
Ok, I agree readabilty matter!
new RegExp() and associated patterns are generally [pretty slow](http://jsperf.com/slashes-versus-new-regexp) compared to the speed you get with slashes. I don't think we need to be too worried about this, but we should construct the regexp outside of the for loop here.
any reason we are using `authorization` here instead of self?
I forgot about this before, but am remembering now: we'd wanted to alternate darkness on the child nodes. (So, make the 2n ones have a darker background color—say darken 15%.) At that point, I think it should be pretty clear on device.
if `this.media` isn't set by now, it would have failed at line 33, so this should be moved up before that. Also,

    this.media = this.media || { 'poster' : '' };
Done. In general, the tests no longer share any menus, menuitems, or commands.

In this case, I combined all of the "should add new menu item..." tests.
We might not want to make this an official public API yet -- not sure we have time to think through exactly what the right API is that will serve all the various use cases (e.g. warning markers, etc.).  For now it seems ok to have the color editor code just reference _codeMirror directly.
This is used in Replace command. Can you try it (Ctrl/Cmd-H) and see the translation of with is correct in that context? You should see Replace {TextToSearch} With {TextToReplace}.
Would be good to document these consts.  E.g. what case does each one refer to?  Is it a multiplier, an addition, or what?
Oh I see... didn't notice that...

I'm closing this pull request then and will submit a new one following your directions soon
One more thing... I imagine we eventually want a way to dynamically add menu sections, but I feel I need to limit the scope of the work for this sprint, so I opted for statically defined constants for now.

-----Original Message-----
From: Narciso Jaramillo [mailto:reply@reply.github.com] 
Sent: Wednesday, May 23, 2012 11:07 AM
To: Ty Voliter
Subject: Re: [brackets] Menu, MenuItem, key bindings, and Command API (review only) (#927)

> +
> +    /**
> +     * Adds a new menu item with the specified id and display text. The insertion position is
> +     * specified via the relativeID and position arguments which describe a position 
> +     * relative to another MenuItem or MenuGroup. 
> +     *
> +     * Note, keyBindings are bound to Command objects not MenuItems. The provided keyBindings
> +     *      will be bound to the supplied Command object rather than the MenuItem.
> +     * 
> +     * @param {!string} name - display text for menu item. "---" creates a menu divider
> +     * @param {!string} id
> +     * @param {?(string | Command)} command - the command the menu will execute.
> +     * @param {?(string | Array.<{key: string, platform: string)}>}  keyBindings - register one
> +     *      one or more key bindings to associate with the supplied command.
> +     * @param {?string} relativeID - id of menuItem or menuItemGroup the new menuItem 
> +     *      will be positioned relative to.

How are menu item groups defined and positioned? I don't see any API functions for them here.


---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/927/files#r869787
The TODO comment got left in here...
Oh, right, that was actually what I was planning to do. I suppose I forgot, but I'll add it. Now if we only had some kind of wildcard matching like Stratego's `[FArg(farg) | _]` mechanism...
Try to remove any whitespace formatting changes from the main diff and add as either a first or last commit.
I'm interested in pulling this in, but I don't like messing with the Gemfile. Are these changes really needed?
Surrounding the button with <h3> was a hack to get it to align properly with the text to the left of it
These additions should not be in your pull request right?
Some people prefer to have their methods return right away if null conditions are met because it can reduce nesting.
Ex:
`if(!angleDTO.Files.Any()) return null;
rest of your method here, not nested in an if or else`

It's strictly a readability thing and I'm not even sure how I feel about it, but I figured I'd mention it in case you like it.
Thanks for your hard work, Adam.
源地址存个变量吧.
```javascript
var srcPath = [src, name].join(sep)
  , isDir = fs.statSync(srcPath).isDirectory()
copyFile(srcPath, isDir ? dst + name + sep : dst)
```
If you call `postAJAXLoad()` with a form (jquery) element, all inputs are reset. 

That's only applicable for forms for adding stuff where you want the inputs to go back to being blank after the addition.
Nit: Consider using @example.com email addresses in tests in case you end up sending some emails accidentally.
I don't see this as any different from 'success'. A successful validation will result in a redirect. If the tester wants to validate that the redirect occurs, then they just need to instantiate the expected page object with a suitable wait.
you know, there should probably not be a wait in those places at all...after all, someone might want to test that we notice if you give mismatched passwords.
That would be an awesome idea.
Alright I'll file a follow up ticket in 0.6 then, thanks.
Can you not just put this code in `start` too?
These do have to exist separately since they aren't in the main layout (the primary and alt media divs are copied and inserted as needed)
Is this object likely to grow at any point? As things stand, seems a bit redundant to first declare the variable, then assign the variable, then return an object literal with a single item for that variable. Of course, if you're expecting this to change, this might be moot.
I don't see a reason for object creation here, we can declare getFileName as a normal js function
The `FirefoxCom.requestSync(` statement below shall have same exact logic: `this.initialize` and promise resolution
remove this line, instead of comment out.
Sure.  What would you prefer? `juggle'`? `juggle-main`? Calling `partial` twice?
You can just make it a default param:
```scala
def apply(testMode: Boolean = false)
```
rasti chera aslan  min-height dadi ?!
you need to clear the trailing bits after the end of the bitarray, i.e. add special code for the last chunk you're writing (using a mask). That's an assumption made throughout the bitarray code, not doing it breaks stuff.
Yes, this was the fix for the logout stuff that was missing :)
I'm assuming this is intentional?
Any reason why it's not with the other env vars few lines below and allowing the user to provide it's own value?:

```bash
KATELLO_HOME=${KATELLO_HOME:-/usr/share/katello}
KATELLO_ENV=${KATELLO_ENV:-production}
```
Just a style nit, but let's remove all the leading whitespace in between the var name and value in this let binding.
@stard0g101 it's not really your bad. When close() is called the handler should be destroyed and not assumed to be usable anymore, much like calling fclose() on a file handler, or closing a db connection. It's not *needed* to clean up because in most cases close is just called at the end of the request, but you can clean up references to make sure stuff gets garbage collected ASAP.
Hm, utility function which wraps original string and uses correct encoded based on the Python version should probably live in `libcloud.utils.py3`.
This example is no longer true and should be removed.
Will do.  Just out of curiosity where is it happening that `_ASTROPY_SETUP_` gets injected into the module namespace?
It looks like only this process ever touches this table, so maybe it should be `private` instead of `public`. Or at least `protected` if you want to read it out-of-band for debugging, but still protect against concurrent writes.
Just for my own education: is this conditional more pythonic than `if not username:` (the latter being shorter and accepting also False as "default")?
Please don't change the qt version without updating the gitian win32 scripts, I'd prefer to do this in a different pull.
Yeah I know the advantages, I just think it is unreadable :(

On 24/03/2012, at 20:46, Mark Story
<reply@reply.github.com>
wrote:

>> @@ -607,7 +607,12 @@ public function testItemEnclosureLength() {
>>        $File = new File($tmpFile, true);
>>
>>        $this->assertTrue($File->write('123'), 'Could not write to ' . $tmpFile);
>> -        clearstatcache(true, $tmpFile);
>> +
>> +        if (50300 <= PHP_VERSION_ID) {
>
> It does save you from accidentally doing assignment when you want equality.  `null = $foo` is an error, which is one feature of yoda conditions.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/cakephp/cakephp/pull/579/files#r600663
Thanks for that @fredj. We don't want to the sandboxes and GeoExt user extensions to diverge from their counterparts in the GeoExt SVN repo. The sandbox and GeoExt.ux dirs should just be copies, and updating them from upstream should always be straightforward.
What made you want the tree node to be expanded? Any particular reason?
why making it abstract and then extending it with an empty class ?
Btw, you should probably add an interface
search/replace detected :)
I think all four of these should probably be unicode literals - i.e. `u"u = {u}'héllo'"`. 8-bit strings with non-ascii characters are a bit of a headache.
This isn't the correct regular expression for Python variables. They can't start with numbers. 

There's also Python 3 Unicode variables, but I don't think anyone will blame you if you don't support those :)
in your next pull, see if you can get rid of one of these types with \@XmlElementWrapper
Is it necessary to change the order of inclusion? If so, let's write a comment.
I *think* this is more correct than it was. Now `git_reference_name_to_id` will return ENOTFOUND if the ref isn't well-formed, rather than a generic -1. What do you think?
try libssh2-1-dev instead
extractRequires is actually a public method
ok transblocks removed
Nit: it seems to be an emerging Firefox convention to include a trailing comma at the end of an object literal. The archaic JavaScript strict interpreter in SpiderMonkey used to warn about it, since it wasn't legal ECMAScript, but that is changing (or has already changed?). And including a trailing comma means that the next time you add a property to the object, you don't need to change the last line of the previous property declaration.
s/False/True :)
https should be preferred.
using $this->_helper->json(array("success" => true)); is better that just exit; in this case.
also corrected in portletFeedSaveAction
nit: something like \W at the beginning as to not match say `text/plain; notthecharset=ascii`.