labofsahil/github-event-dataset-2012 huggingface dataset:
This may be a stupid question, but what does TUIO (Touch user interaction option?)
тоже самое
```
NSLog(@"Before: %i", [first hammingDistance:second] );
```
Попытайтесь найти придумать решение, которое сможет обойтись тыкания нуклеотидов наугад. Вообще это, конечно, не критично в данном случае, но поискать решение можно и просто для тренировки.
Хоть в требованиях этого и не было — очень хорошей практикой считается выносить подобные числа в константы. Чтоб можно было оперативно поменять число.
Это не ошибка, но хорошей практикой считается держать категории в отдельных файлах. Например  Cell+Mutator.h  и Cell+Mutator.m

http://macdevelopertips.com/objective-c/objective-c-categories.html
Тут не совсем дело в удобстве, это как написать:
if((a < b) == false)

Так никто не пишет, но всем пофиг, конечно =)
Could you add the required field spans to the fr locale as well?
We can let method_missing proxy this through
Don't really like having to use `RACReplaySubjectUnlimitedCapacity` to denote unlimited stashing, since it belongs to `RACReplaySubject`.
Maybe just a flag to toggle between stashing unlimited values and only the latest value is enough?
These should be initialized in the `before` block, and copied, due to some memory management oddities with Specta (we might get crashes if they're left here).
Maybe include the thx pattern here too?
I don't believe you need to add the `crypted_password` field to the factory. You can use password / password_confirmation as you normally would. I could be wrong tho
Ooops, that's right. Currently we're not using the isValid() anywhere and I actually doubt we need it. Nevertheless, I'll correct ... It should not change anything though.
It moved to the downloadPolicy. Just need to clean that up.
Isn't the option to include the current projections just there to work around the problem that mne_process_raw cannot add, but only replace the projections in the file?

I thought it would be good to have the option to only have the ECG SSP projectors in the file, in mne-python we can make an "add_projectors" function which keeps the existing projectors.
if there is only EEG channels like here it should be only 1 subplot (not 3)
nothing ,they came with the sample code and i forgot to delete them
Locale variables are nicer to debug :)
Sure thing.

When I get to a real keyboard, I'll rewrite. Do I resubmit changes through fit or just reply to this email with the edit?

Carolyn


Sent using a mouse-sized keyboard with feigned autocorrect intelligence.

On Dec 25, 2012, at 10:23 AM, Mike Fiedler <notifications@github.com> wrote:

> In soft_skills.rst:
> 
> > +professionals are spreading themselves too thin on projects and
> > +deliverables that do not serve the organizations mission. One way
> > +to focus the efforts of Operations is to answer these questions and
> > +to ensure that the Operations organization, whether insourced or
> > +outsourced, is delivering services that provide the most value.
> > +
> > +Protection of Information and Services
> > +--------------------------------------
> > +
> > +Often the Operations professionals in an organization are the people
> > +who most completely understand the technical risk to organizational
> > +assets from an IT perspective. Senior management within an organization
> > +will usually understand financial risks, risks from competition,
> > +manufacturing risks, etc. but they often do not understand IT risks
> > +because of the deep-dive technical expertise required to comprehend
> > +threats to the organization.
> The word "risk" features heavily in this sentence. Any way to make it less risky? 
> 
> —
> Reply to this email directly or view it on GitHub.
>
I rewrote it. See the branch.
I prefer this syntax, but if you want I can convert it without problems
You should add `use Symfony\Component\HttpKernel\Event\FilterControllerEvent` because of the use of this class on line 104
I don't know nginx well, but I wonder if the above 2 items are strictly needed?
Great thanks, I have added examples of this as well
这个是日志啊？ 不用加句号吧？
There are only 2 ways to do this:

1. Iterate over known interpolation methods and search for them in the string
2. Iterate over interpolations parsed from the string and replace if a corresponding method is found.

As of today, method #1 is being used.  #2 is much faster, but it requires placing rules on how to parse the string for interpolation keys. Supporting ":idwhat" as "20what" is just silly, imho.  Requiring non-word characters (or some other token) around your interpolation keys is the norm.
The following content add a new feature : 
When an operator remove an Entity from this list , the MobSpawner associate with this Entity can't be generate on the DMWorld


Xephi
Hi Dan, is there a reason this was deleted?
In this case the javadoc seems pretty wrong, like it was copy/pasted from somewhere else?
use: log.warning("Consumer is rejecting delivery with tag [{}] - requeue [{}]", deliveryTag, requeue)
since the current code will create the message anyway even if warnings are not enabled.
it's there: 1677
Добавь идентификатор. В конце концов, не использовать параметр легче, чем
потом бегать и менять все, когда ВНЕЗАПНО понадобится. А рано или поздно
понадобится, инфа 200%.
In fact, it's a generic test to see if applying format then parse produces the original date. It's not specifically linked to the authentication date.
In the future, we maybe will have a new ServiceTicketValidator with this extra parameter (after CAS protocol revision). So far, we only have this one to customize...
Love it!

--------------------------------------
John C Bland II
http://www.johncblandii.com
http://www.johnandseason.com

On May 8, 2012, at 12:31 PM, Jesse Wolgamott wrote:

>> +		move_a = @fighter_a.last_move
>> +		move_a_type = move_a.type.to_sym
>> +		move_b = @fighter_b.last_move
>> +		move_b_type = move_b.type.to_sym
>> +
>> +		if move_a_type == :strike and move_b_type == :strike
>> +			if move_a.ranking > move_b.ranking
>> +				@loser = @fighter_b
>> +				@winner = @fighter_a
>> +			else
>> +				@loser = @fighter_a
>> +				@winner = @fighter_b
>> +			end
>> +		elsif (move_a_type == :strike and move_b_type == :block) or (move_b_type == :strike and move_a_type == :block)
>> +			#there HAS to be a cleaner way here
>> +			@loser = @fighters.sample
> 
> Check it: http://rubyfiddle.com/riddles/98466
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/RubyoffRails/Episode2/pull/9/files#r789593
I will git on these changes.

Thanks
Josh Kurz
I think you wrote a little bit too fast here, it's "Har tagits..." not "Hare tagits...".
again, why store something that is used once
Can you add a check here to see if the instance is already a ScaledVector instances as well?  I remember we had some weird recursive scaling where a huge number of scaled vectors were nested together when they could have been replaced with a single one with an updated scalar.
We don't do any input validation in the models as of today, but might be good to do that.
I'm also worried about using the eviction manager for expiration handling. There should be separate components, but this is outside the scope of this.

I'd put a TODO to re-think this though.
Can you please use [dot syntax](https://github.com/github/objective-c-conventions#expressions) here (`spec.UTF8String`)?
This function needs a doc string -- take a look at contour to see how we share docstrings between module helper code and the axes wrapper funcs.
I would like to see a pixel shift test (e.g. the whole image shifted across by 1 pixel).

Similarly, it would be nice to test a sub-region with a pixel shift. (i.e. move the x axis across by 1 pixel).
I know it is not consistent throughout mpl, but we are trying to standardise on the ```"""``` style docstring. Would you mind updating?
```fh``` is the traditional "file" handle/pointer variable name.
Yes, but:

a) I don't know how to do that. It almost feels like this if _remove_method were an instance method, rather than a placeholder.
b) I did this to be consistent to `artists.py`, like you mentioned.

Thanks for the feedback!
why the '()' this is Python not C :)
согласен. погоды не делает. никто не решил но уже 3 высказались )
ok. I will include them today or tommorrow. Should I do a diffrent pull request?
Yes, and it is included in this pull request.
I don't this line should be removed. When calling `moveTo` we expect the `movestart`, `move`, `moveend` event sequence to occur.
We need a naming contest!
The sanitize method was designed to be a place holder for all such types of functionality.  Please move code into sanitize action.
@christianv : I checked again and getJqueryObject is not used anywhere else. It really is a redundant function that jQuery handles better itself.
Be explicit about the character type you want (use RegQueryValueExA or RegQueryValueExW)
Hmm I think not. We are not trying to get just the first child but all the leftest (visually) blocks.
I'm not sure where you got those numbers from, but that's not what I'm getting: I'm getting considerably less than that, and I have a low powered server running apache.

It's easier to debug with aliases, and it doesn't do any harm.
Is it possible to add a ``@param`` annotation for parameter ``$id`` which defines a more specific range of types; something like ``string|array``, or ``null|MyObject``?
added scaling variables
Thanks, I tried to come up with ways a city could be impressive, and being well planned into a giant piece of art was my favorite.
ctrl + shift + c in eclipse ;-) geht schneller
Does this remove a change? I thought it was for deleting files you dont want to be tracked anymore. Maybe I've been using it wrong all this time :(
Isn't it `sage --sh` to enter the Sage shell?
We can use the new JS interop for JSONP, actually. Much better! :)
Ah awesome!
Thank you ;)
I don't understand what you're looking for here.
Same in the save method.  ``return false``
saving CPU cycles is a sure thing, remember than only IE as `.currentStyle` so other browsers will go though the longer function-call-heavy path. Considering that `getStyle` is used a lot on animations, we should always have the fastest possible path here to avoid jerkiness.

That being said, **if** it's safe to do `style || getComputedStyle` on this part of the code then we could integrate this default behavior right inside getComputedStyle `if (this.style[property]) return this.style[property];` etc. as the rest of the uses of getComputedStyle are always as a default when the element.style cannot be retrieved (getOpacity, setFilter on #2411, etc)
Please describe in the function comment how the "event" parameter is used.  Which fields of it are read in this function.  It seems clear that the mouse position will not be used, for instance.
Тут ничего не поломается из за того, что я interval_t туда запихнул?
Add a space between String> and namespace
Ok. Is everything else checked and ok? Before I make thousand of "useless" commits... :)
I would use jQuery `.data()` to attach this to `.posts` DOM element?

You can create array easier with simply `[]`.
:-)

Good idea to sort by updated time, probably for our portal this is the best choice for now, when it will be small. Facebook does much more complex things here.
Aaa. Then you should make it a template:

```
BICIKELJ_INFO_URL = 'http://www.bicikelj.si/service/stationdetails/ljubljana/%d'
```

So it is clear that it is a template.
I did nothing, just an empty line like everywhere else. Will check what happened
There is one space after `"`. Be precise!
Then put empty action there. :-)
See how we are populating fields in `AuthoredResource`.
Why not a class attribute, like above? You do not need method for this?
On Thu, Mar 8, 2012 at 7:28 PM, Olivier Delalleau
<reply@reply.github.com>
wrote:
>> @@ -128,9 +128,15 @@ def filter_variable(self, other):
>>          if (other.type.dtype != self.dtype):
>>              raise TypeError('Incompatible dtype', (self.dtype,
>>                                                     other.type.dtype))
>> -        if (other.type.broadcastable != self.broadcastable):
>> +        if any(bi and not obi
>
> Sorry, I'm not qualified to review this PR, but please make sure you import any from theano.gof.python25 to remain python 2.4-compatible.
>

Good catch. I replaced any with numpy.any. Does that do the job?
I thought python:Depends already pulled setuptools, might be wrong?
Good riddance to bad (`preload`) rubbish. :D
The point of gettext_lazy is to trick our text extractors into thinking we're running gettext, but not actually run gettext until later.  We need the strings to be inside a _() call so that the translation utils pick them up.

One way to do it is this:
  -  import gettext_lazy as _
  - Change the constructor code so that it stores its argument in _text instead of text
  - Add the text property that returns unicode(self._text).  This is when the translation will actually happen
  - Keep the code that creates the buttons the same as before.
He meant that there's a  not needed }, and that he may have got confused by the if.
This wouldn't compile at all.
s/BarcodeScanActivity.this/this/
Is this appropriate to expose? I feel like this is an implementation detail and should be hidden from the public interface.
Okay, I will comment it, but it basically means "Only watch properties on *subclasses* of CMObject, and the aclIds property on CMObject itself".
Yeah otherwise that's going to break.

you need to pass along maxsize, which is the only Queue argument, 

http://docs.python.org/library/queue.html
ah yeah, nevermind
If "enclosed promise" is `deferred.promise`, just name it, this will be less ambiguous.
Just the `.replace(...)` calls ? Should the `(info.chara...`part have the same indentation as the `.replace(...)` or as the lines before them ?
I think these should be static members of the class.
Probably a good idea to "cache" a reference to `$(elem).css('transform')` so you don't have to call `$.fn.css` 2 or 3 times.
I think you can do this in one line `s= u' .join...`
You can test that with:

    reply=(${^path}/virtualenvwrapper.sh(@,.N))
    (( $#reply ))

But I don't understand, you say that there's no fixed location; maybe I extrapolate to much, but this means that users can install it manually, in which case the file could be not executable.
Otherwise, if there is indeed an installer, which probably (should at least) install the file in a specific emplacement, then it would be better to just check in this emplacement.
convergence of what?  should specify what the convergence criterion is, difference of params between iterations
Just looked this up.  You're passing it a block that handles initialization - that's actually super cool.  Let's do it.
As I understood from the [link](http://devblog.avdi.org/2010/08/02/using-and-and-or-in-ruby/) you posted, `||` and `&&` are for boolean expressions, and `and` and `or` are for chaining expressions together. Since this is an instance of the latter, wouldn't `or` be correct?
Sweet!

There probably should be but robot_monitor is currently way under-implemented, its missing a lot of features, that being one of them.
Ohhh, thanks. Fixed.
also, abbreviations are not welcome in Cocoa/Cappuccino, can you call this _arguments?
Since you're only ever using args[0] here, I don't see the point in using the *args version.  Also, I'd keep just the else branch.  I don't think we want to have the user passing in SB objects.  Just GSObjects.  Developers who need it can use the SBDeconvolve method directly as you did.
Is "12.0.0.1" a good default?
Why is this synchronized? The field is package-private rather than private, so I'm not convinced by this synchronization model: it's unsafe, even if not in the way current calls do stuff but in the way it's likely to be added to if we leave it as folk accessing the field directly.
Instead of javadoc saying "which can be placed on the classpath", is it more accurately something lke "annotation that can be placed on an Application (template), entity or policy to give metadata for when used in a catalog."
Perhaps checkNotNull would be better.
When called from user-code, it looks like this is the first null check so good to throw something better than an NPE later if not running with assertions enabled.
What about just doing a `@mkdir($installDir, 0777, true);` and if that fails then complain to the user?
`rectArrayForCharacterRange` is declared to take an `NSUInteger*` as its last parameter, not `uint*`. See the documentation of [NSLayoutManager](https://developer.apple.com/library/mac/#documentation/Cocoa/Reference/ApplicationKit/Classes/NSLayoutManager_Class/Reference/Reference.html).
I would replace 'remote control api' with 'Selenium RC API', and capitalize 'Selenium'.
Rather than doing this I would use the predicate system of Pyramid.
`:require => false` is specific to the use of `Bundler.require`, which we don't do
`Kernel.is_windows` is not necessary i would only use : 
`main_config.vm.share_folder "v-root", "/vagrant", ".", :nfs => (RUBY_PLATFORM =~ /linux/ or RUBY_PLATFORM =~ /darwin/)`
The compiler of today itself existed even way before this. Before that, there was even another implementation which was completely thrown away as it was more or less "the first attempt". Yet, I don't now the exact year, but it should be at least 2010.
A bit of background: there is a security option in Apache that makes it refuse encoded slashes in URLs. rhelmer and I tried to understand why it is there and couldn't find much. We decided that there must be a good reason for this to be activated by default, and chose to double-encode slashes in fields that we know can contain some. That only means signatures, as I don't think any other field needs a slash. If we end up with a similar problem for an other field, we'll either consider turning that Apache option off, or adding a special case for that field. But we're not dealing with it until it happens, basically.
it should be ``->prototype('scalar')->end()``. Your current tree is broken (you should test your changes)
i already changed it, you don't see the change on my screen are changes visible ;)
Please add the namespace "t:Importance"
А насчет трая, ну да, примерно так и вышло, просто надо в разные переменные парсить
I removed the wrong line. `nodeselected` and `pseudoclasstoggled` are not emitted anymore.
I would suggest changing this assert to:

Assert.true(search_page.is_the_current_page)
Use libra_server as a properly formatted URL for this segment (be sure to default to the correct server too)
Bike shed alert: can we make this variable name a bit less odd? Perhaps `REGISTRATION_INVALID_DOMAIN_MESSAGE` or whatever?
на что указывает этот `_this`?
Ok, shoud be ``first``. Not seeing this on the first look probably means I should not do much reviewing today ^^
If the heading is ``Dummy estimators`` I would probably rephrase the sentence to reflect that, i.e. "When doing supervised learning ... one's estimator ..." and then say "three such strategies for classification."
Maybe write out NLP?
hm ok never mind, I can still use it directly...
подивись який має бути порядок аргументів
http://www.phpunit.de/manual/3.6/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.assertions.assertEquals

і така перевірка як на мене трохи сумнівна, бо при зміні кількості фікстур будуть падати тести
What is the purpose of this loop? And what is the semantic value of "3"?

Is it to ensure that it is possible to connect to the ZK cluster we create?

And is the semantic value NUMBER_OF_ATTEMPTS_TO_CREATE_CONNECTABLE_CLUSTER?
it seems the indentation is wrong
Ok here's a preview:

```d
if (tok2 == TOKtemplate)
{
    Type *t;
    Expression *e; 
    Dsymbol *s;
    targ->resolve(loc, sc, &e, &t, &s);
    s = s ? s : t ? getDsymbol(t) : getDsymbol(e);
    if (!s || (!s->isTemplateDeclaration() && !s->toAlias()->isTemplateDeclaration()))
        goto Lno;
    
    tded = targ;
    goto Lyes;
}
else
{
    Type *t = targ->trySemantic(loc, sc);
    if (!t)
        goto Lno;                       // errors, so condition is false

    targ = t;
}
```
Ah, I forgot to fix this.
And this declaration should be moved down to where it's initialized.
This method may work perfectly well as-is, but I'm a bit surprised by that.  To me this method should return a ModelAndView, since it loads stuff into a Model object.
We need to update only properties marked to update. Otherwise it seems useless calls of not-updated properties.
The default mode should be set when the picker object is created.  The default exists so the user doesn't have to set the type explicitly, we can't rely on the user setting the type explicitly.  

The other types of picker will need to be supported.  If not in this patch, then  a TODO comment should appear around here and an error message should be logged or an exception raised.  Then a new Jira issue needs to be created for the work left to do.
Should one of these be m_width?
Are you sure that we need a wrapped hydrator? After all, `Doctrine\Common\Persistence\Mapping\ClassMetadata` should allow you to access the fields directly (also for writes).
Also here, could you do a quick check and see if `ObjectManager` and `ClassMetadataFactory` interfaces could be used?
should be 2012
ignore removal changes, they will be discarded
This isn't right...

It's re-reading from offset 0 every chunk
I was in two minds whether to or not. Other calls to wu->setTimerInfo generally don't (but they are all supplying count=1, so max is a bit pointless).

I could I suppose pass max only where count was <> 1
Instead of turning it into an if statement, just assign it. I.e.

```javascript
showAdvFilter = queryParams.indexOf("advanced=1") > -1;
```
The color `#999` is repeated 4 times.
Ups. Good catch. Moved the code around and forgot the print line.
This seems overly complicated. It'd be justified with three paths and additional paths in the future being likely. With only one legacy location, I'd go for:

```c
if (fopen ("new path") == NULL) {
  if (fopen ("legacy path") == NULL) {
    ERROR();
    return;
  }
}
```
I would like to do this in a different ticket since this issue has already gotten quite large
this is the same in index.html.  Just copied this over. For now, I don't really care about this, until the use case for international dates are given.  But it's not that hard.
The class instance variable dance fun for ruby to avoid class variable scope leak.
} else {
    qWarning("Cannot generate video thumbnail, thumbnailer func not available");
}

maybe?
Server 2008 falls to the same issue as windows 8, the key is being removed after activation. Server 2000 and 2003 though I totally forgot about and don't have machines available off hand to test with. Let me see if I can fix this so I can test them as well.
You're right, moved to a method.
You can ignore this one =)
Gettext yo?
Updated version. :D
yea i struggled with the name for this one too. I guess we just go with blend color and document it well
Almost always do not use abbreviations in public interfaces.  Rename this to `getNumberOfImages`.
"gui" is a big word. Menus, canvas are all part of GUI. Naming a child of menu div as gui wouldn't be the best choice.
Are you sure credit cards always passed in as integers?
It was just because at first glance I didn't understand the method. It was more clear after running the unit test cases
Holding off on any changes regarding cross platform pathing support till the new PR revolving around cross platform support.
You can use wrench-js to remove directories (https://github.com/ryanmcgrath/wrench-js). Chris and I are both using it in the CLI and Code Processor, respectively, and it works very well.
property can not be disabled. Should read when `wrap` property is set to false.
You don't have to do the assignment at the top, just the declaration, if you feel so inclined, but that is what the style guide says. Putting declarations at the top is done to avoid getting into trouble with JavaScript's screwy scoping rules. Sure this is a trivial case, but it's still better to be consistent about it to prevent getting lazy and then getting in trouble. Fail early doesn't take priority.
`if (typeof start === "Date")` WAT? :) https://speakerdeck.com/u/azproduction/p/2-liektsiia

В программировании есть хороший принцип лучше шире, чем глубже - те нужно избавляться от вложенных стейтментов и стараться спрямлять их.

```javascript
if (typeof start !== "number") {
    throw new TypeError("error of start time format");
}

if (typeof end !== "number") {
     // ....
}
...
```
Немного gramar nazi - people множественное число и без s (peoples - народы)
`@param {Object}  eventOptions`
Even private methods should not start with an uppercase character.
Can you put a blank line between each block?
Good, God. I *hate* parens. I don't know how that happened. I will most certainly rectify this.
This diff block and the preceding block seem to already be merged into the module.
No uncommented code, please.
Ah, I believe I missed this in converting to using AUTODISCOVER_PATH constant. An easy fix.
installed extensions
This test is failing when I running the whole test suite. Any ideas? I didn't dig deeper.
Fair enough. Feel free to make the change.
you forgot to add the `,` here, so this should actually fail.
try adding it at the end of each line, even if it is the last line anyway.
Please make a pull request for the easyconfig file in https://github.com/hpcugent/easybuild-easyconfigs
This file has been reviewed...
Why is there a template file called: `3` in `lib/PSU/Person`? :D
fron > from :)
This should call `parent::__construct`, rather than `parent::CI_DB_driver` and the `EXT` constant has been removed.
This will be fixed to have the correct mode on removal in the next push.
This seems very bad. It's very view-dependent. What if we decide to add a class name to a `tr` or `td`? Then this has to change. Use something that looks at content instead.
this comment isn't necessary.
`message = message || "";` ;)
Ich denke, hier sollte auch gleich $salary gesetzt werden.
Do we add a dependency on sass ~> 3.2 as well? sass-rails only adds a dependency for >= 3.1.10, and compass only specifies ~> 3.1, which won't force users to use 3.2.
Again, no point in calling experimental
I'll try to be a little more bold in the future... :)  yeah, this is better.
weird `§` in front of name here.
I've followed the concept of the former and other includes to include the beginning of the page name.
current secondary profile is whatever is set in the url
please fix ALL of the typos... several "Transactioin" classes/configs still
Could just be Git visualizing, but shouldn't this sub-function be indented by 2 spaces?
Ah ok, does this fix all of the outstanding issues with using Ion Auth and Oracle or was this just the last one on your list?
So this caused a circular dependency. I'm not sure if removing it is the correct choice.
This is still incomplete - we need a way to incorporate the different routing of the objects, possibly determined by the configurations.
Actually, reads fine as-is; adding "need to use" actually breaks the sentence grammatically.
rewrite to extract the _id field without adding it as an indexed attribute
This must be an error. The messageData we are parsing must be complete enough to reach here. If it's not complete, that means we are saving incorrectly some data on the DDBB or we are acepting something somewhere with an invalid (or not complete) format.
Same as above. About to remove.
Drop the logout
Tiago,

To keep things in line with how CorrugatedIron handles optional parameters in other areas of the application, woud it be possible to use a nullable type here? (ie. `public int? Timeout { get; set; }`). Nullable values are a bit more self-documenting and remove the need for magic numbers (which in this case would be zero).
use Json::FastWriter().write instead of creating an object for one time use
nevermind...this doesn't make sense
show should be changed to edit
Should it be `train_size` instead ?
Some syntax errors here?
trailing whitespace on above two lines as well as L37,54,70,92,98, 100, 106, 118, 127, 134 - I think this is leading to issues merging (unable to auto merge at current time)
In case it's not clear, the actual implementation [is in grisaille](https://github.com/contraflows/grisaille/blob/master/lib/grisaille/faker/image.rb#L6) rather than in this repo
next_ тогда должен называться rightmostChild
As previously mentioned, make this a property.
Upcoming information about the iPad 3 indicates it will have a retina display, so get ready for that. We may want to include a TODO note or other marker here to make sure we update this when the specs are revealed.
Maybe for consistency use either version-less schemas or versioned ones.
Nutno doplnit vsechna pole, jinak TEXT si nebere default hodnotu pri vytvareni tabulky
And also not here, still a large group has the LIST permission
`phrasesFile` seems to imply that a `File` object is being passed in, not a string representing a filename. Consider renaming?