labofsahil/github-event-dataset-2012 huggingface dataset:
Unfortunately, due to the way Python works, dir(nhlib.gsim) will only list the imported modules, so you have to import them in the __init__.py (same as using __subclasses__). Notice that I am perfectly fine with a function get_available_gsim() doing the import only when called and such that we do not need to list the modules explicitly. It is just that it should be in nhlib. I am influenced by REST design, in which it is customary to have a resource which returns a list of the available resources. It makes a lot of sense to me.
I feel like we want a logic/action/get.py function for this
Hmm. This seems to be the same as just doing `urllib.unquote()` directly, except that where `urllib.unquote()` would return an empty string if given an empty string, this will return `None` if given an empty string. Is that the purpose of this? The function name `unquote()` doesn't suggest that to me. Is this function necessary or could you just call `urllib.unquote()` when you need it?
Interesting. Removed it.
broken.  this is supposed to give users permission to change their own permissions even if they don't generally have permission to edit accounts.  (arguably it should be handled elsewhere but this is a needed hack for now)
ah, so this is where you were attempting to handle the id caching?  the @local store gets cleared with every request, correct?  Is that all we have now in terms of id caching?
@davilla, @theuni - you are both seeing the issue - can you describe it for me? Is it just on resume? What actually happens? Is it only with navsounds or streams?
This scares me a bit. IsInternetStream is used for a lot of things. It could potentially break stuff, but i suppose the ones you added is likely safe.
Ага, public с подчеркиванием не айс...
Returns a {@link List} of additional {@link Resource} instances. These resources require user confirmation before deletion.
Rename for ValueTableFacetResource
Loading magic.  Would not surprise me if it came from a different place depending on how it is loading, test vs. production and all that.
This empty line should be removed
Apologies, did not see the custom setter. All's good here. :+1:
Properties should be preferred to ivars whenever possible.
Just replace it with some while loops or for loops -- I don't care exactly how, as long as it doesn't rely on break. It's probably possible to do something functional too where you recursively search down the list, but it's not necessary.
S'il va falloir l'ajouter après, c'est correct. Laisse ça comme ça.
oh yes, you are right. There is a mistake checking for is_int($name) after $name was set to string. I will post a new commit.
Then maybe alloc_log_enable should clear it. The point is that we don't want log entries to get mixed up between different logging sessions, since that would silently give misleading results.
One of us added a trailing space here
@jeremy I'll leave the + here since it's output to stdout here.
This same issue happens with the `serialize` method?
@steveklabnik If you look at the [test codes](https://github.com/rails/rails/blob/master/actionpack/test/template/url_helper_test.rb#L440), string likes "http://www.example.com/shop/checkout" and like "/shop/checkout" are available too. I'll send a pull request which contains the examples of strings.
Will do. I am on the road now, so if yog can fix it for me that's fine. Thanks.
This fixes nearby overlong lines noticed while renaming the non-descriptive funs.
Should I drop the change and keep the long lines?
This is way to much complexity for my taste, the previous version was much easier to follow. Also, what is the allCPSMethods check good for? It will only catch literal uses of shift, reset etc but no transitive ones like ``future { ... }``.
I think it is simpler to add a single space. 

It's a small issue in any event, +0.
Yes, but since it would be used only once I will remove it.
Perhaps, as in a few place in sympy now, this could just be a `random` method: `IntegerPartition().random()` or something like that?
>
> That's new to me. In my own writing I've used vida infra...but since
> the docs are not for a math journal, perhaps we can error on the
> non-latin forms for such things.
>

done
What do you mean according to WolframAlpha?  This makes it sound like this function calls WolframAlpha.
```
function = sympify(function)
if function.is_Pow:
```
argh--`if self.is_number` condition.
We are not using this syntax. We use 'except' property instead.
`$data=isset($data->$field) ? $data->$field : null;`
So @drak is can be merged. I will add a changelog later, since I did this as a inline edit..
After discussion, catch OSError and _log.error it. Don't check for existence first.
This whole module needs documentation.
The default option array should be included in `security.authentication_listener.remember_me.options` directly as it is not configurable.
Style: type* name
That doesn't look right to me. @piscisaureus?
I've called the interface IdOrdering, (and I'm going to be pedantic here, sorry) as collation has a very specific meaning in text processing, and if we then go on and include John's suggestion of including a numeric ordering, then we're going outside of the category of collation to a more general ordering. Maybe KeyOrder or something similar would be clearer, perhaps?

But you're right about the naming--the choice was a little rushed. I'll improve those.
s/will return/returns/
prints instead of returns;
why side_effect here?
@imkingdavid Nah. `||` and `&&` are completly analogue. In the first case, a true stops further evaluation, in the second case a false.
This is PHP4. Please also make a _construct() for PHP5 correctness. _constructor() would call this method to construct.
Same as with template events, if this does what I think it does it is not acceptable.
Determines whether users are given "Remember me" option when they visit the board.
Data needs to run through htmlspecialchars before it is output. This essentially unescaped user input.
> I don't want co check their type and cast to Movable* or Rotatable*.

That's pretty weird.  I think you should grep lights by type (e.g. TYPE_DIRECTED) and than say ->getPosition()

Is there GLOBAL light type?
This can't be TCB. Any MP can then set a label to whatever
pls modify to "helloworld"
ひでぶー？？
Not in that case. Just noting that we're lax in other areas and maybe carried that over here. If it's internal it's fine as is.
would be good to comment which field is referred to... 5's meaning is unknown without looking in the model
TODOs without a plan for follow-up aren't acceptable .. We'll soon have them all over the code-base if we aren't careful! Could you please create a JIRA ticket for each of these todos and add a hyperlink to that ticket in the code?

Please hold me to the same standard.
The getAll is returning a REF representation basically to be able to verify on the client if there exists a list that needs to be updated or new one to be created. The searchQuery is only on FULL rep because the inList and notInList is basically useless without a lot of context about the other lists...
To be honest this was mostly copy-and-rewrite from the cuda stuff.
As this code never change inputs_val, this wasn't a problem. But i agree we should take the good habit in case the code change in the futur. Also, having the good habit help prevent that we forget about this.

I updated the PR.
You mean there shouldn't be a space after the commas? Or you mean no space on either side of the equals signs only?
Since having [] as a default parameter is dangerous, I would do something like this instead:
    def __init__(self, ..., inputs_val=None):
        if inputs_val is None:
            self.inputs_val = []
        else:
            self.inputs_val = inputs_val
?What is the difference between ps.id used in the first line and ps.pid used in the second? Should they be the same?
Unrelated hunk
.all() is not needed
ISourceFileLocator had to move into core because I needed to locate source from my source directives filter.
May I use std::string_stream in newFunctionName()?
The `+`s should be surrounded by spaces.
style nits: `window.scrollTo( 200, 200 );`
We will not style Twitter posts. They are styled officially and I like them because they are recognizable. What we need is that Facebook posts are styled in a similar way to Facebook posts on the timeline. Again, to be recognizable.
Do the docs mention that you can only use args, or kwargs, not both?
The `elf.h` header file is not needed for building the formula per-se but for building tools for some of the compilers this tool can create.
`#{bin}/sshpass`
^an unnested
So we'll have an attribute that isn't used instead. I can't care one way or another.
What does 1 mean? This should be a constant defined somewhere. The same for the rest
yeah, no problem
I honestly feel the generator comprehension option, even disregarding the StopIteration issue, is less readable, and therefore *less* pythonic than the `for` loop.

For your other question, I prefer non-L variables, though single characters are fine. I'm more concerned with new code coming in than changing old code without an associated new feature / bug fix, though, so don't feel obligated.
Not using brackets are somehow usual in junit.
It's safe to use them.
I will improve it.
Thx
That makes sense. I'm sorry I didn't think of that approach before.  I'll see I can get that taken care of this evening.
Rather than "without constructor", "with the default constructor"
I don't think so? We want to overwrite any errors that were generated through the filters unless they're already an error
yep, just easier to read.
Can't be null at this point
I think Spout should be with a capital S. Also the website shouldn't be at the top, see the header.txt file.
I hadn't noticed before that this had an inline style. Is there a reason for that? It should be in the stylesheet.
Let's change this to use our standard array looping convention: `this.swatches.forEach(...)`
_getCodeHintList is only for unit testing, not really part of "public API", so this line of code should be moved up above the "public API" comment.
I tried splitting them then realized the tests all need to run in a single window since perf data is tied to the window. Doh, we definitely have a lot of things to solve when we really get into automation for performance. I left a comment.
You forgot to delete this line
i was thinking the same thing while i was doing this 
so totally agree with you.

Also I think are more jsdav tnan apf
The property might not exists!
Could perhaps use some justification, but no obvious issues.
I'm not sure if I understand the downside of an exception here? How would it run the risk of exposing sensative information? Only if it was caught, and when someone catches it whatever happens next is their responsibility. In fact I'd say it's far preferable to have it catchable instead of being something like PHP's errors.
Can you now also remove `trans_type`?
This method may be split in several ones in order to make it easier to understand.
Sure, would be useful to add those, but obviously outside the scope of this ticket.  Let's just have it for now work the way the old system worked... 

1. look in config.json if it's there, 
2. if not, look in git config settings if it's there
3. if not, then prompt for username and password and store oauth token back into git config

Cool?
And also API for `graph_api_url` is strange - it requires called to know how to construct URLs for the API. I would add additional function into `utils` for user picture and call that. So that Facebook Graph API is defined all in one place and callers just give user object or username or something, and size (but again in some predefined way, like constants).
It should be expanded to cover the full RFC, be private to the Content-Length code, or at least its incompleteness should be documented so nobody else decides to use it for a more general purpose.
No, let's leave it.  Based on irc discussion with you/rick, I think it's fine.  My hesitation comes from historical issues.
Like I said in the comment above, I've just copied and pasted most of this code. However, I can see why this might be here: to let the user-agent do what it needs to do before applying the changes from `focus()`. I think we added this to prevent some problems with the user-agent grabbing focus improperly otherwise. Regardless, I don't see problems with these kind of `setTimout` blocks because they're usually UI-oriented.
Just pasting at the end of the video is an issue here, and only if you're right at the end, otherwise, you are going to paste something.
`calls` should probably be `class`. Do these even need a class here anymore?
Same as above for warnings
Mind fixing the indentation on this?
Throw a comment above this line so it's more clear what the regex wants to do.
remove the px
No action attribute needed here.
Mixed http/https content!
OK, as long as it degrades gracefully enough
@harryw A database is also created when a snapshot is specified. Currently we don't have test coverage for this. It's in the backlog
alias this join!
Wondering why this is necessary and why not set autoActivate to true? Can you explain?
Please fix indentation to the 80 characters line boundary
This is a habit of mine because I also run python code on Mac OS, I've always done it this way.
It looks like this does not belong here.
Updated with commit  d68a667e40f7ebe5fd666f490101794718603498
I'd be tempted to move this snippet over to the activation_key.js to minimize the js that is included within the haml.  

Possible approach (not tested) might be:

1. remove this snippet from haml

2. in activation_key.js - add to the KT.panel.set_expand_cb something like:
    KT.activation_key.initialize_new()

   (Note: the panel.js calls this callback function each time a pane is opened)

3. in activation_key.js - add the KT.activation_key.initialize_new() function.  e.g
    initialize_new = function() {
        // confirm that the user opened the 'new' pane... if so, initialize; otherwise, return
        var pane = $('#new_activation_key');
        if (pane.length === 0) { return; }

        // insert the snippet here
    }
   
In the future, we could then easily add other initialization logic to this initialize_new function, if desired.
I'm not a fan of detecting this sort of thing.  Give the snowball analyzer a shot and see if its still necessary.
removed it, it actually became unnecessary with the callback's error parameter and existing checks
Make a method called `GetCacheKey` and clean up this logic so it's more readable.
Not sure, documentation isn't totally cleared - would need to check the code.
You are right that it's defined but not used in ``convolve_fft``. If we do implement that like in ``convolve``, we need some tests for it, so I'm just going to merge this as-is, and will work on a PR that implements this.
You can use `DistutilsFileError` instead--using subclasses of `DistutilsError` will result in different error-handling behavior and not result in full tracebacks.
@luke-jr Ah, I had it inside on another branch, but it got missed in this one. Fixed. Thanks.
Is there any documentation on PROTOCOL_VERSION I can refer to please? How is this decided and how does it cope with branches?
> I wouldn't suggest touching the user's PGP setup, but verifying without touching it. If GPG really needs to keep keys somewhere, ~/.bitcoin/.gnupg or similar makes sense.

OK. Then I misunderstood you. So you're saying the script should pull in a public key from a remote location and use that to verify? That makes sense.
This would create another point of attack though. I figured the best way was to let the users who run the script store the keys themselves, so these can't be modified easily by an adversary.

> contrib/gitian-downloader contains PGP keys. There's also a git repository here on GitHub with signatures of multiple developers for most releases which would be better to use than the SHA256SUMS file (which can only have one signature).

The threat that this script tries to mitigate is that of an adversary replacing the binaries on SourceForge (I made it after reading this thread: https://bitcointalk.org/index.php?topic=113018.0). So the devs in question need to sign the binaries that are linked to on bitcoin.org. Is this the case wrt. to the git repo you're referencing?
As far as I can see, this is not what is signed in this repo at least: https://github.com/bitcoin/gitian.sigs/ - is this the repository you're talking about?
Move all of these to a css file, if they can't be moved revert it. This inline makes it hard to change this file.
Please add some unit tests for non-edge cases.
Just to ensure they work correctly as well.
This should probably be called "Add existing Loomio user to a group". That way, we can make another test for "Add a non-existing Loomio user to a group". Because the second is actually a separate set of functionality which invites the user to Loomio as well as to the group.
Not sure how I forgot this, but thanks
Is this the usage text? Should it not read: git tfs __init-branch__
We all have one of these skeletons! :) Stick it in a package object instead.
Copyright (C) 2012 The IPython Development Team.

:-)
sweep for VirtualBox and correct
We're trying to drop variables with return values. Use two paths; a normal that returns 0, and a `on_error` label that does the cleanup and returns -1.
$len for eg.
Done, but I do not think I need the url helper do I?
+1 to truthy.
This isn't working because you forgot to bring over the id's for specific answers
Sorry.. You should only look at the Reply model. The Tweet model is continually truncated but the replies stay there forever. I think it will make all this easier as well!
How would they be viewing the dashboard if the system is not installed?
Okay, then just leave it.  I notice in Samuel's version, reset is a separate function that sends back a message to the browser, so it'll be cleaned up when we merge his code anyway.
One final change: please move those imports back where they were so the diff becomes minimal :)
Are you sure?

        if (false === $imports = $xml->xpath('//container:imports/container:import')) {
            return;
        }
you should be passing ``$configs``, not an empty array
Do you really need to use a data provider for a single case ?
You should use `preg_quote` to escape the string.
Should it be modified ?
Does this do the right thing if the base repo is not sympy/sympy (I didn't check, so maybe it does)?
Nice work! `[MDdw]o` will need to go before `[MD]{1,4}` and `d|dddd?|ww?` otherwise it will match just the `M` of `Mo`.

Also, we are adding a `dd` token in another pull request so `[MD]{1,4}` can become `[MDd]{1,4}` and drop another 5 bytes.
perhaps 2 lines for the condition would be more readable.
Wait, why not `Yam.post_activity(params)`? Is `post` a more general action which takes other endpoints?
can you move this into YammerUser?
OK, I will reorganize the samples in another pull request.
Why the TypeType ??
Starting with capital letter, we are considering a type, and  two more suffixes "Type" seem to re redundant.
Given the unit tests, are all these main methods really necessary?  If you want to give examples, which i think you should, probably better to do it in the javadoc for the class?
Yep. It's a left-over, from when this was standalone. Will do.
Two # make this line a header
why changing the level here?
Isn't this misaligned?
I am not sure having -safe and -confident in the library's name is needed
I don't really mind, but it makes the name longer
But if this helps catching issue like using non void-safe lib in void-safe application  why not.
Not sure it really justifies being inline, but doesn't really matter either way.
Slightly more usual would be to have a reference parameter (string & filepartname) for the target - since it must always be supplied, but again very minor and not worth changing.
This codesniff error can be fixed by adding the curly brackets.
我觉得有值比不得值好。判断是否有效的时候，代码更直接更可读，而且少判断一次。有效的条件就是 expiredData > new Data()
如果可以为空，你就要多判断一次。条件1 && 条件2，这个肯定不好。
Nit: typo (Building Blocks)
nit: move </dd> to the next line
what about adding an `app.ondownloaderror =  null` here?

and for the app.ondownloadapplied too ;)
Remove the webkit rule for now, it will fire a warning in strict mode.
In this case, I’d suggest to put the date **below** the label instead of next to it — think of other locales for which the text will be even trickier to fit.

```html
<li>
  <small>2012-11-24</small>
  <a> Last updated </a>
</li>
```
Same, `.sharp`
Note: we plan to land a platform patch to remove localStorage access to apps. So please be future-proof
What are the use cases for this activity?
I extracted all animation query to getAllElements for performance.
Maybe I spoke too fast. Let's try that and I will revert if needed.
"no one". And can we say "without" so non-native English speakers catch on?
PS: Yes: you should just remove type, if I wasn't being clear. It's essentially not used in HTML5, like `/>` on input or img tags and convention on our other sites is not to include it.
yeah, that'd protect us if any of our code or the code we depend on monkey patches Object.

I'll add.
Can't check this in. Must handle this better.
OK, thanks for the pointer. Next rev of the pull request will have this refactored.
There isn't anymore -- I need to update this line.
Thanks, I can follow that pattern.

In my opinion, the method should continue to color things red. I was thinking about this earlier and I decided I didn't like the idea of methods explicitly named for their colors producing different colors than their names. But I it would be easy enough to change it to just use the configured color if you or others disagree.
The `CarView` already has those properties. So I guess we should move them to the model so the `CarView` and others can access it.
toSet is not needed here.  SortedSet[Long] is an instance of Set[Long].  I *assume* toSet on SortedSet returns this, but I'm not sure of it.  Removing it is clearest that there is no extra copy.
Good catch... the explicit monoid was a holdover from a prior iteration.. it's clearly correct to use implicits
I wish this could be an implicit hasher...
I guess you are doing this because you want to make this really a TupleConverter. Don't see what you are gaining though, given that we need the whole Packer hierarchy...
I'd rather have a default match all ``, default='.*')`` (or ``''`` depending on how we do regex matches) than having these headers mandatory. It makes it easier to use this with ``curl`` or ``telnet``.
nesting the 2nd ternary in a different indent level would help readability here.
This is a good idea but would take to long to extract and test throughly in the scope of the current task. I've made an issue: https://github.com/AppScale/appscale/issues/108
From the discussion on IRC:
- Character arguments will need to be replaced by single word arguments
- Ordering of command arguments needs to be made more intuitive and consistent (see Thanh's suggestion)
err platform was made for this reason @zml2008 

Also, could use reverse logic and simply check if platform is CLIENT and fail to continue.
Yes, that's a good cleanup, but I haven't been terribly concerned about it..
I owe you an apology for this one, as I saw your initial post on Diaspora Dev, and didnt respond.

I am trying to depreciate as much YML as possible.

I support the 'having a rake file give people admin access' and maybe even make a user from a rake task.

However, I would much prefer it to take command line arguments, rather than read from a file. 

so something like

rake create_admin[username:maxwell, ;password:foo]
or

rake upgrade_admin[username:maxwell]


(that API sucks, but you get the idea)
丁寧な回答ありがとうございました。
私も長文読むときに苦労しますので、ぜひ参考にさせていただきます。
You got trailing whitespace.
Not sure if this is a big deal, but on the last iteration when you find the node, you end up calling steel_avl_node_to_elem twice. Maybe you could save the result of the call each time? Memory vs. Function Call trade off?
why we have this column? remove it if not needed
here as well?
Beside note: I think we should remove the listener in paneTitle.init(). It's not necessary anymore once the window has been initialized. It will save a bit of memory.
Make it explicit, I suggest.
Why don't you simply call ``disableButtons()`` here?
"Get them" should become "Get one" or "Get a compiler".
You can shorten this to `pills += '<span/>'` and jQuery will match up the closing tags for you. Also why not use `i++` on the line above?
Its good to point out the java interfaces are being worked on, but more verbose is a given if someone chooses to use java. I'd replace 'the Java examples are considerably more verbose.' with 'the Java API is less polished.'
seems like the binary format here would be different than the one above (in EdgeListOutput). Is that correct? If so, it would be nice to have the same binary format. If it would be the same, then it would be nice to use the same code to do the actual writing if possible.
Since i and length are scoped just to the loop, you could declare them in the for initialization
for(var i = 0, length = this._togglers.length; i < length; i++) {
@weierophinney is this near to what you had in mind? 

Similarly, I've added an initializer to the ValidatorPluginManager: https://github.com/zendframework/zf2/pull/1890/files#L7R129
@Freeaqingme This is standard getopt usage, not just in ZF, but any library or language that provides getopt-style CLI argument parsing. And I mean down to the syntax used to interact with it.
@Maks3w, He's just accessing the first character in the string using array notation. This is perfectly valid. :)
This test case is too long.

Please split by strict option
I personally read Yoda much better :)
Anyway, good to know this rule. I've rewritten this one to return early, will keep those hints in mind in future.
I'd remove this one. However, I'd likely add "application/hal+json" to it (as that's becoming a common standard).
Fixed all of these -- thanks!
Eigentlich finde ich das schon wichtig, damit man sieht, dass es ein Freifunkprojekt ist.
The inline event binders are only used in the shadow dom. In the background we call addEventListener. Please have a look at the rendered html. There are no event binders anymore...
We don't want methods that augment the prototype. You can easily use splice directly instead.
Sure. I'll do it when I'm back home from work.
note: invoked by RestServiceImpl on Main since it is bound as HttpConductor
this sounds weird. What are you doing here? A comment would be very much appreciated :)
you should add the empty line before the return statements
Its not doing the same, the problem is that converting an array to sql and converting a criteria to SQL are two very different things that need different handling. That is why i started this way, before looking into ways to refactor this.
Do we really need this ? I thought it was useless since the switch to the XML config for phpunit (it is used as the phpunit bootstrap file).

Check that you don't have an outdate phpunit.xml file
``AnnotationRegistry::registerLoader(array($autoload, 'loadClass'))`` is enough as composer already returns the boolean
I think purge() should be called with the locationIds. As the purge will only be applied on those ids, and we don't need to keep them afterwards, it does not make much sense to have to store them in the class.
This:

import operator

reduce(operator.and_, [opt.help == nohelp for opt in optgrp.options])
This class is identical to the other Task quickstarts (tasks and tasks-jsf). All the task quickstarts must be consistent so we either change it in all of them or none of them.
Are you saying I should replace : 
Start the JBoss Enterprise Application Platform 6 or JBoss AS 7 Server

wih:
Start the JBoss Server
??
You can read more here - https://community.jboss.org/wiki/EqualsAndHashCode - in this case you could make email a unique column, and use that as the natural identifier.
sorry, i'm confusing about these two projects, whats the different between _org.jboss.spec:jboss-javaee-6.0_ and _org.jboss.bom:jboss-javaee-6.0_?
Is there a compelling use case for the colon redirect syntax that isn't satisfied by built in shell redirection?
Rather than copying into `part` to do null-termination, you could use a format specifier to indicate the number of characters in the string. For example, something like this:

    printf("%.*s\n", u->field_data[i].len, url + u->field_data[i].off);
Good point. However I'm not really sure about this. Unique stream id is literally a vfsStreamWrapper property so why we are going to generate it in vfsStreamFile.php? On the other hand when I've changed ```s/getStreamId($stream)/getResourceId($resource)/``` because I didn't like invocation ```$this->getStreamId($stream)``` (new developer might wonder "if $stream is an instance of object then why there is no invocation $stream->getStreamId()?") then this started to make sense. In this way if at some point in future we will be able to get resource handler in vfsStreamWrapper then we could only change body of ```vfsStreamFile::getResourceId()``` method and then just drop acceptance for ```vfsStreamWrapper``` in other methods - only ```resource``` will be allowed.

At the end I've made three versions of change:) None of them are perfect for me - I think you just need to decide which one is more acceptable.

https://gist.github.com/3493492 Simple change, just generate id inside vfsStreamFile
https://gist.github.com/3493466 ```s/getStreamId($stream)/getStream($resource)->getStreamId()/```
https://gist.github.com/3493482 ```s/getStreamId($stream)/getResourceId($resource)/```, generate id inside vfsStreamFile
Maybe you should add :cvs-module here.
I think unittestzero is also third party.
I know that it's maintained by @AutomatedTester ... but is pip installable and we should treat it this way.
Weird indenting going on here.
Similar to the above, "<a href="http://opendoors.transformativeworks.org/contact/open%20doors">Contact Open Doors</a>" would be more accessible than "Contact <a href="http://opendoors.transformativeworks.org/contact/open%20doors">Open Doors</a>"
what do you think of the new change? I also realized my extension could probably be using an after_hook.
I need to search multiple instances of SFX so I create multiple Searchers in this same way.  I wanted an easy way to include and configure this functionality.  Abstracting out the Searcher bit and having the outer module tell the searcher what it's AzTitle class is (and hence SFX db connection) was how I did it.  This also allows Sunspot to easily reindex my multiple AzTitles in Solr.

Re: the const_get stuff, I think I was having an issue finding the module, but I'll see it that is still an issue.
should we shoe the combinations kind etc only if its not a snippet?
And uh, correct me if I'm wrong >_>
Would you mind marking it with a different prefix so that it will appear between the real branches and the All page?
Should any of these be localized?
Any chance you could add a comment to this section that explains why it's there? Please include the link to the StackOverflow question that posted this as the solution.
Replace all the magic strings with proper constants
This guy will probably want to change his handle... The clever use of leetspeak is deeply moving.
these should end up just being called `sobel()`, `blur()`, etc. when we get to presenting the library in the form of code snippets. I used overly verbose names here to illustrate that these create and apply filter objects.
There it goes :)
https://github.com/gbentley/ezpublish/pull/2
а другие виды ты не хочешь потестировать?
This should probably be a 200 response instead of a 400
@jcredding - good call.  In 9984c75 I've added the ability to pass an ASSERT_TEST_ROOT env var and it will be honored.  It falls back to `./test`

I also deprecated the `Assert::RakeTasks.for` method and replaced it with `Assert::RakeTasks.install` method that takes no args.
Also, space before the brace?
@nawroth Where are these files located ?
Heh, was the goal to have this working for merge-in? Seems like it's fine to me, but this message makes me want to ask to be sure...
FWIW, the logger can be used even if it hasn't been initialized yet. Currently it hardcodes internally to go into bootstrap.log, which @simong and I have agreed isn't ideal. But if that changes, then it would just go into the default "bootstrap" location with the other logging that happens before oae-logger is init'd
By the way, if you like the idea, this can be totally done in a separate PR.
I think your list here is backwards... :)
If we delete a row, the row key survives until the next compaction takes place. Since CQL3 enforces schemas, also all pre-defined column-keys will remain in that row as "column ghosts". They will however have `null` values regardless of their data type.
Почему бы не использовать property?
continue;  //зачем добавлять элемент, который уже есть в массиве testArr ?
Haha, I can never see in my small editor.  I can fix this.

On Thu, Oct 25, 2012 at 2:24 PM, Daniel Lowe <notifications@github.com>wrote:

> In src/classes/act.cyborg.c:
>
> > @@ -1872,11 +1872,10 @@
> >
> >      if (vict) {
> >          if (!IS_NPC(ch) && (
> > -                (!(tool = GET_IMPLANT(ch, WEAR_HOLD)) &&
> > -                    !(tool = GET_EQ(ch, WEAR_HOLD))) ||
> > +                (!(tool = GET_IMPLANT(ch, WEAR_HOLD)) && !(tool = GET_IMPLANT(ch, WEAR_HANDS)) && !(tool = GET_IMPLANT(ch, WEAR_WRIST_L)) && !(tool = GET_IMPLANT(ch, WEAR_WRIST_R)) && !(tool = GET_IMPLANT(ch, WEAR_FINGER_L)) && !(tool = GET_IMPLANT(ch, WEAR_FINGER_R)) && !(tool = GET_EQ(ch, WEAR_HOLD))) ||
>
> This line is ridiculously long. Please break up.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/TempusMUD/Tempuscode/pull/74/files#r1945434>.
>
>
oh shit I didn't even see the `Function`, that's fine though, nothing wrong with that, the JSON.stringify is fine for now then but we should make sure it doesn't impact builds much, though I dont think it will, it would be simple to just quote and escape the quotes as well
On further thought, is there any reason not to just fold this into `execute:`? I don't think the interface should allow execution if `canExecute[:]` is `NO`.
Can you please change this to an explicit reference to the `+mainThreadScheduler`?
I feel like we've been through this before... An `int` is 32-bits in C, be it a 32-bit or 64-bit machine. And MSDN tells me this is also the case with C#'s `int` type. An `IntPtr` will be the size of a pointer in the machine, which is not the same as an `int` in 64-bit processors, which means you'd be passing a variable with the wrong size. Here it's not that bad because it'd be at the top of the stack, so the other parameters would still be in the right place, but I'm not sure we'd end up reading the lower bytes of the argument (in amd64 more args get passed through registers, so this might end up working because of that).

There's also `int`s being used all over `NativeMethods.cs` where we want an `int`, and it seems to work just fine.
No need to hurry :-)

Just remember: Setting it to `NULL` is actually _all_ that is required (Think that wasn't clear from what I wrote above).
Instead of saving the URL here and passing it through the whole license selection/agreement workflow, why not directly load the redirection template here? Your changes in `metashare/templates/repository/licence_selection.html` and `metashare/templates/repository/licence_agreement.html` would not be necessary then.
needs @verbose
thanks guys for all the feed back. I'll address all these tomorrow or
thursday. Now is Xmas time !
isn't it a duplication with export_raw_to_nitime.py?
... hower args and kwargs should not be. that's for internals only.
oh no... we won't reinvent a new file format... maybe we'll just rely on python serialization / pickle but not more. At least I won't do it :)
You know that this is true at this point.
Thanks! Just a quick note: can we move this to the `registrations_controller`? The `Devise::Controllers::Helpers` only holds helpers that make sense through the whole application (like authentication, where which controller wants to dictate how it work).
Yes Josef is right :)
Good use of `#presence`
Fixed in 9903bed7a535a6379b9eed69ec6c51d2f210ff9b
need the double `\\` here - `Symfony\\Component`..
In this page, a little bit later, they recommend to do it the way I did, http://symfony.com/doc/master/contributing/documentation/format.html#adding-links.

To be honest, I don't mind using double quotes, I just want to make sure that I am doing it the right way.
在此版之后浏览器都被认定为常规浏览器。
此版之后的浏览器都被认定为新式的浏览器。
/cn
It all looks good, but this sign in can probably just be "Sign in with GitHub" and can just go straight to `/auth/github`. We can maybe do that when we removed devise in another pull request though.
Then this else clause would be unnecessary
over here as well
save file as compressed nifti form use `from nipype.utils.filemanip import split_filename`
Habíamos quitado este ítem porque estaba el de abajo ya.
You're right. I'll use the field names
If this function is unused, consider removing it, perhaps?
Try this:

```ruby
def raise_if_not_answered
  raise UnansweredError unless answered? || outbound?
end
```
same here for dialog box
This is not as complex as the previous expectation...
containsKey()?
New tests probably should code against latest protocol which is 12? Don't think it makes much difference for the test itself, but adds further coverage for the latest protocol.
What does `//` do?
I just tried it out with contourf3d_demo2.py.  The autoscaled limits should be: x [-30, 30], y [-30, 30], z [-100, 100].  If you take out the calls to set_*lim(), take out the call to plot_surface(), and leave only the call to the second contourf (and use offset=-30), you will see that the autoscaling works fine (if you use my branch at PR #1289).  If you change the call in contourf3d() to use jX, jY, and jZ for autoscaling, it comes out stretched in the z-direction.

jX, jY, and jZ are juggled around so that the contouring data that was provided in, for example, Y, ends up in jZ, but this is only for the sake of contouring.  The contouring code expects the first two arguments to be coordinate data, while the third is the height data to contour.  If dir='y', then the data in Y is the height data, and Z is the coordinates.  jY becomes the coordinates of Z, and jZ gets the values in Y, but then everything gets flipped back around when 'cset' is passed into add_contour_set().

The current way of calling auto_scale_xyz() is correct, it is just not functioning properly due to other issues that are fixed with PR #1289.
Neither necessary nor relevant--but entirely harmless, and presumably useful if a developer uses eclipse/pydev (which I do not).
PEP8 doesn't address trailing whitespace.  The "no tabs" rule is in PEP8, but I'd rather make it really prominent here.
nit :-) s/test_login_marketplace/test_log_in_marketplace/
Good catch @sbrunner!

I just think we should use the `OpenLayers.Event.element` function.
Like this. The original is, as you say, too restrictive, and also a bit ugly anyway (startsWith...).
What's the reason for the empty comment in this div? Is that some weird IE issue?
moved into super-interface
please don't mix variables and functions in one block:

use

private:

function def 1
function def 2
...

private:

variable 1
...
unless our response text is a self contained descriptor, it's probably better to use "[...] trying to publish: " + err.responseText
it's suppsed to be a sequence when sent on the worker and when displayed. but i've renamed it to :number
The **need** is here to hard, this option is still useful for use case where two administrators manage one server, they both being in rvm group + this setting is susfficient to install rubies without using `sudo`/`rvmsudo` - it is always better to avoid privilege escalation. So I think it would better look like:

> You can ...

Or:

> You might ...

The main point is it might be no longer required but still might be useful.
Yes, table would make it look exactly the same as in viewtopic
No, it doesn't. This should not be inline.
Are you sure that the property "failedTest" exists? Did you maybe forget to declare it?
Are you sure you want to override the parameter ``$nameField``? This often makes code less readable.
Accepted - solved as above.
historically is misspelled
That is the historical behavior. You can witness this by submitting a single phase (or single kept-phase) to the mapred endpoint using curl (w/o the chunked parameter). In that case, you will get a single array back with the results. If you have multiple kept phases (w/o the chunked parameter), you will get an array of arrays. In general I think this is a bad pattern, but I'd rather not break existing code.
Ist für uns Dummerchen unleserlich. Lagere das doch in DBUtil aus - in mehrere Methoden
Just a suggestion, but considering the badge widget may end up fluctuating in terms of structure, copy, style, etc I would suggest loading a DOM fragment and query-selecting/cloning instead of manually creating each element here.

So for example, you could load in fragments for
1. the widget container
2. an individual badge (and then clone this as needed)
2. the notification at the bottom (clone as needed, or just replace text)
Owner comment
There's putAll 

http://docs.oracle.com/javase/1.5.0/docs/api/java/util/Map.html#putAll(java.util.Map)
No, don't move extrahead. This is just for other meta options and stuff.

Use [sekizai](http://django-sekizai.readthedocs.org/en/latest/usage.html) for adding additional JavaScript.
I must say that I don't understand why are you defining a context variable here?
Can you name this function a bit more precisely? Remember, everything you define is global. So this could conflict with something else. Maybe just `initializeMapPanel`. Or even simply more everything inside of `ready` function.
In anonymous functions, put space before `()`. Write out all arguments of the callback, even if you are not using it. So later developers will know what is available.
He he. There is no event in this case. But yes, in general we want all arguments. But this one is useless. There is no real event behind and no data there.
What do you think with this: This can go to a top-level module constant (named with ALL_CAPS).
It should have been renamed to `request` since the SUT requires that object.
It should be `const std::string folder = profilePath.parent_path().string();` if I am not mistaken.

Also, there should be a check if parent path.empty() and then (and only) then should the code below be called.
I'd prefer to have a space between the two lines above and the one below... Instead of this <br>, can you close the p tag above and make a new one for the last line?
You should generally always quote paths in messages e.g.

> Could not load the metadata migrations from '/some/path'

That makes it possible to diagnose issues where an extra space or newline has crept in, such as:

> Could not load the metadata migrations from '/some/path '
ye, makes sense. I have actually just copied it from https://github.com/TrinityCore/TrinityCore/blob/master/src/server/game/Reputation/ReputationMgr.cpp#L78 which cannot be used with an offline player, at least i couldnt.
I think he's concerned about the codestyle ;)
maybe 't -= time_step / accelRate'?
Good point, Bruno. I think this is a too much of a leak from the core example.
```php
$validator = new \DoctrineModule\Validator\ObjectExists(array(
    'object_repository' => $objectManager->getRepository('Application\Entity\User'),
    'fields => array('email')
));

var_dump($validator->isValid('test@example.com')); // dumps 'true' if an entity matches
var_dump($validator->isValid(array('email' => 'test@example.com'))); // dumps 'true' if an entity matches
```
To fix this exception, we should change the exception raised in models.py (end of the __init__.py method of RedisModel) "asap" (blech'style)
It is still showing to me that you changed something here. What?
We had hard time to make the difference between join_cluster and register_node.

Maybe we should only have one of those and not both? Because IMO it's doing kind of the same thing.

Thoughts?
Yea, it was a bit a lot of copy (old code) and edit, will fix it.
Maybe even a dictionary/hashmap in cls.ResourceUtil ?
Cleared etag manually: 5018f21
The initial value should be -1.
I'm not sure this does what you think it does.

`$+commands[]` allows you to check if you can call the command `virtualenvwrapper.sh` (not if the file exists).
Yes, I think new extension still need adjustments to the code.
One possibility is to create a new LTS instance for each subsample, so each has it's own endog and exog.  (I don't expect much overhead doing so.) But in that case I don't think we already have the correct stopping criterion to get a 1st stage set of candidates out of it.

I appreciate your investigation and comments.  And, if you want to work on extensions like the large sample case, then it would be faster to get this finished up.
In nan_dot,  if a != 0, then a * nan = nan.  So both need to be nan_dot.
You don't have to I guess but I prefer it to sticking it in keyword arguments and checking for it. I kind of followed R's lead here so that when it's explicitly included you know it's handled. I guess we could just add missing to the model parameters doc template that's used everywhere. Similar options that should be added to every model is `weights`.
This change should be reverted. You went back to the old CS.
they are different things.
Shareware version I assume.
Good point.  Currently the OpticalPSF doesn't have a flux I noticed, but it would be easy to add one.
Yes, that's very true.  It does a couple of decent-sized FFTs at least...
Yes.  That might be something to consider for #168.
This should be done in the model.

Should be

Project.not_funded_by(current_user.id)
gonna leave it as Strings because guava's Strings is rather poor, you're not likely to use it or get confused (and btw there are quite a few Strings as well as StringUtils so a precendant for repeating also!) .. least bad option
Unsure, actually. I have had my Eclipse using Groovy 2.0.x with no issues, anyway. The other version updates are mostly just trying to keep current, but Qpid 0.18 is useful, I think, and it's generally good practice to keep up with point releases so that we track other open source project statuses. Guava 13 also has some neat new features it would be good yo have access to.

In the end, though, this is all just _nice to have_ stuff...
systemProperties is deprecated; one should now use systemPropertyVariables:
http://maven.apache.org/plugins/maven-surefire-plugin/examples/system-properties.html

Do these sys props get passed in with this config? It looks like it's using the format of the new systemPropertyVariables, but with the element systemProperties...
We need to update the sudo function to something like this to make multiple command execution work as expected:

public static String sudo(String command) { "(test \$UID -eq 0 && ${command} || sudo -E -n -s -- '${command}')" }

e.g. sudo -E -n -s -- 'whoami && whoami'
I think this needs further rewording. "Useful" is better than "used", but I don't like the conflation of indexing (which is where the within document frequency is set) and searching (where it's used). Perhaps something like:

This is used when searching the database to give documents in which the term occurs more often higher weight. (You can also manually set the within document frequency, which can be helpful for instance if you want terms in a document's title to be considered more important than in the documents body.)
I don't think this works... `to_s` returns a string, it doesn't modify in-place.
I started to but mock makes it really hard to introspect arguments in a clean way. If you insist! I can re-write it with mock, it's just awkward and ugly.
Yes, the Python smtlib.sendmail() call uses this concatenated list for everyone who is supposed to get a copy of the message., but this does NOT make them visible in the *header* of the message which all recipients can see. This fixed a bug where CC/BCC recipients were not receiving messages sent using the SMTP API. The code I fixed above this block makes CC recipients visible in the header (so all recipients see the CC list), but this particular piece makes sure those CC recipients actually GET a copy of the message.