labofsahil/github-event-dataset-2012 huggingface dataset:
Does this work right in Velocity? What does the type parameter get resolve to when you call this method from Velocity? List<Object>? Does the DiffUtils utility method properly handle this?
Confirmed, kan weg
значения randomIndex могут повторяться и тогда получиться, что несколько раз будешь менять значение одной ячейки массива dna
В примере используется ДНК длиной в 100 знаков, соответственно процент это количество ячеек, зачем усложнять код в конкретной ситуации? Мы ведь не пишем универсальную программу. 
Если руководствоваться принципами экстремального программирования подобные изменения вредны для кода.
Мне кажется такая конструкция неоптимальной. Могу ошибаться, но, кажется, что каждый цикл будет перелопачиваться весь массив для подсчета количества элементов в нем.
эта строчка бессмысленна, поскольку randomIndex == randomIndexAtDNA из-за того, что описано выше
Почему в интерфейсе нет метода getDNA()?
http://stackoverflow.com/questions/1292862/nsstring-isequal-vs-isequaltostring лучше использовать другой метод
In English Parameter Titles Should Always Written In CamelCase, descriptions are normal text.
Good point. I missed that.
unnecessary indent here. If that doesn't match your editor, check that you have it set to use spaces instead of tabs.
*scheduler
get rid of this debug print statement
is_fixed_ori is always False
epochs : instance of Epochs
    The epochs used to generate the time frequency representation (TFR)
Insert openmrs license
No need to add brackets here
Is it possible to using @Test annotation in abstract class, which would avoid to duplicate all  in final test impl?
Bug fix, this line should have used the $charset variable which is set in line 70
You update each with the same params value?
这样就能够在任何时间随机访问，或者时间过半时开始回放。

这样就允许候随意拖放，或者从节目的中途开始回放。

意译的
Please don't update that
private def notifyCallback(message: AMQPMessage): Unit =
  for(cp <- channelParameters; cb <- cp.channelCallback if !cb.isTerminated) cb ! message
Я дедфуд и упарываю UDisks, мне можно.
Does the introduction of 'utf 8 support' solve a specific bug?
I wanted to keep the number of arguments for the progress callback small, and these 2 values seemed irrelevant. I'll remove them completely.
I'm thinking about something like this: https://gist.github.com/2732200 (just an untested outline)
Welcome to RSpec!

Could you rename those to: `it "should render when true"`? :)
Nothing's wrong with it, that's just what was there before, I just removed the line break that I didn't like looking at.
This is violating the API: `withFlags` is documented as to not change the Cache it is invoked on but that changes need to be invoked on the returned Cache.
Hence I would not expect it to change the current Cache..
This is both insufficient, as Adrian said, and overly broad, as a topology change doesn't mean the originator of this command left.
Pull req: https://github.com/danberindei/infinispan/pull/6
Actually, while awaitTermination() might be clearer to read, it has to be wrapped in a try/catch
Hey, I copied this one from another method and I've since looked around and every setParameters method has different IllegalStateException message and some don't even check this.https://issues.jboss.org/browse/ISPN-2391
That's old naming, particularly in AS envs where you could have multiple cluster partitions, which essentially are multiple clusters. I'll change it.
PEP8: triple quotes for docstrings should be on its own line
I don't see why we can't provide magic numbers (or maybe even strings) for these keys.  
In my eyes, since you have used the ```{}.get()``` method further on, this commented out section should either be removed or the mouse buttons supported.
I just copied the whole thing from the output of boilerplate.py. I could revert it though I'm not sure how to revert it properly in git.
83 columns
Okay, so this looks like the comment was addressed.
You meant imgFilter
oae-hidden instead of display: none
We should avoid using the native JavaScript `removeAttribute` in here since we can use jQuery `removeAttr` [1] for this.

Apart from that I'm not even sure whether we need the `.each` in here.

```javascript
$('link[href*="skin"]').removeAttr('href');
```
Does this work?

Looking a bit further in what this function tries to do, can't we completely remove the `<link>` element?
Instead of removing the `href` attribute?

[1] http://api.jquery.com/removeAttr/
That won't work since you can't add a class to a data attribute. I guess the selector could be changed so that the filter is used to select the object but that won't fill the filter variable anymore. I'll leave this as is.
@christianv Can we remove these divs? I don't think we need them anymore.
please use SG_ADD here too
Ack! I don't know what I was thinking. I'll fix this.
I thought of that but since smeranda's version left that out so did I. Doing the full init on resize would probably be for the best.
If I test for the id and not for the hostname, it would not be possible to specify an id for the created publisher (see example 2)
Newline before Returns
im not entirely sure what the precise deal with this is:     doing /random.jpg throws a 404.  I think the #.jpg is needed to post the image in campfire? not sure... thats just how the kitten script was set up.
I had difficulty understanding whether the stats should be a snapshot or not from inspecting the file and connection statistics implementations. I decided to keep it simple at first.
My concern with that is that if the image resolution is too low the image will be stretched out (I don't think that was their intent) and the solution would be to upload a higher resolution image (and min-width would never be triggered0.
Are you sure ``$section`` has the correct type as ``integer|string`` is not a sub-type of ``string``? Please also check line 133.
Have you created a bug in JIRA for it?

Also please add a `// TODO: remove condition once clearInterval doesn't choke on undefined param`
I thought the node.js idiom was `err == false`?
Just curious: why change this from a function declaration to a function expression?
Did you see this occurring in practice? I don't quite recall how this would behave if the prefs are locked, but I don't think we need to handle that case, since that would only occur in heavily customized Firefox builds.
But isn't the calculation to see if it's bookable the same no matter where the course is displayed?
Why the change from old to PREVIOUS? It doesn't seem to be related to the issue?
is json a dictionary here?
This ignores emacs autosave files, which are by default the file name with a tilde at the end, like `.gitignore~`
Indentation.

And there are easier ways to pass this data if keys in the dict are the same as keys in the document model. Which they should be.
No need for `staticfiles`?
Hm, the only thing you are doing here is making data mutable?
There is no use to limit `#boxHeader` with a class. `#boxHeader` uniquely determine the element (it is an ID).
As I said, this does not seem to me as a good approach. Shouldn't you simply use `default` argument to field constructor?
Please order tag args same as they are ordered in other tags.
Is `urllib` needed?
Where is this file used?
*403 Prepovedano*?
So if this errors out then it will return `{{error, Reason}, State}` but I don't see that in the spec for this function.
This breaks the ability to send whispers as a GM to a DND player.
This is code is repeated in several classes. Maybe you can create a `WilliamResource` which `Invoice` and friends can inherit from with the comment behavior.
Actually how about `subtree`? (Or `descendants`? Since we're assuming 'parent' is obvious, I would think `descendants` is clearly in the opposite direction of the `parent`?)
focussing -> focusing
I think it would be weird for `click` to log coordinates, and `click_coordinates` not to, so I think it should stay in there. Should I add a test for this?
For consistency, please rename this class to 'AbstractEntity'. Also, change its visibility to default by removing the 'private', as all classes extending this base class are in the same package.
`#endif  // NET_INSTAWEB_NGX_URL_ASYNC_FETCHER_H_`
I think this is a really powerful statement - useful when designing tests, acceptance criteria, documentation and design.  Perhaps it's self evident, but is there any further reading on this that we could link to?
i wanted to keep CreateBundle storage agnostic.
Why do we have these changes here?
Since this is a public method, let's document the behavior a bit more clearly:

- Clarify that this is the closest point across _all_ series.
- Does this calculation include invisible series? (It probably shouldn't)
- Is the coordinate system for domX and domY the same as one of the three described at http://blog.dygraphs.com/2011/10/dygraphs-coordinate-systems-33.html? If so, use the same names that kberg does. If it's not, is there a good reason?
Taking out the `show` parameter mindlessly means that the listener for `"code.ext:defaultbindingsrestored"` breaks.
Totally unrelated to correlated noise, but: why not?  What determines which properties do or do not get copied?
ok; I'll reword it.
Is silent failure here a good idea?
Difference in naming for getUnclaimedMachines() and unclaimed(ops).
Same for getAllMachines() and all(ops).
I'd favour methods having same name, with overloading I think.
@ahgittin suggested this moved to EntityUtils, but it feels better here. We know we have a SERVICE_UP attribute, so i think this should always work. will document though.
non t'y vas beaucoup trop fort, ce n'et pas toute la music and movies industries qui font ca. Ce sont les recording industries.
En effet :)
Yes , there is no backslash in C standard syntax outside a string.
Ah, good point. How far back should I test?
This going in our yum repo?
line length is 77
I think you mean "typically by adding it to `settings/local.py` with an awesome editor like Emacs::".
Hey, this call is made too early.
Just curious, why do you need to do100%  + 2px?
Thanks Thomas,

I'll make the change tonight and submit another pull request.

Daley

On Tue, Feb 28, 2012, at 04:01 PM, Thomas wrote:
> > @@ -1740,7 +1740,9 @@ def magic_run(self, parameter_s ='', runner=None,
> >              main_mod = self.shell.new_main_mod()
> >              prog_ns = main_mod.__dict__
> >              prog_ns['__name__'] = name
> > -
> > +        
> > +        prog_ns['execfile'] = 'py3compat.execfile'
> 
> This needs to be the function itself, not the name (i.e. lose the
> quotation marks)
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1452/files#r499306
> 
--

-- 
http://www.fastmail.fm - Does exactly what it says on the tin
looks like a lot less code, though user and password (as well port) shouldn't be assumed
Ahh.  I didn't realize percona added the repo itself.
Which would be? :|
Ah-HEM!  You need to check your parentheses. >:(
Dupa '//' trebuie sa existe un spatiu.
Good catch. I have #if (HSE_VALUE == 16000000) later in this file to choose between 8 and 16MHz but im gonna clean that up.
I've been putting PP first.
These are fine defaults, but we may want to have this configurable - for instance, I can see not all people wanting INFO setting, or wanting to have it log somewhere else
use consts ES_CONFIG & ES_PLUGINS instead of "es.config" & "es.path.plugins" ?
Coding style:
Conditional Execution: One Line >> Three/Four Lines, even contao uses inline-ifs (ternary operator), which just cant be used in this case for 5.2 compat (?: only works in 5.3+)
SQL: Writing the whole query in one line with inline-ifs and shit-load of parentheses is no coding style.

strlen is necessary because "0" is a valid where or order by expression.
I think this is only ever called from MPMetrics, which always called with the Application context, as it should be, so I'm not sure what the point of a Context -> instance map would be.
so, i have not figured out how to do it yet, but given that all of the tests from here to the end of the file are the same, i would be looking for a way to do them using pytest.mark.parametrize
i find it strange that strftime can give you 'Monday', 'January', and '2012' but not the day. at least i think that's what's happening here.
we should use ' instead of "
http://code.google.com/p/selenium/issues/detail?id=4468
Thursday, eh?
ドメイン貧血症に対する皮肉というか、hopefullyなので「ちゃんとやってればそうなるよね」的なニュアンスかなとは思ったんですが、ここで突然ドメイン貧血症の話が出てもこれを読む人には唐突すぎる気がしたのでざっくり無視してます。

誤読の可能性もありますけど。
-f had some weird issue of plain not working, it was quite fun to debug
Same as above, ``k`` could be included in the warning.
I think it should be "as close as possible *to* the"
Note that I've changed your single `;` statement into a `{ }`. This is deliberate since a lone semicolon might issue a warning/error on some compilers.
if you using this practice, please use it all places where you freeing memory
Let's help Martin generate the correct AMIs for each region, and document the appropriate AKIs, then fix up `create_micro_bosh_yml` and then we can remove all this edge-case section of the tutorial hopefully!
Were you not able to use my munin module?

https://github.com/efroese/puppet-munin
It's not about ensuring anything, it's about actually doing it. It should probably be "generateApiClass()" or "includeApiClass()" or something that shows direct action.
Already mentioned this in-code, but this solution lets you cycle through all of the kernels in the matrix, then apply them to the corresponding pixels.
That would require me to be prepared to change a non-huge heaplet into a huge one (which would complicate things somewhat).
I thought you would be comparing elements in two distinct lists: transform and record fields.
Agreed.

Related to previous comment, it could also do a better job of streaming the local set, rather than gathering it all first.
Probably via a lookahead, so it can listen for request from broadcaster.
Ah ic, retry() = prepare for another run(), not actually rerun itself. Had forgotten it's function.

But, I'm still confused..

retry(), will be called _before_ run() has ever been called, it will be called if a prepare() failed in the transaction, correct?
So why does retry() need to do a : doRename(<original name>) ?
Yes it is defined as a URI (URL or URN) in the specification, with no restrictions beyond that.  It doesn't actually have to refer to or locate a resource.
Should this say ```return false;``` ?
You're right.  I'll double check what happened there.
Don't we want to make sure it's positive?
Why "msg" when all other parameters have a nice human readable name? I suggest "message".
Use warnOnly or just catch the expected exception and not everything.
try catch for any assertions.
Looks like your indenting is wrong here.
No callbacks.
Please remove debugging junk
woah... a bit embarassing that I didn't know that one. Thanks :)
The title should be something for the users, not just a repetition of the name. CamelCase isn't something for users...
According to your comment here: http://groups.google.com/group/colorbox/browse_thread/thread/d64609b27148e098

Probably data('group') would be a better name here
Remove above branch, and perhaps do
```
float angle = atan(direction.y, sign(direction.x) * max(abs(direction.x), 0.001));
```
There must be a less tricky way to do that, or we can put it in a new `czm_` function.

Less dynamic branches is generally better.

Strach that...

Actually `atan` is really expensive on the GPU.   Since `direction` is hardcoded, let's change this function to take `angle` instead of `direction`, and then hardcode `angle` for now.  It is possible that the optimizer is already optimizing the `atan` call, but I wouldn't bet on it, especially on mobile.
No technical reason, style preference.  I think horizontal space is too precious to waste on indentation if you can avoid it.  Personally I think readability goes the opposite way, and would rather see functions return early than have 100-line if blocks.  When reviewing this stuff, my comments shouldn't be taken as demands, just things that I would have coded differently if I had done it, and you're free to ignore any of my suggestions if you want :)
we're checking the change in length specifically for the two cases you mention. If the length changes, the polylinecollection needs to know so it can recreate the vertex buffer. if it doesn't change, but the values of the positions change, it needs to update the existing vertex buffer.
Isn't this 0.6.1 as of the last release?
No, the test from this class have been extracted to other test classes. I'll remove the test class.
Note that the value for the row's table is not returned by `editable` in the case where it is `undefined` on a row (its default state).
This comment isn't good.  This should probably just happen when initial data is set in the future, this probably won't need to be directly called.  Comment should just say it adds the post data to the form if provided.
Nope. I can create one if you wish. Where do you want them?
I also recommend testing for :undefined
Is this pep-8, may indent to 1-char after paren above?
I reintroduce this scope in main develop branch of EsupPortail project because this artifact is used by some java classes (not just test classes)
I try to make a habit of it the explicitly check for required dependencies, like Boost is in this this.

Very often, dependencies are optional, and just enhance the features of what's being built.

I don't see this as something that's really required, but I consider it good practice in the easyblocks I write. So, up to you.
タグの件についてドキュメントに追加する> @kentaro
Okay, let's unify these three? four? (essentially identical) wipe clauses.
Not true!  On desktop, `"History" checkbox == { "forms", "history" } engines.`
I personally don't think it's worth it, because the user can just click back into the engine selection to see what they've selected. Confirming that the preferences have been saved seem sufficient to me.
I'm not sure. I think it should try indefinitly.
This is  from another pull request, right? The one that you accepted. 
I was looking at the issue about the infinite loop caused by facebook not sending the right permissions. I realized that sometimes, facebook was sending the permissions, but as request.facebook was set (but to None), we were not checking the new permissions. This change did not eliminate the problem completely on my end, but reduced its occurrence.
But i'm sorry this is in this pull request. It's actually unrelated to timezone issue.
I really like the dividers :sunny: :sunflower:
@var forgotten + comments
RGB please
You do not use this one any longer /methinks
Почему бы не внести это условие под внешний if?

В следующих аналогично.
These arguments are backwards -- it should be 

`@param {String} dialogContainer a jquery....`
Estoy de acuerdo, pero si los PDFs generados están bien ahora, yo aceptaría el merge.
evt isn't an ```OpenLayers.Event```. The latter isn't a type, but just an object containing functions to work with events.
DbgHelp.get returns a wrapper struct. I don't think that an explicit type makes sense there.
Don't depend on the order of evaluation of arguments.
Anyway, all pushed
Just validated its not necessary, but  i'll keep it to maintain consistency with the attribute name.
s/~~The~~/This/

Also, it's only 'true' by default if procedureParameters is null.
This convention is different than elsewhere. Elsewhere it has been 
.div-class
  = markdown(@object.field)

Why markdown HAML preprocessing is needed in this case?
please do not mention this one, we only included it to work if people do guessing. or put it as alternative for Double - it is just an alias
I found that I needed to set this to nil, otherwise it will be non-nil after returning from the call to NSJSONSerialization even if there was no error.
I'm not sure if it's the issue: we generally used 2 spaces to indent in Webots, and never tabulations.
I think it's a good rule to adopt, because every editor see the spaces the same size, while it's not the case for tabulations.
Setting your editor to fix these indentation issues is the best strategy to fix them all.
mocha is using the done callback as the last argument, but anyway I'll change it.
Same thing here: this will not pickle, and if people try to use it with Parallel, it will break.
Actually that is a really good point, no need to remove it unless you want to, I was just wondering what your rationale was.
I have to say I'm not a big fan of injecting code into existing Leaflet classes like this - is there any interest in taking any of these upstream @mourner?

Obviously things like toBBOX are probably a bit too specialised for that, but maybe they should just be separate functions?
it might be nice to keep the minified version around and just add a .min.js after it, just in case we decide to get fancy.
This might be the answer to the serialization bug. However, if we pass NULL, there's no way other threads would know of the dynamic flag.

Maybe creating yet another global as the small, default property for Roxie packages?
:+1: Done
What des this row mean?
Please address this comment
Remove the duplicate return description here (the top one)
This should just be UI.SIZE.
Seriously? I think this makes it easier to read than using a halfWidth or widthByTwo variable, and I don't think the CPU will sweat over dividing a number by 2...

If you feel strong about it, I can change it though. I usually code with the mindset to strive for easier readability of code rather than micro improving expressions that the JVM will optimize away anyway.
pls use section directly. This shouldn't be an own paper, but rather included into the entire manual
I think *"convention over configuration"* is good for this case. Settings trend to grow a lot and become **complex**. So anything we can do to keep it **simple** is a good thing.

I don't know why people use `LOCALE_PATH`. Maybe the reason is that after read the first documentation point they stop and get it done. :)

The Zen of Python is a great and inspiring poem, but it's not a manual. :) Anyway, *"Explicit is better than implicit"*, but *"Simple is better than complex"*. So IMO *implicit* is bad if it creates *ambiguity*, which doesn't seems to be the case. ;)
The point of this is to illustrate most of the features of github.
Is there any way we can do this once, at initialization time?
Sorry, but I don't think I can accept this abusing of the FactoryBean. There must be an approach to this problem that does not violate a core contract of Spring.
Функция для добавления узла в дерево. pNode - родительский узел, node - узел, который нужно добавить.
Sorry my mind was still on invoke interrupter. Yes we should probably do once here.  I'm not sure I understand about the dangling callbacks. If the user provides the callbacks we add them. We then remove them when they are used on their specific ID. We should never trigger the same one again. If the click is not clicked on a menu item, then yes we will not remove the callback. I guess we could remove both callbacks from the dismiss handler since that is triggered every time.
Can you cache this so we only go over the bridge once?
this class must not contain such logic
select id and etag only
FWIW I meant the newline added by using

task = BatchTask(parent=batcher_key)

AND

task.Populate(method, *args, **kwargs)

not the newline after the for loop.

I still maintain that task creation and populate being distinct makes the code more readable.