labofsahil/github-event-dataset-2012 huggingface dataset:
Doesn't that evaluate to

```python
if (key) and (mins_to_expiration is not None):
```

? I want to explicitly check for `None`.
I'm not sure what you mean? Do you want there to be two boolean values instead?
i added this check:


 if (resource instanceof JavaResource)
      {

         ejb = JavaUtils.getJavaClassFrom(resource);
         if (ejb.hasAnnotation(transactionAttributeType.name()))
         {
            throw new RuntimeException(
                     "Current class have already TransactionAttribute annotation!");
         }
         ejb.addAnnotation(TransactionAttribute.class).setEnumValue(
                  transactionAttributeType);
         save(ejb);
      }
      else if (resource instanceof JavaMethodResource)
      {
         Method<? extends JavaSource<?>> m = ((JavaMethodResource) resource)
                  .getUnderlyingResourceObject();
         if (m.hasAnnotation(transactionAttributeType.name()))
         {
            throw new RuntimeException(
                     "Current method have already TransactionAttribute annotation!");
         }
         m.addAnnotation(TransactionAttribute.class).setEnumValue(
                  transactionAttributeType);
         ejb = m.getOrigin();
         save(ejb);
      }
Why was the type (`int`) removed?
Timer should be invalidated before setting to nil.
Well I think that this code is a point for later rewrite anyway :) Probably, I'll make things derived HandleWraps more DRY for 0.9
This is not quite what I meant. What I had in mind is essentially `max(1024, sysconf(_SC_NGROUPS_MAX))`
i think the parenthesis should come after `exists=True`
Can't say I'm happy with PyUnicode_Kind being changed to be a size, but that is a Python issue...
Определись с внешними скобками, ниже ты их не ставишь, например.
Please make this `app/decorators/models/forem/post_decorator.rb`
Why not this:

```ruby
module ActiveRecord
  class FixtureSet
  end

  Fixtures = ActiveSupport::Deprecation::DeprecatedConstantProxy.new('ActiveRecord::Fixtures', '::ActiveRecord::FixtureSet')
end
```
Could we leave this initialization here? Without it we can receive a "non-initialized instance variable" warning
The changes in this file are not related to this commit, could you please remove them?
Typo: "will be interpreted as _an_ escape character"
Okay, if you're testing comment to html this test is fine, but remember it's a two layer transformation you're testing:
source => parsed comment => html

I would rather have two tests, source => parsed comment (which you can easily print) and then source => html (this test).

WDYT?
It's not a name clash; it's a consequence of fixing "originalName" with respect to local suffix.  The only difference here is how the name is printed.

    -    def originalName: Name = nme.originalName(name)
    +    def originalName: Name = nme.originalName(nme.dropLocalSuffix(name))
Yes, that's right. No worries :)
What's the reason final isn't implied when private?
It makes sense, but if the apply was transparent there would be no way to get to that tree.

With the current solution, one can locate the `Apply(qual..` tree, and it has access to both the qualifier and selection. This allowed the IDE to offer hyperlinking to both `List.apply` and `List` on that position.

If we make the `apply` transparent, one will always retrieve the qualifier. Since trees have no way to get to their parent, there would be no way to hyperlink or get to the `apply` node anymore.

Does this solution make your life much harder?
The only way promotions would be applied to an order is if `fire_event` is called with an event which the `Spree::Promotion` uses. The promotions are only applied to `payload[:order]`. I couldn't find where `payload[:order]` is defined, but my hunch is that it is the order in `params` or the `current_order` both of which (as far as I know) are incomplete orders. Do you know where `payload[:order]` is defined?

I could add a check [here](https://github.com/iloveitaly/spree/blob/multiple-product-promotions/promo/app/models/spree/promotion.rb#L51) to ensure that the order is `!completed?` but I wouldn't want to add that unless that was actually a possibility.
can this be wrapped as

```
res = Rational(-1, m + 1)*cos(x)**(m + 1)*sin(x)**(n - 1) + \
      Rational(n - 1, m + 1)*trigintegrate(cos(x)**(m + 2)*sin(x)**(n - 2), x)
 ```
`\ dx`
@catchmrbharath, you were right about the finally block:

```
In [1]: a=1

In [2]: try:
   ...:     raise Exception
   ...: except:
   ...:     raise Exception
   ...: finally:
   ...:     a=2
   ...:     
---------------------------------------------------------------------------
Exception                                 Traceback (most recent call last)
/home/stefan/<ipython-input-2-ea2d083e8b4d> in <module>()
      2     raise Exception
      3 except:
----> 4     raise Exception
      5 finally:
      6     a=2

Exception: 

In [3]: a
Out[3]: 2

```
As you point out, this is wrong.  So can you review #1481 so we can merge it? And then you'll need to merge over master again and change this to be like that.
> I understand your point, and in my opinion it would be the best way. But I don't see how to say at once to all unit what system should be used in priority if they don't come with "intrinsic" system (as it is the case when you get the unit from the system, which sould in principle be the preferred way).

I am not completely sure what you mean. Isn't "getting units from the
system" the only way to do it? If not, why not?
But even if the objects are essentially just cached, they are determined deterministically from premises. So it wouldn't change the behaviour to compare them as well? [Don't bother about the performance difference.]
On Fri, Mar 30, 2012 at 2:17 AM, Julien Rioux <
reply@reply.github.com
> wrote:

> >          denom_others = []
> > -        newargs = list(args[:])
> > +        newargs, numer_others = expr.func(*args).args_cnc()
>
> @smichr Could you implement your idea in a branch so that we can test it?
> I think that since gamma is a function over the complex numbers, all the nc
> terms end up in numer_others. My patch puts them there right away.
>
>
OK, see my combsimp branch.
Not really. The output format is like that of WolframAlpha.

>  —
> Reply to this email directly or view it on GitHub<https://github.com/sympy/sympy/pull/1579/files#r2201497>.
>
>
```
elif e.is_Add or e.is_Pow:
    return e.func(*[a.limit(z, z0, dir) for a in e.args])
```
«в каче<b>ст</b>ве»
This SQL should be rewritten from scratch in another PR. Currently it's messed and inconsistent. So i don't care about SQL code style right now.
The hash isn't meant to be a hash of the code though, just of the schema and the associations definitions aren't part of the schema.
Is this intentional? Seems unnecessary effort?
Oh, cmon Haegin, this is not a big deal :D
For subrequests, you should probably not catch them, to avoid receiving an error page
I would like someone with OpenBSD to verify this.
Hmm, I didn't consider that, but [it looks like it *does* remove duplicates on a case-insensitive basis](http://pastebin.mozilla.org/1698670). In any case, it for sure gets rid of the MultipleObjectsReturned errors we have.
I don't really know. Do you need one?
Btw I would base this permission on the limited set.
I don't like the use of "query" and "queried" together... perhaps "data is retrieved" instead?
"Enabled" is a board setting, therefore should refer to avatars in plural. An individual user may have an avatar, in which case the avatar would "exist" or be "present", or may be permitted to have an avatar, in which case the avatar would be "permitted".
Those should probably be in the drivers too, right?
just so the user could actually debug it
s/disabled/enabled ? I would expect frame times are not collected when fps logging is disabled.
Yup, comment has gone stale. I'll update it.
Sorry. You are both correct. I've got a branch that changes ```coord_dims``` and will test it in the morning before submitting a pull request.
swap node.inputs[0] and self._numop.__name__
It seems that SciPy suppot ``uint8``, ``uint16``, ``uint32`` and ``uint64`` so all unsigned integers. You want a ticket on Assembla or an Issue on GitHub?
не надо ничего говорить, если нечего делать
2001,1,1
"introductory tutorial" is redundant IMO. "testing tutorial" gets the point across.
Is there need for the deleted_object_pks as there is already a list of deleted_objs which contains the needed information.
@geemus Hey! No worries - checking the status and outputting a message sounds perfectly fine and reasonable :)

Do you prefer to write the patch and refactor on your end or did you want me to submit an update?
Perhaps `exposeHeaders = jQuery.trim(exposeHeader).split(/\s*,\s*/)` instead?
There should be coverage for all manip methods
This will propagate the ArrayIndexOutOfBoundsException, consider catching it and rethrowing with a more appropriate error message, since this one will expose the shifted index and may lead to confusion.
We didn't make that change because LazyBigIntArray is a variant of the existing BigIntArray and we wanted to keep the patch small.
Is <em> the convention through all MADlib docs for variable portion in function signatures? Otherwise, it does not seem to stand out immediately to allow new users to understand they can replace "distribution" by "student_t" or "uniform".

Also, is it the case that parameter1, parameter2 and parameter3 have (1) different meanings according to the "distribution" that the user calls, or (2) same for any distributions? If (1), perhaps a link to more actually function signatures should be put in this @usage section (the one in @sa?); if (2), it might be better to change the parameter names to be more informative.
In which case I'd like to see some comment here explaining the expectations placed upon a caller of this function.
Why does this take an `Edge` as an argument if it doesn't use it?
No, I think $strlen on a nonexistent field should return zero.  This fits the operation of other operators, which are as forgiving as possible unless the result is definitively wrong.  Here, I think it makes sense to treat this as an empty string.  However, I think it should be an error to call $strlen on a real field that does not contain a string value.
Though the bling factor of just going to localhost:1234 and getting your index.html up might be worth more than clinging to keeping the namespace to ourselves.
This probably deserves a comment so that somebody doesn't come along in a month and change it to `ENV.cc` and wonder why it doesn't work.
Must have seen it in another formula. I'll get that when I clean up the other bits. 

On Oct 22, 2012, at 3:10, Samuel John <notifications@github.com> wrote:

> In Library/Formula/analog.rb:
> 
> > @@ -0,0 +1,27 @@
> > +require 'formula'
> > +
> > +# Jason Thaxter <jason@thaxter.net>
> I appreciate your work here, but usually we use git blame and git log to assign credit.
> 
> —
> Reply to this email directly or view it on GitHub.
>
Well really it should be factored out into a private method; `add` is getting long already.

```rb
when Symbol then add_symbol(spec)
...

def add_symbol spec
  case spec
  when :x11 then ...
  when :autotools then ...
  when :foo then ...
  ...
end
```
Absolutely right. You convinced me it's better this way :-)
index and plural indices
do end, not {} on multiline blocks.
If requesting /alias/dir/index.php/pathinfo or /alias/dir/index.php?param
scriptFileName == '/var/www/dir/index.php'
But then you need a lot of dispatcher, and it will make the overall vumi architecture bulky and very complex. 

Let image a simple case: i have 5 short code in 2 countries and my programs are using 10 keywords. So i need a layer of dispatcher that will filter according to the international prefix, so only 1 dispatcher is sufficient here. For the the shortcode dispatcher layer, i need 2 dispatcher, one for each country. Finally on the keyword dispatcher layer, i need 5 dispatchers, one for each shortcode. So totally i have 8 dispatchers. Already quite some for this simple example, Text To change has at the moment 11 shortcode in 7 countries... and it's on the increasing trend.

Then when it come to add a new shortcode in one country. I need to review my architecture of dispatcher to 1) add a new dispatcher 2) modify the shortcode dispatcher config of this country.

The thing is that at the moment it's impossible to test and monitor so many vumi dispatcher effectively. When the architecture is changing with a new country or new shortcode. The tree structure of dispatchers has to be changed (some config update + new dispatchers). The maintenance of the architecture will just drive us nuts.

On the other hand having a unique dispatcher filtering according to 3layer of rules, seems at least test-able for the all dispatcher job. When a new country or shortcode is added to the system, you just has to update the routing rule in one file.

I agree to have several simple thing rather than one complex thing, but it seems to me that it's not worth it here. I'd rather have one dispatcher. Still it is doing only one thing and it does it well: having simple behavior if necessary but also can handle complex rules if necessary.
Expiry is probably good just to protect us from an ever-growing redis store. I don't know whether we can detect a failure to send because of lack of airtime though. The SMSSync API doesn't obviously support reporting sending failures.
Agreed on both counts. I'm happy to revert this before merging, but I'd kind of like to keep it in the branch until we have a better solution or are ready to merge.
Why do we need this method for?. I don't see it used.
Another lurker comment, but I think JavaWriter makes a lot of sense; using a DSL for emitting Java source code is much more sane than a template-based approach.

FWIW, I've done the same sort of thing, and have a reusable "sourcegen" DSL, hidden within this joist-util project. You can see some of the unit tests here:

https://github.com/stephenh/joist/tree/master/util/src/test/java/joist/sourcegen

https://github.com/stephenh/joist/blob/master/util/src/test/java/joist/sourcegen/GClassTest.java

It's very similar to JavaWriter, although instead of each method call immediately calling "out.write", it buffers the fields/methods/etc. as data structures, and then outputs them in a final "toCode" pass. This means you can do things out of order, like later add new imports, annotations, etc.

Which, depending on how complex your processor is, can sometimes be useful to have your processor code organized logically rather than solely on the physical order of the resulting Java code.

Anyway, YMMV, and definitely under you don't want to pull in a dependency, even with jarjar'ing, unless you really need it.

Hopefully I'll eventually do more than just lurk, as I've done a bit with APT over the years, and think it can be pretty neat applied to problems like this, I just haven't had the time yet. Great stuff!
This test will always pass, since fail() throws AssertionError.
A couple of comments here:
* the method is misnamed. It shouldn't be "get" because it doesn't return anything. AddSpawnMessages maybe?
* It's scary that the list is exposed. I understand it's for performance reasons, but theoretically couldn't someone accidently do a .clear() and break the server? Do we need a special subclass of list that hides those methods? Something else?
This should use the try structure

lock.coreReadLock(...);
try {
.....
} finally {
    lock.coreReadUnlock(....);
}

This guarantees that the lock is released, even if an exception is thrown.  With locks, it is much better to be safe than sorry.
I would've thought that, too, but I was getting TypeErrors on some of the feeds without those extra guards.
Also, these styles are now consolidated as mixins—if you use

     @include list-component-link;
     @include child-node-link;

you should get the correct styling on phone and tablet.

(You'll have to merge master in first, though.)
Will do in ``brackets-shell``
Added html espcaping to addMenu() and Command.setName() as well

-----Original Message-----
From: Peter Flynn [mailto:reply@reply.github.com] 
Sent: Thursday, May 24, 2012 4:40 PM
To: Ty Voliter
Subject: Re: [brackets] Menu and Command and Shortcut Extensibility implementation (#936)

> -            if (menuBindings.hasOwnProperty(keyCmd)) {
> -                commandStr = menuBindings[keyCmd];
> -                menuID = cmdToIdMap[commandStr];
> -                if (menuID) {
> -                    // Convert normalized key representation to display appropriate for platform
> -                    if (brackets.platform === "mac") {
> -                        shortcut = keyCmd.replace(/-/g, "");        // remove dashes
> -                        shortcut = shortcut.replace("Ctrl", "&#8984");  // Ctrl > command symbol
> -                        shortcut = shortcut.replace("Shift", "&#8679"); // Shift > shift symbol
> -                        shortcut = shortcut.replace("Alt", "&#8997");   // Alt > option symbol
> -                    } else {
> -                        shortcut = keyCmd.replace(/-/g, "+");
> -                    }
> +        if (name === "---") {
> +            $menuItem = $("<li><hr class='divider'></li>");
> +        } else {

name needs to be escaped before adding to HTML (or add after the fact via jQuery)

---
Reply to this email directly or view it on GitHub:
https://github.com/adobe/brackets/pull/936/files#r879003
Nits:
* prevent -> prevents
* Add a space between this var and the function below
This should say: 'dispatches the "contextMenuClose" event'
if you don't want to provide a phpdoc, adding an empty docComment is useless
Can you add documentation why rb_eql doesn't work?
*Translated strings should now be available in PiplMesh.*
Because this is the same stack as all the plugin content, in this case it's actually a good idea to use an arbitrarily high number, like `1000`.

Also, consider naming this `@handleZIndex`, since that's what it is.
`stores.publish`
fixes the pencil icon position
This is soooo much better than iframes, is the plan to move the editors to something like this as well?
Probably doesn't need to be cached. You can just reference `window.history` directly below, right?
does this ever get used? Looks like we can just use _rootElement
correction, into the popcorn.js we build, so will work on export too.
Not sure you need a `DOMContentLoaded` here. Do you?
"registrar" is better than "registrator" (which isn't a work as far as I know).
Er disregard that last note.
rb->busy说明还有数据可以发，rest != rb->rest，说明读了部分数据，不能却掉的。
You need to pull in master
* you should combine them with flatMap
* if you want to block (or whatever semantics is associated with M), just use the extractor, not checkIsomorphism, as the name is misleading
* if you want to check that you went all the way to the last M in your flatMap chain (for example because there is a test there), just do something like that:

```scala
val result = m1 flatMap m2 flatMap ... map finalTestLogic
result must be ('success)
```
It can, I'll do another pull-request for that. It's on my todo list.
It'd be more space efficient to store the data bits in a bitfield.

Something like:

```
class WiegandCard {
  ...
  private
    unsigned long long dataBits;  // bits read from Wiegand interface
    unsigned char bitCount;       // bit position for next read within dataBits

    void readZero() { bitCount++; }
    void readOne()  { dataBits |= (1 << bitCount++); }
  ...
}
```
I think we should round to 7. Let's do it.
Yes, but that's true for the other tests in there as well...
`toggleEditMode`
Hi Vivien,

On Thu, May 24, 2012 at 3:41 AM, vingtetun
<reply@reply.github.com>
wrote:
>> +      param[Storage.ums_mode] = val;
>> +      settings.getLock().set(param);
>> +    }
>> +  },
>> +
>> +  handleEvent: function storage_handleEvent(e) {
>> +    switch (e.type) {
>> +      case 'locked':
>> +        Storage.setMode(Storage.automounter_disable_when_unplugged, 'screen locked');
>> +        break;
>> +      case 'unlocked':
>> +        var settings = window.navigator.mozSettings;
>> +        var req = settings.getLock().get(Storage.ums_enabled);
>> +        req.addEventListener('success', (function onsuccess() {
>> +          Storage.setMode(Storage.modeFromBool(req.result[Storage.ums_enabled]), 'screen unlocked');
>> +        }));
>
> Sadly this code won't work for the moment because bug https://bugzilla.mozilla.org/show_bug.cgi?id=731746 needs to be resolved first. You have to use settings.onsuccess = ... with all the problems coming with it. I suggest to use the SettingsListener.observe helper.

So it is actually working. It was recently landed as part of:
https://bugzilla.mozilla.org/show_bug.cgi?id=743336
I use it from the C++ side as well (so it specifically doesn't use any
DOMRequests)

-- 
Dave Hylands
Shuswap, BC, Canada
http://www.davehylands.com
Can you just clarify for users what they should actually do to resolve this?  E.g.,

```
Creating units from strings requires the use of a specialized
parser for the unit language, which results in a performance
penalty if units are created using strings.  Thus, it is much 
faster to use unit objects directly (e.g., ``unit = u.degree``) 
instead of via string parsing (``unit = u.Unit('deg')``).
```
I'd say that belongs in the docstring for the `Angle` or `*Coordinates` classes, because that's where exact syntax is covered. The idea of this sentence (I think - @demitri wrote it) is to just say the general philosophy.  The intent here is to indicate that you *don't* have to specify what format to use when its unambiguous. E.g., ``Angle('1d2m3.3s')`` should work, and not require ``Angle('1d2m3.3s', unit=u.degree)`` or something like that.  Does that make sense, @taldcroft ?  And if not, how might you suggest rephrasing it to better reflect that intent?
When documenting the initializer, you should put these arguments in a section called Parameters, and use the syntax described in the docstring document (see http://astropy.readthedocs.org/en/latest/development/docguide.html#class-docstrings) - that lets sphinx automatically put thing in the right place.
I don't get this code. Where does this 2 come from?
Nothing else ends with a full stop, why this one?
I disagree. Passing "::somestring" would be invalid either way and the developers fault (you cant possibly cover all mistakes possible here).
But using the type safe check we would at least result in a meaningful exception: "trigger_error(__d('cake_dev', 'Could not find custom validation rule %s'" instead of trying to execute it as a preg match string.
How about this https://github.com/cakephp/cakephp/pull/831

Seems cleaner like that to me.
By the way, what "version 1.2" corresponds to? The version of CGXP? c2cgeoportal?
Use the Sauce (Labs).
I'm not a fan of class level arguments in unit tests, can you move this to each test? It makes it clearer as you don't have to look outside the test to get context.
Make this a set, please
We tend to prefer to steer clear of attributes that might be present, and instead provide a default value of `None`.
@michaelmior The lexer name (from pygments) is Awk and not awk.  That said, simply removing the lexer line would be better since Linguist would then default to Awk for the lexer name anyway, so there's no need to specify it.
I thought about this some more and played with Terminal, and I think what I said before was wrong. If you try to move all the way to the left, it'll fail when you have a vertical split in vim (:vs) or emacs (C-x 3). Better to order the moves:

1. left
2. up/down
3. right

This allows you to move from a long line to a short line (or vice versa) in vim/emacs regardless of the length of intermediate lines. You can pass through shorter lines on the way because those editors remember the column that your cursor started on. For example, to move from (1, 1) to (3, 3), go down twice and then right twice. But to go from (3, 3) to (1, 1), go left twice and then up twice. Does that make sense?
No need for space here.
Cython does use ~/.cython directory to store caches of compiled code (e.g. ~/.cython/inline), but perhaps it makes more sense to store this stuff in the profile_dir?  I don't think we should be creating any new directories outside IPYTHON_DIR.
It will be great if  "-" return "-" or "none" or some other string... because undefined give me problems with reveal.js
Can you do this modification? Thanks
Not using any json-object mapping so that isn't needed
assert hash codes unequal here and elsewhere?
this is an attribute we set on all billboards, I believe it is used by XUL to set specific permissions to the page, here is a reference of when this was added:
http://viewvc.svn.mozilla.org/vc/projects/mozilla.com/trunk/en-US/firefox/4.0beta/details/index.html?revision=81479&view=markup

maybe it's not needed today, people from the product side should decide on that since they added that attribute in the past :)
I don't mind naming them `some-thing.py` which tells the reader it's a script and not something you can import from. Might complicate matters if you need to do unit tests and run monkey patches on it. However, the mock library solves that with the patch() trick.
Please add a log here saying that this service was initiated, like most of others services do. It's really useful when debugging on dev/stage.
Use hyphenated names so it's dead obvious that it's not a python module. 

Also, perhaps consider using some sort of prefix for postgres stored procedures maybe. Like "proc-nightly-builds"
What do these vars do? They are overridden down below.
Need to change this to "(type > 1) and amark", since amark won't be set in nback-1. Or just always set amark to False above.
Python trick.. instead of doing this:
  { 'website': 'http://foo.bar', 'link': 'http://more.foo.bar/' }

I think it looks nicer to do this:
  dict(website='http://foo.bar', link='http://more.foo.bar/')

Nice work on plugging in the templates!
This doesn't need any code pushed, just an app record created.

I think options to uses_app would be nice, one for stack and make the code directory optional. Then we could re-use the helper in this test too.
Currently the Client doesn't know if the request will be secure or not, only the Request does (the URL is parsed in Request).

Also, if redirection support is added, and if it's added in Request, when redirecting from http to https for example, the original Request would need the other manager to create a new Request.
якщо boolean, то false
@stloyd: Sorry, I am not sure what lines above you mean or what exactly you would factor out :/
There should be a space after `if` and such, see [coding standards](http://symfony.com/doc/current/contributing/code/standards.html) for more details.
@meandmymonkey Thanks for making this PR. I know that a lot of time passed but could you also pass locale (`$this->getLocale()`) to the logger?
I've made a change but it is still not perfect when the default strategy is involved and an exception is thrown. But I don't know how to fix this elegantly (and the way it's done in HttpContentRenderer is far from perfect too).

The only idea I have is to create a `RequestAwareInterface` and when a service implements that, we automatically call the setRequest when we enter the request scope and remove it/reset the previous one when leave the request scope. I've not had a look at how we can implement that, but does not look easy to do that without performance impact.
this is useless as the DI extension always set it
I don't think a FormBuilderInterface would need a setParent method because it doesn't need to be acesses from outside the class. And how the hierachy is implemented can be left to the implementing developer.
BTW, if this PR is merged, a global configuration option could be set in `FrameworkBundle` to change the behaviour easily.
I used boolean for `EXCEPTIONS_OFF` and `EXCEPTIONS_AUTOTUNE` to maintain backward compatibility.

As `KernelInterface::handle` use these values as argument and as it's widely used outside symfony components, I thought it would be better.

I can change it, sure. So should I change values for every constants and introduce a BC break, or should I only change the value of `KernelInterface::EXCEPTIONS_ON` ?
I think this change should be reverted. Explicit braces are more readable than having to check what the precedence is for ``||`` and ``&&``
Not sure how that'd be possible. You can't "unhash" it to send to Gmail for authentication.

The typical solution I see to prevent all of GitHub from accessing it is to rename this file
`application.conf.example` and replace the sensitive information with dummy text.
Then add the real `application.conf` to `.gitignore`.

Now every developer will need to maintain the true `application.conf`, unless we write
a script to automatically do it. Obviously, this adds an annoying manual step, but we can't
make passwords world-visible...
add spaces before and after - ```} else {```
The deal is that this file never gets used, right?
How about doing a toLowerCase here.  That would essentially make column names case insensitive.  CQL3 does this.
libjped is also mandatory and as for Boost - maybe it would be good what boost libraries we're using, boost is very general.
Is there any reason why this information is included in the readme? It seems better described in a make-file.
Fair enough, I'll fix that
This should probably account for all non-panel children inside `this.element`. Tabs is implemented such that you can include arbitrary elements inside the container.
On Thu, Sep 27, 2012 at 7:16 PM, vdumoulin <notifications@github.com> wrote:
>
> But then if someone writes a new supervised cost function and forgets to override the constructor, it will still raise an error when the cost is called. Would it be an improvement over raising the exception at instantiation?

These things shouldn't change throughout the life of the object (or
even at creation) so I'd just go ahead and make them class-level
attributes, i.e.

```python
class Foo(object):
    supervised = False
```

And since supervised training is kind of naturally a special case of
unsupervised training (you can do unsupervised training on a dataset
with or without labels, but the case is not symmetric with supervised
training) I think it's fair to default it to False.
I am afraid..no :|
Should not turn off when airplane mode terminates if anything is turned on during airplane mode.
Out of curiosity, is it the same whitelist mechanism than WebTelephony (ie full URLs) or is it just on an origin base?
I'm checking visual images and I've discover a small discrepancy in pending message style (sometimes there is opacity of 0.6 and other is opacity 1...)  Otherwise we should take this first approach and then, after talking with UX, I will add a new style if is needed in a new pull request. Thanks Steve for the tip!
I think it would be better to give each li its own id and set their styles individually.
This will cause problems if we later add a new li that needs a different height.
Maybe we should call |holdAndAnswer()| if `handledCalls.length > 1` to handle multiple calls properly.
Is it still used, can't see it in the dom...
reuse the lock: `var lock = settings.createLock();`
What means 'm' in this case? What means 'n'? Please add explicit naming so external contributors can easily understand it.
So are we going to re-enter here so that eventually we will remember the icon? Like, we enter once while the installation is pending and then again when it's successful? It's not really clear to me from this patch. I wonder if it'd be clearer to always remember the icon and then just remove it if the user cancels.

Also, did you mean ```&&```?
Is there a issue number for that so you can link it in the code?
add a test for this guy too please
no l10n?
I only see one return statement there. Were you referring to blank lines? Perhaps you could be less ambiguous in your comments, @pdmars. :-)
Don't think so, since if you don't include the flag it doesn't do anything special
Yes, you're right. Thanks for your note !
This should read `center`, shouldn't it?
You say that these functions are private, but they actually are not.
is that bad ;)

I'll fix that
yes, if you implement named you would override getName
Why the () around result?
'A notification of how many departments have currently been migrated to GOV.UK'
ok, I'll update this comment line.
Ignore that comment
Specifically these bits.
I'd like to not lose this - it allows attribute access to go a bit faster.
What's the benefit of catching the exception here? You can also let `testSerliazibility` throw `Exception`. JUnit will print the error message just as well.
As i understand then subclassing ````Control```` in this case wouldn't work?
It seems to me that it could be using ````:unless => [:ms_uia, :win32].include?(SpecHelper.adapter)```` and moved to global window_spec.rb (delete also #child spec from win32 window_spec, if it exists.
Combine let*'s
add this to the query selector
In fact that's almost implementation of `future`. Only visible difference is that you can pass in prototype.
This test needs remained to say something about the user having a password.
As discussed previously, the interesting part here is that the *chooser* of the password controls the account once the email is verified.  So unlike our current implementation, an accidental click give the attacker *full* control. 

I think we need to make sure that unless it's the same device, the password is required to complete verification.
this isn't the first time I've made this error.  let's see if I can avoid it in the future...  :)  fixed.
As above, should delete_key have its own test?
I looked into this, and unfortunately it's not possible. You can execute the source command, but its effect is limited to a single exec task.
Sure, I'll amend the commit, if that's all right.  This is an impetus to finally set up directory-local variables in emacs, so I can keep all these project-specific settings straight.
Confirm with @nvcWilson. From what i remember deb last comments seems like the session is not require because the page will auto redirect.
What is the advantage of doing this over just using a simple subclass? As far as I can tell this just adds two new attributes to the Response.

On the other hand, I don't see how the peer_cert, and ca_cert have anything to do with the Response. I get the feeling that this is being done as a convenient vehicle for this data rather than really modeling what is going on.
Does this dump the entire catalog, or just an opaque object reference?

Is there any way you can show the actual differences - what extra and missing resources there were?
...similar to above comment. I think this should read `m < n`.
there's also the convenient `./node_modules/.bin/{hint,recess,uglifyjs}` that npm sets up
Great! Thanks for adding this!
this is just me but I stay away from overly defensive programming , I'd do:

```js
this.options.grep = 'string' == re
  ? re = new RegExp(re)
  : re;
```

same with above
No. This is horrible. ( I know you just rewrite it). Please no side efect in log meesages! It is hard to read
Why did you remove the correct alignment here?
You are right, need to be both and Iterator is more closed. I deleted this commit
I used "addIdentifiers" to keep existing identifiers.
I don't follow you :).

If $useHydrator = true/self::ARRAY_HYDRATE, it runs

    $return[] = $this->getHydrator()->extract($row);

In all other cases (false, or self::ARRAY_ROW), it runs

    $return[] = $row;

Or am I missing something?
With this here, it won't break those setting the options in the attributes. Deprecate it in the future?
Since you use .spinner-input a few times, suggest doing this.$input = this.$element.find('.spinner-input') in constructor and use this.$input throughout this file.  Then this statement may read more clearly if the .val and .attr are split into two separate statements.
yes but this is not enough that is why we run it with a script
Hm, wouldn't this have been possible to hookify? "data-tables" sounds very... table-support-plugin-specific?
Ok, I guess it makes sense. I overlooked that you can still set them independently.
does dart:html always return '' or null when it's not defined?
coool, wow, look at that awesome code. (actually this is just a fun test).
Can we add another jsfiddle demo showing how to use multiple sources. I know this won't work for the time being but maybe it would be good to add in now. If not can you file a ticket for adding this after we stage 1.3.
I don't know how others feel about this, but I prefer to spell out my method names -- I would name this `reverse_chronological`.
If SlashemGeoFields generate Term[V] queries, then Term[V] queries need to
have query-gen support for Solr.

On Thu, May 10, 2012 at 5:32 PM, Adam Alix <
reply@reply.github.com
> wrote:

> > @@ -782,28 +788,54 @@ trait SolrSchema[M <: Record[M]] extends
> SlashemSchema[M] {
> >
> >  }
> >
> > +/**
> > + * A field type for unanalyzed queries. Results in using Term[V]
> queries.
>
> Then do we want separate SlashemGeoFields for Solr / ES?  This should
> behave the same for ES / Solr with the current impl.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/foursquare/slashem/pull/34/files#r806027
>



-- 
Cell : 425-233-8271
Not completely the same, but similar yes.
(arguments are strings, not objects, you can't use position, and you use `%(keyword)s` instead of `{object}`
Application.\_\_init\_\_(self, *args, **kwargs)
no need for the ' : '
Backticks around method name
js 또는 css 내부에서 호출하는 static 파일도 모두 ? 처리를 해줘야지
파일이 변경되었을때 캐시로 인한 애로 사항이 없는 것 같습니다.

다만 문제는 html은 동적으로 생성하기 때문에 mojolicious에서
파라미터로 버전을 넘기기가 어렵지 않은데 css와 js의 경우
파일 자체가 정적으로 존재하기 때문에 해당 처리를 하려면
매번 수동으로 변경해야 한다는 번거로움이 있습니다.
또한 minified 버전이라면 수정이 어려울 수도 있습니다.

확인 결과 js와 css에서 사용하는 static 파일은 3~6개 정도 됩니다.
여기까지 수정하기는 번거로운 부분이 있으며,
일단 추측상 해당 부분은 디자인이 완전히 바뀌지 전에는
변경되지 않을 확률이 높아보여 ? 처리를 하지 않는 방향으로
진행해야 할 것 같습니다.
This one I'll move though. :D
Hm, does this need to run on every request, or could this be in ``__init__``?
This is not deadweight as we test for this functionality
It should be working replacing `process.nextTick` with the actual call. I'll PR again with more tests.
Apostrophe fixed as per comments and pushed to Github. Cross-updated in
Google Code.
-njx

On Thu, Aug 9, 2012 at 7:45 AM, sarken <notifications@github.com> wrote:

> In app/views/user_mailer/invitation_to_claim.html.erb:
>
> > @@ -29,3 +27,31 @@
> >      <li><%= link_to(work.title.html_safe, work_url(work)) %><%= (work.fandom_string.blank? ? "" : " (#{work.fandom_string})") %></li>
> >    <% end %>
> >  </ul>
> > +
> > +<p>
> > +  If these works <b>don't</b> belong to you, please don't delete them! Copy the above links and  <a href="http://opendoors.transformativeworks.org/contact/open%20doors">contact Open Doors</a> so that we can move them for safekeeping.
> > +</p>
> > +
> > +<p>
> > +  If these works do belong to you, but you don’t want them, you can claim them and then orphan or delete them. (For assistance, please <a href="http://archiveofourown.org/support">contact Support</a>.)
>
> "but you don’t want them" Can you change that from a smart/curly
> apostrophe to a straight apostrophe?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/otwcode/otwarchive/pull/786/files#r1339250>.
>
>
Refactored to have fewer `puts` statements, remove the namespace (unnecessary and hard to type), extracted the redundant export code to a method.
this whole block needs to come back in by 2 spaces and a blank line above it would be nice too.

well done on copying the prevailing coding style overall though!
I'd rather get rid of it, yes. It seems specific to your way of working: I have my *liq tests one level higher, or in a separate directory altogether.
Removed pry from the test.rb
Currently this is hard-coded with a Mac master. I can introduce an ENV_PLATFORM parameter, but when the job is triggered it will use whatever the default parameter value is. This would effectively be the same as hard-coding it.
done

On Sun, Jun 24, 2012 at 9:21 PM, Simon Kornblith
<reply@reply.github.com>
wrote:
>>       "maxVersion": "",
>>       "priority": 100,
>>       "inRepository": true,
>>       "translatorType": 4,
>> -     "browserSupport": "gcsibv",
>> -     "lastUpdated": "2012-06-13 15:07:20"
>> +     "browserSupport": "gcsib",
>
> This should just be gcsb
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/zotero/translators/pull/401/files#r1039816



-- 
------
Sebastian Karcher
Ph.D. Candidate
Department of Political Science
Northwestern University
If you add fixed: true, it doesn't create the mouseTarget.
These hints are really good for explaining to a student how to _do_ the problem, but we also usually like to add explanations of _why_ they're doing this. How you want to explain that, if at all, is really up to you, but just a suggestion.
I would prefer the connector SPI uses the Document class rather than the (internal-to-Schematic) BasicDocument. Hopefully that's possible; if not, we should talk about what might prevent doing this. 

Also, do you think it's feasible that we could completely hide the "metadata"?
This method is a bit crazy. Since it is changing a bit, we can make it much better:

* It is reasonably complicated and has no documentation. We use [Yardoc](http://rubydoc.info/docs/yard/file/docs/GettingStarted.md) to document our code. The key parts would be a small example of input / output.
* We can use the Enumerable method [Inject](http://ruby-doc.org/core-1.9.3/Enumerable.html#method-i-inject) to pass an empty Hash `parsed_params` to our iteration.
* This code using index and Ranges seems a bit strange. Could we not use `metadata_key, metadata_value = param.split(":")`?
* The names of the variables `c1, c2, key,  value` are pretty bad. They are very generic and don't help explain the code. In particular, `c1, c2` are especially bad, but I think they can be completely removed.
* Maybe the conditional on line 273 can be changed. If we remove `c1`, it gets better.
* The else condition should just raise the `N3ParameterMalformedException` and I assume we should not be able to recover from it?
We’re [not using squiggly versions](https://github.com/alphagov/styleguides/blob/ecc6de0ac864316a7366ca95227e53f3be640064/ruby.md) at the moment, to avoid unintended upgrading.
Hey Christian, Mike and I worked on this PR together this morning. The problem we had was that in a package install it's in `/var/run/redis.pid` and the source-built init.d script it's in `/var/run/redis_#{port}.pid`. It did feel a bit hacky to do it this way but we weren't sure if we'd have access to the expanded run list to make that decision in the attributes file. I agree, it should be in the attributes, what do you think?
You inherit this - no need to repeat the version number (and worse: The day you upgrade to a new version you have to update all these pom files!).
Changed to allow empty strings in the displayName property. It will still default to field name if displayName is null or undefined.
either way works.  I prefer settings in code as well
Ah yeah, good call. I think originally I was thinking that having information in sub-dictionaries would help "future-proof" it, but since the call is "hwinfo" .. returning a dictionary "hwinfo" seems a bit redundant. I'll remove it.
That makes more sense to me. I will make this change.
Gettin' fancy with teh grammerz.
In general, I think that's a good idea, but it would also be easy to do as a separate ticket. Where do we say we support 'popular'?
I don't think there is any reason to iterate all the results / values, right? this whole block if (results) {....} can be destroyed I think.
would it be easier to stringify and store extra as a string, rather than breaking it out into keys like this?
Add yourself to the contributors list in the pom.
Should be `$this->` because the method is not static `private function getLimitedTableNameWithoutPrefix($table)`
Ik zou deze 2 gewoon in de if statement boven doen. Je kan dan doen:

```js
if (detected_collision) {
  this.flip = !this.flip; 
  this.accel.x = -this.accel.x;
}
```

Right?
No private methods please (protected is good) - this makes it impossible to extend parts of the core.
Also, why does this method need to be separated if it's only used by `_parse_request_uri()`?
:) на вкус и цвет товарища как говорится... На мой взгляд не очень читаемо именно в таком применении (котроль границ с двух сторон), мне даже пришлось какое то время втыкать проверяя равенство результатов. Тем более, что эти макросы развернуться практически в такую же конструкцию как у меня.
вероятность чего?
буду благодарен, если дашь пояснение, я не вижу разницы между выборкой из 4-х элементов или из 100 по 25
On 04.11.2012, at 23:09, VGPETROV <notifications@github.com> wrote:

> In DNAProject/DNAProject/DNAProject/Cell.m:
> 
> > +//
> > +
> > +#import "Cell.h"
> > +#include <stdlib.h>
> > +
> > +const int ARRAY_COUNT = 100;
> > +
> > +@implementation Cell
> > +
> > +-(id) init {
> > +    self = [super init];
> > +    if (self) {
> > +        _array = [[NSMutableArray alloc] init];
> > +        for (int i=0; i<ARRAY_COUNT; i++) {
> > +            int r = arc4random() % 100;
> > +            if (r>=0 && r<25) {
> Для увиеличенной вероятности, в данном случае 100 Sun, 04 Nov 2012 11:00:56 -0800 от s1dney <notifications@github.com>:
> In DNAProject/DNAProject/DNAProject/Cell.m: > +// + +#import "Cell.h" +#include <stdlib.h> + +const int ARRAY_COUNT = 100; + +@implementation Cell + +-(id) init { + self = [super init]; + if (self) { + _array = [[NSMutableArray alloc] init]; + for (int i=0; i<ARRAY_COUNT; i++) { + int r = arc4random() % 100; + if (r>=0 && r<25) { забавно, а почему именно arc4random() %100 ? почему бы не сделать просто %4 и выбирать символ из 0 1 2 3?
> if (r == 0) {
> _array addObject:@"A"];
> }....
> —
> Reply to this email directly or view it on GitHub.
> —
> Reply to this email directly or view it on GitHub.
> 
>
@nanotears интересно! Спросил, потому что нет пока возможности проверить в Xcode. В Delphi, например, названия переменных тоже критичны.
@crmaxx о чём я и говорю, каждый видит что-то своё. Если "заменить" и "обновить" - это одно и то же, зачем два разных слова?
Их таких много: http://stackoverflow.com/questions/2865190/how-to-round-an-float-value-and-convert-it-into-nsinteger-value-in-the-iphone-sd
Этот метод есть у "родителя" (NSObject), так что его не нужно определять в интерфейсе. Достаточно просто сделать свою имплементацию (реализацию) этого метода, тогда мы "перезапишем" метод для нашего объекта.
hey, mike and I were talking about the routes and we think that we should get this fixed merged now, and changed all the url_fors in another branch. What do you think?
This modification shouldn't be in this patch :)
As far as I know the solution actually exists already; you can override the url that is used by setting it in the prefs file for the app, and you can figure out if you are in a stable|RC|Beta build of the app by including and examining the values in `autorevision.h` (fairly certain that I added that to the xcode project).

On 2012/12/29, at 5:17 PM, Barijaona Ramaholimihaso wrote:

> In configs/scripts/Sparkle-setup.sh:
> 
> > @@ -0,0 +1,24 @@
> > +#!/bin/sh
> I am pretty sure you will find a good solution soon, anyway ! 
> 
> —
> Reply to this email directly or view it on GitHub.
> 
> 


--------------------------------------------------------------------------
My Web Sites:
http://dak180.users.sourceforge.net/
single quotes
space between operators
I'm unsure whether I like this change from a user perspective.
q is not a descriptive variable name.
Can we call this pan_x_margin to have better idea in rest of the code how it's different from pan_x ?
I am trying with simple signals like

    >>> t = np.linspace(-.1, .6, 700)
    >>> y = np.zeros(700)
    >>> y[200:400] = np.hanning(200)
    >>> y2 = y + (np.sin(t * 60))

What is the proper filter order for  `sp.signal.decimate()`? The default shifts the signal in time for small q (e.g., 10), and returns a flat line for higher qs (e.g., 20).

`sp.signal.resample()` seems to shorten the signal (e.g. I have a signal that's -100 to 600 ms and `resample` to 7 time points, the signal now ends at 500 ms).