labofsahil/github-event-dataset-2012 huggingface dataset:
Yep, will do.

On Thu, Jul 26, 2012 at 2:36 PM, Ohad Levy <
reply@reply.github.com
> wrote:

> > @@ -6,3 +6,5 @@
> >  :puppetconfdir: /etc/puppet/puppet.conf
> >  :login: false
> >  :require_ssl: false
> > +:single_org: false
>
> can you add a oneliner telling the users whats the diff between the org
> options?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/theforeman/foreman/pull/116/files#r1246943
>
This is a shot in the dark. Check what VARCHAR length a SlugField creates in the database. Then, check what happens if you title an event:

"This is a super long title that checks if the slug is shorten This is a super long title that checks if the slug is shorten This is a super long title that checks if the slug is shorten "

Django 1.4 might have solved this already but I remember it biting me embarrassingly hard in the past.
if at all possible module_load_include should live within a function. when its outside of a function, its executed when the file is read, not when the function that requires it is called.
nit: capitalize your sentences! :)
Its going to upload shit?!
maybe uuid.toLowerCase(locale.English) to be on the safe side? Though that may conflict with whatever hibernate/db thinks lower() means.
The current implementation has slightly more nuanced behavior, which may not be obvious because I have not documented it very clearly. The `AsyncExecutingContext` may be accessed from more than one thread. Consider this example:

```
*Thread 1*                                *Thread 2*                             *time*
async_context['a'] = 3                                                             |
async_context['b'] = 5                                                             \/
async_context._wait()                      async_context['c'] = 3
                                           async_context['d'] = 9
```
In the current implementation, the `_wait()` call will only wait on the two assignments in its own thread because it grabs a copy of the `_pending_update` count. If the proposed three state machine is used it will potentially wait until Thread 2's assignments have finished also. 

I believe the current implementation's behavior is what would be expected, but I could also be convinced the state machine's behavior is more correct.

Either way, I do believe implementing a state machine with clearly defined transitions and states would produce much easier to read code, and that may trump all of this.

Thoughts?
There's really no consuming code--remember that a dockets queue calls process_item, not vice-versa.
Given cross_raw is a ,FEW - can we not use a ,LOOKUP here - if we can it would be substantively faster
Fonéticamente sample es sampol en inglés subnormal!
This is wrong because it won't work with python3, check at the beggining of the file where i did: str = unicode... maybe you should replace that with str = basestring
Is this still needed now that the startup issue on unordered file-systems is fixed?
I'd say that sakaidocs and pages are content as well.
```var done = 0;``` should go on the next line
We are not using them right now. In Java, they are used to display the port number in the debug view, and having a context menu item to open the launch configuration. We might have it in the future.
Oh, now I like it :-)
I do this so the end-user doesn't have to, and it seemed to make sense to me to use the same interface as registering custom checks. Do you have an alternative to suggest?
Проверяете не тот параметр вместо size
Ну и как? Заработало?
А у объекта вашего класса этот метод не виден?
Это почти то же что я посоветовал сделать, немного меняется место где генерируется случайное число. 
https://github.com/cleg/osx-project-1/blob/master/FunnyBiologist/Cell%2BMutator.m#L22
1) точно нужен джиес?
2) создание данных хорошо выносить в before или let
I don't know.  I didn't like the goodbye text.  The whole remorter should be redone to have better story, since remorting should be a big deal.
I recently updated this so that the footer sticks to the bottom, and this change will affect it.  Pull, rebase, then update the -70 for the margin to -100
Dunno if we need this (hard to understand all logic) but maybe drop it to 0 comparing with frame_total?
Have you removed this row (and similarly in module.php) because "extension_id" is a serial row?
Hm, I hadn't thought about that. I'm inclined to think it's ok where it is for now, because right now nothing is using it. I'll keep that in mind if we do become more dependent on listening for events.
ica -> ICA
makes sense.


On 26.12.2012, at 14:34, Alexandre Gramfort <notifications@github.com> wrote:

> In examples/preprocessing/plot_ica_from_epochs.py:
> 
> >  # explained variance.
> >  
> > -ica = ICA(n_components=0.90, max_n_components=100, noise_cov=None,
> > +ica = ICA(n_components=0.90, max_pca_components=100, noise_cov=None,
> I would put n_pca_components=64 here rather than when call of pick_sources_epochs
> 
> —
> Reply to this email directly or view it on GitHub.
>
... Here the inplace makes more sense to me.
ugh, I was under the impression the API returns a `401` or `403` if authentication fails. But doesn't seem to be the case.
I believe this value is used further down. I haven't changed behaviour from the way it previously was anyway - will look to make sure this value is being used but I'm pretty sure it is further down
Shouldn't parser(rh) be wrapperd in a Future as well?
Ah, yes, it's fine - I didn't see that you had removed the 2 other form functions! Merged now - thanks!
This is too long semantically. We already know it's in settings, why add the word?
Less is more.
ToView? Щито?
Refactor is missing here
Ah, it didn't occur;  I don't have experience doing that. I will look into it.  There is also the problem about Prelude.catch , but that doesn't seem to cause a build failure?
You should be able to do 

```
#ifdef MIN_VERSION_directory(1,2,0)
  ...
#else
  ...
#endif
```

Re: catch, I'm not sure.  Why not just do what you did in the other patch, i.e. import Control.Exception qualified and use Exc.catch?  That ought to work fine whether or not catch is exported from Prelude.
this has been update just for testing
Amen.  We should have added add() long ago.
Let's assume for now no data in the db. We can wipe the data in dev before the next upgrade. In the future we would have to write calls to get the proper flavor/tenant from nova and populate them, if something like this comes up again. I agree we need no null values, but having the default make it possible for a bug to enter in. Ie someone forgetting to set flavor or tenant in a insert. 

Sent from my digital shackles. 

On Jun 9, 2012, at 10:54 AM, Tim Simpson<reply@reply.github.com> wrote:

>> @@ -25,11 +27,12 @@ def upgrade(migrate_engine):
>> 
>>     # add column:
>>     instances = Table('instances', meta, autoload=True)
>> -    volume_size = Column('volume_size', String(36))
>> -    flavor_id = Column('flavor_id', String(36))
>> -
>> -    instances.create_column(flavor_id)
>> -    instances.create_column(volume_size)
>> +    instances.create_column(Column('flavor_id', String(36), nullable=False,
>> +                            server_default="1"))
>> +    instances.create_column(Column('volume_size', Integer(), nullable=True))
>> +    instances.create_column(Column('tenant_id', String(36), nullable=False,
>> +                            server_default="0"))
> 
> It's required by the upgrade script. Maybe there's another way to do it? I assumed that default only applied during an upgrade but if it affects the table forever it may be worse that allowing the field to be null.
> 
> Sent from my iPhone
> 
> On Jun 9, 2012, at 10:20 AM, "Nirmal Ranganathan" <reply@reply.github.com> wrote:
> 
>>> @@ -25,11 +27,12 @@ def upgrade(migrate_engine):
>>> 
>>>    # add column:
>>>    instances = Table('instances', meta, autoload=True)
>>> -    volume_size = Column('volume_size', String(36))
>>> -    flavor_id = Column('flavor_id', String(36))
>>> -
>>> -    instances.create_column(flavor_id)
>>> -    instances.create_column(volume_size)
>>> +    instances.create_column(Column('flavor_id', String(36), nullable=False,
>>> +                            server_default="1"))
>>> +    instances.create_column(Column('volume_size', Integer(), nullable=True))
>>> +    instances.create_column(Column('tenant_id', String(36), nullable=False,
>>> +                            server_default="0"))
>> 
>> I don't like that we are setting defaults for the flavor_id and tenant_id, but if setting nullable=False requires a default then we'll do it, if not lets remove them, our code to update these values should take care of that, I mean making sure we set those values. 
>> 
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/hub-cap/reddwarf_lite/pull/110/files#r955768
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/110/files#r955823
Could we change this. This seems like could be a problem, we'll need to keep adding stuff when there's new parameters and the common service class will have to do the unwrapping. Would be nice if just the respective create methods had to know what to expect.

so something like `def create(cls, context, body):`
Remote cache store depends on a Hot Rod *server*?  :)  Even if optional, surely this is just test-scope?
Is this still ok, shouldn't we switch to topology id?
Yeah, good point again. Thanks
yeppers.
Weird spacing near the end here.
This sentence is a bit awkward.  How about:

"When True, remove axes which fully overlap the newly created axes.  Default is False."
For self consistency I would change all signatures in this module from dmin to vmin. Is there a consistency outside of this file which would make ```dmin``` preferable?
None or Node?
Is there a reason we are not waiting for the clock to load before proceeding?
Very specific rule - do you need all of these selectors?
This is a cool way to get the number of days in a month, but I had to look up documentation twice—once for `UTC` and once for `getUTCDate`—to fully understand it. A comment would help greatly.
The draft says that the requestAnimationFrame callback function takes the current timestamp as an argument:
http://dvcs.w3.org/hg/webperf/raw-file/tip/specs/RequestAnimationFrame/Overview.html#the-WindowAnimationTiming-interface
Might want to come up with a minimalistic description of the issue associated with ticket #11771 since the public at large doesn't have access to our ticketing system.
The log message should probably mention that it's only displaying selected dev tags.

Also -- might it make sense to audit these tags as well -- or better yet, audit their effect? e.g. if any tags are set, the cook runner (or perhaps the executable sequence) could add a new audit section saying "Debug features enabled," and the detail could enumerate which tags were set... maybe even a human-readable description of their effects too.
Ao inves de adicionar este tag script, inclua isto no js/app.js (colocando o photoviewer antes)
Do we need the `important` in here?
One current annoyance when trying to debug search problems is having to grep all over the source tree just to find out what code is in play. And so with the new classes, I've tried to make the template provider and processor names exactly match the class name. Would you be OK with following that convention here?
can you please use the CMath::qnorm function here?
what are these functions for? above it seems one can use any machine not just svms? If you implement shallow_copy for SVMs these functions would not be needed right?
SG_CALLOC here
Agreed.  It's still good for merge (wanted to get it in beta3) but could be refactored.
link to https://wiki.sourcefabric.org/display/CS/Wiki
Why is there a min-width on images?
Are you sure ``$v`` has the correct type as ``*`` is not a sub-type of ``string``?
Are you sure you want to override the parameter ``$testMessage``? This often makes code less readable.
in all other places you used `scaleX += 1;` how come this one is different.  I would expect this one to use the same scaling factors as in the other files.  If it needs to be +=7 here, then add a comment explaining why.

This is a question that i've been having, seeing as this scaling code is duplicated in multiple files, would it be possible to to move it to a single helper file and load it from there ?  Would it be useful my js knowledge is not deep enough to answer these questions.
"the only ones she would have said if it were Dragon-Tongue)".....d-d-d-douche

You might try italics. I guess random capitalization might be better.Than. This.
Awesome! Check out the Twitter example, that does JSONP.
There's no need for `(options || {})` here as `_.extend` can handle a null value.
Здесь мы надеемся, что нам никогда не пришлют 153,5 г или предусмотрели это в коде.
Shouldn't this be "HPCC Systems" ?
This stops working if the predicate `pred` is something that returns `true` for falsey values, like `nil?`:
~~~clojure
(boolean (first (filter nil? [1 nil 2])))
;=> (boolean (first [nil]))
;=> (boolean nil)
;=> false
~~~
To fix this, you should call `pred` for the first value:
~~~clojure
(pred (first (filter pred a-seq)))
~~~
Because I'm a chicken. I know that this implementation was reviewed by at least one paper - to me that makes this the default choice.
This is a way of handling CDNs. If someone wants to add a CDN plugin in the future and call it on one of the earlier system plugin events, that'll register it in the system before any other extension tries to. Otherwise, yes, first caller wins.
maybe you're right.
return constant($table->getPeerClassname().'::DATABASE_NAME') == $connectionName;

This is the shortest way I could found, if you have another idea :)
You mean the build.xml ?
I am not sure how the build works then for when I issue `phpunit` it will not load or when I move to `tests` and issue phpunit same effect .
But as a phpunit.xml is there I am able to run phpunit inside tests folder .

    cd tests
    phpunit 

That works!
Any reason this is package-private instead of private?
Okay. Fixed.
Add that the time is in seconds.
Constant string. Space after `,`.
Use jQuery `.each` to iterate over, then you do not need to play with `hasOwnProperty`.
No no, don't style like that. Add a class you want to use in template and match by that. We are in power to modify templates, so it is better to use classes then to use complex CSS rules.
*Nimaš*?
No, you use ready events for local-to-panel initialization. Then you should create ready-like custom event/API (through Deffered) for other panels to wait for map to load.

OK, please read a bit more around, what I wrote and what I send links. And you can combine it together and you will understand. ;-) If not, tell me and I will be more concrete in explanation.
`key` -> `token_key`, `secret` -> `token_secret`.

Please, when I comment something, check also around if it is something similar to fix. Like when I say to have more verbose/longer variable names.
TODO. Check the diff on GitHub.
The ignored files and folders all get a '/' prefixed. Is there a reason for this?
Title should be "500 - Internal Server Error"
True, however if a[0] is false you'll miss that and pass an empty hash instead. Which was the main reason I chose to do it the less concise way.
Where are these before/after log files generated?
>> warn him that this action should only be used if no other TGit application is running.
I agree
h1 for now
This file should be moved to path: getpaid/templates/getpaid/payment_post_form.html
as it will be pointed in app like
template_name = "getpaid/payment_post_form.html"
Isn't this the same as ``float(process.get_memory_info()[0]) / (1024 ** 2)`` ?
у меня есть тип команды и сообщение. Как я тебе узнаю длину первого поля? Щас оно одно, поэтому я не усложняла.
ahould tap take care of this rather?
s/circus/Circus
Accorsing to Wikipedia it's called [HTTP ETag](http://en.wikipedia.org/wiki/HTTP_ETag)
Should we add here any code action?
They don't stretch over the whole panel.
I think this should be sizediff_t to be in line with the documentation, although it doesn't have any visible consequences in this case.
This one isn't copied over. It's just part of the framework and you can import it or not, but you don't have access to the file itself.
Comma and space too much. For constant strings use `'` around them.
Please write similar copyright notice as it is on [wlan slovenija](http://wlan-si.net/) home page (you can link to the same page there).
This file is a stub for forthcoming README, it's incomplete.
Do you want to say "This and all other *.py files are..." so you don't have to repeat this everywhere?
Great that you harmonized these `pad_factor` params to the python-layer style.
逗号后面留个空格。
Yes, just a typo. Should I pull again or are admins able to fix that?
This is different because the keycodes for keyPressHandler aren't the same for keyDownHandler and keyUpHandler. I need keyPress evento to get the real keycode and show this on the input text.
Git is not capitalized in this way.
Location being null when testing feels bad / unnecessary (but I haven't reached the test code to review yet).
Can we force it to be non-null (and do checkNotNull in the constructor) if it must always be non-null in non-test code.
shouldn't this be `$(pwd)/brooklyn-dist-0.4.0-M2/brooklyn` or similar?
J'ai fait plus clair la.
이 부분은 `스크롤링 탭`으로 변경하겠습니다.
I have no idea. it can probably be removed altogether, it doesn't serve much purpose.
Poking around a few other projects, Test* appears to be normal in Python. I'm going to keep it as it is.
закоментований код
Good on ya for specifying a license for your gem.  But now that you've done so, you should provide (and update throughout the project's life) an email address so that you can be contacted e.g. if someone wants to license it under different terms (or just wants to let us know that it's being used).  I recommend using opensource@verticalbrands.com instead of your personal address, to avoid spam.
This is either badly named or API is wrong. Judging from the name of the method it sounds like it's supposed to not do anything and just return True if `fn` is a function or a basestring.
There a parenthesis that is not closed here.
Please extends OBController.
See examples in controller package.
I did a source->fix indentation. It should be better now.
we should not remove the lines above. they are going to be needed as soon as we add the compute service context.
Can we keep that code for now? I am planning to back that up once i am
ready with my patch. If that is stupid idea, yeah, we can delete that
for now. Thanks for reciewing!

KEY

On Jun 29, 2012, at 4:04 PM, David Welch
<reply@reply.github.com>
wrote:

>> @@ -199,9 +199,9 @@
>> BRAINSCutTrainModel
>> ::printANNTrainInformation( neuralNetType& myTrainer, unsigned int No )
>> {
>> -  std::cout << " Error, " << myTrainer.get_MSE()
>> -            << " Iteration, " << No
>> -            << std::endl;
>> +  //std::cout << " Error, " << myTrainer.get_MSE()
>
> Remove commented code
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/BRAINSia/BRAINSStandAlone/pull/90/files#r1078935
Interesting note here. I tried to use `$el.val('' + value)` but jQuery wasn't having it. Looks like the `#val(text)` only sets the value on the DOM but never updates the `value` attribute of the actual tag. Ugh.
These can be @ variables, since they're class level. @@ variables have weird semantics, and should generally be avoided. I think they're fine here though
Would it be better to link to http://c9s.github.com/Onion/ ?
Sosrry I hadn't realised identation had got messed up there.
As mentioned, let's change this to 'change_app_state_to_pending' and take a mock app object as an argument. You could even have a 'change_app_state' method that takes a state value, which this could call. It would allow tests to set arbitrary states.
s/Mis-Match/Mismatch
Is this needed on dev_4_4?
pep8: `1.0 / 2 * alpha` which should even be simplified as `0.5 * alpha`
I think this link should be higher up and more prominent.
This is not the combiner we talked about. Use the simpler

[ self.combine(node.args[0], node_args_k, register=True) for node_args_k in node.args[ 1:] ] ]
This is awesome. It's why developers say "Failure is progress". You had an idea, tried it, failed ... and it made you think about your overall approach. That is progress, because you understand more about the problem in front of you and about potential solutions. It should make you happy :)

The issue that you have is that the `Dog` class relies too much on data that is passed into the `happy` and `unhappy` methods. You have to pass in the right data depending on the context (i.e. depending on which branch of the `if` statement you're on), right? And that can get messy very quickly.

Looking at it the other way around the global namespace (i.e. where your `if` statement is) needs to know too many things about the `Dog`'s concerns.

On top of that you then also have the issue that you don't just want to print out reactions as a list. Instead you're trying to print out a more rich feedback (e.g. that some dogs where petted equally often etc).

May I suggest that you first solve the first issue? Maybe try something like this:

Remove the `happy` and `unhappy` methods for a minute (by commenting them out) and instead have a `reaction` method. This method is later supposed to return a reaction from the dog based on how often it was petted at this point. Now you can have a simple loop like this:

```ruby
dogs.each do |dog|
  puts "#{dog.name}: #{dog.reaction}"
end
```

If the `reaction` method is still empty then go and implement it. What would you expect? Maybe if the dog has been petted at least one time then it is happy? (something simple so you get started quickly ... you can improve it and make it more funny later)

Eg

```ruby
class Dog
  def reaction
    if petted > 0
      "makes a handstand"
    else
      "howls like a wolve"
    end
end
```

Does that make sense? If so then go ahead and reanimate the `happy` and `unhappy` methods and extract those strings to there. Also, have a `happy?` method to ask the dog if it's happy or not.

```ruby
class Dog
  def reaction
    happy? ? happy : unhappy
  end
  
  def happy?
     # what goes here?
  end

  def happy
     # what goes here?
  end

  def unhappy
     # what goes here?
  end
end
```
Sorry, you're not the one going through 1000 lines pom.xml.
I'm going to redo this part -- which does not work as expected with supervisor (but works properly with wgen's version).
I updated mine, but yours is better. I'll steal that.
i think so.
No harm intended :  ``Please``
The middleware should or will only dish out datetimes *without* microseconds. So I think this should reflect that too.
I understand the motivation here, but it makes me wish we had defaults set for all of the db passwords. It is strange that you have to set the cinder db pass even if cinder = false. not sure what to do about it though.
@shacker did you run the images through imageoptim?
Should be "/javascript...." (leading slash)
Monta nimi-arvo-paria samalla rivillä `let`issä menee nopeasti vaikealukuiseksi, joten ne on tapana laittaa kaikki eri riveille
destrukturoinnin voi tehdä myös suoraan parametrilistassa:

~~~clojure
(defn suit [[_ snd]]
  ...)
~~~
Should this be a node attribute?  I kind of consider WS_HOME to have been a bad idea.
I would really prefer if this accepted true/false and true was just munged into -f
If ExitOnSession is false and ListenerTimeout is < 0, this loop will continue forever pointlessly.
already a string no? to_s is redundant
use the methods from Post::File when possible please.
That doesn't look quite right... doesn't convert manage ranges?  Should be ``ascontiguousarray(convert(...), ...)``?
all kwargs, all the time. basically, this adding behavior to Django's builtin `Model.objects.create` function, so we don't want to inspect the kwargs at all, just pass them up to Django.
To escape some possible errors, removed functions could be deprecated in: https://github.com/wp-e-commerce/WP-e-Commerce/blob/master/wpsc-core/wpsc-deprecated.php
Add some more info how this is useful to update a menu, by adding new items or even replacing everything.
`( function` => `(function`
There is an error occurring accessing get_user_name now. 

But I think this method could be moved to base or page.py anyway because it's a global/generic method rather than specific to the login page.
I'm pretty sure there will be issues with this manual date conversion you are using.  There's a private function in JulianDate (convertTaiToUtc) that does it "correctly" but that isn't supposed to be exposed publicly.  Maybe we can just make it a private static function on JulianDate._convertTaiToUtc and use that instead?  It will also make this function cleaner.
Isn't this business logic?
Can we extract all the <script> tags into a separate file, so that both index.html and SpecRunner.html can include it?
Oh, sorry. The reason I split the default compiler into its own enum is just to simplify reasoning about the code (see below; it's used in the help message).
An error will occur when the build script is invoked same as if the existing build scripts are called without the Android SDK being installed or if the environment variables point to the wrong location.
Dictionary which can only have the `name` parameter
It's supported for parity. No need to force devs to fork their code for something that is harmless to call.
Agreed. I submitted an updated pull request.

-brian

---- 
Brian O'Neill
Lead Architect, Software Development
Health Market Science | 2700 Horizon Drive | King of Prussia, PA 19406
p: 215.588.6024blog: http://weblogs.java.net/blog/boneill42/
blog: http://brianoneill.blogspot.com/







On 3/27/12 5:53 PM, "Isaac Rieksts"
<reply+p-612121-079f2f7f504fb76d6e7fb9f32faf102c70dcbcb9-897229@reply.githu
b.com> wrote:

>> @@ -17,11 +18,15 @@
>>      private static Logger logger =
>>LoggerFactory.getLogger(TriggerTask.class);
>>  
>>      public TriggerTask() {
>> +        String threadPoolParam =
>>System.getProperty(THREAD_POOL_SIZE_PARAM);
>> +        if (threadPoolParam != null){
>> +            threadPoolSize = Integer.parseInt(threadPoolParam);
>>    
>
>Feels a little weird to set a static member in a constructor.  Seems like
>it would either be a member or would use and Integer type and only set it
>if it was null.  
>
>I think this is ok since we only create one thread pool.  I would feel
>more comfortable with threadPoolSize being non static.
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/hmsonline/cassandra-triggers/pull/29/files#r612121
Hm quick, just saw that you answered before me asking.. :)
Thank you for throwing instead of returning.
Oops, I missed the exception for POM files (_The <project/> element is always on one line._) to the generic rule : _Line wrapping: Always use a 120-column line width._

The drawback is that the eclipse formatter can not be used because of this exception. :-/
DELICIOUS DELETIONS
including config should be as first thing, since it can influence the rest of the module.
should also test with ubyte[] and friends
Oh wait it returns 0 for int[]. I'm having difficulties figuring the utility of this function.
The `@trusted` debate is one thing, but what I'm worried about here is that this a breaking change to `isSafe` for non-functions, for questionable benefits.
click "view full changes" to see what I'm talking about for this file.
I was also thinking about adding tests, but we should use drt + layout tests :)
A cleaner approach: add .DS_Store to ~/.gitignore_global 

Many have argued that having Mac-specific ignores just adds confusion
+1

The only thing is that this collides a bit with the name of the `ConfigurationSource` enum. We might consider to rename that to something like `MetaDataSourceType` or make it an inner enum of the interface implemented by the sources (`MetaDataSource$Type`).
Target Product: WFK
curly bracket needs to be on the next line
These need `@Override` annotations.
no commas needed
Se puder, cria uma função chamada get_post_terms onde retornará uma array com os objetos de cada termo, que pode ser utilizado nesta função.
Since the current behavior is to allow multiple instances as long as they are on different ports, shouldn't the default value here should be true, not false.
Please keep BC for the unserialization (see how I did it in ``Model\User``). This will currently throw a warning when unserializing an object serialized before the update (coming from some existing session for instance)
Might want to add comment that these need to be sorted in descending order by version, per implementation of GetServiceRuntimeAssembly()
Looks like `first` and `last` are never used.
please use lower case
Clear this out...
You can put it here, remember, these are examples, people may choose to remove this.
manufacture_year -- может быть стоит так назвать?
I ran this code on my dev server a few times, and it appears to work.  (The weighed average utility is usually around 4.4, which seems reasonable.)
Yeah, so it should be $($focusable.get(0)).focus() then?
Not a lot to comment. Added something
May be misunderstanding this line, but if anything the dependency is the other way around (i.e .ign_logging.sh is dependent on ign_log_write)
If you want ot make your code pass, just put in some commented out code about what you would call if it existed.
As I understand this was copy/paste...
In the JPA adapter txAdvice won't be applied on the Polling Task proxy. It is direct Adapter ability
Please add two spaces indentation to comment string
I see. Guess over-documenting this wouldn't be that good :)
Huh, what brought that about?
nein, nein, nein! Which exceptions are we catching? why and under what conditions do they occur? Don't use blanket Exception catches.
Sorry, hadn't noticed this before but I prefer not to use `>=` or anything besides `~>` when specifying versions because it can lead to some accidental issues with dependency upgrades. I know you made these changes to support rack 1.4.x. Mind changing the dependency to `~> 1.4.0`? I'll make sure everything runs ok with the upgrade before merging in.
Acording to things we talk about on meeting:
we need to add ``lib`` to the path, so you need to add
``site.addsitedir("lib")`` line once again.
Immutable List?

Also, do you need/want ordering?

Maybe ImmutableSet?
one too many spaces here
«два лейбла» можно перевести как «две подписи»
I should have mentioned this before, but you should call attention to the fact that `coordinates` is in degrees by calling it `coordinatesDegrees`.
Is this change compatible with Symfony 2.0? We're not ready to break compatibility yet.
It does, I've added that now
As I mentioned above, this should probably be happen when the csv file is initialized; otherwise, in certain modes of running the script, you may get a lot of header rows repeating.
Change to only remove whitespace.
While you're editing this, you can also remove the ` \` at the end of each line.
Ahh, I see. This has changed since the last version I reviewed. There we had getXYZDescriptor. Now it is just _getElementDescriptor_. Definitely better. I didn't like the previous approach. I need to get a handle to the consequences to this approach ...
go up :D
midooni man roo in joziyat hasassam
Yes that is one of the biggest messes in the code generator.  One day when I have time I'll go around and make them all consistent.  Maybe I can pass that off to renato!
These methods (`startup`, `shutdown:`, `suspend:`, etc.) are not meant to ever be called directly. They are instead intended to be called by the system during events which trigger these. Documentation should indicate this and also give some information about when they may be triggered.
Perhaps add a comment such as `// append returns the StringBuilder`, or keep the `StringBuilder` type declaration and add a comment that `StringBuilder.apply is a Function1[Int, Char]`?
Same as:
~~~clojure
(some empty? seqs)
~~~
Add the **splunkVersion** property
Hmm, do we need to register the transaction provider here for the embedded containers?
go ahead open it when you can. Have a look at the rest of this file, its really weird at this poing perhaps just open two at the same time
one for refactor of this file to not make functions properties
and
second for not having a proper way to report errors
We should probably pass the personalMessage string by reference.
The surrounding `if` is unnecessary. `(== (mod n divisor) 0)` already evaluates to the correct boolean value.
This fix has to be done to not get an exception about converting CDbExpression to string.
It's optional subfolder should not be a mandatory otpion