labofsahil/github-event-dataset-2012 huggingface dataset:
There should be no empty line between the docblock and the method in all cases.
mozApps.install() should be return an app object on the [spec] (https://developer.mozilla.org/en-US/docs/DOM/Apps.install), but I only get null, I have no idea to get the app object when installed, so I implemented another method 'get()' to get the app by name.
Automatic tags are shown in the tag selector by default, but they can be turned off, so I think setting this as automatic should be fine.

I'd prefer a tag that at least fits within the tag selector. Maybe something like like ```*Verify imported item type*```?
Do we need/want to keep these comments in?
Added in.
https://github.com/karlanderson/khan-exercises/commit/bc9070432a4019562c05009103693d3598f9f24e
Also wrong indentation
because "_" is ruby default. I aliased the name as "ember-rails" so if you like you can still require "ember-rails"
If this is something that shocks people I will revert :)
const без ссылки не используют, так как нет в этом смысла
функторы принято передавать по значению
Similarly it would be easier to return an integer here as well rather than passing in the pointer.
This check is done each time log directory variable is accessed. I would rather it was not done here.
Actually there's no need to QString() it (I assume it was either in error or to cast way constness). However, toUtf8() produces a temporary QByteArray which stays alive until after the function call. C++ is nice, at times :-)
Comment on first added line
This makes the column names human-readable. Do you know of an easier way to do this? It works.
Come to think of it, why make this configurable at all? Seems like something we should just do by default, all the time, no? Is there a measurable impact on performance if we have this on all the time?
Good catch, header is added locally.

WRT moving things to validator -- this is the kind of thing we need a "plugin" system for, similar to plugging functionality into oae-context. i.e., something that will be a bit better at "bootstrapping" all the modules in a more generic way so they can add Validator.prototype or Context.prototype plugins. Until then, we shouldn't mix dependencies by grouping everything into central modules like this, I don't think.
You'll have to add in a more explicit check than `!someConfigValue` since Booleans could be considered as an error in this example.
The intention was so that the model makes it clear which properties are "core" and which are "additional properties" specific to the provider (e.g., password).
Let's keep just one of the two getters.
I think this isn't necessary. You shouldn't be forced to close the view, nor should the user care that no useful information will be shown (though, at least the offset in the source file will be still printed).
Nope, good catch, I'll i18n that

On Tuesday, December 11, 2012, Jake Holman wrote:

> In manifest.json:
>
> > @@ -26,6 +26,12 @@
> >        "name": "password",
> >        "type": "password",
> >        "required": true
> > +    },
> > +
> > +    {
> > +      "name": "defaultNote",
>
> Is there an i18n string for this?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/zendesk/harvest_app/pull/1/files#r2375189>.
>
>
This was actually the way I first did it, but unfortunately when I tested it on older Firefoxes it didn't work.  (At the moment I don't remember how many versions I had to go back to see it fail, but the first failing version was >= 10.)   I didn't investigate what the problem was since I found a way that works on all versions >= 10.
I've taken the time to configure PHP properly, I've set my timezone and I use `time()` and `date()` extensively. Why would I want CodeIgniter to reset my timezone and break my whole application, when I don't even use the date helper?
Id like to leave this in, since I didn't write this I don't want to appear as though i'm taking credit for another persons work.
It's actually used on line 129 (it's just a pass through for whatever value you give it
I think this was supposed to be removed, right?
Cool.

Sorry about too many comments on the code.
I would love to see this moved into ripple/utils because I can see this being used more often instead of __defineGetter__
Fixed, any other suggestions?
i know the first subscriber model is going to be the right one, but lets make it iron clad and do `SubscriberModel.find(pub.id)`
What happens if people try this key, and it happens to be one that is assigned to a real account in production at some future point?

If its basically guaranteed to be safe, then sure. Otherwise fudge it to be sure or use a placeholder like <api_key> or whatever
unit tests should run successfully each time and should be independent from the data. Here if the service does have a news item that matches the filter you passed the test will fail, which is not correct. So enhance this test to so that regardless of the filter value it will succeed.
We cannot change this one. Otherwise we will break people's apps when upgrading.
What exactly is the problem here? I tried the following in the Chrome JS console:

JSON.parse(JSON.stringify([1, 2, 3, 4, 5])).length -> 5
Is this safe? `#fetch` will raise an exception if you don't provide a default.
I was asleep. The enthusiasm is appreciated, though.
connectionStatus match {
      case None                               => connect
      case Some(Left(r)) if !r.isOpen =>  connect
      case _ => all good
    }
Parece que es a drede: https://github.com/eternicode/bootstrap-datepicker/pull/184.

No tengo idea de cómo funciona, tiene pinta de hack.
How's your schedule look for Tuesday night?  I'll be available.
I hadn't seen this one when I reworked the Viewport today: is that similar to `Viewport#sqlWhere`?
didn't know about that, I thought it was the opposite.
I'll fix that in a future commit.
Thanks!!
Taka jedna uwaga: nie powinno się rzucać obiektami (tracisz stack trace). Dałbym `throw new TypeError("funkcja add wymaga liczb")` zamiast tego.
gatein-dep just released 1.2.0.Beta01 which actually should be used here instead of 1.2.0.Alpha06
Sounds good then!
Another option would be to have CacheValue migrated to o.i.util. ByteArrayKey is already there...
When can the entry be null?
you should use ``class_exists`` to decide whether the test should be skipped
This can be done on the stack in the `else` clause below. There's no need to introduce a memory allocation (and this one is never properly freed anyways).
`STAssertTrue` takes a variable number of arguments – the `stringWithFormat:` call is unnecessary.
Note that elsewhere in `pcolormesh`, the `faceted` option has been deprecated for some time and recently removed in favor of respecting the `edgecolors` kwarg.  Does that make sense to do here as well?
I couldn't figure out how to do roughly on a collection. That is, (map roughly coll) produced a bunch of functions, not their result. We've been down this path before, and I'm racking my brain, but it's not working!
Very minor comment, but I noticed in both resource files you are specifying a default of 120 seconds for @new_resource.timeout which gets passed to this call, so it seems that this line can just be:

def query_tags(tags, agent_ids=nil, timeout)

This would minimize the number of places in the code where one has to look for the default.
yeah you should really - precompute as much as is possible. then there is also no need to use the invsqrt approximation. so just store extra variables (or others processed).
Can you confirm this doesn't break category ordering anywhere else this function is called?
I didn't take the time to see exactly the initialization order that an experiment goes through but it's more robust to not have an ordering dependency.

I don't think calling `set_experiment_created_at` in `set_experiment_enabled` is a good idea since it adds an unexpected side effect to set_experiment_enabled. That's gravitating more towards an `init(experiment)` method that sets both of them, which already exists in some form in `AbTest#save`.

This method is pretty messy though. Not sure what to do about it (if anything).
- If you would like to follow PSR-0, the class name should have a vendor namespace. PSR0 requires the following structure: \<Vendor Name>\(<Namespace>\)*<Class Name>
- 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 "ProjectServiceContainer.php".
Are you sure that ``$events`` (of type ``false|object<Zend\Stdlib\PriorityQueue>``) is traversable?
Add a comment to that effect then please.  Something like:

`// Scale factor to scale button length from 50 to ~400`
Has been changed for all platforms, should be BB specific
Not sure why it would stop executing. Maybe something to do with locking or timeouts? It would occur to me to put ifelapsed => "0" on the command promises, and see if it changes anything.
So you could just append to object stored in a variable here.
Please check all translations. Especially those with "fuzzy". Those were automatically guessed. But of course not always right.
This is not good. You should set to `True` only if it is still set to `False`. :-)
Is this `-1` supported by client code? What happens?

So if I understand correctly, the question is what to do with new panels which do not have layout. Here you are adding it to special column? Is this really necessary? What if we just omit those panels and leave to client-side code to put those panels somewhere?

So here is one question to be decided: should be panels without layout be put somewhere by server side or client side. If former, then you should put it somewhere real, some real column. If latter, you probably do not need to pass those panels through. :-)
You could do something like that, yes. But this syntax is not correct for Django templates.

Or, you could use `.data()` function from jQuery to attach arbitrary data to DOM elements. Like id. And then search for that. (We are using that in #199.
IPB databases? Do you mean IPBoard? How it's related to mooege project? We can't really allow in user/site specific code to master repo.
Yeah but when you use PR (push request) that means you want us maintainers to review the code and merge in to mooege master.
Are these autogenerated, or do they have to be written by hand? Seems like we are duplicated a lot of information.
Надо добавить событие которые перед деспатчем получает конфигурацию, сетит ее и проверяет формат и выкидывет исключение если он не верный
I just didnt think of it, kek.
preferred, not prefered
lepiej zrobic tak jak wyzej
no it doesn't, fixed.
The value of ``indent`` used to specify how many characters to indent for each level (with a default value of 2 if the version without ``indent`` was called). Now you are using ``indent`` as a flag and the value. The api has not changed but the implementation has (the contract is broken). Do you have a way to restore this behavior?
Yeah, I think keeping the requirements low could help adoption
This example is much better. I would just rename it `fetchURI` or `getURI` instead of `readURI`.
well, die client still receives events via defined methods. I'm not quite sure what instead you want to do here? Can you layout your idea in a little bit more detail?
`emit_token`
You know what's the future too ? The EDIT COMMENT feature. And the future is now ;)
No strong preference.  `sizeof(url_)` does seem a bit better.
Why can't we just tell this method to expect a `QSObject` instead of `id`?
Python has a few different string operations for this

    subjects = u""
    for subject in book.subj:
        subjects += u"    | {0}\n".format(subject)

Also, we are dealing with unicode strings, so you need to prepend your strings with `u`
Oh nice, thx.
Unless I misunderstood the purpose of this method, it might be better for it to explicitly return true/false instead of an int/nil.
I believe there's a helper to get you to the `templates` dir.
Agreed. I didn't understand CSS at all when I first built dygraphs, so there are a lot of missed opportunities here. The "labelsDivStyle" option is probably the most egregious example -- there's no reason for it to exist, since the labels div (aka the legend) can be styled via CSS.
RealGalaxy
Barney, once I'm fixing it in one place I can fix it in the other place on this branch...
grrrrr! Our code must be perfect! :)
the deprecated one extends the good one but i think worth leaving both here.

i have renamed this class EntityConfigMap, btw.
Correct. I am updating this class to address some of @ahgittin comments on issue #1 and will address this too.
This just gives the toString of the exception - we need to give the user the stack trace. We could do that in this log and/or as the cause of the IllegalStateException thrown below.

For setting the cause below, I still think we want a try-catch in case there is an exception accessing the list of images - then we'd propagate the "wrong" exception, rather than the user's exception that would be more meaningful to them.

Also, listing all images on AWS can take over one minute on my home network, and have 20,000 images when no user filtering, with one log message for each below (just so you're aware).
Thanks, updated.
Also added --credits-- for the actual author of the test (the reporter)
On Fri, 14 Dec 2012, Hannes Magnusson wrote:

> >    zval **data;
> >    mongo_cursor *cursor;
> >  
> > -  if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "Oz|zz", &zlink,
> > -                            mongo_ce_MongoClient, &zns, &zquery, &zfields) == FAILURE) {
> > -    return;
> > -  }
> > +	if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "Os|zz", &zlink, mongo_ce_MongoClient, &ns, &ns_len, &zquery, &zfields) == FAILURE) {
> 
> Wouldn't z/ been a simpler fix?

Maybe, but using the correct convert_to_*_ex is also a good thing!
In Python, it's ``True`` and ``False``.
that's fine by me, let me investigate and I'll redo.

On 3 September 2012 12:18, Philipp Meier <notifications@github.com> wrote:

> In project.clj:
>
> > @@ -1,5 +1,16 @@
> >  ;; Use leiningen 2
> > -(defproject liberator "0.3.1"
> > +(require 'clojure.java.shell)
> > +
>
> I wan't to get rid of this. Although i like the idea, this only works on
> unix, not on windows. Second, I think the synchronization of the project
> version should be achived by a leiningen plugin that calls out to the scm
> system defined in :scm, see
> https://github.com/technomancy/leiningen/blob/master/sample.project.clj#L271for an example.
>
> We should investigate https://github.com/relaynetwork/lein-release or
> https://github.com/ato/lein-release (i like the changes by ato)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/ordnungswidrig/liberator/pull/9/files#r1516037>.
>
>
Na realidade acho que esse método não faz sentido, é facil pegar o dado direto do dicionário `MONTHS` usando a sintaxe `return self.MONTHS.get(num_month, '')`
Hi @german, thanks for the pull request.  Can we change this to say `obj := Object new.` and remove lines 1-2?
It's good practice to remove the QtModule/ from the headers as in Qt5 they have moved around and will need to be changed, while w/out the module they'll still work :)
This will be updated in next push  which is very likely today
So, I accept that `last` is slow — it says right in the docs that it runs in linear time. And of course I’m happy when a change results in a dramatic performance improvement. But this makes me uncomfortable. `last` may be slow but it has the advantage that it is *extremely* clear as to what it’s trying to accomplish. `(last windows)` — it’s hard to get clearer than that. `(first (rseq windows))` is semi-opaque. An unseasoned reading of it would (A) take a lot longer than `last` and (B) reveal no clues about *why* the code is being written this way.

I’m not a comp-sci guy, I’m not well versed in theory or algorithms, so I can’t say that there’s no good reason that `last` should be slow and that `(first (rseq …))` shouldn’t, and that the latter is the way to go — but I can say that it doesn’t sit right with me. Code should be clear **and** fast.

So what I’m tempted to do here is to wrap the semi-opaque performance optimized `last` replacement in its own function — something like `fast-last` — so at least it can be annotated as to *why* it’s being used as opposed to just plain old `last`. Any reason *not* to do this?
There are white spaces at end of lines and tabulations in this file. Please remove/replace them.
humm. With the correct indentation.
I really think we should use some images here insted of this heavy css but I will ask the UX team by myself.
Soon, when we have the [MPIRequirement](https://github.com/mxcl/homebrew/issues/16678), we can use that instead of the `depends_on 'open-mpi'`
This seems to be duplicated
Выравнивание?
Sorry, I still don't see the use case for adding the individual themes. By default the asset pipeline adds assets/themes to the paths, so in the example above the paths would look like:
["assets/themes/xproject", "assets/themes/genigame", "assets/themes"]

The themes_for_rails helpers in this code construct logical paths to assets like:
xproject/stylesheets/styles.css
So assets referenced using these helper methods will be found because of the default entry on the paths list not because of the individual theme entries.

Can describe how you reference an asset that would take advantage of the individual theme entries?

I noticed this code and I'm asking about it, because the precompiling code in the asset pipeline doesn't seem to handle having multiple ways to reference the same asset. And this code causes that to happen. In the example above the xproject style.css would be available through 2 logical paths: xproject/stylesheets/style.css and stylesheets/style.css
This `implicity_wait()` property does not actually do a wait. It's a setting for how long selenium will implicitly wait and in this case you're setting it to 1 (usually the default is 10). As it's an implied wait you can remove this line altogether and let the implied wait start when it's required :)
I was duplicating what I saw elsewhere, e.g., at https://github.com/mozilla/marketplace-tests/blob/master/pages/desktop/developer_hub/submit_app.py#L15 and throughout that file. I thought perhaps that was a standard, but I assume now that it is not and I shall change all single line comments to use #
Wrong CS here
I thought that it would be nice to allow Markdown here as well. The strong just added readability. Both reverted back.
This quite a bit of logic for a template, don't you think? Fine for now, but guess we should take care to follow one design. So e.g. allways try to define form elements like this in templates (like you did here)?
I've removed the commented out code
Use a proper numbered list?
Ideally we'd have the key size as part of config. This isn't going to change anytime soon, so is fine for now.
Да, лучше унести.
I will contact the original owner, as though the dataset is _primarily_ synthetic, it is not 100% synthetic.  There is a portion (particularly, the transition offsets) that was created by hand or by some real robot, and it is this part that makes the data look "interesting" rather than just a straight or curved line.  I could potentially create a synthetic generator function, but the output would look more artificial than the dataset provided.

In addition, the dataset provided contains reference output I compare with the `KalmanFilter`'s output to check validity.
you could apply np.power inplace which would avoid a copy
for large datasets the memory requirments can be rather large: ``2**max_depth *n_samples`` just for Tree.values. 

btw: please check if ``len(X)`` does not break for sparse X
There is a safe_mask function already.
On Aug 17, 2012 12:16 AM, "Andreas Mueller" <notifications@github.com>
wrote:

> In sklearn/feature_selection/univariate_selection.py:
>
> >          The set of p-values
> >      """
> > -    X = array2d(X)
> > -    y = np.asarray(y).ravel()
> > -    args = [X[y == k] for k in np.unique(y)]
> > +    X, y = check_arrays(X, y)
> > +    if issparse(X):
> > +        args = [X[np.where(y == k)[0]] for k in np.unique(y)]
>
> Also of course the where would be unnecessary.
> I could do a safe_boolean_indexing helper function but that feels a bit
> overkill...
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1025/files#r1393613>.
>
>
I don't really see the point of keeping the `density` attribute on the dense RPs.
spaces please, below too
There's already helper for this you can call. SetClientCredentials extension method. Also just pass the ICredentials.
How about ABSTRACT_DISPLAY_PRESENTATION = 'regular' | 'hover'?

That would allow us to add more atrategies later.
should be `N_BUFFER_PROP_LAST`
Good catch! I always forget about this.
This is actually work started by those failing tests on Windows on ci. I'm
pretty sure the order of read methods is not the same there at each run and
indeed that would fail as it does today even without the need of any new
test.
On Nov 19, 2012 11:07 AM, "Hardy Ferentschik" <notifications@github.com>
wrote:

> In
> hibernate-search-orm/src/test/java/org/hibernate/search/test/id/UnorderedIdScanTest.java:
>
> > +
> > +import org.hibernate.search.test.SearchTestCase;
> > +import org.hibernate.search.test.util.TestForIssue;
> > +
> > +/**
> > + * No test method is needed as we just want to
> > + * verify the SessionFactory is able to start when
> > + * using this entity.
> > + *
> > + * @author Sanne Grinovero
> > + */
> > +@TestForIssue(jiraKey = "HSEARCH-1048")
> > +public class UnorderedIdScanTest extends SearchTestCase {
> > +
> > +	public void testStartup() {
> > +		//Just verify the SearchFactory could be built
>
> We really need a way to test in a more unit based approach. Weak test to
> just say the factory build, hence the issue is fixed.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/hibernate/hibernate-search/pull/353/files#r2164964>.
>
>
Yeah, highly unlikely (read prob. impossible) to happen. I tend to think of most assert's of that nature.
However, you're right it's non-time critical and might as well be a assertex I suppose.
The compiler will probably hoist that anyway...
If you're not using the default format, then don't show it.
Use the new URL (ie. don't add the extra `/query` part) and instead we'll have a redirect for old legacy URLs. 

Also, those `&` should be `&amp;`.
This might be clearer as

``` python
chdict = {
    'changeid' : None,
    ....
}
```

Also, master.addChange should accept a codebase argument, and only do this if it isn't specified. (The default for the argument should probably be None).
%s/from the / to the/
I left out this (half)line.

@whimboo Have You any proposal for the description? 
I asked help on irc and @AutomatedTester began to talk, but he got an i/o error and left.

My first thought was `Tab Title`, but it's too vague, imho.
Why are these indented an additional space?
I think there is a semicolumn missing here, isn't there?
Leaving the `ctor` in the cocos2d classes is safe. You can remove them later if you want. 

Regarding the pull request, sure, I can fix it.  Could you tell me which part could I change?
The code in the pull request should work without the `ctor`s. But I might be wrong...  Have you seen any code that you know is going to fail ? Thanks.
That's how I would go about it (and intended it to be done).
Why ```save=False```?
Was there a reason to not just do a `link_contents` for the `app/views/shared` directory?
while !ok or not ok
enum_domain.rb uses the same registry key doesn't it? 

I can try enumerating on a network with multiple domains and see what results come back.
OK, so I think I see now... you're trying to estimate fewer parameters.  We could probably still do this in a generic way, by simply setting certain coefficient multipliers to zero, but perhaps that obscures things more than what we gain?
dont like this sentence. but have no good replacement. maybe you have?
I like this, it's fun :)
I suspect you did a lot of testing to come up with 100?  lol.  I have something similar to make sure I don't thrash the shader program cache.  You are welcome, but not obligated, to add a `frameCount` to `SceneState` to cleanup everything.  I also plan to expose this to shaders as an automatic uniform, but we don't have to do that now.
Throw if `date` is undefined?
This class is a lot better now - much more consistent with what we've done with `Cartesian3`, etc.
We discussed getting rid of - or not adding them - before.  Don't we freeze the constants so we get a run-time error if someone modifies them?
I'm not sure I understand how it's self documenting?  How do I even document the template object with JSDoc so that users know what properties are available?  Or in you opinion should all template properties be public properties on the object being constructed as well?  If that's the case then maybe I agree about the self documenting portion, but that doesn't fit in all cases (options aren't always public properties).
Hm forgot to remove this line.
Personally i dont like this very much... what if you need more than 1 command of each type /etc ? it also makes the test code harder to read. Tests should be very easy to read so that one understands exactly what and how it is beeing tested.... not sure what this tries to solve... :-/
This may just be me, but it seems cleaner if you added these two error checks as an 'else' condition if typeHashMap doesn't contain either 'service' or 'username'
Can't follow this paragraph at all, and I'm pretty sure "Hence." is not a complete sentence.

 Is it safe to use a person object that's been removed *up until* the point of a save operation? Or should we just avoid using any Person objects after they have been removed from the database. It is not clear.  

When you remove a [Person] object from the database, the object remains valid until the next time the database is saved. You can initiate a database save operation explicitly, by calling [save], or implicitly, by creating a new contact. At this point, any deleted [Person] objects are invalidated. Continuing to use a deleted object **after** the save operation can result in unpredictable behavior, including crashes.
Hmm, that seems really odd.  how does something like `StreamModule.createStream(KrollDict params)` work?
I am not sure it's necessary to remove this, but I will consult with Arthur to be certain.
Please don't hold up a merge for this item.
Do you need to initialize?? Isn't the default going to be 0 and 1
cache document.getElementById. Otherwise its wasted code space, and code size matters a lot on the web.
why are we setting this back to NO?
Interesting to note, the following does not work.

```
    Function readEntries = () {
      reader.readEntries(
          (List<Entry> results) {
            if (results.length == 0) {
              success(entries);
            } else {
              entries.addAll(results);
              readEntries(); // XXX: Not able to resolve closure.
            }
          }, errorHandler);
    };
    
```
This however does work. I have a feeling in C# the previous does work, I could be mistaken. 

```
    Function readEntries;
    readEntries = () {
      reader.readEntries(
          (List<Entry> results) {
            if (results.length == 0) {
              success(entries);
            } else {
              entries.addAll(results);
              readEntries();
            }
          }, errorHandler);
    };
    
```
missign space
package ? how is it a keyword describing the bundle ?
@jtopjian Could you please leave a comment about this change, or rebase your commits to include a description of this change in the commit message? Thanks!
@widget.tabs[0] твой фирст
I like how you've used the try/except here.  <3 <3 <3
Dunno how I managed to swap those. Will fix.
If `c1` and `c2` aren't going to change, then make them `immutable`. That way they don't get accidentally mutated, and it gives the compiler a hint to use in its optimizations.
Nice, please add tests
fyi, you don't actually need to throw, the test runner will perform the comparison of the RenderTree
We have no support for BSD tar anywhere right now, so taking that into account already is probably overkill.

We can fine tune this further if we start looking into BSD/OS X support, and if the current stuff turns out to be an issue.
Wrong author ;-)
Because grunt 0.4.x isn't stable, it should only be a development dependency.
Not sure if this is a good idea, but we use 10px everywhere.
И вообще мы this не пишем явно.
This line in the docstring doesn't make sense; when you're importing, you aren't going through the UI, so it doesn't matter whether the UI prevents this or not. In actual fact, though, the model layer itself should prevent it, not just the UI; it would make sense to note that here.
There is a missing blank space here. ;-)
"it isn't implemented yet!" <--is this comment still actual?
Why would we want to make our core code dependent JUnit? Also, "throw new IllegalArgumentException" is what we do everywhere.
That's what I had initially (!), but it didn't feel right to me, but I am assuming you insist so I'lll change it.
Maybe in another PR.
This looks very weird in Githubs web view because you're not using tabs for indentation.
@samsonasik  Please read the header of the table
You could set this to `lambda { UserMfaSession.create(user) }` instead and remove the need to set the subject in the `nil user` context.  The following `it` assertions should still work.
``` e.g. ``:source:``` -> ``` e.g. ``:sourcedir:```
Lots of extra solution configurations here.  Unless there's some native code involved somewhere, we shouldn't need any of them except Any CPU, so let's delete them.
Is CZML going to expose the `center` property?  Or just rely on being in the center of the reference frame it is defined in?

I exposed the `center` property for the primitive so users don't need to construct their own model matrix for the common, default case of WGS84.
I guess it doesn't matter, since lft is always less than rgt for any given node.  But I can change that if you like.
En fait, je trouve que "cela" est un peu empoulé. J'aime que les posts soient écrits sur le ton d'une conversation informelle. Et je ne dit jamais "cela" à l'oral.
Compile all jade templates into one js file for client side rendering
I know this bit is just refactoring now, but the magic constants of 1, 2, and 7 really would benefit from constants that explain what they mean and are accompanied by rationale either on the constant or here at the call-site.  For example, MIME_TRUNCATE_102K would be a good constant, and I could see the explanation being "Our target was 150k, but that's the biggest size allowed without sending us the whole message."  I am particularly interested in the Type of 1 (Plain text), and MIMESupport of 2 (always send), especially as it compares to not desiring the MIME structure or desiring the MIME blob (type 4).
Tested without the blur(), and it works the same, so I'll take that out.
what are titanium properties?
Need to declare Ti/UI as a dependency and then reference UI.SIZE.
never delete this guy
I'm assuming the idea is here that the ISE comes from 
```scala
println(m.end)
```
Would there be an additional output line from the 
```scala
println(rx.findAllIn(t).end)
```
call?
I don't want to confuse the notion that closures are not supposed to evaluate to a number.  I think of it more like printing an empty string -- it's blank, it doesn't have a special token to indicate an empty string.  Whoever writes the printf can always enclose in quotes or do other things to specially signify an empty closure.
The attribute will default, if not specified, to "-1"; it is used when
And this one you can probably quess.
Don't know if you have the time, but I think these can be automated if we have a dom element (like a text box) and give it focus we should trigger the keyboard to appear. This would allow us to completely automate the test
Good catch, personally I think you should just remove it entirely, I feel like it does bizarre things
Lets revert this
Where is the cost for Consistent read?  Does it slow down writes?  Are reads constant speed?
this looks really bad...who wrote this!!!!!
Incorrect indentation for this whole block
You do not implement this method
Why not "#%06X" % random.randint(0, 0xffffff)