labofsahil/github-event-dataset-2012 huggingface dataset:
Put these two if statements into one. `if ( that.focus !== undefined && item.index !== that.focus  ) {`.

Also here a comment explaining the purpose would be nice.
Since this is an API, we should avoid it changing whenever we change implementation details. I'd recommend using a generic key and hardcoding the array and copying the properties (so that it stays the same regardless of the implementation). Again, could be revisited later as this was already this way.
in previous versions, $body was indeed a reference to document.body. but since integrating in Gaia, it was changed to reference the Evme container- like the $container var.

removed it, and we're now using only our main $container.
I can't review this without more explanation...

the lastModifiedDate = null is a known device storage bug, and a fix is pending, I think.

If we make the change you propose here, where will we handle these files with no date?  Are you suggesting that if a no-date file has the same name and size as a known existing file we treat it as an existing file rather than a new file?  

Don't you need parens around teh || expression, though? I think your code doesn't test size if the date is null
ModalDialog -> this (and this applies to a lot of places)
As I said in a previous comment. Don't use an activity here. The activity use to launch the homescreen will be removed and there won't be a way to start an activity without a user interaction so it won't work.
`enabled: true,`
If there are any, I'd just fix them in this patch.  Note that null is probably okay, as long as we don't try to call getTime() on it.
var [ ... ] and remove the declaration above.
Let's fix that in a followup.
mapa? porque! Para favor use a english name ;)
I don't fully understand this code.  Is the point that it waits 'till the screenshot is actually drawn on screen before beginning the CSS transition?  A comment would be helpful, I guess.
Ugh, I really thought it was a mistake because that’s not how it looks on the [web page](http://mozilla-b2g.github.com/Gaia-UI-Building-Blocks/index.html#widgets/buttons_list/), and I don’t really get the logic…

Long story short, do you think all buttons in the Settings app should be uppercase?
animations are unprefixed since firefox 16 (it turns stable today), I don't know which version of Gecko we use now.
I think this conditional is a vestige of the older version `case` statement and can be safely set all the time now.
Sorry, I want to say these states are not used in the debian model (perhaps due to a spelling mistake).

With or without my changes apt_get* are still calling back but without a matching state for each action.
The source_fix state doesn't have a sens because it change things related to ubuntu so I have removed it from the debian model.

Perhaps I can clean the patch by keeping the apt_get* state, and just correct the fsm tree to have the same spelling than the postinstall script
And remove only the source_fix stuff.
It's comparing the field name to the schema so there's no need to do that here.
> I'd prefer to avoid this as I've seen this pattern cause lots of confusion in the past.

Are you referring to `extend self`?  It's one of my favorite ruby idioms and I use it frequently.  What confusion have you seen it create?

Anyhow, I think it makes sense to put these methods in a module named `BacktraceFormatter`, as that'll be more intention-revealing when rspec-expectations uses it.
#items is private indeed but we need to check that the items have been properly set (especially) there status
e8495db
maybe `.join('\n')`?
Hapi.utils.version();
Maybe ```theme('checkbox', ...```
This could be shorter `parse_url($path) ?: array()`
Actually it's `foolfuuka` but whatever.
Pull this and the rest of your CSS in to a separate stylesheet.
Why not make annotated config an interface with load and save methods, a separate class that has implementations of load/save and the field cache, AnnotatedObjectConfiguration to manage them all, and a new class (or AnnotatedObjectConfiguration reuse) that replaces the current AnnotatedConfiguration.
You didn't close this tag, that's why you had a blank screen.
this could just be elseif + elseif instead of nesting it into the else block.
Yeah, why not? No idea.
*error handling:* there is a specific "_errors.createInvalidReqMissingCommandParameterEH(req)" to describe an error that has a missing parameter (in this case, the "file").

*var declaration:* It's mostly a style thing: I'd like to maintain consistency across the whole codebase.
And I noticed that you did a great job, even without any actually written code style convention (well done so far!).
WebSockets、 XML transport、 Local Storageなどの異なる永続化の方法を利用する場合はこの関数を置き換えることができます。
I don't know if this is intentional or not, but I would like to see more parameters, like `wiki_enabled`, etc.
This is wrong (as in morally wrong ;) ). You shouldn't be setting any @variable from the view layer (helpers belong to the view).
We want to move the sched page to only scheduling components. But here if we get rid of the page, the system won't load. You helped me to work this around this summer. I still don't know the source of this, but added comment in the code.
Doesn't the Die message appear in the error_log?
What if I added hash support to to_event, so you could: to_event(entry, @sitename) and you'd get a LogStash::Event based on your hash?
I've added a singleLine argument which is trumped by returnEvent.
the same like in question
Huh, that seems to work but I don't understand how. The docs say the sort function is meant to return a negative or positive number but this is boolean. Is it just chance that it works or am I missing something?
No call example, I presume it is just `evaluate(options)`?
I think there is a misunderstanding, this feature is not released yet so we're not going to break any add-on. Currently "*" for `contentScript*` are already limited to http/https/ftp, this patch makes `contentStyle*` consistent with that rule.
ok, it was working because I changed client code, so while CompleteRegistration's __init__ called the argument `mozwebqa`, it was actually a selenium object.  The fix is to adjust both client code and this call to `Base.__init__()` - running tests now and will add commit.
@callahad and I agreed that if `authority` is present and not parsable, an implementation should consider the domain to not have support.  A verifier should ignore and verify assertions IFF `public-key` is present.  This asymmetry is important for zero user impact during transitions.
Can this lead to doubly escaped domains ever? Are emails already escaped when they are entered?
__whimboo__ commented:
> Mind adding a comment which explains what are doing here? ...

[SeaMonkey Debug and QA UI][1] addon is a default distributed addon on nightly and aurora channel:
> All nightly builds currently include the Debug and QA UI extension. For release builds you can install [extension from AMO][2]. 


It have a lot of feature as You can see on the wiki, one of them is modifying the title. See [my comment][3] above for pictures!

I thought `titlebar-customization` should provide the same result regardless how this addon is installed or not.
DebugQA modifies the original `titlemodifier` (==`SeaMonkey` or empty on Mac) to `SeaMonkey {Build ID: 20121108013004}` or like, and messes `defaultTitle` up.

These lines do checks like: _debugQA addon is installed/enabled?_, _Current `titlemodifier` seems like debugQA's `titlemodifier`?_ and set a helper wariable (`debugQATitleModifierWorkaround`), which is used in title calculation (`getWindowTitleForNavigator()`).



[1]: https://wiki.mozilla.org/SeaMonkey:Debug_And_QA_UI
[2]: https://addons.mozilla.org/addon/212342/
[3]: https://github.com/mozilla/nightlytt/pull/104#issuecomment-9831511 "2012-10-26 18:33:59"
I wanted to keep close the implementation to original sources (`tabbrowser`'s `updateTitlebar()` and [`getWindowTitleForBrowser()`][1]), therefore I did only the necessary modifications: `this` -> `aBrowser`, stripping the comments and the debugQA workaround stuff.

Let this `nightlyApp.getWindowTitleForNavigator()` along with `nightlyApp.getWindowTitleForMessenger()` keep close to original implementations, please.


[1]: http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabbrowser.xml?rev=9379c0915c6a#775 "Firefox has a nice titlebar calculator"
Just re-factored all this shit.  Defined `Muddle::Filters` with methods to cast to the expected format (`as_nokogiri`, `as_string`), so now the filters return whatever they're using and just make sure they cast to the intended format first.
@joshcooper does that really matter? we're stubbing it at line 66 and incorporating it at line 67. We're not using what the true value would be. The real value could be `C:\My Awesome Directory` and we wouldn't try to match because of the stub.
If you created a Puppet::Settings::DurationSetting (along the lines of the other setting types), all of the scattered logic around parsing the setting into a duration of seconds would be consolidated into a single spot.
@zaphod42 see 7b4ac10
copy/paste monster
We now use pidfile named "pid" inside of lockdir .bitpocket/tmp/lock/ (like in above line that was removed).
Instead of another set of custom colors for these, we need to be making use of global colors across the board. Would be interested in seeing that get fleshed out.
The `dict()` thins is able to take an iterator so you don't need the `[` and `]`.
this should be more like the reporter implementation:

```js

Mocha.prototype.reporter = function(name){
  name = name || 'dot';
  this._reporter = require('./reporters/' + name);
  if (!this._reporter) throw new Error('invalid reporter "' + name + '"');
  return this;
};
```
why srf instead of comprehensible surf?
Will look into this later today or tomorrow.
the underscore must be removed
This part can be optimized (in a different PR) doing array_search directly
you have either normalize strings (to lowercase or to uppercase) or use ILIKE
> So, I added the dot since @forsetti mentioned some tickets are generated of the form ST-1234-login.example.com.

Oh, okay. Never seen those myself but that sounds reasonable.

> Will add a commit soon with our own local implementation which checks for [a-zA-Z0-9] via isupper(), islower() and isdgit() which seem not to rely on locale.

`isupper()` and `islower()` are locale dependent, too. Just check for `(c >= '0' && c <= '9') || (c >= 'A' && c <= 'Z') || (c >= 'a' && c <= 'z')` and let's pretend EBCDIC doesn't exist.
I actually checked this a bunch, and apparently this is now needed. It might be a bug in brew though.
It's not really true...

Our system currently is monitoring more then once a day, and score could be updated quite few times in even few hours. There is a PR to adjust saving of score updates, but as said, it's not once a day, and bundle don't need to trigger any update on our side, we check it on our own... of course AFAIR =)
Can't do fire-and-forget. We need to know if the index is ready to be queried. I think there are ways to do this, but I'd rather have it be slightly slower than have the query service be aware of this.
You need to pluralize the `item` when there are 0 or >1 items.

To minimize ID use, can you use something like `#todo-count strong` instead?
Needs IIFE wrapping: `(function( window ) {` and `use strict`. You should consider namespacing the Todo* vars.

This applies to all the /app files.
Tu pourrais ajouter
```ruby
true # true to return a temp file, false to return the ZipOutputStream
```
Not sure how to handle that, basically both operations are possible in one flush, but as @stof pointed out there is only one state for the document. I rhink the best is to create some tests and look at UnitOfWork again.

Am 01.09.2012 um 15:41 schrieb David Buchmann <notifications@github.com>:

> In lib/Doctrine/ODM/PHPCR/UnitOfWork.php:
> 
> > @@ -631,6 +644,29 @@ public function scheduleMove($document, $targetPath)
> >          $this->setDocumentState($oid, self::STATE_MOVED);
> >      }
> >  
> > +    public function scheduleReorder($document, $srcName, $targetName, $before)
> > +    {
> > +        $oid = spl_object_hash($document);
> > +
> > +        $state = $this->getDocumentState($document);
> > +        switch ($state) {
> what exactly does this mean? if i move a document somewhere else and then (without flush) reorder it, it won't be moved anymore? and the same for creating and removing?
> 
> —
> Reply to this email directly or view it on GitHub.
>
We probably don't need these first two clauses.
Ok, I understand. But why do you use **encode('ascii', errors="xmlcharrefreplace")**?
More logically to use  **encode('utf-8')**.
Then symbols like "Ø" will not replace.
I'll rebuild with packed-groups set to True, and see whether this fixes this issue.
/me agrees with fred. No free-floating trunk requirements.
Space here, please: #main-feature > h2
I think the default way I want to use this is as follows:
```python
u = user()
add_profilepic(self.client, u.get_profile())
```
Either way, I think what you need to create a profile is a client and a user.
Is there anything we don't want in Open Sans? Does it make more sense to apply this to the body and except other selectors that *shouldn't* be Open Sans?
Is this needed? I would expect all files to be included by default (excluding those explicitly listed below).
this might present a small issue with smartvars, as they are normally resolved by their string representation.

need to rethink this - @abenari any ideas?
You missed an 'd' at the end of the job name.
any reason for the no-op done handler?
It's generally a good idea to only call toFixed right when you're about to use it, not when creating it (so it stays as a number for as long as possible, instead of a string)
Sure, it's definitely doable. For basic CSS colors I could do this without any extra code, by just adding an extra random variable. ("blue", "red", "orange"). 

If you want a more sophisticated solution, I could add a library function that would let us define our own colors, or tweak their appearance. (Make "blue" refer to "#369" instead of "#00e", for example)
I looked at custom answer types in the wiki and it didn't seem like the appropriate solution, so I added a hexadecimal type in answer-types instead and that seems to work. (And that way it can be re-used in other hex exercises)
In it's current form it's a bit of a hack, yes. :-) I'm not too fond of storing the complete `GermanJobRequest` in a DB - there will be use cases where you want to simply poll a server (or a collection of servers) for the state of a job (from the CLI, for example) and don't have access to the complete `GearmanJobRequest` object.

Since an handle isn't unique to a host in Gearman (as duplicate local host names will generate the same handles across servers), I decided to use the server address provided to the client itself to inidicate which Gearman server to query for the status of the task. The idea (if I remember correctly) was to allow a client user to query the status of a task by creating a client with the gearman server to query, and then request the status of a handle from that server.

Polling all the connections are a possibility, but might return the wrong task if you have several servers with the same local host name - so either we're talking about a documentation issue (by polling all the connections) or a code issue (by only allowing the user of the library to set which server to poll). The last case solves the issue where you could store the (server, handle) pair in a DB and then request the job status at some point in time. I'm open for both, as long as the limitations are clear for the user.
It's something the appraisal gem does.
Nope, that doesn't appear to work.
Should probably have a file header, especially for license.
Here we sign with the tenant of the principal, but in /api/download/signed we verify with the principal of the current user, which will result in invalid signatures for cross-tenant access.

Signing with the principal's tenant will improve caching, however we would need a custom download endpoint that knows to verify using the target principal's tenant and not the current user's tenant. On the other hand, by moving this up to the API we can just always sign using the current user's tenant. TBH, I think just always signing w/ the current user's tenant should be OK.
You probably want to use the isGroup function for this.
I'd prefer `for (var t = 0; t < _.keys(tenants).length; t++) {` here and throughout the PR.
eager copy pasting ;-)
Any reason why catching ``Exception`` would not be enough?
I would get rid of the `ClassOf` unapply :). To me, it adds one layer of indirection for no real gain: it's private, can be used only from this call site (see your own warning in the comment) and the same could be achieved by writing:

    for (AnnotatinInfo(_, List(Literal(Constant(typeName: Type))), _) <- throwAnnotations) 
      yield mapType(typeName).toCharArray
self.object = nil does release it from its property attributes.
object = nil does nothing.

I remember freeing retained object in -dealloc is suggested convention.
` ! empty($index)` should be better than `$index === NULL`, to filter out bogus keys such as '0'. It also might be a better idea to return FALSE or NULL for non-existing keys.
Коректней будет вывалиться с ошибкой :)
исправил все в последнем коммите, нуклеотидеы сделал через метод - (NSArray*) possibleNucleotides
для переменной DNA переписал сеттер, чтобы невозможно было переопределить переменную извне, возможно сделал это немного криво :)
с extern пробовал разобраться, но не смог пока.
правда такой подход мало что даст если все равно нужно использовать NSMutableArray, в который char просто так не поместить, на сколько я знаю.
It's not used anymore. At the moment, user preferences are stored in permanent cookies (which persist across sessions) so they'll only lose their user preferences if they change PC/browser. In my opinion, both approaches are viable. 

For the moment, I don't think those kind of settings should be stored in a table, but eventually, if we have more important user preferences (major ones), it might be something worth looking into.
Use LinkHandler!
`Package::_findIndex()` uses it: https://github.com/cakephp/cakepackages/blob/master/Model/Package.php#L338
What if it set up the timer on a global concurrent queue, but automatically `deliverOn:`'d to the `RACScheduler` on which the subscription occurred?

In other words, this would keep delivery on the main thread when `+interval:` is called from there, but allows you to customize that through the use of `subscribeOn:`.

If we do this, we should document it very clearly.
:cat: :hankey:
Any reason this can't be on the widget? (nitpick)
same here

```
self.info['bads'] = [name for k, name in enumerate(bad_channels) if isthere[k]]
```
should be

param : type

so 

```
info : dict
    Measurement info (e.g. raw.info)
```
Don't need to set `finishedCallback`
I guess 'cause it's fun.

I'm wondering if there is a good way that users can add custom zsh themes without forking. Maybe that would be more personally interesting to me, seeing as I'm in this for myself and just sharing because I like giving back.
This is misleading. ``validation_constraint`` was also applying on the field directly. The difference is that it was impossible to add several constraints
your use statement is about ContainerBuilder
它 => nginx
There is still a pending dillemma if this should appear in the stdlib at all. Some say yes, some say no (they want to use Either in futures instead of this).

Once it becomes part of the nightly, there will be more loud voices and more clear opinions.
Are you on Python 2.7 or 3.x? There deprecation warnings are turned off by default. Start your interpreter with `-Wd` (IIRC) to make them show up.
нет пользователей - некому выводить
Если тебя не сильно парит совместимость с Qt < 4.8, можно написать `return { TerminalTC_ };`
if use session is false, aren't we defaulting this to false, not true?
According to the Symfony CS `else` must be in the same line as the closing brace.

http://symfony.com/doc/current/contributing/code/standards.html
https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md#51-if-elseif-else
```(if envs (env/create-path test-org envs))``` works fine.  The only time it is any different than ```(if (not (nil? envs)...)``` is if envs is ```false```.  Which I doubt will ever be the case here.
Inverse the test
Should add a patch as it introduces BC break
Won't this fail anyway, regardless of all the extra checks you did in the class?
Doing that now. In fact, I don't even need this method with the deleted flag since I can filter easier.
Leaving it that way means one more request every time the user refreshes the page just for an (in most cases) empty stylesheet?

This is a really bad idea. Please use the asset pipeline for that.

see https://github.com/magiclabs/alchemy_cms/blob/master/app/assets/stylesheets/alchemy/alchemy.css
Don't do asserts for checking user inputs.  Raise a ValueError instead.
Thanks a lot, it work fine :)
Минут десять вчера убила, форматируя его так и сяк, и сдалась. Ладно, попробую ещё раз.
>> +    $req = WWW::Curl::Simple::Request->new(simple_ua => $self, request => $req);
> 
> I am not sure, but might this lead to a leak?

Yep! I've now weakened this reference, in the version I just pushed to github.
There's no way to get to the function at the moment without a mouse.  I haven't gotten this working yet actually.  Also the language needs to be set properly.
In your opinion, how many projects will do the effort to comply to this rule, when they're using spaces in their master/dev/fix/feature branches, and while their PR are using spaces?
This PSR shouldn't force any change in existing projects, just state the current trend, and leave the debate for their own coding style.
Yes. I'll have to think about what is the best set of trade offs long term. But I think if I make the auto-glossary thing not be case sensitive we can have something that works today. I'm looking into that now.
nil can't happen because of ?float-val, and == doesn't care about types --
but sure, sounds good, put it in its own function.

On Mon, Jul 23, 2012 at 2:12 PM, Dan <
reply@reply.github.com
> wrote:

> > @@ -182,7 +182,7 @@
> >          mod-coords ["?mod-h" "?mod-v" "?sample" "?line"]]
> >      (<- [?dataset ?m-res ?t-res !date ?mod-h ?mod-v ?sample ?line ?val]
> >          (rain-vals !date ?row ?col ?float-val)
> > -        (not= ?float-val nodata)
> > +        (== ?float-val nodata :> false)
>
> We are screening out *bad* values -- right?  A useful function, generally,
> would be a wrapper that checked for `nil` or `-9999` and its type variants
> (floats, ints, etc.).  Maybe something like `valid-data?` which is easier
> to follow and more generally useful.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/reddmetrics/forma-clj/pull/176/files#r1219697
>



-- 
Sam Ritchie, Twitter Inc
703.662.1337
@sritchie09

(Too brief? Here's why! http://emailcharter.org)
not sure what this does... docstrings, please.
I don't really mind it here, but I also want to note that the example doesn't really help much, since the user won't have "ndvi" available when they're calling the function .

I'm a little confused by the docstring, actually -- the docstring needs to help the programmer decide what goes in and what comes out. How to use the damned function. When I see "deseasonalized vector", it really doesn't mean anything to me... a vector's a list of numbers! What does that have to do with the seasons/ And "frequency of the data in the supplied time series"... what does that mean, frequency of the data?

Maybe something like

"Accepts an evenly-spaced, time-ordered sequence of measurements (`ts`) and a count of measurements / year (`freq`) and returns a sequence of measurements of the same length smoothed using the deseasonalization technique found at <some url>."
I recommend this:

    unless => "test -f ${plugin_dir}/${name}.hpi || test -f ${plugin_dir}/${name}.jpi"
@apex13 It seems like these are only whitespace changes, can we avoid these?
I think Hilary has switched to just returning milliseconds since epoch, so as long as that one and JS dates are supported, we should be fine.
It's for constructor (type-3) injection, which makes the component re-usable in other non-OSGi IOC containers. But it is also very useful for unit testing :)
There is a similar method in either DocMigrator or PageMigrator that might also need this same change.  L
This not-really-an-open-comment is asking for trouble.
I checked irc logs - you were asking about differences of SGMatrix and SGVector. There is some misunderstanding here - I meant it is ok to use referenced instance of SGMatrix/Vector here, not pointer.
What does `$2y$` mean? http://php.net/crypt talks about using `$2a$`. Perhaps you could write a line to document what this does, as it's not very clear from just reading the code?
You could create a couple of variables with clear names and use them in
filter callback, for example.

On Sat, Sep 1, 2012 at 3:22 PM, Kirill Dmitrenko
<notifications@github.com>wrote:

> In src/event-manager.js:
>
> >
> > -        if (eventCallbacks) {
> > -            this._eventsCallbacks[eventName] = eventCallbacks
> > -                .filter(function (c) {
> > -                    return !!callback &&
> > -                        !(callback === c.fn && (!ctx || ctx === c.ctx));
> > -                });
> > -        }
> > +        eventName.forEach(function (singleEventName) {
> > +            var eventCallbacks = this[callbacksStgName][singleEventName];
> > +
> > +            if (eventCallbacks) {
> > +                this[callbacksStgName][singleEventName] = eventCallbacks
> > +                    .filter(function (c) {
> > +                        return !!callback &&
> > +                            !(callback === c.fn && (!ctx || ctx === c.ctx));
>
> No, to be honest. How we can make it more understandable?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/simple-map/simple-map/pull/4/files#r1512123>.
>
>
In bad conditions update could be delayed of almost 2 seconds.
No I belive that as of rspec 1.8 that line is dropped, presumably because
it is defaulting to rspec for mocking.
On Apr 1, 2012 11:07 PM, "Dan Croak" <
reply@reply.github.com>
wrote:

> > @@ -80,7 +80,7 @@ def configure_action_mailer
> >
> >      def generate_rspec
> >        generate "rspec:install"
> > -      replace_in_file "spec/spec_helper.rb", "mock_with :rspec",
> "mock_with :mocha"
> > +      replace_in_file "spec/spec_helper.rb", "# config.mock_with
> :mocha", "config.mock_with :mocha"
>
> Is there still a `mock_with :rspec` line around with this change?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/thoughtbot/suspenders/pull/76/files#r631446
>
Two is a number I guessed at. I should move the above magic numbers to be defined constants in the header.
Nice catch! I added a #.  There is no need to return false since there is some javascript code binded on this tag anyway.
the string module has a function called fmt.
``$GLOBALS`` does not seem to conform to the naming convention (``^[a-z][a-zA-Z0-9]*$``). Please also check line 29.
Hey, you're the author, not me :)
Maybe put this in a separate helper class instead?
I believe we are prefixing all bundles with cfdc, I assume this would apply to body parts as well, but I am unsure if we should be prefixing the entire sketch name with cfdc.
right. now i see :) didn't notice that you pass the actual url to that method!
No need to default `context` here since it's done for us in `trigger`.
What is the purpose of this method?  Obviously setting "values" but to what?
`count` can be an expensive operation, so `(pred (first (filter pred a-seq)))` would be a better alternative. `first` returns `nil` if the collection is empty, and `pred` makes sure that the right boolean value is returned (for example if we did `(my-some nil? [1 nil 2])`).
Does that look better @ohadlevy ?
It is only eight digit number. Here is example of response:
```
response: {
      user: {
            id: "28727544"
            firstName: "FirstName"
            lastName: "LastName"
            relationship: "self"
            photo: "https://foursquare.com/img/blank_boy.png" 
      }

      ....
      
      contact: {
            email: "example@example.com"
      }

      ....

}
```
This could be done simply by adding `.*`, so to ignore all hidden files and directories.
OK, but i dont know how to get right url in html file: from this

{% static "piplmesh/js/registration.js" %}

to what?

I try: /static/admin/iplmesh/js/registration.js but admin doesnt exits
In get you should just prefill the form from GET and invite user to press a button to make a POST on the form.
Probably in MongoDB data directory. I removed those two lines ;)
Aaa. What you did. Aaa. There are different places. Some are assignments, then operators. And keyword arguments. Those last are without spaces. Otherwise we have them!
Let's avoid making `zd` references in this repo :)
OK Thanks for the feedback, I'm new to creating formulas.
What are common things to test otherwise?

On Feb 22, 2012, at 8:46 PM, Jack Nagel <reply@reply.github.com> wrote:

>> +    system "otool -f /usr/local/lib/libfcgi++.la"
>> +    system "otool -f /usr/local/lib/libfcgi.0.0.0.dylib"
>> +    system "otool -f /usr/local/lib/libfcgi.0.dylib"
>> +    system "otool -f /usr/local/lib/libfcgi.dylib"
>> +    system "otool -f /usr/local/lib/libfcgi.a"
>> +    system "otool -f /usr/local/lib/libfcgi.la"
>> +    # Verify if headers are installed
>> +    system "test -e /usr/local/include/fastcgi.h"
>> +    system "test -e /usr/local/include/fastcgi.h"
>> +    system "test -e /usr/local/include/fastcgi.h"
>> +    system "test -e /usr/local/include/fcgi_stdi"
>> +    system "test -e /usr/local/include/fcgiapp.h"
>> +    system "test -e /usr/local/include/fcgimisc.h"
>> +    system "test -e /usr/local/include/fcgio.h"
>> +    system "test -e /usr/local/include/fcgios.h"
>> +    system "test -e /usr/local/include/fcgi_config.h"
> 
> Can't hardcode /usr/local (has to be HOMEBREW_PREFIX), the otool and test stuff is unnecessary.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/mxcl/homebrew/pull/10398/files#r477291
Syntax error. At least run your shit before submitting a pull request.
Так тоже :)
Renaming formulae doesn't really work in homebrew right now - sorry. We'll need to keep it as `stone-soup` for the time being.
@superdweebie that is surely a better approach, but reduces the scope of this adapter for those who have already defined entities/documents (plus it makes their entities depend on the module).
We could provide it as optional, but it wouldn't help that much.
I would use a property, if your intent is to hide the implementation of summary and allow the updates only by the #update_summary. So the user can do  an_importer.summary
You're not testing the indexing here, i.e. at a certain index you get an expected value. What about http://pastebin.com/6XYKnK1w ?
May want to add dependencyManagement and pluginManagement sections so we can standardize Maven plugin versions and centralize dependency versions.
I think errors are severe enough that the JVM should explode. A genuine bug in Kiji should still probably terminate with an exception and a full stack trace for developers. (Even the JVM itself, when it completely borks out, writes a stacktrace to a file and asks you to upload it to Oracle.)

We should only translate errors caused by user behavior into friendly error messages... otherwise I think we risk masking a serious bug in Kiji that we would like a user to report with a cryptic short message that they just ignore.

But if a user specifies a path that doesn't exist, it should print "file not found: foo.txt" rather than the entire stacktrace of a FileNotFoundException.
Мелочь, но эту строчку нужно вынести за try.
Потому что есть signa.signal вывалится с ошибкой то в finally возвращать старый обработчик не нужно (он и так стоит), да и не из чего (handler еще не присвоет и просто не существует).
Аналогично, например, нужно делать с блокировками без with:

```
l = RLock()
# ....
l.acquire()
try:
  do_something()
finally:
  l.release()
```
One var per line.
maybe also include the offending values(s) here in the returned string, to make the emergency message that nginx will output as clear as possible?
I think I'd change these to int and cast `m_children.size()` to int for the multiplications. Choosing integer types in C++ is always a pain, but I think in general it's better to pick storage types so they make sense and then cast as necessary for arithmetic. Of course, the cast will break if we get over 2 billion children, but I don't *think* that's really a situation we're aiming to support anyway.
Moved check to super.
It looks like it's so we can use bincount.
When you construct a range like that Ruby uses [`#next`](http://www.ruby-doc.org/core-1.9.3/Fixnum.html#method-i-succ) method to get the next element. For numbers this method returns `self + 1`. As you can't reach `-5` moving that way from `-3`, `(-3..-5)` generates an empty range.

Though, you can use `-3.downto(-5).each { ... }` if you wish.
Here's the intialisation of a cache, but I don't see the place where smth is written into it. Can you, please, point it out?
Renamed L_ZVAL_P to Z_LVAL_P as per method override above. Make was throwing errors without this change - assuming this was a typo.
InitializeExperimentsTask - the run function will store the results within YozioHelper
would it be reasonable for this file to include just two imports: one for mulberry, and one for toura-specific stuff?
actually, I've not used archipel.conf at all even for jid. I just included archipel.conf for sake of needed ConfigParser in TNArchipelEntity.
defaced.PNG! Wuhuu \o/
Regarding the units issue, I guess I'd be happy with a comment here along the lines of: "Since we usually consider the physical units for our images to be arcsec (i.e. our pixel_scale is usually thought of as being arcsec/pixel), this implies that the units of k for these power spectra would typically need to be arcsec^-1.  It is the user's responsibility to make sure that is the case."  This could probably replace the sentence "If you want instead...", since we should never encourage the user to deal directly in pixels.  The pixel scale is supposed to be used to convert from physical units (aka arcsec) to pixels.
Is galsim.Shear working in doxygen now?  We used to need galsim.shear.Shear, which shouldn't be necessary anymore, but I thought Rachel had a comment recently that the links with just galsim.Shear were still not working.  Anyway, worth checking this to see if it's ok on this branch.
Maybe remove.  Commented on the api pull request.
Yes good point.

And I think we could add when null fields are ignore to indicate which fields are null.

The same message as  ignored fields could work (the difference is they are dynamically found)   :

"comparison was performed on all fields but  ["lightSaberColor"]"
Replace campaign_friend?
campaign_friend is different from a regular donor or user in that it's a specific person who a campaign is emailing, and the details (like the video link) can be different for every campaign.
should client.close() be in a try-finally block?
When will classpath be null? Should folk be constructing this object with a null classpath and calling load etc?
Maybe it will become apparent as I read through the code.
And another. :)
Here you can also use `"%s.%s" % (User._meta.app_label, User._meta.module_name)`.
Don't you need to free hash?
With the changes you made to `__getitem__`, I think this logic could be simplified closer to the old logic where this method delegated most of its work to `self[name]`. Avoids duplication.
Please delegate to `uncheckedRestartSplunk` here to reduce duplication.
Ah thanks for clarifying, did not struck in first look.
Want to add a filteredJobQueue as well? :)
i think we could do `display: vendor-value(type) args` as well and bypass manipulating vendor-value()
I disagree... even knowing the type expected in them is helpful. More documentation over none.
What wut? What are you wutting about?
I think this is a security vulnerability. Much better to have a list of legal templates instead.
dig - копать. Лучше полностью писать, чем использовать необщеупотребимые сокращения.
А, всё, понял.
\\s матчится на пробел, таб и другие символы-разделители.
You're right. Should be removeObserver:forKeyPath: instead.
Using O_ct(sfp[K_RTNIDX].o) is better.
PreApproval
Should we be hard-coding the variable "evernote_details"?  If anyone else changes the search term, they'll have to update it in a couple places -- maybe a generic name would be better?  I defer to @teodosia, @zacc, @bebef1987, @AlinT, et al.
Here I think `include` is unnecessary.
since last_alpha is always None and it's the only call to _fit_enet_with_strong_rule why do you need this parameter?
typo: skip
```javascript
body.push(UTIL.format(
    '<!DOCTYPE html><html><head><meta charset="utf-8"/><title>%s</title></head>' +
        <body><b>Index of %s</b><ul style=\'list-style-type: none\'>',
    path))
```
menuId is not even used?
ok, I see, Port isn't used by the backend.
Should this class name really be all uppercase?  is there a specific reason for it?  I don't think it qualifies as a constant, so I would think the name should be capitalized camel case.
Added new class method to regenerate
project.py --android_sdk=c:\Android\android-sdk  --blackberry_ndk=c:\bbndk-2.0.0  dummy5 com.macadamian.dummy5 "c:\Users\alexg\Documents\Titanium Studio Workspace" blackberry android

also tried with:
project.py c:\Android\android-sdk  --blackberry_ndk=c:\bbndk-2.0.0  dummy5 com.macadamian.dummy5 "c:\Users\alexg\Documents\Titanium Studio Workspace" blackberry android
Odd comment location. If it's for the `else if` put it inside as the first line
Because I didn't know better. Interesting reading.
->isText()
Yes it could be.  You could also transfer ownership of the rowset instead of linking.
The obvious much needed change.
This code needs to be in a child context e.g., subctx.  The build function can't modify the current level  code is being inserted.
not if you check for eof/fail flags, which will be emitted by the last successful (ie. non-locking) read.
Double space before !=
right. removed
By forms I don't mean HTML like <input> and stuff.
did you mean on_reload?
No.  It was a separate fix for a JavaScript error that I noticed in the Safari error console.

https://poddery.com/custom/github/diaspora-2976-unmatched-tags-error.gif

If you omit that change, the main problem of the posts appearing in the reverse order will still be corrected.  The older version of Safari appears to operate fine with the particular change omitted.
As already noted within my example above my vote is *ArrayUtils* ;)
Thats right. But it could also be called compliance :-)
Anyway, i will put it in /hardware/i2c/master/ecmd.c where the emcd functions of all other modules are. Or do you think i should create a new *ecmd.c for the driver? This would quickly mess up the directory if we dont put each driver in a separate subdir.
BTW, most of this is just whitespace changes from the line endings
remove isElementpreset
misc_room will be null if it's not parseable as Integer. Is that what you intend?
that's just it. it shouldn't be. that's why SeasideProxyModelPriv exists.

further reading: http://en.wikipedia.org/wiki/Opaque_pointer#C.2B.2B
instead of testing if res is nil in each if statement, test if it is null beforehand and throw an error, then continue.
All of these are ```::Rex::ConnectionError```s, just rescue that instead of all these.
As discussed, this should be CDI alternatives.
If this were example code, I could see a benefit to keeping the object literals written as JSON, which would help someone if they wanted to pull the material definition out into a JSON file loaded through XHR or something.  If this is internal implementation, then yeah, no need to make it look like JSON.
Adding documentation here isn't difficult. I must have missed it in my last pass looking where documentation was needed. It's already done, but hasn't been submitted :)
What I wrote to @shunter yesterday: I’d say that even in C#, internal properties should be used sparingly.  But it seems much worse in JS if we name them the same as private fields, especially considering that there’s no enforcement of their accessibility and no good way to find all references.  It’s hard enough for me to figure out what any given pile of code does without constantly wondering if this thing named _foo gets clobbered by someone else.
Again this is line ending whitespace clean up. I can add that to the comment in my pull request so they will expect to see it.
Should show deprecated property value:

[WARN] 'Ti.UI.Media.Sound.sound' is deprecated; Deprecated; use 'url'"
"I don't know about MW, but in iOS and Android, that's exactly what Ti.UI is--an object that provides a bunch of factory methods and constants." That's how it is in Mobile Web too, which is why I don't like the term module. Module =/= object.
I wasn't aware of the broken resolve, and there wasn't a ticket open to fix it. Thanks for catching and fixing it!
but with a log statement
A pimp on the response object could help readability here. `response.sendNeedsAuthorisation(userProvider.realm)`.
If you are woried about performance, use:
`log.info("Set environment variable %s: %s",key, value)`
This will only perform the string interpolation when the logging function is actually called.
Meaning:

    $thread->setRecipients($message->getRecipients());

Correct? This is perfectly fine with me. Will update accordingly.
Recombined in the latest push.
а, у тебя `upload` это не функция. почему?
про present? я на презентации говорил :)
these all caps enums are so C. Use regular symbol notation?
The _only_ stuff which supports ranges of `char` or `wchar` in Phobos is in std.utf. Nothing else. `stride`, `decode`, etc. were made to work with ranges of code units, because you _need_ them to work on ranges of code units. And the only reason that that was done at all was so that lexers could support ranges of code units. Phobos as a whole doesn't, and it would be a pretty big change to make anything else support it.
I'll have to re-bench this, but I do think it is faster this way, since all the control paths trickle down to a lone return statement.
No, I think as the possible gain is more or less understood it can be done later. 
Indeed the implementation won't be DRY at all, let's not mar  this already large commit. 
Generally it sure worths it, but not "here and now". 
(Imagine inserting into the end of 1 million element array that has few blocks of extra capactiy, a realistic case)

About dupTo change - I'm not clear on it. 
On one hand over-writing immutable (that it implies it, but doesn't  ) seems wrong.
On the other hand  I guess it won't be the last time this copyInto static if thingy shows up.
So, I think I'm more in favor of this change.
True, that's always better!

Still, this unittest uncovered 3 different functions that didn't compile! I'll try to see into getting it to actually *do* something, but it might triple this (already long) test's length.
And a bit of stupid wasting of yours and mine time:
> ElementType is a template which is therefore used in template programming. All range-related traits are used in template programming. They're not good for anything else.

Yes. My mistake. I meant naked template programming without using ranges.

> And this trait not only has a name similar to ElementType, but it's doing something similar.

`ElementType` is a mistake as it suppose *range* is the only thing with *elements*. It should be `RangeElementType`.
`MultidimStaticArrayElementType` is like `ArrayElementType` from pull #776.

> But I'd still think that having this trait work with anything other than a multidimensional static array was fundamentally broken

As I said, my concept is *every value is a static array*. You can call it *extended static array* if your want. Yes, it's resonable but will make names longer.
Needs an update :-)
I can get the behavior you want (I think?) with XMLin's ForceArray option:

<code>$ XMLin('<pages><page title="Transclusion Testing">blah</page></pages>', KeyAttr => { page => 'title', r => 'from', n=>'to'}, GroupTags => { pages => 'page', redirects => 'r', normalized => 'n'}, ForceArray => [ 'page' ])</code>
<code>$HASH1 = { page => { "Transclusion Testing" => { content => 'blah' } } };</code>

<code>$ XMLin('<pages><page title="Style">blah2</page><page title="Transclusion
Testing">blah</page></pages>', KeyAttr => { page => 'title', r => 'from',
n=>'to'}, GroupTags => { pages => 'page', redirects => 'r', normalized =>
'n'}, ForceArray => [ 'page' ])</code>
<code>$HASH1 = { page => { Style => { content => 'blah2' }, "Transclusion Testing" => { content => 'blah' } } };</code>
Comment where these came from, when they might change, etc.
yep they are defined in root. just forgot to remove them here
https://github.com/cneves/openmicroscopy/commit/bf5b4671b3974b6657abb80c6a14970f7b735e9a

When we close the connection we now close all service proxies too, and that iterates the _proxies dict. When we first create a BlitzGateway it get a NoProxies() instance as _proxies and during testing there are cases when we close the connection without ever creating actual proxy objects, which would then raise on close. I'm not sure this is seen outside the testing cases but it can certainly happen.
In .NET 3.5, I guess you couldn't mock HttpContext or RouteData directly on `RequestContext`; had to pass them in via its contructor.
No extending Function. I could see this being on Class, though the gain is small, since you can write all the methods for a class when you do `new Class`.
`$this->_categories` never gets set. This method only updates the db if the `_categories` array is manually altered. If that is the desired function then why not *only* use the `_categories` array? Once the db has a different number of categories than the array; it will always hit that RuntimeException.

This change assumes you intended on syncing the db/array by setting `_categories` at the end. Making the hard coded categories be defaults and allowing later added categories via the array or db.

Also after further look, `count()` may not be the best way to verify a change as both arrays could have the same count but different values. If my above assumption is not off base then I think we should alter this method to: Grab the db values, do a diff against the array and add any missing hard coded values to the db, return combined results.
rm 210
intending is off here ..
please don't call ``$router->getRouteCollection()`` twice. This methods triggers the routing loaders.
No, it has moved to https://github.com/jamestyj/barclamp-crowbar/blob/0ba226b2d11f33818d03b8fcc8c7deae8b44a8a0/crowbar_framework/config/initializers/constants.rb, line 11. 

The condition is removed since we don't need it - the value is always the same in all environments.
lo puse así onda a efectos de probar. Hya que cambiarlo eventualemente.
The problem I had was that if you try and fetch classMetadata that hasn't been loaded, then Doctrine throws an exception. I either had to do this or catch exceptions as flow control. I'm not sure there's a workaround. I'll try on IRC next week
Юзай Q_DISABLE_COPY
Yes, but this is converted from a string label to a number by the time it gets to the FigureManagerBase -- see pyplot.figure impl.  So the docstring is correct here that this will be a number

https://github.com/matplotlib/matplotlib/blob/master/lib/matplotlib/pyplot.py#L318
Not sure which is faster, it just makes more sense to me the other way, both because it puts less extraneous stuff inside the `with` statement that doesn't actually require the open file, and because it error-checks the args in the order the user provides them.
Why did you cast it to use triple equals?
need to make sure we are unsetting the value properly -- write a unit test to cover this
Yep, and I just pushed it.
Parsing files and searching for a string is pretty scary. We could implement a marker interface, but then we'd have to instantiate our models.
This looks like the defaults for Facebook. Do they need to be here?