labofsahil/github-event-dataset-2012 huggingface dataset:
Проверка удаления постов
Изменение скрипта

Здесь включены почти все мои исправления за последнее время (те, которых здесь нет просто пока выглядят отвратительно (и в основном касаются ютуб-эмбеддера), поэтому я их отправлю немного позже). Ах да, здесь нет исправления поддержки http://voile.gensokyo.org/ - я сделал с помощью костылей, но мне кажется, что лучше слегка улучшить регекс.
Pull req: Signing support

Signing support [UserStory - As a WebWorks developer, I can sign my bar file
Unit tests have also been created.

Detailed list of changes:
-Updated command-line interface/session object to accept and store signing parameters from user
-Updated native packager call options file to include parameters to sign the bar (if specified)
-Error handling [Handle missing keys]
-Signing-helper class created to get signing key location (Per Eric Li)
Use bootstrap and update info
Update README.md

* Clean RVM instructions
* do not use sudo for gem commands (sudo loses context and most likely will install in wrong place)
* minor notes update.
Additions to OrmLiteDialectProviderBase to create parameterized IDbCommands for Insert and Update

I added some methods to OrmLiteDialectProviderBase to create parameterized IDbCommands for Insert and Update. They are called by extension-methods from IDbConnection. If don't like the approach please get back to me. 
With the parameters the image blobs are not a problem anymore. Serialized complex fields receive also attention (see the test).

Regards
---
Joachim
walidacje w signavio
Don't run tooltipInterval if tooltip is disabled
V1.2

DONE - (04/02/2012) - User Preferences re-work
DONE - (04/02/2012) - Check preference reloads
DONE - (04/02/2012) - Check preference defaults
DONE - (04/02/2012) - Re-order and re-word preferences
DONE - (04/02/2012) - Long click open management pop-up menu & short click open overview mode, option to long lick on overview, option to change?
DONE - (04/02/2012) - Capture all shared preference values on application crash (ARCA)
DONE - (31/01/2012) - Add ability to launch service from wake up i.e. unlock, phone wake
DONE - (31/01/2012) - Clean up LaunchControllerReceiver, remove if required
Ticket/929

Add categories to pagination
xxxxxasdfghhkl
Cool new feature

please pull into master
Fixes: notices & strict `strpos` return value comparison

The first of the commits changes the various places `strpos` is used to strictly compare the return value to `FALSE`. Otherwise, the code doesn't distinguish the case where the needle occurs at the start of the haystack from the needle not occurring in the haystack. Not distinguishing these cases has various consequences. For example, field values can't be the empty string (e.g. "|none" will be parsed as a field with value "|none", rather than a field with value "" and label "none").

The second addresses minor notices for unknown constants and undefined variables & indices, mostly which clutter up things in development.

The third is another minor update, removing an unnecessary string interpolation.
PermssionFailure instead of 404

I've added these two commits that I'm using in my project.

1. Only show Actions that are permitted by the object in the ModelAdmin

2. Show a 404 when not found, but permission error for !canView(), which redirects user to a login page normally.
Fix for SOCIALTW-28
Adds constructor STSSessionCredentialsProvider to take ClientConfiguration.

This allows AmazonDynamoDBClient to connect behind a proxy.
Also adds a missing dependency in pom.xml for jackson-mapper-asl.
こうべ

htmlタグつくんなきゃ
6-лекция

Слабое подобие на конспект.
PLFM-1470: Resolved deadlock issue in recursive ACL update
Fix background color issue for IE9

I dig in the code a little bit and correct the background color for all swatches
Signed the manifesto:)

Nothing more nothing less
Adding support for complete translation

There are simple arrays to complete translation : comment_form and comment_reply_link
Update src/com/mixpanel/example/hello/MainActivity.java

Edits and additions to the comments compliments of the solutions department.
Fixing up README.markdown

Adding export of LANG so make html will work on OSX
consider <title> attribute in original option tags

if the option's already have a title attribute, use it as the title on generated labels, if not, leave the original behavior.
Перелопатил вест перевод, внес много улучшений. (С телефона).

Перелопатил вест перевод, внес много улучшений. (С телефона).
rowend/5-admin-common

Transalete admin common folder
Specify port 80 for connecting to Google

This corrects a failure to connect to Google for posting a sitemaps ping.
Add thumbnail filter for using armstrong presets in templates
Release Tests

This contains the basic infrastructure to write release tests for any modules. Run the shell script releasetests.sh or run the command mvn integration-test -DskipTests -Pintegration-test '-Dtest=*' -Ddatabase_user_name=root -Ddatabase_root_password=password to run the initial/sample test. This by default bundles the module under 1.10.0-SNAPSHOT version. If you want to bundle this under 1.9.0-SNAPSHOT use option                               '-DopenMRSVersion=1.9.0-SNAPSHOT'

All the release tests will run with a dedicated database by name of the 'openMRSVersion' you've specified during running the command. Look for the corresponding properties file in you home/root/application directory of the OS you are working with.

If you want to run specific migrations on your test database use the command 
mvn -P start-test-server jetty:run-exploded -DskipTests
IE7 trailing comma bug

Hi there, 

Bumped into this error while checking in IE7 - apparently it doesn't deal well with the trailing commas!

I have simply removed the 2 commas causing the bug.

Thanks for sharing your code! :)
Added some commands

I have added some more commands like echo, palin, calculator.

I would like to add more commands and contribute to this virtual terminal.
Porting to visual studio 2010
typo in annotations appendix

3 s is too much
Update bbp-custom-reply-notifications.php

Removing $user_id from reply_title() method.  It is unused, and unusable, given the parameters set to '3' on line 21
Update Listener/AbstractBlameableListener.php

Fix to be able to use the bundle with commands that involves data creation.
first parts on french translations

First draft on parts.
Signed Declaration
Update Ukrainian/main/SamsungSettings.apk/res/values-uk/arrays.xml
Removed all unnecesscary drive functions. Added in mecanum wheel capabil...

...ity

Includes the brake functions as well
Custom liblights

Should fix #11. Seems to work but I haven't had time to test too extensively, can anyone verify?
Update permissions in ueventd.thunderc.rc

This fixes issue issue #54 in inferiorhumanorgans repo

Original author - BobZhome@cheerful.com
Nette Framework

For test run use [Nette Framework](http://nette.org) sandbox from [this package](http://files.nette.org/releases/2.0/NetteFramework-2.0-PHP5.3.zip) and replace app dir.

If you need run "string" test with Presenter (Controler) remove `#` before `stringController: true` in *app/config/config.neon*. Default is a microframework way (route callback).

Documen Root is *www* dir in sandbox.
Fixed compiler errors: res folder + renamed fields

* Added empty .gitignore file. Avoids manual folder creation.
* Fields: mTexturePositionX-Y and mWidth, mHeight
WndProc nothrow, Thread.sleep changed to use duration

This commit makes simpledisplay compile under windows. Nothrow was recently added to phobos windows headers, so WndProc must be nothrow. Added a try-catch with asserts(false) on an exception . You probably want to do something better than this, but this at least compiles. 

Thread.sleep(long period) is deprecated, changed it to use dur!"msecs" instead.
if (!control->GetAction(moveAction, action))
  return false;
nextControl = action.GetNavigation();
context.getPackage()

Sent from Windows Phone
From: Владимир
Sent: 2/19/2012 18:06
To: AlexTaran
Subject: Re: [enot] XMLResources major bug fixes. It is now integrated
in page view. Index encoding problem fix. From now on all page IDs are
1-based (c813f0d)
Кто нибудь знает как можно вытащить сюда строчку /manifest/@package
вместо "app.tescact.manual"?

---
Reply to this email directly or view it on GitHub:
https://github.com/Losik/enot/commit/c813f0d7dbe00c46722d688731748e2e99262734#commitcomment-982516
Esse ```unless opcoes[:autores] == false``` pode ser substituído por um simples ```if opcoes[:autores]```, é muito mais legível e não embaralha a cabeça. O mesmo pode ser feito com o mesmo tipo de código que está na função ``submeter_conteudo``.
Can you add a test that check the inplace version is inserted in the graph?
select all .po files, then make directories with .lproj postfix
The alternative was to make routers in akka-actor remote-aware, which is not nice at all. Unless you have a better proposal …
Thanks for notificing that! Fixed in 8bb486a
try to get a link to this hunk
This has been released as v0.9.6, just let me know if you run into any other issues or have questions.
I don't see why yet,

    user=> (let [s "Hello"] 
                 (if (seq? s) 
                   (seq s) 
                   s))
    "Hello"

Note also the difference between `seq?` and `sequential?`. Am I still missing something?
Aight. I'll look at drawing victory lines. Can you guys jump on gtalk?
@sasezaki then please remove zf2 from travis-ci.org (disable github hook on the Services tab). Thank you.
Actually, this caused it to only work for IndexError.
Gotcha. Thanks! I'll get back to you on that in a couple days.
This is necessary since the retry call at line 212 wraps apply_and_push from tools/lib/python/util/hg.py. Looking at that file, it makes a call on line 313 to get_branch(localrepo) which calls 'hg branch'. This will give us an error if the localrepo directory is not an hg repository.

This means that we need to clone before apply_and_push() is called. Then, we need to have cloning done again in apply_patchset as, if an error occurs, we should need to clone again.

Cloning a second time shouldn't cost much time-wise.
We could probably move the clone in apply_patchset into cleanup_wrapper, but the effect will be the same. Either way, we require the one that you re-added here due to the function of apply_and_push().
how about writing it this way?

```javascript
var sound_effect = {};
sound_effect.ding = new Audio('ding.wav');
sound_effect.click = new Audio('click.wav');
sound_effect.clack = new Audio('clack.wav');
```

btw, i tested on local files instead of loading via http, not sure if it matters
I know it's fairly simple to get access to jsc on OSX. For building documentation, we need an OSX solution and a linux solution. Looking right now for a linux xolution.
Thanks. The bug is indeed fixed. I like to see you back at full speed ;-)
This is not guaranteed to actually find a matching plugin for a gameid. A good example there this fails is platforms, which use engine plugins. One could have game entries in the config file, for which the engine plugin (is no longer) installed. Thus plugin would be 0 and gameInfo would be an "empty" game descriptor.
Looks like this at least doesn't break anything for devices with regular screens, that's good. I'm not very happy about eink screen checks everywhere though, do you think you can clean that up a bit?
Line 285 triggers the assert in darray_index which checks that the incoming index is less than darray->length. Same thing for the call to darray_remove in line 286.
Forgot. Are you using command line or a gui?
Ty, totally forgot to check
ok I had removed that because the id mismatched the one on default-images.yaml, but leave it in and make it match.
we should probably re-factor the uiConf player rules check into a separate function
@shingara Yeah that's the intention... I'm trying to clean them up as much as possible before rewriting them.
This issue only occured on some systems. 
Notably, on KDE with `QtCurve` theme.
Oops! Sorry for the overlook :-)
Se puso a azul para que quede el sitio mas o menos parecido, sino va a parecer un arbolito de navidad lleno de botones coloreados
En category repository esta el mismo ejemplo :D
What examples did you try? I would like to try those out with openlibm too.
Again, we only want to test the case we wish to act upon. See my comments a few lines below
fck voll fail sorry
looks good, I like this.
http://pastebin.com/Wu6Ur2nF 17 часов.
Нашли одного моба, сам еще не смотрел. (вечером посмотрю, как  с больницы приду).
И еще некоторые говорят, что есть баг с петами у локов, можно по 50+ петов сразу вызывают...(сам не проверял и не говорят как багать, видел только на скрине на форуме)
i could add that extra field for spel_dbc if someone can tell whos the real one, no time to test now.
I'm not sure why the loader isn't defining davinci.ve.Snap.  I put in a workaround to define it manually.
Do we have to rollback?
не понятен логика этого коммита. Добавлены два файла пустышки для ордеров. Коммит в принципе не несет никакой логической нагрузки
ertetewrte ttw
I will show you how to turn of the wildcard imports.
Вообщем то ничего крутого тут нет - Слегка урезанная версия HandleAddItemCommand
const double * even cleaner.
We should add this new plugin to README.
No worries, we'll do a cleanup push sometime soon.
sorry about that, forgot to add the file to the stage when committing
I'd feel a lot better about this if it follow Vanilla's coding standards.
Dette kan du ikke mene!
copyClippedImageToScreen seems to be a method for safety. But it does not seem to handle the case where the source rect is an invalid rect as far as I can tell.
configurations are associated with a name in defineCache, not in the configuration itself. We can change that, but then we should change defineConfiguration etc.
In the interest of time we have started to use django-modeltranslation with mezzanine.  We did encounter an compatibility with mezzanine's front-end editing but - after much digging around - I found a very simple work around that seems to work fine.

We've posted an example at
https://github.com/catalpainternational/mezzanine_modeltranslation

If if you're still interested in built-in multilingual support in mezzanine perhaps code from modeltranslation can be re-used. It is released under the BSD license.
Works with melee, not so much with arrows.
from iran
we have one counter that reports the total time that queries take.  We could add another, one that we should *never* see - but should it be "slow queries" or should it be "failed queries"?

If we intercept the callback to client code and detect non-null err values there, it will catch many more cases where queries are failing, I lean toward that.
Allow for Proc to be passed to :s3_credentials to allow multiple s3 accounts.
Any idea when this will get into NPM?

Right now I fail with the expected:

node.js:201
        throw e; // process.nextTick error, or 'error' event on first tick
              ^
Error: require.paths is removed. Use node_modules folders, or the NODE_PATH environment variable instead.
    at Function.<anonymous> (module.js:376:11)
    at Object.<anonymous> (/home/zip/node-collect/node_modules/journey/lib/journey.js:3:8)
    at Module._compile (module.js:432:26)
    at Object..js (module.js:450:10)
    at Module.load (module.js:351:31)
    at Function._load (module.js:310:12)
    at Module.require (module.js:357:17)
    at require (module.js:368:17)
    at Object.<anonymous> (/home/zip/node-collect/listener.js:2:15)
    at Module._compile (module.js:432:26)
I did not see this commit and commented on your previous commit https://github.com/dmison/quickstart/commit/b25bc627aee52a64f407723774a0e511aa77607d#commitcomment-856103. This commit takes care of my review comments on the previous commit, so I'm OK with this change.
Yes. We can get it out.
I think in aggressive mode, a missing quality value should be changed to `None`, as I did in my branch: martijnvermaat@aba54d284773f6d05ea1e9c5e50113c5618f70cb

Anyway, I'm not too sure that having two 'modes' is a good thing and would rather see the aggressive behavior be the default.
Also freeze http://paste2.org/p/1911972 look down on movement generators where is the problem ...
Gavin?
Added the Ajax calendar and Category posts widgets
Do not use kOn, only use kForwardOnly or kReverseOnly, which is used correctly in the if statements below
= =我貌似还没弄Logo吧。
Linux支持我不会做……但是WxRuby的话我会尽量用W/L兼容的函数画界面，移植难度应该不大，主要是wine不好弄RM在线调试，离线调试器过于蛋疼了……
I was going to do that, but the code in the python file is different besides the operator. Should I ignore this? Thanks!
There were some and I removed them. Configured eclipse to remove them when saving a file
間違ってコメント削除してしまいました
Let's just leave it until someone wants it more!
shouldn't we use is_null?
That syntax is `markdownCodeBlock`. I am not sure what is your point.
This expects the following DB update:

delete from spell_script_target where entry = 34742;
insert into spell_script_target values
(34742, 1, 19949);

-- spell target positions (Warp Splitter)
delete from spell_target_position where id in (34727,34730,34731,34732,34733,34734,34735,34736,34737,34739);
insert into spell_target_position values
(34727, 553, 26, 419, -25, 0),
(34730,	553, 84.83, 362.09,	-28.1,	0), -- old data
(34731, 553, 56, 428, -25, 0),
(34732,	553, 46.65, 360.87,	-27.71,	0), -- old data
(34733, 553, 94, 416, -27, 0),
(34734, 553, 94, 376, -27, 0),
-- 34735 missing data
(34736, 553, 66, 353, -26, 0),
-- 34737 missing data
(34739, 553, 33, 366, -25, 0);
Unfortunately this is not only this place where the information is missing. Offline-log info is not present on the map as well. And if the user leaves a cache after logging offline, its marked in the list as found. I'll open an issue for that I think.
Would you consider also making a constructor with these parameters as fixed arguments?  I personally find the use of properites extremely frustrating and would much rather create an object with the arguments passed directly to the constructor and then call cluster without having to set properties.
Hm, something like this should be considered for all the shell scripts I think.
I'm curious about the property behavior of `receivedData` — it looks like there aren't any memory management problems, but it's definitely inconsistent. I assume you did it differently for that property for a reason?
higher-order functions (getting method refs, passing them as arguments to functions) are also working now in ceylon-js.
Apologies for my git ineptitude.   I'll get the hang of it eventually.  The renames of the .ngc files are because I mistakenly deleted them and had to restore them.  Thanks for the pointer to the graphical tool.
Yes. It... could be made optional, but I would rather just wait until ``feature/contrib_voting`` is merged to remove this, since it will be a lot less messy.
It might not have the rights to create a new file, though.
do i neeed to have old vmaps folder too for this ?
i extracted maps, then i started the makevmapsimple.bat 
it gave me error
your output directory seems to be polluted blahblahblah
and then after pressing a key it did some work
and then showed error
cmpfail VMAP003 != VMAP004
etc
This == test works because [NSNull null] is a singleton, but I'd rather use
STAssertEqualObjects(row.value, [NSNull null], nil);
revisá-lo e integrá-lo
It looks like it already is volatile.
I think we still need the barrier - it is there to prevent code that precedes the set of value to 0 from being moved after it.
If anyone's curious, this was deleted because you can't use a subquery expression in a VALUES set.
I guess this change was not necessary but it doesn't change the method behavior.
I like the idea here, but..  this will start I/O eagerly, as this constructor is called at the time the binding is satisfied.  It looks ugly, but doing return new Supplier<Server>{... is actually the intended code here
You're right the comment it's confusing. I'm going to change the comment.
copy and pasted from overra :)
This commit breaks the gemspec file so that it is in an invalid format for older versions of Rubygems.  I believe to fix it, this line needs to be changed to:

    s.add_dependency('bson', [Mongo::VERSION])

or something to that effect.  Just updated my project to use `mongo`, `bson`, & `bson_ext` versions `1.6.0` and then tried to push to Heroku and got the following error when running `heroku run gem list`:

    WARNING:  #<ArgumentError: Illformed requirement ["#<Syck::DefaultKey:0x00000008be6b28> 1.6.0"]>
    # -*- encoding: utf-8 -*-

    Gem::Specification.new do |s|
      s.name = %q{mongo}
      s.version = "1.6.0"
     ...
      if s.respond_to? :specification_version then
        current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
        s.specification_version = 3

        if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
          s.add_runtime_dependency(%q<bson>, ["#<Syck::DefaultKey:0x00000008be6b28> 1.6.0"])
        else
          s.add_dependency(%q<bson>, ["#<Syck::DefaultKey:0x00000008be6b28> 1.6.0"])
        end
      else
        s.add_dependency(%q<bson>, ["#<Syck::DefaultKey:0x00000008be6b28> 1.6.0"])
      end
    end
    WARNING:  Invalid .gemspec format in '/app/.bundle/gems/ruby/1.9.1/specifications/mongo-1.6.0.gemspec'

Heroku is running `Rubygems 1.3.7`.  I'm not 100% certain that my suggestion fixes it, I just know that right now it is invalid for older versions of `Rubygems`, and should be yanked.  [Here is a blog post](http://blog.rubygems.org/2011/08/31/shaving-the-yaml-yak.html) written by Tenderlove where he discusses the source of the error and the fix.

Try looking at `indirect`'s comments on this [issue request on the bundler gem](https://github.com/carlhuda/bundler/issues/1392).  He seems to have some good suggestions of what to look for in bundler as well as how to fix it.
Nice work! I need to stick with this for now because it's working with all the quirky things I need it to. I posted an issue on the new project. Hopefully the new module can handle cleanup better and I can make the switch soon.
Hey im trying to use java.io.file to create .dat files such as game.dat player.dat level.dat Im pretty sure it works i just cant get it to write files. But it loads them fine when I generate them
Yet the D auto-tester shows all green. And why would the testcase in pull #707 fail to compile? It's in the test suite, isn't it?

It is of critical importance that bugs that get fixed go into the test suite. Otherwise, we're left spinning around in circles like we are here.
Übernommen, aber die hinteren beiden Sätze wieder angehängt. Auch hier müsste der Verweis auf die "Twitter app used for authentication" geklärt werden.
Importante poner el lenguaje a cualquier documento html (google le da mucha bola a esto)
Shouldn't this message say `IOAcknowledge.ack()`?
Sorry, a misstake. Fixed in 3f28ad8e1a207d55dc08b3c476c1b91fbbff9fb8
{}.
I will respond only to a subset of your points, and then suggest how I think we should move forward.

> In this commit you're reducing test coverage; you should always try to explain why that is.

I deleted the tests individually so that I could highlight what was in place for each deleted test for you as reviewer (rather than writing a long commit comment or even worse a GitHub comment.)  In the case I found where the SyncCryptographer test was not a dupe, I adapted it.

In future I will be more conscientious about identifying grouped commits whose effects must be considered as a whole.

> Note that testSetKeysFromWBO — as I mentioned in my comment — does not parse a real server-side record; it constructs one in a different way from a JSON *payload*, not a complete envelope. There absolutely should be a test that covers the whole process, and you're deleting the last such test! That we don't use SyncCryptographer doesn't mean that testing it is pointless: *it* uses other code that really is live.

There should be a test that covers the whole process *as it is implemented in  the code*, but it is my opinion that the SyncCryptographer tests were actively bad:  SyncCryptographer implements something that you *believe* is the same as the whole process and you *believe* it is well tested.  But SyncCryptographer may or may not implement the same process.  (I claim that the fact that it uses code that is live is beside the point as well: I have faith in our ability to encrypt/decrypt because we carefully test the live building blocks, not because the mangled carcass of SyncCryptographer happens to exercise a code path that happens to use such building blocks.)

Now, for moving forward: how about I commit a patch that adapts these deleted tests and we leave arguing about whether things are minimal/duped to the dogs?
Gary, good job!
I don't know how do you say it in the USA, however we say as "brevity - the sister of talent" ("brevity - the soul of wit" ?).
In this place I want to discuss.
If we have this constructor with special **threshold** and **halfOpenAfter** it looks like we should configure several **CircuitBreakerBeanPostProcessor** for different circuit-breaker policies.
How about to have something like **ChannelSecurityMetadataSource's patternMappings**, than we may have only one instance of **CircuitBreakerInterceptor **...
And in the end config will be like this:
```xml
<circuit-breaker>
   <handler name="*WebService" threshold="2" half-open-after="1000"/>
   <handler name="*JdbcService" threshold="3" half-open-after="100"/>
</circuit-breaker>
```

Another thought: how about to individual `<circuit-breaker>` for components along with `<poller>`?
```xml
<ws:outbound-gateway>
   <circuit-breaker/>
</ws:outbound-gateway>
```
And of top: the same issue may be addressed to securing channels:
```xml
<channel id="inputChannel">
   <security:access-policy send-access="ROLE_ADMIN"/>
</channel>
```
Ah. Very cool. I had not hear of FJ before. This has sparked my curiosity :)
eh eh :) we'll change the door location afterwards :)
Was it hard to find ? :)
Thanks for the tip man. Got that taken care of =)
It can't read the file..No clue why :(
Hmmm, in my case, this does not work unless I add this in my XXTestCase class:
```python
class XXTestCase(unittest.TestCase):

    def setUp(self):
        ...
        self.portal = self.layer['portal']
        ...
```
Start confirmation of power functions
I am getting the following unit test failure on this line:

```
AssertionError: {'achievements': [4, 5, 1, 2, 3], 'id': 2} != {'achievements': [1, 2, 3, 4, 5], 'id': 2}
```
wena! no tenia ni idea que existia
Also I suggest you this fix for x,y var warning when typing make menuconfig : bfd83438e8e32c3f6115972d2a51338b4381d40d.
I've followed the path described in this post: http://blog.macromates.com/2011/locating-bundles maybe something has changed since that post :)
True, now you mention it I think removing the dupe check completely makes sense. In essence a duplicate event could be considered an error in the users code, or one that has been put in on intent. I can confirm jQuery has no dupe check: http://jsfiddle.net/mbArX/
a.indexOf(e)  returns position of e in the a, -1 if no e
I believe the _pyopencl_ dependency is wrong, don't we need 2012.1?
It looks like there are a couple of these `Error` exception types in different modules. That makes it impractical to do `from actionrun import Error`. May as well just call it `ActionRunException` or somesuch.
I believe the reason for `__slots__` here is because we can potentially create many of these objects for every single action_run, job_run, job, and service, which could add up to quite a few. 

I've started to use __slots__ on more of the data objects in a later branch.
On all o' 'em
And perhaps the same to ClusterCoreDaemon and ClusterHeartbeatDaemon. I could do a separate pull request off master for it.
added doc comment :-)
This line makes 0 sense to me
it would be more flexible to leave this open, i.e. return the Future to the caller
Ooops ... *cough* *cough*, uhm, that was a remnant from my original implementation. It's gone now.
Occasionally, but that's why we have this layed out this way, so we can tweak it when we want different details.
Ok, i'll not ask you to change, but for information, this is where you can optimize too: ['text_size'] can be accessed up to 3 times, and then index 0 twice
2 options here:
- tsw, tsh = options['text_size'], and use both variables
- ts = options['text_size'], and then only ts[0]/[1].
Not sure about what's the fastest between both.

But again, was just as an information, like dot lookup!
Inform the user with ".. versionchanged:: 1.4.0" than the class inherit from ModalView, etc.
Crazy renaming - recline?
You could make this a static method. I personally like doing that to have the guarantee that the method doesn't depend on or modify instance state, which is particularly nice for testing purposes.
like ..== setId) ||  origItem->GetVideoInfoTag()->m_iDbId == item->GetVideoInfoTag()->m_iDbId)
1) In general, closing a file didn't close the embedded IFile descendant, which seems like a bug to me
2) It is needed for WEBDAV write, as closing the CurlFile writes a 0 bytes packet that is needed to "commit" the write
Oh.. Ik zal er aan denken
same as above as well :)
boto likes to output info I don't want it to. Is there a better way to approach that problem?
No, that's useful; I think you're just focusing too strongly on our immediate use case. If we want to get details like this in our inbox, we should turn off `-q`. And the message should be less scary in any case.
Displaying images in log scale didn't work unless we set the log10Scaled property to true even after transforming the image data to log scale.

Aynur Abdurazik, SAIC
Ocean Biology Processing Group
NASA/GSFC Code 616  (Mail Stop 616.2)
Greenbelt, MD 20771
Phone: (301) 286-4072







On Mar 12, 2012, at 11:01 AM, Norman Fomferra wrote:

        RenderedImage sourceImage = getSourceImage(raster, level);
+
+        //added by seadas team

Images are created solely for the purpose of displaying raster data. Here, you are _modifying_ a data-inherent raster data property 'log10Scaled'. This is a very bad idea.

---
Reply to this email directly or view it on GitHub:
https://github.com/bcdev/beam/pull/2/files#r546145

--
To UNSUBSCRIBE: Send an e-mail to
seadas_dev-request@seawifs.gsfc.nasa.gov
with the subject, "unsubscribe"
Good point, too much copy/paste ;-)
Calling this `encode` is confusing, because it doesn't on Python 3. Elsewhere, I've called a similar function `unicode_to_str`.
They are now inside a fixed size (six columns) container so that will not happen. The new UI only changes column sizes at cut off points and the column size does not get smaller than 28px; so it should be fine.
this must go into separate function in elastic_tabstops.js
editor must know nothing about elastic tabstops, 

insead of `editor.setUseElasticTabstops(true)` one should do
```js
var ElasticTabstops = require("ace/elastic_tabstops").ElasticTabstops;
new ElasticTabstops(editor);
```
You should not have removed this submodule.
Why are you going through an intermediary method, `_controlTintChanged:`, rather than just calling `controlTintChanged` in the first place?
No trailing newlines.
Style: missing license boilerplate.
Don't call `process.exit()` here, that potentially hides bugs (think off-by-one errors in node's active handle reference counting scheme).
Whitespace-only change, please undo.
The return type ought to be bool.
I think it must be "!pendingEncryptedWritesLock.isHeldByCurrentThread()" .. Can you test with it..
this should have been named simply SQL and have a value of simply "sql". same issue for xpath.

see http://www.day.com/maven/javax.jcr/javadocs/jcr-2.0/constant-values.html#javax.jcr.query.Query.SQL

ok if i go and fix this?
нужны doxygen-комментарии
forced update since this really shouldn't be in the history
I think the only places in Action Mailer too. Maybe in some tests.
We should simply do: `keys.zip(args)`
Do we need the full identifier for the constant?
You can also use relative links, if it's obvious to what they point to.
Thanks a lot.  LGTM.
nope, this needs to be a constant, and unfortunately constants are not allowed to use the police or any armed forces :-(
Yes, I believe that ephemeral class/trait can only have defs in their `Template` tree node so we are good with this set of conditions.

Does it make sense?
tab; please use whitespaces only.
Ditto, and so on below.
Should `euler` and `packages` be unified? Moreover, euler defaults to true (what is it used for?), whereas "xypic-support" defaults to false. This seems inconsistent.
I agree that it could use a better name. All of MatrixExprs are lazy though so it feels strange to give one the name Lazy (e.g. `Transpose(MatrixSymbol('X', 3,3))[1,2]` is evaluated lazily as well.) Maybe `LambdaMatrix`?

I'm not very familiar with DeferredVector. I'd be happy if we could find a way to merge the functionality of it into MatrixExprs eventually. I agree though that this is a separate topic.
`==` -> `is`
A nicer printed rendition, otherwise there is no difference.
I get 16 warnings in `master` and 16 in my branch.
The 'first' element will the first that SymPy gives for function.base.args.  Usually its a coefficient if there is one, then symbol (if no coeff) or then other functions (Not clear about ordering in this case). In any case though, the answer is mathematically correct I guess.
Does the new adjoint class properly set the commutative keyword arg if its arguments are non-commutative?  That is very important.
Explicit `return None` would be clearer.
It's been dropped in Python 3 though.
Yeah, reduce is really a very nice function. I think the standard Python stylebook says to use a for loop. The argument is that it for loops are clearer to more of the Python community. I'm personally fine with the reduce; I've seen this issue brought up in other PRs so I thought I'd bring it up here.
No space before `,`, no space around `-`.
too long; you can simply say `w = options$out.width %n% options$fig.width * options$dpi` (similarly for height below)

`%n%` is defined in utils.R
Renaming the file destroys the original sort order.  Please restore.
I want to see that the warning is properly generated.

Convert the warning to an exception, catch it, and assert it.
Make sure it fails if you remove the warning.
the operators get serialized so you may want to just store the cache key in there
Cool. I can't close it (I don't have contributor access), but I think you can :)
Please don't document it this way. Services should be lazy-loaded, not registered with an instance directly.

And btw, you can simplify it here:

```php
$app['security.encoder.digest'] = $app->share(function ($app) {
    return new MessageDigestPasswordEncoder('sha1', false, 0);
});
```
Please list the services separately.
Indeed, it is so huge that it's almost a rewrite :). Still, it's for the best of Listen.
Can this be worked around, e.g. by calling `uv__once_init()` in `uv_cond_init()`?
@tcp_socket.string raises excpetion: (execution plaform is win7)
    undefined method `string' for #<TCPSocket:fd 5> (NoMethodError)
Failing in TC against MySQL
I think we should avoid using `$atomic` in this section because it's emphatically not atomic.
same here? check defer_rendering
Ok ;) Thanks for the link!
We definitely don't want this. Add a class and use jQuery. INCLUDEJS includes the script after the jQuery call (although you might want to use a generic file name like script.js)
If I didn't, it won't submit here and the assertion below will fail because it will be a Login link rather than Logout [admin]
Use one of $config_name or 'default_style' for the last two comparisons.
In posts table we have 3 columns for this (enable_bbcodes, *_smilies, *_urls) which should be copied and used here aswell, otherwise the post is still not the same as it is in the viewtopic view.

However I guess you can combine them to one "reported_post_text_options" field
Should have a newline here. Please squash into previous commit.
Add a comment explaining what true/false mean.
Problem is, styles table doesn't include paths. It only includes style name, which isn't necessary the same as style path or name in one of style.cfg files. Old paths are stored in other tables that are dropped, new path is stored in new field in phpbb_styles table.

Because of that conversion has to be done in this order:
1. Add new columns to phpbb_styles
2. Convert all styles (code above)
3. Drop old tables (last few lines of code above)
Add a comment explaining what this function is for/what it does.
Please don't use RSpec-specific shortcuts. I'll probably try to convert the files into a meta-format easily parsable by other non-Ruby implementations.
See existing expected files for some hints.

Specifically, avoid

```
be_false -> == false
be_true -> == true
be_empty -> == []
be_nil -> == nil
```

Also, make sure to test for all methods, including unsupported methods (aka methods that raise a `Whois::PropertyNotSupported` error).
@ericf Do you see a downside to just using requires:['handlebars'] ?  I'd like to keep things as simple as possible.
Remember to use "debug": true, otherwise all this is irrelevant.
This suggests to me that DataFrames should have an OrderedDict or similar instead of an array to define column names. That way, this wouldn't be required on every call to within.
You have some unneeded code here. Simply
```SimpleObject obj = new SimpleObject();```
would be sufficient
Should probably copy the dims array in case the source modified it later on.
```cartopy_map``` -> ```cartopy_projection```?
Sadly, I believe this needs to be a comment, not a docstring, so that the method name is outputted rather than the documentation on running test.
Check whether the downloaded disksharemount is the same you added in setUp()
GetCaster() https://github.com/arcemu/arcemu/blob/master/src/arcemu-world/SpellAuras.cpp#L695 can return NULL , I suggest to use GetPlayerCaster() https://github.com/arcemu/arcemu/blob/master/src/arcemu-world/SpellAuras.cpp#L667 and check for NULL.

When rewriting some code pay attention of it's logically the same or if you hid some checks like in this case.
It should be ` . DS . ` (with spaces before/after DS).
Wrong doc comment - no param anymore.
Accessing `self.template_postgis` just above already made a query to the database. If that worked, why is it necessary to create a new connection now?
Interesting, I actually thought the of was required but was omitted in speech as a form of slang. Maybe an American thing?
What if you use a hash instead of a name property? It fits better stylistically with the layer config: 

cron:
  someTask: {time: "...", command: "..."}
  anotherTask: {time: "...", command: "..."}
`.append()` never gets tested?
The drawback of coding directly in the github editor! :-)
It's a temporary table with a reserved name (\_madlib\_*). I don't know what you want to hide more and why.
Could possible be fixed in the function, but local was absent at the point I came along here.
The default value for `WS_SERVER_HOSTNAME` should be `u'::'` to work with IPv6 sockets.
I think, after reporting an issue, we should go back to the config that the issue is reported on, since that seems like it would be more informative for the user.  (They would get to see that their issue was on the correct config, and what it would look like to other people.)
did you mean to add this, or was this left over from debugging?
We should probably find a good home for this Resource at some point. It's being copied over a lot.
Тут, кажется, нужны ещё несколько запятых:
"Таблица 2-2 содержит как параметры, уже рассмотренные нами, так и другие, могущие быть полезными параметры, вместе с описанием эффекта от их использования."
А вообще предложение дурацкое, лучше исправить его на
"Таблица 2-2 содержит как уже рассмотренные нами параметры, так и другие полезные параметры, вместе с описанием того, как они влияют на вывод команды `log`."
I don't think returning None is appropriate in this case; an exception should be raised (with the more helpful error message)
You should also specify in the comments the order of searching through the platform's languages: language, defaultLanguage etc.
I feel like passing info back through the global `c.error` is pretty hacky. Can't this data be returned from the validator somehow?
this line crashes when I boot my configuration. I think the problem is that this line will be called outside the scope of the file loding and the variable for the current-path is not set correctly. This confuses `cabbage-load-bundle-dependencies`.
Update the commit message too?
You shouldn't need to check if it has a parentGroup as all objects are in one.
Same for this method (removed `virtual`).
OK. If _isMatch() does not populate _pathMatches, then multiple calls to getMatch() could be an issue. 

I made _pathMatches a class variable to avoid needing to pass it to _isMatch() by reference, since _isMatch populates it and then getMatch() uses it.

Possible solutions:
- initialise $this->_pathMatches = array() at the very start of getMatch()
- remove _pathMatches as a class variable and pass a local $matches by reference into _isMatch(). This means any overridden child version of isMatch()  will also need to accept $matches (note this var isn't needed for DynamicLandingPage_Route at all).
- return $matches from _isMatch() . Any overridden child will need to return an array to be consistent, or getMatch() will need to handle a mixed return result from _isMatch().
Maybe remove this 3 commented lines?
Works for me.  Thanks.
same as next
No worries.  By the way, just to set expectations, I probably won't be logging on a lot until the new year.   Also, I might want to spend a little more time in private brainstorming about if we can get the same idea while still re-creating test objects.
Ah, actually this has to be wrapped, since 'job complete' will pass `job` as a first argument. If we simply proxy that to `fn`, it'll interpret `job` as an error. I left a comment about it in the code.
Can't you use the built-in strip_tags method for this?

http://api.rubyonrails.org/classes/ActionView/Helpers/SanitizeHelper.html#method-i-strip_tags
Terminal whitespace! @ryanfreng should set up his IDE/git with a cleaner for this. I use a git hook (also catches stray console.log and debugger calls): https://gist.github.com/2948929
Does this mean that we're both going to add a `<link>` tag for the stylesheet and keep the `@import`, so it will load twice? (I'm not sure that would necessarily cause a problem, but it seems a little dangerous.)

If the underlying goal is just to fix up the promises so we're absolutely sure of when all the stylesheets are loaded, it seems like we shouldn't change how we're actually loading the stylesheet (i.e., only create one `<link>` tag pointing at the root stylesheet), but just create multiple dummy `img` elements, one to track each dependency, and set up the promises based on those. That way, we're not changing anything about the actual loading, just how we're tracking the load completion.
Are divs the only elements that can scroll? Seems like any block-level element with overflow set appropriately might scroll. Might be a lot of elements though.
Glenn and I had a quick chat about this. We think loadProject() is asynchronous so we are curious if PerfUtils.addMeasurement("Application Startup") would yield different times if put before or after loadProject().

Ultimately I think we want our measurement for "Application Startup" to measure from initial app launch to when the app is actually responsive to the user. Since LoadProject() is asynchronous we may need a better end point to determine this.
Seems like this should be an `else if`
I tried adding an additional measure point *after* ProjectManager.loadProject() is called. On my machine (a 3 year old Mac Book Pro), the second measure point is always 4 milliseconds after the first, regardless of the size of the project or the number of files in the working set. 

I agree we need an additional measure point for "app is fully started and the user can now interact with it". However, I think that is independent from my changes here. For now the "application startup" time is basically the same before or after projectLoad()
typo. should be ``failedCount++``
REMINDER: need to update new context menu unit test I added in pull request https://github.com/adobe/brackets/pull/1273 to uncomment line that checks for contextMenuClose event.
``{function(*,*)}``
@ray is out so I will fix this. Ray, have a look at this since Javascript is a bit funny on how 0, undefined, and null work compared to native code.
This is the way @cadorn wanted it. The idea is that NPM is managing the install process, not make. The local version depends on this way while doing some install/update stuff.
Why did you remove this block of code?
You should be able to do:

ary.map(parseFloat)

Looks like this works in anything past IE9: https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Array/map
doesn't this require updating the list of illegal case property names?
let me rename that block to what it actually should be: additional-form-content

On Sep 13, 2012, at 3:04 PM, Daniel Roberts wrote:

> In corehq/apps/adm/templates/adm/forms/configurable_admin_adm_item.html:
> 
> > @@ -0,0 +1,33 @@
> > +{% extends "adm/forms/admin_adm_item.html" %}
> > +BAM
> > +{% block js-inline %}
> 
> Should include {{ block.super }}?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
you might have a NPE here so null check.
Not every machine has this same certificate path -- if you're going to add this check that the system is compatible first.
You can use Mock#assert_call_with which is available in version 0.7.1
`vapply(e1[names(e2)], is.null, logical(1))` would be safer
Should this go in a validation specific spec like requires, presence and regex do?
Shouldn't here be:

```
if isinstance(tmp_value, datetime.datetime):
```
Didn't know template authors had access to butter directly, awesome!
so 
 });

_this.attachInputChangeHandler( _countInput, trackEvent, "count", function( te, prop ) {
Do you mean ?

```php
use React\Stream\StreamInterface as ClosableInterface;

interface OutputStreamInterface extends ClosableInterface
{
    ....
}
```
Tiny bit of doc would be nice to be clear what this is.

"Allows a label to be rendered in a form as a field" or something.
效果上一样, 不过多了format 解析~
Seems restrictive to hardcode 0.0.0.0 here. Add another val for host?
We should probably remove this. A start might be to see if we can serve the static content outside of Ruby.
that looks good, yes
typo "hte"
intentionally using different path matchers for different methods (so you could route a POST to a different place as a GET for the same url)
Tip podatka koj se koristi u dvosmjernoj komunikaciji. Frontendu se šalje skup akcija tipa Action
Space after "function".
These feel a little too low-level to be very helpful. We should come up with a good API for dealing with all of this in views and controllers so that we can keep things clean. I was thinking more like `@user.bookmarks` returns the list (maybe with videos and listings eagerly loaded), and some ways to quickly test for membership like I mentioned in my other comment.
Why rename this variable, `buffer` is a fine name?
As above line will get the strlen removed your argument counts :). My intention was to avoid pure char * usage as much as possible and use std::string for this.
Isn't this normally CakeText?
@lorenzo Yes. To follow the PSR-0 we will have the namespace equals to folders, so seems better to have `App\Controller\PagesController` instead of `app\Controller\PagesController`.
Should the doc block be expanded to mention support for the different formats?
Just before that there is `$newData = array();`, if nothing has been added to it and its empty `foreach($newData...` would do nothing. There is no code after the `foreach`
Wont this cause 5.4 strict errors for existing adapters extending this with 2 arguments?
This clause can never match because you've wrapped the whole thing in the same test now.
Can you add a space after // so it complies with coding standards.