labofsahil/github-event-dataset-2012 huggingface dataset:
Yes, but we need only squares. Gravatar has only squares, no? So common denominator is only 50x50, no? Aren't there also smaller? For chat/list of users? So which square avatars do Facebook support? Gravatar supports all?
* a successful validation for a user created by a client site is redirect. 
* a successful validation for a user created on the persona.org site does not redirect (not even to the manage accounts page, which i think is a lack-of-feature). 
* a successful validation for resetting the password is no-redirect with a different message.
* a successful validation being done by a different browser than the one where you created the user requires you to enter a password and click another button (this may become an issue with personatestuser.org, we'll see).

it is four different use cases. i think it's either do this or provide 4 separate POMs for the different expectations.
Without it being in the setTimeout the editor dialog iframe was never becoming the active element when calling focus on _iframe.contentWindow.focus()
what's this guy doing here?
Why don't you just use classes instead of "data-panel"?
Add some simple docs to explain what a LoadGroup is.
You would use fewer characters, and lines, just writing "project" in each error string below. Doing it this way is confusing.
do we still need that option if only one is possible ?
demo 相关的介绍，放到 demo/README.md 文件中。另外，RSDemo 显然无法直接运行，需要用户填入自己的 accessKey, secretKey。任何一个步骤要为客户多想一点。
If typeset_functions is not working correctly, this could fail.

One last thing to check might be to see if there are any aliases on your system that override/conflict with function names. This has suprising and buggy behavior with function declarations.
Thanks, I am on it
Yeah, that totally makes more sense. Done and pushed!
Путь будет зависеть от cwd, лучше его сконвертировать в абсолютный
Centering horizontally is fine by me. :)

Also at some point in the future, it's probably a good idea to swap Twitter Bootstrap out for richard-specific style. No one's expressed interest in doing that, yet. I'll write up an issue for that.
Oh, bugger. I totally forgot about that. Bah. I'll fix that. At that point, we'll have only one test that won't run and that's the "does it kick up a 404 if it's disabled" test. And the answer is "YES!" because there's no urlpattern for it. So we can ditch that test.
Sorry, I've added a 0.1 tag now
Correct, the strings won't be added to the file given to the translators the way it is right now.
`exit_with :general_error` could be more descriptive
what is the concept of channel in Pusher ? Couldn't you use the logger channel for this ?
Oh, and if we stay with the current approach, I would suggest the following name changes:

`operand` -> `other_nddata`
`result_data` -> `result_array` (or `result_arr`)
`self.parent` -> `self.nddata_object` (or maybe `self.parent_nddata`)

I found those three to be difficult to understand at first glance (I had to poke around at the source code to figure out what they do). Hopefully this is basically just a simple find-and-replace.
Another reason to cache the scale/format conversions--each time you look up one of these attributes you get a *different* but equal Time instance.
I would generally expect `mytime.tai is mytime.tai` to hold true, just like a normal static attribute.
It would be nice to explain the `use`, `prefer` stuff at the module level doc.  I thought it was just a list of modes at first until I saw this.
I've noticed that yesterday. github rocks!
This function is currently unused!
For me to know: where are the translations for this?
Change back to delete
I'd be game for adding a followup recursive walk that filled in names.
Done, fixes will be pushed this evening (can't from here)
You don't need to use `paste` with `stop` - it accepts `...` and paste together with no sep.
But `timeit.template` uses the `%` style formatting syntax up to and including Python 3.2. I think it's better to keep using the stdlib code than to carry our own copy of the template.
I didn't know that.  It seems that NamedTemporaryFile is used in several places in IPython.  May I create a context manager so that people can replace the stdlib NamedTemporaryFile to the workaround version when problem is found?
(sound of beeping) wildcards boo..
Yes, you are absolutely right. I missed it. I'll fix it.
Originally, I thought what was Recur Expansion was could be turned into ICAL.Event, after thinking about it more I believe its much better suited to be its own object and consume ICAL.Event(s) once we support more sub-components we can look into abstracting the common pieces like it is done in Lightning. I have tired to be very lightweight here mostly delegating to the component.

I found that I did this is my own code a lot on gaia and its super helpful to have a thin wrapper which we can add functionality to.
No. This was rather a placeholder. I'm going to drop it.
This is  still left.
Oops, sorry... I'm an idiot, I only checked the test files for leftovers in my last checkin.
The template would be used in either case right? I could test the response code to make sure it's 200 and not 302, but this seems to test the entire stack as a bonus.
An empty line would be nice here too.
rm. It's in `postgres_middleware.ini`
hmm, got to get rid of this giant commented out section
Same as above.. I do think that this isn't the right place to handle this, but in run() perhaps is. I don't think we want a single bad file permission etc. to stop the whole run.
Ah, I notice that this pull request and the patch on the relevant sage ticket are different.  Did you perhaps mean that inside of the if-test, it should be `f = f.__call__`?
Was supertype supposed to be rawType?
This is how currently all the other components have it. Should I make this change and also submit a PR for the rest of them?
this path should probably be simplified btw (and currently, it breaks things when trying to run the tests of the bridge standalone unless you created all the parent hierarchy before cloning)
you should typehint the argument as an array
Any particular "why" using used once additional array variable ? (even more it's private)
Because logout_url is no longer stored, no need to change this file.
It is intentional and, interestingly, in the document he requests this string.
I think it should be std::vector instead of vector (much safer)
@charles-dyfis-net not sure I get this line. Did you mean?

    if len(emitter_spec) == 0: continue

My little test

```python
agentConfig = {'custom_emitters': "2,3 , 4 ,,,,," }
print agentConfig.get('custom_emitters')
emitters = [1]
print [s.strip() for s in agentConfig.get('custom_emitters', '').split(',')]
for emitter_spec in [s.strip() for s in agentConfig.get('custom_emitters', '').split(',')]:
    if len(emitter_spec) == 0: continue
    emitters.append(emitter_spec)
print emitters
```

gets me `[1, '2', '3', '4']` whereas with `len(s) == 0` I only get `[1]`.

Did I get this right?

Interestingly enough:

```
Python 2.7.1 (r271:86832, Jun 25 2011, 05:09:01)
[GCC 4.2.1 (Based on Apple Inc. build 5658) (LLVM build 2335.15.00)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> for x in [s for s in [1, 2, 3]]:
...     print x, s
...
1 3
2 3
3 3
```
Sorry - not sure what you mean. I've already deleted those lines.
I don't recognize these port numbers. Could you add comments describing what these lines do?
Looks like we fixed this to be `Ember.$ && Ember.$.Event` so `ember-views` is not a dependency.
For being inexperienced with ruby, this patch was super awesome. Good work! :)
You can use `this.destroy()` here --- `_on` ensures the widget instance is the context of the callback
It's fine to use `that` in these situations, but if you really want to get rid of it, you would do so by using `_on()` to bind to the events instead of using callbacks.
Use `._on()` for event binding; it automatically handles namespacing (`menubar` is actually not the correct namespace anymore) and keeps `this` pointing at the widget.
Nevermind, I see where it's called now.
ну и в других местах надо посмотреть где remark отображается
I would say it does :-) What about including the JS file inside /apps/communications/js or maybe /apps/communications/utils/js? What do you think?
You don't need 3 timers to update 3 UIs. Please correct this in next pull request.
Did you mean to remove the close buttons?
Please don't add extra whitespaces in the beginning of the css rules
These constants are deprecated, but its right in flux so we dont really want to stop using them

    IDBTransaction = IDBTransaction || {};
    IDBTransaction.READ_WRITE = IDBTransaction.READ_WRITE || 'readwrite';
    IDBTransaction.READ = IDBTransaction.READ || 'readonly';

Will fall back for when they stop working complete (unfortunately will still show warnings in the meantime)
Did you need this because the UssdManager wasn't initialized at the time openUI got called?

We should fix this instead, looks like we can `init()` the UssdManager earlier (it does nothing dom-related).
nit: it's hard to follow the relationship between all those `close`, maybe we should rename the method on `UssdManager` to something like `finish()` or `cleanup()`. What do you think?
BTW, do you want me to update the english one so we can take that as the reference? :-)
I'm not sure to understand why the browser app is opened on this case and this is not an embedded search?
Not using this template so far. I guess I'll delete it.
restore now. because it do not checkout head to the working dir when specify the target dir to clone.
you are not testing the template output, let me see if i can come up with a
quick fix

On Tue, Oct 9, 2012 at 5:35 PM, Greg Sutcliffe <notifications@github.com>wrote:

> In test/unit/host_test.rb:
>
> > @@ -511,4 +511,11 @@ def setup_user_and_host
> >      assert_equal h.token, nil
> >    end
> >
> > +  test "host should have a token when created" do
> > +    Setting[:token_duration] = 30
> > +    host = Host.create :name => "foo", :mac => "aabbeeddccff", :ip => "53.45.6.7", :managed => true, :build => true, :architecture_id => 1, :environment_id => 1, :puppet_proxy_id => 1, :domain_id => 1, :operatingsystem_id => 1
> > +    assert host.valid?
> > +    assert_equal Token.first, host.token
> > +  end
> > +
>
> This actually seems to pass with after_save too - what am I doing wrong?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ohadlevy/foreman/pull/2/files#r1796636>.
>
>
This should not always be 't
Perhaps something like `changeImageSource(url, image)` ? But I'm a bit curious as to the XHR method, vs the image load event.
```
var img = new Image();
img.addEventListener('load', myFunction, false);
```
Couldn't you create another image in memory, then on load adjust the source attribute?
temp var for max-min seems useful
If this exception is thrown when you can't find a key, you should call it "KeyNotFoundException".
Add a comment here - why the x1000? Converting to milliseconds, microseconds, something else?
This is necessary, as this project uses bundler.
Would it make sense to combine this class with StreamProxy class?
remove chdict['category'] here, as well
And beyond that, I think we should be supporting returning SKIPPED via Deferred too.
Good point. Fixed in the cleanup I'm doing now.
Probably we can mention how it became root like 

    cd monolog
    composer install --dev

Thanks 
Hari K T
That would be better :)

```php
if  (null !== $path && !is_writeable($path)) {
    throw new \InvalidArgumentException("Not authorized to write to '{$path}'");
}
```
New source has to be provided under GPLv3 or later. See http://ethersex.de/index.php/Contributing
Shouldn't it be:
```
let requirements = manifest[requirer].requirements;
if (id in requirements)
  return requirements[id];
```
environnement -> environment

Also I'd say "environment variable is not set" instead.
Oh nevermind, I fall into `null` trap!
That didn't work.
I carefully changed all the setters the way you described above, and self.options was always undefined in the content-script. No idea why.
@seanmonstar - updated.
To test, I ran the `manage.py create_project` command and this did not break on non-objectstore datasets.  However, it does break the unit tests.  I added `null=True` to the field definition in the model.  I prefer `NULL` over setting the value of a `VARCHAR` to an empty string.  I think there is a clear semantic distinction between a `NULL` and an empty string and that is "put no data" vs "put empty data".  There are also some unexpected consequences in SQL when using empty strings in place of `NULL`.  COUNT will count empty strings but not `NULL`s, we can discuss more later.
Ok, so it seems like we cannot cross this boundary. Then please put the code inside a function within widget.js and call it at the end of the file.
I'm not sure about this. As I read this, this will create new sprints for s= that don't already exist. Part of me thinks it should just go in the backlog. Otherwise every s= typo will create a new sprint and since anyone can fiddle with the s= column in bugzilla, that seems like the worst of the two possibilities (lost in backlog vs. lots of extra sprints).
This logic is almost identical to the logic in `stub` -- might be worth extracting it into something common?
The typo war there before me: https://github.com/rubinius/rubinius/blob/master/spec/core/ffi/platform_spec.rb#L36-L41
… but I didn't catch it…

I'll review and fix this.
it should be "if called with a normal file"
what kind of a random graph is this?
we might as well just use this one, not like it's slow or anything, and ditch the comment
The mixed up parameters should be removed, thus this check would become unnecessary.
s/see its/see, it's/
JsonException
Not sure if this violates CS on ssGWT-lib, but in the future try to keep your lines 120 characters or less in length to make reviewing easier.
Nope, not that I know of. So long as the code works fine, it's formatted properly. :p
Yes, dear. :P
This doesn’t work for 16:9.375 (1024x600) – should be `" [%g:%g]"`.
Is there a good reason to check if `/usr/local/.git` exists instead of just using `which brew`?
alL => all
Actually, I agree -- leaving the line, but commenting it out makes sense, IMO. This gives users one single place to look for default values instead of this file, plus Module.php for the one exceptional default.
what exactly does this mean? if i move a document somewhere else and then (without flush) reorder it, it won't be moved anymore? and the same for creating and removing?
links should be valid HTTP web addresses and should not require a http:// in front of them.
can we somehow output where that block appeared in the mapping file? Otherwise it will be hard to spot the violations.
I would also appreciate some kind of label like "[DEPRECATION]" in the comment to explain the problem.
Is % here the same as "foo".format(dict) ?
under ifcore.a
can't this use pefrom_step in some way? The code is roughly equivalent...
I think he was trying to give an outline of what will be done in this quickstart. It does say "The procedure for adding a user is described below." 
Should I remove that from the beginning since it's added later in the quickstart?
I think this one needs more work.
The convention we've established is not to use `ejbjar` but `ejb` here.
Please use `{ isDevice .. lazyArray }` notation (i.e. add a space next to `{` and `}`)
This feels a bit clumsy as  `std::string::c_str()` can be used to get a zero terminated string, but you can keep it like this.
newline :E
Hmm... all I know is @ednapiranha yells at me when I use these :smiley_cat:
It's pep8! ;) http://www.python.org/dev/peps/pep-0008/#documentation-strings
Should be 'Open Sans'. The light weight will be addad on line #64.
A simple alternating pattern for incoming messages to validate the validity of the "doubler". **This will not work in a distributed environment.**
I believe this was here to assist in the magic of using the module as a
lookup path. It now works without the path munging, so removed it so that
everyone's sys.paths are more standard.
On Nov 9, 2012 10:55 AM, "jikamens" <notifications@github.com> wrote:

> In zipline/data/loader.py:
>
> > @@ -15,17 +15,33 @@
> >
> >
> >  import os
> > -
> > -if __name__ == "__main__":
> > -    import sys
> > -    sys.path.append(os.path.abspath('.'))
> > -    print sys.path
>
> Why are we no longer adding '.' to the path?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/quantopian/zipline/pull/15/files#r2084604>.
>
>
ok - ideally I'd like the two to be merged at the end, so I didn't worry about it.
I'd very much like to keep the C implementations. The LW server(s) do run into CPU limitations. This is mitigated by autoscaling in AWS and memcached but each server that gets started costs money. There are currently three servers running, each one costs over $100 a month.

I would suggest you leave the try/catch block, remove the import of uspace_wrapped and include you note about it. Remove both definitions of spaceCompress, replacing it with your one.
The view should own their rendering state here. If they want to be rendered at the time of ready they can call ensureRendered explicitly. At the framework level rendering should be deferred as long as possible as a child might be registered through _addChild but not explicitly inserted until later. They would still want to be aware that their parent as been inserted into the dom (i.e. ready) but they wouldn't want to render until they are actually inserted into the dom.

I don't think appendTo is necessary. ready is something that is handled by the layout view. If they want to have that behavior they can trigger ready directly.
> It should be 42 imho))

I assumed that that would always be the response from the actual GitHub call, otherwise, what good is it?! ;)
This adds the 'org.hibernate.search.engine' AS7 module, with dependencies on other existing modules and embedded libraries for some of the transitive dependencies. Note that the Jackson ASL libraries are different versions than what's used in AS7, which is why they were embedded in this module.
well I wouldn't want that.
This could be
```
  def generate_n3_urns_from_hash(contents, n3_user_urns={})
```
and then you can skip the next line to ||= it to a hash
This would be better to be

    if (NULL != mapping[action])
I guess I could get rid of newlines with a regex replace. in the spec.
Using the left unit sounds fine to me.
The @param type of $data is StdClass . This is not fixed . Let me know if its ok .
Yes, sorry, I meant empty() ;)
@complexcompulsions correct. @m4tthumphrey, mind sorting that out?
Hardcoding `ezpublish/` folder is not a good idea IMHO. What about people who want to integration eZ Publish in their Symfony application ?
Use a constant instead maybe ?
I guess I can remove it ...
yeah in general but shouldn't we line up the var stuff so it's easier to read? :) lmk what you think - i can switch it back
@@ are terrible - I hate them.  It's bad code usually.  The rails configuration mechanisms are usually the *right* way, a static accessor that returns an object that exposes the config of the object.  Then you can create your own config and inject it into places if necessary.

Some tests are going to need to change configuration at test time, so don't make it too hard to completely replace the config value (and keep config values cached close to the config object, not deep in other classes).
`barcode` is leaked if `osr == NULL` (that is, in the `openslide_can_open()` path).
Maybe a more friendly tag. My Repos, Own Repos.
Can someone explain when this gets called during update_positions in crud (calls class method rebuild! -- what is alias_method_chain doing here?)

and then what this rebuild_with/without method is doing? I'm a bit mystified by it. 

I understand why my earlier method didn't need to iterate over children...
Fixed. We're now just allowing all .js files.
I fixed the descriptions.
Are you passing in the object as a wrong value on purpose? If so, could you change this to something like 'non-existant-user'? It makes the test a bit clearer
Is it necessary to disable recaptcha here? All of our tests are set up to create users as the tenant admin, which seems safer than testing without recaptcha?
Yes, there's no point in waiting a second of the tenant creation has failed.
I think you can underscore for this as well:

if(!_.isNull(name))
Would it be feasible to split file related functionality, sakai doc related functionality and link related functionality out into 3 separate files and include/extend them into the main API? There's a few functions now that specifically check for the type, and it's going to make it a bit harder to maintain.
Also, when I create a new element for this it defaults to a 'slash', but that's not specified here. Should it just be an empty field when a new one is created?
Do we still need console.log here and below?
Please update the JSDocs.
If you use env.getEnv in the exec method, you need to add @api(environments={CommandHelperEnvironment.class}) instead of just @api. Currently, this is not enforced, but it will be at some point, and so it will cause issues in the future.
i can do it right now tough!
@tsetimmy 
yes, that's why I modify the return a little bit.
The name confirmer seems wrong here (and in the module too). It does not do any confirming. It only creates a confirmation message.
Waarom deze code als je hieronder alsnog de juiste righting op flipt?
Yep. I have shortened the code as suggested now. Please note that the last part of the code should read ```$this->alt_message``` instead of  ```$this->alt_messasge``` (typo)
Было бы хорошо более строго указать тип mIndex (вы же знаете, что это NSNumber).
Метод `@throw` относится к исключениям. Если вы программировали на каком-нибудь другом языке программирования, то наверняка слышали про конструкцию try ... catch. В разных языках это делается по-разному, но суть везде одна и та же - вы пытаетесь вызвать какой-то метод в блоке try, и если прогнозируется ошибка, вы её "выталкиваете" и вылавливаете в catch, каким-то образом обрабатываете, программа при этом продолжает выполнение. Опять же, я просто описал сам механизм. На примере можно сделать что-то типа такого:

`@interface Fraction : NSObject
+(float)fractionAsFloat:(int)numerator :(int)denominator; // хочу, чтобы дробь показывалась как вещественное число
@end

@implementation Fraction
+(float)fractionAsFloat:(int)numerator :(int)denominator {
    if (denominator==0) @throw [NSException exceptionWithName:@"InvalidDenominatorInFraction" reason:[NSString stringWithString:@"Denominator can't be zero"] userInfo:nil]; // если знаменатель 0, вытолкнем ошибку
    return (float)(numerator / denominator);
}

int main(int argc, const char * argv[])
{

    @autoreleasepool {
        
        // insert code here...
        float result; // переменная для вещественного числа дроби
        @try {
            result=[Fraction fractionAsFloat:54:0]; // попытаемся получить значение дроби
        }
        @catch (NSException *exception) {
            NSLog(@"Error: %@",exception); // если в блоке try произошла ошибка, то она вылавливается здесь
        }
        @finally {
            NSLog(@"My fraction as float is %1.2f",result); // этот блок выполняется, если ошибки не было.
        }
    }
    return 0;
}`

Данный код - всего лишь маленький пример, в реальности всё используется несколько по-другому. Однако, надеюсь, понимания прибавится. Мы в нашем единственном методе хотим получить дробь в виде числа. И в нём прогнозируем, корректна ли будущая дробь - не равен ли её знаменатель нулю. И если так, то мы не будем продолжать выполнение программы, мы "вытолкнем" ошибку наружу! А чтобы её получить, воспользовались конструкцией try ... catch ... finally
Не совсем понял. Мой последний commit сделанный после pull request'а попадет в финальный pull request? Или надо делать повторный pull request?
как вариант)
I agree that we will want a generic "reveal in finder" option at some point. I think we'll also need a native method to get the extensions folder (in an OS-friendly location). Once we have both, the showExtensionsFolder() function can be re-implemented as revealInFinder(getExtensionPath()). 

This will hold us over in the meantime, and shouldn't cause any issues down the road.
How do you get this to point to our FV map with the popup?
Android? (vs iOS)
Original is not ideal either, and is split into 3 gettext calls “Delete”, “all”, and “entries in this table.”   So those word orders cannot be change either.  This commit is not aimed at changing that, and keeps those same 3 gettext calls, adding only a table entries count with html bold tag.
Even though the $tablesname variable will only have the values "bogons" or "bogonsv6" in this case, I still suggest using escapeshellarg for all user input shell arguments. Also instead of directly concatenating you might want to store the output in a meaningful variable this would make the code more readable.
That means `page.should have_table_row('Posts', "Title" => "Title", :Year => "Year")` would always match? Or do you mean when one has multiple rows containing `th`s?

The former is an always-passing assert, and we shouldn't have something like that. Maybe we could exclude the cells we are matching again?

The latter option is more interesting to me, though I suspect people having multiple td rows also rely on colspan which I believe we not yet fully support. Maybe supporting colspan is should block this issue?

What do you think?
Immer 4 Spaces statt Tabs.
[Done](https://github.com/Thynix/fred-staging/commit/4786e90dec24110db6956ee08bc3a962f4bbc7af).
Isn't ```equal``` more appropriate than ```ok``` here?
icosahedral
hum. How about being able to pass to the plot an order array (order=np.argsort(scores)) to order the plot according to the similarity with eg. the ECG?
it seems weird to sort just to score.
`center_of_mass` and `vertex_to_mni` work with SUBJECTS_DIR - the alternative would be to submit it as argument to each function call, would that be preferable?
if obj is an array don't use for...in with it.
I disagree with this statement. For this use case it's sufficient to synchronize populateMap, as long as it's written to construct a _new_ map and then do this.cache = newMap at the end. Writes are extremely rare, so It's fine to return a millisecond-old cached value, as long as that value is consistent.

With the code as-written, though Rafal is correct: since you're modifying the map rather than replacing it, you'd need to synchronize the read method too. Which would be bad. Since writes are very rare, we must avoid synchronizing in getInstance or read, because that's an unnecessary performance hit.

Personally I think it's confusing to call the Map field "cache". Really the _instance_ is the cached value. And the instance should contain a non-static map. When the GP is changed, you can just throw away instance, and instantiate a new one. To phrase that another way, we should be caching the result of parsing a String (which is a map), not saying that we put the parsed values into a cache.
Writes will be very uncommon comparing to reads so synchronizing is the worst choice for this cache. Instead of implementing your own cache, please look at my comment on the ticket about using the Cacheable annotation. You could add a new service method: VisitType getVisitType(EncounterType encounterType) annotated with Cacheable. Another option is to use any cache class from the guava library.
Thanks for doing this, getting rid of a call to deprecated code
You can put getById, getByUuid, delete, saveOrUpdate in a common HibernateOpenmrsObjectDAO class.
On Tue, Dec 18, 2012 at 9:53 AM, Sadek Drobi <notifications@github.com>wrote:

> In framework/src/play/src/main/scala/play/api/mvc/Results.scala:
>
> >       */
> >      def stream[C](content: Enumerator[C])(implicit writeable: Writeable[C]): ChunkedResult[C] = {
> >        ChunkedResult(
> >          header = ResponseHeader(status, writeable.contentType.map(ct => Map(CONTENT_TYPE -> ct)).getOrElse(Map.empty)),
> > -        iteratee => content |>> iteratee)
> > +        iteratee => content |>>> iteratee)
>
> You should not need EOF, ending the enumerator simply should be enough. If
> it is not then there is a bug somewhere else.
>

I agree with you we shouldn't force EOF to end the enumerator.
But I confirm this problem because I had to send EOF to close it when using
it and Gaetan seemed to have the same kind of pb too.

Pascal



>  —
> Reply to this email directly or view it on GitHub<https://github.com/playframework/Play20/pull/621/files#r2445951>.
>
>
The matter is tat you don't read all tags but only the first one: ``$attributes[0]`` and the array of attributes is **not** ``$attributes`` but ``$attributes[$i]``
oh, it does not exist yet ? It could be a good choice for the next step as many external libraries are starting to build on it :)
extra ``\\`` here
Thank you for your advise!  I took the way of skipping the doctest altogether, instead of trying to get it pass.  The reason is that, according to what is written [here](http://stackoverflow.com/questions/2418570/testing-warnings-with-doctest), one should either turn warnings on explicitly with `simplefilter`, which I did and it didn't work for me, or one should write a decorator.  I chose to consider that writing so much code just to get some warnings actually caught is not worth it.

By the way, the warnings do show in the output of `./doctest`, but, apparently, doctest misses them.

Thank you for telling me about gist :-) I'll use try it out next time.
Еще можно modification time проверять, чтобы весь файл не считывать все время целиком.
Даже лучше. А то все это счастье лагать будет весьма.
A esto es a lo que me refería que no quería hacer, con esto nos estamos atando a trabajar solo con Fechas y no Fechas-Hora
Solo saber cuanto tardó.

De paso veo que hay código comentado que debería haber borrado.

Yo no me haría mucho problema porque esas acciones van a tener que
cambiar cuando hagamos la UI.
when does nocache=1 get put on the querystring?  Or is this not related to that?
This passed because when you use a message-expectation (should_receive), then all you are testing is that the message is sent to the receiver. So you're wanting to make sure that "subject" receives the "replay" method call.

And of course it does on line 33.

Generally -- only use should_receive on a different class... Like if you wanted to make sure that Fighter received a "take a bow"... then you might do something like this pseudo-code:

```
it "should have the winner take a bow" do
  fighter_a = Fighter.new
  fighter_b = Fighter.new
  match = Match.new(fighter_a, fighter_b)
  fighter_a.should_receive(:take_a_bow)
  match.fight!
end
```

We'd be asserting that when I call fight!, fighter_a would be told to take a bow.
!! test removed !!
Fixed comment now as amended commit.
What about integrating better with the StringBuilder on the outside? The String you return is appended to a fresh (?) StringBuilder.
This file can be deleted, it was created for sunspot/solr
")" needs to be before the %.
:suspect: We don't appear to use delete_log in any other place than right here. Something about the way this is written doesn't jive with me.
I can use it as a command parameter so it will be serialized across.
I don't think it matters, but strictly speaking if you're passing a name explicitly I think the `isFQCN` parameter should be `false`.
So as not to interfere with any other tests
When adding tests to an existing module, I should try staying close to existing style. 
make_all_testfuncs() was there (I just renamed it), and since it used module functions rather than class methods, so did I
( and that would have been my default choice anyways in this case).
please add a second ')' at the end of the line.
i was planning to put it back in an update
I initially did but thought you guys wouldn't like it :D
Just following the style set in the rest of the code.
I'm not sure, the functions won't make much sense and each loop is dependent on the data from the previous loop.
I added a comment for each loop that hopefully explains what it does.
Boolean?
Yes, it's the same approach.
To be honest, I use solution-wide analysis by ReSharper and it highlights such members with suggestion to reduce accessibility. It's a best practice and I give consideration to it.

For example, R# says that method isn't used from public scope and can be made protected, then I check why the method isn't used and:
- when method is used, but R# doesn't see usage I mark it as UsedImplicitly (i.e. used via reflection) or PublicAPI (i.e. used from plugins), R# won't suggest to make that method protected again.
- otherwise I change access modifier to protected because it's enough today. Method can be published in future if it ever become needed. This approach is pretty simple and should work well for projects which source code is completely accessible.

Risk that someone will parse back value returned by ToLine I estimate at one level with risk that someone will call private method via reflection, implement external command-type to command-name mapping and so on — there is always risk that someone will reinvent the wheel without looking attentively for existing opportunities.

I have another two additional thoughts today :)
1. The idea of such wrappers seems to be an abstraction layer between high-level code and command-line access implementation details. It's a good idea to separate git access layer, so I doubt that someone will use self-reliant command and specific details at the same time. Probably someone will need, for example, “branch” command name, for other purposes than deleting branch with GitDeleteBranchCmd, but GitComandName method isn't static, so the only way to get it is to create instance of a command just to read command name. If I understand the situation correctly, it would be better to have a separate static class with public reusable constants.
2. Imagine that all methods of GitCommand are public. Then GitCommand will become a candidate for deleting because it will do nothing but aggregating public data. The hierarchy could be simplified with converting GitCommand class to an interface and extracting ToLine method to extension method for that interface.
Why isn't this WT_ERR(__wt_lsm_copy_chunks(...))?
I instantiate the indices here so I can set the cache resident flag in the underlying btree handles.   I did a version that made it possible to cursor.reconfigure with the cache_resident flag, but decided it wasn't worth the effort, I didn't see much reason to reconfigure the object after open.
Most of the LUA code I've seen used single line comments for function docstrings. To be honest I don't really care which style we use as long as it is consistent :)
Please use "lazy val" instead of "def". Reasons were explained in another comment (scala.collections.immutable.List isn't java.util.List but a functional list that has to iterate over its elements to get the size)
Could you add a ``@return`` annotation? I found this a couple of more times on lines: 96, 108, 120, 132.
@denniskeefe - I would consider removing the comments in this file because they don't add any value IMHO and clutter up the code.  I realize it's probably boilerplate Devise-generated, but we should probably remove it anyway.  If we ever need to add new functionality we can always review the Devise docs.
let's go ahead and use r here, and remove the mention of @
I noticed there was an old conflict here. the two lines were almost identical, I chose the one above for now, but let me know what you think. After splitting this in multiple lines I could see that the only changes are that:

  * the first line has:

        .spec-commentary{color:blue;font-style:italic}
        @media print{.no-print{display:none}

  * the second line has:

         article .has-permalink{padding-top:40px;margin-top:-40px}



I did one additional change in this css: there are a lot of fixed px widths set, one of which was 1170px. 

    .container{width:1170px}

This was making the articles too wide in my laptop (I had to use horizontal scrolling). I changed it to 1120px. Is that OK?
Можно создать класс роутинга, в котором будут прописаны правила, какой контроллер вызывать в зависимости от урл
Why don't you use $input ?
OK, changed it with 3d3b527
Does this port have to be an "admin port"? I know 8089 is THE admin port, but you can change it to something else, right?

Or is any port you specify here an "admin port"? I changed per your suggestion, assuming this latter statement is true.
In the meantime, I tried to match the conventions from the rest of the calls
should be more, mocha currently inlines a bunch of xbrowser stuff as well but I really need to rewrite them with component(1)
Ha ha, there [is one](http://celery.readthedocs.org/en/latest/configuration.html#celery-always-eager). :-)))
???

I am saying, that it is not necessary that all stations in range(1, len(stations_nodes)+1) exist in Ljubljana. You should iterate over stations_nodes to see which stations exist.
Strange. This is already fixed in main repository, why it shows it as added line?
Then add a comment. Otherwise it would be probably better to port `django.core.validators.email_re` to JavaScript. It probably does not require match to port, just some correct escaping. Maybe there is even something on Internet about that.

Because if would be better to use the same regex, so that it succeeds or fails exactly in same cases.
No, this is not needed in base. Not all pages will use this library. Move it to home template.
And why are you doing this only for Facebook backend? You should fix it everywhere.
Я заменяю на анонимку чаще всего только для того что бы работала нормальная подсветка синтаксиса, поиск ошибок, и рефакторинг. 

IDE не будет искать вызовы функций внутри строк, а вот в таких анонимках все будет ок. В нашем же случае это конечно не имеет никакого значения т.к. врядли мы когда то будем пытаться искать сколько раз и где вызывается функция `Yii::getPathOfAlias($name)`.

Уберу.
This could be simplified a bit to `fn(expr1, expr2) and compare?(expr2, ..args)`.
I added some more tests to ensure correct handling of failure cases. See [`tests_security.py` L167+](#L3R167)
Can omit the checksum when the stable URL is a git checkout.
Awesome, thanks. I was wondering about the right thing to do there.
For some things, like TH1s, you have a method to do this:

http://root.cern.ch/root/html/TH1.html#TH1:GetDirectory

I'm not sure if there is a general way to do it, however.
Actually thinking about that.  I can not say what happened because I ended up putting those mods back in by hand because the conflict was crap
accidentally put the string in there twice?
Ah okay, then maybe i droppen the wrong methods. @Naios will check this for me ans drop what i forgot.
because you execute this query on char db and factionchange table is on world db
codestyle! missing space between if and (
from bootstrap - not my original
I think this can be simplified to just:

if status:
    projects = projects.filter(status=status)


or possibly:

projects = projects.filter(status=status) if status else projects

The Q() objects are only used above because an or is required.
What do you exactly mean by that? Attaching listeners? Could it be done like for the CLI?
It's the default value from the driver

2012/4/27 Sanne Grinovero <
reply@reply.github.com
>

> > @@ -53,5 +58,10 @@
> >       * property is not set, we'll try this port.
> >       */
> >       public static final int MONGODB_DEFAULT_PORT = 27017;
> > +
> > +     /**
> > +      * The default value used to configure the safe mode {@link
> MONGODB_SAFE}
> > +      */
> > +     public static final boolean MONGODB_DEFAULT_SAFE = false;
>
> should we really default to `unsafe` ?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hibernate/hibernate-ogm/pull/76/files#r743525
>
I think yeah ? it seems explicit
I prefer the { on the same line as function, but apparently the common way in PHP world is to place it below - ++ for consistency
Instead of this weird shift-based loop, if for some mysterious reason, you really want to guarantee order of evaluation, you could have two nested loops, resetting the array each time you start. Not sure which is fastest (your implementation resizes the array repeatedly while this alternative allocates/deallocates at least once too often).
try: except:
This is sort of nice, but I think it might be unnecessary in PHP, where people are more willing to learn by trial an error.
hService to hostedServiceProperties?
`wrap` sound like a weird name here, no? (And yea, I'm nit-picking.)
We want to clear the etag even on older nginx.  I'll clear the etag manually.
I would rather do:


```
iin_keep = np.zeros((n_keep, n_obs))  # best_stage1 --> iin_keep
ssr_keep = np.inf * np.ones(n_keep)
...
for ii in iterator:
    ...
   # replace value that we will pop out (the biggest amongst the previous n_keep kept ones)
   ssr_keep[-1] = ssr_new
   iin_keep[-1] = ii2
   # sort both ssr_keep and iin_keep
   sort_aux = np.argsort(ssr_keep)
   ssr_keep = ssr_keep[sort_aux}
   iin_keep = iin_keep[sort_aux}
```
and then, use all value in (ssr_keep X iin_keep) for stage 2.
can you check for exog_Q instead?
I played around with the testCase and I can't use updateOptions because it doesn't allow to reproduce/test the bug (valueRange, dateWindow is changed). So I changed this to:

DygraphOps.dispatchMouseDown_Point(g, 10, 10);
DygraphOps.dispatchMouseMove_Point(g, 10, 30);
DygraphOps.dispatchMouseUp_Point(g, 10, 30);

to simulate zooming (both y and x-axis). 

While the testcase correctly works for y- axis, it doesn't work for the x-axis because when I dispatch the double click g.dateWindow_ will be set to null and xAxisRange() returns the extremes. I am not sure if this is intended behavior or a bug. 

RIght now it is not possible to set a fixed dateWindow (similar to valueRange)  which is reverted back to when unzooming.
I can refactor the code to introduce a dateRange (similar to valueRange) and use dateWindow similar to valueWindow. 
However I am not sure if you want that or not. 
The easiest fix for the specific pull-request/test case would be to just assert if the yAxisRange() is correct and ignore the xAxisrange()
I think part of Barney's concern was making documentation of the magic numbers go into doxygen (sorry if I misunderstood!) so I just doxygenized them.  (see latest revision)
The antecedent here is confusing.  "these objects" seems to refer to ImageView and ConstImageView.  But you meant images in general.  Recommended text:
```
There are additional flavours
of \texttt{Image} that you might also encounter: \texttt{ImageView}
provides a mutable view into \texttt{Image} instance data, and
\texttt{ConstImageView} an immutable view into \texttt{Image}
instance data.  These may be the type of images returned from
various GalSim functions, but as they work the same way as \texttt{Image},
you shouldn't notice the difference.  See their doc strings for more information.}
This is another part of the file that has some ambiguity as to who it's aimed at.  Does everyone who might use this software know what a build system manager is?
This is Gary's old way.  I'm inclined to actually get rid of the OLD_WAY section, since I don't really grok it, and the new way looks much clearer to me.  But I left it in for now, since that's the way it was.  (I think the OLD_WAY was before he wrote the XTable and KTable classes.)
i did that on the Consumer-Facing side of things in the controller. I thought this was a cleaner step towards obsolescence of the find-by-id style, but i'm open to that if you prefer.
discussion "testing 1"
You've dropped to past tense here. Should be "You already caught a glimpse". Also you can run into the second sentence ", where we defined the ..."
Updated metadata, left the includes for now.
What about checking if `provider::server` exists?
We need <code>self.click_options() </code> before this.
An honest biography for a change.
Might be better to default the ``login_url`` to ``None`` then lookup the URL using the ``reverse`` function rather than hard-coding it here.
If it's the same, this is pretty complex code---needs to be shared, maybe in DropJob then
Also a separate file.
Would rename this to something like "has sent CentComm a message: " so as to not confuse admins.
Right. The GIT_PREFIX env variable is set by git to point at the actual current directory. So by changing dir to the GIT_PREFIX, things are executed in the correct directory
@solidgoldbomb This is where we should switch to uniptr_t if we want to be able to run unit tests?
И можно кешировать предыдущее значение - так  быстрее будет, чем по индексу в массив обращаться.
I've removed this method.
So, which implementation do you think we should use now?

My suggestion: only provide `threshold` as paramater, set it to None by default and use `0.5 * np.max(hspace)` (note that in the doc string) as default threshold.
could you write down the maths problem solved by this PAV?

we have one too with @fabianp for isotonic regression
Recent Cython optimizes this into a C-style `for` loop. Try compiling

    def foo(int n):
        for i in xrange(n):
            print(i)

(And note that you don't even need to declare `i` when `n` is properly declared, so I can even remove some more declarations.)
Why `_net`?
pay attention to indentation @aymas :)
I saw it didn't know what it does. It was a bit late lat night ;) I'll have another look, thanks.
Removed underscore dependency and used this snippet, thanks.
Maybe I have some leftover .so files.  I didn't do a python build.py before testing, so perhaps I'm on an old version, despite your updates.
Ok, I'll merge and then fix in a separate commit.
You are correct, I overlooked this: https://github.com/Exercise/FOQElasticaBundle/pull/86#r661646
This implementation doesn't match the apidocs.

these methods are to retrieve the input param that the user set on the control.  The information you're retrieving now goes in rect and size properties.
Validation failed on line 215. E.g. user has given an empty name. Then user should be able to try again.
also see the pull request to the doctrine/migrations repository: https://github.com/doctrine/migrations/pull/87 (wrote that before)
See comments above, in CSimpleSorterBase::spillSortedToDisk, re. using DynamicRoxieOutputRowArray::getClear()
grouped versions (when implemented) should probably use smaller values.
Self contained URI tests
b var is used here
Is surrounding by parenthesis not equivalent?
I don't know what the coding standards say about when you should create a local variable or not. What are some guidelines for this?

I'm making this change, though. :)
You're right. Int-like strings are automagically converted for no apparent reason. Still, floats and padded integers are not...
These are defaults. I'm guessing we can add to them here, so that you don't have to remember all the variables you want to delete every time?
En fait ce sont les classes propres au jumbotron que j'ai récupéré sur Bootstrap, ces définitions ne sont pas dans la CSS de base de bootstrap, donc je les ai recopié.
simplify 
return valueInCategoryColumn != null
There is no reason to implement at test class level because already I have implemented with uploadHistorypage
Thanks, you are right.
Tää tekee nyt saman asian kun:
~~~clojure
(and (flush? hand)
     (straight? hand))
~~~
Please remove any logging dumps.
This probably isn't testing correctly; super users aren't going to be in the `.admins` set.
Done in 1a8e439.
newName check needs to check for .empty? as newName is always within the properties as "" even whe no name update is used.
I think you should use store_loot from the Msf::Auxiliary::Report mixin to store the gathered files.
The problem becomes that data outside [-1, 1] no longer carries any range meaning.  We don't know whether clipping or scaling is the correct approach.  Functions should accept correctly ranged inputs and produce similar outputs--if they violate those constraints, but documented it well, then the user must still take some form of action in the pipeline to address the issue.
Thanks for the pull request. Sorry I only had the time to review a small part of the code today, will continue tomorrow. However, I noticed that on my box all 2-D tests of the random walker now fail, can you also run the tests and try to fix the code if it also fails for you?