labofsahil/github-event-dataset-2012 huggingface dataset:
=> Unable to remove object at key %1$s, object not found

But generally I wonder if we should log or simply ignore this case, WDYT? What would make sense in a mongo world?
Looks good, but if I remember correctly, indexing into a string isn't the most reliable thing, cross-runtime (not that it matters here). We should probably use `charAt` instead.
typo... ActionType 80 have only two param(entry and timer update type)
zwerg, kologarn door doent work.. no activate in combat
Because if you're compiling with a debug compiler you're likely to be running against a debug system.  And at least with windows if they don't match it seems to go wrong.  I suspect linux doesn't suffer from the same problems.
I will be out of the office until Friday, February 3rd for vacation.  I will not have access to emails.  

Thank you,
Greg Kerstin
Graphic Agenda
> bold = '\x1B[0;1m'

@michaelficarra good thinking :)
Is portal_registration the right place for this?  Wouldn't it be PlonePAS/acl_users?
fix japanese messages
#25 #33
Gradient variables can be modified from the Wizard, but the modification is visible just in the view mode. I haven't added the gradients in the Wizard.
This commit went bizarre.  I was trying a mv (rename) on opacityspheres.py and it registered as a delete.
Probably add this to unacked_responses?
take out the Provider stuff here as it isn't doing anything helpful.  Just inject CacheLoader and use it
Hi, you forgot this :)

include/linux/if_arp.h
(1 / 0)
 	 
	#define ARPHRD_LAPB 516 /* LAPB */
	#define ARPHRD_DDCMP 517 /* Digital's DDCMP protocol */
	#define ARPHRD_RAWHDLC 518 /* Raw HDLC */
+	#define ARPHRD_RAWIP 530 /* Raw IP */
	
	#define ARPHRD_TUNNEL 768 /* IPIP tunnel */
	#define ARPHRD_TUNNEL6 769 /* IP6IP6 tunnel */

Any way THANKS allot :)
@kno Fixed in a more recent revision =) Thanks for reporting!
Can you remove BSONInput.java (which is basically empty now) and SingleThreadedMongoPerfTest.java?
> Nitpicking but the word "classifier" could be confusing for some people as it is usually associated with supervised learning...

Agreed
@isyoon ㅋㅋㅋ
Надо было сразу так сделать, а не парсить по однаму файлу а потом мучаться мусар выпиливать
Rewriting seems good.
you need to ensure open has returned before issuing any commands
C'è qualcosa che non torna comunque pazienza, l'importante è che funzioni.
Tutte le cartelle in assets è come se non ci fossero. Ad esempio se vuoi vedere "/assets/javascripts/simpla_admin/facebox.js" vai in "http://localhost:3000/assets/simpla_admin/facebox.js"
thanks. Will do.
This commit introduces a regression where selecting a VirtualVideoAction results in a "Corrupted data" message.

When the VirtualVideoAction child is added in DLNAResource, the logic decides to have MEncoder assigned as player of choice. However, VirtualVideoAction comes with an MPEG movie which is supposed to be streamed instead of transcoded.
Since the code is the same (except for my above comment), maybe you could consider using a parent class defining the function for all 3 ?
Great! You could have even been so radical to keep the class name `Transformer` since it implements the `ufl.Transfomer` interface, right?
Use the `forEach` method of the `NodeList` returned by `dojo.query`.
The wiki didn't specify, but I specified it and fixed the comments
Sorry for my error putting, i am a greenhand on github , i don't find the location to pm tpruvot. 
Sorry.
@wycats, @tomdale: This breaks the specs for me.
no problem…afaik related to your plugin the changes are rather minor and are working with maven2 as well…
Ok I understand the problem. Thanks for the explanations. In an other order of ideas, I tested the `composer.json` and everything is working fine.
Yes, that is definitely broken.  Thanks for pointing it out!

My goal was to make the "local" and "remote" Pinto APIs identical, so you wouldn't have to make those kinds of distinctions.  But obviously, I didn't get it right.  I will fix this shortly, but I'll probably use a different solution.
Nice, thanx!
Well this is an informational comment on a pattern you can see in coffeescript, where you group the function call as opposed to putting parens around the argument, I believe it also says in the document that this pattern is recommended against, still I feel it is useful to keep here as it is not an immediately obvious pattern and in case it is ever come across somewhere.  (At least that's why I kept it in rather than just removing)
Oh sh.. forgot the paging of 2 items in there, was for testing the pagination on my meagre set of files....
Shouldn't this be returning the delegate return value?
touch.h is missing!
Isn't the .htaccess file denying access to the .swf referenced from the code?
ok it still isn't finding our apps. onClick is getting called however starting the activities throws ActivityNotFoundException

we should have 3 actions

onClick the date launches the alarm clock @ com.android.deskclock.AlarmClock
onClick Liquid Control icon launches com.liquid.control.LiquidActivity
onLongClick Liquid Control icon launches com.liquid.control.fragments.Preformance
I found I needed to place the git checkout in `~/Library/Application\ Support/Avian/Bundles` to get TM to pick the bundle up.
1st: Please only comment on my _latest_ commit.

2nd: Test the _performance hit_ yourself: There's none - I discussed this some month ago and proved myself wrong. Single & double quotes have the exact same processing time. Btw: If you test, then do it with a _high_ number of loop times.
exception is created but not thown
Oh, I see.

Anyways, it look good now. Thanks for a fix.
It's gotten a bit too long, it'd be better to devide this over 2 lines:

```
$label = isset($settings['label']) ? $settings['label'] : (isset($attributes['label']) ? $attributes['label'] : $p);
$label = \Lang::get($label) ?: $label;
```
@qwcode: You're right. This particular issue and test was mainly about ensuring that the `elif` block does not get run when it shouldn't; i don't know what circumstances it would actually be required in.

It doesn't seem to be covered by the test suite, at least: removing the whole block does not seem to induce any failures.
`cancelled` -> `canceled`.
Now that the Text widget is in master and we have a "Simple HTML" category in the new "Widget Sets" group, we should put it there instead.
Default to "List Item" so that user can understand its functionality
Equality of ActorRefs, I assume you mean that if I use actorOf(), actorFor("akka://node@74.74.74.74:2552/user/someActor") and actorFor("/user/someActor") are they all equal?  This I have not tested.
why semicolon?
great catch, fixed.
changing to “non-error condition tests”
This deserves a comment :)
I prefer `to[immutable.Seq]`, even though you get the same thing. The intention is more readable. This is the only place with `to[Vector]` that I could find.
that is true, I will change, thx
If you can figure out how to keep it from happening, more power to you, but I couldn't.  This fixes a deep and vexing loading problem.

I was handling this in the exception code, but I noticed it was being thrown from here, so I though I'd head it off at the pass.

Auto loading is using exceptions and often this method as well, so deep voodoo may be involved, I'm not a core Rails guy (yet).
I'm sure there a way around these goto's e.g. using the value of ret further down to check if an error has happened before and then doingthe rollback instead of the commit. Will even save you doing the announcement call twice.
Right. My mistake for talking too soon.
$this->dataProvider->pagination->pageSize = $this->_pageSize;
сорри
```php
Yii::app()->setGlobalState(Yii::app()->getModule('menu')->menuCache, YII_BEGIN_TIME);
```
не может существовать, потому что алиас уникален, а генерация имени идет от алиаса.
переименуй в XLS
Het lijkt me dat je 't bestand ook weg kan gooien? Als 't niet meer belangrijk is kan 'ie weg, is 'ie nog wel belangrijk dan moeten we 'm misschien een beschrijvender naam geven.
made more verbose.
A to co za Alfa-System? ;)
this just prevent the double refresh when first step into the repo list
Maybe there's another `else` missing here. When there are additional non-option arguments, there are too many arguments and it would be good to show the help.
* Please don't write `inline`. It's superfluous since you already implemented these methods inline.
* Note that I always prefix member variables with an 'm'. Please do the same with `objectBorder`, so that you can drop the `this->` part.
* Do not prefix getters with `get`. This does not match the Qt (and Tiled) coding style.
I know, but thats PHP core paradigm? Why not use it?
Every scalar value will be put into an array.
What do I mean when I call:
```php
findByPks(false) -> findByPks(array(false))
findByPks(true) -> findByPks(array(true))
findByPks('') -> findByPks(array(''))
findByPks(0) -> findByPks(array(0))
findByPks(20) -> findByPks(array(20))
findByPks(array()) -> findByPks(array())
```
The PHP behavior when converting to array is well documented and should be known (or known where to read about) by every PHP developer, why use own checks, which have to be documented separatly?
This asks for the External address of a given internal one, no? If not,  
then I don't understand what this method is good for. There is only one  
test using this in akka-doc.
What exactly do you mean? By default, automatically synthesized ivars are created with an underscore prefix. In this case, the ivar for `cursorColor` would be `_cursorColor`.
And after , and around =
Minor style issue: we usually use single quotes. Otherwise LGTM but I think that @isaacs should weigh in if this is what he had in mind.
Figuring out the best granularity of analysis is best done at report time, not record time. I recommend playing with kcachegrind some time -- after you've "zoomed" in and out a few times on the fly, you'll never want to write that kind of stack walking code again.

And anyway, if you *are* going to do line-by-line tracing in any case, while turning the memory profiler on and off as you go, then you can do this just as easily with either the callback or logging APIs. (Reimplementing the logging API using the callback API is like 5 lines of code, most of them function signatures.) So the complexity of the callback version is never higher.
This `zval_ptr_dtor` isn't right here. `option_buffer` will just get assigned the zval**, the refcount won't be incremented. So this will cause a double dtor.
Backslash is not recommended. Within parentheses, your split string should be fine, e.g.

>>> ('first part' ' second part'
... ' third part') == 'first part second part third part'
True
и что оно делает?
do you mean i should remove this from here? what is the harm of its presence?
It is used for more readability
In this case is better to use the `join.html_safe`, because [`safe_join` escapes the separator](https://github.com/rails/rails/blob/master/actionpack/lib/action_view/helpers/output_safety_helper.rb#L31), and we don't need to escape it since we know what is the separator content.
In general, you don't want to rescue Exception.
Done, looks fine so far :+1:
Not bad.   I can make this part of the ant targets in the future, so we don't remove the original source *and* we still stop using this after a new starr....

For now this is 100% better than what we were doing.  Thanks much!
Should probably point to https://github.com/scala/scala.
I think so. We are using this class: https://github.com/scala/scala/blob/master/src/compiler/scala/tools/nsc/interactive/InteractiveReporter.scala As long as messages get there, we can display them. Right now they would appear as error markers in the editor, but only if the position is correctly set. Everything is also spit out in the console, if the presentation debug flag is set.
Ah, okay. Not a big deal, so let's leave it as is then.
Could be:

``` if optional in [None, 'howard', 'sidreal',;synodic'] ``` ??
I see: we don't want to regulate subheaders. All my comments really just apply to the headers. But I don't see that the docscrape mechanism (when collecting headers) differentiates between the two, so it doesn't look to me like making a whitelist will work. So I guess you can proceed as you wish.
ahhh, I see. I think what you have is fine, then. Sorry for the noise.
Name clashes with `sympy.geometry.point.Point`
note for future refactor:
this.element = ... on line 30 and this if (!this.element) ... else .. part should be placed in createElements method and createElements should be called here.
I wanted the option of sending a separate message for the device in the case it was a simulator, eventually we should try to identify the computer the simulator was running on as well.
Converting RunFunctionSafe to take a `const char*` will make this conversion from string->plstring->string unnecessary
Put in the annotation info and the link to docs (once we have docs publicly hosted and built with every release) it will be valuable.
I think it's a good idea to decorate the mocked version to avoid confusion, so I'd use m_format_exception_info or mock_format_exception_info.
Things indexed by type should go into the `typeMap`.
Ha! Can't believe I missed that :p. If you need a reference on the documentation format we use @luosheng, check out the [TomDoc site](http://tomdoc.org).
Because I dislike variables named ````temp```` (preferring meaningful names), delete line 113, rename ````temp```` to ````credit````, set credit with

    credit = v[:credit] || ''
I think for more involved examples, we should give people a look into the document as they go through the procedure so that they can play along in their minds more easily.
needs a colon.
need a better link format.
I like presenting indexes as "supporting" a query or group of related queries.
general admonitions (e.g. "remember...") are clumsy in these situations. Better to:

- give a link to the spherical section that covers this in great depth.

- say "The distance returned by this query is in radians, you must convert to the units of your choice in your application" (edit this as needed, but you get the idea.)

- put some or all of this after the example, not before
I think the redirect makes sense. Migration needs a tweak to drop the redundant locale. Might be a quick fix, but worth filing a bug
I kind of punted on that... There *is* a mechanism for uniqueness, but only for auto-generated IDs (eg. sect1, sect2, etc). For IDs based on element text or the `name` attribute, no uniqueness is enforced.

This is really a half-baked feature, ugh. :/
No need for the extra space.
Move response creation to separate line.
The auth class or auth management class should have a method for dropping permissions.

This part here should then look like
`foreach ('u_pm_delete' ... 'a_jabber' as $perm) { $auth->drop_permission($perm); }`
Sure, sounds good. Stefan did install sphinx with easy_install so I would expect it to work.
Varargs can't be used as upvalues in closures (unfortunately)
It looks like, on the server, that Y.mojito.ViewRenderer isn't being used. That's slightly inconsistent, but it looks like it'll be OK.
'return' rather than 'result'
add new line between header declaration and if-statement to readability
This shouldn't be necessary, the standard unpackPhase sets $sourceRoot
Unfortunately, the Apache License being so permissive (why we chose it) allows one way compatibility.  

http://www.apache.org/licenses/GPL-compatibility.html
This api change should be documented as a non-compatible change.
return this._searchIndex('fullName', name, regionType);
Looks like there are two spaces in ``its  *op*``.
I don't think so, as we currently list the dtype in the c code. If we add the int128 into discrete_dtypes, the c code won't work.
why is lazy specified up front here? Normally the VM_Linker inspects the graph and if there are any lazy ops, then it uses an appropriate linker. Why is lazy even a parameter of VM_Linker?
It seems like extracting the detailed arguments from the core could be handled within the SQLStmtAdHocHelper factory method, or maybe better the SQLStmt factory method it calls or maybe the constructor that THAT calls.

Actually, I'm not sure what/how the SQLStmtAdHocHelper actually helps -- it just seems to be adding a layer.

Is the idea to keep from exposing the factory function to user stored procs?

I suppose we could give the helper a (further) purpose if we recruited it to mine the core for its details, 
-- if that meant we got insulation between SQLStmt and PlanCore.
Switching from the individual arrays to m_columns is intended to not change the tests in any way. It's purely a reorganization to help with maintenance.
The above comment applies to other instances of thrown Exception.
This method is equivalent to: ```return 'status' in self.dirty_fields if self.id else True```.
Yea that's true. I didn't touch the permalink docs themselves other than adding the deprecated note. I wasn't sure if it would be better to leave them as is until it's removal, remove them completely, or just rewrite them to still document permalink but to mention that people should use reverse instead.
No need to save the user here, we only needed to test the one assert.
strike: "and is important when building Django projects."

what does that add?
IMHO, the 3 day info should be mentioned in desc. i.e. Delete all notices older then 3 days.
What is the purpose of this change? It breaks the leading-text case: http://jsfiddle.net/NzCTZ/
Oh, not for combinators, must be POS.
why not use `jQuery.offset.supportsFixedPosition`?
Is this true??   If it's not continuous value but its value is NULL, what should be m.code?
I think instead of hardcoding these dates, I'd make them all based off of today for whatever today is at the time this code gets run. The reason for that is because I think we want sample data to be recent so that the system shows it properly. If we hardcode the dates, then the sampledata gets old over time.
It is a bit messy and all around at the moment, sql, tls certificate scripts, optional xml dictionaries. doc is mostly .conf.sample but there are few sql and python scripts, so I would just put it in prefix until it is cleaned up.
What does this do if I pass `--use-llvm` or `--use-gcc`?
Agreed re: the sqerl prefix. I'll update the statement name to caps.
We also need a brace here.
Same here, maybe add "and errata" so that people know they can promote errata through a workflow. That's a huge use case for many users, arguably more so than just packages.
consider DRYing up this code. There's LOTS of repetition, here, so it's crying out for some meta-programming with a define_method.  Ask me for ideas if you need some help, here.
hier dachte ich an

```

$props = array();
foreach($mixed as $name => $val)
{
  $props[$name] = self::callableAsString($val);
}
return 'Array ('. implode(',', $props) .')';

```
HS --> LHS
Yes, I totally agree with you @sl4mmy. I was going to change it according to what @kcooney proposed. Since you already made the changes, we could close the pull request.
I think we should use `.prototype` instead of trailing `::`. It just never looks clear to someone unfamiliar with what's going on.
Negative, since the iterator can change the array. Was a terrible bug to fix.
I started including it in the capability, but removed it because capabilities lack lifecycle methods; consequently, once the dojo.subscribe is set there is no way to tear it down.

I also think this _should_ be a default behavior of any asset list in its own scrollable region.
Minor nit: would be nice to have a Windowsy example on Windows (so you know what syntax to use for filenames--I'm assuming you still use `/c/whatever/whatever`). I'll go ahead and merge this without that, though--can fix later.
I thought I had run into a similar cyclical dependency before and it just worked. Are you thinking more like a recursion guard?
Yes, it's overriding the typical use of tooltips. How about this:

"Live Preview: click to stop  (Save file to update)"
This file is still in english. Did you intend to translate this to italian? If you don't translate it, then you should revert your change to src/nls/urls.js.
change `sort -u` to `sort -uV`
Is `options.selected` an array or an object? Usually I like to have

```javascript
for (var key in object) {
    if (object.hasOwnProperty(key)) {...}
}
```

if it's an object and

```javascript
for (var i = 0; i < array.length; i++) {...}
```
if it's an array so that they look different, and then as a bonus you're guaranteed not to rake up properties from up the prototype chain in the first case and to traverse the numbers in order in the second.
my last PR changes this to use the more preceise domain query: 
{
'match': {'domain.exact': self.domain}
}
In general the we use `get_id` instead of `_id` just because `_id` looks like a private variable, and because in template you can't access variables starting with `_`. Merging anyway, just FYI.
Seems like this line should go right at the beginning, i.e. only do any of the above groundwork if it's the right type of form. No idea of that'd ever make the slightest runtime difference, but it's I think a nice convention to be able to look at a view/filter and know right off the bat what doc type it's for.
Optional can help "document" the fact that something can be null or not. You could argue that ```@Nullable``` serves the same purpose, but Optional gives you type safety (and compiler errors), whereas ```@Nullable``` is just an IDE warning.

There are some other marginal benefits, like being able to say:
```String result = getSomething().or("default")```

Instead of the more verbose:
```
String result = getSomething();
if (result == null) {
   result = "default";
}
```
Can the `\n`s at the end of each line be removed?
We probably want to use .First() here, because FirstOrDefault can return null, which we don't want here. It's not really going to make a difference since it's inside of the else and we know there will be at least 1 element, but something to keep in mind.
Method name should be isShowFavorites()
I mean from run.sh to something else. It'd be a bit confusing for new coders.
Perhaps :p
Hahahahah oops. Forgot those.
Drop the ! from this, capitalize 'Save' so it helps point them at the button:

```
Login and Save your project to share
```
The style I put in the guide (wiki) is to omit the leading 0, and I've been doing so whenever I come across one. I know this is not good in javascript.... so could be convinced otherwise, for consistency's sake
This makes me feel like the don't need the settings.ALLOWED_OMEMBED_SITES? Or maybe the values in that could be tweaked a little?
Minifiers/compilers might rename ```StepperManager``` when processing pdf.js. This would introduce a bug, as StepperManager is declared in debugger.js, which might not be processed in the same process.
```globalScope["StepperManager"]``` should prevent this from happening.
It looks like this is testing the content of a file that's been uploaded to S3 by using a private method call to generate a duplicate PDF file.  What we're really testing here, according to the heading `it "should uploads only the Agi IQ when the merging flag isn't set"`, is that it uploads the Agi IQ.  We could stub the methods that return the Agi IQ and the Medistrano PIR so the return clearly identifiable content, then set a `should_receive` expectation on the method that uploads the content to S3, saying it should receive only the predefined Agi IQ content.  That would be a completely independent test with no external dependencies.
The problem was that ```verify_medistrano_pir_exists``` sets one instance variable when the PIR exists and two instance variables when the PIR doesn't exist. I didn't want to return an array with ```@enable_merge_iq_with_medistrano_pir_checkbox``` and ```@error_hint``` because when ```@enable_merge_iq_with_medistrano_pir_checkbox```  is true  ```@error_hint``` is empty. Any idea on how to improve this?
Let's remove this block and add an apache header.
ye h1 vase onvane site aval container besaz ba matn title site tori ke mohtavash dide nashe
va onvan safahat ro h2 bezar
I added a new commit to address this comment.
Can you also figure out how to vertically align the label with the select?
I just mimicked original spec, where was:
# remove glue-specific files
rm -rf app/models/glue/*
rm lib/resources/candlepin.rb
rm lib/resources/pulp.rb
rm lib/resources/foreman.rb
Should we also temporarily disable the checkboxes while the ajax request is being performed?  That way an over anxious user doesn't clickity clickity queuing up requests... :)
Second. It's weird, as I'm sure I've already fixed this file. WIll double check.
Huh? Doesn't this break when using homescreen.gaiamobile.org ?
And this whole thing also isn't wrapped in with which could potentially mean that file gets left open in case an exception thrown.
If we do want to do this, then I could simply rename ``cache_remote`` to e.g. ``download_to_file`` and then add a ``docache`` argument (and make sure the logic works out). What do you think?
Well, I feel that something is needed so that this section has it's own heading in the TOC. I changed to 'Introduction' which is maybe more useful? Or do you still think it should be removed?
validDays在什么地方设的？ factory的时候直接传进来的？ 没看到任何声明和检查的地方呢。
Also, should this perhaps be configurable?
Can you remove this chunk instead of commenting it out?
Ok, then 7f88c13 just needs to be reverted.
We may not need it, although it would prevent double execution via public/assets if we would've run into that problem. If you think we should take it out then by all means go for it.
Should be `foreach (` and `) {`
test the optional argument too
this is the same as in the other file: there are two identical blocks for setting the node information (the other one is in 71-82). We could move that into a dedicated function. It's not a big deal though, as I don't run phpcpd in my CI setup by default :-)
Inderdaad.
Hi, can you please rebase with the most recent source? I changed this variable name to remove underscore, so now your change does not merge well. Thanks!
greenify_watchers.events are not pollfd.events!
You're right.. I just copied it from the user suggestion and didn't think about it. thanks
yes, 
you can test in [here](http://codemirror.net/mode/ruby/index.html)
that putting `%%xxxx`on a line confuse ruby highlighter (also perl, and some others) 
So I use [mode mixer](http://codemirror.net/demo/multiplex.html) to switch mode after`%%` ( i would like to do it on the EOL, but I didn't found a way.) 

I would comment on `cell.js` for more explanation.
`$column` is not being used anywhere, can we drop it?
I feel like we talked about this, but I can't find a record of it. Maybe it wasn't on this PR. Anyhow, there is now a `git_repository_head_tree` function with exactly the same prototype as this in `repository.c` that can replace this function. I suspect it wasn't there yet when this code was written.
If this is a mixin-specific setup, then it should be better named. Something like "facet_setUp". Prepending a `_` indicates it's private and it's not part of the interface for the class, so it shouldn't be used outside of the class.
Yes! Much better!
Please add a point before unittestbase so we know it comes from the same directory and is not a global Python module.

``from .unittestbase import PostgreSQLTestCase``
Another thought is something hybrid. Note how I have a base class for postgres (``PostgreSQLCronApp``). Perhaps we could curry that on load time somehow so we don't need to have the ``database_*`` options on the `CronTabber` class. 

Then, likewise we could have something like a `BugzillaCronApp(BaseCronApp)` in there too to avoid having to add "global" options to the `CronTabber` class.
does `uncached` automatically do `no_transforms`?
we should be mindful of RTL, but yeah
I thought about that, but I have cases where I have a "toolbar" partial in most of my view directories so even though it has the same name as another, it's actually different. So I think this assumption would be safe *most* of the time, but not always. 

Thoughts?
Please don't remove the empty line here.
Why make a class for this?
Just make a Map<Vector, something that encapsulates id and data>
you should use trans or transchoice alternatively, depending if there is a pluralization for the constraint (see how the form theme does it)
I didn't knew that. Thanks, I'll fix it.
should this be specified in the context of the component ?
The FormRegistry being new in 2.1, it seems weird to have a deprecated method in it
I only moved it out of the original `KernelTest` class. I'll try to remove it.
The Play convention is that controllers are pluralized.  Change this file to Checkins.java
Thanks for the reminder. I was testing the gem on a few different machines and didn't want to squash the older version. Forgot to remove this portion from the pull request.
Your suggestion is better.

I love "Who wrote this dreck? Oh me..." :frowning: . Happens to me 2-3 times a day.
Same `update_attributes` play can be used here.
alan be ie fekr nakon
sari too chrome behesh tahvil bedim baraye ie dorosteh mikonim
chon amalan dar bazi jaha manteghe jquery ba zepto fargh dare
you shouldn't use exceptions for non-exceptional conditions. if you expect there to be a colon, look for one. e.g. `bean.split(':')[-1]`
This doesn't seem very thread-safe to me.  How about turning the UTF strings into char*'s by using ustring(blah).c_str() (it's persistent storage and thread-safe) and instead of returning a whole new argv, why not replace the argv[i] entries directly?
fix ur npe shiz
You could use "for i in xrange(IPV6_ASSIGNMENT_ATTEMPTS)" to avoid managing the loop criteria yourself.
Maybe better explain why is commented out by default :)
There actually isn't a classname by default, just #miniTip in the CSS file.
Please change "And" to "Given".
remove these so that it will be like the others
I wasn't suggesting a rendezvous, I was suggesting using Delays.sleep 
rather than Thread.sleep.

Either way, I would resolve this issue as you have and consider the 
wider issue case by case.

On 2012-09-10T16:17:07 BST, Michael wrote:
> In qa/tests/src/org/jboss/jbossts/qa/Utils/CrashRecoveryDelays.java:
>
> >      private static void awaitReplayCompletion(int seconds) throws InterruptedException
> >      {
> > -        Thread.sleep(seconds * 1000);
> > +        Thread.sleep(getDelayFactor() * seconds * 1000);
>
> Fixing all of them using rendezvous on a case by case basis is not a
> trivial task. How would we divide up the work.
> We also have lots of waits in the unit tests and those never/rarely
> fail so such a JIRA should focus on the QA suite only.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jbosstm/narayana/pull/91/files#r1566631>.
>
TIL && So Noted.  Thanks.
The new implementation should just check if the event target is inside the dialog. If it's not, then we should prevent the default action and shift focus back to the dialog. We'll need to track the most recently focused dialog in order to know which dialog to check against.
I used allocate("localhost/") and in that case when I ran get_children() on the root, it returned ["ookeeper"] (should be ["zookeeer"]). That's why I included zkpath in the first place.
why don’t you use a `getElementById` instead?
You did not change the detail of `appclose` and `appwillclose` event.
We have discovered that this way of calling the activity produces a 'bug' in SMS, because we break the 'hash' model of this App. Im gonna review it (there is an issue in git #4324). I think that first of all I have to create a method for calling the activity properly and then apply this patch, what do you think @albertopq ?
Can we do this:

if (!isScrolling) {
    return;
}
nit: `return (this._dialogStacks[origin] && this._dialogStacks[origin].length);`
I think we have an `Enum` traitlets class that should be used instead of `Unicode` here.
Maybe we will refactor this in full JS after.
One suggestion, specify: [default: %default] in the help. That will let the user know there is a default and what it is. Ditto for the others as well.
I removed it and all the specs passed. Want me to push a commit that removes it?
I'd recommend changing this to `@name = name.to_sym` and then remove `.to_sym` from all the places where you pass the name into the constructor or descendant constructor.
Aha, was wondering if that `updated` operation existed. Done.
all apps run repo
Ok, sorry, old convention.
Well, I've talked with the English teacher I know and she told me that both sentences are correct, but the most correct is yours, so... sorry :P
This doesn't belong in a Rails Helper.  Doing this makes this method available to the views, which isn't really what's wanted.
Style: One(?) blank line between class docstring and beginning of code.
This is taking DRY too far IMO. This part is more maintainable (easier to read and modify) before this change.
Why not use CCArray like m_pHandlersToAdd?
I think it is better to use the same codes if there is not any especial reason.
We need exactly "v110_xp" for VS2012. What exactly does VS2010 report as an error when compiling?
Why do you need this? in theory, all these builders should be private and interaction should go through the dialect methods.
Thanks for your multiple edits. SInce at least one is an edit to my edit,
I presume (I'm new to this process) I should:
	1) Cancel my pull request, and
	2) Apply your changes, commit, push
	3) Make another pull request.
	
And, do you have commit privs or are you an outside commenter like me?

Thanks
Ian D
master factory, creates internal components, allowing to replace them
*nitpicking here* I would suggest you use keytake or the like instead of any plus a fun, see http://www.erlang.org/doc/man/lists.html#keytake-3
Especially for physical switches the user may have fewer ports connected to OFTest than exist on the switch. Please change this to `ports >= len(of_ports)`.
I made this stuff pale ruddy brown because that seems to be traditional.
IMHO with https://github.com/gitlabhq/gitlabhq/pull/2188/files#L2R105 it isn't necessary any more. (there was a small discussion in https://github.com/gitlabhq/gitlabhq/issues/2165#issuecomment-11020827)
The `+s` will make sure that newly created files/dirs get the same user:group as the parent (which is git:git).
This should also solve the problems we had when mixing SSH and HTTP push.
Admittedly a slightly dodgy move on my part. My plan was to disrupt the API the least by simply subclassing the strings currently used to store alternative alleles. The idea is that for more complex strings (such as "]chr2:12345]NNT") extra information would get added to the object. Regarding the init function, this is an oddity of Python: because str is immutable, the sequence is stored not at __init__ but at __new__ (http://docs.python.org/reference/datamodel.html#object.__new__)
Also should have been removed from the repository
This actually looks fairly old code; the current master has:


```
    @rename.each do |old, new|
      next unless event.include?(old)
      event[new] = event.remove(old)
    end
```
> Note that execute shortcut doesn't necessary improve comprehension of this code.

No I agree it's harder to read this code, but it's way more optimal and this function something that is frequently used, so reducing closures introduced on each call is very important. Also see comments on line 162 that explains explain that.
It would be okay if we didn't have the domain query string parameter when principalDomain and fromDomain have the same value.

I don't think it hurts to be consistent and always send the domain query string parameter.
this email template was removed and broken out into three new templates
oh, I see, actually not.  Given that now we're going to *not* include type in list_emails response, this function probably will hang around.
Can we have the IMGs removed from <h1>. Deb says this is very bad practice.
I think you meant `if (typeof module !== "undefined" && module.exports)` (avoids ReferenceError when `module` is not defined)
wil/will
Will `zpool upgrade -v` return the same thing on `GNU/kFreeBSD` as it will on `FreeBSD`?
Given you know that options has the right form, is there any special reason it is worth creating the new array here?
I actually switched to mochijson from mochijson2. the actual output sent to the client was "erlangified". Single character strings were converted to integers. I couldn't find a way to prevent it.
This is a minor bug fix, should be 0.1.1.

```python
print "hello"
```

- list 1
- list 2
How about "Limit the number of collaborators of an object (the number of entities an object depends on)" and "Limit the number of an object's dependencies (the number of entities that depend on an object)"?
we already do this lower why did this change?
Maybe extract the above into a separate method?
Would we also use this inside the getDateValidatorMethod()? 
https://github.com/weierophinney/zf2/blob/437ff0b5ddb956a17072889e0a341733cd3b3187/library/Zend/Form/Element/DateTime.php#L140

ISO8601 has some subtle differences to RFC3339 (I didn't realize), so we probably need that changed to RFC3339 regardless in getDateValidatorMethod(). Cheers!
I concur with @Ocramius here -- I prefer to return/continue/break early as then the body of the work is not inside a conditional.
There is unneeded "/" character after #endif
Should be no whitespace here
DataService name here will need to be injected/configurable, but ok for now.
the eval is done on a constant, it's strictlty equivalent to do a file_put_contents() + include imo.
you don't need initial-scale
i just added it as a "catch-all" i'm ok with removing it
Not really a fan of passing the container. But I guess this is because of some restrictions?
We usually load eZ Publish namespace first, but that doesn't really matter. However, it would be nice to get only one, comma separated, `use` statement.
Could `system` not be hardcoded ? Would be better as a property, which would default to `system`, with getter/setter, so that it's easily overridable.

This way a bundle developer could easily implement its own parser for its own namespace, not having necessarily this config key.
@andrerom @emodric `eZExpiryHandler::registerShutdownFunction()` is ok to be called several times. There is a flag check in this method.
I'll need you to submit a new pull request without this problem.
@kracekumar did you load socialite.js while testing?

On 12/20/12, kracekumar <notifications@github.com> wrote:
>> @@ -27,6 +27,24 @@
>>              <i class="icon-thumbs-down"></i>
>>            </button>
>>          </div>
>> +           <div class="btn-group dropup" >
>> +          <a class="btn dropdown-toggle" data-toggle="dropdown"
>> title="Share on Social Networks">
>> +            <i class="icon-share"></i>
>> +            <span class="share-button-label">share</span>
>> +            <span class="caret"></span>
>> +          </a>
>> +         <ul class="dropdown-menu">
>> +          <li>
>> +            <a target="_blank" href="http://twitter.com/share"
>> class="twitter-share"  data-text="hasgeek.tv"
>> datavia="hasgeek">Twitter</a>
>> +          </li>
>> +          <li>
>> +            <a target="_blank"
>> href="https://plusone.google.com/_/+1/confirm?hl=en&amp;url=http://hasgeek.tv/"class="googleplus-share"
>> data-href="http://hasgeek.tv/" data-action="share">Google+</a>
>> +          </li>
>> +          <li>
>> +            <a
>> target="_blank"href="http://www.facebook.com/sharer.php?u=http://hasgeek.tv/&amp;t=Hasgeek.Tv"
>> class="facebook-share" data-href="http://hasgeek.tv/">Facebook</a>
>
> We need to pass exact url in facebook, else facebook will show all
> thumbnails in `hasgeek.tv`
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hasgeek/hasgeek.tv/pull/44/files#r2475653
>


-- 
Cordially
Abdul Rauf (haseeb)
then remove this #TODO
[`include` is an alias for `contains`](http://underscorejs.org/#include), which is already used in `rekapi.core.js`.  Please change this to `contains` for consistency.
At any rate, that sounds like a refactoring - the message is received the same way whether the exception constructor creates it or it's set by the raise call.
maybe :process is a better name?
if the errors are json, are you setting the content type to json?
Missing semicolon at end of line
Just had to check.
Twitterへのリクエストを極力減らすため、
ユーザがフォームに入力したtwitter_nameがnilでないか、DBに格納されている値と違う場合のみ
有効な値としています。
コメント追加いたします。
I'm not sure about this one. To me it seems like bibliography is a book section, but in one of the test cases it has a "host" and in the other it seems like it's supposed to be a book.
Should we just add support for this to BibTeX? I looked through BibTeX standard and keywords is not part of it, so since we're accommodating some customization, might as well be flexible about it?
I believe problemID is already a string, no?
Yep. `appendVars` specifically merges data-ensures and runs them all after appending. (compare to `appendContents`)
Might pick a wider range of denominators to allow more variety.  I'd look at standards to get an idea of what a good range might be.
Yeah it's awful. I don't even grok it yet, although I did clean it up some.

I'll experiment and see if I can get at the actual method inside FileUtils and call it directly, and then wrap that call in a support function in case we need to change it later.
Prefer re-opening eigenclass syntax here

```ruby
class << self
  def file
  end
end
```
One n. Also, link to phpBB profile.
Push this into the codemark `@model`
Not sure this is the best approach, but I couldn't think of a clean way of separating out the path-based and file-object based approach, so trying to generate either key type from the start seems the most generic.
System.out.println() tests nothing
use assert to test smth
If you use it in this single cpp file, please use the static keyword.
Heh, I was thinking this as I was working on those acceptance tests.  I believe that when I set the values to 1 second, it rounded them down to zero days.  Will see what I can do.
No, I didn’t; thanks for catching it. I’ll remove that note and make that Issue.
Why wouldn't this return an error if the user can't be found? The function calling this could just catch that.
I didn't realize a String `u:cam:simong` is what we were passing round the back-end APIs, as opposed to a structured object. I think it remains to be seen which approach will be easier to work with, but my Java experience, understandably, tells me to use a model object:

```
var principal = Principal.fromUuid('u:cam:simong');
var uuid = principal.toUuid();
```

Then we can pass structured principal objects around with low resistence.

If you guys still find this too difficult to work with, I'll let go and refactor, as it is mostly important that we agree on one and move on.

Let me know.
I'd rather not use this forEach. It's not used anywhere inside of the UI or Hilary codebase.
I agree, the shorter version sounds better.
You mean to move them to the companion object?
I don't see how that would help. What I need is a way to print the fully qualified name of any ``JavaElement``. If I'd use ``e.getDeclaringType.getFullyQualifiedName`` then I would need to add logic for printing method's parameters, which is not something I really wanted to do.

``JavaElement.readableName`` is the closest thing I have to get the fully qualified name for any ``JavaElement``. Though, for methods it removes the package declaration (go figure). Of course I could code up something more robust than the hack I have now, but I can't believe there isn't a method/class somewhere for exactly this purpose, so I didn't wanted to waste time re-inventing the wheel.

WDYT?
Technically, you should be using MCWorld here, despite other references in this file. (I'm going to make a bug report for the other occurances, as this pollutes the code).
Ok, not sure how I missed that. Thank you.
Please use braces here.
Either put it on the line above, or put it more indentation and do the same to the rest of the method parameters.
If it's like this...otherwise, no:

`/**
  * ....
  */
`
Да конечно, вы правы. Написал пример, проверил, получил ожидаемый результат. Невнимательность :)
Не знаю откуда взялась привычка - одна строка значит без фиг. скобок.
Стоп! Percentage - это процентный показатель! Хорошо, если length==100, а если нет?

Ну и, собственно, работать с percentage нужно как с процентным показателем - количество элементов для замены ещё нужно получить.
Yeah, I would think so. That would be my expectation...
Can we turn the initializer as a private method?
It seems all we need to do is something like:

```php
$initializer = $this->createInitializer(
    $documentPersister, 
    $reflectionId, 
    $classMetadata->getReflectionClass()->hasMethod('__wakeup')
);
```

And then:

```php
function createInitializer($documentPersister, $reflectionId, $hasWakeupMethod) 
{
    return function (Proxy $proxy) use ($documentPersister, $reflectionId, $hasWakeupMethod)
    {
        $proxy->__setInitializer(null);
        $proxy->__setCloner(null);

        if ($proxy->__isInitialized()) {
            return null;
        }

        $properties = $proxy->__getLazyLoadedPublicProperties();

        foreach ($properties as $propertyName => $property) {
            if ( ! isset($proxy->$propertyName)) {
                $proxy->$propertyName = $properties[$propertyName];
            }
        }

        $proxy->__setInitialized(true);

        if ($hasWakeupMethod) {
            $proxy->__wakeup();
        }

        $id = $reflectionId->getValue($proxy);

        if (null === $documentPersister->load($id, $proxy)) {
            throw DocumentNotFoundException::documentNotFound(get_class($proxy), $id);
        }
    };
}
```
We should have spaces before and after the plus signs. Even though it wasn't there before.
here again


can you add a deprecation warning?

```
import warnings
warnings.warn('Constructing a SourceEstimate object with a filename will be deprecated in v0.6. Use read_source_estimate.')
```
Good idea about the warning. What is the general rule for the minimum epoch length?
`None` is the default return value. Unless you want to be explicit this isn't really needed.
Correct. I actually copied this from master, so I need to fix it there too.
With the stub above, this does nothing
Unfortunately, no luck. Updated [gis](https://gist.github.com/3217282)t.
I don't agree on this one. For the purpose of this article, I'm using the exact same approach than the one documented in the [How to handle File Uploads with Doctrine](http://symfony.com/doc/current/cookbook/doctrine/file_uploads.html) article, and I explicitly refer to that article in mine.

I concede that this is probably not the cleanest approach possible (in our agency we have since opted for a more structured - and also more complex - solution)  but if the reader relies on the official documentation to handle file uploads, then he  will be familiar with what I use in my tutorial.

If you really insist on changing that part I will, but I think it will be ok for most people - and more advanced users will have no problem adapting it to their own approach.
I don't think the config component is documented at all yet.
Agree, that might be helpful, but we would have to parse the config as well (parse the string input from CLI into the proxies object). Let's not worry about it for now...
Because it still lurks in some places. Is there a good reason to remove it entirely?
what if there's an exception raised here?
Do we really want to change the default? @dizzyd, @joewilliams, @hyperthunk?
Still don't know where this line is.
pas de commentaire commençant par #. le # est réservé pour le builder de jelix.
И вообще, делай `query.finish()` после SELECT-запросов. Иначе у скулайта/кутей есть какие-то проблемы с высвобождением ненужных ресурсов.
Еще, кстати, если URL известен, то можно возвращать  и его через setUrls.
``$auth = $config['auth']`` and you're done :)
Entendido. Yo no tenía claro de dónde venía `date`.
Does this mean the verb is NULL for cached requests?
I thought to update pref getters in entire app
You must not rely on exception message
we need to disable the cache while this runs, to avoid a nasty bug

also, we should resave them all AFTER, to
1) be sure it fixed the bug
2) retrigger all the caching we didn't allow to happen
Ok, but the 20-line comment block above needs to be updated.