labofsahil/github-event-dataset-2012 huggingface dataset:
No! The TODO should be on line 51, and MimeType.JSON should be MimeType.JSON_UTF8. Guava does this really really well:
http://docs.guava-libraries.googlecode.com/git/javadoc/com/google/common/net/MediaType.html

All of the content types have charsets baked in!
Missing `public` and bracert should be in new line.
You should use the tag on the filesystem loader instead of handling it in a special way
you should remove the ``getParent`` method to make it inherit the proper one from AbstractType
no need to cast here as ``==`` always gives you a boolean
Can you show me an example?
Please remove it. There is no readon to mark dev-master as an alias for 2.0-dev here (it is useless as the alias for dev-master is ignored if you are not in the master branch, and it is wrong as dev-master is **not** 2.0-dev)
in any case I would not trust precedence and add parenthesis... Hope it's coding standard too
shouldn't be better to use an `InvalidArgumentException`?
This is also not a change in the phpdoc and I would argue that we want a `PropertyPath` here (`ViolationPath` also implements `PropertyPathInterface`).
You're right, my fault! So the final pattern should be ```/(\[[^\[]*\])|(\\)?([MDdw]o|DDDo|d{1,4}|D{1,4}|M{1,4}|ww?|YYYY|YY|a|A|hh?|HH?|mm?|ss?|S{1,3}|zz?|ZZ?)/g```, if you introduce the ```dd``` format too. And if I didn't miss anything again.
Where do we need this?
I prefer false as the default value or use abstract method so that subclasses are forced to make a decision explicitly and consciously.
Is an explicit return not required after the render call? After the before_filter has been applied, it will go on to execute the ```show_summary``` method which is not what we want..
no space after `:`, settingsMenuController cold be placed to local variabele
Could we do this in a Factory Girl factory, and if so should we?
chera position hato injoori midi ?
ali.md/nib 'fixed: bottom left right;'
I've removed it, but made it possible for subclasses to easily implement such a thing in the future (and minimised the number of changes I needed to make at the same time :smile:)
Так мне нужно передавать результат во вложенные вызовы.
в тесте это есть тоже. Для чего?
Hey! <img src="http://cdn.memegenerator.net/instances/400x/21244806.jpg" />
misses a ``u`` prefix, I know this wasn't there before, but just noticed.
I'll comments here too.
> What's the point of displaying currentState in the message if there's no currentState?

It could be any falsy value. I wasn't sure how to phrase the message, so I just made it output "null" or "undefined", depending on what currentState is set to. Is there a better to put it?

> Since this only effects initialization wouldn't it be better to do this check in the code that handles initialState?

If it only affects initialization, we can scrap this commit, but I'm not sure whether it does. I wrote this before the other commit (while I was tracking down the problem), and I just figured it can't hurt to assert that currentState is set.
this User_Profile is an id, the others are classes are you sure its working? maybe you can turn this one into a class just to make it consistant
Hi Amos, was this change required?
Could we make the comment more concise and just describe what it does rather than referring the user to the whole internal dev discussion of what it should do?
L163-L169 should be move to List Menu and have the module handle the keyup event itself.
But the type=reset is not meant to close/cancel a dialog, is for removing the content of an input element right?
I mean is not the same, and we have kind of 3 different button styles, Grey (default), Blue (affirmative), Red(negative).
Remove also change this to `evt.detail.origin` here.
I'd use this.audio.currentTime here instead.
`new Date().getTime()` -> `Date.now()`
nit `date_src` -> dateSrc per our coding guidelines
frame.setAttribute('mozbrowser' ,'mozbrowser');
We shouldn't be using those styles so let me just get rid of them and do some testing... ;-)
The indentation looks wrong here.
expand those rules
`settings.createLock().set(cset)`
Is this being used as a synonym for fill?  If so, please append another word to elaborate.
do you really want to merge that line? :)
Nit: I like when assertions say what they're asserting so test failures are pretty.
`--format=zip` ?
nit: Reading codes I would think if this returns true, it means it will be filtered.  Turn out it is otherwise.
I'm using that in my sub branch, but I can bring it back :)
I guess you want to remove this space :)
This translation is in english.
please add JBool and Boolean:

Metric.from[Boolean] = { (x,y) = if(x ^ y) 1.0 else 0.0 }

and similar for JBoolean.
That's exactly what I was thinking. I'm not sure if there's any technical reason why this would not be preferred.
OK. So clone directly into session.ext assignment.
tearDown?
Document this method, and explain why all it does is validate the app id.
```
resto:    threadID
```
!
e.getMessage() returns a string that usually follows this format: "Unsupported major.minor version 51.0 at java.lang.ClassLoader"

51 indicates the compiler version ( http://stackoverflow.com/questions/10382929/unsupported-major-minor-version-51-0 )

So you could print the exact version of Java they need.
Minor stylistic changes to this and the following function.
The security conscious person in me says that this should be ON by default. If someone turns it off, at least they had to think about doing it, so they've made a conscious decision (or copied someone's bad example somewhere on the internet).
Is the underscore here intended?
I think the name *hash_collision* is easier to understand than simply *hash*, i.e.:

```
hash_collision value:DERIVE:0:U
```
There is missing test case in master in test/fixtures/relative.js for `require('../foo')`. I added it and got failure. Since the fix simplified require.normalize() much of test cases for it became irrelevant.
im down with regJSON but probably not indexJSON, especially since components are supposed to be smaller I dont see people doing something like:

```
config/
  index.json
  prod.json
  dev.json
```

etc
improving trait code
Yep, recreated with cleared whitespaces.
ソーシャルニュースネットワーク → 空白削除
大幅な書き換えとスパゲッティコードからの移行 → 移行の漢字違い
Models、Collections、Views → 空白と余計な点を削除
not by email - it's probably a bad copy/paste.
Sorry, didn't mean to include that with my pull. Fixing.
Extra brackets?
Guess no gain no harm.

Would something like this be helpful? https://gist.github.com/2852267 As far as I can tell that part of the code is the same in all these methods.
Well, some of the coercion is done to handle cases like :validate => array while is much more readable to be a single-value

format: "json" vs format: ["json"]

The validation values might need to be tweaked slight or just some light coercion is necessary.
add validation on the body
add validation for sending place holder (Bug on IE)
Is there going to be a main module in firefox?
What is it going to be?
Typo: this should be `promise.resolve(result)`.
@ozten - I'm not seeing any obvious tests for this condition in tests/verifer-test.js - a test like this would have to be handled by vows.  The QUnit tests never interface with the backend, all the network requests there are stubbed in.  

@callahad - I hear you are heading this up now, could you explore this?  If there is not a vows test for this condition, we should get one.
Strange.  I expected that it would ... The same changes in that commit on dev were in a different pull request from a branch (for issue #1650).  I pulled the change into this branch as well so I could run the mysql test suite.  I don't understand why, after rebasing, this is still showing up as an addition.
You're absolutely right. I have this set back to proxyidp locally.

It does make me wonder if it will break non-shim clients. If they test for auth and provision properties, they will be fine, but if they test for type === 'primary', they won't handle this case. CC @benadida @anantn
Well it doesn't have to be a paragraph :P. I changed it anyway but stuff like this is just a semantics debate; one pro of using span over paragraph is so you don't have to declare paragraph as inline.
nit: missing space between if(
wondering if you're repeating var current_env, var confFiles and confFiles.forEach in each test file, if you can refactor that up a level higher and include it in each one as a one liner?
Sorry,

I using the old code. I have change it to html5.

On Fri, Aug 24, 2012 at 12:31 AM, Hilary Ho <notifications@github.com>wrote:

> In Graphics/index.php:
>
> >  <head>
> > -<meta charset="utf-8" />
> > +<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
>
> Why did you change the Doctype and meta tags?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/nvcHilary/CoinCod/pull/4/files#r1446686>.
>
>



-- 
Regards,
vincent
This could cause problems w/o validation, but this may not be the correct spot to validate it.
This section creates the necessary markup and allows you to stack your HTML for a mobile/tablet layout and then display them differently for larger screens.
don't agree with this change
nooo it needs  to die haha remove alllllll of these
Setting percent widths on things scare me... shouldn't be necessary most of the time. If something's display:block and not floated, you don't need to set a width. If it's not 100%, then why set a percent width instead of a fixed px width?
OK, I finally figured out the issue, you are right. Let me think how to solve this.
Can you document why this is so (just a line or two)? Admittedly, I kind of hacked this piece together. Would be nice to have a proper explanation.
ping @Palleas, you should really fix that I guess?
Its not global, it just needs to be change to private
Store the ID in the datatable else this won't be persisted.
-> `completed: false,`
Any reason you can't just install the ejabberd [binary](http://www.process-one.net/en/ejabberd/downloads)?

Installing erlang and ejabberd from source took like forever.
I agree, was a mistake of mine. I would put it to 2.0.x then if this is prefered :)
Hehe got it :smiley:. We can basically introduce a `config.yml.dist` (like former `config.php-DEVELOPMENT`), with basic settings. The environment is not applicable here IMHO.
I'm not sure you want to run the closure right away :wink:
just to play devils advocate here, rather than using jquery to do native JS things -- something like this?

```javascript
if (self.options.ignoreErrors.indexOf(message) > -1) { //...
```
ex:
```javascript
["foo", "bar", "baz"].indexOf("bar") === 1
["foo", "bar", "baz"].indexOf("snazz") === -1
```
`Array.prototype.indexOf` doesn't exist in all browsers. And the point about jQuery is correct.

Just do something like:
```javascript
for (var i=0, j=self.options.ignoreErrors; i<j; i++) {
  if (self.options.ignoreErrors[i] === message) return;
}
```
make this into an extra list ['amd', 'include','amd.h'] ... since we're using os.paht.join later on...
This doesn't flow "To implement EJB security, you need to The application will use a security domain that needs to defined in the.... " If you start a list with the word "to" you need to start each item with a verb.
nit: dedent to align w/ reverse
I thought that we were only checking that the links were correct for the footer (not that the urls were valid) on each page, with the exception of test_about. This is because the footers are currently identical and it adds a significant delay to the test to do the url validity checking. See test_mission and test_partners for what I'm talking about.
I can't tell in this test who this user is or what their attributes are. I think we should have a helper function like this:
```python
def get_logged_in_client(**kwargs):
    u = user(kwargs)
    client = test.Client()
    client.login(email=u.email)
    return (client, u)
Might as well do it now, eh? We had lots of folks sign up for Mozillians after Mozcamp Asia.
The reason to do the digit thing at the beginning of the version is because there are (or at least were) other keys that could be in the section, such as platform=foo. Until we put in the "must start with a digit" requirement, we were picking those up as fallacious versions.
that's cool :)
brace needs to come in 2 spaces.
Or, you could just call it `is_staff` maybe.
I would make `Therefore Java has to be installed on the node first` the 2nd sentence in this line
That's what appears in the A2 field (the PsycNET translator actually changes it to ED). I'm not sure how this can reliably be split. Either way, it would have to be done in the PsycNET translator either before or after RIS. I'm leaning towards before. I'll cook something up in a little bit.
Oh, I assumed the "" was to cast problemID to a string. (For example, that's [what the CoffeeScript compiler does](http://coffeescript.org/#try:%22%23%7BproblemID%7D%20%23%23%7BproblemSeed%7D%22).)
c5 requires 5.2, so it doesn't matter :P
Well, as other capabilities will be exported in future pull requests, we can just leave this detail for the end.
a user
After looking at this, you should probably refactor some of this into `Gandalf.Collections.Events` as a standalone `findOverlaps` method for any events collection.
What is the benefit of that? I think you lose some API flexibility, because you'd never be able to insert a hard-coded value. IMO the API shouldn't care if it is a dynamic variable or an actual value being sent in the request.
At some point we'll have to do a PrincpalsAPI.getUser check here and determine what the visibility of the User is, as it doesn't make a lot of sense to output "public" profile sections on a "private" user.
I hadn't spotted the constructUpdateCQL method when I made the previous comment.
I would roll these methods into one `upsert` method which uses the `UPDATE` syntax as Cassandra doesn't differentiate between them anyway.
Yeah sure, the config in most of the modules is just stuff I made up to be honest :-).
I think it belongs to this method, as we want to be sure the flag is in sync with the VM requests.
It's a private member, so it's impossible to share it. Just sending it inside a reply is fine: we're sending our reference (that is always up to date, because it's in ours) to another actor. It would only be unsafe if we sent a reference to a reference (say, we're sending `this`, and the member is public so they can read `this.threads`).
"for" or "by"?
Can we fail on error, instead of crashing clients?
```xxlarge```?
Я бы рекомендовал дальнейший код засунуть в if ( self ) { ... } или даже сразу так  написать if ( self = [super init]) { ...  }
Таким образом, у вас несколько раз может поменяться одна и та же  ячейка, т.к. arc4random_uniform не гарантирует уникальность возвращаемых значений.
Если задать процент мутации 100%, то Ваш алгорит не отработает, в условии необходимо указать <=
Уважаемый freetonik, скажите, обязателен ли пересчёт процентов в кол-во в реализации первого задания?

Почему нельзя использовать это удачное совпадение для оптимизации кода? Ведь в первом задании длинна масива гарантировано 100 элементов, поэтому пересчёт будет гарантировано возвращать то же значение. Соответственно логично будет сделать оптимизацию под такую реализацию убрав ненужный в данном случае пересчёт.

Стоит ли такую оптимизацию считать ошибкой?
Программа выводит именно разницу в наборах, если массивы идентичны то разница 0 если же есть различия считаем их и выводим число, обратите внимание [DNA isEqualToArray:sample->DNA] == NO .
Есть ещё один подобный вариант реализации.
Делается массив в ячейках также лежат индексы.
Выполняют его перемешивание (цикл for в котором случайным образом меняют местами элементы)
А дальше выполняют нужное кол-во замен в основном массиве на основе перемешанного массива с индексами.
тут нет "положено" и "не положено". Так же, как нет правила "клади грабли зубцами вниз".  Эти "правила" формируются из опыта.  
Блоки без скобок -- потенциальные источники ошибок. Особенно, если это чужой код. Лучше сразу привыкнуть быть педантом в таких мелочах, чтоб потом не наступать на грабли
shouldn't it also emit a heart beat then?
not to be too pedantic but could you add a whitespace after the comma? (same for the occurences below)
This should use the GROUP_NAME and APP_NAME constants defined in config.h instead of hard-coding "Brackets"
The author order should be: you, me, Wolf, Satya.
No need to check length of the dict, you already verify the keys with verifyjson.
Are these lines left over from writing it? Do they have reason to still be here?
This and /card 2/ can probably be combined to allow 1 or more cards. Like:

    robot.hear /card(?: me)?(?: (\d+))?/i
      count = if msg.match[1]
                       parseInt(msg.match[1])
                     else
                       1
      for i in [0..count] by 1
        msg.send msg.random answers

May need some testing, but you get the idea.
I missed that. I will fix and re-open.
Use ``` t('app.back') ```
There should be a call to `status_file:close()` here --- otherwise you will run out of file descriptors (plus memory leak due to Lua's overhead per each descriptor).
This has been addressed. Errors will now flow through with the broadcast.  CONNECTION and PARSE errors will display a dialog.
Just use alias:

```ruby
alias :track_user :track_customer # for backward compatibility
```
As you foresee too implement Blob to Blob comparison, I'd be in favor of changing this signature to

```
public TreeChanges Compare(Tree oldTree, Tree newTree)
```

which would play nice with 

```
public ContentChange Compare(Blob oldBlob, Blob newBlob)
```
It works on my Ubuntu install, fails on my OSX install :(

Maybe this has to do with numerical roundoff error? I'll try running on my OSX machine with assert_array_almost_equal.
keep_only_data_channels is not documented and titles in not an argument.

I don't really like keep_only_data_channels. When this problem happened in the past we agreed that picks=None was equivalent to keep_only_data_channels=True
Also use result.jobs.forEach rather than the for loop
Couldn't you use one of the existing users, e.g admin by default is the authenticated user in unit tests
In order to understand how Salt does it's configuration management on minions, we'll take look at the ``salt`` ...
You should remove the PEAR link (#1852)
Please wrap the line after the first word crossing 72 chars
改成主动式可能更通顺一点：响应头<header>X-Accel-Buffering</header>传递“<literal>yes</literal>”或“<literal>no</literal>”可以动态地开启或关闭代理的缓冲功能。
Please remove inline style tags. Use `class="error"`
should can_resume and always_run be properties? they shouldn't be changed by anybody
зачем? Парсер один фиг запишет туда или true и false.
Vaya a saber que es eso. Debe venir del CSS del original, creo que no le cambié nada, en la próxima versión de MomentDatepicker lo arreglo.
Oh i see, you're just renaming it as a "promotion OR deletion lock".  Sorry, I get it now.
Check out http://stackoverflow.com/questions/212407/what-exactly-is-the-singleton-class-in-ruby
it would be probably better to put this code in the dedicated section of the class (only when method fork is used)
ahah :) good typo fix !
please unindent
*image* is still a 2d array, right? I mean it is equivalent to

```python
ax.set_slice(['x', 3, 'y'])
ax.imshow(image)
```
Makes sense. Merging.
Accident I'll fixit
Ill leave it up to you to decide a maximum size, i was looking at this (https://npmjs.org/package/netroute) to possibly get the MTU ?
Above, can you surround `==` with spaces, use single quotes around `_id` and put a space after the comma?
NGHUTPR
It'd be instructive to add an assertEqual to exc.message as well.
Any particular reason not to be?  I'd like to conform to your coding conventions.
Same re: class templating
My browser makes that look like a ``` ` ``` rather than a ``` ' ```. Is that the case. If so, was that intentional?
Interesting, I did not know that reshape could lead to a copy.  However, assigning to .shape does make the code more ugly in many cases (because it cannot be chained), and the caveat is not relevant here, since it only applies to cases where ("incompatible") dimensions are reduced/joined, not split.
Do
```
:class:`colors.LogNorm`
```
instead.  Note the backticks.
Don't you mean np.any() here?
I have made suggested changes, and updated the doc string to note that working directory is also added to `sys.path`.
Не жёстко ли константу определять прямо в тесте? У нас при тестировании кэширования тесты по 9 раз прогоняются. Не вылетит ли на втором прогоне этот тест с ошибкой что эта константа уже определена?
Да и добавление линка будет каждый раз, видимо, происходить.
Hmm, XBrowser might be a bit misleading. How about BrowserVendor? By the way, is the preferred style still uppercase for the first char even though this is not a "class"?
[Like] button

Might wanna write

```php
if (!$info || !is_array($info)) return false;
```
@vimishor Because this constructor method has a type hint it is a way of enforcing that a class that implement this interface must use a CacheDriverInterface as the parameter for its construction. But stof does has a point that the construction of an object doesn't need to be enforced at the interface level.
Does this actually work? You're asserting that $a equals the backquoted version of $a.
o_O
yeah, you have to go to leiningen 2, or switch to 1.0.8 for midje, as I said.
@christianv resolve
You should use `event.stopPropagation();` here
It's probably just my ignorance here, but why is it always safe to hardcode this to localhost?
maybe change this name to 'enable' instead. Also, I prefer to use an options hash for readability for optional params but that's just my preference:

```
def new_ab_test(name, options = {}, &block)
enable = options[:enable].nil? ? true : options[:enable]
...
new_ab_test('foo', :enable => true) # reads better
```

versus

```
def new_ab_test(name, enabled = true, &block)
...
new_ab_test('foo', true)
```
_deleteIndex() is a very small method. You should insert the two lines directly instead of calling this method
Are you sure the variable ``$feedStrategy`` exists, or did you maybe mean ``$feednStrategy``?
This class is not autoloadable using a PSR-0 autoloader. If you would like it to be loadable, it should be in the sub-path "ZendTest/Session/SaveHandler/MongoDBTest.php".
На мой взгляд, правильней вычислять пропорцию шильдика:

```ObjectiveC
float bannerHeight = [self bannerHeight];
_shildImageView.frame = CGRectMake(0, 0, bannerHeight * (_shildImageView.image.size.width / _shildImageView.image.size.height), bannerHeight);
```

Тогда ширина шильдика будет всегда пропорциональна его высоте.

>почему width во фрейме была константа "9"?

Собственно, в этом и была проблема :)
maybe "we shared the concerns of Twitter's ..."
Ich habs mir hier einbisschen einfach gemacht. Ich habe keinen weg gefunden gewisse Checkstyle Regeln bei Tests auszuschalten.
I think we can leave options alone and pass them along, whatever they are, e.g. let's remove this line.
We also have `alias` now :(
I don't think that's right. `/account/id/(id)` returns a 404, but `/account/(id)` returns the user.
You're right. I was getting at that with the underscored name :P 

I'll remove the comments completely.
I think we got this working too:

    dynamic document = HtmlPage.Document;
    document.addEventListener("click", handleOnClick, false);
i will add it if i need it for other backends, however the SocketClient has the advantage to use keep alive. which can make all the difference in performance :-)
I would separate validations by attributes instead, since it can get confusing sometimes:

```ruby
validates :title, presence: true
validates :release_date, presence: true
```
Any reason you're changing the since tag here?
Yep, false is 1 in UNIX, maybe we can just use that
right, forgot to update. nice catch
Yea this is other .gitignore candidate
Huh? It removes the index and the input.
нельзя создать пост без тегов? Так задумано?
This should be displayed only to authenticated and staff users, no?
Even if I create PasswordChangeForm I'll still have the same problem....

Anyway Do u want me to create different page/view for Password change, or should I just leave it like it is ?
It looks nice :) What would be better?
`;` missing at the end.
Should just be else...
I'm not sure what getRealPath returns when not found (undocumented) but I'd like to find out so we can strict check.
duplicate querys
Use CheckArgumentNull
RelatedModel?
Bishoped!   taking SHA256 digest of secret to allow for shorter secrets
Also, I suspect that readers will find it a little weird that `reject` only calls `resolve`. I certainly did.
Quote from [Mozilla coding guidelines](https://developer.mozilla.org/En/Developer_Guide/Coding_Style)

> Of course every rule has an exception, but it's important to know the rules nonetheless!

I believe this case and others from mxr are well justified exceptions.
won't => won't be
Seems my merge tool kicks out BOM, running cleanrepo.
VICINITY_MIN is perfect, thank you.
For some reason it works in specs (I think because we explicitly include `TwitterCldr` in several our spec), but it fails in console for me with

```
NameError: uninitialized constant TwitterCldr::LocalizedDateTime::LocalizedTimespan
```

You should use the full name (with all parent modules) for `LocalizedTimespan` class here.
I extracted it from `Collator` to make things separated. And I need an instance because otherwise `@bytes_array` and `@common_count` will be passed from one method into another as arguments that doesn't feel right in this particular case when we have quite a lot of methods involved.
Do we no longer need to convert the locale?  Is the rule no longer at `[:i18n][:plural][:rule]` for some reason?
For example, this line reimplements inspect_server (but in a buggy way, not supporting Unix domain sockets).
Minor changes:
Normally the php-docs are a simple line finished as a dot.
Then a blan line and after if it is needed a extensive description, @todo or others.
Put the #781 is not never necessary on the code, because git let blame from a source code or search git log commits where the #issue is written.
Remember this docs are generated after for developer documentation, see a lot number issues it is not the purpose of document a function.

Also, if you use a previous function, you should put the anonymous author (we all know that was Ligesh the main author) but this is for mention that you are not the fist author, so if old bug is found, we can know the first, second, third author on a class or method, or blame the code with git.

Another thing to change, it that function params are not camel case (just class names and methods). Params are lowercase and separated words by underscore. 

Respect to _()  I don't know if you are importing the l18n.php on this file, but maybe the php code fails because the function it is not defined.

The phpd-doc also require put a @return value, and if you want rewrite fully the function, you should echo (native call more faster) instead the old print (perl behaviour based).

Another improvement it is used NULL instead NULL and you can save call twice to version_cmp if you store the result on a var previously.
you're totally right, these were being called repeatedly but i wasn't paying attention, but this could totally have blown up in my face.
You might also consider extending this from `base_page.html`, for easier visual consistency.
straightforward?  I do not think this word means what you think it means.  ;)
same here, you forgot to remove this one as well
I think your comment is missing some words. Maybe you tried to add markup without wrapping in backquotes?
jclouds labs does, though. unsure. i think its probably a good thing to be able to select modules by groupId, even if we don't do it yet
The return type should be `id`, you changed that as well.
Please remove trailing whitespaces.
The input is: ' a , b and c,d    '. 
Should the list values really be trimmed?
Rather than doing this (encoding into RightSupport the knowledge of our busted SingletonPatch implementation), why not make the following change to right_agent:

https://github.com/rightscale/right_agent/pull/35

This is consistent with RightSupport's own version of the same idea (EasySingleton), with which we hope to replace the right_agent SingletonPatch someday:

https://github.com/rightscale/right_support/blob/master/lib/right_support/ruby/easy_singleton.rb
There's a standalone Linux command that can be used, but that doesn't exist for OSX. Both Linux and OSX have python as standard.

A different strategy would be to hardcode the directories at installation time.
This test looks brittle. Are you trying to match on the prefix specifically?
I think this method should be broken up into smaller methods.  It's best to only test one thing in each method so failures are clear and accurate.  Having comments in your tests is a code-smell that you should create another method where the method name says the same thing as the comment - eg ``test_creating_page_with_existing_URL_does_not_work``
Why is this part of `XMLFetcher`? Shouldn't it be part of `WalkscoreFetcher`?
This will cause a regression. The onclick handler should execute event.tasrget.dataset.action. Otherwise you won't be able to hang up the call.
If savehandler is an instance of AbstractProxy, you will wrap it in a NativeProxy here. Is it really intended ?
Holy if Batman!
My thoughts were just to clean up the code so one file does one thing, instead of having a large #ifdef block.

If you look at how I did navigation, you'll see that I pulled all nav settings completely out of manualcontrol and put them into the pathfollower. This doesn't address the signal routing problem in its entirety, of course.
```javascript
var titleCls = titleElm.classList;
window.onscroll = function (evt) { 
    if (limit < document.documentElement.scrollTop)
        titleCls.add('float');
    else
        titleCls.remove('float');
 };
```
Use C comments in C code; I've tried to keep the code buildable on relatively primitive compilers.
Ich weiß die Doku fehlt hier noch ...
Это делается в supports
Зачем тут StringBuffer?
И необязательно main() метод делать, если пишешь junit тест
Okay. I'll create separate testing and self-documenting starter configuration files and update this.
It _is_ opinionated software... What is an alternative?
Declaring this resource here will cause duplicate file resource declaration if someone ever declare two ppa.

Also this resource is already declared in the apt class:
https://github.com/puppetlabs/puppetlabs-apt/blob/master/manifests/init.pp#L65

So this resource should exist if you declare:

    include apt
    apt:ppa { '...': ... }
Reused in all `_core*` files. Better a function for this.
I don't like this as it does not to what it says. This is equivalent to ``if class_weight is True`` which is not the same as ``if not class_weight is None``.
Still I can do it...
hm maybe I missed something. ``hmm.gmm`` should be the parameter that is passed. I seem to have messed it up and my test didn't catch it.
changed that - thx
Done.

On Wed, Dec 19, 2012 at 9:54 PM, Jonathan Chang <notifications@github.com>wrote:

> In src/gibbs.c:
>
> > +						int yv = INTEGER(annotations)[dd]-1;
> > +						if (yv==-1) p[kk] *=exp(0-maxExp)/sumcn;
> > +						else {
> > +							change = REAL(beta)[kk + yv*K]/nws;
> > +							p[kk] *= exp(change-dv[dd + yv*nd]-maxExp)/sumcn;	
> > +						}
> > +					} else{
> > +						int yv = INTEGER(annotations)[dd]-1;
> > +						if (yv==-1) p[kk] *=1.0/sumcn;
> > +						else {
> > +							change = REAL(beta)[kk + yv*K]/nws;
> > +							p[kk] *= exp(change-dv[dd + yv*nd])/sumcn;	
> > +						}
> > +					}
> > +				} else {
> > +				  change = REAL(beta)[kk] / nw;
>
> Could you include the comment here that you deleted above? Even I get
> confused sometimes as to what I'm computing =P.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/slycoder/R-lda-deprecated/pull/1/files#r2473607>.
>
>
Thanks, I'll remove the do_test commit from this pull.
Will try to do that in later patch.
What blackberry specific properties are there?
There's a `head_css` block in base.html where I intended page specific styles to go.
What does distance indicate?
`Hibernate` > `Hibernate Search`
I thought about this over the weekend and examined the source. The original assumption was that the resolved file was always a Dali file (or resolved by Dali somehow), which is not true for local files. The very operation being done inside the if block is only meaningful to Dali files, so non-Dali files should never get in there anyway.

This confusion is due to the way files are handled. As @ghalliday and I were discussing last week, there are several ways to resolve a file and more than one useful representation of a file for each use. 

The URI naming system would add consistency and hopefully reduce (and trim down) similar representations of files (local, Dali, remote, web?), while providing safe access to Dali (DFSAccess proposal). This code will definitely change on such occasion.

To make it more clear, I can add a flag ```local``` to make this class avoid any Dali resolution, but that would probably be set to ```subFiles.ordinality()``` anyway. Unless I'm missing something obvious, I believe we should, for now, stick with the ```if```s until the URI refactoring.
```rand() << 16``` was still an int (rand's type), so that ```rand() << 16) | rand()``` would still be a 32 bit number with 32 leading zeroes.
If the graph indicated something like whether the file was being used for read and/or write that could simplify things.
This is duplicating code in roxierowbuff (including spelling mistakes) - it should really be commoned up if possible.
These are both 0x0008
зайве
Hm, and as another note, the whole ```{partials: {name : path}}``` construct is not the standard "hogan way" of providing partials.

Hogan instead has a 2nd argument where you normaly provide a hash of the compiled templates that are to be used as partials. The keys of the hash correspond to the names of the partial in the template...
Well this function only check is the version asked by the user is available. Since the 0.0.1 is not going to installed this always will print that the version doesn't exists, of course when the user put a bad version.
Sweet. Haven't known that it can be done in a single command. Can you please update the readme as well?
`self.assertEqual(set(expected), set(sessions))` will work.
The other way we could call this is by doing 

```javascript
calcentral.Api.User.getCurrentUser({
    'refresh': false
}, function(success, userData) {
```

but for now that seems unnecessary right now, I'm definitely open to changing it though.
WDYT?
@yuhunglin Can you set your DocBlockr settings in Sublime to use this:

`"jsdocs_align_tags": "no"`

That way we won't have the extra spaces/tabs after `@param`
@shacker some minor tidying up with styling and where line break on the tags, but i could follow up with that. Not going to block the merge.
Done. My fault: In our above I confused GitExtensions.csproj with GitUI.csproj.
You fixed it in WritableDirectoryCheck but not here
I think you want ```os.path.basename()```
if success
Remove the comment `#< ModuleSet`
I'm not convinced that a `Transformation` class is necessary. Why not just have classes `transform_similarity`, `transform_similiarity_inv`, etc.?

I guess the class saves the cost of recomputing the transformation matrix on a round-trip (forward then inverse transform), but I would imagine (just a guess) that this would be dwarfed by the cost of the matrix inversion.

Is there a use case for calling the transform multiple times (other than a round-trip)?
typo in *neighborhood* ;)
I don't think it can be private - its intent is to be called by parent matchers on their submatchers to construct descriptions, as in the `submatcher_allowed_types` method of `ValidateNumericalityOfMatcher`.
Is this really needed, since it's set in the function above anyway?
After your code changes, this comment is out of date.  Users don't set the buffer usage.  The code figures it out in `_computeNewBuffersUsage`.  I think this comment needs to go; it documents code that no longer exists.
I know its a bit outside the scope of your change, but add some tests for `EquidistantCylindricalProjection` and `MercatorProjection` classes please.  Should be easy.
Right, its really testing which side of the plane the box is on. I'm not sure what is most commonly used. How about positive, negative, and intersect, where positive is the side the plane normal is pointing?
This one looks pretty broken to me.  For instance, this would match it: ZZ+-050708::1234.
Maybe something like this (totally untested): /(Z|(+|-)(\d{2})(:?(\d{2}))?)?$/
drag_end, drag_start??? We should be consistent and use camel casing. e.g., dragEnd, dragStart...
is it possible to make the setting of the stroke color part of the color selection in our side panel?
It is more elegant to implement this method using [Array.filter()](https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Array/filter)
make sense, will do that.
If a user specifies an empty roles() method, roles in DefaultRouter will just be an empty set. This is situation is covered by the following [test](https://github.com/danbev/aerogear-controller/blob/AEROGEAR-426/src/test/java/org/jboss/aerogear/controller/router/DefaultRouteTest.java#L21)
This doesn't sound right: "The returned node has an ownerDocument property points to this document," should it be something like "an ownerDocument property which points to this document" ?
Media Item is not (yet) supported on mobile web and should be marked as such.
Nope, we don't actually support platform tags on method parameters--so we can only clarify that in text.
Bryan, you're right, I am seeing this on MW, my bad.

Max, I'm not seeing the child view on iOS. Maybe it's displaying behind the switch itself, but if so, it's not really useful (it would also violate the expected z-ordering of parent and child controls).

I did see this inverted z-ordering on image view, progress bar and slider, but I could at least see the child image, so I documented it as supported.
"If undefined, defaults to either..."
use "star imports"
I do this isdigit check on the last item of an array a lot as well, should probably make this a utility as well.  may make it more readable.  

something like:  `last_word_isdigit(word, separator="_")`
This line is what sets the data, the next line sets list field data, since it can be dynamically added.  Should comment on this a little better in the code.
Source formatting note: new line needed here as well (one is a property access vs. function call)
Жесть по-моему. Мы собираемся делать отключаемый цсс разве?
If this is faster, could we merge it within `typeid(T).postblit(&t)`?
Why do we need to filter the bad file names? File::fnmatch should take care of the bad file names
Prefer tearDown instead of uninit.
`compile a solr query into an equivalent clojure function`
Где тут два способа?
If I have seen it correctly, than this method is exactly the same like in the SendEventPage class? This is a candidate for an own class which is used in both files.
checkFor...
I guess some people might find it useful, but the check performed is way too simple.
THIS is a smiley face
These two I will have to see what makes sense and possibly contribute them back to refinery.
Case statement for backward compatibility.  Note that now the default for manage_redhat_firewall is set to false instead of true.  This was necessary to allow us to deprecate the parameter.
Hier Ernten wir nun die Früchte vom speichern des WordCount auf dem Dokument....
Der Auftrag kann nun den Preis selber rechnen.... resp... die Pricing-Instanz nach dem Preis fragen....
Sure, I've noticed it after submitting pull request. Do I need another commit?
There is no official one. Ive used following:
http://grepcode.com/file/repo1.maven.org/maven2/org.jvnet.hudson.dtkit/dtkit-junit-model/0.3/com/thalesgroup/dtkit/junit/model/xsd/junit-1.0.xsd
Another one:
http://windyroad.org/2011/02/07/apache-ant-junit-xml-schema/
If you need the callback after the styles have been applied, it might make sense to create a callback in the validation API that is called at the end of `showErrors` after `this.defaultShowErrors();`
If I get this correctly, this can now be moved out to the template instead?
Hi, Actually, that's why we introduced the TemplateFunctions class.
This call should look something like : ${ function.callTemplate("Generic PV1") } etc. etc.
I find the opposite. It's hard to read all small templates, because I need to switch many times and see what bindings they need ;)
The condition inside "if" will never evaluate to true, the "while" loop above it guarantees that.
This whole string actually seems a bit superfluous.  You've already created a custom exception for this case, so the meaning is already clear since you're getting that particular exception.  Also, since the message is already determined by the exception itself, it might be better to just set the message (which should preferably be shorter) in `#initialize` by calling: `super('GoogleAuthenticatorRails must be loaded before ApplicationController is defined.')`
That does indeed fix the issue, thanks for pointing this out.
Apart from logging the error, we need to send with `connection.sendUTF()` a message to the user to show what happened.

The connection is still closed at the end of the function.
Merge cu `@templated`?
Tests should be named basically as sentences describing what they're testing.  In this case, maybe something like "WritePacketGeneratesValidCzmlForSimpleLineString".  Better yet, break this into several tests, one that verifies the time span gets converted to availability, another that checks coordinates, and more that check coordinates in different forms.
why ``$(that)`` ? ``that`` is already a jQuery object
此处少了一个参数
Less API surface area is good - let's remove this and just call directly into `validate`
You'll probably get an Out-Of-Memory here
That last sentence seems superfluous now.
Cindy could you change version to 0.1.0 otherwise "npm list" commands gives out a warning.
The namespace doesn't match the folder. I prefer `Event`, so renaming the folder would be nice.
Oh, indeed - thanks for pointing out ;)
Took me a minute to get to this, but I'd try a pattern match here instead:

```scala
lines(0).split("->").toList match {
  case bundleId :: rest =>
    rest.headOption.foreach { bucketName =>
      val filesForBucket = customBucketMap.getOrElseUpate...)
      ...
    }

    Some(bundleId -> lines.drop(1))
  case _ =>
    None
}
```

Or somesuch.
This will get called for every 'playing' event which is every single song.  I'm thinking this should just happen if the source changes.
Children of File are incorrectly static
You might also consider putting the blocking queue into this class as well.
(not sure on this one, but if you put both in here you would encapsulate everything)
Height typo.
We should also warn that on iOS, we still have the Kroll issue. That is, for a view foo, doing "foo.padding.top = 5" is undefined. It may work, it may do nothing, or it may take effect at any random time later on depending on implementation.
Original cards have this capatalized
Empty array => out of bounds exception?

Also, why fetch plural headers and not the first?
Not a bad idea!  Should be easy enough.
The idea was that in the view we need it in at least two places: dashboard + pulldown menu. In the view it would remain easier to configure, but I don't see why it should be configurable: if a user doesn't have access to an item, he shouldn't see it (IMHO). Filtering it in the getDashboardGroup function would also make it easier to hide empty admin groups.
Not sure what version is on the ci machine but i just checked my local 0.6.10 install and path.sep doesn't exist(see below)

```
C:\Documents and Settings\jkeshavarzi>node
> require("path").sep
undefined
> require("path").dirname
[Function]
```
yeah i do i can commit that as well
This code only checks accessList[0] (local features). Shouldn't you also be checking features for remote pages? They will be contained in the other indexes [1,2,3 etc]

`<access uri="http://www.blah.com/">
     <feature id="blackberry.app" />
 </access`
you need to rebase.
You can now run individual specs using the commit below, which you don't seem to have.

https://github.com/blackberry-webworks/BB10-WebWorks-Framework/commit/2b6b00c09319202633379373350604ee06ec47d9
Oh you're right, its definitely set as a constant. I missed that (I was only looking at the if statements above). id, parentId and folder cannot be changed (meaning these checks here are likely unnecessary).
I think should be better to call it twig params, in the roadmap i've to make a show view
No spacing before the bracket.