labofsahil/github-event-dataset-2012 huggingface dataset:
capacityOfDNA и gene хранят одни и те же значения во всех созданных вами клетках. Есть смысл сделать и то и другое константами и не выделять под них память при создании каждой клетки.
извиняюсь, забыл о способе создания категорий через новый файл...
И я не вижу проверки на случай, если методу вздумается скормить значение свыше 100
Сравнение проходит корректно добавил NSLog для проверки.
Ну и почему не правильно? :))
Можно обойтись и одним циклом - перемешать массив на n-процентов и одновременно с перемешиванием заменить значение элементов.
Зачем вам idx как переменная класса? Что в ней будет? Для чего она?
$this->_request->getPost() já não retorna um array igual ao que você está reescrevendo em $data?
Because there's (almost) no chance to find it in a commit message.
we're returning JSON not text/plain. Also this should not be here, it should be written when the result is ready.
If it were to catch that error, what should it do then?
After doing some testing, i realized this is not the problem, it is because the user is uploading a text file yet the handler is for images. See the comment i left on the ticket, we just need to display to the user an informative error message that they can only upload images in case of an ImageHandler
Why not use the current application charset? (by taking an additional implicit `Codec` parameter)
@shurizzle Is it worth it making this plugin specifically and only for rvm? I don't think it's a dealbreaker for the theme (but I'm not the one making the decision), but I really wonder what's the reasoning behind this.
>As an example, suppose you have...
Comma unnecessary here (no second element inside square brackets)
What it checks here is if the drug is not null, check whether it has a valid drug.concept. Else (which means drug is null) check whether the drugOrder concept is null or not, as both drug and order concept can't be null. Isn't it?
Yes, that's all that line was. My mistake.
Why braces here?
Пусть лучше имя таблицы вместе с открывающей скобкой всегда будет на строчке с `CREATE`.
Si el applicant no está guardado en la base de datos no se le pueden adjuntar archivos.
this is simply wrong. The service is private so nothing ensures you can retrieve it at runtime (if only 1 service uses it as a dependency, it will be inlined and removed when optimizing the controller)
Take a look at using [`to_s`](http://api.rubyonrails.org/classes/ActiveSupport/TimeWithZone.html#method-i-to_s) with a custom date format instead of specifying the formatting string here in the view.  I just did something similar on Time Life to specify an 'xml' format for our API payload:

    # config/initializers/date_format.rb
    [Date, Time, DateTime].each {|klass| klass::DATE_FORMATS.merge!(:xml => '%Y-%m-%dT%H:%M:%S.%L') }
Shuldn't Exporter also have its own namespace? The importer has its own class.

I was thinking shared code would be in STL, then importer and exporter specific would be on their own.
please keep this line
I think you should assign the result of whereGeocodeIn(geocodes) to ```where```
I recommend to reimplement "clear" as a separate action that actually clears the part of the session responsible for the issue list.
This way we
- have one single point holding the information how to sort
- use the "original" index action to restore the index action :)
.. or simply we do what the button says.
Well, you are either passing in the string "0" or an integer otherwise. It's sloppy, it should always be an integer for consistency.
We cant do this anymore. it wont run in dev if we make updates to old versions. plz add a new version and do updates to the existing table.
`status.status` what's the emoji for Fry squinting
In this case you return the coll itself. But you don't really want to check this at all, as `count` is an expensive operation.
javadoc for return is not correct :-)
what equals/hashCode generator are you using? Looks a bit different than what I've seen.
I tend to agree with @WeatherGod here: let's leave this assuming base 10 for now.
this test is failing for me.
judging from the fact that the captured screenshot says "can't find server at crash-stats.allizom.orginvalidpath, i think the "/" needs to go back at the beginning of the path.
If we're to assert that the lock/unlock is successful, it should be in the LockScreen class and not in every test, as the assertion is not related to the test. It would also allow us to have an appropriate failure message. I can follow up with a pull to implement this change.
sur une leul ligne is just too long and doesn't sound as good.
ok, I would just delete the one in main.qml and leave the others (once per component, for the reasons I pointed out)
don't just comment out code, delete it.
listelements and elmeents does not make things easier to udnerstand
Why calculate dx and dy on preserveCenter cannot just be?:
```
    if (this.preserveCenter) {
        this..applyTransform("scale(" + scale + ")");
        this.currentCenter = this.map.getPixelFromLonLat(this.map.getCenter())
    }
    else {
        var current = this.map.getPixelFromLonLat(this.map.getCenter()) : evt.xy;

        var dx = Math.round((current.x - pinchOrigin.x) + (scale - 1) * (containerOrigin.x - pinchOrigin.x));
        var dy = Math.round((current.y - pinchOrigin.y) + (scale - 1) * (containerOrigin.y - pinchOrigin.y));

        this.applyTransform(
            "translate(" + dx + "px, " + dy + "px) scale(" + scale + ")"
        );

        this.currentCenter = current;
    }
```
'used' :-)
If you override call you'll lose some of the functionality Goliath adds. Setting the env into Thread.current, handling streaming responses, calling the async.callback to go back up the middleware chain correctly.

Why did you use call instead of response?
in which case is this useful?
I don't think this is correct. If I'm looking at it right, it would default enabled to FALSE if ASSET_SYNC_ENABLED does not exist
I don't think I can change this.
'user' or 'group' objects will have a 'picture' attribute, this will either be their profile picture, or the default image.
If the picture is the default user/group image, it's better to replace them with the smaller version rather then scaling down the larger version so they look better.
Space missing here.
Why is the questionSetId grouped with the if-else logic for selected/type?
Seems to work now, so will apply the change then.
I don't think it's necessary for correctness, but I'd be happier if values was explicitly initialized.
This part might need some additional testing
- Are you sure ``$length`` has the correct type as "double" is not a sub-type of "integer|null"?
- Are you sure ``array_reverse($dst)`` has the correct type as "array" is not a sub-type of "string"?
- Are you sure you can call ``implode()`` with the given parameters? It seems like at least the 2. parameter (``$pieces``) is missing, no?
if (File.Exists("ygopro_vs.exe") && !File.Exists("devpro.dll")) File.Copy("ygopro_vs.exe", "devpro.dll");
            File.Delete("ygopro_vs.exe");
Cool, I'll update that bit tomorrow.
This one can be shortened to `(__import__('sys')._getframe() if Frame.NATIVE else Frame._getframe()).f_back`, can save some columns.
Changed, I'll commit.
The overridden constructor instantiates a JHttpTransportStream instead of a JHttpTransportSocket
@videlalvaro it is very important comment from @stof I just put the wrong interface here ...
I will only go with the keyword replace (way more readable, if not also faster, nice catch there).
We can't integrate the `0px 0px` default inside the if block because it will not return the default on browsers that lack the X/Y extensions (Opera in this case). **unless** we want to remove this default and handle such cases where needed (Fx module)
Good catch, fixing now.
Ah yes. Updated.
`sort_links` isn't being provided.  I'm not sure whether that context variable isn't needed with the new templates, or whether it needs to be included here.
I don't think context is an issue here.
If you have more than one attribute to set, use version when you pass a dict of values to the `.attr`, not that you call it multiple times.
*povezava*? E-mail?
Use class-based views. Not function views.
(I am not sure if this works, though.)
`for name, field in self.fields.items():` does not work? Then you would not have to access dict again bellow. And it might be more readable code.
Hm. Do we really need data to access it through data? Each form element has its own unique form input name, at least (not to mention ID). Why not using that?
This seems misleading, you use COMM in both cases.
The return type is guaranteed by the method signature, so that isn't necessary, I think...
I got it, that makes sense. Just looks a little weird. Maybe we should add a comment explaining this?
Intended one ?
aura spell script?!?!?
Is there a point to this rather than just terminating the application (which will also dump the exception)?
The `OpenAppCommand` is in line with the existing solution (where command lines parameters are based on the class names). We might want to change that solution, but I think that'd be a separate pull request
Should this be `[permissions copy]` like all the other methods?
100% arbitrary unfortunately. My guess is that Francois was lazy and didn't want to do separate query without time constraints...
the contract is that pigValue is expected to correspond to Thrift type. E.g. if Thrift field is a String and pigValue is Integer, it will throw an exception. 

I am not sure of the right phrasing for it:  second line could be: Pig type for pigValue should match corresponding thrift type.
The only styling nit I'd make here is that a few newlines before each comment block would make this much more readable
This change makes all the unit tests look awful. The correlation id is something that we should solve at the infrastructure level, and avoid polluting all the tests for the domain logic with it.
鬼畜ループ唯一のbreakがここ。
カスにならないとループから出れない。

一度カスになったやつは一生カスなのである。
最初からyかnで答えていえればよかったものを...
Not sure if i fully understand, but it looks like you could get that easier with some like `this._on_abouttoloaddocument = function listener (msg, is_playing_back)` then `listener ` will only be visible in that scope and refer to the function itself.
window_id should be a of type number (it will work with a string too though).
I think these two are flipped?
This seems to be a debugging stuff not sure how to keep it away of the production code. Any idea?
(not sure if we need to change this)

I would prefer the reversed order of the arguments (x, Xi, h) 
I like h last (in case we get default arguments and h as keyword)
order x, Xi   mainly to read it as function of x given Xi and h (not sure about this since standard Kernel notation is K(Xi, x) ? )

with ``x`` and ``Xi`` it's not obvious to me from the names which is which (I often use xi meaning x_i subscript, the ith observation. What does ``i`` stand for in the training set.)
maybe ``data`` instead of ``Xi``, would be more informative

(We need kernel functions for other parts, but I don't know or recall the details.)
This should be wrapped in a try/except block since matplotlib is a soft dependency. You can also delay this import to the method that uses it.
*case-insensitive*. not *case insensitive* (because you have like that bellow - just be consistent).
You don't have to use inline styles any more dude. You can use Cascading Style Sheets… not sure if you've seen this before? http://www.csszengarden.com/ it's crazy what you can do now.
Shouldn't this just do a getattr on the value in the query properties dict?
This probably needs a check on kind put in so I can't just grab anything I want.
sorry, i'm going to make a stink about this one.
please catch only certain types of exceptions, instead of all exceptions.  You should only catch the exceptions you expect, and can handle, meaning you should be minimizing the set rather than maximizing.

I am going to gate the pull request on this fix.
strange indentation (is it tabs and spaces?)
Should this set `log4j.logger.brooklyn.util.internal.ssh=DEBUG`, like many of the other log4j.properties? I think it should.
switched to an if null check -- don't want to use checkNotNull because that would force the string concatenation
Riq, there are some APIs different between Cocos2d-iPhone JSB and chipmunkJS. 
What do you think that we use the chipmunkJS one?

'''js
// In JSB, it uses setXxxx(yyy);
motor.setMaxForce(  0.0 );
 
// In Chipmunk JS	, it uses this.xxx = yyy; 	
motor.maxForce = (  0.0 );
'''
true. forgot this can ofcourse be literal 'true' when it comes as part of the URI
absolutely not.
A testcase should test one unit and not depend on random other things.

The fact this test is failing because of totally unrelated thing is very annoying as it will be masked when it actually fails because of an 343 issue but at that point its discarded being related to 585.
Also inb4 @fwenzel "That's so Java!"
tests is listed as an app in `tests/settings.py`, so it will still import this file and look for models. This way, it will not bother creating an extra DB table for versions of Django that the tests aren't running on anyway.
These seem like `private` methods to me
It might, I look into other refinery plugins to add translations and I found this way to specify the reference [here](https://github.com/resolve/refinerycms-news/blob/master/Gemfile)
Yes, that's the only way it'll work in memcached. The keys with the old version number namespace will never be touched again when it is increased and then memcached will either expire them or clear them lazily when the memory is needed for newer items.
Space is also needed before and after the "=" sign. Please follow Cake's [Coding Standards](http://book.cakephp.org/2.0/en/contributing/cakephp-coding-conventions.html).
ahh.. sure

make a inner static class

private static enum AddressToString implements Function<Address, String> {
   INSTANCE;
   public String apply (Address in) {
       return in.getAddr();
   }
}

then, transform the address collection prior to putAll

addressStrings = Iterables.transform(addresses, AddressToString.INSTANCE);
I know this isn't from you, but I believe this should use lower camel case (`public $firstName;`).
This should be labcoat_cmoalt.
I'm not sure, but I didn't change it from the above.
I commented it because it's broken. I don't actually know if the line is relevant to what the original coders wanted to do, so the question is whether to fix it or just remove it.
using references means this:

```
    Crazy linking action
    --------------------

    I get 10 times more traffic from [Google] [1] than from
    [Yahoo] [2] or [MSN] [3].

      [1]: http://google.com/        "Google"
      [2]: http://search.yahoo.com/  "Yahoo Search"
      [3]: http://search.msn.com/    "MSN Search"
```

instead of like this

```
    Crazy linking action
    --------------------

    I get 10 times more traffic from [Google] (http://google.com/) than from
    [Yahoo] (http://search.yahoo.com/) or [MSN] (http://search.msn.com/ ).

```

Example taken from [The GFM docs] (http://github.github.com/github-flavored-markdown/sample_content.html)
В интерактивном режиме не надо выходить при исключениях.
While it was a bit tedious to set this up, it makes it possible for us to both run the game as an executable script, and also control it grammatically from `Blind::UI::Simulator` testing tool
I think normally we don't cover this step in the READMEs
Is it the app that's purchasing, or are we purchasing an app?  If the latter, I'd suggest is_purchasing_app
ah, maybe
We're forcing anyone that uses thincloud-test to use our setup here?
Ow :shit:, I forgot we support 1.8 here :smile:.

Anyway, what should this `:class` option support? `nil`, `[]`, and maybe `'foobar'`? Those should be good I think?!?

```ruby
>> Array(nil)
=> []
>> Array([])
=> []
>> Array('foo bar')
=> ["foo bar"]
```

Well, in any case @stephenprater, just ignore the `Array.wrap` comments, and let them out there, we probably have it in other places too :)
Maybe "Return scaled image"?

I'm slightly in favor of `rescale` as a function name as opposed to `scale`. Also, I think the parameter and function should have different names. Maybe the parater could be `rescale_factor`.

Also, you might want to add a one-sentence explanation to differentiate this from `resize`, since the names suggest similar operations. (Yes, you can just look at the input parameter to tell the difference, but it helps to a quick comment sometimes.)
This line is completely wrong to begin with...
Could you please check the `dtype` of `K` here?
No, abc and warnings: standard library imports.
BTW, when using the knn graph, should we add some dummy points to make the graph to be connected? RBF may not suffer from this problem.
please remove this comment - might be more useful to refactor all of the above into a single function e.g. nearbyViewInit
> I added you (i think) to have push/pull access to this repo. As far as i know, this means that all your work, as of now, still remains in  your branch until you push the big green button at the bottom of the screen which will automatically merge all the commits into the Khan/kiln-review branch.
>
> Let me know if that's not working, you should be able to merge these changes in whenever without waiting for me, but it's fun to talk about code anyway.

Thanks!  This was one of the things I was trying to figure out how to
mention in the onboarding docs.  (I was wondering why I wasn't showing
up as a member of the khan group on the github khan homepage.)  Is the
best way to do this in the future just to ask someone to add in, just
to ask an existing member?

In the meantime, I'm happy for you to finalize the review in the
branch before pushing to the main branch.  It doesn't matter in this
case, since nobody is using the code in the main branch yet, but it's
closer to pre-commit review which is the method I prefer.  So I'll try
it for a while and see how it goes. :-)

craig
Delta doesn't seem to be used
Good catch. Note that this is only necessary for ruby <1.9
JUnit has a nice

    Assert.assertEquals( message, expected, actual, delta )

especially for _double_s
I was being lazy and didn't change
Should the connectWatcher calls still happen, even if daliHelper != this?  How about disconnect()?
Not sure this is necessary... Simply replacing it by an ```assert``` would make more sense...
RDDE ?
Is comment valid?
Isn't it only true if localtrans created here, i.e. not if transaction passed in and already started.
missing waitToConnect &&
It's only using this value to clear ALL events on the queue, I think. Unfortunately I don't think there is an easy way to know how many there are.
The first reply from the SO thread you linked states: 

>However, some languages (notably Python) use exceptions as flow-control constructs.

Also in the Python glossary](http://docs.python.org/2/glossary.html) under EAFP:

>Easier to ask for forgiveness than permission. This common Python coding style assumes the existence of valid keys or attributes and catches exceptions if the assumption proves false. This clean and fast style is characterized by the presence of many try and except statements. The technique contrasts with the LBYL style common to many other languages such as C.

So I do think it would be quite idiomatic Python. Interesting that you state the Exception is more expensive. What do you expect the % of KeyErrors out of all loops to be? I don't have enough knowledge on Python internals to say which is faster (should probably test it) though from gut feel I'd expect the version that doesn't look up the value twice to be faster.
Don't think you need this. Try without.
unless optionAnnotation.blank?  :)
Tää koodi tosiaan ylikirjoittaa noi Games kansion Start.cb Stop.cb ja Include.cb tiedostot jotka sit includetetaan pääohjelmaan.
Hakee kaikki pelit joilla on gameinfo ja ainoastaan Main.cb includetetaan.
Ah, nice research. Let's leave it at 1024 then. Thanks!
No. The logic will not work with an  AND - not for this one.
Yes. It will fail this, I would like to use this code from Rafael - verificationErrors.append(e.toString()); within my catch. I would refactor this once that is in a shared place -
see previous line note
No need for the null checking, nthChunk always give a valid chunk object.
I don't think this belongs as a seperate FilterType: what if you want to search for favorites with a given text?
Artificial restriction.  This is a fairly simple task that should work just fine on java or a shell session.
I just verified you can complete strip run_cmd() and just use cmd_exec().  Here's how to do that:

1. Remove run_cmd()

2. Add the mixin:

	require 'msf/core/post/common'
	...
	include Msf::Post::Common

3. At line 64, replace it with the following:

	res = cmd_exec(cmd)
I have addressed this in latest commit
"pyxmorph" -> "cmorph"

Search and replace bug ("*.c" -> "*.pyx")? ;)
I find the empty line in these kind of docstrings really weird to read. So I care... but if it's really just me it's not a hill I'll die on.
I think this should become `loadJson` to match all the other loadBlah functions.  Doc and API should change to mirror `loadText`, `headers` should be an optional parameter here just like it is for `loadText`, except `loadJson` should always add the Accept: application/json header (which should be noted in the doc).
Yes, that's the theory.  I thought I saw an improvement when I did it originally, but when I swapped the sort order just now it made little to no difference for horizon views.
As I said already, I wouldn't bother with constants for `agi_defaultImage` and `agi_defaultCubeMap`.  In Cesium, we can use full JavaScript, so using `source` the way we do is OK, as is allocating `Color`, etc.  I suppose the Sandbox examples can follow the same rules since they are also JavaScript code.

Only when Fabric is being authored in pure JSON, do we need to use pure JSON.  Try it - I think it will be cleaner.
Is the default texture really something that should be relied on?  I assume it was more than anything else a fallback in case the referenced texture could not be loaded.  I don't really care, so I'm fine with whatever.
This is cloud based operation so it's moved from `CmdletBase` to `CloudBaseCmdlet`
*decremented
It may not throw an exception (as far as I know, it only throws an exception on iOS) but it also doesn't seem to update the table, which is the important point. Here's my test code:

```

Ti.UI.backgroundColor = 'white';
var win = Ti.UI.createWindow();

var sectionFruit = Ti.UI.createTableViewSection({ headerTitle: 'Fruit' });
sectionFruit.add(Ti.UI.createTableViewRow({ title: 'Apples' }));
sectionFruit.add(Ti.UI.createTableViewRow({ title: 'Bananas' }));

var carrotRow = Ti.UI.createTableViewRow({ title: 'Carrots' });
var potatoRow = Ti.UI.createTableViewRow({ title: 'Potatoes' });
var sectionVeg = Ti.UI.createTableViewSection({ headerTitle: 'Vegetables', footerTitle: 'Thats all' });


sectionVeg.add(carrotRow);
sectionVeg.add(potatoRow);


var table = Ti.UI.createTableView({
  data: [sectionFruit, sectionVeg]
});

win.add(table);
win.open();

var sectionFish = Ti.UI.createTableViewSection({ headerTitle: 'Fish' });
sectionFish.add(Ti.UI.createTableViewRow({ title: 'Cod' }));
sectionFish.add(Ti.UI.createTableViewRow({ title: 'Haddock' }));

table.setData([ sectionFish, sectionFruit, sectionVeg ]);
sectionVeg.remove(carrotRow);
var cardoonRow = Ti.UI.createTableViewRow({ title: 'Cardoons' });
sectionVeg.add(cardoonRow);
```
We should be able to use charCodeAt(). See https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/String/charCodeAt
Поехал код - проверь на табы и пробелы.
Попробуй уменьшить число аргументов

```javascript
var data = {
    start: new Date(),
    end: new Date
};

new Event(data);
```
Lol, this is one of those tiny, almost inconsequential core changes that I hope sparks a loud and passionate discussion. What was the purpose/reasoning of this change?
Let's stay with `.bind( "click"`. Event shortcuts are becoming optional in modular builds and we should promote the most compatible code.
Actions:

Prototype one env per process (including dynamic envs) - Jeff (On Trello)
Put together one test that uses puppet in a real manner - Andy (On Trello)
Identify 2 things that we want to isolate - Patrick (Not yet on Trello)

Patrick, for your action, can you update the hit list with what should
be isolated and a bit of a description about how you see it working.
One you have that just send out an email to platform@puppetlabs.

On Mon, Oct 1, 2012 at 8:47 PM, Jeff McCune <jeff@puppetlabs.com> wrote:
> rca
![overconfident-alcoholic](https://f.cloud.github.com/assets/369495/8997/12644f3a-4491-11e2-9bf8-df78b797050e.png)
Я имею ввиду папку шаблонов в i18n, а не в мастере, к сожалению. :frowning:
I'd rather take `sender` as per the Django documentation: https://docs.djangoproject.com/en/dev/topics/signals/#receiver-functions
Some weird space character.  I killed it.
... justification
remove ", true"
please try to use hamcrest instead of plain junit here (assertThat(xxx, is(...));)
@jburel: More commented out lines.
@gaker @pkriete am i missed something here? I check `PHP_CodeCoverage` package within my php directory, and i couldn't find `getInstance` method there, and like [this build](http://travis-ci.org/#!/toopay/CodeIgniter/builds/692751)  i get Undefined method before i commenting above line(s).

Also, [is this result](http://travis-ci.org/#!/toopay/CodeIgniter/builds/692816) was like you guys expected?

cc : @philsturgeon @derekjones
Please uncomment now that it is translated
from design perspective, this method may be used in any inherited function from AtomBase. For example, a customer created TheirEntry which inherits from our Entry class. In TheirEntry class, they can re-use writeOptionalAttribute to write an optional attribute value which would require validation.
Migrations that have already been executed should not be changed, as this breaks the migration and forces everyone to drop their existing database and rerun `rake db:migrate` from scratch.
Conversions don't need derivatives.  There are no derivatives mentioned here.  What are you getting at?

There is not more to this patch.  convert_value is provided as a utility function for implementations of RendererServices (i.e. renderers that embed OSL), by providing the conversion logic in one place and specifically matching that described in the OSL spec.
this does not support multiple entity managers. You should inject the registry instead.

and we should find a way to register it only when the ORM is configured, to avoid breaking other cases (and we should register the mongoDB, CouchDB and PHPCR ones)
`auto` сам б-г велел использовать перед `channel`. Но это на будущее.
sél*e*ction! :)
There is a tab here
These are click triggers, not bindings, so no need to do anything with `.on()`
Insert one space after `if` to fit with others.
Last time we used a delegate, it somehow resulted in a GC allocation if memory serves. May want to watch out for that.
So are the values here for x86, or is it just that MIPS is different from everything else?
This is odd.
It means that this mechanism is activated by importing the module.
Instead it should be a function like ```installHandler```.
Those match X86 and asm-generic, i.e. these are the defaults unless they are overridden for an architecture.
Back in the day, the gc, rt, and core portions of Druntime were complied into separate libraries and the only interface between them was extern (C) functions.  Some of the motivation for this was to avoid any dependency on a particular implementation because of inlining, etc.  Since then I've decided that it's okay for anything in the runtime to import stuff from the core package (they already were anyway, from core.sys and core.stdc), so there's less reason for the thread_blah() routines than there used to be.  About the only reason remaining is so C code can more easily integrate with the runtime, which is the reason for thread_attachThis(), etc.  Beyond that, I think these could probably be converted to extern (D) functions instead.
This is moved - not removed
I did look at the spec and all I am saying is that if we do that we would need to test every single one of them. Is that the objective here?
In the spec (while i scanned it briefly) it mentioned that characters allowed in property naming must  follow selector expression rules etc.
So i already removed the pattern and it simply is a String (as its always been).
We can raise a different JIRA later if we thing such validation is required.
TBD is OK for Milestones; the motivation for this JIRA was to ensure the doc matches the code, not that it is complete.
Looks like you reverted back to injecting a reference (value injection is commented out). See comment below about runtime resolution.
Does `.plot` accept anything jQuery selectors do? Then you could instead store above `<div>` you generated into a variable and pass it here. so that you do not access it by ID, but by simply a variable.
@Maks3w deprecated doesn't mean gone, though, so the note is still good to have for 5.3 users who may have that setting still enabled.
El número de `#` debe coincidir con la longitud del título
Why is this a singleton?
it was there because ``this.routes_.get(name)`` could not be inferred fully. But ``this.getRoute(name)`` is already documented as returning a ``fos.Router.Route``. So ``var route = this.getRoute(name);`` is enough
Bad indentation only 3 spaces should be 4
In one paragraph, you're not worried about things we don't use in Medley (even if they are universally good skills to have), but in the next paragraph you're worried about websockets, which can't even be used at all, if we're to provide a cross-browser compatible website - http://en.wikipedia.org/wiki/WebSocket#Browser_support
local path is not mandatory, for example, on remote files. This is the second (a bit desperate) attempt to get it, so it doesn't matter much if it also fails.
Do we want this default to change?
So I have checked the whole process and found a remaining issue with cleaning up the target folder from compiled scripts. With this change we do no longer remove those files. It results in:

-rw-r--r--  1 henrik  staff  12354881 Feb 15 15:18 1.5.9-win.zip
-rw-r--r--  1 henrik  staff  14285529 Mar   1 13:59 1.5.9-win-new.zip

So please fix the remove_files call.
That seems to handle two cases - read mode and anything other than read mode.

I was looking at this on iphone https://github.com/appcelerator/titanium_mobile/blob/master/iphone/Classes/FilesystemModule.m#L76
the definition should be set in private and the service accessible through a dependency factory by calling the ``AMQPBackendDispatcher::getBakend`` otherwise we can access a backend with an invalid state as the ``setDispatcher``  might not have been call.
that should be above our apps
Maybe put the closing tag on a new line

--
William Durand | http://www.williamdurand.fr


On Wed, Nov 14, 2012 at 6:54 PM, hardchor <notifications@github.com> wrote:

> In Resources/views/exposeTranslation.json.twig:
>
> > @@ -0,0 +1,12 @@
> > +{
> > +"locale": "{{ locale }}",
> > +"defaultDomains": [{% if defaultDomains %}"{{ defaultDomains|join('", "')|raw }}"{% endif %}],
> > +"messages": {
> > +{% for domain, translations in messages %}
> > +    {% for key, message in translations %}
> > +        {{ (domain~":"~key)|json_encode|raw }}: {{ message|json_encode|raw }}{% if not loop.last %},{% endif %}
>
> Twig ignores the linebreak here when I end with an if statement. Any idea
> how to force a new line / rewrite?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/willdurand/BazingaExposeTranslationBundle/pull/37/files#r2127522>.
>
>
> `a_.href =`

Сейчас скрипт изменяет ВСЕ ссылки в документе на локальные. Как думаешь, так стоит делать? Или лучше где-нибудь оставить оригинальные ссылки на сервер?
Что я и сделал. Прочитай внимательнее пулл-реквест.
why ? I 'hink just missing blank line in L312
sp mistake, should be "under"
Those are not related here and coming from the fix for multiple event registration branch.
Latter they would be merged with that branch or can be removed here.
Shouldn't this still error out if you don't have the webplatform since the framework will be useless?
this needs a unit test.
Document the regex
It would be better readable if you would write it like this
if (tt == null){

}
Yep.  this just requeries the one object saved, before it gets displayed in the list of saved records.