labofsahil/github-event-dataset-2012 huggingface dataset:
It's inactive for the time being but I intend to finish it in time for
release 0.14.
Em 11/06/2012 18:50, "Min RK" <
reply@reply.github.com>
escreveu:

> > +        return
> > +    cmd = get_activate_cmd(line)
> > +    p = Popen(cmd, stdout=PIPE, stderr=PIPE, stdin=PIPE)
> > +    out, err = p.communicate(cell)
> > +    p.wait()
> > +    if err:
> > +        print >> sys.stderr, err
> > +    return out
> > +
> > +_loaded = False
> > +
> > +def get_activate_cmd(line):
> > +    """
> > +    Returns the correct activate command given the platform
> > +    """
> > +    if sys.platform.startswith("linux"):
>
> This should be the default behavior, as the most likely other platforms
> (OS X, freebsd, etc.) will all behave the same as Linux here, rather than
> "Platform not supported".
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/ipython/ipython/pull/1891/files#r963238
>
The problem with the approach you propose is that all of us have slightly different standards as to when the underscores improve readability.  Maybe I like to avoid the "_" as much as possible, only using it in extreme cases.  Maybe other folks find that adding "_" between words *always* improves readability.  When these different standards are applied over a large code base, the end result is any given file has a random 50/50 chance of having "_" in it.  At that point, there is, in effect, no standard whatsoever.  We might have well as said "name the file however you want".

I am not in love with filenames w/o "_" in them and I agree that sometimes they are ugly.  But I am in love in consistency - I like to open up a directly and see that all the files in that directory are named the same way.  I suppose that if we long ago started to put "_" in all of our filenames I would be equally happy.

In summary I find PEP8's guideline on this point to be fine for single author projects, but next to useless when there are many opinions about "_" and readability.
Gotcha, never mind then.
Might be too narrow, on narrow screens? Perhaps keep it 100%, but figure out why it's overflowing the righthand margin?
`assert`
uri.path always contains something, we're checking it above with regex. Concatenation is redundant
Use `url` here instead of manually building the url.
We have a mixin that makes these widths a bit easier: .span(7) will span 7 columns (and float it left) and automatically add the gutter widths as well.
I think we usually go to next line for each selector? /nitpick
Our tests make sure you declare tables as INNODB:


```
AssertionError: There weren't as many occurrences of "ENGINE=InnoDB" as of "CREATE TABLE" in migration 153-add-document-products.sql.
```
Yeah, it is just a normal user. I could probably leave it as `u =` like everywhere else since there is nothing special. And a comment.
Are users unprivileged by default? Can this be explicitly specified when creating the forum and user somehow? Or a comment added?
Totally! That should be a separate test, though. This verifies that the view doesn't throw an unhandled exception. I can fix the comment accordingly.
Or similar?
This seems to be incomplete. Did you mean `common/products_select`?
I was sure there would be a better way, thanks
I believe you can use `Array(base_message['args'])` to conditionally coerce to an Array.
`p1` should be `first_package`
不要直接弹alert。。创建一个alert message吧。有个id为alert-area的div，插入到里面。
You don't need session << study, as it's done by set_uuid
(maybe we should change the name to save_and_set_uuid)
this file should not be executable
Please use the short class name
Two of those should be removed.
only fixed typos here
I use isset because the data_class could be set to null. array_key_exists wouldn't prevent that.
Wouldn't it make sense to have a `$form->getErrorBubbling()` proxy to `$form->getConfig()->getErrorBubbling()`
integer not boolean right?
I just recognize an even bigger problem: $this->kernel is type hinted as `HttpKernelInterface` in Symfony\Component\HttpKernel\Client
But here you assume kernel is a `KernelInterface` which is not necessarily true. `HttpKernelInterface` does not provide  `->getContrainer`.
Sounds sensible. I think I'll rename 'createWithStat' to 'create'. I don't like the name, and only chose it because the compiler would use type erasure to translate Function<byte[], T>.

I'll add NodeDeserializer to the zookeeper package.
Well this is just for now since i wasnt exactly what the final plan is with photos/albums. Im assuming once albums is implmented we can then just have albums for the pages/groups/events and this controller will just be replaced by albums.
Should this be interpolated? I was thinking this would be ERb.
please cite your DGCI paper (with 4 space indentation in order to create a markdown block, cf SphericalAccumulator for instance)
I think that it'll be good to use StringBuilder#append instead of strings concatenations.
И делать два вида смещения не надо. Достаточно одного
source and campaign_template are already parsed into Nokogiri at this point.  No need to parse them again.
'clear' belongs in this list, right?
чо это за говно
А кроме того картинки, которые не нужны, js-ки тоже можно, если они пустые (или по крайней мере в них не нужны закомментированные строки)
maybe pull this onto its own line?
```dart
messages = messages == null ? new Messages.silent() : messages;
var file = new SourceFile(new Path(''))..document = doc;
return analyzeDefinitions(file, messages: messages);
```

up to you though, we use both styles, it's kind of a tossup
Is three_3_10_3 a default panels layout?
If the deployer can configure the allowed ranges, should the message list the valid ranges?
Oh, bash instead of script and interpreter. You're totally right. I'll update it, thanks.
You should probably put in min-/max-pool-size here
Now if something is holding the classloader for this class in memory, the JVM won't shut down for 60 secs.

It seems we need some sort of general usage executor facility.  Other cases like this one have gone the new Thread(...) route, or have used a thread pool that was already available in the subsystem. But there are getting to be enough of them these cases that we should find a better solution.
I'll merge this pull, but will be reverting the this particular change (ivars instead of methods). Unless I'm missing something, let me know why this change was made, curious! :)
Ok, np :). Easiest way to check usually is to run the specs in 1.9 mode and see if it runs anything. In this case you can see it runs the specs too then.
вот это место надо отдельно проверить в RSS клиенте
А почему бы эти методы isEditableNow и isDeletableNow не перенести в TopicPermissionService?
Do you really need to do the lazy compilation here? Unless you actually have a startup time issue, I'd just compile in the constructor.

The (minor) problem with this kind of clever thing is that it changes the "shape" of the object and confuses the JIT. (Though my understanding of these matters is rudimentary, so I could be wrong about that.)
yet more frivolous time-burning! :)  I believe timers should be strictly ordered and cancelable (it just uses nsITimer's logic) even if the background thread has dispatched the notification to the main thread.  So really, 1 and 1 should be fine, or maybe 1 and 2 to be safe.  (Otherwise if your VM hitches your test could fail anyways).
Those needs to be l10nize.
foo({
});
Why are we removing 'org' as a searchable element?
If there is no need to use app.launch here, maybe we can remove the getSelf here if iconURL is just a fixed string.
@ferjm has confirmed to me that he will check the timeout issue on the backend side and we'll come up with a solution ;-)
You don't need to specify WindowManager in itself.
Declare a local variable in WindowManager and then expose to an entry in bottom of WindowManager as public API **if you really think you need this**.
But I don't see you use isFTU elsewhere? Could you explain more?
core apps are *not* removable.
Heh. Good way to handle that Tastypie has changed this.
The associated model should be removed if all references to that model are being removed.
should be `@param string $str`

oh, and could you name the parameter `identifier` too?

and put a blank line before `@param` to make things clear.
I think `eql` is for not comparing object identity ... I agree that this is not the perfect solution but at least it would be consistent.
Same question here about "assert" vs "raise" for the ValueError
Do we want to return here? Even if what we depend on already exists, we still have to build this rule
Did you check the performance of the INNER JOIN? It looks fast, but did you test the performance in your forum?
Agreed. Comment would be useful here
This commit is both for task stuff AND some other stuff. Like this. If you really want me to split it, I can.
Single channel, not multiples - need to fix related javadocs etc
I was pretty shocked too, but I couldn't find anything else and nothing obviously broke.
They are auto-generated and we had no Linux-users to worry about them.
Unfortunately, no.  It was accepted before we had the ironclad "MUST HAVE TESTS" rule.  We still let small mods to code that doesn't have tests in (since it wouldn't be fair to ask you to write tests for the whole module).
good point - it should be `ascii2unicode`.
That's what this is suggesting - in `TestRealWhatever`, or `TestAMQPImplementation`, add a skip if the real whatever, or AMQP, is not available.
I worry that people have used this function as part of their own split_file implementation, and that their code will fail with this change. And, unfortunately, fail silently, except for an exception in `twistd.log`.
i know it's minor but `strrpos` or even better `'packages.json' === substr($jsonUrlParts['path'], -13, 13) ` would be more appropriate
I've been refactoring code to make naming consistent all over the board. Lets agree on calling "annotations" by its name instead of "metadata". So, please, refactor the variable names and method names to use "annotations", "getAnnotation()", "addAnnotation()", etc...
This make sense. I'll change the pull request to work with regex and remove this hardcoded url structure.
Default  settings should be set in config/initial/1_settings
what about ` sudo usermod -a -G gitlab git`?
Oh, that's simple and obvious solution :)
These should really be selenium integration tests, as they are testing the UI functionality, and not unit testing the actual modules core functions
* spacing?
self.survey.sections.first.questions.inject(0){|memo, q| memo += q.correct_answers.size}
@ochameau Although mapping for `./` is really SDK specific so, I'd be ok with removing that section if you guys think it's too much information.

Also @ochameau @wbamberg @mossop I have being wondering if `paths` is a good property name for this. I went with it as it's equivalent of [requirejs](http://requirejs.org/docs/api.html#config) `paths` option, but maybe `locations` or something else would be better.
Isn't it breaking assumptions made here: https://github.com/mozilla/addon-sdk/blob/master/lib/toolkit/loader.js#L246-L248
We're stick to `===` convention unless there is a reason to prefer `==`.
commit added to PR.
Wait, nevermind, I know why this is probably empty. Ignore me.
nit: let instead of var, open { for the object starts on the same line just behind (
i'm wondering if custom user code should be outside of this file and in a separate file? seems like this one could get wild if they put a lot of stuff in here.
No purpose: I used the old validator file as canvas, so I left here this method.
Now grouped in validate() method and objectValidate template.
I tried to add this in, but the dependencies on bundler are a bit more complex than I thought. Maybe we could have this as a different pull request?
This part looks very complicated, don't know, may be better will be to move it outside of task?
@kpieters , I just saw your note about the FormElement helper which uses 'multi_checkbox'... I see what you mean. Doesn't seem like any value to keeping 'multicheckbox' in since it doesn't currently work anyway with FormElement. Carry on... ;) Looks good
The message should still talk about filter specifications as well as FilterChains.
However, it falls outside the MessageInterface - while it's convenient, I'd argue that unless you're storing messages, it's likely not going to be used often. As such, I'd leave it out, and you can either create a wrapper that includes it, or document how to test for it.
Tabs used instead of space.
Shouldn't the string be in a Constant interface? Keep in mind that this application will have multiple languages
Same issue with docstring here.
Has been re-inserted and compiles fine.
I'd tack a class onto this div and move the styling into style.css
Dump the list into an issue. We can tackle them together.
ok - the comment on htmlToCodeMarkup makes things clearer - this however is not returning a actual tree, just the string representation of it.
Korjattu: ei näytetä jos ei merkityksellinen. Tekstejä myös muutettu.
because you guys got mad at me for not putting requirements update anywhere, so i put it both in the project (for someone pulling/using it the first time) as well as in commcare-hq as well.
if-else please. I find shortcircuiting returns jarring.
Remove this since it's commented out?
Added this in 00bb588
use raiseException
Lol!

I guess we should check this with James.
Man you love these blank lines at the end of docstrings, don't you? :P
This should specifically catch ImportError.

Also, you should set a module-level var here, like "using_iptools" or "CIDR_support" or something, that you will test for later. See comments on the test below to see how to use it.
Might want to reconsider the file name at some point, I keep reading this as newgoo...
I wonder why nodelint didn't catch this. I may investigate further.
あと `its(:hoge) { }` とか。
http://shinsukekatsuyama.blogspot.jp/2011/07/rspec-1-it.html
一般的にはここに ENV への依存関係を発生させるのはおすすめしない
Should this be position: absolute; instead of float: right;

I am imagining floating right may push around some content on the page in weird ways.
There is a chance that the `default` client does not exist. Another scenario would be that someone uses more than one redis instance so it would be better to let the user specify the client.
end if (!is_param || (is_param && override)) && lookup_values.any?
missing indetion
As you already have the "with" statement, the file is already closed.
Use a label ID and change the label to: "All workspaces"
@daneden @jeremyworboys I'm 99% sure that jQuery (latest version at least) supports both hyphenated and camel cased property names.
Good call.  Moved both nextTicks to the end of the env callbacks.
this is connected to your _trigger_error()_ comment above. I choose to just ignore the parameter if it is false, and that's why the unit test was also changed to verify that `eZURLAliasML::setLangMaskAlwaysAvailable( false, ... )` does not break the new always_available bit handling.
red, really ? :-)
то, что я заметил вложенный цикл --- это просто удача
Yay for getting rid of that weird case.
cloud_user ? cloud_user : {:uuid => 0, :login => 'anonymous'}

instead
This should just be `add-limit-clause` as it has no side effect.
You should be able to just do return a.config.title > b.config.title
The idea was that if Redis returned an error, that it would be ignored and that cassandra would be attempted.
I'll refactor it so it'll log the error, but otherwise ignores it.
I think that as long as the roles are changed first, that should suffice. Thanks.
Actually, this was my first implementation. Then I thought that the extra-indirection was actually serving one purpose, i.e., documenting what it is extracted out of the ``AnnotationInfo``, which I believe is useful for reading the code. But if you feel strongly about it, I'll change it.
It could probably be done by making the 'companion' actor a member of this class (then `setThreads` can be private). Actually, I think this would be a better pattern, since actors and their companion class are so tightly coupled.
I believe in the ``ScalaJdiEventDispatcher`` we still have some synchronous call to the ``ScalaDebugTargetActor``. Is it possible that any of these synchronous call is sending a ``ThreadSuspended`` message to this actor?
You don't need two sets of parentheses.
`npm install` is enough
I removed a few more comments in 57fc5eddedf96b52f76700b70686359a594b2031. Is that okay?
Use ternary operator to do do a null check on node to avoid an NPE when doing node.getId(), perhaps?
minor - whitespace before (
Same here (filtering on IPs)
Yes, but that points to a problem with how the methods are laid out.  Those two methods should not be public.
Er this is a `public` repository. Could you please remove those credentials really soon? I'd recommend changing your password as well.
description нужен, чтобы можно было вывести состояние объекта в удобном виде. А ещё, суть этого метода такова, что он предназначен как раз для логирования. Поэтому, вместо [obj1 description] можно просто использовать obj1
Не нужно объявлять этот метод здесь - он уже объявлен в NSObject.
А можешь прислать какую-нибудь ссылку на статью или пример использования этого метода?
Тут нужно использовать вычисления с вещественными числами и округление до целого, так как возможна потеря значения.
"m меньше, чем размерность DNA" - с этим абсолютно согласен( не досмотрел..
You're right, my mistake! :)
Sorry about this. Don't know how this got into the commit.
Use var keyword to declare variables.
Well I've only used it with -ruby.  I tested my features against cucumber-ruby before writing the -js impl.
ReactiveCocoa already includes libextobjc. We could use EXTScope and `@unsafeify` here.
Maybe? It's not necessarily a programmer error.
How about 

```
Provides access to diffing functionalities to show changes between the working tree
and the index or a tree, changes between the index and a tree, changes between
two trees, or changes between two files on disk 
```
I dalej: "Pytanie pomocnicze do hasła jest niepoprawne". Szkoda, że to tak ogólnie brzmi. Przydałby się też powód, ale nie wiem, czy system aktualnie na to pozwala.
... adding a titles dict to make the story round.. surprisingly zip is smart enough to handle the unordered dicts, fascinating..
I don't like having to split off notch_filter and band_stop_filter from filter(), but I didn't see an immediately easy way to combine them in the existing filter().
"an"
If I'm reading the code right, the original behavior is that if someone tries to create an encounter of an unmapped type, an exception is thrown, preventing creation of the Encounter.

You've changed that behavior so it now _does_ create the encounter, with an arbitrary visit type.

(As an aside, I think that allowing the global property to specify a "default:visitTypeIdOrUuid" mapping, which if set would avoid the exception, is a better behavior...)
I think we will introduce this for Play 2.1, right?
I think you should remove the "false" as otherwise the if block will never be executed.
Can't you make this the event object? Seems like it would be neater.
include_root_in_json is false by default. Without setting "true" it breaks the build.
if .distribution namespace is kept, should it be created with way or in a separate file?
Can you please at an empty line between `@param ...` and `@return ...` as this is the 'standard' in SF2?
ok .. i am removing the <container> tag .. it just creates another place where the docs can get out dated without any significant value in terms of documentation.
please follow the Symfony2 coding standards for the examples
允许指定的网络地址访问。
Yes, submitted a correction.
On Fri, Feb 17, 2012 at 2:16 AM, Aaron Meurer
<reply@reply.github.com>
wrote:
>> @@ -4,11 +4,82 @@
>>  """
>>
>>  class SymPyDeprecationWarning(DeprecationWarning):
>> -    def __init__(self, value):
>> -        self.parameter = value
>> +    """A deprecation warning which can also hold the information about
>> +    the feature to be removed, when it will be removed, and what the
>> +    developers recommend to use instead.
>> +
>> +    This class is expected to be used with the warnings.warn function
>> +    (note that one has to explicitly turn on deprecation warnings):
>
> It should be turned on by default *in isympy*, but otherwise, it will use the default Python behavior (off by default in 2.6+ if I remember correctly).

Yes, the warning does not show with default settings under the python
interpreter.

Sergiu
Hmm, I have added this same line to my own script, while it does fix the issue of displaying the "disable-with" copy correctly, it doesn't disable the button nor does it revert it back to the original state after wards.
Well yes, if the old Message method is left in the current deprecated state as per your pull request it will be broken and that horrible symbol looks extremely bad if you're casually browsing the code.
The only thing I don't like about putting it directly on assert, is that a careless test that doesn't reset the fmttablelevels can cause hard-to-debug problems later on in other tests that assume the default. Maybe we should reset it back to default after tests? We can catch it in the metatable index function on assert.
I'm not too sure. I wager it would prevent adding a new history item each time. Let me test it...
same here for message string
Please revert this blank line
I would make the extStorage the first entry in the result list, not the last one. As this is the _default_ location, it should be searched _first_ in the methods using this list.
added new property "IgnoreAttribute"
love the copy pasta.
If you don't configure it, your default quota is 5.
Please configure it.
Do you really need to create so many objects?  Couldn't you have 4 static NodeFilterHelpers for the 4 different cases, and just return the appropriate instance?  `NodeFilterHelper` and `NodeFilter` are both stateless, so can be shared/reused...
Wow I have no idea how I managed to forget that file - fixed now.
Indeed. I'm not sure whether to `return false`, `pray()`, or `if () throw;`
are these necessary?  Running the example doesn't appear to produce these files, I don't think.
I've thought about refactoring this code, I'm not sure it would make the code easier to understand/maintain. What happens inside the conditional statements is really rather different in _uniform_y and _proportional_y. Using one method to resolve both these cases, I believe could result in longer and more difficult to understand code. Suggestions welcome of course.
after seeing how many times i have to filter the runs by name, get the list of one run, and do something to it, i am thinking that the manage page needs delete_run(name), activate_run(name) and edit_run(name) after all. that being said, i still think Runs should be sub-objects rather than having to wiggle the locators each time.
"anchors.centerIn: parent" is quite a bit faster (because it doesn't require evaluating complex bindings, among other things)
fault how what *confused*
Yes, but then it's not that readable anymore...
Typo: withing -> within
I need to change it to "pass".

On Wed, Sep 5, 2012 at 8:43 PM, Richard Rodger <notifications@github.com>wrote:

> In lib/parambulator.js:
>
> > @@ -63,10 +63,30 @@ var quantrule = function( foundok, rulename ) {
> >    }
> >  }
> >
> > +var lenrule = function( islenok ) {
> > +  return function(ctxt,cb) {
> > +    var conditionallength = ctxt.rule.spec
> > +    var value = ctxt.point
> > +
> > +    if( !_.isUndefined(value) ) {
> > +      if ( _.isNumber(conditionallength) ){
> > +        var valuelen = _.isObject(value) ? Object.keys(value).length : value.length
> > +        if ( !_.isUndefined( valuelen ) ){
> > +          if ( !islenok( valuelen, conditionallength ) ) {
> > +            return ctxt.util.fail(ctxt,cb)
> > +          }
> > +        }
>
> generally a positive statement is easier to understand than a double
> negative
> so islenok should return true if the len is >= the min
> however, the name of the function should be "pass" - this is consistent
> with childrule
> (and quantrule foundok should be renamed to pass as well )
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rjrodger/parambulator/pull/9/files#r1535925>.
>
>



-- 

*malex*
I'm not sure if we get any benefits from not including ie8, but we officially no longer need to support ie8.
@apex13 Can we use single quotes for the lines that need to change?
some spaces here please
碉堡了，为何Utf-8的是乱码？Git到底识别文本文件的默认编码到底是哪个？
This should not be done!
It breaks all other uses of truncate_string() where the value is not specified.

Instead you should look up the places and set max_length to 120 for the subject....
How about making the return type a bit more specific; maybe use ``string``?
Is it possible to define the element type for the array of parameter ``$messageParameters`` (using ``array<SomeType>``, or ``SomeType[]``)?
空格的正则是\s 所以改为/\s/g就可以了。具体可以看看<http://www.w3school.com.cn/js/jsref_regexp_whitespace.asp>
Ist 0 eine gute Wahl? Eine Node mit Id 0 gibt es. Vllt. besser Id -1? (Detail)
How is this testing that the re-sort didn't happen?
why commented out instead of deleted?
тестил на ruby-1.9.3-p194
Only in platform or cms also?
Nevermind, I looked a little deeper.
This can be `true? result[-1].first_arg`
touche. I can make a little more reasonable ;-).
This is not true. One is that we don't have recent data bout them, the other is that there are not stations nearby.
This two lines are ugly.

Why not have everything something like:

```
layout = self.panels[name].get_layout(columns) or PanelState()
layout.collapsed = collapsed
self.panels[name].layout[columns] = layout
```
Hm, the idea is to open him a page with input box where token is prefllled in. Reason is that sometimes user will need to paste in the token because it does not work. So define form for this, use FormView, validate it, set flag.
This is already default.
Please improve.
Can I use inline css for this padding, because putting it in css file will make faq class less reusable.
Hmm, it's a dict where the key is panel name and the value a list of dependencies. I'm not sure how to do it better than one dict?
You are not importing only module here.
What we decided about this?
I should use ' at args?
Other than the one at https://github.com/gibson042/jquery/blob/571cc6baa56fb8559a7ae860ee74093ee8a85832/src/manipulation.js#L465? What are you looking for, and where? I have never seen documentation for these internal interfaces.
Hi Noel,

Sorry it took me so long to get to this, I got caught up in the conference craziness.  I added a commit to the pull request to check this.

Cheers
Evan
Please merge the changelog entries ;)
How can this class also be used by AutoNSubstitute? As far as I can tell, the class is being introduced by this pull request.
i think, it is better to split this function in TeamCastSpell and TeamRemoveAura or add a new parameter for remove ^^
You should use the provided factory here. It's not an advanced option, it makes things a lot simpler and more flexible to configure. Rather than creating an options class, and then and adapter just use one line:

`$authAdapter = $serviceManager->get('doctrine.authenticationadapter.odm_default');`
> `[].some.call`

That would be `Function::call`. I would need to rely on `Function::bind` too, right?

    Function::call.bind Array::some
To this "wrapping up" section I'd like to add...probably in bold, that

You cannot access native DOM methods and properties directly from a jQuery object, and vice versa. If you see error messages like, "event.target.closest is not a function"' or "TypeError: Object [object Object] has no method 'setAttribute'" you are likely making this common mistake and now have the knowledge to remedy it.
Again, use `INTERNAL_IPS` setting. I really do not understand what and why are you doing this here?
oh I see, I would name the other one `parse_env_section` then, what do you think?
@amitaibu I think access content will be perfectly fine, I don't think we need a special permission.
Why have two?  I don't see RestSubscriber being used
Was the BOM removed here?
License boilerplate?
Doesn't this just work with StringPiece? We use:

StringPiece foo("foo");

if (foo == "bar")

in the code in many places.
Does it contain any non-reclaimable resources like, say, sysv shm segments? If yes, define an exit_process handler in the ngx_module_t. Otherwise, I wouldn't worry about it.
Slightly tricky, I renamed one light_clamped:)
Since Arch Linux is a rolling release distribution, not having the latest and greatest is rarely a problem.

I do not like APT because making deb packages  is notoriously complicated. Instead of fixing the problem, the community has create tools upon tools upon tools, each with its own documentation and caveats. Pacman PKGBUILD files are dead simple. I prefer KISS.

There are a few Linux [distributions](http://en.wikipedia.org/wiki/Category:Pacman-based_Linux_distributions) that use Pacman. Can you check if they have their own AUR?
Notes: needs test whether x and ei broadcasts correctly, I don't see how this well works from just reading the code
Yes: testGetUrlWithExperimentVariationSids and testGetUrlWithExternalProperties and testCollectWithExternalProperties
On Thu, Nov 15, 2012 at 9:30 AM, Paul Brian <notifications@github.com>wrote:

> In rhaptos2/user/__init__.py:
>
> > @@ -45,22 +45,12 @@
> >
> >  APPTYPE = 'rhaptos2user'
> >  VERSION = __version__
> > -_app = None
> > +app = None
>
> I thought the order was as follows:
>
>    1. run *init* (doing nothing but creating a placeholder "app")
>    2. run.py executes, running the make_app function which instantiates a
>    Flask app, and then sets the *init*.app variable to bind to the newly
>    created Flask application
>    3. view.py then is imported, it imports "from rhaptos2.repo import
>    app" and uses app.config app.XXX to perform its decorator based setup of
>    routes etc.
>
> we could either keep app as it is or we could change views.py to from
> rhatps2.repo import _app as app
> or some similar solution.
>
> Or am I missing something?
>
No, this is fine. I'm still wrestling with the tossing around of the
application object to do everything. It just seems very sloppy to me. But
never mind, I'm tired of fighting the framework. Just do it however the
flask folks think it should be done.

>  —
> Reply to this email directly or view it on GitHub<https://github.com/Connexions/rhaptos2.user/pull/9/files#r2139059>.
>
>
dbg is defined in Std.h.  It's my way to have debug statements in the code that are really easy to turn on and off.  If DEBUGLOGGING is #defined, then these write out to *dbgout.  But if not, the dbg macro becomes just `if (false) std::cerr`, so the compiler parses this fine, but trivially optimizes the code away.  There are three levels of verbosity which can be helpful too which are written as dbg, xdbg, xxdbg.
Aha, perhaps this answers my question almost immediately!??
In the `bounds` class above, `xMin` etc. had been changed to camel case.  Is this just a vestige from the past that should be changed?
This is sure a lot of copy/pasted code @dcadams -- isn't there a better way?
On the debian machine, apache is started when apache is installed.

But I'll test it to see what happens when we stop an already stopped apache.
Say what app we just failed to load
No indentation, i remove lines because these var are unused since we must use time_t type and to be comply with MythTV protocol (success response is OK, not something else). Below i call cmyth_rcv_okay. Also cmyth_rcv_okay read all block unconsumed on the socket. Old code read only 2 bytes and may cause a side effect of load the next time.
Not sure if it matters, but this seems to have high probability of being optimized out completely (the business with x). Maybe take a, b and return a+b or something?
Can we use String.IsNullOrEmpty(this.gitPath.Trim()) instead so we don't need the preprocessor directives, and get the same behavior on different frameworks?
No, but it doesn't say it isn't either... the spec is vague, it only talks about "these options", and not about values. We can always fix it later if people complain. I prefer to be strict here, as we can always relax it.
oh, great, thanks!
Artifact. Fixed.
@Tobion no it's just `<?=` who still available, in PHP 5.4 `short_open_tag` is only for `<?`, so you should keep that
Why not call onSourceAdded() for each source here? Otherwise any source that is already connected here won't get the srcRemovedClosure etc hooked up?
We need this same fix in the gallery app.  Want to add it real quick?  Can do it in a followup.
I prefixed the `\` to clarify that this class has no namespace but I have not looked at other stubs where this is the case
I changed the variable names to camel case and updated the commit range for my merge request. 
Is that what I am supposed to do if I make changes to a commit, but it hasn't been merged yet?
Language! :)
We removed this due to spam.
What are you meant to be checking?
All 3 checked in with latest commit.
This needs to be node['cinder'] ... not node['openstack']['cinder']... and defaults for username need to be put into attributes/default.rb
That sounds like a reasonable place to put it. As for pre-installed devstack box, do you mean for dependencies? Preinstalling devstack itself would defeat the whole porpoise.
А что считать за разделитель? Все кроме букв и цифр?
Все if/for/while должны быть с {}
Ты кидаешь эти исключения из try блока. В catch ты их перезаворачиваешь в FormatterException.
`this.selection.destroy()`?
Looks like a good pattern to repeat across all api urls.  `/assertion/:email/:password/:audience/:env?`, `/user_and_assertion/:audience/:env?` etc.
Fair enough, however you should be able to use the following, more pythonic (can't believe I've used that term) statement:

    root = hasattr(self, '_root_element') and self._root_element or self.selenium
Yes. That is going to be step 2 of this refactor. I want to experiment with some things, including a base PageRegion, but I thought it best to get the basic refactor done first so as not to have so many variables.
Yeah I know, This is because we are keeping one widget coming from C++. Its necessary to have it right now. When we change it to qml, I am going to remove all this.
これは本物のSpringの@Errors@インスタンスで、既存のプロパティ、メソッドにアクセス可能で、さらにマップ、プロパティ形式でアクセすることができる、特別な@Errors@オブジェクトになっています。

普通の文章的な句読点と、列挙する意味の句読点（プロパティ、メソッドとかの）が混ざっていてわかりにくい気がします。
I'll have a look at the center of mass problem the next days and try to figure out a better solution than the current implementation.
You should do this test with indices=True and indices=False: it's not the same code path.
Please never use a raw access to np.random: always create your own RNG with a controled seed:
<pre>
rng = np.random.RandomState(seed=0)
coef_init = rng.randn(n_features)
</pre>
I was not aware of that. I guess I'll add a ``multiclass`` parameter  or something?
``zero_one`` is clear. The question was more about the ones that need a continuous prediction.
Wait... f1 doesn't need a continuous prediction, right? So it just works here!
This is only for the ones like ``auc`` and ``average_precision``....
well... ``K`` is actually the number of regression trees to be induced - for binary classification it differs from ``n_classes`` - I tried to be more specific in the docstrings and comments. It is consistent with the narrative documentation and with the reference material (Friedmann's work, ESLII and gbm).

If you still think its not a idea - I'll change it
> What are the pros and cons of y = np.reshape(y, (-1, 1)) vs y = y.reshape((-1, 1)) ? It seems equivalent.

```
>>> len("y = np.reshape(y, (-1, 1))") - len("y = y.reshape((-1, 1))")
4
```

Just 4 chars :)
Yeah, we use UK english convention in SA, so I often do that kind of thing (doesn't excuse the rest of my miserable spelling though.. haha).. better to stick with USA one for this
could be prettified with new mocking helpers
This list of configs should contain everything that's customizable from the tiapp editor, so long as it can be added to the bar-descriptor.  For the time being i see publisher url and copyright that aren't supported here.
use `NULL`
I don't like the idea of setting a member variable with a reference to memory on the stack, but since it's a private member and seems to never get referenced after the function ends, it should be ok. If I'm wrong about referencing it later then this needs to be in the heap.
Why does this still result as added? Just because of the missing rebase?
Because the last read would block until there was a change.
the internal_links and extenal_links methods can be merged into one method to go over the array of links one time only but this will do the job for now
Yeah, this is not done using JS, will have a look what I missed here. Thanks
yeah, but it will be saved like data[:auth_token] = {...}

i meant that i can pass in .set method something like {:auth_token => auth_token, :foo => bar}
if thats desired i can extend it. I prefer the two-step approach, its more explicit and this command is not the daily use-command anyways.
Will ".::" on the end of  the line be rendered as "." or ".:"? If a colon doesn't naturally fit on the end of the sentence, I would put the "::" on a line of its own.
Kind of an awkward name don't you think?
I can access it if I clone the repo but not if I hit it from wget/chrome.
Should "254" be written if "254B" is competing?
Using ruby 1.9 I get the output if I modify it to:

print_line(m)
When there are multiple authors, one line by author looks better.
Yes, I think that "star" is most common English usage.   "asterisks" is correct, but I think for now I'd prefer to keep stars.
Why not extract this to a constant? it's easier to test that way too.
How about setting the environment on each extension when it's added to the environment?

```js

twig.Environment.prototype.addExtension = function(extension) {
    extension.setEnvironment(this);
    this.extensions_[extension.getName()] = extension;
};

twig.Extension.prototype.setEnvironment = function(env) {
   if (null !== this.env_ && this.env_ !== env) { 
       throw new Error("This extension is already associated with another environment.");
   }

   this.env_ = env;
};
Use `DeveloperError`.  Also document the @exception.  @shunter did you write this way back when?
Thankfully, this should be really easy to implement a performance test for.  Let's start out doing it the "right" way, which is how it is now and then we can profile and optimize it later on.  I plan on doing similar performance tests in places where we create way to many temporary object literals and can include these in that evaluation as well.
`GeographicTilingScheme` link is broken in the generated doc.
nope. D is not a function.
I agree this pattern of mutating a passed in param is not great, but it makes the calling code a bit prettier I guess. I just make it consistent.

This 
```
add_x(hash, options)
add_y(hash, options)
add_z(hash, options)
```

might be prettier than
```
hash = add_x(hash, options)
hash = add_y(hash, options)
hash = add_z(hash, options)
```

I actually like the builder pattern better, but that would have been a bunch of refactoring.
ANIMATION_CURVE* Constants where moved into iOS namespace as it was only available in iOS at that time and was marked with deprecation warnings. If this Constants are going to be used as general Constant throughout our SDK, a new ticket needs to be made and the corresponding constants need to updated in the documentation. See the current the [documentation](http://docs.appcelerator.com/titanium/2.0/index.html#!/api/Titanium.UI-property-ANIMATION_CURVE_EASE_IN)
To have this properly default to true, you'll need [TiUtils boolValue:value def:YES], otherwise doing tv.scrollsToTop=tv.scrollsToTop would set it false if undefined.
A simple command that doesn't work on Windows. Wrench.js is stable from our testing, works on all platforms we are targeting, and is already in use in other Appcelerator projects.
Аналогично с фильтром. Так же из имен этих переменных невозможно понять, что они содержат.
Should each of these be const members of the Filters class?
Right now, the user *must* run ```mvn install``` to be able to run the examples, if this is not an accident, it should be better explained here, as I missed that in a first reading.
dはdemocrats[0]なので重複といえば重複なのでなんとか省きたいなー
Are these suppose to be commented out?
we should do 3 sections here and keep jackalope-jackrabbit and jackalope-doctrine-dbal apart. only one of the two is usually required, not both.
Есть какой-то стандарт как называть разные формы http://unicode.org/repos/cldr-tmp/trunk/diff/supplemental/language_plural_rules.html не знаю насколько он распространен, знаю только что используется в рельсах. По нему вместо `0 1 2  n` будет `one few many other`.

Может нам по этому стандарту сделать?

У меня даже есть код:

``` coffeescript
I18n.pluralizationRules['ru'] = (n) ->
  if n % 10 == 1 and n % 100 != 11
    return 'one'
  if (n % 10) in [2..4] and not ((n % 100) in [12..14])
    return 'few'
  if (n % 10) == 0 or (n % 10) in [5..9] or (n % 100) in [11..14]
    return 'many'
  return 'other'
```
This will be slow. I suggest you replace uses of `enforce` with `assert` until we come up with a systematic handling of this matter.
`ProductInterface|null`.
Before the FAQ section, we should include a link to the video.
setter = 'value';

=> 

    setter = name;
missing close parens?
For the development dependencies, you should be able to include `thincloud-test` rather than all of these individually.
unneeded new lines in this file
Format this better, Caps on first letters in a title.
So, I guess it's necessary to store options via jqmData instead of attributes, because at the top (lines 22 - 29), the options are retrived via jqmData, and .data only reads attributes the first time.
WTF ? this is not a copy. Look at the other action
Running this command will not provide the class we are showing below. So why telling to use it ?
For end user, they *should* download the certificate authority file and put that on their server and point the sslCertificateAuthorityPath to the location of the certificate authority file. However, without those certificates, the service bus would still work as before without validation.
no, you're right.  optional now.
We should add a message "Mask must be a power of 2 - 1" to the assert.
To invoke the <i>setBeanClassLoader</i> on the <i>MessageReadingMongoConverter</i>. If its not invoked, 
it would use the default one though.
Interesting; was this really from an issue? https://github.com/SpringSource/spring-integration/commit/bb8409f96f0cca68a36b1dabb96b7f18586233e3 has no reference to a JIRA.

I am not sure I like putting a warning out on every request where this condition arises.
You shouldn't need the `File.join(File.dirname(__FILE__), ` bit here...
インデントは スペース 2つ
i don't understand. what do you mean by "constants"?

the code, as is, will create a new function and execute it immediately, every time. i don't see any value in doing that. from what i can see, it just adds another layer of function definition (and memory use and performance overhead) for what can be done with a simple if statement.

is there any value in this, that i'm missing?
I don't think we should translate developer error messages.
This is the particular place to note. These three static public members. This allows you to declare pre-defined navigation types that look like enums but are actually real classes (NavigationType.Dropdown or NavigationType.Tabs).
Get rid of this class and use CesiumFormattingHelper.ImageToDataUrl instead.
Were the changes above intentional, or were they the result of working based on a CPAN version rather than the version in master in the GitHub repo?
oops `CharField`, not `IntegerField`.
can you undo the unnecessary changes in this method?
Ah, ok. I think Petr should confirm the changes are fine before we accept this pull request. He's back from holiday next week. Thanks for sharing the changes with us.
Consider using an object as a dispatch table instead of chained if/else statements.
It's the same for problem for parameters actually so in theory with infinite computation resources and infinite speed we would indeed add method validation right before everyone and right after everyone.

But amongst the two options, I think that the latter is more useful and natural especially since rogue interceptors changing values are quite uncommon. In fact, they would be written by a user as I don't see a technical platform interceptor do that and in this case, at least in CDI, there will be a way to place them after method validation if required.

The interceptor ordering will be salient based (number based) and method interceptor will have a high but predefined number.

You can add your and my remarks to the proposal. It's nice to see that recorded even though I think we discussed that in the EG a while ago.

By the way the proposals are a nice way for people to see where we came from. This might be the first spec with such documentation of its features construction.
Then you'll be able to directly make this call:

translation.yaml_path
`FUCK_OFF` is shorter and more direct than `MIND_YOUR_OWN_BUSINESS`
xd yes
los ID nunca se ponen. se agregan automáticamente. y la PK tampoco, es el ID que se agrega automático,.
i copy/pasted qvga out of the fabfile
Use the javadoc `@see` here instead of "Refer to"..
is KrollStreamHelper used?
There should be an @file header here.

See: http://drupal.org/coding-standards/docs
New sub-module commit?
i wanted to keep it simple, especially since there could eventually be multiple FOS libs. also there is no ``src`` dir atm ..
Раньше, до того, как вы все перепилили, ежели избранный тред уже находится на странице, вместо его подгрузки происходила фокусировка на существующий на странице тред.
Давно хотел починить ея, да руки не доходили.
`tNum = el.id.substr(13).split('|')[2],`  сейчас фейлится в null
This file still seems to have some indention issues.
Added check. Thanks.
I think we should tell that default domain is `Admin`
The issue earlier we fixed was for OpenEJB . The TransactionProvider was never attempted to be loaded when running with embedded, so they couldn't provide one that worked with their transaction manager.
So if I am to make another extension, I'd need to modify ext/meta.mk as well? and add in a block like https://github.com/blackberry-webworks/BB10-WebWorks-Framework/blob/next/ext/meta.mk#L29 ?
remove and add blackberry?
would this msg end up with both ip4 and ip6? 
is that really what we want?

can you end up with both IP addresses?
Why is this test turned off?
SSL is required as part of the spec, but I would recommend using a rack-ssl component to enforce the transport security instead of baking this into the oauth controllers.
Dużo wyszukiwania po nazwie. Wygląda na to, że trzeba zmienić to na mapę Dictionary<string, Balance>()
This function has a unexplained paramater msg